Electron-Packer hangs

Electron Packer hangs... produces this output and then nothing happens:
PS D:\Code\GitHub\ElectronNET\ElectronNET.WebApp> dotnet electronize build osx
Build Electron Application...
Executing dotnet publish in this directory: D:\Code\GitHub\ElectronNET\ElectronNET.WebApp\obj\desktop
Build ASP.NET Core App for osx-x64...
Microsoft Windows [Version 10.0.15063]
(c) 2017 Microsoft Corporation. All rights reserved.
D:\Code\GitHub\ElectronNET\ElectronNET.WebApp>dotnet publish -r osx-x64 --output "D:\Code\GitHub\ElectronNET\ElectronNET
.WebApp\obj\desktop\bin"
Microsoft (R) Build Engine version <IP_ADDRESS>001 for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.
ElectronNET.WebApp -> D:\Code\GitHub\ElectronNET\ElectronNET.WebApp\bin\Debug\netcoreapp2.0\osx-x64\ElectronNET.WebApp
.dll
ElectronNET.WebApp -> D:\Code\GitHub\ElectronNET\ElectronNET.WebApp\obj\desktop\bin\
D:\Code\GitHub\ElectronNET\ElectronNET.WebApp>
Start npm install...
Microsoft Windows [Version 10.0.15063]
(c) 2017 Microsoft Corporation. All rights reserved.
D:\Code\GitHub\ElectronNET\ElectronNET.WebApp\obj\desktop>npm install
up to date in 1.289s
D:\Code\GitHub\ElectronNET\ElectronNET.WebApp\obj\desktop>
Start npm install electron-packager...
Microsoft Windows [Version 10.0.15063]
(c) 2017 Microsoft Corporation. All rights reserved.
D:\Code\GitHub\ElectronNET\ElectronNET.WebApp\obj\desktop>npm install electron-packager --global
C:\Users\Robert\AppData\Roaming\npm\electron-packager -> C:\Users\Robert\AppData\Roaming\npm\node_modules\electron-packa
ger\cli.js

<EMAIL_ADDRESS>updated 1 package in 6.459s

D:\Code\GitHub\ElectronNET\ElectronNET.WebApp\obj\desktop>
Build Electron Desktop Application...
Executing electron magic in this directory: D:\Code\GitHub\ElectronNET\ElectronNET.WebApp\bin\desktop
Package Electron App for Platform darwin...
I found some similar problems in the electron issue list:
https://github.com/electron-userland/electron-packager/issues?q=is%3Aissue+hangs+is%3Aclosed
I saw this "error" also on appveyor yesterday. I'm not sure why, but I can't build the osx package on my windows device.

had the same issue with the "Start npm install..." maybe we ignore some error code and get stuck?

https://github.com/electron-userland/electron-packager/issues/544

if the next commit doesnt solve the problem, maybe the /c switch is needed https://stackoverflow.com/questions/4291912/process-start-how-to-get-the-output


when i use cesium customshader in tileset,the tileset can not show all

when i use cesium customshader in tileset,the tileset can not show all.
The normal situation as shown below,

but when i  enable ModelExperimental ,the result become that shown bellow,

the code as following shows:
new CustomShader({
    fragmentShaderText : `void fragmentMain(FragmentInput fsInput, inout czm_modelMaterial material) {

    material.diffuse = vec3(1.0, 1.0, 0.0);

    material.alpha = 1.0;
}`

Hi @engineerhe,
I tried your Custom Shader on one of our tilesets and it seems to work fine. Is it possible to recreate this bug in a Sandcastle? If not, are you able to share the tileset you're using so that we can debug further?

@j9liu thanks !
the tileset' url is "http://<IP_ADDRESS>:9627/ygkj/tileset.json"

@engineerhe Thank you for providing the tileset, I tried this locally and I think I see the problem. The good news is this is already fixed on the main branch (to be released next week in the 1.96 release)!
We had a bug in ModelExperimental where .cmpt tiles were not cached correctly. The first inner content will load correctly (which I think is what's happening in your second screenshot), but all the remaining contents end up using information from the first, which can lead to unpredictable results (sometimes duplicating the model several times, other times not rendering at all)
Fortunately, this has already been fixed in https://github.com/CesiumGS/cesium/pull/10524
Here's a local Sandcastle that works on the main branch

One side note about transparency in CustomShader: By default, the shader renders opaque, and material.alpha is ignored. I wasn't sure if you intended to make the model translucent, but if you ever need that, remember to set isTranslucent: true, then material.alpha will adjust the translucency. See a translucent version of the above Sandcastle.

Since the underlying issue was already fixed, I'm closing this issue

@ptrgags It's wonderful.Thanks for your help


Problem with installing scikit-multilearn

May I know how to install scikit-multilearn?
I have tried doing pip install scikit-multilearn from command prompt and it managed to install successfully.
"Requirement already satisfied: scikit-learn in C:\Users\Name\appdata\local\continuum\anaconda3\envs\name_env\lib\site-packages"
But when I tried to run the command after activating my environment, "from skmultilearn.problem_transform import BinaryRelevance"  on spyder, I had the error:
ImportError: No module named 'skmultilearn'
I am using Python 3.6.3 :: Anaconda
Anybody can help me on this?

yes even I am facing the same issue.

This is not an issue of the scikit-multilearn library, this is something on your end.
You might want to check the version of pip you're using to install this library with. If you have Python 2 and 3 installed, there might be pip2 and pip. Also you can validate your installation with the following:
pip install scikit-multilearn
python -c "import skmultilearn"

If it runs through, you have problems with your jupyter notebook or your environment in general, but the package is installed correctly.
If you're using anaconda, I advise you to look at the documentation here.

still can't use this in last version. !!!

ModuleNotFoundError: No module named 'skmultilearn.problem_transformation'

i am using the package at this very moment, no issues here.
from skmultilearn.problem_transform import BinaryRelevance

yeah, I see the source code here make senses. But I make it work with from xx.lp import labelpowerset


"Error: Could not encode" when set mode "development"

I noticed I only dashboard and servers menu

I think this is because start my nodejs using "NODE_ENV=production node index.js" so I set mode code specifically in code
instrument(io, {
  auth: {
     ...
  },
  mode: "development"
});

But then server crash with error
Error: Could not encode
 at _encode (/home/express/node_modules/notepack.io/lib/encode.js:256:13)
 at _encode (/home/express/node_modules/notepack.io/lib/encode.js:162:19)
 at _encode (/home/express/node_modules/notepack.io/lib/encode.js:162:19)
 at _encode (/home/express/node_modules/notepack.io/lib/encode.js:246:17)
 at _encode (/home/express/node_modules/notepack.io/lib/encode.js:162:19)
 at Object.encode (/home/express/node_modules/notepack.io/lib/encode.js:261:41)
 at RedisAdapter.broadcast (/home/express/node_modules/socket.io-redis/dist/index.js:313:33)
 at BroadcastOperator.emit (/home/express/node_modules/socket.io/dist/broadcast-operator.js:109:22)
 at Namespace.emit (/home/express/node_modules/socket.io/dist/namespace.js:170:73)
 at Socket.<anonymous> (/home/express/node_modules/@socket.io/admin-ui/dist/index.js:284:36)

I searched this error and found someone else also hit this error
https://bestofvue.com/repo/socketio-socket-io-admin-ui-vuejs-web-sockets  (at "7. Error serialising sockets with session stores")

I saw this issue https://github.com/socketio/socket.io-admin-ui/issues/7  was closed at 0.1.2 but I am using 0.4.0 and still hit it.

+1 seeing it myself as well

Here is my stacktrace (similar to above)

I'm encountering the same issue

I think I have found the culprit, this should be fixed by https://github.com/socketio/socket.io-admin-ui/commit/6d58a755b4d692970d3f2066903a4d503f334f0a.
I'll publish a new version later today.

Released in 0.5.1. Thanks for your patience :+1:

@darrachequesne I have updated to 0.5.1 but I still get the same problem,
Before login to https://admin.socket.io/

But after log in, no socket/room stats

@darrachequesne I have verified 0.5.1 worked and I would like turn it into feature request: is it possible to make mode setting  a runtime setting, so I can turn it on and off when I need ?
I understand the development mode will be burden to https://admin.socket.io/ but we will host the Admin UI server ourselves so a runtime setting is more appropriate.

@darrachequesne 05.1 also fixed the issue I raised here https://github.com/darrachequesne/notepack/issues/29


Gnosis: start node sync

Overview
Start syncing a gnosis full node to prep for chain integration
References and additional details
https://docs.gnosischain.com/node

also reference current evm init.sh scripts and readiness.sh probes
check for snapshots to bootstrap node

Acceptance Criteria

gnosis full node syncing

Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

Done in #722


remove vanity plates

Because maybe, just maybe, a project owner would prefer his visors to visit him & not you.  ;)
Not sure what to do;  Could lowercase & remove space to auto-fill the GitHub/Twitter names, but that will only be accurate for a %.  Could add extra fields in the people field; I guess extra fields are usually ignored?

lol oops. thanks! I'll get this merged today!

thanks!

Thank you :)


awesome-package-manager

awesome-package-manager
https://github.com/damon-kwok/awesome-package-manager

Please read https://github.com/sindresorhus/awesome/blob/master/create-list.md and https://github.com/sindresorhus/awesome/blob/master/pull_request_template.md and ensure you follow them, then submit a PR.


Transparent emojis on Android when using a bold TextStyle

Emojis are transparent (again? #11108) when using a bold TextStyle.
Wasn't an issue before I updated from ~v0.2.1 to newest commit.
Works fine on iOS.
Steps to Reproduce
import 'package:flutter/material.dart';

void main() => runApp(new MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return new MaterialApp(
      title: 'Flutter Emoji missing Demo',
      theme: new ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: new Scaffold(
        appBar: new AppBar(
        ),
        body: new Center(
          child: new Column(
            mainAxisAlignment: MainAxisAlignment.center,
            children: <Widget>[
              new Text(
                'Emoji: 🍜 12345',
                style: new TextStyle(fontWeight: FontWeight.w900),
              ),
            ],
          ),
        ),
      ),
    );
  }
}

Flutter Doctor
[✓] Flutter (Channel master, v0.2.12-pre.33, on Mac OS X 10.13.4 17E199, locale de-DE)
• Flutter version 0.2.12-pre.33 at /Users/valauskasmodestas/Desktop/flutter
• Framework revision ea30c95dc9 (2 hours ago), 2018-04-13 17:02:14 -0700
• Engine revision 76cb311d9c
• Dart version 2.0.0-dev.47.0.flutter-f76dad0adc
[!] Android toolchain - develop for Android devices (Android SDK 26.0.2)
• Android SDK at /Users/valauskasmodestas/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-27, build-tools 26.0.2
• Java binary at: /Applications/Android Studio 3.0 Preview.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-884-b01)
! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses
[✓] iOS toolchain - develop for iOS devices (Xcode 9.3)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 9.3, Build version 9E145
• ios-deploy 1.9.2
• CocoaPods version 1.3.1
[✓] Android Studio
• Android Studio at /Applications/Android Studio 3.0 Preview.app/Contents
✗ Flutter plugin not installed; this adds Flutter specific functionality.
✗ Dart plugin not installed; this adds Dart specific functionality.
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-884-b01)
[✓] IntelliJ IDEA Ultimate Edition (version 2017.2)
• IntelliJ at /Applications/IntelliJ IDEA.app
• Flutter plugin version 21.2.2
• Dart plugin version 172.3317.48
[✓] Connected devices (2 available)
• LG D802  • LGD8028d47a899                       • android-arm • Android 5.0.2 (API 21)
• iPhone X • A1223CE9-152A-48EA-8C14-94C467CBF812 • ios         • iOS 11.3 (simulator)

