Bug: Slowness in Devtron if Devtron is opened in multiple tabs

📜 Description
It bugs out when Devtron's dashboard is opened in multiple tabs. In this case, Devtron's Dashboard's response time increases drastically. It becomes extremely slow.
Ques - How do I know that it is an issue with opening multiple tabs of Devtron ?
Ans - Well as soon as I closed all the tabs where Devtron was running, Its responsiveness improved drastically. The slowness just vanishes. Have experienced this multiple times.
Ques - What kind of slowness was observed ?
Ans - Page load becomes extremely slow or try connecting terminal for any application's pod and try running few commands. The lag increases extensively.
👟 Reproduction steps

Open multiple tabs of devtron.
Open terminal for any application's pod.
Try running any command, you should experience lag greater than 10 / 20 seconds.

NOTE : It might be the case that certain pages / apis are creating this issue because there have been scenarios where 10+ devtron tabs were opened but still slowness was not observed whereas there have been scenarios where I observed slowness even with 5 devtron tabs being opened.
👍 Expected behavior
There should be no slowness in page loading or terminal etc irrespective of how many tabs are opened.
👎 Actual Behavior
Slowness is introduced.
☸ Kubernetes version
EKS 1.24
Cloud provider

AWS

🌍 Browser
Chrome
🧱 Your Environment
NA
✅ Proposed Solution
NA
👀 Have you spent some time to check if this issue has been raised before?

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

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

AB#5368

@tayalrishabh96 Is this still slow.


Improved: Have a main action menu for accounting (OFBIZ-11780)

see commit message

Closing due to going stale. Superseded by PR 403


Need pseudo code for categorical,boolean slot types and FormAction form setter

Rasa Core version:
0.13.6
Python version:
Python 3.6.4 :: Anaconda, Inc.
Operating system (windows, osx, ...):
windows10
Issue:

I want to add buttons for given restaurant type, when I add more than four buttons, buttons are not getting displayed in the slack GUI
or HOw can add them as categorical variable

I want to use FormAction, but when I import in actions.py, it throws as error name is not recognized

If any pseudo code is available for reference, please do share, with import statements as well
Content of domain file (if used & relevant):
## Generated Story<PHONE_NUMBER>23404498
* greet
    - utter_greet
* restaurant_search
    - utter_ask_location
* restaurant_search{"location": "delhi"}
    - slot{"location": "delhi"}
    - utter_ask_cuisine
* restaurant_search{"cuisine": "chinese"}
    - slot{"cuisine": "chinese"}
    - action_search_foodie
    - slot{"location": "delhi"}
    - utter_goodbye
    - export

## Generated Story<PHONE_NUMBER>540566202
* greet
    - utter_greet
* restaurant_search
    - utter_ask_location
* restaurant_search{"location": "delhi"}
    - slot{"location": "delhi"}
    - utter_ask_cuisine
* restaurant_search{"cuisine": "North Indian"}
    - slot{"cuisine": "North Indian"}
    - action_search_foodie
    - utter_goodbye

## Generated Story<PHONE_NUMBER>399695936
* greet
    - utter_greet
* restaurant_search
    - utter_ask_location
* restaurant_search{"location": "italy"}
    - slot{"location": "italy"}
	- utter_ask_cuisine
* restaurant_search{"cuisine": "chinese"}
    - slot{"cuisine": "chinese"}
    - action_search_foodie
* goodbye
    - utter_goodbye

## Generated Story -4639179087166749998
* greet
    - utter_greet
* restaurant_search
    - utter_ask_location
* restaurant_search{"location": "delhi"}
    - slot{"location": "delhi"}
    - utter_ask_cuisine
* restaurant_search{"cuisine": "chinese"}
    - slot{"cuisine": "chinese"}
    - action_search_foodie
    - slot{"location": "delhi"}
    - export

## Generated Story<PHONE_NUMBER>290237512
* greet
    - utter_greet
* restaurant_search{"location": "delhi"}
    - slot{"location": "delhi"}
    - utter_ask_cuisine
* restaurant_search{"cuisine": "chinese"}
    - slot{"cuisine": "chinese"}
    - action_search_foodie
* affirm
    - utter_goodbye
    - export
## Generated Story<PHONE_NUMBER>26174463
* hi

## Generated Story 0

## Generated Story<PHONE_NUMBER>132798014
* greet
    - utter_greet
* restaurant_search
    - utter_ask_location
* restaurant_search{"location": "hyderabad"}
    - slot{"location": "hyderabad"}
    - utter_ask_cuisine
* restaurant_search{"cuisine": "Italian"}
    - slot{"cuisine": "Italian"}
    - action_search_foodie
    - slot{"location": "hyderabad"}
* goodbye
    - utter_goodbye

## Generated Story<PHONE_NUMBER>050737577
* greet
    - utter_greet
* restaurant_search
    - utter_ask_location
* restaurant_search{"location": "bengaluru"}
    - slot{"location": "bengaluru"}
    - utter_ask_cuisine
* restaurant_search{"cuisine": "chinese"}
    - slot{"cuisine": "chinese"}
    - action_search_foodie
    - slot{"location": "bengaluru"}
* thankyou
    - utter_goodbye

## Generated Story -6686161742149946112
* greet
    - utter_greet
* restaurant_search
    - utter_ask_location
* restaurant_search{"location": "bengaluru"}
    - slot{"location": "bengaluru"}
    - utter_ask_cuisine
* restaurant_search{"cuisine": "Chinese"}
    - slot{"cuisine": "Chinese"}
    - utter_ask_price
* restaurant_search{"price": "300"}
    - slot{"price": "300"}
    - action_search_foodie
    - slot{"location": "bengaluru"}
* goodbye
    - utter_goodbye

## Generated Story -4291197092257407725
* greet
    - utter_greet
* restaurant_search
    - utter_ask_location
* restaurant_search{"location": "hyd"}
    - slot{"location": "hyd"}
    - action_check_city
    - slot{"location": "hyd"}
* restaurant_search{"location": "Hyderabad"}
    - slot{"location": "Hyderabad"}
    - action_check_city
    - slot{"location": "Hyderabad"}
    - utter_ask_cuisine
* restaurant_search{"cuisine": "Chinese"}
    - slot{"cuisine": "Chinese"}
    - utter_ask_price
* restaurant_search{"price": "300 to 700"}
    - slot{"price": "300 to 700"}
    - action_search_foodie
    - slot{"location": "Hyderabad"}
* goodbye
    - utter_goodbye

What you have uploaded is not domain file but stories file(please check it), and you can get some reference on how to use form-action in docs and this repo.
Version of rasa_core used in repo rasa_core==0.11.08.
NOTE :: The version you specified is most likely of rasa_nlu. Also, please specify the import line for form-action you have used, and what is the error otherwise it doesn't help.
Sorry, I can't help you with the first part, as it seems problem with slack rather than rasa.

What you have uploaded is not domain file but stories file(please check it), and you can get some reference on how to use form-action in docs and this repo.
Version of rasa_core used in repo rasa_core==0.11.08.
NOTE :: The version you specified is most likely of rasa_nlu. Also, please specify the import line for form-action you have used, and what is the error otherwise it doesn't help.
Sorry, I can't help you with the first part, as it seems problem with slack rather than rasa.

The 'repo' link is broken. Can you fix it cause I am not able to understand FormAction documentation

The repo was private so the link was not working. I have made it public now.
But, I don't think the repo will help you any more than docs. If you have any question, you can always ask in discussion forum.

@halloTheCoder thanks for referring to the forum :)
The buttons problem is related to slack limiting the amount of buttons allowed, so it's expected behaviour.
Not sure which Core version you're using, but this is the correct import from rasa_core_sdk.forms import FormAction, FormField
Gonna close this now, as the rest is more of a discussion

Thanks all for the reply , it worked now


update: Improve performance, code & remove used import & add error handling

Used built-in Python library functools to make the code faster. Also added error handling.

Thanks for helping! :)


Update KO locale for v3.3.0

Oh after trying updated version, I realized I missed some commits you made difference in English cfg file.
Now I registered to monitor ATOM feed of your cfg file properly.

Thanks. I intend on switching to use the Factorio Mods Localisation project for all of my mods in the near future, so this pull request dance won't be needed any more.


/alertraw command

A command that lets you make a network wide alert using tellraw format :3

Let me know if you get any errors with the new command :3


Add Scroll-To-Top Button

Implementing a Scroll-To-Top Button can provide a convenient way for users to quickly return to the top of the page.
Could you please assign this issue to me under GSSoC'24?

explain expected behavior with Screenshot.

Yeah sure, @techmannih
the expected behavior of the solution will look like this:
https://github.com/Muskansahuincredible/StudyNotion-An-Online-Education-Platform/assets/101971980/99da723f-18c3-4fa6-93c5-c1ecdeb9008e

@techmannih, @Muskansahuincredible Please assign this issue

hey @techmannih please take a look at it and assign this issue to me as I've shown the expected behaviour of the feature which adds visually appealing view and can improve user experience.

thank you for assigning this to me @techmannih and I hope that you guys have solved your issue regarding repo.


[DartFix] omit_local_variable_types doesn't work with final or const

Example
When running dartfix --fix=omit_local_variable_types on the following snippet
final bool foo = true;
const double bar = 3.0;

Expected
final foo = true;
const bar = 3.0;

Actual
final var foo = true;
const var bar = 3.0;

Which produces errors.
Version
Dart VM version: 2.9.0-11.0.dev.flutter-e007545c42 (be) (Tue May 26 22:17:42 2020 +0000) on "macos_x64"

https://dart-review.googlesource.com/c/sdk/+/149361


Updating README.md

Hey!!
I am a beginner here and I would like to work on the README of this project and make changes/additions to the Contributing section

Hey Atharva ,
I have made a sample contribution sample guide for the same previously ,
I have added them to this project,
hope you can improve documentation of the project 🤗

Thanks for the reply brother!
I really like your project.
My idea is to restructure the README a bit to make it visually appealing, should I send a PR with my changes?

Thanks for the reply brother!
I really like your project. My idea is to restructure the README a bit to make it visually appealing, should I send a PR with my changes?

cool👍


Build and deploy shared services as part of make tre-deploy

PR for issue
Closes https://github.com/microsoft/AzureTRE/issues/1556
How is this addressed

