Do not create a random password if a admin_password is provided to the VM module

Module Name
vm
Description
Only create the random password resource if an admin_password has not been provided.
Example Terraform Configuration
resource "random_password" "this" {
  count       = var.admin_password != null ? 1 : 0
...
}

Might not be possible, see #119. Depends on how your custom admin password is generated though.

Might not be possible, see #119. Depends on how your custom admin password is generated though.

In our case, we already have our VMs with a random password stored in the state. With the current config, we will have two random_password resources, but only one that will be used.

@hknutsen  What value will not be known during the planning?

Consider the following example configuration:
resource "random_password" "custom" {
  length      = 128
  lower       = true
  upper       = true
  numeric     = true
  special     = true
  min_lower   = 1
  min_upper   = 1
  min_numeric = 1
  min_special = 1
}

resource "random_password" "default" {
  count = random_password.custom.result == null ? 1 : 0

  length      = 16
  lower       = true
  upper       = true
  numeric     = true
  special     = true
  min_lower   = 1
  min_upper   = 1
  min_numeric = 1
  min_special = 1
}

Assumptions:

random_password.default is the password generated by the vm module.
random_password.custom is the custom password you pass to the vm module.

Running terraform plan throws the following error:
$ terraform plan
╷
│ Error: Invalid count argument
│
│   on main.tf line 14, in resource "random_password" "default":
│   14:   count = random_password.custom.result == null ? 1 : 0
│
│ The "count" value depends on resource attributes that cannot be determined until apply, so Terraform cannot predict how many instances will be created. To work around this, use the -target argument to first apply only the resources
│ that the count depends on.
╵

This is because the random_password resource generates a password. Generated values are not known until the after the resource has been created by terraform apply.
Note that the error message specifies that you can use terraform apply -target=... to work around this, however it seems like a flaky workaround to me.

I would agree if you rely on the output result from another resource within the module for count. But in our case, we are referring to a variable that is either supplied by its consumer or defaults to null.

But if the provided value of that variable comes from a resource, you can run into the same problem.
Basically, the suggested implementation would completely remove the possibility of providing a custom password generated by Terraform.

Leaving the issue open for a little longer though, in case we come up with a solution 🙌

Might be worth taking a new look at this once Terraform 1.9 has been released:

The experimental "deferred actions" feature, enabled by passing the -allow-deferral option to terraform plan, permits count and for_each arguments in module, resource, and data blocks to have unknown values and allows providers to react more flexibly to unknown values. This experiment is under active development, and so it's not yet useful to participate in this experiment.

From changelog.

Might be worth taking a new look at this once Terraform 1.9 has been released:

The experimental "deferred actions" feature, enabled by passing the -allow-deferral option to terraform plan, permits count and for_each arguments in module, resource, and data blocks to have unknown values and allows providers to react more flexibly to unknown values. This experiment is under active development, and so it's not yet useful to participate in this experiment.

From changelog.

As of today, the "deferred actions" feature is still labed as an experiment in the alpha pre-release of Terraform 1.10, so we might have to wait a while for it.
I think we can close this issue for now and rather keep an eye on the changelog for when the "deferred actions" feature moves out from the experimentation stage.
Any other thoughts @equinor/terraform-baseline-maintainers?


Improve exporting of pre-trained models

Importing of a pre-trained models using import_meta_graph(...) has been implemented here.
However, this code seems to initialize the graph with the instantaneous parameter values and not the EMA filtered ones, and this degrades LFW performance a bit.
Figure out how to import a graph and initialize it with filtered parameter values.

This has now been fixed in validate_on_lfw.py.


Enable Platform session cache

Send session outputs to the Platform API.

I'll do the review on this one.


fix(2665): ensure infer_type_names sent only once

/claim #2665
fixes #2665

Changes are required in infer_type_name.rs not here.


Github orgs with many repos prevent the extension from saving data

Using latest Edge, etc.
I have about 300 github repos in my org. When I try to save my settings, this error pops up in the console, but the extension reports success:

Uncaught (in promise) Error: QUOTA_BYTES_PER_ITEM quota exceeded
    at chrome-extension-async.js:46:1
(anonymous) @ chrome-extension-async.js:46

Thanks for opening an issue @ryjones! Indeed, from the chrome.storage docs it looks like items in the storage must not exceed 8Kb (around 8k chars apparently).
My intuition would be that we can add a while loop before the storage.sync.set call below, that trims down the string if its JSON representation exceeds the limit.
https://github.com/orbit-love/orbit-browser-extension/blob/0c186eb48099aa3c2b54e1204522671a1d1f9e97/src/options/options.js#L96-L101


Clean text, docstrings and code

Thanks, changes look good but can they be applied against the base version of the notebook at sequences/P1C1_BehaviourAsPolicy/P1C1_Sequence3.ipynb. Tthe sequences/P1C1_BehaviourAsPolicy/student/P1C1_Sequence3.ipynb version of the book is generated by our ci scripts from that base notebook.

Sure @dcownden, I’ll fix it tomorrow.

@dcownden Done.

That's better, thanks. One more issues though, can you make the pull request to the integrate-feeback branch, or create your own new branch? our CI tools don't like PR's from forked main to original main.

Sure @dcownden, I'll do it in a few hours.


fix edit URL of license in docs

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.96%. Comparing base (cb10cfd) to head (4c47a23).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

:exclamation:  There is a different number of reports uploaded between BASE (cb10cfd) and HEAD (4c47a23). Click for more details.
HEAD has 3 uploads less than BASE

Flag
BASE (cb10cfd)
HEAD (4c47a23)

4
1

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #88      +/-   ##
==========================================
- Coverage   97.12%   90.96%   -6.17%
==========================================
  Files           6        6
  Lines        1250     1250
==========================================
- Hits         1214     1137      -77
- Misses         36      113      +77

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


Version 2.2.1 on Windows installs into wrong directory, causing file not found errors

I did python -m pip install --upgrade pipdeptree to install the package on a Windows 7 machine and it was installed into C:\Program Files\Python38\Lib\site-packages\pipdeptree-2.2.1.dist-info\pipdeptree-2.2.1.dist-info instead of just C:\Program Files\Python38\Lib\site-packages\pipdeptree-2.2.1.dist-info, causing file not found errors when trying to run it.

HELLO???

@kerstin921 Like any other python package, pipdeptree cannot control how/where it gets installed on a particular environment. It could be related to the behaviour of pip on Windows 7. I am not quite familiar with windows but 2 things come to mind for such problems: (1) admin privileges (2) sys.path
You may also try specifying the -t option when running pip install.

Hi,
I already checked privileges and path variables, they're all perfectly set as I recently did a clean, system-wide reinstall of the entire Python framework. Can you at least reproduce this behavior and confirm/refute this is not a bug on your side?

Unfortunately I can't reproduce the issue completely as I don't have access to a windows 7 machine. I can try to reproduce it inside a virtualenv on linux.
Can you provide following info?

Version of python and pip
Previous version of pipdeptree that you upgraded from.

At your end, can you also please try upgrading some other package to confirm that it's not your environment?

PS.: So, to answer your question, pipdeptree hasn't been upgraded from an older version either

Thanks for sharing the info. I tried reproducing this behaviour on linux with python version 3.8.0 and pip version 22.1.2, but couldn't. I tried both, installing the latest version with the --upgrade flag as well as actually upgrading from an older version of pipdeptree. In both cases, the files were copied to the expected location.
I can see pipdeptree in the pip list output, which indicates that pip is able to find the package in spite of the incorrect location. Can you share the complete error output of pipdeptree.
Also, pipdeptree doesn't depend on any other external packages, so it's unlikely that the version of any other package installed in your env could impact it.

Seems this was fixed, and was a broken installation.


Implement In-App Notifications View

Time Tracking: https://its-pro.ucsd.edu/browse/MA-58

use MyMessages API in codebase

[ ] First load view, need to refresh to see messages
[ ] Pagination
[ ] Ordering of messages


refactor into async/await style to see how it sits

I got nerd-sniped by this :)
My thoughts:

I think async/await is an improvement, but I'm curious if you do?
maybe more of an improvement is just extracting the find function?
try-catch all over the place to send different error messages, I think I'd look for a different way to achieve that if this was my code

Absolutely no ego about this, please decline if you don't like it better. I tested it one time with "Dallas" and it worked, and one time with "San Francisco" which it said was mostly water!

