Implementation for #805: JdbcMigrations and new FlywayCallback should have access to core configuration

Introduced a generic ConfigurationAware interface.
This allows to inject Core configuration into resolvers, migrations and
callbacks. Configuration is injected by using a new FlywayConfiguration
interface, which provides read only access to flyway configuration.
Right now, Flyway implements FlywayConfiguration directly.

Is there a status update for this issue? Really need access to the config from within a JdbcMigration myself.

I provided an adapted implementation. Once again: This implementation is backward compatible, because injection of config data is done via an optional interface.

Resolved conflicts. However, currently tests fail that also fail in master, too.

This patch in its current form does way too much. At this point I certainly don't feel like making DbSupport part of the public API. Could you isloate ONLY the FlywayConfiguration/ConfigurationAware part?
Thanks,
Axel

Agreed, moving DbSupport was too much. However, the other two changes work hand in hand and do not make much sense without each other.

Fixed conflicts. Unfortunately, your current changes are all in the code changed by this PR, so this results in a lot of work in keeping it current.

This still does way too much. PlaceholderReplacer and Scanner are now exposed in a public package, there is a new skipDefaultResolvers property, etc.
Please isolate ONLY the FlywayConfiguration/ConfigurationAware part. Also please include the JavaDocs in FlywayConfiguration.

Ok, I worked around exposing PlaceholderReplacer and Scanner (the way it weas in the original PR). I also included JavaDoc as requested. However, there is still one internal api left: DbSupport. In order to replace built in Resolvers, the CustomResolver needs access to the DbSupport Object.
In that case, I can not do a workaround, because of the state of DbSupport. I could (and acutally did in a previous step) refactor the internal part (DbSupport) into a separate, use-at-your-own-risk interface (DbSupportAware?).
What do you think? (waiting on your feedback before merging again)

Please move the CustomResolver  support to a separate pull request. I would like to keep this one to literally the FlywayConfiguration/ConfigurationAware part. It's big enough already with just that.

Ok, I removed the custom resolver part.
When you give me the go, I would start to merge to resolve the conflicts.
I will create a new PR with the custom resolver part.

Squashed commits into one, no benefit in keeping them separate (and makes it easier for you to validate)

Getting closer, but still a little more trimming required
Could you:

remove the original javadoc from the getters (that are now in FlywayConfiguration) in the Flyway class
remove the Flyway.injectConfiguration method and its usages as well as the changes to setXYZAsClassNames
remove the changes to the creation of Scanner objects as they effectively undo the effects of the newly added caching
document the ConfigurationAware.setFlywayConfiguration method and its argument
remove the imports of internal classes from FlywayConfiguration
remove the changes to the existing migration resolvers
remove injectionutils

Javadoc and imports: sure
As for the other changes:

Flyway.injectConfiguration(): This is an important part of the PR. We need a place to perform that automatic injection. In the current version, the most feasible place is the setters. In the second part of the original PR (the replacement of the built-in resolvers), I moved that part into CompositeMigration Resolver - which might be an option, Injection into Callbacks could be done somewhere else, but this would be scattered throughout the code. Do you agree?
Scanner Object: yes, this removes caching again (thus the earlier approach in providing Scanner via FlywayConfiguration). If I move the injection into CompositeMigrationResolver, it would allow the caching again.
Changes to existing Resolvers: I could roll back the constructor changes (still, almost every information needed by the resolvers is part of the FlywayConfiguration, so replacing the constructor arguments seems a logical step). However, the injection steps in both jdbc resolvers are essential. My initial reason for creating this PR (1,5 years ago) was to create a useful GroovyResolver that needed access to placeholder information.
Why remove InjectionUtils? It needly incapsulates the injection part, which is needed in at least three places (jdbcResolvers, callbacks, and compositeResolver).

(sent you an email via your website, might be easier to discuss this via phone)

InjectionUtils is mileading as its name is generic, yet it only works for FlywayConfiguration. Simply add a private method to the Flyway class.
Flyway.injectConfiguration() is something that can be done right before command.execute(...) in Flyway.execute(Command)

Actually, that is what I did. Which works for the callbacks. Resolvers can be injected in the CompositeResolver (which I did). Thats already the second place for injection. Third place is the injection into custom migrations (jdbc/spring jdbc). So it makes sense to put it into a separate helper class (I am open for name suggestions).

OK, I feel we're getting there.
Can you remove that new sqlline dependency?

Sure, sorry, that slipped into it because 1.1.8 can not be resolved from central.

So? No new dependencies please.

I did not introduce a new dependency... It was only an entry in dependencyManagement... phoenix has a dependency on sqlline 1.1.8 which I did replace with 1.1.9, to be able to build it in my corporate environment. It was not meant to be included in the pull request and I did remove it already. (still, might make things easier for developers to put it back in - apart from this PR)

OK. Thanks for clearing things up.


Release azure-cli-profile 0.1.1b2

Reviewers, please verify the following for this PR:

[ ] The PR title has format Release <component-name> <version>.
[ ] setup.py has been modified with the same version as in the PR title.
[ ] HISTORY.rst has been modified with appropriate release notes.
[ ] If releasing azure-cli or azure-cli-core, __version__ defined in __init__.py should also be modified to match.

Release of 'azure-cli-profile' with version '0.1.1b2' successful. View at https://pypi.python.org/pypi/azure-cli-profile/0.1.1b2.

GitHub release created. https://github.com/Azure/azure-cli/releases.


HW Accelerated SHA1 and SHA256

Add Poritng for Sha1 and SHA256 over Cryptocell.
tested with benchmark application, hashing application, and the cc hash integration tests, converted to mbedtls API

@andresag01 I modified according to your style comments


⚠ ClamAV has degraded performance

In df4532d, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 53 ms

Resolved: ClamAV performance has improved in 0e182fa.


[EventRequest] SalesTaxCalculate.Codeunit.al (codeunit 398 "Sales Tax Calculate")

Hi MS,
Please help us in migrating our verticalized solution by adding below events in SalesTaxCalculate.Codeunit.al codeunit.
Standard Code (Screenshot from AddPurchLine function)

Modified code

Event Signature

Thanks
SUD

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.


Do not log users token from the HTTP requests ⚠

That is an really important issue and should be fixed before any further release!!!
Within our server middleware the user authentication token is included in the HTTP logs:
 metadata: {
    req: {
      url: '/search?q=foo&pageSize=45',
      headers: {
        ...
        authorization: 'Bearer eyJhbGciOiJSUzI1NiIsInR5cCIgOiA...
      }
      ...
   }
}

This is a significant security issue. We have in general two ways to remove the header from the logs

Trough the Gateway: remove the header from the request that is forwarded to the upstream services. This however could not be a long saving solution, if we could need it in individual services (e.g. for Policies checking)
Remove the token from the logs in the logging middleware

The option 2 from above was chosen.
The express-winston lib has a headerBlacklist options that just removes the specified headers from the logs. This however could be misleading on the error investigation through the logs. Missing authentication header could be interpreted as the cause of an error. Instead the value of the header is masked:
...
authorization: '[MASKED]'
...


Issue after installing

I've installed g2p_en on my macOS system using setup.py
I created a test_g2p.py file containing the sample code provided in the README. When I run python test_g2p.py I receive the error below. I'm using python 3.7.
Any idea of how I proceed?

Traceback (most recent call last):
File "test_g2p.py", line 1, in
from g2p_en import G2p
File "/Users/Tristan/anaconda3/lib/python3.7/site-packages/g2p_en-2.0.0-py3.7.egg/g2p_en/init.py", line 1, in
from .g2p import G2p
File "/Users/Tristan/anaconda3/lib/python3.7/site-packages/g2p_en-2.0.0-py3.7.egg/g2p_en/g2p.py", line 21, in
nltk.data.find('taggers/averaged_perceptron_tagger.zip')
File "/Users/Tristan/anaconda3/lib/python3.7/site-packages/nltk/data.py", line 660, in find
return ZipFilePathPointer(p, zipentry)
File "/Users/Tristan/anaconda3/lib/python3.7/site-packages/nltk/compat.py", line 228, in _decorator
return init_func(*args, **kwargs)
File "/Users/Tristan/anaconda3/lib/python3.7/site-packages/nltk/data.py", line 506, in init
zipfile = OpenOnDemandZipFile(os.path.abspath(zipfile))
File "/Users/Tristan/anaconda3/lib/python3.7/site-packages/nltk/compat.py", line 228, in _decorator
return init_func(*args, **kwargs)
File "/Users/Tristan/anaconda3/lib/python3.7/site-packages/nltk/data.py", line 1055, in init
zipfile.ZipFile.init(self, filename)
File "/Users/Tristan/anaconda3/lib/python3.7/zipfile.py", line 1222, in init
self._RealGetContents()
File "/Users/Tristan/anaconda3/lib/python3.7/zipfile.py", line 1289, in _RealGetContents
raise BadZipFile("File is not a zip file")
zipfile.BadZipFile: File is not a zip file

It turns out the nltk package was not importing one of the zip files correctly, specifically "'taggers/averaged_perceptron_tagger.zip". Posuer's solution on SO seemed to resolve the error (https://stackoverflow.com/a/38931681/7147480).


🐛 BUG: Pages next() headers mutable in local dev, but not in production

What version of Wrangler are you using?
2.0.9
What operating system are you using?
Windows 10
Describe the Bug
The Pages Functions docs has this example:
const hello = async ({ next }) => {
  const response = await next();
  response.headers.set('X-Hello', 'Hello from functions Middleware!');
  return response;
};

This works perfectly fine in local dev with pages dev, but fails in production due to the next() response having immutable headers there. This leads to frequent frustration for folks since there's no way to get function error logs to debug this inconsistency.
To make this work in prod, you have to do something like this:
const hello = async ({ next }) => {
  const rawResponse = await next();
  const response = new Response(rawResponse.body, rawResponse); // clone response to create mutable headers
  response.headers.set('X-Hello', 'Hello from functions Middleware!');
  return response;
};

Related: https://github.com/cloudflare/cloudflare-docs/pull/4773

The change in #1325 appears to work great, thanks very much! I ran the follow:
export async function onRequestGet({next}){
	const response = await next();
	response.headers.set('x-test', 'hello')
	return response;
}

and was able to see the x-test header when using the new wrangler version both in prod and dev, and an exception in prod when using the current release version, as expected. Woot! 🎉

