Can I configure global IP address where printer is installed in windows?

As far as I've found out, to use the printer programatically in a node project, one has to install the printer globally and then do "add printer" in windows using the IP address that ipp-printer is installed in globally. However, the issue is that I have to change my node code every time I run the code on a different machine. Therefore, is it possible to configure ipp-printer's IP address when it is installed globally? or do I have to work with the default random IP it gets installed on? If it is configurable, then I can probably create a batch file to install the printer on a certain IP and not change the node config variable each time it is used on a new system.

My mistake. It was never the case that one had to globally install the ipp-printer to run it in a program.


Update apache-jena-libs to 4.3.2

Updates org.apache.jena:apache-jena-libs from 4.2.0 to 4.3.2.
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!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.apache.jena", artifactId = "apache-jena-libs" } ]

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

Superseded by #12.


Document common ingressShim.extraArgs use case in chart

What this PR does / why we need it:
Lots of new users don't realize:
(a) They need to create a Issuer/ClusterIssuer themselves
(b) They need to tell ingress-shim the name via extra-args
This PR adds a comment to the helm chart values.yaml to address these issues.
(Ideally the helm would create an ClusterIssuer for you by default, and set these options, if you specify and email address to use with LE.)
Release note:
NONE

Release note:
NONE

/retest

@munnerz the test failures appear unrelated to this change?

The quick-verify failure is down to this change - you need to run ./hack/update-deploy-gen.sh as you've changed the chart version.
The e2e test failures are flakes though yep.
/test e2e v1.8

/lgtm

Hurray! I thought I'd never get these two lines of comments past the red tape! 🎉


New File - Python Filetype Removed After Program Update

Issue Type: Bug
Using the menu 'File', then 'New File', I used to have three options: Text File, Python File, and Jupyter Notebook.  When the software auto-updated from v 1.69.2 to v 1.70.x (or whatever it was at the time, this was a few months ago) it removed the entry for Python File and despite my best efforts in searching for information, and asking for help in a reddit post, I have not found a way to add that filetype back into that menu.  I had a backup of my laptop so I was able to restore that version of the program, but on other computers where I didn't freeze updates the same thing happened; that New File option is gone.  If I allow the software to update it will remove the option, so I have auto-updates disabled at the moment on my laptop.  In my opinion I consider this a bug as the software update should not have removed that setting.
Really, all I want to know is, step by step, how to add that back into the New File menu.  Thank you!
VS Code version: Code 1.69.2 (3b889b090b5ad5793f524b5d1d39fda662b96a2a, 2022-07-18T16:12:52.460Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 x 2808)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.86GB (8.70GB free)

Screen Reader
no

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

dax-language
jia
0.0.9

rainbow-csv
mec
2.4.0

python
ms-
2022.10.1

vscode-pylance
ms-
2022.7.40

jupyter
ms-
2022.6.1201981810

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.9

powershell
ms-
2022.6.3