Thanks @RandomEtc! This is very interesting and helps me — someone who has literally spent maybe 8 hours total in Node — learn more about modern JS idioms through executable code.

I think async/await is an improvement, but I'm curious if you do?

I'm 99% sure yes. And you showed the critical missing bit I couldn't get from my spikes to try and do this on Line 44:
app.post('/random_point', async function(req, res) {

One of those common "oh boy, yeah that's what you get jumping in to real code" moments, but I totally didn't think to declare the request handling function as async, and so I couldn't make use of await work. This is great and I definitely find it cleaner.

maybe more of an improvement is just extracting the find function?

Big improvement, again seemingly made possible by having the top-level function context also being async? (Is that accurate?)

try-catch all over the place to send different error messages, I think I'd look for a different way to achieve that if this was my code

I like this. Cleaner. For different errors, I agree, this was definitely a quick-and-dirty job to build something usable. If it were more of a serious thing I'd think more deeply about both the dev/ops user context for error notification and the end-user ideal UX for these different errors.

I tested it one time with "Dallas" and it worked, and one time with "San Francisco" which it said was mostly water!

I also tested with SF a few times just to ensure it wasn't accidentally saying mostly water consistently, and got it to find a land place each of the 3 or 4 times, so no issue there.
I didn't get into this to learn more about Node/JS in 2019 (only used it because of Turf.js) but I definitely got a lot out of your PR, and I think internalizing the changes you saw can only help me in this weird year we find ourselves in. Thanks again Tom!


not working at all

Just dried it on an div with height 500px. Does not work for me at all, neither it throws any errors :/

Can you show code your code?
18 mar 2017 4:09 PM "Manuel Fink"<EMAIL_ADDRESS>napisał(a):
Just dried it on an div with height 500px. Does not work for me at all,
neither it throws any errors :/
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/NagRock/angular2-auto-scroll/issues/4, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AFiNMj16HS6WLrn7yrGYMnJZAqMs7qXgks5rm_OugaJpZM4MhcWD
.

Closing as cannot reproduce. Please reopen it with example code if it is still not working.


Remove the unwanted content form the Website

assign me this issue so, that I can remove all the unwanted content that has been added to the home page
So, please assign me this task under GSSOC'23 so that I can start working on this issue to fix the issue in your website

I would like to work on this issue. Please assign it to me.

@deepeshgupta3257 done.


Adopt rendered HMTL CSS styles from classic notebook

Right now the "rendered HTML" in markdown cells and the markdown plugin are not fully styled. The solution is to create a new CSS class, something like jp-RenderedHTML and port over the CSS styles from the class notebook, which live here:
https://github.com/jupyter/notebook/blob/master/notebook/static/notebook/less/renderedhtml.less
Assigning to myself :)

Fixed by #706.


UseSynthesizedInitializer rules does cover default initializers

The UseSynthesizedInitializer initialiser does not seem to cover default initializers - only memberwise ones.
See https://docs.swift.org/swift-book/documentation/the-swift-programming-language/initialization/#Default-Initializers
and
https://docs.swift.org/swift-book/documentation/the-swift-programming-language/initialization/#Memberwise-Initializers-for-Structure-Types
In code like this:
internal struct Foo {
    var bar: String = ""

    init() { }

    func myFunction() {
        _ = Self()
    }
}

the init() { } can be omitted, as it will be synthesized.
Classes will also get default initialisers under certain conditions.

Tracked in Apple’s issue tracker as rdar://126948248

So we actually re-implemented this rule in SwiftLint - see https://github.com/realm/SwiftLint/pull/4812
That implementation covers the case mentioned here, but also fixes a bunch of other cases. The unit test examples there should give some idea of additional coverage that swift-format does not have.


[Bug] Something wrong with sqlite

在这次提交中，我们将开发默认的数据库改成了sqlite，但看起来程序出现了一些报错。
94a969ac9c03b3f6645edeb5b5667bc84869a407
2024/02/12 10:19:10 E:/10 Github Projects/12 Opensource Project/1_OpenTeensBackend/dao/sqlite.go:9
[error] failed to initialize database, got error Binary was compiled with 'CGO_ENABLED=0', go-sqlite3 requires cgo to work. This is a stub
panic: Binary was compiled with 'CGO_ENABLED=0', go-sqlite3 requires cgo to work. This is a stub

你在现在的dev上改一下试试？
如果还不行应该是你开发环境的问题了


Add awesome-gpt-prompt-engineering

By submitting this pull request, I promise I have read the contribution guidelines twice and made sure my submission follows it.
⬆⬆⬆⬆⬆⬆⬆⬆⬆⬆

I will be happy to add it here when it gets added to https://github.com/sindresorhus/awesome - https://github.com/sindresorhus/awesome-chatgpt/pull/69#issuecomment-1572580340

Do you want me to create a PR on the other repo ?

@sindresorhus can you explain ?

You need to make the list good enough to be added to the main Awesome project first. This is a good ensurance that it's high enough quality to link to.


Tp2000 401

Why
Improve test coverage for the importer
Increate team familiarity and awareness of this part of the codebase
precursor to resolving and improving the importer
What
test as much of the importer as possible
Understand the separate functions, rename if needed, refactor and improve readability and testability
Notes
If the unit tests pass on CI and coverage for the importer has improved, this should be the main indication of success.
Function API should not change unless necessary
Existing tests should pass without change
All import functions should continue to work
Checklist
no migrations or dependency updates required

Codecov Report

Merging #665 (e702bd4) into master (01264eb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #665   +/-   ##
=======================================
  Coverage   93.63%   93.63%
=======================================
  Files         337      337
  Lines       21684    21684
  Branches     1679     1679
=======================================
  Hits        20303    20303
  Misses       1063     1063
  Partials      318      318

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


Hide dashboard on certain conditions

[x] When the interface is completely hidden (via numpad asterisk)
[ ] When other UI is displayed that overlays the actual game view (eg. settings)

The latter might be a bit more difficult to detect properly. Not sure.

Might want to re-consider how the hidden interface is handled (via asterisk), as this is easily accomplished by checking the visiblity of the FrameInGameBase control.

Hi 👋 I'm asking on behalf of Micka (as a moderator of his stream), he likes to play with HUD hidden while actually driving to remove distractions, but wants to keep Dashboard shown so viewers can still see the inputs.
Basically he installed Better Chat today, but noticed the chat was no longer hidden while playing, so I suggested to use the new checkbox in OpenPlanet itself to hide all UI with the TM toggle, but obviously this also hides Dashboard.
I can think of 3 possible solutions (this issue may not be the right place for this, but since it kinda crosses concerns between 3 repos, figured here is fine):

Option for in Dashboard to exclude itself from being hidden along with the global "hide overlay" option
Option in Better Chat to include itself from being hidden along with the TM toggle (ignoring the OpenPlanet "hide overlay" option and hiding anyways)
Exclude list in OpenPlanet itself to choose which plugins to not hide along with the TM toggle (seems complicated)

Obviously this can get sorta complicated, and I don't want to introduce undue complexity, but one of those solutions would be nice 😄

Thanks for the report! I think the third option makes the most sense. Can you open a separate issue for this here?

The first point has been implemented for a while now, so I will close this issue:

Option for in Dashboard to exclude itself from being hidden along with the global "hide overlay" option


Demo: Usage of the configuration facilities

inkview.h advertise for many functions related to "configuration":
iconfig * OpenConfig(const char *path, iconfigedit *ce);
int RefreshConfig(iconfig **cfg); //update memory copy config from his file
int SaveConfig(iconfig *cfg);
void CloseConfig(iconfig *cfg);
void CloseConfigNoSave(iconfig *cfg);

int ReadInt(iconfig *cfg, const char *name, int deflt);
long long ReadLongLong(iconfig *cfg, const char *name, long long deflt);
const char *ReadString(iconfig *cfg, const char *name, const char *deflt);
const char *ReadSecret(iconfig *cfg, const char *name, const char *deflt);
...

It would be interesting to have a demo dedicated to those functions. I could not find any public repository using this part of the SDK.

I finally found one application using those functions : https://github.com/JuanJakobo/Pocketbook-Read-offline/tree/main


Remove run-tests.sh

At the moment we run TestIntegrationScripts and run-tests.sh. These could really be done at the same time tin go test.

This was fixed in v0.8.5


Add gauge for network point stats

Fixes https://github.com/ecadlabs/tezos_exporter/issues/3

@jevonearth Whoops, I instinctively merged this (unintentionally), hope that's ok!


Adding Operating System Concepts 8th Edition Book

What does this PR do?
Adding Operating System Concepts 8th Edition (2009) Book PDF. Thank You!
Why is this valuable (or not)?
Explains Operating System Concepts very well
How do we know it's really free?
It's a direct link to pdf and it's downloadable.
It's a book pdf

This appears not to be an authorized distribution. Please see CONTRIBUTING.md guidelines.


Documentation does not show AOS downtime

[Enter feedback here]
As per feedback from Lane Swenka, when an environment is linked to PP AOS get restarted and there is a downtime of up to 10min per AOS
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: ae234693-3a0a-d25c-ecd1-abd01ad7580f
Version Independent ID: ae234693-3a0a-d25c-ecd1-abd01ad7580f
Content: Enable Power Platform Integration - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/power-platform/enable-power-platform-integration.md
Service: dynamics-365-cross-app
Product: ****
Technology: ****
GitHub Login: @laneswenka
Microsoft Alias: laswenka

Bug 810997 has been logged to address this issue.


Fixed navbar link

#3 Fixed navbar links (easing)

Perfect! Thank you. Make sure to add yourself under CONTRIBUTORS as well! :)


