Macros not working in post-hooks in dbt_project.yml but working as a model config

Describe the bug
Macros are not working in post-hooks in dbt_project.yml but working as a model config
Steps To Reproduce

Create an empty dbt project with dbt init
Create a macro in macros/grants_test.sql. I'm using this for testing:

{% macro grants_test(rel) %}
ALTER TABLE {{ rel }} SET OPTIONS (description="testing...")
{% endmacro %}

Modify the dbt_project.yml to add a post-hook. Based on the sample project dbt init creates I changed it to this:

models:
  my_new_project:
      # Applies to all files under models/example/
      example:
          + post-hook: [
            '{{ grants_test(this) }}'
          ]
          materialized: view

Expected behavior
I would expect the post-hook to use the grants_test macro correctly. It actually works if instead we add it as a model config in for example: models/example/my_first_dbt_model.sql
Screenshots and log output
Running with dbt=0.19.0
Encountered an error:
Compilation Error
  Could not render {{ grants_test(this) }}: 'grants_test' is undefined

System information
Which database are you using dbt with?

[ ] postgres
[ ] redshift
[x] bigquery
[ ] snowflake
[ ] other (specify: ____________)

The output of dbt --version:
installed version: 0.19.0
   latest version: 0.19.1

Your version of dbt is out of date! You can find instructions for upgrading here:
https://docs.getdbt.com/docs/installation

Plugins:
  - redshift: 0.19.0
  - bigquery: 0.19.0
  - snowflake: 0.19.0
  - postgres: 0.19.0

The operating system you're using:
Ubuntu 20.04
The output of python --version:
Python 3.8.5
Additional context

I think I found the cause for this:
https://github.com/fishtown-analytics/dbt/issues/2583
That fixed the eager render of macros if using +post-hook in dbt_project.yml
Notice the way I'm adding a space after the plus sign +post-hook. That space is not being stripped:
https://github.com/fishtown-analytics/dbt/blob/develop/core/dbt/config/renderer.py#L150
I think this would have to be changed to (k.lstrip('+ ') if isinstance(k, str) else k)

Hey @jmriego, thanks for opening, and nice work figuring this one out!
We don't document support for any whitespace between the + prefix and config keys today. At the same time, I don't think it should cause as much grief as it does. Your proposed fix is straightforward and seems reasonable to me.
Are you interested in opening a PR to make that one-line change?

thanks @jtcohen6 ! sorry, I opened this thinking it was a bug but yes, after noticing I shouldn't have put that space after the plus sign it all made sense. Sure, I'll open a PR about this. Good as a first PR

@jtcohen6 I'm working on the fix in #3526 but for some reason that change has not fixed it completely. It avoids the error message but the hook is not run. I cannot find it in the logs either
Any ideas on anywhere else to check?


Adds User Profile Management, Book Review and Rating, Browsing History, Book Recommendations, Favorite Books.

Related Issue
Closes #2265
Fixes:  #2265
Description
[Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.]

Type of PR

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

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes. Make sure to attach before & after screenshots in your PR.]
Checklist:

[x] I have made this change from my own.
[x] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

@anuragverma108 Please assign it a level 3 as a lot of features were involved in 1 PR.

@anuragverma108 Please merge it and assign it to a greater level according to you if you feel it right

Hi @arunimaChintu, please ensure to wait for the issue to be assigned to you before proceeding next time.

Okay! Sure

@anuragverma108 Could you please assign it a level 3 , as it has a lot of features merged in a single issue

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change


chore(prql-php): Add comment and define to header file

This #define can later be used by the PHP binding for caching.
As per the PHP documentation: https://www.php.net/manual/en/ffi.load.php

The header file should contain a #define statement for the FFI_SCOPE variable, e.g.: #define FFI_SCOPE "MYLIB". Refer to the class introduction for details.

Looks great!
(anyone who knows PHP well, feel free to comment if you have thoughts)


TypeError: can only concatenate str (not "int") to str

https://github.com/att/docker-forensics/blob/379e04e5677e57bce62b5484d386c12e340797d9/mac-robber.py#L133
This fragment should be string not int ?
Tested on: Linux, Python 3.10, pystatx not installed
Traceback:
Traceback (most recent call last):
  File "/tmp/mac-robber.py.orig", line 214, in <module>
    outstr = process_item(dirpath, directory)
  File "/tmp/mac-robber.py.orig", line 147, in process_item
    md5str
TypeError: can only concatenate str (not "int") to str

Sigh... not sure how I missed casting those. It actually worked on the system where I tested it out. It is fixed now.


log dataset and model in new cli

run.sh had that and it was nice to know progress

Do you mean just printing out model/dataset/progress percent after each one?

It looks like the dataset, model, and language (but not progress percent) are already printed out if you use --verbose

opinion on removing verbose option and making it the default ?

I don't have a strong opinion one way or the other, but if we wanted to make that change, we could replace
parser_eval.add_argument('--verbose', default=False, action="store_true", help="verbose mode")

with
parser_eval.add_argument('--quiet', dest='verbose', action="store_false", help="suppress verbose messages")

to just flip the default verbosity (and also rename the cli option name correspondingly).

yeah #78


Error when starting a new site

I'm getting the following error when setting up a new site using the instructions in apero website
Error: Error building site: failed to render pages: render of "page" failed: execute of template failed: template: collection/single-series.html:7:11: executing "main" at <partial "shared/breadcrumb.html" .>: error calling partial: "xxxxx\themes\hugo-apero\layouts\partials\shared\breadcrumb.html:2:55": execute of template failed: template: partials/shared/breadcrumb.html:2:55: executing "partials/shared/breadcrumb.html" at <.UniqueID>: can't evaluate field UniqueID in type *hugolib.pageState
I'll appreciate any help to resolve this issue. I have set the options to add apero to .Rprofile
Here's the result of check_site
― Running a series of automated checks for your blogdown website project...
○ A successful check looks like this.
● [TODO] A check that needs your attention looks like this.

Let's check out your blogdown site!

― Checking config.toml

Checking "baseURL" setting for Hugo...

● [TODO] Update "baseURL" to your actual URL when ready to publish.

Checking "ignoreFiles" setting for Hugo...

○ "ignoreFiles" looks good - nothing to do here!

Checking setting for Hugo's Markdown renderer...

○ All set! Found the "unsafe" setting for goldmark.

― Check complete: config.toml

― Checking .gitignore
| Checking for items to remove...
○ Nothing to see here - found no items to remove.
| Checking for items to change...
○ Nothing to see here - found no items to change.
| Checking for items you can safely ignore...
● [TODO] You can safely add to .gitignore: .DS_Store, Thumbs.db, .hugo_build.lock
| Checking for items to ignore if you build the site on Netlify...
● [TODO] When Netlify builds your site, you can safely add to .gitignore: /public/, /resources/
| Checking for files required by blogdown but not committed...
● [TODO] Found 1 file that should be committed in GIT:
layouts/shortcodes/blogdown/postref.html
― Check complete: .gitignore
― Checking Hugo
| Checking Hugo version...
○ Found 2 versions of Hugo. You are using Hugo 0.93.0.
| Checking .Rprofile for Hugo version used by blogdown...
○ blogdown is using Hugo 0.93.0 to build site locally.
― Check complete: Hugo
― Checking netlify.toml...
○ Found HUGO_VERSION = 0.93.0 in [build] context of netlify.toml.
| Checking that remote & local Hugo versions match...
○ It's a match! Local and remote Hugo versions are identical (0.93.0).
| Checking that Netlify & local Hugo publish directories match...
○ Good to go - blogdown and Netlify are using the same publish directory: public
― Check complete: netlify.toml
― Checking content files
| Checking for validity of YAML metadata in posts...
○ All YAML metadata appears to be syntactically valid.
| Checking for previewed content that will not be published...
○ Found 0 files with future publish dates.
○ Found 0 files marked as drafts.
| Checking your R Markdown content...
○ All R Markdown files have been knitted.
○ All R Markdown output files are up to date with their source files.
| Checking for .html/.md files to clean up...
○ Found 0 duplicate .html output files.
○ Found 0 incompatible .html files to clean up.
| Checking for the unnecessary 'content/' directory in theme...
○ Great! Your theme does not contain the content/ directory.
― Check complete: Content

after executing serve_site()

You are using Hugo 0.93 which doesn't allow Page.UniqueID that's used in the theme (see issue #76). An easy solve will be to manually change Hugo version to (say) 0.89.0. (It's recommended to pin Hugo version too.)
To change Hugo version, go to netlify.toml and change HUGO_VERSION.
[build]
command = "hugo"
publish = "public"
[build.environment]
HUGO_VERSION = "0.89.0"

That worked great! Thank you so much.
For the record,
I installed hugo 0.89.0 with
install_hugo(   version = "0.89",   extended = TRUE,   arch = "auto",   os = "auto",   force = TRUE, )
Then set options with options(blogdown.hugo.version = "0.89.0")


How can I mint multiple NFTs in one transaction?

due high length of instruction for a single minting. it's not possible to mint multiple NFTs under a single TXN yet.

And how to mint in multiple transactions?

Isn't there any way of approving only once for multiple NFTs minting?


⚠ Bofrost.de has degraded performance

In d0fe561, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1585 ms

Resolved: Bofrost.de performance has improved in f108e68 after 23 minutes.


Title bar flashes when editing and onfocus/blur of the window

Prerequisites

[x] Can you reproduce the problem with Debug -> Reload Without Extensions?
[x] Did you perform a cursory search to see if your bug or enhancement is already reported?
[x] Did you read the Troubleshooting guide?

For more information on how to write a good bug report read here
For more information on how to contribute read here
Description
Hello,
After updating to v1.13 I noticed that the title bar started flashing whenever I edit or focus/blur the editor; this did not occur prior. It almost appears like it's constantly losing/applying the window style.
Steps to Reproduce

Open some files
Edit the files and save/undo so the title is updated (displays/hides the "unsaved changes" icon) OR give focus to another window and then refocus the editor.
Repeat step 2

Expected behavior: Modest UI that doesn't flash me.
Actual behavior: Shamelessly flashes me.
Example: https://www.youtube.com/watch?v=CPk84XGO7Ek
Versions

Windows 10 Home
Brackets v.1.13

Additionally, here's my potato's specs.

AMD Athlon II X2 235e Dual-Core Processor (2.70 GHz)
NVIDIA GeForce 6150SE Integrated
6GB DDR2 memory

Thanks for reporting this @SethClydesdale. We will investigate this from our side.
@sobisht Is this anyway related to the fix that went this w.r.t window full resize on multi monitor? We could try reverting the fix and see if that reduces the window flashing.

@ayushrathi15 Could you check and recreate this from our side? More importantly please check if this is happening with 1.12.

@nethip It appears to be an issue with 1.13. I reverted back to 1.12 just to make sure and it's definitely more stable. There's no notable changes to the title bar when editing, focusing, or blurring.
The only thing I did notice was a small white line appearing under the menu bar (file, edit, find..) briefly when focusing/blurring the window; not when editing. It's not as noticeable as the title bar issue in 1.13, in fact I'd say it's quite minor, but I thought I'd mention it anyway just in case it's related.

@SethClydesdale Thanks for your observations. We will work on the fix for the issue.

Hey, been awhile!
I decided to test the latest version (v1.14.2) and I can confirm that this issue is still persisting. I disabled all addons, defaults included, so they're definitely not the causes. I am wondering if perhaps this issue was caused by the border radius that was added to the title bar in v1.13? Because that's the only notable change I've noticed to it.
Though I've reverted back to v1.12, again, I'm curious to test this theory. Is there any way to modify/remove the border radius on the title bar?

I have this issue too

I have this issue too

Welcome to the club!! Don't think it'll ever get a fix, unfortunately. Only solution is probably a new PC, because I know mine is pretty outdated.

Welcome to the club!! Don't think it'll ever get a fix, unfortunately. Only solution is probably a new PC, because I know mine is pretty outdated.


Linux support

Could you also provide Linux executables?

I second this.

Could you also provide Linux executables?
I second this.

I don't use Linux all that often, however if enough requests do come in for Linux, I may make a build for said Platform.
Alot of Functions use the Windows API so some work will def need to be done, before it could release.


Scbe align for k8s (flexvolume umount and also provisioner size)

This change is

@midoblgsm  I created the PR so u can review the code so far.
i still need to add more unit testing commit tomorrow and add the vendor/ubiquity util also inside the ubiquity original repository.
just review it for now and tomorrow i will apply the above as well.

Review status: 0 of 8 files reviewed at latest revision, 3 unresolved discussions.

controller/controller.go, line 229 at r1 (raw file):

	// Validate that the mountpoint is a symlink as ubiquity expect it to be
	realMountPoint, err := c.exec.EvalSymlinks(unmountRequest.MountPath)

This should be implemented in ubiquity, otherwise it will be broken in the next glide up

controller/controller.go, line 253 at r1 (raw file):

		c.logger.Println(fmt.Sprintf("Removing the slink [%s] to the real mountpoint [%s]", unmountRequest.MountPath, realMountPoint))
		err := c.exec.Remove(unmountRequest.MountPath)

This should be implemented in ubiquity

vendor/github.com/IBM/ubiquity/fakes/fake_executor.go, line 1 at r1 (raw file):

Please do not change anything inside the vendor directory, all your changes will get lost in the next glide up
All changes should go to ubiquity repo then bump ubiquity-k8s to the latest ubiquity
Once this is done, don't forget to point glide.yml to the right branch of ubiquity

Comments from Reviewable

Review status: 0 of 8 files reviewed at latest revision, 3 unresolved discussions.

controller/controller.go, line 229 at r1 (raw file):
Previously, midoblgsm (Mohamed Mohamed) wrote…
This should be implemented in ubiquity, otherwise it will be broken in the next glide up

yes, this what i told u in the comment above. i am now pushing the relevat to ubiquity and update it in k8s side.

controller/controller.go, line 253 at r1 (raw file):
Previously, midoblgsm (Mohamed Mohamed) wrote…
This should be implemented in ubiquity

same as above

Comments from Reviewable

Reviewed 6 of 8 files at r1, 3 of 3 files at r2.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

Comments from Reviewable


vscode ipad support plan

I have a question about vscode for IpadOS plan
Sent from PPHub

And what is the question?


Implementing automated deployment

Implementing a deployment routing:
-> live deployment when creating a new tag (or updating the master branch)
-> staging deployment when pushing to / merging into the develop branch

-> linting / formatting


[BUG] unable to import knn searchPipeline and ml methods

What is the bug?
A clear and concise description of the bug.
the methods in the knn, searchPipeline and ml are not available for import
How can one reproduce the bug?
Steps to reproduce the behavior.

install opensearch package in project
no ml, searchPipeline or knn imports available

What is the expected behavior?
A clear and concise description of what you expected to happen.

should have methods available for import

What is your host/environment?
Operating system, version.
"@opensearch-project/opensearch": "~2.13.0"
Do you have any screenshots?
If applicable, add screenshots to help explain your problem.

Do you have any additional context?
Add any other context about the problem.

@toyaokeke which version of the client are you using? We're testing the 3.0.0 beta that has these endpoints generated from the OpenSearch Spec.

@toyaokeke which version of the client are you using? We're testing the 3.0.0 beta that has these endpoints generated from the OpenSearch Spec.

hi @nhtruong , thank you for the quick response 🙏🏿
I am using version 2.13.0

@toyaokeke Can you please try the 3.0 beta? Does this fix your problem?

@toyaokeke Can you please try the 3.0 beta? Does it fix your problem?

appears to work as expected. is there an ETA of when there will be a non-beta release?

The current beta version is going to be very close to the final version. We're trying to get more people to test the beta before releasing it. We're aiming to release by year end.

perfect, thank you! if you like we can mark this as closed since it will be resolved with the latest release


Addressed: Make grid column count a variable #60

Set variable in theme-variables {{columnCount}} -- this is populated through the cs-grid. Have done an initial HS check to make sure it is populating/generating according to the column number.

@TheWebTech For some reason the ' + 1' wasn't rendering when in the logic loops. The |add() function did the trick... so I stuck with it.


Sign the product payload before creating it

[x] use window.nostr.signEvent to sign using Browser extension

Refer https://github.com/nostr-protocol/nips/blob/master/07.md

I am creating an event after creating the product, which might change later


Are you model could be trained on GPU

Hi! I try to run train process, but it's too slow couse of CPU training.
I've found this line in reqiuirments: tensorflow-gpu==1.11.0.
And in readme:

I have trained on an instance with 1 NVIDIA GTX 1080Ti (11GB VRAM) and it takes approximately 14 hours.

Could you please share training code for GPU? Thanks in advance for your reply.

actually the code is GPU training already.
On Tue, Sep 10, 2019 at 12:47 AM Makhaon<EMAIL_ADDRESS>wrote:

Hi! I try to run train process, but it's too slow couse of CPU training.
I've found this line in reqiuirments: tensorflow-gpu==1.11.0.
And in readme:
I have trained on an instance with 1 NVIDIA GTX 1080Ti (11GB VRAM) and it
takes approximately 14 hours.
Could you please share training code for GPU? Thanks in advance for your
reply.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hmchuong/ML-BoneSuppression/issues/5?email_source=notifications&email_token=ADYUTQ5O5GSUTZLMMTVL72LQI2D2LA5CNFSM4IU6FZQ2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HKHU75Q,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADYUTQ6PKTGLTI56HEZZILLQI2D2LANCNFSM4IU6FZQQ
.

--
Chương M. Huỳnh
*Email<EMAIL_ADDRESS>Skype: minhchuong.ryanhuynh
*Phone: *+84962332808


matrix is singular error

Hi, I am using the example's files to test the code.
When I take a 100MB file with 250 data shards and 125 pairity shards and then delete shards 0-49 and 270-279 reconstruction fails with matrix is singular error. Is this suppose to happen? Cause I thought I can lose up to 125 shards and still recover the data.
What am I missing?
Thanks, Amit

Also happens for random 40kb buffer and some other files. For fewer pairity/shards (e.g 50/20) error doesn't occur. Is the likelihood of singularity rises when the size of the matrix rises?

it seems the  rows/cols idx exceeds byte len when generating Vandermonde matrix within Galois Field
https://github.com/klauspost/reedsolomon/blob/master/matrix.go#L271

It would seem that to be safe from this the number of data+parity shards should not exceed 255.
I did some tests, which only seemed to fail if the number of data shards where >255, but it seems like there is another limitation that kicks in when reconstructing.

Hi,
Thanks for the answer.
I think I missed the point of how to avoid this or that maybe it's a bug?
Are you saying that shards+parity cannot excced 255? Cause I didn't infer
that from the documentation. (And also there's no error for those
dimensions)
Sorry for my lack of understanding.
On Apr 10, 2016 21:30, "Klaus Post"<EMAIL_ADDRESS>wrote:

It would seem that to be safe from this the number of data+parity shards
should not exceed 255.
I did some tests, which only seemed to fail if the number of data shards
where >255, but it seems like there is another limitation that kicks in
when reconstructing.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/klauspost/reedsolomon/issues/16#issuecomment-208041209

Are you saying that shards+parity cannot excced 255?

That would be my recommendation. I will do some more tests, but for now, I would stick to that.

Thank you very much
On Apr 10, 2016 21:50, "Klaus Post"<EMAIL_ADDRESS>wrote:

Are you saying that shards+parity cannot excced 255?
That would be my recommendation. I will do some more tests, but for now, I
would stick to that.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/klauspost/reedsolomon/issues/16#issuecomment-208042959

AFAIK - (k + m) cannot be bigger than Galois field GF(2^8) - 1

ok so shouldn't we add a check at the constructor?

That would make sense, yes -  as well as updating the documentation


add 2.2 commits and update os sup doc

@vivmishra - if needed, please add a commits doc link to the 2.2 release notes.


Route action still fires with triggersEnter redirect

This example given on the master branch fails with SSR
FlowRouter.route('/', {
  triggersEnter: [function(context, redirect) {
    redirect('/some-other-path');
  }],
  action: function(_params) {
    throw new Error("this should not get called");
  }
});

if it's helpful, here's my similar code that lead me to test this:
FlowRouter.route('/login-required', {
  name: 'Login Required',
  triggersEnter: [loginRequired],
  action: () => {
    ReactLayout.render(MainLayout, {
      main() { return
You shouldn't see me.
; }
    });
  }
});

function loginRequired(context, redirect) {
  if (!Meteor.user()) {
    redirect('/');
  }
}

Yep. SSR version of triggers and redirects are not handled well.
Tracking them here: https://github.com/kadirahq/flow-router/issues/313


vbump asde deployment extension

0.3.0 will be released soon, vbump the extension.

Pull Request Test Coverage Report for Build 323396583

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

Totals

Change from base Build 322982665:
0.0%

Covered Lines:
20536

Relevant Lines:
47410

💛  - Coveralls


Cleaned formatting a bit and added more Command Aliases (i really lov…

Hi nvbn,
I really love theFuck, and I started using it a while ago. I just wanted to be a small part of it!

Should we close some of these?

This is one negative aspect of Hacktoberfest 😞 Thanks for the reminder, @davidhart82! 👍


[workloadmeta/mock] Use only one mock

What does this PR do?
There are 2 mocks defined in workloadmeta (workloadMetaMock and workloadMetaMockV2) which is a bit confusing. The first re-implements a lot of the functionality whereas the second is just a thin wrapper with a few functions that are useful for testing. I believe V2 is a better option because it's closer to the real implementation and simpler to reason about.
This PR switches all callers to use V2 and deletes V1 to have a single mock. There were some functions that were only defined in V1, but callers were already migrated to V2 in this previous PR: https://github.com/DataDog/datadog-agent/pull/27076
For reviewers: your code is now calling workloadmetafxmock.MockModule() instead of workloadmetafxmock.MockModuleV2() but it's the same thing because after deleting MockModule, I renamed MockModuleV2 to MockModule.
Describe how to test/QA your changes
Skip. It's just tests.

/merge


lxd/init: Improve error messages when failing to bind an address

Note for reviewers (especially @stgraber): I realized that now that we try to listen to the configured network address in lxd init, the behavior is not idempotent anymore. If you try to run again lxd init and use the same network address you'll get an error. Not sure if we care about this.
Signed-off-by: Free Ekanayaka<EMAIL_ADDRESS>

I think that's okay, we never really pretended to be idempotent, storage certainly isn't :)


App crashing ramdomly

Initial RCA: XFixesCreateRegion throwing BadRegion error...root cause for this error unknown.

Solution: Changing the process to get the Damaged area.

Avoiding the use of XDamageSubtract, XFixesCreateRegion and XFixesFetchRegion APIs.

Fetching the Damaged area using event.

Creating branch BadRegionFix for this issue.

Solution 2: re-using the same region id.
create a single region using 'XFixesCreateRegion' and reuse it for every damage event.

Solution2 accepted.
Initial test: looks good ... not crashed at 30FPS for 3hrs.(playing youtube constantly, with random keyboard & mouse input)
stability check: pending :+1:
final test : pending :+1:


Implement DNAME synthesis validation

I think I misunderstood this, as long as the correct EDNS stuff is sent in queries this should never have to be done (...I think).


list zeroswap

https://zeroswap.io/
#3220AC
https://medium.com/zeroswaplabs/step-by-step-guide-to-mint-zee-bep20-with-the-binance-smart-chain-bridge-a74e476a67ad
https://etherscan.io//address/0x104B9B1c41c6764e88EDf1207f498902d840fe2A

not general bridge...


gst-resource-error-quark: base_inference plugin intitialization failed (2)"

Tried running the application with the GPU by updating my pipeline file.Please find the part of the pipeline.
template": "urisourcebin uri="{source[uri]}" ! concat name=c ! decodebin ! video/x-raw ! videoconvert name="videoconvert" ! gvadetect inference-id=inf0 model="{models[person_detection][1][network]}" model-proc="{models[person_detection][1][proc]}" device=GPU name="detection" ! gvametaconvert converter=json method=detection source="{source[uri]}" name="jsonmetaconvert" ! tee name=t ! queue ! gvawatermark ! videoconvert ! autovideosink t. ! queue ! appsink name=appsink",
When i checked the log file i observed the following issue
{"levelname": "ERROR", "asctime": "2019-10-23 10:47:27,566", "message": "Error on Pipeline : gst-resource-error-quark: base_inference plugin intitialization failed (2)", "name": "GSTPipeline"}
Thanks in advance.
I am able to run by adding device=CPU but for GPU i am getting the issue.

Hi @kanagala388 ,
Can you share a bit more details?
Especisally, you can answer to this questions:
Can you share your final pipeline?

Are you running that pipeline within container or in the host? (probably within container)
We need more details to help you resolve your issue.


Cosmetic(ish) - last/first name

Complete example has the labels for first and last names reversed in the GetRandomEmployees method

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

ID: acea0dac-37dd-bea4-7545-3ee37556a033
Version Independent ID: 4d4b3dbc-adf4-5f92-cec5-7393156a3535
Content: Walkthrough: Using BatchBlock and BatchedJoinBlock to Improve Efficiency
Content Source: docs/standard/parallel-programming/walkthrough-using-batchblock-and-batchedjoinblock-to-improve-efficiency.md
Product: dotnet
GitHub Login: @rpetrusha
Microsoft Alias: ronpet

Thanks for your feedback @f3 and good catch!
We're adding this to our backlog and to our community contribution project.
That code can be found at
https://github.com/dotnet/samples/blob/master/snippets/csharp/VS_Snippets_Misc/tpldataflow_batchdatabase/cs/dataflowbatchdatabase.cs
and
https://github.com/dotnet/samples/blob/master/snippets/visualbasic/VS_Snippets_Misc/tpldataflow_batchdatabase/vb/dataflowbatchdatabase.vb

Closing - see dotnet/samples #673


Support for the new Outlets API

We have a new API now: https://stimulus.hotwired.dev/reference/outlets
Would it be possible to implement this?

I thinks its done by this commit: https://github.com/anstarovoyt/intellij-stimulus/commit/1a07f7d8bba8ec89de46965aa46281560dce8359
In version 0.5.232

@PedroAugustoRamalhoDuarte as far as I understand the example above still doesn't work.

It's true, the plugin doesn't know the class of the outlet right now so it can't auto complete the targets of the outlet yet.


plugin missed

from plugins.hackedemails import HackedEmailGrabber
ImportError: No module named hackedemails

This is pending resolution in PR #8.
Comment out all references to HackedEmailGrabber for now until the patch is merged.

At this time there is a newer version of the application which addresses this issue. Aslo note, please us Python2 for launching ./skiptracer.py


System.UnexpectedException: Note can't be saved

Notes won't convert when related to an Account or Opportunity. Notes related to Leads work fine.
Here is a snippet from the debug log:

07:49:22.0 (32469143)|FATAL_ERROR|System.UnexpectedException: Note can't be saved
Class.ConvertNotesToContentNotesService.convert: line 105, column 1
Class.ConvertNotesToContentNotesQueueable.execute: line 41, column 1

Debug Log - Note Can't Be Saved.pdf

Hi @pdxjake,
To help with troubleshooting, I've got some questions for you:
Are you getting this error consistently for all Accounts and all Opportunities in your org?
Have you been able to reproduce the issue in another org?
Is Feed Tracking enabled for Accounts and Opportunities?
Is the Feed Tracking value different for Accounts and Opportunities than for Leads?
In Setup | Notes | Notes Settings, do you have Enabled Notes checked?
If you create a new Account then a new Note then try to convert specifically for that Account do you still get the error? If yes, could you send me the debug log for that test please?
Thanks,
Doug

In troubleshooting today I found a more specific error:

First error: Note can't be saved because it contains HTML tags or unescaped characters that are not allowed in a Note.

Are you getting this error consistently for all Accounts and all Opportunities in your org?
I have no idea. Thanks to the limitations in querying on ContentDocumentLink I can't analyze the note by parent type the way I can with the old note object.
Have you been able to reproduce the issue in another org?
No. I haven't had an issue with this tool in other orgs. Given the impact, I will not be implementing this in their production org until it can function without issues.
Is Feed Tracking enabled for Accounts and Opportunities?
Yes. And Contacts and Leads.
Is the Feed Tracking value different for Accounts and Opportunities than for Leads?
There are different fields being tracked, but they are different objects. None of the objects have "All Related Objects" enabled.
In Setup | Notes | Notes Settings, do you have Enabled Notes checked?
Of course.
If you create a new Account then a new Note then try to convert specifically for that Account do you still get the error? If yes, could you send me the debug log for that test please?
I don't know how to run the job for a specific record. If you send me an example of what the execution would look like in the developer console I'll be happy to try it.

Hi @pdxjake,
Thanks for the follow up.
Here is the code snippet to run in Developer Console to perform conversion for one parent object, simply replace the sample account id 001j0000003ZS1z with your note parent id:
// default options per custom setting
//Convert_Notes_to_ContentNotes_Settings__c settings = Convert_Notes_to_ContentNotes_Settings__c.getInstance();
//ConvertNotesToContentNotesOptions options = new ConvertNotesToContentNotesOptions( settings );

// or, explicitly set options for this run
ConvertNotesToContentNotesOptions options = new ConvertNotesToContentNotesOptions();
options.parentIds = new Set<ID>{ '001j0000003ZS1z' };
//options.deleteNotesUponConversion = true|false
//options.sharePrivateNotesWithParentRecord = true|false
//options.convertIfFeedTrackingDisabledOnParentObject = true|false
//options.conversionResultEmailNotificationAddresses = list_of_email_addresses

// then run batchable
ConvertNotesToContentNotesBatchable batchable = new ConvertNotesToContentNotesBatchable( options );
Database.executeBatch( batchable, 100 );

Although Notes can include HTML, according to the Salesforce documentation here and here there are some limitations:

The content or body of the note, which can include propertly formatted HTML or plain text. Any special characters within plain text in the Content field must be escaped. You can escape special characters by calling content.escapeHtml4().

With that said, the code does try to properly escape special characters per Salesforce's direction, but there's nothing we can really do if the HTML in the note is not properly formatted (e.g. has a start tag  but no closing tag  or if the tags are not properly nested like )
For your test, if you have an idea of which Notes are the offending records, try to put them on a new Account record and try to convert that record using the code snippet above. Once you've identified the offending Note(s) if you could provide me a copy I can try to decipher what in the Note is causing the problem and whether we can handle it in the code.
Thanks!

Hi @pdxjake,
Just following up to see if you had been able to do further troubleshooting or if you've resolved the issue?
Thanks,
Doug

Hi @pdxjake,
I'm working on a new version of the Notes conversion tool at https://github.com/DouglasCAyers/sfdc-convert-notes-to-chatter-notes
I'd love to find out if you ever resolved your issue with the note content that was failing to convert. If you identified certain special characters or something that you had to adjust to make the conversions work for you so that I can either update my FAQ or provide logic in the code to handle the scenarios.
Since this project, "sfdc-convert-attachments-to-chatter-files", is solely focused on attachment conversion I'll be closing this issue.
Thanks,
Doug

Hi @DouglasCAyers ,
I did not resolve the issue but it was probably (99% sure) escaped text. I'm not sure how you'd handle escaped text errors, but an indication that that type of error was the issue - and even better an ID to the offending record - would have helped in the head-scratching.
-Jake F

Thanks for the follow up @pdxjake. I agree, would be nice if Salesforce gave us more meaningful error message. Their documentation says any save error yields System.UnexpectedException: Note can't be saved and that could mean a hundred different things =/
I'll try my best to help narrow down the problem notes in the new version of the note conversion tool.
Thanks!


fix DotArray Access for ServerCollecton, followup to #873

@pniederlag can you show example of code for that pr?

Closing as inactive.


Add command to remove prefixes

As suggested in https://github.com/sindresorhus/atom-autoprefixer/issues/59#issuecomment-492123454

Added new option Autoprefix: Remove Prefixes

Renamed "Autoprefix" to "Autoprefix: Prefix"

updated readme with all the examples

this should close #59
Any thoughts to improve this PR is welcome.

updated with upstream master branch after 4.0.2 version release.

Works great 👍


Fix integration tests GitHub Workflow

:tada: This PR is included in version 3.10.0-next.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.10.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


OAK-10660: DocumentNodeStore: avoid repeated commits of :childOrder in branch commits

This is a proof of concept. It passes the new tests.
The idea is to keep track of previous revisions in the branch, and to augment any change to _childOrder with removals of the previous commits.

Also commented on the jira : ideally it would be possible to reuse dns.getBranches().getBranch(rev) to get the list of unmerged branch commit revisions. That would even be accessible in the Commit directly, as Commit already has a reference to DocumentNodeStore. In which case the introduced wiring down to applyToDocumentStore would not be necessary.

@reschke , @mbaedke , did a draft of an alternative that doesnt need the new wiring : check https://github.com/apache/jackrabbit-oak/pull/1326


Enable GatewayWithAttachedRoutes test

Fix #1201

Codecov Report

Merging #1229 (7510abe) into main (9036533) will increase coverage by 0.10%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1229      +/-   ##
==========================================
+ Coverage   63.16%   63.27%   +0.10%
==========================================
  Files          79       79
  Lines        9831     9833       +2
==========================================
+ Hits         6210     6222      +12
+ Misses       3201     3192       -9
+ Partials      420      419       -1

Impacted Files
Coverage Δ

internal/gatewayapi/validate.go
91.29% <100.00%> (+0.03%)
:arrow_up:

... and 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


AOS action repeat 2~5times

This is:- Question
there is any options  AOS action repeat?
example data-aos="flip-down"  3 or 5 times?

Just add once: true to your AOS init. Like this:
AOS.init({
  once: true
});


Message too long for Discord

Describe the bug
Discord webhooks have a 1024 character limit for the value field. The current implementation exceeds this causing notifications to Discord to fail.
Messages sent to Discord need to be split into multiple 1024-character messages.
Reference:
https://birdie0.github.io/discord-webhooks-guide/other/field_limits.html
https://discord.com/developers/docs/resources/channel#embed-object-embed-field-structure
To Reproduce
Cause a CrashLoopBackOff on a pod that is within the monitoring scope of kwatch.
Expected behavior
A notification successfully sent to Discord via the configured webhook.
Actual behavior
Error is logged within kwatch in regards to a failed notification due to exceeding the 1024 character limit.
level=error msg="failed to send event with Discord: HTTP 400 Bad Request, {\"code\": 50035, \"errors\": {\"embeds\": {\"0\": {\"fields\": {\"6\": {\"value\": {\"_errors\": [{\"code\": \"BASE_TYPE_MAX_LENGTH\", \"message\": \"Must be 1024 or fewer in length.\"}]}}}}}}, \"message\": \"Invalid Form Body\"}"
Version/Commit
0.8.0

same issue!

you manual update the code for limit the character to 1000 character
path: alertmanager/discord/discord.go
the souce code like this

this worked for me
thank


gentx - therealssj

Please make sure your node is up and running now. Join #validators on Discord for help: https://discord.gg/QeJWCrE.


recommended dev workflow

Is there a recommended dev workflow? Some pattern with ibazel to reload on update would be very useful.
I'm new to using bazel for web dev, so not sure if there's some nicer patterns out there. I currently got a simple workflow going with the following:
load("@npm//http-server:index.bzl", "http_server")
load("@com_github_edschouten_rules_elm//elm:def.bzl", "elm_binary", "elm_library")

elm_library(
    name = "lib",
    srcs = glob(["**/*.elm"]),
    strip_import_prefix = "elm",
    deps = [
        "@elm_package_elm_browser",
    ],
)

elm_binary(
    name = "app",
    main = "App.elm",
    deps = ["lib"],
)

http_server(
    name = "server",
    data = [
        "index.html",
    ] + [":app"],
)

I can then run ibazel run server, and reference the script  in index.html.
However, each reload requires a force reload since the file is not hashed, and not sure if there's a way to get live-reload working.

I can give here my experience with dev live reload workflow:
It starts from addition of concat-js to package.json (e.g. via npm install @bazel/concatjs). Then you can add concatjs_devserver target e.g. under assets directory:
$ cat assets/BUILD.bazel
load("@npm//@bazel/concatjs:index.bzl", "concatjs_devserver")
...

concatjs_devserver(
    name = "devserver",
    deps = [":assets"],
    serving_path = "/assets/index.html",
    port = 3001,
)

However, live reloading feature that comes with concatjs_devserver is using livereload protocol which requires some client side js code (downloaded from you concatjs_devserver) included into html frontend source, e.g.:
$ cat ./assets/index.htm
...

...

This somehow worked for me.


Getting 'undefind' error evaluating 'new a.Renderer'

I've loaded Monaco inside an iFrame, the scripts are loaded using a custom URL.
I'm using the AMD version with the requirejs library.
While editing, i will randomly get this error:

It is most likely appears when Monaco tries to bring up the autocomplete box. (so when typing ' = new' in JavaScript for example). I'm getting this error on Safari and Chrome.
require.config({ paths: { 'vs': 'https://vscode.cdn.servconnect.net/vs' } });

window.MonacoEnvironment = {
    getWorkerUrl: function (workerId, label) {
        return `data:text/javascript;charset=utf-8,${encodeURIComponent(`
        self.MonacoEnvironment = {
          baseUrl: 'https://vscode.cdn.servconnect.net/'
        };
        importScripts('https://vscode.cdn.servconnect.net/vs/base/worker/workerMain.js');`
        )}`;
    }
};

require(["vs/editor/editor.main"], () => {
    monaco.languages.typescript.javascriptDefaults.setCompilerOptions({
        target: monaco.languages.typescript.ScriptTarget.ES6,
        allowNonTsExtensions: true
    });
});

I've tried implementing code documentation when Monaco is suggesting code. Which would not work. The documentation (embedded in the monaco.addExtraLib()) would not show up at all, neither did my definition document. When i click on 'Go to definition' Monaco says that there is no def. found, even tough it can suggest my monaco.addExtraLib() code correctly.

To make the whole monaco.addExtraLib() thing more clear; this is what i mean by saying the documentation does not show up. I'm getting the new a.Renderer error when Monaco tries to show the documentation.

Also, i've tried my code at Monaco Playground, and it works fine!

Well, i'm 99.9% certain this has something to do with the marked.js library. Since i had to source that file from the Marked Repo. I can't seem to find the good version because i'm still seeing the undefined error. I fixed the Web Worker myself by adding CORS rules.

A couple of days passed, and i still haven't got any clue how i'm supposed to solve this. I'm not familiar with the source code nor TypeScript in general. It's sad, but i'm ready to switch over to another code editor.
This is how i try to load Monaco:
My webpage loads an iframe and adds the loader.js from another domain. Everything works except for the documentation of my API's. It just won't show, and throws an error saying TypeError: undefined is not an object (evaluating 'new marked.Renderer') at markdownRenderer.ts:84.

Could you provide a minimal example that reproduces the error in the Monaco Playground (https://microsoft.github.io/monaco-editor/playground.html)?

Could you provide a minimal example that reproduces the error in the Monaco Playground (https://microsoft.github.io/monaco-editor/playground.html)?

I can't. My code works perfectly fine in the Monaco Playground. When i download Monaco from the website (no NPM) and try to initialize it on my website, i get an error saying that it can't find /vs/base/common/marked/marked.js. If i source marked.js myself, it shows the TypeError: undefined is not an object (evaluating 'new marked.Renderer') at markdownRenderer.ts:84 error.

So, a couple of days have passed. And i just moved to another code editor. I'm sorry, i never found the problem.


Stop one broker may cause another broker shutdown when enable function woker in broker

To Reproduce

Start pulsar cluster with 3 brokers and functionsWorkerEnabled=true
Stop the broker which is the leader of function worker and the owner of function worker coordinate topic.
Another broker will become leader and then shutdown due to fail to get coordinate topic.

Expected behavior
Should not trigger shutdown when fail to get coordinate topic.
Additional context
When stop the broker which is the leader of function woker and the owner of coordinate topic. Another broker will become leader but coordinate topic did not shift to another broker in time.
There is a schedule task runing only on leader and will get follow error:

ERROR org.apache.pulsar.functions.worker.ErrorNotifierShutdownServiceImpl - Encountered fatal error. Shutting down.
java.lang.RuntimeException: org.apache.pulsar.client.admin.PulsarAdminException: java.util.concurrent.CompletionException: org.apache.pulsar.client.admin.internal.http.AsyncHttpConnector$RetryException: Could not complete the operation. Number of retries has been exhausted. Failed reason: Connection refused: /xxx.xxx.xxx.xxx:8080
at org.apache.pulsar.functions.worker.MembershipManager.getCurrentMembership(MembershipManager.java:78)

getCurrentMembership will throw a RuntimeException when fail to get coordinate topic. And because we are using ErrorNotifierShutdownServiceImpl the broker will shutdown when get exception.
There may be two parts need to be improved:

Add error handle when get coordinate topic.
Do not use ErrorNotifierShutdownServiceImpl, it is not suitable to shutdown broker when function worker get any exception.

The issue had no activity for 30 days, mark with Stale label.


Markdown (Pre-)View Default Options

Markdown (Pre-)View Default Options
Motivation
First of all: I really really like VSCode! I tried a lot of other editor/IDEs before, but none of them felt so comfortable and smooth like this. So I dropped all other apps and started to go my way with VSCode as my "everyday-swiss-army-knife" for coding, texting, writing, note-taking and everything else that needs a text editor.
My job as a solution engineer requires a very good information- and note-taking concept and managment.
One of the most done things day in day out is to take notes.
So I started to find a good suitable note-taking app that supports standard markdown without any (properitary) encoding or something like that.
There are some good alternatives out there, but the biggest problems I have with all of them are:

I have to use a second application with another file/folder structure
I have to leave VSCode to look for my notes
I can't use my beloved VSCode workspaces for a "wiki folder"
No suitable or adaptable shortcuts
Oftentimes a bad search function

And many other little things that makes the daily work disruptive and uncomfortable.
VSCode supports markdown out-of-the-box (perfectly fine!) but with an unpleasant behavior when it comes to read-only markdown files in preview mode.
It always opens the markdown code view first and then you have to open the preview mode in addition. And that not instead of the code view but in addition to the current file as a side-by-side view. It is absolutely ok, that you have this option, but this is very uncomfortable if you want to read-only your markdown files/notes and/or use them as presentation slides!
But that is what you want most of the time...just read your markdown files in preview mode!
Related (but closed/locked) Issue
I found the following feature request I joined...

https://github.com/Microsoft/vscode/issues/54776

...but that was locked a few days ago with the following comment:

Locking as none of the comments are anything but fancy +1s.
If you are actually interested in working on this feature
and have ideas on how to implement it, please let me know
(just open a separate issue and reference this one)

Because the annotator is completely right, I'll go to try my luck here with this brand new feature request including a (hopefully) sufficient description of what I (and by the way, a lot of other users) am/are obviously missing as a feature.
Feature Description
The idea is to have the option to choose the default view of markdown files that will be open in VSCode.
This setting could be named like:

Markdown > Preview > Default

This option should be available as a general user and as a workspace setting.
User and Workspace settings

You could have 3 option to choose:

Preview Mode
Editor Mode
Side-By-Side Mode

In addition you can toggle between both modes with a button and a customizable key shortcut.
"Preview Mode" opens markdown files always in preview mode, "Editor Mode" opens markdown files always in the code view and "Side-By-Side Mode" opens always both views next to each other.
Toggle Button
Whatever mode you choose as "default" you can always toggle between the views with a toggle button.

But switching between modes means to switch the view within the same window.
For reference: this behavior is perfectly integrated in apps like...

Notable
Typora
Joplin

Keyboard shortcuts
The keyboard shortcuts definition could be like that:

Conclusion
VSCode is IMHO the best editor/IDE on the market right now. I know this is a very subjective opinion and I don't want to start another editor war here.
Such a tightly integrated feature as described above would kick out a lot of other apps and would make VSCode much more powerfull and versatile! And it would also make the lifes easier for a lot of ppeople that need to use (markdown) notes day in day out.
You can stay in VSCode most of the time and you can use the very powerful find and search options you have right now. VS Code would getting a perfect note-taking app!
Thanks for reading this request and to all of the developers and contributors! All of you are doing a great job!

Duplicate of #54776
Thanks for the writeup. The problem is not that we don't understand the feature request but that we are looking for help implementing it. if you are interested in trying this, start by taking a look at the builtin markdown extension: https://github.com/microsoft/vscode/blob/36aa903d5a98ba5cf1d46b84c0a8fb3c2106bd8a/extensions/markdown-language-features

Thanks @mjbvz for your reply!
I would really like to contribute but at the end I have to admit, that I am not that good as a programmer to do this. Maybe you could leave this issue (re-)open for the future? Maybe some other contributor has the time and knowledge to do this?


Implement the Contact Us Endpoint

Description
This PR introduces a new API endpoint to handle user inquiries submitted through the "Contact Us" form. The endpoint validates the input data, sends a request to an internal email service endpoint, and returns appropriate responses based on the success or failure of the operation.
Linked Issue
This PR addresses the following issue: Issue #101
Changes Made

API Endpoint Implementation:

Added a POST endpoint /api/v1/contact to accept inquiries.
The endpoint validates the incoming request payload to ensure all required fields (name, email, message) are present and valid.

Data Validation and Sanitization:

Used Pydantic models to validate the request payload:

name: Non-empty string.
email: Valid email address.
message: Non-empty string.

Email Sending:

Integrated with an internal email service by simulating email sending functionality within the app.

Response Handling:

On successful request to the email service, returns a 200 OK status with a success message.
On failure, returns a 500 Internal Server Error with an appropriate error message.

Logging:

Set up logging to capture the flow of operations and errors for better monitoring and debugging.
Logs are written to logs/contact_us.log with a specific format.

Testing:

Added unit tests to cover various scenarios:

Successful inquiry submission.
Email sending failure.
Invalid email format.
Missing required fields.
Empty message.

Performed load testing using k6 to ensure the endpoint can handle a high volume of successful requests.

Load Testing Results
Performed load testing with k6 to verify the endpoint can handle multiple concurrent requests efficiently. The test simulated 50 virtual users for a duration of 5 minutes. The results indicated that the endpoint can handle the load, with the majority of requests being processed successfully.
Files Changed

api/v1/routes/contact_us.py:

Implemented the API endpoint and logic to send a request to the internal email service.
Added logging for better error tracking and monitoring.

tests/test_contact_us.py:

Added unit tests to cover various scenarios for the "Contact Us" endpoint.

Documentation
API Documentation:

Endpoint: /api/v1/contact
Method: POST
Request Body:

name: (string) The name of the user.
email: (string) The email address of the user.
message: (string) The message from the user.

Example Request:
http
POST /api/v1/contact
Content-Type: application/json
{
  "name": "John Doe",
  "email"<EMAIL_ADDRESS>  "message": "Hello, I have an inquiry."
}

Successful Response:
{
  "message": "Inquiry sent successfully",
  "status": 200
}

Error Response (Invalid Email):
{
  "detail": [
    {
      "loc": ["body", "email"],
      "msg": "value is not a valid email address",
      "type": "value_error.email"
    }
  ]
}

Error Response (Email Sending Failure):
{
  "detail": "Failed to send inquiry"
}

Review and Merge
You can kindly review the changes that I have made and if there are any actions I should take, let me know. If you are satisfied with this PR you can merge it.

Resolve your conflicts

@Dev-wonderful I have resolved the conflicts now.

@Dev-wonderful I have updated my code to follow the advised instructions.

Since I don't know where and how the internal email service will be and be implemented, I have presumed that the email service file is in the services directory and I have used the following code as my email_services to simulate that the code is working properly.
import os
import requests
import logging
from dotenv import load_dotenv

load_dotenv()

# Setup logging
logger = logging.getLogger(__name__)

class EmailService:
    def __init__(self):
        self.api_key = os.getenv('BREVO_API_KEY')
        self.receiver_email = os.getenv('RECEIVER_EMAIL')

    def send_email(self, name: str, email: str, message: str) -> bool:
        url = "https://api.brevo.com/v3/smtp/email"
        headers = {
                "accept": "application/json",
                "api-key": self.api_key,
                "content-type": "application/json"
        }
        data = {
                "sender": {"name": name, "email": email},
                "to": [{"email": self.receiver_email}],
                "subject": "New Contact Us Inquiry",
                "htmlContent": f"
Name: {name}

Email: {email}

Message: {message}
"
        }

        try:
            response = requests.post(url, json=data, headers=headers)
            response.raise_for_status()
            logger.info(f"Email sent successfully to {self.receiver_email}")
            return True
        except requests.exceptions.HTTPError as err:
            logger.error(f"Failed to send email: {err}")
            return False

Removed the test file because the previous ones that were on the dev branch were removed to resolve test issues


design/software-architecture: Add what section 7. "Deployment view" covers

For all other sections, except 7. "Deployment view", it is stated what the section covers. Add what this section covers as well? The Deployment view page is currently blank, so I don't know what it will cover.
https://github.com/seedcase-project/seedcase-project/blob/d6086fe2ca077597dc7cf00db9d863055f04139e/design/software-architecture/index.qmd#L57-L61

Many of these things have been fixed, are no longer relevant, or are addressed in other issues. Closing


Properties declared on a class aren't documented until you initialize them with a value.

Make sure these boxes are checked before submitting your issue - thank you!
(If your issue is neither a both bug nor a request, please write in a free style)

[ ] https://esdoc.org/manual/usage/feature.html
[ ] https://esdoc.org/manual/configuration/config.html
[ ] https://esdoc.org/manual/usage/tags.html
[X ] https://esdoc.org/manual/advanced/api.html
[ ] https://esdoc.org/manual/faq/faq.html

Short summary of your issue
Properties declared aren't documented until you initialize them with a value.
Input data for reproducing
Configuration
{
  source: "./src",
  destination: "./doc",
  includes: [
    "\\.ts"
  ],
  excludes: [
  	".*node_modules.*",
  	"\\.d.ts",
  	"\\.js",
    "\\.spec.ts",
    "\\.svg",
    "\\.html",
    "\\.module.ts",
    "index.ts"
  ],
  lint: false,
  plugins: [
    {
      name: "esdoc-standard-plugin",
      option: {
        coverage: {
          enable: true
        }
      }
    },
    {
    	name: "esdoc-typescript-plugin",
    	option: {
    		enable: true
      }
    },
    {
      name: "esdoc-plugin-require-coverage",
      option: {
        required: 0
      }
    }
  ]
}

Codes
this doesn't generate docs for coolThing
export class GenClass {
  /**
   * @desc a cool description
   */
  @Input() coolThing: string;
}

this does generate docs for coolThing
export class GenClass {
  /**
   * @desc the type of blueprint icon that is displayed (look at blueprint docs for mapping)
   */
  @Input() coolThing: string = "";
}

Your environment

OS type and version: Mac OSX El capitan
Node.js version: 6.11.1
ESDoc version: 1.0.1

I've been looking through the source to try to find out where the check for "=" is done but haven't found it yet :/ It looks like it could even be part of Babylon? If you point me in the right direction, I can open a PR


Send Headers on Connection

Hello there,
When connecting my flutter app to rails, Im sending headers to authorize:
cable = ActionCable.Connect(constants.socketHost, headers: {
        "Authorization": constants.socketToken,
      }, onConnected: () {

Is there a way to do the same with actioncable-vue?
Thanks,

Any ideas?
Make sense if I create this issue on rails/actioncable?

I've run into the same issue. Passing authentication tokens in the url as the docs propose is widely insecure and bad practice. Apparently passing headers on the initial http upgrade request is unsupported in browsers:

I guess the next best option is to send the token over the websocket channel after connection, and then start streaming the data that requires auth.

@rafaeliga Like @jakob-lilliemarck said the best way to do this is to pass the token as a query parameter in the connection URL. e.g http://localhost:3000/cable?token=XYZ


Implement panic using 'panic_fmt' (only for nightly rust).

Coverage decreased (-0.2%) to 90.49% when pulling 0ed279823fdc9de225c65b503dcde33d40ea026c on core_panic into 40fb2007667e9c7c80c7eb163eb1cc09ee3f21c6 on master.


Using jQuery, hide all sections except the main section, then create animations to go through each section

A new file traverse.js was created to store the animation and hide the sections by default.


ClusterDescriptor

Related to https://docs.google.com/drawings/d/1-m1azdsBqMA0A6ATYRMfkhyeuOJuGCEI62N5a96LXj0/edit?usp=sharing
This should not be something which is given to UMD, rather provided by UMD.
Change the API so that ClusterDescriptor can itself run create-ethernet-map and create itself. This is used by tt_SiliconDriver (look at it as Cluster class in the diagram). Replace the constructor to call ClusterDescriptor rather than get it as an argument.

Remove test_utils::GetClusterDescYAML as part of this effort (or reuse it's code)

I've completed the first part of not providing CEM externally. @pjanevskiTT is currently working on the other part which removes it from tt_SiliconDriver constructor.

A follow up from #277 is now to reduce ClusterDescriptor's construction not to surface yaml at all.
After this task I think we can close this issue which tracked initial push of pruning cluster descriptor


WIP: H2spec compliance fixes

Running H2spec and fixing the raised compliance issues.
It's a WIP, as I'm still going over them. I'll mark it as "done" once I fix them all, or fix a sufficient amount, or just get tired of doing it.

Coverage decreased (-0.7%) to 96.499% when pulling 85bb3afff0fccb311c4b87a3d34b7694875f75f9 on HoneyryderChuck:h2spec-fixes into 49df702aa2e2d146cc18dbf9ca3159963f260385 on igrigorik:master.

@igrigorik managed to make the local darwin h2spec run with no errors, and then I've realized it was version 1.5.0 . I've added h2spec's latest (2.2.0) to the CI build, which has ~100 more specs, and now there are 51 failing tests. Back to square one.

First off, thank you for working on this! 🙇Second, yikes.. 51 tests?
Stepping back, would it make sense to break this into multiple PR's? Worried that it might get unwieldy to review otherwise, and it would be nice to get the incremental wins without blocking on 100% success.

I guess 51 sounds rather bad, but in most cases just means that the lib doesn't handle certain edge cases, which make it a bit exploitable. But most ruby servers get proxied anyways, so I wouldn't sweat it much.
You could review it by commit, as I organized it (mostly) in a way that each commit handles a specific failure.

This is really great work. I didn't know h2spec was updated. I found it a bit buggy. I look forward to integrating some of these changes into my own upstream http/2 implementation.

@igrigorik left some more changes. If we can agree on the 2 remaining issues, then we could finally have this in master.
@ioquatix even after this is all done, we'll still be 51 specs behind. Hopefully we can count on your help to get that last mile :)

Looking at https://github.com/summerwind/h2spec/issues I'm not sure that h2spec is a good baseline. Basically, it's really great for evaluating the implementation, but as a target, it's got too many issues.
For example, the strict order it sometimes expect responses isn't actually required by the spec. But I don't believe this has ever been addressed: https://github.com/summerwind/h2spec/issues/93
I will have to try it out again with my latest implementation (https://github.com/socketry/http-protocol). I did have some automated testing with the latest h2spec, but even building it on Darwin was tricky if I recall correctly. How hard was it to build/use in travis?

I think it's a good-enough baseline, even with those pending issues. No tool is perfect :)
On the particular issue you mentioned, while I don't see exactly details about that particular interaction, it seems that the peer sends a HEADERS frame + DATA frame, which probably means that the request contains payload, and you shouldn't be sending responses without processing the data (unless Expect: 100-continue). I think that the spec covered well there.
About building in travis, I added a rake task to install it. It's not great, but it's self contained, and it works on my computer (mac) and in travis (linux), so I'd say it's good enough. Feel free to improve it though.

and you shouldn't be sending responses without processing the data (unless Expect: 100-continue). I think that the spec covered well there.

Can you point me at the relevant part in the RFC regarding this? I've often wondered if this behaviour is specified or not. Because it seems to me it should be fine to send a response without receiving the entire request body if you don't care about it.

I can see the logic. Thanks. But it's a bit of a stretch. With HTTP/2 you can fail streams at any point... so reporting the error is not a matter of a 500 status code but perhaps a stream failure. I'd personally want to see this discussed explicitly, but I have scoured the RFCs and never seen anything which seems to explicitly deny sending a response before receiving the request body in it's entirety.

@igrigorik any updates?

@HoneyryderChuck sorry about the delay, started reviewing this earlier but keep stumbling over scope and breadth of all the changes. Given some of the outstanding comments both from myself and others on the thread, I would recommend that we break apart this PR into smaller chunks.. Would you be willing to split this up, so we can tackle individual updates on case by case basis?

@igrigorik I get that this review requires more attention than the usual, but I don't think that breaking this down into smaller PRs would help that much. And frankly, my time has been very limited since the year started.
I don't know if you're reviewing all of the changes in bulk or if you're going commit-by-commit, but can I suggest the second approach if you're not doing it? I've just had another look at the list of commits, and the majority is relatively small and addresses a specific issue/scope. This way, you could simply cherry-pick them into master, and we could decrease the scope of this PR gradually. WDYT?
I get that I started with the wrong scope for this PR and now we're in this state, and your review time is pretty limited, but I still think that this adds more value than maintenance burden overall. I've just seen another compliance issue pop up which I think this PR already covers.

@tiago I can relate to your perspective, since  my available time is also minimal at the moment.
@igrigorik I’m wondering if there is another collaborator that could jump in here and break this down into multiple PRs?

On Mar 3, 2019, at 05:38, Tiago<EMAIL_ADDRESS>wrote:
@igrigorik I get that this review requires more attention than the usual, but I don't think that breaking this down into smaller PRs would help that much. And frankly, my time has been very limited since the year started.
I don't know if you're reviewing all of the changes in bulk or if you're going commit-by-commit, but can I suggest the second approach if you're not doing it? I've just had another look at the list of commits, and the majority is relatively small and addresses a specific issue/scope. This way, you could simply cherry-pick them into master, and we could decrease the scope of this PR gradually. WDYT?
I get that I started with the wrong scope for this PR and now we're in this state, and your review time is pretty limited, but I still think that this adds more value than maintenance burden overall. I've just seen another compliance issue pop up which I think this PR already covers.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Yeah, I was effectively thinking of cherry picking incremental bits.. If someone is willing to break this apart to make it easier to review and digest, that would be super helpful.
p.s. @HoneyryderChuck thanks for all the great work on this.

@igrigorik could you cherry-pick the following commits to master?

https://github.com/igrigorik/http-2/pull/144/commits/ca444bca11739a7d0be1f72c9e89dd79fb238a27
https://github.com/igrigorik/http-2/pull/144/commits/15d84417dba6aa3dd145f314bbbec78a96b28e6f
https://github.com/igrigorik/http-2/pull/144/commits/79712f69d124c80e8e4d64abe78a2c5413e61110
https://github.com/igrigorik/http-2/pull/144/commits/b171a6c6c9d33d967f802e040b47c83271b45aff
https://github.com/igrigorik/http-2/pull/144/commits/49c754f250c7d6a9c28fe9773c5ef1b61895dbd9
https://github.com/igrigorik/http-2/pull/144/commits/4b06d2812ddeba1c9d1189423b1445c1e078396a
https://github.com/igrigorik/http-2/pull/144/commits/8852136158ab668a89bf76aa33595d6c0cec4baf
https://github.com/igrigorik/http-2/pull/144/commits/ddf228343743920df09f54b0d2019b97a59310f9
https://github.com/igrigorik/http-2/pull/144/commits/4db58c5c9a9c3473a4db2abf1a741ea66345ea7a
https://github.com/igrigorik/http-2/pull/144/commits/8c50d263faf75ae6bfa5dcd2ab6d03360410ec7e
https://github.com/igrigorik/http-2/pull/144/commits/5e043d0e3714880af85098a96309e69406b3de58
https://github.com/igrigorik/http-2/pull/144/commits/aa9247736cd44ccf521be44c646790cf46cbb0e6
https://github.com/igrigorik/http-2/pull/144/commits/56296b5b36256913b459606faebe85023ca29862
https://github.com/igrigorik/http-2/pull/144/commits/b282ca9d5acdfa51d1bac631c5290d6b49d64201
https://github.com/igrigorik/http-2/pull/144/commits/2a7a9b5fe2328d51bdde833536695c8a5eac02f6
https://github.com/igrigorik/http-2/pull/144/commits/5d02c543312144012d93138730cc22d121d25a3b

(this one has a puts, consider removing it)

https://github.com/igrigorik/http-2/pull/144/commits/c2a50419f5943065a8a1887cc88625ee4d487cff
https://github.com/igrigorik/http-2/pull/144/commits/d50cadfb014a0fad3e0f02fad62862da728973eb
https://github.com/igrigorik/http-2/pull/144/commits/1d65a15766a5fee422df0299e09a8c70393777ba
https://github.com/igrigorik/http-2/pull/144/commits/0c5d00ca85de8ff959fdb61cacfe8950bf769cbc
https://github.com/igrigorik/http-2/pull/144/commits/215e78c7a22663f9cd71c45cbf4911b212dc3a10
https://github.com/igrigorik/http-2/pull/144/commits/74555c89be2cbc9f3d42ebc4a9f9b8b4ca1c330e

All of the commits above are essentially bugfixes and a lot are one-liners. I left behind the ones involving ivars, as I assume those were the ones you're more concerned with.

https://github.com/igrigorik/http-2/pull/144/commits/0997b6968552eabec899e5347b4ad5c076189c67

this one adds the rake task to install latest h2spec. It also adds the task to the CI build, so that we get some feedback.
I will have some time to work on this end of April, and if you could merge this low-hanging fruit, I could start working on breaking the remainder into separate PRs and see the remaining compliance issues.

@HoneyryderChuck Are you still interested in getting this merged? The author gave me permissions to the repo. We're looking to make improvements to this gem and release a new version.

Hi @mullermp see this discussion

@HoneyryderChuck I understand. http-2 is used with the AWS SDKs (current version and a planned version). Changing that away is difficult. I see you've been very active in improving it with your gem. I'd also like to propose a converge back. @igrigorik added me as a contributor here and I've been making a few fixes, maybe he would be open to adding you as a contributor as well as publish permissions? It seems like maybe a handoff is appropriate but I won't speak for him.
My proposal is this - which simplifies things I think - major version bump http-2 using http-2-next as the base.
Let me know what you think.

@mullermp thx for the reply!

current version and a planned version

Oh, that's interesting. Will the AWS API be allowing HTTP/2?

I'd also like to propose a converge back.

As mentioned in the other thread, I have no objections, just concerned about the "how", given the implied amount of work, as both repos diverged, in ways that mainline may not be comfortable accepting the changes from the fork. My (obviously biased) ideal approach would just be to hard-merge http-2-next into mainline, considering all spec compliance work done, performance improvements, RBS type defs, among other benefits. It's something I've never formally proposed for lack of activity in mainline and some fear of pushback. Not sure how you feel about it, but anything else would probably be a time sink I'd have very little time for. LMK what you think or were planning for, or whether you see some other approach / workaround.

maybe he would be open to adding you as a contributor as well as publish permissions?

Once both repos get merged somehow, I'd be open for it.

Oh, that's interesting. Will the AWS API be allowing HTTP/2?

We currently do this for streaming services such as Transcribe and Kinesis.

My (obviously biased) ideal approach would just be to hard-merge http-2-next into mainline, considering all spec compliance work done, performance improvements, RBS type defs, among other benefits.
Yep, I agree with this, as a major version bump (http-2 version 1.0 since latest gem is 0.11.0). Even better is if we can get this into stdlib ruby as net-http2 (though that gem is taken). Let me evaluate some options and talk to some folks.

Are you on the bundler slack? If you are, find me on there and let's chat on this.

Even better is if we can get this into stdlib ruby as net-http2 (though that gem is taken).

That's probably not desirable: stdlib is losing gems by the version, and for good reason (stdlib gems tend to silently die). net-http2 would probably be a weird name, considering this is a parser, and net-http is a full blown client. There's also (that I know of) no immediate benefit in having this a standard gem.

Are you on the bundler slack?

I'm not, is that by invite?

Try this invite (from google): https://join.slack.com/t/bundler/shared_invite/zt-1rrsuuv3m-OmXKWQf8K6iSla4~F1DBjQ

For anyone who comes across this, a new branch "main" was created that includes this work. https://github.com/igrigorik/http-2/pull/172


Method addition fix it insert special variables as parameters

Special variables should not be inserted as parameters when creating a new method through a fix it action. The usual anObject variable should be used instead.

Cheers


Give the ability to download files into specific locations (CLI)

Requested Feature
While downloading files directly to the folder or directory the terminal is open in, it's a bit of a hassle organization-wise to have to move the downloaded files or execute the commands inside a specific folder, being able to download files and save them in a specific folder, regardless of where the command was executed from could make your downloads easy to manage and organize.
Possible implementation
The best way I see to implement this feature would be to add a setting in the configuration file where you can specify the path to a directory. Should the path be invalid the file could be downloaded to where the CLI was opened from.

Try spotdl download [url] --output /path/to/download


OS X unable to find link

After a quick search or inside a popular movie, when I try to play one of them, an "unable to find link" message appear.
XBMC latest Gotham version, os x Yosemite.

Me too I have this problem on OSX, did you find a solution ?


kernel: add patch and config

Add defconfig file for x86 KVM.
Add 9p patch to always use cached inode.
fixes #17
Signed-off-by: Julio Montes<EMAIL_ADDRESS>

@laijs @bergwolf @sboeuf @egernst @jodh-intel @grahamwhaley  - ptal

@devimc Sorry if I didn't follow here, but what's the kernel config for Xen for ? Future support of Xen as hypervisor ?

This looks good
A couple of comments.

Could you add some docs describing how this patches are going to be used ?
I think could be great to have or first test for this repo that takes that config and and patches and make sure the kernel builds.

@bergwolf changes applied , thanks

@devimc I know that eventually we might have Xen support as an hypervisor in Kata Containers, but we don't have it now. And my understanding is that we're advertising that we have a kernel for Xen but we cannot test it with Kata.

@chavafg  @devimc
Seeing that there are a couple of question regarding if something is need or not.
Could you add this repo to the CI (jenkins) and add a test for this repo that:

build this config in a kernel install it and check all works.

This will  help to avoid @devimc  being testing manually on every added option.

@devimc  as reminder you you add a doc here describing  the usage of this config  patches and probably how will relate with linux fork.

changes applied, thanks

lgtm
just a minor questions, after have any feedback fixed by @klynnrif this looks good to me, we can optimize as needed in follow-up PRs.

@jcvenegas  changes applied, thanks

@klynnrif changes applied, thanks
@bergwolf please take a look

@devimc @bergwolf @sboeuf any update on this?

@jcvenegas -- I talked with Peng on this last night.  I think both of us would rather see a resulting config rather than the minimal defconfig.

@kata-containers/packaging  Hey! could you please take a decision NOW!?

@devimc a decision about what ?
I think there are two things that have been mentioned:

Removing the Xen config as it is not used/tested yet, so no reason to introduce something we didn't test
Do not rely on defconfig as they can include unneeded component.

@sboeuf  dude xen was removed I think you should take a look to the commit before ...

Do not rely on defconfig as they can include unneeded component.
really? how do you know that?

@devimc wow easy, I am sorry I didn't check the new version, I was just trying to summarize what was going on.
About the defconfig, I am just saying that when you apply a defconfig, you don't have control over what are the default for options that you didn't specify into the defconfig file.


Avoid double rounding in Float#format for nonnegative decimal_place

There is no need for the number.round(decimal_places) call since %.*f already performs correct rounding at the necessary precision. Without this PR 1.2345.format(decimal_places: 24) would return "1.234500000000000152766688".

As for why number.round(decimal_places) is incorrect: the rounding is done with (number * 10**decimal_places).round / 10**decimal_places, and the largest power of ten exactly representable by a Float64 is 1e22, so even minute differences can lead to rounding errors if decimal_places >= 23.


fix: set json fragment name optional

Summary

When declaring a JSONFrament input/outputs name is not required.

Example
This works;
    const contract = await setup([
      {
        type: 'function',
        name: 'return_context_asset',
        outputs: [
          {
            name: 'arg0',
            type: 'b256',
          },
        ],
      },
    ]);

This should work also;
    const contract = await setup([
      {
        type: 'function',
        name: 'return_context_asset',
        outputs: [
          {
            type: 'b256',
          },
        ],
      },
    ]);

@QuinnLee, this was implemented here: https://github.com/FuelLabs/fuels-ts/pull/197/files#diff-620d54c98858ccefe59323ac69ab85f1a37cd15f6d11b5aaad6d8c34504a7a4fR4 any solid reason for name to be required?


Adding latest gb files with cy translations

New source-data for gb questionnaires have been provided by the business. This PR accepts those files and creates initial front-end json files to use with the current front-end.
The conversion script has been updated to ensure the json files end with a new line.
It is accepted that the files do not contain a translation for every English term. This is a business decision.

 Add latest (v2.0.0) GB files to suggestions repo and publish to CDN


Adding a bot to a channel via the "startchannel" parameter does not work

Hello,
Bot API 6.0 adds support for t.me links, which can be used to add bots to groups and channels as admins. I'm trying this feature, but it seems to only work for groups:
https://t.me/<bot_username>?startgroup=true&admin=change_info

When I follow a link like this:
https://t.me/<bot_username>?startchannel=true&admin=change_info

It just opens a chat with the bot. There is no channel choice dialog. I'm doing something wrong or it's a bug?

You are doing wrong. There is no startchannel. Use startgroup it will display all chats including groups and channels.

No, startgroup displays only groups. I've tested on macOs, Desktop and iOS clients.

There is no startchannel.

There is: https://github.com/tdlib/td/blob/da4282a6d94c107b3240425b4c72c877eab89b71/td/telegram/LinkManager.cpp#L930

Not all telegram clients support startchannel, i have tested Android and IOS and it just open the bot chat.
Android and IOS display channels/groups using startgroup.

It works correctly in TDLib. Bugs of other clients are better to be reported at https://bugs.telegram.org.


mySQL schema

@lukehollis can you share the database schema with us before we sign off on all this?  @amir-zeldes may want to look at it, and I think I want my spouse to give it a once-over.  Thank you!

Sounds great!  The best way to review this right now is looking at the models, available here:
annis: https://github.com/CopticScriptorium/cts/blob/master/coptic/annis/models.py
ingest: https://github.com/CopticScriptorium/cts/blob/master/coptic/ingest/models.py
texts: https://github.com/CopticScriptorium/cts/blob/master/coptic/texts/models.py
Let me know if you'd prefer a different format, and I'll send it your way.


rotate coordinates in metadata file

Currently decomp uses the following coordinate layout:
O┌───►Y
 │
 │
X▼

But this causes a problem for nextsim which needs to know the concept of Bottom, Right, Top & Left neighbours (BLTR).
In order for the two codes (decomp and nextsim) to agree on which neighbour is BLTR, we need to use the same coordinate system. @timspainNERSC mentioned that we should use axis as in mathematics, as follows:
Y▲
 │
 │
O└───►X

@TomMelt I guess this was resolved by your recent changes?

@jwallwork23 , that's correct. It was fixed by #58.


Toolbar height not saved on different decimal delimitor: 1,23 vs 1.23

On a machine with a different decimal notation, 1.23 instead of 1,23, the localized notation is used when saving the value to the config file, making the value invalid when the program attempts using it.

As 0,03 isn't a valid when loading the config, ppInk will revert to the default value of 0.06, making this unusable via the GUI.
Even if it's modified via the config file manually, every time you persist the changes from the GUI, it will be overwritten.
I would recommend either ensuring both formats are properly saved regardless of the entered text (Aka. converting 1,23 to 1.23), or supporting loading both formats when loading the config (Replacing , with .? Not sure if there are other conversions to be done)

thanks for this report,
commas are used to separate some data so I can not simultaneously , and . as decimal separator
I did my best using 'NumberStyles.AllowDecimalPoint' on all loadings and  '.ToString(CultureInfo.InvariantCulture)' but I did missed the one for toolbar_height.tell me if it is better with this later commit.

Can you send me a compiled version, or release one of them?
I can't compile the project on my PC at the moment, so I can't test it out :/

Fixed with https://github.com/pubpub-zz/ppInk/issues/21#issuecomment-707508019


reset never terminates

Is this C8-specific too?

Or maybe that was because of Nexus blocking port 8080...

Well it works now, so I'm gonna assume it was Nexus.


IPv6

Hi Dave,
Quick question, is Drachtio Server capable of supporting IPv6 addresses? And if so is it capable of using both IPv4 & IPv6 simultaneously?
Thanks.

Sorry, ipv6 is not currently supported

Ok, thanks Dave.
I realise you are the sole dev on Drachtio, but do you have any timeline for IPv6?

Not at the moment, mainly because until now no one has asked for it.  It would be helpful to understand the case / need for it in your environment, if you could share some details

I will get that to you early next week via email. Thanks again.


Closes #32 Add timesheet enhancements

:white_check_mark: Build mentorbot 1.0.103-jytjbtfi completed (commit https://github.com/MentorSource/mentorbot/commit/3abba63ba1 by @rosenkolev)

Codecov Report

Merging #34 into master will decrease coverage by 13.68%.
The diff coverage is 69.97%.

@@             Coverage Diff             @@
##           master      #34       +/-   ##
===========================================
- Coverage    95.5%   81.81%   -13.69%
===========================================
  Files          50       32       -18
  Lines        2645     1413     -1232
===========================================
- Hits         2526     1156     -1370
- Misses        119      257      +138

Impacted Files
Coverage Δ

...orBot.Functions/AzureFunctions/HangoutChatEvent.cs
91.66% <ø> (-8.34%)
:arrow_down:

...ntorBot.Functions/Processors/LocalTimeProcessor.cs
96.66% <ø> (-0.71%)
:arrow_down:

...MentorBot.Functions/Services/TableClientService.cs
27.27% <ø> (-4.83%)
:arrow_down:

.../MentorBot.Functions/Processors/RepeatProcessor.cs
84.37% <ø> (-13.5%)
:arrow_down:

...entorBot.Functions/Processors/CalendarProcessor.cs
48.48% <ø> (-49.02%)
:arrow_down:

...ot.Functions/Connectors/GoogleCalendarConnector.cs
75% <0%> (-15.48%)
:arrow_down:

...t.Functions/App/Extensions/EnumerableExtensions.cs
0% <0%> (ø)

....Functions/Connectors/AzureBlobStorageConnector.cs
69.23% <0%> (-7.24%)
:arrow_down:

src/MentorBot.Functions/App/ServiceLocator.cs
100% <100%> (ø)
:arrow_up:

...rBot.Functions/Connectors/OpenAir/OpenAirClient.cs
96.29% <100%> (-0.28%)
:arrow_down:

... and 45 more

Continue to review full report at Codecov.

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

:white_check_mark: Build mentorbot 1.0.104-tuqlpeim completed (commit https://github.com/MentorSource/mentorbot/commit/683a23f27e by @rosenkolev)

:white_check_mark: Build mentorbot 1.0.105-rgdrmsna completed (commit https://github.com/MentorSource/mentorbot/commit/e9e850d35e by @rosenkolev)

:white_check_mark: Build mentorbot 1.0.106-ialndbid completed (commit https://github.com/MentorSource/mentorbot/commit/0976fe3e6e by @rosenkolev)

:white_check_mark: Build mentorbot 1.0.107-nsleeubc completed (commit https://github.com/MentorSource/mentorbot/commit/00af028e56 by @rosenkolev)

:white_check_mark: Build mentorbot 1.0.108-mmnuaonh completed (commit https://github.com/MentorSource/mentorbot/commit/d8fb4e43ab by @rosenkolev)

:white_check_mark: Build mentorbot 1.0.108-yvmgxtvb completed (commit https://github.com/MentorSource/mentorbot/commit/d8fb4e43ab by @rosenkolev)

:white_check_mark: Build mentorbot 1.0.108-ldcmvvmg completed (commit https://github.com/MentorSource/mentorbot/commit/d8fb4e43ab by @rosenkolev)


Embedded CA truststore classpath name clashes with the one from MobileID Java Client

AuthenticationResponseValidator loads trusted CA certificates from a classpath resource called trusted_certificates.jks located in resources directory of the library. However, when MobileID Java Client is also present on the classpath, another truststore with the same name is defined on the classpath and it is undetermined if SmartID Java Client loads the correct truststore or not.
My suggestion would be to rename trusted_certificates.jks to a more specific sid_trusted_certificates.jks.

Or not load it from resources root.


Watching tab

Pull request checklist:

[x] CHANGELOG.md was updated, if applicable

Description

merge once extension is approved

merging before extension is approved, want to build on top of this PR


更新win10创新者后，vs code中的shell缩进不正常

VSCode Version: Code 1.11.1 (d9484d12b38879b7f4cdd1150efeb2fd2c1fbf39, 2017-04-06T13:58:12.954Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions:

Extension
Author
Version

html-snippets
abusaidm
0.1.0

gitignore
codezombiech
0.5.0

vscode-eslint
dbaeumer
1.2.8

vscode-npm-script
eg2
0.1.9

vscode-JS-CSS-HTML-formatter
lonefy
0.2.3

debugger-for-chrome
msjsdiag
3.0.1

view-in-browser
qinjia
0.0.5

vscode-icons
robertohuertasm
7.5.1

JavaScriptSnippets
xabikos
1.4.0

Steps to Reproduce:

@pyyzcwg2833 please file issues in English

duplicate with #24313 #24363 and #24651

thanks

After my OS was updated to Win10 Creator, the indentation in shell (Powershell, git bash....) is not normal

After command 'clear',  it temporarily returned to normal.

I'm sorry to say that the effort is to be in vain.
After I type 'clear', the shell looks like this

But after I start to type other command ,such as 'node', it doesn't work normally.

FYI, I found that this issue is gone on the latest Win10 insiders build 16170 with the same insider version of VS Code
@jrieken

So I think the issue has something with Win10(Build 15063) .@jrieken

Duplicate https://github.com/Microsoft/vscode/issues/19665


[Event Request] Codeunit 427, ICInboxOutboxMgt, OnForwardToOutBoxOnBeforeHndlInboxPurchLineInsert

[Event Request] Codeunit 427, ICInboxOutboxMgt, OnForwardToOutBoxOnBeforeHndlInboxPurchLineInsert
New event in procedure OnForwardToOutbox
                                HndlInboxPurchLine.TransferFields(InboxPurchLine);
                                OnForwardToOutBoxOnBeforeHndlInboxPurchLineInsert(HndlInboxPurchLine, InboxPurchLine);
                                HndlInboxPurchLine.Insert();

[IntegrationEvent(false, false)]
local procedure OnForwardToOutBoxOnBeforeHndlInboxPurchLineInsert(var HandledICInboxPurchLine: Record "Handled IC Inbox Purch. Line"; ICInboxPurchLine: Record "IC Inbox Purchase Line")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Show nuspec XML for signed packages

Adds the ability to view nuspec xml's for signed packages. I've made sure to disable the save commands where I could find them.
Not sure if this is sufficient, please let me know if I missed something.
Addresses #478

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

Hi @samneirinck Thanks for this! Two small things

can you please sign the CLA so we can accept this?
can you please hide the metadata button editor in this scenario?

Added some logic to hide the metadata button. I'm not convinced of the name, so feel free to change :)
Also signed the CLA in the meantime.
Thanks for the quick feedback!

Thanks!


nginx module: fix typo in proxyWebsockets description

Motivation for this change
This typo has been there since 2017 without anyone noticing.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This typo is also in the stable version, so it should be fixed there as well, right?

This typo is also in the stable version, so it should be fixed there as well, right?

This typo is also in the stable version, so it should be fixed there as well, right?

If we you want to backport it you can do it.

This typo is also in the stable version, so it should be fixed there as well, right?

If we you want to backport it you can do it.

I'll wait til this PR is merged since the manual says that the cherry-picked commit should be in the master branch.

I'll wait til this PR is merged since the manual says that the cherry-picked commit should be in the master branch.


Parsing attributes issue

When i am trying to parse this to json:
<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9"></urlset>

The output is:
"urlset":{"$":{"xmlns":"http://www.sitemaps.org/schemas/sitemap/0.9"}}

Then when i revert this to xml it is not how he was at the first place.

I'm not able to reproduce this issue. And all the tests that check to make sure that reverting a converted XML document back to XML are passing.
'use strict';

const assert = require('assert');
const xml2js = require('xml2js');
const xml = [
	'<?xml version="1.0" encoding="UTF-8" standalone="yes"?>',
	'<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9"/>'
].join('');

xml2js.parseString(xml, (err, result) => {
	if(err){
		throw err;
	}

	let builder = new xml2js.Builder({
		renderOpts: {
			pretty: false
		}
	});

	let newXml = builder.buildObject(result);

	assert.equal(xml, newXml);
});


WIP: initial support for 'cog serve (image)' command

This is a WIP experiment for a more featured local cog serve - #739
This adds new command to the cog cli

spawns docker process to serve model
cog (golang) has http handler to proxy model running in docker
ability to disable CORS for remote SPA
serving static content (SPA mode)
kill cog via ctrl-c causes docker process to be killed

Status:

[x] http proxy to docker container
[x] ctrl-c kills docker container
[x] ctrl-c shouldn't cause error: ⚠ Error getting container logs: signal: interrupt
[ ] ensure docker container is ready before proxying
[x] make CORS support optional
[x] fix copy & paste code from cmdPredict to cmdServe
[x] add SPA support #687
[ ] add local file support #496
[ ] add support for http urls for file inputs (replicate.com says: "Files should be passed as data URLs or HTTP URLs.")
[ ] harmoize cog serve api vs replicate api
[ ] cleanup pass (middleware messes, cleaning abstractions, ...)
[ ] tests!

Signed-off-by: Jesse Andrews<EMAIL_ADDRESS>

Hey I just tried to take this for a spin locally but it doesn't look like serve is registered as a CLI command yet:
~/git/replicate/cog pr/738
$ cog --version
cog version 0.4.1 (built 2022-08-04T18:02:43Z)

~/git/replicate/cog pr/738
$ cog serve --help
ⅹ unknown command "serve" for "cog"

How are you running it?
Sorry if I jumped the gun on reviewing this. Just let us know when this is ready for a look.


Disable intermittently-failing testStubProcessProtocol on Linux

Clear CI while we investigate the cause.
Example failure:
https://ci.swift.org/job/oss-swift-incremental-RA-linux-ubuntu-16_04-long-test/6931/console

@swift-ci please test


Removed all references of GUN from filestore

@endophage removed GUN references for the FileStore as you requested.
The FileStore still does operations on directories (which might be sightly counterintuitive), but at least now it's explicit that you're removing a directory, and should pass a param accordingly.
This also bails if you pass a file, instead of a directory.
Signed-off-by: Diogo Monica<EMAIL_ADDRESS>

LGTM. Just renaming things.


Add functional tests for cFE ES Application Control APIs

Is your feature request related to a problem? Please describe.
Need open source functional tests for certifiability
Describe the solution you'd like
Add functional tests for cFE Application Control APIs
– CFE_ES_RestartApp - Restart a single cFE Application.
– CFE_ES_ReloadApp - Reload a single cFE Application.
– CFE_ES_DeleteApp - Delete a cFE Application.
Describe alternatives you've considered
N/A
Additional context
N/A
Requester Info
Alex Campbell GSFC

duplicate of #806


RichTextLabel's built-in effects are not exposed as push_() functions

Godot version
4.0.2.stable
Issue description
I noticed something while working on https://github.com/godotengine/godot/pull/77117. These RichTextLabel methods exist in the C++ source code, but aren't exposed to GDScript unlike other built-in tags like push_bgcolor():

push_fade()
push_shake()
push_wave()
push_tornado()
push_rainbow()

Having these methods exposed would be useful for consistency with other tags, but also because large text is faster to format using push_*() methods compared to BBCode.
Note that set_process_internal(true) is called in the BBCode handler for those tags (example), not in the push_*() methods (other than push_customfx() which is exposed). This call should be moved to the push_* methods so that it works even if you only use push_*() methods to format your RichTextLabel's text.
cc @Eoin-ONeill-Yokai – is there a reason these methods aren't exposed?
Keywords for easier searching: RichTextEffect

It was probably a mistake on my part on the original MR -- I don't think there should be any problem with exposing the push functions to scripting language. And yes, the set_process_internal should be moved to accommodate.

Isn't push_customfx enough? It's also suitable for script-defined custom fx, and the current list of the builtin ones is pretty arbitrary so I'm not sure they should be exposed as part of the API.

but also because large text is faster to format using push_*() methods compared to BBCode.

How so? Do you mean faster for the programmer, or more performant?

push_customfx() works for custom effects, but there is no way to refer to the built-in effects as you can't grab a handle on them from the scripting API.

How so? Do you mean faster for the programmer, or more performant?

It's more performant to use push_* methods, as well as more secure when BBCode is disabled to prevent any risk of formatting injection from untrusted input.

Hi there, was this ever resolved? I am currently trying to use the built-in custom effects.

Sorry, I've been a bit busy on my end but I should be able to address this sometime this week. Let me put it to the top of my Thursday tasks and I'll try to get it done as it shouldn't be too hard to do.

No worries! I found a workaround but I do think a lot of things could be easier if we had these functions available. :]