WIP: Add typeNameModifier to GQLTypeOptions

fixes #558
This is a first draft. I don't have much experience on large Haskell projects, so I just wanted to get this up and get a conversation going.
It seems inefficient to upack and apply the modifier then pack again, but I'm not exactly sure what the implications are of typeNameModifier :: TypeName -> TypeName.
I do not understand what proxy is doing in the code, so could I please have a bit of an explanation about that?
Also, I think I'll need more direction on writing tests for this change.
Thanks in advance for the feedback!

thanks @bradsherman,
proxy contains only type information. is handy if you want to perform operations on the type without giving it a value.
it is used with data type Proxy: https://hackage.haskell.org/package/base-<IP_ADDRESS>/docs/Data-Proxy.html
which is nothing more than this.
data Proxy a = Proxy

now i thinking that may we should combine  prefixInputType and   typeNameModifier  as a one function. where if you wish you can define different names for input and output types generated from same type.
 typeNameModifier ::  Target -> String -> String

where data Target = Input | Output

or
 typeNameModifier ::  Bool -> String -> String

where True means that target is input

Are you suggesting the user supplies a function like that? For example, if I have a type with original name User, then we allow users to specify a function like:
userTypeNameModifier :: Bool -> String -> String
userTypeNameModifier True original = "MyCustomInputUserTypeName"
userTypeNameModifier False original = "MyCustomOutputUserTypeName"

instance GQLType User where
  typeOptions _ opt = opt {typeNameModifier = userTypeNameModifier}

So in this case we do not have to force the input type name to be Input<TypeName>?

@bradsherman exactly.
i think, in this way we can give user more control.  what do you think ?

Yeah, I like how it gives the user more flexibility. Is there any way they could abuse/misuse that control? For example, by providing a name that clashes with other names? I am just trying to think of extraordinary scenarios, but you would know better than I how those situations would be handled.

i think it is okey. The compiler validates the schema after the type derivation so that name collisions are reported.

cool, I'll push up those changes

@thanks, looks good, since you updated test/Feature/TypeCategoryCollision/Success/API.hs we don't need any additional tests.
if you could update changelog would be nice :

first date on  0.17.0 is wrong must be  0.17.0 - Unreleased
since prefixInputType was not yet released, we can replace it with new approach and update its example.

It seems inefficient to upack and apply the modifier then pack again, but I'm not exactly sure what the implications are of typeNameModifier :: TypeName -> TypeName.

we could text, but I don't know how much performance improvement we could get from that.
Changing strings is easier than text, I think.
but you can open a issue to discuss it, and if others also agree, we can do it.

sorry, tried to move too fast! good catch

sorry, tried to move too fast! good catch

all right :)
request is  Approved.

thank you again


cannot connect after building

Following the development guide, I built the server (tried both with -tags dev and without) and started it:
level=debug starting=metrics addr=localhost:6078
loaded active.de.toml
loaded active.en.toml
level=info event=serving<EMAIL_ADDRESS>shsmuxaddr=:8008 httpaddr=:3000 version=v2.0.7 commit=unset

Connecting to it via http://localhost:3000 redirects to https://localhost:3000 which fails in Chrome (latest) with ERR_SSL_PROTOCOL_ERROR, no further info, nothing printed in the console. System is Debian 11.
If it matters: Chrome is running on macos using an SSH tunnel for port 3000.
How can I get out more debug info and/or solve this problem?

Hey @rkuhn, sounds like you maybe ran into https://github.com/ssbc/go-ssb-room/issues/356? You definitely want -tags=lite to avoid this HTTPS requirement for local testing. Did you try another browser? I wouldn't imagine the tunnel has much to do this with this but you might want to reduce the moving parts to track down this issue... good luck and do let us know!

Thanks, I got it working by using -tags=dev and purging the browser cache (the 301 → https was still in there). However, I also tried accessing the public IP, i.e. in non-local and non-testing mode, and the behaviour was the same. So setting up a proper Room is still something I can’t do.
The next hurdle is that any click on the served page goes to about:blank#blocked (i.e. a white screen). This behaviour persists also after creating the admin user.

@rkuhn

However, I also tried accessing the public IP, i.e. in non-local and non-testing mode, and the behaviour was the same. So setting up a proper Room is still something I can’t do.

Ah, you can't run a non-local / non-testing instance with -tags=dev? It's only for local dev mode?
(if you're seeing this issue on a not -tags=dev build, let's open a new issue with more details?)


node 13.x安装报错

node 版本  v12.10.0
Mac 版本 10.15.2
报错信息
npm WARN deprecated<EMAIL_ADDRESS>ffmpeg-static@2 won't be updated with new ffmpeg binaries anymore. Use ffmpeg-static@latest.
npm ERR! Unexpected end of JSON input while parsing near '...wqMgKK+JgWsAQik5o0HTL'
npm ERR!      /Users/guofeng/.npm/_logs/2020-03-02T11_03_28_462Z-debug.log

You've been GFWed, check network.


Display error message after getting a server response

The aim is display the server validation error. How can I do this?
For example:
The request was sent to server didn't pass a validation and error message was getting. I'd like to display that above validation message under the appropriate field.
How can I do this?

While the Promise has to call resolve() with a boolean, you can set other values right before you call resolve() and use those in the field-messages, e. g.:
return new Promise((resolve, reject) => {
  fetch('https://httpbin.org/get').then(response => {
    this.fieldNameServerMsg = response.serverMsg;
    resolve(response.isValid);
  });
});

<field-messages name="fieldName">

{{fieldNameServerMsg}}

</field-messages>

Hope this response did not come too late. Please close the issue if this resolved it.


Suggesion: Get-BcContainers with indication of the container type

Describe the issue
It would be nice if Get-BcContainers -includeLabel would also specify if the container type is OnPrem, Sandbox, Insider Sandbox or Preview. With the info I would know if I need to append ?tenant=default or not.

Looks messy, but ok...


41 emissions plots and outputs separate from concentration

5-panel plot of total emissions for each pollutant!

I've added comments to the function and edited concentration_layers and concentration so they calculate and add the north arrows! I've also changed it so that the border is only around California, not the output regionl. I tried different ways to see if we can clip it only once outside the for loop, but I haven't been able to get it to work, since we still have to clip the data each time to the output region- but I've tried taking some of the code out of the for loop that I can. The current fastest mine has run was around 2.3 minutes, and without clipping at all it took around 2.0 minutes, so I hope it won't be too big of an issue? I'll take another look for sure!


SLF4J-345: Move delegation check loop from log4j-over-slf4j.jar to slf4j-log4j12.jar

Currently, the check on delegation loop for log4j is done in the log4j-over-sl4fj.jar, whereas the same check for jcl is done in the slf4j-jcl.jar. This causes different behaviour for log4j and jcl:

For log4j, if both log4j-over-slf4j.jar and slf4j-log4j12.jar are present on the classpath, an IllegalStateException is always thrown, even if slf4j-log4j12.jar is not chosen as the actual binding.
For jcl, if both jcl-over-slf4j.jar and slf4j-jcl.jar are present on the classpath, an IllegalStateException is ONLY thrown when slf4j-jcl.jar is chosen as the actual binding. If, for example, logback is also on the classpath and is chosen as the actual binding, then (JCLLoggerFactory class in) slf4j-jcl.jar is never initialized and therefore the delegation loop is not executed.

The log4j behaviour causes an issue for my situation, where I want to redirect log4j logging to slf4j/logback, but my classpath also contains (an older version of) slf4j-log4j12.jar. The actual binding is logback, so there is no real conflict, i.e. no danger of a delegation loop/stack overflow. Still, I'm unable to redirect log4j logging via slf4j/logback using log4j-over-slf4j.jar.
I propose to move the delegation loop check from log4j-over-slf4j.jar to slf4j-log4j12.jar, so that the delegation loop check behaviour for log4j will be the same as for jcl.
See also JIRA SLF4J-345.

I like the symmetry with JCL.


Update to mongodb 3.2.x and update dependencies

Update mongodb to 3.2.x (from 3.x.x)
Update code to @hapi/code 5.3.x (from 5.x.x)
Update lab to @hapi/lab 19.x.x (from 15.x.x)
Update hoek to @hapi/hoek 6.x.x (from 5.x.x)
Update joi to @hapi/lab 15.x.x (from 13.x.x)
Fix eslint warnings

Make use of rest parameters (...args) instead of arguments.
Minor issues
Updates to code related to mongodb update
Change count() reference to countDocuments()
Use atomic operator ($set) for calls to findOneAndUpdate()
Allow test suite to pickup MongoDB URI from the environment.

Sorry for the delayed response on my part. I let one last comment. After that I'll land this and do a release.

I’m out of town for the weekend but I’ll take care of it Monday. Thanks

On May 31, 2019, at 8:55 PM, Reza Akhavan<EMAIL_ADDRESS>wrote:
Sorry for the delayed response on my part. I let one last comment. After that I'll land this and do a release.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks again. 3.x.x has been published and includes these changes 👍


zone-nativescript ERROR on with angular and iOS simulator