No, thank you very much! We appreciate you. I'll give you a heads up when we have a released version!

@Cherry It's now out in<EMAIL_ADDRESS>🎉

@Cherry It's now out in<EMAIL_ADDRESS>🎉

Awesome, thank you!!


Feedback

Create a frame that shows previous sessions of the same stud with the same child, along with any feedback

Moved to lookit/ember-lookit-frameplayer#35

Moved to lookit/ember-lookit-frameplayer#36


Add cargo-deny check to CI

This config protects from:

duplicate crates
copy-left licenses in the dependency tree
RUSTSEC advisories

Note we need to check in Cargo.lock for this to work.

Thanks to @emilk for the help in ureq https://github.com/algesten/ureq/pull/661


增加单元测试

MR Summary:
The summary is added by @codegpt.
The MR introduces unit tests for the DFA functionality, enhancing the codebase's reliability by covering various scenarios. Key updates include:

Setup of DFA transitions, start, and final states in the setUp method.
Addition of tests to validate DFA's string acceptance or rejection based on transitions.
Inclusion of tests for special cases like empty strings and invalid transitions, ensuring the DFA's robust handling of edge cases.

@codegpt /linter

@codegpt /all

@codegpt /all

@codegpt /review


I still see test selectors in production files

Hi ! first of all thanks for the great addon! It makes testing markup agnostic which is something I love! <3
I had a look to the production build though, and I could still see some test selectors in there, so I put together this repo so you can reproduce the issue.
Steps to reproduce:

Run ember build --prod
Have a look in the files that were generated, you will be able to see the data-test-component there

Is this intended ? I was expecting the test selector not to be there right ?
Thanks !

I confirmed the test selector is still present. Any idea @Turbo87 ?


Reject non-distinct token pool before reaching token connector

When sending a POST to /tokens/pools, the name field should be verified as distinct before reaching the token connector. This ensures that the token connector does not successfully create a pool that will ultimately be rejected.

#674


Update toml dependency to 0.8

Seeing this patch, I mainly wonder whether the full machinery of toml is appropriate or whether basic-toml would suffice?

I do not know the answer to that. I don't even remember what I was doing that led to this PR tbh, I got home from work and my terminal had this diff up. What was I doing this morning?

For #10 we'd still depend on toml_edit, so IMO switching to basic-toml isn't helpful.


LoadGen: Update docs/src project.

Removes unused logo.
Updates BUILD.gn to include recently added files.

@christ1ne : ptal. Another doc cleanup patch.

Hi @christ1ne. This is a fairly small documentation cleanup patch. Can you take a look?

LGTM


Feature/pdf improvements

This PR implements the following features requested in issue #474 :

Before
Now
PDF Type

1.
PDF displays date generated
PDF displays no date if status is not published otherwise displays the published at date
Regular

2.
No page break after TOC
Page break after TOC
Regular

3.
No heading in PDF
Heading in PDF (says: This ATBD was downloaded from the NASA Algorithm Publication Tool (APT)
Regular

4.
Centered heading in PDF only displays: Manuscript submitted to _Earth and Space Science_
Centered heading in PDF displays: Manuscript submitted to _Earth and Space Science_ \n This ATBD was downloaded from the NASA Algorithm Publication Tool (APT)
Journal

Screenshots:
No date in un-published regular PDF:

Date in published regular PDF:

Page break after TOC in regular PDF:

Header in regular PDF:
Title page:

Other page:

Header in journal PDF:
Title page:

Other page:

Full sample documents:

Regular PDF, published: test-atbd-1-v2-0.pdf
Journal PDF, _un_published: test-atbd-1-v2-0-journal.pdf

@leothomas do you have a screenshot that shows the date of a pdf that was published before today? I think one of the main issues here was people were generating pdfs and they would always just have the current date

I could generate one by changing the date manually in the database - let me give that a try

Regular PDF with publication date (note current date in top right corner)

@leothomas The proposed changes look good. I found some other issues that may not be related, but leaving them here.
For this I was looking at the Test ATBD 1 v2.0 (fixture data).
Even though the links do not overflow the page anymore, they still overflow onto the margins. Is there something to be done here? Does not seem like a big problem, but flagging.

journal pdf only
Spaces are stripped from links that have multiple words.

@danielfdsilva Thanks for pointing those out. I'm going to merge this PR and open issues for the links


Fix fn update_subscribe in crates/relayer/src/chain/axon/monitor.rs

https://github.com/synapseweb3/forcerelay/blob/7d2779599ac44af08cb002bdd80412678fffd6b7/crates/relayer/src/chain/axon/monitor.rs#L138-L155
When the sender end is closed, this will always return Continue? (I guess the sender is supposed to send Shutdown before dropping, but we can have more robust error handling here, and AxonChain is not actually doing this.)
Originally posted by @blckngm in https://github.com/synapseweb3/forcerelay/pull/271#discussion_r1281374385

Fixed.


executeReplacingMayWithMust triggers a sharing bonus even when executed from a demand

This comes from Blackmail which is a demand effect.  I executed Calendar from the score pile and then the demander got a share card for some reason.

I'm guessing that since another player causes the effect to happen causes it to be detected as a share effect.

Also tracked by https://boardgamearena.com/bug?id=93330.


Add special Parameters modules support

This PR supports adding special Parameter modules when calling the context manager init_empty_weights that force assign the weights of each module into nn.Parameter module. For some specific usecases we might want to replce it with other modules such as bnb.Int8Params from bitsandbytes
Related PR: https://github.com/huggingface/transformers/pull/17901
Related discussion:  https://github.com/huggingface/transformers/pull/17901#discussion_r920230935
cc @sgugger @TimDettmers

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


v3.2.1+ Mock server seems to ignore withHeaders('Content-Type' : 'application/vnd.api+json')

In Groovy version 3.2.7 (using Gradle), when writing a test as
        uponReceiving('a request for')
        withAttributes(method: 'get',
                path: '/profitLossReport',
                query: ['reportBreakdown': someQuery],
                headers: reportingHeader
        )
        willRespondWith(
                status: 200,
                headers: ['Content-Type': 'application/vnd.api+json']
        )

        withBody (mimetype: 'application/vnd.api+json) {}

The mock server always returns application/json.

Hmm, I wonder if it is the period that is causing it to not match.

It's case sensitive it seems. The following worked.
withBody([mimeType: 'application/vnd.api+json']) {                 data {} }

Yeah, Groovy is case sensitive for map keys.


Papers

Add list of papers and which methods come from each to wiki.

I updated the publication list yesterday by include the ANGSD paper. I'm closing this issue.


Query strings with multiple values

Hey guys, this looks like a cool router, but I have a problem with query strings. I would like to use something like this?
https://url.com/id/1?status=1&color=blue&size=1,2,3&merchant=99,15,28,37,45,69
Any idea how to make this work?

Create a router in "routes.php". The route:

Ahhh, I see. Pretty easy. I already had this, but I am using this for search queries and it collides with paramater in routerClass.php, I am idiot. Good work with this router BTW, thank you :)
$conditions[] = 'kekurzu = ?';
$parameters[] = $idaktualni;
if (isset($_GET["stav"]) && !empty($_GET["stav"])) {
$conditions[] = 'stav = ?';
$parameters[] = $_GET["stav"];
}
if (isset($_GET["slovo"]) && !empty($_GET["slovo"])) {
$conditions[] = 'nazev LIKE ?';
$parameters[] = '%' . $_GET["slovo"] . '%';
}


Remove late-initializations in the library, properly wrap ctors from bp::dict

This should be merged only after #12


Pre-select reviewers/assignees/labels in quick open

When editing the set of Reviewers, Assignees or Labels using the quick pick, I expected the current values to appear in the quick open already selected.
There's even some inconsistency right now:

For both Reviewers and Labels, the current values don't appear at all.

For Assignees, the current values appear, but unchecked.

Duplicate of https://github.com/microsoft/vscode-pull-request-github/issues/4084


Facades to do the CRUD?

First of all, thanks for the nice package. I am trying to do category-base lists (tags). Once user subscribe through the form, they will be added under the tag named after the category they are subscribing to.
After reading the docs and this issue, I realized that this can only be done through API calls.
But since I am using the package version (Sendportal Core), it feels like I am going around unnecessarily. It would be nice to have the facades and do the CRUD directly.
Is there any direct ways to do the CRUD?

Yes, just use the repository methods, or even just create your own model calls


Hide hangman word otherwise kinda pointless.

When starting a hangman game, the word is shown, which makes it pointless since everyone can cheat. To avoid this, the word is read as if it were a password and only the person starting the game knows it.

Actually the word isn't shown!

It's okay! I can see how the UI right now could make a user think that their word is displayed (would love suggestions on how that could be mitigated). Anyways, thanks for helping out Devzat!


rolling back organization.yml changes

This PR removes the changes that were made to the organizations.yml file.
If you are suggesting a fix for a currently exploitable issue, please disclose the issue to the prime-reportstream team directly outside of GitHub instead of filing a PR, so we may immediately patch the affected systems before a disclosure. See SECURITY.md/Reporting a Vulnerability for more information.
Test Steps:
Go to the old PR https://github.com/CDCgov/prime-reportstream/pull/8989/files#diff-a3595b762b115fdaacb32ec9e8930def1a645002ecc8cc1ba9102d2ec5512fd8 and make sure that the changes are removed that were added. However, the comment should now have the default sender instead of the test sender.
Changes
Rolled back the organizations.yml changes.
Checklist
Testing

[ ] Tested locally?
[ ] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[ ] (For Changes to /frontend-react/...) Ran npm run lint:write?
[ ] Added tests?

Process

[ ] Are there licensing issues with any new dependencies introduced?
[ ] Includes a summary of what a code reviewer should test/verify?
[ ] Updated the release notes?
[ ] Database changes are submitted as a separate PR?
[ ] DevOps team has been notified if PR requires ops support?

Linked Issues

Fixes #issue

To Be Done
Create GitHub issues to track the work remaining, if any

#issue

Specific Security-related subjects a reviewer should pay specific attention to

Does this PR introduce new endpoints?

new endpoint A
new endpoint B

Does this PR include changes in authentication and/or authorization of existing endpoints?
Does this change introduce new dependencies that need vetting?
Does this change require changes to our infrastructure?
Does logging contain sensitive data?
Does this PR include or remove any sensitive information itself?

If you answered 'yes' to any of the questions above, conduct a detailed Review that addresses at least:

What are the potential security threats and mitigations? Please list the STRIDE threats and how they are mitigated

Spoofing (faking authenticity)

Threat T, which could be achieved by A, is mitigated by M

Tampering (influence or sabotage the integrity of information, data, or system)
Repudiation (the ability to dispute the origin or originator of an action)
Information disclosure (data made available to entities who should not have it)
Denial of service (make a resource unavailable)
Elevation of Privilege (reduce restrictions that apply or gain privileges one should not have)

Have you ensured logging does not contain sensitive data?
Have you received any additional approvals needed for this change?

Closing in favor of another plan


fix(schema): add basic support of Pattern type in schema generation

Change Summary
Since typing.Pattern is not a valid type, an error is raised when generating the schema.
This PR adds a basic support for it.

Related issue number
closes #1767
closes #1269

Checklist

[x] Unit tests for the changes exist
[x] Tests pass on CI and coverage remains at 100%
[x] Documentation reflects the changes where applicable
[x] changes/<pull request or issue id>-<github username>.md file added describing change
(see changes/README.md for details)

Hi PrettyWood,
Thanks for the PR.
I think it would be better if we provide a format, for exemple: 'properties': {'filter': {'type': 'string', 'title': 'Filter', 'format': 'regex'}}
This is because with a model like this:
class Model(BaseModel):
    filter: Pattern = '^regex$'

the property filtermust be a valid regex (and not a string validated by a regex)
It is not the same as filter: constr(regex='^regex$')where the schema must provide the property "pattern": "^regex$") asfilterwill be validated agains the regex ^regex$.
What do you think ?
Also we may need to add this new type in the doc: https://pydantic-docs.helpmanual.io/usage/schema/#json-schema-types

