QR Scanning

What's happening?

When I try to scan qr code I m not able to read any qrcode any solution
How can it be reproduced?

 <QRCodeScanner         onRead={this.onSuccess.bind(this)}         topContent={           <Text style={styles.centerText}>             HOME           </Text>         }         bottomContent={           <TouchableOpacity style={styles.buttonTouchable}>             <Text style={styles.buttonText}>WELCOME</Text>           </TouchableOpacity>         }       />
Build details?

OS ANDROID
"react": "16.3.1",
"react-native": "0.55.1",
"react-native-camera": "^1.0.3",
"react-native-qrcode-scanner": "^1.0.1"

I just had to upgrade to v1.0.3 from v1.0.1. That'll solve it

@JakeRawr How did you upgrade to V1.0.3? I'm facing this issue when i tried doing the same.

No matching version found for<EMAIL_ADDRESS>

@karthick-t2s Hey sorry for misleading you. I updated React Native camera from v1.0.1 to v1.0.3. Not react native qr scanner. I’ll update my solution as well. Hope that helps you.

@JakeRawr that's fine. I'm using react native camera V1.1.4 and the scanning takes too long and sometimes it won't scan at all.

I've found that running on the device works perfectly on both Android and iOS but running in the android emulator doesn't scan the QR code correctly.
Really no idea how to debug it, it simply reads nonsense from the qr code as presented.


fix: Bug fix on backfill action for BaseAspectRoutingResource

Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[x] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable)

Instead of hardcode the routing aspect class in the backfill result, we should actually read from remote GMS backfill result and set accordingly.


Bug in APILoggerMiddleware

Hi,
Great thanks, for this package!
I got bug from django "KeyError: 'content-type' in APILoggerMiddleware"
probably, you should use method ".get()" to get key 'content-type' from response
line 76:
if 'content-type' in response and
response.get('content-type') == 'application/json' or
response.get('content-type') == 'application/vnd.api+json':

Hi,
Thanks for the suggestion.
In the next version I'll fix this issue.
On Wed, 19 May 2021, 4:51 pm Bernar Berdikul, @.***>
wrote:

Hi,
Great thanks, for this package!
I got bug from django "KeyError: 'content-type' in APILoggerMiddleware"
probably, you should use method ".get()" to get key 'content-type' from
response
line 76:
if 'content-type' in response and
response.get('content-type') == 'application/json' or
response.get('content-type') == 'application/vnd.api+json':
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/vishalanandl177/DRF-API-Logger/issues/16, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AD7SDTGLTCMMQPZSPA5S6JLTOONMLANCNFSM45EOJROQ
.

In the next version, It will fix this issue.

The issue is fixed, please update to the latest version 1.0.7.
New features:

Hide Sensitive Data From Logs
Change the default database to store API logs


Notebooks 2.0 // Frontend // Workspaces table // Start, restart and stop workspace

Implement Start, restart and stop workspace actions present in the workspace table actions kebab dropdown.
If workspace is running:

Stop action: display warning popup about unsaved work.

If there is a pending update:

Buttons: "Stop and update", "Stop and defer update", "Cancel".

If there is NOT a pending update:

Buttons: "Stop", "Cancel".

Restart action: display warning popup about unsaved work.

If there is a pending update:

Buttons: "Update and restart", "Only restart", "Cancel".

If there is NOT a pending update:

Buttons: "Restart", "Cancel".

If workspace is not running:

Start action:

If there is a pending update: display warning popup about unsaved work.

Buttons: "Update and start", "Only start", "Cancel".

If there is NOT a pending update, start workbench directly.

/assign


Improve lang lib Module.md fie

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #18855 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #18855   +/-   ##
=======================================
  Coverage   14.95%   14.95%
=======================================
  Files          47       47
  Lines        1230     1230
  Branches      187      187
=======================================
  Hits          184      184
  Misses       1033     1033
  Partials       13       13

Continue to review full report at Codecov.

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


Clarify the 120 nodes limit cluster type in ARO FAQ

It's not entirely clear when we just state: If the cluster is created using User Defined Routing (UDR). We need to set the right expectations by referring to private clusters without any public IPs for outbound.

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

#sign-off


Font type and size

How to change the font size and font type for legend and axis?

It is not possible to change the font-type in the current version.


Use specific http header in ring hash.

https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/upstream/load_balancing/load_balancers#ring-hash
According to the doc above,  we know that while using ring hash lb policy, each request is routed to a host by hashing some property of the request. I wonder whether this property of the request can be specified with some http headers, or is this feature already supported and how should I config it?

Found what I need...


[RFC-007] Implement GitHub app authentication for git repositories.

API change to add new github provider field in GitRepository spec.
Controller change to use the GitHub authentication information specified in .spec.secretRef to create the auth options to authenticate to git repositories when the provider field is set to github
Tests for new github provider field
Updated docs to use GitHub Apps for authentication in source-controller.

Thank you for picking up this topic. Can we also add documentation for the use with GitHub Enterprise?

@dipti-pai please rebase with main, it contains the latest auth and git packages with GitHub App suport.


Support for TimescaleDB's hypertable

Is your feature request related to a problem? Please describe.
Does it support Postgres based TimescaleDB's hypertable?
Describe the solution you'd like
Support for TimescaleDB's hypertable
Additional context
This should potentially help in time-series based data exploration.

Thank you for the feature request!  We are considering integrating the duckDB connector to Postgres in our application.
https://duckdb.org/2022/09/30/postgres-scanner.html
Would this help you get what you need? It sounds like you are interested in TimescaleDB, specifically? What features do you like about TimescaleDB that you are looking for here?

Thank you for the feature request! We are considering integrating the duckDB connector to Postgres in our application.
https://duckdb.org/2022/09/30/postgres-scanner.html
Would this help you get what you need? It sounds like you are interested in TimescaleDB, specifically? What features do you like about TimescaleDB that you are looking for here?

Awesome! Sounds good to me!
I've been fiddling around with some sensors at home and TimescaleDB fits best for the use case.

We support postgres since a while back that would allow you to talk to any postgres compliant databases. Let us know if you would run into any issues!


Article isn't clear on how to handle IAsyncDisposable without IDisposable

If I read the article right, it is OK to implement only IAsyncDisposable without IDisopsable (also for non-sealed classes, the article could be clearer in that regard).
But the article does not show a full example on how the async disposable pattern looks like when IDisposable is not implemented. The first full example shown (in Implements the async dispose pattern) already has IDisposable implemented.
In fact, right thereafter the article then says:

You may need to implement both the IDisposable and IAsyncDisposable

Yes, we already did that right in the example above.
I think the article should be a bit clearer on how to handle IAsncDisposable alone and include a full example for this.

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

ID: 61a961dc-3ebf-2a96-a6eb-2e0b62ab909f
Version Independent ID: d72276e1-9059-815f-dc14-a03a7544ab87
Content: Implement a DisposeAsync method
Content Source: docs/standard/garbage-collection/implementing-disposeasync.md
Product: dotnet-fundamentals
GitHub Login: @IEvangelist
Microsoft Alias: dapine

Hi @bitbonk, thank you for posting this issue. I agree with you that this article needs some work, and the points that you raise are valid. I'll work on addressing this.

Related to #28630

@IEvangelist, you agreed with @bitbonk that the article must explain the use of IAsyncDisposable alone.
But now you have closed this issue together with the PR which reports (only) a potential resource leak.
I still don't find an explanation for the correct implementation of IAsyncDisposable WITHOUT IDisposable in the article. I think this issue is still to be finished and should be reopened.

