fix: add support for EL10

According to the Ansible team, support for listing platforms in
role meta/main.yml files is being removed.
Instead, they recommend using galaxy_tags
https://github.com/ansible/ansible/blob/stable-2.17/changelogs/CHANGELOG-v2.17.rst
"Remove the galaxy_info field platforms from the role templates"
https://github.com/ansible/ansible/issues/82453
Many roles already have tags such as "rhel", "redhat", "centos", and "fedora".
I propose that we ensure all of the system roles have these tags.
Some of our roles support Suse, Debian, Ubuntu, and others.
We should add tags for those e.g. the ssh role already has tags for "debian" and "ubuntu".
In addition - for each version listed under platforms.EL - add a tag like elN.
Q: Why not use a delimiter between the platform and the version e.g. el-10?
This is not allowed by ansible-lint:
meta-no-tags: Tags must contain lowercase letters and digits only., invalid: 'el-10'
meta/main.yml:1

So we cannot use uppercase letters either.
Q: Why not use our own meta/main.yml field?
No other fields are allowed by ansible-lint:
syntax-check[specific]: 'myfield' is not a valid attribute for a RoleMetadata

Q: Why not use some other field?
There are no other applicable or suitable fields.
Q: What happens when we want to support versions like N.M?
Use the word "dot" instead of "." e.g. el10dot3.
Similarly - use "dash" instead of "-".
We do not need tags such as fedoraall.
The fedora tag implies that the role works on all supported versions of fedora.
Otherwise, use tags such as fedora40 if the role only supports specific versions.
In addition - for roles that have different variable files for EL9, create
the corresponding EL10 files.
Signed-off-by: Rich Megginson<EMAIL_ADDRESS>

@mlichvar anything else needed for el10 at this time?  I'll note that the tests tests_ptp_multi.yml and tests_ntp_ptp.yml are failing on centos-10 - I think due to selinux issues.


all: fix typo

thanks for the PR! can you do git commit --amend --signoff and repush?

thanks for the PR! can you do git commit --amend --signoff and repush?

Done

looks like you'll also need to do cd test && go mod vendor. I should've remembered it the first time 😄

implemented


DWL with vue.js does not work (the bundle itself works...)

I have been running DWL for couple of months now and working great! I now tried to include vue.js in my bundle and do not get it working. However it does work when I copy the main-[hash].js bundle to the Django static folder and reference it with:

Furthermore, (several) other libraries and css and sass included in the bundle DO get through when using DWL. So it seems something particular makes DWL not work with vue.js. As there is some issues with django template tags and vue tags both being {{ }}, it is 'common' to change vue tags into [[ ]] ; I tried whether changing it into something else, like {* *}would solve my issue, but that did not solve it.
Further details:
testhtml:
{% load staticfiles %}

<!DOCTYPE html>

    <meta charset="utf-8">
    <title>Webpack Sample Project</title>

{% load render_bundle from webpack_loader %}

{% render_bundle 'main' %}

[[ message ]]

kiekeboe

