Kill Backup Page

/index.php/dashboard/system/backup/backup
This page should go.
It doesn't work at all today.
It's not something we're ever going to make work ourselves.
We only had it in here because someone submitted it in the first place for an earlier version.
Backup is important, and given the number of fully functioning great solutions that are out there, we really shouldn't be spending our time dinking around trying to make concrete5 backup itself. Even when this used to sometimes work, it would die on a big site, and it's a huge security hole to be leaving these backups in web directories, which I'm sure is happening across the web.
Currently says:
Existing Backups
You have no backups available.
Create new Backup
Use Encryption
Important Information about Backup & Restore
Running a backup will create a database export file and store it on your server. Encryption is only advised if you plan on storing the backup on the server indefinitely. This is not recommended. After running backup, download the file and make sure that the entire database was saved correctly. If any error messages appear during the backup process, do not attempt to restore from that backup.
All that should get removed and instead should say:
Backups
Backing up your site is important. Typically your web host can help you make a complete backup of your environment from your hosting control panel. Alternatively, there are additional backup solutions available in the concrete5.org marketplace.
(link: https://www.concrete5.org/marketplace/addons/-/view/?submit_search=1&search_keywords=backup)

@aembler
I am up for removing/modifying the backup page, but I wanted to run down the details first to make sure I am removing the appropriate files.
Change
Files
concrete\single_pages\dashboard\system\backup\backup.php
<?php defined('C5_EXECUTE') or die('Access Denied.'); ?>

<?php echo t('Backing up your site is important. Typically your web host can help you make a complete backup of your environment
    from your hosting control panel. Alternatively, there are additional backup solutions available in the concrete5.org marketplace.'); ?>

    <?php echo t('https://www.concrete5.org/marketplace/addons/-/view/?submit_search=1&search_keywords=backup'); ?>

concrete\controllers\single_page\dashboard\system\backup\backup.php
<?php
namespace Concrete\Controller\SinglePage\Dashboard\System\Backup;

use Concrete\Core\Page\Controller\DashboardPageController;

class Backup extends DashboardPageController
{
}

Delete
Files:
concrete\src\Backup\Backup.php
concrete\src\Backup\BackupTable.php
Constant:
concrete\bootstrap\configure.php
define('DIR_FILES_BACKUPS', DIR_FILES_UPLOADED_STANDARD . '/backups');
Dashboard Help:
concrete\src\Application\Service\UserInterface\Help\DashboardManager.php
'/dashboard/system/backup/backup' => t("Create a backup of your site's database. This feature works best on small sites. For larger sites, we recommend using a database utility like phpMyAdmin or exporting a database dump via command-line MySQL."),
Permission:
concrete\config\install\base\permissions.xml
<permissionkey handle="backup" name="Perform Backups" package="" category="admin">
    <access>
        <group name="Administrators"/>
    </access>
</permissionkey>

Yep, looks like you got everything


Add permissions parameter to Exchange Information API

In the official docs (https://binance-docs.github.io/apidocs/spot/en/#exchange-information) there is the permissions parameters, but it is not available in this connector.
Would you accept a PR for this improvement?

Thanks for proposing but no need. It has already been added in the version 1.5.2: https://github.com/binance/binance-connector-ruby/releases/tag/v1.5.2


Incorrect no

Now is: Event Grid module accepts and delivers events in JSON format. There are currently two schemas that are supported by Event Grid
Should be: Event Grid module accepts and delivers events in JSON format. There are currently three schemas that are supported by Event Grid

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

ID: e648eb2b-ad53-b5ea-c731-5714a0dce6f5
Version Independent ID: 51ab4d15-a9fc-1a14-b908-6e584eccdf9e
Content: Event schemas — Azure Event Grid IoT Edge
Content Source: articles/event-grid/edge/event-schemas.md
Service: event-grid
GitHub Login: @VidyaKukke
Microsoft Alias: vkukke

@vunvulear Thanks for catching that! We have worked up a PR to address this.


waffle.io Badge

Merge this to receive a badge indicating the number of issues in the ready column on your waffle.io board at https://waffle.io/markplagge/NeMo
This was requested by a real person (user markplagge) on waffle.io, we're not trying to spam you.

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


ts errors in js files

VSCode Version: latest
OS Version: latest Sierra

This doesn't happen when I disable the built-in TypeScript and JavaScript Language Features extension.. but then I lose a bunch of functionality....
Steps to Reproduce:

Create new file named test.js add the following to it:

const submitEl = document.getElementById('submit');

submitEl.onclick = (event) => {
  console.log('submit');

  function KeyPress(e) {
    console.log('keypress', e)
  }

  document.onkeydown = KeyPress;
}

Why am I getting ts errors on js files?

Does this issue occur when all extensions are disabled?: Yes/No

Found a solution for this
After searching for what felt like an eternity, I finally found a Stack Overflow answer that resolved this for me.
The issue is "typescript.tsdk": [bad_value] is set to a bad value.  You want to set this to a path which contains a tsserver.js file.  The path in use below is on OSX and it points to the global TypeScript npm install.

Open settings.json
Set (or add) the following property:

{
  ...
  "typescript.tsdk": "/Users/<your_username>/.npm-global/lib/node_modules/typescript/lib/",
  ...
}

Hopefully this helps someone.

Just kidding.... this is still an issue.. I have no idea what fixed it before but this stopped working the next day.. I've just being doing // @ts-nocheck at the top of all of my files... This never used to happen - why is it happening now??

Have you set javascript.implicitProjectConfig.checkJs: true in VS Code or does your project have a jsconfig or tsconfig that sets checkJs: true?

@mjbvz I have not set that on purpose. Let me check and see if that's set, or if anything is changing it to true. Thanks for the tip!

As soon as I upgraded to 1.47 all my purely JS projects started to show TS errors. I don't have TS on my system, nor jsconfig or tsconfig, yet even trivial code produces errors. Before 1.47 it was totally usable for JS/ES projects.
Example of an error to illustrate what I am talking about:
const identity = x => x;

In this example x is highlighted with a red squiggly line showing this error:
(parameter) x: any
Parameter 'x' implicitly has an 'any' type.ts (7006)
Peek Problem (⌥F8) No quick fixes available

Why is it an error? I don't want to fix it.
Obviously, if I try to add a type annotation, it errors showing:
Type annotations can only be used in TypeScript files.ts (8010)
Peek Problem (⌥F8) No quick fixes available

Which is totally fair. At least we know that it understands that this file is not TS.
Why it is bad and makes my life hell:

In the minimap it shows bright red on every line preventing overviewing searches and so on.
In the file view (Explorer) all open files are red. I use colors for modified and newly added files. Now I don't know the git status until I close files.
I don't like seeing errors where there are no errors. :-)

This is my settings.json just to see that it doesn't have any funny business:

Click me to expand:
{

    "gitlens.advanced.messages": {
        "suppressShowKeyBindingsNotice": true
    },
    "editor.fontLigatures": true,
    "editor.fontFamily": "Fira Code",
    "sync.gist": "0aadafae52deaff8d740ba461b9349c9",
    "sync.quietSync": false,
    "sync.removeExtensions": true,
    "sync.syncExtensions": true,
    "sync.autoDownload": false,
    "sync.autoUpload": false,
    "sync.forceDownload": false,
    "window.zoomLevel": 1,
    "breadcrumbs.enabled": true,
    "cSpell.enabledLanguageIds": [
        "asciidoc",
        "c",
        "cpp",
        "csharp",
        "css",
        "go",
        "handlebars",
        "html",
        "jade",
        "javascript",
        "javascriptreact",
        "json",
        "latex",
        "less",
        "markdown",
        "php",
        "plaintext",
        "pub",
        "python",
        "restructuredtext",
        "rust",
        "scss",
        "shellscript",
        "text",
        "typescript",
        "typescriptreact",
        "yml"
    ],
    "[javascript]": {
        "editor.defaultFormatter": "esbenp.prettier-vscode"
    },
    "[json]": {
        "editor.defaultFormatter": "esbenp.prettier-vscode"
    },
    "editor.largeFileOptimizations": false,
    "[jsonc]": {
        "editor.defaultFormatter": "esbenp.prettier-vscode"
    },
    "git.autofetch": true,
    "todo-tree.tree.showScanModeButton": false,
    "javascript.suggest.autoImports": false,
    "typescript.suggest.autoImports": false,
    "[html]": {
        "editor.defaultFormatter": "esbenp.prettier-vscode"
    },
    "search.searchEditor.reusePriorSearchConfiguration": true,
    "search.actionsPosition": "right",
    "editor.minimap.enabled": true
}

There is a project-specific setting.json:

Click me to expand:
{
    "deno.enable": false
}

Please tell me how I can help you guys to fix this problem.

@elazutkin-dynata Do see this when you disable all extensions?

Yep. Just to clarify: I used Disable All Installed Extensions command in the Command Palette. Let me know if you had something else in mind.

Ok. This is a distinct problem from @oze4's.
The error you are seeing is a suggestion. That's why it's underlined with ... instead of with a red squiggle like an error would have been. We show this suggestion in JS files because without a type annotation here, we cannot provide good IntelliSense. To disable suggestions, set "javascript.suggestionActions.enabled": false

Also, you'd need to use jsdocs to add the type annotation since this is a JS file

To disable suggestions, set "javascript.suggestionActions.enabled": false

Done. No change. Restarted. No change. What else do you have? Keep suggestions coming. :-)

Also, you'd need to use jsdocs to add the type annotation since this is a JS file

The whole point is I don't want type annotations nor jsdocs. If I wanted that instead of dynamic languages I would use Java. Please don't ask me to change the language or update humongous projects I am charged to support.

Please share a screenshot of your editor when this problem happens, including the problems view

When you disabled extensions, did you also restart VS Code? Some extension, specifically ones like deno, require a restart to take effect. Make sure this happens when no extensions are enabled

When you disabled extensions, did you also restart VS Code?

Yes.
See my post above: https://github.com/microsoft/vscode/issues/102320#issuecomment-659706777

I take it back. I disabled all extensions, and restarted — the squiggly lines are gone. Must be an extension.
Deno extension is installed by disabled in project settings.

I enabled all extensions and disabled Deno — the problem is gone. It looks like Deno's extension is not ready.
Sorry for polluting the ticket with a different problem and thank you for helping to debug it!

Ok, glad you found the issue. Please make sure to also file an issue against the deno extension as well so they can address this

@elazutkin-dynata I recently installed the Deno extension as well. I'm on mobile right now but I will check if disabling it fixes this for me.
Nice find!

@elazutkin-dynata @mjbvz I can confirm that disabling the Deno extension resolves this issue.
Apologies for wasting your time!


Proba refac

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@ba2af4a). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             dev     #329   +/-   ##
======================================
  Coverage       ?   71.10%
======================================
  Files          ?       84
  Lines          ?     5956
  Branches       ?        0
======================================
  Hits           ?     4235
  Misses         ?     1721
  Partials       ?        0

Continue to review full report at Codecov.

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


BIGTOP-3513. Update deb resources for hadoop-yarn-router and hadoop-hdfs-dfsrouter.

https://issues.apache.org/jira/browse/BIGTOP-3513

I manually tested this on ubuntu-18.04.

 Thanks @iwasakims!


Real User Managment (RUM)

Currently there is no real user management.
Half of function don't event check it.
There are data leaks between current user-system.
Also in few cases people (CuddleBear) use two separate because some settings are global and not per user.

We don't need multiple users levels, just two.

Admin
Normal User

The only difference is that an Admin can make changes to the database setup and add/remove/modify Import Folders. Just about all other settings should be configurable by everyone.
We can also add presets for things like Family Friendly which is currently a separate user account, the admin would select this option and the tags would be blocked on the account and hidden from the user so they could not modify them. Series. Images and Tags should all check not only for the user level, but any of the presets we add.
I don't think we should have separate import folders per user, that's not how JMM was designed and would require a lot of work. Instead we work towards allowing two instances of JMM Server to be run, but that's for another issue. If a user wanted separate content per user then they can just create two users that each block the content of the other.

So basicly its no go for you. There colud be simply shared shares like you would add folder and chcek if other users can access it, different anidb account, currently you can add you gdrive and i will be able to browser and access it - thats bad also whats the point of running twice the app to support multi user env ?
On September 19, 2016 3:16:17 PM GMT+02:00, ElementalCrisis<EMAIL_ADDRESS>wrote:

We don't need multiple users levels, just two.

Admin
Normal User

The only difference is that an Admin can make changes to the database
setup and add/remove/modify Import Folders. Just about all other
settings should be configurable by everyone.
We can also add presets for things like Family Friendly which is
currently a separate user account, the admin would select this option
and the tags would be blocked on the account and hidden from the user
so they could not modify them. Series. Images and Tags should all check
not only for the user level, but any of the presets we add.
I don't think we should have separate import folders per user, that's
not how JMM was designed and would require a lot of work. Instead we
work towards allowing two instances of JMM Server to be run, but that's
for another issue. If a user wanted separate content per user then they
can just create two users that each block the content of the other.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/japanesemediamanager/jmmserver/issues/426#issuecomment-247989817

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

What is the use case of this?
We see folders as repositories of anime. JMMServer must have access to all
folders.
Usage restrictions are in place so you can control what user have access to
something. Maybe you should reask, if we can add a new GroupFilter that can
control what importfolder the user can View, and filter according like tags.
Anyhow, this is real work, and this is an open project. so...

I agree that jmm should have access but not all the users. User should not see each other files/imports until the user that manage it own sources add second use to that access
On September 19, 2016 6:40:48 PM GMT+02:00, Maximo Piva<EMAIL_ADDRESS>wrote:

What is the use case of this?
We see folders as repositories of anime. JMMServer must have access to
all
folders.
Usage restrictions are in place so you can control what user have
access to
something. Maybe you should reask, if we can add a new GroupFilter that
can
control what importfolder the user can View, and filter according like
tags.
Anyhow, this is real work, and this is an open project. so...
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/japanesemediamanager/jmmserver/issues/426#issuecomment-248047283

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

The best way to do it, IMHO, is adding a GroupFilter with IN and NOT IN ImportFolders, then add the posibility of restrict GroupFilters or enable groupfilters to users. That will open the system to have a very good granularity of what users can or cannot view, including some folders.  The only problem i see, is, when a series have some Anime that is sharing anime between import folders. That could be a problem.

thats look good, imo, but also we would need add propel db structure to hold per user settings - not all of them (as some will have to stay global) but some

What i like to see, but need a refactor is the following: One master AniDB account tied to the collection, then unlimited number of AniDB Slaves, TVDB, MAL, Trakt. Every user can have any one of those or inherit permission from a father account, also per type account, stop inheritance. Per example, if inherits an user that have a MAL account, and the user do not have MAL account, if set something as watched, it will be set as watched in the parent account, but if the father account have Trakt, and the client account not, you can turn off trakt inheritance, to not propagate watch status. In this way we can have two types of users, real ones, and restricted ones. All this refactor means we have to refactor the AniDB communication library to support multi seat. Probably grouping commands by login. Not all, since slaves only need a limited command set, watched, etc.
On the restriction zone, we need to probably support groupfilter filtering per user, (like tags/categories).
For other restrictions like folder restriction, we could add a Import Folder GroupFilter Condition.

I like it, we should make this the focus of the next version. :)

