fix schema from docs_src/sql_app_py310

BaseModel does not support fields of type "<class 'types.UnionType'>"

Looking at the change, I would suspect that you were trying an example for Python 3.10 in a lower version of Python (like Python 3.9).
Although I'm not sure with the description of the PR. 🤔
It would be better to start with a question, filling all the template, that would help me/us understand your use case better:  https://github.com/tiangolo/fastapi/discussions/new?category=questions
Given that, I'll close this one, but thanks for the interest! ☕


Enable SQUASHFS support in the default VM kernel

Squashfs is useful for read-only disk images e.g. in case of large auxiliary trusted setup files.

Are all trusted setups provided as filesystem images? Isn't it a very special case? What's wrong with providing it as archive?

Are all trusted setups provided as SquashFS filesystem images? Isn't it a very special case? What's wrong with providing it as archive?

The thing with SquashFS is that you can build the disk image by single command invocation (mksquashfs <dir> <disk-image-file>). This is easy to document, easy to use and doesn't require sudo for example. I'd also imagine that one can do this easily on macOS (at least there seems to be tools in homebrew)?
So with this, one can bundle any set of read-only files into VM as a separate input file. If files are submitted as an archive, I think it may be slightly less efficient (since you need to uncompress & write them through two(?) filesystems - the VM VirtFS and then the host one) whereas I think the SquashFS should be relatively well optimized for just mapping the file entries and uncompressing them directly into page cache.
TL;DR; This isn't only for trusted setups, but it will probably be the most common use case. I think SquashFS is simpler & faster compared to archive.

If it's required only for this setup only, should we add this to builder?

What do you mean by "add this to builder"?

What do you mean by "add this to builder"?

I meant adding SquashFS to default kernel configuration in our VM builder.
Okay, let's enable it by default, I guess it won't hurt.
The implementation looks a bit ugly, I agree, but I suppose there is no easy way to simplify kernel configuration? If so, let's merge it.

let's enable it by default, I guess it won't hurt.

Yeah, this should be totally fine :slightly_smiling_face:

The implementation looks a bit ugly, I agree, but I suppose there is no easy way to simplify kernel configuration? If so, let's merge it.

Not in general. I noticed that the build system starts asking questions if one only enables some of the flags, so in practice all of them needs to be set. Hence my thinking towards some kind of structure with feature name and then set of configs, but I think this is a good candidate for later refactoring, especially if we need to add other configs as well. Not a huge issue right now.


Fix MapToHTMLTableConverter

Hello,
This branch should fix #64. The issue was that the converter checked if key was null for both the key and value. This commit fixes the issue for me, but if there are any problems please let me know.
Thanks,
Alison

@awalter17 and I just discussed a (IMHO) slightly better way of addressing this bug. Stay tuned; push coming imminently.

Ok, I've made the changes @ctrueden and I discussed and I've added some examples to Test.ipynb. So this should be good to merge. But if you notice any issues please let me know.


Add market parameter to tracks() call

Get several tracks endpoint supports market parameter which allows getting linked tracks. https://developer.spotify.com/web-api/get-several-tracks/

thanks for the PR!


Highlight agility shortcuts in Meiyerditch Laboratories

Type
Incorrect behavior
Operating System
Windows
OS Version
Windows 10
Bug description
The level 74 and 93 agility shortcuts in Meiyerditch Laboratories, leading to the true blood altar, are currently not being highlighted by the agility plugin.
Screenshots or videos

RuneLite version
RuneLite version: 1.8.30
Launcher version: 2.4.2
Logs
No response

Added in https://github.com/runelite/runelite/commit/f72a8ab50bf56676f4dfc410b09629433d2b3bf4


Resolve colab compatibility issues in AppendixIII

Issue
As noted in AppendixIII/OpenAI_GPT_2.ipynb ,  we cannot run notebook in Colab for they does not support Tensforflow 1.x anymore.
Solution
To handle this, I changed source repository from OpenAI to nshepperd, and imported tensorflow.compat.v1 for compatibility.
Change Notes

in step 2, changed source repo from !git clone https://github.com/openai/gpt-2.git to  !git clone https://github.com/nshepperd/gpt-2
in step 4, deleted line %tensorflow_version 1.x
in step 7a, imported tensorflow.compat.v1 for compatibility

I did not changed the notes in preface. So please edit if changes are needed.

Good work! Thanks. I'll take a look and see how to promote your work the best way possible. I might just add a link to your work in a readme doc in the directory of the programs to give your work the attention it deserves. 😊

Issue
As noted in AppendixIII/OpenAI_GPT_2.ipynb , we cannot run notebook in Colab for they does not support Tensforflow 1.x anymore.
Solution
To handle this, I changed source repository from OpenAI to nshepperd, and imported tensorflow.compat.v1 for compatibility.
Change Notes

in step 2, changed source repo from !git clone https://github.com/openai/gpt-2.git to  !git clone https://github.com/nshepperd/gpt-2
in step 4, deleted line %tensorflow_version 1.x
in step 7a, imported tensorflow.compat.v1 for compatibility

I did not changed the notes in preface. So please edit if changes are needed.

Thanks to Hnky, the issue of the notebook in AppendexIII has been resolved.
A great thanks to Hnky!


add strip-ansi for karma output on tests

karma tests are failing (probably due to versions change).
The output now has green/red colors and it's better to ignore that for the sake of the tests.

Codecov Report

Merging #187 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #187   +/-   ##
=======================================
  Coverage   86.58%   86.58%
=======================================
  Files          13       13
  Lines         164      164
  Branches       21       21
=======================================
  Hits          142      142
  Misses         19       19
  Partials        3        3

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 04c70dc...483e359. Read the comment docs.


Please change the type of logContext from int to NSUInteger

I found that the DDLog takes a logContext parameter to create a corresponding DDLogMessage which could be used for filtering logs (black list or white list), though the default DDLog macros only pass a value of 0 to DDLog. I think it's a powerful feature to filter log messages at runtime.
Could you please change the type of logContext from int to NSUInteger? I want to use an object address as the logContext. But the size of address (uintptr_t) is 8 bytes on 64-bit OS and 4 bytes on 32-bit OS while the size of int is always 4 bytes on both architectures.
I think it's a good idea to use object addresses to indicate the log context. Doesn't it?

Fixed by #419


feat: Getting since and until tuple from calendar grains

SUMMARY
The first stage of making time picker. return the since and until datetime according to the granularity selected by the calendar
Miro link: https://miro.com/app/board/o9J_kiQxGIU=/
Issue link: https://github.com/apache-superset/superset-roadmap/issues/34
TEST PLAN
adding unit test for this function
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #11418 into master will decrease coverage by 10.61%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master   #11418       +/-   ##
===========================================
- Coverage   66.58%   55.96%   -10.62%
===========================================
  Files         860      408      -452
  Lines       40874    13654    -27220
  Branches     3694     3477      -217
===========================================
- Hits        27215     7642    -19573
+ Misses      13562     5838     -7724
- Partials       97      174       +77

Flag
Coverage Δ

#cypress
55.96% <ø> (-0.68%)
:arrow_down:

#javascript
?

#python
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...uperset-frontend/src/dashboard/util/dnd-reorder.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...rset-frontend/src/dashboard/util/getEmptyLayout.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...et-frontend/src/components/Menu/LanguagePicker.tsx
0.00% <0.00%> (-100.00%)
:arrow_down:

...dashboard/components/resizable/ResizableHandle.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/dashboard/util/getFilterScopeFromNodesTree.js
0.00% <0.00%> (-93.48%)
:arrow_down:

...uperset-frontend/src/utils/getClientErrorObject.ts
0.00% <0.00%> (-89.19%)
:arrow_down:

.../src/dashboard/components/FilterIndicatorGroup.jsx
11.76% <0.00%> (-88.24%)
:arrow_down:

...c/explore/components/controls/withVerification.jsx
9.09% <0.00%> (-87.88%)
:arrow_down:

...src/dashboard/components/gridComponents/Header.jsx
10.52% <0.00%> (-86.85%)
:arrow_down:

...rset-frontend/src/profile/components/Favorites.tsx
0.00% <0.00%> (-86.67%)
:arrow_down:

... and 680 more

Continue to review full report at Codecov.

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

Is this changing the behavior of the current function? We have to be mindful of the thousands of charts out there that have been built based on the current behavior.

Tested it today, this feature enhancement is so AWESOME!! MANY MANY THANKS @zhaoyongjie
this is by far the best community individual collaboration!
We will update the description in a couple of days, and tune the design a bit more!

@mihir174 it's not as easy to pull the draft to review, please chime in and give feedback based on the screenshot, we will need an error message design for Time Range Error here

thank you!!

Hey @zhaoyongjie, thanks for this! Looks great. Here's some design input
Flow for the advanced time range (reflects button states and error messages) -
https://www.figma.com/file/JWaGztdhZS0kS5ruG7x9tB/Control-Panel?node-id=491%3A46097
Button states-

validate is only active when an unvalidated change has been made
apply is only active when changes have been validated and the time range is valid

Error Message -
Invalid Time Range: Start date cannot be after end date
I've attached the error icon as an svg zip
The buttons should also use the Superset styles in storybook

error_solid.svg.zip

For the buttons, you'll want to import Button from 'src/components/Button';.
Those buttons have a buttonStyle prop for those three style variants in the design, e.g. <Button buttonStyle="link"
From left to right, the button styles are:
• Validate: tertiary
• Cancel: secondary
• Apply: primary (and the disabled prop)
For all three, you can see they have a particular min-width instead of just using the text width. For that, just add the cta prop, i.e. <Button cta ...
To see (and play with) all the button variants, you can run npm run storybook from the superset-frontend directory:

Hi @mihir174, Thank you for your reply.
Where do I find the icon for "Edit Time Range" title.

Is there a reason why this does not include "Previous Calendar Quarter"?

sure we can add "previous calendar quarter". Cartel designer did mentioned companies define their quarter differently.
for the rest of the styling issues pointed @zhaoyongjie please straightly follow the design spec to make the changes.
https://www.figma.com/file/JWaGztdhZS0kS5ruG7x9tB/Control-Panel?node-id=195%3A36068

Here are the follow-up PRs I plan to make to address the remaining comments. After this PR is merged, if there are any unforeseeable related issues, I will make myself available to fix them promyly to ensure the master branch's stability

frontend break down to child components
modal => popover(potentially)
improve typescript typing
frontend unit test
remove the validate button

Cypress seems to be stuck. I've seen this on other PRs recently, too. Not sure how to solve this, but maybe file another PR in a new branch?

Cypress seems to be stuck. I've seen this on other PRs recently, too. Not sure how to solve this, but maybe file another PR in a new branch?

if it's ok with you @ktmud 🙏
absolutely filing another PR

Rebasing might help. This could be due to they way the new pull_request_target works with branches that were created before that change.

many thanks to @kristw @ktmud @graceguo-supercat for reviewing this PR! 🙏🙏

@zhaoyongjie Could you also confirm whether this new control work well with existing charts (i.e. any charts with existing custom time range filters)? I hope we have some test cases for them, but it's not obvious by just looking at the PR.

hi @ktmud I'll test it manually right now, on every chart