@IEvangelist I agree. Even with the new change, the article still reads that it is OK to only implement IAsyncDisposable (without IDisposable and yet the article does now show how this is done.
There is a header for the async-only dispose pattern: Implement the async dispose pattern.
And there is a header for the sync+async dispose pattern: mplement both dispose and async dispose patterns.
But both examples implement IAsyncDisposable and IDisposable. Both examples are almost the same.
Event weirder: The chapter Implement both dispose and async dispose patterns begins with the text

You may need to implement both the IDisposable and IAsyncDisposable interfaces

Which seems to convey that were are now introducing IDisposable to the mix, but we did that aready in the example right before that.
After reading the article now, we still don't now how to implement IAsyncDisposable only.

@IEvangelist that was a lousy way to close this issue. You agree that the article needs work but then gets away with just adding a simple warning box?

Hey @bitbonk and @CasperWSchmidt, I've been having trouble with my "at mentions" not notifying me, I've since fixed that issue - but I did admittedly miss this. Sorry, please see #32040 and let me know what you think. Thank you for your patience.

@IEvangelist It's a bit harder to read the markup than seeing it live, but it looks good. When can we expect the changes to be live?

@IEvangelist It's a bit harder to read the markup than seeing it live, but it looks good. When can we expect the changes to be live?

Hi @CasperWSchmidt - it's live now: https://learn.microsoft.com/en-us/dotnet/standard/garbage-collection/implementing-disposeasync#implement-the-async-dispose-pattern


Provide basedir or root option to allow absolute paths for includes

Feature Description
First of all, thank you for your work on this project.  I am migrating from ng-cli-pug-loader (thanks to your helpful reply on one of its issue threads) following the upgrade to Angular 12 and think this implementation is very nice.  I did run into one big issue however but it seems like it should be pretty easy to solve.
Pug allows the use of 'include" to import pug templates from elsewhere in the source code.  It is very convenient to be able to use absolute paths, relative to the root or base of the project, for these includes.
The former solution, ng-cli-pug-loader, did allow this style for includes, my project relied on it heavily.  Btw, this project also had some issues with pug includes, see below for more info on that.
The dependency simple-pug-loader allows this style as well, however you have to provide that value for the basedir or root option in the loader config:
...
const addPugRules = (config) => {
     return webpack_merge_1.merge(config, {
        module: {
            rules: [
                {
                    test: /\.(pug|jade)$/,
                    use: [
                        { loader: 'apply-loader' },
                        {
                            loader: 'simple-pug-loader',
                            options: {
                                basedir: config.context // '/srv/deeply/nested/path/to/project/app'
                            }
                        }
                    ]
                }
            ]
        }
     });
  };
...

I am not entirely sure if this is actually an issue with the simple-pug-loader dependency or what, but it seems modifying the loader rules created by ngx-pug-builder's util.js file takes care of the problem for me at least.
FYI, I also noticed that the former project, ng-cli-pug-loader, had some issues with includes, specifically with apply-loader per this issue, and this was their solution:
function getPugLoaderRules(): string {
  const partialRegex = /\.(include|partial)\.(pug|jade)$/;
  return `
    {
      test: /\\.(pug|jade)$/,
      exclude: ${partialRegex},
      use: [
        { loader: "apply-loader" },
        { loader: "pug-loader" }
      ]
    },
    {
      test: ${partialRegex},
      loader: "pug-loader"
    },`.replace(/\s+/gm, ' ');
}

I'm not 100% clear on the why and how or if this even address the problem I am discussing or a different one, but hoping you know more than me, maybe you can make use of this information.
Let me know if there is anything else I can do, am interested to see how you think this should be addressed if at all.
Thanks again.
Use Case
I can keep my includes like so:
include /src/theme/templates/mixins/shared.include.pug

Instead of what would otherwise be a nightmare:
include ../../../../../../theme/templates/mixins/shared.include.pug

PS, I might have labeled this a bug instead if I had the option, but like I said, I don't know if this is really more an issue belonging to simple-pug-loader or what, all I know is they do provide the ability to pass the correct options object with the rules config, and everything is documented.
This all began when I was unable to compile or serve with my includes like so:
include /src/theme/templates/mixins/shared.include.pug

For the files which contained those includes, the following error would return when attempting to compile:
Error: Module build failed (from ./node_modules/simple-pug-loader/index.js):
[ng] Error: the "basedir" option is required to use includes and extends with "absolute" paths
[ng]     at /srv/project/client/src/app/mods/com/comps/meta/view/com.meta.view.comp.pug line 1
[ng]     at Function.resolve (/srv/project/client/node_modules/pug-load/index.js:73:11)
[ng]     at Object.resolve (/srv/project/client/node_modules/pug/lib/index.js:153:19)
[ng]     at /srv/project/client/node_modules/pug-load/index.js:26:26
[ng]     at walkAST (/srv/project/client/node_modules/pug-load/node_modules/pug-walk/index.js:26:18)
[ng]     at /srv/project/client/node_modules/pug-load/node_modules/pug-walk/index.js:112:20
[ng]     at Array.reduce (<anonymous>)
[ng]     at walkAndMergeNodes (/srv/project/client/node_modules/pug-load/node_modules/pug-walk/index.js:111:18)
[ng]     at walkAST (/srv/project/client/node_modules/pug-load/node_modules/pug-walk/index.js:40:19)
[ng]     at load (/srv/project/client/node_modules/pug-load/index.js:13:10)
[ng]     at Function.loadString [as string] (/srv/project/client/node_modules/pug-load/index.js:55:10)
[ng]     at compileBody (/srv/project/client/node_modules/pug/lib/index.js:82:18)
[ng]     at Object.exports.compileClientWithDependenciesTracked (/srv/project/client/node_modules/pug/lib/index.js:317:16)
[ng]     at Object.module.exports (/srv/project/client/node_modules/simple-pug-loader/index.js:86:29)

This is the relevant piece of code for simple-pug-loader, index.js:
const pugOptions = {
      filename,
      doctype: options.doctype || 'html',
      pretty: options.pretty,
      self: options.self,
      compileDebug: loaderContext.debug || false,
      globals: ['require'].concat(options.globals || []),
      name: 'template',
      inlineRuntimeFunctions: false,
      filters: options.filters,
      plugins: [plugin].concat(options.plugins || []),
      basedir: options.root || options.basedir
    };

You can see how if no options are passed, basedir will be undefined.  I do not have any idea where it goes from there to determine when basedir is actually required.
Anyway, just more info for you.

Hi @gerwintown,
Thank you! That is a great idea. Just released a new version. 🙂


Promote staging to master

An automatic merge is not possible, so I will perform a manual merge in order to update master and complete this PR.

Current coverage is 80.06%

Merging #188 into master will increase coverage by +20.22% as of fd1b507

@@            master    #188   diff @@
======================================
  Files           15      21      +6
  Stmts         2072    2994    +922
  Branches         0       0
  Methods          0       0
======================================
+ Hit           1240    2397   +1157
  Partial          0       0
+ Missed         832     597    -235

Review entire Coverage Diff as of fd1b507

Powered by Codecov. Updated on successful CI builds.


Speed up _.clone() on arrays

This is my test：

arr.concat() has better performance than arr.slice().

Coverage remained the same at 97.01% when pulling 349adc031454587d0e743d193ccea2a3c23a445e on CoderMing:master into 3cd55ea9b40044b34bfe1ad90c25eb3ac777b158 on jashkenas:master.

Coverage remained the same at 97.01% when pulling 349adc031454587d0e743d193ccea2a3c23a445e on CoderMing:master into 3cd55ea9b40044b34bfe1ad90c25eb3ac777b158 on jashkenas:master.

@CoderMing Thank you for taking the effort to submit a pull request.
Would you mind extending the clone test in test/objects.js to add a check that _.clone on an array does indeed return a shallow copy and not the array itself? Once you do this, I'll take care of the merge conflicts for you.

@CoderMing While you're at it, please apply this optimization also to _.toArray and please double-check whether there are any other places where it might be applicable. Thanks in advance.

A quick test of my own didn't confirm the performance advantage. I just noticed that @CoderMing was doing .slice(0) instead of just .slice() in their microbenchmark, which might have disabled an optimization in the JS engine.
Closing this until somebody offers stronger evidence that there is an actual performance benefit.

A quick test of my own didn't confirm the performance advantage. I just noticed that @CoderMing was doing .slice(0) instead of just .slice() in their microbenchmark, which might have disabled an optimization in the JS engine.
Closing this until somebody offers stronger evidence that there is an actual performance benefit.


Add shading from rasterized elevation data #5

@khdlr this is not really finished. But now the two branches for diffuse radiation and elevation need to work on the same code (as the shading involves both), so I think it is better to merge this one now to main to limit the amount of merge conflicts.


fix(qt): allow refreshing wallet data without crashing

Issue being fixed or feature implemented
We re-use refreshWallet method to optimise loading for huge wallets https://github.com/dashpay/dash/pull/5453.
However gui121 backport (via 25f87b9434b98a524d38a97d9fe580acc0fa47ce) added a condition that prevents this logic. Fix it by allowing explicit wallet refresh (force=true).
What was done?
How Has This Been Tested?
Open a wallet with 10k+ txes, do rescanblockchain via rpc console
Breaking Changes
Checklist:

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation
[ ] I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

Updated PR description with steps to reproduce the issue

Open a wallet with 10k+ txes, do rescanblockchain via rpc console

Reproduced crash again without this patch, then with it works :)


document 'list' commands

tabcmd list [projects|workbooks|datasources|flows] --details
-d, --details: print the generic item representation. If listing workbooks, also list all views in the workbook. (This can make a very long list).
Also, TODO

add a count of items at the end of the list?
#84

Add some more detail to close this out https://github.com/tableau/tabcmd/pull/295


fix: importing manim should not trigger pygments.styles.get_all_styles

Overview: What does this pull request change?

Removed the Code.styles_list attribute.
Added new class method Code.get_styles_list().
Rewrote the documentation to say that a list of all styles can be generated by calling a new class method.
The example in the docstring of Code was rewritten to use an explicit code style name.

Motivation and Explanation: Why and how do your changes improve the library?

See https://github.com/ManimCommunity/manim/issues/3713 for issue.
Links to added or changed documentation pages

https://manimce--3797.org.readthedocs.build/en/3797/
Further Information and Comments

First contribution!

Reviewer Checklist

[ ] The PR title is descriptive enough for the changelog, and the PR is labeled correctly
[ ] If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
[ ] If applicable: newly added functions and classes are tested

Hey, I'll take a look at this within the next few days, as I'm a bit busy right now. Just wanted to let you know that we see this PR and intend to review it :)


Logging

I found some code that seems not right to me, and would lead to no logging. If the log severity is Information then LogDebug should be ignored. In case this message should only be logged when the severity is LogLevel.Information it should be just _logger.LogInformation("my message").
                if(_logger.IsEnabled(LogLevel.Information))                 {                     _logger.LogDebug($"Requested challenge request for {token}");                 }
Here more about logging:
https://docs.microsoft.com/en-us/aspnet/core/fundamentals/logging/?view=aspnetcore-2.2
Thanks for the library !

You're right. This check in unnecessary as the logging framework can takes care of the proper level.
Just pushed a fix for this.

Updated the nuget package to version 1.5 and included this fix.


Optional options

Currently, ts-rs translates foo: Option<T> to foo: T | null in TypeScript, but a common pattern is to use #[serde(skip_serializing_if="Option::is_none")] for types that are represented in TypeScript like this: foo?: T. It's a rather specific feature, but one that is very valuable for our usecase.
Note: This PR includes my other PRs, as I want to save myself the merge conflicts, so you may want to just look at the last commit. If necessary, I can split them out later.

Closing in favor of #19, which only contains the relevant commits.


Improvements to emacs inlay hints

only send request if workspace is initialized (emacs-lsp doesn't seem to
prevent sending requests before the initialized notification is sent)
check whether we're still in the correct buffer before sending request

bors r+


Error on clap version when trying to install cli

Looks like a great tool.  I can go in and fix this by grabbing the source and changing it but thought I'd just note it here.  Let me know if you want me to try to create a PR (a bit new at all this).
when trying to use cargo install json_typegen_cli
>cargo install json_typegen_cli
    Updating crates.io index
  Downloaded json_typegen_cli v0.4.0
  Downloaded 1 crate (14.9 KB) in 0.89s
  Installing json_typegen_cli v0.4.0
error: failed to compile `json_typegen_cli v0.4.0`, intermediate artifacts can be found at `C:\Users\jas\AppData\Local\Temp\cargo-installRFMGPW`

Caused by:
  failed to select a version for the requirement `clap = "~2.19.0"`
  candidate versions found which didn't match: 2.33.3, 2.33.2, 2.33.1, ...
  location searched: crates.io index
required by package `json_typegen_cli v0.4.0`

