No CSS styles after build

Describe the bug
No CSS styles after build
To Reproduce
Steps to reproduce the behavior:

git clone https://github.com/Vuepic/vue-datepicker.git
npm install
vite build
no errors

Expected behavior
css file has to be in dist folder, as when installed via npm i @vuepic/vue-datapicker, but it's absent. Don't know is it connected to https://github.com/Vuepic/vue-datepicker/issues/526

npm run build:css
to the rescue!
Thanks for a wonderful package.


sequenceMask method throws error when NDArray dimension is 2 but use_sequence_length is True

Description
Currently sequenceMask in MxNDArray.java throws an error if the dimensions is 2 and use_sequence_length is set True (which currently is always the case).
Expected Behavior
Only throw error when dimensions of NDArray is less than 3 and use_sequence_length is False. If use_sequence_length then throw error if less than 2.
Error Message
sequenceMask is not supported for NDArray with less than 3 dimensions
How to Reproduce?
https://github.com/markbookk/java-d2l-IDE/blob/02a5ce874aa8d3fabc3ce944d5ab765b71c8f6fe/section9_6-7/src/main/java/Main.java#L41
Steps to reproduce

This repo contains multiple projects, but if needed section9_6-7 uses sequenceMask as an example of what will throw an error which shouldn't.

What have you tried to solve it?

Changing the condition to only throw error if the dimensions are less than 2 works fine. This only works though because use_sequence_length is currently always True.
I haven't fully implemented this but another solution is to add a method that allows the user to enter the boolean value use_sequence_length and handle throwing the error based on the boolean.

Environment Info
djl 0.11.0 SNAPSHOT

@frankfliu @stu1130 @zachgk


⚠ Samsung has degraded performance

In e0b6c86, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2599 ms

Resolved: Samsung performance has improved in 5c1794a.


Why doesn't pointer-events: none exist on PDOM root?

It is on the every one of the PDOM elements but not the root. Why? It seems like it would more aggressively block pointer events that we do not want. But it may also be necessary to receive the events that we put on the PDOM root. We should try to move it and see if it changes anything. Would probably be better to be on the root if possible.

@jessegreenberg don't feel confident enough to commit, but we feel like this may help solve an issue we were encountering over in #1137. Not sure though.
Index: js/accessibility/pdom/PDOMSiblingStyle.js
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/js/accessibility/pdom/PDOMSiblingStyle.js b/js/accessibility/pdom/PDOMSiblingStyle.js
--- a/js/accessibility/pdom/PDOMSiblingStyle.js	(revision 839a67b2cb0bc3e6bc97ddf96c7550f408bc2408)
+++ b/js/accessibility/pdom/PDOMSiblingStyle.js	(date<PHONE_NUMBER>730)
@@ -28,9 +28,6 @@
                       // in PDOMPeer.positionElements
                       'position: fixed;' +

-                      // ABSOLUTELY CRITICAL - so PDOM elements do not interfere with rest of scenery input
-                      'pointer-events: none;' +
-
                       // default, to the 'relative' root PDOM element - will change with node transform if focusable
                       'top: 0px;' +
                       'left: 0px;' +
@@ -76,6 +73,9 @@
                       // JUST FOR DEBUGGING! So you can see the PDOM on top of other graphical content
                       // 'z-index: 5000;' +

+                      // ABSOLUTELY CRITICAL - so PDOM elements do not interfere with rest of scenery input
+                      'pointer-events: none;' +
+
                       // a catch all for things that are not hidden by the styling on descendants of the root (for example
                       // there is no other way to hide or style check boxes with CSS)
                       'opacity: 0.0001;' +

@jessegreenberg don't feel confident enough to commit, but we feel like this may help solve an issue we were encountering over in #1137. Not sure though.
Index: js/accessibility/pdom/PDOMSiblingStyle.js
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/js/accessibility/pdom/PDOMSiblingStyle.js b/js/accessibility/pdom/PDOMSiblingStyle.js
--- a/js/accessibility/pdom/PDOMSiblingStyle.js	(revision 839a67b2cb0bc3e6bc97ddf96c7550f408bc2408)
+++ b/js/accessibility/pdom/PDOMSiblingStyle.js	(date<PHONE_NUMBER>730)
@@ -28,9 +28,6 @@
                       // in PDOMPeer.positionElements
                       'position: fixed;' +

-                      // ABSOLUTELY CRITICAL - so PDOM elements do not interfere with rest of scenery input
-                      'pointer-events: none;' +
-
                       // default, to the 'relative' root PDOM element - will change with node transform if focusable
                       'top: 0px;' +
                       'left: 0px;' +
@@ -76,6 +73,9 @@
                       // JUST FOR DEBUGGING! So you can see the PDOM on top of other graphical content
                       // 'z-index: 5000;' +

+                      // ABSOLUTELY CRITICAL - so PDOM elements do not interfere with rest of scenery input
+                      'pointer-events: none;' +
+
                       // a catch all for things that are not hidden by the styling on descendants of the root (for example
                       // there is no other way to hide or style check boxes with CSS)
                       'opacity: 0.0001;' +

It seems like an OK time to try this out, I just requested a spot check in https://github.com/phetsims/qa/issues/1093. If all is well there we can close.

The spot check went through without any big problems, so this can be closed.


Transcriptomics Comparison Tab: select a comparison, green bar items DOWNLOAD, FASTA not working

Trascriptomics experiments and comparison selection toolbar should not have fasta download.
Please remove it.
-Maulik

On Aug 19, 2016, at 2:44 PM, maovt<EMAIL_ADDRESS>wrote:
https://cloud.githubusercontent.com/assets/6361802/17822204/c14b99b0-6623-11e6-83cd-7a4907c2c0b6.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/PATRIC3/patric3_website/issues/877, or mute the thread https://github.com/notifications/unsubscribe-auth/ABLd76kQRFJ1wcC4qVCMqvwnXIN6u53Bks5qhgeQgaJpZM4JoytV.

Removed FASTA button

Will be pushed to the new build

Verified on beta.


Model generates short text no matter how big maximum_tokens is set

Describe the bug
Model generates short text no matter how big maximum_tokens is set
text_generation.yaml
'{
"text-gen-type": "input-file",
"maximum_tokens": 500,
"temperature": 0.9,
"top_p": 0,
"top_k": 0,
"recompute": false,
"num-samples": 10
"sample-input-file": "prompt.txt",
"sample-output-file": "sample_output.txt",
}'
##Output
{"context": "Arnold Schwarznegger is a famous actor and a body builder. He won the", "text": " Mr. Universe title in 1978. This bodybuilder is now 58 years old and his body is absolutely perfect with hard muscle tissues. He has a new movie called [\u2026]", "length": 33, "finished": true, "message": null, "duration_seconds": 4.630121469497681}

This is not the case with other prompts.


Koa: Add test for header added by middleware.

Codecov Report

Merging #193 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #193   +/-   ##
=======================================
  Coverage   89.32%   89.32%
=======================================
  Files          57       57
  Lines        2454     2454
  Branches      305      305
=======================================
  Hits         2192     2192
  Misses        260      260
  Partials        2        2

Continue to review full report at Codecov.

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


[Bug]: Better error message for specifying inputs on chain pipeline in Beam YAML

What happened?
When specifying explicit inputs to transforms in Beam YAML, the error message is difficult to decipher.
For example, running the following pipeline:
pipeline:
  type: chain
  transforms:
    - type: Create
      config:
        elements:
          - id: 1
    - type: LogForTesting
      input: Create

will result in the following error message:
jsonschema.exceptions.ValidationError: 'Create' should not be valid under {} around line 475

Failed validating 'not' in schema['properties']['pipeline']['allOf'][2]['then']['properties']['transforms']['items']['properties']['input']:
    {'not': {}}

On instance['pipeline']['transforms'][1]['input']:
    'Create'

This should instead print something like "Cannot define explicit inputs on chain pipelines"
Issue Priority
Priority: 2 (default / most bugs should be filed as P2)
Issue Components

[ ] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[X] Component: Beam YAML
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Infrastructure
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

.take-issue

@Polber ptal at the PR

@Polber closing this issue as the fix has been merged !

.close-issue


Key Vault Name restriction

You need to add more info to this section

VaultNameNotValid
The vault name should be string of 3 to 24 characters and can contain only numbers (0-9), letters (a-z, A-Z), and hyphens (-)

Update:-
Vault name must only contain alphanumeric characters and dashes and cannot start with a number.
A vault's name must be between 3-24 alphanumeric characters. The name must begin with a letter, end with a letter or digit, and not contain consecutive hyphens.

I have cx whom trying to create KeyVault starting with no. and we have this error
The vault name '6e49f4337c8b' is invalid.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6d30bc0d-38a2-2a52-398c-67eb947df328
Version Independent ID: 48b2bb25-c449-0f7a-f682-413a298138fe
Content: Common error codes for Azure Key Vault
Content Source: articles/key-vault/general/common-error-codes.md
Service: key-vault
Sub-service: general
GitHub Login: @sebansal
Microsoft Alias: mbaldwin

@Shalzayed Thank you for your feedback . We have verified this and have initiated a change request .

The changes will be live once the content team approves it .
Thank you .


Two groups with same name, but different color

My use-case for TM is to have tabs for approx 60-70 different URLs to servers which all would look identical if not for TM. One very important dimension is production vs test. Naturally, I would use color (red/yellow) to separate the two environments - but the same names. That works fine for TM rules, but not for groups (understandable, since group concept is pretty new).
The issue is this:
Tab Rule 1: "S1" (red) - (URL fragment "prodserver.com")
Tab Rule 2: "S1" (yellow) - (URL fragment "testserver.com")
Group 1: "G" (red);
Group 2: "G" (yellow)
TabRule1 is placed in Group 1
TabRule2 is placed in Group 2
If you open the URL "prodserver.com" - Group 1 will be displayed. After that, you open "testserver.com" and both! tabs will be added to Group 2 (yellow). If you open "prodserver.com" again, all three tabs will be added to Group 1...and so on.

Thanks for your feedback. I've implemented a fix, which will only work if the colors for the same group name are different. This will resolve the issue in release 1.0.9.
To avoid mixing tabs between groups, ensure each group has a unique color. Even if groups share the same name, distinct colors will keep them separate. This way, your "prod" and "test" environments won't get confused in the tab management.


[BEAM-12984]: Add support to collection PCollections from composites

Composites were being incorrectly pruned out in the PipelineFragment stage. This PR adds support by recursively adding all transforms and PCollections under a composite necessary to compute the given PCollection.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @KevinGG


[FEA]: Upgrade to RAPIDS 24.04/CUDA 12.4