[x] Made changes to the script register_bundle_with_api.sh to register shared services and also optionally deploy shared services
[x] Add make targets for building, registering and deploying shared services
[x] Made changes to Porter bundles to make them deployable (e.g. template_schema.json was missing in some)
[x] Pass location from resource processor to porter bundles as several bundles depend on it
[x] Manually test e2e in an environment

@marrobi we also have this task on adding shared services to E2E tests: https://github.com/microsoft/AzureTRE/issues/1533
Although @damoodamoo raised a point that this might not be necessary if they are already part of the build, as is done here.
I think they need to be in the build to preserve the behaviour (as in, shared services get deployed as part of make tre-deploy). I can also see the argument of adding some tests for consistency. What do you guys think is best?

/test

/test

/test

/test

After reviewing this on a proper screen I think we should chat about several aspects:

Including the "deploy" action in the flow (through the make file and register-... script) only asks the API to deploy it. There's no feedback loop that checks if the shared service has actually been deployed.
This can create issues when (for example) you call the "install" for the firewall and on the next step, try to install nexus as the firewall data terraform object might fail.
It doesn't work for me - I tried to deploy a new TRE and then run make firewall-install, the deployment eventually failed and rendered the system unreachable as the app gateway started to return http 502.
Looking at the docker logs, looks like TF has crashed without completing the deployment.
I haven't gone through the API code, but what will happen when another make firewall-install is issued? Will it save another resource?
A small thing - why didn't you take the location from the core resource group as it was before this PR? You anyway depend on its name to be right, so using a data object looks fine...

This is great progress on a really thorny piece of work!
What's the expected behaviour for TRE's with a firewall already deployed - will this just post to the API and then essentially no-op? We'll also need to think about passing the firewall rules in via properties - but that's perhaps left to a new task.

@damoodamoo

What's the expected behaviour for TRE's with a firewall already deployed - will this just post to the API and then essentially no-op?

Right now yes, it will no-op at Terraform level.
Perhaps I should change the logic so that API fails when trying to deploy the same shared service of the same version.

We'll also need to think about passing the firewall rules in via properties - but that's perhaps left to a new task.

Yes, let's make that a separate task (perhaps the "Demonstrate updating firewall rule" line you have in item https://github.com/microsoft/AzureTRE/issues/1244 can be extracted into a task for this).

@tamirkamara

It doesn't work for me - I tried to deploy a new TRE and then run make firewall-install, the deployment eventually failed and rendered the system unreachable as the app gateway started to return http 502.
Looking at the docker logs, looks like TF has crashed without completing the deployment.

Thanks for the bug report. I can't reproduce it in my environment -- any chance you have the docker logs saved so I can use it for debugging?
The rest of the questions are the same as David asked, I replied to them above.

@tanya-borisova sounds good - let's try and keep the scope of this work contained and we can revisit edge cases of deploying twice to the API in later work. I'm planning to pull this and test it locally today so will report back...

There is a bug in this version of PR: when resource processor is deploying the firewall shared service, after the deployment of firewall it blocks itself all traffic (as all traffic by default is denied).
I think it should be possible to solve this via creating a Firewall policy first and then providing said policy when creating the firewall. But while I work on this, I'll move the PR back to draft

/test

/test

/test

/test

/test

/test-destroy-env

/test

/test

/test

/test

/test

I don't really know what should be the way forward with this PR. The last 4 runs failed with the same error (permission denied when trying to connect to Docker socket from the build agent):

This doesn't seem to be related to the PR, but doesn't look like a flake either.
I wouldn't like to merge it without succeeding in CI first, since the changes to deployment are significant, would like to make sure it doesn't break CI for anybody else.

I don't really know what should be the way forward with this PR. The last 4 runs failed with the same error (permission denied when trying to connect to Docker socket from the build agent):  This doesn't seem to be related to the PR, but doesn't look like a flake either. I wouldn't like to merge it without succeeding in CI first, since the changes to deployment are significant, would like to make sure it doesn't break CI for anybody else.

I don't know exactly why it happens but might be related to running porter and then trying to run another docker based action. I tested it on a side branch and was able to replicate it easily so merging it like so would mean that all CI validation would be fail going forward.
On the positive side, I suggest to split the shared services deployment activities into a publish step and a register/deploy step much like we do with the other bundles. This accomplishes 2 things:

It makes the pipeline complete successfully. An example run: https://github.com/microsoft/AzureTRE/actions/runs/2092453109
It reduces the total run time, as publishing (and building) the bundles runs in parallel to the deploy-core steps
The change is here: https://github.com/microsoft/AzureTRE/commit/215af780502ea6689da224a82ab902deeee498a0

@tamirkamara I think your proposal is a good idea, at the very least it achieves consistency with the other bundles, as well as reducing the build time, so I will merge these changes in. Thanks!

might be related to running porter and then trying to run another docker based action

Porter runs on resource processor, where the error that I linked is from running docker login on the build agent. So I don't think this would explain what's going on.
@stuartleeks  and I will try to take a closer look at why the build agent error is happening, but that shouldn't prevent this PR from getting merged.

/test

Successful CI run (I needed to run it with deploy_tre_branch.yml because this PR changes the workflows too)

/test-force-approve


Update openid_signing.md

This is a fix for https://github.com/sigstore/docs/issues/163
Summary
Remove duplicate and obsolete info from file.
Release Note
NONE
Documentation
Docs only fix

@jonvnadelberg is this the same content as in #164?


Config changes to reflect the default branch from master to main.  Please merge to complete the default branch renaming job.

Auto branch renaming content.

Docs Build status updates of commit 5592f28:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch adbb4f29-aaf9-42bc-8ba8-b4eca1d764df has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Remove LibreNMS container from Terraform

I just realized that Terraform likes to nuke guests if certain changes are made.  This would be REALLY REALLY BAD for the LibreNMS container because @sbarbaro has already done a TON of work in that container.
Remediation plan:

Get this merged
Backup existing container
Push this to prod, allow Terraform to nuke container in progress
Restore from backup

hail satan


Dashboard

hello again,
it's seems like my friend cannot change anything in the bot dashboard, i tried it myself on my own server and the dashboard doesn't work. i can only see a small loading bar on top-right corner of my screen that disappears after few seconds, but doesn't save the change i made.
btw is there a way to put ALL text into a single text channel (for example affect $osu, $ow, etc. and momo responds to a single text channel)

Can you post what shows up on the console when someone tries to access the dashboard?

There isn't a way to restrict commands to a certain channel. You'll have to edit Momo's permissions in the server and channels

Thanks for the Answer paul
1.https://puu.sh/ywajl.png Screenshot of command prompt Highlight = access dashboard + trying to modify welcome message.

so in short, i have to modify the momox.x.x.jar file?
Thanks for the help

missclick sorry lol

That error is saying you don't have a Reddit api key set in the properties file. As for restricting momo to one channel, you can just set it so momo can't read any channel but the one you want using discord permissions

hey @Flat,
i was checking in bot.properties there are no set for $osu, $ow commands. so i guess it's include rewrite?

Yes there is no way to currently set commands to specific channels.

@Flat no way or no other way? otherwise if i have to rewrite i would gladly try to work on it.
As well as expanding $osu command with more information, but i never heard of a .jar file editor? maybe do you got an hint?

There is no way. You can restrict momo to one channel via Discord. If you want the ability to restrict a command to one channel it will have to be added to momo. You will need to know how to program in Java to implement this - or hope someone else adds the feature in the future.

@Flat thanks for the answer.


when two page have the same css file, the grunt-cache-bust.json DO NOT contain the css file

HTML file one:
<html>
<head>
<link rel="stylesheet" type="text/css" href="../css/item.css" />
</head>
</html>

HTML file two:
<html>
<head>
<link rel="stylesheet" type="text/css" href="../css/item.css" />
</head>
</html>

The css file item.css does not exist in grunt-cache-bust.json

Very strange. Thanks for the heads up

I've tested this in feature/v1.0.0-beta branch and is working fine


exposing common_handlers fixed in docs

Took me quite some time to spot this while I debugged my own code ;).

@GiantCrocodile I'm sorry about that.
I've fixed this with 5dcf7f1 and drafted a new bugfix release, now you can use
app.common_handlers = [Handler()]
app.common_helpers = [Helper()]

as documented.
I'm gonna close this since I've added the functionality and corrected the docs with the opposite behavior.


test php 7.2 on travis

Coverage remained the same at 94.253% when pulling 2f95f39153ab9a71db7c2077cab25a878867e174 on php_72 into 1e7144c9f4c7592f8b68b9aaf2081026e3cce1a9 on master.


Clean up demo project files

Clean up demo project files
Description
Cleans up the Solution Explorer view of the demos, and makes them consistent.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run checks

/bot run checks

/bot run checks

/bot run checks


add vThreshold in meta. fix #1542

Overview
Added vThreshold to meta in workflow.
@lauramaxwell can you apply to other workflows?
Test Notes/Sample Code
Only difference from test code in #1545 is that dfMetrics is derived as follows:
dfMetrics <- wf_metrics %>% map_df(function(wf){
  wf$meta$vThreshold <- paste(wf$meta$vThreshold, collapse = ",")
  return(wf$meta)
})

Full updated script is:
devtools::load_all()

# Prepare Workflows
wf_mapping <- MakeWorkflowList(strNames="mapping")[[1]]
wf_metrics <- MakeWorkflowList(strNames=paste0("kri",sprintf("%04d", 1:12)))
dfMetrics <- wf_metrics %>% map_df(function(wf){
  wf$meta$vThreshold <- paste(wf$meta$vThreshold, collapse = ",")
  return(wf$meta)
})

# Import Site+Study Metadata
dfStudy<-clindata::ctms_study %>% rename(StudyID = protocol_number)
dfSite<- clindata::ctms_site %>% rename(SiteID = site_num)

# Pull Raw Data - this will overwrite the previous data pull
lRaw <- gsm::UseClindata(
  list(
    "dfSUBJ" = "clindata::rawplus_dm",
    "dfAE" = "clindata::rawplus_ae",
    "dfPD" = "clindata::ctms_protdev",
    "dfLB" = "clindata::rawplus_lb",
    "dfSTUDCOMP" = "clindata::rawplus_studcomp",
    "dfSDRGCOMP" = "clindata::rawplus_sdrgcomp",
    "dfDATACHG" = "clindata::edc_data_points",
    "dfDATAENT" = "clindata::edc_data_pages",
    "dfQUERY" = "clindata::edc_queries",
    "dfENROLL" = "clindata::rawplus_enroll"
  )
)

