Create a block to get the current level number

It would require one more RAM variable to keep stored the current background number; doesn't seem to be worth the trouble.


LightNovelPub doesn't seem to be working

The website seems to be loading fine but inside the app the output is no results. The search throws a JSON Parse Error: Unexpected identifier "A" error and also doesn't work.
Version: LNReader 1.0.18

It has the same issue as FastNovel & MTLNovel. They work locally but return an empty array when deployed.
I don't know how to fix this so I'll just update the status to down.

The same error still is shown while searching for novels.

fixed in @724d13173eacb2085d0ba06f903fb58593849044

Fixed in 724d13173eacb2085d0ba06f903fb58593849044


Add a link to a .NET version

Hey there,
I have just released a .NET wrapper for BLAKE3 at https://github.com/xoofx/Blake3.NET
This PR is adding a link to this version in the front readme.md

Thanks! I'll follow up with a link to some JNI bindings too.

(Also tweeted a link: https://twitter.com/Blake3Team/status/1335633286190821378)


Remove need to toggle vending machine button

About the PR

Removes the need to toggle off a vending machine button in order to dispense again.
Closes #15718
Why / Balance

QOL - fewer clicks to dispense multiple items
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Before

After

Changelog

:cl:

tweak: Reduced clicks needed to dispense vending machine items

I don’t see the difference

I don’t see the difference

It might be hard to see, but currently you have to deselect the current selection in order to vend again, so to get three of the same thing would be 5 clicks, but now it would only be 3.

I don’t see the difference

It might be hard to see, but currently you have to deselect the current selection in order to vend again, so to get three of the same thing would be 5 clicks, but now it would only be 3.

ohh sounds good

god bless


Date picker resizing problem

When you drag the date picker box to the right to reduce the input area, the picker will become smaller. It happened only last week.
Please verify. Tq

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a4fb137e-6a79-e538-a90a-0d17c5cfb8f7
Version Independent ID: 3d725d06-378d-4540-4fc4-6959d00eaf9e
Content: Date Picker control: reference - PowerApps
Content Source: powerapps-docs/maker/canvas-apps/controls/control-date-picker.md
Service: powerapps
GitHub Login: @fikaradz
Microsoft Alias: fikaradz

@hpkeong Thanks for your feedback!
@fikaradz Any ideas?

This is a bug which we have fixed.  We expect it to roll out to production in about 3 weeks.

Thanks so much for the quick fix. Appreciate


Resources as path parameters

This PR follows up on #590 based on our conversation at yesterday's meeting.
(It's also stacked on that PR for hopefully easier reviewing)
I implemented more or less exactly the API we discussed, with one change:  I added a single-key-lookup synchronous read method to the API, as it was used in some of the examples and feels much more natural this way than forcing creation of another resource, then calling getAll on that resource instance (which, using its params, filters out the desired key), then stripping away the redundant structure in the response (which would always have the form [[key, [val1, val2,...]]] instead of just [val1, val2, ...], etc.
So, I propose the API in this PR, with two synchronous read routes as documented in skipruntime-ts/server/src/server.ts and www/docs/resources.md, copied here:
Synchronous reads from reactive resources can either access the resource in its entirety or read the data for a single key, using routes:
POST /v1/snapshot/:resource
POST /v1/snapshot/:resource/lookup

The first route returns all of the values in the named resource, using the parameters provided in the JSON-encoded request body.
It instantiates the resource if needed, then returns a JSON-encoded array of key/value entries, with each entry a tuple of the form [key, [value1, value2, ...]].
The second route requires the request body to be a JSON-encoded value with a key field and a params field.
It instantiates the resource if needed, then returns a JSON-encoded array of all values associated to key in the resource.

It would be nice for reviewing to push #590 to origin and then set the base of this PR to that branch.


Update README.md

Reference to --> #15

Hi,
Thanks for the fixes. Some comments in diff :)

I've fixed the doc in the README in 3cf31e3 ;)


CI failures

TRAVIS

[x] on osx oldrel: Rcpp seams missing

GH Actions

[x] on macOS-latest devel: error Package required but not available: ‘golem’
[ ] on windows-latest (3.6): README.md warns that "This document format requires a nonempty  element."

rcpp installed on Travis
osx devel removed form github action


Basic Shooter Code

Completed and tested code that can spin the motors on the Shooter to launch game pieces.
Comprised of 3 new files: Shooter, ShooterIO, and ShooterIOTalonFX

Not ready to merge as of 1/20/2024 3:04 pm
Shooter being changed to a Top-Bottom shooter and will therefore require some testing to make sure motors spin in the correct directions and at fast enough speeds.

Code has been tested and is PR ready :D


Add a survey page when the extension uninstall

https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/runtime/setUninstallURL
Things need to consider:

The version
The browser
The OS

can you guide me, for implementing this, where we have to made changes in & how ? It will be more feasible to understand.

can you guide me, for implementing this, where we have to made changes in & how ? It will be more feasible to understand.

If you are not familiar with extension development, please read https://developer.chrome.com/docs/extensions/

We can use the setUninstallURL https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/runtime/setUninstallURL to set the URL.
In the meantime, we need to figure out the survey services that support:

Free for limited usage (<1000 per month)
Support pre-filled field (by URL query params is ideal) because we need to distinguish different OS, browser, and versions.
Generally available, i.e. not blocked by the Chinese government.

Codewise, it should be straightforward because we just need to gather the os, browser, extension version information and craft an URL in the file: https://github.com/xcv58/Tab-Manager-v2/blob/master/packages/extension/src/js/background.tsx#L11


[SPARK-21045][PYTHON] Allow non-ascii string as an exception message from python execution in Python 2

What changes were proposed in this pull request?
This PR allows non-ascii string as an exception message in Python 2 by explicitly en/decoding in case of str in Python 2.
Why are the changes needed?
Previously PySpark will hang when the UnicodeDecodeError occurs and the real exception cannot be passed to the JVM side.
See the reproducer as below:
def f():
    raise Exception("中")
spark = SparkSession.builder.master('local').getOrCreate()
spark.sparkContext.parallelize([1]).map(lambda x: f()).count()

Does this PR introduce any user-facing change?
User may not observe hanging for the similar cases.
How was this patch tested?
Added a new test and manually checking.
This pr is based on #18324, credits should also go to @dataknocker.
To make lint-python happy for python3, it also includes a followup fix for #25814

cc @HyukjinKwon,  @ueshin  and @cloud-fan

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

Test build #110987 has started for PR 25847 at commit 90559c0.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16091/
Test PASSed.

Test build #110987 has finished for PR 25847 at commit 90559c0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110987/
Test PASSed.

Test build #110997 has started for PR 25847 at commit fb72447.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16101/
Test PASSed.

@srowen, for instance, users could manually throw an exception with python native function execution like udf or rdd.

Test build #110997 has finished for PR 25847 at commit fb72447.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110997/
Test PASSed.

Test build #111002 has started for PR 25847 at commit ff7f248.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16104/
Test PASSed.

Test build #111002 has finished for PR 25847 at commit ff7f248.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111002/
Test PASSed.

Test build #111007 has started for PR 25847 at commit 0652966.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16108/
Test PASSed.

Test build #111009 has started for PR 25847 at commit ffb4d29.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16110/
Test PASSed.

Test build #111007 has finished for PR 25847 at commit 0652966.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111007/
Test PASSed.

Test build #111009 has finished for PR 25847 at commit ffb4d29.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111009/
Test PASSed.

Test build #111077 has started for PR 25847 at commit d6ec7ae.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16162/
Test PASSed.

Test build #111077 has finished for PR 25847 at commit d6ec7ae.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111077/
Test PASSed.

Merged to master.


Make use of clippy?

I played around with cargo clippy and rustler the other day and wondered if there is interest to bring that upstream. I fixed the warnings from cargo clippy --all-features --all-targets in a branch. The suggestions mainly aimed at removing superfluous (as of Rust 2018) lifetime annotations, removing unnecessary captures and replacing references where move is more appropriate due to Copy being implemented for a type. But clippy found two things which might be of interest:

bin.as_mut_slice().write(self.as_bytes()).expect("memory copy of string failed") should use write_all() instead of write, as write does not guarantee to write all at once
ptr.offset(offset as isize) should be ptr.add(offset), as offset can never be a negative number.

Yeah, I've used it in the past. I probably should have run it with all the recent changes I did 🦀


feat(keymaps): Add syntax to filter keymaps from UI dynamically

Resolves: #256
How to Test

Create a keymap with custom filters like { '<C-v>', description = 'filtree open in split', filters = { filetype = 'filetree' } }
Create a new buffer and :set filetype=filetree
The keymap you created in step 1 should only appear in the UI when triggered from the filetree buffer
Repeat steps 1-3 but for a buftype instead of a filetype
Also test with a custom function filter as described in the changes to KEYMAPS.md

Testing for Regressions
I have tested the following:

[x] Triggering keymaps from legendary.nvim in all modes (normal, insert, visual)
[x] Creating keymaps via legendary.nvim, then triggering via the keymap in all modes (normal, insert, visual)
[x] Triggering commands from legendary.nvim in all modes (normal, insert, visual)
[x] Creating commands via legendary.nvim, then running the command manually from the command line
[x] augroup/autocmds created through legendary.nvim work correctly

Please update docs on that doc/legendary-keymap-tables.txt, currently options aren't listed!

It’s documented: https://github.com/mrjones2014/legendary.nvim/blob/master/doc/legendary-tables.txt#L35


uncommited changes only used in template repos without tags

Describe the problem
According to this thread: https://github.com/copier-org/copier/discussions/743 copier is supposed to use uncommited changes by default. But it only does so when no tags exist in the template repository.
To Reproduce
mkdir template
cd template
echo "Hi" > test.txt
git init
git add .
git commit -a -m "1"
git tag v0.0.1
echo "Hi2" > test.txt
copier copy . /tmp/foo
cat /tmp/foo/test.txt

The  result is Hi
Expected behavior
Hi2
Environment

OS: Linux
Copier version: 7.0.1
Python version: 3.11.0
Installation method: pipx

The reproduce steps are wrong. Copier is doing what it is supposed to do AFAICS.
Could you try with this command instead?
copier -r HEAD copy . /tmp/foo

@yajo
mkdir template
cd template
echo "Hi" > test.txt
git init
git add .
git commit -a -m "1"
git tag v0.0.1
echo "Hi2" > test.txt
copier -r HEAD copy . /tmp/foo
cat /tmp/foo/test.txt

results in:
Hi

which acording to your comment here: https://github.com/copier-org/copier/discussions/743#discussioncomment-3453570 should be Hi2 to my understanding

@yajo After thinking about it again today,  I think -r HEAD should not use dirty changes.
-r HEAD is telling copier explicitly to use the vcs HEAD which means "latest commit" in git. So using uncommited changes would be unexpected.
Currently the existence of tags in the git repo changing the behavior is really surprising so I would love to see it gone.
Some options how to design the behavior:
1. By default for local path use checkout

in case there is a version tag it would change from "use latest tag" to "use current checkout"

2. By default for local path use newest git commit

in case there is a version tag it would use the latest tag (currently behavior)
in case there is no version tag use the latest commit (currently it uses the checkout including changes)

Now we would need a way to tell copier to include dirty changes:
2.1 special -r tag
It should be something that is not already used terminology in git, like HEAD. So maybe CHECKOUT or CURRENT. While unlikely it brings the possibility to have a name clash with a branch.
2.2 separate flag in addition to -r
Instead of using -r at all when wanting to use the current directory "as is" have a new flag, e.g. --allow-dirty. When used in combination with -r it should abort with an error.

However, 2.2 seems a good option to me. I'd call it --dirty-head, so when this flag is true, then HEAD does the magic; otherwise, it just behaves as a predictable HEAD would behave in normal git.


[tutorial] Use GitHub branches to generate StackBlitz links

Changes the StackblitzIntro component to take a GitHub tree branch and generate both a GitHub and StackBlitz link from it.

LGTM! @Jutanium


allow changing grafana datasource ssl mode

Allow for Postgres instance where SSL is required.
No CA verification, but that's another show.

I requested a pull request for this about a year ago, but it hasn't been reviewed yet.
#2846

Closed as duplicate of #2846


Bug: getVar does not return null when item doesn't exist

Describe the bug
getVar() doesn't return null when the item I'm attempting to retrieve does not exist
CodeIgniter 4 version
4.1.3
Affected module(s)
IncomingRequest?
Expected behavior, and steps to reproduce if appropriate
Inside of any controller, dump the contents of getVar() with any parameter you'd like:
public function update() {
	var_dump($this->request->getVar('description'));
}

Now, using Postman, I made a raw JSON request (HTTP PUT) with the following body (empty works, too):
{
	"name": "test"
}

Expected behaviour: NULL should be printed, as per the documentation
Actual behaviour: string(0) "" is printed.
Context

OS: Windows 10
Web server: Apache
PHP version: 7.4.3

Resolved via #4909


exclude sending two notification it sent to yourself

exclude sending two notification it sent to yourself

Let's close this for now


API: exception thrown in Watney API when limitThreads is not set

Noticed by a user:
API configuration defaults to value 0 for limitThreads if it is not specified at all in the configuration file. This can happen when the user upgrades to the latest version while still using the old config file that does not contain the new parameter.
This causes an exception down the line when the value is about to be used, as 0 threads is just wrong.
Mitigation:
Add line
limitThreads: -1

(or any other valid number > 0) to config.yml to make things work.

Fixed and v1.2.1 of API released.


Add flush on close

What is this PR doing?
Adding a flush option to the #close method, defaulting to true so that the metrics are flushed before closing the socket.
Motivation
Correctly flush last metrics on close.

@remeh Thank you for merging this fix into master.
Can I somewhere look up the release schedule?
Do you know when this fix will be released?


support diagnostics and goto definition in virtual files loaded in IDE

diagnostics and goto definition in virtual files loaded in IDE
support for :ref type checks

Codecov Report

Merging #262 (a44d9c0) into main (e3dced7) will decrease coverage by 3.54%.
The diff coverage is 10.73%.

@@            Coverage Diff             @@
##             main     #262      +/-   ##
==========================================
- Coverage   50.05%   46.51%   -3.55%
==========================================
  Files         113      121       +8
  Lines       15248    16446    +1198
  Branches     1058     1062       +4
==========================================
+ Hits         7633     7650      +17
- Misses       7615     8796    +1181

Impacted Files
Coverage Δ

packages/transforms/src/index.ts
82.87% <0.00%> (ø)

packages/typescript-plugin-vue/src/index.ts
0.00% <0.00%> (ø)

packages/typescript-plugin-vue/src/init.ts
0.00% <0.00%> (ø)

...e-languageservice/src/contracts/LanguageService.ts
0.00% <0.00%> (ø)

...nguageservice/src/features/CallHierarchyService.ts
0.00% <0.00%> (ø)

...ue-languageservice/src/features/CommentsService.ts
0.00% <0.00%> (ø)

...languageservice/src/features/CompletionsService.ts
0.00% <0.00%> (ø)

...languageservice/src/features/DeclarationService.ts
0.00% <0.00%> (ø)

...-languageservice/src/features/DefinitionService.ts
0.00% <0.00%> (ø)

...languageservice/src/features/DiagnosticsService.ts
0.00% <0.00%> (ø)

... and 26 more

Continue to review full report at Codecov.

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


amp-fit-text padding breaks the amp-fit-text

Describe the bug
The amp-fit-text places the text up against the containing box.
To look nice, how about allowing CSS padding? Or some way to add space between the text and containing box?
To Reproduce
https://codepen.io/iamalta/pen/pZGozr

...

<amp-fit-text width="200" height="64" layout="responsive" class="p2">Life's Sayings Plaques and Stones</amp-fit-text>

I tried increasing width and height of .fixedblock with no success.
Expected behavior
Padding on left/right, top/bottom.
Screenshots
See above link
Desktop (please complete the following information):
any
Smartphone (please complete the following information):
not tested
Additional context

Hi, I think the main goal was to show how the amp-fit-text wraps the text and not sure how adding a padding may help with this. @sebastianbenz WDYT?

The problem is that <amp-fit-text> creates a div, to measure the text at various sizes while trying to find the right one, but does not account for any CSS on the Element itself.
This will also cause problems if you have other CSS (e.g. if you use word-wrap: break-word), that might cause the text to lay out differently.
I think this should be possible to fix. In the short term, you can add any CSS styles that you want to a parent container of <amp-fit-text> (except for line-height).
I created an issue in the amphtml repo to track this.

Thank you @sparhami.
You guys are great! I appreciate all that you do.


ConnectionTimeoutException

Is the following caused by a bug in the library or something to contact mlab.com about?
ConnectionTimeoutException in Collection.php line 466:
No suitable servers found (serverselectiontryonce set): [connection timeout calling ismaster on '*******.mlab.com:*****']

same error here

same error here

same here

This is because your mongodb service is not running. Check the status of your mongodb service by typing :
service mongod status .
If the status is failed or inactive, try to run the service again by typing
service mongod start
and then again check the status of mongodb service

I get this error too. The weird part is that at times it shows up 5 times in a row and the 6th time it works fine like nothing happened before.

@prashant-pokhriyal solved it to me.


POST request specs missing in rendered docs

Running the provided example documentation at https://sphinxcontrib-openapi.readthedocs.io/#how-to-use
    post:
      summary: Create an Evidence
      description: |
        Creates a new evidence record in database.
      parameters:
        - name: evidence
          in: body
          schema:
            $ref: '#/definitions/Evidence'
      responses:
        201:
          description: An evidence.
          schema:
            $ref: '#/definitions/Evidence'

renders as:

I was also unable to get any description of the body in my own api specs using a parameter: block with in: body, or with the openapi 3.0 requestBody: block.  Is there anything I need to configure in order for rendered docs to display a spec of the request body for POST requests?  If this is intended let me know, I'm relatively new to writing openapi docs

This pull request resolves it:
Request body is missing in rendered docs  #47:
https://github.com/sphinx-contrib/openapi/pull/47

I asked this in #47, but I'll repeat this question here too. Is this ticket about what's called :examples: on our docs? Because it looks so. :examples: actuall generates an example snippet based on requestBody. It's turned off by default because in my opinion, it does too much and lacks some testing. I'm going to work on this and improve it in near future.

/cc @albert2126 @ptdel

@ikalnytskyi Yes, at the time I was unable to get any sort of schema to render via any of the methods in the issue in either the example or just some other yaml spec.  Sorry it's been a little bit so it's not as fresh in my mind now.  I can understand not wanting to re-create functionality. However, if @albert2126 has a PR that has high test coverage that fits in form and factor with the existing codebase is there any reason that this can't be merged in, with the untested functionality being removed/replaced?

I never said it was untested. I said it lacks some tests. You see, the examples functionality is better and worse at the same time. It's better because it does not blindly shows you a schema. Instead, it tries to produce and show a valid payload with fake data. It's worse because such functionality has a bunch of nuances, such as different combination of types and formats, etc. What I'm saying is that I'm not quite sure we covered all possible invariants. It works great on all samples specs I tested, yet, I'm not that confident that it works for all the cases, that's why it's turned off by default.

Only my opinion, but I would prefer that a renderer did simply return the schema blindly to me.  I would think that the functionality of dynamically injecting mock data to a produced schema would be outside the scope of producing the schema, and more suited for an extension or external method of some sort.  If this PR implements the simple functionality of producing a valid request body, I would still ask if there are any reasons that it could not be merged. As the saying goes, a bird in the hand is worth two in the bush.
We've had to move on from using the extension as it stands, but it would certainly make life easier to be able to utilize it again if it's rendering is comprehensive.

I would still ask if there are any reasons that it could not be merged.

Yes, there are. Here's an incomplete list if you are interested:

No tests. Not even a single one.
Code duplication.
Hardcoded values of Host and Content-Type in rendered HTTP example (while in fact it's not even an example, but schema).
Bad assumption that a body contains only application/json content that crashes with any other Content-Type value.

Besides, I still do not understand why are you so oppose using :examples: option? Does it break some use case for you? Can you please elaborate on this? Because I finally found some time to work on this, and your use case may help me to understand better what's broken and how can I fix it.

Thank you guys for the discussion.
As a matter of a fact, an example is not enough in most cases because it often does not show all properties that can be used in a POST or PUT request. This makes a developer to open the YAML file to find the full specification on the required request.
The proposed update doesn't touch the example section. It just adds the request body section.
This is how Swagger-UI does.
I'm studying your notes above to improve the code.
Thanks,
Albert

@ikalnytskyi Thanks for the explanation, gotta have tests :slightly_smiling_face: .  The issue was created in regards to other keywords that were not rendering properly in the spec, I don't have any particular issue with examples:
@albert2126 I would take a look at writing tests in pytest and tox.  Most folks will want code coverage included in PRs.  I'd also recommend reading up on TDD.  You're on the right track

@ptdel Thanks a lot for your support. I've added the new option ":request:" to the test_openapi3_success() function.
Let me know guys if I did something wrong. :)

I'm working as an API documentator, so I know that the Request Body section is absolutely necessary. Otherwise PUT and POST method descriptions are incomplete.
I hope also that someone will make long sections (examples and request/response body) collapsible.
In my documents, I can make the whole "openapi" admonition section collapsible, but it would be nice to have the same possible for the sections inside that admonition.

Has this been fixed? I just downloaded and installed package, and did a build. I didn't get any POST request body rendered.

Hi @lilisha100,
Yes it works if you add the :request: option there. The POST request body is not shown by default for backward-compatibility purpose.
This is an example that I just tested on sphinxcontrib-openapi version 0.5.0:
.. openapi:: ../../schemas/cmp.json
:paths:
/customers
:request:
:examples:
When build, the document displays both, the POST request and examples.
Unfortunately, this option is missing in the documentation.

Hi @albert2126 ,
I've tried the :request: option and it works but gives me a warning rst:24: WARNING: Explicit markup ends without a blank line; unexpected unindent. But I don't have that many lines.
My rst is as follow:
.. openapi:: /openapi/auth.yml    :request:
Is there any way to get rid of the warning?

Hi @albert2126 ,
I've tried the :request: option and it works but gives me a warning rst:24: WARNING: Explicit markup ends without a blank line; unexpected unindent. But I don't have that many lines.
My rst is as follows:
.. openapi:: ../../schemas/cmp.json
   :request:

Is there any way to get rid of the warning?

Hi @PabloSanchez95 PabloSanchez95,
In reST such cases are common. Ensure that you have an empty line after the markup block. Try to use spaces instead of the tab before ":request:".


Regular expression behind /* comment */ is not highlighted in TypeScript

let regex = /**/ /foo/;

A new repo is created to converge the efforts for refining syntax files: https://github.com/Microsoft/TypeScript-TmLanguage. Future issues regarding tmlanguage syntax should be opened there. I'm importing this issue to that repo and closing it here.


VLC mediaplayer integration example for android

p4a vlc recipe required

I see you've made some pep8 commits, the log here contains a complete list of style issues to fix: https://travis-ci.org/kivy/kivy/jobs/88336479.

I don't feel that this should lend into kivy itself. It would be much better for you to commit in kivy-garden in order to maintain it later.
We're not going to handle any maintenance on that part, it's too specific i guess.

@ibobalo nice work!
However I share @tito 's sentiment. This is clearly a lot more than just a example.
The right place for this is inside a garden module, garden is essentially a extensions kind of a aggregate which aggregates user created widgets/modules/extensions.

@ibobalo I've created https://github.com/kivy-garden/garden.androidvlc and added you as a maintainer with write access.
Feel free to push your work there, thanks a lot for your work so far. :cherries:


Error: Attribute without value

<form inline></form>

Error: Attribute without value
Line: 20
Column: 53
how could I solve the problem?

same problem for me


OLS-886 snapshot_to_catalog.sh produce 2 channels with multiple bundle versions

Description
The script snapshot_to_catalog.sh produce 2 channels alpha and preview into the catalog.
The channel preview has the latest version and skips all older versions.
The channel alpha has all versions provided to the script, one replacing the closest older one.
Here is a catalog generated from command ./hack/snapshot_to_catalog.sh -s ols-cq8sl,ols-mdc8x
More detail see this PR
---
defaultChannel: preview
icon:
  base64data: ...
name: lightspeed-operator
schema: olm.package
---
image: registry.redhat.io/openshift-lightspeed-beta/lightspeed-operator-bundle@sha256:fd3849cb65fee7586b261c8c67336d1d0e5d98e89ae10f87f7ca7945d585b0ff
name: lightspeed-operator.v0.1.3
package: lightspeed-operator
properties:
  - type: olm.gvk
    value:
      group: ols.openshift.io
      kind: OLSConfig
      version: v1alpha1
  - type: olm.package
    value:
      packageName: lightspeed-operator
      version: 0.1.3
  - type: olm.bundle.object
    value:
      data:  ...
relatedImages:
  - name: lightspeed-service-api
    image: registry.redhat.io/openshift-lightspeed-beta/lightspeed-service-api-rhel9@sha256:9f4b71d0bfd0cdda5f872adf38a792eb7a0f0cba9097496cb1843b05f6f3e890
  - name: lightspeed-console-plugin
    image: registry.redhat.io/openshift-lightspeed-beta/lightspeed-console-plugin-rhel9@sha256:ebe5b5bbe3af1d6089667a001686ad152084a684f021abc52ec2884b7bae790b
  - name: lightspeed-operator
    image: registry.redhat.io/openshift-lightspeed-beta/lightspeed-rhel9-operator@sha256:af95acfcb1e685a8023cad68707e60cb81de2e117ce0d438e6481dbee3b60841
  - name: lightspeed-operator-bundle
    image: registry.redhat.io/openshift-lightspeed-beta/lightspeed-operator-bundle@sha256:fd3849cb65fee7586b261c8c67336d1d0e5d98e89ae10f87f7ca7945d585b0ff
schema: olm.bundle
---
image: registry.redhat.io/openshift-lightspeed-beta/lightspeed-operator-bundle@sha256:f467fbe0cb7dcf39a8caf8218b96e8e31f7077a256aaf95df6d4143cb3f49139
name: lightspeed-operator.v0.1.4
package: lightspeed-operator
properties:
  - type: olm.gvk
    value:
      group: ols.openshift.io
      kind: OLSConfig
      version: v1alpha1
  - type: olm.package
    value:
      packageName: lightspeed-operator
      version: 0.1.4
  - type: olm.bundle.object
    value:
      data: ...
relatedImages:
  - name: lightspeed-service-api
    image: registry.redhat.io/openshift-lightspeed-beta/lightspeed-service-api-rhel9@sha256:aef85c0c8fa86f7d669a54e82b2cb01cbb3f08959c78ac389f95024aae8f5b96
  - name: lightspeed-console-plugin
    image: registry.redhat.io/openshift-lightspeed-beta/lightspeed-console-plugin-rhel9@sha256:7fb081a73bc72f135f7b3fc42306372308e1c73e51149aede4bd63e7d76444f6
  - name: lightspeed-operator
    image: registry.redhat.io/openshift-lightspeed-beta/lightspeed-rhel9-operator@sha256:5bdb7a15cd2f2700841b0959da0c90134551d4fc34abdcd912746d3fbc5d77e4
  - name: lightspeed-operator-bundle
    image: registry.redhat.io/openshift-lightspeed-beta/lightspeed-operator-bundle@sha256:f467fbe0cb7dcf39a8caf8218b96e8e31f7077a256aaf95df6d4143cb3f49139
schema: olm.bundle
---
schema: olm.channel
package: lightspeed-operator
name: preview
entries:
  - name: lightspeed-operator.v0.1.4
    skipRange: ">=0.1.0 <0.1.4"
---
schema: olm.channel
package: lightspeed-operator
name: alpha
entries:
  - name: lightspeed-operator.v0.1.3
    skipRange: ">=0.1.0 <0.1.3"
  - name: lightspeed-operator.v0.1.4
    replaces: lightspeed-operator.v0.1.3

Type of change

[x] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[x] Configuration Update
[ ] Bump-up dependent library

Related Tickets & Documents

Related Issue # OLS-886
Closes #

Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

example updated, PTAL

/lgtm
/approve


Respond status.

Hi there.
I have a problem with status returning.
@model = Model.create(some_params)
respond_with @model

I need to return 200 if everything is ok, and 422 if some validations failed. But I always get 200.
I thought about
respond_with @model, status: @model.errors.any? ? :unprocessable_entity : :success

But as for me it looks not so good
How may I fix it?

That should be the default behavior - please provide a more detailed report (rails/responders versions, controller code or even a test case/sample app that replicates the issue) so we might figure out what might be wrong on your end.

rails 5.0.beta3
responders 2.1.2
class ArticlesController < ApplicationController
  respond_to :js

  def create
    @article = Article.create(article_params)
    respond_with @article
  end

  private

  def article_params
    params.require(:article).permit(:title, :text)
  end
end

When my article has errors I get rollback and http status 200 and my article is created I get status 200 too

Hi, I'm encountering this exact same behaviour, with an out-of-the-box install of respondes.
I'm using the reponders gem v 2.2.0 and Rails 4.2.6 if it helps, and my implementation of responders is completely vanilla.

I mean that it's good idea if model object has been successfully created  responders sends 201 status, if not then return 422 or someone another status code

Hey guys, thanks for creating and maintaining this gem. I'm facing the same issue and unable to tell if this is the intended behaviour for an html/js request. As far as I dug both format would return a 200 OK in case of validation errors. The behaviour since to be existed since this gem was created (when it used to leave in rails core).
While other mime type will respond with a 422 , html and js request doesn't have the same behaviour
Thanks!

PR #176 added the ability to pass in a customized render option, which should allow you to respond with a different status.
I think we should still be returning 422 Unprocessable Entity for any validation errors response, even for html/js, but that's a different change that might break people's apps.

This is now causing issues with Turbo 7. Turbo requires that a success returns a redirect and so 422 should be sent on validation errors for html

@simmerz what about json?

Thanks for pointing that out @simmerz, I understand that this isn't working with Turbo out of the box, and it will require a new release with this breaking change to get it back on track... it's on my radar alongside a few other upgrades I'm doing.

Thanks for pointing that out @simmerz, I understand that this isn't working with Turbo out of the box, and it will require a new release with this breaking change to get it back on track... it's on my radar alongside a few other upgrades I'm doing.

For those experiencing this issue, set 'data-turbo': false on your Devise forms to maintain the existing behavior.

For those experiencing this issue, set 'data-turbo': false on your Devise forms to maintain the existing behavior.

For those experiencing this issue, set 'data-turbo': false on your Devise forms (the submit button, in particular) to maintain the existing behavior.

Note that this won't work if you're using a Turbo native adapter.

For those experiencing this issue, set 'data-turbo': false on your Devise forms (the submit button, in particular) to maintain the existing behavior.

Note that this won't work if you're using a Turbo native adapter.

Thanks, @ghiculescu 👍

Thanks, @ghiculescu 👍

FYI: I'm working on this in #223, it should work with HTML responses now (returning 422) so it should work with Turbo/Hotwire, but I have to look at JS responses which was the original issue / report.
If anyone can/wants to give that a shot, please do and report back. Thanks.

FYI: I'm working on this in #223, it should work with HTML responses now (returning 422) so it should work with Turbo/Hotwire, but I have to look at JS responses which was the original issue / report.
If anyone can/wants to give that a shot, please do and report back. Thanks.


[FLINK-31643][network] Introduce the configurations for tiered storage

What is the purpose of the change
Introduce the configurations for tiered storage
Brief change log

Introduce the configurations for tiered storage.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)

@flinkbot run azure

Thanks for the review 😄 @TanYuxin-tyx @reswqa . Please take a look.

@flinkbot run azure

Finally, CI passed 🤣, merging...


build fail traffic_portal

I am building the traffic port following the instructions of traffic control I created the file *.rpm.Since I want to build it with ubuntu, I converted the rpm file to a deb to build with the command sudo apt-get install *.deb , and i use systemctl start traffic_portal to run traffic port but it gives error and doesn't enable localhost

Ubuntu is not supported. Versions 5.1.6 and 6.1.0 support CentOS7 and CentOS8, and the current master branch builds support CentOS7 and Rocky Linux 8.
Please refer to the documentation for information on what Traffic Control is and how to use it. If you need help troubleshooting or want clarification about something, please ask in our Slack channels or on whichever of the ATC mailing lists would be appropriate.


feat : added the Hijri Date and alined it with the Gregorian Date in …

added feature :

Hijri Date
styling of the Dates

LGTM!

Thank you for your contribution!


test coverage tool

Does anyone know a good test coverage tool? This way we can make sure we have full coverage of our code.

Istanbul? https://github.com/gotwarlost/istanbul

I have absolutely no experience with coverage tools in the JS world yet, so Instanbul sounds fine to me. The most important thing is that it doesn't get confused by babel ES6 transpilation, I think.

Ahah: https://github.com/ambitioninc/babel-istanbul

I copied the test coverage script from redux. Our current status:
=============================== Coverage summary ===============================
Statements   : 100% ( 35/35 ), 1 ignored
Branches     : 100% ( 8/8 ), 1 ignored
Functions    : 100% ( 11/11 )
Lines        : 100% ( 24/24 )

Yay! I had to wipe my node_modules and do another npm install, but after that it all just worked.
We should document the basic commands we offer somewhere so that people know what's available.


⚠ Rumah123 has degraded performance

In c44c727, Rumah123 (https://www.rumah123.com) experienced degraded performance:

HTTP code: 200
Response time: 6114 ms

Resolved: Rumah123 performance has improved in f796ba0.


[ADAM-1634] Add algorithm benchmarks to documentation.

Resolves #1634.

Woot!  LGTM

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/2497/
Test PASSed.

Thank you, @fnothaft!


Extracted @govtechsg/open-certificate

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[docs] Introduce custom Jekyll tags for better glossary integration

Some Context:
Jekyll supports integrations like custom tags (e.g. think {% include %}, but with behavior that you define yourself). @heckj is working on the user journey and is need of an easy way to bring in the glossary work.
TLDR:
This PR implements tags to help you incorporate glossary definitions when needed. For the full spec, see the README that I added in the _plugins/ directory.
CC @chenopis and @bradamant3 who are actively working on user journeys as well

This change is

oh additional note worth mentioning: I had to disable the Github Pages gem in favor of Jekyll gem(s) because the former doesn't support custom plugins (only if they're from "core").
Locally, the site seemed to still render alright---and since we're hosting on Netlify, not GH pages, I think the update should be ok. Will double check once the deploy preview is out.

@abiogenesis-now 👋 If you tell me what to look for on the staging server, I'm happy to verify it.

@zacharysarah Thanks! So the new tags are being used in two places:

In the brief summary of StatefulSets here: https://deploy-preview-5954--kubernetes-io-master-staging.netlify.com/docs/concepts/workloads/controllers/statefulset/

As a tooltip for Pod here: https://deploy-preview-5954--kubernetes-io-master-staging.netlify.com/docs/tasks/inject-data-application/define-command-argument-container/

If those look ok to you we should be good to go I think.

@abiogenesis-now After testing, it looks like the tags on the first page don't render correctly, while the tag on the second page works.
First page (glossary links don't work):

Second page (glossary link works):

ahh @zacharysarah good catch! that is very important 😅
will fix and ping you when updated

@zacharysarah @chenopis when either of you have a chance, do you mind taking a look at this PR again? I know it's rather long, but I've fixed the link issues that Zach pointed out with the previous iteration. Thank you!!
TLDR: Now if you use the glossary_tooltip tag, you get a tooltip AND a link, which either:

goes to the full-link specified in the YAML file of that glossary term
falls back to the glossary entry

Screenshotted below:

@zacharysarah @steveperry-53 PTAL when you get a chance also. I'm not sure who can do the tech review though.

Now if you use the glossary_tooltip tag, you get a tooltip AND a link

🎉 I like it! I'm not sure about tech review, either—but that seems like the sort of thing we should have in our pocket. Maybe @ahmetb? (Hi there!)

Hi, I haven't been following the space and am very much far away from what’s been happening in Glossary work. I do not carry any Jekyll expertise either, so I cannot be very helpful. I’m sorry.

@zacharysarah Well, this looks like pretty standard plugin code. The JS is pretty short and straightforward. Unless you see anything buggy in the preview, I think we can go ahead and merge it. I tested it as best as I could.

Thanks for the review everyone! Just let me know if you have any remaining questions!

@chenopis Agreed, nothing I’m seeing raises concern. @abiogenesis-now :shipit: :tada:

@ahmetb Thanks for responding, no worries!


WIP -- Working to support migration-scoped queries from VTGate

This will be WIP for a few weeks, but I wanted to start having travis build/test it now as I do some testing internally.  I'll ping for a review when ready.
The goal here is to provide basic initial support for running migrations through VTGate. For now I am using ExecuteShards. In the future we use a different fronting endpoint, and can unify with schemamanger/schema_swap. But first step is to be able to route them through VTGate at all.
I'll be adding to this PR as I encounter blockers during my internal tests.

@sougou if you can take a look when you have a chance. This is working for me now, I was able to run a number of migrations through this. I commented on one area we should specifically look at.
Another gotcha I'd like to tackle is:
Even though I'm using ExecuteShards, I had to turn strict mode off. I'd like to have an in-between mode. Everything else other than migrations should be treated as strict. We shouldn't be doing sharding or filtered replication while running a migration.
I have a separate branch for supporting ExecuteShards in the JDBC client.

I'm not sure what's up with travis. The endtoend tests fail, looks like they fail trying to create a json column in mariadb, with maria returning a syntax error. Running them locally works, but they've consistently failed in travis.

Are sure that's the problem?
There's a hanging transaction and then a stacktrace gets printed?
Hanging may be here?

/home/travis/gopath/src/github.com/youtube/vitess/go/vt/tabletserver/endtoend/transaction_test.go:316 +0x1ec

https://travis-ci.org/youtube/vitess/jobs/207189600

Hmm not totally sure how to grok those various stacktraces around the hang. I'm guessing it could be any one of them? I'm not able to reproduce locally, but I did make changes to the DDL transactions in query_executor.go. So will try to see if I can reproduce or narrow it down.

Not sure why I wasn't triggering the failure locally, but I pushed a fix and Travis is happy again!

Not sure why I wasn't triggering the failure locally, but I pushed a fix and Travis is happy again!

Travis usually has higher load and therefore it sometimes exposes races which don't show up on a beefy, idle workstation. Some suggestions to reproduce such a problem:

Compile the test binary (go test -c) and run it in a while true loop in your shell. This allows you to run the test more often. I understand that's not really an option for the slow endtoend test ;(
You could try to run go test with -race which slows down the execution.
Do other things to increase the load on your computer e.g. compile the Linux kernel :)

I've pushed a version where we close the open transaction. I'm going to do some internal testing to see how that goes. I am mildly concerned that the transactionIDs will somehow be impacted in vtgate and the JDBC driver, but we'll see.

The new approach does not work, sort of predictably on reflection. Liquibase tries to call commit at the end, and the transaction does not exist anymore. So vttablet returns:
vttablet: rpc error: code = 10 desc = transaction<PHONE_NUMBER>635636179: not found

I've noticed that commit in mysql is a no-op if there is no transaction. I wonder if the same should happen in vitess.

The connections used by vitess all have autocommit=ON. This is required for connection pools. We have three options:

We could recognize a DDL on the client side and short-circuit the commit that follows.
We can start a new transaction on that same connection by issuing a begin and Recycle it. Maybe implement a BeginAgain call on the TxConn.
We can mark the TxConn as committed and just return it to the pool on the next commit, but disallow other statements.

I've implemented an initial solution using option 2. I'm testing it out internally while travis runs, to see if this works for the Liquibase use-case.

@sougou this worked for me. If you're ok with the approach I will add tests to ensure we are not leaking transactions or anything for the defined use-case.

Thanks for the feedback. Made changes based on suggestion. Testing in my environment now

Thanks!

I saw travis failed after we merged somehow. I just kicked off another build and if it continues to fail I'll try to fix in the morning

Everything looks good on re-run


EnterTransition of HomeActivity  is not work

I can't find any enterTransition animation on the fab in the HomeActivity

Fixed in 039eee4


Chocolatey Helper Methods

Does it make sense to expose UI components to make calls to the Chocolatey Helper Methods?
Such as PinnedItems, DesktopShortCut, etc?
https://github.com/chocolatey/chocolatey/wiki/HelpersReference

I'm not sure I understand this issue.

@RichiCoder1 the thought process here was to expose the Chocolatey Helper Methods for doing certain things, like creating shortcuts, pinned items, etc, for the installed packages.  i.e. when someone really wants a desktop shortcut for an application, and one wasn't created.  Provide a UI to allow them to enter the necessary information, and then click a button, rather than having to do anything else, like drop to the command line, etc.
Not sure if this is really relevant anymore.

I think my vote is no at this point.


Do not retain messages if retain is not available

Following #255, I noticed that s.Options.Capabilities.RetainAvailable is advisory only - it only indicates to the client that retain is unavailable, but does not actually disable the feature. This PR prevents messages from being retained if s.Options.Capabilities.RetainAvailable = 0.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 3 (33.33%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 98.59%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

server.go
1
3
33.33%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
5246

Relevant Lines:
5321

💛  - Coveralls


Update greeter.json

changed end as 24 in the good evening greeting. 0 didn't work

I've been trying to figure that out myself. Many thanks for this PR.


Code coverage

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


[No QA] Create callable worklow for createNewVersion

Details
This change is part of a larger undertaking to clean up our GitHub Actions to make them faster, simpler, and more approachable.
Fixed Issues
$ (partial) https://github.com/Expensify/Expensify/issues/195693
Tests

Merge PR A with the checklist unlocked.
A staging deploy should occur and when it completes PR A should get a deploy comment and should be added to the checklist.
Merge PR B with the checklist locked.
PR B should get a comment saying it's not being immediately deployed.
Merge PR C with the checklist locked and with a CP Staging label.
PR C should be CP'd to staging. It should be added to the checklist and when the deploy completes it should get a deploy comment.
Close the checklist.
A production deploy should occur, and PRs A and C should receive a deploy comment.
PR B should be deployed to staging and receive a deploy comment.

We also might be able to simplify this callable workflow by using secrets: inherit:

If the secrets are inherited using secrets: inherit, you can reference them even if they are not defined in the on key.

This sounds convenient, we just have to decide if it's safe practice. I actually think it will be safe for any callable workflow that we own and which comes from the same repo

Created an issue for the last actionlint failure: https://github.com/rhysd/actionlint/issues/174

https://github.com/Expensify/App/actions/runs/2659622994

🚀 Deployed to production by @chiragsalian in version: 1.1.84-13 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


hard to copy agent id from the agent api.   from gui

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
and also to have both that and api key ready to use agnt and an agent. thanks.
its jus really hard to grab and a copy button would be great
📝 Additional Context
No response
Acknowledgements

[X] My issue title is concise and descriptive.
[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for the maintainers to understand and evaluate this request.

@arwyn6969 this is now fixed


Twipsy 'live' binder causes slowdown

When using 'live' binder and reloading a large  (80kb of HTML) containing Twipsys several times, page slows down a lot, resulting in heavy lag. Refreshing page fixes that. Forcing 'bind' binder fixes it permanently.

yeah - that's why it's an option - delegation of every hover can get slow for sure :)


Issue with image 14.4.35602.0

I'm having trouble with the latest BC image.
Same error for sandbox and onprem.
Previous image 14.3.34444 works fine.
I have tried removing all containers and images and reinstalled docker, but with no luck.
Suggestions?

SCRIPT:
install-module navcontainerhelper -force
$accept_eula = $true
$containername = 'bcno'
#$navdockerimage = 'mcr.microsoft.com/businesscentral/sandbox:14.3.34444.35600-no'
$navdockerimage = 'mcr.microsoft.com/businesscentral/sandbox:14.4.35602.0-no-ltsc2019'
#$navdockerimage = 'mcr.microsoft.com/businesscentral/onprem:no'
$appbacpacuri = ''
$tenantbacpacuri = ''
$license = 'REPLACE WITH LICENCE LOCATION'
$additionalParameters = @()
if ($appbacpacuri -ne '' -and $tenantbacpacuri -ne '') {
$additionalParameters = @("--env appbacpac=""$appbacpacuri""","--env tenantBacpac=""$tenantbacpacuri""")
}
$credential = get-credential -UserName $env:USERNAME -Message "Using Windows Authentication. Please enter your Windows credentials."
New-NavContainer -accept_eula:$accept_eula -containername $containername -auth NavUserPassword -Credential $credential -includeCSide -alwaysPull -doNotExportObjectsToText -usessl:$false -updateHosts -assignPremiumPlan -shortcuts Desktop -imageName $navdockerimage -additionalParameters $additionalParameters -licenseFile $license -usebestcontaineros
Setup-NavContainerTestUsers -containerName $containername -password $credential.Password

Just copy/pasted your script and it seems to work fine.
I suspect that the docker file system is corrupt - and you might need to reset to factory defaults (which removes all images and containers) - and then try again

I tried factory reset, but still same error. I have also tried uninstalling and running this script:
https://success.docker.com/article/how-to-completely-remove-docker-in-windows-10
Still the same problem.
It seems like it has to do with my computer, as i tested it on a colleague's computer, and it worked.
But another colleague had the same problem earlier this week and for him it helped to reinstall.
Do you have another suggestion? If not, I will try to reinstall again.

Very very very very strange
Could you post the full output of the container creation here?
(then I can see various version numbers)

Give me a minute, I'll just run the script again :)

PS C:\WINDOWS\system32> C:\Sharepoint\Auzilium AS\Gruppeområde - Dokumenter\Løsninger\Kom i gang\CreateDockerContainer.ps1
NavContainerHelper is version <IP_ADDRESS>
Host is Microsoft Windows 10 Pro - 1903
Docker Client Version is 18.09.0
Docker Server Version is 18.09.0
Pulling image mcr.microsoft.com/businesscentral/sandbox:14.4.35602.0-no-ltsc2019
14.4.35602.0-no-ltsc2019: Pulling from businesscentral/sandbox
Digest: sha256:e47d4b7e7856b96f99c21ce3daa666ec028ae84f173c56334b53cbe2f7adad92
Using image mcr.microsoft.com/businesscentral/sandbox:14.4.35602.0-no-ltsc2019
Removing container bcno
Removing bcno from hosts
Removing C:\ProgramData\NavContainerHelper\Extensions\bcno
Creating Container bcno
Using license file C:\Sharepoint\Auzilium AS\Gruppeområde - Dokumenter\Løsninger\Kom i gang\AuziliumLicense.flf
Version: 14.4.35602.0-NO
Platform: 14.0.35570.0
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.17763.678 (ltsc2019)
Host OS Version: 10.0.18362.295 (1903)
A better Generic Container OS exists for your host (1903)
Using generic image mcr.microsoft.com/dynamicsnav:generic-1903
Pulling image mcr.microsoft.com/dynamicsnav:generic-1903
generic-1903: Pulling from dynamicsnav
Digest: sha256:010767c45e06fd4d59a005a3e5e4ce47cb3f4e7994b4b9f7a208e2d018e6e0ca
Generic Container OS Version: 10.0.18362.295 (1903)
Generic Tag of better generic: <IP_ADDRESS>
Using locale nb-NO
Using hyperv isolation
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Files in C:\ProgramData\NavContainerHelper\Extensions\bcno\my:

AdditionalOutput.ps1
AdditionalSetup.ps1
license.flf
MainLoop.ps1
SetupNavUsers.ps1
updatehosts.ps1
Creating container bcno from image mcr.microsoft.com/dynamicsnav:generic-1903
b178bb1f1b88d1a4cf0f3126724aa0d87c3dc3c2e1a6e12051d5fd7676dbeb04
Waiting for container bcno to be ready
Installing Business Central
Installing Url Rewrite
Installing OpenXML
Installing DotNetCore
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
Copying Web Client Files
Copying Windows Client Files
Copying PowerShell Scripts
Copying ConfigurationPackages
Copying Test Assemblies
Copying TestToolKit
Copying UpgradeToolKit
Copying Extensions
Copying ClientUserSettings
Copying ReportBuilder
Changing Database Server Collation to Norwegian_100_CS_AS
SQL Server 2017 transmits information about your installation experience, as well as other usage and performance data, to Microsoft to help improve the product. To learn more about SQL Server 2017 data processing and privacy controls, please see the Privacy Statement.
Copying Cronus database
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Starting Business Central Service Tier
Installation took 121 seconds
Installation complete
Initializing...
Starting Container
Hostname is bcno
PublicDnsName is bcno
Using NavUserPassword Authentication
Creating Self Signed Certificate
Self Signed Certificate Thumbprint 255959AD2AA4E3610CAF5670A92A2E3931BC942D
Modifying Service Tier Config File with Instance Specific Settings
Restarting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
The data is invalid. (Exception from HRESULT: 0x8007000D)
at Set-NavSiteAuthenticationSettings, C:\Program Files\Microsoft Dynamics NAV\140\Web Client\Modules\NAVWebClientManagement\NAVWebClientManagement.psm1: line 654
at New-NavWebSite, C:\Program Files\Microsoft Dynamics NAV\140\Web Client\Modules\NAVWebClientManagement\NAVWebClientManagement.psm1: line 631
at New-NAVWebServerInstance, C:\Program Files\Microsoft Dynamics NAV\140\Web Client\Modules\NAVWebClientManagement\NAVWebClientManagement.psm1: line 501
at , C:\Run\SetupWebClient.ps1: line 26
at , C:\Run\navstart.ps1: line 152
at , C:\Run\start.ps1: line 121
at , : line 1Error
Installing Business Central
Installing Url Rewrite
Installing OpenXML
Installing DotNetCore
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
Copying Web Client Files
Copying Windows Client Files
Copying PowerShell Scripts
Copying ConfigurationPackages
Copying Test Assemblies
Copying TestToolKit
Copying UpgradeToolKit
Copying Extensions
Copying ClientUserSettings
Copying ReportBuilder
Changing Database Server Collation to Norwegian_100_CS_AS
SQL Server 2017 transmits information about your installation experience, as well as other usage and performance data, to Microsoft to help improve the product. To learn more about SQL Server 2017 data processing and privacy controls, please see the Privacy Statement.
Copying Cronus database
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Starting Business Central Service Tier
Installation took 121 seconds
Installation complete
Initializing...
Starting Container
Hostname is bcno
PublicDnsName is bcno
Using NavUserPassword Authentication
Creating Self Signed Certificate
Self Signed Certificate Thumbprint 255959AD2AA4E3610CAF5670A92A2E3931BC942D
Modifying Service Tier Config File with Instance Specific Settings
Restarting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
The data is invalid. (Exception from HRESULT: 0x8007000D)
at Set-NavSiteAuthenticationSettings, C:\Program Files\Microsoft Dynamics NAV\140\Web Client\Modules\NAVWebClientManagement\NAVWebClientManagement.psm1: line 654
at New-NavWebSite, C:\Program Files\Microsoft Dynamics NAV\140\Web Client\Modules\NAVWebClientManagement\NAVWebClientManagement.psm1: line 631
at New-NAVWebServerInstance, C:\Program Files\Microsoft Dynamics NAV\140\Web Client\Modules\NAVWebClientManagement\NAVWebClientManagement.psm1: line 501
at , C:\Run\SetupWebClient.ps1: line 26
at , C:\Run\navstart.ps1: line 152
at , C:\Run\start.ps1: line 121
at , : line 1
Initialization of container bcno failed
At C:\Program Files\WindowsPowerShell\Modules\navcontainerhelper\<IP_ADDRESS>\ContainerHandling\Wait-NavContainerReady.ps1:45 char:17

...             throw "Initialization of container $containerName failed" ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (Initialization of container bcno failed:String) [], RuntimeException
FullyQualifiedErrorId : Initialization of container bcno failed

PS C:\WINDOWS\system32>

The error:

Did you add other parameters (like isolation hyperv) to the command?
(that is not in the sample above)

Same script, only license location that is different:
install-module navcontainerhelper -force
$accept_eula = $true
$containername = 'bcno'
#$navdockerimage = 'mcr.microsoft.com/businesscentral/sandbox:14.3.34444.35600-no'
$navdockerimage = 'mcr.microsoft.com/businesscentral/sandbox:14.4.35602.0-no-ltsc2019'
#$navdockerimage = 'mcr.microsoft.com/businesscentral/onprem:no'
$appbacpacuri = ''
$tenantbacpacuri = ''
$license = 'C:\Sharepoint\Auzilium AS\Gruppeområde - Dokumenter\Løsninger\Kom i gang\AuziliumLicense.flf'
$additionalParameters = @()
if ($appbacpacuri -ne '' -and $tenantbacpacuri -ne '') {
$additionalParameters = @("--env appbacpac=""$appbacpacuri""","--env tenantBacpac=""$tenantbacpacuri""")
}
$credential = get-credential -UserName $env:USERNAME -Message "Using Windows Authentication. Please enter your Windows credentials."
New-NavContainer -accept_eula:$accept_eula -containername $containername -auth NavUserPassword -Credential $credential -includeCSide -alwaysPull -doNotExportObjectsToText -usessl:$false -updateHosts -assignPremiumPlan -shortcuts Desktop -imageName $navdockerimage -additionalParameters $additionalParameters -licenseFile $license -usebestcontaineros
Setup-NavContainerTestUsers -containerName $containername -password $credential.Password

could you try to remove the -usebestcontaineros

I see that the reason why it selects hyperv isolation is that you are running docker 18.09.0.
That version and all versions before only supports process isolation of windows server.
You should really reset to factory again (sorry for the download needed afterwards) and update docker to 19.03.2
and then try again.

Removing -usebestcontaineros worked :)
I updated the docker on a previous attempt, but I probably selected an earlier installation file when I tried again. I'll see if it still works after updating.