This is a regression in Skia, introduced in:
https://github.com/google/skia/commit/5ddb30862509967eca24ba3831cc11ed5396eee7
Filed https://bugs.chromium.org/p/skia/issues/detail?id=7878

Has this issue gotten any further in being resolved?

It occur with v0.5.1.

@najeira can you please also check in beta 0.6.0?

I think this was fixed in Skia with https://skia-review.googlesource.com/c/skia/+/150467 , which rolled into Flutter earlier this week.

Note that I've tried building the engine and running the example above with all new all the things, and I may be doing something wrong, but I'm still reproducing the original issue. Unfortunately, I can't seem to capture an skp which actually demonstrates the issue at the moment. However, putting some print statements down inside Skia shows that Skia's FreeType believes that Noto Color Emoji does not have a path, but Flutter's FreeType does seem to think that the same Noto Color Emoji does have a path. I'm not entirely sure what the difference between these FreeTypes might be, since even if I rebase Flutter's FreeType up to FreeType's (real) current master it seems to still think Noto Color Emoji has an outline. At this point I don't think Skia is really doing anything incorrectly, but something about the way FreeType is being built in Flutter maybe?

It appears that the Flutter build of FreeType is trying to autohint the glyph and creating an empty path instead of just bailing on the autohinting. I'm not sure why Skia's build isn't doing this, but if autohinting is forced the same behavior can be seen. I think this is an issue in FreeType (if a glyph doesn't have any outline at all it can't really be autohinted), I'll see if there's any way to work around it.

SkScalerContext_FreeType has code that sets the FT_LOAD_NO_AUTOHINT flag if kForceAutohinting_Flag is off.  But it's gated behind an #ifdef SK_BUILD_FOR_ANDROID_FRAMEWORK:
https://github.com/google/skia/blob/master/src/ports/SkFontHost_FreeType.cpp#L880
The emoji in this example renders as expected if I remove the #ifdef.
Should this code be enabled in Flutter's build of Skia?

I'm so happy this issue is progressing.

It works on v1.0.0

@najeira thanks for confirming.
I'm going to close.
Feel free to add a comment to have it reopened if you still experience this issue or create a new one.


list of participants

Just waiting for the pdf version to be added before merging.

