http://localhost/laravel-filemanager? 404

i got 404 error when click browse server on url http://localhost/laravel-filemanager?type=Images&CKEditor=editorId&CKEditorFuncNum=1&langCode=en
And provide the followings :

Operating system : win10
Laravel version : 5.8
Package version : dev-master
Steps to reproduce your issue : i already following all the step, and try to reinstall and googling but don't fix the issue i got
Screenshots of browser console :

also i run my web without php artisan serve, but run it through "localhost/folder_name"

Show list route by type php artisan route:list anh check route of lfm

update .env with your real url
example:
APP_URL=http://localhost:8000
reload config and php artisan serve again
now check the file manager
http://localhost:8000/laravel-filemanager?type=Images&CKEditor=article_ckeditor&CKEditorFuncNum=0&langCode=en
hope this help

I have the same issue. An when ein run php artisan config:cache comes
``
67|             require $configPath;
68|         } catch (Throwable $e) {
69|             $this->files->delete($configPath);
70|

71|             throw new LogicException('Your configuration files are not serializable.', 0, $e);
72|         }
73|
74|         $this->info('Configuration cached successfully!');
75|     }
``

This only happens if the file manager Config is in it.


freeze pynndescent to last working version

freezes pynndescent to 0.5.12 which is the latest working version. 0.5.13 is available but unable to correctlly load dumped models (https://github.com/lmcinnes/pynndescent/issues/247)

Oh you're right.
I'll commit that as well

Sorry, one more thing: can you confirm that the tests pass in a clean environment? (python3 -m pytest tests/)
I will add a Github Action for this for next time, sorry for the inconvenience :D


Remove the git dependencies from package.json

I would like to ask you to change the two dependencies from package.json that depend on a Github repo. These are
Adler32cs
Filesaver.js
The reason I must ask you to remove the git link and change it to a particular version is that for some of us working inside restrained environments (some firms or industries). We cannot just allow the fetch of any dependency from github. Hence if its a Github link its blocked. If you change it to a version it would go through the normal checks and be approved.
I also do not see a downside under normal circumstances arising through this change.
Please let me know if this is possible on your end.
Thanks.

Fixed in #1432


Run cypress tests with GitHub actions

https://github.com/marketplace/actions/cypress-io#headless

Resolved in #53


[move] No owned to shared object upgrades

Throw abort on sui::transfer::shared if the object is not new

Fixes #5835

We might also need to update docs/examples?

Thanks for the simplification (tag: @lxfind )

Just double check, it can detect sharing unwrapped objects as well, right?

Just double check, it can detect sharing unwrapped objects as well, right?

Correct, I added a block of comments in there for that case too.

Just double check, it can detect sharing unwrapped objects as well, right?

Correct, I added a block of comments in there for that case too.

I updated the PR title and description to make this a bit more clear


Multiples module merge

Substantial bug fixes and improvements to multiples algorithms. Greatly improved accuracy and stability.

here it breaks a test in test/codes_tests/test_smalln.py and most of test/codes_tests/test_multiples.py
(probably these need to be updated to the new changes, I see multiple "CodeException: Not enough parameters in call, missing ['over']" errors)

Sorry, I missed this. I'll take a look at the tests. Are you OK with me changing them?

See my e-mail. I have fixed the smallN issue, but the multiples problem is deeper.

so encounters needs to be updated? I willopen an issue report for this...


Error: Error in function call

terraform plan -out out.plan
Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be
persisted to local or remote state storage.
Error: Error in function call
on k8s.tf line 36, in resource "azurerm_kubernetes_cluster" "k8s":
36:             key_data = "${file("${var.ssh_public_key}")}"
|----------------
| var.ssh_public_key is "~/.ssh/id_rsa.pub"
Call to function "file" failed: no file exists at /home/nipun/.ssh/id_rsa.pub.

@nipuns11 Please open a new GitHub issue in case you have any questions.


Added infallible error macro

Added macro for infallible errors in error.rs.
Fixes #1222.

This macro looks correct but it is not actually used anywhere.

Is this PR still live or can we close it @PaarthAgarwal ?

@tcharding s/close/take over/

Woops I hit "close" too quickly, now I have to take it over :) Will do it now.

Didn't read the message properly.


Destructuring in a for loop header has misleading error message when the value being looped is not iterable

Description
Compare these two snippets attempting to loop over an integer (error messages inlined):
// error: cannot loop over integer
#for value in 1 {}
//            ^

// error: cannot destructure values of integer
#for (x, y) in 1 {}
//   ^^^^^^

In both snippets, the problem is that integers can't be looped over, but in the second case the error message is misleading, as it suggests that the problem only occurs after a value was already successfully extracted. This is of course not likely to happen with integers, but arguments values have the same problem: "cannot destructure values of arguments".
For comparison, this is the error when iterating over an array whose elements aren't pairs of values:
// error: cannot destructure integer
#for (x, y) in (1,) {}
//   ^^^^^^

Reproduction URL
No response
Operating system
No response
Typst version

[X] I am using the latest version of Typst

Fixed by https://github.com/typst/typst/pull/3308


candle::RadialLight has an offset origin point

For some reason RadialLight has an offset origin. It mean that even setting origin to (0, 0) breaks everything which makes no sense to me
    // ...
    const float BASE_RADIUS = 400.0f;
    // ...
    RadialLight::RadialLight()
        : LightSource()
        {
        if(!l_texturesReady){
            // The first time we create a RadialLight, we must create the textures
            initializeTextures();
            l_texturesReady = true;
        }
        m_polygon.setPrimitiveType(sf::TriangleFan);
        m_polygon.resize(6);
        m_polygon[0].position =
        m_polygon[0].texCoords = {BASE_RADIUS+1, BASE_RADIUS+1};
        m_polygon[1].position =
        m_polygon[5].position =
        m_polygon[1].texCoords =
        m_polygon[5].texCoords = {0.f, 0.f};
        m_polygon[2].position =
        m_polygon[2].texCoords = {BASE_RADIUS*2 + 2, 0.f};
        m_polygon[3].position =
        m_polygon[3].texCoords = {BASE_RADIUS*2 + 2, BASE_RADIUS*2 + 2};
        m_polygon[4].position =
        m_polygon[4].texCoords = {0.f, BASE_RADIUS*2 + 2};
        Transformable::setOrigin(BASE_RADIUS, BASE_RADIUS);
        setRange(1.0f);
        setBeamAngle(360.f);
        // castLight();
        s_instanceCount++;
    }

Hello @chu65536 !
Are you having any specific trouble with this? What is the use case where you need to change the transform origin of a light source instead of the position?

I think it would be nice to make this moment more intuitive. But I understand perfectly well that it already works in most cases and may not be worth the effort. In short, the problem can probably be closed