Thank you for the report. I had not noticed that the clap version had been yanked. Will fix as soon as possible. If you want to try it in the meantime there are precompiled binaries under releases ( https://github.com/evestera/json_typegen/releases ).

Fixed with a21d643 / release v0.4.1.
Thank you again for the report, and let me know if you have any further issues. 🙂


Generate zod schemas instead?

As the title says, who would be interested in generating zod schemas instead of typescript? What I see is often we have to manually create zod schemas for input validation before it gets passed to kysely, so if we could automatically keep our zod schemas in sync with the database that would be ideal!
Just curious if anyone else is looking into this?

it would be nice and will save lots of effort.

I just happened to find this repo: https://github.com/maktouch/kysely-zod-codegen

I just happened to find this repo: https://github.com/maktouch/kysely-zod-codegen

@elitan Oh wow this looks great thanks for the link. Would make sense to merge this upstream, perhaps with an option to toggle it on/off.

Is @maktouch interested in opening a PR? Great work!

I'd like that! Wasn't sure if it would be accepted so I forked it cause I needed it right away.
There's a few downsides to it, it seems like I need to also generate using non-zod types for it to work correctly with kysely. It seems to work well for selects, but craps out for updates and inserts and I'm not too sure why.

I was just looking for this exact thing and came across @maktouch 's fork before this thread. I'd love to see something similar land here (or to bring that solution upstream)
@maktouch do you have any examples or reproductions of those issues?

@jjdonov hey, thanks for letting me know. I've enabled issues on the repo. I'll try to add reproduction for the non working inserts and updates.

Your solution should generate three shapes for each entity, insertable/updatable/selectable

Yeah I think that's the issue --- I needed it pretty quickly so right now I generate both from kysely-codegen and kysely-zod-codegen. Shitty hack, and I hope to find time to actually generate the three shapes for each entity.

Can I help with it?

I'd like that! Wasn't sure if it would be accepted so I forked it cause I needed it right away.
There's a few downsides to it, it seems like I need to also generate using non-zod types for it to work correctly with kysely. It seems to work well for selects, but craps out for updates and inserts and I'm not too sure why.

FYI I have been writing zod types for Kysely and I think with smt like this in the @maktouch generator PR it could be solved
import { Insertable, Selectable, Updateable, Kysely } from 'kysely';
import { z } from 'zod';

// have not test @maktouch gen but assume this is done from DB Schema
export const fooSchema = z.object({
  foo: z.string(),
});

// Then add this to generator
export type FooTable = z.infer<typeof fooSchema>;
export type Foo = Selectable<FooTable>;
export type FooCreate = Insertable<FooTable>;
export type FooUpdate = Updateable<FooTable>;

// use it here
export type DB = {
  foo: Foo;
};

export const db = new Kysely<DB>({
  //...
});

Happy to help, what needs to be done.
@maktouch @RobinBlomberg great work.

I've been using my fork for almost a year now for founderpath.com
Works great. It generates a giant file though. It generates both the typescript one and the Zod ones, and they have the same name.
I'm shy to promote it because it's quite different, and I killed all the tests 😅
Please do feel free to take some logic from my fork to put it back here.

@RobinBlomberg and others.
I have a working(some test failing) version of @maktouch fork.
What would be an acceptable PR to land here.
Gen zod types based on a config flag OR always zod and TS types(no flag) ?
lmk 👍🏼

imho, zod types should be behind a flag.

In the meantime, I wrote a janky script to generate zod types from the output of kysely-codegen.  I tested it successfully against all snapshots.

Script
Nothing is more permanent than a temporary solution 😉
import { generate as generateZod } from "ts-to-zod";
import prettier, { Options as PrettierOptions } from "prettier";

const defaultBanner = `/**
 * This file was generated by kysely-codegen and ts-to-zod
 * Manual edits will be overwritten.
 */

`;

export type GenerateOptions = {
  banner?: string;
  footer?: string;
  prettierOptions?: PrettierOptions;
};

export async function generate(
  src: string,
  {
    banner = defaultBanner,
    footer = "",
    prettierOptions = {},
  }: GenerateOptions = {},
): Promise<string> {
  let hasPostgresInterval = src.includes("IPostgresInterval");

  // multiline replacement regex https://stackoverflow.com/a/45981809

  // remove comment
  src = src.replace(
    /\/\*\*.* This file was generated by kysely-codegen.*?\*\//s,
    "",
  );
  // remove unneeded imports
  src = src.replaceAll(/import.*?;/gs, "");
  if (hasPostgresInterval) {
    // ts-to-zod is unable to parse IPostgresInterval from import
    // reference node_modules/postgres-interval/index.d.ts
    src =
      `export interface IPostgresInterval {
  years: number;
  months: number;
  days: number;
  hours: number;
  minutes: number;
  seconds: number;
  milliseconds: number;
  // toPostgres(): string;
  // toISO(): string;
  // toISOString(): string;
  // toISOStringShort(): string;
}` +
      "\n" +
      src;
  }
  // remove Generated type
  src = src.replace(/export type Generated.*?;/s, "");
  src = src.replaceAll(
    /: Generated<(.*?)>/g,
    (match) => "?: " + match.slice(": Generated<".length, -">".length),
  );
  // remove array types
  src = src.replace(/export type ArrayType<T>.*?;/s, "");
  src = src.replace(/export type ArrayTypeImpl<T>.*?;/s, "");
  src = src.replaceAll(
    /ArrayType<(.*?)>/g,
    (match) => match.slice("ArrayType<".length, -">".length) + "[]",
  );
  // remove json column type
  src = src.replaceAll(/JSONColumnType<(.*?)>/g, (match) =>
    match.slice("JSONColumnType<".length, -">".length),
  );
  // remove DB export
  src = src.replace(/export interface DB {.*?}/s, "");

  // remove and simplify ColumnType
  const columnTypeRegex = /export type (.*?) = ColumnType<(.*?)>;/g;
  const matches = [...(src.matchAll(columnTypeRegex) ?? [])];
  for (const match of matches) {
    const original = match[0];
    const typeName = match[1];
    const types = match[2];
    const reducedTypes = [...new Set(types.split(/ \| |, /))];
    src = src.replace(
      original,
      `export type ${typeName} = ${reducedTypes.join(" | ")};`,
    );
  }

  // zod programmatic api https://github.com/fabien0102/ts-to-zod/blob/main/src/core/generate.test.ts
  const { getZodSchemasFile, errors } = generateZod({
    sourceText: src,
    skipParseJSDoc: true,
  });

  // console log and throw errors, if any
  if (errors.length > 0) {
    for (const error of errors) {
      console.error(error);
    }
    throw new Error(`ts-to-zod generate failed`);
  }

  // generate zod types
  let schemas = getZodSchemasFile("./database-types.ts");
  // remove unneeded imports
  schemas = schemas.replaceAll(/import.*?;/gs, "");
  // add back zod import
  schemas = `import { z } from "zod";` + "\n\n" + src + schemas;
  // remove comment
  schemas = schemas.replace("// Generated by ts-to-zod", "");
  // concatenate types and schemas
  schemas = banner + schemas + footer;
  // format result
  schemas = await prettier.format(schemas, {
    ...prettierOptions,
    parser: "typescript",
  });

  return schemas;
}

I recommend integrating the script with kysely-codegen‘s programmatic api to do everything all in one go.

maybe relevant:
https://github.com/fabien0102/ts-to-zod


libEGL warning: failed to open /dev/dri/renderD128: Permission denied

The following warnings appear when running mujoco environments on a headless machine using egl.
I've set the MUJOCO_GL environment variable to egl using the export MUJOCO_GL=egl command.

libEGL warning: failed to open /dev/dri/renderD129: Permission denied
libEGL warning: failed to open /dev/dri/renderD129: Permission denied
libEGL warning: failed to open /dev/dri/renderD128: Permission denied
libEGL warning: failed to open /dev/dri/renderD128: Permission denied

As they are warnings not errors, the program terminates normally, but it feels a bit slow.
#209 seems to have solved a similar problem by using the nvidia/cudagl image,
but I'm using a real machine not a virtual one, so the solution cannot be applied directly.
Any help would be greatly appreciated.

If you have full control (i.e. root privileges) on this machine, it might simply be a matter of correcting the permissions on /dev/dri/renderD129 (usually by adding your user to the group that owns the device).

If you have full control (i.e. root privileges) on this machine, it might simply be a matter of correcting the permissions on /dev/dri/renderD129 (usually by adding your user to the group that owns the device).

After I add myself to the group, the warning message has changed to

libEGL warning: DRI2: failed to create dri screen
libEGL warning: DRI2: failed to create dri screen
libEGL warning: DRI2: failed to create dri screen
libEGL warning: DRI2: failed to create dri screen

I just checked that even with MUJOCO_GL set to egl, the following test code does not take up any GPU.
from dm_control import suite
env = suite.load('cartpole', 'swingup')
for _ in range(100000):
    env.physics.render()

The same code utilizes about 60 MiB of GPU memory on my PC that has a physical display.
The graphics driver I installed on my machine is nvidia-headless-470-server. Can this be the cause?

Sorry for the long silence, been working on getting the MuJoCo 2.1.1 release out of the door. Are you still looking for help on this? If you are, can you give a bit more detail on your system setup?

No problem. I've attached a list of packages installed on that machine. What kind of details do you need?
installed_packages.txt

It looks like you don't have any Nvidia graphics driver installed at all, only compute ones. I think if you can install https://packages.ubuntu.com/focal/nvidia-driver-470-server that might solve it.

Looks like https://packages.ubuntu.com/focal/libnvidia-gl-470-server is actually the minimal thing that's required.

libnvidia-gl-470-server works like magic! Thanks a lot.

libnvidia-gl-470-server works like magic


drivers: bt_nxp: delete non monolithic support

Current code does no longer support the non monolithic feature.
Hence, removing unnecessary code.

@axelnxp, please check

should the CONFIG_NXP_MONOLITHIC_BT just be deleted then?

should the CONFIG_NXP_MONOLITHIC_BT just be deleted then?

Agree. Zephyr platforms now do not support non-monolithic WiFi and BT.
There is no point in having user configuration parameters that cannot be changed.

should the CONFIG_NXP_MONOLITHIC_BT just be deleted then?

Agree. Zephyr platforms now do not support non-monolithic WiFi and BT. There is no point in having user configuration parameters that cannot be changed.

This Kconfig is still usefull, and it's used for the following cases:

A soc could enable this nxp KConfig to mention the support of this feature. For example RW61x soc supports this and not MCXW71.
To have some CI coverage for BLE on RW61x. CI on Zephyr doesn't fetch the blobs and the monolithic feature needs them. Not fetching the nxp blobs will lead to build failures for BLE apps. This is why we use CONFIG_NXP_MONOLITHIC_BT=n on the sample.yaml (used by twister) so that the build doesn't try to link the binary blob on the final image.

@axelnxp what is your POV on this ?

should the CONFIG_NXP_MONOLITHIC_BT just be deleted then?

Agree. Zephyr platforms now do not support non-monolithic WiFi and BT. There is no point in having user configuration parameters that cannot be changed.

This Kconfig is still usefull for the following reasons:
* A soc could enable this nxp KConfig to mention the support of this feature. For example RW61x soc supports this and not MCXW71.

* To have some CI coverage for BLE on RW61x. CI on Zephyr doesn't fetch the blobs and the monolithic feature needs them. Not fetching the nxp blobs will lead to build failures for BLE apps. This is why we use CONFIG_NXP_MONOLITHIC_BT=n on the sample.yaml (used by twister) so that the build doesn't try to link the binary blob on the final image.

@axelnxp what is your POV on this ?

I agree, especially for the second point.
@butok @decsny this PR mainly aims to remove selecting DT partition when the monolithic feature is not enabled, this behavior doesn't have sense on platforms like MCXW71. MCXW71 doesn't support the monolithic feature, and we don't want to use DT partition either, because it forces the use of mcuboot alongside since @decsny changes enabling mcuboot support on MCXW71. Or we would need overlays everywhere, and that's not what we want to do.
So basically, this select is not relevant anymore, but the monolithic kconfig still is.

should the CONFIG_NXP_MONOLITHIC_BT just be deleted then?

Agree. Zephyr platforms now do not support non-monolithic WiFi and BT. There is no point in having user configuration parameters that cannot be changed.

This Kconfig is still usefull for the following reasons:
* A soc could enable this nxp KConfig to mention the support of this feature. For example RW61x soc supports this and not MCXW71.

* To have some CI coverage for BLE on RW61x. CI on Zephyr doesn't fetch the blobs and the monolithic feature needs them. Not fetching the nxp blobs will lead to build failures for BLE apps. This is why we use CONFIG_NXP_MONOLITHIC_BT=n on the sample.yaml (used by twister) so that the build doesn't try to link the binary blob on the final image.

@axelnxp what is your POV on this ?

in this case the following things appear to me to need to change if I understand you correctly:

the samples for RW which set this kconfig to n should be set to build only since ci will not flash the radio binary, it will not actually run
the description of the kconfig is now wrong if its being used to indicate support instead of configure something like youre saying above. which, if it's not, then why are we changing it from default y to explicit n in tests? it seems like it is for configuring still, but you just make it not work when set to n, >>
probably the appropriate thing to do is to make a new kconfig to indicate support set by soc, and then that kconfig is a dependency of this one
the line you are deleting should probably actually be moved to rw soc kconfig

@decsny,
Updated, does it look good for you now?


Implement slab level cache for remote frees

@nwf-msr observed that we could improve the performance of remote deallocation if the producer does more work on building lists for each slab before returning to the original allocator.  This could improve producer/consumer scenarios further.
### Tasks
- [ ] Implementing statistics to estimate performance wins.
- [ ] Implement version that doesn't work with snmalloc hardening.
- [ ] Implement version with hardening as well

@licenser @darach are you still using snmalloc. If so do you have any benchmarks that represent your workload?  We have some ideas that would benefit your kind of workload for Tremor.

Hi Matt,
We sure are. In tremor, and I have a project at work ( axiom now ) where I
think it's a great fit.
We'll ask the community as well
Cheers,
Darach.
On Thu, Sep 14, 2023, 22:15 Matthew Parkinson @.***>
wrote:

@Licenser https://github.com/Licenser @darach
https://github.com/darach are you still using snmalloc. If so do you
have any benchmarks that represent your workload? We have some ideas that
would benefit your kind of workload for Tremor.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/snmalloc/issues/634#issuecomment-1720082479,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABX5MW5MOJBESOQQAOE7FLX2NQVNANCNFSM6AAAAAA4YXRFRI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Awesome thanks. Any benchmarks we can run would really help us in justifying the engineering work.

Our CI benchmarks are snmalloc based https://www.tremor.rs/benchmarks/ - relatively boring UI there.
The benchmarks we ran a year or two ago for tremor ( and mimalloc before it, and jemalloc before that ) are here:
https://github.com/tremor-rs/tremor-runtime/blob/main/bench/README.md
They have changed enough though in themselves and we rewrote the runtime and connectors so the benchmarking
code works differently so YMMV

I've spent a while playing with various cache strategies (though nothing very sophisticated, just sort of seeing what the low-hanging fruit was like).  For the two workloads I've tried, for three interesting choices of hashes, I currently have these message counts:

Workload
No caching
Perfect assembly
4-way direct hash

msgpass
7,205,380
552,781; 35 rings
1,076,843

xmalloc-test
2,388,551
317,057; 5 rings
317,065

The hash here is inspired by https://github.com/skeeto/hash-prospector but is sort of "a third" of that:
  hash = slab;
  hash *= 0x7feb352d;
  hash ^= hash >> 16;
  return (hash & 3)

This is now completed.  #677 landed in main, and the paper appeared at ISMM'24.


Plugin not working on .TS config file

Issue workflow progress

Progress of the issue based on the
Contributor Workflow

[ ] 1. The issue provides a reproduction available on
Github,
Stackblitz
or
CodeSandbox

Make sure to fork this template and run yarn generate in the terminal.
Please make sure Mesh package versions under package.json matches yours.

[ ] 2. A failing test has been provided
[ ] 3. A local solution has been provided
[ ] 4. A pull request is pending review

Describe the bug
Using a .ts config file instead of YAML caused issues for me. When I try to load certain plugins, they don't work, while the same config in the YAML file does.
For example, in the case of Prometheus, my YAML config file looks like this:
plugins:
  - prometheus:
      endpoint: "/metrics"

With this configuration, I can open the /metrics address and see it working. However, in the TypeScript config file:
plugins: [{
      prometheus: {
        endpoint: "/metrics",
      },
}]

the above code doesn't work.
I have the same problem with Auth0, which leads me to use the "useAuth0" enveloped plugin instead. The same issue occurs with blockFieldSuggestions and csrfPrevention.
The reason I'm sure the plugins don't even get called is that when I add these plugins to my config file without having the package installed, it doesn't give me an error. In contrast, the YAML config file throws an error if the plugin is not installed.

To Reproduce Steps to reproduce the behavior:
To reproduce the issue, create two config files: .meshrc.yaml and .meshrc.ts.
Then, add this to your YAML file:
plugins:
  - prometheus:
      endpoint: "/metrics"

Start the server, and you should get an error that the plugin is not installed.
Now, do the same with the TypeScript config file and add this to your config without installing the plugin:
plugins: [{
      prometheus: {
        endpoint: "/metrics",
      },
}]

The server will start.

Expected behavior
The expected behavior is to get the same error if the plugin is not installed. Currently, not throwing an error indicates that the plugin doesn't work in the TypeScript config.

Environment:

@graphql-mesh/graphql: 0.98.10
@graphql-mesh/cli: 0.91.1
NodeJS: 20

Additional context

Could you create a reproduction on CodeSandbox? So we can help you better! Thanks!

I just found the problem.
This is how I was adding my plugins in the TypeScript configuration:
plugins: [{
    auth0: {...},
    prometheus: {...},
    // other plugins
}]

However, the correct way is to put each plugin in a separate item of the array:
plugins: [
  {
    auth0: {...},
  },
  {
    prometheus: {...},
  },
  // other plugins
]

I think adding TypeScript examples for configurations alongside YAML will help avoid this confusion.


📖  Remove BMO vars from docs

There are still some leftover documentation which was needed when BMO was part of CAPM3. These are not needed anymore and misleading also perhaps.

But I'm afraid tide is not going to merge it. Let's see :)