-usebestcontaineros will allow you to run process isolation on newer versions of docker.
Not sure why it would cause this problem - but good if it works.

Just tried to run the same and force hyperv - still works here...

It still works after I updated the docker, without -usebestcontaineros. Hard to say what's wrong when it works for most people. Thanks for the help :)

Gr8


feat: replace less with minus

Closes https://github.com/bensadeh/tailspin/issues/95
It's mostly done. I need to fix some bugs (like opening a huge file, somehow breaks the pager and takes a lot of time) and especially hope that https://github.com/arijit79/minus/pull/114 gets merged so we have the follow-mode with minus as well :)
I'll also add more docs if you approve the code (I don't want to remove more lines if you don't like them ;-;).
You can try it out if you want :)
but now I should really go to bed

Hi again @TornaxO7 and thanks for putting this PR together. I haven't tested the branch out yet, but here are some of my thoughts:

Since replacing the default pager is quite a large change to the core functionality (or presentation) of the highlighted output, I suggest putting outputting to minus behind a flag like so: tspin --minus app.log. This will allow us to have both, eventually swapping out one as the default pager and putting the other behind a flag later on.
Instead of modifying the current code, I'd like to extend the functionality by adding a minus reader trait implementation (This is the open-closed principle).
Same with the writer and presentation traits, I'd like to have them because it decouples them nicely and lets us have the logic for stdin and stdout in a more isolated way.
One more thought about the writer trait. Is minus reading everything from memory? Currently, tailspin writes a temporary file with the highlighting applied. I don't know how this is done with minus , but if it is all in-memory, it could be an issue with large files as large log files could be larger than the available memory.
I like the EOFSignaler you made as its own type, but let's not introduce it in this PR to keep it as small as possible if that's okay.