(5 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
pyindex848cf:30577861
nodejswelcome1cf:30587006
fc301958:30595537
2e4cg342:30596373

I tried to get the image to display in the bug report here but all I can do is get a link to the image...
Screenshot of New File menu before software update

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.72.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.72.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

The bug report is based on the fact that updating the program removes functionality which I would like to keep.  At this point in time I would rather not update the software.  Thanks.

Text File: Built-In
Python File: Python

Based on your screenshot, this is not built-in functionality.  You probably have an outdated Python extension that's not compatible with the current VSCode version and that needs to be updated too.

Based on your screenshot, this is not built-in functionality. You probably have an outdated Python extension that's not compatible with the current VSCode version and that needs to be updated too.

Thank you for your reply.  I have a couple additional questions; I'd appreciate your help:

How would you suggest I determine what's causing the issue?  I really liked having that capability in the menu.
Is there a way I can manually add file types to the 'New File' menu?


Add origin point in original CRS to global and delc/delr attributes

Fixed in https://github.com/axiom-data-science/modflow2netcdf/commit/d28d990d1214a4338054a16e4e0812af3dd87733


Animate the animation (3, 6, 9, 12..., hours)

The ▷ of "Control | Now « – ‹ – › – » ⊕Grid▷HD" looked promising.
However instead of showing me how a typhoon moves 3, 6, 9, 12..., hours from now, it instead stopped the winds completely.
Therefore consider adding an additional button, which would be just like the user hitting a  "›" button every five seconds.

Of course you would then need a slider, so the user could adjust the seconds, not just 5.
And also smooth out any jumps from a three hour change in position.
And also add a reverse mode.


tflite_runtime pip package compiled from source does not run

System information

OS Platform and Distribution:
Ubuntu 18.04 running in Windows 10 WSL
TensorFlow installed from:
Binary, however the tflite_runtime installed from source.
TensorFlow version:  2.1.0
Python version: 3.6
GCC/Compiler version: 7.4.0

Describe the current behavior
Compiled the tflite_runtime from master using:
./tensorflow/lite/tools/pip_package/build_pip_package.sh
and installed:
pip install --upgrade tensorflow/lite/tools/pip_package/gen/tflite_pip/python3/dist/tflite_runtime-2.1.0-cp36-cp36m-linux_x86_64.whl
Trying to run load an tflite  file:
python -c "from tflite_runtime.interpreter import Interpreter; Interpreter('foo.tflite')"
gives the following error:
Traceback (most recent call last):   File "<string>", line 1, in <module>   File "/home/fresve01/.local/lib/python3.6/site-packages/tflite_runtime/interpreter.py", line 203, in __init__     _interpreter_wrapper.CreateWrapperFromFile( AttributeError: module 'tflite_runtime.interpreter_wrapper' has no attribute 'CreateWrapperFromFile'
Describe the expected behavior
The tflite file should load without error.
Also the README.md file with build instructions should be updated since there is no swig dependency any more.
Standalone code to reproduce the issue
./tensorflow/lite/tools/pip_package/build_pip_package.sh
pip install --upgrade tensorflow/lite/tools/pip_package/gen/tflite_pip/python3/dist/tflite_runtime-2.1.0-cp36-cp36m-linux_x86_64.whl
python -c "from tflite_runtime.interpreter import Interpreter; Interpreter('foo.tflite')"

Hi nemo42,
I'm working on this -- meanwhile, would you try changing following code pieces?
on tensorflow/lite/tools/pip_package/setup.py
diff --git a/tensorflow/lite/python/interpreter.py b/tensorflow/lite/python/interpreter.py
index e3d7d04be1..7015ffd670 100644
--- a/tensorflow/lite/python/interpreter.py
+++ b/tensorflow/lite/python/interpreter.py
@@ -35,15 +35,15 @@ if not __file__.endswith('tflite_runtime/interpreter.py'):
   # rule.
   # pylint: disable=g-inconsistent-quotes
   _interpreter_wrapper = LazyLoader(
      "_interpreter_wrapper", globals(),
      "tensorflow.lite.python.interpreter_wrapper."
      '_pywrap_tensorflow_interpreter_wrapper')
   # pylint: enable=g-inconsistent-quotes

   del LazyLoader
 else:
   # This file is part of tflite_runtime package.
-  from tflite_runtime import interpreter_wrapper as _interpreter_wrapper
+  from tflite_runtime import _pywrap_tensorflow_interpreter_wrapper as _interpreter_wrapper

on tensorflow/lite/tools/pip_package/setup.py
diff --git a/tensorflow/lite/tools/pip_package/setup.py b/tensorflow/lite/tools/pip_package/setup.py
index 0895e30255..5a7fa0bed8 100644
--- a/tensorflow/lite/tools/pip_package/setup.py
+++ b/tensorflow/lite/tools/pip_package/setup.py
@@ -160,7 +160,7 @@ LIB_TFLITE = 'tensorflow-lite'
 LIB_TFLITE_DIR = make_output('libdir')

 ext = Extension(
-    name='%s._interpreter_wrapper' % PACKAGE_NAME,
+    name='%s._pywrap_tensorflow_interpreter_wrapper' % PACKAGE_NAME,

Hi nemo42,
I'm working on this -- meanwhile, would you try changing following code pieces?

Hi teijeong,
Applying the patches above seems to have solved the issue, thanks!

@nemo42
Please close this thread if it solves your question. Thanks!

Good to hear that! Closing this.

Actually this change is not applied to the repo yet, please wait to close this issue until the change makes its way to the code.

Fixed by https://github.com/tensorflow/tensorflow/commit/6a701314351845ecbc6641ecd2f68e21b582c3f8


Update project dependencies

Closes #103

A bit unsure what's happening here, cause it's only the lock file that's changed. How come they weren't any changes to package.json?

You don't need to change the package.json to update the project dependencies. When you run npm update, it updates the dependencies following the constraints specified for the major/minor version.


Power from Beyond advances when boarding a ship

Mission stage advances inappropriately upon boarding a ship as part of a space based encounter
Details
When searching for a temple as part of a Power From Beyond mission, if there is a space based encounter where you enter a ship, the quest advances to the "Follow Distortions" stage. Normally this advancement triggers once you land on the planet.
I have seen this behavior from boarding a bounty hunter ship as well as part of the Juno's Gambit encounter.
This did occur with the 0.0.3 SFCP.
Game Version
V<IP_ADDRESS>
Language
English (EN)
Have you recreated this issue on a new game?
No, I cannot recreated this issue.
Reproduction Steps

Travel to a planet that is specified by a Power from Beyond mission.
Have a space based encounter that has you enter a ship in orbit of said planet
Watch the Power from Beyond Mission advance when the ship is boarded

(Also listed as "No, I cannot recreate this issue on a new game" as I didn't try it. But I did have this occur in two different ways in my current NG+ save)

I can verify from the way to quest is scripted that this will happen.

We might be able to ignore locations with the following keywords:

LocTypeShip [000141F1]
LocTypeShipInterior [000141F2]
LocTypeShipExterior [000141F3]
LocTypeOrbit [00016504]

It might also be that we can change the RequireExactLocation property on DefaultQuestChangeLocationScript to True (as it is currently false). Really, we need a save in orbit over a relevant planet with a ship that can be boarded to test.


Short solution needed: "Redis pubsub example" (python-redis)

Please help us write most modern and shortest code solution for this issue:
Redis pubsub example (technology: python-redis)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

Redis pubsub listen example


Unable to re-expand a collapsed page preview for a chapter

Describe the bug
When expanding a chapters pages in the book view and re-collapsing it, one is not able to expand it again.
Steps To Reproduce
Steps to reproduce the behavior:

Go to '/books/foo' that has at least one chapter with pages in it
Click on 'x pages' to expand the page preview for the chapter
Click on 'x pages' to collapse the page preview for the chapter
When clicking on 'x pages' again, nothing happens

Expected behavior
One should be able to expand the pages list of that chapter again.
Screenshots
--
Your Configuration (please complete the following information):

Exact BookStack Version (Found in settings): BookStack v0.27.4 (lastest release)
PHP Version: 7.2
Hosting Method (Nginx/Apache/Docker): Docker behind Nginx reverse proxy

Additional context
--

Can confirm, From my testing issue will often appear on Firefox, Not so much on chrome.
Thanks for reporting @avnkx .


Render frontend template on the django app

It was more or less done when the Countdown template #49 was completed by JesseyH. I reviewed his branch and it was more or less similar to my implementation.


Make 'mime' field in Body optional

This does not seem like the ideal way to do this since Option have to be
parameterized with a type, see the introduced example code for specific
example.
I see a couple of alternatives:

Add a method for unsetting the mime type
Clearly document what mime type is the "no mime type". This should
possibly be part of an example above set_mime

The first as a semver minor change, and the second is a semver patch
change.


Do I need dir in my form?

When I create an upload template do I have to include the dir form field in my form for it to work? :smile:

I've done some testing and no, it doesn't seem like you need to have this field in your form. However you will still need it to be present in the database table you are uploading file to.
Thanks, I will update my documentation.


[5.2] Delete unnecessary unit tests

Still tweaking this. Don't merge yet, please.

Why?

@GrahamCampbell https://twitter.com/taylorotwell/status/706142963701719040
TL;DR: The methods seeInElement, etc. are already being tested with integration tests in FoundationInteractsWithPagesTest which are better.

Integration tests don't replace unit tests. They just compliment them. Having complete test coverage from both is never a bad thing.

Meh, just read Taylor's tweet. I'll let him decide.
My vote here is :-1: for removing them. We should only remove them if updating them in the future becomes a mess.

I feel this is out of control: https://github.com/laravel/framework/pull/12639/files#diff-5b5766814f4efa52aba6c06a81febb90L156
The amount of code needed to mock something related to the crawler is absurd. TBH I was unable to test all the methods I wrote with that style.
That's why I came with this solution that looks much better: https://github.com/sileence/laravel-framework/blob/tests/5.2/deleteUnitTests/tests/Foundation/FoundationInteractsWithPagesTest.php
Easier to write, understand, maintain, etc. I want other developers who decide to contribute to the integrated test package to write these kind of tests instead of the other ones.

I feel this is out of control: https://github.com/laravel/framework/pull/12639/files#diff-5b5766814f4efa52aba6c06a81febb90L156

I agree there.

The amount of code needed to mock something related to the crawler is absurd. TBH I was unable to test all the methods I wrote with that style.

I also agree that in those cases, the tests are a bit silly.

Maybe just remove the tests with crazy mocking, and keep the others?

@GrahamCampbell all of them have crazy mocking and are covered in the other file.

@GrahamCampbell all of them have crazy mocking and are covered in the other file.

I don't agree. Ones depending on mockInput are fine, but mockOptGroupSelect is crazy.

I agree basically all of these are pretty over-mocked.


Improve the Error Fingerprint algorithm

Looks like this was added to the upstream.  I try to find some time to put together a PR
https://github.com/errbit/errbit/commit/9c40727dc527c43d2f2c3dedbcf94b9dec2aaa4c

Thanks @captmorgan  :+1:
I've made a few specific improvements to reduce duplicate reports in cases I've come across, but I really like the idea from upstream of having the strategy being configurable.


Vm shutdown

monitor_xenVM & xen_api

Could be done as a function but we need to run the monitor service in the background for multiple VM's at the same time as a daemon. I explored involving threading in the code and making each thread (a function call) a daemon but wasn't sure of the performance and feasibility so had to take the easy way out. Would you suggest that we go ahead and use threads?

Will see if I can figure out the exact string. As xl clears all the domains and properties on shutdown there was no way we could have the exact string and there is little to no documentation on that behavior while it still does get triggered on any random string, will see if we can figure out the exact values.

pyxs
Yes we should be able to replace all places where xl or xenstore is invoked. Will work on that.

@richiedjohnson Richie could you please check the changes and let me know if we need to have any further modifications.

The Monitoring has been implemented as a part of xen-api itself but as threads.

The whole http post request for the update of status to web server has been replaced by a ZMQ solution with a PUSH PULL pattern. Now a server runs on the local network through which the communication is enabled thus reducing latency and also enabling parallel acknowledgements based on the number of slave (pull) workers.

Closed stale Pull Request.


[Feature][Tenant]Support users to choose whether to turn off multi-tenancy function

Describe the feature
Support users to choose whether to turn off multi-tenancy function, mainly to solve the problem of 1. Some users do not need multi-tenancy in real scenarios, and use one user to perform tasks; 2. Some users cannot apply for sudo permission

支持用户选择是否关闭多租户功能,主要是为解决1.部分用户真实场景不需要多租户，使用一个用户执行任务；2.部分用户申请不到sudo权限
todo list

[ ]  Supports turning off multi-tenancy function. The page cannot select tenant related function, and the tenant code is default

[x]  #4128 Complete the code sudo usage specification. All script tasks execute commands using sudo-u username command

good job

close by #4217 #5129


Upload-it: a demo Jupyter notebook extension

Split off from the uploader pull request.

Hi Yu, I have addressed the comments and now I am pushing this change as I see you "approved" stamp.


Issues with Buffer/Uint8Array when using this module with browserify

Hi!
This issue might be misplaced, sorry if that's the case. But lets start here and see where we end up.
So my use case is using hyperswarm-web in peermaps-web and I run into the error buf.readUInt32BE is not a function which in turns happens inside hypercore which depends on the uint64be module, which in turn tries to make the buf.readUInt32BE() call.
So the problem is that there is a mismatch between Buffer and Uint8Array somewhere and I tracked it down to the .read() method that calls b4a.alloc() https://github.com/hypercore-skunkworks/b4a/blob/main/browser.js#L33-L37
If I just rename browser.js inside b4a to _browser.js then everything just works since browserify replaces Buffer properly. Not sure if the fix should be inside b4a or if we should just remove the b4a dependency altogether and replace with Buffer.

After some clarification on Discord, using b4a is by design and it was a choice to base everything on the typed array interface in hyperdrive but this doesn't play nicely with hypercore@9. The solution was to use random-access-memory@3 until hypercore@10 will be in the dependencies for hyperdrive.


Add GenerateFromPasswordWithLimit and Calibrate.

Time and memory limit the derivation, and allow calibration - no cgo
needed.

Alright: firstly, thanks for submitting this PR!
As you've probably figured, the reason for mentioning cgo in the README is that it gives us some visibility into available memory (at a cost of platform-specific code, and it's still guesswork) so that we don't enable package users to shoot themselves in the foot.
Most of my comments here therefore revolve around that:

Don't let package users do Dumb Things (tm). This package exists because the scrypt library in the std. lib. doesn't have the friendly API that the bcrypt lib. has.
Sane defaults wherever possible.
Keep the API as small as we can.

Hopefully my comments make sense in light of these goals.

@tgulacsi I didn't get a notification on your changes, so sorry for missing this! Thanks for incorporating my feedback. Appreciate the effort.


Pagination

The property items expects an ActiveRecord collection of instances. The sage component calls kaminari gem methods. Will revisit at the end of the first phase of the documentation update

The property items expects an ActiveRecord collection of instances. The sage component calls kaminari gem methods. Will revisit at the end of the first phase of the documentation update


small clarification in the tutorial docs for cli arguments

In the optional cli arguments section, it said "To make a CLI argument optional, use typer.Argument() and pass a different "default" as the first parameter to typer.Argument(), for example None:" and the example was:
from typing import Optional

import typer
from typing_extensions import Annotated

def main(name: Annotated[Optional[str], typer.Argument()] = None):
    if name is None:
        print("Hello World!")
    else:
        print(f"Hello {name}")

if __name__ == "__main__":
    typer.run(main)

I thought it was confusing because when I read it for the first time i though i should write it like this
def main(name: Annotated[Optional[str], typer.Argument(default=None)]) -> None

which is wrong, so i clarified the text a little.

Hi, thanks for the PR!
This particular sentence in the documentation is already being addressed by https://github.com/fastapi/typer/pull/983, so I'll go ahead and close this one. Thanks again though!


Better desktop testing support with improved mocking

Good work and sorry for the delay. None of my comments are blocking so please resolve them to your own satisfaction and then :shipit:

Reviewed 19 of 19 files at r1.
Review status: all files reviewed at latest revision, 4 unresolved discussions.

test/www/jxcore/lib/testUtils.js, line 35 [r1] (raw file):
Why not make this work for mocked on desktop? Should be easy, right? Just track what you are pretending the state is.

test/www/jxcore/lib/wifiBasedNativeMock.js, line 24 [r1] (raw file):
This assert will always pass so what is the point? I think you mean to check networkStatusCalled

test/www/jxcore/lib/wifiBasedNativeMock.js, line 195 [r1] (raw file):
Couldn't you just use pipe to set up data forwarding in both directions?

thali/NextGeneration/thaliWifiInfrastructure.js, line 192 [r1] (raw file):
Rather than hard coding in test code into our production environment (which I admit sometimes is necessary) why not just reach in and manually override routerServerPort from the tests?

Comments from the review on Reviewable.io

Reviewed 1 of 19 files at r1, 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 4 unresolved discussions.

test/www/jxcore/lib/testUtils.js, line 35 [r1] (raw file):
Done.
This is already mocked on desktop.

test/www/jxcore/lib/wifiBasedNativeMock.js, line 24 [r1] (raw file):
Done.
Thanks for catching that - a clear mistake from my side.

test/www/jxcore/lib/wifiBasedNativeMock.js, line 195 [r1] (raw file):
Pipe would have worked here as well, but I used the more verbose way of wiring things, because I wanted to be prepared for the cases where we want to simulate certain socket connection errors or data timeouts etc. in the mock layer. I admit this is a bit pre-mature preparation since we aren't doing that kind of "advanced mocking" at the moment, but I'd like to leave it like this still.

thali/NextGeneration/thaliWifiInfrastructure.js, line 192 [r1] (raw file):
The advertised port override is a bit different thing than the router server port. When the Wifi code is used from the mock, the router server is still started, but it isn't used for anything. In the advertisement, we don't want to advertise the router server port, because that is not matching to where the other peer's mock is hosting a server than hosts the "proxy TCP server".
This is a bit of a hack to make the mock work, but I wanted to do it this way so that due to the mocking, I don't have to change the way the production Wifi code deals with the router server.
I have a better approach in mind that would be splitting the whole Wifi code into two parts (base and extension parts) and the mock would only use the base that doesn't do the things mock doesn't need. However, this would be some amount of extra work just for mocking sake so would not like to go there at this moment.

Comments from the review on Reviewable.io


Fix invalid method overrides in StdlibCollectionUnittest

This becomes an error when the fix in https://github.com/apple/swift/pull/24484 lands.

@swift-ci Please smoke test


Fix for node-gyp rebuild on Windows (cmd/powershell)

As described in #349.
Apologies if it's not the right way to do it, initially I tried to use copies but that didn't seem to have the option to change a name.

Hey it's better than nothing. Thanks


airy, gaussian beam pattern pixel_scale (error ? )

Hi! frbpoppy has been at the core of my work, thanks a lot for making it !
I found an issue in the pixel_scale for airy and gaussian beam patterns, which is taken as follows in the code:

But, if I go find the fwhm ( in pixels ) in the beam_array given for gaussian and airy beam patterns, I find it to be 94X2 and 30X2 pixels wide respectively. Looks like the pixel scales should have been fwhm/188, and fwhm/60 respectively. Could you please look into it?

Hi Harshini,
Unfortunately having left academia, I no longer have the time to maintain frbpoppy, nor double check prior programming. You might be able to try contacting Yuyang Wang, who is delving into frbpoppy at the moment.
Best of luck!
Cheers,
David


[Subtask] support schema pre event for Gravitino server

Describe the subtask

add corresponding SchemaPreEvent
generate pre-event in SchemaEventDispatcher
add test in TestSchemaEvent
add document in gravitino-server-config.md

Please refer to the implemetation of table pre event in #5539
Parent issue
#5317

It has a reference. I think I’ll be able to take this one. Please assign it to me, and I’ll work on it within a few days.

Hi @FANNG1,
I noticed that in AlterTableEvent, this.tableChanges is set to tableChanges.clone(),
while in AlterTablePreEvent, this.tableChanges is assigned directly as tableChanges.
Should it also be tableChanges.clone() in AlterTablePreEvent?

Good catch, @orenccl ! I commented related issues in the PR.


Add warning to the docs that it should not be used when security is a must

This approach greatly reduces the number of characters to guess. A big fat warning should be added to readme about that.

This depends on the configuration parameters but you are right - if $charactersMin < $charactersMax < $bitsRange < actual length of user's password the end password is weaker than the one in a standard approach.
I will add the more obvious warning.

Warning added.


Update structs to remove 'zero type structs' for consumers.

This is related to servo/core-text-rs#62 to remove the warnings when building.
The offending types for core-text are CGAffineTransform and CGContextRef.
Other warnings in the crate should also be cleaned up or silenced. The method for the two listed types can be found at rust-lang/rust#27303

@jdm I've raised this to track the work for this crate.


Using bootstrap in conjunction with Cosmos

I styled most of my CSS inline using Radium. However, some of my components rely on Bootstrap (which is included via a link tag in my templates).
There is a line in the README about including external Javascript and CSS files:
config.entry.push(path.join(process.cwd(), 'injectBootstrapTags.js'));

What is this injectBootstrapTags.js file? I basically only need to add bootstrap.css to the main page. How would I go about this?
Thanks!

In that file just inject the script/style tags from JS, for example like so:
var script = document.createElement('script');
script.setAttribute('src', '<filepath>.js');
document.body.appendChild(script);

var link = document.createElement('link');
link.type = 'text/css';
link.rel = 'stylesheet';
link.href = '<filepath>.css';
document.head.appendChild(link);

This file will be added to the bundle. You might want to use unshift instead of push to make sure that the file gets at the beginning of the bundle.

Brilliant, thank you so much for the answer!
Also, as another quick question, what folder does webpack server look for files? In your example, where should I place <filepath>.css?
I added the CDN for bootstrap, but I have some other files I'd like to add.

It's looking in process.cwd() (current working directory)


Rancher App & Martketplace Chars autorefresh too frequent.

It looks like all the added charts repos are refreshing every 5min or so.  Is there a way to configure it ?
With the Bitnami Repo Rancher seems to pull a lot more, (probably due to some internal errors) Each pull from the Bitnami Chart repo is 6MB. After running this for a week, the bandwidth monitor showed 3GB Data pulled.
time="2021-02-28T21:43:17Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:19Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:21Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:23Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:26Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:28Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:30Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:32Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:35Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:37Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:40Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:42Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:45Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:47Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:50Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:52Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:55Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:43:57Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:00Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:00Z" level=error msg="Error during subscribe websocket: close sent"
time="2021-02-28T21:44:02Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:04Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:07Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:09Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:12Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
W0228 21:44:13.063662       1 warnings.go:67] extensions/v1beta1 Ingress is deprecated in v1.14+, unavailable in v1.22+; use networking.k8s.io/v1 Ingress
time="2021-02-28T21:44:14Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:17Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:19Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:22Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:25Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:27Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:29Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:32Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:34Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:37Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:40Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:43Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:46Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:48Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:50Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:53Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:55Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:44:58Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:45:00Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:45:03Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:45:05Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:45:07Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:45:09Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"
time="2021-02-28T21:45:11Z" level=info msg="Downloading repo index from https://charts.bitnami.com/bitnami/index.yaml"

Is this just me having this issue ? It looks like each repo is queried once a minute.

@mikekuzak are you using the new UI to configure this catalog or the multicluster one? There are a handful of issues that are related to the bitnami chart specifically or at least occur when it is used, one work around for your issue may be to use the github chart instead. https://github.com/bitnami/charts.

via terraform
  catalogsV2 = {
    "codecentric" = "https://codecentric.github.io/helm-charts",
    "elastic"     = "https://helm.elastic.co"
    # "bitnami"     = "https://charts.bitnami.com/bitnami",
    # "ibm"         = "https://raw.githubusercontent.com/IBM/charts/master/repo/stable",
  }

# V2 Catalogs (Helm3)
resource "rancher2_catalog_v2" "catalogs_v2" {
  for_each = local.catalogsV2

  provider   = rancher2.admin
  cluster_id = rancher2_cluster_sync.cluster-sync.id
  name       = each.key
  url        = each.value
  depends_on = [rancher2_cluster_sync.cluster-sync]
}


nanopolish extract HDF5 error?

Running nanopolish 0.7.1
When trying to run nanopolish extract (./nanopolish extract /home/sguertin/170623_A_data/re-calls/pass-only/workspace -o /home/sguertin/170623_A_data/20170623_A_nanopolishextract.fa), I get the following error:
HDF5-DIAG: Error detected in HDF5 (1.8.14) thread<PHONE_NUMBER>21504:
#000: H5F.c line 604 in H5Fopen(): unable to open file
major: File accessibilty
minor: Unable to open file
#001: H5Fint.c line 1085 in H5F_open(): unable to read superblock
major: File accessibilty
minor: Read failed
#002: H5Fsuper.c line 294 in H5F_super_read(): unable to load superblock
major: Object cache
minor: Unable to protect metadata
#003: H5AC.c line 1320 in H5AC_protect(): H5C_protect() failed.
major: Object cache
minor: Unable to protect metadata
#004: H5C.c line 3574 in H5C_protect(): can't load entry
major: Object cache
minor: Unable to load metadata into cache
#005: H5C.c line 7954 in H5C_load_entry(): unable to load entry
major: Object cache
minor: Unable to load metadata into cache
#006: H5Fsuper_cache.c line 476 in H5F_sblock_load(): truncated file: eof = 62047, sblock->base_addr = 0, stored_eoa = 225373
major: File accessibilty
minor: File has been truncated
Is the root cause of this an hdf5 issue, a nanopolish issue, or something else entirely?

Hi,
Based on the last error minor: File has been truncated it seems there is a problem with one of your fast5 files. Maybe you ran out of disk space when basecalling?
Jared

Possible, but irritating if so. I'll check - thanks!


Appreciate this page!

The examples on this page worked flawlessly and I am glad for this.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: bbf00e2a-56e1-3da7-859a-a7563dd9148b
Version Independent ID: 2866c1ea-6a4e-27e5-ce7f-34153eaf6c94
Content: Build a Xamarin.Forms app with Azure Mobile Apps
Content Source: articles/mobile-apps/azure-mobile-apps/quickstarts/xamarin-forms/index.md
Service: mobile-services
GitHub Login: @adrianhall
Microsoft Alias: adhal

@abhidotnet thanks for the feedback! Glad that it's helping you.
CC @adrianhall


com.jameskleeh.excel.Font issue

Hi,
Using the simple example
sheet {
row { "test1"}
}
and the library via maven :
    <dependency>
        <groupId>com.jameskleeh</groupId>
        <artifactId>excel-builder</artifactId>
        <version>0.4.0</version>
    </dependency>

Java 1.7 .0_45, Grails 2.2.5, Groovy 2.0.8, poi 3.16
I'm getting an exception as follows:
the number of constructors during runtime and compile time for com.jameskleeh.excel.Font do not match. Expected -1 but got 2. Stacktrace follows:
org.codehaus.groovy.grails.web.servlet.mvc.exceptions.ControllerExecutionException: Executing action [exportAuftragspositionen] of controller [de.interseroh.dienstleistungsportal.reporting.ReportingController]  caused exception: Runtime error executing action
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:744)
Caused by: org.codehaus.groovy.grails.web.servlet.mvc.exceptions.ControllerExecutionException: Runtime error executing action
... 3 more
Caused by: java.lang.reflect.InvocationTargetException
... 3 more
Caused by: java.lang.IncompatibleClassChangeError: the number of constructors during runtime and compile time for com.jameskleeh.excel.Font do not match. Expected -1 but got 2
at com.jameskleeh.excel.Sheet.row(Sheet.groovy:176)
at com.jameskleeh.excel.Sheet.row(Sheet.groovy:151)
at de.interseroh.dienstleistungsportal.reporting.Excel2007BuildExporter$_exportData_closure1_closure2.doCall(Excel2007BuildExporter.groovy:20)
at com.jameskleeh.excel.Workbook.handleSheet(Workbook.groovy:114)
at com.jameskleeh.excel.Workbook.sheet(Workbook.groovy:55)
at de.interseroh.dienstleistungsportal.reporting.Excel2007BuildExporter$_exportData_closure1.doCall(Excel2007BuildExporter.groovy:19)
at com.jameskleeh.excel.ExcelBuilder.build(ExcelBuilder.groovy:55)
at com.jameskleeh.excel.ExcelBuilder.output(ExcelBuilder.groovy:40)
Any idea what could be causing this. Is there a minimum Groovy version requirement?

Perhaps it is a groovy compatibility problem 2.0.x is pretty old. As you can see the Font class is about as simple as it gets
https://github.com/jameskleeh/groovy-excel-builder/blob/master/src/main/groovy/com/jameskleeh/excel/Font.groovy

Could also be a JDK issue. I compile this library with Java 8

No feedback in 3 weeks so I'm closing this


Need simple Template syntax checker

As I am building templates, the most common problem I run into is improper syntax of the various <% ... %> tags.
This goes beyond checking "count(<%) vs count( %> )".
Initially, I don't need to check PL/SQL code ( <%! ... %> ) or PL/SQL expressions ( <%= ... %> ).

Directives for Build Templates ( extends() and block() ) have been added.
Directive for include() just needs to be moved to the TE_SYNTAX package.
Directive for template() still needs to be added.

Parse/Decoders for include() and template() Template Directives are now in the TE_SYNTAX package.
Usage of the parsers need to be included into the TEPLSQL package.
However,  in order to still support 10g, these need to be disabled at compile time (due to usage of regexp_count() )


If there aren't enough items to fill the Table height, user cannot trigger a loadMore operation

🐛 Bug Report
Right now loadMore is triggered via scrolling in the Table (from what I can tell). If there aren't enough items to fill the table body, the user can't scroll the table and thus can't trigger loadMore.
🤔 Expected Behavior
Perhaps the table should loadMore automatically if there aren't enough items to fill the window?
😯 Current Behavior
See description
💁 Possible Solution
I've been playing around with adding a useEffect to the TableVirtualizer that looks something like this:
  useEffect(() => {
    if (!collection.body.props.isLoading && collection.body.props.onLoadMore && !state.isAnimating) {
      if (bodyRef.current?.offsetHeight > state.virtualizer.contentSize.height ) {
        collection.body.props.onLoadMore();
      }
    }

  }, [collection.body.props.isLoading, state.virtualizer, state.isAnimating, bodyRef])

Bit finicky cuz it is triggering loadMore one extra time than is required... Need to dig to see when exactly state.isAnimating changes. Ideally this useEffect would only trigger when isLoading=false and after any new items finish loading+resizing in the table. Might be a transition delay issue?
🔦 Context

💻 Code Sample

🌍 Your Environment

Software
Version(s)

react-spectrum

Browser

Operating System

🧢 Your Company/Team

🕷 Tracking Issue (optional)

I assume this issue is also affecting ListBox because the logic is the same there (in Virtualizer)?

Doesn't happen in ListBox because it sets sizeToFit="height" I think


Compatibility with tectonic

Description
I've been trying to use the tectonic front-end. Today I ran into an issue with the microtype spacing support.
$ tectonic microtype.tex
Running TeX ...
error: microtype.tex:3: Package microtype Error: The spacing feature only works with pdftex 1.40
(microtype)                or newer. Switching it off.

See the microtype package documentation for explanation.
Type  H <return>  for immediate help
error: halted on potentially-recoverable error as specified

Any idea what's missing from the tectonic engine for this to work? Tectonic is based on XeTeX, so I was surprised it wasn't supported.
Please also indicate the engine(s) with which the issue occurs: pdfTeX/LuaTeX/XeTeX
tectonic (based on XeTeX)
Links to <tex.stackexchange.com> discussion if appropriate
Minimal example demonstrating the issue
\documentclass{article}
\usepackage[spacing]{microtype}
\begin{document}
Hello, bug!
\end{document}

Of the microtypographic features, only protrusion is supported by xetex (see the table on p. 6 in microtype.pdf). So as tectonic is based on xetex, it is expected behaviour that the spacing feature doesn't work. I don't know whether tectonic can be persuaded to use pdftex instead, but that would be the only way to get the extra spacing.

Ah, I see. Thanks for clarifying. I'll stick with pdftex for this document.


Question ?

how can fixed 3 decimals digits after comma with area measurements.

This can be solved by creating custom units. https://github.com/ljagis/leaflet-measure#units.


Misleading error message when project arguments points to folder that lacks 'tsconfig.json'

create new folder with name '1', run compiler passing path to this folder as a value of 'project' argument.
Error message:
error TS5014: Failed to parse file '1/tsconfig.json': Unexpected token u.

+1 This happens with simple typos too.
For example, tsc -p -src or tsc -p .src/ when I meant tsc -p src. The typo is on us, but I propose that the error should say

No tsconfig.json is found at the specified folder .src/

:+1: ran into this myself.


configure-scheduler.sh script breaks Kubernetes config

Updating Kubernetes scheduler config using the script & (GAS) config in this repo, breaks Kubernetes 1.26 config:
./telemetry-aware-scheduling/deploy/extender-configuration/configure-scheduler.sh -f ./gpu-aware-scheduling/deploy/extender-configuration/scheduler-config-tas+gas.yaml
It messed up the last volume specification in the config so that there were couple of extra lines for the last volume specification.
Besides fixing that in the script, I think script should take a backup of the config file, and show user a diff of the changes it did.
Maybe also ask user whether changes should be accepted (unless something like "--yes" is specified), like e.g. Debian configuration packages script updates do.
You might also consider using kustomize to do the file updates, as that is actually designed for semantic updating of k8s YAML files.
(Which still leaves the issue of upgrade overwriting the changes, as mentioned in https://github.com/intel/platform-aware-scheduling/issues/86.)

Hi @eero-t
Thank you for flag this and for your suggestions. We are looking into them.

Hi @eero-t
would you be able to share both the output of the configure-scheduler.sh run (the output of the command you pasted above) and resulting kube-scheduler.yaml output? I'm expecting the script changed the file in a way that the scheduler would not start anymore. If that's not available, could you share the current kube-scheduler manifest?
I would like to understand what was the problem in the first place and without it it's pretty much impossible.
As for the other suggestions, I have added them to our backlog. Will let you know when they will be picked up

As script did not provide a backup (and I was not the person updating that cluster to k8s 1.26), I do now know what the starting point was.
If I run the command now (with fixed config), it outputs following:
# ./telemetry-aware-scheduling/deploy/extender-configuration/configure-scheduler.sh -f ./gpu-aware-scheduling/deploy/extender-configuration/scheduler-config-tas+gas.yaml
Manifest file is located at: /etc/kubernetes/manifests/kube-scheduler.yaml
Scheduler config file is located at: ./gpu-aware-scheduling/deploy/extender-configuration/scheduler-config-tas+gas.yaml
Scheduler config destination will be: /etc/kubernetes
Version of the image used in the kube scheduler is: 26
Version of the KubeScheduler API: v1
Will proceed to copy the scheduler configuration to its destination path: /etc/kubernetes.
Determined that the appropriate nodeSelector key would be: control-plan

And does not change kube-scheduler.yaml any more.
Which looks like this:
...
  hostNetwork: true
  priorityClassName: system-node-critical
  securityContext:
    seccompProfile:
      type: RuntimeDefault
  volumes:
  - hostPath:
      path: /etc/kubernetes/scheduler-config-tas+gas.yaml
    name: schedulerconfig
  - hostPath:
      path: /etc/certs
    name: certdir
  - hostPath:
      path: /etc/kubernetes/scheduler.conf
      type: FileOrCreate
    name: kubeconfig
status: {}

After the initial run of the script, there were additional path: & type: lines for the last (kubeconfig) volume, in addition to  path: & type: lines you see above.

Hi,
I tried to use the same command as you did and this was my output:
Command
./telemetry-aware-scheduling/deploy/extender-configuration/configure-scheduler.sh -f ./gpu-aware-scheduling/deploy/extender-configuration/scheduler-config-tas+gas.yaml
Initial manifest file
   - mountPath: /etc/kubernetes/scheduler-config.yaml
     name: schedulerconfig
     readOnly: true
   - mountPath: /host/certs
     name: certdir
   - mountPath: /etc/kubernetes/scheduler.conf
     name: kubeconfig
     readOnly: true
 hostNetwork: true
 priorityClassName: system-node-critical
 securityContext:
   seccompProfile:
     type: RuntimeDefault
 volumes:
 - hostPath:
     path: /etc/kubernetes/scheduler-config.yaml
   name: schedulerconfig
 - hostPath:
     path: /etc/certs
   name: certdir
 - hostPath:
     path: /etc/kubernetes/scheduler.conf
     type: FileOrCreate
   name: kubeconfig

Output
  volumeMounts:
    - mountPath: /etc/kubernetes/scheduler-config-tas+gas.yaml
      name: schedulerconfig
      readOnly: true
    - mountPath: /host/certs
      name: certdir
    - mountPath: /etc/kubernetes/scheduler-config.yaml
    - mountPath: /etc/kubernetes/scheduler.conf
      name: kubeconfig
      readOnly: true
  hostNetwork: true
  priorityClassName: system-node-critical
  securityContext:
    seccompProfile:
      type: RuntimeDefault
  volumes:
  - hostPath:
      path: /etc/kubernetes/scheduler-config-tas+gas.yaml
    name: schedulerconfig
  - hostPath:
      path: /etc/certs
    name: certdir
  - hostPath:
      path: /etc/kubernetes/scheduler-config.yaml
      path: /etc/kubernetes/scheduler.conf
      type: FileOrCreate
    name: kubeconfig

The problem I found is here :

-- hostPath:
path: /etc/kubernetes/scheduler-config.yaml
path: /etc/kubernetes/scheduler.conf
type: FileOrCreate
name: kubeconfig

From what I could tell the root cause is this line https://github.com/intel-innersource/libraries.orchestrators.resourcemanagement.platform-aware-scheduling/blob/master/telemetry-aware-scheduling/deploy/extender-configuration/configure-scheduler.sh#L162
sed -i "/$scheduler_config_file/d" "$MANIFEST_FILE"
I started with the default manifest file (deploy/extender/scheduler-config.yaml), but when I specified a new one -f ./gpu-aware-scheduling/deploy/extender-configuration/scheduler-config-tas+gas.yaml, the $MANIFEST_FILE would be scheduler-config-tas+gas.yaml not scheduler-config.yaml so sed can't find and won't remove it.
Not sure if this is exactly the same problem as in my case, the scheduler was still working (the scheduler pod started successfully, and then I was able to schedule the demo pod according to the TAS health-demo policy).
I created a follow-up item for us to work on to address this particular problem.
In terms of priorities, I think we will be starting with 2 items first: fixing the bug above and having the file create backups as I think these are more urgent. I'll be picking these up in the next sprint (following week).
I will update this issue once they will be released.

IMHO YAML modifications should be done with something that actually understands YAML (e.g. Python script using pyyaml module), otherwise it's too easy for the script to break things.

Hi,
Apologies for the late reply. I logged an issue in our backlog about this problem (it was also referenced here https://github.com/intel/platform-aware-scheduling/issues/86), but we weren't able to pick-up the work so far.
I'm working on a new release for TAS, but unfortunately this change did not make it.
My plan is to look this week/Monday at what changes we can work on in March and then what we plan for the next quarter. I'll be able to give a more specific date then.
Just to keep things clear, I will continue to update this issue regarding the changes required for the configuration  script  and just update issue#86 with regards to your last request.


Scrollbar and playlist positioning is not equal

Apologies if this isn't a bug but instead a design choice! I noticed that there's some weird spacing going on with how the scroll bar is rendered in the playlist view, as well as where the playlist stops being rendered at the bottom.
Refer to this annotated screenshot:

Distance A (at the bottom) is unexpected - is the playlist meant to be rendered all the way to the edge of the whitespace that it sits in? I would suggest extending it to the edge.
Distance B and C should be extended to the vertical start and stop of the rendering area for the playlist.

I need those hidden bars ( top and bottom ) to hide the rows that shouldn't be visible when scrolling up and down.
If they are removed, there would be redraw leftovers from those scrolling rows. This is the only workaround there is to it.
The playlist arrows have the same spacing ( from playlist top to the up arrow and from playlist bottom to arrow bottom ).
You can try it out for yourself to understand it, if you remove the code from line 709 to 777 in Panel_Playlist.js
and play around with the scrolling it will happen...


Zoom.ZoomRooms version 5.12.2

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

The default MSI silent switches are not sufficient for this package -


test(steps): organise test workflows in testdata folder

Partial fix for #13610
Motivation

Modifications

Verification

cc/ @isubasinghe


Move more complicated panel editing logic into the store

This builds on #590 (so look at that PR first) and moves the more complicated panel editing logic into a PanelEditingSlice into the store. The initial editor state is now in the store, making sure you can't (for example) edit a panel that doesn't exist. It also adds support for editing a panel and having it move between groups. This is what that looks like:
https://user-images.githubusercontent.com/428023/193431778-e9201073-c1b7-42ec-a7af-3b8ae1d2c75a.mov

Interesting :).
When you said "Moving to the store", I guess you are talking about the browser storage.
Is it in place until the API is ready to store the dashboard ?

Interesting :). When you said "Moving to the store", I guess you are talking about the browser storage. Is it in place until the API is ready to store the dashboard ?

In this case I'm talking about the "dashboard state store" (which we use a library called zustand for). If you've ever built an app with something like Redux, it's a similar idea where the page level state for a dashboard is kept in one place, along with actions for manipulating its state (e.g. visual editing actions). See the DashboardStoreState in the DashboardProvider for what I'm talking about.
Sorry, should've been more specific! 😂


Merging latest template-service-flask changes

The AWS login failed in CI, this might be due to the settings of codebuild, @pakb did you update the codebuild terraform project for service-icons and applied it ?

no I haven't done anything on the terraform side, I'll start digging into that.
I'll also add what your latest PR on template-service-flask contains later

I'll open another PR with a cleaner git log


AKS-Private API server

Is there an existing issue for this?

[ ] I have searched the existing issues

Description
No response
New or Affected Resource(s)/Data Source(s)
No response
Potential Terraform Configuration
No response
References
No response

work completed:
Fully private AKS w/ outbound_type = UDR | bring your own vNet
Azure RBAC w/ Azure Entra Groups
Key Vault
still need to be completed:
default node pool settings
clean up code


Pagination: Integrate with view controller

This pull request integrates the pagination mechanism introduced in https://github.com/spotify/HubFramework/pull/132 into HUBViewController. Whenever the user is about to scroll to the bottom of a view, the next page of content is automatically loaded.
The "Really Long List" feature in the demo app has also been rewritten to support this new pagination feature, instead of having to perform work on a background thread.
Implements the view controller part of https://github.com/spotify/HubFramework/issues/28. Will not close the issue until the content programming guide has been updated :)

@spotify/objc-dev

Current coverage is 92.39% (diff: 100%)

No coverage report found for master at 5f0b3be.

Powered by Codecov. Last update 5f0b3be...2f398a9

🎉 Nice!


Support for Azure OpenAI

Great tool and idea. 👍
Would love to see support for Azure OpenAI as well, as this currently one of our go-to AIs. 😀

We are also interested in this. We are very happy with Crowdin AI support of Azure OpenAI because we are a regulated company and unable to use OpenAI directly.
However, given that this is an official Crowdin package it would be great to have the same models supported as Crowdin AI :)

