Cognitive Services is not the same as AI services [Bing Enrichment needs extra steps]

Before you open an issue, please check if a similar issue already exists or has been closed before.
You can also find details on Troubleshooting Common Issues. You can use these tools to help gather additional logs and details to include in  your issue.
:warning: Please DO NOT include confidential information in your issue on GitHub. :warning:
Bug Details
Describe the bug
A clear and concise description of what the bug is.
There is a diferent API call for AI services, and for Bing Enrichment, which means you ALSO have to create and accept the terms for Azure AI Services multi-service account. without it, you get the error:
Resource Group Name: "infoasst-neurartpubs"
│ Account Name: "infoasst-enrichment-cog-uxkyk"): unexpected status 400 with error: ResourceKindRequireAcceptTerms: This subscription cannot create CognitiveServices until you agree to Responsible AI terms for this resource. You can agree to Responsible AI terms by creating a resource through the Azure Portal then trying again. For more detail go to https://aka.ms/csrainotice
Please recognize this is not the AI services error, but the CognitiveServices error for the API call
Steps To Reproduce

create new sub
make deploy with correct local.env
...

What is the expected behavior?
Screenshots
If applicable, add screenshots to help explain your problem.
Information Assistant details
Please provide the following details. You can simply include a screenshot of your Info panel as well.

GitHub branch: [e.g. main]
Version or Latest commit: [obtained by running git log -n 1 <branchname>
What region is your Azure Open AI Service in?
What ChatGPT model are you using?
model name: (i.e. gpt-3.5-turbo, gpt-4)
model version: (i.e. 0613)
What embeddings model are you using?

Additional context
Add any other context about the problem here.
If the bug is confirmed, would you be willing to submit a PR?

[ ] Yes
[ ] No

This is documented in the known issues at https://github.com/microsoft/PubSec-Info-Assistant/blob/main/docs/knownissues.md#this-subscription-cannot-create-azureaiservices-until-you-agree-to-responsible-ai-terms-for-this-resource.
Azure AI Services (formerly Azure Cognitive Services) require this once per subscription. This is not the same as Azure Open AI Services nor Azure Bing Search APIs, as those are separate services.

It is not though, that's why I'm bringing it up with the tags. You have to create both an AI services AND an Azure AI Services multi-service account. Notice the diference in both the icon and kind in the attached screenshot.
Of course, you could just add to the know problems the extra step of making both, but that's the whole point I'm raising. You'll need to accept both terms.
Note this is a new sub and a new tenant so the IMPORTANT flag in the know issues can also be ammended to add this.

Interesting, thanks for calling this out as we have not seen this before. That is not how it is "supposed" to work AFIK. I will assign an engineer to investigate and update any docs accordingly.

The documentation has been updated to reflect the need for both account terms.


Kiosk mode not working

I installed from scratch the latest version from playstore.
I toggle on kiosk mode and nothing changes. Version 1.4.66

I installed from scratch the latest version from playstore. I toggle on kiosk mode and nothing changes. Version 1.4.66

Does the pencil icon on the platforms not disappear when you turn on kiosk mode?

Yea it dissappears, but path icon is still there. I guess it works,
(partially?) Or is that the intended behaviour. Thanks
On Sat, 16 Sept 2023, 11:40 Post-Mortem, @.***> wrote:

I installed from scratch the latest version from playstore. I toggle on
kiosk mode and nothing changes. Version 1.4.66
Does the pencil icon on the platforms not disappear when you turn on kiosk
mode?
—
Reply to this email directly, view it on GitHub
https://github.com/TapiocaFox/Daijishou/issues/533#issuecomment-1722244386,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWLUFSTCYWBXGKWL2TMLFCLX2W27RANCNFSM6AAAAAA425L5V4
.
You are receiving this because you authored the thread.Message ID:
@.***>

That's the intended behavior. You can still add paths and sync in kiosk mode.

https://github.com/TapiocaFox/Daijishou/issues/423


Preparing for 0.15.0 release

Hello everyone!
Since the last release of remark (v.0.14.0) was on Sep 15. 2016, I think it would be appropriate to cut a new release :)
I would like to use this issue to track any work that's needed in order to do the release (@gnab, it would be great if you take a quick look at it as well, just in case I've missed anything).