# Create Mapped Data
lMapped <- RunWorkflow(lWorkflow = wf_mapping, lData = lRaw)$lData

# Run Metrics
lResults <-wf_metrics[1] %>% map(~RunWorkflow(lWorkflow=.x, lData=lMapped))

dfBounds <- lResults %>%
  imap_dfr(~.x$lData$dfBounds %>% mutate(MetricID = .y)) %>%
  mutate(StudyID = "ABC-123") %>%
  mutate(SnapshotDate = Sys.Date())

dfSummary <- lResults %>%
  imap_dfr(~.x$lData$dfSummary %>% mutate(MetricID = .y)) %>%
  mutate(StudyID = "ABC-123") %>%
  mutate(SnapshotDate = Sys.Date())

# Create charts
lCharts <- unique(dfSummary$MetricID) %>% map(~Visualize_Metric(
    dfSummary = dfSummary,
    dfBounds = dfBounds,
    dfSite = dfSite,
    dfMetrics = dfMetrics,
    strMetricID = .x
  )
) %>% setNames(unique(dfSummary$MetricID))

# Run reports
strOutpath <- "/Users/jwildfire/Documents/github/gsm-playground/gsm_site_report_db.html"
Report_KRI( lCharts = lCharts, dfSummary = dfSummary,  dfSite = dfSite, dfStudy = dfStudy, dfMetrics = dfMetrics, strOutpath = strOutpath )

Notes:

This will also close #1541

cc: @samussiah Note the updated approach for threshold. For reporting/charts, there will be a comma separated list available at dfMetrics$vThreshold that can be parsed as needed.


Francetv: use m3u8 when f4m video isn't available

Fixes #5981
I didn't investigate why f4m format doesn't work.

@dstftw  These modifications could be merged: there no reason to fail if one available format raises an error.


Fix deleting pool with members leaves member network config on bigip

@richbrowne
What issues does this address?
#582 Deleting pool with members leaves member network config on bigip
What's this change do?
Rearranges how pool members are deleted and when subnet hints are processed,
to ensure that subnets and associated network config is cleaned up after deleting
a pool that has members.
Where should the reviewer start?
lbaas_builder.py
Any background context?
No.

Please create the PR's for mitaka and newton and I will merge all at the same time.


Large repo push http  : consumption of ram

I tried to push a large repository via http. Everything goes well until the transfer is finished eating ram becomes important (Gogs). It's a shame Gogs is so light yet in other cases. To see if this is not a technical limitation caused by the protocol, but I think it is possible to optimize.

https://github.com/gogits/gogs/blob/master/routers/repo/http.go#L362 input, err = ioutil.ReadAll(reqBody) is the line that makes Gogs use a lot of RAM when pushing/pulling over HTTP. Not sure why that block is required either to be honest @Unknwon ?

@bkcsoft good point! Maybe you could try to use a continuous reader see if it works?

I can try :)

This will be a much larger task than I anticipated... Involving goroutines and channels :cry:

Excited to see movement here :)

@Unknwon is there a reason for not using the post-recieve-hook instead of the callback that requires that line? seems like it would be way easier to implement it correctly that way :smiley:

@bkcsoft no worries, I'll take a deep look. Not sure why does it matter with post-recieve hook?

Clean up the code for http.go a bit: https://github.com/gogits/gogs/commit/c041273dd334fc7a45ca6cb67ea2c2b492c4a5ed
Still working on the input, err = ioutil.ReadAll(reqBody) thing.

The issue is the callback is reading refname, oldsum and newsum from the http-request. This is why it needs to store it (unless you do magic), unless you use the post-recieve-hook which would give us this automagically :)
Also, I'm still working on this :laughing:

@bkcsoft oops, hope does not break your code too much.... :sweat:

@Unknwon some breakage, easy to fix though :smiley:
and I need to refactor anyhow :laughing:

I took a look and wondering why ReadAll is needed at all.
The code is finally creating a Reader out of the buffer, why not directly using reqBody as the Reader ?
It is originally an io.ReadCloser...

This is fixed by #3748.

Just implemented Git hook delegation, so this problem can be ultimately solved by post-receive hook. Reopen as a reminder for myself.

This problem should be completely solve by https://github.com/gogits/gogs/commit/d521e716dd59617dbbb637a3e8028bf4a5c6f849.


Flags and Flag Categories

Summary
Adds the flags and flag categories. Needs a lot of clean up. Barebones add categories and flag resources. The nav has been updated to include link for flagged resources. The
Zube Card Referenced 60
Routes to Check

/flags: Displays the organizations that has flags (really, the flags and their associated organization by org id)
/flag_categories: Displays the categories w/ link to edit (modify) and form to add category

Note
Modified the app generator to generate in erb files and no stylesheets
Migrations to Run: Yes, should have two tables
TODO

[ ] Clean up views and controllers

Also probably a good idea to include at least some basic unit tests on this.


A question

I am sorry to interrupt you ,I  am very intersted in your paper,but I can not open this link,Could you share the 360-Degree Video Head Movement Dataset? Thanks

Hi, I think the dataset has become private. Need to contact the original authors for the dataset.


fix node build a bit

Pre-merge checklist

[X] I am very sure this PR could not affect performance.

We might end up doing reverts or what not, but adding missing dist seems like a good idea

bors r+

@eatonphil cannibalized this PR to also add your fix for intel macs. Now we only need to deal with go build failure :weary:

bors r+

bors r+


Update documentation

This should go in

the YABA repo's README
the BETYdb docs.
These docs should be referenced in the PEcAn API documentation

User content:

Link to interface
Description/screenshots of data tables/shapefiles (see #54), point out example data

Developer content:

Endpoints
How to deploy interface

Done in #69


Market Order for MarketAPI

Hi Hanhha,
If I'm not mistaken you didn't implemented market orders in your Market API. I can only see limit orders ?

No, I didn't. At that time I only did what was safe :D


405 not Allowed error on POST call with gulp proxy

Expected behavior
POST call currectly redirect to proxy port 9000
Actual behavior
redirect errror : 405 (Method Not Allowed)
Setup

http-proxy-middleware: 0.15.0
server: connect/express/browser-sync... + "express": "^4.13.4"
other relevant modules

Gulp task
gulp.task('connect', ['styles'], function() {

  var serveStatic = require('serve-static');
  var serveIndex = require('serve-index');
  var app = require('connect')()
    .use(require('connect-livereload')({port: 35729}))
    .use(serveStatic('.tmp'))
    .use(serveStatic('app'))
    // paths to bower_components should be relative to the current file
    // e.g. in app/index.html you should use ../bower_components
    .use('/bower_components', serveStatic('bower_components'))
    .use(serveIndex('app'))
    .use(require('express')());

  //////////////// PROXY START ////////////////

  // https://github.com/chimurai/http-proxy-middleware#context-matching
  var proxy = require('http-proxy-middleware');
  var options = {
    target: 'http://localhost:9000',
    changeOrigin: true
  };
  app.use('/api', proxy(options));

  //////////////// PROXY END ////////////////

  require('http').createServer(app)
    .listen(8000)
    .on('listening', function() {
      console.log('Started connect web server on http://localhost:8000');
    });
});

Stackoverflow answer: http://stackoverflow.com/questions/37302712/405-not-allowed-error-on-post-call-with-gulp-proxy

I've made a minimal example for connect:
https://github.com/chimurai/http-proxy-middleware/blob/master/examples/connect/index.js
Try moving the proxy up, instead of configuring as the last middleware.
Not really sure what this is doing: .use(require('express')());

Thank you for your clean example @chimurai, i removed require('express')() (not really sure about it too) but nothing changed, then i also removed serveIndex('app') and everithing seem to work fine now.
maybe there was some conflict with serveStatic('app').


SVN Checkout Issue

Thanks for this docker container, trying to get it going but have this issue.
Getting this error when doing a docker-compose up
svn: E170013: Unable to connect to a repository at URL 'svn://svn.code.sf.net/p/sauerbraten/code'
svn: E210005: No repository found in 'svn://svn.code.sf.net/p/sauerbraten/code'
ERROR: Service 'srv' failed to build: The command '/bin/sh -c svn checkout svn://svn.code.sf.net/p/sauerbraten/code/ srv' returned a non-zero code: 1

I've got the same issue, please let me know if you managed to fix it.
I tried changing svn:// to http:// and https:// but then I got a truncated http response body instead


Update domains-admin-takeover.md

Adding a note that points out that you'll need to have prior access to the unmanaged directory if you're attempting to do an internal takeover. Otherwise, you'll need to perform an external takeover.
The documentation doesn't specifically point this out, leaving some folks to believe they can somehow create an account using an unmanaged/unverified domain and subsequently gaining admin access via PowerBI.

@barclayn

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Merge two Sorted Arrays

Kindly approve the pull request as soon as possible!!!
Thank You !!

Please kindly commits the code as mentioned in ReadMe.


Firingrate norm for BVCs should be unscaled

I noticed that the BVC firingrates weren't scaling correctly if max_fr is not 1. I've traced the source of the issue to self.cell_fr_norm. This value is initialized based on scaled firingrates, but then applied before scaling when calling get_state(), and thus cancels out the subsequent scaling. See L1543 and L1679 of Neurons.py
import numpy as np

from ratinabox import Agent, Environment
from ratinabox.Neurons import BoundaryVectorCells

MAX_FR = 1

np.random.seed(10)

env = Environment()
ag = Agent(env)
BVCs = BoundaryVectorCells(ag, params={"min_fr": 0, "max_fr": MAX_FR})

for i in range (500):
    ag.update()
    BVCs.update()

min = np.asarray(BVCs.history["firingrate"]).min()
max = np.asarray(BVCs.history["firingrate"]).max()

print(f"{min:.4f} to {max:.4f}")

gives 0.0000 to 1.0165 (FYI: you can see there's a slight overshoot of the max, which might be for another issue.)
But, if you set MAX_FR = 10, you still get 0.0000 to 1.0165.
I'll create a PR in a moment to propose a solution.

Minor edit: I hadn't pulled the latest version. The error is still present, but the exact values for both MAX_FR values are 0.0000 to 1.0088.

Oh yes I see that is a bit of a bug, it totally cancels itself out. Not sure how I'd missed that/when it crept in. I'll take a look at your PR and leave this open until it's solved


lowBytesIncrease.py returning abnormally on OSX

First off, thank you for your work on honggfuzz.
I am having problems getting the external fuzzer to work on a fresh install of honggfuzz on High Sierra.
I am running the command sh run-honggfuzz-on-badcode1-with-externalfuzzer.sh after building the badcode binary and receiving the following error message:
[2019-07-15T07:48:32-0400][E][11505] subproc_System():446 Command '../externalfuzzers/lowBytesIncrease.py' returned with exit code 1 [2019-07-15T07:48:32-0400][E][11505] input_prepareExternalFile():377 Subprocess '../externalfuzzers/lowBytesIncrease.py' returned abnormally [2019-07-15T07:48:32-0400][E][11505] fuzz_fetchInput():331 input_prepareFileExternally() failed [2019-07-15T07:48:32-0400][F][11505] fuzz_fuzzLoop():395 Cound't prepare input for fuzzing
The run-honggfuzz-on-badcode1.sh script works as expected, it is only the external fuzzer that is failing. I am seeing that lowBytesIncrease.py script is receiving an argument of /dev/fd/7. When using the external fuzzer with other Mac default apps, for example Safari, I am seeing Safari trying to open /dev/fd/7 when called by the fuzzer.

I don't work with MacOSX myself, but does it have /dev/fd at all?

Yes, it does have /dev/fd

Could you try fuzzing with -Q, then the lowBytesIncrease.py script will tell you what the problem is. But, generally, this script has been submitted by someone loong time ago, what's your goal with using it?

Hmm, so this is the output I got with -Q
`$ ../../honggfuzz -Q -n1 -u -f inputfiles -c ../externalfuzzers/lowBytesIncrease.py -- targets/badcode1 FILE
cmdline:'targets/badcode1 FILE', bin:'targets/badcode1' inputDir:'inputfiles', fuzzStdin:false, mutationsPerRun:6, externalCommand:'../externalfuzzers/lowBytesIncrease.py', timeout:10, mutationsMax:0, threadsMax:1
------------------------[  0 days 00 hrs 00 mins 00 secs ]----------------------
Iterations : 3
Mode [3/3] : Feedback Driven Mode
Target : targets/badcode1 FILE
Threads : 1, CPUs: 2, CPU%: 0% [0%/CPU]
Speed : 6/sec [avg: 0]
Crashes : 0 [unique: 0, blacklist: 0, verified: 0]
Timeouts : 0 [10 sec]
Corpus Size : 1, max: 8192 bytes, init: 2 files
Cov Update : 0 days 00 hrs 00 mins 00 secs ago
Coverage : edge: 0 pc: 0 cmp: 0
---------------------------------- [ LOGS ] ------------------/ honggfuzz 1.9 /-
[2019-07-16T07:42:09-0400][E][918] subproc_System():446 Command '../externalfuzzers/lowBytesIncrease.py' returned with exit code 1
[2019-07-16T07:42:09-0400][E][918] input_prepareExternalFile():377 Subprocess '../externalfuzzers/lowBytesIncrease.py' returned abnormally
[2019-07-16T07:42:09-0400][E][918] fuzz_fetchInput():331 input_prepareFileExternally() failed
[2019-07-16T07:42:09-0400][F][918] fuzz_fuzzLoop():395 Cound't prepare input for fuzzing`
But it actually works if I just take out -c ../externalfuzzers/lowBytesIncrease.py
I'm brand new to the fuzzing world, but my end goal is to fuzz some of Apple's PrivateFrameworks (/System/Library/PrivateFrameworks/). I eventually want to do instrumented fuzzing against the libraries, but Apple doesn't supply headers for their PrivateFrameworks, so I'm trying to work around that. I figured I would start external fuzzing in the meantime, e.g. against their default PDF viewer Preview (which loads, for example, the CorePDF private framework), and can just be launched with something like /Applications/Preview.app/Contents/MacOS/Preview /path/to/file.pdf. Which brings me back to the original problem, where Preview is trying to open /dev/fd/7 instead of a file in the corpus directory.

Spring cleaning, please re-open if needed

also encountering unexplainable sporadic problems with the /dev/fd functionality on osx


Do not create NSDate with incorrect 0 value

Not sure about this, but just an idea. Creating NSDate with a timeIntervalSince1970 = 0 is misleading, especially in Swift. If it does not exist we should use nil as the default value.

Sounds good to me! I think the failing test is here: https://github.com/AppHubPlatform/apphub-ios/blob/master/AppHubIntegration/AppHubIntegrationTests/AppHubIntegrationTests.m#L68
We can just change it to be:
view.accessibilityLabel isEqualToString:@"buildIdentifier:LOCALbuildName:LOCALbuildDescription:This build was downloaded from the App Store.buildCreatedAt:buildCompatibleIOSVersions:1.0"]

Not very pretty, I know.
Once we merge this, will release v0.3.0 to cocoapods/npm as well. :ship:

@arbesfeld updated. Rebased on top of previous pull request commit.


Fix Encrypted Notes Rust and improve the README file

Currently, the Rust canister panicks with [Canister r7inp-6aaaa-aaaaa-aaabq-cai] Panicked at ‘already mutably borrowed: BorrowError', src/encrypted_notes_rust/src/lib.rs:149:77).
To mitigate this, I propose removing the users_invariant checks from the user_count function. My reasoning is this: (1) IMHO, it’s not possible to corrupt the canister state via the public API to violate this invariant right now, and (2) enabling this check at each call of user_count would require refactoring the code to avoid borrowing already borrowed data. I don’t think it’s worth implementing such a refactoring at this point.
Additionally, the PR adds instructions on how to run Rust unit tests and tidies up the Dockerfile.

Tested the local deployment with sh ./deploy_locally.sh and afterwards executed Rust tests docker exec container_id cargo test. The app works on http://localhost:3000/ and 2 Rust unit tests have succeeded.


Components

Second Milestone

Add calculator components.

thank you 🙏🏽  @dasileker for your time, reviewing my code.


VPP TCP stack "secret" support

communicate the secret from CRI shim to VPP agent

not applicable anymore


API integration

I want to be able to use h2ogpt as an API service, how would I be able to do that? I am sorry as I am new to the project and am unsure if questions go here.

in particular, in what manner is the ingestion invoked in the gradio UI so that I could be able to model that as a REST endpoint

There is a gradio client for all operations: https://github.com/h2oai/h2ogpt/blob/main/docs/README_CLIENT.md
For some basic ops see: https://github.com/h2oai/h2ogpt/blob/main/docs/README_CLIENT.md#h2ogpt-gradio-wrapper
For advanced ops see "steps3" test that goes through many things with extra asserts you don't need: https://github.com/h2oai/h2ogpt/blob/main/tests/test_client_calls.py#L2101
There is no REST API apart from just chat as given in the curl test: https://github.com/h2oai/h2ogpt/blob/main/docs/README_CLIENT.md#curl-client-api


We need a brand

We can't keep languagedepot.org. It's also confusing between LF and LD (comment from Chris). Per a conversation with Mike we think it would be confusing to use the name Language Forge for this new version of Language Depot.
Add suggestions to this issue. A good logo would also be nice.
criteria:

must have an available domain name
LF and LD get confused, so we'd like to avoid that again

EDIT from @myieye:
Some more things to consider:

Beneficial if it's not too similar to Language Forge (e.g. avoid the word "Language")
Cool names don't always need to be super descriptive (e.g. Twitter, Bloom, The Combine)
It would be nice to have a name that commuicates vision rather than just describing the tool

Off-the-cuff idea, and I don't know that I like it much because basic geometric shapes with letters on them already feature in so many of our products... but maybe a cube with the letters L, E, and X on the three visible sides of the cube?

I like Garden and Library, though one thing Chris mentioned when I talked to him is that there's a lot of confusion between Language Forge and Language Depot, just because the names are so similar, so something that gets away from Language something would be better at giving it a distinct title. I kinda like Dictionary Depot because it keeps Depot in the name for users to track that it's the same thing that it used to be.

Some other names I came up with

codex depot
dictionary depot

I kind of like Language Locker (trademark available, .com squatted, .org free, .app free)
Pros:

Makes you think of securely keeping your data
Not dictionary specific

Cons:

Doesn't make you think of sharing the data

Please submit any new suggestions by the 9th, and submit a ranked list of 3 names from this issue by the 11th. Next Friday we will discuss them and try to find our favorite name as a group using that ranked list of names.

How about LexVerse?

I also like LexDrive

Some ideas from ChatGPT, or inspired from ChatGPT ideas:
FolkFusion
FolkLoom
LexLoom
LexWeave
LexLore
LexiLink
LexiWeave
LingoLoom
LingoWeave
LingoNet
TalkTrove
I tried to get it to give me some single-concept ideas like Twitter or Bloom, but it just regurgitated more of the same 😂

LexNest

Looking at this issue again to see what the options are I have two thoughts:

How much of the confusion between Language Depot and Language Forge came because of the Language part, and how much because of inadequate branding on Language Depot.
Names that focus on lexical data kind of leaves OSE in the cold

Yeah we talked about OSE, it's kinda the outlier as far as data storage goes so it does make naming difficult if everything needs to fit under the name.
As far as the confusion, I think it's just the similarity in the name as even people very well acquainted with the products slip up and use the wrong name.

Alright, we might be getting out of hand with all these suggestions. Let's keep it to 5 per person.

Here's a few more:
WordWeave LingoLink LinguaLoom
Here are my sessions if anyone is curious
https://chat.openai.com/share/900e523f-0a1d-4333-a308-2422fb017fef
https://chat.openai.com/share/88610493-ec5e-44c3-ba1e-1c53618b4f18
I need to do some more vetting to check domain availability.
Also, I imagine we need to check if these terms are trademarked

@alex-larkin You're a lot more polite than me when you interact with ChatGPT 😆

My two-cents, for more single-concept ideas:

glossia.org (Greek for "language" or "tongue")
vernacular.org (I'm iffy on this, but it is available)
arborofgod.org (for garden theme!)

I'm a big fan of glossia. It's flashy, concise, and mysterious in a good way. Great brand name. It's almost too big for LD, haha, but I would totally swipe it.

@josephmyers glossia is really catchy.
Can you give us a link regarding the Greek meaning?
It seems to me that "Glossa" is a Greek work, but "glossia" is an English suffix based on "Glossa".

I really like Glossia too.
Regarding ChatGPT politeness, I've tried being more curt but the cognitive dissonance it causes me makes me prefer to put in the occasional "please", even though I know it's not needed 😂

"Vernacular" and it's cognates in other languages can have a negative connotation. Also, at least in minority language contexts, it's a less common term that not everyone knows. So those factors make me hesitant to use that term.

After paring down my suggestions, I checked the last seven or so for trademarks and domain availability. Then I did a Google search on the remaining terms to see what was out there. After that filtering, I am left with just one suggestion:
LingoWeave
The .com, .org, and .net domains are free. I couldn't find any US trademarks (after a quick search on their database) and I didn't get any relevant Google search results for either "Lingoweave" or "Lingo weave".
I affirm that I really like some of the other names ya'll have submitted, and will probably be ranking them highly too!

I just googled "glossia in greek," and Google picked it up, but it does appear that the actual Greek is "glossa," like you said. More accurate to say it's the suffix for "language" and "tongue," coming from the Greek root. (It's still a great pick, though :D)

"Vernacular" and it's cognates in other languages can have a negative connotation. Also, at least in minority language contexts, it's a less common term that not everyone knows. So those factors make me hesitant to use that term.

Yeah, I was iffy on it, too. Maybe that's why it's available...

Final list of names:

Glossia there's already a trade mark for this name
LDrive
Open Ark
[The Open] Language Garden
Language Locker

We've got a brand! LexBox is it. Now we need to update some stuff for the new name


Report due date should be shown in a more readable format

Report Due Date on the reports dashboard is being shown with a long timestamp. It should be shown in a more user friendly format:

2024-05-31 or
May 31 2024

All this needs is a datetime function on the page to translate it from its timestamped database value.

Fixed. Closing.


Invert execution orders of post, on_error and shutdown middleware hooks

Hi,
I've further noticed that we could introduce a slight improvement to make middlewares follow the "onion" execution order. For example, if a message is wrapped with pre_send of multiple middlewares, it's generally better to run post_send in reversed order.
I've then inverted the order of post_execute, shutdown, on_error, and post_save so that all existing middleware hooks have an onion-ish invoke order.
A practical use case is that some of my middlewares do modifications to add a mark in pre_send and remove it in post_send. However, without this improvement I couldn't make the mark invisible to other middlewares.
The unit tests I added could fail because of other issues: https://github.com/taskiq-python/taskiq/pull/358 and https://github.com/taskiq-python/taskiq/pull/357
As always, let me know if you have suggestions or concerns, and thanks for the review in advance!

Please fix lints and tests before the review.


Provide option to discard moduleId: module.id from newly created components

Since there is an option to run and build application with webpack, there should be option to not add moduleId: module.id in components.
As suggestion I would rely on nativescript-dev-webpack package to check if it moduleId: module.id should be added.

Thanks for the suggestions, @ph55! Since NativeScript 6.0 is webpack-only, nativescript-schematics won't add moduleId to the components metadata starting with version 0.7.0 (yet to be released).


Crash (in Python) on an URL from www.msnbc.com

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2019.01.16. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[ x] I've verified and I assure that I'm running youtube-dl 2019.01.16

Before submitting an issue make sure you have:

[x ] At least skimmed through the README, most notably the FAQ and BUGS sections
[x ] Searched the bugtracker for similar issues including closed ones
[ x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2019.01.16
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

youtube-dl -v 'https://www.msnbc.com/hardball/watch/bill-maher-trump-should-be-impeached-1426509891917?cid=sm_npd_ms_fb_hb'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.msnbc.com/hardball/watch/bill-maher-trump-should-be-impeached-1426509891917?cid=sm_npd_ms_fb_hb']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.01.16
[debug] Python version 2.7.14 (CPython) - Darwin-18.2.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.3.4, ffprobe 3.3.4, rtmpdump 2.4
[debug] Proxy map: {}
[ThePlatformFeed]<PHONE_NUMBER>917: Downloading JSON metadata
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 174, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/usr/local/bin/youtube-dl/main.py", line 19, in
File "/usr/local/bin/youtube-dl/youtube_dl/init.py", line 472, in main
File "/usr/local/bin/youtube-dl/youtube_dl/init.py", line 462, in _real_main
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2002, in download
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 804, in extract_info
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 870, in process_ie_result
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 793, in extract_info
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 508, in extract
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/theplatform.py", line 411, in _real_extract
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/theplatform.py", line 345, in _extract_feed_info
IndexError: list index out of range
Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
youtube-dl crashes when given the following URL (that plays OK in a browser):
https://www.msnbc.com/hardball/watch/bill-maher-trump-should-be-impeached-1426509891917?cid=sm_npd_ms_fb_hb

I'm also getting this on MSNBC.
youtube-dl -v 'https://www.msnbc.com/rachel-maddow/watch/barr-scrambles-to-rewind-spying-conspiracy-theory-embrace-1489359427929'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.msnbc.com/rachel-maddow/watch/barr-scrambles-to-rewind-spying-conspiracy-theory-embrace-1489359427929']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.04.24
[debug] Python version 3.7.3 (CPython) - Linux-5.0.0-0.1-MANJARO-x86_64-with-arch-Manjaro-Linux
[debug] exe versions: ffmpeg 4.1.3, ffprobe 4.1.3, rtmpdump 2.4
[debug] Proxy map: {}
[ThePlatformFeed]<PHONE_NUMBER>929: Downloading JSON metadata
Traceback (most recent call last):
  File "/usr/bin/youtube-dl", line 11, in <module>
    load_entry_point('youtube-dl==2019.4.24', 'console_scripts', 'youtube-dl')()
  File "/usr/lib/python3.7/site-packages/youtube_dl/__init__.py", line 474, in main
    _real_main(argv)
  File "/usr/lib/python3.7/site-packages/youtube_dl/__init__.py", line 464, in _real_main
    retcode = ydl.download(all_urls)
  File "/usr/lib/python3.7/site-packages/youtube_dl/YoutubeDL.py", line 2008, in download
    url, force_generic_extractor=self.params.get('force_generic_extractor', False))
  File "/usr/lib/python3.7/site-packages/youtube_dl/YoutubeDL.py", line 807, in extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "/usr/lib/python3.7/site-packages/youtube_dl/YoutubeDL.py", line 873, in process_ie_result
    extra_info=extra_info, download=False, process=False)
  File "/usr/lib/python3.7/site-packages/youtube_dl/YoutubeDL.py", line 796, in extract_info
    ie_result = ie.extract(url)
  File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/common.py", line 529, in extract
    ie_result = self._real_extract(url)
  File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/theplatform.py", line 411, in _real_extract
    return self._extract_feed_info(provider_id, feed_id, filter_query, video_id)
  File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/theplatform.py", line 345, in _extract_feed_info
    entry = self._download_json(real_url, video_id)['entries'][0]