Just a wild guess, maybe tide pod has to be re-deployed in prow cluster?


OLS-147: Create a skeleton OLS operator

Description
Added initial version of operator.
Type of change

[ ] Refactor
[x] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library

Related Tickets & Documents

Closes https://issues.redhat.com/browse/OLS-147

Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

/lgtm

/approve


Sufficient data for estimation

The inner join between genotypes and phenotypes can remove all occurences of a given genotype or phenotype. This will still produce a result at the moment and shouldn't. Needs to be invetigated.

Has not been raised again. closing


Support ONNX export on torch.float16 device

As per title.
Test still missing. Partly fixes https://discuss.huggingface.co/t/convert-gpt-j-to-fp-16-onnx/30294

is the torch onnx export on fp16 stable

I've tried on a single model - and did not try to load into an InferenceSession. I will add a test for it, thanks.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Added Netgate pfSense Plus to VPN Gateway list

Added Netgate pfSense Plus to VPN Gateway list. I'm the product manager at Netgate (apaxson@netgate.com). Feel free to reach out to me if you have questions.

@TazzzRed : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver @yushwang - have you had a chance to look at this yet?

++ @Mario @.***> - Hi Mario,
Did you work on validation of pfSense Plus for the 3rd party VPN device?
Context - we don't just accept PR from the partners, but need to go through a set of validation steps which Mario owns before we add a vendor/model to the table.
Thanks,
Yushun
From: Audian Paxson @.>
Sent: Thursday, April 15, 2021 3:29 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Yu-Shun Wang @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Added Netgate pfSense Plus to VPN Gateway list (#72700)
@ktoliverhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fktoliver&data=04|01|yu-shun.wang%40microsoft.com|4463127d9e4143e5488808d9005deefc|72f988bf86f141af91ab2d7cd011db47|1|0|637541225702632633|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=7wfkZ44tyAJTmpH1aknU3fco8177506Y1sqk0FQKOAI%3D&reserved=0 @yushwanghttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fyushwang&data=04|01|yu-shun.wang%40microsoft.com|4463127d9e4143e5488808d9005deefc|72f988bf86f141af91ab2d7cd011db47|1|0|637541225702642631|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=arVfYRnz%2B2n%2BvhpiPEjvFLda6lkr24%2BN3NZLmiSvnTw%3D&reserved=0 - have you had a chance to look at this yet?

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F72700%23issuecomment-820768050&data=04|01|yu-shun.wang%40microsoft.com|4463127d9e4143e5488808d9005deefc|72f988bf86f141af91ab2d7cd011db47|1|0|637541225702642631|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=XX2CUUtCDD8Ki0sFtr%2Bo0LKkbe5P3ezF6Q%2FPEJlhmk8%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACJB5C7TZCFLMMAN3SNP5S3TI5SERANCNFSM4ZXTYJCA&data=04|01|yu-shun.wang%40microsoft.com|4463127d9e4143e5488808d9005deefc|72f988bf86f141af91ab2d7cd011db47|1|0|637541225702652622|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=0%2B3QhSXyv0wJBnEBUaPD1wtLDfWTm13qbnhLAcilcpk%3D&reserved=0.

This username is awesome, but I do not own the validation steps in this case :P

@yushwang Could you check on Mario's GitHub username?

I don't think he is active on GitHub. I'll check the status offline on what we do with the pfSense.
Thanks,
Yushun
From: Kristine Toliver @.>
Sent: Thursday, April 15, 2021 3:59 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Yu-Shun Wang @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Added Netgate pfSense Plus to VPN Gateway list (#72700)
@yushwanghttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fyushwang&data=04|01|yu-shun.wang%40microsoft.com|d34b948bceb747d6ba1b08d900621280|72f988bf86f141af91ab2d7cd011db47|1|0|637541243481335547|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=1fOfSZjcHWeeI6Jo3V72pUbXhiSP2raCZBK%2FukNEQuc%3D&reserved=0 Could you check on Mario's GitHub username?

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F72700%23issuecomment-820783069&data=04|01|yu-shun.wang%40microsoft.com|d34b948bceb747d6ba1b08d900621280|72f988bf86f141af91ab2d7cd011db47|1|0|637541243481335547|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=ohWcM1aeH8FVFzw0oPThX71vwIwRbFI845X209u2s3w%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACJB5C3TKXQ5YSDJGUXM2BLTI5VTVANCNFSM4ZXTYJCA&data=04|01|yu-shun.wang%40microsoft.com|d34b948bceb747d6ba1b08d900621280|72f988bf86f141af91ab2d7cd011db47|1|0|637541243481345542|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=H1kbnTj5BLkO1niEeR5WkfS4I%2BayFoND5zJSMj8TB2M%3D&reserved=0.

#please-close
We will need to validate the connectivity on our backend before adding any new vendor/device to the list. I have reached out to @TazzzRed via email to start the process.


oc new-app detection for Python

I realise this possibly needs to be a bug report against 'oc' in 'openshift/origin' repo, but wanted to raise it here first in case there is something I am missing.
In:

https://github.com/openshift/origin/blob/b7de777c90fd892cd746c905c17d92710eae3de8/pkg/generate/source/detector.go#L69

The code for detecting Python when using 'oc new-app' against a repository, and which thus then triggers S2I using the images from this repository is:
// DetectPython detects Python source
func DetectPython(dir string) (*Info, bool) {
	return detect("python", dir, "requirements.txt", "config.py")
}

The problem is why is it checking for config.py?
I cannot think of any valid reason why it would look for config.py as I know of no convention in Python where a file by that name is anything significant.
I suspect it should be looking for setup.py.
That would make more sense given that assemble only treats requirements.txt and setup.py as anything special for triggering Python package installation.
if [[ -f requirements.txt ]]; then
  echo "---> Installing dependencies ..."
  pip install --user -r requirements.txt
fi

if [[ -f setup.py ]]; then
  echo "---> Installing application ..."
  python setup.py develop --user
fi

Comments?

@rhcarvalho I have a sense you already fixed this in new-app, right?

(confirmed. was fixed here: https://github.com/openshift/origin/pull/6587/files)


[feature request] easy import of DESCRIPTOR, Node

Since it is hard to get DESCRIPTOR and Node modules, can we add it to bblfsh directly?
I suggest adding to __main__.py something like this:
import importlib

DESCRIPTOR = importlib.import_module(
        "bblfsh.gopkg.in.bblfsh.sdk.%s.uast.generated_pb2" % bblfsh.sdkversion.VERSION).DESCRIPTOR
Node = importlib.import_module(
        "bblfsh.gopkg.in.bblfsh.sdk.%s.uast.generated_pb2" % bblfsh.sdkversion.VERSION).Node

Then everyone can just do from bblfsh import Node, DESCRIPTOR.
Can make a PR if you approve.

We have it as ast2vec part: https://github.com/src-d/ast2vec/blob/master/ast2vec/bblfsh_roles.py but I think it is more bblfsh related things.

Yes, that's ugly as Black Death, the init.py file should import all those.

@juanjux btw, can we just somehow avoid in in the directory path? It also solves this problem.

I tried several alternatives, using symlinks, renaming the in to in_ in the protofiles... this one is the one that causes less pain and should not matter much if we import everything from __init__.py.

Implemented in 2.6.0.


Site with documentation doesn't work

Site with documentation doesn't work http://lafikl.github.io/perfBar/

Sorry, it's fixed now.
http://wpotools.github.io/perfBar/


wrong year on latest updates

Almost made me not want to use because it didn't seem to be an active project.  I was looking in craft plugin store when I noticed.

Hah, thanks.


Way to get onChange of terminal?

I am scratching my head on this one, I don't want to get the full command on enter, I instead want to bind to every change that occurs when typing in the terminal (we use a fully simulated pty backend, so we must send a character at a time).
It is not clear if this is possible in the current release, am I missing something?

Apologies, wrong project, this is what having many tabs open does.


defineModel不会生成相应的d.ts,vscode无代码提示

Describe the bug
 let { visible } = defineModel<{
    visible?: boolean;
  }>();

 const props = defineProps<{
    visible?: boolean;
  }>();

  const emit = defineEmits<{
    (event: 'update:visible', data?: boolean): void;
  }>();

Reproduction
none
System Info
System:
    OS: macOS 12.4
    Memory: 441.52 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 17.5.0 - /usr/local/bin/node
    Yarn: 1.22.18 - /usr/local/bin/yarn
    npm: 8.8.0 - /usr/local/bin/npm
  Browsers:
    Chrome: 104.0.5112.79
    Safari: 15.5

Used Package Manager
yarn
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Yeah, it hasn't been implemented yet.

Waiting Volar plugin API to be more powerful...


a11y:fix(bal-radio, bal-checkbox, bal-select): Key Board Navigation

Current Behavior
Selects, Radios, Checkboxes and Radio and Checkbox Groups need to be checked and refactored for keyboard usage as described here and here.
Check focus state as well.
Expected Behavior
see above
Steps to Reproduce
regular usage and keyboard navigation
Version
13.4.4
Browsers
No response
Device
No response
Code Reproduction URL
No response
Additional Information
No response
Code of Conduct

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

@GinaBiondo-aperto can you tells a bit more what accessibility features are requested?

Check focus with tab on safari

Hi @GinaBiondo-aperto
I tried to follow the conversation but I am not fully sure, what we need to adapt.
I listed the keyboard actions from the documentation and kindly ask you to confirm, what is requested.
List:

[ ] Down Arrow : Moves focus to the next option.
[ ] Up Arrow : Moves focus to the previous option.
Multiple Selection:
[ ] Space : changes the selection state of the focused option

Checkbox:

[ ] Space:	Activates the checkbox
[ ] anything else, which is not mentioned in the documentation?

*Radiobutton:

[ ] Tab and Shift + Tab: Move focus into and out of the radiogroup. When focus moves into a radiogroup, if a radio button is checked, focus is set on the checked button. If none of the radio buttons are checked, focus is set on the first radio button in the group.
[ ] Space: Checks the focused radio button if it is not already checked.
[ ] Right Arrow and Down Arrow: Moves focus to the next radio button in the group, unchecking the previously focused button, and checking the newly focused button. If focus is on the last button, focus moves to the first button.
[ ] Left Arrow and Up Arrow: Moves focus to the previous radio button in the group, unchecking the previously focused button, and checking the newly focused button. If focus is on the first button, focus moves to the last button.

I am having an additional qustion regarding the usage of the arrows. At the moment we are scrolling the page when using the up and down arrows. Will the current behaviour be obsolated or do I miss a something?
Thank you for your feedback!
Magdalena

Hi @mmarinkov,
Your listed behaviors are correct. I would like to add something here:
Listbox (select):
Focus state => If nothing is selected when opening this listbox, the first option is focused. If there are selected options, the first selected option is focused when opening.
Checkbox:
No special behavior here. The user navigates with tab key and select with space.
To your question about scrolling behavior: Scrolling with up and down keys is still possible. It should only be disabled if I am in a list box or in a radio group.
You can see the correct behavior in the examples of the native elements.
Select does not scroll when the listbox is open because I can only operate the overlay and not the content below it: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/select
With radio buttons, the page scrolls when I am in the first element and press the up key or in the last element and press the down key: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/radio
I hope I was able to answer your questions. If you have any further questions, please feel free to contact me.
Best regards,
Caro

Will be done it those issues

#1446
#1447


Unsupported OS: freebsd

Sorry if this not related to this repo, but I just can't figure out where to dig further.
$ ./gradlew npmInstall

FAILURE: Build failed with an exception.

* What went wrong:
Could not determine the dependencies of task ':nodeSetup'.
> Unsupported OS: freebsd

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Total time: 4.971 secs

My build.gradle is:
buildscript {
    repositories {
        mavenLocal()
        jcenter()
    }
    dependencies {
        classpath('org.springframework.boot:spring-boot-gradle-plugin:1.2.0.RELEASE')
        classpath('com.moowork.gradle:gradle-node-plugin:0.8')
        classpath('com.moowork.gradle:gradle-grunt-plugin:0.6')
        classpath('be.filipblondeel.gradle:gradle-gulp-plugin:0.1')
    }
}

Os is FreeBSD 9.

Thanks for reporting this. Yes, it's the OS check. Will try to fix this.

I have no FreeBSD to test on, but it should now detect FreeBSD as a linux variant and install Node for binaries for Linux. Please reopen an issue if this does not fix it.

FreeBSD is not linux, installing linux binaries won't work…


[Bug]: Failed to parse document,[ERROR]Internal server error: not enough values to unpack (expected 2, got 1)

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
main
Commit ID
0
Other environment information
No response
Actual behavior
Embedding model:text-embedding-v2
Chunk method:Book
When I created the knowledge base and configured it as follows, after the configuration was completed, I uploaded a docx file to the knowledge base and encountered the following error message: [ERROR] Internal server error: not enough values to unpack (expected 2, got 1).

log info :
Error -2 connecting to redis:6379. Name or service not known.
Traceback (most recent call last):
File "/ragflow/rag/svr/task_executor.py", line 137, in build
cks = chunker.chunk(row["name"], binary=binary, from_page=row["from_page"],
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/ragflow/rag/app/book.py", line 139, in chunk
res = tokenize_table(tbls, doc, eng)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/ragflow/rag/nlp/init.py", line 138, in tokenize_table
for (img, rows), poss in tbls:
^^^^^^^^^^^^^^^^^
ValueError: not enough values to unpack (expected 2, got 1)
Expected behavior
No response
Steps to reproduce
Embedding model:text-embedding-v2
Chunk method:Book

When I created the knowledge base and configured it as follows, after the configuration was completed, I uploaded a docx file to the knowledge base and encountered the following error message: [ERROR] Internal server error: not enough values to unpack (expected 2, got 1).

Additional information
No response

log info :
Error -2 connecting to redis:6379. Name or service not known.
Traceback (most recent call last):
File "/ragflow/rag/svr/task_executor.py", line 137, in build
cks = chunker.chunk(row["name"], binary=binary, from_page=row["from_page"],
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/ragflow/rag/app/book.py", line 139, in chunk
res = tokenize_table(tbls, doc, eng)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/ragflow/rag/nlp/init.py", line 138, in tokenize_table
for (img, rows), poss in tbls:
^^^^^^^^^^^^^^^^^
ValueError: not enough values to unpack (expected 2, got 1)

fixed


improve the styling of the landing page .

by making changes in the css file

Improve the styling of the landing page /assign

please assign me under jwoc

Sorry for the very late reply have you worked on this already?

Yaa definately I am interested , thanks for assigning.

Hey! I noticed that the issue #167 is still open. I'd love to help out and contribute to the project.

I would like to work on this under JWOC


Merging my personal branch to official repository

Addition of two new action :

hls_vector_generator -> action that generates a vector in FPGA and writes it on HOST memory
hls_parallel_memcpy -> action that read(write) data from(to) HOST memory during multiple iterations

These actions were created to experiment FPGA <-> GPU memory exchanges on AC922 with CAPI2.0
Example usage of these actions with GPU can be found here:
FPGA+GPU example applications on AC922

Accepted this merge to test it as a branch of snap with jenkins


Clearly PromotedIsBig is not doing what the topics describes

As was already reported in #885 PromotedIsBig is clearly not doing what the topics describes. And even though the topic was updated (as mentioned in #885) the topic is actually nonsense.
First sentence in the topic:

Sets a value that indicates whether the promoted action is displayed on the ribbon with a large icon.

This is not the case at all.
Later with the example:

However, because the PromotedIsBig property is true, it will appear before the Sales Quote action in the client.

This is indeed the case and to me this seems to only thing PromotedIsBig is doing. So, let that be described in the first sentence of the topic and nothing else. Indeed, you might want to explainn that this worked differently on the Windows client.
BTW: what is this line in the example code actually telling:
PromotedCategory = Category5;  // PromotedActionCategories = New Document

Shouldn't PromotedCategory be set to New?

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

ID: 0865ca2c-f735-3e44-16da-a78607034dda
Version Independent ID: 46b76622-068b-c705-8e2b-346e7fda961e
Content: PromotedIsBig Property - Business Central
Content Source: dev-itpro/developer/properties/devenv-promotedisbig-property.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

We'll take a look.

Thanks for the feedback. We've updated the article. The update will be published soon.


Need settings to prohibit moving widgets with the mouse

Now it is too easy and make some disorder to the beginners - crashes ideal order done by developer

Done in 3.2.3

added support for dragging and resizing restrictions for widgets by URL parameters: nodrag=1 and noresize=1


UnicodeEncodeError while reading image data

Env:
Windows 10, Python 3.9.
Issue:
Got the UnicodeEncodeError while processing example tesseract image data with make run.
UnicodeEncodeError: 'charmap' codec can't encode character '\xfc' in position 57: character maps to <undefined>
Solution:
Specify utf-8 encoding while reading the data.
out = open(output_file, "w", encoding='utf-8')

Thanks for catching. See: https://github.com/ReceiptManager/receipt-parser-legacy/commit/4a0d275cbf2713dbe904e8ec294634c2c3eba0f2

Does it work now?
Regards,
William

Yes, it's good now. Thank you.


x/playground: deploy may take over 10 minute and timeout

I've ran into this when deploying the Go Playground with Go 1.11.1 recently (see CL 140097), where the cloud build failed because it took more than the default 10 minute timeout:
ERROR: (gcloud.app.deploy) Cloud build failed. Check logs at https://console.cloud.google.com Failure status: UNKNOWN: Error Response: [4] DEADLINE_EXCEEDED
In the recent past, some builds have taken around 9~ minutes. They were faster earlier on.
People have run into the 10 minute timeout in other contexts too. E.g., see https://stackoverflow.com/questions/50046545/google-app-engine-build-timed-out-during-deployment.
For now, this issue is to gather data and see if this keeps happening. We can then decide on how to best deal with it. If you run into this, please post data here. /cc @bradfitz @andybons
Workaround
The easiest temporary workaround I'm currently aware of is to increase the timeout before the gcloud app deploy step. For example:
gcloud config set app/cloud_build_timeout 900 # 15 mins

Note that it's a global config value.

I wonder if this build command includes building the docker container? I have noticed that building playground docker container became very slow. Specifically fetching packages is slow. Since I was not sure if it was my network, I did not create ticket for that.

I wonder if this build command includes building the docker container?

Yes, it does. That's the part that takes up to 8-9 minutes.

Specifically fetching packages is slow.

I didn't notice that to be the slow part, but thanks for pointing out. I'll watch it more closely next time.

A data point from today's deploy (for #25462): the cloud build step took 10 min 58 sec. (It was within my 15 minute timeout.)

Another data point from today's deploy: the cloud build step took 10 min 43 sec.

Two data points for the cloud build step from today's deploy (in #30397), using Go version 1.12:

12 min 4 sec
11 min 27 sec

By now, it's clear we are reliably beyond the 10 minute mark.

I'm going to send a fix for this issue by documenting the command to set app/cloud_build_timeout to a value higher than the default 10 minutes.
If we discover a better solution in the future, we can improve it then.

Is it possible it is a problem of app engine? https://groups.google.com/forum/#!topic/google-appengine-go/fBdzFHZ1D5U

This issue was related to the time it takes to build the Docker image using Cloud Build, which grew from 8~ minutes to 12~ minutes over time, and how it intersected with the default timeout of 10 minutes. The linked thread doesn't seem to be directly related.


initial attempt at planet data converter

This one needs a bit of help, but I think is pretty close.
I got it mostly working, though couldn't figure out why it was using 0.4.0 of the CLI, until I pushed the branch and realized I'd accidentally branched off of the Austria and Brazil converter branch that I was investigating. But I had managed to get it working locally.
Then I switched to 0.5.0, pleased to remove my dummy 'bbox' that I stuck in, but it's now stuck. I get:
% fiboa convert planet_afb -o ~/repos/data-fiboa/output/planet.parquet
fiboa CLI 0.5.0 - Convert 'planet_afb'

Getting file(s) if not cached yet
'dict' object has no attribute 'delete'

I couldn't find where the 'delete' was coming from, or where it should be. Any ideas?
This PR also raises some issues, I mostly highlighted as comments in the code, the main one being that this is not about converted a fixed, online dataset, but is meant to be a tool to converter any Planet field boundary output. So it'd ideally have like a -i flag to point at a file instead of a default online location, or even require an input argument for this conversion. So curious how to approach this. It will be similar for DigiFarm and OneSoil, two other converters I'd like to make, as those are the three main field boundary providers that I know of.

The delete issue has been fixed, I think. It only appears with Python 3.12 or later, which I don't have yet.
With the changes relating to extracting ZIP files, handling multiple input files, and caching, it's not overly easy anymore to pass in the source files easily. The -c option which we've used before now accepts a folder with files included and then the converter needs to specify the filenames in the folders. Also, a -i option could only handle the simple usecase of a non-extractable files.
Have a look at be_wa, which has also external data that you need to pass in from an external source. There's a DATA_ACCESS flag, which you can fill with instructions and then you can use -c and SOURCES in combination. Would that work for you?

Ok, I checked the improvements, they all look great. I updated the docs on this a bit, and also added determination method. So I think it's ready to merge.
thanks!


How can I test my own method on this benchmark ?

My method is based on gpt-3.5-turbo, and i wanna test it on this benchmark? Could you please instruct me how to do it. How to custom the evaluation process. Thanks a lot !

for example, my method can be used in the form of :
reasoner = CustomReasoner()
response = reasoner.reason(query)

How can it be tested in your benchmark, thanks

Hi,
Thank you for your interest in our work. Please save the responses to a file and then follow this. Before evaluation, ensure that you have processed these responses in JSON format and that the file format adheres to our guidelines.


docs: fix typo on deno import example

This PR change the example of deno usage.

@ghoullier Wow I really managed to screw this up 🤣 Thanks for the PR! Will add a second commit to append "/main.ts" so that it matches what's on arktype.io.


UniFi - Work around User input malformed issue with config flow

Description:
For some reason config flow doesn't work properly with integers, instead generating a "User input malformed" message first time trying to validate input.
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

This seems like a workaround for a frontend bug

@balloob Yes, some people can't use the component at all, where as others can do a work around by reinitiating the config flow.

If you accept this work around I will do the same fix for deCONZ as well.

So we should fix this in the frontend?

I'll create one

The real issue has been solved. No need to continue with this one.


Allow field names that start with numbers

Problem
Right now the field names can only contain numbers, but they may not start with them. This prevents from easily introspecting existing schemas where this is the case.
Suggested solution

Introspection policy/configuration: allow users to customize how to transform field names which aren't compatible into another compatible name
Upon introspection, prefix fields starting with a number with a standard character, e.g. "_"

Additional considerations
This will likely involve considerations in the Prisma Client API, since field name can't start with numbers.

Hi there,
As part of our effort to clean up our GitHub repo, we’re closing this issue to help focus on features and improvements that align with our focus areas and those of our community.
This issue was originally created for internal tracking purposes, but if you feel this is important to you, feel free to reopen the issue and get like-minded users to upvote as we will prioritize fixing issues based on community interest.
For more insight into our priorities and how we’re thinking about the future of Prisma, please see our blog post: https://www.prisma.io/blog/prisma-orm-manifesto.
Thank you for your understanding and support!


Feature Request: Profiles

Hi,
I know that this is a bigger thing ...
xMarks had this feature and I would like to request it for xBrowserSync, too.
It would be great to have profiles. Each bookmark (or folder) can be assigned to a profile. By default it is automatically assigned to the default profile. Let's assume that we further have a profile called "work" and one "home". When configuring the app or extension it should be possible to choose the desired profile - and only those bookmarks would be synced.
This would allow to have all bookmarks at one place but only synchronise (and display) the ones that you really need on a specific device.
I have bookmarks that I use only at work, others only at home and some at home and at work.
Also a profile for a mobile device would be helpful.
I hope my explanation is clear enough ...
Best regards, Tom

Hi @Metal-Frog, interesting suggestion. Just to be clear, are you saying only bookmarks in the selected profile would be created in the local browser but you would still see bookmarks from all profiles within the xBrowserSync app /extension?

Hi,
when you've selected the profile "home" you see only those from that profile. None from the others.
If you add a new bookmark it is visible in all profiles. If you would like to change this you have to call the GUI to edit the profiles.
The following two picture should help for better understanding:
https://static.makeuseof.com/wp-content/uploads/2014/07/LastPass-Chapter4-xmarks-profiles.jpg
https://static.makeuseof.com/wp-content/uploads/2014/07/LastPass-Chapter4-xmarks-profile-choose.jpg
Best regards, Tom

Right I see, thanks for the clarification. I think this would make a good feature for v2 once all the core syncing functionality has been implemented and major browsers supported.

Hi,
I would definitively love to have this in xbrowsersync !
Moving for the -now dead- xmarks, profiles is the only feature I really miss.
But obviously it makes sense to finish major browsers support first :-)

+1 on this feature.  Currently using "syncmarcx" until profiles are implemented.  There are lots of bookmark sync options, but the profile setting is what made xmarks unique and useful.
Looking forward to seeing it here!

+1 it would also help the user of Vivaldi, since your extensions create is own folder on a way Vivaldi's user don't see it, and if they use the main folder bar, Vivaldi and the extension go crazy when you try to erase something from the bookmarks.
Profile folder will help them to create a profile Vivaldi (the time the browser will be officially supported)

Has there been any movement on this?
I'm also looking forward to profiles. Currently I have two separate syncs; one for my home computers and one for my work computers. But there are many bookmarks that are in both (development reference, local news, etc). I'd love to have one sync that has some folders that always show up, some that only appear on work computers, and some that only appear at home.
This would also help accidentally creating duplicates. I regularly go through each sync to add and remove bookmarks in common folders. But it's possible that I add a site on my work computer and store it in one folder and then go home to bookmark it at home but put it in a different folder. So when I do my sync, I create extra links.

+1 from me for this feature.
It could be as simple as a search filter. i.e. from a user perspective:

Select folders to sync,
Assign profile name
Persist to server

+1 for this feature, IMHO this is an absolutely necessary feature for any bookmarks sync service


Recommended network_compression_threshold?

Hi,
not sure where to ask this, but what would be the recommended network_compression_threshold for velocity when behind tcpshield?

256 on velocity and -1 on your spigot instances seems to work best for most people. If you're bandwidth limited on your spigot machines, 256 on both is good enough.


How to generate a title for a given new text data？

After I trained a model, how can I generate a title for the text I entered? Because I saw that the data entered in the prediction in your code is a DF format containing the body and the title, but in practice, I only want to generate a title for the given text, so I don't need the title when predicting, then I What should I do?
Thank You!

It’s all there in the code ... please run the code in the tutorial and let me know if you have any questions!


feat: add support to missed dotnet versions (.net9, .net8, .net7)

Added support to missed dotnet versions (.net9, .net8, .net7)

.net9 is unsupported currently in AppVeyor.

Added manualy


MQTTNet doesn't seem to be adhering to the spec regarding Topic Alias

The Server MUST NOT send a Topic Alias in a PUBLISH packet to the Client greater than Topic Alias Maximum [MQTT-3.1.2-26]. A value of 0 indicates that the Client does not accept any Topic Aliases on this connection. If Topic Alias Maximum is absent or zero, the Server MUST NOT send any Topic Aliases to the Client [MQTT-3.1.2-27].
https://github.com/hivemq/hivemq-mqtt-client/issues/631#issuecomment-2164879437

Is anyone looking into this??


Move apps/sprayer to program/example and convert scripts to programs

Move apps/sprayer to program/example-spray and program/example-replay respectively and convert scripts to programs.

This looks very well done.
However, I would really like the "Hello world" experience to be simpler than this, and I think that requires some revision of our APIs.
How about we spend some brain cycles in the next release on making it more trivial to write a first app and then revise and merge this then?

@lukego I simplified this a bit and added detailed explanations of the code while adding the getting started guide to the source tree. We should be able to see if the explanation gets more or less complicated when revised for possible API changes. I have one problem though: I can not require require("program.example-spray.sprayer"). The file is there but I get this error:
program/example-spray/example-spray.lua:4: module 'program.example-spray.sprayer' not found:
	no field package.preload['program.example-spray.sprayer']
	no file './program/example-spray/sprayer.so'
	no file '/usr/local/lib/lua/5.1/program/example-spray/sprayer.so'
	no file '/home/max/snabbswitch/deps/luajit/usr/local/lib/lua/5.1/program/example-spray/sprayer.so'
	no file '/usr/local/lib/lua/5.1/loadall.so'
	no file './program.so'
	no file '/usr/local/lib/lua/5.1/program.so'
	no file '/home/max/snabbswitch/deps/luajit/usr/local/lib/lua/5.1/program.so'
	no file '/usr/local/lib/lua/5.1/loadall.so'
stack traceback:
	core/main.lua:96: in function <core/main.lua:94>
	[C]: in function 'require'
	program/example-spray/example-spray.lua:4: in main chunk
	[C]: in function 'require'
	core/main.lua:46: in function <core/main.lua:32>
	[C]: in function 'xpcall'
	core/main.lua:101: in main chunk
	[C]: in function 'require'
	[string "require "core.main""]:1: in main chunk

Any hints in debugging this would be welcome. The file src/obj/program/example-spray/sprayer_lua.o does exist.

Oh, could this be related to the dash (-) character in the directory name? Should I replace it with underscores?

I think that it is the dash, yes. I have been substituting underscores, but maybe there is a way to make the dash work too.

Yup it was the dashes. Fixed.

I somehow missed the text on this PR and only saw the code. Awesome with friendly prose! Let me read...

Oh I just added the prose like yesterday so you couldn't have known. :)

I would have known if you had pushed it as a new commit instead of discarding the history with a rebase :-)

@eugeneia Wow! I love the new getting started guide :-) Great job!
Logistic question. Will it be awkward to have the example programs upstream in the repository? In #470 I am making snabb --help print all the available programs, and it might be confusing if this included ones that are meant as sample programs rather than end-user tools.
Generally it may be a reasonable idea to support writing apps/applications in separate repositories rather than forks of the main repo. Have the Makefile pick them up somehow? But this is also something we can worry about later if it is not needed now.

You know you kind of introduced me to git rebase -i so you reap what you sow I guess. ;P
I don't feel like it will be awkward to have the examples in the main tree. The "example_" prefix makes it quite clear that these are not "normal" programs. I personally am always glad if I clone a repository and have some simple examples included that I can digest.
If we had something like a "production LTS only operator use" release, then we could simply tag a HEAD with the examples removed.
One thing that I see is that the example_replay is actually a useful program. example_spray is not though (as far as I can tell). It would be really elegant to have the "examples" just be simple but useful production programs/apps that just happen to be written and documented with special care.

If only I could rebase my Sent Mail folder so that I never recommended git rebase :-)
I take your point. There does not seem to be much harm in including a couple of examples, even positive if they would morph into real programs some time. I also think it makes a lot of sense to keep the Getting Started guide based on the current API because we do want to support that for a reasonable length of time and who knows when a new one will really land (it would be a shame to churn the API prematurely).

So time to merge?
That would allow us to close #330 and #405 and welcome new scripters with welcome arms. @plajjan and @jeffbrl might have some feedback but we could take that on new Issues/PRs rather than holding up this one.

Sure, go ahead! :)


