[fix][s]: Fix none handling for readme data for package list view in snippets.html

refs: #236
Fixed none handling for package readme value.

Will merge now. But let's think about how to do this going forward.


Add function to get PDf from Wiley server based on DOI

What does this PR do?

Add function to get PDf from Wiley server based on DOI
Add corresponding unit test

How can a reviewer manually see the effects of these changes?
Run unit tests with `pipenv run pytest
What are the relevant tickets?

https://mitlibraries.atlassian.net/browse/DLSPP-30

Requires Database Migrations?
NO
Includes new or updated dependencies?
NO

Currently working on putting the CLI from earlier back in with a different PR which should help show the overall flow

CLI is not needed (and in fact not part of the functionality). Just adding the IT is sufficient.

Withdrawing this PR, will refactor based on updated info from Wiley


problems with compiling stan after 2.34.0 on a server

Hello, may I ask you for some possible solutions? I have a VM and run my Stan programs there (it's Centos machine)
There were no problems doing it for Stan 2.33.1, but when I decided to install newer versions, I've got the following error:
src/cmdstan/main.o: In function `tbb::interface7::task_arena::~task_arena()':
main.cpp:(.text._ZN3tbb10interface710task_arenaD2Ev[_ZN3tbb10interface710task_arenaD5Ev]+0x11): undefined reference to `tbb::interface7::internal::task_arena_base::internal_terminate()'
src/cmdstan/main.o: In function `tbb::interface9::global_control::~global_control()':
main.cpp:(.text._ZN3tbb10interface914global_controlD2Ev[_ZN3tbb10interface914global_controlD5Ev]+0x1): undefined reference to `tbb::interface9::global_control::internal_destroy()'
src/cmdstan/main.o: In function `stan::math::init_threadpool_tbb(int)':
main.cpp:(.text._ZN4stan4math19init_threadpool_tbbEi[_ZN4stan4math19init_threadpool_tbbEi]+0x38): undefined reference to `tbb::interface7::internal::task_arena_base::internal_initialize()'
main.cpp:(.text._ZN4stan4math19init_threadpool_tbbEi[_ZN4stan4math19init_threadpool_tbbEi]+0xe6): undefined reference to `tbb::interface9::global_control::internal_create()'
collect2: error: ld returned 1 exit status
make: *** [make/program:80: examples/bernoulli/bernoulli] Error 1
rm examples/bernoulli/bernoulli.o examples/bernoulli/bernoulli.hpp

This error appears with versions 2.34.0 and later. So I wonder if something has been changed at that time.
There is a similar discussion topic on discourse https://discourse.mc-stan.org/t/cmdstan-linking-error-on-linux-server/34133, but I can't use the proposed solution via conda, because I don't have it on my server. Although I may try it or keep working with 2.33.1
My local make file is here:
https://gist.github.com/aakhmetz/5d7a1e8280f11ef11347af5e0a713a82

I've found it. Another system version of TBB was installed (4.1-9.20130314.el7), so I removed it and now it works


FORMS-12203: Added localization changes and test cases

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes and the overall coverage did not decrease.
[ ] All unit tests pass on CircleCi.
[ ] I ran all tests locally and they pass.

Retry tests

@rismehta I have made the changes as per the comments.
Kindly review.


Is this working on Business Process Flow fields, in Customer Service hub?

Hi,
Just ask, is this 'addCustomFilter' working on Business Process Flow fields, in Customer Service hub?
I just tried and it is NOT working.
(it is working fine in traditional Dynamics 365 pages)
Thank you for any help.
Looking forward you reply.
Example codes:
Xrm.Page.getControl("header_process_parentaccountid").addPreSearch(addSiteFilter);
function addSiteFilter() {
var siteFetchQuery = "<filter type='and'><condition attribute='businesstypecode' operator='eq' value='515610000'/></filter>";
Xrm.Page.getControl("header_process_parentaccountid").addCustomFilter(siteFetchQuery);
}
Regards,
Zhenyu

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

ID: 3e69f226-7f17-8146-4044-786991cecd5b
Version Independent ID: 98c694a5-c0ba-c7e9-9e13-9291642af80a
Content: addCustomFilter (Client API reference) in Dynamics 365 Customer Engagement
Content Source: ce/developer/clientapi/reference/controls/addCustomFilter.md
Service: crm-online
GitHub Login: @KumarVivek
Microsoft Alias: kvivek

Thanks for your feedback. We are closing this issue because we are either unable to find relevant information or reproduce this. If you're still experiencing the issue described here, please reopen or create a new issue.


skel/module.el minemacs-core-modules not equal

minemacs-core-modules (skel/module.el) not equal to  (core/me-module.el)
i think maybe core module should not be modified
Should it be removed from module.el
if you want keep it.  it should be equal default  me-defaults & me-bootstrap should be delete
skel/module.el
(setq minemacs-core-modules
      '(me-defaults      ;; Tweak Emacs defaults
        me-splash        ;; Simple splash screen (inspired by emacs-splash)
        me-bootstrap     ;; Bootstrap straight.el
        me-core-ui       ;; Theme and modeline
        me-keybindings   ;; general.el, which-key, hydra, ...
        me-evil          ;; evil, evil-collection, evil-mc, ...
        me-completion))  ;; vertico, marginalia, corfu, cape, consult, ...

core/me-module.el
(defcustom minemacs-core-modules
  '(me-splash        ; Simple splash screen
    me-keybindings   ; Keybinding (general, which-key, hydra, ...)
    me-evil          ; Emacs as Vim (evil, evil-collection, evil-mc, ...)
    me-core-ui       ; Core UI (doom-themes, modus-themes, doom-modeline, ...)
    me-completion)   ; Completion (vertico, marginalia, corfu, cape, consult, ...)
  "MinEmacs enabled core modules."

Yep, you're right. At some time, I've changed the way the modules are loaded. The me-defaults and me-bootstrap cannot be disabled. I should update the skeleton modules file.
Thank you for reporting this

ok thks ~


Infinite Loading Screen -App is Broken

Hi,
I'm running into a critical issue with this app. When I login to the app, I see a spinning loader that is stuck there forever. I haven't made any changes to the app and I'm using the 1.0.7 version from the master branch. I've tried all other versions and none are working.
I'm using thingsboard.cloud which seems to be 3.6.1PAAS
This is very worrying as it seems to render the app completely useless.
Thanks

Hello @macso-philipperivard!
You mentioned you haven't made any changes, but you need to change the ThingsBoard endpoint as mentioned there in Step 3. .
Please do it and try again :)

I have done that already of course. If you actually run this app on your system you will notice the same issue as I notice.

It works well for me.
Could you please provide the following details?
**Platform type and version (CE, PE, or Cloud with corresponding version; DB type)
**Deployment type and hosting OS
**Steps to reproduce
**Code/client used to achieve the result (with payload example, auth type)
**The desired behavior of the platform or, basically, the user story
**The nature of the question and end goals you have in mind
**Modifications or custom code done
@macso-philipperivard.


[Feature Request] : Video support

Many many thanks for this great package :+1:
Video inclusion support could be the missing piece of this excellent puzzle.
See for example the multimedia or the media9 Latex packages.
Touying is the best slides package for typst IMHO.
Wish you the best !

