Choose an SAP HANA Database

Tutorial URL: https://developers.sap.com/tutorials/hana-clients-choose-hana-instance.html
Please specify the step you are referring to 5
My login happens after these steps are executed:

sudo su - hxeadm
hdbsql -n localhost:39013 -u SYSTEM

ip addr fails with: "* 257: sql syntax error: incorrect syntax near "ip": line 1 col 1 (at pos 1) SQLSTATE: HY000"

Thanks for the good feedback.  I had meant for the ip addr command to be executed at the Linux shell and not in the hdbsql tool.  I have changed the order of the commands in the tutorial to make this clearer.  The change will appear in the next update to the tutorial.
Regards,
Dan van Leeuwen

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


EngineInterfaceObject: initialize late extensionObject

EngineInterfaceObject may have extensionObject set late. Specifically,
internal WScript.InitializeProjection() initializes WinRTPromise, which
is later than EngineInterfaceObject::Initialize() thus missing
extensionObject->Initialize(). Add extensionObject->Initialize() for
this case.
Fixes a list of nightly projection failures.

Thanks @dilijev @meg-gupta


Trying to improve user experience on authentication

I'm trying to adapt the authentication to my needs and somehow improve user experience. Given the various possibilities, I would like to share my thoughts and accept suggestions.
In the sequel, please note I`ve removed from my app providers that do not provide email information. In fact, I chose only facebook, google and linkedIn. In addition, assume the user is not yet logged in.
Case 1: User registered via Provider (e.g. facebook), tries to sign in w/ email and password
In case the email is the same as the one provided by facebook, in my opinion the best user experience in this case would be to receive a message recommending to sign in with facebook (instead of simply saying the 'username or password is incorrect').
To do this, I added the following to the local strategy.
if (user.provider !== 'local') {
    return done(null, false, {
        message: 'Please, sign in with ' + user.provider
    });
}

In case the email is different, it is obviously impossible to detect the user has a registered facebook account.

Case 2: User registered “locally” tries to sign in w/ provider whose email is the same
In this case it would be a good experience for the user to authenticate into the same account. For this to work, I would add an email search query in “saveOAuthUserProfile()"
// Email search query
var emailSearchQuery = {
    email: providerUserProfile.email
};

// Define a search query to find existing user with current provider profile
var searchQuery = {
    $or: [mainProviderSearchQuery, additionalProviderSearchQuery, emailSearchQuery]
};

Case 3: User registered “locally” tries to sign in w/ provider whose email is different
Given that the user is not logged in, there is nothing to be done in this case. This action would create another user account, which is bad, but unavoidable.

Case 4:  User registered via Provider (e.g. facebook), tries to sign in w/ another provider (e.g Google) in which his email address is the same:
For this to work, given that the search query can now match by the email address, it is necessary to check if the provider is already configured, with practically the same code as if the user was trying to add the provider:
// Check if user doesn't have that provider data already configured
if (user.provider !== providerUserProfile.provider && (!user.additionalProvidersData || !user.additionalProvidersData[providerUserProfile.provider])) {
	// Add the provider data to the additional provider data field
	if (!user.additionalProvidersData) user.additionalProvidersData = {};
	user.additionalProvidersData[providerUserProfile.provider] = providerUserProfile.providerData;

	// Then tell mongoose that we've updated the additionalProvidersData field
	user.markModified('additionalProvidersData');

	// And save the user
	user.save(function(err) {
		return done(err, user);
	});
} else {
	return done(err, user);
}

Case 5:  User registered locally with two different accounts (for any possible reason), tries to add in one of the two accounts a provider that was already registered in the other:
If a user does that, then one of the accounts would become unaccessible via the provider that is common to both accounts. Although this is a minor issue, I provided a solution in my app.

Regards

@igorauad has this been resolved?

@ilanbiala this discussion was not continued, so I guess you could close. Perhaps this could be further considered in #343


Update README-MULTISITE.md

I update the parameters according to this PR: https://github.com/ceph/ceph-ansible/pull/7315 In addition, where rgw_zonesecondary: True and rgw_zonemaster: False, I change rgw_zonegroupmaster to be False as those lines below:

https://github.com/ceph/ceph-ansible/blob/main/README-MULTISITE.md?plain=1#L396
https://github.com/ceph/ceph-ansible/blob/main/README-MULTISITE.md?plain=1#L417
https://github.com/ceph/ceph-ansible/blob/main/README-MULTISITE.md?plain=1#L520
https://github.com/ceph/ceph-ansible/blob/main/README-MULTISITE.md?plain=1#L535
Add note at line 205 here https://github.com/ceph/ceph-ansible/blob/main/README-MULTISITE.md?plain=1#L205

jenkins test centos-container-all_daemons

jenkins test centos-container-purge

jenkins test centos-container-all_daemons

jenkins test centos-container-update

@quangln94 you missed adding Signed-off-by: YOUR_NAME <your_email> to the commit. This is usually done by running git commit -s but you can do this by adding it manually via git commit --amend and then force pushing the new commit onto this branch.

@alimaredia i don't know how to add another commit in this PR. I edited it directly on github web. So can you add commit for me please: my email<EMAIL_ADDRESS>Thank you so much

jenkins test centos-container-all_daemons

@guits should I just open up another PR with @quangln94's changes?

closing this one in favor of #7327


load graphviz-react async

Description
Load graphviz-react asynchronously
Related issue(s)
Resolves #1044

/retest

As I was reviewing it I noticed that IntersectionObserver won't kick in if the browser viewport is smaller than graph area height ~500px. I guess we're fine with that.

/test pre-busola-integration-test-k3s-namespace

/test pre-busola-integration-test-k3s-namespace

/test pre-busola-integration-test-k3s-namespace

/test pre-busola-integration-test-k3s-cluster


Reuse materials, if those are duplicated

Description
It was found that when loading glTF, if there are multiple materials with the same material index and different attributes (skinning and morphing), the reuse of the materials does not occur as expected. To improve this, i attempted to change the keys indexed to reuse the material to include not only the material index but also other attributes.
This problem occurs because Rhodonite Materials with different attribute values are repeatedly assigned to the rnMaterials using the same material index as the key.
https://github.com/actnwit/RhodoniteTS/blob/8d7a9aa90df7a78e8f0c1934194699581565509c/src/foundation/importer/ModelConverter.ts#L785
TODO

[x] Implement to change the key of rnMaterials from the material index to a hash of the material's attribute value
[x] Draw a test glTF to verify that the Material is actually reused
(call the Rn.Material.getAllMaterials() method before and after the change in Rhodonite and check the behavior by comparing the number of its elements. The glTF used for testing is the following)

skin-cylinder.zip

It looks great to me:)


[Datafactory] Adding new API to get GitHub access token

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[ ] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Can one of the admins verify this patch?

Automation for azure-sdk-for-go

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

error:
operationId: Factories_GetGitHubAccessToken
scenario: Factories_GetGitHubAccessToken
source: response
responseCode: '200'
severity: 0
errorCode: OBJECT_ADDITIONAL_PROPERTIES
errorDetails:
  code: OBJECT_ADDITIONAL_PROPERTIES
  params:
    - - tags
      - location
      - eTag
      - properties
      - type
      - name
      - id
  message: >-
    Additional properties not allowed:
    tags,location,eTag,properties,type,name,id
  path: ''
  title: /definitions/GitHubAccessTokenResponse
  description: Get GitHub access token response definition.
  position:
    line: 3252
    column: 34
  url: >-
    specification/datafactory/resource-manager/Microsoft.DataFactory/stable/2018-06-01/datafactory.json
  jsonPosition: !<tag:yaml.org,2002:js/undefined> ''
  jsonUrl: >-
    specification/datafactory/resource-manager/Microsoft.DataFactory/stable/2018-06-01/examples/Factories_GetGitHubAccessToken.json

Seems like example does not return the type that is described in the swagger. Please fix.

@hvermis there are bunch of existing Model Validator Errors not related to your PR. Please fix them in the future ;)

Will address them in next PR

In reply to: 419502847 [](ancestors = 419502847)


Improves diagnostic information for 'Any' type

Discussed here:
https://github.com/godatadriven/pydantic-avro/issues/100

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (bcbdeee) 91.12% compared to head (7ed0488) 90.27%.

Files
Patch %
Lines

src/pydantic_avro/base.py
0.00%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #101      +/-   ##
==========================================
- Coverage   91.12%   90.27%   -0.85%
==========================================
  Files           2        2
  Lines         214      216       +2
==========================================
  Hits          195      195
- Misses         19       21       +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Thanks for the pull request! I will approve and merge as soon as I can. I se that some the tests in our pipeline are failing. (Failure to install poetry on 3.7 it seems). I will first need to fix this before I can get this pull request through.

Thanks for your contribution. Will merge the changes.


Add live editing and normalize package.json

Resolves #18
We're using browsersync instead of livereload because the livereload module for hexo is broken.
Additionally, expands package.json with appropriate links and license information.

Looks good to me!


Runtime Error

Hello there,
I'm currently trying to get it working.
I've seem to sucessfully install the run.bat , at least it states that it is successful.

only thing that seems to be out of place is "unable to connect to <IP_ADDRESS>:6555:6555"
The AnkuLua app states no error when starting or selecting the script.
Then I start the game, press start for the script and receive the empty message box (which seems to be normal according to other posts?)
after some time I'm receiving the following error:

here it also states the issue with "unable to connect to <IP_ADDRESS>" ?
I've already tried to reboot my phone and install the deamon again (same window)
System:
device: Huawei P20 Pro
android: Android 8.1
AnkuLua: 8.1.1-trial
According to older posts it states that Android 8.0 an higher is not supported, but I can't find this information in the current readme-file, so I suppose it's working now?

 Latest version of Ankulua however does not require daemon for android 8+, just have to switch settings in there to Media Projection screenshots and touches by Accessibility. This at least works fine for me.

Now I'm getting the following error:

Which version of Ankulua are you using?

Okay, so found a thread that seemed to be related to the issue you posted. The issue it seemed to be was the ads that are shown? The entire conversation is in Chinese and was pre-Ankulua 8.0 it seems, but it was the same error regarding Media Projection issue, on the same game TW server, and he said that the error didn't occur when there wasn't an ad. Also, he said there didn't seem to be a way to remove the ad. I think he rebooted the phone (or rebooted Ankulua) and it wasn't there eventually.
Give it a shot, and see if it solves the issue. Hopefully, since I personally have not run into this issue. Also as a side-note, running the free version of Ankulua might have ads, but the real issue is that there is a time-limit. I personally suggest investing in the app as I feel it is worth it to run for long periods of time and it isn't the most expensive thing in the world that I've seen. $50 for life is pretty good, especially when it can be used for MORE than just this game.

https://github.com/29988122/Fate-Grand-Order_Lua/blob/master/README.md#android-81-and-above
Refer to this section for more detail.
You can still reopen the ticket if you encountered any issue!


As a developer, I want to better support for initializing an infrastructure deployment

As a:
Developer
I want:
Enhanced support for spk infra so that it validates my environment is configured before terraform template initialization. Allows me to copy custom bedrock templates to the cli tool to deploy and stores it locally.
So that:
Better support for my customer scenario
Describe the solution you'd like:

[ ] spk infra init check
[ ] Copying the template of AZ simple and managing in a .spk/environments folder
[ ] Support for git URL for spk infra create

Additional context:
Related to - https://github.com/microsoft/bedrock/issues/590
Does this require updates to documentation?:
Yes

#615 Duplicate


chore: remove getAPIResponse

Testing

Trade-offs

Dependencies

Screenshots

:tada: This PR is included in version 5.0.9 :tada:
The release is available on:

v5.0.9
GitHub release

Your semantic-release bot :package::rocket:


[RIP-9] Add en doc of Operation FAQ

What is the purpose of the change
RocketMQ Developer Guide doc
Brief changelog
Add en doc of Operation FAQ
Verifying this change
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.05%) to 44.778% when pulling fb48f4def633f19a7c49a5c23b9e67df2de90f16 on husterxsp:develop into 6a573a773a87edbcdcc138614b756019fc280988 on apache:develop.


[#72] Implement sustainability architecture checklist

This is the newly opened second part of the ticket https://github.com/GermanZero-de/klimaschutzmonitor/issues/72

I added two commits regarding my comments. But we can discuss that if you think something does not fit.


Prolog Formatting done

Pull Request
Description
Please include a summary of the changes and the related issue.
Closes #
Type of change

[ ] Adding a new Resource
[ ] Adding a new Category
[x] Documentation enhancement or fixes
[ ] Website enhancement or fixes

Checklist:

[x] I read carefully CONTRIBUTING.md
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation

Screenshots (if appropriate):
Please add screenshots to help explain your changes.
Additional

done


Reproducing the results from readme file

Hi there, how can I reproduce the results seen that can be seen in the readme file? Is it possible to reproduce those results (photos + the corresponding labels about whether the photo is classified as fake or not) within python or one has to use Atlas? Also, is it possible to insert and test custom photos or only those provided by you?

Hello,
if by "reproducing the results" you mean train the best model we achieved, then yes you can do it in simple python. You can find the hparams in the UI. When you have the parameters, you can either retrain the model on your data or download the pre-trained model if you wish.
For your question about custom data, you can put whatever data you wish in the correct path and the pipeline with train a model on that data. Keep in mind that the repo doesn't provide inference code, it only provides training code, so you may have to write that on your own.
Hope this answers your questions?

Where could we download the pre-trained model for evaluation please?


Update RecurringCost SDK to match updated API response

Everything looks good on this. Code looks good and matches the API. Approved.


Bluetooth headphones stopped working correctly with latest upgrade

Hello,
since the last upgrade, my bluetooth headphones won't work properly whereas Spotify or Skype do. Please, could you indicate what to provide in this specific case?
Thanks

OS:
Issue Descriptions:
Steps to Reproduce:

We don't officially support Bluetooth headsets as it is impossible for an OSS project to get access to enough headsets to effectively support the multitude of different "standards" and intrerfaces.
If they worked with an older version they will probably start working again with a newer version.  It is probably caused by an upstream issue inside Chromium.  You're just going to have to wait this out 👍
Sorry if that isn't very helpful but it's the best we can do 👍


BlackJack in python project

Type of change

[✅ ] Added new project
[ ] Bug fix
[ ] New features
[ ] Enhanced documentation

Changes proposed in this pull request
added a new project

Project name :

BlackJack

Short description :

We all know about the popular card game BlackJack. Here I have created a simplified implementation of it in python where the computer acts as the dealer. It's a single-player game. The player can bet a certain amount of money. The project is easy to read and follow and contains comments as necessary.
Checklist

[ ✅] The code is Python 3 compatible
[✅ ] I have added relevant comments in my file to make the code understandable
[ ✅] I have added my project to a folder that includes all the necessary files

Looks good thanks for contributing 👍


Is it possible to directly use libsecrets.so and get the key?

Hi. Is it possible to use libsecrets.so library in a custom Android or a Linux app and call the generated method and get the key?

@deastr yes it's possible, see #61 but as long as you obfuscate your class which gets the secrets then it's hard to figure out exactly how to do that


Items not loading

Hi i tried the new trainer update and got to a problem. I can connect to my console but when i hit load items, nothing is showing up. I tried to refresh and play around in the code and nothing seems to work. Thanks !

Game version?
Sent with GitHawk

You have to wait a few minutes after loading the game and then click the load items button. If that doesn't work you may need to get the DLC which fixed it for me.


implementing new crop positions: 'top-center' and 'bottom-center'

useful when cropping vertical 9:16 images or creating Material Design cards.
I think that the image used in unit tests should be changed before writing new tests for 'top-center' and 'bottom-center' positions

@connorjburton can you give this guy a looksie?

Will take a look this afternoon (y)

Hi @salaros, this looks good, and good refactor too. Will put this through its paces tomorrow. In the mean time could you just fix the scruitinzer issue (just need spaces inside parentheses on the switch) and would you be willing to update the tests to reflect these additions?
Thanks

Thanks @salaros, this all looks good, i've tested it locally.
Do you have time to update the tests to reflect this please?
Also just noticed the docs don't seem to mention these options anywhere, or I am just being blind @jarednova

Sorry for messing up the PR, but I've just pulled commits from the wrong origin (before pushing new tests)

Thanks @salaros, I'll be able to review and merge this monday

All good to go, will merge soon! Thanks @salaros

@jarednova - can you merge this or give me permission to push via cli without requiring travis ci check please? Thanks

All done! Thanks @salaros and @connorjburton for your help on this guy!


Example request adds "content-type: application/json" header in GET request

What happens?
I have two endpoints which are almost identical in the openapi.json file (GET /api/client and GET /api/account).
GET /api/account works as expected, but GET /api/client is causing my backend to throw an exception because the request is sent with "Content-Type: application/json" and my backend is trying to deserialize the non-existent body the request.
I can't share the full openapi.json file, but here's an excerpt of the two endpoints:
"/api/client": {
  "get": {
    "tags": [
      "Client"
    ],
  "summary": "Get clients",
  "operationId": "ApiEndpointsClientIndex",
  "parameters": [...],
  "responses": {...}
  },
},
"/api/account": {
  "get": {
    "tags": [
      "Account"
    ],
    "summary": "Get accounts",
    "operationId": "ApiEndpointsAccountIndex",
    "parameters": [...],
    "responses": {...}
  },
}

When testing an example request for GET /api/account, it does not add the header and I get a valid response.

When testing an example request for GET /api/client, the Content-Type: application/json header is added which which causes the exception on my backend.

What did you expect to happen?
I would expect to either both requests to have the included Content-Type header by default, or none of them to have it, since they have almost identical definitions in the json file.
OpenAPI Document
No response

Hello! If you update the client, you will find that we no longer set any content type and leave it up to the user!

Hey @amritk, unfortunately I can't use the latest version of the client - it's crashing the UI. I've reported the issue here #3165.

@amritk, I just tested this with 1.25.17 and it seems that the issue is resolved.
Thanks!

This still happens to me using v1.25.58.


[Bug Report] - Flame Lily Seeds not spawning on 1.18.2

Describe the bug
Using the 2.0.0 version of this mod for Minecraft 1.18.2, Flame Lily Seeds no longer spawn in desert biomes.
To Reproduce
Steps to reproduce the behavior:

Go to a desert biome and fly around in Creative/Spectator.
Repeat in multiple different worlds.

Expected behavior
Flame Lily Seeds should spawn in desert biomes, like in previous versions of the mod, unless their removal is intentional(?) There's also no crafting recipe for them, unlike Ender Lily Seeds, so since they don't spawn they don't seem attainable in Survival.
Screenshots
If applicable, add screenshots to help explain your problem.
Mod Info (please complete the following information):

Forge: 40.1.51
MC Version: 1.18.2
Mod Version: 2.0.0

Additional context
Add any other context about the problem here.

Fixed!


添加worker-list

这个 PR 实现什么功能/修复什么问题:
添加worker-list
是否需要 backport 到之前的 release 分支:
None
/cc @swordqiu @yousong

Codecov Report

Merging #901 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #901      +/-   ##
=========================================
- Coverage    7.62%   7.62%   -0.01%
=========================================
  Files         355     355
  Lines       55105   55105
=========================================
- Hits         4204    4199       -5
- Misses      50504   50507       +3
- Partials      397     399       +2

Flag
Coverage Δ

#aFlag
7.62% <ø> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/util/seclib2/seclib.go
91.48% <0%> (-4.26%)
:arrow_down:

pkg/util/rbacutils/rabc.go
68.56% <0%> (-1.14%)
:arrow_down:

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 7fb6731...5f30f4f. Read the comment docs.

/lgtm
/approve


Change the gradle build image to gradle:jdk11-alpine

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Replace the Gradle build image to gradle:jdk11-alpine.
Why
The open-jdk:11-jdk is not working fine in some environment.
Known limitations

Reference Server Preview is available here:https://anchor-ref-pr731.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr731.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr731.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr731.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr731.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr731.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr731.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr731.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr731.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr731.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr731.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr731.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr731.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr731.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr731.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr731.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr731.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr731.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr731.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr731.previews.kube001.services.stellar-ops.com/


Market Liquidity

I am a beginner in finance, so I do not understand why we assume the agent's action does not impact the market attributes in stock trading. That means the trading actions occur close to the market closure. Can anyone explain the problem in detail? Some assumptions (market liquidity)make me confused.

The agent is trading on a small volume that is around 100 shares at max( can go higher). But if you have a large order volume you may affect the prices, that is changing the history. The transactions are fulfilled based on a order book where people bid and ask for their prices. A higher order volume will likely fulfill many orders leading to changes in price. So we assume the actions by our agent does not change the prices else our train data would be invalid.

If the prices change in that way, the trading agent cannot detect the market trend by the listed indicators, right? Can we call the market change is sentiment?

I guess not. Actually you are training on historical data, so if your agent is able to change it somehow, then it may lead to invalid training. We just assume that whatever our agent does, it will not affect the prices. The changes will be unpredictable also

@thy-hoang-nn would you like to do market simulation (say on limit order Book, or liquidation model)?  It is a challenging and interesting task.

@XiaoYangLiu-FinRL Yes, I am trying to understand the concept and simulate this type of market.

If you have some progress, please refer to the FinRL-Meta repo: https://github.com/AI4Finance-Foundation/FinRL-Meta


[SMALLFIX] Fix path printed by generate-tarball.sh

${TARGET} is already an absolute path

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/12310/
Test PASSed.

LGTM


Add button to toggle auto-completion in the SQL editor and corresponding configuration option

When connected to Postgres database with hundreds of thousands of tables or with highly complex structures the auto-completion in the SQL editor makes editing real pain because of editor lags for several seconds due to the generation of the auto-completion lists (I guess). Could you please add a switch to turn the auto-completion on/off and an configuration option for it's default setting (on/off)? Thank you.

Further proposals:
Also, what's somewhat related is that the initial loading of the database structure (in the left pane) blocks the WUI. It would be nice if this was somewhat asynchronous, just indicating somehow that it's not completely loaded yet and progressing to show what's already scanned but in a way it does not block the rest of the WUI (so that one could connect and start typing SQL immeditately without the need to wait until the left pane loads completely). Thank you.

When you say hundreds of thousands of tables what do you mean by that? Does your server have many schemas with a lot of tables in them or all in one schema?

@sosedoff Sorry for the delay.
Many tables per schema (normalized database; TimescaleDB with Promscale ingesting Telegraf measures via Prometheus remote write protocol or something similar to this could be a good example).

I think this edge case is just way off the general usage of this tool, e.g if you have a huge data set and lots of objects, it would probably be better to use something else. I don't know enough about other DB UI products and how they stack up in that regard, but i have a feeling there might be the same exact issue. I suggest you implement the change in your own fork.


use chatwork notation

close #5
use chatwork notation to post message.
it is using this notation,
"body=[info][title]#{params[:message]}[/title][/info]"

but could use additional field if wanted.
body=[info][title]#{params[:title]}[/title]#{params[:message]}[/info]"

ok it workd.


⚠ Bofrost.com has degraded performance

In 6762493, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 855 ms

Resolved: Bofrost.com performance has improved in 277af4d.


insufficient client permissions

So i was able to set up everything the Steam part is working fine and typing !verify also works but after i give the bot my name it says this in chat " Something went wrong, either retry !verify or contact an admin please!" and this in my console "{"id":2568,"msg":"insufficient client permissions","failed_permid":28}" it also says this every once in a while in the console "Error while querying users: [object Object]" and idea on how to fix it?

That error message should now display something useful.  https://github.com/nikitavondel/steam-ts/commit/d2b2050002357e998761ec6c4245a256216c6534
And I can tell your query client doesn't have all the permissions it needs. Try creating it via an admin account ^.

how would i do that? sorry but im not that familiar with this stuff

I am also getting this error and I can't seem to find a log from the Bot to see whats going wrong...
I even gave it Server Admin Perms...

I suggest connecting to your teamspeak server from your root account (the one you had to fill in a token for at the beginning) and then create the query client. Or you could dive into your teamspeak permission settings and try to figure what doesn't allow you to create query clients with all the permissions.

Oh now i understood it. I already tryed that, i tryed the standart server query and one that i created but both gave the same error. Maybe you can tell me what permissions it is missing exactly? Then i can ask my host to unlock them.

When you give the bot your nickname it will execute the following query commands:

clientfind
clientinfo
sendtextmessage

So it'll need permission for these three.

Alright the bot has all the permissions now but everytime someone trys to verify they get "User was not found on the TeamSpeak server, use !verify to restart the process." and my console is getting spammed with "Error while querying users: {"id":2568,"msg":"insufficient client permissions","failed_permid":27}" which is a different error then the one i had last time.

It's because it needs even more permissions to query the users (obtain all data about current online users), I suggest giving it all permissions since having to manually grant every permission be a bit annoying.


Cljs macros

This allows the repeat and all macro's to be used inside the clojure-turtle.core namespace. See #15
Note: cljc and cljs classpaths had to be divided, I believe this is good practice so clojurescript code is not included during clojure compile.

Very nice!  Thanks for looking into this and figuring it out!
Being able to figure this issue out is a necessary step in moving towards the possibility of targeting the JS runtime without any JVM dependencies.
I like the division of cljc and cljs code, too.


Submitting the form without typing anything

I'm using your package in a single field form with autovalidate=false and ignoreBlank=false. I managed to get auto focus when the user comes to the page. But I'm facing an issue if the user submit the form without typing any key. In that case, the validator of the TextFormField seems to be not called as well as _phoneNumberControllerListener. So in the RaisedButton.onPress() method, formKey.currentState.validate()=true and the form is successfully submitted. I'm aware that this use case can be valid. But in my case, the user has to fill the phone number field. So I would like to raise an error if the user submit the form without entering a single character.
I think that it can be achieved by exposing the value of provider.isNotValid = false; in the method _phoneNumberControllerListener. Or it could be something like this provider.isNotValid=! ignoreBlank.
Do you think that my request is doable?

Hi, @catside295 thanks for raising this issue I checked it up and found out it wasn't behaving as expected. The form doesn't validate if it's empty when ignoreBlank is set to false. I'll try to work on a fix now

I just pushed a fix, thanks for the contribution


Forign key

ForignKey example in the table the department column name automatically ends with the _id suffix like department_id


Add hospital capacity

Related to https://github.com/epidemics/covid/pull/384

@mathijshenquet Would you consider restructuring the export?
This adds 13th argument to be passed around and repeats parts of the code ... :/
What it does (and is common for other attributes):

Reads a DF from a CSV file configured in the config (BTW see epimodel.read_csv)
For each country, if it is in the DF, filters out present keys (all or from a given set), replaces NaNs.

For files with dates, it outputs a series for each key rather than just the keys.

(Then e.g. timezones are treated specially - but a simple lambda function should handle it?)
Then it adds the result as an attribute to ExportRegion

(See also the slack note)
@hnykda Also interested in your take - I may be missing something ..

I think I missed this review @hnykda. I will take a look at it again when Jan and Jacob decide if we still do this feature.

IMHO restructuring should happen in its own piece of work. Going from 12 to 13 arguments doesn't really change anything. Reacted in slack wrt. that. So I propose to merge this as is (with rebase etc.) to get the data out, but that is independent from what I think about refactoring.

highjacked this, rebased on master and fixed some comments

Fine with me


Create CODEOWNERS

To enforce reviews from the X-Ray team.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #109 (80c1e83) into main (b7f4fd5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #109   +/-   ##
=========================================
  Coverage     65.10%   65.10%
  Complexity      193      193
=========================================
  Files            17       17
  Lines           854      854
  Branches        132      132
=========================================
  Hits            556      556
  Misses          229      229
  Partials         69       69

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 b7f4fd5...80c1e83. Read the comment docs.

Already exists: https://github.com/aws/aws-xray-java-agent/blob/main/.github/CODEOWNERS
Closing in favor of modifying that.


Examples comparing to isort's heuristics in the documentation seem outdated

I've tried the examples of isort breaking code that are given here and it doesn't seem like any of those are broken by isort:
https://usort.readthedocs.io/en/latest/why.html#comparison-to-isort
Based on my testing, they haven't been broken since isort 5 (released July 2020) for sure so I'm a bit surprised to see those listed in the documentation that was added just 6 months ago. I can understand the benefit of usort being designed with safety as a priority but the way it's compared to isort is currently unfair.
isort doesn't really seem to advertise safety as a priority (though I'm pretty sure it's nonetheless considered important to it since isort 5) so that is still a good selling point (I find the mention of the strictly-parsed tree and how it causes bugs in isort to be particularly convincing), it would be good to not show code examples that isort doesn't actually break though :)

That's fair! The examples on that page don't misbehave on isort 5.x anymore.  That page (including some of the bugs) was from our experience running 4.x in a megarepo, and what drove us to start this project rather than trying to piecemeal "fix" isort.
There are still some major differences between our approaches.  I'll post some ideas in followup comments, but didn't want to delay confirmation, thanks for reporting.

I did some manual fuzzing of isort to try to come up with better thoughts on how we're different.  In addition to our "first, do no harm" principle, I think using more words:
An import sorter should never take valid code and make it invalid
We get most of this safety by using an actual parser (LibCST) that means it's unlikely we will output mangled lines, and we put care into ensuring that there aren't a lot of branches that are hard to test.
# horizontal whitespace is allowed around dots
import a . b . c

# backslash continuations should count as horizontal whitespace too
import a.b\
.c.\
d

# any name of cimport is silently dropped
import cimport

# but aliases of cimport get mangled
import a as cimport

# formfeed is a valid horizontal whitespace, and this gets mangled and the "from a" ends up on its own line
from z import x
from a\x0cimport y

# nbsp is a valid horizontal whitespace (as is full-width space, U+3000)
from mcimportface\xa0import y

# backslash ending a comment isn't a continuation, but this gets mangled
import a # comment \
import b

An import sorter should never take invalid code and make it valid
This is mainly about syntactically-invalid code, and less about import-time side effects which we consider an acceptable risk and I wish didn't exist.
# turns into two lines, "import a" and "import b"
import a from b

# no commas get "corrected"
import foo bar baz

# parens anywhere
from a import )()()(b
from ((a import b))
from a import (((((b)))))
from a import b)))))

# mixed tabs count as 1 space, not 8
if True:
        import z
\t\t\t\t\t\t\t\timport a

# import line gets dropped
from foo import \ # comment
int

# backslashes get removed
from \a\ \import\ \b\\

# some spaces get removed
from a as c import y

# turns into two imports of a
import a as b as from c

# import line gets dropped
from \
a \
import \

int

I think the only case where we silently "fix" invalid code in usort is moving __future__ imports to the top within a block.
An import sorter should make understandable decisions, even if you disagree with them
...and ideally be able to explain them.
# the "c" isn't considered an import, thus counts as a barrier, I think?
import z
import\
c
import a

# these aren't considered an import either?
from.import(b)
from.a import b

Configuration should be minimal, and behave the same universally
It shouldn't matter what version of Python you're using, or what you have on your sys.path, or what OS you're on.  This causes us to make some compromises (e.g. what names are considered stdlib, or case-insensitivity) that I imagine spark disagreement though.
The one exception to the environment we have is inference of first-party name under the assumption you're running it from a vcs repo.
We assume the union of all py3 stdlibs currently.
It's okay to leave a file alone, or crash as long as you don't overwrite it, but do so as gracefully as you can.
# infinite loop
import (

# pathlib error
import //
# but this "works"
import //a

# IndexError
import as as as

# IndexError
import \

We had one bug this week that raised KeyError, but generally we  raise something like SyntaxError pointing to a relevant part of the source.
Conclusion
I recommend we:

Remove the "Unlike isort" paragraph entirely (it doesn't apply to isort 5, and hopefully nobody is using isort 4 anymore)
Move the "µsort operates on a strictly-parsed syntax tree" paragraph up, as it does a pretty good job explaining the structural difference (although "grammar object" feels awkward, maybe rewrite a little)
Make it clearer that we have less configuration, and consider that a good thing (lead-in to the case-insensitive paragraph)
Make it clearer that the four issues listed are fixed, but there are many more like them that still exist (maybe linking to this comment)
Mention our stdlibs are better https://gist.github.com/thatch/c75844447263df10b8f1c94811ad3525 and have better provenance/local reproducibility.  (among others, peg_parser is missing on 3.10, and sre is incorrectly force-included on 3.10)
Mention something about cython as being a non-goal (it's not strictly Python, and I doubt LibCST wants to parse it)

Is this expected?
--- a/backend/src/hatchling/builders/sdist.py
+++ b/backend/src/hatchling/builders/sdist.py
@@ -8,7 +8,7 @@
 from copy import copy
 from io import BytesIO
 from time import time as get_current_timestamp
-from typing import TYPE_CHECKING, Any, Callable
+from typing import Any, Callable, TYPE_CHECKING

 from hatchling.builders.config import BuilderConfig
 from hatchling.builders.plugin.interface import BuilderInterface
@@ -19,7 +19,10 @@
     normalize_relative_path,
     replace_file,
 )
-from hatchling.metadata.spec import DEFAULT_METADATA_VERSION, get_core_metadata_constructors
+from hatchling.metadata.spec import (
+    DEFAULT_METADATA_VERSION,
+    get_core_metadata_constructors,
+)
 from hatchling.utils.constants import DEFAULT_BUILD_SCRIPT, DEFAULT_CONFIG_FILE

 if TYPE_CHECKING:

@ofek yes.
The first change is due to µsort using case-insensitive, lexicographical sorting — the reasoning for this is the second section of the Comparison to isort section of the "Why µsort" page. We believe this is more consistent overall, and more predictable as someone looking for a specific name in the import.
The second change is because µsort attempts to reflow long lines to match black. If you don't use black, but would like to configure the target line length for µsort, you can just set black's line length in your pyproject.toml and µsort will obey that: https://usort.readthedocs.io/en/stable/guide.html#tool-black

Thanks! I do use black so I wonder why the line length isn't read

I think this is because µsort is only looking for the closest pyproject.toml, which in this case is backend/pyproject.toml, and that does not have a black config.  Adding a copy of the [tool.black] section to that file should get the behavior you expect from µsort. I'll open a task to consider how to handle nested config files like that.


Update da-s3-client, da-sfn-client to 0.1.28

About this PR
📦 Updates

uk.gov.nationalarchives:da-s3-client
uk.gov.nationalarchives:da-sfn-client

from 0.1.27 to 0.1.28
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "uk.gov.nationalarchives" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "uk.gov.nationalarchives" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, commit-count:1

Superseded by #63.


Add username tag to resources deployed to Azure

Modify ARM template to add username tag to deployed resource group

implemented in #1825


change SimpleJobFunc parameter

Codecov Report

Merging #4 (8265ee8) into master (37268ee) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
+ Coverage   99.03%   99.09%   +0.05%
==========================================
  Files           3        3
  Lines         208      220      +12
==========================================
+ Hits          206      218      +12
  Misses          2        2

Impacted Files
Coverage Δ

job.go
98.21% <100.00%> (+0.48%)
:arrow_up:

Continue to review full report at Codecov.

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


snakemake API call is not parallel when running for more cores

Snakemake version
6.7
Describe the bug
running snakemake from the command line with --cores 30 everything works well, it will parallelise the jobs, and run several at the same time. However, when I run the same command through the API (from python), with the command snakemake.snakemake(..., cores=30) until version 6.4 it works well (same way as command line), but since version 6.5 the API is broken: it will run jobs in sequence, one after the other, even if each job requires only 1 core, but 30 were provided.
Minimal example
6.4 and earlier:
command line: snakemake --cores 30: runs as expected, jobs running in parallel
API: snakemake.snakemake(..., cores=30): runs as expeced, jobs running in parallel
6.7 (6.5>=):
command line: snakemake --cores 30: runs as expected, jobs running in parallel
API: snakemake.snakemake(..., cores=30): runs not in parallel, rather in sequence, one job after the other. even if no job requries more than 4 cores.

I have the exact same issue. Dropping to 6.4.0 also fixes it for me.

Was wondering if there has been any progress on this since last year? @johanneskoester


Fixed Spelling Error

I have read through the documentation and found a few spelling error. Also fix the syntax to correctly reference the "Introduction" section.

Looks good!  Thanks for catching those!


Possible concurrency issue when using cupsctl while printing

...or how my /etc/cups/cupsd.conf became an html file.
The weirdest thing must have happened when I executed cupsctl WebInterface=no and cups stopped working.
After a couple failed restart attempts via systemd, I looked at /etc/cups and noticed my cupsd.conf had become an html file with "WebInterface no" appended to it.
$ sudo tail /etc/cups/cupsd.conf

<INPUT TYPE="SUBMIT" NAME="CHANGESETTINGS" VALUE="Change Settings">

                                                                    </FORM>

CUPS and the CUPS logo are trademarks of Apple Inc. Copyright © 2007-2019 Apple Inc. All rights reserved.

WebInterface no

At the time I issued the cupsctl WebInterface=no I was printing (and in fact I found a half-printed page in the printer later) so I guess this must be some weird concurrency thing?
Of course replacing cupsd.conf with cups.conf.default I was able to restart cups and restore its functionality.
I'm running cups 2.3.3 on Manjaro (the package version is 2.3.3-3).

Attaching the resulting cupsd.conf, in case you want to identify what html it is.
cupsd.conf.txt

I think this was a bug we fixed in OpenPrinting CUPS already...


Tests/formatting

Hey,
now all the tests are parametrised.
This is quite useful because it makes easy to quickly add more tests.
In addition, pytest generates exactly one test for each test case. This way you can see exactly which test case is causing the test to fail. That is very useful in case of nasty bugs ;-)

There are too many exceptions for me to add to SonarQube whitelist, would you mind swapping the address literals to constants, for example something like:
IPV6_ADDRESSES = [
    '2606:4700:4700::1111',
    ...
]

TEST_CASES_IPV6 = [
    IPv6(),
    IPv6(IPV6_ADDRESSES[0]),
    IPv6(IPV6_ADDRESSES[0], 80),
    IPv6(f'[{IPV6_ADDRESSES[0]}]:80'),
    IPv6(f'[{IPV6_ADDRESSES[0]}]:80', 8080),
]

That way I wouldn't need to manually approve a hundred different ones.

Oh yeah, apparently I've forgotten to replace pylint and flake8 in this project with Ruff. I should probably get to it.

There are too many exceptions for me to add to SonarQube whitelist, would you mind swapping the address literals to constants, for example something like:
IPV6_ADDRESSES = [
    '2606:4700:4700::1111',
    ...
]

TEST_CASES_IPV6 = [
    IPv6(),
    IPv6(IPV6_ADDRESSES[0]),
    IPv6(IPV6_ADDRESSES[0], 80),
    IPv6(f'[{IPV6_ADDRESSES[0]}]:80'),
    IPv6(f'[{IPV6_ADDRESSES[0]}]:80', 8080),
]

That way I wouldn't need to manually approve a hundred different ones.

Yes, no problem

I tried to do as you suggested and it is not working... Can you add the exception manually? Or can I do that?

I'll take a better look after work

It was nothing complicated, the test result simply included the quotes of the string literal. An easy fix was to make Python treat the IP address via repr instead of str in the test case.

can we not just add the the exception? The code will be much cleaner then....

Okay, fine, I'll spend the weekend telling Sonarcloud to ignore the false positives. You can revert the changes to the addresses.

Okay, fine, I'll spend the weekend telling Sonarcloud to ignore the false positives. You can revert the changes to the addresses.

Actually, I saw that you did already a lot in the code. Let's change the code ;-) I will do the rest this week
sorry for the misunderstanding

Here we are :-)

Not sure why the unit tests for the pull request aren't running, looking into it.

Ah, whatever. It already passes the tests anyway, so I'll force-merge this. Dunno why it's not running in the pull request itself.


Allow "watch mode" on npm scripts after ejecting

I have successfully ejected a create react app I created but now I want to do 2 things:

Disable wepack server because I need to continue development on another server (apache)
Allow watch mode on the compile scripts that come with React.

If this is already possible please include how to do this in the documentation.
Thanks, great work.

I don't quite understand what you mean by

Allow watch mode on the compile scripts that come with React.

I'm also not sure what "disabling" development server means.
Can you describe your desired workflow in detail?

Sorry for not being a bit more detailed.
Watch mode: simply put, to be able to do the equivalent of webpack --watch and have the files compile everytime a change is made
Disable Webpack Dev Server: in my particular case is used create-react-app to scaffold the UI, but now I need this to work on the actual server along with my server scripts (apache + intersystems cache database). So I won't be using/needing Dev Server anymore.
Hope this explanation is a little better. Thanks for the quick reply.

Do I understand correctly this is the same request as #1070?

Would you want watch to run in production or development mode? That is, would you want to see sources minified or not?

Ok, that sounds pretty much like what I need. For the record, I did search the issues for similar requests but did not see this one.
Is this in the works? If not, are there any workarounds to achieve watching  files?

Got it. Let’s track this in #1070 then. It’s not a priority right now but we’ll want to address this eventually.

Thanks for helping Dan, as always.
El El mar, 6 de dic de 2016 a las 8:35 a.m., Dan Abramov <
<EMAIL_ADDRESS>escribió:

Got it. Let’s track this in #1070
https://github.com/facebookincubator/create-react-app/issues/1070 then.
It’s not a priority right now but we’ll want to address this eventually.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/facebookincubator/create-react-app/issues/1172#issuecomment-265150847,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABNmSugU59XbsyW7kTOX8c6ZKAXl64OCks5rFWSygaJpZM4LFFym
.


Exception proposal

Avalonia 0.8
<Window xmlns="https://github.com/avaloniaui"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        xmlns:rxui="clr-namespace:Avalonia;assembly=Avalonia.ReactiveUI"
        mc:Ignorable="d" d:DesignWidth="150" d:DesignHeight="100"
        Width="350" Height="150"
        x:Class="MessageBox.Avalonia.Views.MessageBoxWindow"
        Title="AvaloniaAppTemplate">
   <Button Honest="No exception"/>
</Window>

Often when avalonia cant render window , it just ignores it, without any exception

I don't think this can happen with the current master because XamlIl should produce a compile-time error.

I often get compile time errors for problems with the XAML.  If it's a problem during resolution of a bound property that usually doesn't happen until runtime.  That may be getting better with the stuff @Gillibald mentioned.


🐛 Bug Report: broadcast type notifications can be missing by single user

📜 Description
When user A and user B are logged in Backstage at the same time, and a broadcast notification posted, if user A marks that notification as read, and then user B will unable to view the broadcast notification unexpectedly. And when user B navigates to /notifications route, there will be a 404 response received.
👍 Expected behavior
User B should also be able to view the broadcast notification.
👎 Actual Behavior with Screenshots
An error was thrown, and user B could no longer view the broadcast notification after user A marked it as read.
https://github.com/user-attachments/assets/3cfc60f6-0e7d-4087-a0c9-97b67a3d3061
👟 Reproduction steps
Two users logged in with different browsers, send a broadcast notification, user A marks the notification as read, and then user B navigates to /notifications route
📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

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

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, but I'm happy to collaborate on a PR with someone else

Mostly caused by https://github.com/backstage/backstage/blob/a236f099db650bd55e89b76fe2f93b63eafdda66/plugins/notifications-backend/src/database/DatabaseNotificationsStore.ts#L141-L150


Quick question about 16.6.0

In my script I have
adb shell "settings put global window_animation_scale 0.0"
adb shell "settings put global transition_animation_scale 0.0"
adb shell "settings put global animator_duration_scale 0.0"

Does that mean with 16.6.0 I don't need these lines anymore?

@d4vidi, any ideas?

That's exactly what it means


Consider re-providing Firefox addon now that Webextensions are out

It looks like the addon used to support Firefox previously but stopped to some times ago.
However, Firefox is now supporting Webextensions as their exclusive standard and provide a very close support, close to some small exceptions. Those should be documented on https://developer.mozilla.org/en-US/Add-ons/WebExtensions/Chrome_incompatibilities
about:debugging in Firefox is only reporting a few bunch of errors/warnings:
Reading manifest: Error processing background.persistent: Event pages are not currently supported. This will run as a persistent background page.
Reading manifest: Error processing version_name: An unexpected property was found in the WebExtension manifest.
Reading manifest: Error processing options_page: An unexpected property was found in the WebExtension manifest.

Regarding first line, background.persistent isn't currently supported by Firefox. It looks like setting this to false result in having events pages rather than background. Here this is a warning because Firefox does not support events one while it does for background.
Sources:

https://developer.mozilla.org/en-US/Add-ons/WebExtensions/manifest.json/background
https://bugzilla.mozilla.org/show_bug.cgi?id=1282971
https://bugzilla.mozilla.org/show_bug.cgi?id=1253565#c0

Second line seems to refer to a manifest attribute not part of the spec as of now and also seems to be a warning: https://bugzilla.mozilla.org/show_bug.cgi?id=1380219
Last line is a bit more tricky.
Spec is still talking about options_page (https://browserext.github.io/browserext/).
However, Google deprecated this in favor of options_ui with Chrome 40 (https://developer.chrome.com/extensions/optionsV2) and Mozilla is also supporting this last one, but without the chrome_styleattribute.
I also submitted the .crx extension on www.extensiontest.com
Here's the result:
Here's the results:
{
  "compat": [],
  "errors": [],
  "warnings": [
    {
      "message": "Unsafe assignment to innerHTML",
      "locations": [
        {
          "file": "js/sweetalert2.min.js",
          "line": 1
        },
        {
          "file": "js/sweetalert2.min.js",
          "line": 1
        },
        {
          "file": "js/sweetalert2.min.js",
          "line": 1
        },
        {
          "file": "js/sweetalert2.min.js",
          "line": 1
        },
        {
          "file": "js/sweetalert2.min.js",
          "line": 1
        },
        {
          "file": "js/sweetalert2.min.js",
          "line": 1
        },
        {
          "file": "js/sweetalert2.min.js",
          "line": 1
        },
        {
          "file": "js/sweetalert2.min.js",
          "line": 1
        },
        {
          "file": "js/sweetalert2.min.js",
          "line": 1
        },
        {
          "file": "js/sweetalert2.min.js",
          "line": 1
        },
        {
          "file": "js/sweetalert2.min.js",
          "line": 1
        },
        {
          "file": "js/jquery-3.1.0.min.js",
          "line": 2
        },
        {
          "file": "js/jquery-3.1.0.min.js",
          "line": 3
        },
        {
          "file": "js/jquery-3.1.0.min.js",
          "line": 3
        }
      ]
    }
  ]
}

About the jquery part, someone offered to get rid of it in PR #6
The other file is minified and my knowledges in JavaScript probably are a bit low to take care about it.
Finally, it's worth noting Edge does also allow WebExtensions now, but with some spcificities either.
PS: I tested the addon through the about:debugging tool by loading it as temporary addon (I didn't packaged it). It seemed to work correctly. I didn't have the notifications, but this could be caused by the fact this mode doesn't ask for permissions. This can be the cause of this missing feature.
For what it's worth I tested on Firefox Nightly (59), but stable channel currently on 57 shouldn't have many differencies on what I listed.

Hey, thanks for the feedback!
We dropped support for Firefox a few months ago after an update took 3 months in queue to be approved, and by the time it did there were already several updates done to the service. I'm not sure if their approval process for extensions has changed with these new standards you speak of, but I don't see any harm in trying again.

I guess there was maybe some lack on their side back at a time where XUL and Jetpack addons already had their death announced but Webextensions were clearly not mature enough.
Now that those are definitely dead with the release of Firefox Quantum/57, I think (and hope) they made what's needed for the amount probably way higher to addons to approve.
If this was going to take too much time, it's possible to ask news on #webextensions on their IRC.
I might also try to ask news to some people if this was needed.
Especially, now differences for developping addons on the different main browsers (Firefox, Chrom[e/ium], Edge, Opera) shouldn't be that big with Webextensions adopted as a common standard, and we can hope those differences will only decrease over time.

We'll see what we can do about it then. Thanks for reporting in!

Done.
https://addons.mozilla.org/en-US/firefox/addon/listen-moe-extension/


Why there are Required and nillable separated tags?

I readed that both tags are WIP, but i didn't understand why we have two tags with the same meaning and the verification for required is already checking if it's string nil ""

I think that less required fields means a cleanner code. Another possible improvement would be set Required as true or false by default.

can this be changed to support only nillable flag?

I'll have a look at this!

instead of required and nillable, we might be better off using the notnull validation


Configuring a modal

I'm not quite sure I understand how to configure a modal. In my current app, I have it working but not sure how I managed to do that.
These are my routes:
const routes = {

  Login: {
    screen: Login,
    navigationOptions: {
      header: null
    }
  },

  Main: {
    screen: Blurts,
    navigationOptions: {
      header: null,
      headerLeft: null,
      gesturesEnabled: false,
    }
  },
}

export default routes

In another file, I imported the routes object (see above) and passed it as the first argument in the StackNavigator
const CameraStack = StackNavigator({
  Camera: {
    screen: CameraView
  }
})

const Navigator = StackNavigator(routes)

const Root = StackNavigator({
  Navigator: {
    screen: Navigator
  },
  Camera: {
    screen: CameraStack
  }
}, {
  mode: 'modal',
  headerMode: 'none'
})

This somehow works. Does this mean a Modal configuration object should be the last object in the StackNavigator? Any help would be appreciated.
Could I somehow embed the Camera route in the routes object and get this to work?

Guys, is this repo still actively solving issues?

I will typically have a stack navigator set up as a modal as my root navigator, like you have here. That allows me to open a modal over all other screens.
You wouldn't be able to put the Camera route inside of routes because it would just be treated as a normal screen (like Login or Main). Hope that helps clarify things a bit

having a modal navigator at root will cause exactly the issue as described here: https://medium.com/4thought-studios/better-modals-in-react-navigation-56c649e0249 is there any way around this?


Bug with number input box in Firefox 73.0.1 (64-bit)

Up front, love what I've seen of the framework so far, thanks!
I'm going through the tutorial, and I'm at the Input Type paragraph here.  I'm running the exact script copy-pasted from the tutorial there.
When I test that page in Firefox 73.0.1 64-bit (no issue in Chrome/Edge), it behaves as expected for the first 16 characters.  Then, starting when I type the 17th character in the input box, it changes the number in the input box after I've typed it.
For example, I input<PHONE_NUMBER>123456 and it works as expected: I have<PHONE_NUMBER>123456 in the input box, and in the text box below.  When I type the 17th character, 7, as soon as the debounce expires, the input box changes from "12345678901234567" to "12345678901234568".
Worth noting that the input box appears to populate correctly (at least until the next digit is input after the expiration of the debounce cycle, at which point it populates with the updated contents of the input field).  That makes me suspect there is type checking of the input field itself going on that contains some kind of typing issue.  I'm not a web dev, though, so that's purely conjecture.

Just confirmed--same anomalous behavior on FF 68.6.0 on Android.

Thanks for flagging this. I'll take a look at this issue next week.
In the meantime, could you please add the line in1.debounce = 0 to input_demo and check. I would like to rule this out as a debounce issue.

Confirmed, same behavior with debounce = 0.
async def input_demo(request):
    wp = jp.WebPage()
    in1 = jp.Input(type='number', a=wp, classes=input_classes, placeholder='Please type here')
    in1.debounce = 0
    in1.div = jp.Div(text='What you type will show up here', classes=p_classes, a=wp)
    in1.on('input', my_input)
    return wp

Only difference is that instead of waiting for the debounce to lapse, the input character now changes almost immediately. The delay in the change is now the round-trip latency to the server (tested by increasing LATENCY in justpy.env), suggesting a server-side issue.
I'll do some more digging if I get a chance tonight.

Hi Eli, did you manage to check this? I get a similar issue on Firefox 68.10.0esr following the login example in the sessions tutorial.
As soon as I fill in a username and password, I get a Firefox pop-up requesting whether I wish to save the login & password, even though I did not yet submit the form.
This only happens on Firefox, no similar issues on Chrome or Edge. But I have never encountered similar issues with Firefox in other login forms.

Anybody has an idea where the difference between Firefox and the other browsers could originate from? I don't use Firefox so am not aware of its foibles.

would say fascinating ...

Confirmed with http://jpdemo.bitplan.com/ and Firefox 103.0.2

The problem relies in the type handling
https://github.com/justpy-org/justpy/blob/45041188267e692046ab26a53d40dc43969c1532/examples/tutorial/input/input_demo4.py#L12
With input type=number the value of the input is converted to a js Number and thus to a double-precision 64-bit binary format IEEE 754 but this conversion is only applied when updating the page after handling the event. Thus, the event handler receives the unconverted value which is then for certain numbers changed after updating the page.
Example (in python):
In[1]: x=111111111111111120.0
In[2]: int(x)
Out[2]:<PHONE_NUMBER>11111120

# number that can not be precisely described as float
In[3]: y=111111111111111121.0
In[4]: int(y)
Out[4]:<PHONE_NUMBER>11111120

So when setting the input field to<PHONE_NUMBER>11111121  the event handler receives<PHONE_NUMBER>11111121 and sets the div text accordingly but during the update of the page the value of the input field is converted to js Number and thus is displayed as<PHONE_NUMBER>11111120

see https://www.w3schools.com/jsref/jsref_max_value.asp


Initialization fails with async tree content.

With asynchronous tree content population, [tree] bound field is initially uninitialized before data comes back from backend. This causes exception in TreeInternalComponent.ngOnInit:
core.umd.js:3472 EXCEPTION: Uncaught (in promise): Error: Error in ./TreeComponent class TreeComponent - inline template:0:15 caused by: Cannot set property '_indexInParent' of undefined
TypeError: Cannot set property '_indexInParent' of undefined
    at TreeInternalComponent.ngOnInit (http://localhost:8081/node_modules/ng2-tree/bundles/ng2-tree.umd.js:5883:38)

Current workaround is to initialize bound field with some dummy TreeModel object - but it's ugly (well, it can be used as poor mans "Loading" indicator but you can better than that ;) ).
The latter reminds me that it'd be also great to have a initialization substitute content (like
To replicate the issue use something like this:
// 1 - import required classes and interfaces
import {TreeModel} from 'ng2-tree';

@Component({
  selector: 'myComp',
  // 2 - set [tree] attribute to tree object
  template: `<tree [tree]="tree"></tree>`
})
class MyComponent {
  // 3 - make sure tree object confirms to TreeModel interface
  public tree: TreeModel;

  constructor(private someService: SomeService) { }

  ngOnInit(): void {
    this.someService.getAsyncData().then(tree => this.tree = tree)
  }
}

@dandee thanks! Currently I am working on tests - cause it is hard to make changes without them - once finished - will fix this :)

@dandee It is fixed in version<EMAIL_ADDRESS>


Memory constrained GitHub Runners stuck in idle

Bug Description
We've had reports that github-runner-operator deployed runners occasionally stop receiving jobs.
I did some investigation and discovered when oom-reaper is invoked and kills the VMs, the service does not recover.
I was able to get the runners to recover by rebooting the system; restarting the charm and LXD did not fix the issue.
To Reproduce
N/A; from my investigation it would appear this is caused by the runners running out of memory and oom-killer.
Environment
late/edge
"charm": "github-runner",
"series": "focal",
"os": "ubuntu",
"charm-origin": "charmhub",
"charm-name": "github-runner",
"charm-rev": 4,
"charm-channel": "edge",
"charm-version": "45cfce1",
"exposed": false,

Relevant log output
https://pastebin.ubuntu.com/p/vQ887s42J8/

Additional context
From my investigation notes:
This turned out to be a memory issue.
oom-killer killed a bunch of the VMs on xlarge/2*
xlarge/2 was stuck in the following state:
[Thu Feb  2 15:07:36 2023] Out of memory: Killed process 3018113 (qemu-system-x86) total-vm:17906732kB, anon-rss:44264kB, file-rss:0kB, shmem-rss:6236532kB,
[Thu Feb  2 18:36:52 2023] Out of memory: Killed process 3016510 (qemu-system-x86) total-vm:17915952kB, anon-rss:43820kB, file-rss:0kB, shmem-rss:5447228kB,
[Thu Feb  2 22:25:03 2023] Out of memory: Killed process 548735 (qemu-system-x86) total-vm:18131036kB, anon-rss:45532kB, file-rss:0kB, shmem-rss:4885312kB,
[Thu Feb  2 22:41:01 2023] Out of memory: Killed process 547841 (qemu-system-x86) total-vm:18166784kB, anon-rss:48156kB, file-rss:0kB, shmem-rss:4816036kB,
[Thu Feb  2 23:05:39 2023] Out of memory: Killed process 2953011 (qemu-system-x86) total-vm:18049308kB, anon-rss:43928kB, file-rss:0kB, shmem-rss:2984476kB,
[Thu Feb  2 23:59:43 2023] Out of memory: Killed process 2951372 (qemu-system-x86) total-vm:18185036kB, anon-rss:44996kB, file-rss:0kB, shmem-rss:2964956kB,

03 Feb 2023 00:07:41Z  workload   maintenance  Reconciling runners`