[aes, dv] Add SVAs for SCA countermeasures

This resolves #15705 and is related to #15433.
I've tested the assertion by playing around with the PRNG and they work correctly. Currently running a full regression to verify this doesn't introduce failures. So far things look good.

Thanks for your reviews, I've fixed the typos and will now merge the PR.


init devices presets example

make presets list, output field. Disable keyboard and add Copy to clipboard button

Closing for now -- will revisit for next release.


[HOLD for payment 2024-12-19] [$250] Split- Deleted workspace is still shown in recents list

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.69-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:
Precondition:

Account has at least two workspaces

Go to staging.new.expensify.com
Go to workspace chat
Split an expense in the workspace chat
Delete the workspace
Open FAB
Click Split expense under Quick action section
Enter amount > Next

Expected Result:
~In Step 9, the selected workspace will not display "Unavailable workspace"~
~In Step 11, "Hidden" will not be shown every time the selected workspace is clicked~
In Step 7, the deleted workspace should not be displayed in the recents list.
Actual Result:
In Step 7, the deleted workspace is displayed in the recents list.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/9282360a-c6d9-4293-9936-5e001bb4786a
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021864448103150318798
Upwork Job ID:<PHONE_NUMBER>150318798
Last Price Increase: 2024-12-04

Issue OwnerCurrent Issue Owner: @sakluger

I think the actual bug here is that the deleted workspace shows up in the recents list. We shouldn't show the deleted workspace in any selectable list for choosing where to submit or who to split with.
All the other weird behavior after selecting the deleted workspace is irrelevant because it shouldn't be there in the first place.

Hi, I'm Nicolay from Callstack - expert contributor group - and I would like to work on this issue.

Find a root case - will provide a fix today

Hey @Ollyws, could you please complete the BZ checklist before the payment date tomorrow? Thanks!

Thanks for the regression test steps. I added a step to verify that the deleted workspace did not show up in the list, but otherwise, looks great. 👍

$250 approved for @Ollyws

Thanks everyon!


RME bug with items 459, 460, 2249, 2250

Actual but with RME, nothing can be done at our side.
https://github.com/hjnilsson/rme/blob/64ef3b5e2ac25a6e1d1d3ab7db957e2c52bde5f2/source/map_drawer.cpp#L1132-L1141
It hardcodes items 459 and 460 as colored. This make our 459 and 469 borders full colored tiles and make our invisible stair 2249 also invisible at map editor, same for 2250 nothing special.

And also this:
https://github.com/hjnilsson/rme/blob/23a8fb15a1a90cf369261559536259400ff458ef/source/sprites.h#L24-L27


Add dismissable message on the demo page, that describes what's happening

Will need testing if users need this, for now closing.


Add index.html template

<!DOCTYPE html>
<html>
  <head>
    <meta charset="UTF-8" />
    <title>Résumé</title>
  </head>
  <body>
    ― Hello World! 😉
  </body>
</html>

The index.html file has been created and the template is added.


Management API and web interface overview

This moves the management API overview and the web interface documentation to the platform overview section.
Also UIs & APIs have been renamed to "Use the API".

All checks have been fixed.
@pipo02mix I left Organizations on the top level for now as a concept. The reason was that
a) I don't know how you can add sub pages to a page with content properly. They always appear at the end then.
b) Organizations are not that unimportant so I kept it prominent.


I'd like to join this project's community

Your public repo
https://github.com/mashbean/gov4git-identity-public.git
Your public branch
main

Approved

 Follow up
On Tuesday, 08-Oct-24 14:09:08 UTC by Gov4Git dev
@mashbean was added to the community.


update from upstream

chore(deps): update rust:1.75.0 docker digest to 2176747
chore(deps): update rust:1.75.0 docker digest to 166aa82
chore(deps): update rust:1.75.0 docker digest to ac8c4cb
chore(deps): update returntocorp/semgrep docker tag to v1.57.0
chore(deps): update actions/upload-artifact action to v4.2.0
chore(deps): lock file maintenance
chore(deps): update returntocorp/semgrep docker tag to v1.58.0
chore(deps): update actions/upload-artifact action to v4.3.0
chore(deps): update dorny/paths-filter action to v2.12.0
chore(deps): update npm to >=10.4.0
chore(deps): update dorny/paths-filter action to v3
chore(deps): update github/codeql-action action to v3.23.2
chore(deps): update alpine docker tag to v3.19.1
chore(deps): lock file maintenance
chore(deps): update returntocorp/semgrep docker tag to v1.59.0
chore(deps): update docker/metadata-action action to v5.5.1
chore(deps): update rust:1.75.0 docker digest to 503aee4
chore(deps): update rust:1.75.0 docker digest to e173089
chore(deps): update rust:1.75.0 docker digest to fb477b5
chore(deps): update returntocorp/semgrep docker tag to v1.59.1
chore(deps): update github/codeql-action action to v3.24.0
chore: use internal console, not the terminal for debugging

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
104

Relevant Lines:
148

💛  - Coveralls

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


Attached blob error message should match other resource types

Storage Explorer Version: 1.23.0
Build Number: 20220329.2
Branch: hotfix/1.23.1
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Monterey 12.1 (Apple M1 Pro)
Architecture ia32\x64
How Found: From running test cases
Regression From: Not a regression
Steps to Reproduce

Expand one storage account -> Blob Containers.
Create a blob container -> Attach the blob container via Azure AD or SAS.
Add the attached blob container to the Quick Access node.
Detach the attached blob container -> The QA blob container displays as broken.
Right click the broken QA blob container -> Click 'View Error...'.
Check whether it is easy to get the root cause from the error message.

Expected Experience
It is easy to get the root cause from the error message intuitive.
Actual Experience
It would be better to make the error message more intuitive.

Additional Context
Here is the screenshot for queues.

Verified this issue on the build main/20220414.17. Fixed.


error in image

layou-sitelen.png has the glyph in the wrong order for the bottom right side

I don't have the original GIMP file, so in the meantime I replaced the diagram with the typo with the sitelen pona photo. cb5af197f801993a316cdfcf9ac2fa4a278b4dce


Do you have a plan to  add  wiki pages?

As the title says.

Sorry, missed this issue, @mrtylerzhou. Currently we don't, but you're welcome to ask away and I'll add specific documentation to address your questions.


webpack  Uncaught Error: Cannot find module "."

Do you want to request a feature or report a bug?
bug

What is the current behavior?
Uncaught Error: Cannot find module "."
    at webpackMissingModule (index.js?3672:20)
    at _import (index.js?3672:20)
    at eval (index.js?3672:27)
    at Object../src/router/index.js (app.js:1038)
    at __webpack_require__ (app.js:679)
    at fn (app.js:89)
    at eval (main.js?1c90:1)
    at Object../src/main.js (app.js:1023)
    at __webpack_require__ (app.js:679)
    at fn (app.js:89)