task "major" must be integer (starting ibot 3.37); remove scheduled tasks (deprecated)

most of the changes are removing scheduled tasks.
ensuring task "major" is an integer only applies cases where we pass "major" as config.  for the CLI, that's only in the alice tasks.  (new lines 42-43 in src/lib/tasks.ts)

@ibraheemk starting with ibot 3.37, you'll need to use this updated CLI to start Alice webhook.


Allow setting UAC level

There's more to UAC than just on/off, but, apparently, the solution is complicated.

Duplicate of #47


Monitor supervisord with prometheus

Testing plan: Test-deployed on staging.

We could probably figure out how to setup groups such that prometheus had read-only access to the socket if we wanted to preserve the user-level isolation.
That said, IIRC the model for Prometheus is that the exporter doesn't accept any input (it just exports) and thus is a relatively low security risk?
Also, is there a benefit in having a Prometheus user with this change?

The node-exporter is low-risk because it's nominally firewalled away, and only serves a webpage.
The prometheus user is still useful for the prometheus server, but becomes less so on most other hosts with this change.
I thought about read-only access to the socket -- but I don't think there's such a thing.  It needs to do XMLRPC over it to send the status command, so if the mode on the socket were 750, it can't send the command.  Quick test using the postgres user:
postgres@alexmv-prod:~$ ls -l /var/run/supervisor.sock
srwxr-x--- 1 zulip postgres 0 Jul 14 22:07 /var/run/supervisor.sock
postgres@alexmv-prod:~$ supervisorctl status
error: <class 'PermissionError'>, [Errno 13] Permission denied: file: /usr/lib/python3/dist-packages/supervisor/xmlrpc.py line: 560