Updating with latest developments.
Instead of using RUM, @maxpiva suggested the following.

What about this, we leave users as is, we only verify optional auth to the users, but we add another layer  devices, and devices can be anything we support, here is there thing, every time you run a new client, new plex client, new nakamori, you need to register the device, after registration, you can map what users have access to what device. We change signatures with userid with a session, and we add a few Device registration a User authorization methods kinda -> Register Device() (Plex like) -> Authorize Device(string token) -> session = AuthorizeUser(token, user, password). First, on multiuser, only Trakt will be available for extra users. AniDB is impossible without a child account, since the user and you should share the same collection, TvDB and MovieDB are metadata providers, so no need in here....MAL is almost the same as AniDB as it tracks collections. You could give a friend your ip, and tell him to run Shoko Client, Device Auth -> token -> Device Registered. After that, you could give your friend his own user, or a friend user, it could be connected to TRakt or not since admin can map whatever device to whatever user/users.

Admin will be able to grant / remove access to certain sections in Server, Desktop or other clients.

So you planing to do same 'session' thing as the auth in apiv2 does now. But settings per user would be also great ;-)

Should get on Discord to discuss in real time. :)


feat(ecc-utils-design): add components

Description

Update the ecc-utils-design package by adding new components that will improve the overall usability and design consistency across various ECC packages. These components will be created using the base components from the Shoelace library and will be using design tokens as CSS properties to ensure a consistent and cohesive design language.
Components to be Added:

[x] #141
[ ] Services List / Service
[ ] Service Info

Possible solution(s)

Additional context

Hey @ltwmori & @JaeAeich, Can you guys please decide on what components can be developed first as needed per your projects? Form elements are the one such I can think of as told by @JaeAeich earlier!

I would say the hierarchy should be somewhat as follows:

inputs
labels
accordion
buttons
switch
tabs
tooltips

PS: This is based on the general trend I have seen in both mine and @ltwmori's package :+1:

@git-anurag-hub should I take any of these because you are assigned to input?

Since form component is almost complete other than methods as mentioned here #157. Let's take other components for the development. @JaeAeich can you please put down what components will you need in the TES & WES here?

After the create component, we will need to create runs component for browing tasks and workflows. An Accordion is what both of the component will need. Functionality we can expect from the said accordion would be:

Certain field for basic information like Run Id, status and aditionally, one or two buttons for easy implementation of delete runs feature or update run feature.
Pagination, knowing there isn't a set standard for pagination yet but luckily both TES and WES have a similar schema where where we have a next page cursor which can be used to fetch the later runs.
The uncollapsed view should contain rendering of object, I believe recursively due to lack of structure.
Uncollapsed view should maybe have an specific footer we could leverage to show meta data like date of creation etc.
Optionally we could also have select feature for them where in if the user want to bulk deletion, it can be performed.

We would also like to be able to edit task from the runs component itself so we would also require the form component.

Hey @git-anurag-hub, have you started working on this component that @JaeAeich just described or can I take it up?

Sure @SalihuDickson


fix generateReplaceMap not work well when input is not unique

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


feat(*): refactor error handling to preserve stack traces on platform exceptions

Description
Related Issues
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] The analyzer (melos run analyze) does not report any problems on my PR.
[ ] I read and followed the Flutter Style Guide.
[ ] I signed the CLA.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

@rrousselGit Hey Rémi, should min Dart SDK version be moved from 2.12 to 2.16 on pub.dev with this change?
https://api.flutter.dev/flutter/dart-core/Error/throwWithStackTrace.html was added in Dart 2.16 I believe.
After updating firebase_messaging_platform_interface from  3.1.3 to latest 3.2.0 I started getting:
/C:/Users/Blah/AppData/Local/Pub/Cache/hosted/pub.dartlang.org/firebase_messaging_platform_interface-3.2.0/lib/src/method_channel/utils/exception.dart:13:11: Error: Member not found: 'Error.throwWithStackTrace'.
    Error.throwWithStackTrace(exception, stackTrace);
          ^^^^^^^^^^^^^^^^^^^

Moving back down to 3.1.3 in pubspec.yaml avoided the above:
dependency_overrides:
  firebase_messaging_platform_interface: 3.1.3

This changes has broken people whose Dart SDK < 2.16 and make people disappointed.