Tbh, this project is now a bit old and I'm aware there are lots of pending improvements and updates. I've created a new issue (#38) to tackle a new release any moment in the future... I just hope for now is at least somehow useful.
Thanks for the feedback!


correct spelling mistake

Coverage increased (+2.6%) to 75.325% when pulling daf1a7c8c0f62f5dbc8d48158d6748d0527cc551 on EdwardBetts:spelling into 3e1849d3aaa1b9a325ad6d054acf695fddb4efe9 on visionmedia:master.

Thanks!


Fix: Readiness probe

Whilst there is a a readiness probe in the code-base, it was not being
added to the Docker image!
In addition, there was a bug in which the readiness probe always used
HTTP even when SymmetricDS was in HTTPS mode. This fixes that to
alter the protocol depending on the mode.
Also includes a bit of a code-cleanup.

👍


adds gallery to this branch

splits branch and adds alternative images

Merges styles into this branch and adds js for slideshow into script.js.
I'm still thinking about ways to write reusable html.
I could use Javascript to insert html elements into each webpage.
Or I can use , but MDN advised against doing so.
Or I can just combine all elements into one flie :( and control display

the tag didn't show up in the previous comment. I mean html link.

I left some of the alt tag capitalized because they are art forms and titles.

The selector.slides img .pic somehow shows all images together. I tried removing img tag and it works.

The selector.slides img .pic somehow shows all images together. I tried removing img tag and it works.

That's because there's no space between the two. It should be img.pic.


Bug: D3D11On12CreateDevice doesn't return a valid ID3D11Device value

Which crate is this about?
windows
Crate version
0.48
Summary
D3D11On12CreateDevice does not return a valid ID3D11Device; the code in my example is what I believe to be a copy of the D3D1211On12 C++ sample.
Toolchain version/configuration
Default host: x86_64-pc-windows-msvc
rustup home:  C:\Users\XXX.rustup
installed toolchains
stable-x86_64-pc-windows-msvc (default)
1.65.0-x86_64-pc-windows-msvc
active toolchain
stable-x86_64-pc-windows-msvc (default)
rustc 1.70.0 (90c541806 2023-05-31)
Reproducible example
Check out the 0.48 tag on windows-rs and modify the direct3d12 sample like this:
diff --git a/crates/samples/windows/direct3d12/Cargo.toml b/crates/samples/windows/direct3d12/Cargo.toml
index e14f98827..1a7299582 100644
--- a/crates/samples/windows/direct3d12/Cargo.toml
+++ b/crates/samples/windows/direct3d12/Cargo.toml
@@ -12,6 +12,8 @@ features = [
     "Win32_Foundation",
     "Win32_Graphics_Direct3D_Fxc",
     "Win32_Graphics_Direct3D12",
+    "Win32_Graphics_Direct3D11",
+    "Win32_Graphics_Direct3D11on12",
     "Win32_Graphics_Dxgi_Common",
     "Win32_Graphics_Gdi",
     "Win32_Security",
diff --git a/crates/samples/windows/direct3d12/src/main.rs b/crates/samples/windows/direct3d12/src/main.rs
index b437ce623..06c98e1be 100644
--- a/crates/samples/windows/direct3d12/src/main.rs
+++ b/crates/samples/windows/direct3d12/src/main.rs
@@ -2,7 +2,7 @@ use windows::{
     core::*, Win32::Foundation::*, Win32::Graphics::Direct3D::Fxc::*, Win32::Graphics::Direct3D::*,
     Win32::Graphics::Direct3D12::*, Win32::Graphics::Dxgi::Common::*, Win32::Graphics::Dxgi::*,
     Win32::System::LibraryLoader::*, Win32::System::Threading::*,
-    Win32::UI::WindowsAndMessaging::*,
+    Win32::UI::WindowsAndMessaging::*, Win32::Graphics::Direct3D11::*, Win32::Graphics::Direct3D11on12::*,
 };

 use std::mem::transmute;
@@ -297,6 +297,12 @@ mod d3d12_hello_triangle {

             let frame_index = unsafe { swap_chain.GetCurrentBackBufferIndex() };

+            let device11 = create_d11_device(&self.device)?;
+
+            if device11.is_none() {
+                println!("None");
+            }
+
             let rtv_heap: ID3D12DescriptorHeap = unsafe {
                 self.device
                     .CreateDescriptorHeap(&D3D12_DESCRIPTOR_HEAP_DESC {
@@ -787,6 +793,19 @@ mod d3d12_hello_triangle {

         resources.frame_index = unsafe { resources.swap_chain.GetCurrentBackBufferIndex() };
     }
+
+    fn create_d11_device(pdevice: &ID3D12Device) -> Result<Option<*mut Option<windows::Win32::Graphics::Direct3D11::ID3D11Device>>> {
+        unsafe {
+            let ppdevice:Option<*mut Option<ID3D11Device>> = None;
+            let ppimmediatecontext: Option<*mut Option<ID3D11DeviceContext>> = None;
+
+            D3D11On12CreateDevice(pdevice, D3D11_CREATE_DEVICE_BGRA_SUPPORT.0, None, None, 0,
+                ppdevice, ppimmediatecontext, None  )?;
+
+            Ok(ppdevice)
+        }
+    }
+
 }

 fn main() -> Result<()> {

Crate manifest
No response
Expected behavior
I would expect my create_d11_device to return a valid ID3D11Device.
Actual behavior
println!("None") is called
Additional comments
No response

You're effectively passing null pointers where the function expects storage space for your OBJECTS, or in Rust terms there's no possible way for the call you're doing to affect the contents of ppdevice when it's not mutable.
let mut ppdevice: Option<ID3D11Device>> = None;
let mut ppimmediatecontext: Option<ID3D11DeviceContext>> = None;

D3D11On12CreateDevice(pdevice, D3D11_CREATE_DEVICE_BGRA_SUPPORT.0, None, None, 0, Some(&mut ppdevice) , Some(&mut ppimmediatecontext), None)?;

Thank you, that works for me.


Feature: Integration of data from Monitoring Survey.

Description of the issue
Currently the data from the monitoirng survey is not included in a graphic. This can also be considered as tool chains.
Ideas of solution
Integration of this chart at the protocolls page:
.
We can take the code from Kaifeng and include it as a Dash ap.
Workflow checklist

[x] I am aware of the workflow in CONTRIBUTING.md
[x] I have created a branch for my feature

[ ] Fixed MVP
[ ] Adjust styling to Use Cases
[ ] Review Text


Demultiplication workflow

This workflow contains 3 demultiplexers: mkfastq, bcl2fastq and bcl-convert.
A fastqc report is generated automatically.

The ingestion/cellranger_demux can be removed and replaced by demultiplication, no?
@tverbeiren Is this PR ready for merge?

Yes, ingestion/cellranger_demux can be removed.

As far as I'm concerned this PR can be merged.

@tverbeiren
I notice workflows/demultiplication/config.vsh.yaml contains:
functionality:
  name: demux
  namespace: demultiplication

Shouldn't this be called demultiplex instead of demultiplication? Also, the name and namespace is not consistent with the path of the config file.

Proposal:
Contents of workflows/demultiplex/demultiplex/config.vsh.yaml:
functionality:
  name: demultiplex
  namespace: demultiplex
# ...

Integration test is not working on CI but it does work locally


[Ticket] [PMT] Fix Mobile View Bugs + Responsiveness

Description
GeorgiaCORE is an application where users can create customized chat-bot interfaces by building out trees showing possible paths of navigation in the chatbot.
This ticket may require some additional investigation into how to replicate the bugs.
We have noticed a few issues with how the chatbot component is rendered on mobile devices. However, we do not yet have a consistent way to replicate these bugs. In this ticket, we would like to investigate why the component/text is rendering strangely on specific dimensions and fix these bugs if possible.

Acceptance Criteria

[ ] Mobile view is displayed responsively across all dimensions

Other Notes

Please look at the GeorgiaCORE Setup section in the ReadMe to understand how to work on the project
Ask Manu or Jay for help on anything here if needed! (We both worked on this project last semester!)

I can't seem to replicate the display issues on the chatbot page. Since this bug is similar to the one I did for the previous sprint, I think the fix I implemented last sprint might've fixed this one as well.


Arithmetic Operations in Projections Produce Zero Results

If we modify a column in the output clause of a SELECT statement, the result has zero rows in it:
How To Reproduce

Create a tableterrier=# CREATE TABLE xxx (id INT);
CREATE TABLE

Load the tableterrier=# INSERT INTO xxx VALUES (1),(2),(3);
INSERT 0 3
terrier=# SELECT * FROM xxx;
 id
----
  1
  2
  3
(3 rows)

Query the table by adding 1 to the id column:# SELECT id + 1 FROM xxx;
 + id + INTEGER
----------------
(0 rows)

Was fixed by #705.


Feature 'Azure Spot Virtual Machines for virtual machine scale sets' not supported in Azure China 21Vianet due to dependency

Feature does not support in Azure China 21Vianet.
Azure Spot Virtual Machine is not supported in China Cloud. It will reports internal error when add spot agent pool in china cloud. Can we add one description on this limitation?
https://docs.microsoft.com/en-us/azure/virtual-machine-scale-sets/use-spot#limitations
https://github.com/MicrosoftDocs/azure-docs/issues/85223#issuecomment-993469409
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 5fc40e8e-558d-171c-2fd9-82b67f89d221
Version Independent ID: 9a757f12-cf86-2095-0f6a-37397024e3a5
Content: Add a spot node pool to an Azure Kubernetes Service (AKS) cluster - Azure Kubernetes Service
Content Source: articles/aks/spot-node-pool.md
Service: container-service
GitHub Login: @zr-msft
Microsoft Alias: zarhoads

@zr-msft
Can you please check and add your comments on this doc update request as applicable.

Thank you for the feed back.
This information has already been added to
https://docs.microsoft.com/azure/virtual-machine-scale-sets/use-spot#limitations
#please-close


truncate data refresh percent change

Fixes
Fixes #734 by @AetherUnbound
Description
Did some basic string truncation on the data refresh percent to show 3 digits after decimal point.
Testing Instructions
Added relevant test cases
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I'm going to go ahead and close this PR, @prasanna4567 feel free to reopen it if you find you have the time and capacity to do so 🙂


fix: Distance unit fixed

Should fix the kia_uvo problem: Not valid unit for device class distance #812

This should match the property approach of the rest of the file with a setter. On my phone so hard to link to an example but I can later.

Here is an example of unit being used.
Values being stored here: https://github.com/Hyundai-Kia-Connect/hyundai_kia_connect_api/blob/2ca8b233c0c012d702f511702bb40991914f42c1/hyundai_kia_connect_api/Vehicle.py#L143-L145
and setter getter: https://github.com/Hyundai-Kia-Connect/hyundai_kia_connect_api/blob/2ca8b233c0c012d702f511702bb40991914f42c1/hyundai_kia_connect_api/Vehicle.py#L258-L266
Plus from the EU API this must be set correctly.

I took the following code as template as the odometer is not throwing this kind of error:
@property
def odometer(self):
    return self._odometer

@property
def odometer_unit(self):
    return self._odometer_unit

@odometer.setter
def odometer(self, value):
    float_value = get_float(value[0])
    self._odometer_value = float_value
    self._odometer_unit = value[1]
    self._odometer = float_value

Therefore I thought the unit setter function is missing and I added them. Why do you think this is not correct?

Oops you are correct. The rest was in before this PR.


SFEVL53L1X::setROI sets wrong Y value

Looks like a typo in the C code:
https://github.com/sparkfun/SparkFun_VL53L1X_Arduino_Library/blob/1773c55cb9fe2427d5f6ceef2aaf080b380fc88f/src/SparkFun_VL53L1X.cpp#L273
It probably should be:
_device->VL53L1X_SetROI(x, y);

Good catch, fixed with a separate pull request


wip: extract tymodel for default only

Currently has two problems.

TyModel is not version dendent.

Currently, TyModel is hardcode into TyModel.scala and is being used globally.
Usages of global TyModel is as follows:

Parsing NameTy
Various functions related to NameTy
Containment check for MapObj

Hack in NameTy

This is also related to hack in NameTy, as there are circular dependencies between NameTy functions and TyModel, before you just hardcoded TyModel and as it was provided from the start it did not cause problem. Now, as TyModel itself should also be parsed out of json and NameTy functions are invoked when parsing, it causes problem.
It seems that to resolve above two problems, you should make above three usage sites not to use global TyModel but version-dependent TyModel as parameter, currently working on this solution.

I tried to solve the first problem with https://github.com/es-meta/esmeta/pull/180/commits/7ccad177a77bb5e2ef23f254068886b4dff07a36

Currently checking e2e test failing.

a8d0643 fixes the e2e test.

I merged this PR after refactoring the code. Thanks! Because of the dependency issue, I dropped the feature for version-dependent support of the type model. This PR only supports the JSON-based management of the type model.


Add support for "nbsphinx-toctree" cell tag

... in addition to plain old cell metadata.
Rendered: https://138-210404706-gh.circle-artifacts.com/0/html/subdir/toctree.html

Hello @mgeier! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file src/nbsphinx.py:

Line 220:80: E501 line too long (89 > 79 characters)


New line text problem

Since version 1.20.0, line breaks in command output are not displayed correctly, all text is displayed on a single line

Do you have a sample script to share on this issue? On which platform?

Same problem here. Simple Python Script
print(f"Line 1\nLine 2")
results in
Line 1 Line 2
expected
Line 1
Line 2

Are you using Windows or OSX? I 'am unable to reproduce your issue:

I can confirm this.
Using curl works fine.
Whith a browser as. client. It looks different.

hmmm .. so, not really a bug.
It is the way how the browser interprets the returned text.

A great thing would be to optionally add tags to ensure correct formating, if the request comes from a browser.
<!DOCTYPE html>
<html>
<body>
  <pre>
... webhookd-output ...
as
multiline
  </pre>
</body>
</html>

Maybe with help of this check:
func isBrowserRequest(userAgent string) bool {
    browsers := []string{"Chrome", "Firefox", "Safari", "Edge", "Opera"}
    for _, browser := range browsers {
        if strings.Contains(userAgent, browser) {
            return true
        }
    }
    return false
}

I have also noticed a bit of weirdness with this release when it comes to line breaks, but I haven't been able to figure out the exact cause.  I've personally seen it remove a line break in some cases and also add blank lines in other cases.  This isn't a huge deal for me as I was able to work around this issue by simply returning the response as a JSON value.
Even in the example screenshot above, there's an extra line break added between Line 1 and Line 2; with one \n, there shouldn't be a blank line between those two.

Also, I'm not sure I like the idea of wrapping responses in HTML in a browser. If possible, I would personally rather have the completely raw response so that the page can be saved and not result in an HTML file.

Agree. Default behavior should be completly raw respose. As an option it would be nice to have HTML enclosed output for quick and dirty solutions.

Regarding HTML concern, you can rely on SSE events to get the RAW output stream, and have each event (aka lines) surrounded with an HTML tag. There's an example in this repo's tooling directory: tooling/html/console.html

Indeed, the 1.20 has an unsuspected extra line break. It's fixed. Thanks for the report!


#31: Reviews endpoints

-
-

Issue
Related to https://github.com/ripe-tech/ripe-registry/issues/31

Dependencies
• https://github.com/ripe-tech/peri-templating-api-js/pull/1

Decisions
API for peri-templating service, more specifically the reviews.

Animated GIF
--

@gcandal @BeeMargarida check the renaming of the methods and notice that the name of API broker methods must always end with the name of the entity

@gcandal @BeeMargarida check the renaming of the methods and notice that the name of API broker methods must always end with the name of the entity

@joamag even though it's a POST, that just happens because we need to pass a JSON as param. If it was not for that limitation, semantically it would be a GET (has no side effects), that's why we named it as getX.


Add input data validation/length limitation

Currently any data length can be used in description and NFT fields

data validation/length limitations are in place and working correctly


Garbage on screen race condition cases

How to reproduce:
Have a 60fps video that starts with a mostly white image with large black patches, and play on real hardware.

Expected result:
Normal black is black

What is happening is that the screen starts completely black, and most of the screen is white. So, the encoder choses to use the lines codec and fill the screen whit white, using later frames to add back the black areas.
However, the lines codec on the se30 is slightly over-enthusiastic and the amount of available cpu cycles, and cannot be truly executed in the time set up.
So the code is still turning the original black to white, when the second frame wants to put black back on top of the white (which isn't there yet). After this frame, the preceeding one finishes to fill screen with white, replacing the new black by white, which will stay there, as the encoder is convinced that those parts of the screen are black.
In general, this is not a problem, because:
a) In a typical 24 fps flim, there are two or 3 cycles available, so the screen update batch has less risk to slip behind
b) The error is quickly corrected as flim updates
However, a) does not apply in a 60fps flim, and b) when a flim has a long staying black piece in an otherwise white settings (for instance black borders), b) never occurs.
Fix would be to refuse to update when the preceeding frame isn't completed. This will unfortunately break sound in those cases.


Remove iOS version name normalization to 3 parts

Use case
We specify app version and build numbers in pubspec.yaml. Some app releases we want to have a two part version number, like 2.7, for other app releases we want a three part version number, like 2.7.3.
In pubspec.yaml we write version: 2.7+25 in the first case and version: 2.7.3+25 in the second case.
For both Android and iOS we want the app version number to match the number we have in pubspec.yaml.
Observed Behaviour
When Flutter generates the files for Android, the version number from pubspec.yaml is used verbatim, when it generates for iOS it is "normalized" to three parts. That is, 2.7 becomes 2.7.0.
The offending code is here:
https://github.com/flutter/flutter/blob/415401102ca165c2c727288e542eabd049ad4b62/packages/flutter_tools/lib/src/build_info.dart#L332-L334
Proposal
Remove the iOS special case that normalizes to three parts.
Solution
It is not, and never have been, a requirement to have three part version numbers on iOS. We have released apps on the Apple App Store for the past 10 years and used two part version numbers since the beginning.
The above referenced piece of code in build_info.dart references som Apple documentation that describes version numbers as three part. The author that created the offending piece of code must have read the documentation much to literally and mistaken it for official Apple policy.
The piece of code was introduced in merge request #27743.

I also want to mention that I find it odd that there is validation and normalization of the iOS version name but no validation or normalization at all for the Android version name. The documented and actual requirements and limitations for iOS and Android are almost identical and are 100 % identical in practice so there is absolutely no reason to have a different approach to the two platforms.


Fix include logic where include path is subdirectory of excluded path.

Fixes https://github.com/wp-cli/i18n-command/issues/172.
Changes in this PR:

Fix containsMatchingChildren(), so $root_relative_path doesn't have trailing slash. [#]. This is because $include and $exclude are trailing-slash-trimmed, so if $root_relative_path isn't, this clause won't be true as expected.
When the file iterator gets to the included folder, which is a subfolder of an excluded folder, it will get here, so I created a new if block to handle that case.
Unit test for this case: including a folder inside an excluded folder.

@swissspidy I'm not rushing but just want to make sure if there is any action item on my side.


Issue 127 no token - Work In Progress

Can this be closed? It seems we are not going this direction anymore with the new redesign.

I'm still debating this in my head... I still do think having the concept of "Simple Forms" is a valid addition to the architecture and is one that's needed by the new UI...

I've since realized that this type of simplification should be done within the UI layer and not within the form layer. So the concept of Simple Forms is unnecessary. Thank you @mrrobot16 @cbrzn for your work here and for exploring both options.
Closing for now.


Debug issues with PudlTabl output objects and direct to SQLite outputs

The database produced by the direct-to-sqlite ETL is (at leaset) slightly different from the old datapackage based database, and this is causing issues with the pudl_out / PudlTabl class that we rely on to pull denormalized data tables from the database. We can't run the tests without the outputs working, so this means we don't know how broken everything is with the new DB 😅 See #1195
Known problems

[x] Many of the output objects complain because there's no id field in the new database tables (this includes at least pudl_out.plants_eia860, pudl_out.gens_eia860, pudl_out.frc_eia923, and pudl_out.bf_eia923)

Thankfully, once the tests were up and running, it seems like all the current output routines, mcoe etc. work fine -- just those ID column references needed to be removed.


resolve proxy objects before sending them to actions

This should ensure that actions are always given a resolved object instead of a proxy (so every single action doesn’t have to test and resolve them). See #621 for more discussion.
Changing this method should catch all of the scenarios (interface controller, triggers, etc.).
One example of something that wouldn’t work before this change: Random Track → Add to Playlist… → Some Playlist

... so every single action doesn’t have to test and resolve them

Could you elaborate? Don't quite understand this.
This method definitely seems more robust than the last one :)

... so every single action doesn’t have to test and resolve them

Could you elaborate? Don't quite understand this.

This is sort of covered in #621, but I can (hopefully) clarify. Without this change, proxy objects are sent to actions as proxy objects. So getting certain things out of the object won’t work. For instance, objectForMeta: won’t give you what you expect because the metadata is in the object the proxy points to, not the proxy itself. The same is true for more fundamental things like identifier. An action can get around that by testing for proxies and resolving them if needed, but a plug-in developer would have to 1) know that and 2) add the extra code to every action. (Or they could just add the check to actions where it could be a problem, but they’d have to know a lot about the way the core app works to figure that out.)
If you and I didn’t even know this was a potential problem, what chance do others have? :-)
So with this change, there’s less to know and less to code for plug-in devs, and going along with that, I feel like it makes everything more reliable. Actions can just be written with the assumption that they’ll always be passed a “real” object.

P.S. can I just say...

If you and I didn’t even know this was a potential problem, what chance do others have? :-)

YES! There was one thing that's been annoying me about the 'Paste as Plain Text' action, in that it never worked with the 'Clipboard Contents' proxy (because it would try and paste some unresolved object). I spent quite a few days trying to figure it out. Guess what, I didn't even know this was a potential problem, but now it works :D

Cool! I expect there’ll be a lot of things that should have always worked, but didn’t for unexplained reasons.


CTHUB - Model Year Report Showing Multiple Entries

Describe the task
Model year reports in CTHUB are currently showing multiple entries for the same year and have different values except for the LDV sales.
Purpose
The purpose of this task is to address the presence of multiple entries for the same year in the reports, ensuring that only one entry is displayed for each year.
Acceptance Criteria

[ ] One MY entry per supplier in CTHUB with correct information

Additional context

The discrepancies in the model year reports is most likely due to the compliance obligation table pulling too many rows with outdated data.

Hey team! Please add your planning poker estimate with Zenhub @emi-hi @JulianForeman @tim738745


Feature Request: Updating typings.

Upgrade typings to 1.3.0
New tsconfig:
{
"compileOnSave": false,
"compilerOptions": {
"declaration": false,
"emitDecoratorMetadata": true,
"experimentalDecorators": true,
"module": "commonjs",
"moduleResolution": "node",
"noEmitOnError": true,
"noImplicitAny": false,
"outDir": "../dist/",
"rootDir": ".",
"sourceMap": true,
"target": "es5",
"inlineSources": true
},
"files": [
"../typings/index.d.ts"
]
}
New typings.d.ts:
declare var module: { id: string };
Rerun npm i.
ambient is gone. Everything is moved to globals. index.d.ts has references to all d.ts files.
ng build works.
Thanks to my Ops group for needling this out.

I achieved the same thing without modifying tsconfig.json.

Edit typings.json and replace all references to ambient to global (eg. ambientDependencies becomes globalDependencies).

Update the ref path in typings.d.ts to use index.d.ts of browser.d.ts.
  /// <reference path="../typings/index.d.ts" />

Delete typings/ directory

Run typings install

Grab your drink of choice, and begin resolving issues with typings. For me, I had to move 3 dependencies into globalDependencies and I have no idea why. ¯\_(ツ)_/¯

Dupe of https://github.com/angular/angular-cli/issues/816


Add a command to build a binary of a Quarkus app

Adds a command that generates, then executes a tasks.json task that builds a binary

After you have used the command once, you can build a binary either through the command or <Ctrl>+<Shift>+B > Quarkus: Generate a binary

The command checks to make sure GRAALVM_HOME is set before building the binary, and provides a link to the Quarkus docs if it's not set up.

This PR addresses one aspect of #335
Signed-off-by: David Thompson<EMAIL_ADDRESS>

Demo: https://youtu.be/5WgSg_FzzN8

I rebased this PR and added a section about collecting telemetry when the command is run.


Rewrite linux root cert install to require an RPM that installs certs

Eliminates all the functions/lines supporting the current method that crawls a web site.
# Install root certs, if the root cert url is provided
# Requires a url to an rpm, or the name of a package in currently-enabled yum repos
if [[ -n "${ROOT_CERT_RPM}" ]]; then
    yum -y install "${ROOT_CERT_RPM}"
fi

OBE in watchmaker and latest AMIs


No warnings with Sass <IP_ADDRESS>e0f33b (Selective Steve)

Fixed in https://github.com/sass/sass-spec/pull/529


DB fails to start after upgrade is interrupted

We are using Image pgautoupgrade/pgautoupgrade:14-dev
We are seeing an issue on larger DBs that the init container running this image takes longer than the default thresholds and gets terminated.
Upon the restart the DB fails to start with:
postgresql 11:42:50.%2N INFO  ==> ** Starting PostgreSQL **
2024-08-15 11:42:50.466 GMT [1] LOG:  skipping missing configuration file "/bitnami/postgresql/data/postgresql.auto.conf"
2024-08-15 11:42:50.466 GMT [1] FATAL:  "/bitnami/postgresql/data" is not a valid data directory
2024-08-15 11:42:50.466 GMT [1] DETAIL:  File "/bitnami/postgresql/data/PG_VERSION" is missing.

It seems that after the upgrade is interrupted the upgrade is not retried leaving the DB in a broken state.
Aside from incrementing the liveness probe timeout/failure threshold for the init container running this image is there something that could be set so that the upgrade is retried?

@miguelhar How big is your database, and how long is the default timeout on the platform you're using?
Thinking things through a bit, my initial thought that retrying the upgrade without increasing the timeout value is pretty likely to not result in a success.  The upgrade scripting itself is mostly a case of:

Move things into a directory structure that pg_upgrade (the official PostgreSQL cli tool for upgrading) can work with, then
Run the pg_upgrade cli utility

While the are some sanity checks done first (ie here), the actual upgrade piece is done using pg_upgrade.
If pg_upgrade has been terminated part way through... I kind of doubt it'll be able to happily continue on and complete what wasn't done in a previous run.

Long story short, I reckon you'll need to restore your database files back to how they were before the upgrade, then try the upgrade again with a longer timeout threshold.
Sorry I don't have better news nor suggestions @miguelhar. :frowning:

@justinclift thank you for your response, the size varies but increasing the livenessProbe/failureThreshold to 20 seems to do the trick

Awesome.  I bet that took a bunch of time and effort to work through and get happening.
Those two terms (livenessProbe, failureThreshold) show up as Kubernetes related things.  Are you managing Kubernetes yourself, or are you using the offering from one of the big vendors?
Asking because it might be useful for future people to directly mention the relevant vendor here, so others that hit the same issue can see whether it relates to them. :smile:


how send to specific number ?

iam looking how send to specific number, but i just only found this. pelase help me

You can check the documentation for sending to a number
https://www.postman.com/supply-meteorologist-96562615/baileys/documentation/9yuuao2/baileys-api-v2-0-0?entity=request-14456337-84543cb5-9782-4d4d-8a10-f4488bbdf35d
for the contents of the message object and options, you can check the official baileys documentation.
Sorry for not being able to provide swagger documentation that is easily accessible, maybe if there is time I can make it.


Update and implement Calbacks

Implement callbacks improvements.
https://blog.langchain.dev/callbacks/

No changes are needed.


Don't force the developer to put the "super call" as last function in initializer list (strong_mode)

Please!!! this looks so ugly!!! super must be first in Java, BaseClass-call must be first in C++ - in Dart it should be the last function - not what we want...

If it's only the way it looks that is a problem, I think it is something we can learn to live with.
The reason for wanting to move it last is that it will be executed after the initializer list. Putting it where it is actually executed makes reading and understanding the code easier.
It doesn't have to be that way, we could easily change it to execute before the initializer list, but the current style is already to write the call at the end of the initializer list, and the VM has always executed it there, so there is no compelling reason for changing it at this point.

The main reason IS readability!!! I really don't care what the current style is if the current style is bad. As said above - Dart is the only language I know where the super-call is the last function in the initializer-list.
'Dart should follow the conventions and not doing it's own business if it's not necessary.

Java has the super call first, but they also run into issues because of this:
// In T.java.
class T {
  T() {
    virtual();
  }
  public static void main(String[] args) {
    new TT();
  }

  void virtual() {}
}

// In TT.java.
class TT extends T {
  final int fin;

  TT() {
    super();
    this.fin = 499;
  }

  void virtual() {
    System.out.println(fin);
  }
}

This program prints "0". This means that the final field fin has two values: 0 and 499. Dart fixes this, by ensuring that all (final) fields are initialized before they can be accessed. This implies that all fields  must be set before the super-constructor body can be invoked.
The current semantics allow the super call to be at any position, but that doesn't actually mean that the super constructor is invoked at the time of the super call. In fact, it only initializes the super fields at that time. The actual super constructor-body is only invoked at the end of the initializer list. This means that Dart (so far) had a two-phase initialization: first the fields, then the bodies.
This sounds relatively simple, but in practice it is very hard to implement, and very confusing when users actually want to depend on the order in which constructor bodies are invoked. It is completely unexpected that a super call in an initializer list doesn't actually invoke the super constructor at that time.
The simple solution is to have the super call at the end of the initializer list, which is when the constructor call actually happens. This makes the location of the super constructor consistent with the time of execution. (As a benefit it simplifies our implementations, but that wouldn't be a reason for the change).
I understand that this might require projects to change their conventions, forcing them into a style that isn't familiar to them. In our experience (and we have used this convention for a very long time) this style quickly becomes natural.
While we understand that our convention is now different from other languages, we do think that the benefit (having a clear simple semantics of how objects are created) outweighs the cost of adapting code and getting used to the new style.

:+1: :ok_hand: + Thanks!


Do you have an annotation tool for custom data?

I need tool to annotate custom data
please, provide it
thanks

about how to annotate custom data, please refer to #11

I see that, but I'm still wondering if i need to create json file manually.
I appreciate your answer, Thanks

yeah, you should convert your annotations to a json file(COCO format), including word-level and character-level annotations

Thank you very much, I appreciate that
Then learning data according to detectron2 manual, right?

train a pre-trained model on Synthtext with word-level and character-level annotations, and then use the model to generate weakly supervised annotations on your custom datasets. all of it is based on detectron2 framework.


Only include query params that are not null

This would greatly simplify setting queries.
E.g.
history.pushState(null, '/path', { thisQueryParam: 'yes', notThisQueryParam: null })
->
/path?thisQueryParam=yes
In other words, include params that are null. In cases an empty param is needed an empty string would be sufficient. I think this is a logical behavior that most developers will understand.

Are you suggesting the default should be changed? Is there an option we can pass to qs to make this happen?

Just checked the qs documentation and you can use the skipNulls flag if you need this behavior. Closing.


incompatible cache fix

Another small change. This basically is a fix for the edge-case where there might be multiple instances of styleq being used with different configuration options.

I'll add a test, but here's an explanation in the meantime:
Imagine two different instances of styleq have been created. One with inline style merging disabled and one with it enabled.
Now if the first instance is used, and then the second instance is used with the same arguments, you will end up using the cached value from the first instance when calculating the result for the second instance.
With different configurations to the factory, there will be different results to the same arguments, but before this change they all share the same WeakMap as a cache which will lead to incorrect results.

Inline style merging doesn't affect how the cache is populated

Ok. I see that caching is disabled when merging inline styles. There is still the question of incompatible transform functions. If there are two different instances of styleq with incompatible transform function, that will lead to incorrect caching as well.

No it won't, because the cache always consumes the return value of the transform. If different objects are returned, different paths through the cache are created. See the built-in transform for example https://github.com/necolas/styleq/blob/main/src/transform-localize-style.js

Oh I missed that. That also explains why you chose to transform an entire object and not individual entries. Thanks, that clears up a few other questions I was trying to figure out.


"Table does not exist" error when query is executed after "Key already exists" error

I'm using this great library together with JsStore, in an iterative process of reading log records from many JSON files, processing and then storing into IndexedDB via SqlWeb + JsStore, and it works great.
However, like the title says, if I get an error of "key already exists", when the script moves to the next query (often on a different table), I get the error of "table does not exist", when it definitely exists.
I use connection.runSql(query).then(...).catch(...) but it still raises the table error for all following queries, until the whole process eventually stops.

Seems interesting. I will try to reproduce this & if you have something that you can give me, that would be great.

A brief explanation of what my script does: read logfile with records in JSON format, put the records into a list, process the list to find a type of events, say sessions, and insert to a sessions table via SqlWeb like this:
let query = new SqlWeb.Query("INSERT INTO " + table + " values='@val'");
query.map("@val", data);
connection.runSql(query)
   .then(function (rowsAdded) { if (rowsAdded > 0) { console.log('Success'); } })
   .catch(function (err) { console.log(err);  });

And then do the same for other types of events in the same list of records, then repeat for many JSON, everything on client side.
Sometimes if I get the "key already exists" error, it is caught and it continues working, but other times I get the "table not exists error" for all the following tables until it completely crashes, or this one, which instantly stops the whole process:

Let me know if there's anything I can tell you!

There was a small query flow issue with jsstore. I have fixed it and released it in new version - 2.11.0. Please install the latest version of jsstore and let me know if fixed or still getting the issue.

Thank you! I've been trying to reproduce the error and it seems to be fixed.
Will let you know if it comes up again.


missing in the explanation of ExpressRoute

Hello
I feel that the explanation of ExpressRoute is different.
In No46, officeapps.live.com is supposed to go through a fast route.
In No81, only odc.officeapps.live.com does not go through ExpressRoute.
However, cdn.odc.officeapps.live.com does not follow the fast route.
Therefore, I think cdn.odc.officeapps.live.com should be listed in No81.
how is it?
__
[Enter feedback here]

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

ID: 66bd8666-3566-a901-b25b-eb6baba6264a
Version Independent ID: e9f6310f-2207-8c81-d31a-3d5a80beb26b
Content: Office 365 URLs and IP address ranges - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/urls-and-ip-address-ranges.md
Service: o365-administration
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@kelleyvice-msft Please add your comment and help us in resolving this issue. Thanks.

This issue has been resolved in a previous update.


Question - does this work with Rancher?

I'm looking to use this with Rancher. Is this feasible?
My question is: Question - does this work with Rancher?

My understanding of Rancher is that it provides a GUI front end for managing Docker hosts, images, containers, volumes, etc. It is just a nice way for a person to interact with one or more Docker hosts in a browser.
This library provides a Java wrapper around the Docker remote API. As such, it is just a nice way for your Java application to interact with a Docker host.
Both things just interact with docker, and that's it. I don't see why you would have a problem using Rancher + this docker-client library any more than you would have a problem using Rancher + the docker command-line client.

Thanks for answering this @johnflavin. Sounds cleared up to me.


proposal: Add type position to guru describe output

It would be nice to have command like 'Go to type definition' in code editors. To help implement this we need to know where type of current (under cursor) identifier is defined.
So proposal is to add "typepos" to json output of describe identifier.
{
	"desc": "identifier",
	"pos": "testdata/src/describe-json/main.go:16:8",
	"detail": "value",
	"value": {
		"type": "I",
		"typepos": "testdata/src/describe-json/main.go:21:6",
		"objpos": "testdata/src/describe-json/main.go:12:6"
	}
}

P.S. I am ready to create PR just wanted to discuss this before

CC @alandonovan

Also cc @ianthehat

Ping @alandonovan and @ianthehat.

Ping @alandonovan and @ianthehat.

Yes, it seems like a reasonable feature. If you send me the CL I will review it. Thank you.
BTW, longer term, the Go team at Google will maintain a canonical LSP (language server protocol) implementation, which will subsume most of the features of Guru. In hindsight some of its features such as pointer analysis-based callgraph alias queries do not carry their weight (performance, implementation complexity) and should be dropped or replaced by simpler approximations. I expect that the feature described in this issue is part of the core LSP feature set.

I am not understand, I go google or Microsoft ??


Use targetRowsPerSegment for single-dim partitions

Description
When using single-dimension partitioning, use targetRowsPerSegment (if specified) to size segments. Previously, single-dimension partitioning would always size segments as close to the max size as possible.
This change restores the intended behavior that existed prior to the refactoring done in #8141 :
https://github.com/apache/incubator-druid/pull/8141/files#diff-c285c81cfeec663a227ddf5e241d9effL57
https://github.com/apache/incubator-druid/pull/8141/files#diff-fd61251f2512217838c09a27d1d49a0bL335
Also, change single-dimension partitioning to allow partitions that have a size equal to the target or max size. Previously, it would create partitions up to 1 less than those limits. This change, makes the behavior consistent with the behavior of the row-count-based limits in DynamicPartitionsSpec.
Also, fix some IntelliJ inspection warnings in HadoopDruidIndexerConfig.

This PR has:

[x] been self-reviewed.
[x] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[x] added unit tests or modified existing tests to cover new code paths.

I'll also add that if the default is change in the future, this makes
refactoring easier. I think we should use constants instead of literal
values generally because they make refactoring easier and make it easier to
understand why a particular value is used.
On Wed, Oct 9, 2019 at 1:54 PM Chi Cao Minh<EMAIL_ADDRESS>wrote:

@ccaominh commented on this pull request.
In
indexing-hadoop/src/main/java/org/apache/druid/indexer/DeterminePartitionsJob.java
https://github.com/apache/incubator-druid/pull/8624#discussion_r333227957
:

@@ -661,7 +661,7 @@ protected void innerReduce(Context context, SortableBytes keyBytes, Iterable<Dim
}

     // See if we need to cut a new partition ending immediately before this dimension value

   if (currentDimPartition.rows > 0 && currentDimPartition.rows + dvc.numRows >= config.getTargetPartitionSize()) {

   if (currentDimPartition.rows > 0 && currentDimPartition.rows + dvc.numRows > config.getTargetPartitionSize()) {

When I was adding tests, I was expecting partitions sizes to match the
expected target but they were sized to one less than the target because of
this line. In practice, since the partition sizes are much larger than 1,
the difference is negligible, so I'm ok with reverting this change too.
Related to your suggestion, I'll update the PR description to explain why
the single-dim partitioning is being changed to use targetRowsPerSegment.
In
indexing-hadoop/src/test/java/org/apache/druid/indexer/DeterminePartitionsJobTest.java
https://github.com/apache/incubator-druid/pull/8624#discussion_r333227998
:

@@ -51,6 +52,11 @@
@RunWith(Parameterized.class)
public class DeterminePartitionsJobTest
{

@Nullable
private static final Long NO_TARGET_ROWS_PER_SEGMENT = null;

The named constant can be useful for tools that do not display the
parameter names (e.g., diff tools like the github PR diff). Perhaps the
name can be improved (e.g., DEFAULT_TARGET_ROWS_PER_SEGMENT)?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-druid/pull/8624?email_source=notifications&email_token=AAPSYCWOZZQVHBXS3IL47LLQNZAHPA5CNFSM4I5HFRZ2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCHOOWQY#discussion_r333227957,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPSYCWTXLEWBX26MVHWORLQNZAHPANCNFSM4I5HFRZQ
.

--
Jad Naous
Imply | VP R&D
650-521-3425
<EMAIL_ADDRESS>


Add mfunc to ignored comment strings automaticallyc

If you are using the premium version of the plugin users will have the mfunc minified when using the HTML compressor this needs to be added by default in order to prevent user's fragment caching from not working.
Needs to be added to the Ignored comment stems
Looking for a solution now.

Proposed fix via pull request

Closed by #10


ERROR: [facebook]<PHONE_NUMBER>15642: Cannot parse data

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I'm running yt-dlp version 2023.07.06 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
canada
Provide a description that is worded well enough to be understood
URL:  https://www.facebook.com/820615443/videos/848889063315642
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
>yt-dlp.py -vU "https://www.facebook.com/820615443/videos/848889063315642"
[debug] Command-line config: ['-vU', 'https://www.facebook.com/820615443/videos/848889063315642']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (zip)
[debug] Python 3.10.5 (CPython AMD64 64bit) - Windows-10-10.0.14393-SP0 (OpenSSL 1.1.1n  15 Mar 2022)
[debug] exe versions: ffmpeg 4.3.2-2021-02-02-full_build-www.gyan.dev, ffprobe 4.3.2-2021-02-02-full_build-www.gyan.dev, rtmpdump 2.4
[debug] Optional libraries: sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>Current Build Hash: cedf44a9cf3c241520a692c1006d276e6f434d4d945727c47cb1a70d32953f4a
yt-dlp is up to date<EMAIL_ADDRESS>[facebook] Extracting URL: https://www.facebook.com/820615443/videos/848889063315642
[facebook]<PHONE_NUMBER>15642: Downloading webpage
[facebook]<PHONE_NUMBER>15642: Downloading webpage
ERROR: [facebook]<PHONE_NUMBER>15642: Cannot parse data; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "C:\Transmogrifier\yt-dlp.py\yt_dlp\extractor\common.py", line 710, in extract
    ie_result = self._real_extract(url)
  File "C:\Transmogrifier\yt-dlp.py\yt_dlp\extractor\facebook.py", line 707, in _real_extract
    return self._extract_from_url(real_url, video_id)
  File "C:\Transmogrifier\yt-dlp.py\yt_dlp\extractor\facebook.py", line 654, in _extract_from_url
    raise ExtractorError('Cannot parse data')

Duplicate of #4311
Extractor seems to be completely broken now, all links affected


Add primitive cache for MKL-DNN sum(elemwise_add operator

Description
This PR cached the created sum primitive to reduce the creation overhead, which is helpful to improve the performance of current MKL-DNN sum (elemwise_add) operator.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@pengzhao-intel @TaoLv

Thanks for the improvements. Could you share the performance data?

@mxnet-label-bot add [pr-awaiting-review, MKLDNN]

Merging now. Thanks for your contribution.


Enable pagination for Droplets list

Fixes #205

So, the tests are failing because I change the URL to always explicitly pass the page number. However, even once I fixed that, all of the fixtures are for a paginated account! I could fix that by modifying the fixtures, then I should probably write at least one test for the paginated case. That's a lot of work, though, and this works for my use case. I'll try to get back to cleaning it up, but no promises.

Hi @lachesis! Perfect, thanks for this, was going to work on fixing this at the weekend, saved me the job :smile:
I've rebased to fix a bug and to add the new specs, and change the existing fixtures with the new stubs in light of these changes, see #212 for details.


images for 3.5 release

Any chance of getting Docker images of 3.5-rc2 or 3? Other repos eg golang do early previews of docker images, and it would be really helpful. Currently I am upgrading from 3.4 but its kind of inconvenient.
cc @ncopa

the 3.5 was just released. we also provide an alpine-minirootfs.tar.gz that could be used for creation of the image.

Having a 3.5 image ASAP would be great. Who is able and going to take care of this?

It seems https://github.com/gliderlabs/docker-alpine/pull/240 handles getting 3.5 support into this repo, but the image tag's are either not built, or not appearing on the Docker Hub as yet...

@CpuID This is why docker hub still not upgrade: image is updated via pull requests to the docker-library/official-images GitHub repo.

this one needs to be merged: https://github.com/docker-library/official-images/pull/2489

Aha, makes sense. Thx :)
Nathan Sullivan
Sent from a mobile device

On 27 Dec. 2016, at 8:01 am, Natanael Copa<EMAIL_ADDRESS>wrote:
this one needs to be merged: docker-library/official-images#2489
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

(you can close this now!)

I still cannot see gliderlabs/alpine:3.5 in tags on docker hub: https://hub.docker.com/r/gliderlabs/alpine/tags/ - any news on that?

@miketheman just do alpine:3.5.

@jbergstroem Thanks for the tip - I indeed tried that already. The gliderlabs version (which my stack is based upon) contains some differences (noted in http://gliderlabs.viewdocs.io/docker-alpine/build/ "Differences") that we currently use.

@miketheman oh; I didn't know that. Apologies.

I don't think there is any difference any more.
On 4 Jan 2017 12:35 p.m., "Mike Fiedler"<EMAIL_ADDRESS>wrote:

@jbergstroem https://github.com/jbergstroem Thanks for the tip - I
indeed tried that already. The gliderlabs version (which my stack is based
upon) contains some differences (noted in http://gliderlabs.viewdocs.io/
docker-alpine/build/ "Differences") that we currently use.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gliderlabs/docker-alpine/issues/235#issuecomment-270361090,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdcPGC36DDsWXymEPlVlYTL72jHmzRqks5rO5H_gaJpZM4LCYQm
.

@justincormack Here's what I get:
$ docker run -it --rm gliderlabs/alpine:3.4 ash
/ # apk-install
fetch http://alpine.gliderlabs.com/alpine/v3.4/main/x86_64/APKINDEX.tar.gz
fetch http://alpine.gliderlabs.com/alpine/v3.4/community/x86_64/APKINDEX.tar.gz
OK: 5 MiB in 11 packages
$ docker run -it --rm alpine:3.5 ash
/ # apk-install
ash: apk-install: not found

apk-install is a wrapper script that only does apk add --update "$@" && rm -rf /var/cache/apk/*.
Since then apk got its own flag to do the same so you can now do: apk add --no-cache ...

@ncopa Thanks for the update! That's great that apk now has a flag for it. Is the gliderlabs/alpine image now deprecated, i.e. will not receive updates? What about the Fastly-enabled packages distribution?

The fastly mirror is the default for the official images now.
On 4 Jan 2017 1:11 p.m., "Mike Fiedler"<EMAIL_ADDRESS>wrote:

@ncopa https://github.com/ncopa Thanks for the update! That's great
that apk now has a flag for it. Is the gliderlabs/alpine image now
deprecated, i.e. will not receive updates? What about the Fastly-enabled
packages distribution?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gliderlabs/docker-alpine/issues/235#issuecomment-270366988,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdcPG9RWuD3EyWNVd_Svaq8ByZHYfCYks5rO5qXgaJpZM4LCYQm
.

Closing as 3.5 has been released for some time now.


ValueError and AttributeError

Hi,
I‘m trying to making "run.py" work for Pre-training, but I got ValueError and AttributeError, and I didn't find a solution, can you help me to check it? Thank you very much!
Traceback (most recent call last):
File "/home/T3090U3/anaconda3/envs/hxf/lib/python3.8/site-packages/sacred/experiment.py", line 312, in run_commandline
return self.run(
File "/home/T3090U3/anaconda3/envs/hxf/lib/python3.8/site-packages/sacred/experiment.py", line 276, in run
run()
File "/home/T3090U3/anaconda3/envs/hxf/lib/python3.8/site-packages/sacred/run.py", line 238, in call
self.result = self.main_function(*args)
File "/home/T3090U3/anaconda3/envs/hxf/lib/python3.8/site-packages/sacred/config/captured_function.py", line 42, in captured_function
result = wrapped(*args, **kwargs)
File "run.py", line 20, in main
dm = MTDataModule(_config, dist=True)
File "/home/T3090U3/PycharmProjects/hxf/METER/METER-main/meter/datamodules/multitask_datamodule.py", line 19, in init
self.dm_dicts = {key: _datamoduleskey for key in datamodule_keys}
File "/home/T3090U3/PycharmProjects/hxf/METER/METER-main/meter/datamodules/multitask_datamodule.py", line 19, in
self.dm_dicts = {key: _datamoduleskey for key in datamodule_keys}
File "/home/T3090U3/PycharmProjects/hxf/METER/METER-main/meter/datamodules/coco_caption_karpathy_datamodule.py", line 7, in init
super().init(*args, **kwargs)
File "/home/T3090U3/PycharmProjects/hxf/METER/METER-main/meter/datamodules/datamodule_base.py", line 60, in init
self.tokenizer = get_pretrained_tokenizer(tokenizer)
File "/home/T3090U3/PycharmProjects/hxf/METER/METER-main/meter/datamodules/datamodule_base.py", line 25, in get_pretrained_tokenizer
return RobertaTokenizer.from_pretrained(from_pretrained)
File "/home/T3090U3/anaconda3/envs/hxf/lib/python3.8/site-packages/transformers/tokenization_utils_base.py", line 1672, in from_pretrained
resolved_vocab_files[file_id] = cached_path(
File "/home/T3090U3/anaconda3/envs/hxf/lib/python3.8/site-packages/transformers/file_utils.py", line 1271, in cached_path
output_path = get_from_cache(
File "/home/T3090U3/anaconda3/envs/hxf/lib/python3.8/site-packages/transformers/file_utils.py", line 1494, in get_from_cache
raise ValueError(
ValueError: Connection error, and we cannot find the requested files in the cached path. Please try again or make sure your Internet connection is on.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "run.py", line 16, in
def main(_config):
File "/home/T3090U3/anaconda3/envs/hxf/lib/python3.8/site-packages/sacred/experiment.py", line 190, in automain
self.run_commandline()
File "/home/T3090U3/anaconda3/envs/hxf/lib/python3.8/site-packages/sacred/experiment.py", line 347, in run_commandline
print_filtered_stacktrace()
File "/home/T3090U3/anaconda3/envs/hxf/lib/python3.8/site-packages/sacred/utils.py", line 493, in print_filtered_stacktrace
print(format_filtered_stacktrace(filter_traceback), file=sys.stderr)
File "/home/T3090U3/anaconda3/envs/hxf/lib/python3.8/site-packages/sacred/utils.py", line 528, in format_filtered_stacktrace
return "".join(filtered_traceback_format(tb_exception))
File "/home/T3090U3/anaconda3/envs/hxf/lib/python3.8/site-packages/sacred/utils.py", line 568, in filtered_traceback_format
current_tb = tb_exception.exc_traceback
AttributeError: 'TracebackException' object has no attribute 'exc_traceback'

Hi, I'm not sure if this is because of the Internet connection issue as suggested. I think one way to get around it is to first download the pretrained tokenizers and weights from huggingface and then store it in the huggingface cached directory.

Sorry to bother you again. My text encoder selects “text_ Roberta”, so I downloaded "Roberta base" in the hugging face, and then made the changes in the following figure in the "meter_module.py" file, but the previous error will still be reported. Is it the wrong change I made? Or do I need to change something else? The command I run is "Python run.py with data_root=/home/t3090u3/pycharmprojects/hxf/meter/meter-main/meter/datasets/data num\u gpus=8 num\u nodes=1 task\u mlm\itm\clip\bert per\gpu batchsize=32 clip16 text\u Roberta image\u size=224”
Error:
------------------ 原始邮件 ------------------
发件人:                                                                                "zdou0830/METER"                                                                                @.>;
发送时间: 2022年6月16日(星期四) 凌晨0:49
@.>;
<EMAIL_ADDRESS>主题: Re: [zdou0830/METER] ValueError and AttributeError (Issue #22)
Hi, I'm not sure if this is because of the Internet connection issue as suggested. I think one way to get around it is to first download the pretrained tokenizers and weights from huggingface and then store it in the huggingface cached directory.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Hi, this issue seems related https://github.com/huggingface/transformers/issues/10901. not sure if it will help

Maybe change to python3.7 works


Settings module now has ability to change during a run

Settings can now be modified while running. Needs a bit more work.

Current coverage is 99.21% (diff: 94.23%)

Merging #210 into master will decrease coverage by 0.16%

@@             master       #210   diff @@
==========================================
  Files            23         23
  Lines          1596       1646    +50
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1586       1633    +47
- Misses           10         13     +3
  Partials          0          0

Powered by Codecov. Last update b75ac86...df80d24

Ok, looks ready to be reviewed. What do you think (@jameshensman)? I think this will be pretty useful in tests. E.g.:

Can test against float32 and float64 in a single file (if high precision is needed for a certain test)
Can temporarily switch raise or lower jitter
Demos / experiments can set the configs, for more guaranteed reproducibility

I think this looks great!
Could you make a quick notebook explaining how to use it?

I added the settings notebook to index.rst so that it show up in readthedocs. I also added a few lines to the notebook explaining where the gpflowrc file is and how to use it.


Client Allocation Request for: monkdharma

Client Allocation Questions
Core Information

Name: monkdharma
Website / Social Media:
Region: Greater China
DataCap Requested: 10 TiB
Addresses to be Notarized: f1ma4xksjin3ca3my2stltv4durqxiwvropsgenli
Notary Requested: Emma Russell

Thanks for applying! I would like to ask a few questions for due diligence purposes.
Can you share more details about this data?
Is it public or encrypted?
Where and how will you store you data?
How many copies are you planning to store?


[pulsar-clients]Support nested struct for GenericRecord

Motivation
Currently, GenericRecordBuilder only supports primitive types, e.g. int, long, string. But it doesn’t support struct type.
Modifications
Support nested struct for GenericRecordBuilder, for example AVRO
Verifying this change
Unit Test Pass
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

run java8 tests

rerun java8 test

run java8 tests

run java8 tests

Oh, I see. Seems JSON type is not supported yet.


May I bundle mruby-set to the standard?

As we did for mruby-io etc. in the past.

Sure!

Done. Thank you!


Implement attack mission and battle system logic

When you go to attack someone the button to do so is not available and it says "no mission selected"

Go to Fleet
Click on some combat ships
Click on "Continue" and select the planet
You can see an error when u want to select the mission "Attack"

You should be able to attack someone even with only 1 ship

Thanks for the issue report! This is correct, the attack mission and battle system logic is not implemented yet. Therefore it doesn't show up yet on the fleet dispatch screen.
I'll change the label to mark this as an enhancement.
This is one of the next major milestones for this project to implement.

Ops sorry I didn't realize, you are right
Amazing work anyway! ;)


seems phantom autocomplete not works?

what's wrong? the auto-complete for node repl is ok, why not phantom command?
http://phantomjs.org/repl.html

Phantoms is not Node.js, nor does it run directly on Node.js, so their respective REPL states are irrelevant.

@JamesMGreene i know phantomjs is not node, but the official document of phantomjs said that it support auto-complete, the fact is that it does not, so what's wrong?
the following picture is from http://phantomjs.org/repl.html

@chapgaga, what version of phantomjs are you using? Auto-complete is working for me with version 2.0.0.

I have the same problem, after installing phantomjs2 using npm.

I have the same issue on Linux with phantomjs version 2.1.1.

Same issue here<EMAIL_ADDRESS>on Linux 64-bit.


📰Poster - Magic Mirror Project Introduction and Kick Off Notification

AMC Seminar Sign Up

Link to previous seminar meeting minutes issue:

#176
1. Oral / Poster Selection

Select one of the following selections for Oral or Poster:

Oral should be prepared for 15 - 30 minutes, and no more than 45 minutes, including Q/A.
Posters should be prepared for 5 - 10 minutes, and no more than 15 minutes, including Q/A.

[ ] 🗣 Oral (15 MIN)
[x] 📰 Poster (5 MIN)

2. Paper Information
The Poster contains 3 main part all listed in the project link:

The project background info.
The project mind chart for planning.
The detailed plan for phase I. (to be defined.)

3. After Party 🍻

Everyone will be welcome. You can join or not join as you want (not required by the AMC), and change your mind at any time.

After Party will be started right after all the meeting agenda had been fulfilled.
We will find a bar and drink beers and talk whatever we want. (we can expect that it will be related to meeting topics of the day)
The bill will be paid AA

[ ] 🍻 YES! I'll definitely join the After Party!
[x] 🤔 Maybe. I'd like to decide later.
[ ] 🛌 No, I'm afraid not this time.

4. New Friend

Whether you have a new friend comes with you to the club.

[ ] 👥 I'll bring a new friend
[x] 👤 There's no new friend from me

Artificial Intelligence & Machine Learning CLUB, 2018 - 2019 AI ML Club - AMC Member Manual

求 师兄 挂靠 !

@Chen-Dixi  has subordinated to @newip. You can subordinate to @gaojiuy. The last seat for you 😢


Why not make the proxy part of the image?

So you are using Traefik in another container and proxy through it to the Odoo container. While this follows the typical one-container-one-function paradigm .. it's not totally without precedent to have something like Traefik running in the same container as Odoo, and letting tini get PID 1.
One might argue that the Odoo container should only have the Odoo process and not have its environment cluttered with another process but in reality, even Odoo will spin up other processes when you increase workers in the config.
So why do this?
For one thing, it keeps deployment simpler since you just spin up one container.
Removes the single point of failure: If the reverse proxy container goes down, all nodes behind it are useless.
Is more scalable: As you add more Odoo nodes, your reverse proxy node could eventually be saturated. Having a reverse proxy in the Odoo container means that as I scale up the number of containers, I am also scaling up the number of reverse proxies in parallel.
I have been using this paradigm in production for almost a year and have been very pleased with the results.  Traefik looks enticing though and I might switch to using that instead of nginx.
Thoughts?

@crazy-projects, my use is the same as yours .. SSL termination is at the AWS ELB level. I think this is actually the typical case, so nginx .. or whatever reverse proxy is used ... is just for the sake of getting things working  in Odoo that require it, and should not do more than that.  There are things that are specialized for SSL termination (and security) in production environments so its best to use those things.
Along those lines .. the thing I do different is I use AWS WAF to prevent access to certain URLs, such as the dbmanager. This is just more load I can take off the containers by having this delegated to WAF instead of the reverse proxy inside the container.  If you check your traffic carefully you might see, as I did, a lot of bot and scanner requests coming in  (especially from jorgee, zmeu and yunjiankong) -- in some cases this exceeded legitimate Odoo traffic, so handling this with the reverse proxy in the same container as Odoo is not ideal.

Hi everyone.
It's best to have 1 app per container, always.
The only benefit of doing the other way around would be when you want to boot a production-ready Odoo and bind it to port 80 & 443 directly, but if you need more than 1 Odoo container running together into the same machine or swarm (no matter if they are 2 servers for the same database, or 2 completely different projects), you always need an inverse proxy to balance traffic among them in some unknown and possibly unstandard ways.
The fact of the inverse proxy being your SPF here is not true. It just one of the failure points you have. You could lose network, your database server could stop, an exception while updating an addon could render Odoo unbootable, or your ISP could be burned by a nuclear weapon.
There are strategies to mitigate each of those scenarios, but it's best that deployers investigate those and apply them at their bests. For your comments I can see that most of you are using nginx. We recommend and pseudo-support Traefik, but thanks to not applying what you ask us to do here, you can use the inverse proxy of your preference.
This image is focused on Odoo, and that's how it will continue to be. This standpoint also simplifies support on our side.
Thanks for your understanding.

Completing @Yajo's comment, we have our full stack for Odoo deployment developed, but that's as a Docker compose project which separates each service per container. See the branch https://github.com/Tecnativa/docker-odoo-base/tree/scaffolding for the scaffolding, and other Tecnativa/docker-* projects for the rest of containers.

Regarding .."The fact of the inverse proxy being your SPF here is not true ...You could lose network, your database server could stop".
True in your case, apparently.  If you don't have redundancy for those things as well, as I do, then you're obviously operating at an entirely different level. I actually do run my instances as if my AWS datacenter could be nuked, which is why I have multi AZ deployments for everything and redundancy for everything so if I was to use the setup from this project Traefik would be the SPF. To me there is never a question of "if you need more than 1 Odoo container" .. I always run multiple load balanced instances in-case one goes down. In a production environment, how is this even an option?
All that is fine. I guess the underpinning point here, is that the philosophy of this project is incompatible with someone wanting to run a nuke proof setup, which is what I was seeking to find out.

This thread just hit my daily absurdity limits, blocking.

@sylnsr I think you are very wrong on all your statements and probably our project is far more solid rock, or at least we are more skilled in Docker topics, but hey, do what you want. We don't want people with your attitude in this community.


ValueError: max() arg is an empty sequence

Hi, when I try to install it (after checking requirements) I got this error:
root@localhost:~/kickthemout# python3 kickthemout.py
CRITICAL: Can't open /proc/net/dev !
CRITICAL: Can't open /proc/net/route !
CRITICAL: Can't open /proc/net/dev !
Traceback (most recent call last):
File "/root/kickthemout/kickthemout.py", line 756, in
defaultInterface = getDefaultInterface()
File "/root/kickthemout/kickthemout.py", line 154, in getDefaultInterface
network, netmask, _, interface, address, _ = max(iface_routes, key=lambda item:item[1])
ValueError: max() arg is an empty sequence
I've tried so much things but no one didn't work. Please help me, THANKS!

Are you install kali linux in termux?.the problem is the same as mine


Linting node_modules

Plugin is showing errors/warnings in node_modules.
It's quite common to navigate to source of a lib, which you are using.
However, if code doesn't match your TSLint rules, then VSCode report it as error/warning.

Closing as the solution is to use linterOptions.exclude in TSLint config.
https://palantir.github.io/tslint/usage/configuration/


Various cleanup in template, and improvements to 404 page

See individual commits for details

@usha-mandya @StefanScherer ptal

The improved 404 issue creation link is very useful!

Yes, I initially just wanted to move the script, then thought about what me bothered all the time "I found a broken link" issues; well, they basically reported "I (somehow) arrived on an URL that doesn't exist".
Nothing we can do about those unless we know why they arrived there (did it ever exist? did they make a typo when entering the URL? did someone write a blog-post and made a typo? did we forget to update a link or add a redirect?)


Inline anchors are missing their ids

In the epub output I'm getting a nice chapter for the bibliography, but it's generating anchor links with empty ids, like so:
[] ...

They come out fine in the HTML version.

Hm, if I add extra explicit anchors in a bib entry with [[anchor]]  they also get empty ids, so something is very funny here.

Narrowing further: any anchors inside list items of any kind are missing ids.

Okay, I can't get any inline anchors of any kind to work!

I made some progress on this:

Harmonizing the converter code with that for the html5 converter in a few places helped, in particular the case for :ref in convert_inline_anchor wasn't adding the id when it should. That fixed the anchors not having ids.
For some reason, anchors in a bibliography chapter aren't getting put in the document's catalog (it's actually completely empty apart from the section anchor!). So the generated links actually work, but we get a lot of warnings about them pointing to unknown anchors. I haven't figured this out yet.

I'll put up a PR for the first half, if anyone has any bright ideas about the second half that would be welcome!

Excellent news!

anchors in a bibliography chapter aren't getting put in the document's catalog (it's actually completely empty apart from the section anchor!). So the generated links actually work, but we get a lot of warnings about them pointing to unknown anchors.

Based on what you wrote above, it seems the problem is that you are using the double bracket syntax for anchors instead of the triple bracket syntax. For reasons that have to do with how AsciiDoc was originally implemented, bibliography anchors must use triple brackets (e.g., [[[name]]]).

No, we are using the triple brackets. It's very weird - it all works fine in the html output, so it's not a bug in core, it must be something in this gem.

BTW, I suspect there's a good reason not to do this, but seems worth asking: would it make sense to make the epub3 converter a subclass of the html5 converter? They have a lot in common, and in this case I fixed part of the issue by copying the (probably more widely used and tested) code from the html5 converter.

It didn't used to be possible to subclass the built-in converters. A ton has changed since this converter was developed, so that's now a very real possibility. However, it will require a major version bump, so I think we need to find our way to 2.0.0 in order for that to happen (much like the path the PDF converter is currently on).

it all works fine in the html output, so it's not a bug in core, it must be something in this gem.

I wouldn't doubt it as this converter is very old. It's in need of a lot of work to bring it up to speed with what's been going on in core.

I wouldn't doubt it as this converter is very old. It's in need of a lot of work to bring it up to speed with what's been going on in core.

Yeah, I can see that things have changed, but I'm not quite confident enough to offer a more thorough refactoring, so I'm just going to see if I can find a targeted fix.

Phew, I think I tracked it down. It's an interaction between core and asciidoctor-epub3's include processor. I've described it in https://github.com/asciidoctor/asciidoctor/issues/3356 - I think ultimately it should be fixed in core.

For the moment I can work around this with my PR applied and putting the bibliography inside another section, i.e.
= Dummy section

[bibliography]
== Bibliography

- [[[item]]] This works now

See https://github.com/asciidoctor/asciidoctor-epub3/issues/205 for the second half.

Another solution, as I mentioned elsewhere, is to put the bibliography style directly on the list.
= Bibliography

[bibliography]
- [[[item]]] This works now

Since Asciidoctor EPUB3 calls for the special document arrangement, I think it's reasonable that it be the one to normalize the section style so it can be used without this modification to the document (and, in turn, semantics).

Keep in mind, that workaround won't produce valid DocBook, but if the focus is on AsciiDoc to EPUB3 conversion, that doesn't matter. You could also write it as:
ifndef::backend-epub[]
[bibliography]
endif::[]
= Bibliography

[bibliography]
- [[[item]]] This works now.

That would make it work in all environments.


ComponentGroup: Changing the Layout / Bounds has no effect

Lots of good intentions in this class. You can add components and modify the layout and bounds.
Unfortunately, the layout / bounds are only applied during the add process while making the component entries, so changing those values on the fly has no effect.
Two ways forward perhaps:

Reapply on update, possibly using a hashcode or something to decide if a new layout needs to be applied thus avoiding unnecessary updates
An a method like reLayout that the user's can call to apply all the changes.

cc @hobnob

Technically this is done, however it needs the work in #70 to make it really useful.


Sway unit test (i.e. #[test] fn) compilation

OK, this should be good to go!
Highlights

Adds support for multiple entry points to IR. Adds a tests for serializing entry functions to/from IR.
Enables ASM generation for libraries (to support test function entry points).
Track entry points through ASM generation so we can return entry point metadata as a part of the compilation result. This doesn't affect ASM generation, but allows tools using sway-core as a library to work with different entry points.
Updated E2E test harness with a new test category "UnitTestsPass".
Added E2E tests with multiple unit tests for each type of Sway program (library, script, predicate, contract).
Gets forc test working with pretty output!

Here's the output from the new lib_multi_test:

Test running implementation
Currently, it seems like there's no publicly accessible approach to execute a script directly from a custom entry point. As a result, for each test, we patch the bytecode with a JI instruction that jumps from after the data section setup to the test's entry point. This is a bit hairy, but works for now!
As a follow-up, we may want to consider adding support upstream in fuel-vm for executing scripts directly from a given entry point. Even if this was only exposed from the interpreter API, this would make the forc-test implementation quite a bit cleaner and avoid the need to patch the bytecode.
Alternatively, when building projects we could return more metadata along with the compiled output (whether in memory or bytecode header) to indicate how to work with different entry points in a more reliable manner (rather than the magic const offset currently used in forc-test).
TODO

[x] Add include_tests flag to BuildConfig, allowing forc to trigger compilation of test functions.
[x] Include #[test] fns as entry points within dead code analysis.
[x] IR and ASM generation for test entry points.
[x] Add forc build --tests.
[x] Add forc test.
[x] Always include test fns in TyProgram (for DCA), but omit from IR if not building tests.
[x] Work out how to iterate over different entry points during forc test.
[x] Only generate tests for top-level "members" (not all dependencies).

Follow-up:

#3260
#3261
#3262
#3263
#3264
#3265
#3266
#3267
#3268

Closes #1832.

First, I can't wait to start using this!
Will give a more thorough look over, but one thing I noticed right away is:

Updated E2E test harness with a new test category "UnitTestsPass".

We currently have subdirectories "should_pass"/"should_fail", but these are legacy artifacts from a time before @otrho introduced the test.toml file, which is now the canonical way to declare whether a test should pass or fail. We should remove these subdirectories as they're confusing and just add noise.
However, that could be a small dedicated PR to avoid crowding this one with unrelated changes.

@mohammadfawaz thanks for the review!

Are we planning on disallowing certain opcodes in the tests? For example, we should presumably disallow storage operations, logging (?), contract operations in general, etc.

Ahh good point! Yes denying storage or contract specific operations makes a lot of sense, I've opened another follow-up issue https://github.com/FuelLabs/sway/issues/3278. Maybe logging is something we want to work out how to support for providing feedback, e.g. similar to printing to stdio and using cargo test's --nocapture flag?

Would you say this in a state that we can advertise it already? I assume not before documentation is written including limitations etc.

Yeah perhaps it's worth holding off until we get some docs up?

Are we meant to update sway/Cargo.toml to add the new crate?

Good spot! I wonder why I'm not getting warnings about this :thinking:  it looks like it has still been included within the Cargo.lock file - perhaps a recent cargo update now implicitly includes all nested crates as members or something? I'll add this either way!


Option to limit OrbStack disk size

I don't want my Docker taking up that much space, so if there's a limit, then I can tell when I should clean stuff up.
If this sets the /Users/k/.orbstack/data/data.img file size, that would be extra nice because it would let me tell more easily how much space my home folder is taking up
Related

https://github.com/orbstack/orbstack/issues/29

Is it mainly the Buildkit cache that you're concerned about?

Not anything specific, more like the whole thing. Don't really know how BuildKit cache plays into it :/

This is no longer planned, sorry. OrbStack now cleans build cache automatically, and will also clean unused images automatically in the future.

Sad to read that this thing will not be implemented. I stumbled twice over this issue and got hope for a fix.
My setup is a bit special, but it makes the Mac not usable. Problem: Insufficient Disk space!
So, Orbstack takes 123G from my 256G HDD and this leads to 8G left free on the HDD. The thing is, that I don't need 123G reserved for a few docker images, which are in sum not taller then 2G.
$  du -h ~/.orbstack/data/data.img
123G	/Users/User/.orbstack/data/data.img

$ docker images --format "table {{.Repository}}\t{{.Tag}}\t{{.Size}}"

REPOSITORY                      TAG             SIZE
xxxxxxxxxxxxxxxxxx              latest          616MB
signoz/frontend                 0.35.1          61.6MB
signoz/query-service            0.35.1          54.8MB
signoz/signoz-otel-collector    0.88.3          164MB
signoz/signoz-schema-migrator   0.88.3          28MB
dependencytrack/apiserver       latest          307MB
dependencytrack/frontend        latest          59MB
bitnami/zookeeper               3.7.1           493MB
signoz/alertmanager             0.23.4          60.5MB
clickhouse/clickhouse-server    23.7.3-alpine   615MB
signoz/locust                   1.2.3           85.2MB
jaegertracing/example-hotrod    1.30            18.6MB
gliderlabs/logspout             v3.2.14         31.7MB

Like the RAM and CPU limit, isn't it possible to limit the image disk size for this file?

@sebb-m0 This feature wouldn't help with what you're seeing. To find the culprit, please share the output of docker system df along with a report from Help > Upload Diagnostics.

Time Machine does not allow you to exclude ~/Orbstack.
> tmutil addexclusion /Users/jscheel/OrbStack
/Users/jscheel/OrbStack: The operation couldn’t be completed. Invalid argument

When I was using Docker Desktop, there was this option called Disk Image Size. And I was under the impression that disk space used by Docker Desktop would never exceed this number. And I don't see this option in OrbStack.
The option was helpful for me so I know when to run docker system prune -a, and it helps in not putting my computer to some critical states (I had a warning that my system disk had ~50MB of free space today).


Accept EAI_AGAIN as DNS failure

The error code apparently depends on some configuration. Adding this
one in order to have tests passing in my development environment.

What is your configuration? Can you please add a comment?

I am not sure where it comes from. I can say for sure it is not related to using Node 6 (it fails with Node 4 too). Might be related to my enterprise’s DNS since I get EAI_AGAIN on two systems.


Does it support running on Windows?

In principle, yes. For guidance on installing abspy on Windows, please refer to FAQ here. However, note that we haven't tested it on Windows.


Change books to book in Hadoop InputFormat example

The example books.xml file does not contain a books element. It does
contain a catalog element and many book elements. Previously,
records.count() would return 0. Now, it would return 12.

Nice catch! Could you please check if there are similar instances in the documentation please?

Current coverage is 90.19% (diff: 100%)

Merging #159 into master will not change coverage

@@             master       #159   diff @@
==========================================
  Files            14         14
  Lines           663        663
  Methods         605        605
  Messages          0          0
  Branches         58         58
==========================================
  Hits            598        598
  Misses           65         65
  Partials          0          0

Powered by Codecov. Last update 0c74b41...34e60a9

Ah yes. You're correct there are a few other usages of <books> and </books> under the Features heading, but I think they should be changed to <catalog> and </catalog> respectively. I will update the PR.

Having looked at the diff of my update prior to commit. The usages of <books> and </books> under Features are currently correct because the examples are given inline and before the example books.xml file is linked to:
-* `rowTag`: The row tag of your xml files to treat as a row. For example, in this xml `<books> <book><book> ...</books>`, the appropriate value would be `book`. Default is `ROW`.
+* `rowTag`: The row tag of your xml files to treat as a row. For example, in this xml `<catalog> <book><book> ...</catalog>`, the appropriate value would be `book`. Default is `ROW`.

That said, they'd still be correct if I did update and consistent with books.xml. Completely up to you.
Also, in the API examples where the file is written out the root element is changed from catalog to books. I think it's nice to demonstrate that this can be done, so might be best to leave those as books. Unless your original intention was to keep the root element the same of course!

Yup, thanks. Let's just merge this one. Thanks! merging to master.

Yeap, I am going to merge this. Thanks.