{##}

The js (render_bundle) needs somehow to come after the vue tag reference in the html file!
This works:
{% load staticfiles %}
{% load render_bundle from webpack_loader %}

Webpack Sample Project

[[ message ]]
kiekeboe

{##}
{% render_bundle 'main' %}

@musicformellons Could you please share your config of webpack for vuejs?
I am struggling putting together Django and VueJS.
Thank you.

I jumped the fence into javascript land... No more Django... Sorry... Good luck.

I'm using Vue.js with Django Webpack loader without problems. Here is my index.html template.
{% load render_bundle from webpack_loader %}
<!DOCTYPE html>

        <meta charset="utf-8">
        <title>App Title</title>
        {% render_bundle 'public' 'css' %}

        {% render_bundle 'public' 'js'%}

@rawberg I don't have a problem with template part, these few tags are easy.
I was wondering about webpack config with hot reload, could you please share that? Thanks.

I haven't gone through the trouble of setting up hot reload but good old fashioned refresh seems to work pretty well with this setup.
module: {
        loaders: [
            {
                test: /\.vue$/,
                loader: 'vue'
            },
            {
                test: /\.js$/,
                loader: 'babel',
                exclude: /node_modules/
            },
            {
                test: /\.json$/,
                loader: 'json',
                exclude: /node_modules/
            },
            {
                test: /\.css$/,
                loader: ExtractTextPlugin.extract({fallbackLoader: "style-loader", loader: "css-loader"}),
                exclude: /node_modules/
            },
            {
                test: /\.less$/,
                loader: ExtractTextPlugin.extract({fallbackLoader: "style-loader", loader: "css-loader!less-loader"})
            },
            // handles all references to these files in JS or CSS, use copy webpack plugin
            // for images references in HTML or anywhere else
            {
                test: /\.(png|jpg|gif|svg|gif)$/,
                loader: 'file-loader',
                query: {
                    name: 'img/[name]-[hash].[ext]'
                }
            },
            {
                test: /\.(ttf|eot|woff|woff2)$/,
                loader: 'file-loader',
                query: {
                    name: 'font/[name]-[hash].[ext]'
                }
            },
            {
                test: /\.(txt|ico)$/,
                loader: 'static-loader'
            }
        ]

@rawberg Thank you. This is very helpful.


Update documentation, round 2

This time we try to clean up a lot of the README, after its structure
was reorged in #4345.
cc @calebcartwright
Closes #4336

Excellent, thank you!


Black theme

Another idea - having an AMOLED-friendly true black theme.

I think this can be implemented easily, only need to modify src/theme/colors.ts. At the moment, I don't have a test device with amoled so I can't test the colors.
If you want, you can give me the corresponding colors for each values similar in the format below, then we can test it.
// src/theme/colors.ts
const LIGHT = {
  PRIMARY: '#e15f41',
  POSITIVE: '#218c74',
  NEGATIVE: '#b33939',
  TITLE: '#1f1f1f',
  PRIMARY_TEXT: '#434343',
  SECONDARY_TEXT: '#595959',
  DISABLE: '#bfbfbf',
  BORDER: '#d9d9d9',
  DIVIDER: '#f0f0f0',
  PLACE_HOLDER: '#8c8c8c',
  BACKGROUND: '#f5f5f5',
  AREA_HIGHLIGHT: '#fafafa',
  ERROR: '#c44569',
};

export const DARK = {
  PRIMARY: '#e15f41',
  POSITIVE: '#33d9b2',
  NEGATIVE: '#ff5252',
  TITLE: '#fafafa',
  PRIMARY_TEXT: '#f5f5f5',
  SECONDARY_TEXT: '#d9d9d9',
  DISABLE: '#d9d9d9',
  BORDER: '#bfbfbf',
  DIVIDER: '#595959',
  PLACE_HOLDER: '#595959',
  BACKGROUND: '#1f1f1f',
  AREA_HIGHLIGHT: '#141414',
  ERROR: '#c44569',
};

export default {
  LIGHT,
  DARK,
  Light: LIGHT,
  Dark: DARK,
};

@jerameel was interested in supporting this issue, as an AMOLED black theme would be a good option. I can help you with the color codes that you requested, but I'm not sure where to get them from. Can you please tell me the steps?

You can just do something like this and just replace the colors to what is better for amoled screens...
file is located at src/theme/colors.ts if you want to do the PR yourself
export const AMOLED_DARK = {
  PRIMARY: '#e15f41',
  POSITIVE: '#33d9b2',
  NEGATIVE: '#ff5252',
  TITLE: '#fafafa',
  PRIMARY_TEXT: '#f5f5f5',
  SECONDARY_TEXT: '#d9d9d9',
  DISABLE: '#d9d9d9',
  BORDER: '#bfbfbf',
  DIVIDER: '#595959',
  PLACE_HOLDER: '#595959',
  BACKGROUND: '#1f1f1f',
  AREA_HIGHLIGHT: '#141414',
  ERROR: '#c44569',
};


Hydrating a component with dangerouslySetInnerHTML coming from SSR fails and shows empty

so basically all richtext fields fail to show coming from SSR.
It seems a React v16 bug that it's still open:
https://github.com/facebook/react/issues/11789
There's a PR to fix it but Dan Abramov didn't like it...
https://github.com/facebook/react/pull/11792
We can use old render method but I guess it's not future proof, and console complains.
/cc @robgietema @tisto

temporally resolved that problem via internal state.
{
canShow: false
}
componentDidMount() {
handleAllow()
}
handleAllow just sets canShow: true
additionally have added loader until component will mounted.

@Yauheni-blr Workaround:

Use the old render react-dom method instead of hydrate.
Call .toString() in all elements that use dangerouslySetInnerHTML which can be tedious.

@sneridagh

have a pain when render method will be deprecated :)
upd: same story with lifecycle hooks.

P.S. improvement the quality of the code as usually better than destroyed app.

Fixed in release 1.5.2


file db.js is created inside config and promises is handled with mongoose

What is the change?
created a new folder named config and created the file inside it called db.js and put the database connection code inside it.
Related issue?
close: #320
How was it tested?
yes i have tested the code
Checklist:
Before you create this PR, confirm all the requirements listed below by checking the checkboxes [x]:

[x] Have you followed the Contribution Guidelines while contributing.
[x] Have you checked there aren't other open Pull Requests for the same update/change?
[x] Have you made corresponding changes to the documentation?
[x] Your submission doesn't break any existing feature.
[x] Have you tested the code before submission?

Screenshots or Video:
Add a screenshot or demo video if appropriate.

your master branch have different code in app.js and develop branch has different code in app.js. I have forked from master branch and making pull request to develop branch causing conflict

should i forked develop branch and do it again or you will resolve it by yourself


Implement Table::AsyncReadRows.

This fixes #1487.

This change is

I think we should implement this will callbacks for now. This is too easy to use wrong, and too many layers of type wrapping.

Thanks, I will review when I get back to work tomorrow.
To be clear: I think we will want something like futures for asynchronous streaming reads, it is just that we are not ready for them yet.  What I am thinking about is something like RxCpp (which I believe you introduced me to), so application would be able to say things like:
Stream<Row> rows = table.AsyncReadRows(...);
// Call the lambda for each result and return a new stream, note that this is done asynchronously ...
Stream<Bar> bars = rows | transform([](Row r) -> Bar { ... });
// Aggregate the Bars into some kind of total
future<Total> final = bars | accumulate(Total{}, [](Total acc, Bar b) -> Total { ... });
final.then([](Total t) { std::cout << t << std::endl; });

But I do not think we have the time or the enough understanding of this stuff to design it correctly, so I am going with callbacks for now.
Somewhat unrelated, are you sure about that parallel version? "[Read] ..  should not be called concurrently with other streaming APIs on the same stream. It is not meaningful to call it concurrently with another AsyncReaderInterface::Read on the same stream since reads on the same stream are delivered in order."
https://grpc.github.io/grpc/cpp/classgrpc_1_1internal_1_1_async_reader_interface.html#adf87c602036d69158950b7299d0aae70

Somewhat unrelated, are you sure about that parallel version? "[Read] .. should not be called concurrently with other streaming APIs on the same stream. It is not meaningful to call it concurrently with another AsyncReaderInterface::Read on the same stream since reads on the same stream are delivered in order."

This is only parallel row processing - not reading. The first two examples instruct gRPC to read more data only after the future<> returned by Process() is satisfied. The parallel example allows for N simultaneous Process() to run, but gRPC still gets only one AsyncReaderInterface::Read() at a time - this is implemented in AsyncRowReader.

Should we close this PR?

Closing in favor of #2701.


[Concurrency] Guard use of async calling convention.

Certain targets don't support the async calling convention, so we first
add the feature check to avoid breaking the codegen/runtime while doing
gradual rollout for different targets.
LLVM patch introducing feature: https://reviews.llvm.org/D96802

@swift-ci smoke test

@swift-ci smoke test

@swift-ci please smoke test linux

@swift-ci smoke test and merge


Add password toggle into react text field

Looks like you already have password toggles:

Looks like you already have password toggles

Hi,
thank you for your interest in our issue :-)
Password toggles exist only for web package implementation. See https://github.com/lmc-eu/spirit-design-system/blob/main/packages/web/src/js/Password.ts and https://github.com/lmc-eu/spirit-design-system/tree/main/packages/web/src/scss/components/TextField#password-toggle.
For React and web-react package there is no such implementation. See https://github.com/lmc-eu/spirit-design-system/blob/main/packages/web-react/src/components/TextField/TextField.tsx and https://spirit-design-system-storybook.netlify.app/?path=/docs/components-textfield--template.
We want to be able to use such functionality also in our React components when TextField is of type password.
Hope this is helpful.

Already added.


High-level questions, to help user take decisions

Is your website internal?

Yes => auto-select SPA. (FYI :ssr config.)

Does the content of your website change only occasionally?

Yes => auto-select Pre-rendering and GitHub Pages / Netlify.

Do you need a database?

Auto-select accordingly.
A neat thing is that it would enable us to show warnings if the user's selection doesn't make sense and/or isn't recommended.

It could be a good alternative entry point for people which are not sure of what technology they want/need.
We probably should define a global flow to choose which Bati UI to show on which conditions.

I can't think of some kind of flow right now but maybe there will be some, e.g. if "No database" then some questions will become irrelevant. We'll see I guess 👀.
Maybe there are some questions we want to always show the user? For expert users it could be pesty, but I'm thinking a solution could be to make answering questions optional, so that expert users can go right ahead and select the tools they want.
As for the UI, maybe we can show the questions on a compact left sidebar? (If we don't need some kind of flow in the foreseeable future.)

Questions helping you choose:

Do you like JSX? [i]
Is your website internal? [i]
Does content change only occasionally? [i]
Do you need to persist data? [i]

With compact yes/no toggles like on mobile UIs.
On a tangent, it could be neat to show a confirmation:

✅ No issues detected.

⚠ 2 warnings:

You selected I like JSX and Vue. While Vue does support JSX, it's a second class citizen and we recommend using Solid or React instead.
You selected Content changes only occasionally and Vercel. For websites with content that changes only occasionally (e.g. a few times a day), we recommend using a static host such as GitHub Pages or Netlify.

Maybe there are some questions we want to always show the user? For expert users it could be pesty, but I'm thinking a solution could be to make answering questions optional, so that expert users can go right ahead and select the tools they want.

A wizard UI would be for Junior devs and new Bati users, and the current UI for users that have already used it before I think. So I think that both UI will need to exist at any moment.

As for the UI, maybe we can show the questions on a compact left sidebar? (If we don't need some kind of flow in the foreseeable future.)

That's something I would need to test

On a tangent, it could be neat to show a confirmation

That's a good addition, could be added rather sooner than later if it's simple enough 👍

Not the be-all-end-all, but maybe some inspiration:


Killed for the long recording

GPU memory will be OOM  when the recording time is too long ,for example 1 ~2 hours, any update plan for this scenerio?
by the way, great job! great work!

I have no plan to solve that as it's well outside of my own use case, sorry! You can try other models, or try changing the parameters to use a different batch size etc. - but depending on which model architecture you're using it may also have some kinda memory leak which might be causing the issue.
You can also always chunk your input audio into smaller clips (e.g. using ffmpeg), separate each smaller segment, then join the outputs!
If you decide you want to improve audio-separator to make it handle larger input files more gracefully like this yourself, PRs are welcome :)


add methods to enable debug of SQL statements

Closes #141

added Debug and DebugWith to Store
added Debug and DebugWith to typed stores as well

Done @dpordomingo


added scripts for eTRansport

@SarahYS , Could you attach the pdf file that you provide us by email, please?

Thannk you, @SarahYS !


update contributor guide and portal help pages for v6

Look into whether we can add alternate slot descriptions and use to generate documentation for different audiences (e.g. data contributor vs. internal user of data model)

Add more info about how data are collected & processed (i.e. our clinical data workflow)


Boss: Elkemist

Mechanic: Round Clock
Is crafting the "ultimate potion" which gives win condition levels of positive modifiers to self upon completion. Crafting progress is shown (in modifiers?) and can be stalled/regressed via Stun.

Should include self modifiers that are applied unpredictably and counter different strats to give value to modifiers predict. Examples:

reflecting damage
causing attacks to inflict poison scaling with target's block


Create a pong game using python

please assign it to me

I have created the code for this problem. I can send it to you if you can assign it to me.

@AparajitaSingh121416

@geekcomputers , please merge the pr


Google wont let me sign in, says app or browser unsafe

Checklist:

[ x] I have checked that there are no issues with similar or the same content
YOU SHOULD CHECK CLOSED ISSUES ASWELL

[ x] I have checked the FAQ (https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/wiki/FAQ) and the answer I am looking for is not there

[ x] I have double checked and can reproduce the issue

OS:
Windows
GPMDP Version:
4.6.1
Issue Descriptions:
Google wont let me sign in, any attempt to sign in is met with the same error saying the browser or app I'm trying to use won't "keep my account safe" and refuses to let me log in. I've adjusted all my Google security settings and it still won't allow me to log in
Steps to Reproduce:
Open app, put in log in info, get error.

https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/3545


[Enhancement] Allow specifying diagnostic highlights manually

Hi,
So I've recently changed the way highlights are show in my config to highlight the entire line including the sign column a specific colour.

Unfortunately, this also unintentionally changes the signs used by neotree to have the same background colour highlight. Which I don't want/didn't intend, but it turns out that neo-tree uses my sign highlight not the base Diagnostic{Error|Info|Warn} etc.
It would be nice to be able to decouple these two things, so I don't have to plan my sign highlights around neo-tree as well.

Right now it calls vim.fn.sign_getdefined to get both the symbol and the highlight. How about if the component took an optional config object that looked like this?
  default_component_configs = {
    diagnostics = {
      symbols = {
        hint = "H",
        info = "I",
        warn = "!",
        error = "X",
      },
      highlights = {
        hint = "DiagnosticSignHint",
        info = "DiagnosticSignInfo",
        warn = "DiagnosticSignWarn",
        error = "DiagnosticSignError",
      },
     },
     ...

Another thought I had was to just have a single 'highlight_prefix' config option which could be concatenated with the diagnostic level text to get the highlight group.

@cseickel that proposed API looks good to me since it doesn't force the user to specify icons manually or to specify highlights, and they can presumably fill out as much or as little of both tables independently.
Re. adding a prefix I guess that would also work, but I slightly prefer the above example since it feels more straightforward but not by much 🤷🏿

I added the component config option in main. It will take a little while for the next release because I have a half finished preview feature in main right now which is taking longer than expected.
Using the main branch, you can set your highlight groups like this:
  default_component_configs = {
    diagnostics = {
      highlights = {
        hint = "DiagnosticSignHint",
        info = "DiagnosticSignInfo",
        warn = "DiagnosticSignWarn",
        error = "DiagnosticSignError",
      },
     },
     ...

@cseickel that works like a charm, thanks 👍🏿


Added Pointer Events support in iOS 13.2

As the bugs mentioned in 526f26e has been fixed in iOS 13.2, the status can be changed to full support.
For reference, look for the according <rdar://problem/...> in the WebCore ChangeLog.

releasePointerCapture Issue: <rdar://problem/53127223>
buttons Issue: ~<rdar://problem/55394548>~ <rdar://problem/52778374>

@Fyrd I don't think I can resolve this failure...

@kiding Okay, 13.2 is now separate, thanks for discovering that.

Thanks!


make_fixed_from<...>

Also floating-point literals for elastic equivalent.

There is already a task to investigate constexpr literals.


BUG:  Increase the font size of headings which are not visible clearly

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The headings are not clearly visible and the font size is less. I will increase the font size and make it visible.

Please assign this issue to me.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

I'm a GSSOC'24 contributor
please assign this issue to me

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The headings are not clearly visible and the font size is less. I will increase the font size and make it visible.

Please assign this issue to me.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

assigned!
happy open-source contributing 🎉


Drop Smart Gateway from Community Operators

Smart Gateway is being developed at https://github.com/infrawatch/smart-gateway-operator
and will be released under the quay.io/infrawatch application catalog.
Signed-off-by: Leif Madsen<EMAIL_ADDRESS>

CC: @J0zi @dmesser @aneeshkp
Just fyi this is the PR that will just drop Smart Gateway for now. I may re-submit it in the future once I understand the ideal catalog sources that allows for our upstream and downstream releases and documentation to live in harmony. At this point the Smart Gateway in the catalog is very old and should not be used.


Add our standard redis connection code

Codecov Report
Attention: 16 lines in your changes are missing coverage. Please review.

Comparison is base (b648739) 90.67% compared to head (b8de8ce) 85.53%.

Files
Patch %
Lines

pool.go
61.90%
12 Missing and 4 partials :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
- Coverage   90.67%   85.53%   -5.15%
==========================================
  Files           6        7       +1
  Lines         193      235      +42
==========================================
+ Hits          175      201      +26
- Misses         12       24      +12
- Partials        6       10       +4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


DG cleanup

Changes
Related Issues/ PRs

Fixes # (issue)

Codecov Report

Merging #271 (a8abc60) into master (bd91108) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #271   +/-   ##
=========================================
  Coverage     66.05%   66.05%
  Complexity      751      751
=========================================
  Files           131      131
  Lines          2604     2604
  Branches        261      261
=========================================
  Hits           1720     1720
  Misses          768      768
  Partials        116      116

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Unet3+ with resnet34 (and higher) crash

Unet3+ with a backbone > resnet34, e.g. resnet50, resnet101, ... caused the following error
resnet 18 and resnet34 are working OK
version 0.8.0
´´´
File "train.py", line 153, in run
self.train_epoch(data_loader)
File "train.py", line 203, in train_epoch
y_hat = self.model(img)
File "/home/pd/initze/anaconda3/envs/aicore/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "/home/pd/initze/anaconda3/envs/aicore/lib/python3.7/site-packages/torch/nn/parallel/data_parallel.py", line 161, in forward
outputs = self.parallel_apply(replicas, inputs, kwargs)
File "/home/pd/initze/anaconda3/envs/aicore/lib/python3.7/site-packages/torch/nn/parallel/data_parallel.py", line 171, in parallel_apply
return parallel_apply(replicas, inputs, kwargs, self.device_ids[:len(replicas)])
File "/home/pd/initze/anaconda3/envs/aicore/lib/python3.7/site-packages/torch/nn/parallel/parallel_apply.py", line 86, in parallel_apply
output.reraise()
File "/home/pd/initze/anaconda3/envs/aicore/lib/python3.7/site-packages/torch/_utils.py", line 428, in reraise
raise self.exc_type(msg)
RuntimeError: Caught RuntimeError in replica 0 on device 0.
Original Traceback (most recent call last):
File "/home/pd/initze/anaconda3/envs/aicore/lib/python3.7/site-packages/torch/nn/parallel/parallel_apply.py", line 61, in _worker
output = module(*input, **kwargs)
File "/home/pd/initze/anaconda3/envs/aicore/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "/isipd/projects/p_aicore_pf/initze/code/training/lib/models/unet3p/unet3p.py", line 208, in forward
h2_PT_hd4 = self.h2_PT_hd4_relu(self.h2_PT_hd4_bn(self.h2_PT_hd4_conv(self.h2_PT_hd4(h2))))
File "/home/pd/initze/anaconda3/envs/aicore/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "/home/pd/initze/anaconda3/envs/aicore/lib/python3.7/site-packages/torch/nn/modules/conv.py", line 423, in forward
return self._conv_forward(input, self.weight)
File "/home/pd/initze/anaconda3/envs/aicore/lib/python3.7/site-packages/torch/nn/modules/conv.py", line 420, in _conv_forward
self.padding, self.dilation, self.groups)
RuntimeError: Given groups=1, weight of size [64, 64, 3, 3], expected input[2, 256, 64, 64] to have 64 channels, but got 256 channels instead
´´´

Right, ResNet50 and up have a different internal representation of their feature maps... Sorry for missing that in the first implementation. Should be an easy fix, will look into it :+1:

Interesting, could be related to the multi-GPU then. Could you try what happens when you remove this line: https://github.com/initze/thaw-slump-segmentation/blob/master/train.py#L75?


Aliased link before math causes math to disappear

Describe the bug
When there is some KaTeX math sandwiched between two links (specifically observed when the links point to the same note, but are aliased as different names), the math between them doesn't appear.
To Reproduce
Add the following block to your note:
**Show that the collection of [[Topological Space|open sets]] $\mathcal{C} = \{B_{1/m(m+1)}(1/m) \mid m \in \mathbb{N}\}$ is an [[Topological Space|open cover]] for $S$**

Quartz will not render it correctly.
Expected behavior
It should render the math and both links.

Screenshots and Source

Link to Example
Link of Quartz Repository (and file) with causing issues: https://github.com/lilsharu/course-notes/blob/main/content/Math 297 Midterm 2 Review.md
Link of rendered website page: https://lilsharu.github.io/course-notes/Math-297-Midterm-2-Review
Desktop (please complete the following information):

Quartz Version: 4.2.3

Thanks for opening this issue, and the detailed problem description. I'll look into this tomorrow.

@lilsharu I have found the issue and proposed a potential fix.
The problem appeared to be caused by the output of rehypeMathjax not being explicitly set to html.
I have tested your example case. It appears to process correctly:
**Show that the collection of [[Topological Space|open sets]] $\mathcal{C} = \{B_{1/m(m+1)}(1/m) \mid m \in \mathbb{N}\}$ is an [[Topological Space|open cover]] for $S$**

Could you please confirm the proposed fix works on your end as well?
Once again, thanks for reporting this issue.

That does not seem to fix the issue. I shouldn't be using MathJax (I'm using the default "katex") and even after applying the patch, the behavior still remains

@lilsharu thanks for the confirmation.
I have pulled your Quartz fork for debugging. The issue appears to be resolved when updating to the latest version.
Result when running npx quartz build after updating your fork:

The issue you're experiencing is caused be a previous bug related to parsing wikilinks inside tables. The regular expression responsible is incorrectly assuming the | used to split the second wikilink was part of a table. This issue has already been resolved in the latest version of Quartz.
To summarize: updating your fork to the latest version of Quartz should resolve the issue you're experiencing.
Details on how to update can be found here: https://quartz.jzhao.xyz/upgrading
Please let me know if you require further assistance.

Oh I see, thank you so much!


Why train-annotations-machine-imagelabels.csv missing the labeled bounding boxes?

Hello Thanks for releasing this treasure trove of images.
I am wondering why there is not labeled bounding boxes in train-annotations-machine-imagelabels.csv.

Not all images with image-level labels are annotated with boxes, "only" a subset of 1.7M images. You should therefore download the file containing the bounding boxes from the appropriate section of the open images download site (train-annotations-bbox.csv)
Best,

OK,Thank you for you reply!


csl-json: fix date parts label for empty date

Fix the property name of the date parts when the date is empty. It should be date-parts, not date_parts.

Change to single quotes, to match other, similar code.

I don't have the environment to test this right now, but it seems like a simple enough change.

Thanks @larsgw!


.babelrc not being used in application upgraded from v2 to v3

Hallo.
I am trying to use a .babelrc File in an Application that was created with Version 2 of razzle and later upgraded to version 3. With both Versions 3.1.3 and 3.0.0 i get the message "Using .babelrc defined in your app root", however that file is not used. I can write something completely invalid or leave it without the required preset and the application builds anyways and works correct. Same when configured correctly with a additional plugin that we require now. The Plugin is not being loaded.
Any help in tracking down the issue that prevents razzle from working correctly would be appreciated as i suspect its something internal in the packe. When i create a completly new application, it works as expected.
Are there maybe some Dependencys that need manual update ?

Where is your .babelrc located in the project?

In the root folder, at same level as package.json and razzle.config
(Thanks for reaching out)

Did you update all razzle-* packages?

Yes. As i thought it might be a Bug in Version 3.0, i upgraded everything to 3.1.3 and tested there. Although we could not keep that Version as it broke some things in our release pipeline that we will have to fix first.
Everything is in our case is razzle, razzle-plugin-typescript, razzle-plugin-scss.

It seems to be related to the plugin razzle-plugin-typescript, after adding it to a newly scaffolded solution with create-razzle-app and adding the razzle-plugin-typescript, a custom .babelrc is being ignored.

Did you add useBabel: true option to the typescript plugin?


url changes for tutorial pointing to repo/repo issues

In the "Questions? Ideas? Spotted a Bug?" section at the bottom of Rollup.js Tutorial, Part 2, the links to https://github.com/jlengstorf/learn-rollup-js/ give a 404. I think it's just the "-js" that's giving the grief.
Great tutorial. Thanks.

Thanks — fixed!


Foo

Description

Motivation and context

Test plan

Checklist

[ ] My code follows this project's code style.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] My change is adequately tested.

Terms

[ ] My contribution follows the contributing guide.
[ ] I agree to follow the code of conduct.

Just checking rulesets


community: fix error in sambastudio embeddings

fix error in samba studio embeddings  result unpacking

@ccurme can you please take a look at this minor fix


Remove sync.Pool for relation-identifier buffers.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

6 of 7 (85.71%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 28.064%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/ds/relation.go
6
7
85.71%

Totals

Change from base Build<PHONE_NUMBER>4:
-0.1%

Covered Lines:
980

Relevant Lines:
3492

💛  - Coveralls


Updated Android project to match iOS functions

On iOS, the mobileNetworkCode and related fields always return information based on the SIM installed in the device. This means the actual network code that the device is using is not available.
On Android we have a choice between getting the SIM or the actual network MCC & MNC data. The SIM and network code are different when the device is roaming for example.
To be consistent between the platforms, the functions should use the SIM information exclusively on Android.

Thanks for the elaboration. It makes sense


Javascript files references missing from head section of pages

Hello!
Im having some difficulty with this module. The module itself is installed, and appears to be working correctly -- Etherpad is working on its own, and the Notes module configuration page is showing that it is successfully connecting to the API.
Navigating to a notes page within a space results in a page that appears to render properly in all areas except where you would expect content to show.
After noticing that the dropdowns in the nav menu weren't working on notes-related pages, I took a look at the page source and noticed a significant difference between these pages and others within HumHub.
Notes-related pages:
    <!-- start: Meta -->
    <meta charset="utf-8">
    <title>SITE-TITLE</title>
    <!-- end: Meta -->

Other pages:
    <!-- start: Meta -->
    <meta charset="utf-8">
    <link rel="stylesheet" type="text/css" href="/assets/3a39f9a6/jquery.atwho.css" />
<link rel="stylesheet" type="text/css" href="/assets/a8f0f84/mail.css" />
<link rel="stylesheet" type="text/css" href="/assets/2a5e6a63/linklist.css" />
<script type="text/javascript" src="/assets/a45496b6/jquery.js"></script>
<script type="text/javascript" src="/assets/3a39f9a6/jquery.caret.min.js"></script>
<script type="text/javascript" src="/assets/3a39f9a6/jquery.atwho.min.js"></script>
<script type="text/javascript" src="/assets/2a5e6a63/fileuploader.js"></script>
<script type="text/javascript" src="/assets/a8f0f84/mail.js"></script>
<script type="text/javascript" src="/assets/2a5e6a63/spacechooser.js"></script>
<script type="text/javascript" src="/assets/2a5e6a63/linklist.js"></script>
<script type="text/javascript" src="/assets/2a5e6a63/searchmenu.js"></script>
<script type="text/javascript" src="/assets/2a5e6a63/spaceHeaderImageUpload.js"></script>
<script type="text/javascript" src="/assets/a8f0f84/activies.js"></script>
<script type="text/javascript">
/*<![CDATA[*/
 $.ajaxSetup({
			data: {"CSRF_TOKEN": "API-KEY-HERE"},
			cache:false
			});
/*]]>*/
</script>
<title>SITE-TITLE</title>
    <!-- end: Meta -->

In addition, Chrome's console is logging a "$ or jQuery is not defined" message for all other JS references on the notes-related pages that depend on jQuery, so it looks like HEAD section of pages related to the module is missing quite a bit. Any ideas?
Thanks!

i have the same problem - any hint how to solve it?

I'm having the same problem, the Notes module doesn't load if the new Wiki module is activated.  The logs show a PHP fatal error in protected/modules/wiki/WikiModuleEvents.php on line 31 -  "Call to a member function isModuleEnabled() on a non-object" when opening a note. The error messages in Chrome are probably only a result from the interrupted page loading process. Maybe an incompatibility between the Notes module and the Wiki module?

Same problem, any solution ?

The solutions of @sgregori works.


descriptor: digest example link should be SHA-512

Signed-off-by: Stephen J Day<EMAIL_ADDRESS>

LGTM

LGTM


DEV: Adjust build, setup lint

This PR has a lot of changes, but logically

Update the MakeFile to be current: remove unused commands, update to include complete setup and lint
Update settup.py to install the packages.
Remove the semsearch_pkg level so that we can import semsearch and ml_api directly
Formatted code with black. And fixed the rest of the linting errors manually. This is a major source of differences.

Testing and ci will be setup in a separate PR.

Closing, will start a new one for cleaner merge


Update aws-java-sdk-dynamodb to 1.12.504

About this PR
📦 Updates com.amazonaws:aws-java-sdk-dynamodb from 1.11.1034 to 1.12.504
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-dynamodb" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-dynamodb" }
}]

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