@ktmud,

I have tested all charts manually, all can use the new time picker without problems.
tested all generated by load_examples -t  command charts, no problems too.
original custom time range can mapping to "time range string" split by separator, It will be parsed by the new time range control.


VLAZ: Watch thumbnailchecker.com

VLAZ requests the watch of the watch_keyword thumbnailchecker\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

thumbnailchecker\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Regent: Assertion failed in rdir/plugin/src/regent/flow.t with metaprogramming

I've not managed to work out a way to do this without metaprogramming, but I have a file that causes this to fail at line 244 (  assert(flow.is_valid_node(from_node) and flow.is_valid_node(to_node))). Full stack error trace:
...Legion/legion/language/src/rdir/plugin/src/regent/flow.t:244: assertion failed!
stack traceback:
        [C]: in function 'assert'
        ...Legion/legion/language/src/rdir/plugin/src/regent/flow.t:244: in function 'add_edge'
        ...gion/language/src/rdir/plugin/src/regent/flow_from_ast.t:341: in function 'add_input_edge'
        ...gion/language/src/rdir/plugin/src/regent/flow_from_ast.t:803: in function 'close'
        ...gion/language/src/rdir/plugin/src/regent/flow_from_ast.t:842: in function 'transition'
        ...gion/language/src/rdir/plugin/src/regent/flow_from_ast.t:919: in function 'open_region_tree_node'
        ...gion/language/src/rdir/plugin/src/regent/flow_from_ast.t:930: in function 'open_region_tree_top_initialize'
        ...gion/language/src/rdir/plugin/src/regent/flow_from_ast.t:969: in function 'open_region_tree_top'
        ...gion/language/src/rdir/plugin/src/regent/flow_from_ast.t:998: in function 'open_region_tree'
        ...gion/language/src/rdir/plugin/src/regent/flow_from_ast.t:3319: in function 'stat_for_list'
        ...
        ...gion/language/src/rdir/plugin/src/regent/flow_from_ast.t:3196: in function 'block'
        ...gion/language/src/rdir/plugin/src/regent/flow_from_ast.t:3310: in function 'stat_for_list'
        ...gion/language/src/rdir/plugin/src/regent/flow_from_ast.t:3445: in function 'fn'
        /home/aidan/Legion/legion/language/terra/src/terralist.lua:150: in function 'map'
        ...gion/language/src/rdir/plugin/src/regent/flow_from_ast.t:3196: in function 'block'
        ...gion/language/src/rdir/plugin/src/regent/flow_from_ast.t:3504: in function <...gion/language/src/rdir/plugin/src/regent/flow_from_ast.t:3491>
        ...e/aidan/Legion/legion/language/src/regent/passes_hooks.t:41: in function 'optimize'
        /home/aidan/Legion/legion/language/src/regent/passes.t:52: in function </home/aidan/Legion/legion/language/src/regent/passes.t:48>
        test2.rg:34: in function 'generate_symmetric_pairwise_task'
        test2.rg:54: in main chunk

I've attached a small regent program that causes the error. The error goes away if you require parts1 * parts2, however the strategy for calling this does not allow for this requirement.
Attaching the file will help, 1 sec doh.
rdir_bug.txt

The immediate workaround for this is to run with -fflow 0. The main cost is that you can't use SCR, but it avoids a number of pernicious bugs like this one that are unlikely to get fixed any time soon.

Ok, that works - is there a way to get Regent to do a "preprocessing" type thing (like -fpretty 1) that generates Regent code that can be read by the Regent parser? I'd like to see if this is happens with the same code without the metaprogramming or not, but would prefer to not have to rewrite the code entirely.

Oh, this can definitely happen without metaprogramming. The conditions that trigger it are having two regions of the same type that Regent cannot prove to be disjoint, in combination with accessing those regions later on in the body of the task. It's a long outstanding bug in RDIR that is a result of design decisions that are difficult to change at this point.
I've been thinking about making Regent assume that interfering region requirements are disjoint---which would be like the constraint you wrote, but would be checked dynamically instead of statically. That wouldn't fix the underlying issue (which is really that the algorithm to construct RDIR probably needs to fundamentally change) but it would fix most instances where we see symptoms like this.

SCR is gone, this is no longer relevant.


PVP Connection timeout

PVP loading times out, can not enter the game! The network status is normal, the following is the error code of the server, but I do not understand... I hope you can help me! Thank you very much!

You didn't upload the images. Reasons for failure:

YgoMaster sub process failed to start in time
Duplicate tokens
Didn't allow SessionServerPort through firewall

您没有上传图片。失败原因：

YgoMaster子进程未能及时启动
重复的令牌
不允许 SessionServerPort 通过防火墙

Hello, just now the picture may be network problems caused by the normal upload, now the picture has been successfully loaded

The sub process fails to connect
https://github.com/pixeltris/YgoMaster/blob/126132294b8f14d4c9086f65ac9597920ead9c71/YgoMasterServer/Pvp.cs#L819
It connects using these:
https://github.com/pixeltris/YgoMaster/blob/126132294b8f14d4c9086f65ac9597920ead9c71/YgoMaster/Data/Settings.json#L6-L7
And it connects to this:
https://github.com/pixeltris/YgoMaster/blob/126132294b8f14d4c9086f65ac9597920ead9c71/YgoMaster/Data/Settings.json#L4-L5

子进程连接失败
https://github.com/pixeltris/YgoMaster/blob/126132294b8f14d4c9086f65ac9597920ead9c71/YgoMasterServer/Pvp.cs#L819
它使用以下方法进行连接：
https://github.com/pixeltris/YgoMaster/blob/126132294b8f14d4c9086f65ac9597920ead9c71/YgoMaster/Data/Settings.json#L6-L7
它与此有关：
https://github.com/pixeltris/YgoMaster/blob/126132294b8f14d4c9086f65ac9597920ead9c71/YgoMaster/Data/Settings.json#L4-L5

Excuse me, how can I change it? thank you

You need edit you settings file so that MultiplayerPvpClientConnectIP can connect to SessionServerIP.

您需要编辑设置文件，以便 MultiplayerPvpClientConnectIP 可以连接到 SessionServerIP。

Should I change this "SessionServerIP" to the IPV4 server address of my WIFI?  I'm sorry!

You can't connect to <IP_ADDRESS>. You need to change MultiplayerPvpClientConnectIP to something like localhost. This is mentioned in the documentation https://github.com/pixeltris/YgoMaster/blob/master/Docs/PvP.md#setting-it-up-on-wan

You can't connect to <IP_ADDRESS>. You need to change MultiplayerPvpClientConnectIP to something like localhost. This is mentioned in the documentation https://github.com/pixeltris/YgoMaster/blob/master/Docs/PvP.md#setting-it-up-on-wan

Thank you for your help! Now I want to know if I can change some Settings so that the entire deck order is not affected by shuffling, and I've turned on no shuffling, but now the deck is still scrambled by retrieval. Thank you for your help!

https://github.com/pixeltris/YgoMaster/blob/3b46942003c3c6d55c3fc0ffe1919053855a301b/YgoMaster/Data/Settings.json#L18

https://github.com/pixeltris/YgoMaster/blob/3b46942003c3c6d55c3fc0ffe1919053855a301b/YgoMaster/Data/Settings.json#L18

I've changed that. However, the order of the deck will still be shuffled by retrieval.

Did you reopen YgoMaster.exe? The cards should appear in the order that they appear in the deck file which may be different to the order they are displayed in the UI. You may need to manually edit the deck file.

I did restart it, but after the retrieval, the order of the cards still changed.

您是否重新打开了 YgoMaster.exe？卡片应按照它们在套牌文件中的显示顺序显示，这可能与它们在 UI 中的显示顺序不同。您可能需要手动编辑甲板文件。
I did restart it, but after the retrieval, the order of the cards still changed.

It works for me. The exact order of the cards are displayed can be different but the cards are correct.

它对我有用。卡片的显示顺序可能不同，但卡片是正确的。

The initial hand is fine, but then the order of the deck changes if the retrieval class cards are activated.

If the deck is shuffled by a card effect the deck will be shuffled. If the deck isn't shuffled by a card effect the draw order will stay the same. It isn't possible to disable shuffling by card effect.

如果牌组被卡牌效果洗牌，则牌组将被洗牌。如果牌组没有被卡牌效果洗牌，则抽牌顺序将保持不变。无法通过卡牌效果禁用洗牌。

Is there really no way, but thank you very much for your answer!

You can use a specific seed which will shuffle the same way each time you start a duel but any random card effect / action will change the seed.
https://github.com/pixeltris/YgoMaster/blob/3b46942003c3c6d55c3fc0ffe1919053855a301b/YgoMaster/Data/Settings.json#L19
To disable all shuffles you would need to find the shuffle function in duel.dll which would require reverse engineering the dll.

您可以使用特定的种子，每次开始决斗时都会以相同的方式洗牌，但任何随机的卡牌效果/动作都会改变种子。
https://github.com/pixeltris/YgoMaster/blob/3b46942003c3c6d55c3fc0ffe1919053855a301b/YgoMaster/Data/Settings.json#L19
要禁用所有随机播放，您需要找到需要对 dll 进行逆向工程的随机播放功能。duel.dll

This is too difficult for a non-professional like me to understand, but thank you very much for your patient reply and wish you a happy life!

您可以使用特定的种子，每次开始决斗时都会以相同的方式洗牌，但任何随机的卡牌效果/动作都会改变种子。
https://github.com/pixeltris/YgoMaster/blob/3b46942003c3c6d55c3fc0ffe1919053855a301b/YgoMaster/Data/Settings.json#L19
要禁用所有随机播放，您需要找到需要对 dll 进行逆向工程的随机播放功能。duel.dll

你好，我想知道是否可以定义抛硬币的正面和背面，谢谢!


[bug]: How do you update InvokeAI?

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
8GB
What happened?
I've tried updating by git pull and it did not work, is there a good documentation or tutorial on how to update it?
C:\Users\ninja\InvokeAI>git pull fatal: not a git repository (or any of the parent directories): .git
I'm not seeing any solution on the documentation.
Screenshots
No response
Additional context
No response
Contact Details
No response

Hi @ninjasaid2k
git pull will sync your local InvokeAI if and only if you initially installed it by running git clone and then making a venv e.g. via conda. Given the error message, I think you have installed via install script. In this case, you can update it by re-running the install script.
However, we will have v2.2 releasing in the coming days (it is in a RC stage now). Suggest holding off for that to get the newest goodies.


fix: add Abhishekfm

resolves #1452
Checklist for profile addition

[x] Name the PR fix: add <your-username>.
[x] You've added file named your-username.json in the profiles directory.
[x] You've linked the PR to the correct issue.

Other additions/changes

[x] Changes made in this PR are mentioned below
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.
[x] I've read the contributing guidelines

This PR adds/fixes

Notes

Please add .json at the end of your filename. This is needed so that the workflow can detect your profile.
Let us know if you need any help! 🙂

I have added it and Thank you for helping

Looks good to merge. Thank you for your contribution! 🎉

Thank you so much
This was my first contriution


React hooks not generated