Bizarre how it affected all the units.
I rebooted xlarge/2 first, then noticed there was an active runner and a test got scheduled.
None of the others recovered, I rebooted those, they also started registering as online.

Will close this for now, please let us know if this is still an issue


Android Build Fails Due to Deprecated API and Unsafe Operations in R8 build

Hello,
I encountered several issues while building my Flutter app using the salesiq_mobilisten library (version 6.3.0):
Deprecated API: Some input files use or override deprecated APIs. Here's the specific file:

MobilistenPlugin.java (path: C:\Users\Dell\AppData\Local\Pub\Cache\hosted\pub.dev\salesiq_mobilisten-6.3.0\android\src\main\java\com\zohosalesiq\plugin\MobilistenPlugin.java)

Build Failure Message:

FAILURE: Build failed with an exception.
ERROR: Missing classes detected while running R8. Please add the missing classes or apply additional keep rules that are generated in E:\work\projects\w-mobile\build\app\outputs\mapping\release\missing_rules.txt.                                                                                Chat$Department and 4 other contexts)celize.Parcelize (referenced from: com.zoho.livechat.android.modules.notifications.sdk.entities.SalesIQNotificationPayload$C

FAILURE: Build failed with an exception.
* What went wrong:
Execution failed for task ':app:minifyReleaseWithR8'.
> Compilation failed to complete

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

What was in missing_rules.txt
-dontwarn kotlinx.parcelize.Parcelize
Gradle:
id "com.android.application" version '8.3.0' apply false
id "org.jetbrains.kotlin.android" version "1.9.10" apply false

Please advise on how to resolve these deprecated API and unsafe operation issues, and any additional steps to fix the R8 build failure. Thank you!

@mohamedshasho , As there hasn't been any recent activity, we're closing this issue for now. If you need further assistance or would like us to revisit this, please don't hesitate to let us know, and we'll be happy to reopen it. In the meantime, check out our latest release.
Thank you for understanding. Please feel free to write back to us at<EMAIL_ADDRESS>if the issue persists.


docs: add detailed info how to use influx_inspect to check if I use TSM or TSI

System info: [Include InfluxDB version, operating system name, and other relevant details]
influxdb 1.5.4
Steps to reproduce:
$ docker run --name mytest -d influxdb:1.5.4
cb4256853c00c6f389af44fb3a27ce09fe72f8c02f7a2a39b986b0a90839549f
$ docker exec mytest influx_inspect report -help
Displays shard level report.

Usage: influx_inspect report [flags]

    -pattern <pattern>
            Include only files matching a pattern.
    -exact
            Report exact cardinality counts instead of estimates.  Note: this can use a lot of memory.
            Defaults to "false".
    -detailed
            Report detailed cardinality estimates.
            Defaults to "false".

$ docker exec mytest influx_inspect report
report: lstat : no such file or directory

Expected behavior: [What you expected to happen]
Some info about existing database contents, showing me whether I use TSM or TSI. The --help contents for influx_inspect should explain how to actually use the command to get some correct result, possibly by showing example command line.
Actual behavior: [What actually happened]
Trying to use influx_inspect report fails with unintelligible error. Similarly for influx_inspect dumptsm, I have no idea how to actually use it after reading the --help. I tried running both of them with various options, but never managed to get any meaningful info.
Additional info: [Include gist of relevant config, logs, etc.]
The influxdb 1.5 upgrade guide only repeatedly says "use influx_inspect buildtsi"; but the docs for the tool are super sparse and cryptic. Anyway, I can't find out a way how to call influx_inspect to check whether I have TSM or TSI indexes before running the upgrade (buildtsi), and to verify whether the upgrade worked successfully after the upgrade. Without being able to check this, it doesn't make much sense to me to even attempt the upgrade, as I don't know how to check if the call succeeded.

@akavel we have a dedicated docs repo: https://github.com/influxdata/docs.influxdata.com; I think that this issue should be filed over there. /cc @stevebang

This issue was moved to influxdata/docs.influxdata.com#1703


Estimates: Billed items section in detailed view

[x] Summary of billed items need to re-arrange

Remove amount received field
vat field show after sub total amount
Total amount= subtotal + vat

issue closed


Not logged in

Is it possible to check if the user is signed in or not?

This is quite old, I always forgot to answer.
However: If _spotify.GetStatus() returns null and the spotify webhelper + spotify is running, the user should be not logged in.
(Note this example is in the latest API Version)


PNASNet-5-Large --TypeError: forward() missing 1 required positional argument: 'x_right'

~/anaconda36/lib/python3.6/site-packages/torch/nn/modules/module.py in call(self, *input, **kwargs)
487             result = self._slow_forward(*input, **kwargs)
488         else:
--> 489             result = self.forward(*input, **kwargs)
490         for hook in self._forward_hooks.values():
491             hook_result = hook(self, input, result)
~/anaconda36/lib/python3.6/site-packages/torch/nn/modules/container.py in forward(self, input)
90     def forward(self, input):
91         for module in self._modules.values():
---> 92             input = module(input)
93         return input
94
~/anaconda36/lib/python3.6/site-packages/torch/nn/modules/module.py in call(self, *input, **kwargs)
487             result = self._slow_forward(*input, **kwargs)
488         else:
--> 489             result = self.forward(*input, **kwargs)
490         for hook in self._forward_hooks.values():
491             hook_result = hook(self, input, result)
TypeError: forward() missing 1 required positional argument: 'x_right'
pytorch1.0

What is your version of pretrainedmodels?

@Cadene
0.7.4
I test it in pytorch 0.4.1 and It can work.
I also use fastai.... Sorry, maybe it's the problem.

Pretrainedmodels should work fine with fastai. What was the problem?
On Thu, Dec 20, 2018, 15:58 Zhang Boheng<EMAIL_ADDRESS>wrote:

Closed #116
https://github.com/Cadene/pretrained-models.pytorch/issues/116.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Cadene/pretrained-models.pytorch/issues/116#event-2038588447,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEdvLkBLaJCkgCrFmleoAqC5oZ469EtDks5u66WSgaJpZM4ZcS3E
.

@Cadene
for example :
def pnasnet5large(pretrained=True):
model = pretrainedmodels.pnasnet5large(num_classes=1000, pretrained='imagenet')
model.conv_0 = nn.Sequential(OrderedDict([
('conv', nn.Conv2d(4, 96, kernel_size=3, stride=2, bias=False)),
('bn', nn.BatchNorm2d(96, eps=0.001))
]))
model.last_linear = nn.Linear(4320, 28)
model.avg_pool = nn.AdaptiveAvgPool2d(1)
return model
learn = create_cnn(
data,
pnasnet5large,
ps=0.5,
cut=-3,
path=path,
metrics=[acc],
callback_fns=[ShowGraph]
)
learn.model = nn.DataParallel(learn.model)
it gives :
~/anaconda36/lib/python3.6/site-packages/torch/nn/modules/module.py in call(self, *input, **kwargs)
487 result = self._slow_forward(*input, **kwargs)
488 else:
--> 489 result = self.forward(*input, **kwargs)
490 for hook in self._forward_hooks.values():
491 hook_result = hook(self, input, result)
~/anaconda36/lib/python3.6/site-packages/torch/nn/modules/container.py in forward(self, input)
90 def forward(self, input):
91 for module in self._modules.values():
---> 92 input = module(input)
93 return input
94
~/anaconda36/lib/python3.6/site-packages/torch/nn/modules/module.py in call(self, *input, **kwargs)
487 result = self._slow_forward(*input, **kwargs)
488 else:
--> 489 result = self.forward(*input, **kwargs)
490 for hook in self._forward_hooks.values():
491 hook_result = hook(self, input, result)
TypeError: forward() missing 1 required positional argument: 'x_right'
Another one is :
def senet154(pretrained=True):
model_name = 'senet154'
model = pretrainedmodels.dict[model_name](num_classes=1000, pretrained='imagenet')
w = model.layer0.conv1.weight
model.layer0.conv1 = nn.Conv2d(4, 64, 3, stride=2, padding=1, bias=False)
model.layer0.conv1.weight = torch.nn.Parameter(torch.cat((w, w[:, 2, :, :].unsqueeze(1)), dim=1))
model.last_linear = nn.Linear(512 * 4, 28)
model.avg_pool = nn.AdaptiveAvgPool2d(1)
return model
learn = create_cnn(
data,
senet154,
ps=0.5,
cut=-3,
path=path,
metrics=[acc],
callback_fns=[ShowGraph]
)
learn.model = nn.DataParallel(learn.model)
It can work.

@Cadene
It's the problem of create_cnn.
I read their codes and rewrite create_cnn, and everything works.
Thank u!

@Bonsen Can you expand on how you rewrote create_cnn? I am interested in this and it would be good to have it documented somewhere as well.


Rocket. <IP_ADDRESS> My opinion

Hello , I would like to say that I hated The New Rocket <IP_ADDRESS> , no plugin those I wore it works now, and most are not updated , I simply hated the new rocket again, and if you do not upgrade to old version that worked with old plugins will give up service to you , I am disappointed .

rocket <IP_ADDRESS> just killing my server now .... i can't wait while everybody ask for the plugins


Arcade expression is not being resolved when fields are being added from parent layer to related table pop-up in Public Notification

Arcade expression is not being resolved when fields are being added from parent layer to related table pop-up in Public Notification
Test web map: https://statelocaltryit.maps.arcgis.com/apps/mapviewer/index.html?webmap=b302600c5c884ab280e2ff6dc2e1fc78

@azizaparveen I was just looking at this on production and with the map you shared above it looks like it's working okay. https://webapps.maps.arcgis.com/apps/instant/notification/index.html?appid=3be62b2f598f4c368d924db1ab694121

@sarahmcdonald7 sorry I removed the arcade expression from the pop-up right before last release, I put it back again and now the Export function is failing with a console error
Try this map : https://statelocaltryit.maps.arcgis.com/apps/mapviewer/index.html?webmap=b302600c5c884ab280e2ff6dc2e1fc78
Test app: https://statelocaltryit.maps.arcgis.com/apps/instant/notification/index.html?appid=ba01c51f50d4458bb025db559f962ff1
https://statelocaltryit.maps.arcgis.com/apps/mapviewer/index.html?webmap=b302600c5c884ab280e2ff6dc2e1fc78


Add generic APIResponse Interface

Make the IAPIResponse interface generic to correctly set the Response value type.

All interfaces added in bcb8665. Now need to:

[ ] Add documentation
[ ] Order them in file
[ ] Add them to the npm package


figure out a policy for k3s version bumping

It can be confusing to users that we don't have an explicit policy for when we bump k3s versions. We should document it somewhere at least. E.g. "We bump support the last 3 minor releases of k3s and we bump whenever we release a minor version of Kairos" or some similar rule.

Let's arrange a call to discuss this

It's not only confusing but it's easy to not follow the rule because the auto updater just updates the latest one. Right now for example, if you check the k3s package you will notice that these are the current versions:

1.30.0
1.28.5
1.27.9

In reality, 1.29 should be there and 1.27 should be out

@cheese converting it into a pluing via sysext would be fantastic, and it would allow us to only maintain one set of images, I think we've discussed it already but I'm not sure what was the final decision around this if there was any @kairos-io/maintainers

Installing k3s is the job of the provider. I guess it could install it as a systext plugin but that would mean we still ship multiple images.
The discussed benefit would be there is we made it a bundle but then the provider is no longer responsible for the installation. Given bundle options can be passed down through the Kairos config, maybe that wouldn't be too different.
In other words, we are discussing removing the provider-kairos altogether, no?

The provider (as a concept) takes the kairos config and returns a new config back, having done some changes to the system in between. Maybe we can keep the concept but not use it for k3s? Thus, deprecate the provider-kairos but keep the concept around (for other providers)?

@jimmykarily good point, yeah I think the concept of provider needs to stay, also we have alpine where sysext is not a thing so maybe we cannot get rid of the 2 images issue in the end. But at least if k3s was offered in a bundle/sysext option, then we might not need to ship all those different versions but just the provider and then it makes sure to install the version you request or so

By specifying the K3S_VERSION at build time, all we do is make sure the right luet package is installed: https://github.com/kairos-io/kairos/blob/a5396bc38757651925e863b9569b00a613c8e920/images/Dockerfile.kairos-opensuse#L205
If and only if the luet package writes to paths that are writable at runtime, your suggestion is possible if we include all luet packages in the provider image somehow (without installing them at build time?) and let the user define whether remote fetching is also allowed or not (for versions there were not there are the time of building). If we make k3s a sysext, then it should be possible to install it at runtime.
Summarizing the steps:

Make k3s luet package installable at runtime (not writing to read-only paths)
Include all available k3s luet packages in the ISO (for offline installations)
Allow the agent to fetch newer k3s luet packages from the internet with a new option (e.g. allow-remote: true)
Change the provider to install k3s using luet packages (either the offline or those fetched from the internet)
Build only one provider image artifact for all k3s versions

The only problem will be that newer k3s luet packages will not be part of what we test in our CI before we release. We could have pipelines testing them as they are created but in any case, any release will only be tested to work with the shipped versions. Would the users risk installing a "non-supported" version then?

Relying on luet to install packages at runtime somewhat breaks the expectation of an immutable system.


ReportText vs TextFile

SHould these actually be 2 different objects?

I think we should probably just remove ReportText in favor of TextFile.  Now that the actual reports have changed formats, and are no longer really returned as evidence objects (instead they are attributes of the TurbiniaResult directly), I think it's safe to remove ReportText.

Quick update:
I think it's fine to remove the ReportText object, but we'll still want to keep or refactor the FinalReport.  To do this we'll have to make sure that all of the Tasks that have a ReportText Evidence object as output actually puts the content that was going into that object into TurbiniaTaskResult.report_data instead.  I think that's true for most cases though.


Support credential provided through environment variables

To provide initial support for Workbench, we have decided to support the specification of a server credential via environment variables. These are:
CONNECT_SERVER: the URL of the server
CONNECT_API_KEY: the ApliKey for the publisher
CONNECT_SERVER_NAME:  the nickname of the server. If not specified, use the hostname of the server URL.
Do not allow the deletion of a credential specified via the environment variables.
Do not allow overlap of URL or Name with other credentials.

This ended up not being a solution in Workbench. It worked on desktops since the user space .bashrc was being sourced, but in VSCode on the web it doesn't work.
We are switching the behavior to what is described in #2088


Azure Identity Iron Semester for Python

The Azure Identity library provides Azure Active Directory token authentication support across the Azure SDK. It provides a set of TokenCredential implementations which can be used to construct Azure SDK clients which support AAD token authentication.
This issue tracks the new features, reliability improvements, and the release of minor updates to Azure.Identity for the Python language during September - December 2020

This epic ended in Dec 2020


Multiple TextFields.

Greate component. I have  a small problem.
Lets say I have 3 textfields I'm populating with autoCompleteStrings = [String]. And on textchange everything works as it should.
But I also have 2 textFields I would like the tableview to function as a "picker" instead. So when the specific textField becomeFirstResponder i would like to populate the tableView with strings and the one on [String][0] should be pre-filled in the textField.
Edit: I can't seem to hide one textFields tableView when another textField becomeFirstResponer
Does it make sense?
Pre-Salamat, ;)