Typst upstream doesn't natively support video, so neither does touying. But you can use GIFs and Typst Preview extension to display them.
---Original---
From: "François @.>
Date: Sun, Feb 18, 2024 00:08 AM
To: @.>;
Cc: @.***>;
Subject: [touying-typ/touying] [Feature Request] : Video support (Issue #6)
Many many thanks for this great package 👍
Video inclusion support could be the missing piece of this excellent puzzle.
See for example the multimedia or the media9 Latex packages.
Touying is the best slides package for typst IMHO.
Wish you the best !
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thanks @OrangeX4  for this feedback I was not knowing this.
I've just tried with an animated  .gif, the image is indeed displayed but not "animated" in regular pdf clients.
Don't know if a special presentation tool is needed to the the animated .gif.
Anyway many thanks for your excellent package !

You can use slide mode https://touying-typ.github.io/touying/docs/external/typst-preview
Thanks for liking touying.
---Original---
From: "François @.>
Date: Sun, Feb 18, 2024 03:28 AM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [touying-typ/touying] [Feature Request] : Video support (Issue#6)
Thanks @OrangeX4  for this feedback I was not knowing this.
I've just tried with an animated  .gif, the image is indeed displayed but not "animated" in regular pdf clients.
Don't know if a special presentation tool is needed to the the animated .gif.
Anyway many thanks for your excellent package !
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Many thanks @OrangeX4 , it works indeed well with slide mode for vscode :+1:

It looks like video/multimedia support is on its way, the following PR is just merged
https://github.com/typst/pdf-writer/pull/31

Directed related: https://github.com/Enter-tainer/typst-preview/issues/242
@fbob @OrangeX4

You can use slide mode https://touying-typ.github.io/touying/docs/external/typst-preview Thanks for liking touying.
…
---Original--- From: "François @.> Date: Sun, Feb 18, 2024 03:28 AM To: @.>; Cc<EMAIL_ADDRESS>Subject: Re: [touying-typ/touying] [Feature Request] : Video support (Issue#6) Thanks @OrangeX4 for this feedback I was not knowing this. I've just tried with an animated .gif, the image is indeed displayed but not "animated" in regular pdf clients. Don't know if a special presentation tool is needed to the the animated .gif. Anyway many thanks for your excellent package ! — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>


TestCentric.Cake.Recipe release 1.3.0 has no content

Due to an error in the release process, the content folder, which actually contains all the recipe's cake files is not present in the nuget package. A new 1.3.1 release is needed and 1.3.0 will be de-listed.

:tada: This issue has been resolved in version 1.3.1 :tada:
The release is available on:

GitHub Release
NuGet Package


config({timeout: 100}) fails on Node v18

I'm trying to set a custom timeout
 const sdk = require('api')('@...');
 sdk.config({ timeout: 100 });

but getting an error:
>> ERROR:
TypeError [ERR_INVALID_ARG_TYPE]: The "emitter" argument must be an instance of EventEmitter or EventTarget. Received an instance of AbortSignal
    at Object.fetch (node:internal/deps/undici/undici:11404:11)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)

using node v18.14.1 (npm v9.3.1) and api v5.0.7
Have tried running the same code on Node v19 and it worked fine.

@ignasv Can you check if it's still a problem on Node v18.14.2? It looks like they updated their undici dependency to resolve some issues with timeouts.

Getting the same error on v18.14.2

I have Node 18.16.0 installed locally and am no longer seeing this:
const sdk = require('./dist')('@developers/v2.0#3p5ha36lk2u2qoa');
sdk.config({ timeout: 100 });

sdk.getOpenRoles()
  .then(({ data }) => console.log(data))
  .catch(err => console.error(err));

DOMException [AbortError]: This operation was aborted
    at Object.fetch (node:internal/deps/undici/undici:11457:11)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

It wouldn't surprise me if there was a bug in undici in the 18.14 family that's since been fixed.


cannot parse XML as table

Hi @LaurensRietveld !

http://data.culture.fr/thesaurus/sparql supports only GET, returns only XML (disregards Accept header), and doesn't return the precise Content-Type expected.
I used the old yasgui (http://yasgui.org/short/-NJ26ximw) to display a simple table from that endpoint.
I tried with the new yasgui (https://api.triplydb.com/s/mIXe4sbf) but it displays raw XML instead of a table. Maybe it's looking for the precise Content-Type whereas the server returns application/xml?

$ curl -I  "http://data.culture.fr/thesaurus/sparql?query=PREFIX+skos%3A+%3Chttp%3A%2F%2Fwww.w3.org%2F2004%2F02%2Fskos%2Fcore%23%3E%0D%0Aselect+%3Fprop+%3Fdomain+%28count%28*%29+as+%3Fc%29+%7B%0D%0A++values+%3Fp+%7Bskos%3ArelatedMatch+skos%3AcloseMatch+skos%3AexactMatch%7D%0D%0A++%3Fx+skos%3AinScheme+%3Chttp%3A%2F%2Fdata.culture.fr%2Fthesaurus%2Fresource%2Fark%3A%2F67717%2FT69%3E%3B+%3Fp+%3Fo.%0D%0A++bind%28replace%28str%28%3Fp%29%2C%22%5E.*%23%28.*%29Match%24%22%2C%22%241%22%29+as+%3Fprop%29%0D%0A++bind%28replace%28str%28%3Fo%29%2C%22%5E%28.*%29%2F.*%24%22%2C%22%241%22%29+as+%3Fdomain%29%0D%0A%7D+group+by+%3Fprop+%3Fdomain+order+by+%3Fdomain+%3Fprop%0D%0A"
HTTP/1.1 200 OK
Date: Thu, 09 Jan 2020 13:59:18 GMT
Content-Type: application/xml

Thanks a lot, before migrating, I'll make sure to have fixed this :+1:

This is now fixed

@LaurensRietveld the link https://api.triplydb.com/s/mIXe4sbf still shows the same, I guess the fix is not yet deployed?


Asynchronous requests for mix_data_collector

As the size of current portfolios is a bit bigger than I though, doing requests asynchronously is currently considered.

The efficiency drop of treating non-trading days (which is still a temporary fix) would be solved together.


Hosting

How can I host this bot myself?

host it with your bot token, CLOSED

There is an egg for this on pterodactyl if you use that.


Ability to sign in to Datasette as a root account

I'm going to draw the line here: default Datasette supports authentication but only for a single user account ("admin"). Plugins can then add support for multiple user accounts, social auth, SSO etc.

Originally posted by @simonw in https://github.com/simonw/datasette/issues/699#issuecomment-636498770

I'm calling this the root account now, for reasons discussed in these two comments: https://github.com/simonw/datasette/issues/699#issuecomment-636510647

For the first version of this I'm not going to use passwords at all. I'll implement this:
$ datasette fixtures.db --root

The --root option will cause Datasette to output a URL with a one-time-use token in it which, when clicked, will authenticate the user as the root account (by setting a signed cookie).
Signed cookie means Datasette needs a secrets recipe. I'll open a new issue for that.

The URL for this will be:
/-/auth-token?token=xxx
The token will be generated by Datasette on startup and will only be valid for a single request, at which point it will be used to set a signed ds_actor cookie and then redirect to the homepage.

I'm considering this done. I'm going to leave it to plugins to implement a web-based sign-in flow for accounts (at least for the moment).

Documentation: https://datasette.readthedocs.io/en/stable/authentication.html#using-the-root-actor


Added support for "AI" Eco

How does "AI mode" works? Is it linked with the lack ok nanoe in last version? How can we control nanoe from integration? Function is still present in app ...


Add polling option for watching files

🚀 Feature Proposal
As an alternative to using file streams, add an option to poll for files for legacy compatibility.
Motivation
A lot of other tools offer this options (Webpack, nodemon). My current use-case is that docker volumes on windows don't support file watchers: https://github.com/docker/for-win/issues/56
Example
jest --watchAll poll would do the trick 😄
Pitch
Compatibility from other systems 👍

Hey @rickhanlonii,
For someone whom is not familiar with the Jest codebase, what package makes sense for this feature? jest-watcher?

jest-haste-map 🙂

@SimenB Thanks!

Actually, I'm going to bow out of this one. Found a workaround that suits me just fine, so I can no longer justify the time spent here. (For future googler's with the same issue: https://github.com/dzek69/docker-windows-notifier)

@rickhanlonii Thanks mate, I'll do some more investigations.

or :
...
"scripts": {
  "test:watch": "onchange '**/*.test.js' -i -v -p -- npm run test",
  "test": "jest"
}

+1 to this... I'm having the same problem while developing using Docker for Windows...
For nodemon you can use this: nodemon -L and for React you can use this: CHOKIDAR_USEPOLLING=true react-scripts start, but there's no alternative for jest ;__;
I'm just going to leave another workaround alternative here too: https://github.com/merofeev/docker-windows-volume-watcher


New command: Get profile card property

Usage
m365 tenant people profilecardproperty get [options]
Description
Retrieves information about a specific profile card property
Options

Option
Description

-n, --name <name>
The name of the property to retrieve. Allowed values: userPrincipalName, faxNumber, streetAddress, postalCode, state, mailNickname, customAttribute1, customAttribute2, customAttribute3, customAttribute4, customAttribute5, customAttribute6, customAttribute7, customAttribute8, customAttribute9, customAttribute10, customAttribute11, customAttribute12, customAttribute13, customAttribute14, customAttribute15.

Examples
Retrieve information about a specific profile card property
m365 tenant people profilecardproperty get --name customAttribute1

Default properties
No response
Additional Info
Request:
GET https://graph.microsoft.com/v1.0/admin/people/profileCardProperties/{profile card property name}

https://learn.microsoft.com/en-us/graph/api/profilecardproperty-get?view=graph-rest-beta&tabs=http

Looks fine!

Nice! How are we going to display the information in non-JSON output modes where we can't show nested properties?

Nice! How are we going to display the information in non-JSON output modes where we can't show nested properties?

I'd say: just as we do normally: so without the nested properties...

Will be tough to make it non-JSON output-friendly.
Only thing that could work is by transforming the object to something like:
{
  "directoryPropertyName": "customAttribute1",
  "displayName": "Cost center",
  "nl-NL": "Kostencentrum"
}

But not sure if this would be clear enough.

Hmmm, i do like how that looks though.
We generally pipe through the API output one on one though. Let's keep it that way to stay consistent, that would be my stance.

Hmmm, i do like how that looks though.
We generally pipe through the API output one on one though. Let's keep it that way to stay consistent, that would be my stance.

Yes definitely, but this would be only for non-JSON outputs. We should leave the JSON output untouched.

+1 on that formatting. That would greatly improve readability for the outputs other than JSON.

Will be tough to make it non-JSON output-friendly.
Only thing that could work is by transforming the object to something like:
{
  "directoryPropertyName": "customAttribute1",
  "displayName": "Cost center",
  "nl-NL": "Kostencentrum"
}

But not sure if this would be clear enough.

If you would have a language code with two letters, it would not be very clear:
@odata.context               : https://graph.microsoft.com/v1.0/$metadata#admin/people/profileCardProperties/$entity
de                                    : Kostplatz
directoryPropertyName  : customAttribute1
displayName                   : Cost center
nl-NL                               : kostenplaats

What if we picked displayName-nl-NL etc, just like in the options?

Then didplayName nl-NL would be more clear according to me. Without the dash.

We should also drop @odata.context from the text output as it doesn't add any value

We should also drop @odata.context from the text output as it doesn't add any value

With the right accept header, this property is not returned.


Fix GHC not building with musl

Motivation for this change
Fixes binary GHCs segfaulting on pkgsMusl. See #129247, #118731.
All Nix-built Haskell compilers build now on pkgsMusl:
NIX_PATH=nixpkgs=. nix-build --no-link --expr 'with import <nixpkgs> {}; { inherit (pkgsMusl.haskell.compiler) ghc884 ghc8104 ghc901 ghcHEAD; }'

Key changes included (see commits for details):

Switch ghc8102Binary to use GHC HQ musl bindist instead of patching glibc bindist with musl.
Use this better binary bindist to bootstrap all nix-built GHCs for pkgsMusl
Add missing hardeningDisable = [ "pie" ] where needed.
Swith to build GHC without sphinx by default for musl (cross already does this) to avoid huge amount of build dependencies, some of which may not work with musl yet (or break frequently). This will reduce maintenance burden for projects like static-haskell-nix significantly, and make it cheaper to build musl-based GHC on Hydra.

Add new options to control this.

Add new structure to declare which ncurses version and similar dependencies the bindists have, and a postUnpack step to ensure at build time that those are actually correct.

The build now complains when your expectation is wrong regarding what ncurses a bindist needs.

Fix i686 build of ghc8102Binary as a drive-by contribution.
Lots of explanatory extra comments.

Extra review needed for

My usage of buildPlatform, targetPlatform etc.
There are some some cross options that I slightly touch. People who care about cross Haskell should ideally check out if everything remains working.

TODO

[x] I need some help to make the Hydra evaluation correct, see top WIP commit.

provided by @sternenseemann in https://github.com/NixOS/nixpkgs/pull/129289#pullrequestreview-699418026

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase 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.

CC people who care about this and may want to try whether it breaks anything, and others qualified to review:
@NixOS/haskell @expipiplus1 @peti @bgamari @nomeata @domenkozar @cdepillabout @dtzWill @lunaris @NorfairKing @maralorn @sternenseemann @dtzWill @Ericson2314 @matthewbauer @aveltras

Looks like I broke the nixpkgs manual with my new doc-related config options:
ghc.mk:184: *** BUILD_SPHINX_PDF=YES, but `xelatex` was not found. Install `xelatex`, then rerun `./configure`.

https://github.com/NixOS/nixpkgs/runs/2986064008?check_suite_focus=true#step:5:3174
Not sure why yet.

Will need to target staging as this is a mass rebuild (probably via  pandoc? I haven't investigated yet).

Well pandoc is not really the point here. I believe we use the 8102 binary to build our 8104 ghc? So this is a full haskellPackages rebuild. We normally do those on haskell-updates, so I don‘t think this needs to target staging, but haskell-updates.

Result of nixpkgs-review pr 129289 at a84215ba run on aarch64-linux 1

29 packages marked as broken and skipped:

grass
libvmi
mvapich
python38Packages.dask-gateway
python38Packages.dask-jobqueue
python38Packages.dask-ml
python38Packages.dask-mpi
python38Packages.dask-xgboost
python38Packages.datashader
python38Packages.fipy
python38Packages.pygmt
python38Packages.streamz
python38Packages.stumpy
python38Packages.tensorflowWithCuda
python39Packages.dask-gateway
python39Packages.dask-jobqueue
python39Packages.dask-mpi
python39Packages.dask-xgboost
python39Packages.fipy
python39Packages.pygmt
python39Packages.streamz
python39Packages.tensorflowWithCuda
qemu_xen
qemu_xen-light
qemu_xen_4_10
qemu_xen_4_10-light
qlandkartegt
qubes-core-vchan-xen
yosys-bluespec

6987 packages skipped due to time constraints:

EBTKS
agdaPackages.agda-categories
agdaPackages.agda-prelude
agdaPackages.cubical
agdaPackages.functional-linear-algebra
agdaPackages.generic
agdaPackages.iowa-stdlib
agdaPackages.standard-library
apacheHttpdPackages.mod_tile (apacheHttpdPackages_2_4.mod_tile)
apostrophe
...

2 packages built successfully:

argp-standalone
haskell.compiler.ghc8102BinaryMinimal

5 suggestions:

warning: missing-patch-comment
Consider adding a comment explaining the purpose of this patch on the line preceeding.
Near pkgs/development/libraries/argp-standalone/default.nix:31:53:
   |
31 |        lib.optionals stdenv.hostPlatform.isDarwin [ patch-argp-fmtstream ]
   |                                                     ^

warning: missing-patch-comment
Consider adding a comment explaining the purpose of this patch on the line preceeding.
Near pkgs/development/libraries/argp-standalone/default.nix:32:52:
   |
32 |     ++ lib.optionals stdenv.hostPlatform.isLinux [ patch-throw-in-funcdef patch-shared ];
   |                                                    ^

warning: name-and-version
Did you mean to pass pname instead of name to mkDerivation?
Near pkgs/development/compilers/ghc/8.10.2-binary.nix:135:3:
    |
135 |   name = "ghc-${version}-binary";
    |   ^

Near pkgs/development/compilers/ghc/8.10.2-binary.nix:133:10:
    |
133 |   inherit version;
    |          ^

warning: unclear-gpl
gpl2 is a deprecated license, please check if project uses gpl2Plus or gpl2Only and change meta.license accordingly.
Near pkgs/development/libraries/argp-standalone/default.nix:59:5:
   |
59 |     license = licenses.gpl2;
   |     ^

warning: missing-patch-comment
Consider adding a comment explaining the purpose of this patch on the line preceeding.
Near pkgs/development/libraries/argp-standalone/default.nix:32:75:
   |
32 |     ++ lib.optionals stdenv.hostPlatform.isLinux [ patch-throw-in-funcdef patch-shared ];
   |                                                                           ^

I don‘t think this needs to target staging, but haskell-updates.

Done.

I've addressed all feedback.
static-haskell-nix is working quite well with these changes: https://github.com/nh2/static-haskell-nix/pull/98
Ready to merge to haskell-updates from my side.


Bug : selfhosted, can't redirect the /api route to a specific port

Describe the bug
i am hosting the oneUpTime locally using docker on my server local IP and a specific port 8080 and i can't get the dashboard to work. i am getting this error :

To Reproduce
Steps to reproduce the behavior:

setup the .env to use a static ip and a local port in the .env,
run the docker compose command

Expected behavior
i can access the dashboard
Screenshots

Desktop (please complete the following information):

OS: macOs 12
Browser: chrome

Deployment Type
self hosted using docker

You need to specify the port as well in the HOST env var


DND to download out of codespaces doesn't work

Drag a file from the explorer to Finder

It tries to download, but the browser says "Couldn't download - No File"

But right click -> download works fine

Yeah we could not get this to work via the chrome data transfer protocol, thus we adde the download action as an alternative solution sort of.
So this is a limitation right now.

It also is not working for Desktop + SSH for me right now, which I think used to work?
But in cases where it doesn't work is it possible to disable the DND entirely?

Yeah we should disable when we know it does not work, there is another issue for this.
Next milestone I can discuss to put this on the plan so I have time.

Desktop + SSH no longer works because previously we had code that handled VS Code remote resources here. However @alexdima removed that code because it was a temporary workaround and there are no plans to support this again.
As for the drag and drop from Codespaces, the following happens: we detect a remote scheme and write the download_url here and we rewrite the URI, however when the file is downloaded the URI can not be accessed and thus the download can not happen.
So it seems that the only way for this to work was if the URL to access a particular resource in Codespaces was accessible from the outside which is not.
It is not possible to write a full content of the file in the downloadURL data transfer type. Reference
Thus I do not see how we can fix this. I am open for ideas though. @bpasero might have some since he looked in this area recently.
We can not disable the DND entirely because it is still possible to DND inside the explorer from one folder to another. And when the user starts the drag we do not know where he will want to drop..

@isidorn only in the desktop case: the vscode-remote-resource: URL could be converted to an http: URL by doing url.replace(/^vscode-remote-resource:/, 'http:')

however when the file is downloaded the URI can not be accessed and thus the download can not happen

I think why this happens is the interesting question because the code seems to be doing the right thing:
https://github.com/microsoft/vscode/blob/7a767570e85a4a3eec1c86bfa8d7b88067a87fa5/src/vs/workbench/browser/dnd.ts#L277-L277
Specifically FileAccess.asBrowserUri is proper for creating a link that can be embedded into the browser and I verified that in Codespaces (web) this produces a HTTPS URI.
I believe this may be an issue with service workers implementing that file access in the Codespace case (e.g. https://bugs.chromium.org/p/chromium/issues/detail?id=741778&q=service worker download&can=2)

Just for reference, here's the example HTTPS URI that gets created when I DnD an app.js from the smoke test express codespace:
https://isidorn-microsoft-vscode-smoketest-express-m4hq.github.dev/assets/35436398758ECA9A1BFEB672CCDC768E9785/40955/vscode-remote-resource?path=%2Fworkspaces%2Fvscode-smoketest-express%2Fapp.js&tkn=SOME_TOKEN
If I use this URI just from the browser I can not access the file. Not sure if it should just work but I would expect the file got get download if I just access it.

This may require a service worker to be running to implement the fetch, not sure it would work directly from the browser address bar.
//cc @legomushroom


GFGP Implementation Cleanup

This PR cleans up the original implementation of GFPG

FreeMCG PR merged, so this PR can be closed.


New Contribution

Contributor:<PHONE_NUMBER>
SSH Public key: ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIBoxlLCty8xOw+WE9jx3LxjhL+2OHTJrCTjInLFVaWZ7
Attestation: [any form of attestation for contribution (e.g., signature) - Optional]

Hi sir, @zvorcuit I am ready to contribute, thx~

why killed?

@zvorcuit Hi sir, I'm ready now, can I contribute?


Fails to compile against MuPDF

Tried to compile on a raspberry pi. I assume this is some sort of version compatibility issue.
Compiling ipptransform.c...
ipptransform.c: In function ‘xform_document’:
ipptransform.c:2337:12: error: incompatible type for argument 1 of ‘fz_scale’
fz_scale(&base_transform, xscale, yscale);
^~~~~~~~~~~~~~~
In file included from /usr/include/mupdf/fitz/context.h:6,
from /usr/include/mupdf/fitz.h:11,
from ipptransform.c:23:
/usr/include/mupdf/fitz/geometry.h:305:26: note: expected ‘float’ but argument is of type ‘fz_matrix *’ {aka ‘struct fz_matrix_s *’}
fz_matrix fz_scale(float sx, float sy);
^~
ipptransform.c:2337:3: error: too many arguments to function ‘fz_scale’
fz_scale(&base_transform, xscale, yscale);
^~
In file included from /usr/include/mupdf/fitz/context.h:6,
from /usr/include/mupdf/fitz.h:11,
from ipptransform.c:23:
/usr/include/mupdf/fitz/geometry.h:305:11: note: declared here
fz_matrix fz_scale(float sx, float sy);
^~~~~~~~
ipptransform.c:2346:40: error: incompatible type for argument 2 of ‘fz_new_draw_device’
device = fz_new_draw_device(context, &base_transform, pixmap);
^~~~~~~~~~~~~~~
In file included from /usr/include/mupdf/fitz.h:46,
from ipptransform.c:23:
/usr/include/mupdf/fitz/device.h:329:12: note: expected ‘fz_matrix’ {aka ‘struct fz_matrix_s’} but argument is of type ‘fz_matrix *’ {aka ‘struct fz_matrix_s *’}
fz_device *fz_new_draw_device(fz_context *ctx, fz_matrix transform, fz_pixmap *dest);
^~~~~~~~~~~~~~~~~~
ipptransform.c:2409:7: error: too many arguments to function ‘fz_bound_page’
fz_bound_page(context, pdf_page, &image_box);
^~~~~~~~~~~~~
In file included from /usr/include/mupdf/fitz.h:56,
from ipptransform.c:23:
/usr/include/mupdf/fitz/document.h:475:9: note: declared here
fz_rect fz_bound_page(fz_context *ctx, fz_page *page);
^~~~~~~~~~~~~
ipptransform.c:2528:21: error: incompatible type for argument 1 of ‘fz_pre_translate’
fz_pre_translate(&transform, 0.0, -1.0 * y / yscale);
^~~~~~~~~~
In file included from /usr/include/mupdf/fitz/context.h:6,
from /usr/include/mupdf/fitz.h:11,
from ipptransform.c:23:
/usr/include/mupdf/fitz/geometry.h:417:38: note: expected ‘fz_matrix’ {aka ‘struct fz_matrix_s’} but argument is of type ‘fz_matrix *’ {aka ‘struct fz_matrix_s *’}
fz_matrix fz_pre_translate(fz_matrix m, float tx, float ty);
^
ipptransform.c:2531:16: error: incompatible type for argument 1 of ‘fz_concat’
fz_concat(&transform, &transform, &back_transform);
^~
In file included from /usr/include/mupdf/fitz/context.h:6,
from /usr/include/mupdf/fitz.h:11,
from ipptransform.c:23:
/usr/include/mupdf/fitz/geometry.h:290:31: note: expected ‘fz_matrix’ {aka ‘struct fz_matrix_s’} but argument is of type ‘fz_matrix *’ {aka ‘struct fz_matrix_s *’}
fz_matrix fz_concat(fz_matrix left, fz_matrix right);
^~
ipptransform.c:2531:28: error: incompatible type for argument 2 of ‘fz_concat’
fz_concat(&transform, &transform, &back_transform);
^~
In file included from /usr/include/mupdf/fitz/context.h:6,
from /usr/include/mupdf/fitz.h:11,
from ipptransform.c:23:
/usr/include/mupdf/fitz/geometry.h:290:47: note: expected ‘fz_matrix’ {aka ‘struct fz_matrix_s’} but argument is of type ‘fz_matrix *’ {aka ‘struct fz_matrix_s *’}
fz_matrix fz_concat(fz_matrix left, fz_matrix right);
^
ipptransform.c:2531:6: error: too many arguments to function ‘fz_concat’
fz_concat(&transform, &transform, &back_transform);
^
In file included from /usr/include/mupdf/fitz/context.h:6,
from /usr/include/mupdf/fitz.h:11,
from ipptransform.c:23:
/usr/include/mupdf/fitz/geometry.h:290:11: note: declared here
fz_matrix fz_concat(fz_matrix left, fz_matrix right);
^~~~~~~~~
ipptransform.c:2533:14: error: incompatible type for argument 1 of ‘fz_concat’
fz_concat(&transform, &transform, &image_transform);
^~~~~~~~~~
In file included from /usr/include/mupdf/fitz/context.h:6,
from /usr/include/mupdf/fitz.h:11,
from ipptransform.c:23:
/usr/include/mupdf/fitz/geometry.h:290:31: note: expected ‘fz_matrix’ {aka ‘struct fz_matrix_s’} but argument is of type ‘fz_matrix *’ {aka ‘struct fz_matrix_s *’}
fz_matrix fz_concat(fz_matrix left, fz_matrix right);
^~
ipptransform.c:2533:26: error: incompatible type for argument 2 of ‘fz_concat’
fz_concat(&transform, &transform, &image_transform);
^~
In file included from /usr/include/mupdf/fitz/context.h:6,
from /usr/include/mupdf/fitz.h:11,
from ipptransform.c:23:
/usr/include/mupdf/fitz/geometry.h:290:47: note: expected ‘fz_matrix’ {aka ‘struct fz_matrix_s’} but argument is of type ‘fz_matrix *’ {aka ‘struct fz_matrix_s *’}
fz_matrix fz_concat(fz_matrix left, fz_matrix right);
^
ipptransform.c:2533:4: error: too many arguments to function ‘fz_concat’
fz_concat(&transform, &transform, &image_transform);
^~~
In file included from /usr/include/mupdf/fitz/context.h:6,
from /usr/include/mupdf/fitz.h:11,
from ipptransform.c:23:
/usr/include/mupdf/fitz/geometry.h:290:11: note: declared here
fz_matrix fz_concat(fz_matrix left, fz_matrix right);
^~~~~~~~~
ipptransform.c:2541:50: error: incompatible type for argument 4 of ‘fz_run_page’
fz_run_page(context, pdf_page, device, &transform, NULL);
^~~~~~~~~~
In file included from /usr/include/mupdf/fitz.h:56,
from ipptransform.c:23:
/usr/include/mupdf/fitz/document.h:496:76: note: expected ‘fz_matrix’ {aka ‘struct fz_matrix_s’} but argument is of type ‘fz_matrix *’ {aka ‘struct fz_matrix_s *’}
void fz_run_page(fz_context *ctx, fz_page *page, fz_device *dev, fz_matrix transform, fz_cookie *cookie);
~~^
make[1]: *** [../Makedefs:142: ipptransform.o] Error 1
make: *** [Makefile:27: all] Error 1

@WesGilster What OS? What version of MuPDF?

Builds fine on my Raspberry Pi Zero W running Raspbian Buster:
pi@pi-zero-w-1:~/src/istopwg/ippsample $ cat /etc/os-release
PRETTY_NAME="Raspbian GNU/Linux 10 (buster)"
NAME="Raspbian GNU/Linux"
VERSION_ID="10"
VERSION="10 (buster)"
VERSION_CODENAME=buster
ID=raspbian
ID_LIKE=debian
HOME_URL="http://www.raspbian.org/"
SUPPORT_URL="http://www.raspbian.org/RaspbianForums"
BUG_REPORT_URL="http://www.raspbian.org/RaspbianBugs"
pi@pi-zero-w-1:~/src/istopwg/ippsample $ apt -qq list libgnutls* mupdf*
libgnutls-dane0/stable,now 3.6.7-4+deb10u7 armhf [installed,automatic]
libgnutls-openssl27/stable,now 3.6.7-4+deb10u7 armhf [installed,automatic]
libgnutls28-dev/stable,now 3.6.7-4+deb10u7 armhf [installed]
libgnutls30/stable,now 3.6.7-4+deb10u7 armhf [installed]
libgnutlsxx28/stable,now 3.6.7-4+deb10u7 armhf [installed,automatic]
mupdf-tools/stable,now 1.14.0+ds1-4+deb10u3 armhf [installed]
mupdf/stable,now 1.14.0+ds1-4+deb10u3 armhf [installed]

We've dropped MuPDF for ippsample...


DisplayServer.tts_get_voices() returns empty on Linux with official binaries

Godot version
4.0.beta3
System information
Linux Xubuntu 22.04.01
Issue description
DisplayServer.tts_get_voices() returns empty. For example, print(DisplayServer.tts_get_voices()) prints "[]".
Speech Dispatcher is installed (spd-say 0.11.1), spd-say -L returns a long list of voices, spd-say "hi" says hi as expected.
Steps to reproduce
print(DisplayServer.tts_get_voices())
Minimal reproduction project
TTSexample.zip

Speech Dispatcher is installed (spd-say 0.11.1), spd-say -L returns a long list of voices, spd-say "hi" says hi as expected.

If you built the engine from source, did you have speech-dispatcher development libraries installed at the time of building? The build system should print a message stating that it wasn't found otherwise.

Hi - thanks very much for the reply. I downloaded the binaries rather than
built from source. Many thanks
On Tue, 25 Oct 2022 at 18:39, Hugo Locurcio @.***>
wrote:

Speech Dispatcher is installed (spd-say 0.11.1), spd-say -L returns a long
list of voices, spd-say "hi" says hi as expected.
If you built the engine from source, did you have speech-dispatcher
development libraries installed at the time of building? The build system
should print a message stating that it wasn't found otherwise.
—
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/67863#issuecomment-1290917162,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3ZZQSCDV3QZOGS6OH4Z46LWFALOLANCNFSM6AAAAAARNVQVX4
.
You are receiving this because you authored the thread.Message ID:
@.***>

Seems like official betas are built without speechd enabled, it's probably result of missing pkg-config file, many distros seems to be shipping speechd without it.

Thanks bruvzg for diagnosing the cause. Hopefully it will get resolved in due course, in the meantime I've just called spd-say using OS.execute as a short-term workaround.

I came here while testing lesleyrs/clipboard-narrator with Godot 4.0 Beta 10. I got the following error on Linux Mint 21 with speech-dispatcher 0.11.1:
W 0:00:00:0769   _ready: TTS is not supported by this display server.
  <C++ Source>   servers/display_server.cpp:269 @ tts_get_voices()
  <Stack Trace>  main.gd:52 @ _ready()

If I understand correctly this problem still exists. I wasn't able to find this issue directly because the error message wasn't documented here. So now at least it should be easier to find information about why tts is sometimes not working on Linux.

Relevant issue in the Godot buildroot repo - https://github.com/godotengine/buildroot/issues/7
Adding fontconfig should be as easy as adding it to the config files, but there's no speechd for the buildroot, and it would require making packages for the multiple dependencies which are missing as well.
We actually do not need 99% of the speechd to build Godot, only 3 header files (one in case fontconfig and libudev), so it might be worth including it as the thridparty folder of the Godot main repo instead of using system headers.

For anyone else who finds this: the issue appears to be present in the RC1, but it actually relies on some system libraries.
Knowing nothing about speech generation and how Godot's works, to summarize: Godot on Linux relies on the system to have text-to-speech libraries installed.
In my case, I had to install (via pacman) spd-say, festival, and espeakup before it worked. Running spd-say -L should list something other than a dummy voice.  Once that worked, Godot's TTS APIs worked beautifully (DisplayServer.tts_*).

Hey @nightblade9 thanks for these additional tips. Possibly this would be a good addition to the docs. Would you mind creating an issue there?

There's already an issue for this: https://github.com/godotengine/godot-docs/issues/5823

There's already an issue for this: godotengine/godot-docs#5823

I think this issue is more about contributing to Godot itself or building from source, neither of which apply to my case; I'm just a consumer of Godot who knows little to nothing about TTS.

Hey @nightblade9 thanks for these additional tips. Possibly this would be a good addition to the docs. Would you mind creating an issue there?

Sure, I can do that. I also spoke to @akien-mga about how to get this to work when exporting (TL;DR it isn't trivial for Linux), I think that info should also go somewhere in the docs.

Since this requires users to install specific libraries, it would be best if Godot could detect missing libraries and show an alert the first time the project tries to use text-to-speech.

For anyone interested, my docs PR is open here: https://github.com/godotengine/godot/pull/73408
Thanks for the suggestion @pwab


Hide pre-release features section of advanced settings pane.

Summary
Hide pre-release feature section of advance settings pane
Here’s the PR for the server: https://github.com/mattermost/mattermost-server/pull/7847
Cc @dmeza
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] Ran make check-style to check for style errors (required for all pull requests)
[ ] Ran make test to ensure unit and component tests passed
[ ] Added or updated unit tests (required for all new features)
[ ] Has server changes (please link)
[ ] Has redux changes (please link)
[x] Has UI changes
[ ] Includes text changes and localization file (.../i18n/en.json) updates
[ ] Touches critical sections of the codebase (auth, posting, etc.)

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-031c85c1ef3f8b02a.spinmint.com
Test Account 1: Email<EMAIL_ADDRESS>| Password: passwd
Test Account 2: Email<EMAIL_ADDRESS>| Password: passwd
Instance ID: i-031c85c1ef3f8b02a

@dmeza @jwilander can we set the config flag for this to "false" to test it? It's EnablePreviewFeatures

I will update the spinmint

Done

Spinmint test server destroyed


Cake 4.3 compatibility

Install Cake 4.3 and remove the Migrator and Test Suite Light dependencies

We found out with @vidrascus that the tables were truncated at the end of our tests.
This issue here will handle the cleaning of the DB the CakePHP 4.3 way https://github.com/vierge-noire/cakephp-test-suite-light/issues/49

@vidrascus I have a release candidate Cake 4.3 compatible on that branch: #126-TSL-2.4
Most changes required are from the test suite light itself.
Feel free to share your views on the approach!
We will need this CakePHP issue to be solved before release.

The factories are as such 4.3 compatible.


Include a way to assign headers direct from the call

In the issue #20 @CezarLuiz0 asked about a better way to assign headers and I think this is reasonable.
The proposal is:
Client.Photo.save({
  params: { category: 'family' },
  body: {year: 2015, tags: ['party', 'family']},
  headers: {
    'Authorization': 'token MYTOKEN'
  }
})

The way axios work looks interesting.
https://github.com/mzabriskie/axios


use map to map source id and source name; some clean work.

Cool project. Hope my tiny change helps improve it.

I think mapping could be a better way to manage the source, especially when you have more source in future, which I bet you will. Actually, if we think a step further, should we really use a number to map the source? How about

var sourceMapping = {
  'Xiami': 'xiami',
  'QQ': 'qq'
  ...
};

function getSourceName(sourceName) {
  return sourceMapping[sourceName];
}

And call them from AngularJS with a name instead of a number?

Helped clean some code formatting. A space after comma is recommended; always using a semicolon to end expression is recommended too in Javascript.

Besides, the naming can be improved. I saw lower_case_with_underscore and lowerCamelCase mixed. It's better to use the CamelCase as Javascript naming indicates.
One thing I noticed and I'm also not sure is that when I selected '虾米' as the source under "精选歌单", there is nothing, which is the case in both code before and after the change.
Maybe add some unit tests to your project?

This is the one I was trying to say!

@junhan-z , I think the Author @listen1 comes from a python background(correct me if I am wrong),  and I agree with you that it would be good if it follows best practice in JavaScript. Furthermore, many development tools can be added to this project, such as sass, npm, jslint, unit tests, live reloading. It also would be better to follow some style guide.
I have to say that there are a lot of things going on with JavaScript today, it is kind of too hard to catch up.

thanks a lot for @junhan-z 's kind and constructive advices.
@richdho Yes, i'm from Pythoner and newbie to js(as you have seen, listen1/listen1 is first a python server project). I'm very interesting about dev tools adding in the future. I think we can open an issue for that.
And I'll spend time to have a full code style view recently, any help is welcome.

@richdho Auh, the naming is not a big issue, as long as we stick to one style instead of mixing two of them, right? The author is awesome and I believe the code will become much better in future.
@listen1 I'm a noob to js too!

In fact, I was thinking about rewriting/restructuring the app, making it more developer friendly.  But as you may have guessed, this will be time-consuming. Whether doing it or not depends on how much development work to come . If there will be only small fixes and features, I think rewriting it would be going too far. Keeping it this way is just fine.

I think you can provide a detail list for development to do. I think feature development and restructure are both have their own priority. If developer friendly also provides efficiency for new feature releasing or fix bug, that will be high priority.

I will open a new issue for this. This is a cool project, just want to do it in a cool way;)

@richdho No I don't think rewriting is needed. The author and other developers can improve it gradually. I was wondering what's your opinion of adding unit tests for it, which I think can be a huge improvement as saves review time and makes the code more stable.
@listen1 Just found that you have made a few commits, nice. I'm gonna close this PR. Actually I haven't done too much change to this repo either.

Actually, I don't need to close it.. Just updated my forked repo and now you can merge my change if it passes your review.

@junhan-z I opened this issue #48 for more unit tests/restructuring discussion, please take a look

@junhan-z that would be nice!

@listen1 I see. Will do.


Complete the javadoc of all APIs

HSEARCH-3369: Search 6 groundwork - Add javadoc to the POJO bridge APIs
HSEARCH-3370: Search 6 groundwork - Add javadoc to the remaining engine APIs
HSEARCH-3371: Search 6 groundwork - Add javadoc to the ORM mapper APIs
HSEARCH-3372: Search 6 groundwork - Add javadoc to remaining POJO mapper APIs
HSEARCH-3373: Search 6 groundwork - Add javadoc to backend-specific APIs

I took this opportunity to remove @author tags with my name from most APIs. I didn't change other @author tags because, well, that felt wrong. I'll ask permission for that and do it another day.

Pull Request Test Coverage Report for Build 1

40 of 48 (83.33%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.009%) to 89.134%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

mapper/pojo/src/main/java/org/hibernate/search/mapper/pojo/mapping/definition/programmatic/impl/DelegatingPropertyMappingContext.java
0
1
0.0%

mapper/pojo/src/main/java/org/hibernate/search/mapper/pojo/mapping/definition/programmatic/impl/InitialPropertyMappingContext.java
0
1
0.0%

backend/elasticsearch/src/main/java/org/hibernate/search/backend/elasticsearch/types/dsl/impl/ElasticsearchIndexFieldTypeFactoryContextImpl.java
19
22
86.36%

backend/lucene/src/main/java/org/hibernate/search/backend/lucene/types/dsl/impl/LuceneIndexFieldTypeFactoryContextImpl.java
19
22
86.36%

Totals

Change from base Build 278:
-0.009%

Covered Lines:
18408

Relevant Lines:
20652

💛  - Coveralls

About the authors you can remove my name from DifferentSessionFactoriesIT header.

Applied most of your suggestions and merged. Thanks!


Create AWS VPC Site - Wrong Navigation Item

Environment

Application Lab

Describe the bug
Exercise 4.1 tells you to navigate to Cloud Credentials instead of AWS VPC Sites

also reported by @JeffGiroux  fixed in b910bb287deff4abdbd932032cdd4391be8edac3

Also found in this section too...
Exercise 1: Tag AWS Site

In VoltConsole switch to the System context.
Navigate the menu to go to “Manage”->”Site Management” and click on “Cloud Credentials”.

Needs to be Manage -> Site Management -> AWS VPC Sites
At lab guide location...
/state3/vsite.html


scp bus

when i run xboxmouseleykeyboard it say scp bus failed to initialise

i got the same problem

You need SlimDX Runtime installed (x86)
Could you check if you have that installed by going through your Control Panel?

 When I run "XboxKeyboardMouse 24 02 2017 - x86_64" I receive the same error and after I click 'ok' I receive this:

Any help would be greatly appreciated!

This worked for me: https://github.com/fqlx/XboxKeyboardMouse/issues/52
You have to install ScpToolkit and go into ScpToolkit Monitor (Legacy) and click install. I recommend uninstalling after you're done playing, but make sure the XboxKeyboardMouse program is closed first, because when it wasn't my computer completely crashed.

The last time I had issues with this, it was before I got the ScpVBus installer and device sim to behave better.  Basically the mock controller devices had accumulated in the Device Manager.  Resolving was a matter of unplugging any physical 360 peripherals (if any), opening Device Manager, deleting all of the "Xbox 360 Peripherals" entries, and trying again.

Closing; We should document this better with #102


Storage migration target - support 2 phase sync

Fixes bug where 2 phase sync was not supported when using new storage layer.

Testsuite passed


Compability with Citizens

Hello! I started use ScoreboardStats plugin today on my setuped server. But i have a problem in game. When plugin load, names of npc's will be the same as on the picture:
https://pp.userapi.com/c840225/v840225844/1c780/Kz1POdbzUhA.jpg
I don't know what to do, please help!
P.s Plugin for npc's - Citizens. I use russian words for npc's nicknames

Looks like another plugin is setting those names. ScoreboardStats only changes the sidebar. I guess it's an incompatibility with an nametag plugin. You could try to enable compatibilityMode and install ProtocolLib.


How do I generate a single(or several) image before starting the lengthy training session?

Need: giving a prompt, a seed number, select IF or SD, and generate several images as previews before the training.
This is important to confirm the prompt is ideal (even if there are RNGs in play).

Just for image generation, you can refer to diffusers. They provide friendly interface to use diffusion models in few lines.
But the 3D model conditioned on the text can be very different from the image from the same text. So I guess it is hard to set the expectation based on generated image.


Make runtimeclass names configurable

Currently the operator is hard-coding the RuntimeClass names The RuntimeClass names are closely tied to the runtime handler configurations that are made available via the payload image. IOW, the RuntimeClass names are closely tied to the runtime payload image.
The current set of RuntimeClass names (kata, kata-clh, kata-qemu) can be the default, with the option to specify a different set of RuntimeClass names via the CRD as supported by the runtime payload.

cc @fidencio @jensfr

Investigated what it'll take to add this support. It's a simple addition to the CRD and will provide flexibility to the runtime payload providers to decide what runtime classes they want to provide


DM decryption should have option to be automatic

i'm building a relay that i've decided to use DMs as a control interface (including a custom workaround for clients without NIP-42) for administering the relay
however, it's a bit clunky to use nostrudel DM interface to test/use it
it would be nice if there was an option to automatically decrypt all DMs or at least a checkbox next to the decrypt all for a specific, open chat

There should be an option now in "Performance settings" to automatically decrypt DMs in the alpha version

alpha meaning https://next.nostrudel.ninja right?

released in v0.39.0


cat /dev/random | slacktee.sh

Makes the script very unhappy.

Thank you for bringing up this interesting issue.
I think this is tough to handle in slacktee.sh because cat /dev/random returns invalid characters too.
Since slacktee.sh is expecting that all of the input are valid strings (currently, it doesn't support binary data), it crushes slacktee.sh.
Since -f option stores all of the input to the file and uploads it to Slack, it may be able to handle cat /dev/random. However, cat /dev/random outputs data infinitely, so slacktee.sh waits it forever and cannot upload the result file, I guess.
Do you have any suggestions to handle this issue?

I'm closing this issue as won't fix, but please feel free to let me know, if you think we should handle this issue somehow. I'm looking forward to hearing your suggestions!!


TaskScheduler Version seems to be incorrect on Win2012R2

When creating a new Showing the TaskEditor with an existing TaskDefinition the "Configure for:" shows an empty ComboBox.
The TaskDefinition Settings say it is Compatible with V2_2 but the TaskServer itself (remote Win2012R2) says HighestSupportedVersion V1. This only seems to be a problem when connecting remotely. I tried the TestClient localy on the server and everything looks fine.
Originally posted: 2013-12-02T09:50:18

Fixed logic in 2.1.0
Originally posted: 2014-04-17T16:58:29


Inheriting coordinate systems?

I often draw sketches made of polylines with parametrized coordinates, OpenSCAD-style, then extrude them, and then sketch other stuff on top of extrusions. It's really annoying that every time a new face is taken, a new origin of the coordinate system is produced, and all my prior calculations become invalid, so i have to fight the offsets of the new origin to apply them.
It would be nice if there were builders that don't redefine the coordinate system origin, or otherwise a way to preserve them at least for some axes.

If you provide an example of exactly the problem your are seeing I'll be able to provide better help. Here is my guess at the problem you might have:
from build123d import *
from ocp_vscode import show_all

pts = [(0, 0), (1, 0), (2, 1), (0, 0)]
with BuildPart() as p1:
    with BuildSketch() as s1:
        with BuildLine() as outline:
            l1 = Polyline(*pts)
        make_face()
    extrude(amount=0.5)

with BuildSketch(p1.faces().sort_by(Axis.Z)[-1]) as s2:
    Circle(0.25)

show_all()

s1 is using the default workplane - Plane.XY - so the Polyline is where one expects.  s2 isn't using the default workplane but one created from the top face of p1.  When creating a Plane from a Face the origin is set to the center of that Face which means that the Circle is no longer at the global origin.
There are many ways to create a workplane that allows one to control the origin (and other parameters like the x direction), for example Plane.XY.offset(0.5) (the height could be derived from p1 instead of being provided Plane.XY.offset(p1.part.bounding_box().max.Z).
To summarize, BuildSketch needs a Plane which can be generated in many ways, including from a Face - see the Plane class for more information.

That combination of things isn't a valid way to create a Plane but unfortunately there is no indication of that.
Try: Plane(origin=(...), x_dir=(1,0,0), z_dir=p1.faces().sort_by(Axis.Z)[-1].normal_at()) where ... is the global position of where you want the origin to be.


Automated cherry pick of #2581: fix rbd disk delete with snapshot

Cherry pick of #2581 on release/2.12.
#2581: fix rbd disk delete with snapshot

/lgtm

/approve


No color literals raises warning on variable in map

I am seeing an issue where a warning is raised when a color variable is used as a map value. Specifically when the variable shares the same name as a valid CSS color.
To reproduce:
$black: #000;
$colors: (
  font-color: $black,
);

$ sass-lint --verbose ./test.scss
./test.scss
  3:16  warning  Color literals such as 'black' should not be used as map identifiers  no-color-literals

✖ 1 problem (0 errors, 1 warning)

I am running sass-lint 1.5.0.

I am working on a pull request for this issue.
The fix I was able to get working was to ensure that the map value's parent was not a variable.

The reason we decided against was because it actually used to raise the same warnings in scss-lint and we were looking for parity. This fix may have to wait until our 1.6 update is out as we've changed a few of the rules a fair bit.  Thankyou though for taking the time, I'm sure we're far along enough now to diverge from scss-lint

More than happy to wait for 1.6, I have changed my code to use my fork/branch for now anyways, so I can live without this being merged for a little bit.
My only thoughts on the warning are that they are based on the name of the variable, but there isn't a corresponding rule for that variable name not being allowed, the warning would fit better if there was also a rule that disallowed common CSS color names as variables.

Yep, I think that makes perfect sense. The whole ethos with sass-lint has been for small focused rules so that would suit perfectly.
Thanks again

First things first: thanks so much for the long awaited 1.6.0 — almost all issues I had are gone now, well except for this one, I can reproduce this behaviour with 1.6.0. Any ideas on this one?

@webgefrickel thanks for the kind words, we're glad 1.6 is sorting a lot of problems out, it's a massive relief for us to get 1.6.0 out with the work we've had to get into it including getting some big issues sorted out in the AST.
As for this issue there's a PR currently open that now 1.6.0 is out we'll be looking to get merged in soon!
#539
There should be a patch release very very shortly too to fix a few of the extra issues we've come across and more regular updates should now be around as we've got the structural change and AST update out of the way!


⚠ Bofrost.fr has degraded performance

In 1acbb40, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 893 ms

Resolved: Bofrost.fr performance has improved in dfb36d8 after 6 minutes.


[SPIKE] ui component-like interface

Spiking with some UI ideas. Eventually we will want to support JSX and react components inline. But we're also curious to explore something that looks like Kino. This branch is to play with ideas in that space.
To try out this branch, you can download the branch and use this Srcbook.
Simple input example
Writing a code cell that looks like the following:
import {input} from '@srcbook/ui';

const name = await input({label: 'Your name'});

console.log(`Hello, ${name}!`);

will dynamically render an <input /> labeled Your name and wait for the user to enter input before proceeding.
Waiting on user to enter value

After entering a value

Notes
There are all sorts of issues with this approach and how extensible / robust it is. For one, the example DSL is elegant using await, but in reality the input is likely part of a bigger form. How do we construct composite components? We can't always assume an input is used one time, so the await likely won't work (at least not in all cases). Do we then use some sort of callback that fires each the input changes? Do we fire on each change, or only on a submit? How do we update or remove elements from the server (e.g., remove form on submit and replace with some other component)? How do multi-step workflows work?

Closing because we're looking into supporting react components in Srcbook


Polyfill Symbol if not exists

React native Android don't have a Symbols. So, polyfill them if not exists

Great!

Released as 5.2.2

Hmm, is that usage of global safe here? Is it transformed to window somehow later? I didnt check this is browser:(

Afaik yes. And I think it is more likely that global exists than that window exists on RN.


Add option to set flat fee of 1 sat/vbyte

As suggested by @dergigi in issue #321.
This pr contains the cheapest solution I could think of.
Privacy concern: It leaks application usage data to github, because it queries a static file from github.
The new static fee rate source option queries a raw json file from /src/main/resources/com/sparrowwallet/sparrow/preferences/feeRatesSource-static-1-sat-per-vbyte.json
If this solution doesn't meet the code quality standard of sparrow, maybe someone can use it as starter and solve the data source issue for the json more elegant.

Thanks for this - added in 1b61a78 without the need for the external call.


odoo package fail to run: No module named 'werkzeug.posixemulation' (nixpkgs-unstable)

Issue description
The executable provided by the odoo package raise an error when started because of a missing python package:
$ odoo
Traceback (most recent call last):
  File "/nix/store/hbqm1kr8fvs1djbh9llhf9sahnc7iqb1-odoo-15.0.20211029/bin/.odoo-wrapped", line 6, in <module>
    import odoo
  File "/nix/store/hbqm1kr8fvs1djbh9llhf9sahnc7iqb1-odoo-15.0.20211029/lib/python3.9/site-packages/odoo/__init__.py", line 134, in <module>
    from . import http
  File "/nix/store/hbqm1kr8fvs1djbh9llhf9sahnc7iqb1-odoo-15.0.20211029/lib/python3.9/site-packages/odoo/http.py", line 58, in <module>
    from .tools._vendor import sessions
  File "/nix/store/hbqm1kr8fvs1djbh9llhf9sahnc7iqb1-odoo-15.0.20211029/lib/python3.9/site-packages/odoo/tools/_vendor/sessions.py", line 29, in <module>
    from werkzeug.posixemulation import rename
ModuleNotFoundError: No module named 'werkzeug.posixemulation'

Technical details
building shell
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.91, NixOS, 22.05 (Quokka)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.5.1`
 - channels: `"nixpkgs-22.05pre349906.7f65e4abd5e"`

@mkg20001 I see you are declared as a maintainer, is it a known issue?

Yes
I have a fix here https://github.com/NixOS/nixpkgs/pull/157689

Thanks for the quick answer, I guess I can close this then.

It's not fixed yet so let's leave this open

Once PR is merged this can be closed


[DOC] updates to forecaster and transformer extension template

This PR contains various corrections and updates to extension templates.
Forecasting extension template:

clarified that fh in private methods is always ForecastingHorizon
incorrect docstring for update, was copy-pasted from predict
replaced "core logic" with proper descriptions of where this is called from. @chrisholder will be happy about this.
uniformization of content, various minor improvments to formulation

Transformer extension template:

docstrings for transform and inverse_transform were mixed up
clarified that self return is reference to self

typo resolved, @lmmentel


small cleanup

cleanup hardhat config

Coverage remained the same at 96.661% when pulling 651408c1e631f25f53fbf40c6a5ba198a6c8a21c on matt-small-cleanup into 2c22fffacb37f7cf1fa5717ad34014ed50493950 on main.


Feat: added OCI image manifest support

This PR is the implementation of the spec defined in this PR.
Tested with [COSE, JWS] + [OCI artifact manifest, OCI image manifest] on ACR.

Codecov Report

Merging #241 (e978099) into main (c90d521) will decrease coverage by 0.37%.
The diff coverage is 16.00%.

@@            Coverage Diff             @@
##             main     #241      +/-   ##
==========================================
- Coverage   73.31%   72.93%   -0.38%
==========================================
  Files          23       23
  Lines        1746     1755       +9
==========================================
  Hits         1280     1280
- Misses        372      381       +9
  Partials       94       94

Impacted Files
Coverage Δ

registry/repository.go
27.94% <12.50%> (-4.27%)
:arrow_down:

notation.go
59.86% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[ic-pagination]: remove carousel variant guidance

What component is this related to?
ic-pagination carousel variant
Why do we need it?
This ticket is to remove the carousel alert and guidance from the guidance tab for ic-pagination. The carousel variant has been de-prioritised and can be removed from the site.
Related open issues
[Link to other GitHub issues]

Design to provide images


Update github.py

There is no error_reason

Merged in https://github.com/lepture/flask-oauthlib/commit/78979c6a46472b933696f8a110b05726e7cd0f4b
Thanks.


Synthesis using multiple gpus

Hello,
I get the following error when trying to do the synthesis with multiple gpus:
ValueError: Variable Tacotron_model/inference/inputs_embedding already exists, disallowed. Did you mean to set reuse=True or reuse=tf.AUTO_REUSE in VarScope? Originally defined at:
tacotron/models/tacotron.py", line 112, in initialize
'inputs_embedding', [len(symbols), hp.embedding_dim], dtype=tf.float32)
Is the synthesis with multiple gpus not supported?
Thanks a lot for your help.

@marzieh-razavi Hello.I also got the same problem.
I do not know whether it is correct, but I modified it as follows.
line 25 of tacotron/synthesizer.py
before:

with tf.variable_scope('Tacotron_model') as scope:

after:

with tf.variable_scope('Tacotron_model', reuse=tf.AUTO_REUSE) as scope:

@marzieh-razavi Hello.I also got the same problem.
I do not know whether it is correct, but I modified it as follows.
line 25 of tacotron/synthesizer.py
before:

with tf.variable_scope('Tacotron_model') as scope:

after:

with tf.variable_scope('Tacotron_model', reuse=tf.AUTO_REUSE) as scope:

Thank you very much


Notification: Use gotham_citizen_match_print().

A typical notification message will be :

Notification from [xxx@yyy] : Test message.

You might need to show a few custom variables just like it is done for the seen module, for example.
So lets add it.

Fixed by 8130e4636dce188bb41901ee2866b54c083a71de


August release: nested

In IE, there is a showstopper bug in August release.
Having a view with <template> tag inside, even as simple as:
<template>
  <template if="true">
    Yes
  </template>
</template>

Results in a Stack overflow in IE 11. My application currently does not work at all in IE 9-10 but right now I can't tell if the issue is linked to this or not.
It works in latest FF and Cr, but both have native support for <template>.
I tried to put back the webcomponentsjs polyfill that was used in previous release, but it did not make any difference.
This is critical enough that I have to rollback to September release.

@jdanyow @martingust Do either of you have time to look into this?

In both IE11 and Chrome, the Initial call to compile lands here.
In IE11, source is an instanceof HTMLUnknownElement and source.content is an instanceof DocumentFragment.
In Chrome, source is an instanceof HTMLTemplateElement and source.content is instanceof DocumentFragment.

Both IE11 and Chrome go here next.  In IE11, currentChild is an HTMLUnknownElement representing the inner

Next both IE11 and Chrome go here

In compileElement both IE11 and Chrome go here

This is where IE11 and Chrome diverge.  In step 4, compile is invoked a second time.  Chrome lands here because the inner template element has a content property.  In IE11 the inner template element's content property is undefined, causing it to land here and get stuck in a loop of compile, compileNode, compileElement, compile, compileNode, compileElement, compile, compileNode, compileElement, compile, compileNode, compileElement, ...

@EisenbergEffect, I'm a little out of my depth here, maybe this tells you something?

Yes...yes it does. Something is going wrong with template bootstrapping it sounds like. I'll see if I can get an windows instance up and running so I can debug where that is going wrong. But, this definitely helps. It's something we can fix.

I wonder if the fix for this will also fix https://github.com/aurelia/templating/issues/161

I strongly suspect it will.

Same issue with aurelia 18 september master github source

This needs to be fixed with a high priority. It is blocking any upgrade to August release or later (for me at least).

Don't know if this information helps (might be obvious to you folk) but the problem goes away if you go back to using Html Template Imports.
jspm install aurelia-html-import-template-loader

.plugin('aurelia-html-import-template-loader')

I think it's an issue with incorrectly bootstrapped nested templates in IE for the text loader. We have a new version coming out this week with a fix for that.

Fantastic - thanks!

Fixed in master. Going out in the next release within a day or two.


line chart hide only x labels and responsive height

Version: 1.0.2

how to hide only x labels and leave it on points hover?
how to fix height when responsive?

Hiding only the labels is not possible in v1. It is however possible in v2.
Fixing the height could be done the same way as #423 by setting maintainAspectRatio to false.
V2 documentation is availabel at http://nnnick.github.io/Chart.js/docs-v2/


Going fullscreen makes video darker

Not sure if this is an mpv issue, however, whenever I go fullscreen, the video becomes significantly darker. This is not the case with 2015.04.03 (the 'stable' version on the gentoo repository), only happens with 2015.11.02 and 2015.11.10 (the other 2 versions there).
https://www.youtube.com/watch?v=q858mp9M2wQ

That's unrelated to youtube-dl.

The only thing that changed in between tests was the youtube-dl version, but alright, I'll post on mpv's bug tracker.


Websocket connections under load test

While using the Load Test option in SOAPUI, when each thread wants to open a different websocket connection to the same server but with different URIs, like each URI has a sessionid in it, how does the plugin treat those multiple connections? Even if "Close connection between each request" option is not selected, I think the plugin should still keep a separate connection for each URI. Currently, if I do not select that option, the Load Test fails and I think the reason is multiple threads are trying to use the same connection since the destination IP is the same. I may be wrong in that assumption, could you please clarify?

The connection is cached with the connections name as the key. If you dynamically change the URI or query parameters at the URI this will be effective only after a re-connect.
Please use the latest version v2.0.6, since there was a bugfix in this version regarding connection re-use in LoadTests.

Thanks. I am using 2.0.6.
I use a constant connection name but I use the Server URI as a variable, something like "${#channel}" where a groovy script sets the channel parameter at the beginning of a test case. Here, channel is the full URI like ws:/// and sessionid changes for each Test Case execution. So, in that case, the websocket connection cache can only be used only within the execution of the same Test Case, right? I mean, if you have multiple Receive/Publish Websocket Message steps in a test case which share the same "channel", they are going to use the same connection but in a Load Test, each Thread and different executions of the same Thread will use different connections.
Should I also use connection name as a variable?

The connection's name i not expanded, which means a variable or expression is not evaluated and transformed into a value.
For the test case you like to run, i recommend to use the 'Close Connections: [ ] between each request' option in the LoadTest Options.

"Close connections between each request" will cause each websocket step in a test case to open a new connection, right? That means, there is no connection reuse.

"Close connections between each request" will cause a close of a connection after each TestCase run . Every TestStep in a TestCase will re-use a connection opened in a TestStep before in the same TestCase.

Are you saying this for websocket plugin? For HTTP requests (SOAP or REST), I think this is not the way it works; if you have multiple HTTP requests in a test case, each of them will open a new connection before sending the request and and close it after receiving the response.

Yes, this is a special behavior for websocket, due to the fact that this is not a request/response pattern like REST or SOAP.

I believe it would be better to change this logic. The parameter (Close connection after each request) is asking for closing the connection after each request, so it should cover the websocket steps as well - it should not make an exception for websocket connections, otherwise this is going to cause confusion and wrong assumptions and you will need to explain this exception to everyone who uses this plugin.
I believe that the default behaviour for websocket connections should be using the same connection until a "Drop Websocket Connection" test step is called or the execution of the Test Case is ended (assuming the server definition is same for all steps). If "Close Connection after each Request" parameter is selected then this behaviour should be overridden - Each Receive / Publish test step should close the connection after it is done. This will make the usage of the parameter consistent with HTTP based requests.
Just my 2 cents...

Thank you for sharing your opinion.
For websocket connections a 'request' spans from connect until disconnect and it's lifetime can not easily be bound to SoapUI TestSteps.
Unfortunately plugin developers can not customize or extends the LoadTest options dialog. Thus i have take what is given and document the meaning.
With the implemented logic i wanted to span the lifetime of a websocket connection over the hole LoadTest run. For one reason, i don't wanted to loadtest the websocket handshake but the message stream.


Clearer guidance on test credential usage

Description
Some folks have tried using their own credentials in tests instead of using the AzureRecordedTestCase.get_credential utility. This approach can work in live mode, but real credentials are incompatible with playback tests now that OAuth interactions are sanitized completely from recordings; a dummy implementation of get_token() is necessary to avoid sending token requests in playback. Additionally, get_credential use is necessary to honor AZURE_TEST_USE_*_AUTH environment credentials.
This PR updates our test guidance to explicitly encourage the use of the utility method and explain why it's necessary.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

/check-enforcer override


Note how to use Visitors

It takes a few paragraphs of searching to notice that in order to actually use the visitor,
you must call path.traverse(MyVisitor).

I think that this section is just explaining the concepts and not about the babel api

Yeah, I don't think adding the api to it aids in teaching the concept

Then perhaps what we should add is a few handy links to each concept being
used in a simple example.
On Nov 25, 2016 10:24 PM, "James Kyle"<EMAIL_ADDRESS>wrote:
Yeah, I don't think adding the api to it aids in teaching the concept
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thejameskyle/babel-handbook/pull/111#issuecomment-263041612,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJFPw4lzPG_713AjAKSchk9zaguQEaQks5rB6Z4gaJpZM4K6jky
.

I'm not sure what you mean

What I mean is that path.traverse() isn't even present in the handbook until used in an example for 'visitors-in-visitors' - the handbook assumes a certain level of familiarity with Babel that I don't think the intended audience actually has.
So while it's great to explain each part of traversal, it would be even better to relate each example to a fully-working piece of code and/or a link to ASTExplorer so the concept can really stick.


[WIP] Bookshelf mapping to volunteers tables

Proposed solution
Implementing bookshelf mapping to Volunteers module tables

Tradeoffs
None

Testing Done
This is initial mapping, I'm working on implementing Mocha tests

looks good, worth to integrate with the multi module implementation (#110 or #120)

@OriHoch @amotenko @natekonimbo
Hi, all.
I'm not sure this is the correct place to ask this.
In my prev PR I created migration script for creating tables for volunteers module. In volunteers table i created composite PK on user_id department and event columns, but it kinda messes things up, so i want to create column id and define it as a PK. The problem is that i cannot create a migration that drops existing PK and creates an new one. So there are two possible solutions:

Create a migration that drops (or renames) the existing table, and then create a the volunteers table from scratch with the correct columns.
Edit existing migration script to create the correct tables. This solution is a bit hacky and will only work if the migrations were not deployed already to staging/production, and will require everyone to delete their dev.sqlite files.

I need an advise on this one. Although my spidey-sense tells me that the first one is the way to go...

previous PR was already merged to master which means it was also deployed to staging
so I think best option is the first one  - adding a new migration which does what's needed..

OK. Thanks.

I wonder, if we have migration scripts that drop tables, and those scripts
run every time the system is deployed, how come we don't lose data?
On Wed, Mar 8, 2017 at 9:22 AM Ori Hoch<EMAIL_ADDRESS>wrote:

previous PR was already merged to master which means it was also deployed
to staging
so I think best option is the first one - adding a new migration which
does what's needed..
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Midburn/Spark/pull/126#issuecomment-284966826, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHHcKKS5rjxirWvckcoEmwdrYxyEP4g8ks5rjldAgaJpZM4MTI5S
.

@amotenko the scripts run only once

@LeonFedotov Hey. Updated the code according to your comments, Please review.

@ArtyomKa maybe you can come by tomorrow for our meeting and we can discuss the api's?

If you mean the meeting at ironsource, I'm usually there.

awesome, see you tomorrow!

Spec:
https://docs.google.com/document/d/13GuReh_rGuWyFl-HWmRQl9c9EajN1u71f0k3ZRh_aEc/edit

Hi
Please change the reply of
volunteers/departments/:id/volunteers
to the following format:
{
"profile_id": "1",
"email"<EMAIL_ADDRESS>"first_name": "Roe",
"last_name": "Sebastian",
"role": "Manger",
"is_production": true,
"phone": "+67.7.777.7777",
"got_ticket": false
},

please include the department id and name on the general volunteer api
/volunteers/volunteers
{
"profile_id": "1",
"email"<EMAIL_ADDRESS>"first_name": "Roe",
"last_name": "Sebastian",
"department": "Medical",
"department_id": "Medical",
"role": "Manger",
"is_production": true,
"phone": "+67.7.777.7777",
"got_ticket": false
},

@LeonFedotov Hey. Friday @eyalliebermann and I made progress integrating the spark api and volunteers module. Please review again so we can merge this PR and deploy to staging.

@ArtyomKa Hey, sorry for the delay, let me know when you are free so we could have a phone/meet to go over the changes as theres lots of them, in the future lets have less code in each pr so we could streamline this process.

@ArtyomKa I don't see any logical problems, would love to see some tests for the functionallity to avoid regressions, good job on the cleanup. i think since we have a tight time frame were good to go for now. lets keep it up with the tidy code ya ^_^

lets refresh the request...


Add tokens and token_log_likelihoods to ChooseBest response

Closes #22

change the version in setup.py to 0.0.10


Support input file containing list of software for 'download' feature

Description of the new feature / enhancement
The download feature is really nice. Really helps in instances of offline systems and not needing to head to the web every time IA requires us to update. In its current state, it appears the download feature only supports passing in a single app to download.
Now this isn't really a huge issue as it's easy enough to script out and build a download string from a txt file of needed software but it would be cool if the download feature could support passing in a file containing the software you'd like to download so it's done in a one-liner and a bulk download launches.
Proposed technical implementation details
No response

Similar with #4028


Caching cuDNN kernel choices

Pytorch as a functionality to cache the chosen cuDNN kernels when the input dimensions are identical to previous calls:
torch.backend.cudnn.benchmark = True

This improves throughput of a network by 30~40% (again, when the input dimensions of the network do not change).
Here is where the caching is implemented in Aten:
https://github.com/pytorch/pytorch/blob/358fb51e773b8ad509ec270caee5ec1c51d82f38/aten/src/ATen/native/cudnn/Conv.cpp#L340
Do you have any plans to add similar functionality? It would greatly improve throughput of our DNN engines built with Flashlight.
Thanks

@ThisIsIsaac  - Thanks for the suggestion. Yes, this is in our plans and we would like to support it.

@vineelpratap  Thanks for the reply. Do you know approximately when it will be coming? I'd love to contribute and help expedite the process.

If you want this early, it might be better if you work on it and send us a PR. Here is the code pointer where we choose the cuDNN algorithm - https://github.com/facebookresearch/flashlight/blob/master/flashlight/autograd/backend/cuda/Conv2D.cpp#L79

Thanks. I'll take a look.

@vineelpratap  I'm going to use huge portions of code from Pytorch's Aten. According to their license it seems like we can use it.

@vineelpratap Is there a reason why only 2D convs use cudnn, and not others?

@vineelpratap
ping!

Hi,

Currently, we don’t have modules for ConvNd in the framework.
No, we don’t have it set. I think it would be easy to implement after we have a flag for ‘cudnn.benchmark’

@vineelpratap

How do I test for correctness?

how do I set benchmark = true? Currently, I am testing by giving it as a parameter in conv2d but this probably shouldn't be the way it is actually implemented, as you would have to pass true to every call of conv2d.

Hi,
Thanks for taking care of this ! Regarding your questions -

How do I test for correctness and throughput? Any internal code available? or should I just go with my simple profiling code?

Here are some code pointers. You might have to adapt them for your use case.
For correctness - https://git.io/fjQvF, https://git.io/fjQvb
For throughput - https://git.io/fjQvA

how do I set benchmark = true? Currently, I am testing by giving it as a parameter in conv2d but this probably shouldn't be the way it is actually implemented, as you would have to pass true to every call of conv2d.

We can have a function in fl::cuda namespace similar to https://git.io/fjQv1 called void setCudnnBenchmark(bool val). You can also check the implementation details for af::setMemStepSize(...), af::setSeed(FLAGS_seed), af::setFFTPlanCacheSize(...) in https://github.com/arrayfire/arrayfire/ for inspiration.

How do I concurrently use 2 different versions of flashlight? I have a reference copy of flashlight at /home/isaac/flashlight and a development copy at /home/isaac/flashlight_dev. Let's say I want to compile the built-in examples first using the reference copy and then using the development copy. How would I do this without having to build from scratch every time?

From taking look at tests/CMakeLists.txt, Benchmark.cpp was not being built on make test. In fact, Benchmark.cpp seemed to be not part of any default builds. So I am going to add:

build_test(${FLASHLIGHT_TEST_DIR}/nn/Benchmark.cpp)

to tests/CMakeLists.txt. Let me know if there is a better way done internally.

@vineelpratap ping!

Hi,
Regarding your questions
1 - Not sure if it is possible. @jacobkahn do you have any ideas how to accomplish it ?
2 - We don't want to include in tests as this is not a unit test. It is just a benchmarking tool that we use for performance tuning.

@ThisIsIsaac — some of the things in #72 accomplish this: https://github.com/facebookresearch/flashlight/pull/72/files#diff-f7a8e4a67d72800f1f92fac76935d2e6. We're still discussing how to make this implementation cleaner, but I think this will make a nice performance difference.

The ability to do this was added in https://github.com/facebookresearch/flashlight/pull/187.


Add polling to multicall

For multicall to be more useful in contexts where we need to poll, a polling option should be added. See useContractPoll for inspiration.

Actually, it's simpler to just use it inside setInterval. Closing.


ObjectDetection namespace does not exist in the mentioned nuget packages

I have followed the instructions exactly, but I get the error: "The type or namespace name 'ObjectDetection' could not be found"
Here is a repo with the code up to this point in the tutorial (adding the namespaces): https://github.com/jpayne0061/ObjectDetection

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

ID: c2f95783-f8d3-b884-a67e-131eba39f59c
Version Independent ID: 622956b7-6556-7d5b-a6d1-436ceb9e68e6
Content: Tutorial: Detect objects using an ONNX deep learning model - ML.NET
Content Source: docs/machine-learning/tutorials/object-detection-onnx.md
Product: dotnet-ml
GitHub Login: @luisquintanilla
Microsoft Alias: luquinta

Hey, @jpayne0061, that namespace isn't meant to be found within any of those NuGet packages. It's something you create a bit later on in the tutorial. (I agree, it is confusing to start off right away by adding "usings" to namespaces that don't exist yet.) If you continue with the step-by-step, you'll create classes in directories with those namespaces, and that error will disappear 👍.
Just for reference: I did just try to code along with the whole tutorial from scratch, and can confirm that it all works properly -- no compile or runtime errors. (Though the ML still has some learning to do, based on the results I saw it output. 😏)

I will add one thing that could/should be improved: the default ".csproj" is created with nullable enabled, but the code in the project is not compliant with that, and produces (6) warnings.
I would suggest either explicitly telling users to remove the <Nullable>enable</Nullable> line if it exists in their csproj, adding default values to the properties that need them, or making them nullable.

That was it, thank you!


Include ChromeDriver

Currently Selenium is usually not enough to launch tests as it may require ChromeDriver or GeckoDriver to work on. Would it be possible for this project to include ChromeDriver as well?

I don't think this is within the scope of this project; it doesn't make sense to bundle any browser-specific driver within. I understand from a convenience point of view, but it would make understanding the release schedules pretty hectic. Which driver(s) would we bundle? Why? How often are they updated (and, by proxy, how often would we have to release). Seems like an unnecessary maintenance burden to me...

I was just looking to something similar to https://www.npmjs.com/package/selenium-standalone and I didn't find any good library in PHP except this one :+1:
It would be cool if there was a way for newbies to launch selenium without pain by installing just a single package. Something similar to what Laravel Dusk does.

As for maintenance I think it can be done with a simple PHP script:

check for updates
download new binaries
run basic tests
make PR

Something to download the latest driver could be workable, yeah. But I wouldn't bundle the binaries in this repo (as the JAR already is). :+1:

Possible solution:

install laravel/dusk,
copy chromedriver out of it
remove laravel dusk
PROFIT
:rofl:

There could be additional repositories/packages setup to handle this. selenium-driver-chrome, selenium-driver-firefox, ....

Btw there is already an initiative from Joomla you know about. https://github.com/joomla-projects/selenium-server-standalone
Did you have a chance to get more steps into merging the forces?

@DavertMik I am open to further pursue this. https://github.com/joomla-projects/selenium-server-standalone/issues/25


container stop: kill conmon

Old versions of conmon have a bug where they create the exit file before
closing open file descriptors causing a race condition when restarting
containers with open ports since we cannot bind the ports as they're not
yet closed by conmon.
Killing the old conmon PID is ~okay since it forces the FDs of old
conmons to be closed, while it's a NOP for newer versions which should
have exited already.
Signed-off-by: Valentin Rothberg<EMAIL_ADDRESS>

@giuseppe's working on the conmon side: https://github.com/containers/conmon/pull/62#event-2529821192
@mheon @baude PTAL

Actually, this should only be in restart - otherwise, the cleanup process won't fire

@vrothberg You broke windows

@vrothberg You broke windows

Let's merge then! :-D

Actually, this should only be in restart - otherwise, the cleanup process won't fire

Done. Now using syscall instead of unix.

[+0037s] # github.com/containers/libpod/libpod
[+0037s] libpod/container_internal.go:1169:14: undefined: syscall.Kill
[+0038s] make: *** [Makefile:170: podman-remote-windows] Error 2

I don't think you got all of it... I guess Windows' syscall package is missing the define?

We need to go via os.FindProcess()

Whacked the tests.

LGTM

@rhatdan @baude @giuseppe @vrothberg @TomSweeneyRedHat PTAL

/lgtm
/hold
cancel when success tests.

/hold cancel


cask/exceptions: Actually print actual hash

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

The test failures are unrelated, so I'll count that as a success.

Thanks, @0az!


⚠ Panoramas has degraded performance

In ed33255, Panoramas ($SITE_URL_PANORAMAS) experienced degraded performance:

HTTP code: 200
Response time: 9302 ms

Resolved: Panoramas performance has improved in 9f4ab44 after 6 minutes.


fix link to clint

Coverage remained the same at 100.0% when pulling 34d84c307e650cfc8647e78052eb6b3531fee6cc on skierpage:patch-1 into 8753518f756edfe1f25ba465eb392be5de1c8d88 on danielecook:master.


Add move constructors and move assignments

We should implement move constructors and move assignment operators for user-defined classes such as Mata::Nfa::Nfa, Mata::Nfa::Delta, Mata::Util::NumberPredicate or Mata::Util::OrdVector, Mata::Nfa::Post, Mata::Nfa::Move.
Moreover, we should consider implementing some functions operating with R-value references to see where they can be used.
After some basic profiling, it turns out it can help with performance in some cases even without explicitly using std::move().

Some move constructors and move assignments are going to be introduced in #195. Then I will compare the efficiency of the default move operations with manual implementation of them (which seems to be much faster) in a separate PR.


Remediate shellcheck findings

Issue #, if available: NA
Description of changes:

Removed the "s" on sleep 10 and 15
Ran shellcheck and resolved all of its findings

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Done!


Datetime format not proper

I am trying to format the new Date() to 08/06/2021 11:00:00 AM PDT format but unsuccessful. Below is the codepen to reproduce the issue.
https://codepen.io/msampathkumar/pen/KKWBmrO
Please help resolve the issue.
OS: Windows
Browser: Chrome
Inputmask version (5.0.6)

@sampathkumarm , @GKersten,
Can you have a try with the latest version.  5.0.7-beta.40


KAFKA-12648: fix flaky #shouldAddToEmptyInitialTopologyRemoveResetOffsetsThenAddSameNamedTopologyWithRepartitioning

This test has started to become flaky at a relatively low, but consistently reproducible, rate. Upon inspection, we find this is due to IOExceptions during the #cleanUpNamedTopology call -- specifically, most often a DirectoryNotEmptyException with an ocasional FileNotFoundException
Basically, signs pointed to having returned from/completed the #removeNamedTopology future prematurely, and moving on to try and clear out the topology's state directory while there was a streamthread somewhere that was continuing to process/close its tasks.
I believe this is due to updating the thread's topology version before we perform the actual topology update, in this case specifically the act of eg clearing out a directory. If one thread updates its version and then goes to perform the topology removal/cleanup when the second thread finishes its own topology removal, this other thread will check whether all threads are on the latest version and complete any waiting futures if so -- which means it can complete the future before the first thread has actually completed the corresponding action

shouldAllowRemovingAndAddingNamedTopologyToRunningApplicationWithMultipleNodesAndResetsOffsets is deadlocking. We need to be able to have its be async a little or else it can't make progress

Ah good catch @wcarlson5 , I need to move the offset reset logic out of the actual removeNamedTopology call and make sure we don't block on the offsets being removed until the get() -- otherwise we get deadlock if we have two Streams clients and try to remove a named topology from both from a single thread. (To clarify for anyone else, this is not really a realistic/recommended usage pattern for real applications, but it helps keep the tests simple and makes for a more intuitive blocking behavior anyways)
Anyways I'll push a fix for this, and then we should be good to go 👍

Anyways I'll push a fix for this, and then we should be good to go 👍

Thanks @wcarlson5 for the report! @ableegoldman please feel free to move on afterwards.

Test failures are unrelated. Merging

Merged to trunk

@wcarlson5 @ableegoldman I'm wondering if we could still have live lock scenarios like this: say we have two topologies A and B, and two threads a and b each on a different KS instance. And each thread tries to remove one topology at a time, getting the future to make sure it is cleaned up, BUT they did it in a different order.

Thread a calls remove(A) and gets a futureA.
Thread b calls remove(B) and gets a futureB.
Thread a calls futureA. get() trying to delete offsets, and after that it tries to remove(B), but gets blocked on topology A not removed by thread b yet.
Thread b calls futureB. get() trying to delete offsets, and after that it tries to remove(A), but gets blocked on topology B not removed by thread a yet.

Would that be a concern?

@guozhangwang yes that would be a concern.
I think we need to document that these calls need to be made in the same order with the same topologies for each client. That is what KSQL does to make sure it works

I think we need to document that these calls need to be made in the same order with the same topologies for each client. That is what KSQL does to make sure it works

Sounds good, thanks for the clarification!


We no longer collect information on unique users so remove code.

This code is unused and can be removed
Trello: https://trello.com/c/WkNfnASe/402-remove-code-recording-unique-users

Actually; before this goes in we should undo the scheduled tasks in TF first.

I assume you mean the tasks that copy the data from the performance platform to the S3 bucket? These have to all be removed at some point when the PP is switched off. Maybe we could do them all together?