IndexError: list index out of range

I can confirm this is still an issue with version 2020.09.20

$ youtube-dl --verbose https://www.msnbc.com/hardball/watch/bill-maher-trump-should-be-impeached-1426509891917?cid=sm_npd_ms_fb_hb
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'https://www.msnbc.com/hardball/watch/bill-maher-trump-should-be-impeached-1426509891917?cid=sm_npd_ms_fb_hb']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.09.20
[debug] Python version 2.7.18rc1 (CPython) - Linux-4.19.128-microsoft-standard-x86_64-with-Ubuntu-20.04-focal
[debug] exe versions: ffmpeg 4.2.4, ffprobe 4.2.4
[debug] Proxy map: {}
[NBCNews]<PHONE_NUMBER>917: Downloading webpage
ERROR: Unable to extract bootstrap json; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 797, in extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 532, in extract
ie_result = self._real_extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/nbc.py", line 398, in _real_extract
'bootstrap json'), video_id, js_to_json)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 1010, in _search_regex
raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract bootstrap json; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

It seems that it's trying to use the "nbc" extractor while the url is for "msnbc", a different platform.
I can imagine this is probably a regex issue but I wouldn't know where to begin with fixing this.

Fixed with the latest version(2020.11.17).


Think about reconceiving boot as a (LiftRules)=>LiftRules function

More thoughts in this ML thread.
Basic idea is that boot goes from being a no-arg function that does mutable setup, to being a function that takes an immutable LiftRules parameter and updates its values to a final immutable LiftRules that is returned and then installed as the final rules state. State mutability would be captured in things like FactoryMakers in this conception, while most values would simply be immutable.
The transition here would be interesting, and this may push this into being something we should do for a hypothetical Lift 4, potentially.

Another interesting thought here would be breaking down the large number of LiftRules settings into separate immutable classes, which in turn can get their own boot functions. That also involves the discussion of whether it would be a net good to break out LiftRules into multiple sub-things, though, and there's never been a clear consensus one way or the other.


use openblas instead of lapack for slycot testing

This PR fixes a problem in Travis CI testing against the slycot test case.  In the latest version of slycot (0.3.3), we switched to use OpenBLAS as the default in slycot/setup.py and so the previous code (which used lapack) was failing its tests.  The change is just to add installation of openblas for the slycot test case.

Coverage remained the same at 77.882% when pulling eb89ac96e6cf99d166e67328c164482cc7418949 on murrayrm:fix_travis-slycot into 601b58152080d89575cc677474ec7714e1a34ee2 on python-control:master.


feat: publish releases to luarocks

neotest is a common dependency across Neovim plugins. Using luarocks may alleviate the need for users to specify their plugins' dependencies in their plugin manager. (e.g., vim-plug or packer).
See also:
https://teto.github.io/posts/2021-09-17-neovim-plugin-luarocks.html

If this MR is accepted, a package on www.luarocks.org will need to be added, which can be done from only one account. See https://github.com/luarocks/luarocks-site/issues/173.

Very cool thanks for the PR! One question on the copied directories, do you know if this is this handled by packer by adding them to the RTP? I'd be worried that if a user installs it from luarocks, they wouldn't see the docs

Very cool thanks for the PR! One question on the copied directories, do you know if this is this handled by packer by adding them to the RTP? I'd be worried that if a user installs it from luarocks, they wouldn't see the docs

Good question.
Packer uses hererocks under the hood, and there's an open issue to change that.
This blog post seems to suggest that packer does (or will?) copy directories in the copy_directories directive to the installation root.

OK just wanted to check thank you. One thing, I've tried adding nvim-treesitter as a dependency but it fails with Error fetching file: Failed downloading http://github.com/nvim-treesitter/nvim-treesitter/archive/vscm.zip - failed downloading http://github.com/nvim-treesitter/nvim-treesitter/archive/vscm.zip, is that an issue specific to nvim-treesitter? Is that supposed to resolve to something? The corresponding URL in the rockspec in this PR doesn't resolve to anything either