When I run it on Android get request work correctly. But on IOS it return nativescript error(not server).
{"line": 994,"column": 38,"sourceURL": "file:///app/tns_modules/nativescript-angular/zone-js/dist/zone-nativescript.js"
Which platform(s) does your issue occur on?

iOS 11.4 Iphone 6 emulator

Please, provide the following version numbers that your issue occurs with:
✔ Getting NativeScript components versions information...
⚠ Update available for component nativescript. Your current version is 4.0.1 and the latest available version is 4.1.1.
✔ Component tns-core-modules has 4.1.0 version and is up to date.
✔ Component tns-android has 4.1.3 version and is up to date.
✔ Component tns-ios has 4.1.1 version and is up to date.

"dependencies": {
    "@angular/animations": "~5.2.0",
    "@angular/common": "~5.2.0",
    "@angular/compiler": "~5.2.0",
    "@angular/core": "~5.2.0",
    "@angular/forms": "~5.2.0",
    "@angular/http": "~5.2.0",
    "@angular/platform-browser": "~5.2.0",
    "@angular/platform-browser-dynamic": "~5.2.0",
    "@angular/router": "~5.2.0",
    "install": "^0.11.0",
    "nativescript-angular": "^5.3.0",
    "nativescript-drop-down": "^4.0.0",
    "nativescript-filter-select": "^1.2.9",
    "nativescript-floatingactionbutton": "^4.1.3",
    "nativescript-fontawesome": "^1.0.0",
    "nativescript-image-cache": "^1.1.6",
    "nativescript-loading-indicator": "^2.4.0",
    "nativescript-localstorage": "^1.1.5",
    "nativescript-open-app": "^0.2.0",
    "nativescript-permissions": "^1.2.3",
    "nativescript-phone": "^1.3.1",
    "nativescript-pulltorefresh": "^2.1.0",
    "nativescript-ripple": "^2.0.0",
    "nativescript-snackbar": "^2.0.1",
    "nativescript-theme-core": "~1.0.4",
    "nativescript-ui-autocomplete": "^3.7.0",
    "nativescript-ui-dataform": "^3.6.0",
    "nativescript-ui-sidedrawer": "^3.5.1",
    "reflect-metadata": "~0.1.8",
    "rxjs": "^5.5.0",
    "rxjs-compat": "^6.2.0",
    "tns-core-modules": "^4.0.1",
    "zone.js": "^0.8.4"
  },
  "devDependencies": {
    "@angular-devkit/core": "~0.5.5",
    "@angular/compiler-cli": "~5.2.0",
    "@ngtools/webpack": "~1.9.4",
    "babel-traverse": "6.24.1",
    "babel-types": "6.24.1",
    "babylon": "6.17.0",
    "clean-webpack-plugin": "~0.1.19",
    "codelyzer": "^3.0.1",
    "copy-webpack-plugin": "~4.3.0",
    "css-loader": "~0.28.7",
    "extract-text-webpack-plugin": "~3.0.2",
    "lazy": "1.0.11",
    "nativescript-dev-typescript": "^0.7.1",
    "nativescript-dev-webpack": "^0.11.0",
    "nativescript-worker-loader": "~0.8.1",
    "raw-loader": "~0.5.1",
    "resolve-url-loader": "~2.2.1",
    "typescript": "~2.6.2",
    "uglifyjs-webpack-plugin": "~1.1.6",
    "webpack": "~3.10.0",
    "webpack-bundle-analyzer": "^2.9.1",
    "webpack-cli": "~2.0.14",
    "webpack-sources": "~1.1.0"
  }

Is there any code involved?
{
"line": 994,
"column": 38,
"sourceURL": "file:///app/tns_modules/nativescript-angular/zone-js/dist/zone-nativescript.js",
"originalStack": "ZoneAwareError@file:///app/tns_modules/nativescript-angular/zone-js/dist/zone-nativescript.js:994:38\nfile:///app/tns_modules/tns-core-modules/http/http-request/http-request.js:87:37\nUIApplicationMain@[native code]\nstart@file:///app/tns_modules/tns-core-modules/application/application.js:264:26\nrun@file:///app/tns_modules/tns-core-modules/application/application.js:284:10\nbootstrapNativeScriptApp@file:///app/tns_modules/nativescript-angular/platform-common.js:158:26\nbootstrapApp@file:///app/tns_modules/nativescript-angular/platform-common.js:92:38\nbootstrapModule@file:///app/tns_modules/nativescript-angular/platform-common.js:81:26\nanonymous@file:///app/main.js:7:57\nevaluate@[native code]\nmoduleEvaluation@[native code]\n[native code]\npromiseReactionJo
CONSOLE ERROR file:///app/tns_modules/nativescript-angular/zone-js/dist/zone-nativescript.js:569:26: Unhandled Promise rejection: A error occured ; Zone: <root> ; Task: Promise.then ; Value: Error: A error occured HttpError@file:///app/shared/error/http.error.js:5:14 [<root>]
httpResponseHandler@file:///app/helpers/httpResponseHandler.js:63:38 [<root>]
onHttpStreamError@file:///app/shared/http/http.service.js:118:58 [<root>]
onHttpStreamError@[native code] [<root>]
error@file:///app/tns_modules/rxjs/operators/catchError.js:104:39 [<root>]
_error@file:///app/tns_modules/rxjs/Subscriber.js:132:31 [<root>]
error@file:///app/tns_modules/rxjs/Subscriber.js:106:24 [<root>]
onError@file:///app/tns_modules/@angular/http/bundles/http.umd.js:1641:39 [<root>]
file:///app/tns_modules/tns-core-modules/xhr/xhr.js:151:26 [<root>]
forEach@[native code] [<root>]
emitEvent@file:///app/tns_modules/tns-core-modules/xhr/xhr.js:150:25 [<root>]
_setReadyState@file:///app/tns_modules/tns-core-modules/xhr/xhr.js:218:31 [<root>]
file
CONSOLE ERROR file:///app/tns_modules/nativescript-angular/zone-js/dist/zone-nativescript.js:571:22: Error: Uncaught (in promise): Error: A error occured
HttpError@file:///app/shared/error/http.error.js:5:14 [<root>]
httpResponseHandler@file:///app/helpers/httpResponseHandler.js:63:38 [<root>]
onHttpStreamError@file:///app/shared/http/http.service.js:118:58 [<root>]
onHttpStreamError@[native code] [<root>]
error@file:///app/tns_modules/rxjs/operators/catchError.js:104:39 [<root>]
_error@file:///app/tns_modules/rxjs/Subscriber.js:132:31 [<root>]
error@file:///app/tns_modules/rxjs/Subscriber.js:106:24 [<root>]
onError@file:///app/tns_modules/@angular/http/bundles/http.umd.js:1641:39 [<root>]
file:///app/tns_modules/tns-core-modules/xhr/xhr.js:151:26 [<root>]
forEach@[native code] [<root>]
emitEvent@file:///app/tns_modules/tns-core-modules/xhr/xhr.js:150:25 [<root>]
_setReadyState@file:///app/tns_modules/tns-core-modules/xhr/xhr.js:218:31 [<root>]
file:///app/tns_modules/tns-core-modules/xhr/xhr.js:74:37 [<root

Hi @aturan23,
I tested your case with latest NativeScript 4.1 and nativescript-geolocation v.4.2.6, however, was unable to recreate any issue with the plugin. For your convenience, I am attaching a sample project. Please review it and make the needed changes, which will allow us to recreate the issue.
Archive.zip

I had the same (see issue #139), seems to be an issue with the iOS simulator only.

Hi @aturan23 @abhayastudios,
Are you still able to recreate the issue with the latest nativescript-geolocation version 4.3.1?

@tsonevn unfortunately still happening indeed...

Hi @abhayastudios,
I have tested the case with our demo project on iOS simulator(iPhone 7 iOS 12), however, was unable to hit this problem. Do you have any sample project, which can be used for recreating the issue?

@tsonevn I'll try to reproduce in a minimal project, although as I wrote in issue #139 as long as this happens only on the simulator, it is not that important to me.

@tsonevn I was able to reproduce the problem with your sample project. The things is that the first time when the iOS simulator runs, it works fine. However, if you cause the app to restart (through livesync or running tns run ios again) without closing the simulator(!), then also with your sample app I will get Timeout while searching for location!.
Hope this helps.

@tsonevn I has always been enabled (for me it is set to Freeway Drive). I remind you that the first time the simulator starts it works just fine.


Link to another node module

I have a few node modules in my project, one of which, module B uses code defined in module A. I have listed module A as a dependency of module B using the "dependencies" directive of the binding.gyp, but when I call function functions in module B, I get undefined symbol errors for code that is defined in module A. This tells me that node-gyp is not linking module B against module A to resolve the code. Is there a way to resolve this behavior?

When you say "module", do you mean separate .node files?  Perhaps you can post your binding.gyp(s)?

Yes I mean separate .node files
{
            "target_name": "classificationTesting",

            "sources": [...
            ],

            "libraries": [
                "<!@(pkg-config --libs opencv)",
                "-lboost_system",
                "-lboost_filesystem",
                "-lboost_regex",
                "-lvl",
                "<!@(pkg-config --libs tinyxml)"
            ],

            "include_dirs": [
                "api",
                "<!(node -e \"require('nan')\")"
            ],

            "cflags": [ "-std=c++11", "-w", "<!@(pkg-config --cflags opencv)"],
            "cflags!": ["-fno-exceptions"],
            "cflags_cc!": ["-fno-rtti", "-fno-exceptions"],
        },
...
        {
            "target_name": "pairwiseTraining",

            "sources": [...
            ],

            "dependencies": [
                    "classificationTesting
            ],

            "libraries": [
                "<!@(pkg-config --libs opencv)",
                "-lboost_system",
                "-lboost_filesystem",
                "-lboost_regex",
                "<!@(pkg-config --libs tinyxml)"
            ],

            "include_dirs": [
                "api",
                "<!(node -e \"require('nan')\")",
                "pipelines/classification",
                "pipelines/imagePrep"
            ],

            "cflags": [ "-std=c++11", "-w", "<!@(pkg-config --cflags opencv)", "-fpermissive"],
            "cflags!": ["-fno-exceptions"],
            "cflags_cc!": ["-fno-rtti", "-fno-exceptions"],
        }

I don't think you can make that work without either:

Linking B directly to A so that ldd or otool -L prints B as a dependency of A, or
Patching node.js / libuv to open shared objects with dlopen(RTLD_GLOBAL).

I'd probably split off the shared code into a static library ('type': 'static_library' in your binding.gyp) and link that into both shared objects.

I do actually do this with node-ogg and node-vorbis (libvorbis needs to link to libogg). The implementation has always been pretty hacky (involving changing node-ogg's "type" to shared_library, re-exporting all of libogg's symbols in the generated .node file, and more hacks), but in the end, the vorbis.node file ends up linking to the ogg.node file to access the libogg symbols:
$ otool -L build/Release/vorbis.node
build/Release/vorbis.node:
        @rpath/ogg.node (compatibility version 0.0.0, current version 0.0.0)
        /usr/lib/libstdc++.6.dylib (compatibility version 7.0.0, current version 104.1.0)
        /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1226.10.1)
        /usr/lib/libgcc_s.1.dylib (compatibility version 1.0.0, current version 915.0.0)

Probably I will factor the common code into a static library rather than hack around the nodejs limitations


Updated all cross-platform functions and types up to 8 June 2016

I have not updated two OSX-specific functions,
uiDarwinMarginAmount() and uiDarwinPaddingAmount().
I'm not exactly sure how this should be done.

Sorry about it being in multiple commits. Also, the previous pr (I closed) did not update, so I made a new one. My apologies.

git rebase -i origin/master
don't touch the first line.
from line 2 to last line replace all the "pick" by "fixup".
save and then push with --force.


Use -isystem to suppress warnings from esp-idf headers

When folks use the esp-idf includes (via
embuild::build::CInclArgs::try_from_env("ESP_IDF")) to build c code in
their crates, depending on the warnings they enable for their code, some
warnings might be emitted for code in the esp-idf headers (for example,
unused arguments).
Use the -isystem flag to mark the esp-idf provided header paths as
"system" provided, which suppresses the warnings which would be emitted
from headers included from these paths.

Thanks!


How do you specify trigger when suggestions is a function?

In your examples you explain how to specify the trigger character with values list but not when we give a fonction as suggestions.

You can use following config to specify trigger when suggestions come from a Function
{
    trigger: "@",
    values: Function
}

Thanx @avcs06


[FORMATTING] Can't handle array type

Input data
Which SQL and options did you provide as input?
CREATE TABLE public.todos (
  assigned_to UUID[] NOT NULL
);

Expected Output
No change or:
assigned_to UUID [] NOT NULL

Actual Output
Error: Unable to format SQL: Error: Parse error: Unexpected "[] NOT NUL" at line 27 column 21
Usage

How are you calling / using the library? - Vscode format document / Shift + Option + F
What SQL language(s) does this apply to? - Postgres
Which SQL Formatter version are you using? - v1.6.0

Please read the FAQ that you can find in the README. Your problem is solved
by correct configuration and by not using the outdated VSCode plugin.
On Sat, Sep 7, 2024, 02:24 Mooli @.***> wrote:

Input data
Which SQL and options did you provide as input?
CREATE TABLE public.todos (
assigned_to UUID[] NOT NULL
);
Expected Output
No change or:
assigned_to UUID [] NOT NULL
Actual Output
Error: Unable to format SQL: Error: Parse error: Unexpected "[] NOT NUL"
at line 27 column 21
Usage

How are you calling / using the library? - Vscode format document /
Shift + Option + F
What SQL language(s) does this apply to? - Postgres
Which SQL Formatter version are you using? - v1.6.0

—
Reply to this email directly, view it on GitHub
https://github.com/sql-formatter-org/sql-formatter/issues/777, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA43OOEV6SBOPGZQAS424LZVI2SBAVCNFSM6AAAAABNZOBTKKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGUYTCMZRGA4DENY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.>
[ { @.": "http://schema.org", @.": "EmailMessage",
"potentialAction": { @.": "ViewAction", "target": "
https://github.com/sql-formatter-org/sql-formatter/issues/777", "url": "
https://github.com/sql-formatter-org/sql-formatter/issues/777", "name":
"View Issue" }, "description": "View this Issue on GitHub", "publisher": {
@.***": "Organization", "name": "GitHub", "url": "https://github.com" }
} ]


refactor: Alias function for safe_sign_transaction called sign

High Level Overview of Change
Added alias function for safe_sign_transaction called sign
Context of Change
safe_sign_transaction is too long and unnecessary -- potentially deprecate/remove in xrpl-py 2.0
Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Refactor (non-breaking change that only restructures code)
[ ] Tests (You added tests for code that already exists, or your new feature included in this PR)
[ ] Documentation Updates
[ ] Release

One thing to double check: how do the docs handle the aliases?

Docs work


Add new corpus

As discussed on encode.ru, there are some pretty bad limitations with our current corpus.  See my initial post in that thread for details.  I think that thread has gotten a bit out of hand (feels like design by committee), but I still plan on putting together a new corpus, which should obviously be added to the squash benchmark.

I've just created a new project for this: https://github.com/nemequ/squash-corpus (name and location may change in the future).


Add OTA handlers as McAfee Embedded Control updaters

Once installed into an IDP system, the device manager OTA handler will not be able to perform an update if MEC/solidcores is enabled.
For example:
McAfee Solidifier[5089]: McAfee Solidifier prevented unauthorized execution of '/var/lib/python-device-cloud/otapackage/install.sh' by process /bin/bash (Process Id: 5632, User: dc)
Can we add the OTA handlers (standard and offline) as McAfee updaters in solidcore.conf ?

I think this issue is not a bug, more an IDP config issue.  Closing.


Tag for 0.8.1 not published

This makes it hard to know what changed before updating.

I'll add a tag. Nothing changed about the code, I just updated links that pointed to the old repository location.

Done.


Remove impossible nyan cat achievement

The nyan cat achievement should be removed because the nyan cat has been removed from Minetest Game 0.4.16.
Otherwise, there would be an impossible achievement.

I think it should rather be replaced by a "PB&J Pup" achievement

PB&J Pup has been removed as well.
If PB&J Pup is in an external mod now, the achievement should be added there, not in the awards mod.


Back up chat css

https://trello.com/c/9ubwVFum/72-adding-css-chat-component
https://trello.com/c/xPijtP3M/77-display-the-bottom-of-the-chat-list-instead-of-the-top
https://recordit.co/p013aTlHaf

The back up chat design is not breaking the project.


Sort package versions when dbt deps fails due to package not available

dbt deps
Running with dbt=0.20.1
Encountered an error:
Could not find a matching version for package tailsdotcom/dbt_artifacts
  Requested range: =0.4.5-a2, =0.4.5-a2
  Available versions: ['0.4.0', '0.4.1', '0.2.1', '0.2.0', '0.5.0', '0.4.3', '0.3.0', '0.1.0', '0.4.2', '0.4.4']

It would be nice if the available versions were sorted in ascending release order.

@NiallRees Makes sense to me! This feels like an uncontroversial improvement.


DBZ-8224 allow field name without topic qualifier

make filtering match documentation

@Naros @vjuranek Any comments? If not please feel free to merge.

As I made the changes suggested by @vjuranek, to avoid creating a new PR since this one is locked, I merged these changes directly into main and backported the change to the 2.7 branch.


Delete all OOM

Environment

Nautobot version (Docker tag too if applicable): ghcr.io/nautobot/nautobot:2.1.4-py3.9
Python version: 3.9.18
Database platform, version: postgres:13
Middleware(s): uWSGI/nginx - whatever comes with the container + custome uWSGI config

Steps to Reproduce

Select # of (IP's) in this case 56k:

Press Delete all
Get error:

Expected Behavior
I would just get a list viewing all the IP's with the option to delete them all. Similar to deleting any other object.

Observed Behavior
<class 'MemoryError'>

Thanks for the report!
I'm going to point the finger at https://github.com/nautobot/nautobot/blob/develop/nautobot/core/views/generic.py#L1197-L1202 as a starting point. It seems to me that if we're using _all, we should just carry that flag forward, rather than trying to construct a pk_list consisting of 56000 individual entries.

Please consider this at the same time: https://github.com/nautobot/nautobot-app-golden-config/blob/4de3f3778596bfac6ab2318373b0d8aee2323bcf/nautobot_golden_config/views.py#L281-L322

Please consider this at the same time: https://github.com/nautobot/nautobot-app-golden-config/blob/4de3f3778596bfac6ab2318373b0d8aee2323bcf/nautobot_golden_config/views.py#L281-L322

@itdependsnetworks I'm not sure I follow - what about that function are you asking us to consider?

Related: #2459.

Please consider this at the same time: https://github.com/nautobot/nautobot-app-golden-config/blob/4de3f3778596bfac6ab2318373b0d8aee2323bcf/nautobot_golden_config/views.py#L281-L322

@itdependsnetworks I'm not sure I follow - what about that function are you asking us to consider?

@glennmatthews in GC, we have used the same pattern, so if you are adjusting this pattern here, just ask to consider if it will still work for GC.

Also an issue: the rendered confirmation page is not paginated - so even if the pk_list isn't the issue, Django will probably tip over trying to render a non-paginated table of 55000 records in a single page.

For both "edit all" and "delete all" cases, instead of a secondary page, we could possibly use modals - a simple "confirmation" modal for "delete all", and a modal containing the BulkEditForm for "edit all"?
For the "edit selected" and "Delete selected" cases, we could keep the existing UI for now.

Alternately, instead of a modal, just skip rendering the table of related objects in the confirmation page in the "select all" case, and just list the filters and total count in its place?


Monaco cannot display the written code and its theme is not controlled by Slidev based on the light/dark mode.

Describe the bug

Monaco cannot display the written code. The code block is blank:

Its theme is not controlled by Slidev based on the light/dark mode:
dark mode:

light mode:

(The code in Monaco editor was writen after npm run dev)
When downgrading the Slidev version (such as v0.31.4), these bugs were fixed.

To Reproduce
Steps to reproduce the behavior:

Create a new Slidev project
Add {monaco} after the language id
npm run dev
Toggle light/dark mode

Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome, Edge
Slidev version: 0.35.5

Yeah, I found that too. PR welcome if you find any clue on how to fix it


Cloud presenters and helpers (and tests) should pass Rubocop

Best I can tell, this is running now.


.au-text-input--number: monospaced font, or font-variant property?

Right now, if we add the au-text-input--number it uses our monospace font like so:
It feels kind a bit inconsistent, having different font for number inputs.

I think we could make use of the font-variant: tabular-nums property, and keep the letter spacing, so it looks a bit like this instead:

Are we expecting the au-text-input--number class to always be combined with <input type="number">?
As an example, mobile number inputs could have non-numeric symbols like plus signs, parentheses, etc. and may be better suited to using simple <input type="text"> whilst inheriting default text input styling.
This puts the onus of input data cleanup back on the developer, but allows for much more forgiving (and predictable) text inputs from users.
Related:
https://bradfrost.com/blog/post/you-probably-dont-need-input-typenumber/
https://html.spec.whatwg.org/multipage/input.html#number-state-(type%3Dnumber)
https://developer.mozilla.org/en-US/docs/Web/CSS/font-variant-numeric


archivesBaseName in build.gradle of app module

What is the purpose of archivesBaseName in build.gradle of app module?

This issue can also be extended for GIT_SHA variable that is added to buildConfigField. Seems like both (archivesBaseName & GIT_SHA) variables are not being used in the project.

Hi @gokhanbarisaker, archivesBaseName is a gradle convention for setting the name of the output file. I use this to customise the file name, e.g. to plaid-0.5.5-release.apk.  See the gradle docs for more info.
The GIT_SHA build config isn't currently being used. I made that available as it can be extremely useful for crash reporting (see this great article).  It could be removed but I don't see any point in doing so as it might be useful in the future and it's nice to demonstrate this technique.

Awesome trick, thanks :+1:

@nickbutcher How can you add buildFlavor in archivesBaseName? Can you share how did you configure that?

android {
    ...

    defaultConfig {
        ...
        archivesBaseName = 'myDefaultName.'
    }

    // Specifies one flavor dimension.
    flavorDimensions "version"
    productFlavors {
        full {
            versionNameSuffix "-full"
        }
        demo {
            versionNameSuffix "-myDemoSuffix"
        }
    }

    variantFilter { variant ->
        def name = variant.flavors*.name
        archivesBaseName = 'my-flavor-name-is-' + name + '-awesome'
    }
    ...

You can override archivesBaseName anywhere in the android block. versionNameSuffix can already be used to differentiate between APKs generated for different flavors, but you can have more control if you like by setting archivesBaseName in the variantFilter function.


wt command-line cannot consistently parse more than one argument

Bottom line: the more arguments I put on wt the less likely it is to work. This is currently unusable even for launching the app clean.
Something like this, in a run dialog:
wt new-tab -p "PowerShell Core"; split-pane -H -p "Windows PowerShell"; split-pane -p cmd

Seems to work less than one time in five on my computer. Most of the time I get two panes, not three, and occasionally only one.
Environment
Windows build number: Microsoft Windows [Version 10.0.18363.657]
Windows Terminal Version: 0.9.433.0

Steps to reproduce
Run the command above from a run dialog a half dozen times
Expected behavior
The window should open the same way every time, with three panes, one for each of the shells I have in Windows...
Actual behavior
Sometimes just PowerShell Core, sometimes both PowerShells, rarely all three including cmd.

I can confirm that I'm seeing the same behaviour.

If i run wt with multiple arguments I always get:

split-pane : The term 'split-pane' is not recognized as the name of a
cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

If i just run wt split-pane -H it will work fine

not recognized

You need to escape the ; so powershell doesn't consume it.

I can confirm that wt does something similar on my machine. Repeating the following command: wt ; split-pane -V ; split-pane -V has resulted in:

Two vertical panes, with the right-most vertical pane also split horizontally (what?)
Two vertical panes
No panes, just the default tab

I am seeing the same thing.
Repro
It repros with a very simple scenario...
Execute this multiple times and often times the last split-pane does not execute:
wt.exe new-tab; split-pane -V ;split-pane -H

I also reproduce it.

split-pane ,new-tab only works from Run, address bar and search bar, but does not work if executed from powershell or cmd

It works properly now! Thanks!


Give user a message when they have no responsibilities

In the test environment, users with the Initial Authorizer (e.g., mibarra-ia) role are able to login to ECMPS 2.0 with the CBSI Stage bypass enabled (screenshot 1).  When no facilities are returned for ECMPS 2.0 users, these users will receive the following message:
You have no facilities for which you are responsible. Please login to CBS to review and add facilities for which you are responsible.
Screenshot 1

Need message for user.

Thanks @annalbrecht, the first comment in this ticket has been updated with the message text. Please review, as needed, and update the status from Requirements & Design Need to Ready for Dev, thx.

@mark-hayward-erg Done: https://github.com/US-EPA-CAMD/easey-ecmps-ui/pull/1492

@EvanSun220 you will need to recreate the original PR so the necessary changes can be merged into develop at the appropriate time.

Roles that show the "no responsibilities" message: initial authorizer, data submitter,

Verified in test:

Roles that show the "no responsibilities" message upon login: Initial Authorizer, Sponsor, Submitter, Preparer
Roles that do not display the "no responsibilities" message: ECMPS Admin, ECMPS Analyst, CBS Admin


Typo "Jupyter notebook"→"Jupyter Notebook"

https://docs.microsoft.com/en-us/azure/cognitive-services/computer-vision/quickstarts/python-analyze

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

adding to a new PR
#please-close


fix: upgrade configs for Jest 24

[x] This is a small change

Description:
The motivation is that since Jest 24 setupTestFrameworkScriptFile is deprecated, and setupFilesAfterEnv should be used instead.
Recently, @cs01 suggested his changes in pull request #1162 (and closed it, unfortunately), so the idea of this pull request is to refine it and extend to all the affected places in Detox codebase.
In the pull request, I've bumped Jest to 24 in every project it is required and made sure to fix all the warnings I've stumbled across.

And I'd like to take this opportunity to thank @d4vidi for fixing demo-react-native-jest in https://github.com/wix/Detox/commit/02be7b84b8784dc53ae2fd5ec943ae74477b02f0, that helped this pull request to happen and hopefully will help with resolving https://github.com/wix/Detox/issues/1263 .

Lol, happy to help :)

Glad to see this being fixed. Yeah I made a PR back in Feb but it didn't get any response and the bot autoclosed it. This one looks better anyway 😄.


Chat completion endpoint cannot remember its previous messages

Confirm this is an issue with the Python library and not an underlying OpenAI API

[X] This is an issue with the Python library

Describe the bug

With this request, llm should answer my last question. But it says it doesn;t know. Is ChatPromptTemplate with chat completion endpoint supposed to remember the previous messages or I should manually added them in the latest user prompt?
Chat completion takes a list of messages

Response saying don't know previous message

To Reproduce
This is my code for setting up using semantic kerner which uses openai package in the end to send the request, basically it add the previous messages into the ChatPromptTemplate.
system_message = """
You are a technical supporter that help users' questions, you should give 1-2 sentences to explain the answer and code examples for the question based on only the following contexts.
You may find the contexts provide information for multiple potential answers, you can give up to 3 most relevant answers separated by numbers like 1. 2. 3.

Follow this pattern to answer the question:
Contexts:
- Document path_title1 : page_content1
&&&
- Document path_title2 : page_content2

Question: question

Answer: answer
"""

prompt_template = """
Contexts:
{{$context}}

Question: {{$input}}
"""

def preprocess_messages(messages : list):
    transformed_messages = []
    transformed_messages.append({"role": "system", "message": system_message})
    for message in messages:
      role = "assistant" if message["role"] == "bot" else message["role"]
      transformed_message = {"role": role, "message": message["content"]}
      if (role != "system"):
        transformed_messages.append(transformed_message)
    return transformed_messages

def create_chat_prompt_template_instance(kernel : Kernel, messages : list = []):
    req_settings = sk_oai.AzureChatRequestSettings(max_tokens=2000, temperature=0, extension_data={"chat_system_prompt": system_message})
    req_settings.unpack_extension_data()
    config = PromptTemplateConfig(completion=req_settings)
    template = ChatPromptTemplate(prompt_template, kernel.prompt_template_engine, config)
    if (messages and len(messages) > 0):
      processed_messages = preprocess_messages(messages)
      for message in processed_messages:
        template.add_message(message["role"], message["message"])
    function_config = SemanticFunctionConfig(config, template)
    return kernel.register_semantic_function("ChatBot", "rag_chat", function_config)

  # simplified calling
  messages = [{"content": "Q1", "role": "user"}, {"content": "A1", "role": "bot"},{"content": "Q2", "role": "user"}]
  query=messages[-1]['content']
  context['context'] = combined_documents
  context['input'] = query

  previous_messages = messages[:-1]
  chat_function = create_chat_prompt_template_instance(kernel=kernel, messages=previous_messages)

  response = await kernel.run_async(chat_function, input_context=context)

Code snippets
No response
OS
Ubuntu
Python version
python3.9
Library version
openai1.0

It seems like different wording will bring gpt's memory back, this is confusing

This does not look like a bug in the SDK (more like a prompt engineering question or a problem in the underlying model), so I'm going to go ahead and close this issue.
For help developing with the OpenAI API from fellow developers, I recommend the OpenAI Discord server!


Create scorecard.yml

Changes
Add a scorecard check that runs:

daily
on push to main

This job is created through the official scorecard template.
The SCORECARD_TOKEN has been created through the tekton-robot account and published as an org secret, so that other repos may used it as well in future. The token expires in 1year from today.
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/kind misc

@vdemeester @JeromeJu can I have an LGTM as well? 🙏


Highlights not cleared when cancelling vim.ui.input

Checklist

[X] Have you updated the plugin to the latest version on main branch?
[X] Have you checked the Breaking Changes issue?
[X] Have you read through :h nvim-surround to see if there might be any relevant information there?

Describe the bug
When cancelling the vim.ui.input prompt, any created highlights will not be cleared and stay permanently in the buffer.
To Reproduce

Move to any word in a buffer and type ysiwT.
Press <C-c>.

Expected behavior
The highlight should be cleared.
Additional context
None

Interesting... continuing the discussion here this seems like even more of a niche problem than I thought; this only happens when you use <C-c> to terminate the input, as opposed to <Esc> (which I normally use).

I think this is just the way Neovim's vim.fn.input / vim.ui.input work in general: cancelling normally, i.e. by closing the command line (I mapped this to ii, works with <esc> by default) returns nil as an input. However, <C-c> as a keyboard interrupt is somehow considered an error. Maybe this behavior should be changed in Neovim / Vim  itself.

While I agree, I don't think that's going to be changed anytime soon for parity reasons with Vim haha, as vim.fn.input() is just an internal call to VimL's input()(to my knowledge).


Support External Files Provided by TS Plugin

Problem
Support the new external files option that TS plugins can provide: https://github.com/Microsoft/TypeScript/pull/15308/
We currently allow TSServer plugin extensions to register themselves for custom language modes. The angular extension for example could register the angular plugin with support for the ng-html language. There were a few problems with the current implementation:

We start automatically syncing all ng-html files with TSServer.
If the angular plugin is not active, TS treats these ng-html files as TypeScript code

Potential Fix
On potential fix we discussed previously: add a plugin property on open requests. TypeScript would look at the plugin property and only start validating the file if has the corresponding plugin.

Tracked upstream by https://github.com/Microsoft/TypeScript/issues/17151

Discussions and planning work done for July. Proposed API: Microsoft/TypeScript#17151
Moving to August since we need an API change on the TS side to support this. This change would likely come with TS 2.5

Moving out to at least TS 2.6

Closing as out of date. Discussion of this feature is at https://github.com/Microsoft/TypeScript/issues/17151


Fix Decal clamping to positive values not being applied to RenderingServer

Follow-up to https://github.com/godotengine/godot/pull/61492.

This means the Decal editor gizmo could be incorrect compared to the actual rendering if you enter a negative value in the inspector.

Thanks!


Added poisson_rvs to Owl_distribution

Following the last PR #498, I've added poisson_rvs to Owl_distribution_generic and Owl_distribution as a module.
I've also replaced the loggamma function with the cephes implementation lgam (this was on the TODO).
On a side note, I find some of the documentation in owl_ndarray_maths_map.h slightly confusing. I'm still not entirely sure why exponential_rvs uses FUN24 and gaussian_rvs uses FUN25. I'm using FUN24 to define poisson_rvs, as like exponential_rvs it only has one parameter. It would be nice to have a more detailed description of these functions and maybe more informative names at some point? Just a thought.

It would be nice to have a more detailed description of these functions and maybe more informative names at some point

I particularly agree with this comment. Touching the C interface is very non-straightforward and would be good to have a large section, maybe in CONTRIBUTING.md that explains what is going on and how things work.

From what I can understand, LGTM

There will be one chapter dedicated to the underlying c core in the book, to help others in understandibng what happens behind the scene so they can modify the code.


brew-pull-circle.rb: Fix folder creation

Looks like I found a solution to my problems: https://github.com/Linuxbrew/brew/issues/759
This fix broke it: https://github.com/Linuxbrew/homebrew-developer/commit/7bc4927ddfb349d0c885c749b4350b99aa087c04
Not sure why the FileUtils::mkdir_p "#{tap}/#{issue}" do syntax does not work. The code inside the do/end block was never executed ...

The block form of mkdir_p is a Homebrew-specific extension that was removed in one of the 1.7.0 changes. Homebrew's mkdir do still works though.


What are the s3 permissions for pulumi login + pulumi stack select -c

What happened?
Currently getting access denied errors when attempting to create a stack. Obviously works as an Admin
commands:
pulumi login s3://xxxx
pulumi stack select dev -c

error: could not create stack: blob (key ".pulumi/locks/dev/51a7bb66-cf44-42dc-8a07-9d46c5ee1f06.json") (code=Unknown): AccessDenied: Access Denied
        status code: 403, request id: CSGEJDAD9NF65T23, host id: 8iPG4ObxwQDvxF+Bu1T9PRW9Qs0T1eCAaEEI4gmSjZFX/uXZAva180m+C0NERYfc8YbvrHv6OBE=

What are the least privilege permissions for pulumi s3 backend? Don't see them documented here
Seems like there would be a mix of KMS + S3 Bucket + S3 Object calls
Steps to reproduce
aws role with no permissions and a s3 bucket without a policy and a default s3 bucket KMS key
pulumi login s3://xxxx
pulumi stack select dev -c

Expected Behavior
successful s3 login
successful stack creation + operations
Actual Behavior
pulumi stack select dev -c --non-interactive --logtostderr  -C
error: could not create stack: blob (key ".pulumi/locks/dev/51a7bb66-cf44-42dc-8a07-9d46c5ee1f06.json") (code=Unknown): AccessDenied: Access Denied
        status code: 403, request id: CSGEJDAD9NF65T23, host id: 8iPG4ObxwQDvxF+Bu1T9PRW9Qs0T1eCAaEEI4gmSjZFX/uXZAva180m+C0NERYfc8YbvrHv6OBE=

Output of pulumi about
CLI
Version      3.45.0
Go Version   go1.19.2
Go Compiler  gc

Host
OS       darwin
Version  12.6
Arch     x86_64

Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

actually answering my own question
        {
            "Sid": "AllowListBucket",
            "Effect": "Allow",
            "Action": [
                "s3:ListBucket",
                "s3:GetBucketVersioning"
            ],
            "Resource": "arn:*:s3:::<bucketname>"
        },
        {
            "Sid": "KMS",
            "Effect": "Allow",
            "Action": [
                "kms:Encrypt",
                "kms:Decrypt",
                "kms:GenerateDataKey"
            ],
            "Resource": [
                "*"
            ]
        },
        {
            "Sid": "AllowUpdateState",
            "Effect": "Allow",
            "Action": [
                "s3:PutObject",
                "s3:GetObject",
                "s3:DeleteObject"
            ],
            "Resource": [
                "arn:*:s3:::<bucketname>/*"
            ]
        },


Unselect all in Confirmed Case Trajectories by Districts graph

Thanks for all your hard work maintaining this website, it has been really useful.
Can you consider adding an unselect all option in the districts graph, it would be really useful to have. I want to see stats for my own district alone and having to unselect the other 8 districts takes a lot of time when this is done daily.

The Trajectories graph is drawn with a charting library c3.js. This feature comes with the library. We will try to resolve this, but unfortunately, there is no time frame for a fix. Feel free to issue a pull request, if possible.

Closing this issue - unselect all feature exists with alt+click, found this in a closed c3js PR.
https://github.com/c3js/c3/pull/1533

Thank you for mentioning it, we will try to add a short description in the website to use the atl+click to hide all.


Additional strange symbols near filename and encoding

environment

vim: 7.4.1689
vim-airline: 0.9
OS: Ubuntu 16.04
Have you reproduced with a minimal vimrc: yes
vimrc config link
terminal: xfce4-terminal and pantheon-terminal
$TERM variable: xterm-256color
color configuration (:set t_Co?): t_Co=256
if you are using Neovim: No
does it happen in Vim: Yes

Before creating new issue, i have read almost all similar issues by keyword "symbol".
actual behavior
I have a problem with weird unknown symbols in the fileformat placement and encoding sections.
Here is screenshot without let g:airline_powerline_fonts = 1 in .vimrc , i've commented it.

I want to NOTICE, these symbols are not undisplayed arrow symbols, because i see it in normal, font-enabled mode too

I have tried to edit
let g:airline_section_y (fileencoding, fileformat) from doc/airline.txt
by inserting
let g:airline_section_y = 'NOT_HERE' into the .vimrc, but it gave me only this

let g:airline_section_x = 'NOT_HERE' X section - the same nothing

[x] For the test, I even deployed vim in docker, latest ubuntu
[x] I disabled ALL plugins by PlugClean (vundle), and i still see this symbols
[x] Fonts: powerline fonts
[x] :echo "\ue0b2" Prints left arrow

Please, tell me how can i remove them?

Either your $TERM configuration is wrong, or your terminal does not work correctly.

Hello @chrisbra ,
I have the same problem. Can I ask you for help? I don't know what is bad set in my terminal:
echo $TERM
xterm-256color

well most likely your terminal is not really xterm compatible. What terminal are you using?

I'm using Tilix and Yakuake. Now I found out that in Yakuake is statusline alright. But in Tilix not. I tried to compare ENV set in every terminal - there are some differences but cannot if there is some important.

I don't know neither of them. However, what usually happens is, that Vim sends some terminal escape sequence to query the terminal (because it pretends to be xterm compatible) but then the terminal does not understand those and simply echos those on your screen. And that is wrong. So you need to figure out, what the correct $TERM variable for your terminal is. It is usually nothing about your environment variables.
Setting some terminal related options like
set t_u7=
set t_RV=

or some similar settings might fix it.

Thx @chrisbra for your answers. I found out that Tilix is GTK based and there could be some problem with xterm compatibily (issue on github/tilix).


 When you type an invalid month (e.g. 43), then you can set a 4-digit year, when it's only meant to be a 2-digit year.
It is expected that when you enter an invalid month, you can still only enter a 2-digit year.

This is now in progress! (#6)


Scaffold for Statistics block on Landing pages

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
What
Adds a statistics block to display CSV data on landing pages
Why
Trello card
How
Screenshots

Rendered version: https://govuk-frontend-app-pr-4261.herokuapp.com/landing-page

@KludgeKML Thanks for your review. I've removed the unnecessary modelling and tidied up the commit history.


Export to JSON

Steps to reproduce the problem:

Log flows
Want to read flows from other language from python
Can't export one or multiple flow to JSON

Any other comments? What have you tried so far?
Ideally I could export all the flows matching a filter to JSON
System information
Mitmproxy: 4.0.4
Python:    3.7.0
OpenSSL:   OpenSSL 1.1.0i  14 Aug 2018
Platform:  Darwin-17.7.0-x86_64-i386-64bit

Found a way to do it by hooking up a script to mitmdump.
from mitmproxy import http
import json

def response(flow: http.HTTPFlow) -> None:
    if (flow.request.method == "x"):
        if (flow.request.host == "x"):
            if (flow.request.path == "x"):
                with open('out.json', 'w') as out:
                    json.dump(flow.response.text, out)

Also mitmdump -s script.py "~u mypath" should probably filter the requests the script receives

@cwardata Hello there! Please, use this script to export flows into json - https://github.com/mitmproxy/mitmproxy/pull/2861
We can't get it merged because of the other blocking issue, but it should work well.
Please, reopen if you need anything :)
Good luck!

@kajojify Thanks a lot. I was kind of confused at first about how to use mitmproxy / mitmdump but now it's all good and I think it's one of best software ever made! Thanks a lot to all the team!


Reverting fix done in #573 due to regression.

Platforms Impacted

[x] iOS
[ ] macOS

Description of changes
This change reverts the fix done in b9ff9f0691b161d51b1bb4ee9badf3ae4b616609 (#573) as it caused a regression in client apps where the text labels were squeezed into a smaller width by the accessory view on larger screens (iPad).
Let's revert the regressing change for now and prepare a proper fix in a separate PR.
Verification
Ran the demo app containing the reverted code to ensure the Table View Cell works correctly and confirming that the bug fixed by #573 reproduces again.
Pull request checklist
This PR has considered:

[ ] Light and Dark appearances
[ ] VoiceOver and Keyboard Accessibility
[ ] Internationalization and Right to Left layouts
[ ] Different resolutions (1x, 2x, 3x)
[x] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)

Microsoft Reviewers: Open in CodeFlow

can you write description that says which commit has you are reverting rather than the PR number, please?

make sure when you squash, it also has the commit hash description for keeping the history.


Fix the i386 build by signing the temp dylib linked to by a test.

@swift-ci test

@gottesmm Can this be responsible for failing tests in #24898?


process manager: CPU statistics are much lower than actual

Issue Type: Bug
The process management of vscode shows that the CPU usage is almost 0.
But in fact, it takes up most of the CPU.
Even higher than Chrome’s occupancy.

 If you still see this problem, can you try changing the MacOS Activity Monitor with "View > All Processes, Hierarchically"?

https://user-images.githubusercontent.com/9758711/109244687-1747db00-781a-11eb-821e-850e2ca4c004.mov
This is a video I recorded. It can be seen that the CPU counted by vscode is not consistent with the CPU counted by the system monitor.
And my fan has been spinning frantically, indicating that the CPU is not idle like the vscode statistics.

Thanks! It looks like there's an orphaned process that is causing the high CPU usage. @alexdima Have you investigated this kind of problem in the past or know someone else on the team who has? I guess as a first step, @axetroy, could you try running with extensions disabled and see if you still see this behavior?

cc @bpasero @deepak1556
@RMacfarlane the next steps would be to try to find out which of our processes is becoming orphaned, by looking at the problematic process command line arguments.

Yup I would start with command line to identify which child process got orphaned.
@axetroy can you provide the output of ps -p <PID> | xargs and also take a sample of the process using the following command,
xcrun xctrace record --template 'Time Profiler' --output /tmp/code.trace --attach <PID>

@deepak1556
https://user-images.githubusercontent.com/9758711/109306490-35ddbe80-787a-11eb-886a-7a1ef6bf7175.mov
Here is the pid information
 1903 ??        20:30.21 /Applications/Visual Studio Code - Insiders.app/Contents/Frameworks/Code - Insiders Helper (Renderer).app/Contents/MacOS/Code - Inside

from now, I don’t know why it has this isolated process
But I guess that the process may not be killed when the window is reloaded.
https://user-images.githubusercontent.com/9758711/109306857-bc929b80-787a-11eb-9871-129daf72e86c.mov

@axetroy please pipe the ps output to xargs otherwise it will be cut off to the terminal width, in the above output we don't see the command line arguments because of it and it is important to know the arguments for us to identify which process it is.

@axetroy please pipe the ps output to xargs otherwise it will be cut off to the terminal width, in the above output we don't see the command line arguments because of it and it is important to know the arguments for us to identify which process it is.

But I guess that the process may not be killed when the window is reloaded.

that is a possible scenario, but I haven't been able to repro this.

PID TTY TIME CMD 17713 ?? 1:00.84 /Applications/Visual Studio Code - Insiders.app/Contents/Frameworks/Code - Insiders Helper (Renderer).app/Contents/MacOS/Code - Insiders Helper (Renderer) /Users/axetroy/.vscode-insiders/extensions/dbaeumer.vscode-eslint-2.1.14/server/out/eslintServer.js --node-ipc --clientProcessId=17644

oh! This seems to be a problem with eslint.
Thank you for your help. Should this issue be closed? Or vscode can guarantee that the extension cannot have orphaned processes

vscode can guarantee that the extension cannot have orphaned processes

Not possible with the current implementation, as extensions are free to choose how they spawn the process using node.js child_process api.

Should this issue be closed?

Will let @alexdima or @RMacfarlane decide on that, let me know if I can help further.

We cannot control what extensions do w.r.t. their spawned processes, so I suggest we continue tracking in https://github.com/microsoft/vscode-eslint/issues/1191


feat(prettier): when calculating the remaining width, the docs of the remaining commands should be included

Current dependencies on/for this PR:

main

PR #1441   👈

PR #1442

PR #1443

This stack of pull requests is managed by Graphite.

Adding @camc314 for review because I didn't work on this part 😅

Merge activity

Nov 20, 3:58 AM: @Boshen started a stack merge that includes this pull request via Graphite.


table progress

I think the default progress bar wastes too much horizontal space. The progress bar itself is not very informative and the metric names are unnecessarily repeated in every line.
I have added a new verbosity option verbose=3 which prints the values in a more compact table, which looks something like this:
[0]Step/Epoch | [1]Time | [2]loss | [3]mean_io_u | [4]sparse_categorical_crossentropy_loss
[5]val_loss | [6]val_mean_io_u | [7]val_sparse_categorical_crossentropy_loss

###[0]###|###[1]###|###[2]###|##[3]##|###[4]###|####[5]###|##[6]##|####[7]###
       0 |  147.8s | 282.181 | 0.126 | 282.181 | 1022.609 | 0.189 | 1022.609
       1 |  110.8s | 169.502 | 0.202 | 169.502 | 1018.721 | 0.209 | 1018.721
       2 |  110.9s | 145.796 | 0.229 | 145.796 |  932.413 | 0.245 |  932.413
       3 |  110.8s | 133.016 | 0.246 | 133.016 |  706.056 | 0.285 |  706.056
       4 |  110.5s | 124.921 | 0.258 | 124.921 |  825.170 | 0.280 |  825.170
       5 |  110.5s | 113.782 | 0.277 | 113.782 |  647.269 | 0.311 |  647.269
       6 |  110.5s | 108.972 | 0.284 | 108.972 |  574.973 | 0.343 |  574.973
       7 |  110.5s | 100.025 | 0.304 | 100.025 |  524.471 | 0.369 |  524.471
       8 |  110.5s |  94.226 | 0.319 |  94.226 |  540.708 | 0.370 |  540.708
       9 |  110.5s |  91.576 | 0.328 |  91.576 |  482.633 | 0.394 |  482.633
 317/595 |   55.7s |  83.841 | 0.382 |  83.841

I like this! I remember that Skorch had a progress bar like this and it was nice. Should we consider having an option for embedding the name of each column directly into the header instead of having this separate mini-table? I know that we are trying to reduce horizontal space but for simple cases it might look nicer.
Take a look in cell #17 from this notebook how they render their tables.

Sure, if the metric names are short enough.
Would be even better to remove the metrics [4] and [7] from the example above because they are duplicates

Codecov Report

Merging #127 (c70b77a) into master (68efe27) will decrease coverage by 0.68%.
The diff coverage is 17.46%.

@@            Coverage Diff             @@
##           master     #127      +/-   ##
==========================================
- Coverage   79.32%   78.64%   -0.69%
==========================================
  Files         110      110
  Lines        5210     5267      +57
==========================================
+ Hits         4133     4142       +9
- Misses       1077     1125      +48

Impacted Files
Coverage Δ

elegy/model/model.py
69.17% <ø> (ø)

elegy/callbacks/progbar_logger.py
63.52% <17.46%> (-14.44%)
:arrow_down:

elegy/data/array_adapter.py
90.16% <0.00%> (-0.16%)
:arrow_down:

Continue to review full report at Codecov.

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


iCheckbox using class="flat" will not trigger javascript

I've been struggling with this one for days and days.
I have tested this in form.html to ensure there wasn't something in my app causing it.
I've got an input :
<input type="checkbox" id="MyCheckbox" class="flat">
with javascript which adds an event listener:
function checkboxvalidate() {
    if (document.getElementById('MyNewCheckbox').checked) {
        alert("checked");
    } else {
        alert("You didn't check it! Let me check it for you.");
    }
}

document.getElementById('MyNewCheckbox').addEventListener('change', checkboxvalidate);

So long as class="flat" is in the input, the javascript will not trigger. As soon as I remove the class, javascript works fine.

Seems to be working fine.
https://jsfiddle.net/mickysp/pcsz147v/4/
I did change the id of the inputfield though. (you missed out the "new")

@micksp Your jsfiddle does not include the css from the class="flat". Please test this in form.html and I'm pretty sure any other html in the project.

For that you'll need to listen to ICheck events  ifChecked or ifUnchecked something like:
$('#MyCheckbox').on('ifChecked', function(event){
  alert("checked");
});

Now, if you don't want to use ifUnchecked and ifChecked you can use ifToggled that check both of them and then trigger change event on your input, so that way is general and your event will work
$('yourInput').on('ifToggled', function(e) {
  $(e.target).trigger('change');
});

ok yeah I've made this even worse by placing it in a datatable. Thanks for the help. I'm going to try and figure out how to get around the id field.

for anyone else finding this-
`    var checkboxes = document.getElementsByClassName("invcheckbox");
for (var i = 0; i < checkboxes.length; i++) {
document.getElementById(checkboxes[i].id).addEventListener('change', checkboxvalidate(checkboxes[i].id));
}
function checkboxvalidate(checkboxid) {
    //console.log(checkboxid);
    $('#' + checkboxid).on('ifChecked', function (event) {
        $('#tags_1').addTag(checkboxid.replace(/_/g, ' / '));
    });
    $('#' + checkboxid).on('ifUnchecked', function (event) {
        $('#tags_1').removeTag(checkboxid.replace(/_/g, ' / '));
    });
}`


Dashboard not working

The dashboard https://stats.galaxyproject.eu/d/000000020/public-galaxy-servers?orgId=1 doesn't work - is this repo still maintained @martenson or did this move somewhere else?

I was responsible for running this on EU and it worked while I was there. Now I guess @mira-miracoli @sanjaysrikakulam are responsible for why it isn't working. It used to be a process running directly on the grafana VM that provided data for that dashboard. It'd be super cool to have it running again

@hexylena yeah I agree - getting an idea of where a Galaxy instance is on a map is super useful!

no nothing moved. This code just collects the information and aggregates it, ready for grafana and influxdb ingestion. The process running that needs to be on grafanas VM in the current setup.

The json_dir (pgs_web_dir) was not getting created due to permission error. I manually made it (set it to 777, which is not ideal) for testing and ran the cron job. Now the dashboard is updated. I will keep an eye on and add a task to our pgs role to set up the proper permissions.
May I ask how this list is updated? Looks like the list needs to be updated because I see lots of request errors and this is probably due to the non-existence of the probed site.

Ahh easy fix at least.

May I ask how this list is updated? Looks like the list needs to be updated because I see lots of request errors and this is probably due to the non-existence of the probed site.

Yes, it's just whichever sites are listed in https://github.com/galaxyproject/galaxy-hub/tree/master/content/use, it is expected that there are a lot of request errors, that is normal. Some sites aren't public (and aren't filtered out), some are old and deprecated. It would be nice to eventually prune that list, but that's more likely to cause protracted discussion for a small benefit (and I guess the project likes the large number of servers listed there.)

URL is fixed in readme, good catch @sanjaysrikakulam !

Cool, thank you for the info. This issue can now be resolved.

Thanks so much for fixing it @sanjaysrikakulam, it's appreciated!


Apache Zookeeper: Periodic connection loss

What happened:
After migrating from Calico to Amazon VPC CNI Addon, we observed problems with Strimzi Kafka, more precisely with Apache Zookeeper.
Strimzi installs a network policy by default to allow communication between Zookeeper pods (and other Strimzi related pods).
So in a cluster with no network policies besides these default Zookeeper policy, the following happens in a namespace with 3 zookeeper replicas:

Leader election is successful, Zookeeper pods communicate without problems
After ~ 5 Minutes zookeeper pods lose connection
Connection is established again, (Leader election)
After ~ 5 Minutes zookeeper pods lose connection
... (repeat)

Attach logs
In the Zookeeper logs the connection loss usually shows up like this:
[myid:2] - ERROR [LearnerHandler-/<IP_ADDRESS>:60242:LearnerHandler@714] - Unexpected exception causing shutdown while sock still open
  java.net.SocketTimeoutException: Read timed out
...
[myid:2] - WARN  [LearnerHandler-/<IP_ADDRESS>:60242:LearnerHandler@737] - ******* GOODBYE /<IP_ADDRESS>:60242 ********

or
[myid:2] - ERROR [LearnerHandler-/<IP_ADDRESS>:42736:LearnerHandler@714] - Unexpected exception causing shutdown while sock still open
  java.io.EOFException
...
[myid:2] - WARN  [LearnerHandler-/<IP_ADDRESS>:42736:LearnerHandler@737] - ******* GOODBYE /<IP_ADDRESS>:42736 ********

What you expected to happen:
Established connections do not get dropped (periodically).
How to reproduce it (as minimally and precisely as possible):
We were able to make it easily reproducible by installing Zookeeper from Bitnami and applying a Strimzi-like network policy:
Install zookeeper with 3 replicas
helm repo add bitnami https://charts.bitnami.com/bitnami
helm upgrade --install zookeepertest bitnami/zookeeper --version 10.2.5 --set replicaCount=3 --set logLevel=INFO

Install "Strimzi-like" network policy:
apiVersion: networking.k8s.io/v1
kind: NetworkPolicy
metadata:
  name: netpol-zookeepertest
spec:
  ingress:
  - from:
    - podSelector:
        matchLabels:
          app.kubernetes.io/instance: zookeepertest
    ports:
    - port: 2888
      protocol: TCP
    - port: 3888
      protocol: TCP
  - from:
    - podSelector:
        matchLabels:
          app.kubernetes.io/instance: zookeepertest
    ports:
    - port: 2181
      protocol: TCP
  - ports:
    - port: 9404
      protocol: TCP
  podSelector:
    matchLabels:
      app.kubernetes.io/instance: zookeepertest
  policyTypes:
  - Ingress
status: {}

Observe Zookeeper logs.
Anything else we need to know?:
Something interesting we observed:

The issue can be "fixed" by adding an additional network policy, allowing ingress to additional ports in the range ~ 40000-65000

apiVersion: networking.k8s.io/v1
kind: NetworkPolicy
metadata:
  name: default-policy
spec:
  ingress:
  - from:
    - podSelector:
        matchLabels:
          app.kubernetes.io/instance: zookeepertest
    ports:
    - port: 40000
      endPort: 65000
      protocol: TCP
  podSelector:
    matchLabels:
      app.kubernetes.io/instance: zookeepertest
  policyTypes:
  - Ingress
status: {}

Wild guess why this workaround works: The range 40000-65000 is the range of the source ports (as you can see in the logs above: GOODBYE /<IP_ADDRESS>:60242. Maybe there is a bug in the policy agent causing a loss of state after x minutes. After the state loss the communication to the source port 60242 is no longer known/accepted. With the workaround policy, however, the port range is explicitly allowed.
Environment:

AWS EKS, Kubernetes 1.26
CNI Version: amazon-k8s-cni:v1.15.4-eksbuild.1, aws-network-policy-agent:v1.0.6-eksbuild.1

@xashr - This is similar to https://github.com/aws/aws-network-policy-agent/issues/144. We have a fix for this and I can provide you a release candidate image if you are willing to try it out.

@jayanthvn - Is there an RC newer than v1.0.7-rc1 ? Sounds like #144 is not fixed yet according to Rez0k?

@jayanthvn We are staying with Calico for now, but I ran a short test with the rc3 image in a separate cluster. The issue seems to be solved with that image. Thanks!


Create AzureHttpClient singleton

Description:

Converted the HttpClient to a singleton.
Implemented changes to send HTTP request to Azure Functions app only if the current location is over 30 meters of the last location of the last request.

Type of change:

[ ] Bug fix
[ ] New feature
[x] Fix/new infrastructure

Background context of the changes:

For making the HttpClient available to all our app's pages, I've created a class called "AzureHttpClient", so every page could use the same HttpClient, instead of creating the same instance, with the same logic over and over again.
For reducing the load on our Azure Function, an HTTP request will be sent only if the current location is over 30 meters on the last request sent.

Scenarios that were tested during this implementation:
All different modes of the application (running, in the background and killed)

Great job


required_pull_request_reviews in github_branch_protection isn't applying

Hi there,
Terraform Version
0.9.11 (Last time I knew this worked was 0.9.5, but we upgraded and it started having this issue)
Affected Resource(s)

github_branch_protection

We are seeing a similar output with the vcd_vapp.ip, but I'm not sure that its the same issue. We aren't seeing it with any other resources.
Terraform Configuration Files
resource "github_branch_protection" "myrepo_master" {
    repository = "${github_repository.myrepo.name}"
    branch = "master"
    required_status_checks {
        include_admins = true
        strict = true
        contexts = []
    }
    required_pull_request_reviews {
        include_admins = true
    }

}

Debug Output
https://gist.github.com/gregswift/5a934aa4a638701ec1a73123b84fb788 (encrypted with hashicorp gpg key)
Panic Output
No panic
Expected Behavior
The configuration should be set in GitHub to require reviews and include admins.
Actual Behavior
The setting isn't taking so every time we run terraform apply it wants to re-apply the setting.
github_branch_protection.deployment-control_master: Modifying... (ID: myrepo:master)
  required_pull_request_reviews.#:                "0" => "1"
  required_pull_request_reviews.0.include_admins: "" => "true"

Steps to Reproduce

Create github_branch_protection that matches the one above
terraform apply

Important Factoids
This is against GitHub:Enterprise 2.8.  The vcd_vapp thing I was talking about above is that we have it set to 'allocated', which is a thing in the documetnation that has worked for a while for us, but now it keeps storing the IP in the state and wanting to change it back to 'allocated' every run. This doesnt actually change anything as far as we can tell, whichs i why it appears simiar, in that it keeps wanting to apply the change. but here it isnt applying the change. With vcd_vapp it seems more of a statefile handling issue.
References
terraform-providers/terraform-provider-vcd#29 for that issue

Hey @gregswift, thanks for the issue!
I believe #26 should contain the fix that you need. Please ping if it doesn't solve the issue you're seeing though. Thanks!

@grubernaut so this wont be anywhere until 0.10 ?

@gregswift Correct, we can release v0.1.1 of the github provider (probably should do that now), and it'll be present in the latest 0.10 beta of Terraform.


Update NTTileView

:green_heart: Summary

FIX: X position of tile allowed it to be partially draw off-screen
UPDATE: Renamed DataSource methods

:+1:


Location Onboarding

tldr:

asking for location permission
handling error cases
urging to start a certain start location
introduces floating-ui for the selection behaviour
localizable texts are in a separate file
use the location result after onboarding has been finished

(that is relatively poor, maybe there's a better way, because currently it's using useEffect)

https://github.com/user-attachments/assets/b7f206f2-c69b-4d1d-9381-684e5532d62c

@opyh location bias is in, merge when approved

Thanks for all the nice cleanups! :D


chore - move python solutions from root to /python

@neetcode-gh wanted to get your +1 on this before merging. figure we should probably keep the root dir cleaned up now that we have more languages coming in?

Thanks! This is good, but can you leave the original python files so it doesn't break any of the links on the site. I'll delete them up after I update the site with the new links, probably tonight.
Feel free to merge after resolving the conflicts.

@neetcode-gh gentle nudge on this one (though realize you have a day job, I actually just started at Google today, feel free to reach out!), we should remove the files in root ASAP so we don't get more contributions/PRs against those files/in that dir


Unable to install composer offline manually (composer.phar)

Hi
Tried to install composer manually in below procedure
1.Set the path variable with the location where php.exe exists
2.Downloaded latest composer.phar , copied that in php.exe location
3. Tried to run echo @php "%~dp0composer.phar" %*>composer.bat in the same php.exe path in command prompt
4.Closed that command window again reopened command window with same path tried to give command composer -V
Then i observed the following error
could not open input file : E:/wamp64/bin/php/php7.1.9/composer -V
could you please help me with this.

As I stated in the previous issue #98 , if you cannot use the windows installer then it makes it very difficult to find out what has gone wrong.
There are plenty of You Tube videos about installing Composer on Windows, and not all of these make use of Composer-Setup: perhaps one of these may help you spot what you have done wrong.
There are also some old notes in this repo about installing manually: https://github.com/composer/windows-setup/wiki/Manual-installation This explains how to use php from the command-line
and takes you through each step of installing and setting up Composer.

Closing, as no further comments.


Make Console more extensible

Problem description
Would like to be able to better extend the Console utility by allowing for adding more/alt commands.  In particular, I'm looking for a load command that can deal with file wildcards and skip invalid urls.
Preferred solution
Currently, many things are final and private in the Console module and I would like to make them non final and protected.  @jeenbroekstra
Are you interested in contributing a solution yourself?
Yes
Alternatives you've considered
No response
Anything else?
No response

All for making (some) things extendable but I would advise against making things protected for that purpose. If you want to make something explicitly available for third party implementation/extension, make it public.

Using protected makes sense where a class is intended to both be used by users and be extended by developers. Public methods would be visible to users, while protected methods would not.
A good use case is for abstract classes. Something like an abstract class SensibleFoo could have a public bar() method with sensible input checks and a protected abstract internalBar() method that allows developers to implement the code without having to duplicate the input checks.
I think we do something along those lines for our base connection implementations.


Airdrop

2nd month anything from keybase promised airdrop.

I am not recieved anything!! 2 month how joined and nothing

I AM RECIEVED GUYS! THANK YOU VERY MUCH!!!


text to speech functionalitiy added..

text to speech functionality is added ..further machine learning algorithm will be implemented


[Feat] Allow for push-only configuration of web3Inbox

Describe the solution you'd like

Pilot partners such as Spot are looking to leverage web3inbox SDK to bootstrap Push UI, without involving additional Chat functionality.
web3Inbox should allow for the chatProvider logic to be disabled completely via query parameter config

Suggested implementation

Allow for a new chatProvider=disabled value in w3iProxy, which disables chat functionality entirely when specified.
If chatProvider=disabled

land the user on /notifications instead of /messages by default after logging in
hide the Messages tab item in the UI and
hide Chat-specific settings

Additional context

See thread: https://walletconnect.slack.com/archives/C03RWGSBCUU/p1682433835879819

Instead of making a push focused approach, a generic ui manipulation approach would be better for partners that will only want the chat ui

Instead of making a push focused approach, a generic ui manipulation approach would be better for partners that will only want the chat ui

@devceline Can you adjust the description pls based on what you've now implemented in terms of the more generic approach? 🙏


[Question] Merging LoRA weights with lmsys/vicuna-13b-v1.5 instead of liuhaotian/llava-v1.5-13b

Question
Hi I finetune my own model with LoRA, new-v1.5-13b-lora-665k-custom using finetune_lora.sh but have trouble merging the LoRA weights with lmsys/vicuna-13b-v1.5 backbone.
Can anyone shed some light? Thank you!
The command I used is

python scripts/merge_lora_weights.py \     --model-path ./checkpoints/new-v1.5-13b-lora-665k-custom \     --model-base lmsys/vicuna-13b-v1.5 \     --save-model-path ./checkpoints/merged/new-v1.5-13b-lora-665k-custom

And error is below

Loading LLaVA from base model...
Loading checkpoint shards:   0%|                                                       | 0/3 [00:00<?, ?it/s]/home1/XXX/.conda/envs/llava2/lib/python3.10/site-packages/torch/_utils.py:831: UserWarning: TypedStorage is deprecated. It will be removed in the future and UntypedStorage will be the only storage class. This should only matter to you if you are using storages directly.  To access UntypedStorage directly, use tensor.untyped_storage() instead of tensor.storage()
return self.fget.get(instance, owner)()
Loading checkpoint shards: 100%|███████████████████████████████████████████████| 3/3 [00:19<00:00,  6.36s/it]
Some weights of LlavaLlamaForCausalLM were not initialized from the model checkpoint at lmsys/vicuna-13b-v1.5 and are newly initialized: ['model.mm_projector.0.bias', 'model.mm_projector.0.weight', 'model.mm_projector.2.bias', 'model.mm_projector.2.weight']
You should probably TRAIN this model on a down-stream task to be able to use it for predictions and inference.
/home1/XXX/.conda/envs/llava2/lib/python3.10/site-packages/transformers/generation/configuration_utils.py:392: UserWarning: do_sample is set to False. However, temperature is set to 0.9 -- this flag is only used in sample-based generation modes. You should set do_sample=True or unset temperature. This was detected when initializing the generation config instance, which means the corresponding file may hold incorrect parameterization and should be fixed.
warnings.warn(
/home1/XXX/.conda/envs/llava2/lib/python3.10/site-packages/transformers/generation/configuration_utils.py:397: UserWarning: do_sample is set to False. However, top_p is set to 0.6 -- this flag is only used in sample-based generation modes. You should set do_sample=True or unset top_p. This was detected when initializing the generation config instance, which means the corresponding file may hold incorrect parameterization and should be fixed.
warnings.warn(
Loading additional LLaVA weights...
Loading LoRA weights...
Merging LoRA weights...
Model is loaded...
Some non-default generation parameters are set in the model config. These should go into a GenerationConfig file (https://huggingface.co/docs/transformers/generation_strategies#save-a-custom-decoding-strategy-with-your-model) instead. This warning will be raised to an exception in v4.41.
Non-default generation parameters: {'max_length': 4096}
Your generation config was originally created from the model config, but the model config has changed since then. Unless you pass the generation_config argument to this model's generate calls, they will revert to the legacy behavior where the base generate parameterization is loaded from the model config instead. To avoid this behavior and this warning, we recommend you to overwrite the generation config model attribute before calling the model's save_pretrained, preferably also removing any generation kwargs from the model config. This warning will be raised to an exception in v4.41.
Traceback (most recent call last):
File "/home1/XXX/.conda/envs/llava2/lib/python3.10/site-packages/transformers/generation/configuration_utils.py", line 558, in save_pretrained
raise ValueError(str([w.message for w in caught_warnings]))
ValueError: [UserWarning('do_sample is set to False. However, temperature is set to 0.9 -- this flag is only used in sample-based generation modes. You should set do_sample=True or unset temperature.'), UserWarning('do_sample is set to False. However, top_p is set to 0.6 -- this flag is only used in sample-based generation modes. You should set do_sample=True or unset top_p.')]

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/project/LLaVA/scripts/merge_lora_weights.py", line 22, in
merge_lora(args)
File "/project/LLaVA/scripts/merge_lora_weights.py", line 10, in merge_lora
model.save_pretrained(args.save_model_path)
File "/home1/XXX/.conda/envs/llava2/lib/python3.10/site-packages/transformers/modeling_utils.py", line 2364, in save_pretrained
model_to_save.generation_config.save_pretrained(save_directory)
File "/home1/XXX/.conda/envs/llava2/lib/python3.10/site-packages/transformers/generation/configuration_utils.py", line 560, in save_pretrained
raise ValueError(
ValueError: The generation config instance is invalid -- .validate() throws warnings and/or exceptions. Fix these issues to save the configuration.
Thrown during validation:
[UserWarning('do_sample is set to False. However, temperature is set to 0.9 -- this flag is only used in sample-based generation modes. You should set do_sample=True or unset temperature.'), UserWarning('do_sample is set to False. However, top_p is set to 0.6 -- this flag is only used in sample-based generation modes. You should set do_sample=True or unset top_p.')]

@haotian-liu
Do you think it is caused by the non-default max_length or do_sample=True? Thanks!

Same issue here

I solved this issue by downgrading the Transformer version with pip install<EMAIL_ADDRESS>


It seems that float inf, nan do not follow lisp convention

Float are handled in sxp writer as :
  def to_sxp
    case
      when nan? then 'nan.'
      when infinite? then (infinite? > 0 ? '+inf.' : '-inf.')
      else to_s
    end
  end
end

R6rs, guile use nan.0, +inf.0, -inf.0 rather than nan., +inf., -inf.
Shouldn't these conventions be followed ?
--
See

https://www.gnu.org/software/guile/manual/html_node/Reals-and-Rationals.html
http://www.r6rs.org/final/html/r6rs-lib/r6rs-lib-Z-H-12.html

Seems reasonable to me. I believe @artob wrote this originally. Hearing no concerns, I'd say we make the change.

Looks good to me.

Thanks you both for your quick feedback and follow-up.

I’ll release an update to the gem today.

It seems femtolisp follows this convention too
    if (tok[0] == '+') {
        if (!strcmp(tok,"+NaN") || !strcasecmp(tok,"+nan.0")) {
            if (pval) *pval = mk_double(fl_ctx, D_PNAN);
            return 1;
        }
        if (!strcmp(tok,"+Inf") || !strcasecmp(tok,"+inf.0")) {
            if (pval) *pval = mk_double(fl_ctx, D_PINF);
            return 1;
        }
    }
    else if (tok[0] == '-') {
        if (!strcmp(tok,"-NaN") || !strcasecmp(tok,"-nan.0")) {
            if (pval) *pval = mk_double(fl_ctx, D_NNAN);
            return 1;
        }
        if (!strcmp(tok,"-Inf") || !strcasecmp(tok,"-inf.0")) {
            if (pval) *pval = mk_double(fl_ctx, D_NINF);
            return 1;
        }

https://github.com/JuliaLang/julia/blob/master/src/flisp/read.c#L72-L90


Password errors do not show up in linode create flow

Example response:
{"errors": [{"reason": "Password must contain at least 2 of these 4 character classes: lowercase letters, uppercase letters, numbers, and punctuation", "field": "root_pass"}]}

This was a matter of the response headers omitting Access-Control-Allow-Origin which has been fixed.


Limit to 81 characters

Wonderful idea ! I try transfer from PC to Raspberry Pi, but rezult was:
C:\Python27>python serio.py -s paramiko-master.zip -p COM7 -b 115200 -d /home/pi
/alfa.zip
20 / 81
40 / 81
60 / 81
80 / 81
81 / 81
Uploaded 81 bytes from paramiko-master.zip to /home/pi/alfa.zip
C:\Python27>
alfa.zip have only 81 bytes, bat source have 348304 bytes. What I do wrong ? Thank you.

I modify in this way:
def put(self, source, destination):
data = open(source, 'rb').read()  ## to read binary file
data_size = len(data)
and now work fine. Thank you. Useful program !

Updated code to open binary file in binary mode explicitly. Thanks!


[Docker-Gen] Basic Dockerfile generation

Created the basic infrastructure around Dockerfile generation - currently supports generating just top level statements, but with no parameters.
The order of the statements is enforced, so that the first line must be a FROM clause.
Unit tested, integration tests to come once the full functionality exists.

Also can update this to be more in adherence with what Charlie did with the mkSQL stuff in the Postgres generator


Istio-pilot stuck waiting, "installing agent" on baremetal

On baremetal deployments of kubernetes 1.22 with charmed kubeflow 1.6/stable SQA is seeing istio pilot get stuck with the message "installing agent"
Looking at the logs in the pods, the unit actually seems to have the agent up, the charm is able to finish coming up and idle out, but the status never updates.
Here is a juju crashdump from the Kubernetes, the workers have logs for all the running pods:
https://people.canonical.com/~alexb/juju-crashdump-kubernetes-maas-2022-09-17-04.07.29.tar.gz

Hey @asbalderson, this seems to be an issue related to juju itself, what version of juju where you running? If the charm is at the "installing agent" stage none of our code has been executed yet, so we don't really have much control around that.

Thanks @DomFleischmann I'll follow up with the juju team then

I think this can be closed, but please reopen if I'm incorrect


Generate and verify concrete candidates in buckets(of size MaxV)

@zhengyangl Please review.
We might need some tweaks to the order in which 'components' are specified to match the global cost order.

LGTM. Need to get the failed cases passing, though.

LGTM


Move non returning types from the runner to sementic (class, type, instance of)

Further, these should not be parsed as expressions - rather be treated as magical structures that must be in the root scope of a file


AttributeError: '_tqdm_cls' object has no attribute '_lock'

Describe the bug
  File "/Users/codingl2k1/.pyenv/versions/3.11.4/lib/python3.11/site-packages/datasets/load.py", line 1034, in get_module
    data_files = DataFilesDict.from_patterns(
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/codingl2k1/.pyenv/versions/3.11.4/lib/python3.11/site-packages/datasets/data_files.py", line 671, in from_patterns
    DataFilesList.from_patterns(
  File "/Users/codingl2k1/.pyenv/versions/3.11.4/lib/python3.11/site-packages/datasets/data_files.py", line 586, in from_patterns
    origin_metadata = _get_origin_metadata(data_files, download_config=download_config)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/codingl2k1/.pyenv/versions/3.11.4/lib/python3.11/site-packages/datasets/data_files.py", line 502, in _get_origin_metadata
    return thread_map(
           ^^^^^^^^^^^
  File "/Users/codingl2k1/.pyenv/versions/3.11.4/lib/python3.11/site-packages/tqdm/contrib/concurrent.py", line 70, in thread_map
    return _executor_map(ThreadPoolExecutor, fn, *iterables, **tqdm_kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/codingl2k1/.pyenv/versions/3.11.4/lib/python3.11/site-packages/tqdm/contrib/concurrent.py", line 48, in _executor_map
    with ensure_lock(tqdm_class, lock_name=lock_name) as lk:
  File "/Users/codingl2k1/.pyenv/versions/3.11.4/lib/python3.11/contextlib.py", line 144, in __exit__
    next(self.gen)
  File "/Users/codingl2k1/.pyenv/versions/3.11.4/lib/python3.11/site-packages/tqdm/contrib/concurrent.py", line 25, in ensure_lock
    del tqdm_class._lock
        ^^^^^^^^^^^^^^^^
AttributeError: '_tqdm_cls' object has no attribute '_lock'

Steps to reproduce the bug
Happens ocasionally.
Expected behavior
I added a print in tqdm ensure_lock(), got a ensure_lock <datasets.utils.logging._tqdm_cls object at 0x16dddead0>  print.
According to the code in https://github.com/tqdm/tqdm/blob/master/tqdm/contrib/concurrent.py#L24
@contextmanager
def ensure_lock(tqdm_class, lock_name=""):
    """get (create if necessary) and then restore `tqdm_class`'s lock"""
    print("ensure_lock", tqdm_class, lock_name)
    old_lock = getattr(tqdm_class, '_lock', None)  # don't create a new lock
    lock = old_lock or tqdm_class.get_lock()  # maybe create a new lock
    lock = getattr(lock, lock_name, lock)  # maybe subtype
    tqdm_class.set_lock(lock)
    yield lock
    if old_lock is None:
        del tqdm_class._lock  # <-- It tries to del the `_lock` attribute from tqdm_class.
    else:
        tqdm_class.set_lock(old_lock)

But, huggingface datasets datasets.utils.logging._tqdm_cls does not have the field _lock: https://github.com/huggingface/datasets/blob/main/src/datasets/utils/logging.py#L205
class _tqdm_cls:
    def __call__(self, *args, disable=False, **kwargs):
        if _tqdm_active and not disable:
            return tqdm_lib.tqdm(*args, **kwargs)
        else:
            return EmptyTqdm(*args, **kwargs)

    def set_lock(self, *args, **kwargs):
        self._lock = None
        if _tqdm_active:
            return tqdm_lib.tqdm.set_lock(*args, **kwargs)

    def get_lock(self):
        if _tqdm_active:
            return tqdm_lib.tqdm.get_lock()

Environment info
Python 3.11.4
tqdm '4.65.0'
datasets master

Hi ! I opened https://github.com/huggingface/datasets/pull/6067 to add the missing _lock
Could you try that this works for you ? I wasn't able to reproduce your the error on my side

Maybe it is a bug of tqdm? I think simply use try ... except AttributeError ... wraps del tqdm_class._lock should work.

Yes it looks like a bug on their end indeed, do you want to open a PR on tqdm ?
Let me see if I can find a workaround in the meantime

I opened https://github.com/huggingface/datasets/pull/6068 if you want to try it out

I opened #6068 if you want to try it out

This fix works! Thanks.

Awesome ! closing this then :)
We'll do a patch release today or tomorrow


[Feature][UI] Support users can enter Kerberos configuration from the UI

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description

Remove the input box of hostname, port and databasename and replace it with the input box of jdbcurl parameter. Oracle servicename or Sid also needs to be removed.

Add Kerberos principal, Kerberos KeyTab and Kerberos krb5conf input boxes in the hive data source. If Kerberos startup state is true, it will be displayed, and false will not be displayed.

The following interface input format needs to be changed

/datasources/  POST  --createDataSource
/datasources/{id}  PUT  --updateDataSource
/datasources/connect POST  --connectDataSource
This is a sample parameter
{
"id": null,
"name": "zbg01_mysql",
"note": "test",
"dbType": "MYSQL",
"props": {
"user": "root",
"password": "123456",
"jdbcUrl": "jdbc:mysql://zbg01:3306/mydb",
"kerberosPrincipal": "",---The type is hive. Kerberos needs to be passed
"kerberosKeytab": "",---The type is hive. Kerberos needs to be passed
"kerberosKrb5Conf": "",---The type is hive. Kerberos needs to be passed
"props": {
"autoDeserialize": "yes",
"allowUrlInLocalInfile": "true"
}
}
}

yes,i also need it

I'm willing to develop this feature.

I'm willing to develop this feature.

so perfect


[Feature]: Include Static solver

What are you trying to do and how would you want to do it differently?
I would like to have a static solver.
Is it something you currently cannot do? Is this related to an issue/problem?
The Verlet solver is slow for linear undamaged problems. For load introduction it is a lot of computation time.
Implementation details
Check the Julia ODE Solver and try to create some form of iterative static solver.
Would you be willing to submit a PR?
Yes

Hello Fancisco,
if you need support, feel free to contact us.
Christian
Von: Francisco A. Yapor Genao @.>
Gesendet: Donnerstag, 22. August 2024 07:05
An: PeriHub/PeriLab.jl @.>
Cc: Christian Willberg @.>; Author @.>
Betreff: Re: [PeriHub/PeriLab.jl] [Feature]: Include Static solver (Issue #181)
Any interest in the adaptive dynamic relaxation method used in An adaptive dynamic relaxation method for quasi-static simulations using the peridynamic theory by B. Kilic  https://www.sciencedirect.com/science/article/pii/S0167844210000479 & E. Madenci? I started to get around the code, but I'm not ready to submit a PR yet.
—
Reply to this email directly, view it on GitHub https://github.com/PeriHub/PeriLab.jl/issues/181#issuecomment-2303793031 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AIHBTC6H6X5TAPWSXKTLZNTZSVWPXAVCNFSM6AAAAABMSGWQBWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBTG44TGMBTGE .
You are receiving this because you authored the thread.Message ID: @.***>


make source_id a string too

It's also not a real number. Oops.

@sethherr I beg your indulgence one more time :)


ensure => exported with array_type/append does not work as expected

file (before):
export JAVA_OPTIONS="-server -Xms512m"
modification
shellvar { "JAVA_OPTIONS":
ensure       => exported,
target       => "file",
value        => ["-Djavax.net.ssl.trustStore=test"],
array_type   => "string",
array_append => true,
}
file (after):
JAVA_OPTIONS="-server -Xms512m -Djavax.net.ssl.trustStore=test"
used components
Puppet client+server: 3.4.3
herculesteam-augeasproviders_core (v2.1.1)
herculesteam-augeasproviders_shellvar (v2.1.1)

I've added some code to this issue, which I believe should fix it.
It seems for some reason that it only converges in two runs though, causing the unit tests to fail. @domcleal do you have any idea why the value's insync? is not triggered on the first run?

It seems properties are only synced in the ensure property is already in sync…

Changes Unknown when pulling 434e38294b5efa8642af4e0319a7af07db73a7a0 on raphink:issue/10 into ** on hercules-team:master**.

I've added a manual sync of value when property is unset or exported, but I'm not sure if this is the best way to do it (at least, it fixes the issue).

Changes Unknown when pulling 353bbf9884f3be88a3f8eed5da7bd3e820567b5d on raphink:issue/10 into ** on hercules-team:master**.

@domcleal what do you think of the solution?

@raphink I'm a bit out of it so can't comment much..

@domcleal what do you think of https://github.com/raphink/augeasproviders_shellvar/commit/353bbf9884f3be88a3f8eed5da7bd3e820567b5d? Do you know of another way to do that?

No, unless you can call the underlying provider methods directly there, or inside the provider itself - but it looks convoluted, so probably not easy.