Hi @hugocarlmartin , you can actually do that by setting your textfield's text attributes to String[0] right after you populated autoCompleteStrings
And hiding suggestions when another textField was selected has been added. Thanks for reporting.


tree: cleanup

This PR contains some refactoring and reworking of the tree module. Some rough edges have been smoothed already.
With the rework we also close some long open TODOs and FIXMEs, as well as clearing the pathways for node identification/access which we want to have for more fine grained accesses.

I've added a draft impl for fetching nodes based on their pivots, although to finish this with a meaningful API we need to stabilize a version of Pivot Key from #29


Update API docs to use OAS 3

The Transaction Service's Swagger docs currently use v2 of the OpenAPI Specification, which doesn't work with some of the modern generative tools that only target OAS3. Upgrading will make the API easier to work with for local code generation (querystring filter classes, etc).

Thanks for the suggestion. It seems that we will have to migrate our dependency drf-yasg: https://drf-yasg.readthedocs.io/en/stable/readme.html#openapi-3-0-note


[Bug]: Loss computation with auto_mix_prec = True

Describe the bug
Hello,
I'm training some models with auto_mix_prec = True. But at some point, my model start getting nans. I tried to debug and found out that some tensors after forward pass end up with NaNs. I was checking online and I found this
So my question is:
Why in: https://github.com/speechbrain/speechbrain/blob/891318f5950c337bb951912bf64bd5973af7c908/speechbrain/core.py#L938
the loss computation is done outside of with torch.autocast() context manager? Following the PyTorch link above, it says that loss and forward pass should be inside autocasting.
thanks!
Expected behaviour
The context manager should be able to handle the NaNs.
To Reproduce
No response
Versions
No response
Relevant log output
No response
Additional context
No response