It also seems the version of plenary that is available is outdated (missing the filetype module) and so isn't compatible. It seems like the scm version is a manually uploaded version, not an automatic zip of the latest code as I had first assumed. I think for a proper implementation, I'll need to alter the existing release workflow to push a luarock release. We'll also need the other libraries to do the same before this can be used by users

Yup. I'm facing similar issues with one of my repos that I'm trying to make compatible with luarocks. Let's mark this as draft for now, and I'll get back to it when I know more and have a solution.

It seems like the the scm version is a manually uploaded version,

The scm version installs the one from the "source control manager", git usually. So it should install the latest version. Problem is you have to tell luarocks to install dev/"scm" versions otherwise it doesn't see them.
Looks like there are issues with the nvim-treesitter rockspec (and the git tags appeared after the rockspec, starting with neovim 0.8). I've tried to fix the issues to upload a stable rockspec to luarocks:
https://github.com/teto/nvim-treesitter/pull/new/release-rockspec-0.8.1
but I still have an issue: with the "builtin" build type of luarocks, luarocks tries to  compile the test files of nvim-treesitter and some of them are broken on purpose which makes the install so I am not sure which way to go to work around that (changing the build type removes the 'builtin' install which helps skip lots of boilerplate). I will ask on luarocks. Thanks for the PR.

so I've gone the Makefile way and have a working build at https://github.com/teto/nvim-treesitter/tree/release-rockspec-0.8.1. Will try to merge this in nvim-treesitter/test it in nixpkgs

@rcarriga plenary.nvim now has an automated release workflow :partying_face:
https://github.com/nvim-lua/plenary.nvim/pull/439

so I've gone the Makefile way and have a working build at https://github.com/teto/nvim-treesitter/tree/release-rockspec-0.8.1. Will try to merge this in nvim-treesitter/test it in nixpkgs

@teto I'll merge your makefile approach with the automated release workflow and open a PR to nvim-treesitter.

Awesome to hear! I'll try get some time to integrate an automated release with the semantic release neotest already has. I'd love to have versioned releases :grin:

I have just published a GitHub action to simplify LuaRocks releases.

I have updated this PR with a workflow that automatically publishes to LuaRocks whenever a release is published, along with a LuaRocks badge in the readme (assuming the existence of a nvim-neotest/neotest package).
Notes

For the release workflow to work, someone with a Luarocks account will have to add their API key to this repo's GitHub actions secrets.
Due to a shortcoming in LuaRocks (https://github.com/luarocks/luarocks-site/issues/188), the neovim label has to be added manually, for this plugin to show up in https://luarocks.org/labels/neovim

Adding the API key (screen shot)

Hey apologies for not looking at this recently, but thanks for keeping it maintained and improving it! I love having it all handled by the action :grin:

Hmm it doesn't seem to have triggered, any idea why?

Hmm it doesn't seem to have triggered, any idea why?

That's strange. I've been having a similar issue in which my release workflow doesn't publish the action to the marketplace (it just creates a github release).
Maybe the published trigger isn't the right one for this workflow. I'll debug it in my fork and open another PR with a fix.


.dbeaver folders created in projects without DBeaver nature

Description
System information:
Windows 10
DBeaver IDE <IP_ADDRESS>909221748 installed into existing Eclipse 4.14 install
Describe the problem you're observing:
On startup of eclipse, DBeaver scans the workspace and creates .dbeaver folders in every project that does not contain one.
The issue was supposed to be fixed by #6968 some years ago.
DBeaver Version
DBeaver IDE f. Eclipse V<IP_ADDRESS>211061524
Operating System
Windows 10
Database and driver
irrelevant
Steps to reproduce

Import a project without DBeaver nature
Folder .dbeaver will be created - expectation: only if project has Dbeaver nature

Additional context
No response

Hi, could you, please, look if the .dbeaver folder contains any files?

It contains only a file project-settings.json. Content ist just {"id":""}

I can also confirm the above is correct. The .dbeaver files are created in all projects whereas I would only expect dbeaver to interact with the active project.

#17349


Refactor IBC testing package part 1

Description

ref: #106

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md)
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[x] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[x] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Note sure why this is failing now:
go test
--- FAIL: TestKeeperTestSuite (2.13s)
    --- FAIL: TestKeeperTestSuite/TestUpdateClientEventEmission (0.05s)
        client_test.go:622:
            	Error Trace:	client_test.go:622
            	Error:      	Not equal:
            	            	expected: &types.Header{SignedHeader:(*types.SignedHeader)(0xc0013142f0), ValidatorSet:(*types.ValidatorSet)(0xc00184a600), TrustedHeight:types.Height{RevisionNumber:0x0, RevisionHeight:0x2}, TrustedValidators:(*types.ValidatorSet)(0xc0019864b0)}
            	            	actual  : &types.Header{SignedHeader:(*types.SignedHeader)(0xc001874570), ValidatorSet:(*types.ValidatorSet)(0xc001b23680), TrustedHeight:types.Height{RevisionNumber:0x0, RevisionHeight:0x2}, TrustedValidators:(*types.ValidatorSet)(0xc001b236b0)}

            	            	Diff:
            	            	--- Expected
            	            	+++ Actual
            	            	@@ -10,13 +10,5 @@
            	            	    Time: (time.Time) {
            	            	-    wall: (uint64)<PHONE_NUMBER>7271170818,
            	            	-    ext: (int64)<PHONE_NUMBER>2,
            	            	-    loc: (*time.Location)({
            	            	-     name: (string) "",
            	            	-     zone: ([]time.zone) <nil>,
            	            	-     tx: ([]time.zoneTrans) <nil>,
            	            	-     extend: (string) "",
            	            	-     cacheStart: (int64) 0,
            	            	-     cacheEnd: (int64) 0,
            	            	-     cacheZone: (*time.zone)(<nil>)
            	            	-    })
            	            	+    wall: (uint64) 89770754,
            	            	+    ext: (int64)<PHONE_NUMBER>9,
            	            	+    loc: (*time.Location)(<nil>)
            	            	    },
            	            	@@ -96,13 +88,5 @@
            	            	      Timestamp: (time.Time) {
            	            	-      wall: (uint64)<PHONE_NUMBER>7271170818,
            	            	-      ext: (int64)<PHONE_NUMBER>2,
            	            	-      loc: (*time.Location)({
            	            	-       name: (string) "",
            	            	-       zone: ([]time.zone) <nil>,
            	            	-       tx: ([]time.zoneTrans) <nil>,
            	            	-       extend: (string) "",
            	            	-       cacheStart: (int64) 0,
            	            	-       cacheEnd: (int64) 0,
            	            	-       cacheZone: (*time.zone)(<nil>)
            	            	-      })
            	            	+      wall: (uint64) 89770754,
            	            	+      ext: (int64)<PHONE_NUMBER>9,
            	            	+      loc: (*time.Location)(<nil>)
            	            	      },
            	Test:       	TestKeeperTestSuite/TestUpdateClientEventEmission
FAIL
exit status 1
FAIL	github.com/cosmos/ibc-go/modules/core/02-client/keeper	2.176s

The IBC testing refactor will be a multi part refactor.
Part 1 introduces the new design of the testing package

global time held by coordinator and chains only need to update time before executing messages
paths/endpoints are added. Most functionality will happen via endpoints and relaying will happen via paths

Part 2 refactor tests to use the new design and remove all the old functions
Part 3 will implement a TestingApp interface so outside repos can use the testing package
Part 4 will update docs and do polishing as needed

I've tested that SetupNew works (this will be renamed to Setup when the old functions are removed)

Codecov Report

Merging #114 (bf0e153) into main (4e145d8) will decrease coverage by 1.42%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #114      +/-   ##
==========================================
- Coverage   65.22%   63.79%   -1.43%
==========================================
  Files         127      130       +3
  Lines        8413     8601     +188
==========================================
  Hits         5487     5487
- Misses       2561     2749     +188
  Partials      365      365

Impacted Files
Coverage Δ

testing/chain.go
2.53% <0.00%> (-0.02%)
:arrow_down:

testing/coordinator.go
0.00% <0.00%> (ø)

testing/endpoint.go
0.00% <0.00%> (ø)

testing/events.go
0.00% <0.00%> (ø)

testing/path.go
0.00% <0.00%> (ø)

testing/simapp/test_helpers.go
0.51% <ø> (+0.02%)
:arrow_up:


Testing

tested the folowing classes

[ ]  Book

[ ]  Person

[ ]  Student

[ ]  Teacher

[ ]  Classroom

[ ]  Rental

[ ]  trimmer

[ ]  Capitalize

Im sorry i accidentaly merged the pull reques that merge was reverted in this pull request https://github.com/andgarzonmal/ruby-school-library/pull/8


Header

Add Dan

research into fip and other requirements

Designed by @amablex for common navigation. Will need documentation and code for design system.


docker-compose not working

For dev env. it is given to copy the settings.py.
I could not make it work without the copy step in docker-compose test env., too.

Hi @wurstbrot. Thanks for sharing. What do you mean by "docker-compose test env"? What environment are you using? can you check with:
docker/setEnv.sh
or
ls -l docker-compose.override.yml
You have added the comment in the quickstart instructions which is the "release" env by default.
Also you have commited this on master.  Usually we ask to create PRs on the dev branch alghough being just a doc push it can be discussed; still will need to be integrated to dev branch is confirmed.
Thanks!

It is the same as https://github.com/DefectDojo/django-DefectDojo/pull/1474

@wurstbrot I think you probably have a docker-compose.override.yml link remaining which is why you're seing this issue.
It may be related to this: https://github.com/DefectDojo/django-DefectDojo/pull/1479
What's your OS? some OSes may checkout this file as a working symbolic link.
For me  (linux) it was checked out with in contents the name of the destination (docker-compose.override.dev.yml)
If you remove the file, you shouln't have a problem.