We are now planning to revamp the CLI, one of the things would be to allow the configuration of direct connections to all major AI providers.
Currently, the CLI allows you to configure either a direct connection to OpenAI or to use Crowdin as an AI provider. If you have Azure OpenAI configured in Crowdin, that configuration will be consumed by the CLI context harvester. Please note that if Crowdin is selected as the AI provider, a code will be sent to the AI via Crowdin.


Idea list (moved from design_spec.md)

Ideas
tldlist command

Now that tldlist.com has updated their UI and introduced domain availability search I could scrap the site and create a command using the data.

Aliases: tld, domain

Language pack implementation
A cog must be able to use the language system like this:
from discord.ext import commands

class Cog:
    def __init__(self, bot):
        self.bot = bot
        self.lang = self.bot.get_lang('cmd_cog')

    @commands.command(pass_context=True)
    def quak_quak(self, ctx):
        self.bot.say(self.lang.get('quak.reply', ctx))

Bot.get_lang() MUST return a class
...whose get() can be called to fetch requested string from the language pack

The returned class of Bot.get_lang() MUST:

have a required param filename and path_obj in __init__()
consist of get() function
consist of reload() function
have a local dictionary for YAML with the passed filename in each language pack

if unable to find any, set it as None instead

OR
Example translation file:
"Hello, world!": "您好世界"