Sorry for late pdf version :(


package.json build issue for shared lib

Not sure if this is a problem on Firebase side or some nx config, but after being able to deploy functions with a shared library I'm now gettings this error message in the Firebase logs:
Build failed: npm ERR! cipm can only install packages when your package.json and package-lock.json or npm-shrinkwrap.json are in sync. Please update your lock file with npm install before continuing

Same here, it is working with the local emulator but when I deploy into Firebase then I get an error due to the shared lib import:
Detailed stack trace: Error: Cannot find module '@org/my-pkg'

In the generated dist folder, the dependency @org/my-pkg is present in the node_modules, so my assumption is that Firebase is just ignoring the content of the node_modules and trying to rebuild the dependencies so the shared lib is deleted.

In our case, we were able to fix by modifying the predeploy entry in firebase.[APP_NAME].json to:
npx nx build [APP_NAME] --with-deps && cd dist/apps/[APP_NAME] && npm install
The addition of npm install generates package-lock.json locally and sidesteps what appears to be a package.json and package-lock.json mismatch when generated on the GCP side.
This seems like an okay temporary workaround, though I'd like to know why GCP is generating a conflicting lock file during the build.

I tried, but cannot get it working, here is what I get in return:
i  functions: preparing codebase default for deployment

Error: Failed to load function definition from source: Failed to generate manifest from function source: Error: Cannot find module '@org/my-pkg'

As a dirty fix, I inlined the lib into the Firebase app. Would be interested in a reliable solution @simondotm.

@sbuys solution seems to be the only way this works now for projects that import buildable libraries - ie. it is necessary to use a v5+ of npm and run npm i in the functions dir prior to deploy otherwise I'm seeing Error: Error parsing triggers: Cannot find module '@myorg/nodelib3' type errors from the Firebase CLI.
This is unpleasant as it increases deployment time.
I dont understand why this is necessary - for deployments with no buildable library imports, firebase tools deploys fine.
Only when a local dependency is added with "@myorg/nodelib3": "file:libs/nodelib3" does this become an issue, and this is exactly what the firebase docs say is supported.
I'm using firebase tools version 11.16.1

I've tried running npm pack on the imported libraries so the dependency is "@myorg/nodelib3": <EMAIL_ADDRESS>instead, but same issue

I'm wondering if the firebase CLI is somehow delegating to the root workspace node_modules and package.lock and doesn't find @myorg/nodelib3 in there (which is correct), so it bails

This is the code in firebase-tools https://github.com/firebase/firebase-tools/blob/54a3f85534f4d42129c1e49f74eea78e2872ef97/src/deploy/functions/runtimes/node/triggerParser.js
I'm wondering if its time to move to es modules instead of commonjs. Or just bundle functions with rollup to side step this whole issue now.

 My issue was caused by something unrelated.

Maybe this helps someone (probably me in the future!):
I'm using the suggested workaround from here and here in combination with this of running this in the predeploy scripts like this:
npx nx build appname --with-deps && cd dist/apps/appname && npm install --package-lock-only
After the build command npx nx build appname --with-deps, there's now a dist/apps/appname folder with a package.json file and a node_modules/@myorg/shared package.
Running npm install [--package-lock-only] will see the folder in node_modules, so the resulting package-lock.json will contain this section:
    "node_modules/@myorg/shared": {
      "version": "0.0.0"
    },

Now, when I delete node_modules/ and run npm ci (which is what happens during cloud function deployment!), it will fail because the local dependency couldn't be resolved.
Workaround:
If I do this instead:

npx nx build appname --with-deps
cd dist/apps/appname
rm -rf node_modules or npx rmdir-cli node_modules/ <-- please tell me if theres a better cross platform folder delete!
npm install

I end up with this portion in the package-lock.json file:
    "node_modules/@myorg/shared": {
      "version": "0.0.0",
      "resolved": "file:libs/shared"
    },

Everything else in the file is identical.
Now npm ci works.
We went back to using the default behavior (ie without the suggested workaround of running npm install during predeploy), but perhaps this is useful to someone who requires a package-lock.json or runs into other issues.
Versions:

package-lock.json lockfileVersion: 3
npm: 9.3.0
nx-firebase: 0.3.4
firebase-tools: 11.17.0 <-- yes, there's 11.19.0 available, but that one is a bit broken right now

Thanks for the detailed info @ciriousjoker - I'd like to get to the root of this issue as I am able to deploy fine without needing to generate a package-lock file, but there are a ton of variables - nx version, plugin version, firebase cli versions, node versions etc.

Deployment does work fine without a package lock json. I just originally did the npm install workaround because of another issue and npm install breaks deployment because of the wrong package lock.
Now I fixed the other issue and deployment without any additional npm install works fine. Let's hope it stays that way.

closing this, can re-open if anyone else sees the issue again.


Update Nonce.java

Final variables for silly micro optimization.

I am closing this in favor of a single PR, see https://github.com/braintree/braintree_java/pull/47


Update build-deps.sh to use scalalib 5.7

scalalib had a version bump from 5.6 to 5.7.
Make updates to build-deps.sh to use new version.
https://github.com/ornicar/scalalib

it uses the latest version


Add CreateCSharpParam TypeFactory Implementation

As part of Implement Methods as Collections or Sets, there is a need to convert an input parameter to an output parameter type while constructing a method representation of an input operation. A type factory method should be implemented in the SCM plugin that will handle this conversion so the parameters can be properly constructed while building a method's signature.

With this PR, we've added the initial implementation of the type factory. There is still follow up work to properly convert an input parameter to the correct Parameter output type. That work should be handled as part of the remaining work of this issue.


Add test matrix for windows ctrd versions

Adds test matrix for the current windows ctrd versions
follow up from https://github.com/containerd/nerdctl/pull/533#issuecomment-982772530

the 1.5 failure is the flake #528: msg="error unpacking image: lease \"241796900-Uv38\": already exists"


Document alternative solution

This enables Polylang thus all posts can be managed, e.g. by wp post list
~/.wp-cli/commands/polylang-enable/command.php
<?php

if ( ! defined( 'PLL_ADMIN' ) ) define( 'PLL_ADMIN', true );
if ( ! defined( 'PLL_SETTINGS' ) ) define( 'PLL_SETTINGS', true );

~/.wp-cli/config.yml
require:
  - commands/polylang-enable/command.php

Please consider mentioning it. It took me 20 minutes to find out.

See https://github.com/diggy/polylang-cli/blob/master/src/Commands/BaseCommand.php#L12
Also note https://github.com/diggy/polylang-cli/blob/master/src/Commands/Post.php#L225
Why would mentioning this in the polylang-cli documentation be useful or relevant?

Project usually have an "Alternatives" section in the readme.

@szepeviktor I've been thinking about this, and I'm not convinced this is a viable, alternative solution. Defining the constants is simply not enough; wp post list might work, but for other pll commands I was hoping simply passing args and assoc args to the core command counterparts would suffice, but it does not, take a look at these commands I just implemented:

https://github.com/diggy/polylang-cli/blob/master/src/Commands/Term.php#L344
https://github.com/diggy/polylang-cli/blob/master/src/Commands/Post.php#L192

Ok-Ok.
Finding those constants to make wp post * work took me a long time. I thought it worth sharing.

Instead of alternative solution I would call it Viktor's hack.


Added conditional publisher to run rcov only for JOB_RUBY_VERSION 2.1.5

@mknapik @rmldsky please verify (the additional template content is taken from : http://jenkins.nielsensocial.u2i.eu/job/(master) socialguide 2/# )

You need to bump up a version of cookbook in metadata.rb as well.


Add license file

The MIT license requires that the text of the license accompany the source code.  This pull request adds an appropriate license file to this repo.

Thanks for this! If you rename it LICENSE from LICENSE-MIT it will make it slightly more consumable(as I have no plans to dual license it) and I'll merge it in.

File name is changed.


Update ioc_changes1.rst

Running ec deploy bl01t-ea-ioc-02 2023.11.2 results in No such command 'deploy'

Also regarding the comment "# deploy the new version of the IOC to the local docker / podman instance"... Is that not the intention of the command ec ioc deploy-local? I believe in this case it just happens to be on the local machine

I quite like the namespaces because it feels more explicit - but what you have suggested also makes sense. I have no strong opinion at this stage


Add module to parse Marta response and return a match if found.

Takes in a location name and a list of marta arrival times, The next arrival of the given location will be returned or :error if no match is found.

I'm taking this (can't assign).

in #20

fixed in #20


update iTFT purchase agreement

The asset version of the iTFT purchase agreement seems out of date.
See https://docs.google.com/document/d/162H9QDswivRNwDhxIDmvodzXNsrNtmR9fuGZKshEE_w/edit#
for the most recent version, mentioning "conditionality". Let's get that in there!

Nevermind, I was on the wrong branch >________<


lost code autenticator

please help me i lose my code authenticator my account lakebtc is blocked

Please contact customer support via<EMAIL_ADDRESS>with your IP address.


Child terms for apoptosis

I will be working on creating child terms for
apoptosis, in order to be able to annotate some C.
elegans genes. Please let me know if you have any
suggestions, comments, requests<EMAIL_ADDRESS>Reported by: ranjanak
Original Ticket: "geneontology/ontology-requests/3255":https://sourceforge.net/p/geneontology/ontology-requests/3255

thanks www.movied.org


Fix [

test and [ should be identical and both WinOS compatible.
This updates the code, duplicating the working test to [ and removes [ from the build exclusion.
A v fmt -w . run was done to fix the failing CI format style test.

I'll split this into fixing the current repo issues with format and deprecation, then open this on top of that fix.


Fix prompt assembly for llama

UPDATE:  I found many issues with the llama templating after fixing the omission of the human message.  I found the following issues:

EOS/BOS being applied incorrectly in between turns
System Message was incorrectly formatted with a [INST] where it wasn't supposed to be
Proper handling of multi-turn
etc.

I added lots of tests for the following cases:

Multi turn
Single turn
Multi turn without system message
Single turn without system message

For the tests,  I put a redundant decoded version of the string  so that you can see the prompt which will help with code review and understanding

@winglian ok this is ready for your review

@winglian I fixed quite a few issues that I found.   We all mostly use ChatML for conversation by now (openorca, openhermes, dolphin), so it would be amazing to have more testing for other formats as well. The prompt formatting is something that I have seen issues with before, so it's great to see that it's being handled now!


aggregations: per-aggregation bookmark

Currently, the aggregation bookmarks are unique for a specific event type (e.g. the document type file-download-bookmark is stored in the stats-file-download). This is not handy since there might be multiple aggregations configurations for a single type of event (e.g. one aggregation on the bucket level, and one on the file level for downloads).
This creates the following scenario:
(Assuming we have two aggregations, aggr-A and aggr-B, on the event type file-download, no bookmarks and N events from 2018-01-01 to date 2018-06-31)

aggr-A runs, starting from the oldest event, on 2018-01-01
aggr-A finishes, processing all N events and setting the bookmark for the latest date that was processed, i.e. 2018-06-31, with document type file-download-bookmark under the aggregation's alias stats-file-download.
aggr-B runs, gets the file-download-bookmark doc type from the alias stats-file-download, which was previously set to 2018-06-31. There are no events after this date, so no aggregations are produced for aggr-B

The solution is to create bookmark document types based on each aggregation name, e.g. aggr-B-bookmark.

Closed via #68


Add language translations

Some folks have noted it would be beneficial to have this repository translated to other languages. One contributor has already volunteered to translate the repository into Russian (woohoo!).
If anyone has the time and drive to help out with other languages, please reply below with the proposed language and I'll give you the "go-ahead"... just to make sure two people don't create translations in the same language. Thank you!

I'm interested in translating this to Portuguese-BR :)

@acamilass Awesome! Please go ahead and submit a pull request when you're finished (or a draft pull request as you work).
Thank you!

Hi! Any help with Russian language needed? :)

@nas5w I'm interested in translating this to chinese

@simplefeel Yes, please do! Thank you!

@nas5w Hi! I'm interested in translating this to Vietnamese

@phanlyhuynh wow, that's awesome! Please do!

@nas5w Spanish I can!

@Zardosh yes, I suspect they have given up. Please do submit!

I know you already claimed this, but i would like to help you, if its ok.

@rossi-p Oh, I have already created a PR with it translated, but I'd appreciate if you could review it and suggest any changes!

@nas5w , I am interested in helping with Malay 🇲🇾 translation. Malay is similar to the Indonesian 🇮🇩 language

@asyraf-labs yes please! Thank you!

Thanks @nas5w , I'm working on it 👨🏻💻💪

@nas5w
Should have known this project earlier! This is a great roadmap to front-end dev.
I am interested in helping with Korean🇰🇷 translation. 😃

I'm interested in helping with Indonesian ID Translation

@nas5w  I am eager to translate this into Uzbek language :D

@shoxruxpro yes please!!

Hi! I am keen helping with German 😃

@Daniel-Singer perfect, thank you!

@nas5w  I'm interested in translating this to japanese :)

@miily8310s I would really appreciate that, thank you!

I am interested in Bangla translation

@mirajus-salehin yes please!

@nas5w I'm interested in translating to Spanish :)

Can I help you to translate in spanish too :+1:

@LeonAvancini yes please! That was a while ago so I think the Spanish translation is all yours if you want it.

@nas5w Excelent :).
This will be my first collaboration

Hi, I would like to try my hand at translating it in French ^^


fix(results-table): pass 0 value to formatValue instead string

When result table contains 0 value, it breaks because empty string is passed to formatValue function in this case.
This causes failure to conversion to string (and anyway is wrong).

@k0machi can you also take a look at this one - it's a bug preventing showing results in perf-simple-query tests

@k0machi can you also take a look at this one - it's a bug preventing showing results in perf-simple-query tests

You have a merge conflict here

@k0machi can you also take a look at this one - it's a bug preventing showing results in perf-simple-query tests

You have a merge conflict here

fixed


[Feature Request] Also store binaries using Github Releases

redscientist.com (the website that hosts the only official binaries) is blocked by Malwarebytes, and adding it to the exceptions list appears to have done nothing.
My idea is to make the Redscientist website the main way to download RTC, but also host RTC binaries using Github Releases.

Fixed by disabling Malwarebytes to install RTC, then reenabling it.


explosion-effect config description inconsistency

Hi, I want to report inconsistency in the documentation vs actual server value
https://purpurmc.org/docs/Configuration/explosion-effect_2
Key: end-crystal.*.explosion-effect (and possibly others like respawn anchor or bed in the nether)
It is said in the documentation

explosion-effect
default: DESTROY
description: The type of explosion effect. Available values are NONE, BREAK, and DESTROY.

But when I try to set it to "BREAK" it is not accepted by the server:
[09:05:13 ERROR]: Unknown value for blocks.end-crystal.baseless.explosion-effect! Using default of BLOCK
[09:05:13 ERROR]: Unknown value for blocks.end-crystal.base.explosion-effect! Using default of BLOCK

Also in the documentation, the "BLOCK" option is nowhere to be seen in any config.

This is actually a bug in Purpur that was pointed out by Billy. I've gone ahead and created your discussion post into an issue here: https://github.com/PurpurMC/Purpur/issues/1340 Sorry for the confusion.

no worries, thank you @granny


Check listpicker selection and selection intex when there are no elements

Ditto for listview
Also see  thread at
https://groups.google.com/forum/#!topic/mitappinventortest/Hb3onC-dmjg

PR #642

@halatmit I take it that by merging #642 this can be closed. Please reopen if that's not the case.


📎 Implement stylelint/no-irregular-whitespace

Description
Description
Implement no-irregular-whitespace

[!IMPORTANT]

Please skip implementing options for now since we will evaluate users actually want them later.
Please ignore handling extended CSS language cases such as sass and less.
Please skip custom function since we haven't syntax model yet.

Want to contribute? Lets you know you are interested! We will assign you to the issue to prevent several people to work on the same issue. Don't worry, we can unassign you later if you are no longer interested in the issue! Read our contributing guide and analyzer contributing guide.

Might I take a shot at this? 🙂

@DerTimonius
A person is already implementing the same rule for JavaScript https://github.com/biomejs/biome/issues/2511#issuecomment-2202751073
We can definitely reuse the same logic across grammars

@DerTimonius
https://github.com/biomejs/biome/pull/3333 has been merged :)
Can you please take a look?

will do!

🚀


Create internation3Lcommunicationcorp

mablbank
Problem Statement
What is the problem you're trying to solve?
Related Issue
Fixes #...
Proposed Changes
How do you like to solve the issue and why?
Checklist

[ ] I have read the contribution guidelines
[x] All commits are signed with git commit --signoff
[ ] My changes have reasonable test coverage
[ ] All tests pass with make test
[ ] I ensured my PR is ready for review with make reviewable

in


can't compare offset-naive and offset-aware datetimes

I tried to import a json file that was just exported.
settings.py contains USE_TZ=True
Line number: 1 - can't compare offset-naive and offset-aware datetimes
Traceback (most recent call last):
File "/apps/local/lib/python2.7/site-packages/import_export/resources.py", line 361, in import_data
if self.skip_row(instance, original):
File "/apps/local/lib/python2.7/site-packages/import_export/resources.py", line 266, in skip_row
if field.get_value(instance) != field.get_value(original):
TypeError: can't compare offset-naive and offset-aware datetimes
Same with CSV

@manelclos can you add failing tests?

@bmihelac sure, will do. I'll look further into the issue too.

Tests are ok for me. Also, I have latest code working on another installation. USE_TZ seems not to be the problem. It also works on PostgreSQL. It may have something to do with the previous, original values.
Field defined as:
models.DateTimeField(auto_now_add=True)

When exporting:
ORIGINAL:  2014-02-13 13:38:04.813096+00:00 <type 'datetime.datetime'>
EXPORT:  2014-02-13 13:38:04 <type 'str'>

When importing:
INSTANCE:  2014-02-13 13:38:04 <type 'datetime.datetime'>
ORIGINAL:  2014-02-13 13:38:04.813096+00:00 <type 'datetime.datetime'>

Hi @bmihelac here is what I think we should do:
If the import works ok, just go ahead, this happens for me in different installations, and is also backwards compatible.
Then, if we get the TypeError exception, try to make the datetime timezone aware and compare again:
>>> value = '2014-02-13 10:37:38'
>>> format = "%Y-%m-%d %H:%M:%S"
>>> dt1 = datetime.strptime(value, format)
>>> dt1
datetime.datetime(2014, 2, 13, 10, 37, 38)
>>> from django.utils import timezone
>>> dt2 = timezone.make_aware(data, timezone.get_current_timezone())
>>> dt1.strftime(format)
'2014-02-13 10:37:38'
>>> dt2.strftime(format)
'2014-02-13 10:37:38'

https://docs.djangoproject.com/en/dev/topics/i18n/timezones/#troubleshooting
Then, I'll have only the microseconds problem, but that should be solved by adding a DateTime widget with the proper format, right?

@manelclos what do you think about checking  USE_TZ setting instead of catching exception.
We would basically replace this line with:
dt = datetime.strptime(value, format)
if settings.USE_TZ:
    dt = timezone.make_aware(dt, timezone.get_default_timezone())
return dt

@bmihelac Yes, makes more sense, though the impact is also bigger. I'll try the suggested change. Thanks!

Unfortunately strptime does not allow %z

@bmihelac can you please review?


catalog details query parameters

cortex catalog details currently shows these options:
$ cortex catalog details -h
usage: cortex CLI catalog details [-h] [-i] [-g] -t

options:
  -h, --help            show this help message and exit
  -i , --includeHierarchyFields
                        List of sub fields to include for hierarchies. Only supports 'groups'
  -g , --groups         Filter based on groups, which correspond to the x-cortex-groups field in the Catalog Descriptor. Accepts a comma-delimited list of groups
  -t , --tag            The entity tag (x-cortex-tag) that identifies the entity.

However,  https://docs.cortex.io/docs/api/get-entity-details indicates that only these options are supported:

hierarchyDepth
includeHierarchyFields

So, need to update this subcommand to remove:

groups

and add:

hierarchyDepth

Also noticed that includeHierarchyFields is currently a boolean.  It should accept a string.


0.8 custom id generate invalid code

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

in sqlAll

the id type is
type ID string

Expected Behavior 🤔
working
Steps to Reproduce 🕹

WIP
Your Environment 🌎

Tech
Version

Go
1.16.?

Ent
0.8.?

Database
-

Driver
-

BTW, code in 0.7

Thanks for reporting this @wenerme. This happens also in v0.7.0, right?

@a8m no, v0.7 is working, rollbacked to v0.7, the &sql.NullString{} syntax is ok

Hey @wenerme, the issue was fixed and works fine now in master (see example in https://github.com/ent/ent/pull/1470).
Thanks for reporting this issue 🙏


Bot does not create its own admin role.

Normally most bots create themselves an admin role but for whatever reason PossumBot does not which limits its "hacking" abilities such as not being able to use !myakish

@MonkoTubeYT Is the role being created at all?

Nope.

@MonkoTubeYT Weird. It creates a admin role for me. Make sure the admin box is ticked when you invite it.

I'm pretty sure I did that but I'll try again. Also, I know a huge server that was overthrown because of your bot. https://disboard.org/server/408339131882536971 be careful with this and don't get in serious trouble.

I tried again and it gave itself a role. I must've not ticked the administrator box while inviting it every other time.

@MonkoTubeYT Alright. I made this bot just for fun and i never used it on any server except mine. Also how do you know it was my bot specifically?

The bot literally said "Playing with knives" on its game activity, they didn't even bother changing it in config.

@MonkoTubeYT Wait do you have a working link to the server?

@MonkoTubeYT Anyways i think that server admins with that much people in their server should be more careful. Thanks for the heads up.

Yeah, I don't have the invite but the guy who used your bot posted a video about it and claimed he made the bot. https://www.youtube.com/watch?v=pNNpLxrildY

Found the invite link, I was banned so I can't join back...
https://discord.gg/5aXeAxh

@MonkoTubeYT Wow skid level 100. Thank you.

Lmao

@MonkoTubeYT Pretty funny situation though.


Add names and links to NuGet packages in the README

Make it easier for anyone searching for a list of the packages and their name/links

I agree with sorting the table by package name.
Also: PInvoke.AdvApi32 is missing.

BTW, thanks for doing this. I've wondered how to get the NuGet package badge on the page the way other projects do (and I've obviously never looked at the markdown for those projects, since it seems quite simple).

I've wondered how to get the NuGet package badge on the page the way other projects do (and I've obviously never looked at the markdown for those projects, since it seems quite simple).
Projects started with manually done shields but everyone seem to migrate to http://shields.io/ now

Another idea: as it will be easiest to simply ship all nuget packages together any time I ship anything, I'm inclined to say that displaying the nuget badges/shields with version number may be less useful than some stat that varies among those packages. Perhaps download count?

I've shown the download count, not so high for now but i'll grow :)


Suggestion for improvement of cache folder

Hi,
tried Gitlist the first time today and the only "annoyance" I came across was a hardcoded cache directory path.
What do you think about the following, which should ensure that old configuration files won't break:

change the INI value from boolean to boolean or string (that is: true, false or the directory path)
implement logic so that the path is used according to the config

If you feel this is viable, I'd be willing to implement it and send a pull request. Any comments would be welcome.
Side-note: unless app.cache is meant to be the defining factor for caching, the code that checks whether the cache directory is writable should be  after the reading of config.ini.
// Oliver

Why is the cache path inside the app a problem? It's gitignored and if you really want to change the path then why not symlink?

@garygreen I'm not sure what you're trying to say. I didn't insinuate that Git (and therefore .gitignore) was or should be involved.
It's a natural thing to want to separate program from program data and it has been proven as a sensible solution many times over. One good reason is backups. You don't want a cache to clutter your backups and you don't want every web application to store their cache inside the webroot either - well, if you're an admin that is.
Symlinks do not always work, only bind-mounts provide the required transparency. If I had to do a backup, what would I do? Exclude all symlinks? What if the author then decides to put a symlink into the source itself? Essentially the current scheme complicates things for any use case on a real server.
It's fine on a developer machine, but on a server you want to be able to assign ACLs and permissions in a certain way and make sure that setting them recursively for a web application opens up an attack surface.

@assarbad Fair enough. Not much is actually being cached at the moment (actually, I think nothing is since it's been removed?). It wouldn't be too difficult adding a config option for the cache directory anyway, maybe a PR?

@garygreen what's PR in this context?

Pull request. I think the main dev has been inactive for some time so unlikely it'll get implemented, in any case a PR would be a good start. It would enable yourself and others to see the implementation and at least maintain their own fork and cherry pick it.

@garygreen of course, didn't think of pull request.
Fair point, I'll see when I can squeeze in some time regarding this. Thanks.


Fixes to texture handle changes

Ugh, I did it wrong. The find_file methods that did the name to IC handle lookups did more than just that -- that's also where the ImageInputs were opened (if not yet opened), duplicates found, 'used' flags updated, errors reported, etc.
It wasn't symptomatic until we started messing around with invalidating the files in the cache, which doesn't invalidate the handles themselves (and shouldn't), but at that point the files would be closed, and the re-opening wouldn't happen properly without the call to find_file, which of course wouldn't happen at all since the handle was already retrieved.
The solution is to split into separate "find" (just convert name to file cache handle, creating if necessary), and "verify" (open and other housekeeping and error checking). It's mostly just splitting the existing find_file in half, then cleaning up the many many consequent changes that make use of that properly. The name-based functions just call find, and then pass to the handle-based functions. The handle-based functions need to call "verify" on the handle it has, to ensure that the handle is ready for use. Once this split was done, it also became clear that a thread_info pointer needed to be passed to some routines where we didn't pass it before. There shouldn't be any performance hit from this, it's just re-apportioning the same amount of work across two functions.
Pulling on this piece of yarn unraveled much of the sweater, but I think I've got it put back together correctly now.

Updated the review with further changes:
More IC/TS perthread fixes
App-owned Perthead pointers (the ones preallocated by create_perthread_info) had no mechanism for clearing their per-thread microcaches of tile references even after they were marked as "purge." That's because the actual purging was previously done in the get_perthread_info function that was used to retrieve the true thread-specific-pointers. This could lead to inconsistent texture results for different threads, if an invalidate() for that file had happened at just the right time.
The fix the strategy is that now get_perthread_info takes an optional Perthread* argument, which if supplied is used, and only if NULL is passed does it retrieve and/or allocate a TSP. Then in either case, it does the other housekeeping (including purging, when requested) before returning the pointer (either the supplied one passed in, or the created/retrieved one).


Removing unused imports

After code analysis of the project, some unused imports were found.
Starting a new branch to reorganize them,
and beginning with unused java.util.Arrays import in AccountList class.

Thanks! But please don't start fixing minor issues all over the code base now. Like I mentioned on the mailing list, this has the potential to make it much harder to merge the long-living pgp_mime_preparation branch later.
I think you would learn a lot more if you started writing some unit tests for K-9 Mail.
PS: Unused imports can be found and fixed automatically. It's something I will do myself as soon as the other branch is merged.


Server sometimes throws "Http: connection closed before message completed"

Error:
error: Uncaught (in promise) Http: connection closed before message completed
    at deno:core/01_core.js:106:46
    at unwrapOpResult (deno:core/01_core.js:126:13)
    at async Object.respondWith (deno:ext/http/01_http.js:202:27)

Here is some discussion: https://github.com/denoland/deno/issues/11595

The aforementioned discussion mentions that one must correctly await promises. Testing with using await event.respondWith seems promising.

Still occurs:
error: Uncaught (in promise) Http: connection closed before message completed
          await event.respondWith(response);
          ^
    at async Object.respondWith (deno:ext/http/01_http.js:202:27)
    at async https://deno.land/x/edcb@v1.0.0-alpha.2/utils/actions/listen.ts:29:11


Fix the printing of RawPlutusData

Some recent change in the codebase led to pretty printing of Raw Plutus Data to invoke a pretty printer. This is problematic for two reasons:

OpShin mimiced the behaviour of PyCardano to print RawPlutusData in the most precise way possible
The pretty printer re-orders dictionaries (i.e. sorts them). This may lead to the representation of the dictionary not being faithful to the original representation anymore (specifically, could lead to a different CBOR hex)

Another way to tackle this might be to change the behaviour of the pretty printer (inherited from CBORSerializable) to not sort entries in dictionaries anymore (as this could also break CBOR matching)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (df5ba28) 84.51% compared to head (08b3ce3) 84.51%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #301   +/-   ##
=======================================
  Coverage   84.51%   84.51%
=======================================
  Files          27       27
  Lines        3093     3093
  Branches      758      758
=======================================
  Hits         2614     2614
  Misses        358      358
  Partials      121      121

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

You can also try removing this decorator from the class definition:
# @dataclass(repr=False) <------ this line forces it to the repr from parent class, which uses pretty print.
class RawPlutusData(CBORSerializable):


Implement commands for API resource management

It'd be better to have support for API resource management, including,

create API
update API scopes
delete API
list API scopes

Hi @shashimalcse , I'd like to work on this issue. Could u pls assign this to me?

@pavinduLakshan i have assigned you. Thanks

list command : https://github.com/shashimalcse/asgardeo-cli/pull/14

create command : https://github.com/shashimalcse/asgardeo-cli/pull/22

delete command : https://github.com/shashimalcse/asgardeo-cli/pull/23


Update version

Coverage remained the same at 91.853% when pulling dfdb10ac95e7d545d53598ffb899abc8423e4999 on jansav:update-package into 4b7c79cd0ef750789e84ffbdb380f2bb47f7e884 on Houston-Inc:v1.0.0.


Tooltip - best way to grab element to use as content

What's the best way to specify an element in the DOM that I want to use as tooltip content? The way I'm currently doing it, I add it to the jBox window via the onOpen() event callback and then calling the setContent() method. Doing it this way, I have to clone my DOM element otherwise it doesn't exist the next time the tooltip is rendered.
I'm wondering if there's a better way to do it.
Here's my code... this renders a tooltip to the right of a feature description link.

    feature 1

This is the best feature ever! Click here for more info

Thats how I usually do it too.
Depending on the use case, I sometimes append the element to the jBox content area rather than cloning and then I hide all elements exept the current one in the onOpen method.

Interesting... I can see how that would work. By the wayt, where is the best place to ask questions?


build: Simplify zephyr interface header dependency on generated files

Simplify zephyr interface header dependencies on the generated files
offsets.h and version.h
All targets that depend on zephyr header files must depend on both
zephyr_interface and zephyr_generated_headers.
Missing zephyr_generated_headers leads to a rarely triggered
rare race-condititon for source files for these generated header files.
Example:
zephyr/include/zephyr/linker/linker-defs.h:26:10:
| fatal error: offsets.h: No such file or directory
|     26 | #include <offsets.h>
|        |          ^~~~~~~~~~~
|  compilation terminated.
Depending on zephyr_generated_headers target is harder than depending
on zephyr_interface because zephyr_generated_headers is a custom command
and must have been declared before adding the dependency.
Since zephyr_interface is an interface library it does not have this
order requirement.
Simplify this by adding the generated headers offsets.h and version.h
to the zephyr_interface target instead and remove the
zephyr_generated_interface target that app has no dependency on.

I had misunderstood the original issue, it is not that the 'app' didn't have the dependency.
The target that was missing the dependency was an external built project, and it could not add the dependency on the generated headers because it was declared before the custom target.
This makes fixing the dependency harder.

@tejlmand The interface headers include offsets.h, so all users of the zephyr headers depend on offsets.h
This includes all libraries built, which must depend on both zephyr_interface and zephyr_generated_headers.
Because zephyr_generated_headers is a custom target instead of an interface library like zephyr_interface it has different properties, which ExternalProjectAdd needs to work around in order to get the dependency right.
ExternalProject_Add(my_external_project
    ...
    DEPENDS          zephyr_interface zephyr_generated_headers
)

Fails with the following error:
get_property could not find TARGET zephyr_generated_headers.  Perhaps it
  has not yet been created.

So here we are dependent on the target already having been created.
The alternative solution is this:
ExternalProject_Add(my_external_project
    ...
    DEPENDS          zephyr_interface
)
add_dependencies(my_external_project zephyr_generated_headers)

I have already added this as a workaround downstream.
However with the change in this PR there is no longer a requirement to depend on zephyr_generated_headers.
ExternalProject is not a special use case, this is part of zephyrs example on how to solve the use case of an externally built library.
See  samples/application_development/external_lib, here if the mylib in that sample was depending on zephyr headers it would have the same problem that we are facing.

@tejlmand The interface headers include offsets.h, so all users of the zephyr headers depend on offsets.h

that's not what I see when searching for offsets.h in the generated dep files for a simple project:
$ grep offsets.h (find . -name '*.d')
./zephyr/arch/arch/arm/core/aarch32/cortex_m/CMakeFiles/arch__arm__core__aarch32__cortex_m.dir/scb.c.obj.d: /projects/github/ncs/zephyr/build/zephyr/include/generated/offsets.h \
./zephyr/arch/arch/arm/core/aarch32/cortex_m/CMakeFiles/arch__arm__core__aarch32__cortex_m.dir/exc_exit.S.obj.d: /projects/github/ncs/zephyr/build/zephyr/include/generated/offsets.h \
./zephyr/arch/arch/arm/core/aarch32/CMakeFiles/arch__arm__core__aarch32.dir/isr_wrapper.S.obj.d: /projects/github/ncs/zephyr/build/zephyr/include/generated/offsets.h \
./zephyr/arch/arch/arm/core/aarch32/CMakeFiles/arch__arm__core__aarch32.dir/swap_helper.S.obj.d: /projects/github/ncs/zephyr/build/zephyr/include/generated/offsets.h \
./zephyr/arch/arch/arm/core/aarch32/CMakeFiles/arch__arm__core__aarch32.dir/prep_c.c.obj.d: /projects/github/ncs/zephyr/build/zephyr/include/generated/offsets.h
./zephyr/arch/arch/arm/core/aarch32/mpu/CMakeFiles/arch__arm__core__aarch32__mpu.dir/arm_core_mpu.c.obj.d: /projects/github/ncs/zephyr/build/zephyr/include/generated/offsets.h \
./zephyr/arch/arch/arm/core/aarch32/mpu/CMakeFiles/arch__arm__core__aarch32__mpu.dir/arm_mpu.c.obj.d: /projects/github/ncs/zephyr/build/zephyr/include/generated/offsets.h \
./zephyr/lib/libc/common/CMakeFiles/lib__libc__common.dir/source/stdlib/malloc.c.obj.d: /projects/github/ncs/zephyr/build/zephyr/include/generated/offsets.h \
./zephyr/lib/libc/minimal/CMakeFiles/lib__libc__minimal.dir/source/time/gmtime.c.obj.d: /projects/github/ncs/zephyr/build/zephyr/include/generated/offsets.h \
./zephyr/lib/libc/minimal/CMakeFiles/lib__libc__minimal.dir/source/stdout/stdout_console.c.obj.d: /projects/github/ncs/zephyr/build/zephyr/include/generated/offsets.h \
./zephyr/CMakeFiles/offsets.dir/arch/arm/core/offsets/offsets.c.obj.d: /projects/github/ncs/zephyr/kernel/include/kernel_offsets.h \
./zephyr/kernel/CMakeFiles/kernel.dir/kheap.c.obj.d: /projects/github/ncs/zephyr/build/zephyr/include/generated/offsets.h
./zephyr/kernel/CMakeFiles/kernel.dir/init.c.obj.d: /projects/github/ncs/zephyr/build/zephyr/include/generated/offsets.h \
./zephyr/kernel/CMakeFiles/kernel.dir/xip.c.obj.d: /projects/github/ncs/zephyr/build/zephyr/include/generated/offsets.h

But please note that zephyr_interface is not called interface because it's the interface API for Zephyr but because it's a CMake interface library. A CMake interface library is used to populate properties for the build system and to pass those around, it is not suppose to create artifacts on the disk, ref: https://cmake.org/cmake/help/latest/command/add_library.html#interface-libraries

An INTERFACE library target does not compile sources and does not produce a library artifact on disk.
...
An interface library created with the above signature has no source files itself and is not included as a target in the generated buildsystem.

Fails with the following error:

But that can be controlled by relocating the definition of that target.

However with the change in this PR there is no longer a requirement to depend on zephyr_generated_headers.

but it muddies what a CMake interface library is.
As well as it doesn't clearly considers the CMake issue linked on target dependency when multiple targets depends on a created file. This means there could still be concurrency issues lurking in the dark, which may suddenly appear when using another build tool (ninja vs. make) or different number of build jobs.
So for the above and without a deeper investigation, then i'm reluctant to approve those changes for something that is working in general case, and because I believe there are better ways to handle this, for example to define the generated headers target earlier.
Also note that it was not ExternalProject itself that was a special use-case, but the fact that your ExternalProject is depending directly on offsets.h.

So for the above and without a deeper investigation, then i'm reluctant to approve those changes for something that is working in general case, and because I believe there are better ways to handle this, for example to define the generated headers target earlier.
Also note that it was not ExternalProject itself that was a special use-case, but the fact that your ExternalProject is depending directly on offsets.h.

The external project is not depending on offsets.h, it's depending on zephyr/ztest.h
This is the call-chain:
include/zephyr/app_memory/app_memdomain.h:9,
subsys/testsuite/ztest/include/zephyr/ztest_test_deprecated.h:16,
subsys/testsuite/ztest/include/zephyr/ztest_test.h:14,
subsys/testsuite/ztest/include/zephyr/ztest.h:51,

Which gives this error:
/home/joakim/repos/ncs/zephyr/include/zephyr/linker/linker-defs.h
zephyr/include/zephyr/linker/linker-defs.h:26:10:
| fatal error: offsets.h: No such file or directory

but it muddies what a CMake interface library is.
As well as it doesn't clearly considers the CMake issue linked on target dependency when multiple targets depends on a created file. This means there could still be concurrency issues lurking in the dark, which may suddenly appear when using another build tool (ninja vs. make) or different number of build jobs.

The zephyr_interface library already serves this purpose for for various other generated header files though.
SYSCALL_LIST_H_TARGET
DRIVER_VALIDATION_H_TARGET
KOBJ_TYPES_H_TARGET
These are all custom commands or targets, that needs to be generated before users of the zephyr interface headers can be compiled.
I supposed you could keep all of these in zephyr_generated_headers and then have zephyr_interface depend on zephyr_generated_headers, or demand that all users of zephyr interface headers needs to depend on zephyr_generated_headers and link against zephyr_interface.
This would have to be properly documented though.
I think in the end having a single target to link or depend on is better, and makes future support cases or similar issues easier.

@tejlmand  Can you have a look at my reply?

Hi @tejlmand, is there any updates on this issue or a recommended way forward?


Influx uses the first header in a file not the one closest/enclosing a reference.

I have a reference to [[influx notes]] at the bottom of a daily note.

But the influx data at the foot of that file shows a header from the top of the file, do today, not the expected 3:06 PM trying to influx.

Influx uses the first header in a note as a header for that note. Other then that, it only processes hierarchical data (bullet lists). As such, this is expected behaviour per se, but I may add an option to disable note headers in Influx.

Partially fixed in latest version.


PDF: wrong generation

Empty cover page:

Wrong layout:

XML namespace updated from https://www.metanorma.org/ns/rsd to https://www.metanorma.org/ns/ribose.


Exporting PDF To Excel

Hi
With pdf imported when selected tables exported some of the files have table contents all over other columns on other pages?
Please advice my document is few pages long with about 5 columns
Charlotte
<EMAIL_ADDRESS>

Can you share the PDF with us that gave you this problem? Please either upload the PDF through the Github interface, send a web-accessible link, or email it to me directly.

Hi
Thank you for your reply.
It was a bank statement with many pages.
Regards
Charlotte
On Sat, Nov 19, 2016 at 11:19 PM Jeremy B. Merrill<EMAIL_ADDRESS>wrote:

Can you share the PDF with us that gave you this problem? Please either
upload the PDF through the Github interface, send a web-accessible link, or
email it to me directly.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tabulapdf/tabula/issues/559#issuecomment-261740293,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AViDOcdYTicvtmM47h0e-m2Hifxu7OlHks5q_2fkgaJpZM4KNTSM
.

Hi @CBranford:
I'm sorry to hear that Tabula did not work correctly for your PDF. And your reason for not wanting to share the document makes total sense. But, without the PDF, I can't replicate the issue, so I can't offer any advice on how to fix it. I hope that this issue will be fixed in future releases of Tabula, so I encourage you to try out the next version in a few months.

Hi
Let me submit but confidential please
On 21 November 2016 at 15:51, Jeremy B. Merrill<EMAIL_ADDRESS>wrote:

Hi @CBranford https://github.com/CBranford:
I'm sorry to hear that Tabula did not work correctly for your PDF. And
your reason for not wanting to share the document makes total sense. But,
without the PDF, I can't replicate the issue, so I can't offer any advice
on how to fix it. I hope that this issue will be fixed in future releases
of Tabula, so I encourage you to try out the next version in a few months.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tabulapdf/tabula/issues/559#issuecomment-261943496,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AViDOZOdh3r0saENukQnA2_KXGPrsEWuks5rAaHigaJpZM4KNTSM
.

--
Charlotte Branford

Okay, email it to me: jeremybmerrill at gmail dot com. I will keep it confidential.

thanks
On 21 November 2016 at 16:00, Jeremy B. Merrill<EMAIL_ADDRESS>wrote:

Okay, email it to me: jeremybmerrill at gmail dot com. I will keep it
confidential.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tabulapdf/tabula/issues/559#issuecomment-261945453,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AViDOXs5u3wrJodz7NFuY4uh5aQBU6rZks5rAaPlgaJpZM4KNTSM
.

--
Charlotte Branford

Hi,
I am looking for help to convert Marathi language PDF file conversion to excel which is having images too.
Can someone help me.
Thanks in advance.
Regards,
Dinesh Jain


using other languages plugins

I'm using vue-monaco together with the webpack plugin , and I want to add synthax checking for yaml. I found a monaco-yaml module for this, but I m a bit confused on how to use it together with vue & webpack. does anybody have done this (even for other languages) ?

Also trying to do this, so far it seems impossible.

Nope sorry
On Mon, 13 Sep 2021, 1:51 pm Jared Allard, @.***> wrote:

Also trying to do this, so far it seems impossible.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/egoist/vue-monaco/issues/21#issuecomment-917772192,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPAP3OJHJAI472WMQGUMUTUBVKKJANCNFSM4GTPRNUA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


BUG FOUND IN Musicoco The application is not installed after downloading

Project Information

Repository:https://github.com/DuanJiaNing/Musicoco
Project Name: Musicoco

Expected behavior
After downloading, the application should be installed easily in the mobile device without any difficulty.
Actual behavior
Describe Error (There is a problem parsing the package) comes after downloading the application when I try to install
How to reproduce
1-Download the application
2-After downloading, try installing the application
3- see bug

Browser/App version: v2.2.0
Operating system: 6.0.1
Smartphone: Galaxy j7

Recording Of The Bug

My Github Account
https://github.com/facebook-786

where have you dowload apk file from, is here: https://github.com/DuanJiaNing/Musicoco/blob/master/apk/Musicoco-v2.2.0.apk

Yeah, I downloaded .apk from here

sir @DuanJiaNing  please review my post

I have tried to install the apk on my own android device and it successed, could you just send the apk file
which you already download to this emal address<EMAIL_ADDRESS>so i can check the apk file.

I have sent you an apk file to your email address. Please check
please check this apk file on 6.0.1 android  version or 6.0.0
thanks you

it works fine on me


Updates torque to 2.11.3

Changes since last update:

Limited sprite radius to a maximum of 255px
Fixed custom marker-file functionality in Firefox
Fixed inertia mismatch of Torque's canvas layer
Added opacity get/setter to canvas layer in Google Maps
Fixed issue with _super being removed from listeners

@javisantana

cc @alonsogarciapablo

LGTM. @fdansv, don't forget to add an entry to NEWS.

Done, thanks for the reminder!


Device Request: Pulsar Keyboard

Requesting support for a keyboard with very similar software to an already supported brand (DREVO)
https://www.pulsar.gg/pages/download-nova-optical-pro
Thank you for the consideration.

can you add the label Area: Hardware Support to your request?

Apologies if this is a silly question, but how exactly would I add that label? I'm not exactly sure how that is done. Thanks for your patience!


Adding TimeZone Support

This is a first draft on optional timezone support.   I could be convinced either way if this should be included by default.

:clap:

As this gem doesn't have any optional feature or config yet, I would add timezones by default as long as it doesn't add any performance issues or some kind of overriding conflict.
Anyway, I don't know the codebase deeply, so consider this just as an opinion.

@esaborit4code We are going to be adding a config come 2.0. Which we will wrap this into.  I want optional loading of all data, so we can keep our memory usage down so we can be as useful as possible across the board.

Closing in favor of #317,   Timezones will be extended by default, but tzinfo is not a dependency.


Add skeleton @appcues/tooltip trait

There's a lot of similarities between the standard modal and a tooltip container, so I've reworked DialogContainerViewController and DialogContainerView to be a superclass for both.
The tooltip trait here doesn't do much yet (ie no pointer), but it handles the case with no target element and otherwise it always positions the wrapper 20pt below the target rectangle:

With target rectangle
Without target rectangle

Obvious lots more to do, but wanted to start with a small PR to make the reworking Dialog bit clearer to follow.

@iujames on a file-by-file basis, you can select the 3 dots in the top right and hide the annotations. Alternatively I should be able to update the codecov config if we don't find it useful at all (I really don't)
github_checks:
    annotations: false


email.html:158 Uncaught ReferenceError: firebase is not defined

Duplicate of #35 - closing this one.


Prevent authenticated users from accessing other users' profiles

Currently, any authenticated user can access another user's profile at http://localhost:5132/api/Profiles/{username} by simply knowing their username. To ensure that only users who own the username can view the associated profile, access needs to be restricted.

Fixed with #163

Fixed with #164
To solve the issue, I removed the \profiles\{username} endpoint and replaced it with a new endpoint \profiles\me. When calling this endpoint, you will receive information about the authenticated user.
// ProfilesController.cs
using Application.UseCases.Settings.Queries;
using Application.UseCases.UserManagement.Queries.UserByUserName;
using MediatR;
using Microsoft.AspNetCore.Authorization;
using Microsoft.AspNetCore.Mvc;
using WebUI.Contracts.UserManagement.UserByUserName;

namespace WebUI.Controllers
{
    [ApiController]
    [Route("api/[controller]")]
    public class ProfilesController : ControllerBase
    {
        private readonly ISender _sender;

        public ProfilesController(ISender sender)
        {
            _sender = sender;
        }

-        [HttpGet("{userName}", Name = "GetUserProfileByUserName")]
-        public async Task<ActionResult<UserByUserNameResponse>> GetUserProfileByUserName([FromRoute] UserByUserNameResponse request)
-       {
-            var query = new UserByUserNameQuery { UserName = request.UserName };
-
-            var result = await _sender.Send(query);
-
-            var response = new UserByUserNameResponse()
-            {
-                FirstName = result.FirstName,
-                LastName = result.LastName,
-                UserName = result.UserName,
-                CreatedAt = result.CreatedAt
-            };
-
-           return Ok(response);
-        }

+       [HttpGet("me", Name = "GetCurrentUser")]
+        [Authorize]
+        public async Task<ActionResult<UserByUserNameResponse>> GetCurrentUser()
+       {
+          return Ok(await _sender.Send(new GetUserProfileQuery()));
+      }
+    }
}

I implemented an interface called ICurrentUser that provides methods to retrieve authenticated user information.
// ICurrentUser.cs
using System.Security.Claims;

namespace Application.Common.Interfaces;

public interface ICurrentUser
{
    string? Name { get; }

    string GetUserId();

    string GetUserEmail();

    bool IsAuthenticated();
    void SetCurrentUser(ClaimsPrincipal user);

    void SetCurrentUserId(string userId);

    IEnumerable<Claim>? GetUserClaims();

}


Save augmented images for debugging (extension of SemanticSegmentationData?)

🚀 Feature
For the purpose of semantic segmentation, it would be nice to be able to save the augmented images (after kornia, albumentations, icevision...) for debugging purposes .
Motivation
For debugging purposes it would be aweome to see, how the selected parameters affect the images. Of course this could be done separately, but it would be great if there was the possibility to save the date with simple flags.

Pitch
I would suggest additional flags in SemanticSegmentationData with "saveImagesTrue" or/and a folderpath for storing these data

Alternatives
No hard directly integrated functionality, but an example how to use existing functions to restore this behavoir.

Additional context

Hi @andife, thanks for the feature request! We did previously have some callback functionality that would allow you to access the samples / batch at various points in the pipeline. It has issues at the moment but is something we could look to clean up to enable this sort of use case 😃


PIX runtime not found when building from registry.

Describe the Bug
Since the project uses an overlay-port to load the PIX runtime, when referencing it using the registry, the package cannot be resolved from the root ports. This currently breaks the registry. To make it work again, the overlay port should also be added to the registry.

The same applies to the D3D12 Agility SDK (see #95).


Getting ExtractorError: No video formats found Error

Checklist

[x] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.06.06
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-f', u'bestvideo+bestaudio', u'http://d1z78r8i505acl.cloudfront.net/media/lkJunHwrVYlPx/3f50585e/stream.mpd', u'--verbose']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.06.06
[debug] Python version 2.7.17 (CPython) - Linux-5.4.0-81-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.8, ffprobe 3.4.8, rtmpdump 2.4
[debug] Proxy map: {}
[generic] stream: Requesting header
[redirect] Following redirect to https://d1z78r8i505acl.cloudfront.net/media/lkJunHwrVYlPx/3f50585e/stream.mpd
[generic] stream: Requesting header
WARNING: Falling back on generic information extractor.
[generic] stream: Downloading webpage
[generic] stream: Extracting information
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 2525, in _real_extract
    self._sort_formats(info_dict['formats'])
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 1374, in _sort_formats
    raise ExtractorError('No video formats found')
ExtractorError: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

Downloading video hosted on amazon(probably), The mpd file is public, when I try to download it using youtube-dl tool I am getting ExtractorError: No video formats found error

DRM protected.

@kikuyan : No workaround to it?

Better to ask VideoHelp Forum.


Bug

When you change the PC time, the countdown value is changed together, even if the start-time and end-time values are set

Same issue,  looks like it ignores start time value and uses current time;
@Tiagospem  was you able to solve this somehow?

@bahramhasanov any countdown has this problem, I solved it by creating my own countodwn, theoretically you need to compare the server time with the clock in javascript and subtract the time difference, that way whenever there is a change in the computer clock, it will work correctly


Docs about RxJava are not clear

I'm having difficulties understanding your support to RxJava. From the main readme I thought you had Observables that call onNext everytime something in the db changed (if the query is interested in the change), but then I can only find documentation for Single, that can only emit once (so there are no further notifications of db changes). What's the right way?
Your product seems really good, but the documentation is scattered throughout too many places, I'm still unable to find examples of inserts and deletes and I have to go to the examples to find some good select examples.

Hi the Rx support can be summed up in this way and does support what you're looking for:
Store operations (insert/update/delete)
Query operations:
toObservable() - the query results are returned as an Rx Observable sequence
toSelfObservable() - the result itself is emitted as an Rx Observable and then emitted again if any sql statements are executed affecting the table

Store operations (insert/update/delete)

This brings me to the EntityStoreInterface page, where I can find no example on insert, update or delete operations.
I've figured out some of the stuff I needed, but an improvement in the documentation could help a lot.

There is an example project using RxJava & Android illustrating many of those https://github.com/requery/requery/tree/master/requery-android/example. That being said there are a large number of supported cases (Rx, pure blocking API, Java 8 CompletionStage etc) it is difficult to provide an example for all API combinations but it will improve over time.

Closing for now, if more info is needed please clarify


int32 need to be changed to uint32

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description

As above, there are some fields in both http and tcp connection pool in DestinationRule which value default to 2^32-1，but the field type is int32。Maybe type of those fields should be changed to uint32 ?
Version
1.18 or later

Additional Information
No response

In envoy as well as inside istiod these are uint32 which is how the defaults can be set to values which are otherwise not possible to express in the api. I doubt we can change scalar types in a public api at this point but I agree the comment, although technically correct, seems odd without some inside context.

ah, sorry. It is in the api module.
https://github.com/istio/api/blob/master/networking/v1alpha3/destination_rule.pb.go

I have created a new issue in the api project. Thanks.
https://github.com/istio/api/issues/3267


calling getValue() for each attribute in AttributeBehavior

Q
A

Is bugfix?
no

New feature?
yes

Breaks BC?
I hope not

Tests pass?
haven't checked

Fixed issues
none

calling getValue() for each attribute (instead of once for all attributes) and passing attribute name along as a parameter to value function so that value function can return different values depending on current attribute

what is the use case for this? wouldn't it be better to add multiple behaviors in this case?

Just seeing that this breaks the build. I'll fix this.
Anyway - use case: I want to return a value for an attribute depending on the current value of the attribute. That is, I want to a type conversion. I want to convert a DateTime to a time stamp or string on beforeUpdate/beforeInsert and the timestamp/string back into a DateTime on afterFind.

@cebe you asked:

wouldn't it be better to add multiple behaviors in this case?

The behavior is actually the same for each attribute. I just want to be able to change an attribute's value depending on the original value of the attribute. So running the value function once for all attributes won't do because that would set the same new value on all attributes. So I want to have the attribute's name in the value function to identify the attribute`to get its value. Does this make sense to you?
I can of course do that with a custom attribute which is what I'm doing right now. I just thought the existing AttributeBehavior could benefit from this. Any thoughts?

it could maybe, even though that is out of the original scope of it. Need to check it in more detail.

I basically suggested the change to AttributeBehavior so I can do what I've done in the following subclass. Maybe that makes it easier to understand than my previous comment does.
`final class DateTimeConversionBehavior extends AttributeBehavior
{
const DATETIME_FORMAT = "Y-m-d H:i:s";
public $dateTimeAttributes;
function init()
{
    parent::init();

    if (empty($this->attributes)) {
        $this->attributes = [
            BaseActiveRecord::EVENT_BEFORE_INSERT => $this->dateTimeAttributes,
            BaseActiveRecord::EVENT_BEFORE_UPDATE => $this->dateTimeAttributes,
            BaseActiveRecord::EVENT_AFTER_FIND => $this->dateTimeAttributes,
        ];
    }

    if ($this->value === null) {
        $this->value = [$this, 'getAttributeValue'];
    }
}

function getAttributeValue($event, $attributeName)
{
    switch ($event->name) {
        case BaseActiveRecord::EVENT_BEFORE_INSERT:
        case BaseActiveRecord::EVENT_BEFORE_UPDATE:
            return $this->convertToMySqlDateTime($event->sender->$attributeName);
        case BaseActiveRecord::EVENT_AFTER_FIND:
            return $this->convertToPhpDateTime($event->sender->$attributeName);
    }
}

function convertToMySqlDateTime($phpDateTime)
{
    return $phpDateTime === null ?
        null :
        $phpDateTime->format(static::DATETIME_FORMAT);
}

function convertToPhpDateTime($mySqlDateTime)
{
    return $mySqlDateTime === null ?
        null :
        date_create_from_format(static::DATETIME_FORMAT, $mySqlDateTime);
}

}
`

Won't make it into 2.0.


mscratchcsw defined using mcause.mpp instead of mstatus.mpp

The pseudocode definition of csrrw access to the mscratchcsw CSR uses mcause.mpp for the previous privilege level. However, if mtvec.mode is not CLIC mode, mcause.mpp does not mirror mstatus.mpp and is read as 0. Should the definition use mstatus.mpp instead of mcause.mpp, so that the operation is the same in non-CLIC mode?

mscratchcsw should not exist in non-CLIC mode, need to clarify text

closed by pull #258?  default text that CSRs added by CLIC mode appear zero unless specified elsewise.

Issue addressed by above commits.


Multiplexer matchers

[x] Multiplexer

[x] Method
[x] Path
[x] Url
[x] Status Code
[x] Headers
[x] Query

Pending to add tests.


release-notes: explain libgcc changes

Description of changes
This commit adds (quite a bit of) text to the release notes to try to help out-of-tree Nix packages which are downstream of nixpkgs and encounter breakage due to the libgcc changes.
Things done

Built on platform(s)

[ ] 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/)
23.05 Release Notes (or backporting 22.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

[ ] Fits CONTRIBUTING.md.

Note: I will be offline from sometime in the next few hours until after the end of the month.
Please don't hesitate to edit this PR aggressively if necessary.  I'm unlikely to be able to respond to review comments before the release deadline.

Pushed a fix to the NixOS manual.
By the way, why or null?

By the way, why or null?

Because stdenv.cc might not have a libgcc output.  Two cases come to mind:

Future: using a non-gcc compiler (clang?)
Current: even our gcc does not always create a libgcc output.  Here are the current conditions which cause its creation, but these may (and almost certainly will) change in the future.

Note: I will be offline from sometime in the next few hours until after the end of the month.
Please don't hesitate to edit this PR aggressively if necessary. I'm unlikely to be able to respond to review comments before the release deadline.

I'm back and pretty close to conquering my backlog.  Not sure if this is still being considered for the release (would technically be a "backport" at this point), but I can respond to comments now.

Because stdenv.cc might not have a libgcc output.

Shouldn't compilation of things that depend on libgcc fail in those cases?

There are many situations where stdenv.cc doesn't have a libgcc output:

pkgsStatic
cross-compilers to platforms which lack dynamic linkers (arm-none-eabi)
a hypothetical future non-gcc-based stdenv

Shouldn't compilation of things that depend on libgcc fail in those cases?

Unfortunately no.  This is the big problem with libgcc.  The glibc developers have decided to dlopen() it and not build with -lgcc, so the dependency is completely invisible to nixpkgs and patchelf.  This is the root cause of all our troubles.


Add GitHub Action workflow for CI

This is an initial workflow to enable CI using GitHub Actions. It makes use of
reusable workflows and composite actions defined in govuk-infrastructure. The
CI workflow is initially only used to support deployments to the new EKS
infrastructure and not to immediately replace the CI workflow in Jenkins.
The CI workflow makes use of individual jobs to run each testing and linting
tool. This allows the processes to run concurrently and improves the visibility
of status in the GitHub UI.
Most jobs make use of reusable workflows as they are consistent across the
majority of our repositories - this allows us to easily maintain them in a
single place. However, the test-ruby job (and/or integration-tests) is defined
per repo as backing services and other dependencies vary greatly between repos.
Backing services (such as databases) are created using a composite action
instead of a job service. This is because they contain a lot of shared config
and makes it easier for us to maintain.

Publishing API has a bunch of after test tasks including running pact tests that need to be incorporated.


Add support for rich state display in Jupyter Notebooks.

As of #290, the katas now use a version of IQ# that includes the enhancements from https://github.com/microsoft/iqsharp/pull/81. By calling the WithJupyterDisplay extension method at simulator creation time, the %kata magic can thus be extended with support for this new functionality.

I can have a look at this. To implement correctly there is an open issue in the IQSharp project which probably needs to be solved first.

This should be addressed in the next release following https://github.com/microsoft/QuantumKatas/pull/344. I'll close this issue for now, and reopen if it turns out more work is needed. Thank you @cgranade for flagging this and @vivanwin for working on this!


Removed public-read acl from amplify publish method

*Issue #1599
*Description of changes: As mentioned here, S3 ACLs is a legacy access control mechanism that predates IAM. I don't see the value of making each individual object public through ACL since bucket policy already does that. Therefore I removed the acl public read from the upload parameters.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This is implemented by the PR https://github.com/aws-amplify/amplify-cli/pull/2369


BUG: Questionable macOS wheel contents

Describe your issue.
I'm not sure if this really is a bug. Merely I found it very unusual so I thought I'd check if it was deliberate. It has broken our pipelines and whilst that's more due to missasumptions on our end, I want to be sure before I start trying to fix it.
The latest release (1.8.0) of wheels for macOS includes both universal2 and separate architecture variants. Both the arm64 slice of the universal2 wheel and the arm64 wheel targets macOS >= 12 leaving macOS M1 Big Surs to compile from source?
Inside the universal2 wheel there is a mixture of fat and thin dylibs including one i386 slice:
> for file in scipy/.
    printf '%s\t%s\n' $file (llvm-lipo -archs $file)
end
scipy/.dylibs/libgcc_s.1.dylib		x86_64 i386
scipy/.dylibs/libgcc_s.2.dylib		arm64
scipy/.dylibs/libgfortran.3.dylib	x86_64
scipy/.dylibs/libgfortran.5.dylib	arm64
scipy/.dylibs/libopenblas.0.dylib	x86_64 arm64
scipy/.dylibs/libquadmath.0.dylib	x86_64

In particular, I find that seemingly redundant i386 slice and the lack of an arm64 variant of libquadmath unlikely.
Is the lack of an macOS 11 M1 compatible wheel and the above cocktail of binaries types intentional?
Reproducing Code Example
N/A

Error message
N/A

SciPy/NumPy/Python version information
1.8.0 1.22.2

Can maybe ping @isuruf for opinion maybe. I think we've been leaning toward ditching the universal2 and just shipping the separate wheels long-term?

Is the lack of an macOS 11 M1 compatible wheel and the above cocktail of binaries types intentional?

To be clear, you should be able to install a binary directly on MacOS M1, if you are using a recent enough OS.

Yes, you need MacOS 12.x for M1 because of issues on 11.x. I'm sure you could dig up the issues/reasons we had for this.. by they were serious like segfaults with 11.x on M1.

I'll close this, since it is working as intended--you need MacOS 12.x for M1 binary install b/c of serious segfault issues. I think Ralf worked a bit with Apple folks on that one too.

From a quick search of our issue tracker: https://github.com/scipy/scipy/issues/13409#issuecomment-971217036

Maybe a separate issue would be appropriate for the "cocktail" inside the universal2 binary if it can be demonstrated that it actually causes a problem.

I'll leave this open until i.e., Isuru comments on the wheel contents maybe

Maybe a separate issue would be appropriate for the "cocktail" inside the universal2 binary if it can be demonstrated that it actually causes a problem. You didn't report an actual i.e., test failure there?

The cocktail has knocked out PyInstaller support but that's more PyInstaller's problem but other than that it doesn't appear to have broken anything. Like I said, I just wanted to confirm that it is intentional. Given how hard it is to get anything resembling macOS M1 CI/CD, I thought that that wheel might have had to be released untested.

In particular, I find that seemingly redundant i386 slice

That's because the gfortran toolchain we are using for x86_64 has a libgfortran.dylib that has both x86_64 and i386 slices.
It'll be good to remove that i386 slice to reduce a little bit of space.

and the lack of an arm64 variant of libquadmath unlikely.

This is expected because libquadmath doesn't support arm64/aarch64.

This is expected because libquadmath doesn't support arm64/aarch64.

Ahh so libquadmath is an optional dependency?

Ok, thanks Isuru--I think I can close this then. Ralf did carefully run our full test suite on his M1 Mac, but @bwoodsend is correct that the CI situation there is unfortunate of course.
Shrinking wheels is of interest to us too, we've had some "bloating" recently I think.

Ahh so libquadmath is an optional dependency?

Yes. It's only needed for np.float128 support which is only supported in x86_64 and not arm64.

Perfect. Thanks for clearing that up for me. The redundant i386 only wastes 150kB so it's probably not worth pulling teeth over.


Remove dependency of LanguageServerProtocol on TSC

As in https://github.com/apple/sourcekit-lsp/issues/936, LanguageServerProtocol should be usable in any context without any dependencies - it's a generally useful module for eg. any LSP client written in Swift to use. Remove the dependency on TSC.

Tracked in Apple’s issue tracker as rdar://117565087


[WiP] Datepicker for time fields

Resolves #785.
Currently there's no way to specify PM times.

Do staff members really want a date-picker? When we had it in prod - it sounded like it just made the TA experience harder. Maybe we just need a format validator?

I'm just going to close this PR. The issue is still open so we can have it as a reminder.


Replace log.fatal with error return?

should the log.fatal lines be replaced by an error return? e.g. at https://github.com/Foxboron/go-uefi/blob/5faf8e43ee9b7163b277b7a2d6282f1b904ca1de/efi/attributes/attributes.go#L112

They probably should! The main problem has been to try figure out a good way to structure the top-level APIs in a way that makes sense. The result is that many of the functions are just hacks to iterate quickly, it's easier to ignore error handling :smile:
I'd gladly take patches else I'll iterate a bit while working on covering more with integration tests :) The relevant issues with attributes.go is how to set the appropriate os.OpenFile file modes in a way we can truncate the file (say we want to remove dbx with an empty EFI_SIGNATURE_LIST signed with KEK) and write to it. I haven't figured all of it yet.

They probably should! The main problem has been to try figure out a good way to structure the top-level APIs in a way that makes sense. The result is that many of the functions are just hacks to iterate quickly, it's easier to ignore error handling :smile:
I'd gladly take patches else I'll iterate a bit while working on covering more with integration tests :) The relevant issues with attributes.go is how to set the appropriate os.OpenFile file modes in a way we can truncate the file (say we want to remove dbx with an empty EFI_SIGNATURE_LIST signed with KEK) and write to it. I haven't figured all of it yet.

Ah so you are still trying to flesh these APIs. So this issue is moot.
I'm not yet ready to submit any PR. I'm still trying to figure out how Secure Boot actually works, and this code and sbctl are helping making things clear. Thanks for creating this and putting it in the open :-)