Let me know if you're stuck on some parts and we'll see if we can simplify or change some things around.

Since replacing the default pager is quite a large change to the core functionality (or presentation) of the highlighted output, I suggest putting outputting to minus behind a flag like so: tspin --minus app.log. This will allow us to have both, eventually swapping out one as the default pager and putting the other behind a flag later on.
Instead of modifying the current code, I'd like to extend the functionality by adding a minus reader trait implementation (This is the open-closed principle).
Same with the writer and presentation traits, I'd like to have them because it decouples them nicely and lets us have the logic for stdin and stdout in a more isolated way.

ooooof, I though that we replace less with minus so we don't have any external dependencies anymore and minus will be fully written in rust xD Welp... bad communication from my side. I'll have to revert my changes then... ._.

One more thought about the writer trait. Is minus reading everything from memory? Currently, tailspin writes a temporary file with the highlighting applied. I don't know how this is done with minus , but if it is all in-memory, it could be an issue with large files as large log files could be larger than the available memory.

good point! If I'm seeing it correctly minus buffers the text, which should be displayed in a String but we can fix this by handling a buffer on our own (or use your file) and set the text manually which minus should display.

hm... so would you like to have minus as the default pager?

hm... so would you like to have minus as the default pager?

I think the discussion of whether to have minus as the default pager can come later down the line.
For the first version, I'd like to be able to use it with the --minus flag for example. This way, we can release a new version with this feature and let people test and try out. I can also add an option to set an environment variable to have this as the default for those who want.