If the current behavior is a bug, please provide the steps to reproduce.
1、clone https://github.com/rawbin-/vue-webpack-test
2、npm install && npm run dev
3、see the browser console

https://github.com/rawbin-/vue-webpack-test
What is the expected behavior?
the following code both run as expected
// router/index.js
import Vue from 'vue'
import Router from 'vue-router'

Vue.use(Router)

// this piece of code works
// const _import = (relativePath) => {
//   if (process.env.NODE_ENV === 'development') {
//     return require('@/components/' + relativePath).default
//   } else {
//     return import('@/components/' + relativePath)
//   }
// };

// the following code does not work
// Uncaught Error: Cannot find module "."
const _import = (relativePath) => {
  const fullRelativePath = '@/components/' + relativePath;
  if (process.env.NODE_ENV === 'development') {
    return require(fullRelativePath).default
  } else {
    return import(fullRelativePath)
  }
};

const HelloWorld = _import('HelloWorld');

export default new Router({
  routes: [
    {
      path: '/',
      name: 'HelloWorld',
      component: HelloWorld
    }
  ]
})

If this is a feature request, what is motivation or use case for changing the behavior?
changing the fullRelativePath for other reason
Please mention other relevant information such as the browser version, Node.js version, webpack version, and Operating System.
Node 8.11.1  MacOS High Sierra  10.13.4 tested on webpack 3.6.0  2.6.1,and both fails
related to this  https://github.com/webpack/webpack/issues/4921

full dynamic impports/requires are not supported. Also check the warnings/errors you get while compiling.

@sokra Do you think there could be a way to make the runtime error less cryptic?

@sokra
const _import = (relativePath) => {
  const fullRelativePath = '@/components/' + relativePath;

  if (process.env.NODE_ENV === 'development') {
    // this works
    return require('@/components/' + relativePath).default

    // Uncaught Error: Cannot find module "."  ???
    // return require('@' + fullRelativePath.slice(1)).default

    // console error: Uncaught Error: Cannot find module '@/components/HelloWorld'
    // return require(`${fullRelativePath}`).default

    // with warning: Critical dependency: the request of a dependency is an expression
    // console error: Uncaught Error: Cannot find module "."
    // return require(fullRelativePath).default
  } else {
    return import(fullRelativePath)
  }
};

If you use the webpack 4+ version, try installing CLI.
npm install --save-dev webpack-cli


how to make an event that serves to hide the HUD, and another event that does the same but in reverse

Describe your problem here.
I'm talking about hiding the life bar, the arrows, basically everything, and that there is another event capable of reversing the effect
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x86/x32
Did you edit anything in this build? If so, mention or summarize your changes.
No response

function onEvent(name, v1, v2)
	if name == 'HUD visibility' then
		setProperty('camHUD.visible', not getProperty('camHUD.visible'))
	end
end

function onEvent(name, v1, v2)
	if name == 'HUD visibility' then
		setProperty('camHUD.visible', not getProperty('camHUD.visible'))
	end
end

You can just replace 'not getProperty ('camHUD.visible')' with false

no? you can't turn it back on if i did

no? you can't turn it back on if i did

setProperty('camHUD.visible', true) --this should work

you do know that's doing the same thing i did?

Well I just found out that you can just set the hud's opacity by doing this
setProperty('camHUD.alpha', [int]);

cool but that makes the script longer

You can just replace 'not getProperty ('camHUD.visible')' with false

but wouldn't that just set it to false ? the point of
not getProperty('camHUD.visible')

so that it's something you can toggle, otherwise you would need a second event to turn it back on

Thanks! but couldn't you make a smoother transition instead of disappearing all at once?

Thanks! but couldn't you make a smoother transition instead of disappearing all at once?

function onEvent(name, v1, v2)
    if name == "HUD visibility" then
        if string.lower(v1) == "on" or v1 == "" or v1 == " " then
            doTweenAlpha("HUDvisibleTween", "camHUD", 1, v2, "sineOut")
        elseif string.lower(v1) == "off" then
            doTweenAlpha("HUDinvisibleTween", "camHUD", 0, v2, "sineOut")
        end
    end
end

v1 should be "on", " " or empty if you want to make the HUD visible, and "off" if you want to make it invisible.
v2 is for the duration of the tween, a smaller duration makes the tween appear or disappear faster.

THXXXX!!!!!


Add template filter to parse human-readable file size to bytes

Motivation for changes:
Some of the plugin, for example rss does not produce the torrent_size entry field, but for some sites, it actually contains human-readable size like 114.5MB, 14GB, or 1.91TB in the torrent title.
In some cases, we need to filter out the torrents that are larger or smaller than we expected size. In this case, a filter to parse human-readable string to actual bytes number is necessary.
Detailed changes:

Added a jinja filter parse_size  to support parse the string like 114.5GB to<PHONE_NUMBER>48 bytes int

Config usage if relevant (new plugin or updated schema):
tasks:
  download_filtered_size:
    rss: https://example.com/?token=xxxx
    accept_all: yes
    no_entries_ok: yes
    if: # rejects torrents size which are larger than 100GB
      - "title|re_search('\d+(\.\d+)?\s*\w+B')|parse_size > 100 * (1024**3)": reject

To Do:
Current PR is a proof of concept, if you have any ideas about it, welcome comment.

[ ] tests
[ ] document

I like the idea. Maybe we should remove the anchors from the regex so that less regex in the template itself is needed? And maybe we make an optional parameter to control whether it assumes SI units or not, e.g. 1 kb|parse_size == 1024, but 1 kb|parse_size(si=true) == 1000. kib would of course always be 1024.

Maybe we should remove the anchors from the regex so that less regex in the template itself is needed? And maybe we make an optional parameter to control whether it assumes SI units or not

@gazpachoking Updated. Please check again and feel free to comment :)

I thought of one more question... should it be case sensitive? I'm thinking people will only use this filter on fields that are very likely to have a size in it, so we probably don't need to be so strict about the casing.

I thought of one more question... should it be case sensitive? I'm thinking people will only use this filter on fields that are very likely to have a size in it, so we probably don't need to be so strict about the casing.

Updated. I've added an argument that can let user match case-insensitive, and false by default.
Case-insensitive matching will significantly increase the wrong match rate, considering a title like this, which are really common in most RSS feed:
Symphony No.9 - Ludwig van Beethoven [FLAC][96kbps][5.78G]

A case-insensitive matching will match 96kb instead of 7.58G.

That's a good point. We probably could have gone with it several iterations ago, but I can't help tweaking. 😄  What about a negative lookahead at the end of the regex to eliminate that kind of error? Something like (?![A-Za-z]). Or maybe just use a word boundary at the end \b

What about a negative lookahead at the end of the regex to eliminate that kind of error? Something like (?![A-Za-z]). Or maybe just use a word boundary at the end \b

I adjusted the regex with lookahead syntax, it should work without wrong match:
(?P<digit>\d+(?:\.\d+)?)\s*(?P<unit>[KMGTPE]?i?B(?!\w))

If you think there is no problem with this regexp, please reply and I will push the new commit later.

I believe this regex should work for 99% of entries. And for scenarios that this filter can't handle, users can chain the re_search and other jinja syntax to make it work for their scenarios.

Awesome, thanks!


Add clean-backups Command

Added a clean-backups command to manually prune old backups.
Related docs PR MinecraftServerControl/MinecraftServerControl.github.io#14
Resolves #327

looks good. thanks


Add scala and sbt versions needed for Gatling tests

These versions of scala and sbt are needed by the gatling plugin used by
the performance test project.

There's a sneaky bug thats bitten.
The depends_on only provides a solution for build ordering. The Dockerfile for sbt has an explicit version defined:
FROM onsdigital/jenkins-slave-scala:2.11.8
You may wish to parameterise the version of scala.

I'm going to take a different approach with the ONS specific config that means we could possibly remove some of this extra config. I will like to go through it with you and get your opinion sometime. Lets merge this for now, but I may need to revert at a later date.

No problem.
I missed the earlier question about building with later SBT versions.
Sorry, I have not built sbr-control-api or address-index-api with later sbt versions. Both of these use 0.13.13, and I haven't tried any sbt versions in the 1.x series.
Were you thinking of unifying the SBT version used to simplify the pipeline's needs in the short term?

That would be ideal. From what I seen it wasn't trivial. The desires of the developers is to use the latest version so this could be an exercise for them I guess