Describe the bug
With codegen configuration that I expect to generate React hooks, there are no hooks in the generated output. I'm using hooksImportFrom: "@apollo/react-hooks", but the issue happens with or without that configuration set.
To Reproduce
I've reproduced the issue in this codesandbox:
https://codesandbox.io/embed/graphql-codegen-hooks-issue-2nswv.
Look at the generated types.tsx file and verify there are no hook functions.

Hi @jcelliott !
Your config syntax is not correct, the - creates array instead of object with keys. It should be:
schema: schema.graphql
documents: document.graphql
generates:
  types.tsx:
    plugins:
      - typescript
      - typescript-operations
      - "typescript-react-apollo"
    config:
      withHooks: true
      withHOC: false
      withComponent: false
      hooksImportFrom: "@apollo/react-hooks"


Feature/apply sequenceT

#421

@raveclassic nice, thanks. Could you please squash the commits while merging?


fix(transformer): turn on react preset by default

#2969
#2968  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Boshen and the rest of your teammates on  Graphite


Is there an .nccrc.json file or something?

I see in the documentation that when you call ncc programmatically you can pass in options like explicit externals, and there is some control from the command line too. Can we configure these options via a config file of some kind as well?

Yes run ncc help to see the CLI options.
https://github.com/zeit/ncc/blob/master/src/cli.js#L21

But can those options be put in a json file instead of on the cli?

No, ncc doesn't support json config files.
You can create your own script to support this:
const config = require('ncc-config.json');
const ncc = require('ncc');
const { code, map, assets } = await ncc('src/index.js', config);

@styfle Would it be useful to add these values to the README.md file? I wanted to see them at a quick glance and couldn't see them.

It is in the readme: https://github.com/zeit/ncc#programmatically-from-nodejs

Oh I meant:
Options:
  -o, --out [file]         Output directory for build (defaults to dist)
  -m, --minify             Minify output
  -C, --no-cache           Skip build cache population
  -s, --source-map         Generate source map
  --no-source-map-register Skip source-map-register source map support
  -e, --external [mod]     Skip bundling 'mod'. Can be used many times
  -q, --quiet              Disable build summaries / non-error outputs
  -w, --watch              Start a watched build
  --v8-cache               Emit a build using the v8 compile cache

Fixed in #478


Mixin props are not merged into component

Version
3.0.0-beta.14
Reproduction link
https://jsfiddle.net/y2bvfzdk/4/
Steps to reproduce

Execute reproduction

What is expected?
mixin prop should be rendered
What is actually happening?
nothing is rendered

I also observed this bug while attempting to port to Vue 3.

@pikax
It fixed in 3.0.0-beta.15 development build. But it doesn't work in 3.0.0-beta.15 production build.
https://jsfiddle.net/dxybujrm/


Serializer caching / model_name bugginess?

I have a resource, where I have defined the model_name because, it's a namespaced STI model, and the name doesn't quite line up with what I want the resource name to be.
module Api::Jsonapi
  class ProjectNoteResource < OpportunityResource
    model_name 'Opportunity::ProjectNote'
  end
end

When I attempt to use the API to create one of these models, I get the following exception:
Internal Server Error: undefined method `new' for nil:NilClass /gems/jsonapi-resources-0.9.0.beta1/lib/jsonapi/resource.rb:478:in `create_model'

When I looked into it, I see the exception is happening in this method:
module JSONAPI
  class Resource
    class << self
      def _model_class
        return nil if _abstract

        return @model if defined?(@model)
        return nil if self.name.to_s.blank? && _model_name.to_s.blank?
        @model = _model_name.to_s.safe_constantize
        warn "[MODEL NOT FOUND] Model could not be found for #{self.name}. If this a base Resource declare it as abstract." if @model.nil?
        @model
      end
    end
  end
end

Basically the line return @model if defined?(@model) ... it doesn't seem to be behaving correctly.  @model is nil ... but defined?(nil) actually returns "local-variable" -- so it's returning nil.  This is causing the exception.
The line was changed as a part of PR this commit:
https://github.com/cerebris/jsonapi-resources/commit/827f5db494ff8add581371db2fe6ac7f1c9abded#diff-e6cf31c573bd5f2277b1f16812943df4R986
Where it was changed from the original return @model if @model to return @model if defined?(@model) ...
@DavidMikeSimon was this intentional?  Apologies if I am off base?  For now, I have monkey patched that line back to the original,  and things are working again for me.
Thanks again for the gem!  It has taken some investment from me but I am loving it and I believe in the long run this approach is really going to pay off.

This may be related? https://github.com/cerebris/jsonapi-resources/issues/654

@kevinmtrowbridge I believe you are correct; the change to the line you referenced was a mistake on my part.
@dgeb Thank you for correcting this. :+1:


[Infra] [WIP] Add test report listener to delta-spark and bucket test suites by estimated runtime

Which Delta project/connector is this regarding?

[ ] Spark
[ ] Standalone
[ ] Flink
[ ] Kernel
[X] Other (Infra)

Description
Add test report listener to delta-spark to record test runtimes. Logs test metrics and writes metrics to a csv file and uploads it as a test artifact.
We can use the output of this job to learn how to better bucket our slowest + largest tests.
https://github.com/delta-io/delta/actions/runs/10949526852?pr=3694

How was this patch tested?
Locally and GitHub CI.
Does this PR introduce any user-facing changes?
No.
Results
Slowest Suites
We just use AAA, BBB, CCC to differentiate the shards
+---------+-----------------------------------------------------+------------------+
|shards|test_suite                                           |total_duration_min|
+---------+-----------------------------------------------------+------------------+
|AAA      |MergeIntoDVsWithPredicatePushdownCDCSuite            |36.09             |
|BBB      |MergeIntoDVsSuite                                    |33.26             |
|BBB      |MergeIntoDVsCDCSuite                                 |27.39             |
|BBB      |cdc.MergeCDCSuite                                    |26.24             |
|BBB      |MergeIntoDVsWithPredicatePushdownSuite               |23.58             |
|AAA      |MergeIntoSQLSuite                                    |23.01             |
|CCC      |MergeIntoScalaSuite                                  |16.67             |
|AAA      |deletionvectors.DeletionVectorsSuite                 |11.55             |
|BBB      |stats.DataSkippingDeltaV1ParquetCheckpointV2Suite    |8.26              |
|AAA      |DescribeDeltaHistorySuite                            |7.16              |
|BBB      |ImplicitMergeCastingSuite                            |7.14              |
|AAA      |stats.DataSkippingDeltaV1JsonCheckpointV2Suite       |7.0               |
|AAA      |UpdateSQLWithDeletionVectorsSuite                    |6.03              |
|CCC      |commands.backfill.RowTrackingBackfillConflictsDVSuite|5.97              |
|CCC      |DeltaSourceSuite                                     |5.86              |
|AAA      |cdc.UpdateCDCWithDeletionVectorsSuite                |5.67              |
|BBB      |stats.DataSkippingDeltaV1Suite                       |5.63              |
|AAA      |DeltaSourceLargeLogSuite                             |5.61              |
|AAA      |stats.DataSkippingDeltaV1NameColumnMappingSuite      |5.43              |
|BBB      |GenerateIdentityValuesSuite                          |5.4               |
|AAA      |commands.backfill.RowTrackingBackfillConflictsSuite  |5.02              |
|AAA      |ImplicitStreamingMergeCastingSuite                   |4.77              |
|CCC      |DeltaVacuumWithCoordinatedCommitsBatch100Suite       |4.73              |
|AAA      |CoordinatedCommitsBatchBackfill1DeltaLogSuite        |4.64              |
|CCC      |DeltaLogSuite                                        |4.6               |
|CCC      |IdentityColumnIngestionScalaSuite                    |4.36              |
|CCC      |DeltaVacuumSuite                                     |4.22              |
|AAA      |columnmapping.RemoveColumnMappingCDCSuite            |4.12              |
|AAA      |DeltaSuite                                           |4.05              |
|AAA      |UpdateSQLSuite                                       |3.99              |
|CCC      |typewidening.TypeWideningInsertSchemaEvolutionSuite  |3.92              |
|AAA      |cdc.DeleteCDCSuite                                   |3.9               |
|AAA      |CoordinatedCommitsBatchBackfill100DeltaLogSuite      |3.86              |
|CCC      |rowid.UpdateWithRowTrackingCDCSuite                  |3.83              |
|AAA      |expressions.HilbertIndexSuite                        |3.75              |
|CCC      |DeltaProtocolVersionSuite                            |3.71              |
|CCC      |CoordinatedCommitsBatchBackfill2DeltaLogSuite        |3.68              |
|BBB      |CheckpointsWithCoordinatedCommitsBatch100Suite       |3.59              |
|AAA      |ConvertToDeltaScalaSuite                             |3.59              |
|AAA      |typewidening.TypeWideningTableFeatureSuite           |3.49              |
|BBB      |cdc.UpdateCDCSuite                                   |3.42              |
|CCC      |CloneTableScalaDeletionVectorSuite                   |3.41              |
|AAA      |IdentityColumnSyncScalaSuite                         |3.33              |
|CCC      |DeleteSQLSuite                                       |3.31              |
|AAA      |CheckpointsWithCoordinatedCommitsBatch2Suite         |3.19              |
|CCC      |DeltaSourceIdColumnMappingSuite                      |3.18              |
|BBB      |rowid.RowTrackingMergeCDFDVSuite                     |3.18              |
|AAA      |rowid.UpdateWithRowTrackingTableFeatureCDCSuite      |3.12              |
|CCC      |UpdateSQLWithDeletionVectorsAndPredicatePushdownSuite|3.01              |
|BBB      |rowid.RowTrackingMergeDVSuite                        |2.97              |
+---------+-----------------------------------------------------+------------------+