Is this a new feature, an improvement, or a change to existing functionality?
Change
How would you describe the priority of this feature request
High
Please provide a clear description of problem this feature solves
RAPIDS has begun generating matrixed builds of 24.04 for CUDA 12.4. We should align our builds to that when the time is appropriate.
Describe your ideal solution
Changes as needed to the versions of cuDF and RMM
CI checks
smoke tests (including container image builds)
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I have searched the open feature requests and have found no duplicates for this feature request

Superceded by #1717


Cannot run/no response when running

I pasted the full script from Multiagent-Debate on pycharm，and tried to run the script code on cmd/anaconda prompt, but no response

Screenshots

First you need to install metagpt correctly. Please refer to: https://docs.deepwisdom.ai/main/en/guide/get_started/installation.html#install-in-development-mode

First you need to install metagpt correctly. Please refer to: https://docs.deepwisdom.ai/main/en/guide/get_started/installation.html#install-in-development-mode

Thank you very much for your answer. I guess I had some minor problems when I first installed it. I will troubleshoot the problem and continue to try your project.

First you need to install metagpt correctly. Please refer to: https://docs.deepwisdom.ai/main/en/guide/get_started/installation.html#install-in-development-mode

Thank you very much for your answer. I guess I had some minor problems when I first installed it. I will troubleshoot the problem and continue to try your project.

I have successfully run the project code, thank you again for your reply


Chromium and Firefox both fail a11y on buttons — Option 1

Adrian's testing shows both Chromium and Firefox have accessibility problems when display: contents is applied to buttons.
This PR sets all browsers with any bugs to "partial support".

Closing as this has been resolved in a separate change.


Parent divs being removed

Not sure if i'm doing something wrong but it seems to be removing parents divs from items and arranging their children instead.

I get the same thing but in different environments.  In my case, the parents in my code are html .


Uncaught Exception: Attempting to call a function in a renderer window that has been closed or released. Function provid

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.0.1
Time: Mon Mar 14 2016 16:31:56 GMT 0100 (W. Europe Standard Time)
Error:
Attempting to call a function in a renderer window that has been closed or released. Function provid
Error: Attempting to call a function in a renderer window that has been closed or released. Function provided here: Object.<anonymous> (C:\Users\ynaut\AppData\Local\GPMDP_3\app-3.0.1\resources\app\build\inject\GPMInject\interface\mouseButtonNavigation.js:8:8.
    at BrowserWindow.callIntoRenderer (C:\Users\ynaut\AppData\Local\GPMDP_3\app-3.0.1\resources\atom.asar\browser\lib\rpc-server.js:201:19)
    at emitTwo (events.js:92:20)
    at BrowserWindow.emit (events.js:172:7)

This issue was created automatically inside the uncaughtException handler

Closing this automatically reported issue as a duplicate of #470


Add Ireland flag

@skeevey Hey! :smile:
This looks great! Thank you for sending this PR. Merging this.
Keep contributing! :smile:


Allow disabling OVN chassis per cluster member

Documentation preview available at: https://linuxcontainers.org/lxd/docs/pr.10088/

@tomponline would have made mitigating the issue I had this week a fair bit easier ;)


Improve performance by reducing redundant work

Fixes #19.
This PR should improve squisher's performance both when compressing images and when returning already-squished images.
This PR:

Adopts toktx's stdout output mode. This should help avoid a lot of I/O when squishing large images.
Avoids writing input images to disk if they're already cached.
Avoids decoding input images unless they need to be resized.
Moves the temporary file writing code into the toktx function. This lets us skip that step if the image is already compressed. compress_image was removed in favor of using tempfile to manage the temporary input file.