the missing "settings.py" issue is only present when mounting the docker volume (in dev setup) if the unix user inside of the container fails doing that copy.
the easy fix is to do the copy manually before starting the app but this is only required when the volume is mounted.
the "clean" fix is setting a coherent unix user id in Dockerfile.django but this depends on each host!
it's here:
USER 1001
that uid needs to be the one running the application on the host (see https://github.com/DefectDojo/django-DefectDojo/pull/1045#issuecomment-482736718 )
this was discussed in
https://github.com/DefectDojo/django-DefectDojo/issues/1036
https://github.com/DefectDojo/django-DefectDojo/issues/1026
https://github.com/DefectDojo/django-DefectDojo/pull/1028

you are right, for master it is working after changing fullcalendar version.


Fix Bug in Resource Ref Verifier

Fix bug in eng/common/scripts/Verify-Resource-Ref.ps1

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

Is there an issue for this or did you just spot it some other way?

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Cannot attached to container

Whenever I start a new container in interactive mode, I find myself stuck at the "Service 'w3svc' started" prompt and am unable to navigate away from there. Unless I Ctrl-C to stop the container.

You can override the entrypoint at docker run time from ServiceMonitor.exe to something else. Try this example below
docker run -it --entrypoint powershell microsoft/iis


Text editor steals focus from "Find in Files" dialog

Tested versions

Reproducible in 4.3-dev3.
Not reproducible in 4.2.1-stable.

System information
Godot v4.3.dev3 - Debian GNU/Linux 11 (bullseye) 11 - X11 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3060 Ti () - 11th Gen Intel(R) Core(TM) i5-11600K @ 3.90GHz (12 Threads)
Issue description
Text editor steals focus ~when nothing is selected before invoking~ from "Find in Files" dialog.
Steps to reproduce

Open any script. ~Don't select anything.~
Hit "Control + Shift + F" shortcut.

My attempt to fix
Tried to investigate "editor/find_in_files.cpp" ("FindInFilesDialog::_notification") and found that if replace
callable_mp((Control *)_search_text_line_edit, &Control::grab_focus).call_deferred();
with
_search_text_line_edit->grab_focus();
the problem is gone.
Not sure about "Doesn't work more than once if not deferred..." comment. It works normally subsequently (at least on my Linux with v.4.3-dev3).
Minimal reproduction project (MRP)
N/A

I also have this issue in 4.3-dev3 and not in 4.2.1-stable
I'm not at my computer right now; I'll get my exact specs later. But these should be correct -- I copied them from a recent github issue of mine:

Manjaro Linux #1 SMP PREEMPT Sat Sep 23 10:10:02 UTC 2023 - X11 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 750 Ti (nvidia; 535.129.03) - AMD FX(tm)-8350 Eight-Core Processor (8 Threads)

Bisected to 40d69c25d5b4238b6372d4c0f5fd10a8f0a64e9e #86441
CC @Mequam @dsnopek

Tried to investigate "editor/find_in_files.cpp" ("FindInFilesDialog::_notification") and found that if replace
callable_mp((Control *)_search_text_line_edit, &Control::grab_focus).call_deferred();
with
_search_text_line_edit->grab_focus();
the problem is gone.

Tested and this doesn't seem to work in current master.
CC @bruvzg @KoBeWi - maybe we can improve the Find in Files focus handling logic to make it less dependent on the timing of events.

(tried single window mode, as it can't even happen with multiple windows).

For me on Linux (Fedora 40, KDE Plasma on Wayland, X11 DisplayDriver) the issue is only reproducible with multiple windows actually, and not reproducible in single window mode.

Huh I tested with multiple windows and seems like you can focus the editor, it just doesn't accept input.
Still can't reproduce the bug though.

This has been a little tricky for me to reproduce. It's not reproducible on my system (Ubuntu 22.04) with i3 (my usual environment) or GNOME. I used multiple window mode in both cases, and tried both Forward+ and Compatibility (above, someone said it happened for them only on Forward+ and not Compatibility).
However, I just setup a Fedora 40 VM in VirtualBox, and I am able to reproduce the issue there! (using Compatibility, since getting Vulkan running in VirtualBox is a pain)
Strangely, that is also GNOME, however, it looks like it's running on Wayland and so using xwayland. I wonder if that's the difference? On my host machine, I'm using real X11 in order to take advantage of the proprietary nvidia drivers.
It'd be great if other folks who are experiencing this issue could share:

The name of their window manager or desktop environment?
Whether or not you're using real X11 or xwayland?

In any case, I'll experiment with some changes to the code that was committed in PR https://github.com/godotengine/godot/pull/86441 to see if I can find a workaround.

I reproduce the issue on Fedora 40 KDE Plasma/KWin on Wayland myself, so using XWayland. I can test tomorrow if it's indeed not reproducible on the same system with KDE X11.

I just posted draft PR https://github.com/godotengine/godot/pull/93682 which fixes the issue in my testing.
I don't know if it reintroduces the bugs that PR https://github.com/godotengine/godot/pull/86441 aimed to fix, though, and I'm going to attempt to determine that.
In any case, I'd appreciate it if other folks who are experiencing the bug described here could test if my PR fixes it for you as well!


Edit language of allowmodtorun

make it more informative and less accusatory towards any particular mod.

Thank you!


Support of alternative payment methods

Hi team,
are there any plans to support alternative payment methods such as iDeal, Bizum, Przelewy24, and others?
Thank you.

Hi @JozefGajdos we have plans to support APMS but this would be in our upcoming releases.


ChatGPT o1 models - "Unsupported value: 'stream' does not support true with this model."

When requesting on o1 models, this error shows up:
{
  "error": {
    "message": "Unsupported value: 'stream' does not support true with this model. Only the default (false) value is supported.",
    "type": "invalid_request_error",
    "param": "stream",
    "code": "unsupported_value"
  }
}

According to https://platform.openai.com/docs/guides/reasoning/beta-limitations, there's currently no support for streaming on the o1 models as of this moment.

Thanks for reporting, will look into that today

fixed in https://github.com/animalnots/BetterChatGPT-PLUS/releases/tag/v1.9.0


[10.x] Fix return type and convert function to arrow function

This PR addresses two changes in the code. First, it fixes the return type of the renderSections method from array to string. Second, it converts the function from a regular function to an arrow function for a more concise syntax.

@driesvints Then why, where the render method is used return type string?

@driesvints  class View implements the Renderable interface with a render method that returns a string


fixed non-working code example for draft-posts

What kind of changes does this PR include?

Minor content fixes (broken links, typos, etc.)

Description
Added missing .then() to the await to make the example work correctly

Hello @ChristianK43, thank you for bringing this up and your contribution! I just tested it and the example actually doesn't work, so good catch!
However, adding a .then() is actually not required in this case. The true reason for the error is that the order of operations is wrong - await must be executed before trying to call .filter().
If we add brackets to ensure the desired order of operations, the code works:
const posts = (await Astro.glob('../pages/post/*.md'))
  .filter((post) => !post.frontmatter.draft);

Alternatively, we could also use a new variable to avoid the brackets and make the code more explicit:
const posts = await Astro.glob('../pages/post/*.md');
const nonDraftPosts = posts.filter((post) => !post.frontmatter.draft);

I'd be fine with any of those alternatives. @delucis @sarah11918 what do you prefer?
Once we've found a consensus, you could update your PR accordingly and we could get this moving forward! :)

Thanks @ChristianK43 for the catch, and @hippotastic for reviewing !
I think my preference is for the two seperate statements. As you say, it more explicitly shows the filtering step.

Thanks @hippotastic  for the review and correction! I'll be happy to update my PR once you've agreed on a solution. :)

I like the second option as well 😇, very clean

One more vote here for option 2! Thanks @ChristianK43!

Ok @ChristianK43 - feel free to update your PR with the second option then and we're ready to merge! :)

done! :)

Thanks for your first contribution to the Docs, @ChristianK43 !  🥳  Welcome to the Face Pile!


PolyLine shifting can crash

make_polygons(0.5m) of PolyLine { pts: [Pt2D { x: 5217.6908, y: 5241.742 }, Pt2D { x: 5217.683, y: 5241.7357 }, Pt2D { x: 5212.1705, y: 5244.2894 }], length: Distance(6.0854) } failed: Line from Pt2D { x: 5217.6908, y: 5241.742 } to Pt2D { x: 5217.683, y: 5241.7357 } too small
Camden and other maps crash for reasons I don't understand -- the road center line is OK, but when we try to shift the polyline, the result doesn't break upfront, but it later crashes when trying to make polygons.
Short-term fix is to avoid the crash and just return a circle instead of a thickened polyline:

Actual fix is to make polyline shifting correct and robust. I never meant for my horrible math to last this long; I really want to investigate https://github.com/jbuckmccready/cavalier_contours or something else.

Another example where a bug somewhere in this implementation is causing a mess:

It's actually fine in the map_editor:

(So maybe this is the problem with trimming center points from both ends?)
It looks fine in OSM, https://www.openstreetmap.org/node/282416719#map=19/51.46319/-2.54542

trimmed_road_geometry in the map_editor is totally fine for this road, even after shifting the result. There's some difference happening elsewhere in the importer. I am baffled.

In map_editor, trimmed points are
PolyLine::new(vec![     // length 120.6564m
  Pt2D::new(2608.9265, 2146.7426),
  Pt2D::new(2726.1538, 2140.5341),    //<PHONE_NUMBER>0000001, -6.2085000000001855 (+ 117.3916m @ Angle(356.96837717355965 degrees))
  Pt2D::new(2728.6358, 2138.9885),    // 2.481999999999971, -1.5455999999999221 (+ 2.9239m @ Angle(328.08850253534797 degrees))
  Pt2D::new(2728.7598, 2138.6709),    // 0.12399999999979627, -0.31759999999985666 (+ 0.3409m @ Angle(291.32707776309627 degrees))
])

that matches importing by the end of InitialMap. OK. Over in the final map... Road.center_pts also match. Great, so the problem is somehow in the code that calculates lanes...

Here we go! For the lanes, we first shift the center points to the left edge of the road, then gradually shift it right. That's causing the problem here -- the center points are OK, shifting them left is mostly OK but winds up with that bend. If we then shift the partly bent line, the bend explodes.

This is if we shift relative to the road center. The yellow center-line is still funky, because of get_left_side...


Add support for interfaces, part 3: automatically add __typename

Summary:
In #52, I added support for interface types, but with the simplifying
restriction (among others) that the user must request the field
__typename.  In this commit, I remove this restriction.
The basic idea is simple: we preprocess the query to add __typename.
The implementation isn't much more complicated!  Although it required
some new wiring in a few places.
Issue: https://github.com/Khan/genqlient/issues/8
Test plan:
make check

Ah, good point.  I'll just make a separate issue for that; there are some other potential conflicts.


fix env variable for ML.NET telemetry

@natke @JRAlexander please let me know if you'd like me to update ms.date on this one as well

@KathleenDollard FYI


LOC CHECKIN | Microsoft/azuredatastudio master | 20190718

@kevcunnane this PR appears to be localizing branding strings.  Do you know if this is correct?