I've rebased from master and updated - just waiting for CI then will merge down


[CI] Correct port GHA bug

LGTM

Sorta wonder if once this is merged it will fix that permissions issue. Since the GH_TOKEN fix might have been part of that issue.

/backport SKIP release/v4.0

/backport SKIP release/v3.0


[Feature Request] Support alternative vector DB's other than Pinecone

Hi there,
Congrats on the launch! The library looks very clean, and I like the interfaces. I would love to see some alternative stores implemented, and would be happy to contribute some myself.
Is there anything I/we (other contributors) should be aware of when thinking about submitting a PR like this?
Thanks again!

Hey Ibrahim 👋
Support for more vector DBs and model providers is definitely high up on the priority list, and PRs are more than welcome 😄
I added some notes to the contributing guidelines on getting the dev env setup, but aside from that, I don't have any specific guidance.
When I find some time, I'll try to create issues tracking some of the low hanging fruit.
Thanks!


nixos/firefox-syncserver: enable recommendedProxySettings

Description of changes
The syncserver requires a Host header identical to the one provided by the client, so let's make it so.
I'm not sure if this should be done by (a) enabling recommendedProxySettings or (b) adding the single stanza that actually deals with Host alone. Thoughts?

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@pennae

kind of wondering why other existing setups worked fine without (we know of at least ours, and probably a couple others).

Maybe you've enabled recommendedProxySettings globally (services.nginx.recommendedProxySettings)?

did you get any errors?

Firefox kept getting 401 on its requests to /1.5/4/info/collections. It then took me a surprisingly large amount of time to coax the actual error from syncserver-rs (because it's normally built with any logging above the level if info not compiled in at all).


[Bug] "RuntimeError: Client Ray installation incompatible with server: client is 2022-12-06, server is 2023-06-27"

Search before asking

[X] I searched the issues and found no similar issues.

Note: In this documentation, this problem is mentioned. The solution is "In above cases, you will need to check if the client ray version is compatible with the images version in the ray cluster's configuration." However, I'm not able to solve the issue. Please see the description below.
KubeRay Component
Others
What happened + What you expected to happen
I am trying to connect to a ray cluster from a pod in the same cluster.
However, the default ray cluster is on Python 3.7 and ray 2.3.0.
My client pod, on the other hand, uses Python 3.9 and ray 2.5.1 (My client application requires the Python version to be above 3.9)
I tried to deploy a ray cluster using a different image, but I couldn't manage to find a Ray image that is compatible with Python 3.9 and ray 2.5.1.
 It is not clear which image uses Python 3.9 and ray 2.5.1.
Image pull failed when trying to pull older images.

Could you please help me with this? Thank you so much!
Reproduction script
To deploy the ray cluster, I followed the instructions on the quickstart documentation by executing the following commands:
! helm install raycluster kuberay/ray-cluster --version 0.5.0
! helm repo add kuberay https://ray-project.github.io/kuberay-helm/
! helm install kuberay-operator kuberay/kuberay-operator --version 0.5.0
! helm install raycluster kuberay/ray-cluster --version 0.5.0

I also tried
helm install raycluster kuberay/ray-cluster --version 0.5.1 --set image.tag=6a8d0f-py39-cu117
In the client pod:
#Python 3.9
import ray
ray.init(address="ray://<IP_HEAD_POD>:10001")

However, I got this error when trying to connect to the ray cluster.
"RuntimeError: Client Ray installation incompatible with server: client is 2022-12-06, server is 2023-06-27".

Hi, Ray client is currently not actively maintained by the Ray community. I strongly recommend you using ray job submit instead.

Are you saying the Python SDK is not actively maintained? From the documentation, it seems to be actively maintained though.
We need to use the @ray.remote decorator for our project.

I tried
ray job submit -- python script.py  , but got the following error:
ERROR utils.py:1403 -- Internal KV Get failed

Same question. Has this been solved?

what I find really confusing is that the error doesnt output the version but a date, which itself does not have a release on github nor PyPI

Hi @Tongyuang @chris-aeviator, as I mentioned https://github.com/ray-project/kuberay/issues/1265#issuecomment-1646165782, Ray client is not actively maintained by the Ray community. Please use ray job submit instead. Thanks!

Ray submit is not at all a replacement for the ray client. How can one utilize ray inside a running python job (e.g. in spark) in a way similar to how the client works with @remote

Am 27.09.2023 um 00:42 schrieb Kai-Hsun Chen @.***>:

Closed #1265 as completed.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

@chris-aeviator If you have any related feedback about the Ray client and ray job submit, you can open an issue in the Ray repository. Thanks!


Remove duplicate PrintLog.js file

This file is a duplicate of api/logging_api/models/PrintLog.js, no need for it.

very motivational!!!

Looking cool Allen!


update backup docs

ensure datetimes are datetimes

Coverage decreased (-0.03%) to 71.348% when pulling c8fe390a19b202ff1cc3579b537821fc8d26c2a8 on SzySteve:docs/backups into 6c335ad9d69e77d90c03f68d7ed93706ba900899 on linode:pending-docs-updates.

Coverage remained the same at 71.375% when pulling c8fe390a19b202ff1cc3579b537821fc8d26c2a8 on SzySteve:docs/backups into 6c335ad9d69e77d90c03f68d7ed93706ba900899 on linode:pending-docs-updates.


BUG: Different results on Linux/Mac and Windows

Expected behavior and actual behavior.
Attached is a geopackage with six polygons that are pairwise equal, meaning I have three geometry pairs. I want to check for equality with the within predicate of the STRtree, but get unexpected results on Linux and Max, and expected results on Windows.
Results on Windows (expected):
[0 0 1 1 2 2 3 3 4 4 5 5]
[0 2 1 4 0 2 3 5 1 4 3 5]
Results on Linux and Mac:
[0 0 1 1 1 1 2 2 3 3 4 4 4 4 5 5]
[0 2 1 4 3 5 0 2 3 5 1 4 3 5 3 5]
Steps to reproduce the problem.
import geopandas as gpd
from shapely import STRtree
gdf = gpd.read_file("...")
tree = STRtree(gdf.geometry.values)
left, right = tree.query(gdf.geometry.values, predicate="within")
print(left)
print(right)

Operating system
I have tested both locally and on push to GitHub (tests run on Ubuntu, Mac and Windows with python 3.11 and 3.10).
Shapely version
2.0.1. Tested with pip and poetry.
linux_windows.zip

I'm getting the expected results on Mac (M1) with GEOS 3.11.2, shapely 2.0.1 (built locally):
>>> tree.query(gdf.geometry.values, predicate='within')
array([[0, 0, 1, 1, 2, 2, 3, 3, 4, 4, 5, 5],
       [0, 2, 1, 4, 0, 2, 3, 5, 1, 4, 3, 5]])

What version of GEOS is available in your installed versions of shapely?  print(shapely.geos_version)
Because the smaller polygons look like they almost share an edge with the larger polygons, there might be a predicate robustness issue coming from GEOS that causes different results on different platforms.  I've seen cases where the intersection between overlapping polygons does not always return true for a within predicate of either of the original polygons, and I'm also finding that to be the case when I calculate the intersection of your polygons 1 and 3 and then check via shapely.within against the original polygons.

There isn't anything special that shapely is doing here; it is only applying the within predicate using GEOS to the initial results from STRtree.

I'm running GEOS 3.11.1.
It's difficult for me to test this in pure GEOS on other platforms than Windows.
But since you get correct results, it might be a GEOS issue.


403 when using DWD in appengine to use admin sdk

Hi guys! Great work, but running into some issues with the new auth flow.
I am going for unattended access- i.e. generating reports- to the admin SDK. I stepped away from the GAPPS API's for the last year, and came back to quite the change!
Attended flow is super simple, but unattended seems a lot more complex. I have tried: AppAssertionCredentials, ServiceAccount, and even using the flow covered in the admin sdk python quickstart.
Can you clue me in? How is unattended API access supposed to function now? When I try- using a DWD account with it's client ID granted scopes in GApps admin settings- I get a 403.
Appreciate the clarification. Does your team have a slack I can join for this stuff? The documentation is currently nigh useless due to being very  convoluted.

By unattended, do you mean accessing these APIs as the administrator of a domain?

The documentation is currently nigh useless due to being very convoluted.

Please gratuitously use the "send feedback" button on the docs - feedback is collected internally as bugs.

Thanks- yeah, I just continue to run into different and conflicting data.
I believe yes, that is what I need. The goal is to be able to have my application run API calls on a scheduled basis, without needing a user to manually accept the consent screen.
This is running in GAE fyi.
credentials = ServiceAccountCredentials.from_json_keyfile_name(             SERVICE_ACCT_JSON, scopes=SCOPES) credentials.create_delegated(sub=USER_EMAIL) self.service = build('groupssettings', 'v1',credentials=credentials) self.admin_service = build('admin', 'directory_v1',credentials=credentials)

Your code is missing just one critical piece. credentials.create_delegated returns a new set of credentials and doesn't modify the original object. So try this:
credentials = ServiceAccountCredentials.from_json_keyfile_name(
    SERVICE_ACCT_JSON, scopes=SCOPES)

delegated_credentials = credentials.create_delegated(sub=USER_EMAIL)

self.service = build('groupssettings', 'v1', credentials=delegated_credentials)
self.admin_service = build('admin', 'directory_v1', credentials=delegated_credentials)

Thanks! Still getting a 403, but that is progress- 'user email' - would that be a actual user, or would it be the service account email?

For the directory API, as far as I know it needs to be the email of an
actual admin in the domain. I'm not 100% sure on that. (We maintain the
client library, not the APIs themselves)
On Fri, Apr 15, 2016, 4:06 PM jessicawisesc<EMAIL_ADDRESS>wrote:

Thanks! Still getting a 403, but that is progress- 'user email' - would
that be a actual user, or would it be the service account email?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/google/oauth2client/issues/502#issuecomment-210677789

I'm closing this issue as you haven't responded in several days. If you still have questions, feel free to reply and we'll re-open the issue if needed.


Return flutter result on all public plugin methods in ios

Added flutter result calls to all methods exposed to flutter side. Without this, calling await on any of these methods cause the application to wait indefinitely as the Future will never complete.

Thanks!


Rename files

It be nice if there'd be an option to append the number of the post and the position of of the image in the post (1st, 2nd and so on) to the file name for sequential viewing.

This tends to be done automatically by tumblr's own system, not counting out-of-order reblogs.


Simplify Kafka integration using WithContext

Note that this is still not working, something is pushing messages in (context, item) order and not (item, context) order

This works now.


status

I want to know if this proyect is abandoned, because on packagist appears abandoned.

My impression was that robrichards/xmlseclibs provides more features and a larger community. Some people also have such special requirements that cannot be provided in a generic package. So they implemented their own solution.
On the other hand, the last release of robrichards/xmlseclibs was in April 2022. Is this package still maintained?
What do you think about it? Is it reasonable to maintain two similar packages for this use case?
I would be happy to hear your opinion.


Add official docker image

add   velacp dockerfile and ui dockerfile
add official docker image with gitlab ci

I can help if needed

@lxd5866 Sure. You are more than welcome to do it.


Update HTTPSession.java

Correcting problems regarding issue https://github.com/NanoHttpd/nanohttpd/issues/356

This might break during keep alive. Because HTTPSession constructor is called only once, but execute is called many times.

Well spotted @grragesh
Indeed, it cannot be done like this. v3.0.0 will add better ways to handle the protocol but it's still a mess :(

Thank you @LordFokas


deploy-app: add network policy handling, enable for preview

https://trello.com/c/H53K45It
This does a few things, but luckily should only have an effect on preview for now:

adds secondary .apps.internal routes to each of the preview api apps
adds a new step in the make deploy-app process which will attempt to create the network policies the newly created app is going to need. Do read the extensive comments on this, because it's quite carefully considered (!) and should hopefully work even in the face of simultaneous release procedures of both parties of an ingress/egress relationship.
these ingress/egress relationships are defined in the vars yaml on the "egress" side of the relationship by defining a egress_to_applications key. These should automatically get picked up as ingresses for the referred app using a little bit of yq magick. Again, here i've only defined these egress relationships for the preview stage and designed the relevant steps so that they're no-ops for stages that have nothing relevant defined.

Note this PR doesn't actually make any apps actually use the private urls yet - that comes later once we're sure bringing up the routes and policies works reliably enough. The idea is I merge this and then that allows me to hit "re-release all apps" for preview, which once I've done will allow us to rely on this new setup being persistent (when releasing from my machine without the code upstreamed, any random releases resulting from app merges will knock my alterations out).
I've also added a add-all-app-network-policies for panic-mode repairing the network policies if it turns out I haven't got the release procedure right.

As @katstevens noticed and I forgot to mention in the description - this PR only moves the APIs to internal addresses. This is because the frontends all live on the same domain under different paths, relying on PaaS' gorouter to do the actual routing to the correct frontend. This can't be done with internal connections because internal connections aren't actually handled on an http level at all - they're raw tcp connections.
We could possibly move the frontends to private addresses at some point if we actually set up the per-path routing on the ... router, directing requests to different frontend apps itself. But for now, they're protected by the basic auth nonsense, which we'll just have to keep in place for the time being.

So, to check my understanding by working through an example, doing a release of the api app would involve:

Creating the api-release app as normal, with cf push
Adding the following network policies, which are doubled up in case we're simultaneously releasing search-api:

cf add-network-policy api-release --destination-app search-api-release
cf add-network-policy api-release --destination-app search-api

cf add-network-policy search-api-release --destination-app api-release
cf add-network-policy search-api --destination-app api-release

Waiting 60 secs for api-release to successfully bind to services
Unmap the routes from api and waiting for any in-progress requests to complete
Carry on as normal (deleting old api app etc)

Questions arising:

Do we ever need to cf remove-network-policy? Would this be at the unmap stage?
Should we add the policies before or after the initial sleep, i.e. will the add-network-policy fail if the new .apps.internal route isn't there yet for whatever reason?
Do we already have yq installed on Jenkins?

Adding the following network policies, which are doubled up in case we're simultaneously releasing search-api

Not just that, it would also add corresponding policies for each of the FE apps that rely on it, and the router.

Do we already have yq installed on Jenkins?

Great question. I will find out.

Do we ever need to cf remove-network-policy? Would this be at the unmap stage?

They will die with the app as it's deleted.

Should we add the policies before or after the initial sleep

I don't know - I think we may have to experiment here, because on the other hand it's equally possible that the policies might take a few seconds to have an effect after creation...

I've improved the "error" messages, take a peek.

I'm going to merge this so I can at least get an idea about whether it works as I expect it to when used in anger.


Xeno:Deep learning library made only using Numpy

Xeno is a scalable deep learning library made from only using Numpy library, and do inculcates most of the features provided by famous libraries like Keras or sklearn. It is based on linear stacking of layers, and compared to more efficent DAG(Directed Acyclic Graph) stacking done by tf.keras, but this is the best I could!
For features head over to the ReadMe.rst!

@bhaveshlohana can you please check the PR for the merge.
Thank you.

merged


Add contextual backgrounds for .table-inverse

Addresses #19317

Dry-er fix coming in #19863. Thanks though!


nil hookManager in tokenizer.lua

Hi,
with the latest commit 56e80f6a0f39fc6267a6af11626705b7e53add5a, when I start a multi model server,
th tools/rest_multi_models.lua -model_config en2fr/server/rest_config.yml -port 9201

an error if sending the request,
Error in application: tools/rest_multi_models.lua:113: unicode error in line ./tools/utils/tokenizer.lua:340: attempt to index field 'hookManager' (a nil value)

no such problem with rest_translation_server.lua
hookManager is not included in global variable ?
PS: my text is already tokenized before sending to the server, so I choose space for mode
Thanks!

Hi @silverguo - let me know if this works.

Hi,
@jsenellart 11f20f9 is an empty commit ?

strange - yes. #480 is the one.

problem solved, thanks 👍


Existing chore error on New chore

Issues duplicate chore error when saving new chore

I'm able create new chore successfully

Let me test again on my phone. This error occurs mostly on my phone.


Natively support a date range picker

Use case
Date pickers are great, but sometimes users need to pick a range of dates spanning days, weeks, months, or even years.
It avoids having to use two fields like so:

(Yes I know it's the same date! It's just to show the two fields.)
Why would you use two fields when you could use just one?

Proposal
There's packages for date range pickers (https://pub.dev/packages/date_range_picker, https://pub.dev/packages/flutter_date_pickers) but it's not official and it's a part of the material design specifications. Android has this functionality so should Flutter. This should be natively supported in Flutter and as far as I can tell, it isn't.

(Source: Pickers)

Hi @Reprevise
Closing this issue as there is already duplicate #50133
Thank you for contribution

CC @darrenaustin

Related PR: https://github.com/flutter/flutter/pull/50546


[BEAM-4691] (do-not-merge-yet!) Move perf tests to a separate directory and rename them (conventionally)

This PR renames and reorganizes jenkins Job definitions in ./test_infra/jenkins directory
Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Spark

Go

---
---
---
---
---

Java

Python

---

---
---
---

Run seed job

Run seed job

Run seed job

Run seed job

Run seed job

I did a small test in order to check if we can refactor the code this way (split to directories, change to CamelCase). Below are some things I've found out:

I am not sure if I can change the job_00_seed.groovy name. Jenkins didn't change the glob pattern used to search for other job definitions when the name was Seed.groovy (like it wasn't able to find the seed job when it's name changed). Am I missing something here? Is path to seed job hardcoded in the workspace/jenkins config (see: the seed job run that cannot find jobs using the old "job_" pattern).

I cannot import CommonProperties everywhere when the directory structure is more than 1 level deep. This is described here:  LINK. I will try to simplify the directory tree to be one level and check if it works for our case.

I think we should split the CommonProperties file (former common_job_properties). It's a "god object" to me and knows way too much.

Another good thing of having directories would be the possibility to hide PreCommitBuilder in pre_commit directory (and so on) - so this is more than just clean structure.
CC: @kennknowles @swegner request for comments :)

This is awesome, thanks for working on this. A couple thoughts:

I am not sure if I can change the job_00_seed.groovy name.

I suspect this is a bootstrapping problem. i.e., remember that when you run the seed job, it's running the currently configured job from master, and generating and publishing new job definitions from it. The currently published job looks for job_*, so you might need a two-step process, i.e.:

Rename the files as planned, but also keep the original job_00_seed.groovy version, and in that file add both glob patterns
Run that job to generate the new seed job definition
Once your change is merged to master and the new seed job is successfully running, submit the change to remove the old seed job.

I cannot import CommonProperties everywhere when the directory structure is more than 1 level deep.

Bummer. thanks for investigating this.

I think we should split the CommonProperties file (former common_job_properties). It's a "god object" to me and knows way too much.

+:100: Any improvements we can make to this code is welcome. I recently stumbled up this page on Design patterns in Groovy
CC @Ardagan

@swegner @Ardagan  ok, today I learned a little bit more about the possibilities we have. I've done another PR to dig into this from a different angle (https://github.com/apache/beam/pull/5915).
I think we cannot have both: directories and common files. I successfully moved HadoopInputFormatIOIT to a separate directory but no matter what I tried (keeping CommonProperties in .jenkins and import or moving it to a "common" directory and then import) I couldn't run the seed job successfully.
One last idea that comes to my mind is to merge job files that have the same category. We would then have PerformanceTests.groovy, PostCommit.groovy, PreCommit.groovy, etc. I consider this idea because:

we could have multiple jobs per one file
we could extract some handy methods for creating jobs per each category. Such methods could be hidden in corresponding files (eg. method for creating a post-commit job would reside in PostCommit.groovy class)
because we're using groovy here, methods for creating jobs could accept a map as a parameter. Such map can be very readable and clean. We do something like this in FileBasedIOIT
we could keep stuff like CommonProperties etc. where it is now and use it as before. At the same time, category specific methods can remain hidden in categories' files (eg. kubernetes related methods could be hidden in a PerformanceTests.groovy file).

The drawbacks I see:

it may be harder to find a specific job in a common file
it may be easier to break many things at once due to common code (IMO, we're vulnerable to this either way)
???

WDYT? I ask for your opinions first because you recently worked on some refactoring and maybe have some thoughts in this matter.

I am also considering this as an alternative solution, but it would require much more effort (maybe worth it): https://jenkins.io/doc/book/pipeline/jenkinsfile/

Hi Łukasz,
Thank you for looking into this. Out of what I've seen so far:
If we stay in current JobDSL world:
I like your idea to generalize code for common job categories and define jobs via methods that accept map as parameters. If we can extract that map into separate file, that would be even better.
Current amount of files is just too huge for amount of work that is done. In the state of pre- and post- commit world it already moves in the direction of splitting job definition into logic and data files. I believe finishing this work would be a good thing to do.
If we move to jenkinsfile world:
I believe it is worth at least looking into before rejecting. Main reasons I would vote for switching to jenkinsfile is that it is jenkins native way of doing things.
Good write up is mentioned on job dsl wiki.
My opinion is that we should give jenkinsfile a try and make decision based on estimated amount of work required.
One of the starting points can be auto-convert plugin that converts free-style jobs to jenkins pipeline.
We can first try it on combination of forked repo and local jenkins docker instance to be safe.

@Ardagan1 thanks for all the ideas and links you posted! I agree with all your points.
I saw on the mailing list that you have a tutorial on spinning up Jenkins via Docker - really looking forward to this! While we have it it will be totally safe to test the options that are left (Jenkins files) and decide which way is best.  I'll stay tuned.
Meanwhile, I'm closing this PR (nothing more to do here).

@lgajowy The doc is work-in progress. Dumped docs in my fork for now, will move them to corresponding wiki once we come up with one. Please see it here:
https://github.com/Ardagan/beam/blob/JenkinsDoc/devdocstemp/jenkins.md
Feel free to update doc if you find any bugs.


Upgrade search app to Elastic Search 5.4

This upgrades elastic search, and fixes indexing so that it works on 5.4.
Also integrates the new 5.4 search code in ext. I'm not 100% convinced this shouldn't live in the .org codebase, but just following existing convention of where the docsearch code was.
Test failures are unrelated to this change.

Is it possible to upgrade latest 6.x?

@safwanrahman that breaks a lot of what we do, so I want to keep things here until we have more time to do a full rewrite. This is an incremental step.

I've rebased this on master here: https://github.com/italia/readthedocs.org/tree/upgrade-es-rebased
It fails for me against ES 5.6.8 with the following:
{u'index':
  {u'status': 400, u'_type': u'page', u'_index': u'readthedocs', u'error': {
    u'caused_by': {
      u'reason': u'For input string: "de3a71b6edf5001ece51f4ea16ca4758"', u'type': u'number_format_exception'
    },
    u'reason': u'failed to parse [id]', u'type': u'mapper_parsing_exception'
  },

I've done some work here: https://github.com/italia/readthedocs.org/tree/upgrade-es-rebased
Now we are indexing again, updated the search regarding filtering and faceting (now called aggs) to what ES 5 expects.
Still broken:

search for single project returns nothing
searching on the homepage returns the following error. Fun fact is that lang is already a keyword :unamused:

TransportError(400, u'search_phase_execution_exception', u'Fielddata is disabled on text fields by default. Set fielddata=true on [lang] in order to load fielddata in memory by uninverting the inverted index. Note that this can however use significant memory. Alternatively use a keyword field instead.')

Closing this, hoping to be replaced by #4183


Fix "strip trailing whitespace"

TBR
This might fix the flaky "strip trailing whitespace" preference.

:+1:


Fix write response metadata field(measure/stream)

Fix

[ ] Add a unit test to verify that the fix works.
[ ] Explain briefly why the bug exists and how to fix it.

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Fixes apache/skywalking#.
[ ] Update the CHANGES log.

Codecov Report

Merging #330 (afee42b) into main (9552688) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #330      +/-   ##
==========================================
- Coverage   40.59%   40.57%   -0.02%
==========================================
  Files         110      110
  Lines       11775    11775
==========================================
- Hits         4780     4778       -2
- Misses       6526     6528       +2
  Partials      469      469

Files Changed
Coverage Δ

banyand/liaison/grpc/stream.go
2.15% <0.00%> (ø)

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


🐞 [Bug]: pricing calculator: balance less than 1 TFT is acceptable

Is there an existing issue for this?

[X] I have searched the existing issues

which package/s did you face the problem with?
Dashboard
What happened?
https://app.testlodge.com/a/26076/projects/40893/runs/757884/run?run_suite_id=1250709&selected=33254510
Steps To Reproduce
No response
which network/s did you face the problem on?
Dev
version
038404b
Twin ID/s
No response
Node ID/s
No response
Farm ID/s
No response
Contract ID/s
No response
Relevant screenshots/screen records

Relevant log output
.

will fix the test case to accept zero.


arch: arm: core: cortex_a_r: enable the VFP unit on boot for FPU_SHARING

The FPU is already disabled by the z_arm_svc function when the first thread starts. Therefore, disabling the FPU at boot is unnecessary for lazy FPU; instead, it must be enabled to handle floating-point instructions before the lazy FPU works.
This fixes #79585.

@ithinuel @bbilas , this can still make it to the 4.0 release as it is a bug fix. Can you help review.

@ithinuel @bbilas , this can still make it to the 4.0 release as it is a bug fix. Can you help review.

I had nothing to do with this bug.

@ithinuel @bbilas , this can still make it to the 4.0 release as it is a bug fix. Can you help review.

I had nothing to do with this bug.

Sorry meant to ping @bbolen

It looks ok to me but I don’t know the cortex-a side very well yet. @bbolen do you remember why was this only enabled when not sharing the FPU?

@mmahadevan108 @bbilas @ithinuel @bbolen Hi, I've checked with our board and everything looks good. Can we go ahead and merge this PR?


Planes ignore transform

Bevy version
0.4
Operating system & version
Latest Void Linux
What you did

Spawn cube with size 1.0 at (0,0,0)
Spawn plane at (0,-5,0)

What you expected to happen
I expect the plane to be below the cube
What actually happened
The plane bisects the cube (as if it were at (0,0,0))
Additional information
Specifying a plane with translation (5, 3, 4) for example will still spawn it at (0,0,0)
Screenshot:

Minimum code to reproduce:
use bevy::prelude::*;

pub fn setup_world(
    commands: &mut Commands,
    mut meshes: ResMut<Assets<Mesh>>,
) {
    commands
        .spawn(PbrBundle {
            mesh: meshes.add(Mesh::from(shape::Plane { size: 2.0 })),
            transform: Transform::from_translation(Vec3::new(0., -5., 0.)),
            ..Default::default()
        })
        .spawn(PbrBundle {
            mesh: meshes.add(Mesh::from(shape::Cube { size: 1.0 })),
            transform: Transform::from_translation(Vec3::new(0.,0.,0.)),
            ..Default::default()
        })
        .spawn(PbrBundle {
            mesh: meshes.add(Mesh::from(shape::Cube { size: 1.0 })),
            transform: Transform::from_translation(Vec3::new(1.,1.,0.)),
            ..Default::default()
        })
}

#[bevy_main]
fn main() {
    App::build()
        .add_plugins(DefaultPlugins)
        .add_startup_system(setup_world.system())
        .run();
}

As a control case, can you try to reproduce this error on master? I haven't been able to recreate this error on the master branch.

As a control case, can you try to reproduce this error on master? I haven't been able to recreate this error on the master branch.

Appears to be fixed on master, closing.

Appears to be fixed on master, closing.


chore(ci): trigger browserstack iff key is set

:white_check_mark: Build karma 2563 completed (commit https://github.com/karma-runner/karma/commit/1e4c0435f8 by @johnjbarton)


Remove btn-base class

Current Behavior
Currently, the btn-base class gets suggested with autocomplete when typing btn although it shouldn't be necessary to get a button of the correct default size, the class should be removed.
It should also be removed in the docs:
https://github.com/skeletonlabs/skeleton/blob/3f2287605da10b0dffc693523f5d0852cfd7b04d/src/routes/(inner)/elements/buttons/+page.svelte#L33
Steps To Reproduce
Got to:
https://github.com/skeletonlabs/skeleton/blob/3f2287605da10b0dffc693523f5d0852cfd7b04d/src/lib/styles/elements/buttons.css#L34
The class is included and will therefore get suggested during autocomplete
Anything else?
No response

Removed as part of: https://github.com/skeletonlabs/skeleton/commit/d94732d702c43bd030e67bf382dfbb7b2f1b9a54


[Angular 10] Using *ngIf breaks layout order

Environment

CLI: 7.0.7
Android Runtime: 7.0.0
NativeScript-Angular: 10.1.0
Angular: 10.1.0

Describe the bug
Using *ngIf over an element/component breaks layout order unless the element/component is wrapped in another layout.

To Reproduce
just try:
<StackLayout>

</StackLayout>

Expected behavior
First label should be rendered before Second label.
Sample project
https://github.com/mohammadrafigh/dynamic-component-issue
Additional context
I'm not sure if it just happens with StackLayout or not but as a workaround wrapping First label like this fixes the order problem:
<StackLayout>
    <StackLayout>

    </StackLayout>

</StackLayout>

If it helps, the problem occurred after upgrading to Angular 10.0.0 but since the core module was in "rc" I thought this might be a known issue and would be fixed in final release, So this may be an inconsistency problem with Angular rendering behavior and Nativescript runtime.

This is also a bug with Angular 9, see here: https://github.com/NativeScript/nativescript-angular/issues/2176

Same problem here. Appeared after upgrade to Angular 10 and NS 7.0.

This issue is now fixed with @NativeScript/angular 10.1.3 published now 👍

Hi @NathanWalker. I'm facing this issue in a large messaging application. I can't update to Angular 10 yet because there are plugins with no NativeScript 7 compatibility. Shouldn't this be the case to have a fix for this issues also available in the @nativescript/angular 9 package?

@rickwalking can you try this release: https://github.com/edusperoni/nativescript-angular/releases/tag/8.21.2 ?


unable to run

I'm getting following error
ankush@Lenovo:/media/Study/Projects/showandtell/models/im2txt$ bazel-bin/im2txt/run_inference   --checkpoint_path=${CHECKPOINT_PATH}   --vocab_file=${VOCAB_FILE}   --input_files=${IMAGE_FILE}
INFO:tensorflow:Building model.
INFO:tensorflow:Initializing vocabulary from file:
Traceback (most recent call last):
File "/media/ankush/Study/Projects/showandtell/models/im2txt/bazel-bin/im2txt/run_inference.runfiles/im2txt/im2txt/run_inference.py", line 85, in
tf.app.run()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 48, in run
_sys.exit(main(_sys.argv[:1] + flags_passthrough))
File "/media/ankush/Study/Projects/showandtell/models/im2txt/bazel-bin/im2txt/run_inference.runfiles/im2txt/im2txt/run_inference.py", line 55, in main
vocab = vocabulary.Vocabulary(FLAGS.vocab_file)
File "/media/ankush/Study/Projects/showandtell/models/im2txt/bazel-bin/im2txt/run_inference.runfiles/im2txt/im2txt/inference_utils/vocabulary.py", line 50, in init
assert start_word in reverse_vocab
AssertionError

@ankush-06
Where did you place the checkpoint files?
Can you run ls -l --block-size=K in your Pretrained-Show-and-Tell-model folder?

Thanks for replying so soon.
Can you share your full error log (the previous one was truncated)?
Which tensorflow version are you using?

I'm using ubuntu 16.04, python 2, tensorflow 1.2.0
Here is a screenshot of the error
https://image.ibb.co/dvSY2k/3.png

Seems like there's an issue with the word vocabulary. Are you using the word_counts.txt file given here?
Run cat word_counts.txt | head.

ashish@Ashish-Lenovo:/media/ashish/Study/Academics/BTP/Projects/showandtell/models/im2txt/Pretrained-Show-and-Tell-model$ cat word_counts.txt | head
a 969108
 586368
 586368
. 440479
on 213612
of 202290
the 196219
in 182598
with 152984
and 139109

i solved the problem, but results are not accurate. Any suggestions to improve performance ?

Can you try the 1M like mentioned here? Keep us updated on that thread. :smile:
Closing this issue because it's solved.

To be really honest, i could'n really solved that issue :) but i somewhat overcame that. The problem was that the program was not able to locate the "word_counts.txt" file. I tried everything. I was getting frustated. So, i ran the commands in a separate python file("xxx.py"):
with tf.gfile.GFile("word_counts.txt") as f:
word_map = list(f.readlines())
word_map = [line.split()[0] for line in word_map]
then i imported this word_map list in the vocabulary.py file.
from xxx import word_map
reverse_vocab = word_map