ooooof, I though that we replace less with minus so we don't have any external dependencies anymore and minus will be fully written in rust xD Welp... bad communication from my side. I'll have to revert my changes then... ._.

No problem, my friend.
Like I mentioned earlier, I have tried to decouple the highlighting logic from the presentation logic. The idea behind this is that we should be able to easily swap out these parts since they don't have anything to do with each other.
We already have a less presenter, so I suggest we start with creating a minus presenter that will be used when we rung tailspin with the --minus flag.

good point! If I'm seeing it correctly minus buffers the text, which should be displayed in a String but we can fix this by handling a buffer on our own (or use your file) and set the text manually which minus should display.

I suspect that having minus read from our temporary file will solve any memory issues, but I might be on thin ice here. In any case we need to take into account large files.
So to sum up, a way forward could be creating a minus presenter which implements the Present trait and still reading from the temporary file the same way less is doing. It will be used with the --minus flag.
Lastly, it is important that the minus is able to pick up newline entries somehow. This is already handled with the temp file, as we continuously watch the original file and write to the temp file if new entries come in.

I just noticed that minus does not allow to read directly from file like less does, so my approach above would not work in the current version.
I still think it is worth looking more into. I'd like to take another look at integrating minus when I have time. I've tried to make tailspin be agnostic when it comes to how things are presented (to pager vs. stdout etc.). How we display the output can be configurable. It would be interesting to get user feedback on how the minus pager works.
So to sum up, I'll have a look at this again as well, and if there aren't any big blockers (performance, memory etc.) we can look into integrating tighter with tailspin.
Open to thoughts and your ideas as well.