And supervisor itself doesn't have any concept of permissions to limit via.  So our only other option is to chmod 770 and chgrp to prometheus.
The other option is to run a second node_exporter as the zulip user, so it's not running the majority of the node_exporter code.  I don't think that saves much.

OK, I think that's probably a reasonable risk profile.  Merged, thanks for the explanation @alexmv!


CSRF refresh then retry doesn't work if the PUT request has a body

POST probably suffers from the same bug. The body has already been disposed and can't be reused for a retry.
I can probably move the retry up a level to get around this.

I worry that it will be confusing to implicitly retry in some cases and in others return an error saying they need to manually retry.
At the same time, I'm guessing we lose out on the benefits of the async pattern if we move this up a layer and hold onto the body until we get a response. Good catch Jason, thanks.


⚠ Samsung has degraded performance

In 0794023, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2098 ms

Resolved: Samsung performance has improved in 9af6231.


CCM Task Sequence Deployment getting Error 0x00000001

Hello,
i'm getting Error 0x00000001 in the Task Sequence and don't know what this means and what i should do to get rid of this error.

Can you help me?
Best Regards
Tobias

Hello,
Did you fix this issue? I'm getting the same error.
Reece

Hi,
i also ran into that Error. Does anyone has a solution for that Problem?
BR
Max