[x] PR to master + merge
[x] Create a new Github release from master with tag 0.15.0
[x] Commit the new remark-0.15.0.min.js to gh-pages branch (downloads folder)
[X] Write and update release notes (HISTORY.md) (PR: #609)
[ ] As part of the release notes update, we should also review the Wiki to make sure it's up-to-date with the features
[X] Update the list of contributors (PR: https://github.com/gnab/remark/pull/605)
[ ] Update index.html in gh-pages branch - update demo slideshow with new features

I've probably missed something, so please feel free to comment & add more to this.

Maybe update the list of contributors. We can get the names from the git log with the following command
 git log | grep Author | cut -d ' ' --fields=2- | cut -d '<' -f 1 | sort | uniq > contributors.txt

Something similar should be done with the wiki repository

Looks good @peterj! I believe I used to also update the index.html in the gh-pages branch, and if there are any new features worth mentioning, the demo slideshow could be updated to present these in addition to the wiki being updated.

Hello!
How to get this version on CDNs? :)


DOC: tab sync based on same tab name using sphinx-tabs

Reference issue
None related.
What does this implement/fix?
When selecting an OS tab in the documentation you can also use sphinx-tabs to easily sync all tabs related (i.e. with the same name). See the video attached. This implies adding the dependency and some minor modifications.
Additional information
None.

Pinging @melissawm since she is attending SciPy Conf '23 as Sprint organizer

Seems like a good idea to me, thanks @RobPasMue

This implies adding the dependency

This works as-is with sphinx-design (see https://sphinx-design.readthedocs.io/en/latest/tabs.html) which is already a dependency, so no need to add a new dependency.

This works as-is with sphinx-design (see https://sphinx-design.readthedocs.io/en/latest/tabs.html) which is already a dependency, so no need to add a new dependency.

Thanks @rgommers - indeed, I tried it first with sphinx-tabs. @melissawm here at SciPy Conference mentioned to me that the dependency had been dropped some time ago in favor of sphinx-design. That's when I started investigating how to do it with sphinx-design. I just edited the PR description 😄

Thank you @RobPasMue! That's a good addition 😃
Could you also sync venv/conda on this page?
For the next commit, could you please add in its message the following to skip some of the CI: [skip actions] [skip cirrus]. We only need to build the doc for doc only PRs.

Sure, you can count on it @tupui! Working on it right away =)


Fix diagnostics messages (correct filename and output is cleared when compilation is successful)

fixes #44

Would also close #57

I pushed a new version where diagnostics are grouped per file, and are sent together with join_all as suggested.
However I am unable to find a document where typst reports several errors, to try if it works as expected. Can typst return several error per file? it seems like it stops after the first error (hence always on a single file).


Enhance mock_iot to support describe_endpoint and register_certificate_without_ca

We wanna enhance the moto framework by adding two functions in iot_mock module.
Currently describe_endpoint and register_certificate_without_ca are what we need, and probably there will be more coming but depends on what we wanna implement in near future.
I have a branch already implemented the describe_endpoint with unittest working as well.
Now working on register_certificate_without_ca.
What we want is that we could push a branch so that we could git clone it in our pipeline, and have PR reviewed/merged as soon as possible that we don't have to keep a repo in our pipeline always.
Thank You.

register_certificate_without_ca  is done with unittest as well

Hi @zhil3, thanks for raising this.
PR's are always welcome!

@bblommers Thanks for the reply. But I don't have permission to push a branch thus how to raise the PR?

Thanks @bblommers for the info.
And could you please help up speedy up the PR process by any chances?
We are kind of in an urgent status of using those functions inside our pipelines.
Thank You


Consolidate retry logic

Toward #2042.
This PR tries to rationalize the retries in our system tests into three types:

Retry methods which raise particular exceptions (the class @daspecster implemented already)
Retry methods until the result passes a predicate (generalizing the backoff stuff in logging_).
Retry methods until a predicate on the instance's __self__ passes (e.g., until job.state == "done").

The PR adds classes for each of these categories, and attempts to use them where appropriate, replacing manual time.sleep() calls and existing "backoff" functions / loops.
It also tweaks the tox -e system-tests runner to attempt all system test modules, even if one fails, and tries to run them in order from least to most flake.
BTW,  cannot figure out how to get the translate tests to run at all, and there is one of the error_reporting tests which blows up at me.

@tseaver nice! Did you happen to try this on any of the pubsub failures? I tried retrying them last night but they still failed.

Lint failure
system_tests/retry.py:163:9: E301 expected 1 blank line, found 0

Really great work!
How did you decide which flaky test cases to target?

@dhermes Thanks!  I was tackling those which fail routinely, first on my box and then on Travis.  I imagine we haven't whacked the last mole yet.

@dhermes Any remaining issues?  I think we disagree about the third RetryInstanceState class:  should we ask @jgeewax to be the tie-breaker?  I believe @daspecster is mostly offline through the end of next wee,.

My argument for a tie-breaker: fewer classes is better :grinning:

Mine:  the third class makes the intent clearer:  I still have to puzzle out exactly how your examples work every time I look at them.

LGTM. Let's get this merged before it gets stale


Allow users to specify multiple adapters for workspaces

Add an adapter registry where users can register custom adapters if desired.
Add a workspace_type field to the Workspace model to indicate which adapter should be used.
Modify urls to take the workspace_type as an argument.
Add a mixin to a class variable containing the adapter for use in views.
Modify templates to handle having multiple workspace types (and urls) instead of just one.
Update docs.

@jmcarson if you have a chance to review, here's a pull request implementing multiple adapters. If not, don't worry about it!


Fix typo in AddingTags document

I have fixed a value in json file because it was not consistent with the previous sentence.

For example, to set the color of the tag "critical" to orange

Oops. 🙂

Documentation change only, CI not required.


NULL model weighting

NULL model (semivamiogram values set to 0 for each lag) in automatic semivamiogram fitting is not weighted as other models near the origin and sometimes it causes strange situation when NULL model is better than other models due to the large errors generated by the weighting algorithm. It should be changed (so, NULL model must be weighted OR models in semivamiogram regularization process must be compared to a different model).

"Null weighting" is removed. Error calculation is changed:

instead of the mean absolute error mean squared difference is calculated (MSE penalizes outliers more than MAE).
every model is weighted,
is_weighting bool attribute for TheoreticalSemivariance class,
instead of zeros kriging models are compared to simple regression model (least squares fitted to the experimental semivariogram).
Tests pass but tutorials must be checked before I close this issue.


doc: fix most :zephyr-app: paths

During testing I've found one bad path and looked if there is more.
I've used this oneliner to find more:
rg :zephyr-app: | awk '{ print $3 }' | while read dir; do test -d $dir || echo $dir ; done | grep '^samples' | grep -v '<' | sort | uniq

Threre are still some broken path references which not so easy to fix:
samples/boards/pine64_pinetime
samples/net/dhcp_client
samples/sensor/adxl362
samples/shields/lmp90100_evb/thermocouple
samples/video/mt9m114

Great, thanks for this!
Looking at some of the "not so easy" ones, I think you might be able to address them too, if you're interested of course.
For example samples/shields/lmp90100_evb/thermocouple --> samples/shields/lmp90100_evb/rtd
The two mt9m114 video samples also seem trivial, from a quick look
Thanks again!

Regarding the compliance check errors (https://github.com/zephyrproject-rtos/zephyr/actions/runs/5560869089/jobs/10158196453#step:9:1) the errors should be self explanatory. You can amend your commit and then force push to your branch.

Added more fixes, signed. Still left few path unfixed (updated PR/commit message).

    samples/boards/pine64_pinetime

You can point to button sample (see 92e317c) and update the sentence before ("... you can, for example, use the button sample")

    samples/sensor/adxl362

Point to accel_polling sample instead (see 5306db05afcb615bc51cde97338562818680fc5c)
Thank you!

    samples/boards/pine64_pinetime

You can point to button sample (see 92e317c) and update the sentence before ("... you can, for example, use the button sample")

    samples/sensor/adxl362

Point to accel_polling sample instead (see 5306db05afcb615bc51cde97338562818680fc5c)
Thank you!

Looks like you forgot the second one in the latest update, and incorrectly fixed the Pine64 link with it instead of having it point to the button sample? Thanks!

updated


Use creation rules on startup.

It would be nice to have the ability to select a creation rule to use on start-up to open a specified page or to create a new one, to facilitate daily note taking or for simply having a startpage. And the option to open a specific folder or query as well.
Optional: schedule to open different pages according to time or weekday.

This is an interesting idea! What to do on app start up is pretty straightforward, but what about if you just leave Tangent open forever? Would you expect this "start-up" creation rule to run then as well?

Now that is a interesting question, in the case you leave it running, it would be more akin to automating page creation.
You probably don't want the creation rule to run while the user is writing, so you would be prompted to postpone while the editor is being used.

What's the benefit of having the page be created automatically, but be blank, vs having the page created when you request it? If pages were created automatically, you might have several days where you didn't actually interact with that generated page and now you have blank notes, no?

You're right, it would be dumb to have a bunch of empty pages, probably better if the page were only to be created with the user's confirmation and only then, but that isn't much different to just scheduling on a agenda, so it is pointless.
Better to stick to on-new-session creation.


Chrome Page issue

After I put my Gmail account it keeps showing me that is incorrect.

What should I do?

Google logins are not supported. Create a password in your account settings and login using your email address.


Some problem about SDL

Hi, I meet a SDL error on ubuntu 3090 GPU (Binit -Waring:unable to set VSync! SDL Error: Taht operation is not supported) when I run the command. Would you like to provide some instruction for me? Thank you.

same problem. How to solve this? many thx


[BUG] Power BI Connector cannot read int64 Timestamp Columns

Bug
Describe the problem
Steps to reproduce

Create a Test File (here using Databricks)

spark.conf.set("spark.sql.parquet.outputTimestampType", "TIMESTAMP_MILLIS")
spark.sql("create table test_data using delta location 'abfss://MYPATH/repo/test';")

from datetime import datetime, timezone
spark.createDataFrame([{"date": datetime.now(tz=timezone.utc)}]).write.option("mergeSchema", "true").mode("append").save(MYPATH)

The resulting delta file: repo.zip

Read it using fn_ReadDeltaTable in Power BI:

let
    Source = AzureStorage.DataLake("MYPATH", [HierarchicalNavigation = false]),
    DeltaTable = fn_ReadDeltaTable(Source)
in
    DeltaTable

Observed results
Column is empty:

Expected results
Column should correctly display the data, as does databricks

Willingness to contribute
The Delta Lake Community encourages bug fix contributions. Would you or another member of your organization be willing to contribute a fix for this bug to the Delta Lake code base?

[ ] Yes. I can contribute a fix for this bug independently.
[x] Yes. I would be willing to contribute a fix for this bug with guidance from the Delta Lake community.
[ ] No. I cannot contribute a bug fix at this time.

I have to admit I don't really understand the Power Query Code of the Connector :)

FYI: Reading the parquet directly in power bi does work


Training Page for Product Owner Website

Describe the Issue
As a platform admin having a training resources for newly onboarded teams, introducing them to cloud services
Additional Context
Add any other context, attachments or screenshots
Acceptance Criteria

Create a new page in WordPress for the new Public Cloud website (work with Pilar on this)
Create the content for the new page: re-use the training announcement email content + the list of free AWS training from the StrackOverflow
Sign off by Olena

Blocked until Content Design team resolves a technical issue

Hi @abibat-adesina, could you please remind me why this ticket is blocked, are you waiting for Pilar?

Provided page to review today - confirmed - page is now live https://digital.gov.bc.ca/cloud/services/public/get-support/#training


feat: added toastify for better user feedback

Closes #29

[x] The toast notification should appear in the top right corner of the screen and disappear after a short delay.
[x] The notification should be styled to match the app's design and branding.
[x] The toast should display different messages based on the action that was taken.
[x] The feature should be implemented using a popular library such as React-Toastify.
[x] This issue will require changes to both the front-end and back-end of the application.

@sumana2001 I just resolved merge conflicts and I think now we are good to go.


[Feature][Client] Flink1.15 CDCSOURCE add PostgreSql2Starrocks

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Flink 1.15 cdcsource add PostgreSql to StarRocks

Flink 1.15 增加 cdcsource，适配 PostgreSql 同步至 StarRocks 数据库
Use case
Flink 1.15 cdcsource add PostgreSql to StarRocks

Flink 1.15 增加 cdcsource，适配 PostgreSql 同步至 StarRocks 数据库
Related issues
no
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

PR 已提交 #1933


Update tests for decode_utf8 changes

Makes decode_utf8 default in tests
Add a way to parametrize client options in tests
Test exceptions with decode_utf8 both on and off

Test failures were expected, that's a remaining bug with deluge 2, python 3, and decode_utf8 on.


Missing skia library

Observed Behavoiur
If a compiled application is distributed without skia.dll (so/dylib? Can't check) the behaviour of the end application is to silently terminate without complaint - i.e. there is no indication that the application was started in the first place - hello world without the 'Hello World'.
Desired Behaviour
Warn used skia.dll could not be found
As it is rather easy to miss packaging a dll with an executable when testing (even scarier - could hit distribution) a missing skia.dll status should be notified to the user so the developer can kick himself (or herself)...

Added easy 'how to break skia' as part of a repo for illustration
https://github.com/peardox/Compositing

Hello @!
Well, there is no warning for a Delphi limitation. Our code throws an exception with the proper message but the exceptions in the initialization section are not displayed. Docwiki itself warns of this: https://docwiki.embarcadero.com/RADStudio/Sydney/en/Exceptions_(Delphi)#Raising_and_Handling_Exceptions

Note: Raising an exception in the initialization section of a unit may not produce the intended result.

The good news is that in our next version there will be no such problem as we will have static linking on Windows, that is, there will be no dll as a dependency. It is likely that by the end of the month the launch will take place

Is there any solution for this case?

Non
Télécharger TypeApp pour Android
Le 19 avr. 2023, 14:58, à 14:58, valber-custodio @.***> a écrit:

Is there any solution for this case?
—Reply to this email directly, view
it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID:
<EMAIL_ADDRESS>

The good news is that in our next version there will be no such problem as we will have static linking on Windows, that is, there will be no dll as a dependency. It is likely that by the end of the month the launch will take place

So... ~1,5 years later... will it be at all, Vinícius? 🥲


Problem n+1 query

When i used association in my export rails console  are joins for each relation model,That`s  very bad ...

@mhi20 if your ActiveRecord::Relation has an association, you should eager load it before passing it to render to avoid N+1.

Closing for now. Re-open if it's unclear.


fix wsgi by returning bytes

Related to https://github.com/mesosphere/marathon-lb/issues/37
Thanks for looking at it!
On side note we should switch to something else like bottle but my python-fu might be limited.

Seems reasonable to me. Thanks!


Some suggestions about Brutusin json form

Hi guys,
I recently found your very great works, Json form, it is so flexible and easy to use.
It is very suitable for me, but I think if something will be enhanced(or just I don't know how to use it correctly), it could be much flexible. Many thanks to Mr. Ignacio del Valle Alles and all that contribute to the project.

More clear documents to be supported, I will offer some code as example I guess.
How to externally set a value by column on Schema Resolver module and customized error message to a certain column of form properties and user defined validation function on submition?
Ex.

bf.schemaResolver = function (names, data, cb) {
    var schemas = new Object();
    var schema = new Object();
    if (data.user_password1 != '' && data.user_password2 != '' && data.user_password1 === data.user_password2) {
    } else {
        BrutusinForms.user_password.addError("Re-enter password, please!"); // Sample
    }
	schema.title = "Confirm Password";
	schema.type = "string";
	schema.value = data.user_password1; // Failed
	schema.readOnly = true; // Failed
	schema.required = true; // Failed
	schema.value = "TEST"; // Failed
	schemas[names] = schema;
	setTimeout(function(){cb(schemas)},100); // in order to show asynchrony
};

I made some modification to your code, and I feedback to you as the attachment.
a. Seperate each enum item to {'value':'dog', 'text': 'Dog'}
b. Some multilingual issues.
c. provide zh_CN(Simplified Chinese) and zh_TW (Traditional Chinese) language files.
brutusin-json-forms-bootstrap.js.txt
brutusin-json-forms-bootstrap.js.txt
brutusin-json-forms-lang-en_US.js.txt
brutusin-json-forms-lang-zh_TW.js.txt
brutusin-json-forms-lang-zh_CN.js.txt
brutusin-json-forms.js.txt

I Franco, some questions here:
Regarding:

Please give us examples of what you mean.
What you are trying to achieve? Share also the root schema
As an attachment the modifications are very difficult to inspect and merge. The standard procedure is preferred that is: fork the project, commit and push your changes, and send us a pull request. More info here https://help.github.com/categories/collaborating-with-issues-and-pull-requests/

Thanks!


Erreur lors de l'accès avec une perte de session

https://sentry.io/guillaume-poussel/cyclo-paca/issues/438736854/
ErrorException: Trying to get property of non-object
  File "storage/framework/views/cae8e29dfa4057901fd388bbf8b51fc25a20bf28.php", line 21, in handleError
    <?php echo e(Auth::user()->name); ?>
  File "storage/framework/views/cae8e29dfa4057901fd388bbf8b51fc25a20bf28.php", line 21, in include
    <?php echo e(Auth::user()->name); ?>
  File "storage/framework/views/f93dbfd15e41a284933f9109043861589d3eae78.php", line 95, in include
    <?php echo $__env->make('layouts.dashboard', array_except(get_defined_vars(), array('__data', '__path')))->render(); ?>
  File "app/Http/Middleware/HttpsProtocol.php", line 16, in handle
    return $next($request);
  File "www/index.php", line 60
    );
...
(49 additional frame(s) were not displayed)

ErrorException: Trying to get property of non-object (View: /home/gpoussel/cyclopaca/coupe/resources/views/layouts/dashboard.blade.php)
  File "storage/framework/views/cae8e29dfa4057901fd388bbf8b51fc25a20bf28.php", line 21, in handleViewException
    <?php echo e(Auth::user()->name); ?>
  File "storage/framework/views/f93dbfd15e41a284933f9109043861589d3eae78.php", line 95, in include
    <?php echo $__env->make('layouts.dashboard', array_except(get_defined_vars(), array('__data', '__path')))->render(); ?>
  File "app/Http/Middleware/HttpsProtocol.php", line 16, in handle
    return $next($request);
  File "www/index.php", line 60
    );
...
(46 additional frame(s) were not displayed)

ErrorException: Trying to get property of non-object (View: /home/gpoussel/cyclopaca/coupe/resources/views/layouts/dashboard.blade.php) (View: /home/gpoussel/cyclopaca/coupe/resources/views/layouts/dashboard.blade.php)
  File "storage/framework/views/cae8e29dfa4057901fd388bbf8b51fc25a20bf28.php", line 21, in handleViewException
    <?php echo e(Auth::user()->name); ?>
  File "app/Http/Middleware/HttpsProtocol.php", line 16, in handle
    return $next($request);
  File "www/index.php", line 60
    );
...
(40 additional frame(s) were not displayed)

Non reproduit depuis 3 mois → clos


[BACKLOG-22369] Added support for the AWS SDK's default provider chain to the S3 CSV Input step.

@pentaho/moseisley Please review.

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'plugins/s3csvinput/core' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'plugins/s3csvinput/core' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/s3csvinput/assemblies/plugin/src/assembly/assembly.xml
  plugins/s3csvinput/core/pom.xml
  plugins/s3csvinput/core/src/main/java/org/pentaho/di/trans/steps/s3csvinput/S3CsvInputDialog.java
  plugins/s3csvinput/core/src/main/java/org/pentaho/di/trans/steps/s3csvinput/S3CsvInputMeta.java
  plugins/s3csvinput/core/src/main/resources/org/pentaho/di/trans/steps/s3csvinput/messages/messages_en_US.properties
  plugins/s3csvinput/core/src/test/java/org/pentaho/di/trans/steps/s3csvinput/S3CsvInputMetaNewInjectionTest.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes: +0.3% (click to expand)
org.pentaho.di.trans.steps.s3csvinput.S3CsvInputMeta

Complexity Change: -4% :small_red_triangle_down:
Instruction Change: -0.6% :small_red_triangle_down:
Line Change: -1% :small_red_triangle_down:
Method Change: -0.5% :small_red_triangle_down:


Service layer for rewriting request IDs

The request IDs sent back to the client in the ResponseEnvelope might not always match up with the original RequestEnvelope if there were additional requests made by the retry/auth middleware.
This typically doesn't matter since users will be using Client which doesn't expose the response envelopes, but advanced use cases might want a layer that rewrites IDs.

Nevermind, can't really think of a practical use for this at the moment. Could be reopened later if necessary.

I forgot that the input request envelopes actually don't have a request ID at the moment.
In case the request ID is specified, should be resolved by #36


Lost K8S autoscaled nodes contacted indefinitely

What you expected to happen?
When a Kubernetes node is removed from our autoscaled cluster, Weave continues trying to contact it until an operator manually rmpeer & forget's the node via the command line.
What happened?
weave-net-mlqj4 weave INFO: 2018/05/14 14:30:16.028571 ->[<IP_ADDRESS>:6783] attempting connection
weave-net-p5mjh weave INFO: 2018/05/14 14:30:17.357611 Discovered remote MAC 5a:b9:a1:0c:57:a6 at 1e:bc:cb:84:0d:2a(ip-10-0-3-37.us-east-2.compute.internal)
weave-net-mlqj4 weave INFO: 2018/05/14 14:30:19.091097 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: connect: no route to host
weave-net-2bgzv weave INFO: 2018/05/14 14:30:25.250138 ->[<IP_ADDRESS>:6783] attempting connection

... about once a minute
How to reproduce it?
Remove a Kubernetes node (terminate the instance)
Anything else we need to know?
Kubernetes 1.9.4, kubeadm cluster on AWS.
Versions:

$ weave version
weave 2.3.0
$ docker version
Client:
 Version:      17.05.0-ce
 API version:  1.29
 Go version:   go1.7.5
 Git commit:   89658be
 Built:        Thu May  4 22:10:54 2017
 OS/Arch:      linux/amd64
$ uname -a
Linux ip-10-0-3-160.us-east-2.compute.internal 4.8.0-59-generic #64-Ubuntu SMP Thu Jun 29 19:38:34 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.5", GitCommit:"f01a2bf98249a4db383560443a59bed0c13575df", GitTreeState:"clean", BuildDate:"2018-03-19T15:59:24Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.4", GitCommit:"bee2d1505c4fe820744d26d41ecd3fdd4a3d6546", GitTreeState:"clean", BuildDate:"2018-03-12T16:21:35Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}

Logs:
Grepped out a relevant host:
NFO: 2018/05/11 00:40:34.163026 Launch detected - using supplied peer list: [<IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS>]
INFO: 2018/05/11 00:40:34.175099 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2018/05/11 00:40:34.176893 ->[<IP_ADDRESS>:6783|5e:31:34:2d:2e:e9(ip-10-0-1-64.us-east-2.compute.internal)]: connection ready; using protocol version 2
INFO: 2018/05/11 00:40:34.177060 overlay_switch ->[5e:31:34:2d:2e:e9(ip-10-0-1-64.us-east-2.compute.internal)] using fastdp
INFO: 2018/05/11 00:40:34.179623 ->[<IP_ADDRESS>:6783|5e:31:34:2d:2e:e9(ip-10-0-1-64.us-east-2.compute.internal)]: connection added (new peer)
INFO: 2018/05/11 00:40:34.200111 ->[<IP_ADDRESS>:6783|5e:31:34:2d:2e:e9(ip-10-0-1-64.us-east-2.compute.internal)]: connection fully established
INFO: 2018/05/11 00:40:34.206113 sleeve ->[<IP_ADDRESS>:6783|5e:31:34:2d:2e:e9(ip-10-0-1-64.us-east-2.compute.internal)]: Effective MTU verified at 8939
INFO: 2018/05/11 00:40:35.102470 [kube-peers] Added myself to peer list &{[{e6:4b:b7:4c:df:d0 ip-10-0-1-128.us-east-2.compute.internal} {c2:18:9c:74:e7:f2 ip-10-0-3-64.us-east-2.compute.internal} {aa:84:b9:7a:13:5a ip-10-0-3-114.us-east-2.compute.internal} {86:41:ea:22:67:e5 ip-10-0-3-73.us-east-2.compute.internal} {e6:d6:b7:a2:bf:26 ip-10-0-1-41.us-east-2.compute.internal} {86:f7:66:2b:87:19 ip-10-0-1-200.us-east-2.compute.internal} {7a:e5:64:52:cd:43 ip-10-0-3-93.us-east-2.compute.internal} {1e:bc:cb:84:0d:2a ip-10-0-3-37.us-east-2.compute.internal} {22:02:44:cb:1a:e5 ip-10-0-1-109.us-east-2.compute.internal} {5e:31:34:2d:2e:e9 ip-10-0-1-64.us-east-2.compute.internal} {8a:50:e4:5b:fc:b3 ip-10-0-1-43.us-east-2.compute.internal}]}
INFO: 2018/05/11 00:40:35.968265 Discovered remote MAC 5e:31:34:2d:2e:e9 at 5e:31:34:2d:2e:e9(ip-10-0-1-64.us-east-2.compute.internal)
INFO: 2018/05/11 00:40:36.070057 Discovered remote MAC e2:05:43:3e:09:b9 at 5e:31:34:2d:2e:e9(ip-10-0-1-64.us-east-2.compute.internal)
INFO: 2018/05/11 00:40:36.070427 Discovered remote MAC b2:1b:27:4c:a4:52 at 5e:31:34:2d:2e:e9(ip-10-0-1-64.us-east-2.compute.internal)
INFO: 2018/05/11 00:40:36.215259 Discovered remote MAC 3a:3c:9f:e7:7d:08 at 5e:31:34:2d:2e:e9(ip-10-0-1-64.us-east-2.compute.internal)
INFO: 2018/05/11 00:42:00.633047 ->[<IP_ADDRESS>:6783|5e:31:34:2d:2e:e9(ip-10-0-1-64.us-east-2.compute.internal)]: connection shutting down due to error: read tcp4 <IP_ADDRESS>:50037-><IP_ADDRESS>:6783: read: connection reset by peer
INFO: 2018/05/11 00:42:00.633138 ->[<IP_ADDRESS>:6783|5e:31:34:2d:2e:e9(ip-10-0-1-64.us-east-2.compute.internal)]: connection deleted
INFO: 2018/05/11 00:42:00.640213 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2018/05/11 00:42:00.640585 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: connect: connection refused
INFO: 2018/05/11 00:42:00.646541 Removed unreachable peer 5e:31:34:2d:2e:e9(ip-10-0-1-64.us-east-2.compute.internal)
INFO: 2018/05/11 00:42:03.457897 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2018/05/11 00:42:03.458262 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: connect: connection refused
INFO: 2018/05/11 00:42:05.602050 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2018/05/11 00:42:05.602441 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: connect: connection refused
INFO: 2018/05/11 00:42:11.717182 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2018/05/11 00:42:11.717543 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: connect: connection refused
INFO: 2018/05/11 00:42:21.712218 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2018/05/11 00:42:40.031406 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: connect: no route to host
INFO: 2018/05/11 00:42:53.518432 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2018/05/11 00:42:56.575443 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: connect: no route to host
INFO: 2018/05/11 00:43:12.400009 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2018/05/11 00:43:15.455412 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: connect: no route to host
INFO: 2018/05/11 00:43:41.529976 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2018/05/11 00:43:44.607391 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: connect: no route to host
INFO: 2018/05/11 00:44:32.622034 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2018/05/11 00:44:35.679413 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: connect: no route to host
INFO: 2018/05/11 00:45:27.154253 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2018/05/11 00:45:30.207380 ->[<IP_ADDRESS>:6783] error during connection attempt: dial tcp4 :0-><IP_ADDRESS>:6783: connect: no route to host
INFO: 2018/05/11 00:46:46.325098 ->[<IP_ADDRESS>:6783] attempting connection

.. and so on for days.
-->
See also issue #2797

This is the same as #3021; see particularly https://github.com/weaveworks/weave/issues/3021#issuecomment-349001695

Read that issue.  How often is "eventually restart"?  It seems to be never?

I was thinking things like kernel upgrades, power-downs, hardware failures would lead to restarts of the Weave Net pods.

Okay, so its expected that we'll have these errors.  It created some concern for us as our nodes cycle with some frequency based on spot price bidding, so many of the weave pods are logging numerous old node contact attempts constantly.  It feels like it should give up after some, long (configurable?) time.

I'm not saying the behaviour is correct, merely noting my comments last time it was raised.
Happy to have an issue open, and/or to receive PRs improving it.

Understood.  My apologies, I thought this was as intended.

@bboreham  in your comment https://github.com/weaveworks/weave/issues/3300#issuecomment-388865354 you talk about "having an issue open".
Isn't this issue, https://github.com/weaveworks/weave/issues/3300, sufficiently reported on to narrow down the cause and fix / improve this?

@frittentheke I believe so.
The cause is that there is no code to re-fetch the peer list from Kubernetes after startup.

Hi, we have the same issues,  weaveworks/weave-kube:2.3.0  also.
Is there a fix?   would 2.4.0 upgrade solve this?

No fix yet.

@bboreham What would be needed for a fix? Is there a PR or something I can contribute on? I had this issue in a cluster today where we scale the nodes up and down a lot and I had to manually remove the local .db file and restart all the nodes to clean them up. I'd love to work on automating a solution, either by integrating it in weave or in an additional pod/container.

Something needs to wake up periodically, re-fetch the peer list from Kubernetes and supply it to the Weave Net router process.
It's a little fiddly since we have two binaries, one linked with Kubernetes client-go and one not.
Here is the code that fetches the peer list from Kubernetes:
https://github.com/weaveworks/weave/blob/master/prog/kube-utils/main.go#L28
Here is code that handles a REST call to replace the peer list inside the router:
https://github.com/weaveworks/weave/blob/master/router/http.go#L16
and then write a test.

The problem here would really be that we don't want to drop immediately nodes that are gone as it could just be a temporary issue. We probably need a "timeout" on which we consider the nodes dead.
I'm actually quite surprised that this is not handled automatically and doesn't seem like a minor issue to me, so it's probably worth discussing further and a bit more in detail, what do you think?

It's not a problem to delete a node from one peer's list of targets; if the node comes back it can add itself into the cluster and everyone will talk.
Then again, why are you deleting nodes from Kubernetes on a temporary basis?

Ok, so yo
We scale down for autoscaling purpose: on a testing cluster, we want to scale to 0 the autoscaling group on AWS to make sure we don't waste money.

The total impact that I am aware of is some failed connection attempts and associated log noise, so I consider it a minor issue.  If you have evidence of some real consequence please open a new issue with the details.

The cluster today had major issues, I will open a different one so that we can have a proper discussion.

we want to scale to 0 the autoscaling group on AWS to make sure we don't waste money

Those nodes are gone for good, not temporarily.

Yes they were gone for good. I'll open a dedicated issue with the full description, it is probably better.

I feel like I should delete the five previous comments here since they'll just confuse future readers.

It's okay for me if you want to delete them, go on.

To add some context: Our Kubernetes clusters consist of a few dozen nodes
allocated to a specific EC2 instance type, but we also run a controller
which bids on spot instances to save cost.  These instances semi-frequently
get removed as the spot prices fluctuate, so the node comes down and
another is spun up.  Over a few weeks we wind up with quite a lot of 'lost'
instances.  All part of normal operation.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/weaveworks/weave/issues/3300#issuecomment-415479571,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABLTrmHezeOvs-Ww0L6GDQ8OqZV3mSEdks5uTtecgaJpZM4T96wf
.

Thanks for adding a bit more context. Did you ever experience issues like the one reported in #3384 ?

No, I haven't had that particular problem.
On Fri, Aug 24, 2018 at 11:53 AM Raffaele Di Fazio<EMAIL_ADDRESS>wrote:

Thanks for adding a bit more context. Did you ever experience issues like
the one reported in #3384
https://github.com/weaveworks/weave/issues/3384 ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/weaveworks/weave/issues/3300#issuecomment-415817915,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABLTroIPxj0Reqk1--9O5lygtWka1p76ks5uUC-ZgaJpZM4T96wf
.

By the way the issue seems like an easy to fix, if it's just a log noise. Those reconnect attempts need to have a global retention so the nodes which are down more than a week at least will be cleaned up. IMHO

AFAIK, there is no recommended automated fix and it's actually good info that you run Weave 2.4.0. I'd say to continue on #3384 which I opened as suggested by @bboreham

k8s 1.11.3 ,  AWS,  weave  2.4.1
still the same :(

Do the logs show more information with the new version? Can you post them?
I'd be trying this soon myself, but any info would be great.
On Fri, Sep 28, 2018, 10:43 Dmitry<EMAIL_ADDRESS>wrote:

k8s 1.11.3 , AWS, weave 2.4.1
still the same :(
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/weaveworks/weave/issues/3300#issuecomment-425366372,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AApv1KbY1t0h73--hZV3hzvsyIgVvJEGks5ufeFBgaJpZM4T96wf
.

quick update on this issue please see
https://github.com/weaveworks/weave/issues/3401#issuecomment-421293844 and https://github.com/weaveworks/weave/pull/3399
I am planning to add forget once #3399 is merged. Basically watch for k8s node delete events and perform necessary steps on receiving a node delete event.

Do the logs show more information with the new version? Can you post them? I'd be trying this soon myself, but any info would be great.
…
On Fri, Sep 28, 2018, 10:43 Dmitry @.***> wrote: k8s 1.11.3 , AWS, weave 2.4.1 still the same :( — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3300 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AApv1KbY1t0h73--hZV3hzvsyIgVvJEGks5ufeFBgaJpZM4T96wf .

no it is exactly all the same as before... I was just hoping in was fixed over time...
@murali-reddy that sounds great 👍

@murali-reddy Sorry to bother and for the silly question, but does this PR mean that the issue reported as fixed in 2.4.1 (https://github.com/weaveworks/weave/releases/tag/v2.4.1 ) still aren't fixed and there could be cases where reusing IPs can lead to the same problems reported for example in https://github.com/weaveworks/weave/issues/3384 ?

@Raffo the issues we reported as fixed are believed to be fixed.
"this PR" (which? Murali named two) is fixing other things.
No problem was ever identified in reusing IPs.  Re-using node_names delayed reclaim of IP ranges, but that was fixed in 2.4.1.

Sorry I meant "this issue", not PR. Thanks for the clarification.

Whats the status on this issue?
I have weavenet components in one of my clusters reporting 7 hosts unreachable. I looked into it on my end and it appears that all of these IPs that weavenet is reporting as unreachable were deleted from the cluster autoscaler days ago.
I'm running on aws, kops and k8s 1.10.3. We are also using a number of spot instances on this cluster so there's a chance that we're seeing more churn than average with these node groups.
Manually running a command like weave rmpeer <peer_id> doesn't seem like viable long term workaround, is there any other way to avoid this or an alternative remedy?

Hi @m0rganic , recently this was fixed, as you can see in another issue I had open that was relevant (I think) to these old nodes cleanup tasks:  https://github.com/weaveworks/weave/issues/3427#issuecomment-435193838


Thank you!!

Just found this plugin - it's amazingly useful. Thanks for making it.
Just FYI (and maybe to include in the "working configs" dataset), my setup is:
iPad pro using Blink -> mosh to remote host -> tmux -> neovim master
My relevant config looks like this:
local autocmds = {
	textyank = {
		{'TextYankPost', '*', 'silent! lua vim.highlight.on_yank({timeout=250})'},
		{'TextYankPost', '*', 'if v:event.operator is "y" && v:event.regname is "" | OSCYankReg " | endif'}
	};
}

Thanks again. This plugin made my day.

Thank you I'm glad you appreciate the plugin :) I'll make sure to point people to this issue if they share a setup similar to yours.


[BUG Bash] delete_blobs doesn't work with azurite

Describe the bug
When use delete_blobs, azurite refused with status code 400
Exception or Stack Trace
python exception:
  File "FileOperator_spec.py", line 172, in empty_data_folders_does_not_raise
    self.data_container_client.delete_blobs(*blobs)
  File "venv/lib/python3.9/site-packages/azure/core/tracing/decorator.py", line 83, in wrapper_use_tracer
    return func(*args, **kwargs)
  File "venv/lib/python3.9/site-packages/azure/storage/blob/_container_client.py", line 1302, in delete_blobs
    return self._batch_send(*reqs, **options)
  File "venv/lib/python3.9/site-packages/azure/storage/blob/_shared/base_client.py", line 322, in _batch_send
    process_storage_error(error)
  File "venv/lib/python3.9/site-packages/azure/storage/blob/_shared/response_handlers.py", line 110, in process_storage_error
    _LOGGER.warning(
Message: 'Unexpected return type % from ContentDecodePolicy.deserialize_from_http_generics.'
Arguments: (<class 'NoneType'>,)

azurite debug error:
2021-11-24T04:11:02.906Z bdacffce-65a7-410d-b728-cb53fdc240fc info: BlobStorageContextMiddleware: RequestMethod=POST RequestURL=http://<IP_ADDRESS>/devstoreaccount1/tmp-data?restype=container&comp=batch RequestHeaders:{"host":"<IP_ADDRESS>:10000","user-agent":"azsdk-python-storage-blob/12.9.0 Python/3.9.7 (macOS-11.6-x86_64-i386-64bit)","accept-encoding":"gzip, deflate","accept":"*/*","connection":"keep-alive","x-ms-version":"2020-10-02","content-type":"multipart/mixed; boundary=batch_89ba4f78-4cdc-11ec-aad1-acde48001122","content-length":"4024","x-ms-date":"Wed, 24 Nov 2021 04:11:02 GMT","x-ms-client-request-id":"89bc7e7e-4cdc-11ec-aad1-acde48001122","authorization":"SharedKey devstoreaccount1:W+FzGbp15qwKN5hKiIXXQ/z13ILlF7bBK4XKPRaakj8="} ClientIP=<IP_ADDRESS> Protocol=http HTTPVersion=1.1
2021-11-24T04:11:02.906Z bdacffce-65a7-410d-b728-cb53fdc240fc info: BlobStorageContextMiddleware: Account=devstoreaccount1 Container=tmp-data Blob=
2021-11-24T04:11:02.906Z bdacffce-65a7-410d-b728-cb53fdc240fc verbose: DispatchMiddleware: Dispatching request...
2021-11-24T04:11:02.906Z bdacffce-65a7-410d-b728-cb53fdc240fc error: DispatchMiddleware: Incoming URL doesn't match any of swagger defined request patterns.
2021-11-24T04:11:02.906Z bdacffce-65a7-410d-b728-cb53fdc240fc error: ErrorMiddleware: Received a MiddlewareError, fill error information to HTTP response
2021-11-24T04:11:02.906Z bdacffce-65a7-410d-b728-cb53fdc240fc error: ErrorMiddleware: ErrorName=UnsupportedRequestError ErrorMessage=Incoming URL doesn't match any of swagger defined request patterns.  ErrorHTTPStatusCode=400 ErrorHTTPStatusMessage=undefined ErrorHTTPHeaders=undefined ErrorHTTPBody=undefined ErrorStack="UnsupportedRequestError: Incoming URL doesn't match any of swagger defined request patterns.\n    at Object.dispatchMiddleware [as default] (/usr/local/lib/node_modules/azurite/dist/src/blob/generated/middleware/dispatch.middleware.js:41:30)\n    at /usr/local/lib/node_modules/azurite/dist/src/blob/generated/ExpressMiddlewareFactory.js:49:42\n    at Layer.handle [as handle_request] (/usr/local/lib/node_modules/azurite/node_modules/express/lib/router/layer.js:95:5)\n    at trim_prefix (/usr/local/lib/node_modules/azurite/node_modules/express/lib/router/index.js:317:13)\n    at /usr/local/lib/node_modules/azurite/node_modules/express/lib/router/index.js:284:7\n    at Function.process_params (/usr/local/lib/node_modules/azurite/node_modules/express/lib/router/index.js:335:12)\n    at next (/usr/local/lib/node_modules/azurite/node_modules/express/lib/router/index.js:275:10)\n    at blobStorageContextMiddleware (/usr/local/lib/node_modules/azurite/dist/src/blob/middlewares/blobStorageContext.middleware.js:73:5)\n    at /usr/local/lib/node_modules/azurite/dist/src/blob/middlewares/blobStorageContext.middleware.js:15:16\n    at Layer.handle [as handle_request] (/usr/local/lib/node_modules/azurite/node_modules/express/lib/router/layer.js:95:5)"
2021-11-24T04:11:02.906Z bdacffce-65a7-410d-b728-cb53fdc240fc error: ErrorMiddleware: Set HTTP code: 400
2021-11-24T04:11:02.906Z bdacffce-65a7-410d-b728-cb53fdc240fc error: ErrorMiddleware: Set HTTP body: undefined
2021-11-24T04:11:02.906Z bdacffce-65a7-410d-b728-cb53fdc240fc info: EndMiddleware: End response. TotalTimeInMS=0 StatusCode=400 StatusMessage=undefined Headers={"server":"Azurite-Blob/3.14.3"}

Code Snippet
            blobs = self.data_container_client.list_blobs(name_starts_with="")
            self.data_container_client.delete_blobs(*blobs)

Setup (please complete the following information):

Python Version: 3.9
SDK Version: azure-storage-blob==12.9.0
azurite: 3.14.3

@blueww / @tasherif-msft can you folks please look into what happened here?

@morland96
Per the request Uri, this is a Blob Batch request.
RequestURL=http://<IP_ADDRESS>/devstoreaccount1/tmp-data?restype=container&comp=batch
Azurite still not support Blob Batch.
We already have an issue to track it: https://github.com/Azure/Azurite/issues/287

Hi @morland96 please track on Azurite as per @blueww's rec. I will proceed to close this issue.

This occurs with azure.storage.filedatalake,DataLakeFileClient alone.

python 3.7
azure-core 1.21.1
azure-identity 1.7.1
azure-storage-file-datalake 12.5.0

with DataLakeFileClient(
  account_url="https://myadlg2.core.windows.net"
  file_system_name="someblobcontainer",
  file_path="somefolder/somefile.txt",
  credential=DefaultAzureCredential()
) as client:
  result = client.delete_file()

  File "/mnt/workhdd/projects/python/py-ent-adlgen2/venv/lib/python3.7/site-packages/azure/storage/filedatalake/_path_client.py", line 261, in _delete
    process_storage_error(error)
  File "/mnt/workhdd/projects/python/py-ent-adlgen2/venv/lib/python3.7/site-packages/azure/storage/filedatalake/_deserialize.py", line 139, in process_storage_error
    'Unexpected return type % from ContentDecodePolicy.deserialize_from_http_generics.', type(error_body))
Message: 'Unexpected return type % from ContentDecodePolicy.deserialize_from_http_generics.'
Arguments: (<class 'NoneType'>,)

This occurs with azure.storage.filedatalake,DataLakeFileClient alone.

python 3.7
azure-core 1.21.1
azure-identity 1.7.1
azure-storage-file-datalake 12.5.0

with DataLakeFileClient(
  account_url="https://myadlg2.core.windows.net"
  file_system_name="someblobcontainer",
  file_path="somefolder/somefile.txt",
  credential=DefaultAzureCredential()
) as client:
  result = client.delete_file()

throws:
  File "/mnt/myproject/venv/lib/python3.7/site-packages/azure/storage/filedatalake/_path_client.py", line 261, in _delete
    process_storage_error(error)
  File "/mnt/myproject/venv/lib/python3.7/site-packages/azure/storage/filedatalake/_deserialize.py", line 139, in process_storage_error
    'Unexpected return type % from ContentDecodePolicy.deserialize_from_http_generics.', type(error_body))
Message: 'Unexpected return type % from ContentDecodePolicy.deserialize_from_http_generics.'
Arguments: (<class 'NoneType'>,)


[Forge] Build info are not produced when --no-cache is used

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (ac80261 2024-02-24T00:17:06.154246094Z)
What command(s) is the bug in?
forge build --build-info --no-cache
Operating System
Linux
Describe the bug
Hello,
I tried to extract AST of a project using forge build info but I get an issue when I try to recompile a project. I need to pass --no-cache argument to the forge build command to make sure I recompile every file and get coherent types IDs in the AST.
The problem is, when the cache is forcefully disabled, the build info are never generated (even if there is no previous compilation).
This is likely due to this line (https://github.com/foundry-rs/foundry/blob/master/crates/config/src/lib.rs#L673).
I am sure there is a good reason for this but can an analysis be made to evaluate the refactor needed to remove the cache condition? I would be happy to implement the changes needed.
In the meantime, I removed the cache condition for the build info and it seems to work fine (at least on the forge init project)

Also ran into this issue

Hello, is thee any update or information needed to perform the analysis ?

Closed by #7347


Kernel specs don't appear until I reload the window

Testing #11755

Run ipython kernel install --name "venv1-kernel3" --user
No new entry in the kernel picker until I reload the window

No new entry in the kernel picker until I reload the window

Yes, this is a known issue, we don't monitor the these directories for changes nor do we query them from time to time.

No new entry in the kernel picker until I reload the window

Yes, this is a known issue, we don't monitor the these directories for changes nor do we query them from time to time.

Maybe I meet the same question, but I can't be quite for sure. Yesterday I update my R and change the path of updated R, but I find that I won't find my updated Irkernel until I open the R ternimal, and the old Irkernel  is not a valid path but it is still in the select kernel list.
If this is not a bug, could you give me some help?

the old Irkernel is not a valid path but it is still in the select kernel list.
If this is not a bug, could you give me some help?
@kebuAAA
please could you file a separate issue

OK，I feel so sorry for bothering you

OK，I feel so sorry for bothering you

please don't apologize, i think you've misunderstood my comment, i do want to help you and an study you're running into this issue.
I'd like to keep your issue separate, else it gets difficult when we have multiple different issues in the same place.
please do file an issue and I'll be glad to help you get  unblocked

Closing for now, now one else has reported this.


Added Doc.isA for Header, Main, and Footer

Closing this to test the github behavior of the missing commit.


Support Instrumentation Scope on all signals

We currently map instrumentationScope for traces as service.framework.{name|version}(https://github.com/elastic/apm-data/blob/cee5ac3fc5f2ee66f156133992882cae758e2b66/input/otlp/traces_test.go#L567-L578) but not for signals and logs.
This was reported here: https://discuss.elastic.co/t/logs-dotnet-logger-category-where/363304

Similar issue over here I think https://github.com/elastic/sdh-apm/issues/1443


Validation error in TestAllocationAliasing on AMD Ryzen™ 9 7950X

Hello
When running VulkanSample application's tests with my AMD 7950X (AMD Radeon(TM) Graphics), I get a validation error caused by TestAllocationAliasing(). Running the tests with my Intel Arc A770 works without any problems.
When I continue the tests, there is another issue with ProcessDefragmentationPass, but I will create a separate issue for this.
Vulkan Memory Allocator Sample 3.0.1
Vulkan API version used: 1.3
physicalDeviceProperties:
    driverVersion: 0x8000FA
    vendorID: 0x1002 (AMD)
    deviceID: 0x164E
    deviceType: 1 (INTEGRATED_GPU)
    deviceName: AMD Radeon(TM) Graphics
    limits:
        maxMemoryAllocationCount: 4096
        bufferImageGranularity: 1 B
        nonCoherentAtomSize: 128 B
physicalDeviceVulkan11Properties:
    maxMemoryAllocationSize:<PHONE_NUMBER> B (2.00 GB)
physicalDeviceVulkan12Properties:
    driverID: 1 (AMD_PROPRIETARY)
    driverName: AMD proprietary driver
    driverInfo: 22.40.40 (AMD proprietary shader compiler)

MEMORY HEAPS:
heapCount=3, typeCount=8
Heap 0: 268435456 B (256.00 MB) DEVICE_LOCAL, MULTI_INSTANCE
    Type 0: DEVICE_LOCAL
    Type 4: DEVICE_LOCAL, DEVICE_COHERENT (AMD), DEVICE_UNCACHED (AMD)
Heap 1:<PHONE_NUMBER>6 B (15.31 GB)
    Type 1: HOST_VISIBLE, HOST_COHERENT
    Type 3: HOST_VISIBLE, HOST_COHERENT, HOST_CACHED
    Type 5: HOST_VISIBLE, HOST_COHERENT, DEVICE_COHERENT (AMD), DEVICE_UNCACHED (AMD)
    Type 7: HOST_VISIBLE, HOST_COHERENT, HOST_CACHED, DEVICE_COHERENT (AMD), DEVICE_UNCACHED (AMD)
Heap 2: 268435456 B (256.00 MB) DEVICE_LOCAL, MULTI_INSTANCE
    Type 2: DEVICE_LOCAL, HOST_VISIBLE, HOST_COHERENT
    Type 6: DEVICE_LOCAL, HOST_VISIBLE, HOST_COHERENT, DEVICE_COHERENT (AMD), DEVICE_UNCACHED (AMD)

Conclusions:
- 2 heaps are DEVICE_LOCAL, total 512.00 MB.
- 2 heaps are HOST_VISIBLE, total 15.56 GB.
- 1 heaps are DEVICE_LOCAL and HOST_VISIBLE, total 256.00 MB.
- No types are HOST_VISIBLE but not HOST_COHERENT.
- No types are not DEVICE_LOCAL and not HOST_VISIBLE.
- 4 types are AMD-specific DEVICE_COHERENT or DEVICE_UNCACHED.
- No types are LAZILY_ALLOCATED.
- A buffer with VERTEX_BUFFER usage can be created in some non-DEVICE_LOCAL type.
- An image with OPTIMAL tiling and SAMPLED usage can be created in some non-DEVICE_LOCAL type.

Testing budget...
Testing aliasing...
  size: max(1441792, 8896880) = 8896880
  alignment: max(65536, 65536) = 65536
  memoryTypeBits: 255 & 255 = 255
Testing allocation aliasing...
VUID-vkAllocateMemory-pAllocateInfo-01713 ║ Validation Error: [ VUID-vkAllocateMemory-pAllocateInfo-01713 ] Object 0: handle =
0x1cbd17a8e20, type = VK_OBJECT_TYPE_DEVICE; | MessageID = 0xe9a2b96f | vkAllocateMemory: attempting to allocate 314572800
bytes from heap 0,but size of that heap is only 268435456 bytes. The Vulkan spec states: pAllocateInfo->allocationSize must be less than
or equal to VkPhysicalDeviceMemoryProperties::memoryHeaps[memindex].size where memindex =
VkPhysicalDeviceMemoryProperties::memoryTypes[pAllocateInfo->memoryTypeIndex].heapIndex as returned by
vkGetPhysicalDeviceMemoryProperties for the VkPhysicalDevice that device was created from (https://vulkan.lunarg.com/doc/view
/<IP_ADDRESS>/windows/1.3-extensions/vkspec.html#VUID-vkAllocateMemory-pAllocateInfo-01713)

I will look into this and try to find a solution for the problem.


posix: fix pthread thread specific data cleanup

The pthread_key_data allocated in pthread_setspecific() is never deallocated. Due to that pthread_setspecific() is returning ENOMEM after using it specific amount of times, depending on size of k_malloc `s underlying pool.

Thanks for the review @cfriedt, we've added the test you suggested, could you take another look?

@ycsin could you also take a look?


Menu and input box highlighting

See https://github.com/ArthurSonzogni/FTXUI/issues/928
After adding a new chat and then going back to input mode, normal mode does not work correctly.

Solved, see attached link. And below:
It looks like the main_container still contains the previous tab_selection.
On the other side the main_renderer is renderering the new one.
I am sorry updating ftxui::Component isn't as easy/functional as updating ftxui::Element. That's a design flow that is now difficult to remove.
In your case I wouldn't execute:
tab_selection = ...

because this would create a new instance.
I would do:
tab_selection->DetachAllChildren();
for (size_t i = 0; i < tab_entries.size(); ++i) {
    tab_selection->Add(MenuEntry(tab_entries[i]) | ftxui::size(ftxui::WIDTH, ftxui::EQUAL, 25));
}

This has now been remedied.


support sctl explain

Changes proposed in this pull request:

support sctl explain command,facilitate the users to locate problem

Accept.

Pull Request Test Coverage Report for Build 6871

0 of 29 (0.0%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 71.623%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-proxy/sharding-proxy-backend/src/main/java/org/apache/shardingsphere/shardingproxy/backend/text/sctl/ShardingCTLBackendHandlerFactory.java
0
2
0.0%

sharding-proxy/sharding-proxy-backend/src/main/java/org/apache/shardingsphere/shardingproxy/backend/text/sctl/explain/ShardingCTLExplainParser.java
0
9
0.0%

sharding-proxy/sharding-proxy-backend/src/main/java/org/apache/shardingsphere/shardingproxy/backend/text/sctl/explain/ShardingCTLExplainBackendHandler.java
0
18
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-proxy/sharding-proxy-backend/src/main/java/org/apache/shardingsphere/shardingproxy/backend/text/sctl/ShardingCTLBackendHandlerFactory.java
1
28.57%

Totals

Change from base Build 6868:
-0.1%

Covered Lines:
10805

Relevant Lines:
15086

💛  - Coveralls


Useless files in root directory

in tofu's root directory, these files can probably be deleted / moved:

Debug_JINTRACMesh.pdf
Debug_JINTRACMesh.svg
XICS_mask.npz
TODO_EndOfVisit_20181212.txt

Yes for all, good point
can you do it on devel ?

Rectification, I'm doing it now


Update compileSdk & targetSdkVersion 34 to 35

Description
Currently, the project's targetSdkVersion is set to 34. It is necessary to update it to 35 to remain compatible with the latest versions of Android and the new Play Store requirements. This update will also bring the latest optimizations and security features.
It is important to thoroughly test the application after this update to detect possible regressions and to ensure that all features are working properly.

https://developer.android.com/about/versions/15/get?hl=fr
https://developer.android.com/about/versions/15/setup-sdk?hl=fr

Changes should be made in the file: build.gradle.kts (Module:app)

https://github.com/Drumber/Kitsune/blob/master/app/build.gradle.kts

namespace = "io.github.drumber.kitsune"
    compileSdk = 34
    buildToolsVersion = "34.0.0"

    defaultConfig {
        applicationId = "io.github.drumber.kitsune"
        minSdk = 26
        targetSdk = 34
        versionCode = 33
        versionName = "2.0.1"

        testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"

        buildConfigField("boolean", "SCREENSHOT_MODE_ENABLED", screenshotMode)
    }

Acknowledgements

[X] I have searched the open and closed issues and this is NOT a duplicate.

The latest release (v2.0.2) now targets SDK 35.


Window Position is offset on multiple monitors

The window position on multiple monitors on OS X is offset by the arrangement bounds offset in the y direction when displaying on a secondary monitor.
To see this go to System Preferences > Displays > Arrangement and set offset your primary display like the following:

Then open your menubar application from the secondary display (will be offset down by the amount of offset of the secondary display.

This is a regression from https://github.com/maxogden/menubar/pull/61. See https://github.com/maxogden/menubar/commit/924236e60b921eb41487146946fc6cdfcdfc6dde#diff-168726dbe96b3ce427e7fedce31bb0bcL86
A bug should [also] be opened with Electron.

Ah sorry for this.
Instead of #64 we could fix this in electron-positioner currently y is set to 0 on all tray positions.
https://github.com/jenslind/electron-positioner/blob/master/src/Positioner.js#L35
This issue should be fixed by just using screenSize.y instead of 0.

+1 fixing this one. Now only the main monitor shows the pops up windows

This should have been fixed now. Just make sure you are using electron-positioner 2.0.1.
@jingweno

Now only the main monitor shows the pops up windows
What version of electron are you using?

@jenslind:

What version of electron are you using?

electrcon: 0.31.2, menubar: 2.2.1, electron-positioner: 2.0.0

Try use electron 0.30.2 or later, that should solve the issue.

With this fix, I am able to get the window on both monitors. I agree that the electron-positioner fix seems to be the best option as that's effectively what I did but my version was much more hacky ...

Updated #64 to use 2.0.1 of electron-positioner instead of my original fix. Tested as working in my OS X environment with the stated configuration.

@paullryan
No need to bump the version: https://nodesource.com/blog/semver-tilde-and-caret

... the caret version, ^1.2.3 permits versions from 1.2.3 all the way up to, but not including, the next major version, 2.0.0.

@jenslind I understand that it allows all the version in between but I would consider this a critical patch, thus a bump. However it's your projects so if you want to reject the pull just let me know and I'll delete it.

Sorry to bring this up again, but i'm still having this issue. This looks like it was resolved a while back but when i open my menu bar app on a second monitor on OSX 10.11.6 the app shows at the top of the height of the primary monitor, as per the first comment in this thread.
electron: 1.4.0, menubar: 5.1.0 (electron-positioner: 3.0.0)

Ah it turns out i no longer need to provide a y option to drop the menu slightly down from the menubar. Removing my y option solved this for me.


Add examples dir in release tar

To fix https://github.com/istio/istio.io/issues/7854
With istio-1.7.0-beta.1 manifests/examples directory is missing
[u@master-node istio-1.7.0-beta.1]$ ls manifests/
charts  deploy  profiles

The manifests/examples directory consists of the following. Which is used with VM examples.
[u@master-node istio]$ ls manifests/examples/
customresource  multicluster  user-gateway  vm

@shamsher31: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
c7e7e0fcf4a250f9869c87f4acff2cd49d518d36
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test unit-tests_istio

/test unit-tests_istio


fix for integer Info columns not in latest release on pip

I followed the instructions in the README and installed with
pip install igv-reports

However this gave me version 1.0.1. It seems like a number of important fixes are not included, importantly the ones in bbdf1e8
For example, the version of igv-reports I got off of pip has the following for varianttable.py;
def render_value(v):
    """Render given value to string."""
    if isinstance(v, float):
        # ensure that we don't waste space by insignificant digits
        return f'{v:.2g}'
    elif v.startswith('http://') or v.startswith('https://'):
        return create_link(v)
    else:
        return str(v)

but the one I cloned off GitHub here has this, which fixes some bugs I was hitting;
def render_value(v):
    """Render given value to string."""
    if v is None:
        return ""
    elif isinstance(v, float):
        # ensure that we don't waste space by insignificant digits
        return f'{v:.2g}'
    elif isinstance(v, str):
        str_val = v.replace('"', '')

        if str_val.startswith('http://') or str_val.startswith('https://'):
            return create_link(str_val)

    return str(v)

Can there be a new release pushed to pip that includes these fixes? Thanks.

Apologies for the delay.   I don't know what happened there, but I just re-tagged as v1.0.2 and pushed a new release to pypi.


turn gunicorn viewvc into a service

ignore the puppetfile nonsense

🍔🍔🍔 (testing unicode)

🍔🍔🍔🍔 bit more testing

should be text/plain, silly
🍔🍔🍔🍔

Root


Patrick Rice - Module 1 Changes

Hopefully I'm doing this right.  Let me know if I missed anything.
Thanks,
Patrick

:+1:


style: Truncate long model names

Long model names are now truncated with the extra letters replaced with ellipsis. I also added option to display the full model name on hover.

Awesome! When doing UI changes like this, could you please take the habit of adding a before/after screenshot? It makes the reviewing easier

Alright. I'll do that from now on


salt: 3001.1 -> 3002 and change maintainer

Motivation for this change
Salt v3002 got released: https://docs.saltstack.com/en/latest/topics/releases/3002.html
Also, remove @aneeshusa as maintainer and add myself instead as already done in #99060. (cc @doronbehar)
Things done

Checked that a salt master and minion work.

[x] 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"
[ ] 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
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 101480 1

1 package marked as broken and skipped:

pepper

1 package built:

salt


Fix deleted topic name

I wrongly typed the default name for deleted topics (d'oh!): _deleted_topic_ should instead be _deleted-topic_. This PR fixes that error and adds a check for the fromPb method in TopicId.

Coverage decreased (-0.009%) to 84.251% when pulling 8d0c0488038aadf334aa2281222779d2333ed8b0 on mziccard:fix-deleted-topic into fe4c90cba1be85c6885676eb1fa0214318448845 on GoogleCloudPlatform:pubsub-alpha.


feat: plugin for generating responsive images

Summary
Closes #147.
What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[ ] Docs
[ ] Build-related changes
[ ] Other, please describe:

If changing the UI/CSS related code, please provide the before/after screenshot:
Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)

You have tested in the following browsers: (Providing a detailed version will be better.)

[ ] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge
[ ] IE

If adding a new feature, the PR's description includes:

[x] A convincing reason for adding this feature
[x] Related documents have been updated
[ ] Related tests have been updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.
Other information:

think that looks pretty good :smile:

How does it look with 0.7s

I like this idea, but for lazyload I prefer using lozad.js combined with a markdown-it plugin, so that I can still use the markdown image syntax while writing articles. :D

@h404bi You will be able to lazy-load Markdown images as well, but if you want to use your own stuff you can disable it.

I like this idea (the image optimization), but for lazyload I prefer using lozad.js combined with a markdown-it plugin, so that I can still use the markdown image syntax while writing articles. :D

plugin now works with standard markdown images as well :)
@egoist do you think it's ready or is there something still missing?


fix: Remove iOS HiddenSelect workaround

Closes
Alternative to https://github.com/adobe/react-spectrum/pull/7103
I have tried this out in iOS 18.0.1 and our hidden input isn't enabling the next/previous arrows which come with form filling anyways. So it doesn't really matter if it's removed.
We could also attempt to fix the behaviour for next/previous.
Can someone try the next/prev arrows in iOS 17? we suspect it wasn't working there already.
✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

not sure if i tested this correctly but this is what happens when i try using the next/prev arrows in a form. doesn't really seem to work too well for some of the components. i'm on iOS 17.6.1
https://github.com/user-attachments/assets/303fc820-207b-48a1-ba83-628423329d3a

not sure if i tested this correctly but this is what happens when i try using the next/prev arrows in a form. doesn't really seem to work too well for some of the components. i'm on iOS 17.6.1

@yihuiliao can you pop it out into a new tab? I found that works better. otherwise the iframe seems to interfere.

okay this is what's happening on main right now.
https://github.com/user-attachments/assets/a9c36ec5-9819-4990-8c52-12e25aeda339
i also briefly tested the next/prev arrows on this pr, same story, and it doesn't seem to be any different from what's happening on main

@yihuiliao thank you for confirming. Next/Previous do skip over our Pickers/Selects

Correct me if I'm wrong but this is the direction that we chose to go with at the end of otter time since it seems our support for the prev/next button has always been a bit finicky


ADD: more settings to backup file

So that the user can have a seamless experience when importing a vault, please add:

[ ] language
[ ] currency
[ ] chains/tokens (can be skipped when autodetect is implemented)

@yvebe we discussed with JP that whole user app settings should be saved so user has seamless importing experience.
Yes chains get saved. But I think we could improve that with auto detecting the tokens. agree?


Service Catalog removal initial code and scenario

SC removal application

Requires KUBECONFIG to be set
removes SC helm release, deletes service instances and serivce bindings but the secrets is not removed

/retest

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


What to do if create app package says "Incorrect Program Format"

Pl provide steps on what should be done if following error is encountered on "Create an app package"
Severity	Code	Description	Project	File	Line	Suppression State
Error		PRI175: 0x8007000b - Processing Resources failed with error: An attempt was made to load a program with an incorrect format.	AppSystem.Wap	GENERATEPROJECTPRIFILE	1
Executing the project is running file. The above error comes up only on "Create an app package". Have tried with AnyCPU and x86/x64 platforms.

Hi, error code means that the format of DLL or executable being loaded is invalid. Are you seeing this error through Visual Studio?

Yes. Have both x64 and x86 dlls in the application folder. Package format is neutral because we load relevant dll depending on platform using assembly resolve.

@absinghal,
That error usually implies that the packaging process encountered corrupted PRI (resource) files. Regenerating your assets and any resources may help. If that doesn't work, please file a Feedback Hub item reproducing the error, for our devs to investigate. Please file it under the category Developer Platform > Desktop Bridge.

In order for the issue to be complete for future reference, it is same as described here:
https://developercommunity.visualstudio.com/content/problem/1135173/cant-create-app-packages-for-a-windows-application-1.html
Still waiting for the fix as mentioned in the post.


Editing light_specular is not available in the DirectionalLight3D inspector settings

Tested versions
4.2.2 stable
System information
Windows (I think it's not matter)
Issue description
Hello. According to the documentation, Light3D has a light_specular property. DirectionalLight3D inherits from it, however, when added to the scene, in the inspector properties it is not possible to edit light_specular. (At the same time, the official tutorial on fake global illumination directly advises creating a DirectionalLight3D and setting to zero its specular.)
OmniLight3D and SpotLight3D are fine in this regard.
Steps to reproduce
Create a scene, add DirectionalLight3D.
Minimal reproduction project (MRP)
N/A

Looks like it's not implemented:

Thanks for the report! Consolidating in https://github.com/godotengine/godot/issues/80737.


Upload faulty files

Rename a MOV file to PDF and upload where PDF is required. Also work with smaller video files, wav files to ensure that the file is smaller than file size limit
also try uploading a shell script code file. it should not execute on server side

can be tested fairly quickly. must be done before go live
@suresh12 @Ramu-kandimalla

@Ramu-kandimalla here are some additional guidelines for this. can be shared with the team
To prevent malicious file uploads, especially those where users rename files to bypass restrictions, consider these measures:

File Type Validation:

MIME Type Checking: Check the file's MIME type on the server side. Even if a MOV file is renamed to a PDF, its MIME type (e.g., video/quicktime vs. application/pdf) will reveal the true format.
Magic Number Inspection: Look at the file’s "magic number" (first few bytes of the file). Each file type has a unique identifier in these bytes, making it harder for renamed files to bypass validation.

Content Scanning:

Library-Based Validation: Use a library specific to each file type (like PDFBox for PDFs) to validate the structure of the uploaded file. If the file structure doesn’t match the expected format, reject it.

Restrict File Size:

Set a file size limit for each type to avoid oversized malicious files.

Sanitize and Secure File Storage:

Store files outside of the web-accessible directories and avoid direct access to uploaded files through their URLs.

These checks can help ensure that your application only accepts genuine files of the expected type, even if users attempt renaming tricks.
Here's sample magic numbers and sample code that can be used
Many file types use a "magic number" (specific byte sequences at the beginning of the file) to identify their format, regardless of the file extension. Here are some common file types and their magic numbers:

File Type
Magic Number (Hexadecimal)

PDF
%PDF (0x25 0x50 0x44 0x46)

JPEG
0xFF 0xD8 0xFF 0xE0

PNG
0x89 0x50 0x4E 0x47

GIF
0x47 0x49 0x46 0x38

ZIP/Docx/PPTx
0x50 0x4B 0x03 0x04

MP4
0x00 0x00 0x00 0x18 0x66 0x74 0x79 0x70 0x6D 0x70 0x34 0x32

MOV
0x00 0x00 0x00 0x14 0x66 0x74 0x79 0x70 0x71 0x74

Java Code to Validate Magic Numbers
Here’s a sample Java code that reads the first few bytes of a file and checks the magic number against expected values:
import java.io.File;
import java.io.FileInputStream;
import java.io.IOException;

public class FileMagicNumberValidator {

    // Define magic numbers for file types
    private static final byte[] PDF_MAGIC = {0x25, 0x50, 0x44, 0x46}; // %PDF
    private static final byte[] JPEG_MAGIC = {(byte) 0xFF, (byte) 0xD8, (byte) 0xFF, (byte) 0xE0};
    private static final byte[] PNG_MAGIC = {(byte) 0x89, 0x50, 0x4E, 0x47}; // PNG

    public static void main(String[] args) {
        File file = new File("path/to/your/file.pdf");
        try {
            if (isValidPDF(file)) {
                System.out.println("The file is a valid PDF.");
            } else {
                System.out.println("The file is not a valid PDF.");
            }
        } catch (IOException e) {
            System.out.println("Error reading the file: " + e.getMessage());
        }
    }

    // Function to validate if a file is a PDF based on magic number
    public static boolean isValidPDF(File file) throws IOException {
        return hasMagicNumber(file, PDF_MAGIC);
    }

    // General function to check if file has specific magic number
    private static boolean hasMagicNumber(File file, byte[] magicNumber) throws IOException {
        try (FileInputStream fis = new FileInputStream(file)) {
            byte[] fileBytes = new byte[magicNumber.length];
            if (fis.read(fileBytes) != fileBytes.length) {
                return false; // File is too short to have the magic number
            }
            // Compare magic number with file bytes
            for (int i = 0; i < magicNumber.length; i++) {
                if (fileBytes[i] != magicNumber[i]) {
                    return false;
                }
            }
            return true;
        }
    }
}

Explanation

hasMagicNumber: This method checks if the file's starting bytes match the provided magic number for a specific file type.
File Validation: Call the appropriate file-checking method (isValidPDF, etc.) to confirm the file type.

This code can be extended to support other file types by adding corresponding byte arrays for each file type’s magic number.

Hi @atulgupta2024  Please confirm the severity for golive

pr is merged  in dev env

dev testing is done for  it, qa can test it @Ramu-kandimalla

Closing this Ticket, reference #2401


added one more fix

Codecov Report

Merging #47 (f5c8f2a) into dev (d224001) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev      #47   +/-   ##
=======================================
  Coverage   18.29%   18.29%
=======================================
  Files          28       28
  Lines         246      246
  Branches       33       33
=======================================
  Hits           45       45
  Misses        199      199
  Partials        2        2

Impacted Files
Coverage Δ

src/Constants/pages.js
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


Access to errors in simpleMutation / queries

it seems that GraphQL errors are thrown into console as JS errors, which makes it difficult to catch and analyze.
Is there some way to access the errors?

https://www.apollographql.com/docs/react/features/error-handling.html
Now I see it is a wrapper of Apollo client so we shell do Error handling referring to Apollo.

Because the plugin implements its own flavor of Apollo instead of letting the user construct their client, I guess there is no easy way of hooking on the onError option in the Apollo client.

The user can construct their own apollo link instance. You can pass the link while setting up the GraphQL Plugin :)
If this is not enough, we could allow the user the construct an own client too, maybe.

@phortx
Thanks for looking into this, but since reading TypeScript is a bit hard for me, I am struggling to modify the code.
But I guess this will solve some other problems, I read in the doc that

It is inefficient to dispatch SimpleQuery and SimpleMutation in Vuex stores.

I believe if the Apollo client is constructed by user, then the user can use it for both VueX queries and non-Vuex queries.

It is inefficient to dispatch SimpleQuery and SimpleMutation in Vuex stores.

Where did you read that? It's not inefficient at all :)
Apollo has nothing to do with Vuex. Plugin-GraphQL is a bridge between Vuex-ORM and Apollo.
We could provide the onError hook in the configuration of this plugin, that shouldn't be any problem :) Thanks for the hint!

@phortx
from here https://vuex-orm.github.io/plugin-graphql/guide/custom-queries.html
Marked yellow
Yeah that's what I thought, it shouldn't be any problem dispatching an action there but hey, I didn't wrote the documentation :)

It says it's not a clean solution. It is efficient, but from a architectural view not ideal ;)

Right....OK whatever you say boss haha.
Right now I am still a bit confused about the barrier b/t apollo link and apollo client. But that's ok, I will figure it out eventually, but I just would like to consult if there is any barriers of implementing the WS version of apollo-link in the current stack.
https://github.com/apollographql/apollo-link/tree/master/packages/apollo-link-ws
There official link settings is like:
import { WebSocketLink } from "apollo-link-ws";
import { SubscriptionClient } from "subscriptions-transport-ws";

const GRAPHQL_ENDPOINT = "ws://localhost:3000/graphql";

const client = new SubscriptionClient(GRAPHQL_ENDPOINT, {
  reconnect: true
});

const link = new WebSocketLink(client);

Websocket is really cool!
I don't think it's complicated to bring apollo-link-ws into this plugin. The biggest barrier is that I don't have any time currently for working on this plugin :(


swift problem, have to cast everything and can not save

Hi. I am using this library in swift 4 and xcode 8.3.3. My problem is I have to cast almost everything and this is a problem because I can not save my changes to the file. The values returned are not same as the ones in the objective-C function bodies.
for example
file = BRAOfficeDocumentPackage.open(filePath)!
let worksheet: BRAWorkshhet = file.workbook.worksheets[0] //this is supposed to return a BRAWorksheet. But not for me. It returns Any for me. So the line does not compile and gives me this "can not assign value of type Any to BRAWorksheet"

So I have to cast it now
let worksheet = file.workbook.worksheets[0] as! BRAWorksheet
This is fine for reading from files. But writing becomes a problem because the worksheet returned by casting is a COPY of the one in the workbook, not the original reference. so when I file.save(), nothing is saved to the file.
I have a feeling that I have done something wrong with importing the framework, though I followed what was written in the guide step by step, except I used cocoa pods.
Can Anyone tell me the problem here?? thanks

Same, cannot save...


Hotfix PS Tables

I think I did it.

@schachfeld can you increase the "hill" values for the king and then merge this?


all: gofmt -w -r 'interface{} -> any' .

This PR replaces interface{} with any. Additionally, modifies the Fmt mage task to ensure usages of any in the future.
Follows #9688

@bep I think we should merge this PR to avoid possible conflicts with other PRs. But these conflicts are easy to fix by running the command gofmt -w -r 'interface{} -> any' .


[AVM Module Issue]: add support for Azure Provider v4.0

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Feature Request
Description
Modify the module to support azurerm v4 (see 4.0-upgrade-guide for change log and migration hints).

Accommodated for in https://github.com/Azure/terraform-azurerm-avm-res-managedidentity-userassignedidentity/releases/tag/0.3.3


git clone/checkout inserts CR characters causing --check-only to malfunction

Environment:
This issue only relates to genversion usage on Window OS.
Conditions for --check-only to malfunction:
This issue only occurs when the generated version source file is recreated when executing git commands such as git clone or git checkout. On Windows OS when the git configuration includes core.autocrlf=true git expands LF characters to CRLF when git creates or overwrites the generated version source file.
Suggested Fix:
Strip all CR character from the generated source file string before the --check-only validation of the file contents occurs.
Linux/Mac Side Effects:
AFAIK there are no side effects ... git does not expand LF to CRLF on Linux/Mac.

Thanks for the reply; I can see it now. The flag might bring only more confusion because of git doing its things nevertheless.
Oh, that was a quick adoption :) Good to hear everything is in order. Thank you.


db_config.json does not support custom ports

Our on-premise instance is provisioned with non-standard ports, but copilot's db_config.json does not support specifying ports.

Thanks for your report @OrkhanAlikhanov! We will take care of it.

This has been merged and resolved in the latest docker image.

Incredible turn around time! Thanks!


Updating security reachout email

Description
Updating security reachout email from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>Issues Resolved
N/A
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Note that opensource-codeofconduct@ is likely correct at the moment.

@varun-lodaya Even if we don't change it, it seems like the force behind updating the security email to use opensearch domain should be following the same principles as the one governing all our reporting processes

@varun-lodaya Please update this PR to replace all instances of the security email in this repo.

@peternied the code of conduct email alias is a different one, we don't have replacement of that yet right @peterzhuamazon ? Also, rebase was messing up the history, created a new PR - https://github.com/opensearch-project/.github/pull/204/commits. We can close this and track the change there.


Fixes Rainbow Score to use correct Reporting Protocol

Currently, the Rainbow scores show the scores of the final agent, as opposed to a re-evaluation of the best intermediate network.
The new README looks like this: https://github.com/prabhatnagarajan/chainerrl/tree/fixed_scores/examples/atari/reproduction/rainbow

Did you include examples/pretrained/ by mistake?


Update package.json

Bump axios to the latest version

Duplicate of #215

Duplicate of #215

Fixed in #215


Try compile against Go 1.6

cross fingers

Merged here:
commit be59cd12f9951190d8a791b6a352f3ffca1c889b
Author: Term1nal<EMAIL_ADDRESS>Date:   Mon Feb 29 16:47:50 2016 -0800
fixes #190 Add go 1.6 to travis


Rebased version of #13.

These are the changes from #13 rebased to the current master.

@yonahforst @support-fetchsky @zsajjad
Hi! It will be very helpful if this branch is merged.
Is there anything wrong? Please tell me if there is something I can do.
Thanks!