zbundle fails with internal error during start

[Mon24 07:16] - PrefabCore.py     :1123:j.prefabcore         - INFO     - RUN:smartctl --version
[Mon24 07:16] - PrefabSmartmontools.py:27  :j.prefabsmartmontools - INFO     - smartctl version 6.6 is present
Traceback (most recent call last):
File "/etc/register_capacity.py", line 15, in
_, resp = client.api.RegisterCapacity(data)
File "//lib/python/JumpScale9Lib/clients/grid_capacity/api_service.py", line 167, in RegisterCapacity
resp = self.client.post(uri, data, headers, query_params, content_type)
File "//lib/python/JumpScale9Lib/clients/grid_capacity/http_client.py", line 62, in post
return self._handle_data(uri, data, headers, params, content_type, self.session.post)
File "//lib/python/JumpScale9Lib/clients/grid_capacity/http_client.py", line 58, in _handle_data
res.raise_for_status()
File "//lib/python/requests/models.py", line 937, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 500 Server Error: Internal Server Error for url: https://capacity.threefoldtoken.com/api/nodes
[Mon24 07:19] - ErrorHandler.py   :257 :j.core.errorhandler  - ERROR    - 500 Server Error: Internal Server Error for url: https://capacity.threefoldtoken.com/api/nod