I also have the same issue.
It never seems to actully even try to run the Invoke-CMApplyDriverPackage.ps1 at all.
Can't even find any attempt in the smsts.log

Circling back on this one... as I've recently tried implementing MDM and I'm stuck in this exact same spot. It shows no attempt to trigger the script in the SMSTS.log files.
Any ideas?


tt_user is working?

yt-dlp --use-extractors "/home/elias/yt_dlp/extractor/tt_user.py"  --cookies-from-browser firefox<EMAIL_ADDRESS>in linux, it says no suitable extractor found.
when i use
yt-dlp --use-extractors tiktok:user  --cookies-from-browser firefox<EMAIL_ADDRESS>the empty json error appears.
trying to install wheel again:
yt-dlp-ttuser is already installed with the same version as the provided wheel. Use --force-reinstall to force an installation of the wheel.
so how i can use it to download the vids of a tiktok profile?
yt-dlp --list-extractors | grep tik
DetikEmbed
tiktok:effect (CURRENTLY BROKEN)
tiktok:live
tiktok:sound (CURRENTLY BROKEN)
tiktok:tag (CURRENTLY BROKEN)
tiktok:user (CURRENTLY BROKEN)
vm.tiktok

pip install --break-system-packages --user --force-reinstall *.whl
DEPRECATION: Loading egg at /usr/local/lib/python3.11/dist-packages/gallery_dl-1.26.7.dev0-py3.11.egg is deprecated. pip 24.3 will enforce this behaviour change. A possible replacement is to use pip for package installation.. Discussion can be found at https://github.com/pypa/pip/issues/12330
Processing ./yt_dlp_ttuser-2024.3.18-py3-none-any.whl
Installing collected packages: yt-dlp-ttuser
  Attempting uninstall: yt-dlp-ttuser
    Found existing installation: yt-dlp-TTUser 2024.3.18
    Uninstalling yt-dlp-TTUser-2024.3.18:
      Successfully uninstalled yt-dlp-TTUser-2024.3.18