Ah so you are still trying to flesh these APIs. So this issue is moot.
I'm not yet ready to submit any PR. I'm still trying to figure out how Secure Boot actually works, and this code and sbctl are helping making things clear. Thanks for creating this and putting it in the open :-)

sbctl still uses sbsigntools for secureboot operations. But you could almost substitute all the tools with the stuff inside ./cmd. It should be noted that you can totally sign and enroll keys using go-uefi, but the API is tedious and not really ready for people adopting it.
Once I'm happy with the API I intend to mark a release and actually port sbctl over. But until then I need to read some more API designs and figure out how things make sense :)
Thanks for the encouraging words and please do open issues for any API ideas or stuff that is confusing in the code. I'll gladly take the feedback to figure out these things!

sbctl still uses sbsigntools for secureboot operations. But you could almost substitute all the tools with the stuff inside ./cmd. It should be noted that you can totally sign and enroll keys using go-uefi, but the API is tedious and not really ready for people adopting it.
Once I'm happy with the API I intend to mark a release and actually port sbctl over. But until then I need to read some more API designs and figure out how things make sense :)
Thanks for the encouraging words and please do open issues for any API ideas or stuff that is confusing in the code. I'll gladly take the feedback to figure out these things!


Make pagesize query parameter configurable.

Add commandline flag to change pagesize query parameter while still
defaulting to 50.