Hi @Gatux
Thanks a lot for your precious remarks! You're absolutely right!
i added format: regex and the doc.
Do not hesitate to tell me if you see anything else wrong / to be modified

Hi @PrettyWood
Looks good to me ! Thanks !

👍 thanks so much.


Got certificate claim challenge marked complete before it should be possible

Challenge Name
Claim Your Front End Development Certificate
Issue Description
I have got this checkpoint marked as complete before finishing all prerequisite challenges.
The certificate is not granted in fact, but the "Confirm" buttons in step 2/4 & 3/4 do not seem to work, and "Go to my next step" buttons are unblocked.
Browser Information

Browser Name, Version: Any
Operating System: Any
Mobile, Desktop, or Tablet: Any

Screenshot

@smsrkr Thank you for reporting this issue. I have resolved this in an open PR #16664

My certificate checkpoint is still marked as complete.

I'm closing this issue as stale since it hasn't been active lately. If you think this is still relevant to the newly updated platform, please explain why, then reopen it.


Image in PDF doesn't load

Tested in both Chrome and Firefox. Image in page 10 of the attached PDF (labeled page 8 in the document) doesn't load.
The Chrome native viewer and Adobe Reader both load the image properly.
temp_portfolio658660046914130129 (1).pdf

I can confirm this using Windows 10 x64, Firefox 44.0 and the latest PDF.js development version. It's unfortunate that there are no console messages about this.

Same behaviour on Linux Debian stretch (testing) and trunk pdf.js version. No console logs.