Successfully installed yt-dlp-ttuser-2024.3.18

so how i can use it to download the vids of a tiktok profile?

assuming you also installed yt-dlp with pip, you can just do yt-dlp "URL"
if you didn't install yt-dlp with pip, then don't install the plugin with pip. see the README

i have copied tt_user.py manually as well installed through python here:
/home/elias/Downloads/_linux/yt-dlp/yt_dlp/extractor/
then:
~/Downloads/_linux/yt-dlp/yt-dlp https://www.tiktok.com/@veganlifengo
WARNING: The program functionality for this site has been marked as broken, and will probably not work.
[tiktok:user+TTUser] Extracting URL: https://www.tiktok.com/@veganlifengo
[tiktok:user+TTUser] veganlifengo: Downloading user webpage
[download] Downloading playlist: veganlifengo
[tiktok:user+TTUser] veganlifengo: Downloading page 1
[tiktok:user+TTUser] veganlifengo: Downloading page 2
[tiktok:user+TTUser] Playlist veganlifengo: Downloading 29 items of 29
[download] Downloading item 1 of 29
[TikTok] Extracting URL: https://www.tiktok.com/@veganlifengo/video/7284278300401323296
[TikTok]<PHONE_NUMBER>401323296: Downloading video feed
WARNING: [TikTok] Expecting value in '': line 1 column 1 (char 0). Retrying... (attempt 1 of 4)
[TikTok]<PHONE_NUMBER>401323296: Downloading video feed
WARNING: [TikTok] Expecting value in '': line 1 column 1 (char 0). Retrying... (attempt 2 of 4)
[TikTok]<PHONE_NUMBER>401323296: Downloading video feed
WARNING: [TikTok] Expecting value in '': line 1 column 1 (char 0). Retrying... (attempt 3 of 4)
[TikTok]<PHONE_NUMBER>401323296: Downloading video feed
WARNING: [TikTok]<PHONE_NUMBER>401323296: Failed to parse JSON (caused by JSONDecodeError("Expecting value in '': line 1 column 1 (char 0)")); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U; trying with webpage
[TikTok]<PHONE_NUMBER>401323296: Downloading webpage
```
same error if i use cookies from firefox

searching for other yt-dlp paths in entire linux install, no other is found.
what i must do?

~/Downloads/_linux/yt-dlp/yt-dlp --version
2023.11.16

sorry, with latest yt-dlp is working. thanks


Take commit hash of mruby too

This PR seems to add the following information into the lock file:
mruby_version:
  version: 2.0.1
  release_no: 20001
  git_commit: 246179518d8e3641dcb026fb24e8b06f9878faa8

Is this information just output and not used?
It seems redundant that version is repeated like mruby_version.version, so I think just mruby is better.


greeter_server killed at startup and executable deleted!

What version of gRPC and what language are you using?
compiled latest from master as of this morning using bazel 4.2.0, C++
What operating system (Linux, Windows,...) and version?
macOS  Big Sur, 11.6
What runtime / compiler are you using (e.g. python version or version of gcc)
➜  grpc git:(master) g++ --version
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/4.2.1
Apple clang version 13.0.0 (clang-13<IP_ADDRESS>)
Target: x86_64-apple-darwin20.6.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

What did you do?
Trying to run greeter_server using info here:
https://grpc.io/docs/languages/cpp/quickstart/#try-it
What did you expect to see?
Expected greeter_server to run.
What did you see instead?
It is killed right away and the executable deleted!
Here's more info:
➜  build git:(master) pwd
/Users/vamsee.lakamsani/git/ot/grpc/examples/cpp/helloworld/cmake/build
➜  build git:(master) make -j
[ 22%] Built target hw_grpc_proto
[ 33%] Built target greeter_async_server
[ 44%] Built target greeter_async_client2
[ 55%] Built target greeter_client
[ 66%] Built target greeter_callback_client
[ 77%] Built target greeter_async_client
[ 88%] Built target greeter_callback_server
[ 94%] Linking CXX executable greeter_server
ld: warning: direct access in function '_sk_X509_NAME_deep_copy' from file '/Users/vamsee.lakamsani/.local/lib/libssl.a(ssl_x509.cc.o)' to global weak symbol '_sk_X509_NAME_call_free_func' from file '/Users/vamsee.lakamsani/.local/lib/libgrpc.a(ssl_transport_security.cc.o)' means the weak symbol cannot be overridden at runtime. This was likely caused by different translation units being compiled with different visibility settings.
[100%] Built target greeter_server
➜  build git:(master) ls -rtl ./greeter_server
-rwxr-xr-x  1 vamsee.lakamsani  staff  30814912 Oct 29 11:29 ./greeter_server*
➜  build git:(master) ./greeter_server
[1]    93695 killed     ./greeter_server
➜  build git:(master) ls -rtl ./greeter_server
ls: ./greeter_server: No such file or directory

Anything else we should know about your project / environment?
The other executables like greeter_async_server, greeter_callback_server are starting fine. My end goal is to test server reflection using  grpc_cli ls localhost:50051 as documented here. Seems like server reflection is enabled only for the greeter_server
Is there a bazel way to build these quickstart examples like greeter and route guide given that bazel is recommended over cmake for building grpc c++? Thanks.

Please hold on to this. I suspect some malware scanning software on my company-issued laptop is falsely flagging this executable and deleting it!. Working with our IT security admins to resolve it. I will close after my suspicion is confirmed. thanks.

Anyone you should be restart files with cmd than idk lel

I'm going ahead and closing this.


[#95] refactor to new groupId:metricbot/bmarwell

Codecov Report

Merging #96 (6c4adf7) into main (20ff272) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 6c4adf7 differs from pull request most recent head 1ed107b. Consider uploading reports for the commit 1ed107b to get more accurate results

@@            Coverage Diff            @@
##               main      #96   +/-   ##
=========================================
  Coverage     50.23%   50.23%
  Complexity      103      103
=========================================
  Files            26       26
  Lines           846      846
  Branches         70       70
=========================================
  Hits            425      425
  Misses          395      395
  Partials         26       26

Impacted Files
Coverage Δ

.../io/github/bmarwell/twitter/metricbot/web/App.java
0.00% <ø> (ø)

...ell/twitter/metricbot/web/events/MentionEvent.java
0.00% <ø> (ø)

.../metricbot/web/events/PotentialTweetProcessor.java
0.00% <ø> (ø)

...tter/metricbot/web/events/TweetProcessRequest.java
0.00% <ø> (ø)

...ricbot/web/events/UnprocessedTweetQueueHolder.java
0.00% <ø> (ø)

...twitter/metricbot/web/factory/TwitterProducer.java
0.00% <ø> (ø)

...metricbot/web/listener/DataSourceInitListener.java
0.00% <ø> (ø)

...r/metricbot/web/listener/TweetCleanerListener.java
0.00% <ø> (ø)

...r/metricbot/web/listener/TweetMentionListener.java
0.00% <ø> (ø)

.../metricbot/web/listener/TweetResponseListener.java
0.00% <ø> (ø)

... and 16 more

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