In Python:
print(locale('Hello, world!', 'zh_cn'))

There should be no need for info.yaml, considering removing it
OR
Write a custom Bot.say(), other relevant functions & inherited Embed class instead that checks message author's language preference (via database).
Example translation file:
"Hello, world!":     # The key itself would be considered as "en_us"
  zh_cn: "你好，世界！"

Fetching translated string is as easy as:
self.bot.say("Hello, world!") # replies with 你好，世界！" if user's lang pref is zh_cn

# pass say() translate param if you don't want it to be automatically translated.
self.bot.say("Hello, world!", translate=False) # Can be easily replaced with Ctrl+F'ing "translate=False" if you want to use cogs that contains this on other bots

# lang param will be required if there's no database module loaded

Object tree (in database)
id
|- type "server"
|-- superuser: array (either role or member ID, server owner's ID included)
|-- admin: array (either role or member ID)

NOTE: Bot owner(s), by default, is a superuser of all servers the bot is in so the bot owner(s) will be able to provide support to any servers using their bot.
NOTE: Superuser and admin WILL BE ENTIRELY SEPARATE. Superuser cannot perform any adminstration commands while admin can.

But let's be honest that could be bypassed with built-in eval command anyway, and it's an essential command for bot support, which means a certain trust between the bot owner(s) and the server owner(s) is needed. The eval command will log who and what code did said person run.

Database specs
Malena and Liara's suggestion: table.get(key) which means creating a new table class
Music
Auto Pause/Unpause
Definitely not Inspired by HcgRandon's Ayana.

Pause the current song and list when there's no listeners (aka non-bots) in the VC for minutes (able to set via config file)

play/add Command

Usage: play/add <link/tag> [from time/start|till time/end|from time/start till time/end (in MM:SS format) ]

Example: play BRAIN POWER from 1:00 till end or play BRAIN POWER from 1:00 assuming that BRAIN POWER is a tag

Supports YouTube/SoundCloud/Twitch
Supports playing from/until a time/ within the period

song Command

Only the requester, superusers/admins (user/roles) and bot owner can control the current song.

To prevent infinite loop abuse, the superusers/admins (user/roles) can set how many times can a song repeat/restart/goto

By default unless they are specifically specified as infinite, if only one was set the value applies to the rest as well

This also applies to how many times can the same link get added into the queue.

If majority of the listeners in a server wanted to repeat a song infinitely it's already remedied by local-storage system

play

If a video is geo-blocked and the bot exists on other VPNs, the other VPNs will be used to download and deliver/stream the music file back to the server and play

pause
restart
goto <time (in MM:SS format)>
repeat

skip Command

Skips either the current song, the playlist (if it was requested) or queue

queue command

repeat
shuffle

Music local-storage system
NOTE: A way to prevent getting IP banned by YouTube for scraping too much.

Either store it locally or fetch the files from other servers

Tags system (get, set/edit/create, give, remove/delete and block command groups)

☞Ｐａｋｏ#9400<PHONE_NUMBER>55524352) suggested using PEG parser instead of regex, looking into it.

Available specifiers for <object>s
NOTE: String section is only used when the object is known, for example the bot itself would be bot

Guild:

String
guild
server
the guild
the server

Bot itself

String

bot
the bot

get

role <tag/role>

Gives role to the user

role <tag/role> for <user>

Gives role to specified user (ONLY WHEN SAID USER HAVE PERMISSION)

info from <object>

info of <object>

Object could be either a server/guild string, a channel (mention/name/id), a user (mention/name/id) or bot/the bot string

role

SET (e.g. set role <role tag>)

At this point it requires a multiline codeblock in YAML config for settings and will be loaded with YAML parser.

In a way JSON is also supported since YAML parser is also technically a JSON parser.
safe_load() will be used.
Format:<tag>:
  who_can_get:

Example:mature:

GET (e.g. get role <role tag>)

info (in a way replaces userinfo)

Aliases: info from, details, details from
GET (e.g. get info from <object>)

User/channel/role name/ID/mention

For name search:

User: the default choice, but for the sake of it, u: or user: prefix, case-insensitive
Channel: add a c: or channel: prefix, case-insensitive
Role: add a r: or role: prefix, case-insensitive

server
bot

setting

Aliases: settings
GET (e.g. get setting from <tag>)

Aliases: setting from, settings from

SET (e.g. set settings of <tag>)

Aliases: setting of, settings of

Tag types

Generic
Music

Compactible with Rotom's music cog

Regex will be used to parse subcommands

NOTE: Unless specified otherwise, if a tag type isn't mentioned in the subcommand it means it's not usable for said type.
load and unload bot-owner command groups

cog/cogs

Supports multiple cog names at once, separated by space

lang/language

Supports multiple lang names at once, separated by space

API

Pretty much essential for multiserver communication (for region-based music playing mainly)

Annnnd other people's requests

https://owo.whats-th.is/ce77f8.png https://owo.whats-th.is/bbc4ac.png (okay not too shabby actually)
https://owo.whats-th.is/6564dd.png (a reminder in light of two recent mention spams)

All ideas are separated into individual issues except database section (since it's not need anymore).


[Bug] Billing Extension dead Installer.php on version 2.2.194

Hi, this is the only way I found to contact you to inform that there is a bug on your compilation that is making impossible to enable the module.
Updated from versions 2.2.191 (working perfectly) to 2.2.194
ERROR LOG:
ValueError: PDO::exec(): Argument #1 ($statement) cannot be empty in /modules/addons/BillingExtension/core/BillingExtension_Admin/Installer.php:2995
Stack trace:
/modules/addons/BillingExtension/core/BillingExtension_Admin/Installer.php(2995): PDO->exec()
/modules/addons/BillingExtension/BillingExtension.php(44): BillingExtension\BillingExtension_Admin\Installer->Upgrade()
/admin/configaddonmods.php(0): BillingExtension_upgrade()
{main}

WHMCS [eg. 8.9.0]
PHP [eg. 8.1]

I will try to rollback to previous version for now.
Thanks

@EddiePiazza solved in some way? @Kian987 updates on this bug? Thanks


Too many redirects when adding nginx-proxy-letsencrypt container

Bug description
When accessing sub.hostname.tld, I get infinite redirects.
letsencrypt-nginx-proxy-companion image version
Info: running letsencrypt-nginx-proxy-companion version v2.0.0-58-g8743298

nginx-proxy-companion configuration
I used the exact configuration on the ReadMe (3 steps nginx proxy -> letsencrypt -> nginx), except for the domain. I also tried the other examples with docker-compose and got the same result.
Containers logs

letsencrypt-nginx-proxy-companion container
2021-03-05T04:35:47.084089341Z Info: running letsencrypt-nginx-proxy-companion version v2.0.0-58-g8743298
2021-03-05T04:35:47.801249276Z Generating a RSA private key
2021-03-05T04:35:48.744341688Z ..............................................................................................................................++++
2021-03-05T04:35:48.831769961Z ...........++++
2021-03-05T04:35:48.832505273Z writing new private key to '/etc/nginx/certs/default.key.new'
2021-03-05T04:35:48.834040676Z -----
2021-03-05T04:35:49.062968776Z Reloading nginx proxy (nginx-proxy)...
2021-03-05T04:35:49.611820470Z 2021/03/05 04:35:49 Generated '/etc/nginx/conf.d/default.conf' from 3 containers
2021-03-05T04:35:49.613519844Z 2021/03/05 04:35:49 [notice] 87#87: signal process started
2021-03-05T04:35:49.613795995Z Info: a default key and certificate have been created at /etc/nginx/certs/default.key and /etc/nginx/certs/default.crt.
2021-03-05T04:35:49.649403245Z Info: Creating Diffie-Hellman group in the background.
2021-03-05T04:35:49.649462120Z A pre-generated Diffie-Hellman group will be used for now while the new one is being created.
2021-03-05T04:35:49.674840919Z Generating DH parameters, 2048 bit long safe prime, generator 2
2021-03-05T04:35:49.887634515Z Reloading nginx proxy (nginx-proxy)...
2021-03-05T04:35:50.428295401Z 2021/03/05 04:35:50 Contents of /etc/nginx/conf.d/default.conf did not change. Skipping notification ''
2021-03-05T04:35:50.431149648Z 2021/03/05 04:35:50 [notice] 113#113: signal process started
2021-03-05T04:35:50.527873438Z 2021/03/05 04:35:50 Generated '/app/letsencrypt_service_data' from 3 containers
2021-03-05T04:35:50.527917412Z 2021/03/05 04:35:50 Running '/app/signal_le_service'
2021-03-05T04:35:50.544622429Z 2021/03/05 04:35:50 Watching docker events
2021-03-05T04:35:50.591791928Z 2021/03/05 04:35:50 Contents of /app/letsencrypt_service_data did not change. Skipping notification '/app/signal_le_service'
2021-03-05T04:35:50.817800536Z Sleep for 3600s
2021-03-05T04:36:46.990162349Z 2021/03/05 04:36:46 Received event start for container dad373c48a03
2021-03-05T04:36:51.980404465Z 2021/03/05 04:36:51 Debounce minTimer fired
2021-03-05T04:36:52.057902052Z 2021/03/05 04:36:52 Generated '/app/letsencrypt_service_data' from 4 containers
2021-03-05T04:36:52.057957794Z 2021/03/05 04:36:52 Running '/app/signal_le_service'
2021-03-05T04:36:52.962576442Z [Fri Mar  5 04:36:52 UTC 2021] Create account key ok.
2021-03-05T04:36:53.060018220Z [Fri Mar  5 04:36:53 UTC 2021] Registering account: https://acme-v02.api.letsencrypt.org/directory
2021-03-05T04:36:53.473047905Z [Fri Mar  5 04:36:53 UTC 2021] Registered
2021-03-05T04:36:53.537440129Z [Fri Mar  5 04:36:53 UTC 2021] ACCOUNT_THUMBPRINT='...'
2021-03-05T04:36:53.776871705Z Reloading nginx proxy (nginx-proxy)...
2021-03-05T04:36:54.378334928Z 2021/03/05 04:36:54 Generated '/etc/nginx/conf.d/default.conf' from 4 containers
2021-03-05T04:36:54.383546090Z 2021/03/05 04:36:54 [notice] 145#145: signal process started
2021-03-05T04:36:54.384516327Z Creating/renewal sub.hostname.tld certificates... (sub.hostname.tld)
2021-03-05T04:36:54.970896663Z [Fri Mar  5 04:36:54 UTC 2021] Using CA: https://acme-v02.api.letsencrypt.org/directory
2021-03-05T04:36:55.060159039Z [Fri Mar  5 04:36:55 UTC 2021] Creating domain key
2021-03-05T04:36:58.674623459Z [Fri Mar  5 04:36:58 UTC 2021] The domain key is here: /etc/acme.sh/...
2021-03-05T04:36:58.681882076Z [Fri Mar  5 04:36:58 UTC 2021] Single domain='sub.hostname.tld'
2021-03-05T04:36:58.742717564Z [Fri Mar  5 04:36:58 UTC 2021] Getting domain auth token for each domain
2021-03-05T04:36:59.602153586Z [Fri Mar  5 04:36:59 UTC 2021] Getting webroot for domain='sub.hostname.tld'
2021-03-05T04:36:59.808111720Z [Fri Mar  5 04:36:59 UTC 2021] Verifying: sub.hostname.tld
2021-03-05T04:37:02.347275793Z This is going to take a long time
2021-03-05T04:37:02.347318647Z Info: Diffie-Hellman group creation complete, reloading nginx.
2021-03-05T04:37:02.347325257Z Reloading nginx proxy (nginx-proxy)...
2021-03-05T04:37:02.347331167Z 2021/03/05 04:37:02 Contents of /etc/nginx/conf.d/default.conf did not change. Skipping notification ''
2021-03-05T04:37:02.347337015Z 2021/03/05 04:37:02 [notice] 171#171: signal process started
2021-03-05T04:37:02.412294118Z [Fri Mar  5 04:37:02 UTC 2021] Success
2021-03-05T04:37:02.441531105Z [Fri Mar  5 04:37:02 UTC 2021] Verify finished, start to sign.
2021-03-05T04:37:02.473625599Z [Fri Mar  5 04:37:02 UTC 2021] Lets finalize the order.
2021-03-05T04:37:02.479519112Z [Fri Mar  5 04:37:02 UTC 2021] Le_OrderFinalize='https://acme-v02.api.letsencrypt.org/acme/finalize/'
2021-03-05T04:37:03.272850780Z [Fri Mar  5 04:37:03 UTC 2021] Downloading cert.
2021-03-05T04:37:03.278971443Z [Fri Mar  5 04:37:03 UTC 2021] Le_LinkCert='https://acme-v02.api.letsencrypt.org/acme/cert/'
...
2021-03-05T04:37:03.643770588Z [Fri Mar  5 04:37:03 UTC 2021] Your cert is in  /etc/acme.sh/...
2021-03-05T04:37:03.650584218Z [Fri Mar  5 04:37:03 UTC 2021] Your cert key is in  /etc/acme.sh/....
2021-03-05T04:37:03.666884065Z [Fri Mar  5 04:37:03 UTC 2021] The intermediate CA cert is in  /etc/acme.sh/....
2021-03-05T04:37:03.676344606Z [Fri Mar  5 04:37:03 UTC 2021] And the full chain certs is there:  /etc/acme.sh/...
2021-03-05T04:37:03.781917580Z [Fri Mar  5 04:37:03 UTC 2021] Installing cert to:/etc/nginx/certs/...
2021-03-05T04:37:03.790106267Z [Fri Mar  5 04:37:03 UTC 2021] Installing CA to:/etc/nginx/certs/...
2021-03-05T04:37:03.807423595Z [Fri Mar  5 04:37:03 UTC 2021] Installing key to:/etc/nginx/certs/...
2021-03-05T04:37:03.818398860Z [Fri Mar  5 04:37:03 UTC 2021] Installing full chain to:/etc/nginx/certs/...
2021-03-05T04:37:04.206212624Z Reloading nginx proxy (nginx-proxy)...
2021-03-05T04:37:04.752304131Z 2021/03/05 04:37:04 Generated '/etc/nginx/conf.d/default.conf' from 4 containers
2021-03-05T04:37:04.754930731Z 2021/03/05 04:37:04 [notice] 198#198: signal process started
2021-03-05T04:37:04.756239500Z Sleep for 3600s

your nginx-proxy container
WARNING: /etc/nginx/dhparam/dhparam.pem was not found. A pre-generated dhparam.pem will be used for now while a new one
is being generated in the background.  Once the new dhparam.pem is in place, nginx will be reloaded.
forego     | starting dockergen.1 on port 5000
forego     | starting nginx.1 on port 5100
dockergen.1 | 2021/03/05 04:32:51 Generated '/etc/nginx/conf.d/default.conf' from 2 containers
dockergen.1 | 2021/03/05 04:32:51 Running 'nginx -s reload'
dockergen.1 | 2021/03/05 04:32:51 Watching docker events
dockergen.1 | 2021/03/05 04:32:51 Contents of /etc/nginx/conf.d/default.conf did not change. Skipping notification 'nginx -s reload'
2021/03/05 04:32:56 [notice] 50#50: signal process started
Generating DSA parameters, 4096 bit long prime
dhparam generation complete, reloading nginx
dockergen.1 | 2021/03/05 04:33:04 Received event start for container 915d8d387c99
dockergen.1 | 2021/03/05 04:33:04 Generated '/etc/nginx/conf.d/default.conf' from 3 containers
dockergen.1 | 2021/03/05 04:33:04 Running 'nginx -s reload'
nginx.1    | sub.hostname.tld ... - - [05/Mar/2021:04:33:18 +0000] "GET / HTTP/1.1" 200 612 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/89.0.4389.72 Safari/537.36"
nginx.1    | sub.hostname.tld ... - - [05/Mar/2021:04:33:21 +0000] "GET / HTTP/1.1" 304 0 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/89.0.4389.72 Safari/537.36"
nginx.1    | sub.hostname.tld ... - - [05/Mar/2021:04:33:22 +0000] "GET / HTTP/1.1" 304 0 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/89.0.4389.72 Safari/537.36"
nginx.1    | sub.hostname.tld ... - - [05/Mar/2021:04:33:23 +0000] "GET / HTTP/1.1" 304 0 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/89.0.4389.72 Safari/537.36"
nginx.1    | sub.hostname.tld ... - - [05/Mar/2021:04:33:23 +0000] "GET / HTTP/1.1" 304 0 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/89.0.4389.72 Safari/537.36"
nginx.1    | sub.hostname.tld ... - - [05/Mar/2021:04:33:24 +0000] "GET / HTTP/1.1" 304 0 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/89.0.4389.72 Safari/537.36"
nginx.1    | sub.hostname.tld ... - - [05/Mar/2021:04:33:26 +0000] "GET / HTTP/1.1" 304 0 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/89.0.4389.72 Safari/537.36"
nginx.1    | sub.hostname.tld ... - - [05/Mar/2021:04:33:29 +0000] "GET / HTTP/1.1" 200 612 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/89.0.4389.72 Safari/537.36"
nginx.1    | sub.hostname.tld ... - - [05/Mar/2021:04:34:38 +0000] "GET / HTTP/1.1" 304 0 "-" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/89.0.4389.72 Safari/537.36"
dockergen.1 | 2021/03/05 04:35:25 Received event die for container 915d8d387c99
dockergen.1 | 2021/03/05 04:35:25 Received event stop for container 915d8d387c99
dockergen.1 | 2021/03/05 04:35:25 Generated '/etc/nginx/conf.d/default.conf' from 2 containers
dockergen.1 | 2021/03/05 04:35:25 Running 'nginx -s reload'
dockergen.1 | 2021/03/05 04:35:25 Contents of /etc/nginx/conf.d/default.conf did not change. Skipping notification 'nginx -s reload'

Docker host

OS: Ubuntu 20.04
Docker version: Docker version 20.10.4, build d3cb89e

Additional info
I tested without the letsencrypt container (only Step 1 and Step 3), and I could access the the sub.hostname.tld. It's only after doing all 3 Steps will I get this redirect error.
I use Cloudflare for DNS and disabled Automatic HTTPS. I used Flexible mode for SSL, but I tried changing to No SSL and got same error still.
Does anyone know how to solve this problem? I believe this should be the simplest case for this, but I can't seem to get it working...

Hi. Are you sure your app (or Cloudflare) isn't sending 304 redirection to HTTP which, in turn get redirected to HTTPS by nginx-proxy, then redirected again, etc ?
Please note that this container (nginx-proxy/letsencrypt-nginx-proxy-companion) "only" purpose is to:

enable the automatic creation of certificates from containers env var.
put them in the right place with the right name so they can be used by nginx proxy's docker-gen template.
reload the nginx proxy when needed (cert creation or renewal).

So redirection errors cannot be caused by it, because it does not handle http request at all. If there is a redirection issue, only nginx-proxy/nginx-proxy can be causing it.

I am using cloudflare and having the same issue

Hi @farhadmammadli , could you try the steps I listed here ? https://github.com/evertramos/nginx-proxy-automation/issues/238#issuecomment-798224003


Tuesday 22-10-2024 Roll call

Roll Call!
Leave a comment with these two things:

A summary of your study using only emojis
Something you'd like to share, anything goes! (within respect)

emoji cheat sheet

🤪

hi

GOOD MORNING!

Morning!

good morning

Good Morning

salut

hi

🖐

Bonjour !

Bonjour 🎉

✌


Fix integration tests pipeline for API 21

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 69.98%. Comparing base (2529684) to head (c6765b9).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2197      +/-   ##
===========================================
+ Coverage    69.93%   69.98%   +0.05%
===========================================
  Files          726      726
  Lines        26967    26999      +32
  Branches      4526     4526
===========================================
+ Hits         18858    18895      +37
  Misses        6830     6830
+ Partials      1279     1274       -5

see 28 files with indirect coverage changes


OpenEye logging with a --verbose flag.

          (doing the official review thing)

Could you make OE logging an option via a --verbose flag?
Originally posted by @apayne97 in https://github.com/choderalab/asapdiscovery/pull/518#pullrequestreview-1662157576

@kaminow and I use the OE logging in separate text files to keep track of the errors, if you let it all go to stdout, if you're doing stuff in parallel it will all get overlapped and is impossible to parse


Simple output

I really like schema.rb as a documentation method and want to use dbxray as a way to replicate it. The current output of dbxray is extremely detailed and verbose (specifically the :raw part of each entry).
I would like to get a less detailed output somehow, one that retains the basics (column name and type and uniqueness and defaults etc), and maybe puts foreign keys or indexes or constraints in a single line or a two entry map (which columns, title).
Thank you so much, this tool is very very cool.

This is a great suggestion! I will find a way to work it in.

OK, I've added an :include-raw? option so that the raw output is opt-in. How's that work for you?

Looking great! I think there are some ways to make it work better for my use-case but I can do that with normal Clojure function-writing.

awesome! but also feel free to share your ideas, the library's very alpha so I love hearing what folks would like to see


[Suggestion] Natural briges between islands

Can be helpful for journey between islands, especially between different layers separated from each other.
I think it may look like spaghetti caves, may be more flat, or it can be like giant vines, ice spikes or roots, may be for different biomes different types

This mod isnt being updated anymore but this would also be too difficult to add anyways.


python whois package

From looking at the code and if I understand correctly the whois package being used is: https://pypi.org/project/whois . This is an old package and from the tests I did it is failing on many queries (mostly issues with parsing the response).
Are there any plans to move to the following whois package:  https://pypi.org/project/python-whois/ . It seems to work better on most queries. Also, it does direct queries and not via the whois executable.
Note that there may be some confusion as the debian package python-whois is actually using the package https://pypi.org/project/whois .

I've been bitten by this bug too. I'd like to get information on domains using whois, but the module fails with many of them.

Perhaps it could use rdap instead:  https://pypi.org/project/whoisit/

Package python3-whois has been replaced with custom WHOIS client implementation.


Repo Imports

It would be useful to be able to import scripts from GitHub repos (or other providers) to maximize re-usability.
The syntax could look something like this:
import 'github:remi05/uial/samples/taskbar.uial'

Added in the following commit.

Note that only public GitHub repos are supported and that the files are pulled from the master branch.


Nested list items fail spectacularly

Would be nice to have these, so we can show lists inside of the video column

You can do something like this for managing nested lists:
`ul {
list-style-type: disc;
margin-top: 5px !important;
margin-bottom: 5px !important;
margin-left: 5px !important;
padding-left: 20px !important;
}
ul ul {
list-style-type: circle;
padding-left: 10px !important;
margin-left: 10px !important;
}
ul ul ul {
list-style-type: square;
margin-left: 10px !important;
padding-left: 10px !important;
}
ul li {
margin-top: 1em;
}
`

Thanks, @chuckdude. The basic ul becomes the left-hand Video column, so that's not quite what I'm going for, but I see what you mean about the nesting.


[T4A7][W13-B2]YongLinHan

Ready for review

@ylhlh you seem to have good grasp of TDD :+1:
Some comments added. Please ack & close the PR after reading comments.


[Snyk] Security upgrade python from 3.9.12-slim-buster to 3.11.0b1-slim-buster

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

Dockerfile

We recommend upgrading to python:3.11.0b1-slim-buster, as this image has only 81 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

436
CVE-2022-1664 SNYK-DEBIAN10-DPKG-2847944
No Known Exploit

400
Information Exposure SNYK-DEBIAN10-GCC8-347558
No Known Exploit

400
Information Exposure SNYK-DEBIAN10-GCC8-347558
No Known Exploit

778
OS Command Injection SNYK-DEBIAN10-OPENSSL-2807585
No Known Exploit

778
OS Command Injection SNYK-DEBIAN10-OPENSSL-2807585
No Known Exploit

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Happy go lucky - why bumping across a minor python version, who would want a bot to do that? ... and even a beta target, crazy.


Boolean query works only with 0/1

This return a list of size 0.
Model.find(Model.class, "favorite = ?", "true")

where favorite is a boolean type inside Model class.
I read a bit about the issue and turns out this works fine (using 1/0 for true/false)
Model.find(Model.class, "favorite = ?", "1")

It'd be nice to be able to query boolean fields using true/false string instead of 0/1.

This is also happened in query builder.
Select.from(Model.class).where(Condition.prop("is_From").eq(true)).count()

I have to change true with "1"

We will take care of this to add as feature of sugar.

This is now solved in PR #553

This is still not working, Can anyone verify ?


⚠ TorrentLeech has degraded performance

In 42648f3, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 1367 ms

Resolved: TorrentLeech performance has improved in 9448dd6 after 6 minutes.


Tags 0.6.1, 0.6.0 missing

The latest versions on https://pypi.org/project/waybackpack/ are 0.6.1 and 0.6.0, but there are no such tags in this git repo.

Thanks, now fixed!


Playing around with the Offline Issue on iOS 15. (Both Brandosha’s Fork & Game Play)

So for the last few days I’ve been playing around trying to figure out how to get the offline mode to work again on both Brandosha’s Fork & the Original Game Play Fork. I’ve noticed that Brandosha’s Fork lasts offline for about 5 minutes if you have it closed then when you try to go on it say without any data or wifi connection it goes to offline. As to Game Play Fork in general if I try to go offline after closing and killing any data or wifi signal, safari immediately says I don’t have any connection and that leads to the offline issue. I haven’t tried playing around with the experimental features yet as iOS 15 itself is down to its final days of the Beta Testing and will soon launch Into the Public Update itself. Here’s a Screen Recording of what I mean.

Unfortunately GitHub is not letting me upload any screen recordings of screenshots. If there’s anywhere I can submit them please let me know. I would like for you guys to see the issue. Thank You so Much and Happy Saturday! 😊

cc @brandosha

Closing this as a dupe of #210 which I'm currently using to track offline support.


Social Preview Image for the Repo

Description

Create a social preview image used for the repo (Mikaela will upload this, and it'll be the preview image when the link to the repo is shared
This can be an image using Hacktoberfest's official fonts and colors for this year
1280×640px

Describe alternatives you've considered

N/A

Additional context

The image should have the repo name, some fun graphics that are Hacktoberfest themed and convey that this is an open source project for iOS developers of any experience level to contribute to!
Somewhere include my name Mikaela Caron as the maintainer

Please assign this to me

Please assign this to me

Hi @sarafurd  we haven't started Hacktoberfest yet, so I'm not yet assigning issues, but I will starting October 1st!

Would be happy to work on this @sarafurd

Would be happy to work on this @sarafurd

@chloebeau it'll be one person per issue!

I've tried to work on your issue, and I realized that it's something that a designer should do. I'm unassigning myself from the issue so that someone else can pick it up. Sorry for any inconvenience.

No problem! It doesn't have to be fancy at all, using Hacktoberfest's colors and fonts from the website would be totally fine
@chloebeau do you want to take this issue?

@chloebeau any progress on this?

Un-assigning because there's been no progress in 2 weeks

I've been trying some ideas. What do you think?

Love that!! Can you add my unicorn emoji after my username?

Ok. I had it there initially but I removed it because it didn't match the color scheme. Can you assign this to me. Here is  the new version.

For this can you just use the emoji, rather than the profile pic?

Here is the final version with your emoji and some grammar fixes

@rramirez-dev one tiny change, can you add a space after my name and the emoji
Also did you make this in Figma? If so DM me your email and I can add you to the official Figma board, and you could copy this there, so that I have the original file too, thank you!
And this can count as a Hacktoberfest PR!! Create a PR, and add this as a banner image to the DocC documentation!

Here is the updated image. I will also create a pull request

.

Closed with #240


improved py validation functions

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

Checked


Feature Request: use basename for extension-less files

I've been using this library upstream in a project of my own, and I've run across an issue that I'm not sure if it should be handled down here in this library or further upstream.
Would it be possible for this library to also handle "extension-less" files? (eg: Makefile, CNAME, Dockerfile)
I would think that for a file without a ., the basename would be returned instead.
extension('Makefile')
// 'makefile'

Of course, { preserveCase: true } can also remain as a feature:
extension('CNAME', { preserveCase: true })
// 'CNAME'

If that doesn't seem to belong here in this library, I totally understand and will handle it upstream instead. If you're interested, I'm more than willing to write a PR.

I also understand this would constitute a breaking change, since extension('Makefile') currently returns ''.
An alternative to a breaking change would be to put this behavior behind it's own option:
extension('Makefile', { basenameFallback: true })

I think I'd be fine with a breaking change. It makes sense for extension-less files to return the full name.

All right, I will work on a PR for you to review. :smile:

Please try if /[^.]*$/.exec(filename)[0] can satisfy all cases.

Turns out the single regex was slightly faster, so I went with it. Released 3.0.0 with this change.

Wonderful! Thank you very much!

And 3.0.1 is out. I totally forgot to support paths :)

Ah yeah, I was going to ask about that next, but it seemed like this library assumed you would use "basename" instead of a full path. (which is what I started using upstream)


Reverse key-filter arguments?

Currently, the order is { predicate : key }. Would it make more sense to have { key : predicate }?

I think {key : predicate} is more intuitive as we read from left to right*. But in the end, this is the same dilemma as with get/put:
(get.a.b *  f >> put.x.y is the most readable form as a whole (IMHO), while
(f * get.a.b >> put.x.y) makes more sense in terms of understanding what operations are applied to the data.
I think I prefer {key : predicate}, but I don't have a strong opinion about it.


Added badges and lanyards.

Default price. See email from Mary.

Hi @snim2 -- the change itself looks great, but the Travis checks are failing because you've not committed the changes to the output/ branch.  Could you run make build (or just wok) and then commit and push the changes that doing creates?

And if you've got any suggestions for how to make it easier not to be caught out by this, let me know.

Ah, so this happened because I edited the files from the in-browser editor in GitHub. I'll check everything out when I'm back in a sensible operating system later today. In the mean time I'll generate a PR to clarify this in the README.

Ah -- it hadn't occurred to me that the new way of doing things would prevent us from using GH's in-browser editor.

@inglesp I've not followed this at all, but can't the build just happen on Travis?

@d0ugal I'll have a go...


cmd/scollector: Propagate FullHost setting

I noticed our scollectors reported different host tags for internal metrics.

Good catch. Thanks.


billy peng ui extract

Closes #820
Description: Replaces Ni components w Tm

ok so i got voyager to run w the component library by going into node_modules/vue-loader and running yarn. This shouldn't be necessary of course, but might have something to do with this:
https://github.com/vuejs/vue-loader/issues/678
also as can be seen in the circle ci test it's a babel issue not understanding "import". I found a similar babel issue here:
https://github.com/babel/babel-loader/issues/149

wait now this branch it's working as expected after rm -r node_modules && yarn
might have been the naming corrections i made on tendermint/ui
circle is still failing of course : \

I guess that Babel is not correctly applied to the imported components.

@faboweb You're right, I just merged a PR to export babel-ified components.

I spent time trying to fix the tests here and on @tendermint/ui, without much progress. The fact that Babel is transitioning to version 7, Vue is transitioning to version 3, and Jest is falling behind [1] is making this extra difficult. There's a complex dependency chain filled with incompatibilities, and CircleCI throws another wrench into the gears.
An option we can look into in the near future is dropping Babel. We can start using ES modules in Electron already, with things like this [2][3]
[1] https://github.com/facebook/jest/issues/4842
[2] https://gist.github.com/smotaal/f1e6dbb5c0420bfd585874bd29f11c43
[3] https://cli.vuejs.org/guide/browser-compatibility.html#modern-mode

I'm getting errors like this with yarn testnet:
         ERROR in ../ui/src/components/TmPageHeader/TmPageHeader.vue
         Module not found: Error: Can't resolve 'vue-loader/node_modules/vue-style-loader' in '/Users/pz/Projects/ui/src/components/TmPageHeader'
          @ ../ui/src/components/TmPageHeader/TmPageHeader.vue 4:2-326
          @ ../ui/src/index.js
          @ ./node_modules/babel-loader/lib!./node_modules/vue-loader/lib/selector.js?type=script&index=0!./app/src/renderer/components/wallet/PageWallet.vue
          @ ./app/src/renderer/components/wallet/PageWallet.vue
          @ ./app/src/renderer/components ^\.\/.*$
          @ ./app/src/renderer/routes.js
          @ ./app/src/renderer/main.js
          @ multi (webpack)-dev-server/client?http://localhost:9080 webpack/hot/dev-server ./app/src/renderer/main.js

         ERROR in ../ui/src/components/TmPart/TmPart.vue
         Module not found: Error: Can't resolve 'vue-loader/node_modules/vue-style-loader' in '/Users/pz/Projects/ui/src/components/TmPart'
          @ ../ui/src/components/TmPart/TmPart.vue 4:2-320
          @ ../ui/src/index.js
          @ ./node_modules/babel-loader/lib!./node_modules/vue-loader/lib/selector.js?type=script&index=0!./app/src/renderer/components/wallet/PageWallet.vue
          @ ./app/src/renderer/components/wallet/PageWallet.vue
          @ ./app/src/renderer/components ^\.\/.*$
          @ ./app/src/renderer/routes.js
          @ ./app/src/renderer/main.js
          @ multi (webpack)-dev-server/client?http://localhost:9080 webpack/hot/dev-server ./app/src/renderer/main.js

I used module.exports in the index.js of https://github.com/nylira/vue-field and we've been using vue-field without problems for a long time. I wonder if we can solve a lot of these issues by not ES6 modules in https://github.com/tendermint/ui/blob/develop/src/index.js

This looks pretty decent. Awesome work @okwme and @nylira . What is the status of this PR? What is missing?

@faboweb still need to improve code coverage, although there's not really any new code besides VrToolBar. Actually maybe removing code damaged the coverage report as the missing governance tests now have a larger effect?
I think we need to figure out a system for deciding what goes into the ui library. There are a lot more common components but they're not being used outside of voyager atm. Should they be included in the library? in the PR? if they stay in voyager should we rename them all to Tm prefix? Vr? I did a search and replace on ni- -> tm- to fix inherited style changes, not sure if there are other implications that need to be considered.
Also still need to get them published on npmjs.com, i'm not sure how that permission works. @nylira you have access right? will future updates always need to go through you or should we automate it somehow? I hate seeing npm packages that are behind their project repos...

99% of this PR was due to @okwme awesome efforts. Great job.
I'd suggest renaming Vr modules to Tm just to keep everything under the same namespace. We can keep single-use Tm components in Voyager for now, but as we continue to build front end projects we'll likely reuse them--and that's when we can move them out from this repo.
If it's simple, I propose removing governance components from the coverage report. We're not working on governance yet. (We will be soon, as first version of governance is coming into the SDK).
I have access to deploying new modules to the tendermint npm organization. I can give you access too, @okwme. But you need to enable 2FA on your NPM account first. Let me know. I don't know if we can automate publishing because every publish requires you to enter your OTP. How do we give our CI the OTP without compromising security?

renamed Vr as well as the remaining Ni prefixed components
removed governance from the coverage report
merged/resolved everything from develop

@nylira I've added 2FA to my npmjs.com account, my username is okwme. Happy to publish tendermint/tm and edit this pr to pull directly from the npm repo.
As for automating w CI I think you're probably right and it's not worth it for the security concern. Maybe instead we just add a git hook to check for the version number and give a reminder to npm version patch and npm publish?

Awesome! :)

e2e tests failed but pass for me locally 😢
trying again...