I think pull request 200 is a good fix for this, but simply being able to change the pageSize parameter to a larger value works great for an account I'm using with 100+ apps. It also reduces the number of request that have to be made instead of iterating over multiple pages.

@chrisperelstein have you tested to what number you can increase the page size?

Fixed by #216.


Markdown tables

Type of changes

[ ] Bug fix
[x] New feature
[ ] Documentation / docstrings
[ ] Tests
[ ] Other

Checklist

[ ] I've run the latest black with default args on new code.
[ ] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[ ] I've added tests for new code.
[ ] I accept that @willmcgugan may be pedantic in the code review.

Description
Adds support for Markdown tables.

Codecov Report
Patch coverage: 36.00% and project coverage change: -3.47 :warning:

Comparison is base (51f70fa) 98.31% compared to head (4345f51) 94.84%.

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

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2977      +/-   ##
==========================================
- Coverage   98.31%   94.84%   -3.47%
==========================================
  Files          74       74
  Lines        7949     7998      +49
==========================================
- Hits         7815     7586     -229
- Misses        134      412     +278

Flag
Coverage Δ

unittests
94.84% <36.00%> (-3.47%)
:arrow_down:

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

Impacted Files
Coverage Δ

rich/markdown.py
90.65% <36.00%> (-9.02%)
:arrow_down:

... and 3 files with indirect coverage changes

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


Why get_primitive_library is using self.strs

@gatecat
I'm having issue with get_primitive_library, when it calls to_logical_netlist it passes self.strs, this causes cellDecls names to no longer reference strings inside primLibs.strList but strings inside self.strs.
This is unclear to me, why we are doing it this way. I assume this was done due to some devices not having strList set and their names were referring to global names.

Have you seen the discussion behind this in #47 ?

May bad, did not see this issue.


Changed import statement to angled

While building AFOnoResponseSerializer fetched through Cocoapods 1.0.1, Xcode 7.3 couldn't find an AFNetworking header file. Changing the import statement to <AFNetworking/AFURLResponseSerialization.h> fixed the issue for me.

Hi @mattt, thank you for all your amazing work! This is my first pull request ever, so I hope I'm doing it right! A simple fix to an import problem with an AFNetworking dependency.


Changes in install generator template

Rails command rails destroy feature:install was not working
Change install generator to take input from user so table name / class
name can be defined by user.
Adds specs for install template using generator_spec gem. Could not add migration spec. Had some
problems with time stamp.

Coverage decreased (-59.6%) to 37.333% when pulling 0dec1ee52c1399c4176e1a35fb87c76e434e6a5b on laertispappas:lp/generator_destroy_fix into 37ea89a1dfbb6a19f56d4e14212f67a36e7d8889 on mgsnova:master.