Superseded by #467.


Meteor - sergeyt:typeahead - Not injecting when inside an {{#each}}

I have several instances of sergeyt:typeahead working in my webapp. However today I am building a table out of data from query. One of the columns needs to allow for possibly selecting from a collection or if the user wants to enter their own value.
This has worked for me in other parts. Now it appears that when using an {{#each}} statement in the html and when the typeahead is inside the {{#each}} that it does not receive the injecting.
I believe this may be due to the fact that inject is typically done on the rendered which is run before the {{#each}} has run and created the dom elements. And if that is the case how would I go about then placing the inject on these newly generated elements?

@atheriault try to place typeahead input into separate template and init it with Meteor.typeahead in rendered event handler in this template.

AFAIR this is fixed now


feat: Allow passing in custom instance profile role

Description
Allows passing in a custom instance profile role and custom autoscaling role to the service pass role policy.
Also includes minor provider version upgrade https://github.com/hashicorp/terraform-provider-aws/pull/37740.
Motivation and Context
Closes: https://github.com/terraform-aws-modules/terraform-aws-emr/issues/26
Breaking Changes
No.
How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

This PR is included in version 2.3.0 :tada:


Fix cfn-nag violations

./msam-events-release.template
| WARN W58
|
| Resources: ["Collector", "AlarmUpdater"]
| Line Numbers: [-1, -1]
|
| Lambda functions require permission to write CloudWatch Logs
Failures count: 0
Warnings count: 2
./aws-media-services-application-mapper-release.template
Failures count: 0
Warnings count: 0
./msam-core-release.template
| WARN W58
|
| Resources: ["IncomingCloudwatchAlarm", "UpdateNodes", "UpdateConnections", "UpdateFromTags", "SsmRunCommand", "ProcessSsmRunCommand", "UpdateSsmNodes", "APIHandler"]
| Line Numbers: [-1, -1, -1, -1, -1, -1, -1, -1]
|
| Lambda functions require permission to write CloudWatch Logs
Failures count: 0
Warnings count: 8
./msam-dynamodb-release.template
| WARN W78
|
| Resources: ["Channels", "Events", "Layout", "Settings", "Content", "Alarms", "CloudWatchEvents"]
| Line Numbers: [51, 75, 152, 186, 200, 242, 298]
|

DynamoDB table should have backup enabled, should be set using PointInTimeRecoveryEnabled

WARN W74

Resources: ["Channels", "Events", "Layout", "Settings", "Content", "Alarms", "CloudWatchEvents"]

Line Numbers: [51, 75, 152, 186, 200, 242, 298]

DynamoDB table should have encryption enabled using a CMK stored in KMS

| WARN W58
|
| Resources: ["DefaultSettingsResource"]
| Line Numbers: [6]
|
| Lambda functions require permission to write CloudWatch Logs
Failures count: 0
Warnings count: 15
./msam-browser-app-release.template
| WARN W10
|
| Resources: ["MSAMAppBucketCloudFrontDistribution"]
| Line Numbers: [23]
|

CloudFront Distribution should enable access logging

WARN W70

Resources: ["MSAMAppBucketCloudFrontDistribution"]

Line Numbers: [23]

Cloudfront should use minimum protocol version TLS 1.2

| WARN W58
|
| Resources: ["MSAMWebContentResource", "MSAMWebInvalidationResource"]
| Line Numbers: [164, 214]
|

Lambda functions require permission to write CloudWatch Logs

WARN W35

Resources: ["MSAMBrowserAppBucket"]

Line Numbers: [146]

S3 Bucket should have access logging configured

| WARN W41
|
| Resources: ["MSAMBrowserAppBucket"]
| Line Numbers: [146]
|
| S3 Bucket should have encryption option set
Failures count: 0
Warnings count: 6
./msam-iam-roles-release.template
| FAIL F5
|
| Resources: ["InstallationManagedPolicy"]
| Line Numbers: [278]
|

IAM managed policy should not allow * action

WARN W13

Resources: ["InstallationManagedPolicy"]

Line Numbers: [278]

IAM managed policy should not allow * resource

| FAIL F39
|
| Resources: ["InstallationPolicy"]
| Line Numbers: [243]
|

IAM policy should not allow * resource with PassRole action

FAIL F4

Resources: ["InstallationPolicy"]

Line Numbers: [243]

IAM policy should not allow * action

| WARN W12
|
| Resources: ["InstallationPolicy"]
| Line Numbers: [243]
|

IAM policy should not allow * resource

WARN W11

Resources: ["EventsRole", "DynamoDBRole", "CoreRole", "WebRole"]

Line Numbers: [6, 47, 82, 205]

IAM role should not allow * resource on its permissions policy

| WARN W76
|
| Resources: ["CoreRole"]
| Line Numbers: [82]
|
| SPCM for IAM policy document is higher than 25
Failures count: 3
Warnings count: 7

Several '*' resource issues required suppressing in the IAM template. That template is general and applied first by compartmentalized groups before actual resource ARNs are known from the following templates.


Public files don’t get served until dev server restarts for the first time

(Crosspost from https://forums.meteor.com/t/flow-router-public-files-dont-get-served-until-dev-server-restarts-for-the-first-time/39763)
I have a simple app that uses Flow-Router. In my public directory, there are some files that I'd like to be served. This seems to work always when running meteor without flow router. However, when I start the dev server with my route definitions, the static files are not served at all and result in a 404 no matter how many times I clear the cache and refresh. I have to edit some file to trigger a server restart before it works as expected.
Is there something I can do to debug what's going on? Serving public files is supposed to work out of the box with flow router, right?
Here's my routes.js:
import { FlowRouter } from 'meteor/kadira:flow-router';
import { BlazeLayout } from 'meteor/kadira:blaze-layout';

import '../../ui/layouts/app-body.js';
import '../../ui/pages/index-page.html';
...
import '../../ui/layouts/not-found-body.html'

FlowRouter.route('/', {
  name: 'home',
  action() {
    BlazeLayout.render('appBody', { main: 'indexPage' });
  },
});

// ...
// Two more basic routes that have nothing to with /public or similar
// ...

FlowRouter.notFound = {
  action() {
    BlazeLayout.render('notFoundBody');
  },
};

Other info: The routes are defined similarly to the newest TODO example in /imports/startup/client/routes.js and get imported early on the client-side.
I've used Meteor 1.5 but just upgraded to 1.6-rc.5 and the issue still persists.

I have the similar issue. But my app crash when i`m trying ti pass any FlowRouter params, like 'id'
FlowRouter.route('/editproduct/:id', {
name: 'editProduct',
action: function(params) {
BlazeLayout.render('pageContent', {wrapper: 'editProduct' });
}
});
without FlowRouter param '/:id' my public files loads correctly in template

Solved. In my case the error was in absolute url of  static files (css/bootstrap.min.css) loaded as (/editproduct/css/bootstrap.min.css) - that was wrong. Static files should be imported like this:


scikitlearn_mnist example objective function is accuracy, why goal is set to MINIMIZE in config.json?

change goal to MAXIMIZE in advisor_client/examples/scikitlearn_mnist/config.json.

It is the issue of this example. Actually we should minimize the minus accuracy.


WIP: Bump Sentry JavaScript 7.1.1

[x]  Build pass
[x]  Test pass
[x]  Validate on Android
[x]  Validate on iOS
[x] Update changelog

Is this still WIP? it's not draft

Adding @jennmueng in case he wants to review. I just skimmed, are there any breaking changes?

Adding @jennmueng in case he wants to review. I just skimmed, are there any breaking changes?

Yes, the most noticeable one was the new way of writing the level of breadcrumbs.
On React Native we added the following changelog:
## Breaking changes

By bumping Sentry Javascript, new breaking changes were introduced, to know more what was changed, check the [breaking changes changelog](https://github.com/getsentry/sentry-javascript/blob/7.0.0/CHANGELOG.md#breaking-changes) from Sentry Javascript.

this may be enough for showing the break changes?
additionally, just like in RN, I'd like to release this version as a beta.

We're on version 0.x.x so we should be fine with that note.
We plan to merge this on July 4th once Lucas is back online

Any updates on this? Is there an ETA for a new version? Thanks in advance!

@bruno-garcia  can we release this as an beta?


Shader TIME in editor

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Ubuntu 17.04 - Godot v3.0.alpha.custom_build.3ae1cff
I don't think it's desirable to have the TIME variable behave in editor as I'm game. There's two options either turn off TIME in editor or have a way to reset the time to 0 through a button or something. I think 2nd option would be better for users

Something also annoying is the fact that, if you prototype a shader using TIME, you just can't use "update changes" mode, because the shader will animate only when you do things in the editor, so it's better to use "update always".

Note: The feature described here was implemented in https://github.com/godotengine/godot/pull/31070, but it wasn't merged.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Line charts not displaying tooltips

Bar and pie charts show data points on mouseover, but line charts do not seem to be. This is seen locally and on https://marianogappa.github.io/chart/
Tooltips are referenced at https://github.com/marianogappa/chart/blob/master/line.go#L107
Are line chart tooltips currently supported?

I haven't spent much time on tooltips yet; they may be inconsistent between chart types or broken. We just haven't had the use case yet, and generally coding things we don't test leads to buggy code.
I'm glad to see that you're trying chart! Can you describe your use case? What would you expect exactly? I might get to work on it this weekend.
Cheers!

Can you describe your use case?

Simple timeseries trending, usually the output of an Awk script that counts things on a time or date stamp:
2016-01-01, 100
2016-01-02, 105
2016-01-03, 95

This utility is a bit simpler to use than gnuplot and R/ggplot, and the tooltips in the chart formats where supported are useful.  I'm looking at line.go to see if I can figure out how to port them from bar.go or scatter.go

@kmatt my bad; the lack of tooltips was a bug. Fixed in the above commit.
This new release has the fix on it
https://github.com/marianogappa/chart/releases/tag/v1.0.2
Tooltips now appear
https://marianogappa.github.io/chart/
Regardless of the bug being fixed, is the feature satisfying your use case? Thanks in advance for the review. Let me know if you find any other use cases chart is not enabling for you.
If approved, please close the Issue. I will close after some time of inactivity.

Yes, those work well!


The font of the choice isn't good looking

The font of the choice isn't good looking.
Because I think the font of the choice is ugly.
Choose the other font.

Please find some examples of fonts that would look better and paste their link, name, and a picture of them here, Darren.
@Darren20080511


Unable to upgrade Az version to latest on microsoft hosted Windows agents

Issue: Unable to update Azure CLI to 2.37.0.
Scenario: Using Azure pipelines, tried to upgrade the Az cli version to the latest with the below script.
az upgrade --all --yes -y
The script runs for well over 40 mins and is stuck in installing msi stage.
Kindly could you help on upgrading the script on the Microsoft hosted machines.

@jiasli for awareness

This is similar to https://github.com/Azure/azure-cli/issues/18363 but not exactly the same since you are using Windows agent.
az upgrade currently doesn't work in Azure DevOps agent.
Solutions
You may either

Wait for Azure DevOps agent to upgrade to the latest Azure CLI by itself (no action from your side)
Install Azure CLI from the official source: https://docs.microsoft.com/en-us/cli/azure/install-azure-cli-windows?tabs=azure-cli

hi.. I know there was no solution a few months back, but is there another workaround for this aside from waiting for the host to update itself? getting security advise from azure to upgrade to the latest cli version.


Unable to get deluge to work after a system crash

Ok I had this problem in the past that I solved but this fix don't seem to work anymore :(
Basically MY main SSD failed so I had to do a full system reinstall, but for the life of me I ca not get deluge to run the script everything else and I mean everything else is working as intended but deluge doesn't, I used python 3 instead of 2.7 could this be the problem?

Ok it seems its python 3 as I downgraded to 2.7 reinstalled all the "pip" bits re-downloaded a video and it worked first time

Python 3 should work. What errors were you getting?
Sent from my iPhone

On Oct 12, 2017, at 14:59, morpheus2n2<EMAIL_ADDRESS>wrote:
Ok it seems its python 3 as I downgraded to 2.7 reinstalled all the "pip" bits re-downloaded a video and it worked first time
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

This is all I get, the CMD windows pops up for like 5 seconds before vanishing.
Where as when I went back to Pyhon 2.7 the windows pops up and stays until its finished.

2017-10-12 18:24:57 - delugePostProcess - INFO - Deluge post processing started.
2017-10-12 18:24:57 - readSettings - INFO - C:\Users\morph\AppData\Local\Programs\Python\Python36-32\python.exe
2017-10-12 18:24:58 - delugePostProcess - ERROR - Not enough command line parameters present, are you launching this from deluge?
2017-10-12 18:25:50 - delugePostProcess - INFO - Deluge post processing started.
2017-10-12 18:25:50 - readSettings - INFO - C:\Users\morph\AppData\Local\Programs\Python\Python36-32\python.exe
2017-10-12 18:25:50 - delugePostProcess - ERROR - Not enough command line parameters present, are you launching this from deluge?
2017-10-12 18:55:49 - delugePostProcess - INFO - Deluge post processing started.
2017-10-12 18:55:49 - readSettings - INFO - C:\Users\morph\AppData\Local\Programs\Python\Python36-32\python.exe
2017-10-12 18:55:50 - delugePostProcess - INFO - Successfully connected to Deluge

Looks like your command line parameters aren't getting sent to python
delugePostProcess - ERROR - Not enough command line parameters present
I would check and make sure you have python 3 appropriately added to your system path so that it can receive command line parameters
https://stackoverflow.com/questions/2640971/windows-is-not-passing-command-line-arguments-to-python-programs-executed-from-t

Sorry for the long delay, I have fixed this issue now, I uninstalled and removed all traces of Python from my System and started over and its now working fine, All I can think is there was a conflict or missing file somewhere they throw a spanner in the works.


[Doc]: OpenAI Server Command Line Args Broken

📚 The doc issue
In v0.4.1 (and before), there was full documentation for params:
https://docs.vllm.ai/en/v0.4.1/serving/openai_compatible_server.html#command-line-arguments-for-the-server
In v0.4.2 and master, its empty:
https://docs.vllm.ai/en/v0.4.2/serving/openai_compatible_server.html#command-line-arguments-for-the-server
Suggest a potential alternative/fix
No response

actually it doesn't fixes things.. yet...

actually it doesn't fixes things.. yet...

When I built the docs locally, the CLI arguments did show up. Not sure what is different in the CI environment.

https://github.com/vllm-project/vllm/pull/4729
https://readthedocs.org/projects/vllm/builds/24329938/ '"python -m sphinx -T -b html -d _build/doctrees -D language=en . $READTHEDOCS_OUTPUT/html" depends on openai package,  when you uninstall openai package in your local env and run the command above, the problem occurred.


Error with cross-fetch in postgres-js & supabase-js

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
I just upgraded to the latest Supabase-js package 2.33.1 when I try to use the api,
Uncaught (in promise) SyntaxError: The requested module '/node_modules/.pnpm/cross-fetch@3.1.8/node_modules/cross-fetch/dist/browser-ponyfill.js?v=bd2992b8' does not provide an export named 'default' (at PostgrestBuilder.ts:1:8)
I uninstalled node_modules and tried to reinstall them and I still get the error, all when running supabase-js
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Go to '…'
Click on '…'
Scroll down to '…'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

System information
System:
OS: macOS 13.5
CPU: (8) arm64 Apple M1 Pro
Memory: 185.33 MB / 16.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 18.14.2 - ~/.nvm/versions/node/v18.14.2/bin/node
npm: 9.5.0 - ~/.nvm/versions/node/v18.14.2/bin/npm
Browsers:
Brave Browser: <IP_ADDRESS>
Chrome: 116.0.5845.110
Safari: 16.6
npmPackages:
@builder.io/qwik: ^1.2.8 => 1.2.8
@builder.io/qwik-city: ^1.2.8 => 1.2.8
@supabase/supabase-js: ^2.33.1 => 2.33.1
undici: 5.22.1 => 5.22.1
vite: 4.3.5 => 4.3.5
Additional context
Add any other context about the problem here.

With nextjs same issue please review and correct possible import errors

Seems like an issue on cross-fetch: https://github.com/lquixada/cross-fetch/issues/120#issuecomment-1617172860
Can you run npm up @supabase/postgrest-js and see if the error changes? We're about to replace cross-fetch with using node-fetch directly which can possibly fix this.

@soedirgo - I'm also seeing this runtime error with the Vite framework.
What logs do you require exactly? Just a package-lock or something more specific?

To elaborate - I've got @supabase/supabase-js in a submodule that the main project is importing.
This submodule works fine when linked instead of installed which may suggest the issue is one of the pre-deps for @supabase/supabase-js not importing the updated version of node-fetch?

Alright after some hair pulling I got this to play nice with Vite.
It looks like some of the modules need explicit aliasing otherwise Vite / ESBuild / Rollup seems to get confused and import the wrong file.
Merge this into vite.config.js:
import { defineConfig } from 'vite';
export default defineConfig(({mode}) => ({
	// ... presumably other existing config ... //
	resolve: {
		alias: {
			'cross-fetch': 'cross-fetch/dist/browser-ponyfill.js',
			'@supabase/node-fetch': <EMAIL_ADDRESS>			'websocket': 'websocket/index.js'
		},
	},
}));

@hash-bang I'm referring to the Import trace for requested module logs above - the actual error messages should be above it.
Do you have a repo where I could reproduce this issue? I'd like to make this work OOtB if possible.

@soedirgo after playing around a bit it looks like this only seems to be a problem when Supabase is a sub-dependency (i.e. project imports NPM which then imports Supabase).
I've set up a simple demo of us using Supabase here its using our in house wrapping NPM Vitel which gives us a thin wrapper around Supabase to handle some company specific things.

I don't see any relevant warnings in that demo. Am I missing a step?

In any case, does upgrading to supabase-js v2.33.2 help?

I assume you're doing the "Open in new Tab" thing then opening the console there?
If so that's, odd because its the first error that shows up for me:

One question though - if I'm right in assuming that node-fetch is the backend wrapper to ponyfill the fetch() API, is there a reason its not just falling back to using the native version for frontend imports if its already present?

I'm having this problem too, with a Vite build. When we were still on cross-fetch, the alias hack worked for me. But it no longer does with @supabase/node-fetch.

@mikestopcontinues please provide a GitHub repo with the reproduction of this issue. I tested the one @hash-bang provided above and cannot replicate the issue he is having either. It's also harder to download files off Stackblitz to run them locally to dig into the issue further.

@silentworks - apologies, I didn't know you had issues downloading the project.
Steps to reproduce:

Download the Stackblitz export
Run npm run dev
Navigate to page with the devtools console open to see the above error with node-fetch


ASAP-Alchemy: CDD downloading

Description
This PR adds an interface to query the CDD vault for experimental data using the web API. This will be used when making predictions based on FECs.
Todos
Notable points that this PR has either accomplished or will accomplish.

[ ] Provide an easy interface to pull CDD data
[ ] Add multiprocessing to do many requests at once
[ ] expose pulling from the CDD on the predict CLI

Questions

[ ] Question 1 ..

Status

[ ] Ready to go

Developers certificate of origin

[ ] I certify that this contribution is covered by the MIT license as defined in our LICENSE and adheres to the Developer Certificate of Origin.

Codecov Report

Merging #820 (7bf1ab0) into main (1e83214) will increase coverage by 20.31%.
Report is 18 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files

Im gonna merge by hand here, as the failing RTD build will block the merge queue


BUG -psycopg error

Description of the issue
We get import error of psycopg on AWS lambda. I initially thought it was due to imap-data-access version0.10.0. Initial debugging pointed to that because I tried creating new lambda layer with imap-data-access version 0.7.0 through 0.10.0 and deploy the CDK each time with new layer. All versions worked besides version 0.10.0. Therefore, I thought that was the issue.
Then to test why version 0.10.0 was causing the issue, I created a local working branch. I did not make any change to imap-data-access beside updating version to 0.10.1rc2 on it. Then I tried creating package locally using this command python -m build --sdist and then copying the zip file to sds-data-manager/lambda_layer/python and update lambda_layer_construct.py to pip install that independently like this
        code_bundle = lambda_.Code.from_asset(
            layer_dependencies_dir,
            bundling=cdk.BundlingOptions(
                image=runtime.bundling_image,
                command=[
                    "bash",
                    "-c",
                    (
                        "pip install -r requirements.txt -t /asset-output/python && "
                        "pip install imap_data_access-0.10.1rc2.tar.gz -t /asset-output/python && "
                        "cp -au . /asset-output/python"
                    ),
                ],
            ),
        )

        super().__init__(
            id=f"{id}-Layer", code=code_bundle, compatible_runtimes=[runtime], **kwargs
        )

When I deployed with above changes through CDK, the issue went away on AWS lambda.
But when everything is deployed through requirements.txt and through GitHub deploy action, it raises this issue on AWS lambda. I am not sure now what the issue is.
Steps to reproduce the issue
Deploy CDK to AWS through GitHub Actions or locally from this snapshot of sds-data-managercommit
Expected behavior (What should happen)
No error
Actual behavior (What does happen)
{
"errorMessage": "No module named 'psycopg2._psycopg'",
"errorType": "ModuleNotFoundError",
"requestId": "150f8d02-8a1c-41c2-bd29-11a9f79778ec",
"stackTrace": [
"  File "/var/task/SDSCode/create_schema.py", line 29, in lambda_handler\n    Base.metadata.create_all(db.get_engine())\n",
"  File "/var/task/SDSCode/database/database.py", line 27, in get_engine\n    return create_engine(db_uri)\n",
"  File "", line 2, in create_engine\n",
"  File "/opt/python/sqlalchemy/util/deprecations.py", line 281, in warned\n    return fn(*args, **kwargs)  # type: ignore[no-any-return]\n",
"  File "/opt/python/sqlalchemy/engine/create.py", line 599, in create_engine\n    dbapi = dbapi_meth(**dbapi_args)\n",
"  File "/opt/python/sqlalchemy/dialects/postgresql/psycopg2.py", line 690, in import_dbapi\n    import psycopg2\n",
"  File "/opt/python/psycopg2/init.py", line 51, in \n    from psycopg2._psycopg import (                     # noqa\n"
]
}
Code Snippet:
Code

Additional notes, affected areas, and suggested fixes
Files and resources that caused this issue could be these:

imap-data-access - version 0.10.0
mismatch of psycopg = "^3.1" and psycopg2-binary = "^2.9.9" in pyproject.toml file
GitHub workflow yaml not installing dependency for psycopg

I don't what the real issue is still. This issue was fixed by re-releasing imap-data-access version 0.10.0 to 0.10.1 https://github.com/IMAP-Science-Operations-Center/imap-data-access/releases/tag/v0.10.1


Reverse Polish Notation

Is your feature request related to a problem? Please describe.
Reverse Polish Notation also known as postfix notation
1.) Create an empty stack to hold operands.
2.) Read the expression from left to right.
3.) If the element is an operand, push it onto the stack.
4.) If the element is an operator, pop the required number of operands from the stack, perform the operation, and push the result back onto the stack.
5.) After reading the entire expression, the final result is left on the stack.

@Kumar-laxmi please assign this to me under SSOC'23. I'm ready to contribute in all 4 languages

#include
#include
#include
using namespace std;
int evaluateRPN(const string& expression) {
stack operands;
stringstream ss(expression);
string token;

while (getline(ss, token, ' ')) {
    if (isdigit(token[0])) {

        operands.push(stoi(token));
    } else {

        int operand2 = operands.top();
        operands.pop();
        int operand1 = operands.top();
        operands.pop();

        if (token == "+") {
            operands.push(operand1 + operand2);
        } else if (token == "-") {
            operands.push(operand1 - operand2);
        } else if (token == "*") {
            operands.push(operand1 * operand2);
        } else if (token == "/") {
            operands.push(operand1 / operand2);
        }
    }
}

return operands.top();

}

Use this code
#include
#include
#include
#include
#include  // For isdigit
using namespace std;
int evaluateRPN(const string& expression) {
stack operands;
stringstream ss(expression);
string token;
while (getline(ss, token, ' ')) {
    if (isdigit(token[0]) || (token.size() > 1 && token[0] == '-' && isdigit(token[1]))) {
        operands.push(stoi(token));
    } else {
        int operand2 = operands.top();
        operands.pop();
        int operand1 = operands.top();
        operands.pop();

        if (token == "+") {
            operands.push(operand1 + operand2);
        } else if (token == "-") {
            operands.push(operand1 - operand2);
        } else if (token == "*") {
            operands.push(operand1 * operand2);
        } else if (token == "/") {
            operands.push(operand1 / operand2);
        }
    }
}

return operands.top();

}

Fix some bugs

@rajchaudhary99 If you would like to contribute to this issue, you can raise a new issue by mentioning the programming languages in which you would like to work in.


Fix activerecord 8.0 version

coverage: 97.065%. remained the same
when pulling 1d256700d3d08cf88eef2f02b4a315b63f61b330 on fix_ar8_version
into 8506d202ef28b65f048b4de8cc88fc4f7e029622 on 3.0.


No voice recorded in Whatsapp or Telegram services

I am trying to send a voice message with Whatsapp,  the app seems to record but sends an empty voice message. There is no Microphone setting for Franz allowed on the Mac OS System Preferences>Security.
Am I doing something wrong or is this a bug?

I also face the same problem before. But then i found the solution to enable the sound from
Right click the service --> Edit Page --> Enable Audio


Corrected key of Deployment Server Version

@EricSmekens : Thanks for your contribution! The author, @minewiskan, has been notified to review your proposed change.

#sign-off


[Rust Planner] part 1: devbox shell

Summary
This PR adds basic support for Rust, focussing on devbox shell. The next PR will support devbox build.
Rust nix package
It relies on the rustup nix-package, which is the recommended tool for managing the rust toolchain of various versions. In particular, amongst other tools, rustup manages the compiler (rustc) and package manager (cargo).
It's likely there are more sophisticated approaches to doing this, but for now this should suffice. For reference, other sources to look at are:

Rust Nix wiki
Rust section of Nix Packages Manual, in particular the Overlays part.
User Blog Post, which also mentions leveraging a Rust Overlay.

Shell.InitHook
To respect the rust-version in Cargo.toml, I need to call rustup default <version> as a shell.InitHook. However, this does mean that a user specifying a shell init hook in devbox.json will entirely override the Planner's inithook. The fix seems to be to have the planner set a Shell.PreInitHook. So, I implemented this PreInitHook. WDYT?
How was it tested?
adding cases to testdata/ for rust stable (currently, version 1.63.0) and version 1.62.0
devbox shell
> go install ./cmd/install

> cd testdata/rust/rust-stable
> devbox shell
> rustc --version
rustc 1.63.0 (4b91a6ea7 2022-08-08)

> cd testdata/rust/rust-1.62.0
> devbox shell
Installing nix packages. This may take a while...
Starting a devbox shell...
info: using existing install for '1.62.0-x86_64-apple-darwin'
info: default toolchain set to '1.62.0-x86_64-apple-darwin'

  1.62.0-x86_64-apple-darwin unchanged - rustc 1.62.0 (a8314ef7d 2022-06-27)

(devbox)
> rustc --version
rustc 1.62.0 (a8314ef7d 2022-06-27)
(devbox)

Overriding the shellhook by user in devbox.json
> cd testdata/rust/rust-1.62.0-custom-shell-inithook
> devbox shell
Installing nix packages. This may take a while...
Starting a devbox shell...
info: using existing install for '1.62.0-x86_64-apple-darwin'
info: default toolchain set to '1.62.0-x86_64-apple-darwin'

  1.62.0-x86_64-apple-darwin unchanged - rustc 1.62.0 (a8314ef7d 2022-06-27)

hello world
(devbox)
> rustc --version
rustc 1.62.0 (a8314ef7d 2022-06-27)

Notice that the Rust version is correct, AND the user's init hook ("hello world") ran. This was possible because of the Shell.PreInitHook

Current dependencies on/for this PR:

main

PR #92

PR #97   👈

This comment was auto-generated by Graphite.

Spoke to @loreto. May explore using the Oxalica overlay instead.

Moving back to draft until ready for re-review.


Circular Links

Hi, ... Got the Private Pages Info, ... Thanks.
Does the graph navigation handle Circular Links between pages.
When I run the app, ... it just runs and runs.   It never finishes.

Do you mind sharing your code? I have tested with a test page where I created circular links between two pages, but it still works. You also might want to juggle some optional configs defined by Typescript.
Here's my page that acts as the root node: https://lizard-flax-e38.notion.site/Test-bc95525429244d169cad7f6725cb5b34?pvs=4
Running the scraper gives me the following result, without hanging:
{
  "nodes": [
    {
      "id": "bc955254-2924-4d16-9cad-7f6725cb5b34",
      "title": "Test",
      "type": "page",
      "spaceId": "fda5cc70-a0d3-4f12-9b33-2ec0c26ab6ca",
      "parentId": "none",
      "cc": 2
    },
    {
      "id": "65786275-82a8-4c32-8bd7-11d49a3ff7c6",
      "parentId": "bc955254-2924-4d16-9cad-7f6725cb5b34",
      "spaceId": "fda5cc70-a0d3-4f12-9b33-2ec0c26ab6ca",
      "type": "page",
      "title": "A"
    },
    {
      "id": "175b9592-7cf5-4452-be34-da896731654b",
      "parentId": "bc955254-2924-4d16-9cad-7f6725cb5b34",
      "spaceId": "fda5cc70-a0d3-4f12-9b33-2ec0c26ab6ca",
      "type": "page",
      "title": "B"
    }
  ],
  "links": [
    {
      "source": "65786275-82a8-4c32-8bd7-11d49a3ff7c6",
      "target": "bc955254-2924-4d16-9cad-7f6725cb5b34"
    },
    {
      "source": "65786275-82a8-4c32-8bd7-11d49a3ff7c6",
      "target": "175b9592-7cf5-4452-be34-da896731654b"
    },
    {
      "source": "175b9592-7cf5-4452-be34-da896731654b",
      "target": "bc955254-2924-4d16-9cad-7f6725cb5b34"
    }
  ],
  "errors": []
}

I used the code as is from the info you gave me, .... the page id is the blog.
And I used the project from the example scapper and replaced index.js with this code
import { NotionGraph } from "@graphcentral/notion-graph-scraper"
import fs from "fs"
import { NotionAPI } from "notion-client";
/**

example of how to use @graphcentral/notion-graph-scraper
*/
;(async () => {
const notionApi = new NotionAPI({
activeUser: process.env.NOTION_ACTIVE_USER,
authToken: process.env.NOTION_TOKEN_V2
})
const notionGraph = new NotionGraph({
maxDiscoverableNodes: 10,
maxDiscoverableNodesInOtherSpaces: 10,
verbose: true,
unofficialNotionAPI: notionApi
})
const graph = await notionGraph.buildGraphFromRootNode(
// Some random Japanese blog
'95fcfe03257541c5aaa21dd43bdbc381'
)
console.log(graph.nodes.length)
console.log(graph.links.length)
await new Promise((resolve, reject) => {
fs.writeFile(test0.json, JSON.stringify(graph), (err) => {
if (err) reject(err)
else resolve(``)
})
});

process.exit(0)
})()

@jshingler but you configured process.env.NOTION_ACTIVE_USER and process.env.NOTION_TOKEN_V2 right?
what is the output when you run it?

and the japanese blog should be accessible without using the custom notionApi argument because it's already public

Yes I Configured the environment variables, ....
I am not using the Japanese Blog, ... I just didn't want to expose my private page id
Here is  a portion of the output, it just keeps repeating
[12/4/2023, 9:44:24 PM] # total successful requests: 0
[12/4/2023, 9:44:24 PM] # current requests: 0 / # items in the queue: 0
[12/4/2023, 9:44:24 PM] # total requests sent: 0
[12/4/2023, 9:44:24 PM] # total successful requests: 0
[12/4/2023, 9:44:24 PM] # current requests: 0 / # items in the queue: 0
[12/4/2023, 9:44:24 PM] # total requests sent: 0
[12/4/2023, 9:44:24 PM] # total successful requests: 0
[12/4/2023, 9:44:24 PM] # current requests: 0 / # items in the queue: 0
[12/4/2023, 9:44:24 PM] # total requests sent: 0
[12/4/2023, 9:44:24 PM] # total successful requests: 0
[12/4/2023, 9:44:24 PM] # current requests: 0 / # items in the queue: 0
[12/4/2023, 9:44:24 PM] # total requests sent: 0
[12/4/2023, 9:44:24 PM] # total successful requests: 0
[12/4/2023, 9:44:24 PM] # current requests: 0 / # items in the queue: 0
[12/4/2023, 9:44:24 PM] # total requests sent: 0
[12/4/2023, 9:44:24 PM] # total successful requests: 0
[12/4/2023, 9:44:24 PM] # current requests: 0 / # items in the queue: 0
[12/4/2023, 9:44:24 PM] # total requests sent: 0
[12/4/2023, 9:44:24 PM] # total successful requests: 0
[12/4/2023, 9:44:24 PM] # current requests: 0 / # items in the queue: 0
[12/4/2023, 9:44:24 PM] # total requests sent: 0
[12/4/2023, 9:44:24 PM] # total successful requests: 0
[12/4/2023, 9:44:24 PM] # current requests: 0 / # items in the queue: 0
[12/4/2023, 9:44:24 PM] # total requests sent: 0
[12/4/2023, 9:44:24 PM] # total successful requests: 0
[12/4/2023, 9:44:24 PM] # current requests: 0 / # items in the queue: 0
[12/4/2023, 9:44:24 PM] # total requests sent: 0
[12/4/2023, 9:44:24 PM] # total successful requests: 0
[12/4/2023, 9:44:24 PM] # current requests: 0 / # items in the queue: 0
[12/4/2023, 9:44:24 PM] # total requests sent: 0
[12/4/2023, 9:44:24 PM] # total successful requests: 0
[12/4/2023, 9:44:24 PM] # current requests: 0 / # items in the queue: 0
[12/4/2023, 9:44:24 PM] # total requests sent: 0
[12/4/2023, 9:44:24 PM] # total successful requests: 0
[12/4/2023, 9:44:24 PM] # current requests: 0 / # items in the queue: 0
[12/4/2023, 9:44:24 PM] # total requests sent: 0
[12/4/2023, 9:44:24 PM] # total successful requests: 0
[12/4/2023, 9:44:24 PM] # current requests: 0 / # items in the queue: 0
[12/4/2023, 9:44:24 PM] # total requests sent: 0
[12/4/2023, 9:44:24 PM] # total successful requests: 0
[12/4/2023, 9:44:24 PM] # current requests: 0 / # items in the queue: 0
[12/4/2023, 9:44:24 PM] # total requests sent: 0
[12/4/2023, 9:44:24 PM] # total successful requests: 0
[12/4/2023, 9:44:24 PM] # current requests: 0 / # items in the queue: 0
[12/4/2023, 9:44:24 PM] # total requests sent: 0
[12/4/2023, 9:44:24 PM] # total successful requests: 0

Yeah if it keeps showing zeroes, it shouldn't be about circular links. There should be something about not being able to retrieve the information. Let me try with my own private pages and get back to you

Looking at this again tonight, ... here is the error message that shows up in test0.json.
d.map is not a function.  in notion-utils/build/index.js
{"links":[],"nodes":[],"errors":[{"name":"TypeError","message":"d.map is not a function","stack":"TypeError: d.map is not a function\n    at s (file:///Users/jshingler/Archive/Workspaces/graphcentral/notion-scrape-example/node_modules/notion-utils/build/index.js:1:4072)\n    at s (file:///Users/jshingler/Archive/Workspaces/graphcentral/notion-scrape-example/node_modules/notion-utils/build/index.js:1:4308)\n    at s (file:///Users/jshingler/Archive/Workspaces/graphcentral/notion-scrape-example/node_modules/notion-utils/build/index.js:1:4308)\n    at s (file:///Users/jshingler/Archive/Workspaces/graphcentral/notion-scrape-example/node_modules/notion-utils/build/index.js:1:4308)\n    at s (file:///Users/jshingler/Archive/Workspaces/graphcentral/notion-scrape-example/node_modules/notion-utils/build/index.js:1:4308)\n    at s (file:///Users/jshingler/Archive/Workspaces/graphcentral/notion-scrape-example/node_modules/notion-utils/build/index.js:1:4308)\n    at s (file:///Users/jshingler/Archive/Workspaces/graphcentral/notion-scrape-example/node_modules/notion-utils/build/index.js:1:4308)\n    at s (file:///Users/jshingler/Archive/Workspaces/graphcentral/notion-scrape-example/node_modules/notion-utils/build/index.js:1:4308)\n    at s (file:///Users/jshingler/Archive/Workspaces/graphcentral/notion-scrape-example/node_modules/notion-utils/build/index.js:1:4308)\n    at s (file:///Users/jshingler/Archive/Workspaces/graphcentral/notion-scrape-example/node_modules/notion-utils/build/index.js:1:4308)"},{}]}

It works on VERY simple private pages, ....  Seems to be React-Notion-X has issues with something on the more complex pages.


Is there a plan for null-safety on yaml_edit?

Is there a plan? Would love to use it.
I can also help if someone can point me in the right direction. I started doing this on a fork on my own but I got stuck on deepEqualsMap() on equality.dart and didn't know how to deal with mismatching types. I'm fairly new to Dart and generic programming in general.

+1

@asartalo we are definitely going to migrate it -- not sure quite when :D
But if you're interested in working on it, please do get started, feel free to file a WIP pull-request and ping me for feedback..
Otherwise, I'm sure we'll do it eventually as it's the same code we have in the pub client :D

Fixed and published as yaml_edit 2.0.0


Stackoverflow when using transactions_unordered(n) on subscribe_pending_txs

Version
v0.17.0
Platform
Darwin MacBook-Pro.local 21.6.0 Darwin Kernel Version 21.6.0: Mon Aug 22 20:19:52 PDT 2022; root:xnu-8020.140.49~2/RELEASE_ARM64_T6000 arm64
Description
I am trying to handle each pending tx in its own thread using tokio. When used below code, its running for sometime and then it get aborted with stack overflow error like
thread 'tokio-runtime-worker' has overflowed its stack
fatal runtime error: stack overflow

Note: Getting same issue, even when tried with n=5 in transactions_unordered(n)
I tried this code:
let mut tx_stream = client.subscribe_pending_txs().await?.transactions_unordered(10);
while let Some(tx) = tx_stream.next().await {
    if let Ok(pending_tx) = tx {
        tokio::spawn(async move {
            Self::handle_transaction(&pending_tx).await;
        });
    }
}

can you please post the entire code for client and your ethers dependency from cargo.toml?

I have figured it out. The issue was not with transactions_unordered(n) function from ethers-rs. It was due to the tokio library in which i was spawning a thread for handle_transaction and due to recursive function call in handle_transaction, it was overflowing its stack. I didn't handled recursion properly. Its a bug in my code not in ethers-rs and Its fixed now, hence closing it.


Updated instructions for pyusbiss 0.2.0

Changed the instructions for the upcoming release of 0.2.0 of pyusbiss.
Please test the HEAD of pyusbiss and report any issues before accepting this PR.
If all goes well then I will release 0.2.0 and you can accept the PR.

Coverage remained the same at 16.541% when pulling 0dc481cd25f9ef0e89dbb19f112302ace0ef0a06 on DancingQuanta:usbiss into 30b12c95be18d687e6ab713e9d3e3e7d1c1f2a4d on dhhagan:master.

Hey @DancingQuanta This may take me a while to check, as the only SPI-USB adapter I had was lent to the people doing the JOSS review. I'll try and track down another one, but no guarantees. If someone else has one and can check the PR, I would be happy to merge.

Okay, I will put a notice out in the issues that people raised during my non-version phase.

Update. I found another one at home and thus will try and get this done tomorrow as soon as this blizzard rolls through!


Update build_and_test.yml permissions

Add read only permissions to github action

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Watchdog timer causes issues when updating code via Android Studio

I'm mentoring a couple teams this season, and both have run into problems when updating their code with Android Studio, where the Robot Controller app takes a few minutes to launch after updating.
I believe this is caused by a watchdog timer that exists in the Rev Control Hubs, that attempts to restart the RC app if it fails to report that it's alive for 10 seconds. The problem is that it takes roughly 10 seconds for the app to update, so the watchdog tries to restart the app while it's being updated, which causes it to go into some weird error state that can take minutes to recover.
I haven't dug super deep into this myself; I met with someone who has dug into this and explained it to me, so this may be a game of telephone 😉 They found a way to increase the watchdog timer from 10 seconds to 20 seconds, and that completely solved the problem, so that's pretty definitively the cause. I believe the watchdog process is called FtcAccessPointService, and only exists on the Rev Control Hubs (the 4 Google results for "FtcAccessPointService" all point to the Control Hub OS).
IMO the best solution would be to have the watchdog detect when the app is being updated, and disable itself until the update is finished. I'm not sure how difficult that is to implement, so an alternative solution could be to simply increase the timer value. For now, my workaround has been to have the Rev Hardware Client open while working, and force launch the app after updating.

If you are just tweaking values, may I recommend FtcDashboard.....
--Alan
Coach FTC #16072, Quantum Quacks
On Tue, Jan 10, 2023 at 7:59 PM FromenActual @.***>
wrote:

I'm mentoring a couple teams this season, and both have run into problems
when updating their code with Android Studio, where the Robot Controller
app takes a few minutes to launch after updating.
I believe this is caused by a watchdog timer that exists in the Rev
Control Hubs, that attempts to restart the RC app if it fails to report
that it's alive for 10 seconds. The problem is that it takes roughly 10
seconds for the app to update, so the watchdog tries to restart the app
while it's being updated, which causes it to go into some weird error state
that can take minutes to recover.
I haven't dug super deep into this myself; I met with someone who has dug
into this and explained it to me, so this may be a game of telephone 😉
They found a way to increase the watchdog timer from 10 seconds to 20
seconds, and that completely solved the problem, so that's pretty
definitively the cause. I believe the watchdog process is called
FtcAccessPointService, and only exists on the Rev Control Hubs (the 4
Google results for "FtcAccessPointService" all point to the Control Hub OS).
IMO the best solution would be to have the watchdog detect when the app is
being updated, and disable itself until the update is finished. I'm not
sure how difficult that is to implement, so an alternative solution could
be to simply increase the timer value. For now, my workaround has been to
have the Rev Hardware Client open while working, and force launch the app
after updating.
—
Reply to this email directly, view it on GitHub
https://github.com/FIRST-Tech-Challenge/FtcRobotController/issues/555,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAF3Y75CPON3X7ZCUKKRJKDWRYAXHANCNFSM6AAAAAATXN7HNA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I managed to get a log when this issue occurred. We were working on updating an OpMode, and had multiple successful uploads from Android Studio while connected wirelessly with ADB. However we finally experienced the problem again, where the RC app didn't launch for a long time (we stopped waiting after a couple minutes). I opened up Logcat and saved the contents here:
fail_log.txt
I'm not familiar enough to understand everything in the log, but some things I've noticed:

11:13:12 - I believe that's when Android Studio started updating the app.
11:13:22.767 (~10 seconds later) - the FtcAccessPointService detects the RC app hasn't been alive for 10 seconds, and attempts to relaunch it.
11:13:23.864 - an uncaught null pointer is thrown in the Window Manager.

Seems to me the RC app was still being updated when FtcAccessPointService attempted to relaunch it. My guess is that the Window Manager tried to render something that wasn't yet updated, causing the null pointer.  However that doesn't explain why it didn't attempt to restart the app 10 seconds later.
I've also noticed something called FtcRobotControllerWatchdogService. It seems each time we updated the code, that service is force stopped and restart is scheduled in 1000ms. However there's only 1 instance in the log where it appears to start, which is at 11:12:15.923. Curiously, that's right before we encountered the problem with the app not launching, so perhaps that's related? Like I said, I don't fully understand everything here, so I may be off-base.
I tried to recreate the problem with my own laptop, but have been unsuccessful; it's only occurred when my students upload code from their laptops. Mine is relatively new and faster than theirs, so this problem may be more prevalent on older/slower machines.
I hope this is helpful! I'll keep trying to track this down when I can, though time is short with our competitions coming up.

Hi All,
We've been dealing with similar issues. And also have been experiencing weird glitches when it comes to program execution. We found so far that "running" the program once before actually running it eliminates the issues.
It is totally possible that our issues could be unrelated but we can confirm encountering similar upload delays. To reduce that delay we started using scrcpy to remote control the control hub screen to manually launch the app.
Hope this helps,
Lukas
-Mentor/Alum #4511

We found so far that "running" the program once before actually running it eliminates the issues.

I think we've been running or OpMode between updates, and have still encountered this problem. But my memory could be wrong, so I'll try to be aware of whether we're running the OpMode at least once to see whether there's a correlation.

we started using scrcpy to remote control the control hub screen

Haven't heard of this before, I'll have to try it out! It's also possible to launch the app via the Rev Hardware Client, which is what I've had my students use for simplicity.

Little Update: After a unfortunate qualifiers we ended up finding out some cabling issues that may have been part of our issues related to our weird program glitches.

Ok one last update... It turns out that there were a multitude of issues we encountered.

Defective Tamiya connectors on our battery that caused intermittent power outages
Using some function in OpenCV that caused a memory crash after 10 seconds

My suspicion is that as there were more crashes (in our case because of OpenCV bugs) we then ended up with longer wait times because of some exponential backoff timer somewhere, thus leading to longer waiting with uploads after android studio.


Prepare Release for System.ClientModel 1.0.0

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.


Algorithm for choosing what names to display has problems

Choose any dataset and crank the radius out to 50 or 70 ly or more, and ALL the named stars are in the central third or less.
Even if we restrict the central 70 LY to F and G stars, you get a total of 2 names in the outer half.
Oddly, if you grab the 270 LY set, and restrict it to just O and B stars (600 or so) out to 270 LY, the names ARE distributed well.
We need to increase the value of "distance from the center of the plot" by some amount.
On a closely related note, and thus the same issue, I think that "number of names to display" needs to be a value in settings.

number of names to display is in the settings.
Check the stars preferences section.

Ahhh.  I missed that

But we still need to tweak the distance component somehow to avoid the "all the stars cram up in the middle" problem.

Max # of stars to display has been  limit in BOTH  of the star display algorithms I put forth. I even indicated how many CHV showed, and why.
I also explicitly indicated that the only reason I did not  declare a hard limit was b/c that was putting the cart before the horse. But there HAS to be a limit.
Consider searching for G  V stars and then displaying. The are ALL going to have extremely high values.
20 or 30 tops has been my ballpark suggested beginning limit/total of diplayed star names (  see the original spreadsheets I sent ) from the start. But there was no way of knowing if that was optimal before seeing it.
Sent from my Verizon, Samsung Galaxy smartphone
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Rick Boatright @.>
Sent: Sunday, March 21, 2021 1:12:55 AM
To: ljramones/trips @.>
Cc: Subscribed @.***>
Subject: [ljramones/trips] Algorithm for choosing what names to display has problems (#262)
Choose any dataset and crank the radius out to 50 or 70 ly or more, and ALL the named stars are in the central third or less.
Even if we restrict the central 70 LY to F and G stars, you get a total of 2 names in the outer half.
Oddly, if you grab the 270 LY set, and restrict it to just O and B stars (600 or so) out to 270 LY, the names ARE distributed well.
We need to increase the value of "distance from the center of the plot" by some amount.
On a closely related note, and thus the same issue, I think that "number of names to display" needs to be a value in settings.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/262, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNVP3HNRED3B7AKJFKLTEV55PANCNFSM4ZRFTBIQ.

Agreed. That was bound to be the tricky bit.
Diminish range value.
Sent from my Verizon, Samsung Galaxy smartphone
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Rick Boatright @.>
Sent: Sunday, March 21, 2021 2:13:48 AM
To: ljramones/trips @.>
Cc: Subscribed @.***>
Subject: Re: [ljramones/trips] Algorithm for choosing what names to display has problems (#262)
But we still need to tweak the distance component somehow to avoid the "all the stars cram up in the middle" problem.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/262#issuecomment-803519564, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNXJ5W46SF5OD52TVGDTEWFBZANCNFSM4ZRFTBIQ.

There is a max number for labels in the star preferences dialog where you pick size and color of stars.
Sent from ProtonMail Mobile
On Sun, Mar 21, 2021 at 09:49, cegannon1 @.***> wrote:

Max # of stars to display has been limit in BOTH of the star display algorithms I put forth. I even indicated how many CHV showed, and why.
I also explicitly indicated that the only reason I did not declare a hard limit was b/c that was putting the cart before the horse. But there HAS to be a limit.
Consider searching for G V stars and then displaying. The are ALL going to have extremely high values.
20 or 30 tops has been my ballpark suggested beginning limit/total of diplayed star names ( see the original spreadsheets I sent ) from the start. But there was no way of knowing if that was optimal before seeing it.
Sent from my Verizon, Samsung Galaxy smartphone
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Rick Boatright @.>
Sent: Sunday, March 21, 2021 1:12:55 AM
To: ljramones/trips @.>
Cc: Subscribed @.***>
Subject: [ljramones/trips] Algorithm for choosing what names to display has problems (#262)
Choose any dataset and crank the radius out to 50 or 70 ly or more, and ALL the named stars are in the central third or less.
Even if we restrict the central 70 LY to F and G stars, you get a total of 2 names in the outer half.
Oddly, if you grab the 270 LY set, and restrict it to just O and B stars (600 or so) out to 270 LY, the names ARE distributed well.
We need to increase the value of "distance from the center of the plot" by some amount.
On a closely related note, and thus the same issue, I think that "number of names to display" needs to be a value in settings.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/262, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNVP3HNRED3B7AKJFKLTEV55PANCNFSM4ZRFTBIQ.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

What is the default?
Sent from my Verizon, Samsung Galaxy smartphone
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: LJ Ramones @.>
Sent: Sunday, March 21, 2021 9:55:04 AM
To: ljramones/trips @.>
Cc: cegannon1 @.>; Comment @.>
Subject: Re: [ljramones/trips] Algorithm for choosing what names to display has problems (#262)
There is a max number for labels in the star preferences dialog where you pick size and color of stars.
Sent from ProtonMail Mobile
On Sun, Mar 21, 2021 at 09:49, cegannon1 @.***> wrote:

Max # of stars to display has been limit in BOTH of the star display algorithms I put forth. I even indicated how many CHV showed, and why.
I also explicitly indicated that the only reason I did not declare a hard limit was b/c that was putting the cart before the horse. But there HAS to be a limit.
Consider searching for G V stars and then displaying. The are ALL going to have extremely high values.
20 or 30 tops has been my ballpark suggested beginning limit/total of diplayed star names ( see the original spreadsheets I sent ) from the start. But there was no way of knowing if that was optimal before seeing it.
Sent from my Verizon, Samsung Galaxy smartphone
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Rick Boatright @.>
Sent: Sunday, March 21, 2021 1:12:55 AM
To: ljramones/trips @.>
Cc: Subscribed @.***>
Subject: [ljramones/trips] Algorithm for choosing what names to display has problems (#262)
Choose any dataset and crank the radius out to 50 or 70 ly or more, and ALL the named stars are in the central third or less.
Even if we restrict the central 70 LY to F and G stars, you get a total of 2 names in the outer half.
Oddly, if you grab the 270 LY set, and restrict it to just O and B stars (600 or so) out to 270 LY, the names ARE distributed well.
We need to increase the value of "distance from the center of the plot" by some amount.
On a closely related note, and thus the same issue, I think that "number of names to display" needs to be a value in settings.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/262, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNVP3HNRED3B7AKJFKLTEV55PANCNFSM4ZRFTBIQ.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/ljramones/trips/issues/262#issuecomment-803584512, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPXLNRGYVT6AOPZPTFSFL3TEX3DRANCNFSM4ZRFTBIQ.

the default is 30.

This has been subsumed by #259 and probably can just be closed.


"Display as" on integer data variables shows address in preview instead of value

To repro:

Create a new file
Input 1234 in Hex view
Switch to linear view
Press 2 to create an int16_t
Right click on data_0 and hover over Display as
Notice all previews show 0 instead of the value of the data var

I can reproduce this. This only happens when you select the token for the data variable itself. If you select the value token it works. Probably just a luck of judgement of selected token


Bug

For some reason the time that is suppose to be total length of the song seems to be showing the song current time at playing

fix bug

sorry, an error ,,,

Yes confirmed. this is because of a bug in Google Drive file serving. Chrome is not affected but some other browsers are (IE and Firefox at least). I;ve let the Google Drive team know about this bug. Please follow #31 for updates.

And thanks for the repport!


[Refactor] Drop finalizerTask, add FlowAction and CompilerService

Before this PR

Some projects wehre this plugin was applied were failing with:
Unable to make progress running work. The following items are queued for execution but none of them can be started:.
After this PR

Removed the finalizer task
Introduced a BuildService CompileBuildService that collects all the compiler errors and then reports the collected failures before it closes.
for gradleVersion >= 8.6, we introduce a FlowAction that will get the exceptions at the end of the build & report them. FlowActions are only introduced in 8.6 though, so for lower gradleVersions we use a BuildListener

==COMMIT_MSG==
[Refactor] Drop finalizerTask, add FlowAction and CompilerService
==COMMIT_MSG==
Possible downsides?

Looks good! Just a few comments!

Released 1.7.0


ThinLiquid-patch-1

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket: