rust-tools is incompatible with latest neovim nightly

After merging neovim/neovim#15504, rust-tools is incompatible with neovim nightly (hover actions and inlays don't work).

Oh cool thanks guys. Had the same issue.

For me, it is completely not working on the nightly build. Ether RustRunnables and RustDebuggables are complying about a nil index.

Should be fixed now. Please let me know if I missed anything, else I will close this in a day

Not entirely sure if related in any way, but now I'm getting this in my lsp.log:
[ERROR][2021-09-16 06:36:03] .../vim/lsp/rpc.lua:462	"rpc"	"rust-analyzer"	"stderr"	"Panic context:\n> \nversion: b73b32147 2021-09-06 dev\nrequest: textDocument/codeAction CodeActionParams {\n    text_document: TextDocumentIdentifier {\n        uri: Url {\n            scheme: \"file\",\n            cannot_be_a_base: false,\n            username: \"\",\n            password: None,\n            host: None,\n            port: None,\n            path: \"/Users/misko/devel/rust/ip2vlan/src/main.rs\",\n            query: None,\n            fragment: None,\n        },\n    },\n    range: Range {\n        start: Position {\n            line: 55,\n            character: 8,\n        },\n        end: Position {\n            line: 55,\n            character: 8,\n        },\n    },\n    context: CodeActionContext {\n        diagnostics: [],\n        only: None,\n    },\n    work_done_progress_params: WorkDoneProgressParams {\n        work_done_token: None,\n    },\n    partial_result_params: PartialResultParams {\n        partial_result_token: None,\n    },\n}\n\nthread '<unnamed>' panicked at 'assertion failed: check_disjoint(indels)', crates/text_edit/src/lib.rs:191:5\n"
[ERROR][2021-09-16 06:36:03] .../vim/lsp/rpc.lua:462	"rpc"	"rust-analyzer"	"stderr"	"stack backtrace:\n"
[ERROR][2021-09-16 06:36:03] .../vim/lsp/rpc.lua:462	"rpc"	"rust-analyzer"	"stderr"	"   0: _rust_begin_unwind\n   1: core::panicking::panic_fmt\n   2: core::panicking::panic\n   3: text_edit::TextEditBuilder::replace"
[ERROR][2021-09-16 06:36:03] .../vim/lsp/rpc.lua:462	"rpc"	"rust-analyzer"	"stderr"	"\n   4: ide_assists::handlers::destructure_tuple_binding::edit_tuple_usages\n   5: ide_assists::assist_context::Assists::add_impl\n   6: ide_assists::handlers::destructure_tuple_binding::destructure_tuple_binding\n   7: ide_assists::assists\n   8: ide::Analysis::assists_with_fixes::{{closure}}\n   9: std::panicking::try\n  10: ide::Analysis::assists_with_fixes\n  11: rust_analyzer::handlers::handle_code_action\n  12: std::panicking::try\n  13: <F as threadpool::FnBox>::call_box\nnote: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.\n"


seamail:  all messages show as unique bubbles

Can you talk more about this one?

We group messages by time now, so only closely-related posts end up together, FWIW.

The idea being that each message would have its own bubble rather than having multiple messages within the same.   Currently sending two messages in a row groups them together like so rather than having two separate bubbles.

Right, but isn't that good?

I feel it is more common for them to be split out, definitely on iOS but also on most of the messaging apps I'm seeing for Android.   I feel it also adds more clarity to the user.

Hmm. I sampled Android messenger (SMS), Hangouts chat, and Facebook messenger and they all have similar styles. They put each message in their own bubble, cluster bubbles close together based on time and omit extra user icons and timestamps for the cluster

.

I guess I'm struggling to see why it's better with separate bubbles. If I send two things in quick succession, isn't that the same as sending one thing with a line break?

One reason is transparency, it allows the user to know the difference between someone sending things with random line breaks vs. multiple messages.   (And some people use multiple messages for emphasis.)
The other is that it is common practice.   Users are used to seeing it.   If you have the option of maintaining the experience users are used to and doing something different, generally one should err on the former unless there is a very compelling reason to do the latter.
The reason I suggested using bubbles was to maintain the aesthetic common in most messaging software.   I'd honestly get rid of them rather than grouping multiple messages in the same bubble, especially if we are using the same UI elements for forums which I worry might be confusing.


Feature request: make machine learning and graph tabs switchable

I guess both of those tabs require a backend and at least for our use case of a standalone lightweight ClickHouse UI they don't make sense.
It seems to be fairly easy to make those switchable, e.g. via env variables.
I could work on a PR if there's the chance for it to get merged.

I made the tabs switchable in my fork.
Please see https://github.com/dgloeckner/ClickCat/tree/improve-ux


Next

Codecov Report

Merging #15 (8621884) into master (3efb0e6) will decrease coverage by 0.53%.
The diff coverage is 89.58%.

:exclamation: Current head 8621884 differs from pull request most recent head d6b49f2. Consider uploading reports for the commit d6b49f2 to get more accurate results

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
- Coverage   95.05%   94.52%   -0.54%
==========================================
  Files          70       72       +2
  Lines        2994     3158     +164
==========================================
+ Hits         2846     2985     +139
- Misses        148      173      +25

Flag
Coverage Δ

unittests
94.52% <89.58%> (-0.54%)
:arrow_down:

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

Impacted Files
Coverage Δ

aiopenapi3/v20/glue.py
86.15% <50.00%> (-2.13%)
:arrow_down:

aiopenapi3/log.py
73.68% <73.68%> (ø)

aiopenapi3/model.py
88.67% <76.27%> (-6.75%)
:arrow_down:

tests/conftest.py
97.59% <83.33%> (-1.12%)
:arrow_down:

aiopenapi3/base.py
92.94% <86.79%> (+2.71%)
:arrow_up:

aiopenapi3/loader.py
95.91% <96.29%> (-0.75%)
:arrow_down:

aiopenapi3/cli.py
95.34% <100.00%> (+0.11%)
:arrow_up:

aiopenapi3/openapi.py
92.40% <100.00%> (+0.84%)
:arrow_up:

aiopenapi3/plugin.py
98.48% <100.00%> (+0.12%)
:arrow_up:

aiopenapi3/v20/schemas.py
100.00% <100.00%> (ø)

... and 21 more

Continue to review full report at Codecov.

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


开发环境的 tree-shaking 问题

Describe the bug
I have a public 'utils' directory in my project, structured like the figure, export all the methods in 'index.ts', so that they can be  import in utils/index.ts, but that will cause all uitls in the development environment to be loaded.

example: import { isUndefined } from '@dzh/utils'，I only import one of these methods, and all utils were imported

import { isUndefined } from '@dzh/utils/src/is';
Must be import to a specific directory to not be so.
So there is no good way to deal with this kind of problem in the development environment, in addition to utils there will be public hooks and so on。
Reproduction
https://github.com/Miofly/vite-dev-tree-shaking
System Info
System:
    OS: macOS 12.3.1
    CPU: (8) arm64 Apple M1
    Memory: 83.66 MB / 8.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.0.0 - ~/.nvm/versions/node/v16.0.0/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 6.14.15 - ~/Documents/mio/new-mio/mio/node_modules/.bin/npm
  Browsers:
    Chrome: 103.0.5060.134
    Safari: 15.4
  npmPackages:
    @vitejs/plugin-vue: ^3.0.1 => 3.0.1
    vite: ^3.0.1 => 3.0.4

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I think this is a duplicate of #8237


How to distinguish a method property from a bound callback property?

Suggestion
I expected ThisParameterType helper to be able to distinguish between methods (unbound function properties) and callbacks (bound functions), but it doesn't want to do it:
class Cls {
  method() {}
  callback = () => {}
}

type aaa = ThisParameterType<Cls["callback"]>;
// Aaa = unknown

type Bbb = ThisParameterType<Cls["method"]>;
// Bbb = unknown -- WHY? How to distinguish?

Playground link

🔍 Search Terms
method property from a bound callback property this
💻 Use Cases
I want to enumerate all plain properties of some type (e.g. a class instance), but skip methods. Currently I do it with:
export type NonMethodPropertyNames<T> = {
  [K in keyof T]: T[K] extends Function ? never : K;
}[keyof T];

export type NonMethodProperties<T> = Pick<T, NonMethodPropertyNames<T>>;

but unfortunately, this NonMethodProperties also skips callback properties. E.g. in this example:
class Cls {
  constructor(public a = 10, public callback: () => void) {}
  method() {}
}

...I would expect NonMethodProperties<Cls> to return { a: number; callback: () => void }, but unfortunately it skips not only method, but also callback since I couldn't find a way to distinguish them.

The method in your class don't have a this argument, so ThisParameterType<> does not work on them. There's the open issue #28548 to automatically add a this argument to class methods.


LR Schedule

Hello,
thanks for this nice work!
I have a question regarding the LR schedule. I've noticed differences between the descriptions outlined in the paper and the settings in the actual configuration file (specifically for RetinexFormer on the LoLv2-real dataset). You are also using larger crops than for the other datasets. Could you kindly provide some insight into the reasoning behind these differences?
Best,
Eduard

The illumination changes within a single image in this dataset are relatively large.
A larger receptive field is needed for the network to learn this illumination information.
Thus, we use a larger crop size.


ORAS Demo added to asr.dat, show full TID on screen (16 chars)

No need for new release, but this should be there for the next one I guess.
Also, I was hessitant about adding SSB demo and ORAS demo to secure_values.c, but the existence of asr.dat makes it somewhat pointless. What do you think?

I would tend to agree that keeping additional games to asr.dat if possible would be best. If upstream 3ds_hb_menu can pass svdt the title ID, this would not only render secureGameFromFilesystem unnecessary, but let us boil whichSecureGame down to essentially just SECURE_CONFIG and SECURE_UNKNOWN from the start.
As for right now, I also think we should keep demos to asr.dat. If the SSB demo has the same offsets and files, secureGameFromFilesystem will catch it anyway. The ORAS demo cooperates well enough with svdt that I don't think we need a filesystem-based guess for it (and the same goes for all the Pokémon 'main' games for now).


Send error.transaction_id and error.parent_id only if there is trace_id

What does this PR do?

Context - reported in the forum.
APM Server will reject Error events that contain parent_id and/or transaction_id but do not contain trace_id. This PR ensures that this restriction is enforced.
In addition, it provides logs to analyze the reason for getting error captures that are not adhering this.
Checklist