Slowest Tests
+-----------------------------------------------------------------------+--------------------------------------------------+------------------+
|test_suite                                                             |test_name                                         |execution_time_min|
+-----------------------------------------------------------------------+--------------------------------------------------+------------------+
|deletionvectors.DeletionVectorsSuite                                   |huge table: delete a small number of rows from tab|6.38              |
|GenerateIdentityValuesSuite                                            |basic                                             |5.37              |
|deletionvectors.DeletionVectorsSuite                                   |huge table: read from tables of 2B rows with exist|4.12              |
|IdentityColumnSyncScalaSuite                                           |alter table sync identity on delta table          |2.99              |
|MergeIntoMaterializeSourceSuite                                        |materialize source for non-deterministic source fo|1.64              |
|expressions.HilbertIndexSuite                                          |continuous and 1 to 1 for all spaces (TestCase(9,6|1.35              |
|expressions.HilbertIndexSuite                                          |HilbertStates caches states (9)                   |1.23              |
|DeltaSourceSchemaEvolutionCDCIdColumnMappingSuite                      |schema log initialization with additive schema cha|0.92              |
|cdc.CDCWorkloadSuite                                                   |replication workload                              |0.88              |
|coordinatedcommits.InMemoryCommitCoordinator1Suite                     |should handle concurrent readers and writers      |0.77              |
|IdentityColumnIngestionScalaSuite                                      |merge insert                                      |0.75              |
|UpdateSQLWithDeletionVectorsSuite                                      |nested data support                               |0.74              |
|cdc.UpdateCDCWithDeletionVectorsSuite                                  |nested data support                               |0.74              |
|DeltaColumnDefaultsInsertSuite                                         |Column DEFAULT value support with Delta Lake, posi|0.73              |
|stats.DataSkippingDeltaConstructDataFiltersSuite                       |Test when the query contains EqualTo(Literal, Lite|0.7               |
|DeltaVacuumWithCoordinatedCommitsBatch100Suite                         |correctness test                                  |0.69              |
|DeltaVacuumWithCoordinatedCommitsBatch100Suite                         |vacuum after purging deletion vectors             |0.67              |
|DeltaErrorsSuite                                                       |Validate that links to docs in DeltaErrors are cor|0.65              |
|IdentityColumnIngestionScalaSuite                                      |insert into values                                |0.64              |
|DeltaSourceLargeLogSuite                                               |maxFilesPerTrigger: change and restart            |0.64              |
|coordinatedcommits.DynamoDBCommitCoordinatorClient5BackfillSuite       |should handle concurrent readers and writers      |0.63              |
|DeltaSourceSuite                                                       |maxFilesPerTrigger: change and restart            |0.63              |
|IdentityColumnIngestionScalaSuite                                      |append v1                                         |0.63              |
|DeltaCDCStreamIdColumnMappingSuite                                     |column mapping + streaming - allowed workflows - u|0.62              |
|IdentityColumnIngestionScalaSuite                                      |insert into select                                |0.62              |
|DeltaVacuumSuite                                                       |vacuum after purging deletion vectors             |0.61              |
|schema.CheckConstraintsSuite                                           |can't add unparseable constraint                  |0.6               |
|skipping.clustering.ClusteredTableDDLDataSourceV2NameColumnMappingSuite|validate RESTORE on clustered table - column mappi|0.59              |
|RestoreTableScalaSuite                                                 |path based table                                  |0.58              |
|IdentityColumnIngestionScalaSuite                                      |append v2                                         |0.56              |
|DeltaArbitraryColumnNameSuite                                          |create table                                      |0.56              |
|DeltaVacuumSuite                                                       |correctness test                                  |0.54              |
|CustomCatalogSuite                                                     |RESTORE a table from DummyCatalog                 |0.54              |
|DeltaSourceIdColumnMappingSuite                                        |column mapping + streaming: blocking workflow - re|0.51              |
|schema.InvariantEnforcementSuite                                       |reject non-nullable top level column              |0.51              |
|DeltaSourceIdColumnMappingSuite                                        |column mapping + streaming - allowed workflows - u|0.5               |
|DeltaSourceIdColumnMappingSuite                                        |column mapping + streaming: blocking workflow - dr|0.5               |
|DeltaCDCStreamNameColumnMappingSuite                                   |column mapping + streaming - allowed workflows - u|0.5               |
|DeltaCDCStreamIdColumnMappingSuite                                     |column mapping + streaming: blocking workflow - re|0.49              |
|DeltaCDCStreamNameColumnMappingSuite                                   |column mapping + streaming: blocking workflow - re|0.49              |
|DeltaSourceLargeLogSuite                                               |maxBytesPerTrigger: change and restart            |0.48              |
|DeltaSourceSuite                                                       |maxBytesPerTrigger: change and restart            |0.48              |
|DeltaGenerateSymlinkManifestSuite                                      |incremental manifest: partitioned table           |0.48              |
|DeltaCDCStreamIdColumnMappingSuite                                     |column mapping + streaming: blocking workflow - dr|0.47              |
|UpdateSQLSuite                                                         |nested data support                               |0.47              |
|DeltaCDCStreamNameColumnMappingSuite                                   |column mapping + streaming: blocking workflow - dr|0.47              |
|SnapshotManagementParallelListingSuite                                 |Backfills are properly reconciled with concurrentB|0.46              |
|DeltaSourceSchemaEvolutionCDCIdColumnMappingSuite                      |detect incompatible schema change while streaming |0.46              |
|RestoreTableScalaSuite                                                 |cdf + RESTORE                                     |0.46              |
|CustomCatalogSuite                                                     |Shallow Clone a table with time travel            |0.46              |
+-----------------------------------------------------------------------+--------------------------------------------------+------------------+

Another observation of mine: not sure if our hashing is even/fair enough.
The max difference between any two threads in a given shard is 68% (i.e. one thread has 68% more tests assigned to it than the other).
DSL: Scala 2.12.18, Shard 0
[info] Test group 0 contains 40 tests
[info] Test group 1 contains 34 tests
[info] Test group 2 contains 33 tests
[info] Test group 3 contains 36 tests
DSL: Scala 2.12.18, Shard 1
[info] Test group 0 contains 24 tests
[info] Test group 1 contains 37 tests
[info] Test group 2 contains 35 tests
[info] Test group 3 contains 28 tests
DSL: Scala 2.12.18, Shard 2
[info] Test group 0 contains 32 tests
[info] Test group 1 contains 19 tests
[info] Test group 2 contains 29 tests
[info] Test group 3 contains 31 tests

24 minute disparity between first and last


Q: How to get perfect snapshot of WKWebview content

I try to generate a perfect snapshot of the content of a WebView. The content is bigger than the screen and I get the size of the content handover by JS into the iOS app.
The following image shows a blue UIImageView with the snapshot. The green part is the body of HTML, fixed by setting width and height in JS the same width and height is handover into the app to set the WebView to the same size.

The challenge is that I get a whitespace on the right and the bottom inside the snapshot. The whitespace is from the HTML content (tested by setting background-color in css).
I tried different methods of SnapshotKit for the WebView including scrollView. Any idea what I miss? How I can prevent the whitespace and clip it to a given size by JS?

This is a known issue, and unfortunately, there is no good solution yet. Maybe you can try to detect the bottom white content and cut off it.


New BabyBeast Asset and Animations – Crystal Theme💎

Scope
We need to create a new BabyBeast asset with a Crystal Theme🧊 for OD Hack 10. The design must maintain consistency with the current app's BabyBeast style, specifically the Baby Rabbit, while introducing unique crystalline elements. The goal is to enhance the app's variety and visual appeal with this themed asset.
How BabyBeasts are currently designed?

BabyBeasts are designed as cute animals with a consistent style: rounded proportions, pastel tones, and clean, minimal details.
The Baby Rabbit model is the reference. It is stylized with simple shapes and subtle animations that make it feel alive in the app.

Requirements

Design the Crystal BabyBeast💎:

Use the Baby Rabbit as a base for style and proportions.
Add crystal-like features such as translucent or glowing body parts.
Ensure the design feels cohesive with the current BabyBeasts.

Create Animations:

Add animations that emphasize the crystalline elements:

Subtle shimmering effects.
Pulsating glows on crystal accents. 💠

Maintain Consistency:

Match the style of the current BabyBeasts, including similar proportions, color palettes, and aesthetic logic.

How to implement?

Download and use any pixel art toolkit.
Create and Animate the new Beast.
Save it to Assets/BabySprites in this repo, please give us the asset with their respective Sprites.
Let us know with pr and talking directly to me https://t.me/juandixcode or in the community group: https://t.me/+aqxlW9ACIt5jODEx

Cool Tools to Use

For creating and animating pixel art, we recommend using Aseprite. It’s a powerful tool for creating sprites, animations, and other pixel art elements or use another Toolkit of your choice.

How to apply?
Follow the contribution guidelines in the README file.
Let me know by tagging me or @jimenezz22 😇🫶🏻

Could I take a shot at this?
I will like to work on this design; can you please assign me when ODHack start?

Hi, I'm Josué, and I'm part of the Dojo Coding community. I have experience in programming languages such as Angular, React, C#, .NET, C++, PHP, Python, Java, Node.js, and JavaScript, among others. I'm passionate about software development and enjoy contributing to open-source projects, applying my knowledge to solve problems in a practical way. Additionally, I’m working on the project Citizen Of Arcanis, where I’ve had to use Unity, so I’m also skilled with this tool.
I will use the Baby Rabbit as a base and edit it in LibreSprite, adding the necessary features to adapt it to the crystal theme. I will incorporate details such as translucent or glowing body parts and shimmering crystal accents to create a cohesive crystalline design. Additionally, I will add subtle animations, like shimmering effects and pulsating glows on the crystal features, ensuring the asset remains consistent with the original BabyBeasts style.

Hello I'm a frontend developer in web3 I have contributed in previous OD Hacks I have made over 70+ contribution in past OD Hacks I intend in solving this issue with the following approach :

Understand the Base Model (Baby Rabbit)
UI will use the Baby Rabbit as a base and edit it in LibreSprite, adding the necessary features to adapt it to the crystal theme.  Maintain rounded shapes and minimalist details.

Design Crystal Features
Create translucent crystal elements on the body with a subtle gradient (soft blues, purples, or greens).
Add a glowing effect for areas like the crystal accents and edges.

Create Animations
Shimmering Effect: Use CSS keyframes to simulate light reflecting off the crystals.
Pulsating Glows: Animate glowing accents using box-shadow or filter: glow for pulsating effects.

Ensure Cohesion with Existing Models
Make sure the crystal elements feel like a natural extension of the current BabyBeasts, maintaining consistency in proportions, style, and animation flow.

Hello I'm a frontend developer in web3 I have contributed in previous OD Hacks I have made over 70+ contribution in past OD Hacks I intend in solving this issue with the following approach :
Understand the Base Model (Baby Rabbit): I would start by using the Baby Rabbit as a base, editing it in LibreSprite to introduce crystal features while maintaining its rounded, minimalist design.
Design Crystal Features: I’d create subtle, glowing crystal elements using soft gradients in colors like blues, purples, or greens.
Create Animations: I would implement shimmering effects through CSS keyframes and create pulsating glow animations using box-shadow or filter properties.
Ensure Cohesion: Finally, I would make sure the new design blends seamlessly with the existing BabyBeasts, preserving consistency in proportions, style, and animations.

I’d love to work on this task.

Can I attempt this issue?

Hello @jimenezz22 Can I handle this issue I'll complete it in 2-3 days

Would love to contribute to this issue, I'm a developer but also love illustration, so being able to create a character for ByteBeasts would be awesome.
To solve this issue I will:

Create the character design using pixel art and matching the ByteBeasts style.
Create the animations
Submit the PR including assets and sprites

Hey @jimenezz22, I will use the Baby Rabbit as a basis, keeping its style and dimensions, to design the Dragon BabyBeast. The baby will consist of dragon-inspired parts, such as small wings, scales, horns, and a dragon-like tail, but it will maintain the overall look of the current BabyBeasts. Tail flicks, wing flutters, and minor fiery effects will be added as animation. I will make and animate the asset in Aseprite, save into the appropriate repository folder, and submit for PR review.

@sandragcarrillo you have the issue, any bug, blocker, new feature or idea, let me know via telegram please


scripts/sanitycheck: Add --cmake-only option

Add an option that only invokes the cmake phase of sanitycheck.  This
can be useful for any testing that only needs to initial generation
phase of cmake, for example device tree.  Also useful if we want to
just generate compile_commands.json files from cmake via:
./sanitycheck -xCMAKE_EXPORT_COMPILE_COMMANDS=1 --cmake-only
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

that does not seem to work... Getting errors

that does not seem to work... Getting errors

what errors?

Selecting default platforms per test case
Building testcase defconfigs...
Could not build defconfig for em_starterkit_em7d_tests_crypto_mbedtls_crypto.mbedtls_config-sanitycheck
	see: /home/nashif/Work/zephyrproject/zephyr/sanity-out/em_starterkit_em7d/tests/crypto/mbedtls/crypto.mbedtls/config-sanitycheck.log
Could not build defconfig for em_starterkit_em7d_samples_subsys_console_echo_sample.subsys.console.echo_config-sanitycheck
	see: /home/nashif/Work/zephyrproject/zephyr/sanity-out/em_starterkit_em7d/samples/subsys/console/echo/sample.subsys.console.echo/config-sanitycheck.log

and in the log:
-- Configuring done
-- Generating done
-- Build files have been written to: /home/nashif/Work/zephyrproject/zephyr/sanity-out/em_starterkit_em7d/tests/crypto/mbedtls/crypto.mbedtls
/bin/sh: line 2: config-sanitycheck: command not found

Selecting default platforms per test case
Building testcase defconfigs...
Could not build defconfig for em_starterkit_em7d_tests_crypto_mbedtls_crypto.mbedtls_config-sanitycheck
	see: /home/nashif/Work/zephyrproject/zephyr/sanity-out/em_starterkit_em7d/tests/crypto/mbedtls/crypto.mbedtls/config-sanitycheck.log
Could not build defconfig for em_starterkit_em7d_samples_subsys_console_echo_sample.subsys.console.echo_config-sanitycheck
	see: /home/nashif/Work/zephyrproject/zephyr/sanity-out/em_starterkit_em7d/samples/subsys/console/echo/sample.subsys.console.echo/config-sanitycheck.log

and in the log:
-- Configuring done
-- Generating done
-- Build files have been written to: /home/nashif/Work/zephyrproject/zephyr/sanity-out/em_starterkit_em7d/tests/crypto/mbedtls/crypto.mbedtls
/bin/sh: line 2: config-sanitycheck: command not found

What did you invoke sanitycheck to show this failure?

What did you invoke sanitycheck to show this failure?

./sanitycheck -xCMAKE_EXPORT_COMPILE_COMMANDS=1 --cmake-only
The failures are in the generated build.log

./sanitycheck -xCMAKE_EXPORT_COMPILE_COMMANDS=1 --cmake-only
The failures are in the generated build.log

I wasn't able to reproduce your build error, ran into a few others that should now be fixed.

ok, better now, fails on unit tests:
i9:zephyr((HEAD detached from upstream/pull/16852)): sanitycheck -xCMAKE_EXPORT_COMPILE_COMMANDS=1 --cmake-only -T tests/unit/
JOBS: 40
Cleaning output directory /home/nashif/Work/zephyrproject/zephyr/sanity-out
Selecting default platforms per test case
Building testcase defconfigs...
Filtering test cases...
4 tests selected, 183 tests discarded due to filters
total complete:    2/   4  50%  failed:    0Traceback (most recent call last):
  File "/home/nashif/Work/zephyrproject/zephyr/scripts/sanitycheck", line 3523, in <module>
    main()
  File "/home/nashif/Work/zephyrproject/zephyr/scripts/sanitycheck", line 3452, in main
    ts.instances)
  File "/home/nashif/Work/zephyrproject/zephyr/scripts/sanitycheck", line 2420, in execute
    self.goals = mg.execute(cb, cb_context)
  File "/home/nashif/Work/zephyrproject/zephyr/scripts/sanitycheck", line 1423, in execute
    goal.handler.handle()
  File "/home/nashif/Work/zephyrproject/zephyr/scripts/sanitycheck", line 584, in handle
    with subprocess.Popen(command, stdout=subprocess.PIPE, stderr=subprocess.PIPE, cwd=self.outdir) as proc:
  File "/usr/lib64/python3.7/subprocess.py", line 775, in __init__
    restore_signals, start_new_session)
  File "/usr/lib64/python3.7/subprocess.py", line 1522, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: '/home/nashif/Work/zephyrproject/zephyr/sanity-out/unit_testing/tests/unit/lib/crc/misc.crc/testbinary': '/home/nashif/Work/zephyrproject/zephyr/sanity-out/unit_testing/tests/unit/lib/crc/misc.crc/testbinary'

now fails on unit tests

Hmm, you sure you have the latest version?  You should never get to line 1423 as the code has:
                    if state == "finished":
                        if goal.handler and not options.cmake_only:
                            if hasattr(goal.handler, "handle"):
                                goal.handler.handle()
                                goal.handler_log = goal.handler.log


Document release process

Currently only me and @marcoacierno know that we can automatically release strawberry on PyPI. We should document this, and maybe make a bot that suggests to add a Release file :)

We could use this for the bot: https://probot.github.io/

Somebody seemed to already got inspired: https://github.com/autopub/autopub

I'm just gathering the bits (for what it's worth):

Comment: https://octokit.github.io/rest.js/#octokit-routes-pulls-create-comment
Delete a file: https://octokit.github.io/rest.js/#octokit-routes-repos-delete-file
Get contents: https://octokit.github.io/rest.js/#octokit-routes-repos-get-contents
Creat or Update: https://octokit.github.io/rest.js/#octokit-routes-repos-create-or-update-file
Delete file: https://octokit.github.io/rest.js/#octokit-routes-repos-delete-file

Note: I haven't found a trivial way to delete and update in a single commit through the GithubAPI, but for the sake of code simplicity, this might be acceptable.

@blaggacao I think at some point we should move to autopub (and also migrate everything off of circle-ci).

Note: I haven't found a trivial way to delete and update in a single commit through the GithubAPI, but for the sake of code simplicity, this might be acceptable.

what do you mean with this? :)
PS. I think maybe we could close this issue for now, as we have a bot that comments and tells contributors to add a RELEASE.md file, that might suffice for now :)

what do you mean with this? :)

I noticed that both, CHANGELOG modification & RELEASE deletion would produce a commit through github API, autopub / your solutins.
A bot would still be nice (eg. hosted on now.sh) just for the sake of easy adoption... But that's probably not something for your todo list, I'd agree with that. :)

I’m going to close this as it seems the bot is being quite helpful :D


build(nexus): bump nexus version automation

Fixes

Fixes WB-NNNNN
Fixes #NNNN

Description
What does the PR do?

🤖 Generated by Copilot at 53b21d1
Added a new nox session bump-version to automate version updates for the wandb package or core. The session is still a work in progress and needs to be implemented.
Testing
How was this PR tested?
Checklist

[ ] Include reference to internal ticket "Fixes WB-NNNNN" and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Ensure PR title compliance with the conventional commits standards

🤖 Generated by Copilot at 53b21d1

bump-version waits
to automate package updates
a winter task, perhaps

implemented elsewhere


How to open Hyper in the working directory

[x ] I am on the latest Hyper.app version
[x ] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: Windows 10
Hyper.app version: 1.4.8

Issue
How to open hyper in the working directory?

There is a plugin for this but it has never worked for me 🙃
Take a look at awesome-hyper there may have been something added since I last checked 👍

https://github.com/hharnisc/hypercwd


[backport] Support comparison operators for complex numbers

Backport of #1636

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 82148c6885725ee6b508bf38c6ae76cfa25a4825, target branch v5) succeeded!

LGTM!


Inject HOC decorator only injects properties from class stores without any methods.

From how I've used mobx-react in the past, it should inject class methods (actions) aswell.
But for some reason it only seems to inject class properties (observables)
CodeSandbox Example

@inject(({ test }) => test)
Inject is supposed to return a set of props. Inject uses for in loop to merge the returned props with original set of props. However @action and @computed props are not enumerable, therefore ignored by for in loop.
Also note that this would be undefined during await this.props.fetch() call, because the action is not bound to store instance.

If you want to inject an class instance, just inject the entire object instance, rather than trying to destruct it into the props of the injected component. What you are doing here is very confusing, because props would not be a plain object anymore, but a kind of half-arsed class instance (with all the members, but not the prototype or the correct this binding). this.props will never be a TestStore instance, but you are treating like it is one, by calling fetch on it, and thereby probably breaking all the assumptions fetch might have from the fact that it was defined in the TestStore class. For example by tring to make super calls.
In other words, just leave test as a whole thing and write this.props.test.fetch().

i suppose is there a work around?

@michaelspeed Please open the new issue if you have more details about what you are solving. This issue has been closed a long time ago.


[Feature]: Forward click from SplitterResizeHandle

Describe the feature
Hello 👋,
I would love to handle a reset panel size when the user dbclick on the SplitterResizeHandle but it's not possible because the click and dbclick event is not forwarded.
Additional information

[ ] I intend to submit a PR for this feature.
[ ] I have already implemented and/or tested this feature.

Hi @Barbapapazes ! The current does forward all the needed event, including double click. I think you might be capturing the wrong event, should dblclick instead of dbclick
https://stackblitz.com/edit/c63utr9r?file=src%2FApp.vue
https://developer.mozilla.org/en-US/docs/Web/API/Element/dblclick_event

Thanks! 🙌
It works but:

The ResizeHandle must have a width
The user must click on the ResizeHandle

https://github.com/user-attachments/assets/671315b9-3df8-4b47-9e59-fe0199543d98
In this recording, when I click on the ResizeHandle, the event is logged but if I'm just before (or after) it does not works but I can drag-n-drop panels. You can reproduce in your stackblitz.


Add tsconfig and @typescript-eslint/no-floating-promises lint rule

As discussed in Slack, forgetting to await a Promise is a frequent pain point. This adds a TypeScript-based lint rule that picks this up and gives you errors like this:
/Users/mattb/Desktop/usdr/arpa-reporter/src/server/lib/audit-report.js
  590:5  error  Promises must be awaited, end with a call to .catch, end with a call to .then with a rejection handler or be explicitly marked as ignored with the `void` operator  @typescript-eslint/no-floating-promises

/Users/mattb/Desktop/usdr/arpa-reporter/src/server/lib/subrecipients.js
  44:9  error  Promises must be awaited, end with a call to .catch, end with a call to .then with a rejection handler or be explicitly marked as ignored with the `void` operator  @typescript-eslint/no-floating-promises

/Users/mattb/Desktop/usdr/arpa-reporter/src/server/routes/application_settings.js
  9:3  error  Promises must be awaited, end with a call to .catch, end with a call to .then with a rejection handler or be explicitly marked as ignored with the `void` operator  @typescript-eslint/no-floating-promises

/Users/mattb/Desktop/usdr/arpa-reporter/src/server/services/validate-upload.js
  118:5  error  Promises must be awaited, end with a call to .catch, end with a call to .then with a rejection handler or be explicitly marked as ignored with the `void` operator  @typescript-eslint/no-floating-promises
  122:5  error  Promises must be awaited, end with a call to .catch, end with a call to .then with a rejection handler or be explicitly marked as ignored with the `void` operator  @typescript-eslint/no-floating-promises

✖ 5 problems (5 errors, 0 warnings)

These existing errors are also fixed in this PR.
Though we are not using TS yet, TS supports some basic typechecking of vanilla JS files based on type inference, JSDoc comments, and "implicit any" (there are some important limitations). Right now running tsc produces 23 errors that are mostly due to the lack of explicit type definitions, but I don't plan to spend time right now fixing these or actually converting things to TS; I just added TS so that we could use the TS-based lint rule. Also, I've currently scoped this rule and the tsconfig file only to the src/server folder to keep things simple, but if we move to TS in the future, we'll want to expand that to also typecheck tests, Vue components/other clientside code, etc.
Note that there is also a non-TypeScript lint rule, no-floating-promise that tries to enforce this in a much weaker way (i.e. only in a single file and with less type awareness) -- this TS version is superior since it evaluates type information across multiple files.

Nice, you got it working!
@igor47 I'd be curious if you can get this change to play nicely with your IDE, and if you find it helpful.


Lint tests helper files

Depends on #9985

@andy-ms  why is this change depended on #9985 ? they seem to be independent.

@yuit: You're right, this didn't really depend on that.

:+1: I'm always for linting more things!


Error: error getting topics from app - unexpected end of JSON input - error occurred while beginning pubsub messages

Hi,
I am performing the grayscale tutorial example i.e. make run-api-rs the following error is occurring.
ERRO[0000] error getting topics from app: unexpected end of JSON input  app_id=image-api-rs instance=wasm-oci-demo scope=dapr.runtime type=log ver=1.4.2
ERRO[0000] error occurred while beginning pubsub messages: error getting topics from app: unexpected end of JSON input  app_id=image-api-rs instance=wasm-oci-demo scope=dapr.runtime type=log ver=1.4.2
INFO[0000] dapr initialized. Status: Running. Init Elapsed 9.416037000000001ms  app_id=image-api-rs instance=wasm-oci-demo scope=dapr.runtime type=log ver=1.4.2

The following video (which starts just as the error occurs) demonstrates the above error in action.
https://youtu.be/A5EhO7cNTCw?t=392
It seems that dapr list is displaying the correct output (in terms of apps, ports etc.)
wasmedge@wasm-oci-demo:~$ dapr list
  APP ID        HTTP PORT  GRPC PORT  APP PORT  COMMAND               AGE  CREATED              PID
  go-web-port   3500       40659      8080      ./web-port            36s  2021-10-05 02:38.25  334410
  image-api-rs  3502       34493      9004      ./target/debug/im...  23s  2021-10-05 02:38.38  334530
  image-api-go  3501       40639      9003      ./image-api-go        9s   2021-10-05 02:38.52  334658

Is it possible that the grayscale needs to be added to the config i.e. https://github.com/second-state/dapr-wasm/blob/main/config/conf.json
Any suggestions would be greatly appreciated.
Thanks
Tim

Thanks I will give this a try and let you know how it goes.
Thanks again.
Tim


City view gold yield display gets clipped with sufficiently high yield numbers

Is that an issue in the vanilla UI as well?

I haven't actually played a full game through without your mod so I don't know.

I can confirm that it is an issue with the vanilla UI as well. I don't have a game where it is quite as bad, though. Just the decimal of the gold running off.

I think the best solution is probably going to be stretching out that top bar on the citypanel a bit

Is this the kind of thing you were looking for chaorace?

This is how it looks when you have a lower yield city.


Adding French

Is this the same as #8?

Yes, with a fix for Zéro.
Rémy
Le mer. 10 févr. 2016 à 17:20, Mark Seemann<EMAIL_ADDRESS>a
écrit :

Is this the same as #8 https://github.com/ploeh/Numsense/pull/8?
—
Reply to this email directly or view it on GitHub
https://github.com/ploeh/Numsense/pull/31#issuecomment-182456031.

Would it make sense to reopen #8 and close this pull request, then?
Continuing work on #8 preserves the history of that pull request. Otherwise, this one essentially becomes a reboot.

Hello @ploeh
I was seriously considering doing the french version when I saw this PR.
What is the status on it?

@N0ct3, the overall status for the Numsense project in general is that it's no longer active. It seems that no-one uses the library, and both contributors and reviewers have long since ceased activity.
I can definitely still go through the motions of merging a pull request and publishing a new version to NuGet, but as I don't speak French, I wouldn't be able to review any contribution like that.
Unless you're keen on doing the work in order to learn some F#, you're probably better off directing your energy towards something with a more promising future.

Thanks for the answer. I will follow you advice and focus on something else ;)


Passenger access possibilities

As a passenger I want to be able to get access to my own user interface so that later I can send requests or view info about the trip.
Acceptance criteria
62.1 As a passenger I want to be able to press "I am a passenger" button so I get access to my interface.

This user story is edited upon the feedback from the customer (Truong) that passengers shouldn't have to sign in for more friendly application.

Regarding this user story, it is connected to the #66 regarding the selection of the type of user.
Acceptance criteria mention the need of pressing a button to be redirected toward a passenger interface / driver interface for both issues.
A prototype of this selection page is done for the #66 in the corresponding branch (Driver_Access_Possibilities).
Therefore, no need to make this page from scratch =)

The user story doesn't mention anything about making the page from scratch, it could be done after the driver access possibilities :)

The user story doesn't mention anything about making the page from scratch, it could be done after the driver access possibilities :)

I know, but it does mention being able to click on a button, just like #66. But there is not user selection page in the master, so it needed to be done in order to fulfil the acceptance criteria. I just inform the person that will take this user story that work has partly been done since #66 is related, so we don't do double work :soil.

(misscliked and closed the issue by accidente haha)

I will add a dependency to #66 to avoid double work then =)


Implement ASPEN in and out.

Implement in and out nodes that process the ASPEN RTP encapsulation of MPEG transport streams.
ASPEN is the initial focus. It should be possible to process other MPEG-TS encapsulations of media, such as SMPTE 2022-2, using the same codebase. These additional formats will be tracked as separate issues.

[ ] Unpack transport streams to PES stream payloads.
[ ] Pack grain payloads as PES payloads into transport streams,
[ ] Uncompressed mono-essence video streams. SMPTE RDD 37.
[ ] Uncompressed audio streams - twit-biddling required. SMPTE ST 302.
[ ] Ancillary data streams. SMPTE ST 2038.

Created separate module tesladon to provide generic, stream-based processing of MPEG-TS for node.js.


Streaming tokens does not work

The documentation on how to do streaming is here https://js.langchain.com/docs/modules/agents/how_to/streaming
If I follow this, streaming still does not work. I've created an example repo https://github.com/rednebmas/langchain-streaming-bug

@dosubot seems unlikely the model is not actually streaming as requested, the tell me a story chunk is very long.

Thank you for the reproducible example - I think this is a regression. Will dive in and fix 👍