Additionally, I've removed the URI image support temporarily as it appears that squisher does not currently resolve paths relative to the input file path. I'd like to reintroduce that in a follow-up PR and add a test case for it to ensure the paths are correct.
It turns out that toktx does not actually have an stdin input mode (https://github.com/KhronosGroup/KTX-Software/issues/637), so I wasn't able to make any changes on that front.


PixmapPacker - stretch-padding

See https://code.google.com/p/libgdx/issues/detail?id=1257

There is a duplicatePadding option in PixmapPacker, which I assume is what was being requested. Closing this.


Notifications sent twice on action

make a inclusion request
go to request detail page
try to accept the request
add comment in "Additional comments" box
Click accept

Bug:
the mail is sent twice, once for accept, the second time for the additional comment.
See:

Current state
The current simplified service flow when executing an action is as follows (code):

get the specific request action
execute the request action
if there is additional data (assuming it is only for comments)

create a comment

Most notifications are registered when executing an action. The additional data is not available there though. This means we can send the action notification but after that another comment notification may be registered as well.
Possible solutions
Introduce an attribute on the base request action
Similar to the log_event attribute, an attribute such as create_comment can be introduced. This attribute will then be checked in the base execute_action of the requests service before creating a comment

Since we usually override the request action for a notification, we can also specify this attribute.
In the action, we still would not have the additional data but at least we would not send the action notification and the comment notification on one action.
If there is a comment, the user/guest could checkout the request timeline, if interested.

Impact
This will have a rather small impact on the general application.

Specify the attribute with a default value of True
Adapt it for the current actions with notifications.

Move the creation of a comment to the base request action
Creation of the comment could be moved from the service to the request action. Additionally, the attribute for create_comment can be introduced as well, to allow easy configuration for the base method logic.

Since we usually override the request action for a notification, we can also specify this attribute.
We are able to include the comment in the respective action notification - thus only sending one notification with all the information.
Moving the creation of the comment to the request action would allow to further customize the action in the future. If the provided additional data will not always be a comment, it can be dealt with accordingly on the specific action level, where we should have enough context. Thus providing a lot of flexibility for the future.

Impact
This will have a large impact on the general application.

Specify the attribute with a default value of True
Adapt it for the current actions with notifications.
An additional function parameter has to be added to the execute action (explicit data=None, implicit **kwargs) and adapted for every subclass.
Move implementation from service to request action
Adapt calls (e.g. RequestActions, RequestService )

A possible solution would be to go with the attribute only first, to have a quick solution for the general issue.
Then follow up with moving the comment creation to the request action as a next step, as it would probably lead to braking changes and a new major version?


Added more data in database and added 11 images for dashboard tiles

One user is added in Database(Dungali Shreehari) and his courses (MEL-151 and MEP-151)
Each course in course table except from Course1 to Course8 of Database now contains details of 4 students
Course1 to Course8 of Sneha Chauhan in Database are only for demo purpose for dashboard, to show different images in tiles.
11 images are added in images folder for tiles in Dashboard. These images will repeat if number of tiles exceed 11.

@PurviGoyal7 the way you presented your PR is highly appreciable.


Bump min. version for evaluator parallelism feature flag

Since we suspect fine-grained parallelism may make the OOM issues worse, we don't want to enable it on 2.14.0-2.14.5, even though they support it.
Merge / deployment checklist

[ ] Confirm this change is backwards compatible with existing workflows.
[ ] Confirm the readme has been updated if necessary.
[ ] Confirm the changelog has been updated if necessary.

Note that when we ramp up the feature flag, customers running old versions of the CodeQL Action may have fine-grained parallelism enabled despite using an older version of the CLI.  This is probably an acceptable risk, given that most customers will be running the up-to-date Action, and running on the older CLI versions will only rarely cause breakage.

Good point, but I agree the risk is low. Fine-grained parallelism should work just fine if it gets enabled on 2.14.0-5.


hopefully fixed the merge conflicts for joycells work

this is all Joycell's work, i just refactored her code to fit into our main.handlebars and hope it works

Not needed!


pre-train model result

just try pre-train model and result shows：
Accuracy: 0.1019
mAP: 0.8815
Is it correct？
And I train model under gtx 1070, shows CUDA error:out of memory.
Whats your environment？

The code list the directory names for constructing the label and  os.listdir returns arbitrary order , so the label is not same as author.
https://www.tutorialspoint.com/python/os_listdir.htm

Thanks for your attention to our work.

As @tranduytrung answered, it is caused by the different orders returned by os.listdir in different file storage systems. We will soon fix it with a mapping from type names to category indices.
We use one GTX 1080Ti to train in the given configuration. If you get a out of memory error, you may need to reduce the batch_size in config/train_config.yaml to make it suitable for your environment.

Could you plz update the tools you used to change .off to .npz and .npz to .off?
大佬给下mesh简化的方法，还有npz文件和off文件互转的工具（跪了，从npz还原回off永远模型都是破的，而且长得很诡异QWQ）

Yes, we will soon release the codes to convert from .off to .npz files. But the conversion from .npz to .off is not easy to achieve, since it lose some information.
The simplification methods are discussed in #6 , but we are also looking forward to a better simplification method.
(The broken cases do exist, because we restrict the number of faces in the simplification process. But the whole appearance should be correct.

For each data file XXX.off in ModelNet, we reorganize it to the format required by MeshNet and store it into XXX.npz. The reorganized file includes two parts of data:

The "face" part contains the center position, vertices' positions and normal vector of each face.
The "neighbor_index" part contains the indices of neighbors of each face.

大佬问下你在从npz还原回off怎么做的？
我是：

取‘face’中 每一行的3~12部分，即vertices' positions
去除这些vertices' positions重复部分，生成一个vertices list
通过搜寻vertices在vertices list的index，取代‘face’中的vertices' positions，生成新的 face list
根据off格式排列vertices list与 face list，生出off档案

大佬问下你在从npz还原回off怎么做的？
我是：

取‘face’中 每一行的3~12部分，即vertices' positions
去除这些vertices' positions重复部分，生成一个vertices list
通过搜寻vertices在vertices list的index，取代‘face’中的vertices' positions，生成新的 face list
根据off格式排列vertices list与 face list，生出off档案

@Lordores  你好，我并没有直接做过npz到off的转换，转成npz只是为了更方便地进行训练。关于你的转换方法，描述看起来是没有问题的，如果结果有问题的话，你可以贴一下关键代码及得到的模型情况。

如果没有使用npz到off转换 的话，请问下Figure 6的特征可视化部分是怎么得到的？

如果没有使用npz到off转换 的话，请问下Figure 6的特征可视化部分是怎么得到的？

由于特征可视化中需要指定每个面的颜色，我们使用了自己的渲染代码，根据面的特征值和及npz文件中存储的顶点位置生成可视化结果。


Error generating exe

hi，server reported an error when I generated the payload

I am on OSX montery and Teamserver is on Windows

I Use OSX to TeamServer is OK

So just to confirm, changing to using OSX as your os for the teamserver to run on fixed your issue?


Karabiner for NUMLOCK fix

Hi, I have been using the following in the Karabiner private.xml in El Capitan for my QuickFire TK Keyboard.
<?xml version="1.0"?>
<root>
    <devicevendordef>
        <vendorname>CM_Storm</vendorname>
        <vendorid>0x2516</vendorid>
    </devicevendordef>

    <deviceproductdef>
        <productname>QuickFire_TK</productname>
        <productid>0x0011</productid>
    </deviceproductdef>
    <item>
        <name>Quickfire TK</name>
        <identifier>private.quickfiretk</identifier>
        <device_only>DeviceVendor::CM_Storm, DeviceProduct::QuickFire_TK</device_only>
        <autogen>
            __KeyToKey__ KeyCode::KEYPAD_CLEAR, KeyCode::VK_IOHIKEYBOARD_TOGGLE_NUMLOCK
        </autogen>
    </item>
</root>

How do I get the same working in Karabiner-Elements in Sierra? Can't I?

Hi @raj-blippar , I have the exact same keyboard and same issue as yours. I have created https://github.com/tekezo/Karabiner-Elements/issues/719. I don't think you can get the same working in Karabiner-Elements for now. Without this feature the numberpad of compact keyboard like TK-Storm or TK Stealth or other similar brands/models won't function correctly.  I hope @tekezo will have a chance to take care of this. He is quite busy now. We need to be patient.
I brought a new Mac mini which shipped with 10.12 OX. Because OS 10.12 cannot use the Karabiner, I have to downgrade it to 10.10.
Thanks.

Any progress?

Can any one give me some hints or documents about the structure of Karabiner-Elements, so that I can work on to add/fix such feature?

I have a realforce 87u and want to use the embedded numpad via the NumLock key.


cssrewrite "internal server error" (suggested patch)

Hi,
My twig layout contains this:

    {% stylesheets output='/bundles/mysite/css/mini.css' filter='?yui_css,cssrewrite'
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>    %}

    {% endstylesheets %}

The generated HTML looks like this (in dev mode):
<link rel="stylesheet" media="screen" href="/app_dev.php/bundles/mysite/css/mini_reset_1.css"/>
<link rel="stylesheet" media="screen" href="/app_dev.php/bundles/mysite/css/mini_jquery.fancybox-1.3.4_2.css"/>
<link rel="stylesheet" media="screen" href="/app_dev.php/bundles/mysite/css/mini_nivo-slider_3.css"/>
<link rel="stylesheet" media="screen" href="/app_dev.php/bundles/mysite/css/mini_style_4.css"/>

However the cssrewrite is causing issues.  When I use the cssrewrite filter all the css file links that get output have this message:

[exception] 500 | Internal Server Error | ErrorException
[message] Warning: strpos(): Empty delimiter in /var/www/mysite/vendor/assetic/src/Assetic/Filter/CssRewriteFilter.php line 56
[1] ErrorException: Warning: strpos(): Empty delimiter in /var/www/mysite/vendor/assetic/src/Assetic/Filter/CssRewriteFilter.php line 56

The error is because during the while loop in the third condition below targetDir becomes empty.

            // pop entries off the target until it fits in the source
            if ('.' == dirname($sourcePath)) {
                $path = str_repeat('../', substr_count($targetPath, '/'));
            } elseif ('.' == $targetDir = dirname($targetPath)) {
                $path = dirname($sourcePath).'/';
            } else {
                $path = '';
                while (0 !== strpos($sourcePath, $targetDir)) {
                    if (false !== $pos = strrpos($targetDir, '/')) {
                        $targetDir = substr($targetDir, 0, $pos);
                        $path .= '../';
                    } else {
                        $targetDir = '';
                        $path .= '../';
                        break;
                    }
                }
                $path .= ltrim(substr(dirname($sourcePath).'/', strlen($targetDir)), '/');
            }

To be honest I really don't understand what that these conditions and loops are trying to achieve, but what I do know is that if I comment out or remove the third condition so it looks like this:

            // pop entries off the target until it fits in the source
            if ('.' == dirname($sourcePath)) {
                $path = str_repeat('../', substr_count($targetPath, '/'));
            } else {
                $path = dirname($sourcePath).'/';
            }

It works perfectly.  This is because the links in my css look like "../images/mypic.png" by the way, so adding dirname($sourcePath) i.e. where the css is coming from you get what you expect and I can't honestly think of under what condition you would want anything else than that, I'm not even sure why the first condition is needed. If the third condition is some special handling could it be made to fall back to the second one if this error occurs?
Please let me know if I am using cssrewrite wrong but no matter what I try I always end up back with this problem, I can't believe I'm the only one?!

Sorry, I got confused with some caching and thought it was working when it wasn't ... to make it work I need the code to use "targetPath" not "sourcePath", i.e. like this:

            // pop entries off the target until it fits in the source
            if ('.' == dirname($sourcePath)) {
                $path = str_repeat('../', substr_count($targetPath, '/'));
            } elseif ('.' == $targetDir = dirname($targetPath)) {
                $path = dirname($sourcePath).'/';
            } else {
                $path = dirname($targetPath).'/';
/*
            } else {
                $path = '';
                while (0 !== strpos($sourcePath, $targetDir)) {
                    if (false !== $pos = strrpos($targetDir, '/')) {
                        $targetDir = substr($targetDir, 0, $pos);
                        $path .= '../';
                    } else {
                        $targetDir = '';
                        $path .= '../';
                        break;
                    }
                }
                $path .= ltrim(substr(dirname($sourcePath).'/', strlen($targetDir)), '/');
*/
            }

The CssRewrite filter is not compatible with the @MyBundle notation currently. You need either to reference the CSS files in the web/ folder (where they are available too when you install the assets as they are in the public folder) or to use the CssEmbed filter to embed images in the CSS


AttributeError: 'T5Attention' object has no attribute 'relative_attention_bias' when using load_checkpoint_and_dispatch to load T5-11b

System Info
accelerate version: 0.20.3
OS: ubuntu 18.04
python: 3.9
torch: 2.0.1

Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[x] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[ ] My own task or dataset (give details below)

Reproduction
Step to reproduce the bug:

Install related libraries by pip
Run the below

from transformers import T5Tokenizer, T5ForConditionalGeneration
from accelerate import init_empty_weights, load_checkpoint_and_dispatch, infer_auto_device_map
from transformers import AutoModelForSeq2SeqLM, AutoTokenizer, AutoModel
import torch

model_name = "t5-11b"
tokenizer = T5Tokenizer.from_pretrained(model_name)
with init_empty_weights():
    model = T5ForConditionalGeneration.from_pretrained(model_name)
device_map = infer_auto_device_map(model)
model = load_checkpoint_and_dispatch(
model, "/nas/huggingface_pretrained_models/models--t5-11b/snapshots/90f37703b3334dfe9d2b009bfcbfbf1ac9d28ea3/pytorch_model.bin",
device_map=device_map)

Expected behavior
The modules are loaded to the device.
However I get this error:  AttributeError: 'T5Attention' object has no attribute 'relative_attention_bias' . Below is the full traceback:
Traceback (most recent call last):   File "/nas/thuchk/NLLB_transformers/movie_subtitles_translation/T5_huggingface/batch_infer_sharded.py", line 66, in <module>     full_result = T5_batch_infer(model_name=model_name, data_path=intputdata_path, batch_size=1)   File "/nas/thuchk/NLLB_transformers/movie_subtitles_translation/T5_huggingface/batch_infer_sharded.py", line 27, in T5_batch_infer     model = load_checkpoint_and_dispatch(   File "/nas/.conda/envs/subtitle_translation/lib/python3.9/site-packages/accelerate/big_modeling.py", line 486, in load_checkpoint_and_dispatch     load_checkpoint_in_model(   File "/nas/.conda/envs/subtitle_translation/lib/python3.9/site-packages/accelerate/utils/modeling.py", line 1116, in load_checkpoint_in_model     set_module_tensor_to_device(model, param_name, param_device, value=param, dtype=dtype)   File "/nas/.conda/envs/subtitle_translation/lib/python3.9/site-packages/accelerate/utils/modeling.py", line 142, in set_module_tensor_to_device     new_module = getattr(module, split)   File "/nas/.conda/envs/subtitle_translation/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1614, in __getattr__     raise AttributeError("'{}' object has no attribute '{}'".format( AttributeError: 'T5Attention' object has no attribute 'relative_attention_bias'
I run the debugger and find out that when I print the module T5Attention it doesn't have the relative_attention_bias layer, but tensor_name variable value has that layer. This discrepancy causes the error.

I don't understand what you are trying to achieve with this code (that I can't reproduce). Why not use directly?
model = T5ForConditionalGeneration.from_pretrained(model_name, device_map="auto")

I was trying to load a model larger than my GPU size. The below function load each module to the CPU and then dispatch them to GPU one by one. But when I run it I got the error above.
model = load_checkpoint_and_dispatch(
model, "/nas/huggingface_pretrained_models/models--t5-11b/snapshots/90f37703b3334dfe9d2b009bfcbfbf1ac9d28ea3/pytorch_model.bin",
device_map=device_map)

I was trying to load a model larger than my GPU size.

Yes, the code above will do exactly that.

I am having exactly the same question here. I have been doing some research but i just cannot find the difference between from_pretrained and load_checkpoint_and_dispatch. Why is the Accelerate tutorial on making inference using big models specifically mentioned load_checkpoint_and_dispatch not just from_pretrained? Are they the same under the hood?

The tutorial for Accelerate shows you how to do things for any model, it just takes a Transformer model as an example. The Transformers library is a tighter integration, so for any Transformer model, you should use from_pretrained


Remove all legacy graph data/support on August 17th, 2015

Oh boy, this issue will be open for a while!

:skull:


Multiple electricity interconnectors

Adds slides and sliders for interconnectors 2 through 6.
Supports minor changes in the ETEngine API.
Makes adjustments to charts and tables to show the new interconnectors.
Adjusts sankey charts to hide series with no demand.
Adds the excellent "Select curve from a scenario" feature written by @noracato.

Looks good! Great work and the scenario selection works flawlessly! Haven't tested super thoroughly, but @michieldenhaan and @AlexanderWirtz will do that next week on beta


Storage & Compute release 2024-11-04

Storage & Compute release 2024-11-04
Please merge this Pull Request using 'Create a merge commit' button

reviewed for changelog


Section 5: ChatPlayer

@defreez I think this section is done. I went through the whole tutorial up to this point. Everything worked as expected.
As i was going through, i did find and fix a few typos in sections 1 to 4. I put these in the last commit https://github.com/RaiderSoft/semantic-godot/pull/25/commits/268bf9b043efff099536428f023f6746e1c0abe8. Let me know if i should instead put these somewhere else.
Since you opened this pull request, i can't request you as a reviewer. Should i request myself as a reviewer and then approve it?

Thanks for putting this together. I added a warning about having an empty _Ready method in ChatPlayer.
LGTM


Alarm trigger is not accurate，how  to check it?

PINPOINT v1.8.1

INFO:
my application's  alarm rule: ERROR RATE= 10%.
But the scatter data is as following.

From the above data, alarm was not accurate, and how can i check it or what could be the cause of it?
Thanks.

@fionaliupw
Look at the log when the alarm goes off. Set log level to debug.


[1pt] Choose assets - group task

Choose images, text, icons, fonts.

Font will be Mukta

Icons will be:

Likes
Hamburger
Logo


add sorin config variable to allow full PWD to be printed

I have multiple source code paths for work that I need to be able to see at a glance, otherwise I have to keep typing "pwd" manually to see what directory I'm in. For example, I'll have 6 iTerm tabs open at once with these directories open:
/Volumes/SourceCode/src/git/project1-2014/foo/bar/baz/wibble/
/Volumes/SourceCode/src/git/project1-2014-rel/foo/bar/baz/wibble/
/Volumes/SourceCode/src/git/project1-2015/foo/bar/baz/wibble/
/Volumes/SourceCode/src/git/project1-2015-rel/foo/bar/baz/wibble/
And if all I can see is "/V/S/s/g/p/f/b/b/wibble" in my pwd for sorin, I can't tell what branch I'm in without typing pwd manually.
I've added a config variable to allow the full pwd to be printed instead of the shortened version. Totally new to prezto, so definitely open to comments/suggestions/changes.
Thanks!

Hey @ravishchawla  this repo is not being maintained, see #1239
The community fork is here https://github.com/zsh-users/prezto, please move this issue to zsh-users and we will be able to help you!
zsh-users is an organization dedicated to host Zsh community projects and therefore became the rational home for a community maintained version of prezto.

I want this really badly...  @vanrijn Any chance of you cleaning this up so it can be accepted?  If not, I may just submit this as my own PR shortly...

A two-line theme is better suited for your use case.

Because this was refactored into promptpwd, it may be worthwhile adding the option there... it would be a nice addition that would allow configuration of multiple themes.
If you want to make that change, I can take a look @malikoth

I keep my terminal full screen on my third widescreen monitor, conserving horizontal space is the lowest possible priority for me.  If there were a two-line theme that has the nicely formatted git-info, the attractive colors, the arrows that switch direction depending on the vi-style input mode that your theme has, I might try switching.  But your theme is by far the most functional, I like it.  I just can't ever tell where I am with the one-letter directories.

Wow, thanks for all the feedback guys. Let me address things one at a time...

I want this really badly... @vanrijn Any chance of you cleaning this up so it can be accepted? If not, I may just submit this as my own PR shortly...

@malikoth I don't think I'm going to be able to get to this for a few days. If you want to submit a different PR for this, please feel free and I will close this one out.

A two-line theme is better suited for your use case.

@sorin-ionescu I would love to use a different theme. Here's actually the core problem: the sorin theme is the ONLY one I can find that generates and produces the git info in such a way that it doesn't block my prompt from even showing up until that's done. In other words, all other prezto themes I've tried will freeze my command prompt and not show anything until the git info is produced. I use git for my work repository, and we have a HUGE git codebase (like 3.6GB huge). This means that every time I run a command, which is very frequently, it takes like 2-5 seconds for my command prompt to return because it is delayed until the git info is produced. I like sorin because it's a very nice, compact, elegant theme, but mostly because it doesn't make my command prompt hang in order to show git info. My command prompt is returned to me immediately and then the git info is lazily filled in on the right of my prompt whenever it's ready. If another theme (all themes?) could do this, I'd be open to switching to something more verbose.

Hey @ravishchawla this repo is not being maintained, see #1239
The community fork is here https://github.com/zsh-users/prezto, please move this issue to zsh-users and we will be able to help you!

@facastagnini If @malikoth doesn't get to this before me, should I close this issue out, make the changes suggested, and open an issue with the community fork you mentioned?
Thanks again, all!

should I close this issue out, make the changes suggested, and open an issue with the community fork you mentioned?

No.  Sorin has added other maintainers like belak (and others?  Is @paulmelnikow a maintainer too?) and so this repo is officially alive and well again.  The community fork has been abandoned, and all activity should simply remain here.

I hope you don't mind vanrijn, but I did go ahead and make the PR.  Check it out!  (See above)

Closed in favor of #1310. Thanks for the idea! I hope to get this in soon.

Awesome thanks so much @malikoth!! Sorry I got waylaid on implementing this myself. Thanks so much for the enhancement!! =:)

As a matter of keeping the historical record straight, #1310 was almost immediately superceded by #1312.  If anybody wants to use this feature, look for the details there.


Update install setup

Please confirm that this PR has done the following:

[ ] Tests Added
[ ] Documentation Added
[ ] Description in RELEASE_NOTES.md Added

Adding to RELEASE_NOTES.md (remove section after adding to RELEASE_NOTES.md)
Please add a single line in the release notes similar to the following:
- (#XX)[http://link-to-pr.com] Added feature which does something

Description of PR
A way to update the install setup to make everything easier and more automatic for the user.
For example, this now works
git clone git@github.com:IAMconsortium/pyam.git
pip install -e .[test]
pytest

can you add the license per #121 please?

to get the release notes right I should probably rebase onto #128 (or master once #128 is merged)


WIP re-enable large message test

Seems like chrome really needs to be at 89 for this (current version on travis seems to be 88).

Seems like chrome really needs to be at 89 for this (current version on travis seems to be 88).

Chrome 89 seems to be included in the virtual-environment release of March 9: https://github.com/actions/virtual-environments/releases/tag/ubuntu20%2F20210309.1


MINOR: Replacing for with foreach loop in stream test classes

Thanks for the PR, this is a good readability improvement. Surprising that there are so many old style loops in the code. @dguy, maybe you can review?

@dguy Thanks for pointing it out. It was a mistake. Please check I have fixed it.

@IJuma Do you suggest any further improvements?

test this please

Thanks for the patch @PKOfficial ! LGTM and merged to trunk.


Improve readability of brew upgrade

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

Hi, this is just a small cosmetic change that I find useful.
When upgrading lots of packages at the same time, the list of packages can be hard to read due to the length of the list and the fact that most terminals by default do their line breaks exactly when the line is full and not on word endings. This often leads to chopped up package names and versions. It is not easy to quickly find a package at a glance.
For example, on my machine:

Note the line endings and beginnings.
With this change every package is printed on a new line. Package names are sorted alphabetically:

I'm assuming that the output is for human interaction only and will not be parsed by other tools?
Also, Is the order of the packages important? They are sorted alphabetically now to make packages easy to find at a glance. Should we rather display them in the order they will be upgraded?
Not sure if this change is wanted at all, but I'd be happy for ideas for improvements.

Personally I prefer the prior output even although I agree it's not ideal with extremely large numbers of packages. @Homebrew/maintainers: thoughts?

I think the latter might be nicer. I don't have a strong opinion about either one.

Sorted output is what many other package managers (conda, yum) do. I find it a lot more readable.

I prefer the latter (one per line), though I find the order of upgrading more important than alphabetical — in case of failure failure or requiring an interruption, it’s easier to know what was already done.

Could we add some flag that allows us to format it as described above or even make it silently?

I'd prefer sorted, and ", " if it fits in one line, otherwise "\n".

I prefer sorted by name (or possibly order of installation), and a mild preference for one per line.

As the 'Updated Formulae' section is already sorted alphabetically, I'd prefer keeping the Upgrading N outdated packages: section in order of upgrading.
I'd keep the current format (which is compact and fairly readable when updating often), if it doesn't exceed a few lines (say 3), and switch to line-by-line beyond that.

I feel that switching the formatting conditionally is not worth the effort.

@sjackman I don't think effort is a problem here (looks like it would just be a one-liner). However, I do agree with you on the consistent format.

I think the proposed changes (1 package per line) are easier to read.
A flag or environment variable to sort alphabetically or topo-sorted by dependency/update order would be nice, but if we have to pick only one, then I would say keep existing sorting on update-order/dependency topo-sort as it's trivial to pipe the output into the sort command to get alphabetic sorting.
The only drawback I see is that if someone doesn't set their terminal scrollback history/buffer to be long enough they may lose the beginning of the output if there is an absurdly large number of packages to upgrade...

@MikeMcQuaid Just realized i forgot to squash the commits. It is rebased now.

Perfect!
Thanks so much for your contribution!  You rock @kluen!,


The 'headers' and 'cookies' attributes of the data model classes should be marked as private.

The 'headers 'and 'cookies' attributes are only used internally by data models to make requests, therefore they should be prefixed with double underscores.

Kindly assign a contributor to each issue, so that we both don't end up working on the same thing.

I don't think that is true, if people want to use other places from aniwatch along with this module, instead of making their own headers and cookies they can just use the ones that were generated by it.

Yeah that is true, but I was thinking about leaving them as public for the main "Sakurajima" class. All the data model classes like "Episode" and "Anime" have them as public too. This might create confusion.

I will fix that, also I've invited you to be a collaborator, if you want you can accept the request in your email.

Yeah, recieved the invite. Thanks.

Done in https://github.com/veselysps/Sakurajima/commit/b16c8516df521de794987739a19c1ba5ff72a9ea


Update to Lean 4.10.0

Looks like this broke your build, sorry @abdoo8080; unfortunately the tests did not run because you didn't click the button to approve them!

Oh! No worries! I can fix the errors. I will do so tonight. Thanks for the PR!


Extreme lagginess / jerky movement in desktop Safari

Description
On my 15" retina mbp in Safari, when I view a model-viewer that takes up a lot of the screen, the movement is suddenly very laggy and the model jerks around.

It gets smoother the smaller the viewport is.
Repro:

Safari 12.1.1 (146<IP_ADDRESS>)
Visit https://googlewebcomponents.github.io/model-viewer/
Make the window as big as possible
Try rotating the astronaut

It is even more noticeable if you are in the scaled display:

I have noticed performance on iOS is not as good as on Android. My XS Max is somewhat choppy when I rotate models, but on a pixel 2 it's buttery smooth.
Browser Affected

[ ] Chrome
[ ] Edge
[ ] Firefox
[ ] Helios
[ ] IE
[x] Safari

OS

[ ] Android
[ ] iOS
[ ] Linux
[x] MacOS (10.14.5)
[ ] Windows

Versions

model-viewer: v0.3.1
three.js: r104

Tracking two separate issues here:

Safari is slower than other browsers
At some recent point in history, we introduced crazy jank to our controls

I ran a git-bisect and it appears the jank (2) was introduced in https://github.com/GoogleWebComponents/model-viewer/pull/563
FYI @elalish we may need to revert the configurable FOV change if we cannot land a fix early tomorrow.

I think the issue is due to incorrect compensation for long-running frames. In Chrome, frames always stay in budget, but in Safari I'm seeing the delta between frames flap rapidly between positive and negative across frames.


⚠ Samsung has degraded performance

In 741f525, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2723 ms

Resolved: Samsung performance has improved in 6cded2e.


T024 Kinase similarity: sequence

Description

Talktorial ID: T024
Title: Kinase similarity: sequence
Original authors: @t-kimber
Reviewer(s): @dominiquesydow, @AndreaVolkamer
Date of review:

https://app.reviewnb.com/volkamerlab/teachopencadd/pull/117
Todos
Content

[ ] Theory

[ ] Kinase dataset
[ ] Kinase similarity descriptor: XXX

[ ] Practical

[ ] Retrieve and preprocess data
[ ] Show kinase coverage
[ ] Compare kinases
[ ] Visualize similarity as kinase matrix
[ ] Visualize similarity as phylogenetic tree

Content review

Potential labels or categories (e.g. machine learning, small molecules, online APIs): XXXXXXXX
One line summary: XXXXXXXXXX
[ ] The table of contents reflects the talktorial story-line; order of #, ##, ### headers is correct
[ ] URLs are linked with meaningful words, instead of pasting the URL directly or linking words like here.
[ ] I have spell-checked the notebook
[ ] Images have enough resolution to be rendered with quality, without being too heavy.
[ ] All figures have a description
[ ] Markdown cell content is still in-line with code cell output (whenever results are discussed)
[ ] I have checked that cell outputs are not incredibly long (this applies also to DataFrames)
[ ] Formatting looks correctly on the Sphinx render (bold, italics, figure placing)

Code review

Time it took to execute (approx.):
[ ] Variable and function names follow snake case rules (e.g. a_variable_name vs aVariableName)
[ ] Spacing follows PEP8 (run Black on the code cells if needed)
[ ] Code line are under 99 characters each (run black -l 99)
[ ] Comments are useful and well placed
[ ] There are no unpythonic idioms like for i in range(len(list)) (see slides)
[ ] All 3rd party dependencies are listed at the top of the notebook
[ ] I have marked all code cell with output referenced in markdown cells with the label # TODO: CI
[ ] I have identified potential candidates for a code refactor / useful functions
[ ] All import ... lines are at the top (practice part) cell, ordered by standard library / 3rd party packages / our own (teachopencadd.*)
[ ] I have update the relative paths to absolute paths.
List here unfamiliar libraries you find in the imports and their intended use:

Questions

[ ] TBA

Status

[ ] Ready to go

In case we do add sequence similarity: Substitution matrices are implemented in biotite

https://www.biotite-python.org/apidoc/biotite.sequence.align.SubstitutionMatrix.html#biotite.sequence.align.SubstitutionMatrix
https://www.biotite-python.org/examples/gallery/sequence/blosum_dendrogram.html


local minio s3 endpoint not used by delta extension

Hello there,
I created a secret for local minio, but delta extension doesn't use the endpoint, only region is used, can anyone help to troubleshoot? Or is this a bug?
D CREATE SECRET s1 (
      TYPE S3,
      USE_SSL 'false',
      URL_STYLE 'path',
      ENDPOINT '<IP_ADDRESS>:9000',
      REGION 'm1',
      KEY_ID 'minio',
      SECRET 'CHANGEME123');
┌─────────┐
│ Success │
│ boolean │
├─────────┤
│ true    │
└─────────┘
D SELECT * FROM delta_scan('s3://alluxio/logs/dt2') limit 1;
IO Error: Hit DeltaKernel FFI error (from: While trying to read from delta table: 's3://alluxio/logs/dt2/'): Hit error: 8 (ObjectStoreError) with message (Error interacting with object store: Generic S3 error: Error after 10 retries in 2.086870444s, max_retries:10, retry_timeout:180s, source:error sending request for url (https://s3.m1.amazonaws.com/alluxio/logs/dt2/_delta_log/_last_checkpoint))

From https://github.com/duckdb/duckdb_delta/commit/3e33b4967eac2cb4e6725eac52336fe0e319de59, minio is supported, maybe I not using the latest version?

could you run update extensions in duckdb and report which version of delta you are on?

I was about to raise an issue for the same problem, I already setup SECRET as documented: https://duckdb.org/docs/extensions/delta, I decided to explicitly put tokens to make sure it wasn't another issue.
CREATE SECRET delta_s4 (
    TYPE S3,
    KEY_ID '<key_id>',
    SECRET '<secret>',
    REGION 'us-east-1',
    SCOPE 's3://sample-delta-lake'
)

This is the status of secret after executing: FROM which_secret('s3://sample-delta-lake/delta_s3', 's3')
┌──────────┬────────────┬─────────┐
│   name   │ persistent │ storage │
│ varchar  │  varchar   │ varchar │
├──────────┼────────────┼─────────┤
│ delta_s4 │ TEMPORARY  │ memory  │
└──────────┴────────────┴─────────┘

Now, whenever I try to run SELECT * FROM delta_scan('s3://sample-delta-lake/delta_s3') I'm getting the same error as @mervynzhang
IOException: IO Error: Hit DeltaKernel FFI error (from: While trying to read from delta table: 's3://sample-delta-lake/delta_s3/'): Hit error: 8 (ObjectStoreError) with message (Error interacting with object store: Generic S3 error: Error after 10 retries in 59.883701368s, max_retries:10, retry_timeout:180s, source:error sending request for url (https://s3.us-east-1.amazonaws.com/sample-delta-lake/delta_s3/_delta_log/_last_checkpoint))
I did try running update extensions as you recommend and this is the output I'm getting for that command
┌────────────────┬────────────┬─────────────────────┬──────────────────┬─────────────────┐
│ extension_name │ repository │    update_result    │ previous_version │ current_version │
│    varchar     │  varchar   │       varchar       │     varchar      │     varchar     │
├────────────────┼────────────┼─────────────────────┼──────────────────┼─────────────────┤
│ aws            │ core       │ NO_UPDATE_AVAILABLE │ 42c78d3          │ 42c78d3         │
│ httpfs         │ core       │ NO_UPDATE_AVAILABLE │ 1f98600c2c       │ 1f98600c2c      │
│ delta          │ core       │ NO_UPDATE_AVAILABLE │ e526d8f          │ e526d8f         │
└────────────────┴────────────┴─────────────────────┴──────────────────┴─────────────────┘

However, I'm still getting the same error

could you run update extensions in duckdb and report which version of delta you are on?

v0.1.0 works now. Thanks

@yaguirre I'm going to assume this works for you now as well if you update to v0.1.0 by running update extensions again. If  not, please let me know!

Hi @samansmink I executed the update extensions again and now I'm using version v0.1.0 as shown below.
│ delta            │ true    │ true      │ … │ v0.1.0            │ REPOSITORY        │ core           │
However, I'm still getting the same error, not sure if I'm missing to configure any additional parameter

@yaguirre note that the error could arise also from invalid config. If you are sure your config should be correct, please open a new issue with a reproduction and I will take a look!

@samansmink  I tried running delta_scan with same configuration but this time within a GitHub Codespaces machine and it acually worked there, maybe it has something to do with my local system environment. I'm running a WSL 2 system with following characteristics.
Linux version <IP_ADDRESS>-microsoft-standard-WSL2 (root@941d701f84f1) (gcc (GCC) 11.2.0, GNU ld (GNU Binutils) 2.37) #1 SMP Fri Mar 29 23:14:13 UTC 2024
As you see that is a linux_amd64 compatible system, so as of my understanding from supported platforms shouldn't have any issues. The GitHub Codespaces machine where it worked has these characteristics.
Linux version 6.5.0-1022-azure (buildd@lcy02-amd64-015) (x86_64-linux-gnu-gcc-11 (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0, GNU ld (GNU Binutils for Ubuntu) 2.38) #23~22.04.1-Ubuntu SMP Thu May  9 17:59:24 UTC 2024
Maybe the first one has still some kind of issues, just guessing, but anyways it's working now! Thanks for your support!! 😄

@yaguirre Im not sure how I would go about reproducing that easily. Given that it now seems to work for the most part I will keep this closed. Feel free to reopen an issue if you keep having problems!


MAINTAINERS.md: checklist to update infra ships

As part of the release process, infrastructure ships should be updated, but that was not mentioned in the checklist.

We should update the TODO before merge, also exact update steps would be good (merge command, etc.)


Add :extensions option to Gateway

Array of extensions is passed onto the connection.

Example Usage:
ROM.setup(:sql, [‘postgres://localhost/db', extensions: [:pg_enum, :pg_array])
@solnic how would you suggest testing and documented this?

I would mock it, you can pass in connection object directly to setup via ROM.setup(:sql, [connection]) so you can pass a spy and expect it to have_received(:extension) with expected arguments. It is not our interface but the risk that this interface will change in Sequel is very small. This would be way less overhead than adding a pg enum column to test table and testing the whole thing in integration style.

@solnic specs added

Awesome, thanks :)

Could you update changelog too? Just push to master directly if you have time, you've got commit access now

Done :D


language filter needed in "oopsyraidsy" and "raidboss" trigger

Different servers have different dealing with mechanism, so it should make "opps" and "raidboss" trigger not fired in several conditions to avoiding confusing.
For example, in Hades(EX), players would not dodge"Bad Faith" in CN servers, but "oopsyraidsy" trigger will be fired and give warning of "failure to dealing with mechanism", it confused some CN players. Similar conditions also appear in E4S.
Therefore, we need a language filter in "oopsyraidsy" and "raidboss" trigger to make specific triggers not be fired in specific language.

What do people do differently in E4S? Stacking in Bad Faith is what folks do on my server as well.
My hope for oopsy is that it points out when you're hit by stuff that is avoidable.  If there's a mechanic that everybody just stands in, then maybe oopsy should just not call that out. (Or if we wanted to be more complicated, if ~4 or more people got hit, assume it's intentional and not mention it.)

It might be better to add a GUI Oopsy configuration for this? I realize that's always a lot of work, but lowering the barrier for players to disable this for themselves could be the best way to go, rather than trying to determine intentionality.
(Humorously speaking, we could exempt BLM from Oopsy altogether, unless the avoidable damage inflicts damage down or kills/wipes instantly.)

I think in this case, like triggers, I'd want an option to be off by default since it being on causes confusion like this bug.  In that case, it seems likely that nobody would ever go turn this on.  That makes me lean against having an option.
If determining intentionality seems a bit awkward (and I agree with that), maybe it'd be better to just ignore Bad Faith entirely.  Or, throw a warning if anybody got hit by more than two?
(Re: BLM.  Whenever a BLM gets hit, it will just say "healers adjust" in oopsy and not blame the BLM.  :P)

Oh, I meant more generally a configuration GUI of the same sort Raidboss has. It would get around the whole "should we special-case this?" situation. If players can easily turn off Bad Faith in much the same way they would turn off any Raidboss trigger, I think that would avoid the intentionality issue as well. We already name the Oopsy triggers uniquely to enable this sort of per-mechanic toggling, so I feel like having a wall of checkboxes inside dropdowns could kind of make sense?
BLM could have its own checkbox for whether or not they get included in mechanics. ^_^

Ah, I see.  A general configuration might be nice.  There's no reason we couldn't do that too.
I don't think this issue counts as "wanting to configure something off", because it will still cause confusion and people will think something is wrong vs it being something they don't want.  I think this is what most people do, and so in some senses it's not a mistake even though it's avoidable.

The linked patch above handles the Bad Faith issue.  If there are other mechanics that are being warned about that everybody commonly gets hit by, please let me know.
I'd like to avoid "localized strats" for oopsy unless it becomes absolutely necessary.


Initial readme info

What
Add a readme and a PR template, just getting things started.
How to review
Sanity check the readme please, make sure I'm being clear on purpose.
Who can review
Mikes team

PR looks good to me. Readme is easy to understand, whilst descriptions/instructions are informative and kept to a minimum. Format is good i.e. instructions are separated accordingly with appropriate subheadings , bash commands are easy to identify.


Refactoring PyHPS implementation

Going for a class based approach to make the code more reusable.
Also easier to understand, modify and extend.

Codecov Report
Attention: Patch coverage is 0.67873% with 439 lines in your changes are missing coverage. Please review.

Project coverage is 79.97%. Comparing base (33ac15a) to head (cb35511).
Report is 2 commits behind head on feat/pyhps-support.

:exclamation: Current head cb35511 differs from pull request most recent head c827121
Please upload reports for the commit c827121 to get more accurate results.

Additional details and impacted files
@@                  Coverage Diff                   @@
##           feat/pyhps-support    #3117      +/-   ##
======================================================
- Coverage               82.65%   79.97%   -2.69%
======================================================
  Files                      54       54
  Lines                    9766    10110     +344
======================================================
+ Hits                     8072     8085      +13
- Misses                   1694     2025     +331

@pyansys-ci-bot LGTM.


RBAC provisioning errors : 403 and 409 errors

I am trying to grant DeveloperRead access to users.  In my project setup I have
.
├── developer-read.tf
├── main.tf
├── sa-cloudclusteradmin.tf
└── variables.tf

sa-cloudclusteradmin file creates a new service account with "CloudClusterAdmin" privileges.
Error-1
When I run the tf apply command for the first time I get the below error when assigning DeveloperRead.  I have the confluent_cloud_api_key/secret defined in main.tf file.  The error below - is it because it needs an account with "CloudClusterAdmin" privileges?  If yes, how do I use them in the "confluent_role_binding" block?
error creating Role Binding: 403 Forbidden: Forbidden Access

However the same works when I use the confluent CLI "iam rbac role-binding create" command.
Error-2
Also, when I run the tf apply command again for the second time,  I get an additional error for the CloudClusterAdmin account.  Should it not skip creating the service account if it already exists?
error creating Service Account "app-manager-rbac-sa-non-prod": 409 Conflict: Service name is already in use.

I tried the below to assign the CloudClusterAdmin keys while provisioning..... but it did not work
resource "confluent_role_binding" "first-last-topic-rb" {
  principal   = "User:${var.user_first_last}"
  role_name   = "DeveloperRead"
  crn_pattern = "${var.kafka_cluster_rbac_crn}/kafka=${var.kafka_cluster_id}/group=test*"

  credentials {
    key    = confluent_api_key.app-manager-rbac-kafka-api-key.id
    secret = confluent_api_key.app-manager-rbac-kafka-api-key.secret
  }
}

---
Error: Unsupported block type

Thanks.

👋 thanks for opening an issue!
Created & replied to the 2nd issue (error) here: https://github.com/confluentinc/terraform-provider-confluent/issues/28
Regarding the first one:
Your Cloud API Key (var.confluent_cloud_api_key/var.confluent_cloud_api_secret) should be owned by a principal with Org/Env/CloudClusterAdmin roles that should be able to grant CloudClusterAdmin role.
I've got a quick question: is the end goal to create 2 role bindings: CloudClusterAdmin and DeveloperRead? I would expect that you run Terraform Provider with the Cloud API Key owned by OrgAdmin so 403 is a little bit surprising indeed.
There could be a typo in CRNs so I'd suggest to use data sources instead of variables:
data "confluent_kafka_cluster" "basic" {
  id = "lkc-abc123"
  environment {
    id = "env-xyz456"
  }
}

data "confluent_service_account" "example_using_name" {
  display_name = "test_sa"
}

resource "confluent_role_binding" "first-last-topic-rb" {
  principal   = "User:${data.confluent_service_account.example_using_name.id}"
  role_name   = "DeveloperRead"
  crn_pattern = "${data.confluent_kafka_cluster.rbac_crn}/kafka=${data.confluent_kafka_cluster.rbac_crn.id}/group=test*"
}

Let me know if that helps.

I tried the below to assign the CloudClusterAdmin keys while provisioning..... but it did not work
That's expected, see docs/resources/confluent_role_binding for a list of supported arguments.

Thank you for the reply.
The Cloud API key was created by me and I have the "OrganizationAdmin" role.   Also, the cloud key/secret I am using, it has worked in other tf project where I used it to provision a service account with "CloudClusterAdmin" role.
I used the data block as suggested above but still get the 403 error
data "confluent_kafka_cluster" "non-prod" {
  id = var.kafka_cluster_id
  environment {
    id = var.environment_id
  }
}

resource "confluent_role_binding" "aleksandra-sarac-topic-rb" {
  principal   = "User:${var.user_first_last}"
  role_name   = "DeveloperRead"
  crn_pattern = "${data.confluent_kafka_cluster.non-prod.rbac_crn}/kafka=${data.confluent_kafka_cluster.non-prod.id}/topic=test*""
}

-------------------
error creating Role Binding: 403 Forbidden: Forbidden Access

@bluedog13 could you share your OrgID with<EMAIL_ADDRESS>and our backend team will take a look at it?

Thank you. Have sent the email to the email shared above.
Below is the rbac_crn that was generated from the data block to be substituted in the confluent_role_binding block.
crn_pattern = "crn://confluent.cloud/organization=98a3cce4-255d-4a56-8449-6dcbd65*****/environment=env-*****/cloud-cluster=lkc-*****/kafka=lkc-*****/topic=test*"

@linouk23  -  Thank you for helping resolve the issue.
Below is the fix
I was using
principal   =<EMAIL_ADDRESS>
For the fix to work, the email had to be substituted with the user id for the email
principal   = "User:u-mv****"

The reason for using the email initially was, the confluent CLI does take email for principal.  This behavior is different in the terraform provisioning
confluent iam rbac role-binding create \
    --principal<EMAIL_ADDRESS>\
    --role DeveloperRead \
    --kafka-cluster-id lkc-***** \
    --environment env-***** \
    --resource Topic:test \
    --prefix


Initial functionality to put record to the stream and read from it

PR https://github.com/vivecltd/kinesium/pull/9


Error when delete session

when i delete session by
ctx.session = null

it throws
TypeError: Cannot convert undefined or null to object

and fix it
// if is an empty object
if(ctx.session !== null && typeof ctx.session === 'object' && !Object.keys(ctx.session).length) {
  ctx.session = null;
}

@xiedacon  I think instanceof is a better way to fix this bug. Also thanks for your PR! 😄

that's right, instanceof is much better


Cannot connect to Confluent Kafka with SASL

Affected module
Kafka ingestion
Describe the bug
When I try to connect to confluent Kafak with SASL username/password method, I get a timeout on the test connection.
To Reproduce
Add any test Conflunent kafka parms and try to authenticate with SASL username/password method.
Expected behavior
It to pass the test connection and expected to establish a connection

The issue was  SSL encryption is enabled in the kafka cluster , for Confluent kafka that is a common case.
So in that case you need to add security.protocol:SASL_SSL
Updated the documentation - https://github.com/open-metadata/OpenMetadata/pull/13024


Feature: SVG Support

I know SVG isn't easy to play with, but many notification servers have it, so I'm just asking.

Pretty innovative idea though, thank you very much for it.

+1
I've exported the entire FontAwesome library exported as png in order to have them work with Wired but not without it quirks.

Where do you want support for it?  If in image block, do some apps send notification hint images or app images as SVG?

Where do you want support for it?

Image blocks.

do some apps send notification hint images or app images as SVG?

Hard to tell, if there's a way to verify this, I'd help, but I can guess that no. Telegram, for example, caches the images as JPG, and then sends to the server with the hint images along the new message.
Not sure exactly the reasoning behind the question though, is it about recognizing the image being sent as SVG or (PNG, JPG, ...)? I thought many image-rendering libraries already had this 'built-in'.

Not sure exactly the reasoning behind the question though

I'm trying to understand the use-case.  If all notify clients send PNGs, why do we need SVG support?  Just now I'm remembering that users can manually send images, which is probably the use case.

I thought many image-rendering libraries already had this 'built-in'.

Unfortunately `image-rs does not (which is surprising because it's massive!!!).  I'd have to pull in another library, all of which seem non-trivial.  Will keep looking at options though.

I'm trying to understand the use-case. If all notify clients send PNGs, why do we need SVG support?

My bad, I should've made it clear before. I mostly use notification servers for Shell scripts, it's easier to automate certain processes, such as: recording with a keybind, and display a notification with a timer until it begins. I have a lot, really a lot, of those, and I think many people who uses Linux for ricing and making things easier for them send those notifications directly from scripts as well. The icons, in those cases, would help differentiating them.

Could I ask though if the --icon option shouldn't read it as an image-path as well?
# Instead of only this:
notify-call --hint string:image-path:"$icons/definition.svg" "Love" "Means ..."
# Read as well:
notify-call --icon "$icons/definition.svg" "Love" "Means ..."

We should be handling this case.  IIRC we handle the --icon param as an "App" image, and the other ones as "Hint" images, which is adjusted in the ImageBlock settings.
https://github.com/Toqozz/wired-notify/blob/52dd38c93fceca54c0ecf47ec5c63476577be873/src/bus/dbus.rs#L364
My guess is that it's just not loading the SVG, but should handle that icon param fine.

SVG support has been added in master.  Naturally it only works when loading images from a path (e.g. notify-send -i /home/user/img/ferris.svg "summary" "body").
Let me know if you run into any issues.

@lukelex Do you mean you would want to be able to specify that we just present the image as-is without any scaling? So e.g. set scale_width/scale_height to -1 for no scaling on that dimension?

Exactly. It's often the case that opposing images aren't the same size. (e.g. Volume & Mute)


🐛 分页组件没有国际化

🏷 Version

Package
Version

@antv/s2
^1.26.0

@antv/s2-react
^1.24.0

@antv/s2-vue

Sheet Type

[-] PivotSheet
[-] TableSheet
[ ] GridAnalysisSheet
[ ] StrategySheet

🖋 Description
分页组件 setLang后，下拉框没有国际化，另外，分页组件无法配置size, 看代码是写死了small, 在表格高度设置比较大的时候不是很协调，希望能开放配置

⌨ Code Snapshots
https://user-images.githubusercontent.com/46083022/185349523-33b3e0c7-fedf-4526-bdca-011d6b0691ff.mov

🔗 Reproduce Link

🤔 Steps to Reproduce

😊 Expected Behavior
😅 Current Behavior
💻 System information

:tada: This issue has been resolved in version @antv/s2-v1.28.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version @antv/s2-react-v1.25.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version @antv/s2-vue-v1.3.0-alpha.2 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version @antv/s2-v1.28.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version @antv/s2-react-v1.25.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version @antv/s2-vue-v1.2.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@lijinke666 更新后，设置pageSize， 下拉框无法切换，设置defaultPageSize, 无效
https://user-images.githubusercontent.com/46083022/187186624-e4e277d6-935b-46f5-9428-c78d2bbbeebf.mov


Overscroll Glow prevents interaction before scroll

Environment

Browser: All
Version of smooth-scrollbar: 8.1.7

Issue Summary
Any item inside a scroll container using the glow overflow effect will not be clickable until the user scrolls
Steps to Reproduce

use overscroll glow effect with a container that has a clickable item inside of it
click item (it will not react)
scroll
click item (it will react)

Online demo
https://codepan.net/gist/3158707c4c5d41126a665147176826a5

Thanks for reporting! This issue is fixed in v8.1.8.


Update macOS Image to 10.14 in CI as 10.13 is going to be deprecated in March

https://devblogs.microsoft.com/devops/removing-older-images-in-azure-pipelines-hosted-pools/
Note that Azure DevOps recently added 10.15 as well, so an alternative would be to use macOS-latest or macOS-10.15
https://docs.microsoft.com/en-us/azure/devops/pipelines/agents/hosted?view=azure-devops

Can one of the admins verify this patch?

@mitchdenny @JimSuplizio @danieljurek can you please follow up (review or close)?  Thank you

All pipeline yamls in master are using macOS-10.15... Closing.


Fix signedness bug

this fixes calculation for all years < 1200.

I'm not sure I have authorization to release this—I'm not even 100% sure I know how to build/test it, to be honest—but I'd feel a lot more comfortable about trying if you could add a test for it to your merge request.
Thx for your contribution!

@barefootcoder thanks for the suggestion,
while trying to write unit tests I found one more issue with the function:
Even with the current fix the function seconds_between_years is not symmertic in it's arguments, meaning that seconds_between_years(2000, 2401) != (-1)* seconds_between_years(2401, 2000).
With my first commit ("Fix signedness bug") the difference is only one day though.
The reason for this seems to be that the function in one case counts the leap years upward starting from 2000 and skips 2401, while in the other case it starts from 2401 and skips 2000. I think that the one that skips the larger year is the correct behavior, but I'm not 100% certain on that. I modified the function such that in both cases this is the behavior.
Since thinking about which of the years on the edge of the interval is quiet tricky I would request someone to verify my calculations for the leap years between 0 and 1901 or check some negative dates.
Best, Benjamin

Oh no, I think it's even more complicated:
Since the right year in the usage of this function is passed through safe_year() which ensures that both years satisfy
https://github.com/evalEmpire/y2038/blob/c17a49af2c8866f580ea05f44979e1266bc8b8c9/time64.c#L345-L347
, so it matches the left year in it's number of days. Leaving out the first instead of the last year in the iteration should not make a difference this way.
The function was probably designed in such a way that it is only able to give the correct result for years that satisfy this requirement, such that it does not matter if one skips the first or last year. Maybe one could rename the function from seconds_between_years to something like second_between_years_with_same_num_days or add a comment.
But that also means that the tests for this function like
https://github.com/evalEmpire/y2038/blob/c17a49af2c8866f580ea05f44979e1266bc8b8c9/t/seconds_between_years.t.c#L14
can not be written as straightfoward as one might think.
The dates with which the function can be tested must either both be leap years or both be normal years. Some of the current tests do not satisfy this requirement, but still work because the iteration direction was starting from the lower year 2000:
https://github.com/evalEmpire/y2038/blob/c17a49af2c8866f580ea05f44979e1266bc8b8c9/time64.c#L531-L532
I think this will take some time.
Best, Benjamin

Tests should be fixed now.


Update Pages “TermitesInTreesAndPage”

Automatically generated by Netlify CMS

👷 Deploy Preview for chic-shortbread-66a66b processing.

Name
Link

🔨 Latest commit
6692f138a47d4343fa0765ffe784a91e7dd06b55

🔍 Latest deploy log
https://app.netlify.com/sites/chic-shortbread-66a66b/deploys/655d8b97d5c2770008e6228b


"The newer VHDX format is not supported in Azure. When you create a VM, specify VHD as the format. If needed, you can convert VHDX disks to VHD using qemu-img convert"

i don't see in "qemu-img --help"(version 2.0.0 and 2.7.0) to convert to VHD format

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

ID: 6034e25f-4b16-66ce-eb44-305000dbd625
Version Independent ID: e4a5dcbc-a3a5-4439-bd69-de4a6111d355
Content: Upload or copy a custom Linux VM with Azure CLI
Content Source: articles/virtual-machines/linux/upload-vhd.md
Service: virtual-machines-linux
GitHub Login: @cynthn
Microsoft Alias: cynthn

Thanks for the feedback! We are currently investigating and will update you shortly.

@15ramky I also do not see that option in qemu to convert from vhdx to vhd as per this doc. I will remove that reference from the doc.
CC: @cynthn

@15ramky Any update? Did the above help?


(root) Ingress story for OSM

This is the top level GitHub issue for a) designing b) implementing of a complete Ingress story for Open Service Mesh, which includes:

[ ] HTTP traffic to backend Pods
[ ] HTTPS (mTLS and TLS)

Related Issues:

[x] #3501
[x] #2365
[x] #3700
[ ] #1697
[x] #1020
[ ] #981

Reopening this to continue to track OSM's Ingress Story.

Related issues:

[ ] doc: Ingress #2871
[ ] Document how to install ngnix ingress controller #2827
[ ] doc: Integrate w/ AGIC #2868
[ ] Ingress with ngnix results in 502 bad gateway #2930

tested and demo'ed. Thank you!


Changing default configuration to Normal cast.

As far as I know, there is no way to currently set the default to Normal cast. This means that every time I open the Dota client, my very first game I have to hit the F7 button. Related to this, the F7 button does not work initially. I hit it and then the prompt says "All spells and abilities set to Quick cast", even though it was quick cast to begin with. Then, I have to hit F7 again to actually make it normal cast.
Tl;dr: I have to hit F7 twice in my very first game in order to change from quick to normal cast. Also, the prompt is prompted to "say_team" not the coach.

Yes, this is an initialization bug and it will probably be resolved in the future.
As for switching up the default mode at launch, that can be fixed with code changes of the toggle functions.

Try adding "quick_cast_toggled_off" - no quotes, at the end of your autoexec.cfg file, and let me know if that works


Update improved_epitech_dump.sh

kde plasma ?

the script doesn't download any DE anymore, the user can choose what edition of Fedora to download (like Fedora KDE) before executing the script


Double check ingest 5536fc09-031d-4cad-bf40-88e8e08d3095

https://wellcome-ingest-inspector.glitch.me/ingests/5536fc09-031d-4cad-bf40-88e8e08d3095
It got rebagged; I resent the whole ingest and so two different bags got unpacked. We should double-check the bag that gets replicated to Azure matches whatever is in the S3 replicas.

Fixed.


Add specific highlight group for virtual text

This changes the highlight group used in the ale_virtualtext function. It allows for the styling of virtual text to be different than the default ALEError or ALEWarning allowing vitualtext to stand out better.
Before:

After:

You should use the new highlight group names like you did in your first commit, and set them to default to the other highlight groups. Have a look at the signs or highlight files for examples. Document the new highlights in the section for highlighting.


Ways to indicate "In Progress" within imgpkg

In the beginning,

there was an intent to improve the experience of the user waiting for images to copy (#132)
that idea spawned a spinner (#133)
... which didn't seem entirely "right." because of that, it was pulled off into a separate PR (#134)
which sat for a while, grew stale and was closed.

This discussion is about circling back to that original intent and seeing what we learned from the above-described attempt so that we could realize the intent in a better way.
That discussion started elsewhere like this:
@cppforlife said

we briefly started talking about spinners and what exactly it helps with (and doesnt). discussion wasnt continued im guessing due to other priorities so i've closed it for now. we can start a discussion again if ppl have time.
... from what i recall we never got to an agreement regrading this UX paradigm in carvel (spinners arent anywhere in carvel) ... blockage here is on UX.

@DennisDenuto noted and asked:

Ok... so the original problem (users don't know what is happening in imgpkg because they see a blank screen for a moment) is deemed as a valid problem to solve... but it was the decision to use a spinner to solve that problem that is the issue.
What is the UX paradigm in carvel? Was the option to introduce 'spinners' into this paradigm discussed? or tackled another way, what is the carvel way to show that the program is 'loading' / how do we prompt the user to just chill for a moment?

... aaaaaaannnd... ACTION! 🎬

Nice. Well TIL that there is a notion of a UX paradigm in carvel. I'm gonna take it for granted that this is something we want to strive towards.
That paradigm isn't known to me. I'm hoping to define what that actually is. With that shared definition it should be hopefully easier to see whether a 'spinner' fits well or doesn't, and what might be a better alternative.
Here's a stab in the dark: There is an ambition to have the carvel tools have a more 'uniform' way of displaying information to users. As well as having a 'uniform' way of running / configuring the tools. Although for this convo I think sticking to displaying information makes sense.
Different information that our tools display are (not limited to):

output
debugging
-- internal state
-- progress
informational
errors

ytt:
- output: YAML
- debugging
-- internal state: shows a graph-like structure of nodes
-- progress: n/a because ytt is bound by CPU, memory so output typically happens quickly. Validated with large code bases.
- informational: detection of potentially incorrect usage of command. i.e. flags such as experimental schema
- errors: Structured assertion-like error, "expected vs actual" pointing the user to the problematic yaml file.

imgpkg:
- output: Succeeded/Failed with image/bundle shas
- debugging
-- internal state: line item description
-- progress: currently we do nothing
- informational: detection of potentially incorrect usage of command. i.e. flags such as --non-distributable
- errors: A contextual error with the underlying error revealed too.

I haven't added the other tools like kapp or kbld yet, because i'm already seeing differences in the UX. Which I think reflect the difference in structures of information between the two tools. That difference in structure will lead to a different tailored UX.

Not sure if we should be comparing the output of a tool like ytt and imgpkg. These 2 tools serve a different purpose and the displayed information on the screen hardly would be comparable.
As a example ytt is meant to print to standard output the result of computation that can afterwards be piped into a different tool, like kbld or kapp. While imgpkg output is meant for the Humans to read directly, and not consumable but a different tool. imgpkg is a particular case inside the carvel suite because it cannot be part of a pipeline of commands and need to always be used in a standalone manner, so if a particular UX element would make the UX of our users better we should not shy away from it just because it is not in other tools of the suite.
This being said I believe that argument wise we should try to be as consistent as we can. An example would be all tools should have some sort of debugging capabilities and they all should use the same command-line argument.

I feel like this discussion has grown stale. Do we still believe it makes sense to keep this issue around?
@pivotaljohn @DennisDenuto @cppforlife

As discussed today I am going to close this issue. In the future, we can reopen it if we think it is something that we want to improve the UX of imgpkg.


Make which to take previous finder into account

Oh nice! Thanks:)

I believe this fixes #1150
Some tests are failing
Add a few new tests to make sure the changes work

LGTM.
I wonder though if existing tests in customer apps' can break because of this change (because they depended on the buggy which() implementation).

I think not, if they were using it as it was expected.


Move getReadStream up to DataResource in Data Package (and other API improvements)

The immediate suggestion is about using getReadStream (https://github.com/frictionlessdata/tableschema-js/blob/master/src/table.js#L218) to make a stream or rawStream method on Data Resource.
Aside: I don't think getReadStream needs to return a promise - it can be synchronous I think!
However this is just lead-in to the big message ...
Context
There's some important context here. It comes from thinking a lot about the relationship of the various FD libraries and esp the relationship of something like tabulator (nice stream), tableschema (infer, schema, Table) and Data Package / Resource libs.
Long story short: I think with a few tweaks we can get something really nice as an interface and we can centralize it in one lib (at the moment, whilst the libs are nicely factored you do have to look into too many places -- e.g. this issue ends up being here but relating to 3 other libs).
First I must emphasize a key point:
People don't care about Data Packages / Resources they care about opening a data file and doing something with it

Data Packages / Resources come up because they are a nicely agreed metadata structure for all the stuff that comes up in the background when you do that.

Put crudely: Most people are doing stuff with a file (or dataset), and they want to grab it and read it preferably in a structured way e.g. as a row iterator -- sometimes inferring or specifying stuff a long the way e.g. encoding, formatting, field types.
=> Our job is to help users to open that file (or dataset) and stream it as quickly as possible.
Interface
Here's a stab at at the minimal viable interface focused on the file only case:
const resource = new DataResource(path)

// crude rows - no type guessing unless i told it to do it
resource.rowStream()

// i might also want ...
resource.rawStream()

Here's a more detailed API definition:
// path can be local path or a url or whatever ...
const resource = new DataResource(path)

// this will spit out anything it has inferred and look a lot like a Data Resource object
// crucially it will have path, pathType (local, url etc), format, mediaType (the latter guessed), encoding
// I should be able to set any of these when I construct it if i want ...
resource.descriptor

// ========
// Data access
// now the real stuff i want i.e. getting the contents of the file or info about it

// returns a JS stream object or a file-like object in python
// maybe call this rawStream() if we want to be really clear
resource.stream()

// if this resource can be parsed to rows of objects (like tabular stuff can - or even geojson!)
// this is class JS object stream
// for python the Stream stuff from tabulator-py)
resource.rowStream()

// row stream (or maybe bytestream)
// metadata as first row (in bytestream would be first line)
// cf way that datapackage pipelines stream entire data package over single stdin/stdout pipe
//  this is a bit optional ... (could also implement via a flag to rowStream() or stream)
resource.singleStream()

// ------
// extras

// infer types
resource.infer()

// infer structure (e.g. csv separator etc)
resource.inferStructure()

// if this makes sense
resource.headers

// something like this ...
resource.isTabular

Comments
The above was done for JS but could be common to python and any other language too (tabulator-py Stream would need to have a stream() method rather than open() or similar but that's minor).
At the moment we have all the ingredients for this but we don't provide this in one place often e.g.

https://github.com/frictionlessdata/datapackage-js#resource - Resource is missing the stream methods and we have to call Table and then iter to get objects (and why don't we support Node Stream API for the object streams?)
Python: you have to work your way across tabulator-py, tableschema-py and datapackage-py to put the underlying concepts and approach together.
General point: I'm wondering if we almost want to leave the Data Package stuff to one side and present this front and center - this data interface is what people really want and the Data Package and Data Resource are just the descriptor structures that you happen to need. (This is more a presentation aspect but its important e.g. you'd start examples not loading a datapackage.json but loading a file path)

Asides

More and more think the profile stuff should be a separate library outside of core. 90% of the time I don't want and I don't want to know about it and I don't want to validate before I start -- I just want to load the data! If I want it i can import/require it separately and use it separately to test whatever i have once i've loaded it.

As an example of how this gets in the way: two of the first four items in the first example for datapackage-py deal with validation and the fairly obscure issue of schemas not in the local cache https://github.com/frictionlessdata/datapackage-py

@rufuspollock (cc @pwalsh)
What you've described it's almost exactly what tableschema.Table class does (in reference implementation for Python). For JavaScript for now it missed schema inferring, headers etc. So it's kinda half-implemented.
I see and do understand the problem with one entry point for the user. I also see the friction here even a few things you've described are aside problems like:

e.g. datapackage-py just doesn't have documentation at all so hard to judge it for now
any tabulator knowledge to use tableschema/datapackage is not required. Again it's only docs problem.

But we CAN'T do here:

redesign libs for 2 languages. It's just not possible. At first it looks simple (e.g. your API example). Then you goes to streaming, opening/closing file descriptors, having deal with sql-etc export for Python, different args like descriptor/path etc. At first place I've started to design TabularResource! And have failed to make it good enough for all use cases we have (and there are much more than you've described e.g. Table is a core of goodtables)

What we COULD do:

first

move Table class to datapackage and it will be almost what you've described (with simple improvements exactly what you want). With good docs on data package level etc. Consider Table almost as a TabularResource it's just decoupled from being a subclass of Resource (and it's done for a reason as said above)
OR we even could consider merging tableschema-datapackage (it's really simple). I already have solid amount of duplicated code to work with descriptors/profiles between libs. And now without clear separation between Table Schema/Data Package on specs level (now there is Data Resource and other) it could make sense. It will allow as to have first-class docs in one place.

second

after v1 release we could consider adding additional API to Resource class like rowStream/iter_rows() if tabular etc if still needed. Or even merging Resource/Table if we will be sure it's good enough for all use cases.

@roll i've got a long way through a PoC in JS for all of this here:
https://github.com/datahq/datahub-cli/blob/master/lib/utils/data.js
Let's discuss in the next few days.

FIXED/WONTFIX

Current datapackage-js@v1 now uses a lot from this feedback and from initial data.js. So it's fully cover initial Here's a more detailed API definition functionality and data flow:
// path can be local path or a url or whatever ...
const resource = await Resource.load({path: 'data.csv'})

// Descriptor following Data Resource spec
// Minimal before inferring
resource.descriptor

// returns a JS stream object or a file-like object in python
resource.iter({stream: true})

// if this resource can be parsed to rows of objects (like tabular stuff can - or even geojson!)
if (resource.tabular) {
  resource.table.iter() // array of rows
  resource.table.iter({keyed: true}) // array of keyed rows
  resource.table.iter({stream: true}) // Node Stream of rows (all row options could be used like keyed)
  // etc
}

// infer types
// infer structure (e.g. csv separator etc) - not implemented yet
resource.infer()

// now we have schema but could disable casting
if (resource.tabular) {
  resource.iter({cast: false}) // array of rows without casting
}

// if this makes sense
resource.table.headers

// something like this ...
resource.tabular // true

More detailed API and tutorial - https://github.com/frictionlessdata/datapackage-js#resource

Link to an alternative implementation data.js:

https://github.com/datahq/data.js#datajs


Unusably slow diffs

It'd be fine if just the commit was slow, but using wsligit makes the diffs just too slow to be usable. Is there some way to limit wslgit to just the commits?

Have you tried setting the Windows environment variable WSLGIT_USE_INTERACTIVE_SHELL to false, as mentioned in the Readme?
This should significantly speed up wslgit, but it assumes that you do not rely on anything in .bashrc to be run before git (like e.g. ssh-agent or similar tools).
Other than that, I don't think you can configure different git versions for different commands in VS Code. In the latest release, wslgit only does additional work on the output of a few selected commands (diff not included). In most cases, input and output is directly passed to/from git. So the overhead is likely from bash, which can be circumvented using above flag.

Thanks, setting that variable worked, although I'm not sure why it wouldn't be the default behavior.

Great, thanks for reporting back.
The current behavior is mainly for historic and compatibility reasons. The first versions used interactive bash sessions because wsl.exe was not around back then and it is common to start tools like ssh-agent or gpg-agent for key management in .bashrc, which, by default, is only fully executed for interactive shells.
But I may make the improved behavior the default in the future.


Messages J and h

Is there a reason why message types J and h are excluded from self.keep_messages_types in line 16 of file itch50_message_parser.py? It seems that these messages do not get parsed but they seem to be handled in line 165 of the same file.
Thanks!

That could be an oversight on my part; these message types were added at a later time IIRC. It possible I forgot to add them to the list of messages to parse. I will look into it.

After verification, I forgot to add them to that list when they were added to the specification. They are getting parsed (you can't read an ITCH file without parsing every message), but they are not kept in memory afterward.
I will add time to the next update (no time estimate for this!), but in the meantime, you can add them manually in your code by redefining keep_messages_types:
parser = ITCH50MessageParser()
parser.keep_messages_types = b'SAFECXDUBHRYPQINLVWKJh'

with open(sample_dir + 'BX_ITCH_20190530_ALGN.txt', 'rb') as itch_file:
    parser.parse_file(itch_file)