The images are decoded fine. You can observe this by doing the following in paintInlineImageXObject() in canvas.js
--- a/src/display/canvas.js
+++ b/src/display/canvas.js
@@ -2067,6 +2067,13 @@ var CanvasGraphics = (function CanvasGraphicsClosure() {
         var tmpCtx = tmpCanvas.context;
         putBinaryImageData(tmpCtx, imgData);
         imgToPaint = tmpCanvas.canvas;
+
+        var div = document.createElement('div');
+        var imgData = tmpCtx.canvas.toDataURL();
+        var img = document.createElement('img');
+        img.src = imgData;
+        div.appendChild(img)
+        document.querySelector('#viewerContainer').appendChild(div);
       }

Seems to be an issue with how they're drawn. Quick look seems like the coordinates are messed up, but I'll have to look into it more later.

Skipping the SMask case in CanvasGraphics_setGState "fixes" this all the missing images in this PDF file, hence this issue is unfortunately yet another regression from PR #4161.

Looks like we are not handling AIS=false state properly. Switching mask off for this case helps the case.

Can someone please fix this issue in pdf js. This issue has ruined our experience in an application that is in production. If we dont get a fix we will have to abandon pdf.js and move to a different product. I dont want to do that as I find pdf.js really simple in its UI and dependencies.

I am not sure if it can help debug,
This book doesn't show images, and the front cover is an example of the image not rendering.
https://github.com/tony-pond/books-bug-6967/blob/main/indian_basket_weaving_1903.pdf

The original PDF is an SMask problem, this PDF doesn't have them. I suspect a problem with the JPX decoder. Page 1 has a JPX image, most of the other pages have JBIG2 images which show. Page 11 and 15 have both types, and the JPX image doesn't show. I suggest you create a new issue with your file.


Using the same color syntax to convert in HTML and PDF and manage colors in YAML file

Hi,
I'm starting to convert all my documents word in asciidoc (about fifty) and my goal is to convert these doc in html and in PDF.
So I use asciidoctor and ascidoctor-pdf.
My problem is with the color text.
In the doc I write, for example, [myColorName]#my text# and in the CSS file, I declare a class .myColorName: #hexa_color_code
-> It's OK when I convert in HTML
I would to do the same in PDF conversion; but it doesn't work.
Then I have to write this code in my asciiDoc +++my text+++.
It works in PDF and in HTML but the syntax is more complex and the color code is written in hard in the document and I don't want it
So I modified the asciidoctor-pdf code to add these lines in the file transform.rb
line 208 (block when :span in build_fragment(fragment, tag_name, attrs = {}) before the line # span logic with normal style parsing

  # *********************************************************************************************
  if (!attrs[:class].nil? and attrs[:class] != '')
      pvalue = attrs[:class]
      if pvalue.start_with? '#'
          # hexa color code
          pvalue = pvalue[1..-1]
      else
          # color name -> get color code in the theme file (yaml)
          # (I deliberately add suffix _color)
          pvalue = @theme[%(#{pvalue}_color)]
          if pvalue.nil? or pvalue == ''
              pvalue = '000000'
          end
      end
      fragment[:color] = pvalue
  end
  # *********************************************************************************************

In the same file, I transform variable theme to @theme
And I add in YAML file, the corresponding color myColorName_color: hexa_color_code
Now I can manage color in the same way as HTML (via CSS file ) in PDF (via YAML file).
But as I am a beginner in Ruby and asciidoctor, is the right way to do what I want ?
And do you intend to integrate this type of mechanism in asciidoctor-pdf ?
Thank you,
Best regards,
Jerome
PS : sorry for poor english ;-)

This is a duplicate of #368 (though with a lot more detail). We should take it a step further than what you are proposing and link font settings to a role, which can then be mapped to a phrase.

In #652, I use a better solution that matches issue #368

It will be possible to set font color roles in your theme starting in 1.5.0.beta.2. See #368 (and the theming guide) for details.


Update styles.css

🛠 Fixes Issue
fixes #259
Related Issue/Addition to code

Background not displaying on phone/small screen.

👨💻 Changes proposed

display background on phone/small screen

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)

@all-contributors add @samyberkane04 to the doc


For PHP 7.4 the 'wrench/wrench' version need to be upgraded.

Version 0.8.1 of chrome-php/chrome uses library "wrench/wrench": "^2.0"
This library version uses a depricated implode()
PHP 7.4: Deprecated: implode(): Passing glue string after array is deprecated.
see https://github.com/varspool/Wrench/blob/master/lib/Protocol/Protocol.php
Can the version of wrench/wrench be upgraded to a higher version, please?

This issue is not present in v2.0.11.


Combine independently defined renderers

Is it possible to combine different Renderers (a bit the same way python markdown makes it possible to chain the extensions).

I guess without some code changes it is not possible. Can you please add some details of how it would be used?


Not really an issue but a humble request.

Hello,
Can you please provide or at least give a simple instructions on how to take just the inverted pet frame module from your addon? I'm a fan of default UI, currently using obbles modui but I'd like to pet frame "match" the ToT frame. I have no idea on how to do this, I'd be thankful if you could help.

You can use this addon and set it up so that it looks like the default one and then only use pet improved via options. Disable Unitframes module in modui and use this addon for unitframes instead, I will not begin to make tutorials on how to mod, sry.


Merge calculate_rotation_scan_range into calculate_grid_scan_range

Given that calculate_grid_scan_range can already calculate 1D scans we should be able to replace calculate_rotation_scan_range with it. This will lead to less complicated code and less opportunities for error.

E.g. see https://github.com/dials/nexgen/commit/9534cd37c148e775506a5d3c72c51edba9cbdd8a where the test_calc_rotation_range still passes with minimal changes


How to Get Position of multiple dragged Nodes?

Maybe it's just me being stupid, but i can't figure out how to get the modified coordinates of dragged nodes, when multiple nodes are selected and dragged to a new position together. (dragging itself works, but i cant get the positions into my local state)
For a single node onNodeDragStop() works fine, but it does not get called when dragging multiple nodes at once.
I tried to make use of onLoad using ReactFlowInstance.getElements() to figure out changed node positions by comparing to my local state, but there ran into the problem that the position object of Nodes in the Array returned by getElements() do not change, when moving Nodes around an just contains the initial values.
So i am wondering what is the intended way to handle the position change when multiple nodes are dragged at once?

Hey @pulli7
Could you check out the provider example (source code). There we read out the updated node positions. Does it help?

Thank you, i totally forgot about that option (even started out with the provider example at the very beginning)...
That indeed helps, i can now get the coordinates of the dragged nodes. Feels a bit strange to do it that way for my specific usecase though, as i would like to update my local state only when nodes are dropped, and not while still moving. (I am using your package to build a graphical editor, and constantly cache the data of the Nodes into database with session cookie to prevent users lose their drawed graph if they press F5 or something. So updates happening constantly while dragging are an performance issue for me in this case) But i think i could make it work with additional evaluation of mouse events.
An enhanched variant of onNodeDragStop() that returns array of Nodes and also fires when multiple Nodes are dragged would make it easier for that usecase. (would also be more consistent in my opinion, because why limit the drop event to dragging single Nodes?)

You are right. We might add onSelectionDragStart, onSelectionDrag and onSelwctionDragStop. What do you think about that?

I think that would be a nice addition to the otherwise already great package.
Would make application code simpler an cleaner in usecases similar to mine.

The new event handlers are available in v5.6.0


[Features] DLPack functions

Related discussion: https://github.com/tensorflow/community/pull/180
Finished Functionality:

TFE_ToDlpackCapsule and TFE_FromDlpackCapsule

ToDLPack
import tensorflow as tf
a = tf.constant([1,20])
from tensorflow.python import pywrap_tfe
cap = pywrap_tfe.TFE_ToDlpackCapsule(a)

from torch.utils.dlpack import from_dlpack

th_tensor = from_dlpack(cap) # works well

FromDLPack
import torch as th
from torch.utils.dlpack import to_dlpack

a = th.tensor([1,5,2])
cap = to_dlpack(a)

import tensorflow as tf
from tensorflow.python import pywrap_tfe

tf_tensor = pywrap_tfe.TFE_FromDlpackCapsule(cap)

Things welcome ideas and TODO:

Tests: how to test DLPack in TF's test system? Just test only TF<->cap<->TF? How to test Interoperability with other framework? And seems hard to test C++ API, can I only to Python Test?
Device Type: What's custom device? How is this related to DLPack's device flag? How about AMD's GPU?
Whether TF's Tensor has strides and byte offsets?
Naming: How should I name the API? tf.to_dlpack? or make it under some sub namespaces?

And I have to say I'm not familiar with TF's codebase, but I tried to make it consistent with other codes as much as possible. Please feel free to point out any misuse in my codes.

Tests: how to test DLPack in TF's test system? Just test only TF<->cap<->TF? How to test Interoperability with other framework? And seems hard to test C++ API, can I only to Python Test?

A might relevant question is shall we test on TF 1.0? Such discussion seems to be missing in the RFC. Is the current solution applicable to TF 1.0?

@jermainewang this is a PR to the TF repo and we're not cutting any new releases of TF 1.x from this repo, so no need to support 1.x.

I've addressed the comment above. Feel free to leave more advices.

I just remembered another issue, that the alignment requirement is not checked in TFE_NewTensorHandleFromDeviceMemory. Based on our previous test, gather op on CPU sometimes may failed due to the misalignment. Other than that, in most cases it works fine even with misalignment.

Addressed the issues above

Addressed issue above

Fixed the python symbol problem. Now it's available at tf.experimental.dlpack.to_dlpack and tf.experimental.dlpack.from_dlpack

One potential bug I found is that tf didn't initialize gpu device when import tensorflow as tf. Therefore if directly call from_dlpack, there will raise error Unhandled exception in event loop:. It works well if the device is already initialized. Is there any tensorflow api to do the initialization?

Could you explain more? TF should not be initializing the GPUs import tensorflow as tf this in because we need to give an opportunity to users to call the various configuration APIs in tf.config prior to initialization. We right now initialize the GPUs implicitly as part of executing the first op or creating a tensor.

@jaingaurav It didn't show more information other than Unhandled exception in event loop. And what I'm thinking is just to mention it in the documentation, that you need to initialize the device before using dlpack. Is there any API to ensure the device readiness? Or just say user could execute something like with tf.device("gpu:0"): tf.constant([1.1]) to initialize it?

And what I'm thinking is just to mention it in the documentation, that you need to initialize the device before using dlpack.

Since this is a documentation change, I'd suggest doing that in a separate PR that we can land separately.

I've fixed the pylint error and updated the golden rule prototxt. Hope this could fixed the ci error above.

Hi,
I already manually merged this PR at https://github.com/tensorflow/tensorflow/commit/9cd1a63a74655a95bf7036248a6a8b328d4a2f68 since I didn't want to wait for a round-trip.  I fixed the lint and other minor issues myself.

Thanks and glad to see this merged! When will tf release 2.2?

Thanks and glad to see this merged! When will tf release 2.2?

Yes, unless we have to roll it back for some reason.

Awesome news, and great work everyone.  I've merged the RFC changes as well.  Looking forward to seeing this in 2.2!  We should coordinate some press around this.  Is there an expected release date you can share?

Looking forward to seeing this in 2.2! We should coordinate some press around this. Is there an expected release date you can share?

CC @goldiegadde @wolffg

Looking forward to seeing this in 2.2! We should coordinate some press around this. Is there an expected release date you can share?

CC @goldiegadde @wolffg

Any word here?  We've got our Tensorflow pipeline and accelerated tabular dataloader fully working using dlpack now.  We've got plans to share this in a blog post.  Not sure the timing of that but we would love to coordinate promotion if possible.


refactor adding controlled access tracks; prep for more filetypes/missing values

Travis builds broke, but I doubt it's related to this change 😕


Add Pop-Up to confirm row deletion

Add a popup that lets the user confirm/cancel when they click on the delete button in a table row.

needs to be set up for every delete across site, including admin tools


cypress: 5.3.0 -> 6.0.0

Motivation for this change
Cypress update to 6.0.0
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] 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.

Result of nixpkgs-review pr 105239 run on x86_64-linux 1

1 package built:

cypress


[wrt] Update tct-appwgt-wrt-test

Impacted tests(approved): new 0, update 7, delete 0
Unit test platform: [Tizen]
Unit test result summary: pass 2, fail 0, block 5

BugID: https://bugs.tizen.org/jira/browse/TC-2222
no /home/app/.config/xwalk-service folder

I prefer to merge the PR when the 5 blocking issues are testable.

Close it for now.


Cannot import face_recognition on python3.6.8

Using python3.6.8, cannot import face_recognition, getting the below error:
Command "/home/george/EagleEye/venv/bin/python3.6 -u -c "import setuptools, tokenize;file='/tmp/pip-install-vfg0i0dv/dlib/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /tmp/pip-record-pwnyeh3e/install-record.txt --single-version-externally-managed --compile --install-headers /home/george/EagleEye/venv/include/site/python3.6/dlib" failed with error code 1 in /tmp/pip-install-vfg0i0dv/dlib/
All the other requirements are installed without problems.
Any ideas?

Which os are you using?
Please try using a Ubuntu 16.04 VM.
Try uninstalling face_recognition and dlib.
Then install only dlib and then face_recognition

Linux MInt 18.3
I will set up a vm and let you know,
thanks


[HOLD] Fix New flag showing after Expense description update

Details

Fix New flag showing after Expense description update
Fixed Issues

$ https://github.com/Expensify/App/issues/50019
PROPOSAL: https://github.com/Expensify/App/issues/50019#issuecomment-2397158736
Tests

[x] Verify that no errors appear in the JS console

Login the app.
Create workspace.
Submit the expense in the workspace chat.  There should be only one expense in the chat, so when you click on it, one expense details view is opened.
Open the submitted expense.
Update the description -> The message about the updated Description is created. Make sure there is no New indicator.
Click the header link to return to the workspace chat.
Open the expense again, make sure there is no New indicator for the message mentioned in the step 7.

Try to repeat steps 6-8 for amount/merchant/date/category field updates. Make sure no New indicator appears.
Offline tests

Same, as in the Tests section.
QA Steps

[x] Verify that no errors appear in the JS console

Same, as in the Tests section.
PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/4f6e3240-b3db-4f58-a24e-e87f63d5071a

Not relevant anymore.


Can't create user with numeric group name

https://gist.github.com/shijujckl/62800757bbaec2b36d38972a58e1cc49#file-gistfile1-txt
Not able to start the docker containers when running 'ddev start' in macOS Mojave(10.14.6).
To Reproduce
mkdir newproject
cd newproject
ddev config
ddev start
Expected behavior
Expected to successfull start the containers using ddev start
Version and configuration information (please complete the following information):

macOS Mojave(10.14.6)
Docker Desktop in Mac 3.6.0 (67351)
ddev version v1.17.7

Thanks so much for providing the gist.
It looks like you have a user uid of 310822, which is higher than anybody has ever reported, and higher than linux allows (65536)
I'm really surprised. Is this in a corporate environment or something?
Please show id -a
You might try ddev config global --omit-containers=ddev-ssh-agent and see if it offers any help, but I imagine it will croak when trying to build the web and db images.

Thank you @rfay for your quick response. Please find my answers below.
workspace 310822$ id -a
uid=1663749668(310822) gid=1240132652 groups=1240132652,12(everyone),20(staff),62(netaccounts),79(_appserverusr),80(admin),81(_appserveradm),98(_lpadmin),702(com.apple.sharepoint.group.2),703(com.apple.sharepoint.group.3),33(_appstore),100(_lpoperator),204(_developer),250(_analyticsusers),395(com.apple.access_ftp),398(com.apple.access_screensharing),399(com.apple.access_ssh),701(com.apple.sharepoint.group.1)
I have tried using --omit-containers=ddev-ssh-agent and same is happening for db images

Wow, that gid! Also that uid! I've never seen anything like it, "uid=1663749668(310822) gid=1240132652"
You can certainly create a new user on your machine and use that, but that's a lot of trouble.
This is probably a bug that needs a workaround for situations like yours. On Windows we already do the workaround because the uids and gids aren't posix ones.

I'll experiment with a workaround that could maybe hit the next prerelease.
Could you please see if the current prerelease has potential to solve this problem? https://github.com/drud/ddev/releases/tag/v1.18.0-alpha4
It uses  Debian 11 Bullseye as a base for the web image.
You'll have to do ddev config --omit-containers=db, which will mean you won't have a working system, but it will tell us if bullseye fixes this, and then that's a simple solution.

Hmm, I must have been wrong about this. I created a user with your uid and gid and have no trouble with it (although other apps seem to have trouble).
$ id -a
uid=1663749668(biguser) gid=1240132652(biggroup) groups=1240132652(biggroup),12(everyone),61(localaccounts),79(_appserverusr),80(admin),81(_appserveradm),98(_lpadmin),33(_appstore),100(_lpoperator),204(_developer),250(_analyticsusers),395(com.apple.access_ftp),398(com.apple.access_screensharing),399(com.apple.access_ssh),400(com.apple.access_remote_ae),701(com.apple.sharepoint.group.1)

Is your username really "310822"? Maybe that's the problem, I don't think I've seen anybody with a numeric username before.

Ah, I'm able to recreate your problem with a numeric username, changed "biguser" to "310822"
310822@rfay-tag1-m1 junk % id -a
uid=1663749668(310822) gid=1240132652(biggroup) groups=1240132652(biggroup),12(everyone),61(localaccounts),79(_appserverusr),80(admin),81(_appserveradm),98(_lpadmin),33(_appstore),100(_lpoperator),204(_developer),250(_analyticsusers),395(com.apple.access_ftp),398(com.apple.access_screensharing),399(com.apple.access_ssh),400(com.apple.access_remote_ae),701(com.apple.sharepoint.group.1)
310822@rfay-tag1-m1 junk % ddev start
Starting junk...
Building ddev-ssh-agent
Service 'ddev-ssh-agent' failed to build : The command '/bin/sh -c (groupadd --gid $gid "$username" || groupadd "$username" || true) && (useradd  -l -m -s "/bin/bash" --gid "$username" --comment '' --uid $uid "$username" || useradd  -l -m -s "/bin/bash" --gid "$username" --comment '' "$username")' returned a non-zero code: 6
Failed to start junk: failed to start ddev-ssh-agent: Failed to run docker-compose [-f /Users/biguser/.ddev/.ssh-auth-compose-full.yaml -p ddev-ssh-agent up --build --force-recreate -d], err='exit status 1', stdout='Step 1/6 : ARG BASE_IMAGE
Step 2/6 : FROM $BASE_IMAGE
 ---> c18d0948aa82
Step 3/6 : ARG username
 ---> Using cache
 ---> 70b4f803f925
Step 4/6 : ARG uid
 ---> Using cache
 ---> 415d394f95ac
Step 5/6 : ARG gid
 ---> Using cache
 ---> 4fcd11fff92b
Step 6/6 : RUN (groupadd --gid $gid "$username" || groupadd "$username" || true) && (useradd  -l -m -s "/bin/bash" --gid "$username" --comment '' --uid $uid "$username" || useradd  -l -m -s "/bin/bash" --gid "$username" --comment '' "$username")
 ---> Running in 19fe5d18740b
useradd: group '310822' does not exist
useradd: group '310822' does not exist
', stderr='Building ddev-ssh-agent
Service 'ddev-ssh-agent' failed to build : The command '/bin/sh -c (groupadd --gid $gid "$username" || groupadd "$username" || true) && (useradd  -l -m -s "/bin/bash" --gid "$username" --comment '' --uid $uid "$username" || useradd  -l -m -s "/bin/bash" --gid "$username" --comment '' "$username")' returned a non-zero code: 6'

I have a workaround for you. Please add this in your project as .ddev/docker-compose.username.yaml:
version: '3.6'
services:
  web:
    build:
      args:
        username: shijujckl
        gid: shijujckl
  db:
    build:
      args:
        username: shijujckl
        gid: shijujckl

Then add this as ~/.ddev/ssh-auth-compose.username.yaml:
version: '3.6'
services:
  ddev-ssh-agent:
    build:
      args:
        username: shijujckl
        gid: shijujckl

And see what happens!

Related, and also would be good to solve https://stackoverflow.com/questions/64933879/docker-ddev-unicodedecodeerror-utf-8-codec-cant-decode-byte-0xe9-in-positio/64934264#64934264

@rfay Thank you so much for your work around on this issue. Before seeing your comment I have tried creating another user in my mac and it worked. Also I have tried username.yml files and earlier errors are gone and now getting some permission error
AMBIN01432:junk 310822$ ddev start
Starting junk...
Pushed mkcert rootca certs to ddev-global-cache/mkcert
The hostname junk.ddev.site is not currently resolvable, trying to add it to the hosts file
ddev needs to add an entry to your hostfile.
It will require administrative privileges via the sudo command, so you may be required
to enter your password for sudo. ddev is about to issue the command:
sudo /usr/local/bin/ddev hostname junk.ddev.site <IP_ADDRESS>
Please enter your password if prompted.
Running   Command=sudo /usr/local/bin/ddev hostname junk.ddev.site <IP_ADDRESS>
Failed to execute sudo command, you will need to manually execute '/usr/local/bin/ddev hostname junk.ddev.site <IP_ADDRESS>' with administrative privileges
Creating volume "ddev-junk_nfsmount" with local driver
Building db
Building web
Creating ddev-junk-db ...
Creating ddev-junk-db ... done
Creating ddev-junk-dba ...
Creating ddev-junk-web ...
Creating ddev-junk-web ... error
ERROR: for ddev-junk-web  Cannot start service web: error while mounting volume '/var/lib/docker/volumes/ddev-junk_nfsmount/_data': failed to mount local volume: mount :/Users/310822/workspace/junk:/var/lib/docker/volumes/ddev-junk_nfsmount/_data, data: addr=<IP_ADDRESS>,hard,nolock: permission denied
Creating ddev-junk-dba ... done
ERROR: for web  Cannot start service web: error while mounting volume '/var/lib/docker/volumes/ddev-junk_nfsmount/_data': failed to mount local volume: mount :/Users/310822/workspace/junk:/var/lib/docker/volumes/ddev-junk_nfsmount/_data, data: addr=<IP_ADDRESS>,hard,nolock: permission denied
Encountered errors while bringing up the project.
Failed to start junk: Failed to run docker-compose [-f /Users/310822/workspace/junk/.ddev/.ddev-docker-compose-full.yaml up --build -d], err='exit status 1', stdout='Step 1/6 : ARG BASE_IMAGE
Step 2/6 : FROM $BASE_IMAGE
---> 83f2e55a79bc
Step 3/6 : ARG username
---> Using cache
---> 37f7194cd8b0
Step 4/6 : ARG uid
---> Using cache
---> 2c07d8c1f831
Step 5/6 : ARG gid
---> Using cache
---> ccb58b9fef65
Step 6/6 : RUN (groupadd --gid $gid "$username" || groupadd "$username" || true) && (useradd  -l -m -s "/bin/bash" --gid "$username" --comment '' --uid $uid "$username" || useradd  -l -m -s "/bin/bash" --gid "$username" --comment '' "$username")
---> Using cache
---> 2d084242d2c5
Successfully built 2d084242d2c5
Successfully tagged drud/ddev-dbserver-mariadb-10.3:v1.17.7-junk-built
Step 1/6 : ARG BASE_IMAGE
Step 2/6 : FROM $BASE_IMAGE
---> b6c9152d8231
Step 3/6 : ARG username
---> Using cache
---> a08e4d21f3c0
Step 4/6 : ARG uid
---> Using cache
---> 787ce7d9e780
Step 5/6 : ARG gid
---> Using cache
---> ce07e570e647
Step 6/6 : RUN (groupadd --gid $gid "$username" || groupadd "$username" || true) && (useradd  -l -m -s "/bin/bash" --gid "$username" --comment '' --uid $uid "$username" || useradd  -l -m -s "/bin/bash" --gid "$username" --comment '' "$username")
---> Using cache
---> d1fb5844379a
Successfully built d1fb5844379a
Successfully tagged drud/ddev-webserver:v1.17.7-junk-built
', stderr='Creating volume "ddev-junk_nfsmount" with local driver
Building db
Building web
Creating ddev-junk-db ...
Creating ddev-junk-db ... done
Creating ddev-junk-dba ...
Creating ddev-junk-web ...
Creating ddev-junk-web ... error
ERROR: for ddev-junk-web  Cannot start service web: error while mounting volume '/var/lib/docker/volumes/ddev-junk_nfsmount/_data': failed to mount local volume: mount :/Users/310822/workspace/junk:/var/lib/docker/volumes/ddev-junk_nfsmount/_data, data: addr=<IP_ADDRESS>,hard,nolock: permission denied
Creating ddev-junk-dba ... done
ERROR: for web  Cannot start service web: error while mounting volume '/var/lib/docker/volumes/ddev-junk_nfsmount/_data': failed to mount local volume: mount :/Users/310822/workspace/junk:/var/lib/docker/volumes/ddev-junk_nfsmount/_data, data: addr=<IP_ADDRESS>,hard,nolock: permission denied
Encountered errors while bringing up the project.'

These problems are that you have nfs enabled but have not set it up on your computer. Completely different. See the docs on performance. Sorry, on my phone or I'd give more guidance.


sql-database-elastic-scale-configure-deploy-split-and-merge.md using classic azure portal that no longer available

Creating an issue
To create "Cloud Services" still mentioning classic azure portal https://manage.windowsazure.com/ , but it's not available anymore. Document could be actualized
https://github.com/MicrosoftDocs/azure-docs/blob/master/articles/sql-database/sql-database-elastic-scale-configure-deploy-split-and-merge.md

@SychevIgor - Thanks for your feedback! Could you provide this feedback by revisiting the original document page (on http://docs.microsoft.com) and using the "Give documentation feedback" feature in bottom of the page? That way, your valuable feedback is funneled through our appropriate internal processes where the content authors and subject matter experts will get notified, and issues will get resolved in timely manner.

@SychevIgor - Thanks for reporting this, and apologies for the delayed reply. The references to the classic portal are no longer in the article.

#please-close


Use Rc::try_unwrap for (possibly?) better performance

Putting this up here for discussion.
We can make use of mutability in Node::set_value and Node::remove_value, by turning values of type Rc<Node<...>> into mut Node<...> using Rc::try_unwrap(...).unwrap_or_else(|rc| (*rc).clone()).
Essentially that'll just give us an owned Node if we hold the only reference to it, or it'll clone the inner Rc.
Theory is that if the calling code is basically doing lots of changes on the HAMT in a tight loop, that's going to get sped up, since that'll end up being lots of in-place mutation.
The benchmarks don't look too promising though. I got ~10% improvements.
To test this, just compare the two commits in the PR against each other. Specifically their cargo bench -p wnfs-bench node.
Also, I had to hand-expand the async_recursion macro for set_value and remove_value, because otherwise there would've been lifetime issues with the original versions. We should probably figure out another way, but only if we really want to keep this PR.

@appcypher and I figured out that adding + 'static to K, V and H type parameters would make the lifetime annotations redundant.
It's for sure safe to annotate H: 'static, but for K and V it's not too obvious. It'd limit users of Node<K, V, H> to only put owned things (never references e.g. no &str for keys) into the HAMT. That's the case for all of rs-wnfs, so technically we could just do that? That'd make the code cleaner again and we could use #[async_recursion(?Send)] again.


MQTT over WebSockets: After hours of running time receiving published messages is strongly delayed

Describe the bug
I am using MQTT over secure websockets. The most of the time the device is simply pushing mqtt message in an interval of 5 seconds to a single topic. But sometimes another mqtt client publishes a message to a subscribed topic. Right after the start of the device the published message from the other client is immediately send to the MQTT event callback. Thats fine. But after a several hours of running time the published message of the other client is strongly delayed. Sometimes up to one minute or longer. I have looked at the behavior and found out, that the MQTT message gets immediately into the WebSocket buffer. But the MQTT stack does not get it right away. This behaviour stops when restarting the device.
If using MQTTS instead of MQTT over WebSockets everything works fine.
Please also mention any information which could help others to understand
the problem you're facing:

What target platform are you using?
STM32F4
What have you tried to diagnose or workaround this issue?
Looking at the debug prints of the mqtt and websocket stack
Is this a regression? If yes, have you been able to "git bisect" it to a specific commit?
Not that I'm aware of.

To Reproduce
Steps to reproduce the behavior:

Connect to a broker via MQTT_TRANSPORT_SECURE_WEBSOCKET
Subscribe to a topic X
Publish a message to a topic Y with QoS2 every 5 seconds
Sometimes publish a message to topic X (QoS1) from another client
Watch the delay between publish and receiving this message is rising over the time

Expected behavior
The time between publish a message and getting it through the event cb should be relatively constant.
Impact
I'm not able to deliver the product with this bug.
Logs and console output
Here is a log of this situation. Lines, starting with "!!!->" are comments added by me.

 An hour later there are 30 bytes. And that number is never falling.

@jumo-sballing Can you perhaps share your code or provide a minimal sample which allows to reproduce the issue? I've tried to reproduce this case, according to the description, with the following setup:

local mosquitto server with secure websocket port open
slightly modified MQTT sample from Zephyr, which periodically publishes QoS2 packet to one topic, and subscribes to another one (built with overlay-tls.conf and overlay-websocket.conf.
mosquitto_pub which publishes QoS1 to the subscribed topic

Even after publishing lots of packets, I see no regression in processing time, nor data accrual in internal websocket buffer.

Thank you very much for checking this out. And thanks for the reply. I will setup a small example based on my code.

@jumo-sballing @rlubos I think I just got the same issue, or at least a suspiciously similar issue. If it is the same issue your message should be delayed until there is a second message arriving?
The issue I stumbled upon is triggered when 2 consecutive websocket messages are sent before the first one is read out with websocket_recv_msg().
The issue is around zephyr/subsys/net/lib/websocket/websocket.c line 832 - 848 on main branch. When websocket_recv_msg() is entered with an already existing message in ctx->tmp_buf, you will hang in recv (or return, depending on your timeout settings) until another message is received.

When websocket_recv_msg() is entered with an already existing message in ctx->tmp_buf, you will hang in recv (or return, depending on your timeout settings) until another message is received.

Ah, that would make sense... If two consecutive messages are read with a single recv() call (which could happen, it's stream underneath), websocket_recv_msg() will only process one of them. Then on the consecutive call, it ignores the fact the next message may already be present in the buffer, and tries to read it out from the socket, causing hickups. Nice find @svenstorp !
We have another WebSocket-related issue that proposes a complete rework of this websocket_recv_msg() function (https://github.com/zephyrproject-rtos/zephyr/issues/52327#issuecomment-1337430132). Need to make sure to get rid of this bug during the rework.

Please check PR https://github.com/zephyrproject-rtos/zephyr/pull/53070.

@jumo-sballing @svenstorp Please give #53070 a try, I've briefly tested it with MQTT and it seems to work fine, the updated RX implementation should help with the issue described above as well.

Hopefully fixed by #53070, please reopen if that is not the case.

Sorry for beeing so late at the party. I was sick for a few days. I've tested #53070 with my code and now it works fine. I've got no delay - even after hours.
Thank you very, very much!


Write a contributor communication SOP

As a wrangler I want to understand how  and when to follow up with contributors during the submission process
This goal of this task is to define an SOP about follow up with contributors after an initial expression of interest in contributing data to the HCA or after curation of a publication
The questions about this process are

How to best support non-responsive contributors?
How to improve the rate of response?
How frequently to repeat questions when they don't respond
What is a reasonable escalation policy when a lab member isn't responded to more senior lab members or even funders
Can we improve how we write our emails to get responses?

This SOP should be linked to the general new contributor SOP https://github.com/ebi-ait/hca-ebi-wrangler-central/blob/master/docs/SOPs/contributor_dataset_SOP.md
acceptance criteria

[ ] An contributor follow up SOP accepted by all wranglers in https://github.com/ebi-ait/hca-ebi-wrangler-central/tree/master/docs/SOPs

This SOP will need to be reviewed and success metrics to be considered by how frequently people respond in the first instance and for non-responders, how frequently we need to escalate
We, of course, need to consider which parts of this are under our control and which parts aren't

@mshadbolt @rays22 @clairerye I don't know if there is other things I have missed here

@mshadbolt @rays22 @clairerye I don't know if there is other things I have missed here

I think the questions in the task description look quite complete. I could not think of any major point to add, maybe only a minor one.
I think it is implied in the goals and acceptance criteria, but to make it explicit for the SOP, I think we also need to think about how to track dates of communications/responses. This would be useful for both the success metrics and for the wranglers to know when it is time to nudge a contributor again.

I have started a draft here: https://docs.google.com/document/d/1XC3p8TdgDiJGVuJXYIJppWkBsKWKOhw_fhPLC5A0W3g/edit
Not really ready for a full review yet but if people want to add anything or suggest any changes as it currently is feel free

I received feedback on the draft from Claire, Laura and wranglers.
There may be some minor changes to some of the template emails but I decided it was 'done enough' to put it on to the wrangler documentation website here: https://ebi-ait.github.io/hca-ebi-wrangler-central/SOPs/contributor_communication_SOP.html
I also made some links to the new page from the contributor wrangling SOP.
I also added links to the contributor communication SOP to the visual index.


Why I can't use url("static", "img/favicon.ico") function in my haml file

I want to include some static files in my haml file and use url function like this:
%meta{name: 'msapplication-TileImage', content: 'url("static", "img/favicon.png")'}
%link{rel: 'icon', sizes: '192x192', href: 'url("static", "img/favicon.png")'}
%link{rel: 'apple-touch-icon', sizes: '192x192', href: 'url("static", "img/favicon.png")'}
%link{rel: 'shortcut icon', type: 'image/x-icon', href: 'url("static", "img/favicon.ico")'}
- include_static 'bs3/bootstrap.min.css'

The last line can load static file correctly, others failed. If I should load sth in my haml header to use url function? Maybe there's other method I don't know to make it, I think the docs about The url() function or Include helpers docs lacks some details.
I need some help :)

The correct is
%meta{name: 'msapplication-TileImage', content: '{{=url("static", "img/favicon.png")}}'}
%link{rel: 'icon', sizes: '192x192', href: '{{=url("static", "img/favicon.png")}}'}

@laozhu yes, @cassiobotaro is right, the url function is python powered, so you still have to use the weppy template decorators {{ }} to use it inside the template, both with html and haml.

@cassiobotaro Maybe I am still in Django template style, thanks for your help :+1:
@gi0baro I think in The url() function section should include something about url usage in template, it will be helpful to someone like me migrate from django template or jinja2.

@laozhu have you read the templating chapter? In particular http://weppy.org/docs/0.6/templates#included-helpers and http://weppy.org/docs/0.6/templates#basic-context sections? Maybe I should better describe the usage of url in those sections?

Actually, I read the two section more than 3 times :)
I am not familiar with pure python code in template, this is why I can't get {{=url('static', 'js/app.js')}} usage by myself; I am familiar with Django or Flask url function in template, so I write it like this: url('static', 'js/app.js').
I think The templating system needs better describe for how to include static files (except css and js) in template and how to include route name in template.
And when you show us how writing python in template powerful, this could be a nice example :)

Ok, I've created #117


Animation implementation using the Web Animations API

This builds on the stubs added in #235. I hooked up Transaction in the reconciler, and setup support for the withAnimation global function to apply animations on Binding changes.
https://user-images.githubusercontent.com/13581484/125217689-46be2800-e28f-11eb-91b4-63c5a642fddb.mov

Currently the actual animatable properties of an Animatable View/Modifier are ignored, and all styles are animated.

There doesn't seem to be an API for computing style values at a particular step which made solvers tricky. But I updated it to use solvers by creating an animation, reading the computed styles, then cancelling it for each step, and then creating keyframes from those steps. Seems to work well enough.
I also made the animations interruptible to match SwiftUI.


pg_config returns wrong include directory

Using the postgres image version 9.6.1-alpine, the command pg_config returns wrong definitions for the include directories. After installation of the package postgresql-dev, the header files are located in /usr/include/postgresql/ without the local prefix.
$ pg_config
...
PKGINCLUDEDIR = /usr/local/include/postgresql
INCLUDEDIR-SERVER = /usr/local/include/postgresql/server
...

This should be fixed by https://github.com/docker-library/postgres/pull/247. :+1:


Update runs no test.

I've updated from 1.1.0 to 1.2.0 and now none of my test seems to run. When I look at a ts file in chrome running karma it's empty with a comment saying that the file hasn't been compiled. Has any of the configuration changed?

Hello @wawyed, I just merged a little fix made by @lelandmiller, which SO are you using? (windows, linux, osx?)

Windows.

ok, one minute. I will try to figure out what is going on

Please update to version 1.2.1.


(Refactoring) Support of Proxy for manage page

Relates to #957
3d part of integration with Auth-os Proxy contract

support of Proxy for manage page

Nothing is changed in UI.
@fernandomg @dennis00010011b it is higly recommended to merge it ASAP (of course, if no any concerns), as this PR contains an updated ABI for Proxy, RegistryExec, MintedCappedIdx contracts in the attempt to fix open issues in Auth-os.

@vbaranov
Minor issue on contribution page: incorrect value minCap is displayed for whitelisted investors.

Everything else works well but old auth_os issues  still here(for Minted):

Tier durations is not  updatable if index > 1
Method getCurrentTierInfo always return info for  1st tier (# 0) . As a result investors can’t buy in tier if  maxCap was sold  in previous tier, whitelisted investors can’t buy if tier number !=0  etc.
Crowdsale cannot be finalized before the end of crowdsale  if hard cap is sold out

Minor issue on contribution page: incorrect value minCap is displayed for whitelisted investors.

Thanks @dennis00010011b. Fixed. I'm gonna to merge.


Update manifests/x/xanderfrangos/twinkletray/1.15.1/xanderfrangos.twinkletray.locale.en-US.yaml

Fixes an issue where this package has multiple different monikers
Microsoft Reviewers: Open in CodeFlow

[Policy] Reset Labels


Odysee discover channels

Read from primary category. Channel ids not coming over, it's showing all.

For each homepage, it should pull the respective list.

This was fixed, but the other homepages don't pull their respective lists.


Add Konflux build file

Related Issue: https://issues.redhat.com/browse/ACM-10719
Description of Changes: Adds Dockerfile for Konflux builds
What resource is being added: N/A
Is this a Hub or Managed cluster change?: N/A
Notes: Also contains minor repo cleanup

/cc @rh-dluong @Kyl-Bempah

/cherry-pick release-2.10

/cc @bjoydeep @jlpadilla


Export UMD and ESM

To use this library with Deno, ESM would be required as Deno does not support UMD.
And the easiest thing would be just to export also a dotnet.mjs file (with esm code inside).

I see. (The issue looked a little dead, so I opened a new one)

@Elringus is there an estimate of when the problem will be fixed? (I would love to use this)

It'll probably be a while. First, https://github.com/dotnet/runtime/issues/80045 has to be resolved, then I'll have to find a time to resolve #20, which would likely take months. If you just looking for a way to interop with C# from JS, take a look at https://devblogs.microsoft.com/dotnet/use-net-7-from-any-javascript-app-in-net-7 Aside from TS types generation and auto bindings generation, modern .NET mostly covers this project features.


Consider changing mental retardation to intellectual disability

This would be consistent with GARD
https://github.com/obophenotype/human-phenotype-ontology/issues/3290

This sounds good to me. I think coordinating with OMIM and Orphanet is a good idea too.

is there a preferred term re hearing loss vs deafness where the hearing loss is incomoplete, e.g autosomal dominant nonsyndromic deafness. GHR uses hearing loss which seems preferable http://ghr.nlm.nih.gov/condition/nonsyndromic-deafness

What about where the MR is embedded inside a syndromic label? E.g. ectodermal dysplasia with mental retardation and syndactyly

This will impact 2 terms in protected subsets
[Term]
id: MONDO:0010955
name: ectodermal dysplasia with mental retardation and syndactyly
subset: clingen
synonym: "ECTODERMAL DYSPLASIA WITH MENTAL RETARDATION AND SYNDACTYLY" RELATED [OMIM:600906]
xref: MESH:C538018 {source="MONDO:equivalentTo", source="MONDO:ontobio"}
xref: OMIM:600906 {source="MONDO:equivalentTo", source="MONDO:Leader"}
xref: UMLS:C1833169 {source="NCBI:mim2gene_medline", source="OMIM:600906"}
is_a: MONDO:0003847 ! inherited genetic disease

[Term]
id: MONDO:0014911
name: growth retardation, intellectual developmental disorder, hypotonia, and hepatopathy; GRIDHH
subset: clingen
synonym: "GRIDHH" EXACT [OMIM:617093]
synonym: "Growth Retardation, Intellectual Developmental Disorder, Hypotonia, and Hepatopathy" EXACT [OMIM:617093]
xref: OMIM:617093 {source="MONDO:equivalentTo", source="MONDO:Leader"}
xref: UMLS:C4310720 {source="NCBI:mim2gene_medline"}
is_a: MONDO:0003847 ! inherited genetic disease

Proposed implementation date: Jan 5, pending discussion

note to self - check to see if this was done/can be closed

This wasn't done, there are still a lot of terms with MR in the label.

duplicate of #2310.
Reassigning to Nico.


Extract text directly from file-object / file-content rather than using filename

Maybe this is already possible?
How would i go about to extract text from the content of a file, rather than reading the file itself? Background is that using the upload component in Dash, one gets the content of the file rather than a pointer to the file location.
Perhaps, by accessing one of the internal functions in textract and specifying an extension this is already possible?

Unfortunately there's no way to do that, as textract launches external commands (most notably pdftotext) to process files sometimes.


deb packaging: add support for Ubuntu 20.04

Use the lsb_release output to discover the current distribution distributor ID and release. Use this information as an index into associative arrays providing the package name details for the given release.
Verified using Ubuntu 22.04 LTS and Ubuntu 20.04 LTS.
Signed-off-by: Tim Whisonant<EMAIL_ADDRESS>

Coverage remained the same at 66.091% when pulling 48844144ff0e94f066b11ffdc6c0896780b02df2 on tswhison/deb_pkg into 1adf63c2fc7c44419483b2a911a0b24061b1eaba on master.

I see.  Thanks for the heads-up.  Would setting a reasonable default to DIST_KEY when lsb_release is not detected be an appropriate safeguard?


MatplotlibWrapper is an undefined name

MatplotlibWrapper is an undefined name in gaussian_mixture_model.py and k_means.py.  Undefined names can raise NameError at runtime.
flake8 testing of https://github.com/eriklindernoren/ML-From-Scratch on Python 2.7.13
$ flake8 . --count --select=E901,E999,F821,F822,F823 --show-source --statistics
./mlfromscratch/unsupervised_learning/gaussian_mixture_model.py:137:9: F821 undefined name 'MatplotlibWrapper'
    p = MatplotlibWrapper()
        ^

The same issue is also present in k_means.py but the * import masks it from flake8.

Thanks. This is fixed in https://github.com/eriklindernoren/ML-From-Scratch/commit/d58485e573f99a86b6033f0cd793d01fb6b423a9


subscription page fails for airbnb and coinbase

Added funds today to both organizations and made the contributions.
Everything else seems to work ok but their subscription  page serves and unexpected error
This is specially important for airbnb since they might want to modify some of their subscription amounts for the coming months.

were you logged in? as whom?

logged in as myself (I am an admin of both collectives)

I see this as well, and am logged in as the Airbnb admin user. It seems to first render the page and then a moment after the user info in the top right loads it changes to this page with the error message.

Yes, we have a fix coming. But CircleCI was down for a while today so we couldn't deploy it. It's back up now so waiting for the tests to pass then it will be deployed. Thanks for your patience!

It's deployed now. Sorry about the delay!


Zoomify with variable bin size

I was analyzing a yeast dataset that used fragment sizes as the resolution instead of a set resolution. I used to the digest command, then csort, then cload, and everything works fine using the fragment bedfile. However, when I try to use the zoomify command, I get the following error:
Traceback (most recent call last):

  File "/usr/local/bin/cooler", line 11, in <module>
   sys.exit(cli())

  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)

  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)

  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))

  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)

  File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)

  File "/usr/local/lib/python2.7/dist-packages/cooler/cli/aggregate.py", line 367, in zoomify
    chunksize, lock=lock)

  File "/usr/local/lib/python2.7/dist-packages/cooler/cli/aggregate.py", line 79, in multires_aggregate
    n_zooms = get_quadtree_depth(clr.chromsizes, clr.binsize)

  File "/usr/local/lib/python2.7/dist-packages/cooler/cli/aggregate.py", line 66, in get_quadtree_depth
    tile_size_bp = QUAD_TILE_SIZE_PIXELS * binsize

TypeError: unsupported operand type(s) for *: 'int' and 'NoneType'

I'm guessing zoomify fundamentally uses fixed resolutions and you can't coerce a variable bin size into a fixed one with the command. Any thoughts on how to proceed using built-in cooler commands?

Yep, that's right. HiGlass doesn't support uneven bins yet. However, if you want to represent your map in fragment "units" you can hack that in my making a "pseudo" fixed-binsize matrix, either using fragment numbers, though I like to use the average fragment length rounded to the nearest kb as the fake bin size.
The easiest way to do this would be to dump the bins and pixels into two separate files (cooler dump -t bins, cooler dump -t pixels), modify the bins file appropriately, and then use cooler cload -f coo <pseudobins> <pixels> <out.cool>

Closing this, but the coarsening of variable-sized bins is covered in #81.


Southern Africa pops 1946

CaptainCrape
Awesome mod! However, and I don't know if this has been fixed, but many countries start without political parties for some ideologies, such as a populist party in France. Also, another issue I have found is that the POPs in South Africa, the Central Rhodesian Federation and Namibia are inaccurate for 1946 and 1992 even though accurate population data is out there. Here are the demographics for The Central Rhodesian Federation in 1946 and other years, And for South Africa. Another odd issue I had is Lesotho randomly had a constant, huge increase in White Anglo POPs. Which is very odd considering it doesn't start with any, it received 0 immigration and its only accepted culture is Sotho. Here is an example of that