Please use https://hub.grid.tf/maxux/fullcapacity.flist flist on zbundle to register capacity and tell me if it's fixed or not for you.

@maxux shouldn't we promote this flist?

If we confirm it works, yes

@dmitrydelyukin please download the lastest zbundle build from : https://download.gig.tech/gig-autobuilder/threefoldtech-0-bundle-master-67a2042cce.tar.gz
extract the binary from there and run it with this flist: https://hub.grid.tf/maxux/fullcapacity.flist
that should do the trick.
We'll have to clean up a bit our doc to make sure we point to the proper zbundle build

/etc/start: line 4: /env.sh: No such file or directory
Generating public/private rsa key pair.
/root/.ssh/id_rsa already exists.
Overwrite (y/n)? /etc/start: line 10: js_config: command not found
registring farmer with ID:
Traceback (most recent call last):
File "/etc/register_capacity.py", line 1, in
from jumpscale import j
File "/usr/lib/python3/dist-packages/jumpscale.py", line 2, in
from Jumpscale.core.JSBase import JSBase
ImportError: No module named 'Jumpscale'

@dmitrydelyukin please download the lastest zbundle build from : https://download.gig.tech/gig-autobuilder/threefoldtech-0-bundle-master-67a2042cce.tar.gz
extract the binary from there and run it with this flist: https://hub.grid.tf/maxux/fullcapacity.flist
that should do the trick.
We'll have to clean up a bit our doc to make sure we point to the proper zbundle build

Was testing the Zbundle, but there are DNS errors all over. Not sure if the Zbundle uses a custom DNS,
|                 |
|                 |
|       o         |
|  . o =.o        |
|   + O.ES        |
|    X.O.o o      |
|.o *.B . *       |
|o.o+O.oo+ o      |
|+.oO*o+.o=       |
+----[SHA256]-----+
[Sun21 12:41] - LoggerFactory.py  :236 :logger               - DEBUG    - start re-init for logging
* Will start agent
 9201 ?        Ss     0:00 ssh-agent -a /tmp/ssh-glVPrp66uk3V/agent.1654
12091 ?        S+     0:00 /bin/bash -c ps ax | grep 'ssh-agent'
12093 ?        S+     0:00 grep ssh-agent

* load ssh agent
* load ssh key: /root/.ssh/id_rsa
Identity added: /root/.ssh/id_rsa (/root/.ssh/id_rsa)
Lifetime set to 86400 seconds

registring farmer with ID: eyJhbGciOiJFUzM4NCIsInR5cCI6IkpXVCJ9.eyJhenAiOiJ0aHJlZWZvbGQuZmFybWVycyIsImV4cCI6MTUzMjUwNzUwMywiaXNzIjoiaXRzeW91b25saW5lIiwicmVmcmVzaF90b2tlbiI6IlRORFJoWkhNNHZMR
lJHcFprLVVvZWdabnRkMjEiLCJzY29wZSI6WyJ1c2VyOm1lbWJlcm9mOm1henJhYSJdLCJ1c2VybmFtZSI6ImFyZWhtYW4ifQ.j-
[Sun21 12:41] - capacity_parser.py:61  :pacity.capacity_parser - ERROR    - error fetch location: HTTPSConnectionPool(host='geoip-db.com', port=443): Max retries exceeded with url: /json (C
aused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7fc1c68d0c50>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution',))
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/urllib3/connection.py", line 159, in _new_conn
    (self._dns_host, self.port), self.timeout, **extra_kw)
  File "/usr/local/lib/python3.5/dist-packages/urllib3/util/connection.py", line 57, in create_connection
    for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
  File "/usr/lib/python3.5/socket.py", line 732, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno -3] Temporary failure in name resolution

During handling of the above exception, another exception occurred:

any workaround/solution? do I need to change something in config?

New image is coming
On Mon, 22 Oct 2018 at 23:05, dmitrydelyukin<EMAIL_ADDRESS>wrote:

any workaround/solution? do I need to change something in config?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/threefoldtech/jumpscale_lib/issues/148#issuecomment-431985962,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD8yEHxBQ0DeiHFmAqysIGz_z6NKIk64ks5unjMRgaJpZM4W2LfD
.

Please try again my flist: https://hub.grid.tf/maxux/fullcapacity.flist

Seems to be working now ! The node gets registered and shows on the portal
10670 ?        S+     0:00 /bin/bash -c ps ax | grep 'ssh-agent'
10672 ?        S+     0:00 grep ssh-agent

* load ssh agent
* load ssh key: /root/.ssh/id_rsa
Identity added: /root/.ssh/id_rsa (/root/.ssh/id_rsa)

registring farmer with ID: eyJhbGciOiJFUzM4NCIsInR5cCI6IkpXVCJ9.eyJhenAiOiJ0aHJlZWZvbGQuZmFybWVycyIsImV4cCI6MTUzMjUwNzUwMywiaXNzIjoiaXRzeW91b25saW5lIiwicmVmcmVzaF90b2tlbiI6IlRORFJoWkhNNHZMRlJHcFprLVVvZWdabnRkMjEiLCJzY29wZSI6WyJ1c2VyOm1lbWJlcm9mOm1henJhYSJdLCJ1c2VybmFtZSI6ImFyZWhtYW4ifQ.j-

[Tue23 06:31] - register_capacity.py:19  :j.zbundle_capacity   - INFO     - capacity registered

@abdulgig ok cool, we will promote the flist and update our doc, but happy that you are unblocked

can't start zbundle.
same error
/etc/start: line 4: /env.sh: No such file or directory
Generating public/private rsa key pair.
/root/.ssh/id_rsa already exists.
Overwrite (y/n)? /etc/start: line 10: js_config: command not found
registring farmer with ID:
Traceback (most recent call last):
File "/etc/register_capacity.py", line 1, in
from jumpscale import j
File "/usr/lib/python3/dist-packages/jumpscale.py", line 2, in
from Jumpscale.core.JSBase import JSBase
ImportError: No module named 'Jumpscale'

@dmitrydelyukin which flist are these logs coming from ?

I use https://hub.grid.tf/maxux/fullcapacity.flist
do I need to create new organisation and get new farmer id?

Can u make sure u remove all working folders of zbundle
try
rm -rf ~/.zbundle*

Then try again

Seems @dmitrydelyukin  still have issue with the flist:
Reading package lists...
Building dependency tree...
Reading state information...
redis-server is already the newest version (2:3.0.6-1).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
vm.overcommit_memory = 1
Traceback (most recent call last):
  File "/etc/register_capacity.py", line 17, in <module>
    _, resp = client.api.RegisterCapacity(data)
  File "/opt/code/github/threefoldtech/jumpscale_lib/JumpscaleLib/clients/threefold_directory/api_service.py", line 167, in RegisterCapacity
    resp = self.client.post(uri, data, headers, query_params, content_type)
  File "/opt/code/github/threefoldtech/jumpscale_lib/JumpscaleLib/clients/threefold_directory/http_client.py", line 62, in post
    return self._handle_data(uri, data, headers, params, content_type, self.session.post)
  File "/opt/code/github/threefoldtech/jumpscale_lib/JumpscaleLib/clients/threefold_directory/http_client.py", line 58, in _handle_data
    res.raise_for_status()
  File "/usr/local/lib/python3.5/dist-packages/requests/models.py", line 940, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: https://capacity.threefoldtoken.com/api/nodes
[Tue20 08:54] - RedisFactory.py   :230 :j.clients.redis      - INFO     - redis-server --port 6379 --unixsocket /tmp/redis.sock --maxmemory 100000000 --daemonize yes
[Tue20 08:54] - Jinja2.py         :40  :j.tools.jinja2       - DEBUG    - template render:/opt/code/github/threefoldtech/digital_me/DigitalMeLib/data/schema//templates/template_obj.py
[Tue20 08:54] - Jinja2.py         :40  :j.tools.jinja2       - DEBUG    - template render:/opt/code/github/threefoldtech/digital_me/DigitalMeLib/data/schema//templates/schema.capnp

@dmitrydelyukin can you retry with the same flist, it was updated right now.
If everything is okay for you, I'll promote it.

This document was updated:
https://github.com/threefoldtech/home/blob/master/docs/farmers/zbundle.md
The flist was promoted. Please everybody confirm this can be closed.


model.parse() called incorrectly in certain cases of collection.set()

It seems that if you call collection.set() with an array of mixed models and attributes, and options {parse: true}, Backbone will call parse on the model's attributes.  Perhaps that is the desired behavior, but I definitely didn't expect it.  A quick example:
var TestModel = Backbone.Model.extend({
    parse: function(r, options) {
        if (r.other_value != null) {
            r.translated_value = r.other_value + ' MB';
            delete r.other_value;
        } else {
            r.translated_value = 'Not set';
        }
        return r;
    }
});

var m = new TestModel({other_value: 64}, {parse: true});
var c = new Backbone.Collection([m]);
c.set([m, {other_value: 128}], {parse: true});
if (m.get('translated_value') !== '64 MB') {
    alert('what!!');
}

I created a jsfiddle of it as well: http://jsfiddle.net/qL49q5vb/2/

@ralfthewise first of all you fogget create a collection proto with model attr, second Backbone Debug for Chome it must have for all who develop backbone application.
And here is more normal way to use Model.parse jsfiddle
(With your example you always get a first model attr translated_value)
Open the debuger and you will see 2 models in collection with 64 and 128 mb.
Thanks.


Rework firewall rules to include a broad RFC1918 deny all

There's already good rules in place, but consolidation makes it easier to reason about.
https://help.ui.com/hc/en-us/articles/115010254227-UniFi-USG-Firewall-How-to-Disable-InterVLAN-Routing#option 2

Finished this Sunday evening 7/30. Used the RFC1918 alias to create a rule and block VLAN traffic (confirmed).


Implement workaround for opacity-order bug

Hacky workaround to fix opacity until the bug as detailed in vispy/vispy#1541 is fixed.
See:

#12
https://github.com/Napari/napari-gui/pull/21#issuecomment-433547202

Hello @kne42! Thanks for submitting the PR.

There are no PEP8 issues in the file gui/layers/_visual_wrapper.py !


fix autocomplete property #527

What:
Changes Autocomplete attribute to off.

Why:
Nope is ignored on current Chrome 67 and Opera 54.

How:
Changed the attribute and updated test snapshots

Checklist:

[x] Documentation
[x] Tests
[x] Ready to be merged

[x] Added myself to contributors table

should fix [#527]

Ah, looks like you'll need to update the snapshots. Please see the contributing guidelines to see how to do that. Thanks!

Oh, just noticed it's the react-native snapshots which are a tad harder to update. You can run:
npm run build
npx jest --config other/react-native --updateSnapshot

I think that should do it :) Thank you!

Thanks!

Thx @kentcdodds ! :)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Navbar removed

I have removed the Navbar and made your website lookk cleaner. Please consider this as a HAcktoberfest accepted label to your repo. Regards
Atharva

Why you removed the Navbar @Atharva1802, I neither assigned you this issue nor I asked you to do so.