The pubpsec  of the changed package was correctly updated to mention >= 2.16 (see https://pub.dev/packages/firebase_core/versions)
It seems like pub isn't picking up transitive dependencies constraints thought
We can update all packages to explicitly require 2.16 (even if they weren't directly changed by this PR) if needed. Although it's not going to change the fact that you'd need Dart >= 2.16

@rrousselGit Awesome! Thanks Rémi for the update & work on firebase changes / new Error API.  Much appreciated.

What is going on?!!! Why would you force us who are using less than 2.16 to get screwed over. WTF


Confusing results

I'm trying to use this library, but I get weird results that I can't really explain.
Using this code below:
   const srcCorners = [
<PHONE_NUMBER>28078, 6259800.43254993,
<PHONE_NUMBER>09023, 6259799.53201322,
<PHONE_NUMBER>19169, 6260054.09226945,
<PHONE_NUMBER>0665, 6260028.56590027
    ]
    const dstCorners = [
      0, 0,
      100, 0,
      100, 100,
      0, 100
    ]
    const perspT = PerspT(srcCorners, dstCorners)
    console.log(perspT.transform(491438.780488201, 6259922.52984722))

Which when plotted looks like this:

I've got four corner points and a query point right in the center. I'm mapping this to a rectangle spanning (0,0) to (100, 100).
When I run this code, though, I get [-144.37178139160187, -125.00368410330502], whereas it should be around [54.644979834216294, 45.89394897084863].
Am I doing something wrong here?

@jlouthan Is it possible that we use more than 8 source or destination points to transform?

@jlouthan Is it possible that we use more than 8 source or destination points to transform?

@sebastian-raubach Yeah, I have spent a while testing, and I think there's something pretty wrong.
My testing was basically attempting to map a point in the middle of 4 points to the same 4 points - it should end up at the exact same location. But for many sets of numbers, it doesn't.
I don't have a clear pattern for when it goes wrong. But posting some of my testing code here in case it's useful to someone.
const middle = (corners) => [
    (corners[0] + corners[2] + corners[4] + corners[6]) / 4,
    (corners[1] + corners[3] + corners[5] + corners[7]) / 4,
];
const clean = (corners) => corners.map((x) => x.toFixed(1)).join(',');
function run(src, dest, pt) {
    var perspT = PerspT(src, dest);
    var dstPt = perspT.transform(pt[0], pt[1]);
    return dstPt;
}

const check = (a, b) => clean(a) === clean(b);
const test = (corners) => {
    const out = run(corners, corners, middle(corners));
    const src = middle(corners);
    console.log(
        !check(src, out) ?


single space as name crashes canvas

When user sets their name to  , (one space), the canvas crashes for all collaborators.
Sentry Error ID
47ce49a2fb1c4194adb17bc541b3fec1

looking into it

@nokacper24 thanks for the report!, the fix is deployed 🚀


SetContent->SetHtmlContent

Rename SetContent to SetHtmlContent to make its use more clear.
Fixes #18.

Hi @ryanbrandenburg, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

:shipit: Will need reaction PRs in a few repos I beleive.


Fix exception in PathInfo_ component

This fixes a crash that happens when you try to use the path info dropdown in the file viewer in the Electron app.
For a brief moment before pathInfo.deeplinkPath loads in, it has a falsey value which throws an error in <CopyText />. This is due to a recent change (#19963) which will throw if the text property is falsey, but a loadText property is not provided.
The render of the universal path field is held off until it is ready, like how the local path field is done now.

Thanks for the pull request! This was fixed in #20122.


cannot configure with sqlite3

Hello
I like test with an export to sqlite3 on a debian 11.
but it say checking for sqlite3_open_v2 in -lsqlite3... no
I dont't know where to loook to fix it
here the log from ./configure
checking for workbook_new in -lxlsxwriter... yes
checking for yajl_gen_alloc in -lyajl... no
checking for sqlite3_open_v2 in -lsqlite3... no
checking that generated files are newer than configure... done
configure: creating ./config.status
config.status: creating Makefile
config.status: executing depfiles commands
config.status: executing libtool commands
with python3 , i have proper access to sqllite3 (v2 level).
root@debian11:/home/ladmin/patois/fmptools-master# python3
Python 3.9.2 (default, Feb 28 2021, 17:03:44)
[GCC 10.2.1 20210110] on linux
Type "help", "copyright", "credits" or "license" for more information.

import sqlite3
con = sqlite3.connect('example.db')
cur = con.cursor()
cur.execute('''CREATE TABLE stocks
...                (date text, trans text, symbol text, qty real, price real)''')
<sqlite3.Cursor object at 0x7f8319324730>
cur.execute("INSERT INTO stocks VALUES ('2006-01-05','BUY','RHAT',100,35.14)")
<sqlite3.Cursor object at 0x7f8319324730>
con.commit()
any changes have been committed or they will be lost.
File "", line 1
any changes have been committed or they will be lost.
IndentationError: unexpected indent
con.close()

Here the solution, just add the developper version of sqllite3
sudo apt-get install libsqlite3-dev


Can you create an option so we can install on pinokio?

Hello, can you make a version to easily install it on https://pinokio.computer/?

Very interesting. Actually, it has been on my mind about it. This week we're releasing a number of changes in the new version. I'll be quite busy testing and answering people's questions. I think by next week, I'll definitely go through it and try it out. It's actually a very good way and helpful for people. Thank you for bringing it up.


Fix null errors in SetTestPipelineVersion

Fix null errors in SetTestPipelineVersion
Rename Publish Packages to Upload Packages in azuresdkpartnerdrops-to-nugetfeed.yml

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Swipe and AppBar Actions doesnt work well.

When i use Swipe property i can tap all my icon button widget. exists some space or with near to the device side that cause the problem.

I cant touch all this area...

Update... i fixed it changing the
const double _kEdgeDragWidth = 20.0;
to
const double _kEdgeDragWidth = 0.0;
thats all, Thanks you

Consider to change that constant to a public for change it before init state or implement it as a parameter.

tl;dr: definition of a swipe area
If possible the definition of a designated swipe area might be beneficial.
Having a Listview with an overlay button to open the sidebar, the user might also like to swipe.
If the items also have a swipe action attached to them it might get complicated, so limiting the flutter_inner_drawer swipe area to the button area might help here.


Annotations are broken

Having this code:
searchType: task.done
searchTarget: Walk, Run
datasetName: Walk, Run
folder: Diary
month:
    mode: annotation
    annotation: 🚶♀,🏃♀
    startWeekOn: 'Mon'
    todayRingColor: orange
    selectedRingColor: white
    showSelectedValue: false
    todayRingColor: white

It outputs this.

if i hover that mountains of text, it's like this. looks like it prints the html and does not render it.

Version 1.10.6

Hi @felixsanz,
For some reason the emoji characters (🚶♀ and🏃♀) in your code block or notes are replaced by html img tags.
That's why there are texts like '<img draggable="false" class="emoji" ......' in the rendered view.
You can copy the emoji here.

This is not the first time users met this problem.
So I decide add a function to deal with html img tags.

Hi @felixsanz,
For some reasons, the emoji characters ( and) shown in your code block or notes are replaced by html img tags. (They are not emoji characters) That's why there are texts like '<img draggable="false" class="emoji" ......' in the rendered view.
Please copy the emoji here and replace those in your code block by the copied ones.
By the way, did you install any plugin related to emoji?

Yes i know that the code blocks are replaced by html, but doesn't matter where i copy those emojis, or if i use the two emoji plugins available for obsidian, it's always the same result

Hi,
Please download this example and check if it works.
Here is what I got

Nope, same issue. I'm using Linux in case is related

Mmm…
Will find a Linux machine and do some tests on it.
Thanks for the report.

The emojis can be displayed with no problem in obsidian.

The problem is your calendar that renders a  tag instead of the emoji. Maybe the tag is malformed?

Tracker did not create any html img tag.
Maybe the unicode characters are modified at some point before fed into Tracker.
The first thought about this is the YAML parser.
Tracker uses the embeded YAML parser provided by Obsidian API.
Maybe the parser behaves different on Linux.

Tracker does not create any html img tag.
Maybe the unicode characters are modified at some point before fed into Tracker.
The first thought about this is the YAML parser.
Tracker uses the embeded YAML parser provided by Obsidian API.
Maybe the parser behaves different on Linux.
By the way, could you test this for me?
Use single quotes to wrap the unicode characters
like this
searchTarget: ','

doesn't fix it :(
You're right, it's the YAML parser, check this:
https://github.com/blacksmithgu/obsidian-dataview/issues/118
Even the gear emoji displays wrong

Commit: https://github.com/blacksmithgu/obsidian-dataview/commit/fc7c8e6d65b8e13ebaad94918f971d17385338bb

It's different.
We use the function parseYaml from Obsidian API to get parameters from YAML.
Thus if what we got is an image tag, the only option is to turn it back to a emoji character.

The problem is similar but the solution is not related.
We use the function parseYaml from Obsidian API to get parameters from YAML.
Thus if what we got are image tags, the only option for us is to turn them back to emoji characters.
Will do it.

The alt="" parameter of the image tag already include the emoji, maybe you should insert just the content of the alt="" parameter into the calendar, instead of the whole  tag? If that's a solution looks pretty easy!

The alt="" parameter of the image tag already include the emoji, maybe you should insert just the content of the alt="" parameter into the calendar, instead of the whole  tag?

That was my plan.

Fixed!

Thanks for the fix.
Since image tags could be placed in any field in YAML, the problem should be treated more generally.
Also, we don't want to leave any image tags to the SVG graph building.
Thus, all image tags with alt will be replaced by the text in alt, otherwise replaced by empty string.
A fix has been committed (3810a63) for this.

It works on the calendar but you say that the problem affects the whole YAML options and it's not fixed there.
Example:
...
line:
    title: Weight Log 🚶
...

Will fix it.
Really appreciate your report.

no problem. if you need test specific options count on me

Hi @felixsanz,
Thanks again.
The fix was released with version 1.10.7.
You are still welcome to make pull requests or suggestions.


Javascript is the devil

Have you considered rewriting this in Elixir?

Hey Alex, thank you for your suggestion but I'm not familiar with Elixir. In which way does it improve the current code?


Rename Symbol command not working - text does not change

Environment data
dotnet --info output: 2.0.0
C# Extension version: 1.12.1

VSCode Version: Code - Insiders 1.18.0-insider (de68b077b53d6eeafeaf39699d6384c4b8750a19, 2017-10-16T05:22:08.224Z)
OS Version: Linux x64 4.10.0-37-generic
Extensions:

Extension
Author (truncated)
Version

ng-template
Ang
0.1.7

bracket-pair-colorizer
Coe
0.10.9

erb
Cra
0.0.1

html-leaf
Fra
0.1.3

json-to-ts
Mar
1.5.3

Angular-BeastCode
Mik
5.0.7

material-icon-theme
PKi
2.2.2

path-intellisense
chr
1.4.2

gitignore
cod
0.5.0

githistory
don
0.2.3

tslint
eg2
1.0.11

vscode-ember
emb
0.1.0

prettier-vscode
esb
0.24.0

auto-rename-tag
for
0.0.14

search-node-modules
jas
1.2.0

Angular2
joh
2.7.0

csharp
ms-
1.12.1

debugger-for-chrome
msj
3.3.1

EmberES6Snippets
pha
2.1.2

typescript-hero
rbb
1.5.0

vscode-icons
rob
7.15.0

git-merger
sha
0.3.6

autoimport
ste
1.2.2

(1 theme extensions excluded)

Steps to Reproduce:

Click on a symbol (class name)
Hit F2 (Or right click and choose rename)
Should rename the symbol and any references to it, but it does not, nothing changes.

Hi! Could you provide your OmniSharp Log? You can see this by selecting View->Output from the VS Code main menu and then choose "OmniSharp Log" from the drop-down at the top-right of the Output pane.

I have sane problem.
Best regards

@franklupo: I have the same request:

Hi! Could you provide your OmniSharp Log? You can see this by selecting View->Output from the VS Code main menu and then choose "OmniSharp Log" from the drop-down at the top-right of the Output pane.

Story not riproduce error

I'm having the same problem.
Here's my omnisharp log
Starting OmniSharp server at 2017-12-27 13:01:50
    Target: /Users/douglas/repos/noughts-and-crosses/noughts-and-crosses.sln

OmniSharp server started wth Mono
    Path: /Users/douglas/.vscode/extensions/ms-vscode.csharp-1.13.1/.omnisharp/omnisharp/OmniSharp.exe
    PID: 31198

[info]: OmniSharp.Stdio.Host
        Starting OmniSharp on MacOS 10.12.6 (x64)
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Located 2 MSBuild instance(s)
            1: Mono 15.0 - "/Library/Frameworks/Mono.framework/Versions/5.2.0/lib/mono/msbuild/15.0/bin"
            2: StandAlone 15.0 - "/Users/douglas/.vscode/extensions/ms-vscode.csharp-1.13.1/.omnisharp/omnisharp/msbuild/15.0/Bin"
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Registered MSBuild instance: Mono 15.0 - "/Library/Frameworks/Mono.framework/Versions/5.2.0/lib/mono/msbuild/15.0/bin"
            CscToolPath = /Users/douglas/.vscode/extensions/ms-vscode.csharp-1.13.1/.omnisharp/omnisharp/msbuild/15.0/Bin/Roslyn
            CscToolExe = csc.exe
[info]: OmniSharp.Cake.CakeProjectSystem
        Detecting Cake files in '/Users/douglas/repos/noughts-and-crosses'.
[info]: OmniSharp.Cake.CakeProjectSystem
        Could not find any Cake files
[info]: OmniSharp.DotNet.DotNetProjectSystem
        Initializing in /Users/douglas/repos/noughts-and-crosses
[info]: OmniSharp.DotNet.DotNetProjectSystem
        Auto package restore: False
[info]: OmniSharp.DotNet.DotNetProjectSystem
        Update workspace context
[info]: OmniSharp.DotNet.DotNetProjectSystem
        Resolving projects references
[info]: OmniSharp.MSBuild.MSBuildProjectSystem
        Detecting projects in '/Users/douglas/repos/noughts-and-crosses/noughts-and-crosses.sln'.
[info]: OmniSharp.Script.ScriptProjectSystem
        Detecting CSX files in '/Users/douglas/repos/noughts-and-crosses'.
[info]: OmniSharp.Script.ScriptProjectSystem
        Could not find any CSX files
[info]: OmniSharp.Stdio.Host
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpWorkspaceOptionsProvider
[info]: OmniSharp.Stdio.Host
        Configuration finished.
[info]: OmniSharp.Stdio.Host
        Omnisharp server running using Stdio at location '/Users/douglas/repos/noughts-and-crosses' on host 31180.

I'm having the same issue. Here is my Omnisharp log:
[dbug]: OmniSharp.Stdio.Host         ************ Request ************ {   "Type": "request",   "Seq": 245,   "Command": "/rename",   "Arguments": {     "FileName": "c:\\work\\aircost\\PartDrawings\\Sketches\\RectTaper.cs",     "Line": 7,     "Column": 25,     "WantsTextChanges": true,     "RenameTo": "OffsetX"   } } [dbug]: OmniSharp.Stdio.Host         ************  Response ************  {   "Request_seq": 245,   "Command": "/rename",   "Running": true,   "Success": true,   "Message": null,   "Body": {     "Changes": [],     "ErrorMessage": null   },   "Seq": 1699,   "Type": "response" }

This doesn't reproduce with the latest VSCode and C# plugin, so I'll close out this older issue.


Fix cargo build failed caused by no authentication

What is changed and how it works?
Close #31
What's Changed:
Only those authorized to database-mesh/lrpar can clone project by ssh. Using https instead of ssh when cloning lrpar.

Thanks a lot


[QST] Additional GPU mem reservation when creating a Dataset causes OOM when allocating all GPU mem to the LocalCUDACluster

Hi!
I've run into such a problem:
from dask_cuda import LocalCUDACluster
from dask.distributed import Client

cluster = LocalCUDACluster(
    n_workers=1,                 # Number of GPU workers
    device_memory_limit="12GB",  # GPU->CPU spill threshold (~75% of GPU memory)
    rmm_pool_size="16GB",        # Memory pool size on each worker
)
client = Client(cluster)

# NOTE: Importing Merlin before cluster creation will ALSO create this additional reservation on GPU
from merlin.core.utils import set_dask_client

set_dask_client(client)

import numpy as np
import pandas as pd

df = pd.DataFrame({
    "a": np.random.randint(0, 100, 1000),
    "b": np.random.randint(0, 100, 1000),
    "c": np.random.randint(0, 100, 1000),
})

import nvtabular as nvt

# No matter if I specify the `client` or not, there is an additional reservation on GPU created
# that causes "cudaErrorMemoryAllocation out of memory" here.
ds = nvt.Dataset(df, client=client)

I run this code in JupyterLab on the GCP VM with NVIDIA V100 16GB GPU.
I've also tried nvtabular.utils.set_dask_client and it didn't solve the problem.
Questions:

Is it expected behavior and I don't understand something?
Can't I simply allocate all memory for the cluster and make NVTabular use it?
How shall the LocalCUDACluster be configured then?

@piojanu  what helps with the OOM issues with NVT is the part_size and the row group memory size of your parquet file(s).  you can also repartition your dataset and save back to disk, and that might help with OOM. for LocalCUDACluster args you can read here: https://docs.rapids.ai/api/dask-cuda/nightly/api/
if you have a single GPU you can try to set the row group size of your files and that would help without LocalCudaCluster.
There is a LocalCudaCluster example here:
https://github.com/NVIDIA-Merlin/Merlin/blob/main/examples/quick_start/scripts/preproc/preprocessing.py

Hi!
I have follow up questions:

What is the rule of thumb to set “part_size and the row group memory size”? Make them smaller or bigger? How one influences the other?
What do you mean by “you can also repartition your dataset and save back to disk”? Can you show me some code snippet?

Thanks for help :)

By accident, I've found out that merlin.io.dataset.Dataset.shuffle_by_keys is root of this OOMs.

It makes ops.Categorify OOM even when following the troubleshooting guide:

Setting-up the LocalCUDACluster doesn't help.
Saving with consistent row group size (maximum, row group can happen to be smaller, because in a dask partition which saves one file, there is usually a reminder which doesn't fit a full row group) doesn't help.

I've also tested it and there is no need to shuffle when doing GroupBy on the Dask DataFrame when loading data that was sorted by session_id in BigQuery.

No matter if you set the arguments index and calculate_divisions to the dd.read_parquet.
However, nvt.ops.GroupBy on the same data doesn't return expected number of sessions.


[otbn, dv] Added otbn_dmem_err testcase

[otbn, dv] Added otbn_dmem_err testcase
Following files have been added/ changed in this commit:

otbn_dmem_err_vseq.sv - Sequence to inject errors in dmem.
otbn_model_if - Added invalidate_dmem pin which tells the ISS model
to trash dmem.
otbn_core_model - Calls the invalidate_dmem function in C model.
otbn_model.cc/ iss_wrapper.cc - Set the ivalidate_dmem bit in python
code.
stepped.py - Calls EmptyDmem function in dmem.
dmem.py - Added EmptyDmem function  to trash dmem.

This commit also contains changes to pass a valid bit to specify if the
data in dmem is valid or invalid. This is used in c model to compare
between RTL and ISS model dmem.

Oh, and could you rebase onto latest master? My patch should disappear and this will just be a single commit.

Hi @prajwalaputtappa! I've just force-pushed to rebase over the other change that just went in. It all looks good to me, thanks: I'll merge once CI has finished (and I've eaten my dinner!).

Hi @rswarbrick ,
Thanks for doing force push on my behalf. But CI had failed for no apparent reason and I just rebased and pushed again. Hoepfully it should go through this time.

Hi Prajwala! I'm really sorry, but I meant to merge this before the CRC DV change and forgot, which means you're having to deal with the massive merge conflict. Oops! I've just rebased things locally and have run a couple of tests and all looks good. I'm about to force-push with the results: would you mind taking a look to check that they still match what you expect? If you're happy (and CI passes!), we can merge.

Hi Prajwala! I'm really sorry, but I meant to merge this before the CRC DV change and forgot, which means you're having to deal with the massive merge conflict. Oops! I've just rebased things locally and have run a couple of tests and all looks good. I'm about to force-push with the results: would you mind taking a look to check that they still match what you expect? If you're happy (and CI passes!), we can merge.
@rswarbrick, all changes look good to me. Please go ahead and merge it if CI passes. Thanks for helping with resolving the conflicts.


feat: allow API key to be configured at ImgixAPI instantiation

This PR allows users to pass an API key to the ImgixAPI constructor. This will, in turn, be passed through to each call to request() as an Authorization header. Before this, users would need to pass the API key with each request call. This was overly repetitive given that an API key can be used for many sources under an account. This change should yield a more dry user experience, as the key will not need to be manually specified with each call.
Example
const imgix = new ImgixAPI({
                apiKey: "TEST-KEY"
            });

// DRY request() call
ix.request('assets/{sourceId}')
.then(res => console.log(res));

This PR also includes some slight tweaks to the test suite: more test cases in the browser tests and consolidation of testing constants.

I got a bit caught up yak shaving trying to get this to work consistently. Which all sort of begs the question: should imgix-management-js even work in a browser if it requires the user to pass their API key?


CocoaPods and Swift?

The instructions seem to explicitly exclude the idea of adding this via CocoaPods and using it with Swift. Is this the case? I gave it a shot and was told,
Could not build Objective-C module 'FBLPromises'

Thank you for filing the issue, Ken!
We've been wrangling with CocoaPods for a few days already, and still couldn't come up with a proper Spec for Swift that wouldn't also break Bazel and Swift PM.
We've filed an issue on CocoaPods and are looking forward for any suggestions from the community. It has more details about the issues we're facing. We'll definitely keep you posted on the progress.
Thanks.

Thank you for the reply.
Hmmm. So I'm not crazy. There actually is an issue. How refreshing!

I am facing the same issue. My Podfile looks like:
platform :ios, '9.0'

target 'GooglePromises' do
  use_frameworks!

  # Pods for GooglePromises
  pod 'PromisesObjC', '~> 1.0'
end

and when I do a "pod install", then open the workspace, and build, then it builds OK. But once I include:
import FBLPromises

into one of my project Swift files, then I get the compilation error:
'DotSyntax/FBLPromise+DotSyntax.h': file not found

at line 17 in FBLPromise+DotSyntax+All.h

Any updates on this ?

whether  platform support ios 8?

Hey everyone,
Was able to successfully install the 'PromisesSwift' pod into an all Swift project. Will go ahead and close this issue.
Thanks again for everyone's patience!


Add Dark mode feature

Is your feature request related to a problem? Please describe.
no
Which Program are you part of?

[x] DCP21
[ ] Any other

Additional context
Add any other context or screenshots about the feature request here.

@HarshitPandey251 assigning!!


Add an option to use index swap in import command

Pull Request
Related issue
Fixes #277
What does this PR do?

This PR adds an option to use index swap to prevent downtime while importing data in an existing index.

PR checklist
Please check if your PR fulfills the following requirements:

[x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
[x] Have you read the contributing guidelines?
[x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you @thislg


openthread manual joiner hangs

CONFIG_OPENTHREAD_MANUAL_START=y
I am programmatically trying to call otJoinerStart(). The otJoinerStart() callback is never executed and looking at wireshark the expected 'discover' beacons are never sent.
Here is code that reproduces the issue:

#include <zephyr.h>
#include <logging/log.h>

#include <net/openthread.h>

#include <openthread/thread.h>
#include <openthread/server.h>
#include <openthread/sntp.h>
#include <openthread/joiner.h>

LOG_MODULE_REGISTER(app_sample);

struct k_poll_signal app_join_trigger_signal;

static struct k_poll_event join_trigger_events[1] = {
  K_POLL_EVENT_INITIALIZER(K_POLL_TYPE_SIGNAL,
                           K_POLL_MODE_NOTIFY_ONLY,
                           &app_join_trigger_signal),
};

void app_join_attempt_timer_callback(struct k_timer *dummy)
{
  k_poll_signal_raise(&app_join_trigger_signal, 1);
}

K_TIMER_DEFINE(app_join_attempt_timer, app_join_attempt_timer_callback, NULL);

void app_joiner_callback(otError join_err_code, void *passback)
{
  bool success = false;

  switch (join_err_code)
  {
    case OT_ERROR_NONE:
      LOG_INF("JOIN Success\r\n");
      success = true;
      break;
    case OT_ERROR_SECURITY:
      LOG_INF("JOIN Error - Security\r\n");
      break;
    case OT_ERROR_NOT_FOUND:
      LOG_INF("JOIN Error - Not found\r\n");
      break;
    default:
    case OT_ERROR_RESPONSE_TIMEOUT:
      LOG_INF("JOIN Error - Response timeout\r\n");
      break;
  }

  k_poll_signal_raise(&app_join_trigger_signal, success);

}

void main(void)
{
  otError error;

  LOG_INF("Waiting 30secs...");

  k_poll_signal_init(&app_join_trigger_signal);

  k_timer_start(&app_join_attempt_timer, K_SECONDS(30), K_NO_WAIT);

  k_poll(join_trigger_events, 1, K_FOREVER);
  join_trigger_events[0].signal->signaled = 0;
  join_trigger_events[0].state = K_POLL_STATE_NOT_READY;

  LOG_INF("Calling otJoinerStart\r\n");

  struct otInstance *ot_instance = openthread_get_default_instance();

  error = otIp6SetEnabled(ot_instance, true);
  __ASSERT_NO_MSG(error == OT_ERROR_NONE);

  // Must reset to 0xFFFF or it only tries to join default panid
  error = otLinkSetPanId(ot_instance, 0xFFFF);
  __ASSERT_NO_MSG(error == OT_ERROR_NONE);

  error = otJoinerStart(ot_instance, "TEST1234", NULL, "VENDOR", "MODEL", "SWVER", NULL, app_joiner_callback, NULL);
  if (error != OT_ERROR_NONE)
  {
    LOG_INF("otJoinerStart error\r\n");
  }
  else
  {
    // Wait for otJoinerStart callback to be invoked (***never happens***)
    k_poll(join_trigger_events, 1, K_FOREVER);
    join_trigger_events[0].signal->signaled = 0;
    join_trigger_events[0].state = K_POLL_STATE_NOT_READY;

  }
}

Enabling debug in openthread only shows the following...
NOTE: The 'Failed to send IPv6' might be a clue? Not sure.
[00:00:30.636,505] <inf> app_sample: Calling otJoinerStart

[00:00:30.636,688] <dbg> net_openthread.-MAC-----: RadioState: Sleep -> Receive
[00:00:30.636,688] <dbg> net_openthread.-MAC-----: Idle mode: Radio receiving on channel 11
[00:00:30.636,779] <inf> net_openthread: -CORE----: Notifier: StateChanged (0x00001000) [Ip6Mult+]
[00:00:30.636,810] <inf> net_l2_openthread: State changed! Flags: 0x00001000 Current role: 0
[00:00:30.637,176] <inf> net_openthread: -CORE----: Notifier: StateChanged (0x00000001) [Ip6+]
[00:00:30.637,207] <inf> net_l2_openthread: State changed! Flags: 0x00000001 Current role: 0
[00:00:30.637,420] <inf> net_openthread: -CORE----: Notifier: StateChanged (0x00000008) [LLAddr]
[00:00:30.637,420] <inf> net_l2_openthread: State changed! Flags: 0x00000008 Current role: 0
[00:00:30.637,573] <inf> net_openthread: -CORE----: Notifier: StateChanged (0x01000000) [NetifState]
[00:00:30.637,603] <inf> net_l2_openthread: State changed! Flags: 0x01000000 Current role: 0
[00:00:30.637,695] <inf> net_openthread: -MESH-CP-: Joiner starting
[00:00:30.638,061] <dbg> net_openthread.-MAC-----: RadioExtAddress: 56bbc40387a1d9fc
[00:00:30.638,183] <inf> net_openthread: -CORE----: Notifier: StateChanged (0x00000002) [Ip6-]
[00:00:30.638,183] <inf> net_l2_openthread: State changed! Flags: 0x00000002 Current role: 0
[00:00:30.638,397] <inf> net_openthread: -CORE----: Notifier: StateChanged (0x00000001) [Ip6+]
[00:00:30.638,397] <inf> net_l2_openthread: State changed! Flags: 0x00000001 Current role: 0
[00:00:30.638,610] <inf> net_openthread: -CORE----: Notifier: StateChanged (0x00000008) [LLAddr]
[00:00:30.638,610] <inf> net_l2_openthread: State changed! Flags: 0x00000008 Current role: 0
[00:00:30.639,007] <dbg> net_openthread.-MAC-----: Request to start operation "TransmitDataDirect"
[00:00:30.639,007] <dbg> net_openthread.-MAC-----: Starting operation "TransmitDataDirect"
[00:00:30.639,343] <dbg> net_openthread.==============================[TX len=000]===~
[00:00:30.639,648] <dbg> net_openthread.---------------------------------------------~
[00:00:30.639,678] <dbg> net_openthread.-MAC-----: Finishing operation "TransmitDataDirect"
[00:00:30.639,801] <inf> net_openthread: -MAC-----: Failed to send IPv6 UDP msg, len:56, chksum:2c3b, to:None, sec:no, error:Abort, prio:net
[00:00:30.639,923] <inf> net_openthread: -MAC-----:     src:[fe80:0:0:0:54bb:c403:87a1:d9fc]:19788
[00:00:30.640,045] <inf> net_openthread: -MAC-----:     dst:[ff02:0:0:0:0:0:0:2]:19788
[00:00:30.640,106] <dbg> net_openthread.-MAC-----: Idle mode: Radio receiving on channel 11
[00:00:30.640,258] <inf> net_openthread: -CORE----: Notifier: StateChanged (0x00008000) [PanId]
[00:00:30.640,258] <inf> net_l2_openthread: State changed! Flags: 0x00008000 Current role: 0
[00:00:30.640,319] <dbg> net_openthread.-MAC-----: RadioPanId: 0x3942
[00:00:30.640,441] <inf> net_openthread: -MLE-----: Send Discovery Request (ff02:0:0:0:0:0:0:2)
[00:00:30.640,533] <inf> net_openthread: -CORE----: Notifier: StateChanged (0x08000000) [JoinerState]
[00:00:30.640,563] <inf> net_l2_openthread: State changed! Flags: 0x08000000 Current role: 0
[00:00:30.640,594] <inf> net_openthread: -MESH-CP-: JoinerState: Idle -> Discover

In the reproduction code, if I simply replace the
error = otJoinerStart(...) with error = OT_ERROR_NONE;
and then manually invoke the 'ot joiner start TEST1234' from the shell/cli, it works fine, I see proper wireshark discover packets. The shell invoked command show a very different debug log sequence...
uart:~$ ot joiner start TEST1234
Done
[00:00:36.899,780] <inf> net_openthread: -MESH-CP-: Joiner starting
[00:00:36.900,177] <dbg> net_openthread.-MAC-----: RadioExtAddress: a671cbf48620e21b
[00:00:36.900,451] <dbg> net_openthread.-MAC-----: RadioPanId: 0xc64b
[00:00:36.900,604] <inf> net_openthread: -MLE-----: Send Discovery Request (ff02:0:0:0:0:0:0:2)
[00:00:36.900,604] <inf> net_openthread: -MESH-CP-: JoinerState: Idle -> Discover
[00:00:36.900,909] <inf> net_openthread: -CORE----: Notifier: StateChanged (0x0800800b) [Ip6+ Ip6- LLAddr PanId JoinerState]
[00:00:36.900,939] <inf> net_l2_openthread: State changed! Flags: 0x0800800b Current role: 0
[00:00:36.901,092] <dbg> net_openthread.-MAC-----: Request to start operation "TransmitDataDirect"
[00:00:36.901,123] <dbg> net_openthread.-MAC-----: Starting operation "TransmitDataDirect"
[00:00:36.901,214] <dbg> net_openthread.-MAC-----: RadioState: Receive -> CsmaBackoff
[00:00:36.901,245] <dbg> net_openthread.-MAC-----: RadioState: CsmaBackoff -> Transmit
[00:00:36.905,395] <dbg> net_openthread.-MAC-----: RadioState: Transmit -> Receive
[00:00:36.905,761] <dbg> net_openthread.==============================[TX len=037]===~
[00:00:36.906,280] <dbg> net_openthread.| 01 D8 49 FF FF FF FF 4B | C6 1B E2 20 86 F4~
[00:00:36.906,738] <dbg> net_openthread.| A6 7F 3B 02 F0 4D 4C 4D | 4C CD F5 FF 10 1A~
[00:00:36.907,012] <dbg> net_openthread.| 02 28 00 20 D0 .. .. .. | .. .. .. .. .. ..~
[00:00:36.907,348] <dbg> net_openthread.---------------------------------------------~
[00:00:36.907,348] <dbg> net_openthread.-MAC-----: Finishing operation "TransmitDataDirect"
[00:00:36.907,501] <inf> net_openthread: -MAC-----: Sent IPv6 UDP msg, len:56, chksum:cdf5, to:0xffff, sec:no, prio:net
[00:00:36.907,623] <inf> net_openthread: -MAC-----:     src:[fe80:0:0:0:a471:cbf4:8620:e21b]:19788
[00:00:36.907,745] <inf> net_openthread: -MAC-----:     dst:[ff02:0:0:0:0:0:0:2]:19788
[00:00:36.907,806] <dbg> net_openthread.-MAC-----: Idle mode: Radio receiving on channel 11
[00:00:37.207,855] <dbg> net_openthread.-MAC-----: Request to start operation "TransmitDataDirect"
[00:00:37.207,885] <dbg> net_openthread.-MAC-----: Starting operation "TransmitDataDirect"
[00:00:37.207,977] <dbg> net_openthread.-MAC-----: RadioState: Receive -> CsmaBackoff
[00:00:37.208,007] <dbg> net_openthread.-MAC-----: RadioState: CsmaBackoff -> Transmit
[00:00:37.211,425] <dbg> net_openthread.-MAC-----: RadioState: Transmit -> Receive

I started with Nordic Connect SDK version 1.4.2 of Zephyr and tried various versions up to the 1.4.99 / master.
I am using nrf52840dk_nrf52840

Hi @krbvroc1,
OpenThread API is not thread safe, therefore when calling OT functions from other thread (like main) you need to protect the calls with a mutex, there's a dedicated function for that. Just add:
  struct openthread_context *ctx = openthread_get_default_context();

  openthread_api_mutex_lock(ctx);

before you call the OT functions, and then
openthread_api_mutex_unlock(ctx);

after.
In your case, otJoinerStart() likely cause the scheduler to run and since OT thread has higher priority than main, it could've process the OT engine in the middle of otJoinerStart() call. I've observed similar behaviour with commissioner. I've reproduced your issue locally and adding a proper mutex protection fixes it.

@rlubos Thank you very much... I missed that detail!
Is this for the entire API (every single call)? For example,  otCoapNewMessage() and all the individual functions such as appending URI, etc? Or if you retrieve a request from otCoapNewMessage() is it safe to build the message without locking?

There likely are APIs that are safe to call w/o mutex protection, but each case would need to be considered individually, I think it's safer to assume that you need to lock the OT thread in general.


LICENSE / Open source license documentation

What license are you releasing your code under?

It's MIT, open source. Thanks for raising this, I've made it explicit.


Proper fuzz testing

Now that Go has builtin fuzz testing in v1.18 we can use https://github.com/benhoyt/goawk/blob/master/interp/fuzz_test.go and get rid of the top-level fuzz directory.
However, a lot of things fail because of infinite loops. Can we add a check in interp.execute that fails after 1,000,000 (or so) statements, or a timeout? This should probably be behind a build tag and turned off by default, because it will decrease performance significantly.
And then once we've added good fuzz testing, run the fuzzer again and see what it finds.

Katie Hockman noted on #fuzzing in Gophers Slack that fuzz tests should be fast (https://go.dev/doc/fuzz/#suggestions) and that there's currently a 1 second timeout (https://cs.opensource.google/go/go/+/master:src/internal/fuzz/worker.go;l=31), so we should probably set the timeout here to (say) half a second.


Fix intermittent failure in WorkerCancelledTest

Fixes https://github.com/ballerina-platform/ballerina-lang/issues/13492

Codecov Report

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

@@             Coverage Diff              @@
##             master   #13504      +/-   ##
============================================
- Coverage     63.97%   63.95%   -0.02%
  Complexity      661      661
============================================
  Files          2149     2149
  Lines         94349    94349
  Branches      12436    12436
============================================
- Hits          60361    60345      -16
- Misses        29108    29120      +12
- Partials       4880     4884       +4

Impacted Files
Coverage Δ
Complexity Δ

...allerinalang/net/jms/LoggingExceptionListener.java
50% <0%> (-50%)
0% <0%> (ø)

...a/org/ballerinalang/bre/bvm/WorkerDataChannel.java
77.77% <0%> (-9.26%)
0% <0%> (ø)

...rinalang/runtime/threadpool/ThreadPoolFactory.java
61.9% <0%> (-4.77%)
0% <0%> (ø)

...java/org/ballerinalang/bre/bvm/BLangVMStructs.java
50% <0%> (-1.67%)
0% <0%> (ø)

...org/ballerinalang/bre/bvm/WaitCallbackHandler.java
85.58% <0%> (+2.7%)
0% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


dart-sass-embedded: drop

Description of changes

Archived upstream, functionality moved into dart-sass.
Fails on Hydra: build

ZHF: #265948
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

This change was already included in #263345, I just didn't notice when I first opened this PR.

We can drop it as it's included in dart-sass now


prepped for a release

Hmmm I've tried up and down to get credentials to work from a PR and I don't think it's possible. Merging for the release.


video api added in bidRequest

Type of change

[ ] Bugfix

[X] Feature

[ ] New bidder adapter

[ ] Code style update (formatting, local variables)

[ ] Refactoring (no functional changes, no api changes)

[ ] Build related changes

[ ] CI related changes

[ ] Does this change affect user-facing APIs or examples documented on http://prebid.org?

[ ] Other

Description of change
This PR includes changes to add api array for video mediatype. default is empty.
{
 request: {
        mediaTypes: {
          video: {
            api: [2,7],
          },
        },
}}

Other information
Tests are added for api.

Closing in favor of https://github.com/dailymotion-oss/Prebid.js/pull/5 (rebased + branch-name change)


UI: Improve Error Message for Invalid Team Name Input

Currently, when a user attempts to input an illegal team name, the system only displays a generic error message: Failed to create / join the team. This error message could be more descriptive and provide specific feedback on what went wrong with the suggested team name.
Here are some rules for valid team names that should be communicated in the error message (these are already available in the tooltip menu):

The team name should not contain uppercase letters.
The '-' character is not allowed.
Special characters are not allowed.

By providing a more detailed error message, we can guide the user to correct their input, thus enhancing the user experience.

Mhh that used to work, we have a specifc error message for that which used to be shown if the team name didn't match the regex in the frontend. https://github.com/juice-shop/multi-juicer/blob/main/juice-balancer/ui/src/pages/JoinPage.js#L14

This is how it looks for me (after pressing create=

This was fixed with https://github.com/juice-shop/multi-juicer/pull/174 😉

This was fixed with https://github.com/juice-shop/multi-juicer/pull/174 😉
I wonder if it would make sense to align the error message for this and https://github.com/juice-shop/multi-juicer/issues/179, though, to have a consistent UI. But I'll leave this up to @J12934.

Sorry overlooked this.
To me this is fine right now. I think having a different style for validation and for an actual error which was thrown for a valid input is totally fine.
I'm closing this ticket now. Thank you all for the input and feedback here 🙌


[DOCS]:

Give brief description of what documentation changes were made.
Use Swagger to create an open Api Documentation page

What's your slack username?

gosteve


[SPARK-28002][SQL] Support WITH clause column aliases

What changes were proposed in this pull request?
This PR adds support of column aliasing in a CTE so this query becomes valid:
WITH t(x) AS (SELECT 1)
SELECT * FROM t WHERE x = 1

How was this patch tested?
Added new UTs.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@dongjoon-hyun, @gatorsmile this is another feature that PostgreSQL does support.

ok to test

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11638/
Test PASSed.

Test build #106393 has started for PR 24842 at commit 8a7e7e0.

LGTM pending Jenkins, thanks!

In fact, I was thinking, instead of adding test cases in the SQLQuerySuite, maybe it's better to add test cases in the PlanParserSuite and AnalysisSuite.
The changes in this PR are purely about the parsing phase and running full-blown SQL queries for parsing changes is slow and unnecessary. The Spark PR builder already takes a long time to finish, would be better to cut the cost whenever possible.

Test build #106393 has finished for PR 24842 at commit 8a7e7e0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106393/
Test PASSed.

Test build #106405 has started for PR 24842 at commit 92cc5d2.

Thanks @liancheng for the review. I've moved analysis error test cases to AnalysisSuite. But I have left the positive test case in SQLQuerySuite as I think the value of x column should be tested.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11650/
Test PASSed.

Test build #106405 has finished for PR 24842 at commit 92cc5d2.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106405/
Test FAILed.

@peter-toth can you still add a test to the plan parser suite?

Thank you for ping me, @peter-toth . +1 for the above test case comments. And, this PR also looks good to me.

@peter-toth can you still add a test to the plan parser suite?

@hvanhovell sure, I added a test to PlanParserSuite

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11681/
Test PASSed.

Test build #106438 has started for PR 24842 at commit ddb0554.

Test build #106438 has finished for PR 24842 at commit ddb0554.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106438/
Test PASSed.

Test build #106456 has started for PR 24842 at commit 510eee1.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11700/
Test PASSed.

Test build #106456 has finished for PR 24842 at commit 510eee1.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106456/
Test FAILed.

Test build #106458 has started for PR 24842 at commit 0a00a03.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11701/
Test PASSed.

Test build #106458 has finished for PR 24842 at commit 0a00a03.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106458/
Test FAILed.

retest this please

Retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11764/
Test PASSed.

Test build #106523 has started for PR 24842 at commit 0a00a03.

Test build #106523 has finished for PR 24842 at commit 0a00a03.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106523/
Test FAILed.

Retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/11773/
Test PASSed.

Test build #106533 has started for PR 24842 at commit 0a00a03.

Test build #106533 has finished for PR 24842 at commit 0a00a03.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/106533/
Test PASSed.

Merged to master.
Thank you so much, @peter-toth , @liancheng, @hvanhovell , @gatorsmile !
@peter-toth . Please proceed to #24831 and #24860 .

Thanks @dongjoon-hyun @liancheng @gatorsmile @hvanhovell for the review.
I have prepared https://github.com/apache/spark/pull/24831 for review.
Unfortunately I can't work on https://github.com/apache/spark/pull/24860 next week but you can expect updates on it after that.

Thanks. No problem, @peter-toth !

Sorry for my late response. @peter-toth @dongjoon-hyun Could you submit a follow-up PR to improve it?

@gatorsmile @dongjoon-hyun I opened https://github.com/apache/spark/pull/24949 to add some new test cases. Please let me know if you want more cases.
Please note that I'm also working on https://github.com/apache/spark/pull/24860 and it will add many new tests that cover WITH column aliases.


❌ Software Removal | Posteo

Description
Posteo does not meet the Email Provider Criteria.
Why I am making the suggestion
Posteo does not support MTA-STS, TLS-RPT, and DMARC.
https://www.hardenize.com/report/posteo.net/1620012742
My connection with the software
No connection

[x] I will keep the issue up-to-date if something I have said changes or I remember a connection with the software.

You are using incorrect addresses to test them.


Loving reforge doesn't count towards "AD" in bonus stats in the armor section

Describe the bug
I have Wither Goggles (+49.5% AD and Loving reforge chestplate (+5% AD) but total AD in bonus section is 49.5%
Profile/Location
https://sky.shiiyu.moe/stats/Eonick/Tomato#Armor
Additional Info
Necrotic Recombobulated Wither Goggles and Loving Recombobulated Storm's Chestplate
Expected behavior
I don't know if additive or multiplicative but I expected either 54.5% or 52%
Screenshots

Desktop (please complete the following information):

OS: Windows 11
Browser: Chrome
Version: Latest (Version 101.0.4951.67 on Chrome)

Thanks for the reported, added this to the list in #1393


Album Exchange

i'll probably start keeping track of any ideas for the bot here
have the bot post an album exchange message once a week that people react to
have all reactions shuffled and placed in a list, with each user assigned to another 24hrs later
(i assume the bot has a way to reliably keep track of the time)

I don't think it should be automatic, I think we should be able to make any message where we ask for people to react
And then when whoever the moderator is thinks it's ready they make a command to the bot to read the users who reacted and does the album exchange shuffle thing


Exception section so that we can package things like black

As a response to https://github.com/conda-forge/staged-recipes/pull/12485#discussion_r478619040 we probably want to adjust this policy so that things we already package adhere to it.

We actually already "had that" as a verbal agreement. I never noticed that this was not in the document. Thanks @mariusvniekerk!

Also, say you have a software that has

only 1.*a* releases in the year 2020,
1.*.0 releases in 2021,
only 2.*a* releases onward.
And those 2.*a* releases are actually usable/decent.

presumably at that point the meaning of a differs.  This is chiefly a concern for calver / zerover packages

@mbargull can you rereview?

Is there going to be a vote called for this?

attn: @conda-forge/core

Is there going to be a vote called for this?

Given that this does not change the mode of suggestion vs prescription, I don't think we need a vote.
(But ppl. should still feel encouraged to review, nontheless!)

I think we should get rid of these "shoulds" as they are causing too much confusion. I read both of these shoulds as a "must", ie with an implicit "if".  "Once [IF] a non-prerelease version of such a package is available ..., [THEN] they lose this exception and should [MUST] publish future prereleases"

i'd view that as out of scope of this particular clarification.

i'd view that as out of scope of this particular clarification.

I very much agree.

... [MUST] ...

That change would definitely have to be voted on. Plus, you'd have to do some research and give reasons if such policy would be feasible with the existing diverse software release modes. I'm not saying I am against it, but I'd only support it given enough background information.
If you, however, want to propose another wording that you find less confusion, but which still carries the "should" meaning, we can do that, of course -- I'm definitely supportive on clarifications!

I guess it is fine as is, but I am in favor of not making the problem worse in this PR.  I really do read what is written here as a requirement, so it is disconcerting to me that other there is another, looser interpretation.

On the core dev call today the following was clarified:

Need to vote to adjust this CFEP as per these changes
@scopatz will submit a clarifying PR after this one is in.

@conda-forge/core
This PR falls under the Overall workflow and packaging policies policy, please vote and/or comment on this PR.
This PR needs 50% of core to vote yea to pass.
To vote please leave Approve (yea) or Request Changes (nay) reviews.
If you would like changes to the current language please leave a comment or push to this branch.
This vote will end on Wednesday September 9, 2020.

Total headcount of core atm is 22 according to https://github.com/conda-forge/conda-forge.github.io/blob/d9a90ce895ab87b9c646b0b02b5322818787a51c/src/core.csv
Thus this vote passes.

Thanks Marius! 😄


The style of select options is different from the main design

I'm currious that the design of dropdown is great, but select is under the default options design. Why not make the design just as dropdown.
Select

Dropdown

duplicate of #357

<select> list can not be styled using CSS


bodhi: automate obtaining API token with kerberos

TODO:

[x] Finish code
[x] Write new tests or update the old ones to cover new functionality.
[x] Get consensus with bodhi team where the login method is meant to live.
[x] Update doc-strings where appropriate.
[x] Document this for the team
[ ] Verify on stg this works correctly

I'm just sharing the code, it's far from finished.
Related to #1624
Bodhi tracking issue: https://github.com/fedora-infra/bodhi/issues/4588
RELEASE NOTES BEGIN
N/A
RELEASE NOTES END

Pretty good progress, last thing to solve:
sh-5.1$ kinit -k -t /secrets/fedora.keytab<EMAIL_ADDRESS>

sh-5.1$ python3
Python 3.10.5 (main, Jun  9 2022, 00:00:00) [GCC 11.3.1 20220421 (Red Hat 11.3.1-2)] on linux
Type "help", "copyright", "credits" or "license" for more information.

>>> from packit.utils.bodhi import OurBodhiClient

>>> bodhi = OurBodhiClient(fas_username="packit-stg", kerberos_realm="FEDORAPROJECT.ORG")

>>> bodhi.login_with_kerberos()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.10/site-packages/packit/utils/bodhi.py", line 123, in login_with_kerberos
    self.oidc.auth_callback(f"?{value}")
  File "/usr/lib/python3.10/site-packages/bodhi/client/oidcclient.py", line 176, in auth_callback
    self.tokens = self.client.fetch_token(
  File "/usr/lib/python3.10/site-packages/bodhi/client/oidcclient.py", line 203, in tokens
    self.storage.save("tokens", stored_tokens)
  File "/usr/lib/python3.10/site-packages/bodhi/client/oidcclient.py", line 320, in save
    os.makedirs(parent_dir)
  File "/usr/lib64/python3.10/os.py", line 225, in makedirs
    mkdir(name, mode)
OSError: [Errno 30] Read-only file system: '/home/packit/.config/bodhi'

just confirmed that d38e054 actually resolves the problem on stg

recheck


Revert "SWDEV-301667 - Increase default signal pool to 4096"

This reverts commit 94c7004df87cce4b6b0902b365a96905140312f0.
The reason for revert is: the original change introduces a bug in OpenCL when more than one command queue is created:
https://github.com/ROCm/ROCR-Runtime/issues/186
Also, in the orignal commit no reason for the change is provided.
In addition to that, 4096 is a particularly bad value (too large) as by default the kernel can only provide 4094 events for DGPUs.

Applied as part of
https://github.com/ROCm/clr/commit/c157bfb2022076959c521269b27f34996c1ee730


Use SCSS pre-processor for styling core templates

Using SCSS for development of core templates would make it really easy for managing colors of dark theme and also improve the children selector targeting for styles.

Currently in the dev branch


packaged ID-based metrics

(mostly distribution descriptions) for another PR into MS-CV.
Distributions described by Q1,2,3, mean, sigma, 1.5*IQR outliers - all separate as we discussed some reports can not/want not to report all values.
The proper accessions need to be replaced as soon as this PR is up in the queue.

Obsolete PR, as we have now transitioned to using the PSI-MS-CV. In case some of these terms need to be resurrected in the future, they can be retrieved from the git history.


definitions-parser has fatal crash for nearly-correct header

This crash brought down both CI and publishing, which should not happen. It might not be the fault of the parser.
// Type definitions for non-npm package css-typed-object-model-level-1 20180410.0
// Project: https://www.w3.org/TR/css-typed-om-1/
// Definitions: https://github.com/DefinitelyTyped/DefinitelyTyped
// Minimum TypeScript Version: 3.9

With the error
Error: At 2:50 : Expected one of
        /(, )|(,?\r?\n\/\/\s\s+)/
        /\r?\n\/\/ Definitions by: /

The corrected header is:
// Type definitions for non-npm package css-typed-object-model-level-1 20180410.0
// Project: https://www.w3.org/TR/css-typed-om-1/
// Definitions by: Nathan Shively-Sanders <https://github.com/sandersn>
// Definitions: https://github.com/DefinitelyTyped/DefinitelyTyped
// Minimum TypeScript Version: 3.9

See the breaking PR here: https://github.com/DefinitelyTyped/DefinitelyTyped/pull/45076
and the fix commit here: https://github.com/DefinitelyTyped/DefinitelyTyped/commit/f64a42dc3f414f381e83b674525ce38a9cd3cc6f

The header is no more; closing.


[prim_fsm] Initial round of sparse fsm updates

Partially addresses #9447

this looks good, but just like with prim_cnt, we need to add an assertion that points to the instance. I think I have an example in keymgr.

Ah yes you are right - forgot about that assertion.

I just noticed... in alert handler there is an issue with adding this assertion, since the alert handler does not send out any alerts.
i.e., the sparse FSMs trigger a local reaction that is custom to alert handler.
is there a way to bypass this, or should we just revert to non-standardized FSMs in this case?
@weicaiyang

I just noticed... in alert handler there is an issue with adding this assertion, since the alert handler does not send out any alerts. i.e., the sparse FSMs trigger a local reaction that is custom to alert handler. is there a way to bypass this, or should we just revert to non-standardized FSMs in this case? @weicaiyang

hmmmm.... maybe we should add a parameter bit to the module to inform it where it should not try to check for this assertion? What do you think?

I just noticed... in alert handler there is an issue with adding this assertion, since the alert handler does not send out any alerts. i.e., the sparse FSMs trigger a local reaction that is custom to alert handler. is there a way to bypass this, or should we just revert to non-standardized FSMs in this case? @weicaiyang

hmmmm.... maybe we should add a parameter bit to the module to inform it where it should not try to check for this assertion? What do you think?

Yes, that would be fine with me.
@weicaiyang @sriyerg WDYT?

I just noticed... in alert handler there is an issue with adding this assertion, since the alert handler does not send out any alerts. i.e., the sparse FSMs trigger a local reaction that is custom to alert handler. is there a way to bypass this, or should we just revert to non-standardized FSMs in this case? @weicaiyang

hmmmm.... maybe we should add a parameter bit to the module to inform it where it should not try to check for this assertion? What do you think?

Yes, that would be fine with me. @weicaiyang @sriyerg WDYT?

sounds a good idea to add a parameter bit to enable/disable the assertion. When it doesn't trigger an alert, it should lead to some other things like escalation, right? If so, may be worth manually adding another assertion to check that.

I just noticed... in alert handler there is an issue with adding this assertion, since the alert handler does not send out any alerts. i.e., the sparse FSMs trigger a local reaction that is custom to alert handler. is there a way to bypass this, or should we just revert to non-standardized FSMs in this case? @weicaiyang

hmmmm.... maybe we should add a parameter bit to the module to inform it where it should not try to check for this assertion? What do you think?

Yes, that would be fine with me. @weicaiyang @sriyerg WDYT?

sounds a good idea to add a parameter bit to enable/disable the assertion. When it doesn't trigger an alert, it should lead to some other things like escalation, right? If so, may be worth manually adding another assertion to check that.

It does lead to either a local alert or escalation yest - but given that this is very alert_handler specific, I don't think that it makes sense to make this assertion part of a standardized primitive.
Rather, we could rely on alert_handler DV to test this.

I just noticed... in alert handler there is an issue with adding this assertion, since the alert handler does not send out any alerts. i.e., the sparse FSMs trigger a local reaction that is custom to alert handler. is there a way to bypass this, or should we just revert to non-standardized FSMs in this case? @weicaiyang

hmmmm.... maybe we should add a parameter bit to the module to inform it where it should not try to check for this assertion? What do you think?

Yes, that would be fine with me. @weicaiyang @sriyerg WDYT?

sounds a good idea to add a parameter bit to enable/disable the assertion. When it doesn't trigger an alert, it should lead to some other things like escalation, right? If so, may be worth manually adding another assertion to check that.

It does lead to either a local alert or escalation yes - but given that this is very alert_handler specific, I don't think that it makes sense to make this assertion part of a standardized primitive. Rather, we could rely on alert_handler DV to test this.

yeah agreed. Alert handler by itself is such a special case. @msfschaffner do you want me to add the parameter disable? Or you're fine doing it as part of this PR?


Enable Kokoro MacOS result uploading

The Kokoro MacOS image was updated, so that Python libraries are correctly loaded from pip install x instead of the system default Python libraries. This fixes BQ result uploading.
The BQ upload flags will be moved from PRs to master before merging. The current Kokoro builds show that uploads are successful.

This PR only affects master jobs


[Dialog] [React] Controlled dialog unexpected behavior

Description
When using the Dialog component in controlled mode without an onClose listener and clicking outside the dialog, the visual state is updating even if the open property is still true...
The dialog should stay visible until the open property is updated
Link to Reproduction
https://codesandbox.io/s/heuristic-wood-fvvg2t?file=/src/App.tsx
Steps to reproduce
The dialog is opened by default, and the open property is displayed
When clicking outside the dialog, the dialog become hidden, but the isOpen state is still true
Using the render children prop, we can see that the context's isOpen is different from the open property value
Ark UI Version
0.13.1
Framework

[X] React
[ ] Solid
[ ] Vue

Browser
No response
Additional Information
No response

Thanks for taking your time to report this issue. I move this to @zag-js because this needs to be addressed here

Hi @segunadebayo,
I get this, and it makes perfect sense from a headless/vanillaJS point of view, but this leads to @ark-ui changing the React controlled paradigm of  the React version<EMAIL_ADDRESS>maybe this issue should go back to the chakra-ui/ark repository? 😅
I think that this paradigm shouldn't be remove from a UI library targeting React...
To allow @ark-ui to handle this, maybe it would be possible to provide more granular events to the zag flow: I'm not very familiar with finite state machines, but by quickly reading the Dialog code, it seem that calling for the OPEN or TOGGLE action is emitting the onClose handler...
Wouldn't it be a good idea to create several handlers: the onClose event would be triggered by the DOM event(s) and could be prevented the event.preventDefault() way, and an onClosed event, triggered by entering the "close" state...
When not prevented, the default behavior would remain the same as the current one...
For better understandability, maybe the event names should be more like onDOMClose or whatever...


Change to Plan: Implement CRUD for CRs

Now that API #1768 is done, refactor the work done in FE #788 to fully implement the new API.
There should be a component to:

Create a CR.
View current CRs.
Select a CR to be the active CR getting updated.
Delete a CR.

@michaelmarshall @sethmcknight
Here are my thoughts for implementing CRUD of CRs

In the project detail page, if the user clicks Change to Plan Request, it enters CR mode (instead of opening create modal)

In the CR mode, user could change project details (right now just project name). When the user changes the project name, it will call creating a CR request with Pending status. (at that time, Backend should know that is for changing project name request)

Remember that there can be multiple CRs open at once. So we need a way to view the open CRs and then select one of them in order to enter the CR-mode for the correct CR.

@CarsonF any thoughts on how CR-mode should be implemented?

Yeah we'll store the working change id in a react context. Probably at the app/root level, so navigation to another page doesn't kick you out unless you want to. We can display current change info in a banner in the Header component.
We'll need to update all queries and mutations to pull from this. Though it would be better if that part could be abstracted too.


[code-completion] Fix cache writer for non-decl kind

Thanks to Ben Barham for spotting this: sizeof(~static_cast<uint8_t>(...)) is 4, but we need to write a single byte here. Thankfully this code was probably not being hit in the current caching scheme, which only caches declarations.
Looking at the history, this code was broken by d8fbaa01eb10, which was fixing an MSVC warning in this code. Unfortunately I do not have a way to check if there is still a warning here or for what.  If we see it again, we can figure out the correct fix.

@swift-ci please smoke test

No space left on device

@swift-ci please smoke test Windows

@swift-ci please smoke test macOS


pkgsMusl.openssh: fix build

Description of changes
Build failed because MAXHOSTNAMELEN wasn't declared.

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Has this been reported upstream?

It's just curious that we need this, since it doesn't look like either Void or Alpine do…

It's just curious that we need this, since it doesn't look like either Void or Alpine do…

I just looked at Alpine, they also made a patch, but it is different from mine (https://git.alpinelinux.org/aports/tree/main/openssh/gss-serv.c.patch). I just tested it, it works too. Which one should I keep?

Ah, great digging!  Let's use Alpine's patch, since it's smaller.

Thanks! Great first PR.


Add support for capacity reservation

On hold for now as we need to understand the use-case better

Given oke api supports this, this would be helpful  be supported with tf module as well. when using GPUs/BMs, it would help associate the reserved capacity ensuring a cluster can be created with the requested spec


Update-DbaInstance Installing CU5 for SQL2019 fails, store needs update

Calling Update-DbaInstance fails, store needs an update
Verzeichnis: C:\tmp

Mode                LastWriteTime         Length Name

-a----       25.06.2020     14:28      632044864 SQLServer2019-KB4552255-x64.exe
Update-DbaInstance -ComputerName LOCALHOST -KB 4552255 -Restart -Path C:\tmp -Confirm:$false -verbose
...
AUSFÜHRLICH: [08:51:07][Resolve-DbaBuild] Looking for KB 4552255
AUSFÜHRLICH: [08:51:07][Resolve-DbaBuild] We have 0 builds in store for this Release
WARNUNG: [08:51:07][Resolve-DbaBuild] No info in store for this Release

The build file was updated on 26 June.
You can update the local index of your system, by running Get-DbaBuildReference -Update.


ignored files should have gray color and ignored icon

the following is my neo tree setup, I want that the ignored and dotfiles will be visible

it is working. but the ignored files like node_modules and .env files are not gray.
would be nice if the ignored files are gray with ignored icon

What you actually want is this:
      filtered_items = {
        visible = true,
        hide_dotfiles = true,
        hide_gitignored = true,
      },

That will show them because of visible = true, but still mark them because of hide_gitignored = true.


add(starters): V-dashboard and Vite + React starter

Name
Link

V-Dashboard
https://github.com/wobsoriano/v-dashboard

Vite + React + TailwindCSS Starter
https://github.com/wobsoriano/vite-react-tailwind-starter

Add starter using Vite + React + Tailwind and a dashboard template using Vue 3 and Tailwind.

[x] My item is in the right category
[x] My item is logically grouped below similar items
[x] My item's name and description respects the conventions of the list
[x] My item is awesome
[x] I have read and followed the contribution guidelines

Thanks!