Anyways, those are the only issues i've noticed so far in my main game as South Africa. Keep up the good work!

Just wondering are some of these issues coming from private conversations with other members at the paradoxplaza as I couldn't find the post you have displayed above on the main NWO thread in the paradox plaza forum.

From reddit.

Corrected the demographics... not perfect but a lot better than before. The issue of British in Nyasa-land is mysterious and remains unresolved.


Add last deployment tracking in Zulip deploy processes

It would be good to keep track of the last deployment made in our two upgrade path tools (tools/update-deployment and scripts/lib/upgrade-zulip), so that scripts/purge-old-deployments can avoid purging the last deployment (so you can revert back to it if desired).
Also this would make it easier to revert a bad deploy without having to think.
Finally, scripts/lib/upgrade-zulip should call scripts/purge-old-deployments

I'm working on this.


working with 2.4.7?

Hello thanks for making this,
Just had a quick install on a clean install of magento 2.4.7 but unfortunally at first look doesnt seem like payment crypto methods is loading on the checkout, and some issues trying to save a new blockchain

@torys877  Hi are you available to look at this happy to pay for your time, thanks


RSS Feed creator

Is there a way to extend this adapter to be able to create RSS Feeds? It would be amazing!!
Create some templates with markdown an fill it with values from ioBroker.
https://github.com/im-sticky/markdown-rss-feed-plugin
https://github.com/ioBroker/AdapterRequests/issues/30

Hm I think this is a complete new Adapter in the category like sql influx or history.
For each datapoint you can define a template and a placeholder to insert the value.
Markdown is possible but if I decide to make such an Adapter I would prefer the ejs template engine which is already used in Iobroker. With this engine you can mix HTML,css and JavaScript for further processing the value.
Markdown is only for simple formatting the output. A json datapoint for example there is no possibility to format or extract some values

@oweitman Great Idea... it can be like iQontrol https://github.com/sbormann/ioBroker.iqontrol#modifying-datapoint-configuration - for each datapoint you can configure how to be displayed. Maybe something like "Don't show the datapoint if dp x is false..." or "Show it if dp > 30..."
Hmm...  It could be rssfeed 1.x .. :-)


Implement sosreport role for log collection

Replace ci_script action with a "sos" log collector. By using industry-provided standard, we ensure that collected logs adhere to standards.
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date:

[x] README in the role
[x] Content of the docs/source is reflecting the changes

@dasm I have no context about this PR, is this PR gonna replace our current logs collection logic?

@pablintino yes, at least for edpm piece. Later we can extend it a bit more to cover all possible "log gathering" pieces.

Closing cause must-gather now implements sos-report log collection for compute nodes.


the 'img_size' argument in 'get_original' function in utils/renderer.py

https://github.com/soubhiksanyal/RingNet/blob/master/util/renderer.py
def get_original(proc_param, verts, cam, img_size):
img_size = proc_param['img_size']
...
In the current code, img_size argument is overridden by the proc_param['img_size'.]
Wondering if it's always the case that 'img_size' stored in proc_param should be used or it can actually be passed from the argument.

May i know how you would run the code can you tell me please if you have time make it steps and send to me


Release 0.15.3

Fix handling of functions that return multiple values (#623)

Test Plan
CI

(turns out automerge happens without approvals 😕)

Oki, I'll go ahead and release it then.


Feature Request - Make Bifrost Accessible From Command Line

I have been trying to fully automate the process of downloading the Samsung firmware, I would like to know if it is possible to run this program from the command line in a batch file.
I previously used Samloader for this purpose but it is dead now, if anyone has any alternative suggestions please tell me.
I fully understand if the dev doesn't want to implement this feature.
Thank you in advance.

MrWicked

There's a fork of Samloader that fixes the download error https://github.com/martinetd/samloader

Thank you so much Zachary, I appreciate the help with the issue.


Make an option "Show CC/BCC" field in options when you editing/composing new message

Expected behaviour
Make an option "Show CC/BCC" field in options when you editing/composing new message. Currently there is only an option "Request read confirmation". Add to menu "Show CC/BCC" fields. After taping on this, CC/BCC fields must be shown or hidden.
Actual behaviour
Option: "Always show CC/BCC [ ]" - is not enough. Because I have no any methods to see these fields if this option is disabled, but I'm composing a new message or replying on message without CC-field in source message.
Steps to reproduce

Just to create new message while "Always show CC/BCC [ ]" option is disabled.
Try to see CC/BCC, if you will need.

Environment
K-9 Mail version: 5.0.10
Android version: 6.0.1
Account type (IMAP, POP3, WebDAV/Exchange): POP3

Pretty sure this has changed in the beta. You can tap a down arrow to show CC and BCC fields.

This issue no longer applies to the current development version.


Override aliased providers

Summary
Currently tflocal is only overriding the default AWS provider, but it is common to have more than one AWS provider defined per terraform state, using aliased providers. This is most commonly used when mixing resources in different AWS regions in the same terraform state. It would be great if tflocal could also override these aliased providers.
Current behaviour
Given the following providers:
provider "aws" {
  region = eu-west-2
}

provider "aws" {
  alias  = "us_east_1"
  region = "us-east-1"
}

And no valid (real) AWS credentials configured, this is the output when running terraform against localstack using tflocal:
╷
│ Error: configuring Terraform AWS Provider: error validating provider credentials: retrieving caller identity from STS: operation error STS: GetCallerIdentity, https response error StatusCode: 403, RequestID: 99a8111f-87b0-49e2-8794-fb08b80bdaac, api error InvalidClientTokenId: The security token included in the request is invalid.
│
│   with provider["registry.terraform.io/hashicorp/aws"].us_east_1,
│   on aliased-providers.tf line 1, in provider "aws":
│    1: provider "aws" {
│
╵

This is of course because the aliased provider was not overriden and therefore the real AWS APIs were used instead of localstack.
Proposed implementation
This is where it gets tricky: Terraform enforces aliased providers uniqueness so we can't simply use the same approach that tflocal currently has of creating a new file with the overridden (default) provider. One solution could be to actually modify the aliased providers defined to point to localstack, making use of https://pypi.org/project/python-hcl2/ for that. Such an implementation would also require resetting the providers to the previous state upon any errors or at the end of the execution - no small change!
Workarounds
One possible workaround is to isolate the aliased providers in a separate file (i.e. aliased-providers.tf) and have a new file with the overridden providers (i.e. aliased-providers-override.tf.disabled) that terraform won't normally pick up. Then it's a matter of switching around the file extensions of said files before executing tflocal and restore the initial file names after the tflocal execution. Example using a shell script wrapper:
#!/usr/bin/env bash
set -e

disable_aliased_providers_override() {
  [[ -f aliased-providers-override.tf ]] && mv aliased-providers-override.tf aliased-providers-override.tf.disabled;
  [[ -f aliased-providers.tf.disabled ]] && mv aliased-providers.tf.disabled aliased-providers.tf;
}

trap "disable_aliased_providers_override" ERR
mv aliased-providers-override.tf.disabled aliased-providers-override.tf
mv aliased-providers.tf aliased-providers.tf.disabled
tflocal <terraform command goes here>
mv aliased-providers-override.tf aliased-providers-override.tf.disabled
mv aliased-providers.tf.disabled aliased-providers.tf

And this is what an overridden aliased provider would look like:
provider "aws" {
  alias                       = "us_east_1"
  access_key                  = "test"
  secret_key                  = "test"
  skip_credentials_validation = true
  skip_metadata_api_check     = true
  region                      = "us-east-1"
  endpoints {
    acm        = "http://localhost:4566"
    cloudfront = "http://localhost:4566"
    <any other AWS services that will be using this provider...>
  }
}

Thanks for reporting @ricoli , this is a great proposal, and we should definitely look into supporting this use case. 👍
I'm wondering if we can find an even easier/user-friendly way to support this - for example there exist some HCL parsers in Python out there (e.g., this, or this one), which may come in handy here.. 🤔
Would you have capacity to help spike out a first implementation? We cannot commit to a timeline for this feature from LocalStack side, but definitely happy to help shepherd a contribution if you're up for it..! 🙌

Hi again @ricoli , quick update. We have now added initial support for provider aliases. Can you please give it a try with the latest version of tflocal (0.7).

Hi again @ricoli , quick update. We have now added initial support for provider aliases. Can you please give it a try with the latest version of tflocal (terraform-local>=0.7)? Please keep us posted on how it goes.. Thanks!

hi @whummer - I have tested it and it worked great, no more need for workarounds! Thanks!


x/net: net/internal/timeseries/timeseries.go - Division by zero

What do you expect to see
https://github.com/golang/net/blob/2aa82152a53e7aa6b28c6390d9369ee5a5b63584/internal/timeseries/timeseries.go#L360
if num <=0
What did you see instead?
When  ComputeRange(start, finish time.Time, num int)  function
https://github.com/golang/net/blob/2aa82152a53e7aa6b28c6390d9369ee5a5b63584/internal/timeseries/timeseries.go#L354
is called with num=0 you'll get Division by zero at
https://github.com/golang/net/blob/2aa82152a53e7aa6b28c6390d9369ee5a5b63584/internal/timeseries/timeseries.go#L398

does this actually come up in real code?

No, but if someone is calling this function where num is dynamically assigned, there's possible division by zero

given this is internal code with no direct path for misuse, i don't think this is an issue.


Support of Uint8Array added

Thanks for this! Can you add a test? Also, you need to rebase, I removed Node 0.8, 0.6 from the Travis file.

I have rebased my commit and added a test similar to the ones that were already there.

Awesome! Just one other thing I didn't notice before... can you add:
HAS_UINT8ARRAY = (typeof UInt8Array !== 'undefined')

just like how the HAS_BUFFER exists? Then use HAS_UINT8ARRAY in the ifs before the instanceof checks.
Otherwise if this is run in a browser that doesn't have UInt8Array, it'll throw an error since you can't do instanceof undefined.

Yes, it's good idea. But in the current code I see:
var HAS_BUFFERS = (typeof 'Buffer' !== 'undefined'),
    undefined;

Isn't it, that typeof 'Buffer' is always string? And I don't actually understand the purpose of undefined after the comma.

Woops! :P Might as well fix that to be:
HAS_BUFFERS = (typeof Buffer !== 'undefined')

The undefined after the comma is just a prevention against someone overwriting undefined on the window (which some browsers allow) and breaking the meaning of it.

Merged. Thanks @saferif!

@saferif this has been published in 0.0.2. Sorry for the delay. I forgot to publish the new version after merging.


Cleanup shellcheck errors/warnings

Cleanup shellcheck errors/warnings
Fix a problem with printing errors in ARP Table check
Gracefully handle situations where IP & MAC addresses are missing from the vpp address log files

Coverage increased (+0.2%) to 76.993% when pulling fbb618022049f170de191e3b136226ee622e7e60 on jmedved:master into e9185cb262294645b70e5217431eaa2ec498a699 on contiv:master.


[doc] Add a description explaining that DELETE needs to be run in the batch mode

Purpose
Add a description to explaining that DELETE needs to be run in the batch mode
Tests
no code change
API and Format
no API/Format change
Documentation
Change the documentation

Hi @JingsongLi , Please help to have a look when you're free, thanks.
Add the label above the code, because that it seems strange after adding label in the code block(such as below)

@JingsongLi thanks for the review and merging!


initializing rank_start correct?

I am wondering whether line 228  end_pos = rank_start is correct?  is it not rather end_pos  == rank_end

Thank you for pointing this issue! It should be end_pos = rank_end.
I fixed this in https://github.com/sunnweiwei/RankGPT/commit/d0f338802b0091a9ff4745b9af29051962b960b5.


Removing empty tests directory

Removing empty tests directory

https://tickets.puppetlabs.com/browse/FM-2526