Actually, it was a regression :( caused by #4083
Fix will be live soon.

Upgrading @langchain/core to 0.1.21 should fix this!

I was able to reproduce the original reported bug.
Thanks for the quick fix, Jacob!


[commissioner]  update how Provisioning URL is stored

This PR contains a collection of smaller commit changes/improvements in Commissioner.
[commissioner] update how Provisioning URL is stored
This commit updates how the Commissioner stores the Provisioning URL
(now saved as null terminated string). It also updates the related
APIs (including the public otCommissionerGetProvisioningUrl) and its
use in NcpBase.
[commissioner] iterate over joiners using pointer
[commissioner] make simple getter methods inline
[commissioner] log commissioner state changes

In the new pushed commits, updated msg logs to include the PR number.


Plot#remove_dataset docs fixed

Docs of Plot#remove_dataset were wrong, and I fixed them.
Also, I moved the line alias_method :replot, :plot to where #plot is defined.

Thank you!

:+1:


Stub for SSKeychain doesn't work

Stubbing a class method of SSKeychain doesn't seem to work, here is my code:
id keychainMock = OCMClassMock([SSKeychain class]);
OCMStub([keychainMock passwordForService:[OCMArg any] account:[OCMArg any]]).andReturn(@"Fixture password");

The mocked keychain simply won't return the value that I specified. What could possible cause this?
Note: SSKeychain was installed via CocoaPods. I also saw a similar problem discussed in #67, but the advice of removing libPods.a from "Link Binary With Libraries" didn't work for me :(

Nevermind, removing libPods.a actually worked after doing a clean. Sorry for creating a useless issue.


Add THCDeviceTensor

Name sounds interesting.  What does this do?

it's a subset of fbcuda. It allows one to do (among many other things) tensor indexing inside CUDA kernels without having to calculate ugly offsets based on size/stride.

Sorry about the delay Dominik. I think this is ready to be merged now.

Thanks!
Do you have plans to merge THCDeviceTensor and THCudaTensor? It feels almost like we have two implementations of the same thing... Maybe with a move to templated tensors in TH and more types for cutorch tensors, we can consolidate the two implementations?

They serve different purposes in my eyes: THCudaTensor is meant to implement the Torch C tensor API.
THCDeviceTensor is meant for use in writing CUDA kernels, and can only be used for cases where the number of dimensions is statically known (the type includes the dimension). THCudaTensor does not contain any static information about the size of tensors.

A templated tensor implementation that doesn't have static knowledge of the # of dimensions would make sense to support different datatypes, but Torch long ago decided against that and instead is re-implementing C++ templates via C macros and/or massive code duplication in my opinion :(

Good point re static dimensions.
There's hope that Torch will eventually switch to C++ templates: https://github.com/torch/torch7/blob/master/ROADMAP.md


Reorganize docs

Summary
PR #1224 attempted to leave bread crumbs in the documentation to get
people used to the new package organization and split between Qiskit and
downstream packages (applications, experiments, etc). However, in
practice this has been causing more confusion for users than helping.
This primarily stemming from the use of the word "Experiments" to
describe ignis instead of pointing to qiskit-experiments. This reverts
that PR and starts to starts to reoganize things to be a bit clearer.
Details and comments
TODO:

[x] update top navbar to list experiments too

Should we have the order of the listing of the application modules in the readme and in the documentation dropdown Applications menu consistent with the order in the index.rst. Otherwise, currently I think the orderings are all different


Use ISO 8601 date format for villagers birthday

To help avoid confusion with date formats, I recommend either switching the birthday field to using the ISO 8601 date format YYYY-MM-DD, or creating a new field like birthday-iso8601
While I don't believe birthday year is part of the data, perhaps it would a fun side project to determine birth year based on the release date of the game the villager first appears in. In the absence of a birthday year, you can use XXXX
For example, Annalisa's birthday would be XXXX-02-06 to help recognize that it's February 6th, not June 2nd (without needing to parse the birthday-string field).
Looking at https://animalcrossing.fandom.com/wiki/Annalisa, it appears Annalisa was a new character in New Leaf, which has an original release date of November 8th, 2012 (in Japan), which would make her birth year 2012, and her full birthday 2012-02-06.

Specified the date format in the doc (day/month). Bringing an ISO for the sake of bringing an ISO would just bring more confusion than anything else.
These data are extracted directly from the game (which also uses the day/month for internally) and having a year would "corrupt" the data.
Also, making the date XXXX-02-06 would go against ISO 8601 as omitting years is against the standard.

Just want to add my two cents.
Beside what @alexislours said, that the year is not given by Nintendo and we should use the data how it was dumped from the game, it's a very bad idea to try add an birth year on our own.
Because this information would be wrong and misleading, animals are definitely much older than their release date, you can feel it from conversations with them.
The only thing that I would add, if it given by the game, is the translation for each region like for names. The same applies to catch-phrase and personality. I know that at least the catch phrase is different depends on the language.

I'll add an object containing the catch-phrase translations. The personality doesn't really have an "official translation" so it would have to be mapped for each language by the user. The game use two letter codes such as "AN" for Uchi/Sisterly.


[Bug] Backward inference

Describe the bug
Backward inference is currently not handles correctly.
See this test case:
        ' Backward inference

        ' Bird is a type of swimmer.
        <bird --> swimmer>. %1.00;0.80%

        ' What is a type of swimmer?
        <?1 --> swimmer>?

        5

        ' What is a type of bird?
        ''outputMustContain('<?1 --> bird>?')

        ' What is the type of bird?
        ''outputMustContain('<bird --> ?1>?')

To Reproduce
Type above statements in the console python -m pynars.Console

I think 2 rules should be added:
<P --> M>.
<S --> M>?
|-
<S --> P>?

and
<P --> M>.
<S --> M>?
|-
<P --> S>?

Will review today. Thanks!

@bowen-xu when comparing previous test run on dev with the test run on this branch, the number of failing tests is the same. This is unexpected. I'm going to investigate but my guess is that this change fixed some tests but broke other ones so the total number is ~ the same :)

Fixed


GitHub markdown not rendering on GitHub for Merriweather Sans

Observed behaviour
Typically, I run FontBakery with the --ghmarkdown option, in order to get a report easily readable on GitHub. In fact, I just did this in a project of mine with the latest released version of fontbakery, and it again worked as expected.
However, I advised @EbenSorkin to use this option on his Merriweather Sans project, but for some reason that isn't immediately clear to me, the report isn't rendering with the usual GH markdown dropdowns, tables, etc – it just looks like the raw markdown:
https://github.com/EbenSorkin/Merriweather-Sans/blob/69e90335692f8572b2efc32e498ca60061ca51bc/checks/README.md
(here's a screenshot)

Expected behaviour
I expect it to render with proper styling, as my GH markdown reports do. One example is from Libre Caslon:
https://github.com/thundernixon/Libre-Caslon/blob/f2e1c4e271abfb92753dd4b2c00dd1c698844c97/font_checks/LibreCaslonText-Bold-fontbakery-report.md
(here's a screenshot)

Resources and exact process needed to replicate
I would need Eben to upload his Glyphs sources (or at minimum, the variable TTF) to the Merriweather repo in order for me to reproduce this issue.
In the mean time, I was wondering if there is something obviously wrong in the md file generated in Merriweather, or if others know something that might prevent GitHub from rendering markdown as I expect.
Was this an issue with a slightly older version of FontBakery? If so, I don't see the issue popping up, but we can try updating FB next time we run this.
Just wanted to ask someone about this, and asking here allows others to search for it in the future. Thanks for any insight!

Eben's file is saved in binary format for some reason. C/W his README in the root of his repo where the Markdown renders as expected:

and with your check MD files in the Libre-Caslon repo where they render as expected:

Oh wow, amazing find!
Eben did briefly open the .MD file, which opened in xcode on his computer. Maybe he saved it as a binary there?

I am not sure how it was made binary if it was. The file was made by
fontbakery using the --ghmarkdown checks.md command as part of a check ogf
an existing VF.
The whole line
(TestMerriweather) EbensiMacRetina:Merriweather-Sans ebensorkin$ fontbakery
check-googlefonts
/Users/ebensorkin/virtualenvs/variable_ttf/Merriweather-VF.ttf --ghmarkdown
checks.md
On Fri, Feb 15, 2019 at 9:19 PM Stephen Nixon<EMAIL_ADDRESS>wrote:

Oh wow, amazing find!
Eben did briefly open the .MD file, which opened in xcode on his computer.
Maybe he saved it as a binary there?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/googlefonts/fontbakery/issues/2358#issuecomment-464274533,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACFIXbCLI1qr4_WRm5CbuhQuv1E1m1mbks5vN2qsgaJpZM4a-mMN
.

It may have happened during the transition from the fontbakery output file through XCode and back out as a new save.  I'm not sure why XCode would save a Markdown file as a binary...
Can you confirm that a repeat attempt at execution of the fontbakery command that you used in https://github.com/googlefonts/fontbakery/issues/2358#issuecomment-464281656 without opening the Markdown file in another application fixes the problem?

As an update to this, Eben and I just did a video hangout, and ran through the process again. We avoided opening anything in Xcode, and – surprisingly to me – still ended up with a binary file:
fontbakery ghmarkdown file as binary
Process
We used FontBakery 0.6.11, and ran:
fontbakery check-googlefonts <font_file>.ttf --ghmarkdown test.md

We used the VScode terminal to do this (my typically method), then pushed via GitHub desktop (not my typical route). I don't think either of these things would have an effect, and they didn't in a test I just now ran (outputting this file).
Is there anything else on Eben's computer that might cause this unexpected formatting? We set up a virtual environment with python 3 and a fresh upgrade of FontBakery, but I imagine there's something in the dependencies that might be responsible, somehow...

@felipesanches @graphicore do you have any idea why there might be a within platform, same Py interpreter, same fontbakery version difference in the Markdown output here?  I don't know the answer and am having difficulty finding where the Markdown file write occurs in the library.
cc @EbenSorkin

@thundernixon when you run the test with Ebens version of Merriweather, does it give you a problem.
I just did:

open https://github.com/EbenSorkin/Merriweather/blob/326b72d92e51c535f6f0cb6263297c5d76b6f160/test.md in a tab
click the edit symbol (the pencil next to Raw|Blame|History) which created a fork in my GitHub account
directly without changes clicked "Preview Changes" and got a mystical hint for free…
removed the char https://github.com/graphicore/Merriweather/commit/995c881a8a195e4bc5bda10e2b425b6296880a49
saved with reference to this issue https://github.com/graphicore/Merriweather/blob/graphicore-bad-char/test.md

LOL
That's kind of crazy because i did successfully register that with MS years
ago. Maybe we should get a newer set of Data from MS?
-e.
On Thu, Feb 28, 2019 at 4:02 PM Lasse Fister<EMAIL_ADDRESS>wrote:

@thundernixon https://github.com/thundernixon when you run the test
with Ebens version of Merriweather, does it give you a problem.
I just did:

open
https://github.com/EbenSorkin/Merriweather/blob/326b72d92e51c535f6f0cb6263297c5d76b6f160/test.md
in a tab
click the edit symbol (the pencil next to Raw|Blame|History) which
created a fork in my GitHub account
directly without changes clicked "Preview Changes" and got a
mystical hint for free… [image: selection_229]
https://user-images.githubusercontent.com/393132/53598011-efc6c500-3ba3-11e9-9af4-a181ddaa2910.png
removed the char graphicore/Merriweather@995c881
https://github.com/graphicore/Merriweather/commit/995c881a8a195e4bc5bda10e2b425b6296880a49
saved with reference to this issue
https://github.com/graphicore/Merriweather/blob/graphicore-bad-char/test.md

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/googlefonts/fontbakery/issues/2358#issuecomment-468437212,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACFIXRuhOrhd6PAY4eqGH5tCCHsUh2PIks5vSEQAgaJpZM4a-mMN
.

@felipesanches looks like we have an interesting bug where bad/wrong bytes are injected into output. I would kind of expect that python would choke on that char.
@EbenSorkin what is that char?

Oh - it isn't the code. ( STC ) it is the glyph after that.
But in the Glyphs UI it doesn't look like there is a space or glyph after
the STC.
Because this started life as a Fontlab file that may have something to do
with it.
If I output a UFO file might that shed some light on it? Or is this a
question to bring to Rainer/Georg/Adam?
-e.
On Thu, Feb 28, 2019 at 4:02 PM Lasse Fister<EMAIL_ADDRESS>wrote:

@thundernixon https://github.com/thundernixon when you run the test
with Ebens version of Merriweather, does it give you a problem.
I just did:

open
https://github.com/EbenSorkin/Merriweather/blob/326b72d92e51c535f6f0cb6263297c5d76b6f160/test.md
in a tab
click the edit symbol (the pencil next to Raw|Blame|History) which
created a fork in my GitHub account
directly without changes clicked "Preview Changes" and got a
mystical hint for free… [image: selection_229]
https://user-images.githubusercontent.com/393132/53598011-efc6c500-3ba3-11e9-9af4-a181ddaa2910.png
removed the char graphicore/Merriweather@995c881
https://github.com/graphicore/Merriweather/commit/995c881a8a195e4bc5bda10e2b425b6296880a49
saved with reference to this issue
https://github.com/graphicore/Merriweather/blob/graphicore-bad-char/test.md

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/googlefonts/fontbakery/issues/2358#issuecomment-468437212,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACFIXRuhOrhd6PAY4eqGH5tCCHsUh2PIks5vSEQAgaJpZM4a-mMN
.

vim displays it as ^@ (OS/2 VendorID value 'STC^@' is not)

@EbenSorkin let's figure what we got here ;-) FontBakery should detect this. If you can find out where the char originated, we may or may not have to bring in others.
excerpt from $ hexdump -C ~/Downloads/test.md | less:
00002080  4e 2a 2a 20 4f 53 2f 32  20 56 65 6e 64 6f 72 49  |N** OS/2 VendorI|
00002090  44 20 76 61 6c 75 65 20  27 53 54 43 00 27 20 69  |D value 'STC.' i|
000020a0  73 20 6e 6f 74 20 61 20  6b 6e 6f 77 6e 20 72 65  |s not a known re|

Looks like a NULL character(00). It may very well come from FontLab, I don't know it that good, but maybe it used NULL as a value separator and somewhere in translation it was just expected that the OS/2 VendorID has always 4 chars, so the separator was also copied into the new file …

Spec says achVendID it's supposed to be a data type of tag:

Array of four uint8s (length = 32 bits) used to identify a table, design-variation axis, script, language system, feature, or baseline

So, maybe FontBakery just handles a 3(or 2 or 1)-unit tag, which is padded with NULL at the end, wrongly? cc @felipesanches

Looks also like the warning is a false positive, which comes from treating the tag-type wrongly. Do we know other examples with less than four chars in the vendor ID?

yes, we do. And the code-test is prepared to endure our code is capable of dealing with those cases as well.
https://github.com/googlefonts/fontbakery/blob/785feab22f8ae3fdc3162b4b78b2fd7a204e1299/tests/specifications/googlefonts_test.py#L355-L393

In this case CFA  is represented as 43 46 41 20 where 20 is the space char. But it seems that in a font it is represented as 43 46 41 00, not with a space, but with a NULL.

So, the question now in my head is:
Is the NULL char a valid byte for the purpose of padding vendor IDs? Does it cause trouble to some third-party software? Should we treat it as a font-bug and  emit a WARN (or even FAIL) regarding that?

Yeah, that's kind of the question.
We do the space-padding ourselves:
https://github.com/googlefonts/fontbakery/blob/785feab22f8ae3fdc3162b4b78b2fd7a204e1299/Lib/fontbakery/specifications/googlefonts.py#L652
Maybe we shouldn't.
Then in:
https://github.com/googlefonts/fontbakery/blob/785feab22f8ae3fdc3162b4b78b2fd7a204e1299/Lib/fontbakery/specifications/googlefonts.py#L672
remove NULL bytes from the end.
I doubt that spaces (0x20 bytes) are correct padding.

0x0020 is what I expect for padding
On Thu, Feb 28, 2019, 2:24 PM Lasse Fister<EMAIL_ADDRESS>wrote:

Yeah, that's kind of the question.
We do the space-padding ourselves:
https://github.com/googlefonts/fontbakery/blob/785feab22f8ae3fdc3162b4b78b2fd7a204e1299/Lib/fontbakery/specifications/googlefonts.py#L652
Maybe we shouldn't.
Then in:
https://github.com/googlefonts/fontbakery/blob/785feab22f8ae3fdc3162b4b78b2fd7a204e1299/Lib/fontbakery/specifications/googlefonts.py#L672
remove NULL bytes from the end.
I doubt that spaces (0x20 bytes) are correct padding.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/googlefonts/fontbakery/issues/2358#issuecomment-468463278,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAP9yy2AO2T-bV42cmArvSfjeC1pfSkjks5vSFcKgaJpZM4a-mMN
.

0x0020 is what I expect for padding

How so? It seems to me that a space "could rather" be a legitimate part of a vendor ID, as other non-alphanumeric chars are accepted as well (H&FJ), NULL is put into the font binary if you give it a less than 4 chars string. So, if you want spaces, you got to put them into the binary explicitly. Maybe fontTools should remove the NULL padding when decompiling?
At least I can bust a fontTools roundtrip, look at Out[48]: 'STC\x00':
In [43]: ttfont = TTFont('../testfonts/RobotoSlab-Regular.ttf')

In [44]: ttfont['OS/2'].achVendID
Out[44]: 'GOOG'

In [45]: ttfont['OS/2'].achVendID = 'STC'

In [46]: ttfont.save('./mytestfont.ttf')

In [47]: ttfont2 = TTFont('./mytestfont.ttf')

In [48]: ttfont2['OS/2'].achVendID
Out[48]: 'STC\x00'

In [49]: ttfont['OS/2'].achVendID
Out[49]: 'STC'

In [50]: ttfont['OS/2'].achVendID == ttfont2['OS/2'].achVendID
Out[50]: False

this is now fixed


Web auth/cookies confuse identity for other ships

This happened roughly a year ago, where ~poldec was managing to post to fora as ~taglux, and again just now when someone on stream was posting as my identity, ~palfun. Clearing cookies resolved the issue in both cases, so something in there is getting mixed up, or set incorrectly by eyre?

Does this sound like the kind of thing that can be caused by the node proxy? @galenwp

Yeah, that’s possible. I’m not sure how the identity of a posting ship gets
set / validated.
On Thu, Jun 14, 2018 at 11:07 AM Anthony Arroyo<EMAIL_ADDRESS>wrote:

Does this sound like the kind of thing that can be caused by the node
proxy? @galenwp https://github.com/galenwp
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/urbit/arvo/issues/748#issuecomment-397387183, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABZjS4CTerDbc620pYs7KPOVwiDvb1Yjks5t8qZDgaJpZM4UnlgG
.

--
—
~ravmel-ropdyl
https://urbit.org

I think once you're a posting ship it passes requests through directly? The
purpose of the auth proxy I remember was only multiplexing subscriptions
for webtalk clients without cookies, if you try to post it gets you a
canonical cookie and thus stops altering requests
On Fri, Jun 15, 2018 at 8:05 AM, Galen Wolfe-Pauly<EMAIL_ADDRESS>
wrote:

Yeah, that’s possible. I’m not sure how the identity of a posting ship gets
set / validated.
On Thu, Jun 14, 2018 at 11:07 AM Anthony Arroyo<EMAIL_ADDRESS>wrote:

Does this sound like the kind of thing that can be caused by the node
proxy? @galenwp https://github.com/galenwp
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/urbit/arvo/issues/748#issuecomment-397387183, or
mute
the thread
<https://github.com/notifications/unsubscribe-auth/
ABZjS4CTerDbc620pYs7KPOVwiDvb1Yjks5t8qZDgaJpZM4UnlgG>
.

--
—
~ravmel-ropdyl
https://urbit.org
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/urbit/arvo/issues/748#issuecomment-397650999, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABxXhsFndWi76roJeJnxEYTA4AboUhJtks5t881DgaJpZM4UnlgG
.


Changes for middleware-webpack review

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.soulwu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

LaoGeTaiMeng


Documentation📄: Remove 'Copy to Clipboard' from /common-requests/ pages

What is the current documentation state?
With the new 'Just the Docs' theme, the "Copy to Clipboard" button is unnecessary and can be removed from https://open-horizon.github.io/quick-start/
The new theme provides a Copy icon for code blocks.
Edit this page:
https://github.com/open-horizon/open-horizon.github.io/blob/master/quick-start.md
Where is this stated?
No response
Why do you want to improve the statement?
No response
Proposed Statement
No response
Additional context.
No response

Remove "Copy to Clipboard" from these pages too:

https://open-horizon.github.io/common-requests/use/
via https://github.com/open-horizon/open-horizon.github.io/blob/master/common-requests/use.md

https://open-horizon.github.io/common-requests/contribute/

via https://github.com/open-horizon/open-horizon.github.io/blob/master/common-requests/contribute.md

All of the markdown pages in /common-requests/ need many lint fixes.

I tested the pages that the PR modified.  Everything looks good.


NodeJS Apps as a pm2 Service

There are many ways to deploy a NodeJS app as a service. I've had great results with pm2. Does anybody have other suggestions?
I typically follow these instructions for setup:
https://www.digitalocean.com/community/tutorials/how-to-use-pm2-to-setup-a-node-js-production-environment-on-an-ubuntu-vps
One thing I'm wondering about... We've got different apps as different users. Fieldpapers is fp. OpenMapKitServer is omk. I'm a bit unclear on how pm2 handles that...
Maybe all our NodeJS apps should be under the same user?

Old PM2 Setup Notes
https://github.com/Unitech/pm2
I'm somewhat following this guide for setting things up via pm2:
https://www.digitalocean.com/community/tutorials/how-to-use-pm2-to-setup-a-node-js-production-environment-on-an-ubuntu-vps
First, we do this so we do not have to be sudo to run an app on port 80.
sudo apt-get install libcap2-bin
sudo setcap cap_net_bind_service=+ep /usr/bin/nodejs

Currently, our pm2 setup is very simple. Set port to 80 in config.js. Then, we install pm2 this way:
sudo npm install -g pm2

We want pm2 to be a daemon and restart our app if it crashes or if the server restarts:
sudo su -c "env PATH=$PATH:/usr/bin pm2 startup linux -u ubuntu"

Then vim into /etc/init.d/pm2-init.sh and change PM2_HOME to /home/ubuntu/.pm2. I know, it's werid, but it works.
Then we want to start the app in pm2:
pm2 start api.js
pm2 save

To manage your Node process running you can:
pm2 stop     api
pm2 restart  api
pm2 delete   api

If you want to update to the latest commit and redeploy:
cd /home/ubuntu/PMT-API/
git pull origin master
pm2 restart all

@jflemer-ndp rocked it out with upstart. No need for this. Thanks James!


Linkify Employers

In CSS Writing Modes, we used to link the Former Editors employer names to their website, e.g. Antenna House. Seems like something broke that recently. It would be nice to be able to do that again.

That works, just read the docs for the syntax.  (Unless you mean that you tried the syntax in the docs, and something broke it?)

Yes, confirmed, it works fine if you follow the grammar in the docs.


[NPU] Contribute Ascend NPU Compatibility

[NPU] Support STDiT2-XL/2 training and inference for Ascend NPU
What I do in this commit

opensora/models/layers/blocks.py
opensora/acceleration/shardformer/modeling/t5.py
opensora/utils/device_utils.py
requirements_npu.txt
scripts/inference-long.py
scripts/inference.py
scripts/train.py
setup.py
README.md
docs/zh_CN/README.md

Great! Is this PR valid for the latest v1.2?


total_price for orders

Closes https://github.com/inventree/InvenTree/issues/4442
This PR adds a total_price field (calculated) to the PurchaseOrder and SalesOrder models, so that they do not have to be calculated "on the fly" (and thus require expensive recursive database hits)
Pricing will be calculated in a common currency, allowing filtering/ordering of total_price via the API
Initial Query Results
The total_price annotation on the PurchaseOrder API is expensive:

API Results
Database Queries
After Refactor

1
14
6

5
16
6

10
26
6

77
160
6

TODO

[x] Add new fields to models
[x] Data migration
[x] Add 'total_price' to salesorder table
[x] Allow filtering by total_price on purchaseorder api
[x] Allow filtering by total_price on salesorder api
[x] Hooks for automatically recalculating total_price values
[x] Unit test for PurchaseOrder list API (query count)
[x] Unit test for SalesOrder list API (query count)
[x] Unit test for hooks to update pricing
[x] Fix total price rendering on PurchaseOrder page
[x] Fix total price rendering on SalesOrder page

@simonkuehling you can now sort by 'total_price' on purchase and sales orders (and it no longer makes a bunch of queries)

very nice! :+1:


NoConnectionError Changes Error Object

When we're mapping error response to ermrestjs error object, we're not preserving the actual response data in case of NoConnectionError

This is done in this commit.


Mailbox size

Is there a way to limit the mailbox size and drop old emails? I would like to prevent mailcatcher to consume all available disk space.

Looks like its been sitting in a pull request for a year.
https://github.com/sj26/mailcatcher/pull/375

#375 was merged in a while ago. Please try the beta:
‘‘‘
gem install --pre mailcatcher
‘‘'


Platformatic build fails when tsc is not installed

Executing platformatic build command fails for the js service if tsc is not installed.
➜ platformatic build
[12:22:57.570] INFO (26404): Building service "service-1" ...
[12:22:57.583] WARN: The tsc executable was not found.
[12:22:57.585] ERROR (26404): Building service "service-1" has throw an exception.
    err: {
      "type": "Error",
      "message": "Failed to compile /Users/ivan-tymoshenko/projects/platformatic/test-icc-app-2/services/service-1",
      "stack":
          Error: Failed to compile /Users/ivan-tymoshenko/projects/platformatic/test-icc-app-2/services/service-1
              at ServiceStackable.build (/Users/ivan-tymoshenko/projects/platformatic/test-icc-app-2/node_modules/.pnpm/@platformatic+service@2.0.0-alpha.12/node_modules/@platformatic/service/lib/stackable.js:70:13)
              at async #handleRequest (/Users/ivan-tymoshenko/projects/platformatic/platformatic/packages/itc/lib/itc.js:282:24)
    }

Expected behavior:
➜ platformatic build
[12:22:46.399] INFO (24641): Building service "service-1" ...
[12:22:46.413] INFO: No typescript configuration file was found, skipping compilation.
[12:22:46.415] INFO (24641): ✅ All services have been built.

Fixed in #3112.