DB Config Loading can't cope with a .pgpass with no correct entry in it, or one with incorrect permissions

(You'll want to put "print(DB_URI)" in config.py to help see this)
Imagine a situation where the default DB connection strings are fine. Make sure there is no .pgpass file and there is no Environmental variable.  Run cli, and you should see:

postgresql://ocdsdata:ocdsdata@localhost/ocdsdata

All is good.
Now create a .pgpass file. Leave it empty, or put another entry in for a different database. This scenario could arise when one user account uses several different apps that connect to different Postgres DB's. Now run it, and you'll see:

postgresql://ocdsdata:None@localhost/ocdsdata

And it will crash with

FATAL:  password authentication failed for user "ocdsdata"

This is because we are only checking for the FileNotFound error! If the file is found but there is no entry, it returns None for password and then our script tries to use that.
Also, change file permissions on .pgpass to 644 or something and run it again. It will crash with:

pgpasslib.InvalidPermissions: Invalid Permissions for /home/vagrant/.pgpass: 0o644

The script should catch this error, warn the user, and then ignore the contents of the file!
(See #117 )

Should be covered on 87bdc73141e58ea8040668397a410d1d52fd122b at #117


ANC Subsequent Visit - fixes 02

#256
#267
#259
#154

Changes are pretty much trivial, so I'll go ahead and merge, to pass this to QA


update found document,get it by id an ddelete by id with documentation

update found document,get it by id an ddelete by id with documentation

added


Keep the order defined in the YML lists

TreeSet reorders defined list into ordered set by the name. We should really keep the ordering defined in the config.

@The-Huginn please review


delete curl part if force

Hi,
When we fail stop a running download, sometime server doesn't support resume
curl: (33) HTTP server doesn't seem to support byte ranges. Cannot resume

Actually I remove *.part for example:
23:49:50 in .../aurutils/sync/python-azure on  master ?
» ❯ ll
.rw-r--r-- 12M cyril 25 Mar 23:49 -N azure_3.0.0.tar.gz.part

-> It'll be easy and good to detect this error and so, start from begin the download, and/or force delete of part file before start if force flag is passed.
Thanks for this project
ping @maximbaz ;)

The master switched to wget a while ago, I guess this is some makepkg error?

I have not look yet which call curl, I will keep a look

If it's indeed makepkg, I'm inclined to leave things - solutions to such issues belong to makepkg itself, not its consumers.

https://bugs.archlinux.org/task/29326


目前使用clip4clip在时间均匀采样下只能提取12帧吗

Is there an existing issue for this?

[X] I have searched the existing issues

What would you like to be added?
我想知道clip4clip对一段视频进行向量化时最多只能提取12帧，多余12帧的话会报错
i
mport towhee
from towhee import pipe,ops
from time import time

device = 'cpu'
pipes = (
    pipe.input('path')
    .map('path','flames',ops.video_decode.ffmpeg(sample_type='uniform_temporal_subsample', args={'num_samples': 12})) #
    .map('flames','vec',ops.video_text_embedding.clip4clip(model_name='clip_vit_b32', modality='video', device=device))
    .output('vec')
)
# video_path = r'E:\video_danganguan\0451-2001-006-1139.mp4'
video_path = r'E:\data\video\cartoon.mp4'
s = time()
a = pipes(video_path)
e = time()
print('time:',e-s)

Why is this needed?
No response
Anything else?
No response

hi @Terran0629 , the code limits the max frame to 12： https://towhee.io/video-text-embedding/clip4clip/src/branch/main/clip4clip.py

hi ，代码将最大帧限制为 12：https://towhee.io/video-text-embedding/clip4clip/src/branch/main/clip4clip.py

Thanks for your reply! I'm modifying the frame rate in the source code, so it shouldn't affect its functionality, right?

Changing the frame rate does not affect the model's operation, but the model was trained with 12 frames, so altering it may impact the results.


Feature: turn off class-transformer for an individual route

Hi all,
One pain point I have is when using class-transformer I can't seem to return a buffer because the class-transformer keeps trying to process it.
I use class-transformer for almost all routes but I need some individual routes returning stuff that express can handle (such as buffers) but shouldn't be processed by class-transformer.
The current workaround is to get the response object using the @Res() decorator and calling the send() method with the buffer as the parameter.
I dislike this because it doesn't follow the execution flow of routing-controllers (e.g. none of the interceptors are called before the result is returned to the client).
Cheers!

I wonder how we should do this:

Detect if the action is json typed (@JsonController or @JsonResponse), so the returned object is data object and we should transform it using class-transform and return using res.json(). Otherwise don't transform and use res.send(result).
Detect type of the response - if it's Buffer don't transform, if Stream pipe to response, otherwise transform and send.
Introduce decorator like @RawResponse() to bypass the class-transformer transforming - might be helpfull with big and weird objects from ODM/ORM.

@pleerock @NoNameProvided any thoughts? 😉

As #226 has a broader conversation about the topic I will close this as a duplicate.
Please subscribe to #226 to be notified when the required changes land. Thanks.


Robustify VmaStat

Currently, VmaStat is not defined as typedef.
With GCC, this is not compiling.
Just a small fix!

Good catch, thanks a lot!


Submit the application to FDroid

Even if the interface could affray some people, I am a big user of F-Droid. and I was a bit disapointed to not found Open Food Facts init. So here is my request : is it possible to add Open Food Facts to F-Droid ?

Added the metadata to fdroid. The app builds, but I run into the same problems as  @dalb8 -- crashes because of the missing barcode scanner. Since I don't have any clue about phonegap/cordova, maybe someone could provide some insight on how to integrate the plugin into the buildscript?
Ref: https://gitlab.com/fdroid/fdroiddata/commit/8ecbb5eb6f28a6c6af0f3157f718e3c1aad613c8

We have the next generation version app in F-Droid. Not the same codebase, tough


[CI] Use native arm64 worker in GHAction to build M1 wheel

See https://github.com/dmlc/xgboost/pull/10219#issuecomment-2075847580

We might be able to retire the Mac Mini from BuildKite entirely, to minimize maintenance in the future.

@trivialfis Can you take a look at failing mypy?

It's probably a bug in mypy, I run into it as well after upgrading mypy.

https://github.com/python/mypy/issues/17166


A request for new options.

Thank you so much for the excellent plugin.
I'm using Gimp 2.10.14 (from gimp.org) on Windowe10 64bit.
I use your plugin interchangeably with BIMP - so comparing the action: I have a question and a request - is it possible to add the option:

load settings from file?
apply to layers (or open exported layers)?

An example (very good solution) comes from PaintDotNet ......
https://forums.getpaint.net/topic/6091-scriptlab/

Loading settings from file shouldn't be too difficult to implement. I'll add this feature to 3.4.
I am not sure if I understand the second feature request - is it applying filters (effects) on layers without exporting them, or batch-processing files (basically what BIMP does)?
The former is one of my ultimate goals. However, given how intertwined the export is with the code base, separating the export will definitely not be easy.
The latter should also be doable, but, as indicated, the plug-in code requires substantial refactoring before this is possible. Beside Real Life™ and other projects, this is going to take a long time, I'm afraid.

I would love the option of opening for Export-Layers layer after applying additional procedures.
My attempt to resolve this issue is here (but it would be better without my addition)
http://gimpchat.com/viewtopic.php?f=9&t=17874&p=245620#p245620

Version 4.0 now supports batch-editing layers without exporting them and exporting/importing settings. I suppose this can be closed now. ☺

Thank You very much for the solutions introduced and the constant development of an excellent plugin.
In Gimp-2.10.34 (Win10_64bit) when trying to write (Export Settings) as JSON I get this error:
<class 'export_layers.pygimplib.setting.settings.IntSetting'> is not JSON serializable.
Export/import as *.pkl works as expected.

I tried exporting settings to JSON with PDB procedures containing an integer argument (since the error message indicates the problem is with integer arguments), but I cannot reproduce the problem.
If possible:

Can you provide the entire error message (if there is more than that one line)?
Can you list the procedures and constraints used?
Can you provide the saved .pkl file? I might then be able to reproduce this on my end.

Did you by any chance download the source code before 4.0 was released? I had made some substantial changes to the format of plug-in settings between the releases which could possibly cause some unintended side effects (notably some obsolete settings not being removed or updated).

Version Export Layers: latest release (4.0, August 06, 2023).
I confirm that the problem with export to JSON occurs only for some procedures (when pkl handles it).
No other information about the problem (I will add that in such cases it creates a 0 b json file).

Sample file in attachment.
test_pkl.zip

Thank you, I just released 4.0.1 that should fix this. You should now be able to export setting as a JSON file.

Thank you, I just released 4.0.1 that should fix this. You should now be able to export setting as a JSON file.

Many thanks for solving the problem in no time (although a single export format would have sufficed).
BIMP will now be completely unused.
Topic to be closed.

I agree that one format should suffice, specifically the JSON format as it could possibly be used for automation in the slightest case that someone would e.g. want to automatically create several modifications of the saved file and export with a different configuration every time.
The pickle file format also acts as a fallback in case the json module is missing for some uncommon GIMP installations.


Open to adding the ability to restart workers

Hi y'all!
First off, thank you for a really great Github project. We use it here at RTR and its added a lot of rigor around the server-side rendering from our Ruby app.
Onto the issue. Would you be open to a PR which allows the hypernova master process to restart its workers after a specific number of requests. We currently have a job calling SIGTERM on the workers due to memory leaks resulting from our not using createGetComponent.
We have a lot of top level modules and so we opted to not go that route for now and are wondering what your opinion would be on allowing for a hybrid approach, similar to what unicorn-worker-killer does for Ruby.
Thanks again!

That does seem reasonable to me; although it masks the real problem, it would prevent production problems while it's investigated.

Cool! One of the things we noticed about running without createGetComponent is that its really helpful for local development because the restarts are really fast. I'm not sure how y'all load components in without restarting the server — perhaps I'm totally missing something.

@goatslacker, what are your thoughts?

@ljharb & @goatslacker, I'm going to start off with a PR or perhaps just a hook to be able to send a worker an exit message and will go from there.

Hey @Will-Sommers. Were you able to make any progress on this? Or care to share a bit about your current setup? Thanks!