@kburtram I'm honestly not sure - we need to check with PM/Docs if big data cluster is a protected brand. I suspect so, but we should loop them in. @yualan could you help us on this?

@v-zbsail could you please resolve the merge conflicts on this PR?  Or open a new PR if that's not convenient?


added issue templates

Signed-off-by: ashiscs<EMAIL_ADDRESS>

Sir @leecalcote review required!


[PE-D][Tester C] UG does not how to add emergency contact number

User guide refers to 'field constraints'

But field constraints does not have any information

Labels: type.DocumentationBug severity.High
original: hogretup/ped#2

duplicate


Forums are down

For some reason, our forums are down. I've activated debug mode, and it seems there is a problem with accessing the database. This only seems to have happened after I made a new database, so perhaps there are user conflicts? I'm going to look into this, but first I need to know where database credentials are stored in EsoTalk. Does anyone know, and are they encrypted?

Found database info

Fixed!


GH PR: picker for repository could provide help text and should stay open on focus lost

Testing #114967
When I click on the "+" I think the picker could show a description in the input field what I have to do (pick a repository to create pr in):

It should also not close when I focus out to another window.
Example of how it works better:

Actually I wonder, maybe this is using our generic picker to pick a folder in multi repository setup?

Actually I wonder, maybe this is using our generic picker to pick a folder in multi repository setup?


mission complete

I finished the challenge
thank you

i have found a small bug in this request so i decided to close it and create a new one


[Clarification] Reduce error response code lines in oas

Currently for all services in the OAS the same error response codes are used. Is there a way to reduce the number of lines to prevent the same code showing up again and again? -> asked Iswaryaa:

duplicate, handled in #125


suggestion: add food spoilage and greenhouse block

food will spoilage after a few days
greenhouse allow growing plants in any season

I think theres already a mod that does that. pretty sure i saw that somewhere.
I will be making an expansion mod for photosynthesis which is more "techy" which will include a way to grow crops outside seasons. ive seen other mods adding a single block like a greenhouse glass block but i think thats just way too simple. so im gonna go the hard route and imo better route :P


Ensure students are added to group in EditCommand::execute

Fixes #240
To test

add group -g Test
edit 1 -g Test
show -g Test`
check that the group Test contains the student.

Codecov Report

Merging #242 (b4c956e) into master (de49801) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #242      +/-   ##
============================================
+ Coverage     66.07%   66.11%   +0.03%
- Complexity      793      794       +1
============================================
  Files           106      106
  Lines          2839     2842       +3
  Branches        376      377       +1
============================================
+ Hits           1876     1879       +3
  Misses          860      860
  Partials        103      103

Impacted Files
Coverage Δ

...java/seedu/address/logic/commands/EditCommand.java
97.18% <100.00%> (+0.12%)
: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 de49801...b4c956e. Read the comment docs.


When you will update for PHP 8 support?

please update to php 8 support if there are no issues.

See https://github.com/laravel/nexmo-notification-channel/pull/39


Update pull_request_template.md

Description
Add comments to areas where are description to what to fill, placeholders and checklist.
Fixes # (issue)
Type of change
What is it?

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update
[ ] Refactor
[x] Repo maintenance

Extra information
This is how you will see when editing it:

This is how you will see when previewing it without dropping the comments or forgetting to remove them:

Codecov Report
Base: 49.59% // Head: 49.59% // No change to project coverage :thumbsup:

Coverage data is based on head (8016b53) compared to base (f2723dd).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1842   +/-   ##
=======================================
  Coverage   49.59%   49.59%
=======================================
  Files          70       70
  Lines        9847     9847
=======================================
  Hits         4884     4884
  Misses       4499     4499
  Partials      464      464

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fix outdated link for how to edit plugin code

Current link is dead

Please don't land patches when Travis or Appveyor have gone red, even if it's a harmless error or if the PR can "obviously" not be wrong, because it makes trying to track down the problem when the PR subsequently makes the tree red much harder. For example in this case the tree was red, so I looked at the PR, saw that the Travis and Appveyor builds were red, then spent significant time tracking down those issues before realizing they were probably unrelated to the tree failure.

Sorry about that. Got it, won't do that again.


Add aaguid property to Authenticator type

As mentioned in #15, the aaguid from registrationInfo is needed to determine the type of authenticator used to save the passkey. The name of the authenticator can be determined by comparing against this file from the community sourced list of passkey authenticators.
This PR adds aaguid to the properties of the Authenticator type so implementing apps can access this information and display the passkey authenticator to the user, which is a big part of the "Managing passkeys" section of the Passkey user journeys article published by Google.

Looks good. Would you mind adding a reference to the aaguid index in the Readme somewhere? https://github.com/passkeydeveloper/passkey-authenticator-aaguids/blob/main/aaguid.json

Looks good. Would you mind adding a reference to the aaguid index in the Readme somewhere? https://github.com/passkeydeveloper/passkey-authenticator-aaguids/blob/main/aaguid.json

Absolutely, good idea. I'll handle that!

Looks good. Would you mind adding a reference to the aaguid index in the Readme somewhere? https://github.com/passkeydeveloper/passkey-authenticator-aaguids/blob/main/aaguid.json

Done in a4afeb1. Let me know what you think, happy to add anything else needed :)

Thanks for your work on all of this. I appreciate having someone who knows just a little bit more about passkeys spending the time and energy to polish this package. 😁

Published. 0.3.4

Thanks for your work on all of this. I appreciate having someone who knows just a little bit more about passkeys spending the time and energy to polish this package. 😁

My pleasure! Thanks for this package and for the quick responses on these PRs!


v18.2.0 stops syncing

Describe the issue
Can you reliably reproduce the issue?
If so, please list the steps to reproduce below:

Update 18.1.0 to 18.2.0
Watch logs and see error
Revert to 18.1.0
Watch logs and see everything is good again

Expected behaviour
Smooth sailing
Actual behaviour
repeated log messages:
ERROR: AcceptBlockHeader: block<PHONE_NUMBER>000044356e582f9748f9baf084e5b7946e6386b32620d540830fda is marked invalid
What version of Dash Core are you using?
v.18.2.0 release
Machine specs:

OS: Debian
CPU: 2 vCPU cores
RAM: 4 GB
Disk size: 100GB
Disk Type (HD/SDD): SSD

@Belkaar Recently released Dash Core v18.2.0 cannot sync past block 1800330 on mainnet. Developers are investigating, please do not upgrade to this version for now! Anyone who has already upgraded can revert to v18.1.0 and sync will complete without any further action.

Got same issue with v18.2.0


Multipart with Client / Proxy

I just upgraded to a newer hyper hoping I can use the proxy support but I need to send multipart requests which is currently only possible with a Request<Fresh> and the only way to get one of those is with Request::new, never from a client.
However it looks like the proxy support is public API way contained within the client only.
Can't say I have any ideas how to fix this though :(

Author of multipart here. You can use the Client struct with client::lazy::Multipart via the following method:  http://cybergeek94.github.io/multipart/doc/multipart/client/lazy/struct.Multipart.html#method.client_request

Going forward, hyper won't provide a proxy API built-in. All the pieces are now exposed to allow 3rd party proxy implementations, and multipart, albeit it's a fair amount of work.
I expect both proxy and multipart support to exist in https://github.com/seanmonstar/request, though they don't currently.


OCaml syntax on error messages

https://rescript-lang.org/try?code=DYUwLgBGBcHAlgZzAHmQJ3gOwOYD4IBeCACgEYAaCAJgEoBuIA
Compiling in https://rescript-lang.org/try with .res syntax.
Input code:
let t: list<string> = (1, 2);

Error message:
[E] Line 1, column 22:

This has type:
    'a * 'b
  But somewhere wanted:
    string list

Expected:
[E] Line 1, column 22:

This has type:
    ('a, 'b)
  But somewhere wanted:
     list<string>

Can't reproduce when compiling code with bsb.

Fixed in #143


Does keyd support hyper key?

hi, I use capslock as hyper key by setting in Xmodmap
> cat .Xmodmap
! remap caps_lock to Hyper_l/mod3
clear lock
keycode 66 = Hyper_L

remove mod4 = Hyper_L
add mod3 = Hyper_

I wonder: 1) can I use hyper as modifier in keyd config? I didn't see it in the manual; 2) Is there any way I can do above Xmodmap config in keyd? So that I would only keep keyd and drop the Xmodmap config

Not within keyd, because the Hyper key is an X11 concept. cf #761

Thanks! Yeah I got what I need in #370


Ui/transition screen

The loading bar on the transition screen is now a looping animation, and works on chrome and firefox

Are there other animations that can be done easily with CSS? I don't think that the looping bar works very well for the short duration we're going to have, especially if the total loading time isn't an integer multiple of the animation duration. Could we do something spinning, like the black-and-white version of this?

Most css properties that relate to size, color, and position are animatable. I can do that

just realized im dumb and accidentally put these changes in the other branch that I also just pr'd. Im closing both and fixing the loading bar into a wheel


Compile fresh OpenSSL inside dockerized env

BATCH SCORING PULL REQUEST
This is a pull request into a public repository for Batch Scoring script maintained by DataRobot.
RATIONALE
Batch Scoring script uses Docker container based on manylinux [1] to
produce pyinstaller artifacts (i.e. all-in-one binaries). It turns out
that currently PyPI uses TLS 1.2, and this security protocol is not
supported by OpenSSL distributed inside manylinux container. So when we
build our own CPython interpreter (for the reasons mentioned in
corresponding Dockerfile) and link it against default OpenSSL, the
produced interpreter does not not support TLS 1.2. Therefore, when we
try to run "pip" and other tools that deal with PyPI, they are unable to
establish connection and fail.
[1] https://github.com/pypa/manylinux
CHANGES
In order to workaround this issue, let's compile our own fresh OpenSSL
library and link the produced interpreter against it.
Alternatively, it seems like there's no reason to use manylinux as base
image at all because we don't have dependencies that link against system
libraries. However, such change may be potentially destructive and
should be considered and implemented separately. After all, this is a
fix to unblock batch scoring releases.
TESTING
cd batch-scorint
docker-compose build centos5pyinstaller
make build_release_dockerized

Coverage decreased (-0.05%) to 82.913% when pulling 7284a0b641b3c97f3c758abd418684d6aaf6a982 on ikalnytskyi/use-own-openssl into 5b5f26aa9b2b3246676131feea2cdefcff2c4f74 on master.