Possibly related? https://github.com/speechbrain/speechbrain/issues/1803

Hi @Adel-Moumen  and @AsuMagic
Thank you for quick reply. I'm running an encoder (conformer)-decoder transformer model, similar to the one for Librispeech. However, I'm running it for speech translation.
Could it be related to the Conformer architecture? (I don't see this 'underflow' error in the vanilla transformer)

Hmmm I don't think so. It might come from your data instead. Could you please check if you don't have outliers data (e.g., empty files?) ?

Actually, when I was debugging, I saw that in some cases is Tensors with NaNs produced by the compute_forward. I imagine they can be masked before being sent to loss computation.
However, why this only happens after some epochs? In that, wouldn't it should happen in the first epoch?

Hmmmmm when you are setting auto_mix_prec = False what happens ? Is it working?

Hello! I solved this issue, and it was not related to the model.
the problem goes to the way how SpeechBrain handle the --optimizer_step_limit passed from CLI.
I think this a bug. Let me describe the problem.

Imagine in your hparams.yaml file you have instantiated this scheduler:

optimizer_step_limit: 1000

lr_scheduler: !new:speechbrain.nnet.schedulers.WarmCoolDecayLRSchedule
   lr: !ref <lr_adam>
   warmup: !ref <n_warmup_steps>
   cooldown: !ref <cooldown>
   total_steps: !ref <scheduler_step_limit>
   decay_every: !ref <scheduler_step_limit>*0.1

but then, when you call the training script you call something like this:
 python train.py hparams.yaml --optimizer_step_limit 2000 What would happen is that the initial object lr_scheduler would use !ref <scheduler_step_limit> = 1000 instead of the new value passed in the CLI (ie 2000 steps). Then, during training when reaching epoch 1001, the LR would go below 0, ie negative. That was my issue. The LR was going negative.
I know this happens in when the arg is obtained from CLI but not updated everywhere:
2023-05-16 03:14:36,718 - speechbrain.core - INFO - Info: optimizer_step_limit arg overridden by command line input to: 2000

Pablo

Hey @pplantinga @Gastron, what do you think about this please?

So to me it seems that this is a problem with a specific YAML file, which references the value of optimizer_step_limit. If optimizer_step_limit is passed on the command line, it will not be treated as a YAML override, but as one of the run_opts.
However, it is easy to run into this issue, because the run_opts can also be specified in YAML. Their precedence is: defaults < YAML < commandline. We cannot simply add the run_opts to overrides, as they most run_opts will not be specified in YAML, and would instead raise an error (override for a non-existing key).
Perhaps we could allow the load_hyperpyyaml to take an additional run_opts argument, which would then be applied like overrides, but would not raise an error for non-existing keys.

Perhaps we could allow the load_hyperpyyaml to take an additional run_opts argument, which would then be applied like overrides, but would not raise an error for non-existing keys.

While I don't like the idea of adding a speechbrain-specific argument to hyperpyyaml, it could actually be nice (and sufficiently generic) to convert the current overrides_must_match boolean argument into a new argument, a second overrides dict that doesn't necessarily need to match. A little less cleanly, but more backwards-compatible would be to just add the non-matching overrides argument without removing overrides_must_match. I suppose we could add a deprecation notice on the overrides_must_match and remove it in the far future.
I'm not sure how hard this would be to add, as the current setup assumes all overrides will be treated in the same way. I'm also not sure who all would be able to tackle it right now.

Ah yeah good point about the separation of SpeechBrain and HyperPyYAML.
Another obstacle in changing this is that it would require recipes to call load_hyperpyyaml differently, which would potentially mean changes in every recipe. Maybe it doesn't make sense to change anything.
Perhaps the best fix for this is being careful with referencing the value of optimizer_step_limit in HyperPyYAML, preferring the pattern shown by @JuanPZuluaga  above with a separate scheduler_step_limit argument.

Thanks guys! So I guess that this issue should not be alive anymore? If you feel like that we should left it open then feel free to reopen it :-)