[x] This is a bugfix

[ ] I have updated CHANGELOG.asciidoc
[x] I have added tests that would fail without this fix

Do you have a suspicion why there can be a trace context with a non-empty transaction id but an empty trace id? The only things that come to my mind are race conditions or visibility issues.

No clue 🙁 . This is why I added the logs and asked the reporting user to trigger it.
Sadly, the only thing that came to my mind was concurrency-related as well, but I am hoping it would be something easier to pinpoint. It may be related to the non-fully-dereferenced transactions somehow 🤷♂

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: [Pull request #1537 opened]

Start Time: 2020-11-25T10:00:47.704+0000

Duration: 46 min 37 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
1617

Skipped
12

Total
1629


Intel computation guide

Closes: https://github.com/canonical/data-science-stack/issues/150
Add docs with examples for ITEX and IPEX usage.

@misohu I am not really sure this needs to be a dedicated guide. Can we double check with Andreea (can't tag her here) and @kimwnasptd  if we are missing something here? If all we want is to document some refs for the user to run Intel-based ML examples, then I would add a note box in the Create an Intel GPU-enable notebook section we just published. At the bottom of the section, add a note box saying something like:
"Once created, you can access it (with hyperlink to the section) and run ML workloads on Intel hardware with DSS. See IPEX (with hyperlink) and ITEX (hyperlink) for detailed examples using the Intel extensions for Pytorch and Tensorflow respectively."


upgrading to common 0.6.7

fixing 'Schema not supported'
fixing NPE tests on 0.10.1 when timestamp is required on sinkrecord

This extractor is not used. Hbase and influxdb have each their own.  They are different. I covered the boolean (non basic schema) test in hbase. Influxdb doesnt face the problem


Issues scanning ITF barcode on iOS

Scanning the ITF barcode found at http://www.scandit.com/products/barcode-scanner/symbologies/itf-2-of-5/ works correctly on android, but the plugin doesn't seems to be able to read the barcode in iOS
I'm using version 5.0 of the plugin and tried on both iPhone and iPads
Edit: Using the latest version (6.0.2.) exhibits the same behavior

Hi. Thanks wildabeast team for your barcodescanner plugin.
I'm waiting for your answer, because i've got same issue, event trying using formats option : "formats" : "EAN_13,ITF". Works fine with phonegap app 1.7.2. on Android but can't scan ITF on ios10 (but EAN13 is OK).
Don't have signing key for yet to try online with phonegapbuild.

@wildabeast do you have any insights on what could be happening here? thanks in advance

Hi. Back to give news. I get today my Apple Development ID et signing key for phone gap build and all work perfectly with barcodescanner plugin, including ITF barcode on iOS (testing on iphone)!

I'm having the same problem,  have found a solution? read ITF in ios,  i test in Ipad ios 9.3.2 with barcodescanner 6.0.3

@lucascdsilva the plugin seems don't work for ITF barcode when testing with phone gap mobil app.
But with apple signing key you can test with build.phonegap.com and ITF is working.

Sorry, I do not understand. what's the difference?

I found that my problem is related to the issue #354 ,  ITF with Barcode type - INTERLEAVED 2 of 5

I have the same problem, I explained my situation in #354
@reymarx  @tocrea were you able to solve the problem?

For me the issue was only when testing with phonegapapp. After signing the app from apple dev. I was able to test the app with phonegap build ( and apple testflight, it's same thing) and all works fine, no more trouble with ITF!

@tocrea I believe it is the type of coding of the bar code "ITF - INTERLEAVED 2 of 5"

try read that code bar with IOS, i want only android

Thanks all,
But @tocrea I have this problem after signing the app with an Apple's Development Provisioning and testing it with my ipad.

I have the same problem after signing the app with an Apple's Development Provisioning.

It's fixed in master


switch to sqlite-wasm-http

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (993014f) 100.00% compared to head (b380595) 100.00%.

:exclamation: Current head b380595 differs from pull request most recent head 4cd00a2. Consider uploading reports for the commit 4cd00a2 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #1   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines            2         2
=========================================
  Hits             2         2

Impacted Files
Coverage Δ

src/mbtiles-source.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Arctic Ranger Station wrong Distances

The Ranger Station in the Arctic  has a faulty Influence Range in AD, it is on dirt road 50 (RAW), and not 26 (RAW) as it is now in the Presets. I will fix this with the next update for Presets.json.
Workaround :
fill in the number 48 on Distance (red arrow)
and push Place Building (orange arrow)

Sorry this one i forgot to fix...
i still will look into this one as soon i can


Pantheon updates 2023-06-27

Description of changes
Note that some of the changes here cannot be easily tested without switchboard-plug-* bump.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg test pantheon

Tested filters with gsettings set io.elementary.desktop.wm.accessibility colorblindness-correction-filter protanopia
I am not quite sure about the pantheon portal adding background support, since we enable this in gtk portal as well, and in lexicographic order sounds like the gtk one is preferred :thinking: I specifically tested a reboot cycle and i.e.mail --background auto starts for me, and making requests on com.belmoussaoui.ashpd.demo seems to work, I assume things are fine
switchboard-plug-applications has some re-design!


Any Kubernetes

Do you know any resource to easily mount a nifi kubernetes cluster on google cloud?
thanks!

This is the only one I've seen. We should add it to the README. But I've also been trying to find a k8s operator. Don't think one exists. Might be a fun project to work on. https://github.com/AlexsJones/kubernetes-nifi-cluster

Have a look at https://github.com/cetic/helm-nifi


bazel_3, bazel_4: Explicitly add a dependency on bazel-rc

This resolves an issue where the path to the bazel-rc file is sometimes
optimised so the full hash is not present, and hence not picked up by
the scan as a runtime dependency.

Motivation for this change
Intended to address #155947
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[X] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

It'd be good to get this merged unless anyone has concerns?

Oh yeah sorry forgot about it.

Thanks a lot!


Transform bug fix

In the current codebase the transform tracking disable wasn't being correctly reset, meaning moves weren't getting tracked when they should be.
This is because it was only reset when the enemy mon changed, so I've also added a re-set to occur at the end of battle
I also did some cleanup of the transform tracking disable. The second boolean for force-switch moves is no longer required so i removed it. This was initially used because the transform tracking reset used to occur in the "pokeball sent out" event handler which occurs whenever the player or the enemy sends out a pokemon (thus a force-switch would have incorrectly reset the transform tracking disable due to your swapped pokemon coming out)
Tested this out and with this fix it now correctly resets the isTransformed boolean when the enemy sends out another pokemon and at the end of a battle

Update: added a small fix for the stat marking buttons state updates. The state itself wasn't being correctly updating between pokemon which lead to weird orders between pokemon after you mark a stat
Example scenario of where it was broken before:
you encounter one pokemon, the button cycles from off -> plus -> minus -> off and you mark a "plus" on the pokemon. You run from / defeat this pokemon and then encounter another pokemon which has no stat markings. The buttons now cycle off -> minus -> off -> plus -> minus instead because it retained the "plus" state from the previous pokemon internally


Correction for raw affiliation Montpellier Interdisciplinary Center of Sustainable Agri-Food Systems (UMR MoISA), Montpellier University and French Agricultural Research Center for International Development (CIRAD), Montpellier, France

Correction needed for raw affiliation Montpellier Interdisciplinary Center of Sustainable Agri-Food Systems (UMR MoISA), Montpellier University and French Agricultural Research Center for International Development (CIRAD), Montpellier, France
raw_affiliation_name: Montpellier Interdisciplinary Center of Sustainable Agri-Food Systems (UMR MoISA), Montpellier University and French Agricultural Research Center for International Development (CIRAD), Montpellier, France
new_rors: 05kpkpg04;051escj72;011xg1225
previous_rors: 05kpkpg04;051escj72
works_examples: W4390411341
contact: 3328e51a6062451e2adb09e454d4337d:1402d9b1cb1fa4a0 @ supagro.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Kubelet flag deprecated soon

Is this a request for help?: NO

Is this an ISSUE or FEATURE REQUEST? (choose one): ISSUE

What version of acs-engine?:
all

Orchestrator and version (e.g. Kubernetes, DC/OS, Swarm)
Kubernetes
What happened:
Flag --register-schedulable has been deprecated, will be removed in a future version
related to parts/kuberneteskubelet.service file
What you expected to happen:
using LTS flag instead

@seanknox for awareness


Add support for disabling wallpaper, gtm, olytics

@brandonbk Where/how is disabledFeatures defined and configured?


Failed to persist event type [..] with sequence number [1]

Hi,
I upgraded my project from
Akka : 2.4.2 -> 2.4.17
Slick : 3.1.1 -> 3.2.0
akka-persistence-jdbc : 2.2.8 -> <IP_ADDRESS>
Now I have problem, akka persistence persistAsync failed with Failed to persist event type [..] with sequence number [1] after 10 sec. I check that snapshot queries ran but persist not occurred.
Do you have any idea about my issue?  Which layer akka or slick or persistence jdbc  or configuration layer is related with my issue?
Now I am working to solve it, If anyone who encountered this type of problem help me, I am very appriciate.

Hi, if you create a small test project that has the same problem then I could take a look at the problem. For now, please review all the configuration items and ensure they are configured correctly.

Hi,
Thank you for your interest.  I tried to solve my problem all the day and now I can run akka persistence with my configuration. But it is not a solution. It ran when debuging.
I fork this project and add below lines for debuging to the BaseByteArrayJournalDao class and now persistence working well.
override def writeFlow: Flow[AtomicWrite, Try[Unit], NotUsed] = {
Flow[AtomicWrite]
.map(s => {
println(s"Message in writeflow ${s}")
s
})
.via(serializer.serializeFlow)
.map(_.map(writeJournalRows))
.via(futureExtractor)
}
When remove these lines, persistence is not working :) .  It is strange. Any idea about this.

Hi, that is strange. The println will force some synchronization to occur with IO, but I couln't imagine why this would work. Maybe remove the println and then ask sbt to do a ;clean;compile ?

I've created a small test project called akka-persistence-jdbc-test, It works and does eventing and snapshotting, maybe you can take a look and see what the difference are.

Closing this issue since there has been no more response form the issue author
@tcak76 if this issue still occurs, feel free to reopen


github.init_pkg_repo fails because GitHub added internationalization to its documentation URLs

Initializing a new package repository in GitHub fails due to new internationalization within docs.github.com URLs. GitHub introduced i8n to their docs here (sort of - not exactly defined here, but I couldn't find a specific blog post referencing docs.github.com). https://github.blog/2020-07-02-how-we-launched-docs-github-com/#internationalized-docs
github.init_pkg_repo fails with a 404 when calling https://docs.github.com/rest/reference/repos#create-or-update-file-contents
The new URL for English language is https://docs.github.com/en/rest/reference/repos#create-or-update-file-contents

The upstream package was fixed here: https://github.com/PyGithub/PyGithub/commit/babcbcd04fd5605634855f621b8558afc5cbc515


nixos/switch-to-configuration: reload user units

Motivation for this change
When rebuilding you have to manually run systemctl --user daemon-reload. It gathers all authenticated users using
loginctl list-user and runs daemon-reload for each of them.
This is a first step towards a nixos-rebuild which is able to reload
user units from systemd. The entire task is fairly hard, however I
consider this patch usable as it allows to restart units without running
daemon-reload for each authenticated user.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@edolstra would you mind having another look at this? I switched to a loginctl-based approach now, but I'm not a Perl expert, so I'm not sure if there's still a better solution :sweat_smile:

@edolstra done!

@Mic92 done!

@edolstra @Mic92 any chance to get this merged before the 18.09 branchoff? :)


notebook is not trusted

After running the notebook and saving the output to another notebook.
$ runipy MyNotebook.ipynb OutputNotebook.ipynb

I cannot load the output notebook because of this error while the notebook is trying to load:
Loading [MathJax]/extensions/Safe.js

Error
Notebook OutputNotebook.ipynb is not trusted

So, you can mark the notebook as trusted. The trick is there is JS code in there and it is a new notebook so it is probably going to start out untrusted no matter what. Though I can try and see if we can do anything here. I think the aforementioned strategy is the way to go.

In Jupyter, use jupyter trust ... instead as explained in the docs.

Hi @jakirkham,
I have marked the notebook as trusted. But still cannot load the notebook, and the error message just keeps popping up
Error
Notebook OutputNotebook.ipynb is not trusted

Thanks,
Tuan

Could you share both notebooks in a gist?

Hi @jakirkham,
These are the input and output notebooks: https://gist.github.com/tuanvu216/8ae15f26a46c35c3b6cefbbe5f20fd82
Thanks,
Tuan

This issue might occur, if you are using JupyterHub and Dockerspawner and have the notebooks in a volume, which is mounted to the Docker containers. In such case one might need to remove the corresponding volume as well.


Update transform-data-using-script.md

I'm not 100% certain but there are references to a 5k rows 2mb limit in the article, however I know that the limit for the lookup activity is 5k rows 4mb. I would assume that this limit the same on both can someone from product please confirm?

@nabhishek
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


doc: update readme

name: Pull request
about: Tell us about your contribution
labels: ["pr/release/none-required"]

Thanks for contributing !
Before submitting a pull request, make sure you read about our Contribution notice here: https://spidernet-io.github.io/spiderpool/contributing/development/pullrequest/#need-review
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for your reviewer:
make sure your commit is signed off

这个要同步到  doc/readme

好的，晚点同步


made it work again ;)

fixed errors that  caused build fail @Yalantis

Please, remove changes related with cocoapods

Thanks for the PR. It's my fault - I had to bump the first number in version :(


Updated channel and ifo submission format

Changes the way that ifo and channel information is formatted in our gracedb submissions to align with what Alex and Roberto are requesting. I need the gracedb PR to be merged before I can test this.

Looks good - have tested submitting with this to GraceDB?

@EthanMarx I tested this out and the format is showing up as expected: https://gracedb-test.ligo.org/events/G1188124/view/


cherry-pick: stop using deprecated protobufs for ext vols with docker apps (#4131)

generate external vol protos using the unified container protobufs for apps using docker engine
update external volume docs w/ notes re: docker-created anonymous volumes and how to prevent them
only disable deprecation warnings for Test, re-enable for Compile
Update external-volumes.md

Conflicts:
src/main/scala/mesosphere/marathon/core/externalvolume/impl/providers/DVDIProvider.scala
src/test/scala/mesosphere/mesos/TaskBuilderTest.scala

this will be important to land in dc/os 1.8 ASAP. commenting here because i'm OOO for the next 6 business days.
/cc @spahl

Thanks!


compile issue on aws linux

https://gist.github.com/nyghtowl/f554c8159a213de69337
@agibsonccc

This has been fixed with the most recent PR.

https://github.com/deeplearning4j/nd4j/pull/643


Language Request: Dockerfile

I have use for the dockerfile "language" support.
I dont know what else to add here.


Postman autotests fail on Cloud Foundry (at least AppFog)

Probably it's not feasible to invoke native shared libraries on Cloud Foundry.  We may need to think of an alternative approach for executing NodeJS in these environments.  Possibly we'll need to incorporate portions of the NodeJS buildpack:
https://github.com/cloudfoundry/nodejs-buildpack
2017-07-19T11:17:26.92-0600 [App/0]      OUT [(d)20170719.17:17:26.922 ~165] Loading Dynamic Java byte code from: string:///com/centurylink/mdw/testing/postman/NodeRunn
er.java
2017-07-19T11:17:27.15-0600 [App/0]      ERR java.lang.reflect.InvocationTargetException
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at java.lang.reflect.Method.invoke(Method.java:498)
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at com.centurylink.mdw.services.test.TestCaseRun.run(TestCaseRun.java:210)
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at java.lang.Thread.run(Thread.java:745)
2017-07-19T11:17:27.15-0600 [App/0]      ERR Caused by: java.lang.IllegalStateException: J2V8 native library not loaded
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at com.eclipsesource.v8.V8.checkNativeLibraryLoaded(V8.java:252)
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at com.eclipsesource.v8.V8.createV8Runtime(V8.java:152)
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at com.eclipsesource.v8.V8.createV8Runtime(V8.java:128)
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at com.eclipsesource.v8.NodeJS.createNodeJS(NodeJS.java:84)
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at com.eclipsesource.v8.NodeJS.createNodeJS(NodeJS.java:48)
2017-07-19T11:17:27.15-0600 [App/0]      ERR    ... 8 more
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at com.centurylink.mdw.testing.postman.NodeRunner.run(NodeRunner.java:53)
2017-07-19T11:17:27.15-0600 [App/0]      ERR Caused by: java.lang.UnsatisfiedLinkError: Could not load J2V8 library. Reasons:
2017-07-19T11:17:27.15-0600 [App/0]      ERR    no libj2v8_linux_x86_64.so in java.library.path
2017-07-19T11:17:27.15-0600 [App/0]      ERR    no j2v8_linux_x86_64 in java.library.path
2017-07-19T11:17:27.15-0600 [App/0]      ERR    /home/vcap/libj2v8_linux_x86_64.so: /usr/lib/x86_64-linux-gnu/libstdc++.so.6: version `CXXABI_1.3.8' not found (required
 by /home/vcap/libj2v8_linux_x86_64.so)
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at com.eclipsesource.v8.LibraryLoader.loadLibrary(LibraryLoader.java:79)
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at com.eclipsesource.v8.V8.load(V8.java:74)
2017-07-19T11:17:27.15-0600 [App/0]      ERR    at com.eclipsesource.v8.V8.createV8Runtime(V8.java:148)
2017-07-19T11:17:27.15-0600 [App/0]      ERR    ... 12 more

Fixed by adding updated libstdc++ shared library to LD_LIBRARY_PATH (in asset package com.centurylink.mdw.node).


Add missing pragma once in test headers

Purpose
Related Information

[ ] Design document: ...
[ ] Bench PR: ...

Testing

[ ] This change is a trivial rework or code cleanup without any test coverage.
[ ] This change is already covered by existing tests.
[ ] This PR adds tests that were used to verify all changes.
[ ] There are tests in an external testing repository: ...

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1521

Relevant Lines:
1521

💛  - Coveralls


Import error: Error: Unknown glyph name 'Glyph of Subjugate Demon'

Describe the bug
Gives error if you have Subjugate Demon glyph
To Reproduce
Steps to reproduce the behavior:
Generate data from Exporter Addon and have Glyph of Subjugate Demon on your character.
"Import from addon" on the github page
Expected behavior
Be able to import data without having to edit it out
Sim Links and Screenshots
{"talents":"2350002031023510253500331151-000000000000000000000000000-55000005000000000000000000","glyphs":{"major":["Glyph of Quick Decay","Glyph of Haunt","Glyph of Life Tap"],"minor":["Glyph of Unending Breath","Glyph of Drain Soul","Glyph of Subjugate Demon"]},"class":"warlock","race":"Human","name":"Seedgobrrtt","gear":{"items":[{"enchant":3820,"gems":[41285,40051],"id":39496},{"id":40071,"gems":[]},{"enchant":3810,"gems":[39998],"id":40424},{"enchant":3722,"gems":[],"id":41610},{"enchant":1144,"gems":[],"id":42102},{"enchant":2332,"gems":[],"id":39252},{"enchant":3246,"gems":[39941],"id":39500},{"id":37408,"gems":[41457]},{"enchant":3872,"gems":[39998,40051],"id":40422},{"enchant":1147,"gems":[],"id":40750},{"enchant":3840,"gems":[],"id":49123},{"enchant":3840,"gems":[],"id":40719},{"id":40682,"gems":[]},{"id":37873,"gems":[]},{"enchant":3830,"gems":[],"id":40336},{"id":40698,"gems":[]},{"id":37177,"gems":[]}]},"professions":[{"name":"Enchanting","level":445},{"name":"Tailoring","level":441}],"level":80,"spec":"","realm":"Pagle"}
Desktop (please complete the following information):

OS: WIN 10
Browser: Chrome
Version [e.g. 22]

Additional context
The github has the glyph name set up as the old version with "Enslave Demon" but on the mouseover it says "Subjugate Demon" so idk what's up with that lol

#1371


Erratic SRTM elevations

I compare gpxelevations with gpsprune (http://activityworkshop.net/software/gpsprune/) and googleAPI (for a certain GPX track). Using RMSE (root mean square error) I have got 1,2 meter average difference for gpsprune versus googleAPI while almost 4 meters for gpselevations versus googleAPI/gpsprune (4 times more or 400%).
Looking at plotted values (http://fabianserwis.homelinux.com/p/) I have discovered a few areas of suspiciously erratic elevations
Any explanation?

Nope, I have no idea what could be wrong.
If somebody else have an explanation and is willing to fix it (in case there is a bug), I'm willing to merge it.


Error Popup while using Build a micro app for SAP Mobile Cards booster.

Tutorial URL: https://developers.sap.com/tutorials/cp-mobile-cards-booster-starter.html
Step 2: Create contact card using Boosters
The following error popup is displayed for the booster-Build a micro app for SAP Mobile Cards.
This error is displayed when we try to create more than 3 cards with the same public ip address.

Also please add the link for the documentation mentioned in the error message.
Labels:
Landscape: trail1, trail2, trail3
Label: SCPTest-2011A
Scenario Label: SCPTest-mcard
Priority: High
Best Regards,
Priyanka

Hi @PriyankaShivakumar,
Thank you for raising the issue.
We have forwarded the error to the development team and will ensure that a solution is provided by the next release.
Warm Regards
Sandeep T D S

@sandeep-tds Any updates?

Hi @PriyankaShivakumar
The issue has been addressed. Can you please re-test to confirm the same?
Thank you so much for your support.
@MichaelCzcz  thank you for the reminder.

Hi @sandeep-tds
The issue is fixed now. Thanks.
Best Regards,
Priyanka

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


allow injecting user/group name via environment variable

Little change - but this allows to inject/specify the user/group name that is used during testing.
By compartmentalizing this - we are then able to run multiple tests at the same time and simulate several clients ( "load test" )
cc @mrow4a

Smashbox already allows to run multiple clients in the same time (but on the same host). But anyways, this looks good to me. 👍


From datamol 0.6.0 forwards, required rdkit version should be >2021.09.1

An update for the viz.to_image was introduced in 0.6.0. But there was this rdkit bug: https://github.com/rdkit/rdkit/issues/3101 that was only fixed in  Release_2021.09.1
https://github.com/datamol-org/datamol/blob/cd4561b7e8cdfb1c1ccc034116d6f900cc957b11/env.yml#L24

I actually knew about it but forgot to bump. Will do. Thanks.


No data coming from Poll

I'm trying to implement a Future trait for a struct to keep polling a TcpListener however, when I call the port, it prompts the poll to fire but it always prints out the "no events" printout. If this code just runs in a main function, it works. However, it's as if no data will ever reach it if it's implemented in a Future trait. Am I missing something fundamental?
use mio::net::{TcpListener, TcpStream};
use mio::{Events, Interest, Poll as MioPoll, Token};
use std::io::{self, Read, Write};
use std::time::Duration;
use std::error::Error;

use std::future::Future;
use std::pin::Pin;
use std::task::{Context, Poll};

const SERVER: Token = Token(0);

struct ServerWorker {
    server: TcpListener,
    poll: MioPoll,
}

impl ServerWorker {

    pub fn new() -> Result<Self, Box<dyn Error>> {
        let addr = "<IP_ADDRESS>:13265".parse()?;
        let mut server = TcpListener::bind(addr)?;
        let poll: MioPoll = MioPoll::new()?;
        poll.registry()
        .register(&mut server, SERVER, Interest::READABLE)?;
        Ok(ServerWorker{
            server,
            poll,
        })
    }
}

impl Future for ServerWorker {
    type Output = String;

    fn poll(mut self: Pin<&mut Self>, cx: &mut Context<'_>)
        -> Poll<Self::Output> {

        let mut events = Events::with_capacity(128);

        let outcome = self.poll.poll(
            &mut events,
            Some(Duration::from_secs(5))
        ).unwrap();

        println!("outcome: {:?}", outcome);

        if outcome == () {
            println!("no events");
            cx.waker().wake_by_ref();
            return Poll::Pending
        }

        println!("events: {:?}", events);
        for event in events.iter() {
            println!("event: {:?}", event);
            match event.token() {
                SERVER => {
                    let (mut stream, _) = self.server.accept().unwrap();
                    let mut received_data = [0; 4096];
                    let bytes_read = 0;
                    let _ = stream.read(&mut received_data[bytes_read..]);
                    let received_data_string = String::from_utf8_lossy(&received_data).to_string();
                    return Poll::Ready(received_data_string)
                }
                // We don't expect any events with tokens other than those we provided.
                _ => unreachable!(),
            }
        }
        cx.waker().wake_by_ref();
        return Poll::Pending
    }
}

I appreciate that there could be a more optimal implementation, I'm just using this for educational purposes.

I'm trying to implement a Future trait for a struct to keep polling a TcpListener however, when I call the port, it prompts the poll to fire but it always prints out the "no events" printout. If this code just runs in a main function, it works. However, it's as if no data will ever reach it if it's implemented in a Future trait. Am I missing something fundamental?

You are missing something fundamental, I'm sorry to say quite a number of fundamental things. I highly recommend you read the Mio and Future documentation. Start with the Mio guide: https://docs.rs/mio/latest/mio/guide/index.html and the Future trait: https://doc.rust-lang.org/std/future/trait.Future.html.
Hopefully, after reading those you'll realise that Future is not the place to call Poll::poll. In fact the events returned by Poll::poll are often used to determine what Futures (containing some sort of I/O source) to poll.
Hope this helps. If still have trouble, feel free to ask more questions.

Hey @Thomasdezeeuw thanks for the links but they're introductions and I'm struggling to understand my solution using them. From what I understand, futures get converted to runnable and set on a task queue to be polled until completed. For this code example, I used smol, flume for the queue, and futures_lite to handle the blocking of the thread until the task is complete. In terms of Mio from what I understand you can create an event loop (infinite loop), poll the socket (timeout is optional), and then process the incoming data and then return to the event loop. You can pass that data from the socket to be processed as a future to be processed on the task queue. However, if you poll the socket with a timeout, why can't the socket be polled in a future's poll method? Surely if the socket poll times out, the future's poll method results in a Pending and the future just gets put back on the task queue to run the poll again. When I run it, I keep getting "no events" printout every 5 seconds. It's just that when I send a request to the socket, I get the "no events" printout the moment the data packet is sent to the socket. What's the underlying mechanism that's stopping the data from coming out of the poll? I've seen the main thread blocked by a server that then spins off incoming requests to the task queue multiple times with multiple different crates.

I don't know what you mean by "futures get converted to runnable". Futures themselves are pollable, they don't run on their own.

Yes, this is true, a Runnable is a handle as denoted in the following from async_task documentation:

A handle to a runnable task.
Every spawned task has a single Runnable handle, which only exists when the task is scheduled for running.
Method run() polls the task’s future once. Then, the Runnable vanishes and only reappears when its Waker wakes the task, thus scheduling it to be run again.
Dropping a Runnable cancels the task, which means its future won’t be polled again, and awaiting the Task after that will result in a panic.

If you use smol, would it be easier to use smol::net::TcpStream?

Yes, it would. In my initial post, I stated that:

I appreciate that there could be a more optimal implementation, I'm just using this for educational purposes.

I'm building an async runtime using smol, then use Tokio traits to integrate Hyper into the smol runtime, and now exploring how to integrate Mio polling into the runtime. For web apps in general I just use the listeners that the frameworks or crates provide out of the box.

Are you also making TCP connections?  Poll:poll will only return events for I/O sources that can make progress. Going off the example you provided in your initial post, you only register the TcpListener, that type will only generate an event if one or more connections have been made to it.

Yes, in my initial post, I said the following:

If this code just runs in a main function, it works. However, it's as if no data will ever reach it if it's implemented in a Future trait.

If it's working in the main function instead of being polled in the Future implementation, then I am making connections and sending data over.

Also you code does the following
if outcome == () {

Which is always true as () == () is always true. What you should do instead (and is explained in the getting starting guide here: https://docs.rs/mio/latest/mio/guide/index.html, which you clearly didn't read) is following . .

Thanks this is the problem. The future polling this TCP listener is now working. It of course exits once it's received a message which is what I was expecting, I'm merely exploring the different ways in which I can poll sockets. In my previous code where it was all running in main, I didn't break the loop if if outcome == (), but I thought I did leading to the confusion. I'm now going to look into Waker. Thanks for your time I appreciate it.

I don't know what you mean by "futures get converted to runnable". Futures themselves are pollable, they don't run on their own.

Yes, this is true, a Runnable is a handle as denoted in the following from async_task documentation:

Would be easier if you introduce it as such, I'm not familiar with smol.

I'm building an async runtime using smol, then use Tokio traits to integrate Hyper into the smol runtime, and now exploring how to integrate Mio polling into the runtime. For web apps in general I just use the listeners that the frameworks or crates provide out of the box.

But smol is already an async runtime and so is Tokio. And Tokio uses Mio for it's I/O. So what you're trying to achieve doesn't make a lot of sense... You can look at smol and Tokio for inspiration to build an async runtime on top of Mio though.

@maxwellflitton any update on this?

If your goal is to integrate smol into hyper, you may be interested in this example: https://github.com/smol-rs/smol/blob/master/examples/hyper-client.rs

@Thomasdezeeuw the future now works now I no longer have the if outcome == () { in the code. It was a mistake I made when I thought I did something similar in previous working code and I didn't. @notgull Yes thanks, I remember you helping me out with this a couple of months ago. I don't know if you remember me but I'm writing the async Rust book for O'Reilly. smol is an amazing way of teaching async. The reader gets to learn about how async tasks are processed. In the book readers are implementing a smol runtime, then implementing their own task stealing, and different configurations of queues and threads. They then implement Hyper into the runtime, and they then play with Mio to poll some sockets.
The reader then explores Tokio, and then different design patterns and testing approaches to async in general. @notgull I really love the smol eco system and how it's structured. I've got a presentation with some banks and smol is what I'm going to talk about. We're also using it for the Rust key-value store at SurrealDB.


bug(SSR): Angular Universal Broken after 10.0.0-next.2

Reproduction
After upgrading my app from 10.0.0-next.1 to 10.0.0-next.2, the SSR is broken.
I am not sure which of the component is actually causing this issue.
Here's the error message which I see in log:
NodeInvocationException: Prerendering failed because of error: ReferenceError: HTMLElement is not defined at D:\MyProject\dist-server\main.js:1:1552649 at Module.FKr1 (D:\MyProject\dist-server\main.js:1:1553177) at __webpack_require__ (D:\MyProject\dist-server\main.js:1:295) at Module.dNgK (D:\MyProject\dist-server\main.js:1:2631057) at __webpack_require__ (D:\MyProject\dist-server\main.js:1:295) at Object.6GTT (D:\MyProject\dist-server\main.js:1:591420) at __webpack_require__ (D:\MyProject\dist-server\main.js:1:295) at Object.zkdS (D:\MyProject\dist-server\main.js:1:4807955) at __webpack_require__ (D:\MyProject\dist-server\main.js:1:295) at Object.8b2W (D:\MyProject\dist-server\main.js:1:665835)
Expected Behavior
What behavior were you expecting to see?
It should work without throwing any error.
Actual Behavior
Throws below error:
NodeInvocationException: Prerendering failed because of error: ReferenceError: HTMLElement is not defined at D:\MyProject\dist-server\main.js:1:1552649 at Module.FKr1 (D:\MyProject\dist-server\main.js:1:1553177) at __webpack_require__ (D:\MyProject\dist-server\main.js:1:295) at Module.dNgK (D:\MyProject\dist-server\main.js:1:2631057) at __webpack_require__ (D:\MyProject\dist-server\main.js:1:295) at Object.6GTT (D:\MyProject\dist-server\main.js:1:591420) at __webpack_require__ (D:\MyProject\dist-server\main.js:1:295) at Object.zkdS (D:\MyProject\dist-server\main.js:1:4807955) at __webpack_require__ (D:\MyProject\dist-server\main.js:1:295) at Object.8b2W (D:\MyProject\dist-server\main.js:1:665835)
Environment
Angular: 10.0.0-rc.0
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router, service-worker
Ivy Workspace: <error>

Package                                    Version
--------------------------------------------------------------------
@angular-devkit/architect                  0.901.7
@angular-devkit/build-angular              0.1000.0-rc.0
@angular-devkit/build-optimizer            0.901.7
@angular-devkit/build-webpack              0.901.7
@angular-devkit/core                       10.0.0-rc.0
@angular-devkit/schematics                 10.0.0-rc.0
@angular/cdk                               10.0.0-next.2
@angular/fire                              6.0.0
@angular/google-maps                       10.0.0-next.2
@angular/material                          10.0.0-next.2
@angular/platform-server                   10.0.0-next.6
@angular/pwa                               0.1000.0-rc.0
@ngtools/webpack                           9.1.7
@nguniversal/module-map-ngfactory-loader   9.0.0-next.9
@schematics/angular                        10.0.0-rc.0
@schematics/update                         0.1000.0-rc.0
rxjs                                       6.5.5
typescript                                 3.9.3
webpack                                    4.42.0

Hello @naveedahmed1. Thanks for the issue! This was an intentional change we landed as part of https://github.com/angular/angular/pull/37221#issuecomment-631607065. You can find more details there and in https://github.com/angular/angular/issues/30586.
We will revert the decorator change though as part of another issue that came up with #19454. That should fix these SSR failures. Though ideally, you wouldn't have been affected at all by these decorators as the CLI should have optimizations for this.
Would you still be able to help providing context on this? i.e. do you use the default server builder for doing SSR? Usually the default CLI server builder should remove these decorators and the error should not surface at all. That was part of the reason why we said that this change is okay to land (as CLI consumers should not be affected). Here is the code that removes the decorators.

Thanks @devversion for sharing these details, we are using ngx-build-plus in our Angular app, for server we are using @angular-devkit/build-angular:server but as you mentioned and also mentioned in the comments default builders should handle this, that why are we still seeing this issue?
I have added you to a private repo, if you could please take a look and suggest if I'm missing something?

I believe this should be fixed by the 10.0.0-rc.0 release we published today. Let us know if that's not the case

Yeah, this is fixed quite sure, but I wanted to figure out why it failed in the first place as CLI projects should have never been affected by this change, unless build optimizer is disabled.

I can confirm that the issue has been fixed in 10.0.0-rc.0

@devversion this was the decorator down-leveling issue, no?

@jelbourn Yeah. CLI projects should have not been affected at all in next.2, so either build optimizer code in CLI does not run as expected, or the project has special setup that doesn't run build optimizer. Either way though, downleveling decorators fixes this in SSR as build optimizer would not be needed.
I'm going to check that for the sake of completeness if anything is special in the CLI project, but there is no real action on the Angular components team side.

@naveedahmed1 Thanks for sharing your app. Looks like nothing special in your app. My impression was that CLI would not care about decorators being downleveled or not, but it looks like for the server target, build optimizer is disabled, so it makes sense that this error surfaced. Thanks!


Horizontal Bar removed

Related Issue
Horizontal bar removed
Description
There was a horizontal bar of no use which was occurring because of overflow in 'x', so it was removed.
Type of PR

[x] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)

Checklist:

[ ] I have performed a self-review of my code
[ ] I have read and followed the Contribution Guidelines.
[ ] I have tested the changes thoroughly before submitting this pull request.
[ ] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[ ] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

No need to this type of changes.


Add salvo example

Related to https://github.com/SeaQL/sea-orm/issues/269

@billy1624 Sorry, it's my fault. I have fixed it.


Stylebox "bleeding" outside to the bottom

Operating system or device - Godot version:
Windows 10, AMD R9 390, 3.0 master
Issue description:
Unifying the drawing logic of Tabs and TabContainer, I'm experiencing weird draw results from the tab styleboxes.
The styleboxes aren't defined in any special way. The drawing padding / 9-patch thickness is 4-4-4-0, the content padded a bit further horizontally, and the shadows 2-2-2-0:
t->set_stylebox("tab_fg", "Tabs", sb_expand(make_stylebox(tab_current_png, 4, 4, 4, 0, 8, 4, 8, 0), 2, 2, 2, 0));

Yet the inner part of the image somehow "bleeds" to the bottom - except for the margin parts at the side, resulting in very ugly errors:

I marked the "bleeding" part red on the "Apple" tab.
The Rect2 passed to draw is the green rectangle on the "Banana" tab.
On the TabContainer with the close buttons the effect is also noticeable, the shadowy part of the tabs does not extend under the content panel, but the inner part does (which it should'nt as I draw the content panel below the end of the tab rects).

Does changing the sb_exlands to 2,2,2,1 fix it?

No, sadly not; I tried that already. Any number for the expand margin doesn't change the outcome.

I close this at it is the same issue as https://github.com/godotengine/godot/issues/8366


LaTeX equation not rendering properly

Bug
LaTeX equations are not rendering properly. I thought it was due to a missing "UTF8 statement" in my HTML but it's not
Behavior
This is what I'm getting :
https://i.ibb.co/hCFk59b/rendered.png
Expected behavior
This is what I should get :
https://i.ibb.co/Lh9frf7/render.png
What I already did :
Add
<script src="https://cdn.jsdelivr.net/npm/latex.js/dist/latex.js"></script>
Add
const renderers = {
    latex(node) {
        const generator = new latexjs.HtmlGenerator({ hyphenate: false });
        const { body } = latexjs.parse(node.literal, { generator }).htmlDocument();

        return [
            { type: 'openTag', tagName: 'div', outerNewLine: true },
            { type: 'html', content: body.innerHTML },
            { type: 'closeTag', tagName: 'div', outerNewLine: true }
        ];
    },
}

Add
customHTMLRenderer: renderers in the new Editor({...  array

@2klm
I think It looks not the bug.
Can you add the below css link?
<link
      rel="stylesheet"
<EMAIL_ADDRESS>>

fixed! thx


FALCON-2260 Enhance extension List api to support for user extensions

I have no further comments. Regarding, ExtensionManagerIT, I think it would be a good idea to park a JIRA to ensure trusted extension is not regressed.

Just one comment, the description of the list option needs to be changed:
Currently it is: -list                   List extension jobs and associated entities

@pallavi-rao : I had modified the documentation accordingly and I created the following jira to address the test case.
https://issues.apache.org/jira/browse/FALCON-2262

Thanks @sandeepSamudrala. @PraveenAdlakha, please re-review.

+1


Removed mistakenly added symbol

I assume this symbol should not be there :)

@cla-bot check

@cla-bot check

@cla-bot check

Hi there, I am going to merge this now, but for future reference - in order to properly sign the CLA, make sure you supply the same Github username and email address in the form as the ones you used when you made the commit and pull request.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-27-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3486

Questions ?
Please refer to the Backport tool documentation


[NOT READY TO MERGE] Pushsplits ability.

Just name the config gravitationalPushsplits...
Also, change the description to:
Allows for the ability to perform a "gravitational pushsplit", increasing this value will allow them to be easier to perform, but will cause feeding bugs.


Review README: move index up, merge samples by concept

The change looks bigger than it is. I move the index to the top.
Also rewrote the index section for shopping cart and grpc example.


Feature - Provide the ability for users to "ask" for some columns/models to be documented

A bit more far fetched, but tangata could potentially be used for people to request for models/columns to be documented.
A simplistic solution could be to store a specific string (like TODO) in the schema.yml for the model/column and having the ability to search for those TODO items in tangata.

Relies on #29 - will use meta tags to keep things usable elsewhere in the dbt context.
In the meantime, added a promoted models pane on landing page - this will be a good format to display these also.


[go-livepeer] Re-enable oapi request validation

oapi request validation was disabled in this commit https://github.com/livepeer/go-livepeer/pull/2959/commits/8cc9c5e0dd3f4f8623386098e569c2a1ae1ff7f8 because request validation was rejecting requests that omitted optional parameters.

Tracked internally at https://linear.app/livepeer-ai-spe/issue/LIV-311.


Jaeger Cassandra deployment question

Hi Team
I wanted to use Cassandra as a storage option for Jaeger and have built Cassandra in the Gcp instance group and looking for directions on right template and configmap/deployment yaml to connect jaeger collector pods to Cassandra instances
Any help or reference pointer would help in a big way
@jpkrohling

I was primarily looking for to deploy jager using the operator model within the GKE cluster and would wanted to transport data to Cassandra running on GCE/instance groups.
I have been stuck badly on getting right set of deployment artifacts/yamls to proceed further
Kindly let me know incase of any further questions

My cassandra instance doesn't give a permission to create keyspace and schema by application and it needs to created along with the cassandra bootstrapping - I am just looking for directions on how do i proceed in this use-case @jpkrohling

Creating the schema is described in the documentation: https://www.jaegertracing.io/docs/1.18/deployment/#cassandra

In addition to what @yurishkuro mentioned, here are the sources for the create-schema job that the operator executes while bootstrapping the Cassandra cluster: https://github.com/jaegertracing/jaeger/tree/master/plugin/storage/cassandra/schema .


McMMO Port

Describe what you'd like
An McMMO port for Fabric has been requested several times in Fabricord.

there is already an mcmmo-like mod called RPGStats, it's not the same tho...
It can be a starting point.

Would this be optional?

redcreeper14385: It will be in its separate module, so yes

MCMMO is considering official fabric support in the future:
https://github.com/mcMMO-Dev/mcMMO/issues/4510


Product view page - view by category!

As a user, I want to be able to click on a certain product category and view only products that belong to that category. The store navigation bar should informatively let me know which category I am currently on, and I should be able to click a different category to go to that category.
As an admin, I should be able to browse products by category in the same manner.
As a developer, I want to make sure the database is making efficient queries every time a new product set needs to load, and that the components update as needed. Whenever new products are added or products are updated, these changes need to be reflected for the category views.

Kaitlin and I worked on this issue and created a component named "NavCategory" in "/client/components/nav-category".
We are able to click on a certain product category and view only products that belong to that category. Store navigation bar does not inform of us the category currently viewed. Styles have not yet been added.


Migrated from unittest to pytest

Coverage decreased (-0.5%) to 87.766% when pulling 46811116df4b62cde180932d210c68209a5b51f1 on test-migration into 948b8d2d3fe2ae374986f126dcfc3c0338754f01 on master.

Coverage decreased (-0.5%) to 87.766% when pulling 46811116df4b62cde180932d210c68209a5b51f1 on test-migration into 948b8d2d3fe2ae374986f126dcfc3c0338754f01 on master.

Coverage decreased (-0.5%) to 87.766% when pulling 46811116df4b62cde180932d210c68209a5b51f1 on test-migration into 948b8d2d3fe2ae374986f126dcfc3c0338754f01 on master.

Coverage increased (+9.6%) to 97.872% when pulling 1783b6735e00d59deb27d7df3c89266f902dbc61 on test-migration into 948b8d2d3fe2ae374986f126dcfc3c0338754f01 on master.

Coverage increased (+9.6%) to 97.872% when pulling 1783b6735e00d59deb27d7df3c89266f902dbc61 on test-migration into 948b8d2d3fe2ae374986f126dcfc3c0338754f01 on master.

Coverage increased (+9.6%) to 97.872% when pulling 1783b6735e00d59deb27d7df3c89266f902dbc61 on test-migration into 948b8d2d3fe2ae374986f126dcfc3c0338754f01 on master.

Coverage increased (+9.6%) to 97.872% when pulling 1783b6735e00d59deb27d7df3c89266f902dbc61 on test-migration into 948b8d2d3fe2ae374986f126dcfc3c0338754f01 on master.

Fixes #5

Fixes #12


Update full test + modules update

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/metatdenovo branch on the nf-core/test-datasets repository.
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

I update the test_full.config file + the some modules

there might be minor changes in other modules as well. I will merge this.


Interest in supporting react style event handlers?

This is a nice to have, right now you have to explicitly call the function, where as in react you can pass in just the declaration.
See: https://reactjs.org/docs/handling-events.html

Since VHTML renders to static HTML strings, it is not possible to invoke event handlers. They cannot be serialized to strings.


Received ‘missing ) after argument list’ error, but passes tests.

Challenge http://freecodecamp.com/challenges/waypoint-learn-how-script-tags-and-document-ready-work has an issue. Please describe how to reproduce it, and include links to screenshots if possible.

Yep, I got the same error.

I can pass this test fine....

So I am able to pass the test and will fix the parentheses

Passing the test fixes the parenthesis, but I think the editor checks for any syntax in real-time because it gave me errors as I was writing code.
If that is the case, then I would expect the editor to also detect a lack of any syntax errors in real-time, without me having to click Run Code.

There are () in the description part of the test and there cannot be in the json files... I have fixed a few of these

I do think you are right about it checking while you type in the code... I am not sure how it does that though

@awesomeaniruddh I think you are right on this as I just saw this happen as detailed in the third screenshot below and @natac13 I too would like to know how this happens with some screenshots below from other Waypoints for your reference.

@jwjb Working on that blog now I hope to have it done by days end!

@natac13 Awesome and cannot wait to read it, thanks.

@jwjb Here we go https://natacseanc.wordpress.com/2015/09/07/beginners-guide-to-easy-contributions-to-free-code-camp/ You are the first one to read so any feedback can only help me improve! Thank you for the inspiration and I really hope you after you will be able to fix these issue as well!

@natac13 Posted a variationn on this comment after reading your blog post BEGINNER’S GUIDE TO EASY CONTRIBUTIONS TO FREE CODE CAMP which is really a great read that I will refer to often and hope to use in the future to fix these issues too so thanks a million for documenting your process. Unfortunately, I am unable to give any specific feedback other than to say it looks great and it makes sense, until I become more knowledgeable, but reading through your post and understanding you are a fc3 (Free Code Camp Camper) too, I am inspired to contribute to fcc as well, so thanks for all your efforts and happy coding!


Enhancements

Part of #67.

@danroth27 for design, @javiercn for engineering.

:up::date:
And I just want to double check you're happy with the language on the error page @danroth27. I tweeked it a bit from what was suggested. I'm not 100% happy with it but I don't have any further suggestions.

@ryanbrandenburg LGTM

:up::date:

@Eilon do you approve of the new wording?


Email to full list on Tuesday 10/13 to kick off Global Summit

Please prep test email in civi by next Friday AM for review on Trilogy call. Review specs below to make sure all looks feasible from tech perspective. I'll be on point for copy.
Flag the start of the summit and offer ways to follow along online
Share are our 5 big themes
“We want to hear from you. What do you think is the most important theme we’ll be addressing? Click here to share thoughts”
5 radio buttons with name/email/submit
Register to follow along on live stream (that would help us tackle other github issue around live streaming)?
Follow up for survey responders: send to a fundraising upsell page

Heya @rebeccalendl I'll have to do this.
Please can you provide me with a for the live streaming issues, I'm failing at the internet trying to find that. Thanks.


[Feature Request] Aliases with custom arguments

Is your feature request related to a problem? Please describe.
Not a huge issue, but remembering and typing out commands takes a long time. Especially during streams we need to stop playing the game...reconfigure all the music...then start playing again.
Describe the solution you'd like
I would absolutely love custom aliases! They would be short little custom commands that execute one (or more) longer commands. For example, our server has this running joke about John Cena. It would be awesome to alias the command !cena to execute the following:

!stop
!volume 100
!play https://www.youtube.com/watch?v=blIjjwVgUmE

This would stop the current playlist, pump the volume and play the 🔥 theme song!
Here are some other custom alias ideas...
!battle, stop current music, set high volume, play battle music!
battle = ["stop", "volume 100", "play https://www.youtube.com/watch?v=i649odAK7-M"]

!levelup, stop current playlist, set high volume, play level up sound, set medium volume, play playlist.
levelup = ["stop","volume 100","play https://www.youtube.com/watch?v=8URukvnUYTw", "volume 50", "play playlist chilling"]

!tavern, stop current music, set medium volume, play atmospheric tavern music
tavern = ["stop", "volume 50", "play https://www.youtube.com/watch?v=fIuO3RpMvHg"]

Describe the basic flow/steps of using this feature
How this feature would be used:
The MusicBot admin would setup these custom aliases in config.txt. The would live in the same aliases object with the existing aliases. Each array contains the list of commands to execute in quotes "", for example:
aliases {
    ...
    // Custom Commands
    cena = [ "stop", "volume 100", "!play https://www.youtube.com/watch?v=blIjjwVgUmE" ]
}

Just like standard commands, anyone with the DJ permissions could use them. They use the same prefix !. After typing the custom command !custom the aliased commands are executed in order by as though they had been typed out manually.
Additional context
Nope
Have you confirmed that this idea is not already described here?
https://github.com/jagrosh/MusicBot/projects/1
Yes I searched for custom "alias" and didn't see any related cards.

This is already a feature, take a look at the example config file

Technically one facet of this isn't supported, which is adding arguments as part of the alias, so I'll leave this up

Hi @jagrosh, in the example config I only see options to alias existing commands. That is referring to existing commands by another name, like !play to !p.
// Example 1: Giving command "play" the alias "p":
// play = [ p ]
//
// Example 2: Giving command "search" the aliases "yts" and "find":
// search = [ yts, find ]

This enhancement is about defining custom commands. For example defining the command !custom to execute !play https://www.youtube.com/watch?v=VnwD8zsGl2Y.
If this feature is already supported, that'd be great! Thanks!

Technically one facet of this isn't supported, which is adding arguments as part of the alias, so I'll leave this up

I think macro describes it better as in: run particular commands in a sequence.
That is not possible right now as far as I understand, it would be a great feature though. Sadly my Java foo is to weak right now to even try to implement it.

Any word on whether this feature is planned? It would bring this bot from great to phenomenal.


[Improve][translation] the guide of how-to-release should be translated into English

Description
the guide of how-to-release should be translated into English
Are you willing to submit PR?

[ ] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@jinze1107
Hi, are you still want to work on this issue?


Enabling extension in Edge of the Mist map

I noticed that KYE isn't enabled on Edge of the Mist, which in the arena is a popular place to have RvR fights for guilds. I was wondering what your opinion is on enabling KYE in this map.

I don't think arcdps should be on for RvR's so I am just going to leave it disabled.


Trim the log directory name if it's too long

@lizzha can you share some examples that before and after this change?
@LiliDeng would you validate is it impact legacy_runner of v3?

It doesn't impact v3.
There's a corner case that if we choose multiple test areas, the path will be too long, and when the full path exceeds 256, we cannot create files in the folder.


Update get_transcriptomic_data.py

-tissue_type is renamed as tissue in the biosample output
-Added sra_md5 field in the search url to get the md5sum
-added variable md5 in the module get_data_from_ena to capture the md5sum values from the query
-Added file and md5 fields to be printed in the csv_data
-removed the variable line and changed the tuple row to string in the csv.write line.

To fix travis could you please try to update the python version to 3.8 in https://github.com/Ensembl/ensembl-genes/blob/3ca3ea5fc75e8463a06128e5e26b5d39e9bb9813/pyproject.toml#L18 ?

I'm merging this, we'll fix travis later


[vcpkg-get-python-packages] Fix exported variable

Noticed while trying to use another variable name then PYTHON3 with lensfun.

https://github.com/microsoft/vcpkg/blob/11e79b1072c7f12d84258b77948cda3b166aac59/ports/vcpkg-get-python-packages/x_vcpkg_get_python_packages.cmake#L78-L80
This file isn't installed by python3:x64-windows. So the function still breaks when used with the executable from that port.
https://dev.azure.com/vcpkg/public/_build/results?buildId=98885&view=logs&j=7922e5c4-0103-5f8f-ad17-45ce9bb98e80&t=66eef212-08d6-5e6c-a560-7d01149d0188
CC @Neumann-A

file(COPY "${python_dir}/python3${PYTHON_VERSION_MINOR}.zip" DESTINATION "${venv_path}/Scripts")
probably needs to be generated by the python port itself. Ran already into several issues with embedded python not finding the default scripts but need a good way to compose other modules into that.

Hm. I wonder if the actual problem with this port/function is that pip install bypasses asset caching.

It should only install into the venv not into the installed tree.

It should only install into the venv not into the installed tree.

Yes. And still it is a problem for disconnected builds after --only-downloads. Unlike most tools which also aren't installed into the installed tree.
Pip has its own caching. And it is also possible to pre-download to a local downloads dir and to install later from that dir:
https://pip.pypa.io/en/stable/user_guide/#installing-from-local-packages
Maybe this will eventually need an injection point and instruction on how to prepare and pass in the local dir. Which may require to build python3 first to get matching downloads.
(I'm also looking at jinja2: Repeatedly popping up now as a libsystemd requirement since this port was allowed to enter this registry, https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+libsystemd+jinja2)


Dates need quotes in YAML and they shouldn't

Hey! A bug appeared today. I had this property defined:
  type: object
  properties:
    begins_at:
      description: ISO 8601 date and time the office hold starts.
      type: string
      format: dateTime
      example: 2016-08-04T18:53:38.297Z

The output looked like this:

Notice the example is an empty object!
I wrapped the example in doublequotes and it worked, but I shouldn't need to. The original was valid YAML (according to http://www.yamllint.com/ anyhow):

I have a temporary solution so I'm fine here, but could you take a look into this at some point?

Just experienced the same thing with this field:
    date:
      type: string
      format: date
      example: "2016-08-06"

I thought it might be related to : in the timestamp but seems like -'s trip it too.

regarding your initial issue.
Yaml parser that is used in ReDoc parses dates from yaml to javascript Date instances.
So this is valid yaml, but I'm not sure if it's valid OpenAPI. Although, I can add special handling for this case
Regarding your second issue. Could you add more details?

Hey! Sorry for not getting back to you on this sooner. I've tried recreating the issue now (v1.18.1) and it's not failing without quotes. I did confirm that without quotes that date above is valid Swagger:
$ swagger validate docs/api.yml
docs/api.yml is valid

Not sure if you changed anything but it works. 👍

Thanks for the update!
Glad it works now 😄

@RomanHotsiy
What about using example which is more complex than just the date?
F.e.:
example: {"name": "Name", "start_date": 2018-02-20, "end_date": 2018-02-21} (dates without quotes)
Outputs as:
"start_date": "2018-02-20T00:00:00.000Z", "end_date": "2018-02-21T00:00:00.000Z".

can't you just add quotes?

@RomanHotsiy Adding quotes brings similar results. It seems like it's not solving it in the case of a complex example like the JSON above.

Send please minimal reproducible sample OAS file. I can't reproduce it.

Sure!
You can download it from here:
https://gofile.io/?c=ox1nB9
Or from our API documentation page (URL is navigating to the issue):
https://developers.kenshoo.com/#tag/Ad-Groups/paths/~1api~1v1~1ad_groups/post

@RomanHotsiy any updates?

@RomanHotsiy
As Sharon mentioned you can see the issue here
Or from our API documentation page (URL is navigating to the issue):
https://developers.kenshoo.com/#tag/Ad-Groups/paths/~1api~1v1~1ad_groups/post
should I provide any other information?
Thanks,
Rita


Update apt_packages.txt

Add libgif-dev as a new dependency to be installed on Ubuntu 22.04
Signed-off-by: Rafael Sene<EMAIL_ADDRESS>

LGTM. Thanks.
BTW, this change is proactive as I don't believe anyone on the team has seen it. Correct?

@jjscheel yep; got this one missing while building a brand new dev env for the docs using ubuntu 22.04.


Make it possible to override more options

Allows overriding following options in Cargo.toml:

required_embedded_development_version
other_files
post_install_script

Note that other_files is incompletely implemented, for now it only supports copying files in the same folder as Cargo.toml. In the future we might want to be able to copy folders as well as files located somewhere else than the same folder as Cargo.toml.

🤔
cgi.txt is automatically copied if it exists. Would it be better to use a similar strategy for postinstall.sh? And if we were do to that, should we move them both to a subdirectory and just copy the whole tree, eliminating the other_files option too?
I found all the repetition in the Axis tooling distasteful, and it's possible my suggestion here leaning too far the other way. Then again a thing I love about cargo is how the project layout is always shared between projects; if cargo-acap were to demand a particular layout to avoid configuration, I'd generally be on board for that too.
My ACAP apps also didn't require this functionality. I'm uncertain about other packaging use cases, and specifically which choices would be and wouldn't be ergonomic. Thoughts?

Addendum: one more common reasons I can imagine needing a post-install script is to configure Apache to proxy to an application-provided HTTP server instead of cgi.txt-ing to it. Particularly for Rust apps, proxying makes a ton of sense. Is that why you need it? Is "proxy this to me" a thing we should support directly in cargo-acap?

Addendum: one more common reasons I can imagine needing a post-install script is to configure Apache to proxy to an application-provided HTTP server instead of cgi.txt-ing to it. Particularly for Rust apps, proxying makes a ton of sense. Is that why you need it? Is "proxy this to me" a thing we should support directly in cargo-acap?

Exactly, made an simple example for setting that up here:
https://github.com/johan-bjareholt/acap-rust-http-example
I really want to avoid making a wrapper around axhttp, it's a decent C library but it's not a good library for Rust.

cgi.txt is automatically copied if it exists. Would it be better to use a similar strategy for postinstall.sh? And if we were do to that, should we move them both to a subdirectory and just copy the whole tree, eliminating the other_files option too?
I found all the repetition in the Axis tooling distasteful, and it's possible my suggestion here leaning too far the other way. Then again a thing I love about cargo is how the project layout is always shared between projects; if cargo-acap were to demand a particular layout to avoid configuration, I'd generally be on board for that too.

My guess is that the Axis tooling is designed that way to simply be as flexible as possible. Flexibility can lead to confusion though.
Regarding postinstall.sh it might be a good idea, since there can ever only be one postinstall file.
Regarding otherfiles, I'm not sure. When doing C development, it's pretty common to e.g. put a folder with all .so files in otherfiles as well as bundling static files, and I don't think it's a good idea to mix built files and static files in the same directory.
I could see that to be useful for Rust as well, for example if I would like to bundle GStreamer .so files with a Rust ACAP.
The solution I have right now though don't solve that issue either though, will have to think about another solution...

Regarding otherfiles, I'm not sure. When doing C development, it's pretty common to e.g. put a folder with all .so files in otherfiles as well as bundling static files, and I don't think it's a good idea to mix built files and static files in the same directory (especially if the built files are different depending on architecture).

If we decided that anything in $CARGO_MANIFEST_DIR/foo gets copied verbatim into the .eap package, we could also decide that $OUT_DIR/foo gets copied in too. That would let build scripts in the user's application build or copy target-specific C libraries somewhere to drop them into the .eap.

@willglynn Found some time to update the PR, please take a look at it again!
postinstall.sh is always set, but if it doesn't exist it is ignored on the device.

Is there some progress on this? Post-install script is really a necessary thing to have :_(

@sinnrrr Just use my branch until it is merged, it's very easy to just clone down and then locally do cargo install. I assume that @willglynn is on vacation, so might take some time until it's merged.

I just released cargo-acap v0.1.1 containing these changes.


Radio field inline without label not appear

Package
filament/forms
Package Version
v2.10.20
Laravel Version
v9.3.1
Livewire Version
v2.10.4
PHP Version
PHP 8.1.2
Bug description
If on radio field, include inline, this not appear.
Forms\Components\Radio::make('as')->options(config('global.pago_stat'))->inline()->disableLabel()  <- field not appear
with
Forms\Components\Radio::make('as')->options(config('global.pago_stat'))->inline()->label('pago') <- field appear
Steps to reproduce
Create a form with radio field.
Relevant log output
No response

Fixed by #1933


BUG in store procedure and remove row

SELECT view not working inside store procedure;

example:
BEGIN
select "teste";
END

Remove row not working
example:
on click for remove generation is code

DELETE FROM monitoriZE.host_locais WHERE trap_id = 2 AND box = '' AND service = 'rp' AND name = 'Relogio Ponto' AND type = 'rp' AND id_city = 1 AND id_latlon = 166 AND offset = 1 AND ipmgm = '<IP_ADDRESS>' AND network = '<IP_ADDRESS>/24' AND stat = 'down' AND  statif = null AND countup = 0 AND last_update = '2023-04-14 11:15:41' AND enable = 'Y' LIMIT 1
statif=null not working
statif is null working
USE REMOVE ROW per PRIMARY KEY
sorry my english
I love is client mysql

Hi @domwbz,
I think I understand that one of the two problems you reported occurs when you try to delete a row in a table without a primary key that has a null value.
Regarding the problem relating to procedures, I did not quite understand.
Can you provide some more information? If you have problems with inclese you can also paste me some screenshots or a video/gif.
Thank you very much!

Remove row not working
example:
on click for remove generation is code

DELETE FROM monitoriZE.host_locais WHERE trap_id = 2 AND box = '' AND service = 'rp' AND name = 'Relogio Ponto' AND type = 'rp' AND id_city = 1 AND id_latlon = 166 AND offset = 1 AND ipmgm = '<IP_ADDRESS>' AND network = '<IP_ADDRESS>/24' AND stat = 'down' AND statif = null AND countup = 0 AND last_update = '2023-04-14 11:15:41' AND enable = 'Y' LIMIT 1
statif=null not working
statif is null working

Fixed in 74c136f8334b6972ae55dd8ee0ade09ef8ae3282.


Create a Leaderboard

It is an amazing framework for evaluating LLMs. But there isn't a default Leaderboard, this makes the framework less popular compared to Open LLM Leaderboard, Chat Arena, etc.
I would recommend creating a Leaderboard to increase the popularity of this framework. One possible solution would be to use an average of AR2_Challenge, Hellaswag, MMLU, and TruthfulQA for the leaderboard similar to Open LLM Leaderboard. It would be great to add the AR2_Challenge as proposed in https://github.com/stanford-crfm/helm/issues/1695.
Open LLM Leaderboard is updating their MMLU evaluation https://huggingface.co/blog/evaluating-mmlu-leaderboard. I found that MMLU (original) and MMLU (helm) scores are almost the same even though the evaluation methods are different. I am wondering if this is the same for other tasks. If yes, then the results from this leaderboard and Open LLM Leaderboard could be directly compared, or at least as a good reference.

That's a great suggestion!
We already have pseudo-leaderboards: the main one is this page, where each of the aspects (Accuracy, Calibration, etc.) can be considered a separate leaderboard where the models are ranked by that aspect.
We are missing some things that a leaderboard would have, e.g.:

Visual design and visualizations that makes it look like a leaderboard (see: Papers With Code, Quac, Dynabench, etc.)
Single top-level metric per aspect (win-rate does not fit the bill because it's a relative metric)
Better explanatory notes
Better discoverability

In terms of comparability - this is tricky because we've made lots of small decisions in HELM that diverge from other implementations.
For instance, for MMLU, reproduction would mean running all the subjects (#1335), using the same adapter (#1557), and macro-averaging in the same way (#1328).
I am quite surprised that the scores are so similar though, especially since subject subset choice has a massive impact on the metrics.

@yifanmai This pseudo-leaderboard is way too complex. It is a good reference for professionals, but not good for the general public. It would be good if the leaderboard could be clean and nice.
It is not that surprising that the scores are similar. MMLU (original) calculates the highest accuracy in ["A", "B", "C", "D"], while MMLU (HELM) calculates the highest accuracy in all tokens. A decent LLM should almost predict multiple choice questions in ["A", "B", "C", "D"], so the scores are similar. Actually, MMLU (HELM) should be slightly smaller than MMLU (original).

Definitely agreed that we need to do a better job of summarizing information for the general public.
I'll have to take a look at what OpenLM leaderboard does, but we currently only evaluate 5 of the MMLU subjects, rather than doing a complete evaluation. The accuracy can be change drastically from subject to subject.

@yifanmai Can I know which 5 subjects are used for MMLU HELM evaluation? Open LLM Leaderboard evaluates all 57 subjects. I mean the scores are similar if MMLU (original) and MMLU (HELM) are evaluated on the same subject.

Sure, the 5 subjects are the ones labelled as priority 1 and 2 in run_specs.conf here: abstract_algebra, college_chemistry, computer_security, econometrics, us_foreign_policy
You can also get this information from the MMLU page on the website.

Closed because we now have an actual leaderboard: https://crfm.stanford.edu/helm/lite/latest/#/leaderboard


assign findings to a user to fix

add a feature to allow reviewers to assign issues to a user, so he/she can fix the issues on his/her own.

Hey @matthiasbalke , could you describe the problem in more detail?


Pyautogui does not function on the latest raspberry piOS (Debian Bookworm)

Pyautogui does not function on the latest raspberry piOS based on Debian Bookworm.
I've only tested the piOS, but other Debian Bookworm releases may share the same issue. Pyautogui did work on Debian Bullseye.
Steps to replicate:

Flash a fresh standard full x64 Bookworm based piOS released 10-10-2023 and setup.
Create a python venv and activate it
pip install pyautogui
python
import pyautogui
pyautogui.position() -> (0,0)

You can also do the following :

After a fresh install, run "sudo rm /usr/lib/python3.11/EXTERNALLY-MANAGED" to disable pip protection (please don't do this if you intend to keep using your piOS installation afterwards)
sudo pip install pyautogui
sudo python
import pyautogui
pyautogui.position() -> (0,0)

Problems also persist with pip3.
Mouse movement, clicks and identify position do not function: nothing happens but no errors are raised.
Position will always be 0,0 (or 500,500 depending on your setup).
This issue also affects another mouse module pypi.org/project/mouse/ with identical results for their "mouse.get_position".

The root cause of the incompatibility:
https://www.raspberrypi.com/news/bookworm-the-new-version-of-raspberry-pi-os/

"the move to using Wayland rather than X11 as the display system."

PyAutoGui (and Mouse) use X11 api for their linux support.
As such, the issue is addressed in the following thread:
https://github.com/asweigart/pyautogui/issues/111
I redirect any further discussion regarding piOS or Debian 12.0>= compatibility to the aforementioned Wayland support thread.
The Wayland support thread offer some potential alternatives to pyautogui and mouse functions. Some users in the thread have suggested the following modules: evemu, python-evdev, keyboard, ydotool. Please read the aforementioned thread for further discussion about the features and limitations of each module.

See previous comment.


[wip] Add dhcp hostname option to our dhcp server

Offer the vm name as hostname to the vm
Set hostname of the pod to the vm name to get reasonable dns entries
Set subdomain of the pod th "vms" to give vms their own subdomain in
kube-dns. This avoid clashes with pods which have the same name like
vms.

@vladikr @davidvossel @cynepco3hahue FYI
State:

Tested by hand that fedora accepts the offered hostname (I juse cirros in the test but it ignores the setting)
need to add tests to verify that can reach the vm via .vms.

Signed-off-by: Roman Mohr<EMAIL_ADDRESS>

Just had an irc discussion with @karmab, we think it is best to not use the subdomain right now, since even if we add it ourselves by hand to the vm, pods would not be able to resolve it, since they don't have the subdomain search path either. They would always have to search for myvm.vms.
If there happens to be a name clash between a vm and a pod, something like this will happen: https://github.com/kubernetes/kubernetes/issues/47992#issuecomment-310835820. So only one of  them will be resolvable.
@davidvossel @vladikr thoughts?

ok to test

As a start, what about letting the VM inherit the pod's name at runtime?
If we want to do this, then I'd also consider to

address #782 in order to use pod nams like DOMNAME-45tz
extend our API set a hostname which will then be mapped to PodSpec.hostname (which will then in turn be mapped back to the VM's hostname)

This will keep the whole story pretty clear, also about the interactions between pod and VM.

As a start, what about letting the VM inherit the pod's name at runtime?

We discussed that in #775 and think it is better the other way round.

@fabiand the main argument for going with the vm name is that the pod name needs to be generic and that no one knows the pod name. For vms it is very important that you can easily reach and identify them directly (much more than for pods). The vm name is therefore much more preferable.

However, I am not sure why my dns-related network tests are passing. Without a headless service and a subdomain they should actually fail.

@vladikr I have the suspicion that our network connectivity tests contain an error. It seems like the netcat pod always exits successfully, no matter if it ever received hello-world or not. I added the dns tests to the test table there and forgot to add the headless service. They definitely don't have working dns without it but the tests pass.

@davidvossel @karmab @vladikr could you have another look on this one? Also see the updated PR description on top.

/cc @booxter


consensus: make PBFT serialisation more general

I'm unaware of any specification that N >= K; this module's Haddock only says "The window size itself is pretty much arbitrary and will be fixed by a particular blockchain specification (e.g., Byron)."

Without this patch, deserialization would not round-trip unless the window size
N were at least as large as the security parameter K. That s true in the
current implementation, but it is not necessary in general.

Closing as WRONG. I was confused. It's only inWindow that has length n, both of the pre <> post concatenations have length n + k.

Closing as WRONG. I was confused. It's only inWindow that has length n, both of the pre <> post concatenations have length n + k.
(I've been working on this module's tests, which involve the "old" implementation, which did in fact only hold n signatures.)

:heavy_check_mark:


Writing and learning curves

Solves #153, #165, and #172.

Coverage remained the same at 26.741% when pulling 3afeb3bcda8c1ae47a86b67ed978fbe526982995 on nbgl:master into 383b13e1cddff5990f82b16b59409a38dac1849a on chengsoonong:master.


Layered tilemaps

Not sure if this is the correct repo (might be pxt-common-packages or whatever it's called) but it would be cool to have tilemap layers which is just another tilemap but you can set the z index of it. Then you can also have sprite overlaps on specific z indices and hide and show them.

We don't intend to implement this. Bit too complicated to implement. Current way is set the right tilemap.


Consider forcing HTTPS (or adding an option) for Google Fonts

I've noticed that when I test on localhost I get http:// versions of Google Fonts. As far as I can tell, it's due to this default configuration: https://github.com/typekit/webfontloader/blob/da5faa9f5262188e33000b33b0b9dfc271b5e97c/src/modules/google/fontapiurlbuilder.js#L18
I'd be great if this could just be hard coded to HTTPS rather than protocol-relative. At minimum, It'd be nice to have the option to always use HTTPS regardless.

//link always resolves to your current protocol. If the site you're viewing isn't under SSL it will use http:// if you are it will use https://. If an option was added to force the protocol and it didn't match your current one you'd get a mixed content browser warning.

I understand that.
But there are many cases where you're on HTTP and you still want to load the HTTPS version. That doesn't give a mixed content warning, and is actually what folks still on HTTP should be doing.

If the main page isn't secured what's the purpose of securing includes? Genuinely curious.

We've had reports in the past of users making requests to http://www.google-analytics.com/analytics.js' and getting results for a script that wasn't the official analytics.jsscript. I could never reproduce this myself, when those users switched to requestinghttps://www.google-analytics.com/analytics.js`, they got the correct script. Presumably, since the request wasn't to HTTPS, somewhere in the network pipeline someone was modifying the response. Google has since updated analytics.js to force HTTPS, but other resources still requesting HTTP could experience the same issue.
A second reason is with service worker caching. When testing on localhost (which is technically HTTP even though Service Worker is whitelisted to run on localhost), you end up having to write logic to handle both HTTP and HTTPS, and add both files to the cache (since they're different origins) which is a pain. It's much move convenient to just always request HTTPS.

"The protocol relative URL is now an anti-pattern. If the asset is available on HTTPS, always request it with that."
https://www.paulirish.com/2010/the-protocol-relative-url/

Using HTTPS instead of protocol-relative URL also has considerable performance benefits:
https://jeremywagner.me/blog/stop-using-the-protocol-relative-url

Can we please address this. It's straight up silly as Google now redirects the http to https, causing a reload behind the scenes. It's totally pointless to load it using http anymore.