Set folder names matching names of components on diagram

Is your feature request related to a problem? Please describe.
Names of folders are confusing - rds_1, security-group_1...
Describe the solution you'd like
Use names from components, replace strange characters with single _
Additional context

@genaker This was way easier than I expected, fixed and deployed.
Thanks for the feedback!

Thanks Grate Job!
On Sun, Mar 22, 2020 at 8:32 AM Anton Babenko<EMAIL_ADDRESS>wrote:

@Genaker https://github.com/Genaker This was way easier than I
expected, fixed and deployed.
Thanks for the feedback!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/antonbabenko/modules.tf-lambda/issues/22#issuecomment-602226724,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACGJNZQRKCQSRBXZTX6VQV3RIYVRVANCNFSM4LF5QFHA
.

It can be an issue. I didn't try to run it but
dependency "aws-data" {
  config_path = "../aws-data"
}

dependency "Production-VPC" {
  config_path = "../Production-VPC"
}

dependency "SSH22-SecurityGroup" {
  config_path = "../SSH22-SecurityGroup"
}

dependency "WebNode-LoadBalancer" {
  config_path = "../WebNode-LoadBalancer"
}

where folder actually is lowercased
Possible solutions:

Lowercase this part of code generation also

  config_path = "../webnode-loadnalancer"

Don't lowercase the names of the folders

Fixed in 6d21738 by lowercasing directory name.

Thank! Awesome module!
On Tue, Mar 24, 2020 at 2:52 AM Anton Babenko<EMAIL_ADDRESS>wrote:

Fixed in 6d21738
https://github.com/antonbabenko/modules.tf-lambda/commit/6d217385f296242dba22bc3b2f9981a8713a869d
by lowercasing directory name.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/antonbabenko/modules.tf-lambda/issues/22#issuecomment-603139283,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACGJNZRZPEWISOGI7NQP4LTRJB7GRANCNFSM4LF5QFHA
.

I'm also thinking root folder has name of region. Maybe it is better to name it by VPC name?

No, resources are grouped according to public cloud provider model (AWS account => region (or global) => resource)


Make this module runnable with hashicorp/terraform:light

This package cannot be used with hashicorp/terraform:light - as it requires python runtime.
Perhaps should be stated in requirements?
Alternatively simplify lambda handler packaging with data.archive_file.

Looking at the python code, it's not even clear why a runtime/zip package is even needed. The pip requirements are just for testing, so I'm thinking why bother? The AWS lambda runtime already provides boto3, there isn't anything else the script imports.

Data source archive_file has a very basic set of features (no excludes, no files mask, etc) and most dangerous is that this data source creates archives during terraform plan but I believe plan should not create anything meaningful (terraform apply should).
Python is needed by the terraform-aws-lambda module to build/rebuild a deployment package.
There is already a note in README about how to make this module work on Terraform Cloud Agents and Python will be a requirement for the foreseen future in the Lambda module (unfortunately for some users, I know).


merge libs?

Hey there -
Your lib looks cool, I want to use it instead of some stuff I wrote for react-responsive
Here is my stuff:
https://github.com/wearefractal/react-responsive/blob/master/src/toQuery.js
I think negation might be the only missing feature from json2mq - want to have a look at the code and see if there are any other gaps?

Hey,
Its cool.  Let us discuss to cleanup API.
The main difference between toQuery.js and json2mq now is
json2mq turns {type: 'screen' , maxWidth: 1200}  -> 'screen and (max-width: 1200px)'
toQuery turns {screen: true, maxWidth: 1200} -> 'screen and (max-width: 1200px)'
Am I right ?
In addition to not keyword ,  media query also support only keyword . How should we handle it ?
I introduced type in json2mq because I am doubtful whether
{screen: true} should turn into 'screen' or 'only screen'.
With type  users can add  not and only keywords in string format
{type: 'only screen'}
{type: 'screen and not (handheld)'} etc.
What do you think ?

@akiran I think that only and not are details of the media query that the user shouldn't need to know about. The whole idea is to hide that ugliness away from the user so they can stick to what they already know, JSON - right?

@contra, I agree. It would be nice if every thing is in JSON.
API for not keyword is straight  forward.
{handheld: false} turns into  not (handheld)
API for only keyword is slightly tricky.
{screen: true} should it turn into 'screen' or  'only screen'
Any ideas for solving this ?

@akiran What exactly does the only keyword do?

From this article

The only keyword prevents older browsers that do not support media queries with media features from applying the given styles

From what I understand, only keyword is required for old  browser support.
Can we ignore only for now and mention in the docs that json2mq supports only modern browsers ?

@akiran So it doesn't actually change he behavior of a media query? That seems kind of silly...

Yes, at least that what I understood from that article.

from the css-tricks article

Note the only keyword was intended to prevent non-media-query supporting browsers to not load the stylesheet or use the styles. Not sure how useful that ever was / still is

@akiran I think we can ignore that, it doesn't seem like something in effect today

I will go ahead and make the changes .
API will be
{ screen: true, maxWidth: 1000} -> 'screen and (max-width: 1000px)'
{handheld: false, maxWidth: 1000} -> 'not handheld and (max-width: 1000px)'

@contra  Made a 0.2.0 npm release
Checkout and let me know your feedback


refactor did peer create logic it's not matching the interface

Currently for peer DIDs in create you cannot do build and store at the same time.  Current did-exchange logic should be reviewed to see if we could get rid off separate store call for peer did.

Currently for peer DIDs in create you cannot do build and store at the same time.  Current did-exchange logic should be reviewed to see if we could get rid off separate store call for peer did.


Release Google.Cloud.DataLabeling.V1Beta1 version 1.0.0-beta01

Changes in this release:
First beta release.

Created release for Google.Cloud.DataLabeling.V1Beta1-1.0.0-beta01


Question: Can I get the current heart rate without having to wait for the workout to end?

I'm trying to get the current Heart Rate. I'd like to be able to be able to get it every 5 seconds but 20 or 30 seconds would work as well.
I've noticed that when I query the heart rate was only updated every 5-6 minutes. To get around this I started a workout on my watch. "Mind and body" or "other" this increased my heart rate samples to 5 seconds. Only issue is that I can't seem to query these values until I end the workout. If I query during the workout I get the reading from 5-6 minutes ago.. But as soon as I end the workout, I can go back and see my heart rate throughout at 5 second intervals?
Is there a way to get a realtime heart rate?
what my query and options look like:
const options = {
        startDate: new Date(2021, 9, 24).toISOString(),
        limit: 3,
      }
      AppleHealthKit.getHeartRateSamples(
        options,
        (results) => {

@austinGENEreeve did you find a way to overcome this issue?

@gilador No luck.

@austinGENEreeve I know it's been a while since you posted but have you had any luck as I'm interesting in a similar feature for my app? Thanks


feat(bigquery) : expose billing statistics

@antonpod query-end event should arrive with new key statistics which in case of bigquery has following keys :
"cacheHit" -> jobStatistics.getCacheHit,
"bytesProcessed" -> jobStatistics.getTotalBytesProcessed,
"bytesBilled" -> jobStatistics.getTotalBytesBilled


bugfix/944_Auto_select_if_mtaching_element_is_found

https://github.com/boxture/oms/issues/944

Can you fix the conflicts and merge?


Complement of previous commit

The previous commit wasn't built correctly. This one is.

Hey Victor, what did you change?
this lib is automatically generated based on Bungie's docs.

I changed the HTTPConfig class so you can pass headers to the API, because it is needed to access some resources


Re-parenting of cognitive services parent POM to track 1 parent inadvertently caused the groupId to change from com.microsoft.azure.cognitiveservices to com.azure

Re-parenting of cognitive services parent POM to track 1 parent inadvertently caused the groupId to change from com.microsoft.azure.cognitiveservices to com.azure. The original PR that changed this https://github.com/Azure/azure-sdk-for-java/pull/5202. I believe this work is going to be adding groupIds to the individual POM files to set the group instead of allowing them to inherit the parent's groupId. The complete list of POM files can be found in the original PR link.

Looking at some packaging stuff it seems like this is still an issue. @mitchdenny Do you know if we explicitly decided to not move these packages back to their original groupId? As opposed to com.azure?

No I think it was just a mistake. If we want to override then we should just just set them back to com.microsoft.azure in the child packages.

Yes I suggest that we reset these back.

Was fixed here: https://github.com/Azure/azure-sdk-for-java/pull/5366

Cool yeah I did a little exercise to double check and I didn't find any other packages with the wrong groupid.


Serving only WS: disconnected immediately

I am trying to serve on websocket only websockets. The code looks like this:
from flask import Flask, render_template, Response
from flask_sockets import Sockets

app = Flask(__name__)
sockets = Sockets(app)

@sockets.route('/wscrd')
def echo_crd(ws):
    print "HERE!"
    vcamera = VideoCamera()
    pframes = ProcFrames(vcamera)
    while True:
        message = "UP"
        ws.send(message)
        ws.send("11")
        yield True

if __name__ == '__main__':
    from gevent import pywsgi
    from geventwebsocket.handler import WebSocketHandler

    print "HERE! SERVER!"
    #app.run(host='<IP_ADDRESS>', debug=True)

    server = pywsgi.WSGIServer(('', 5000), app, handler_class=WebSocketHandler)
    server.serve_forever()

Or very close.
The js code gets connected and immediately disconnected.
The /wscrd route is never hit.
The js code is very plain.
Please note that the content of the  loop
while True:
had many different versions... a kind of trial and error .. this is just one. But it never gets called anyway.
I add the js code for refs:
$(document).ready(function(){

var WEBSOCKET_ROUTE_CRD = "/wscrd";
var wscrd = new WebSocket("ws://" + "<IP_ADDRESS>:5000" + WEBSOCKET_ROUTE_CRD);

    wscrd.onopen = function(evt) {
        $("#wscrd-status").append("Connected");
        alert("On_open")
        };

    wscrd.onmessage = function(evt) {
        $("#wscrd-status").append("NewMessage");
        alert("On_message")
        };

    wscrd.onclose = function(evt) {
        $("#wscrd-status").append("Disconnected");
        alert("On_close")
        };

    wscrd.onerror = function(evt) {
        alert("On_error")
        };
};

On the js client side I see the 2 alert: "On_open" and "On_close".

@mariotti please, format your snippet, respect time of other people. Thanks

@oblalex I am very sorry, formatted! I was trying but got troubles and had to leave the keyboard.

Solved part of the problem, I mean part of mine ;) , but I am not sure about the actual behaviour of flask-sockets. This snippet works:
@sockets.route('/wscrd')
def echo_crd(ws):
    print "HERE!"
    vcamera = VideoCamera()
    pframes = ProcFrames(vcamera)
    while not ws.closed:
        #message = ws.receive()
        ws.send("|Start loop|")
        while True:
            xred, yred, xyellow, yyellow = pframes.get_frame_crd(ws)
            ws.send(str(xred))

But wait... the client now is sending a first message to the server, even if it doesn't get read.
Once opened a ws connection, is it possible to start with just the server sending data?
I mean, even by protocol. Does websocket need to have the client starting with a first data?
This in relation to my last comment.

From this document
mozilla.org / Writing_WebSocket_servers
it seems that the server can start sending data. Quote:
"
Either the client or the server can choose to send a message at any time — that's the magic of WebSockets.
"

I have this problem. I have no idea what to do.
This issue is ancient. This is the worst.


Update quickstart create package file

updated file create-and-publish-a-package-using-visual-studio.md.
The sentence "Defina a configuração como Versão" makes no sense in portuguese.

The PR is not being built because the target branch live is filtered in .openpublishing.publish.config.json on live branch.

Hello @deniscapeto,
Thank for your contribution. We are currently reviewing and will communicate the decision taken.
Thanks for your time and support on this.
Kind regards,
Microsoft C&E Localization Team

@deniscapeto :
Thank you for your feedback and suggestion!
We agree with your suggestions, and we will include them into the next article update release.
Thank you for your time and support.
Microsoft C&E Localization Team

The PR is not being built because the target branch live is filtered in .openpublishing.publish.config.json on live branch.


Update changelog

API change check
API changes are not detected in this pull request.

/check-enforcer override


More sc-service config reexports

This PR simply reexports some of the code units that sc_service::Configuration depends on internally. As of now, it is not possible to construct the sc_service::Configuration using just the sc-service crate, but I wanted to do this and figured it would be convenient to apply some changes that I needed as a patch.
This PR does not change any fundamental rules about the code structure, it only adheres to existing ones and solves current issues with them, so I assume it won't be difficult to merge.

bot merge

bot merge

bot merge


Error: Cannot read property 'length' of undefined

Can you share some code? Can't really help you otherwise

I'm guessing the error is coming from this line which would imply an undefined value is passed for textToHighlight when it should be a string. Seems like an error in the user's app code?


[Build] 'char_traits' is deprecated

Problem description
Lastest Velox build failed on Mac.
System information
Velox System Info v0.0.2
Commit: fc8a69f1d7b3af8947f489e65712f19fdddf0c19
CMake Version: 3.27.7
System: Darwin-23.1.0
Arch: arm64
C++ Compiler: /Library/Developer/CommandLineTools/usr/bin/c++
C++ Compiler Version: <IP_ADDRESS>00040
C Compiler: /Library/Developer/CommandLineTools/usr/bin/cc
C Compiler Version: <IP_ADDRESS>00040
CMake Prefix Path: /Library/Developer/CommandLineTools/SDKs/MacOSX14.0.sdk/usr;/opt/homebrew;/usr/local;/usr;/;/opt/homebrew/Cellar/cmake/3.27.7;/usr/local;/usr/X11R6;/usr/pkg;/opt;/sw;/opt/local
Result copied to clipboard!
CMake log
/Users/abei/Code/velox/_build/release/_deps/folly-src/folly/Range.h:197:12: error: 'char_traits<unsigned char>' is deprecated: char_traits<T> for T not equal to char, wchar_t, char8_t, char16_t or char32_t is non-standard and is provided for a temporary period. It will be removed in LLVM 18, so please migrate off of it. [-Werror,-Wdeprecated-declarations]

Closing it.


Invalid save error

After saving the edited file, I cannot load it in the game OR inte save editor itself. The error that I am getting states that the file is invalid and that it can only load json files.

Please post the file in question so I can take a look.

Hey, it might have been the location i was saving it too. I chose my sd card instead, and it loads just fine. Thank you!


Define mandatory/optional attribute tag

It's mandatory when (tag number % 128) < 64 and optional when (tag number % 128) >= 64.
Fix #352

Seems like the obj_attrs_handle_unknown function is missing in the risc-v GNU ld when we ported.  The rule looks reasonable and good to me, thanks.

Got LGTM from two major linker maintainer, gonna merged.


Get related video of YouTube

Checklist

[X] I'm asking a question and not reporting a bug/feature request
[X] I've looked through the README
[X] I've read the guidelines for opening an issue
[X] I've searched the bugtracker for similar questions including closed ones

Question
I would like to know whether it is possible to get related video which is on the right side bar in YouTube through yt_dlp or not.
Is there no other way than using official api?
Any help like information about known alternatives would be appreciated.🙏
Verbose log
No response

This is out of scope for yt-dlp. But you can get this info from the INNERTUBE API (that yt-dlp uses) if you don't want to use the official API
PS: --print-traffic and --write-webpage should help you figure out what requests to make

Thank you for your advice!
I didn't know it , but it sounds interesting and is worth trying.
I will look it up.

It's not quite on topic to discuss here, but if you have any doubts related to innertube, feel free to ask in our matrix/discord servers

I could get the related video which I wanted to , so I leave snippet here.
import requests
res = requests.post(
    "https://youtubei.googleapis.com/youtubei/v1/next/",
    json={'videoId': 'emrt46SRyYs', 'context': {'client': {'clientName': 'WEB', 'clientVersion': '2.20210223.09.00'}}},
    params={'key': 'AIzaSyAO_FJ2SlqU8Q4STEHLGCilw_Y9_11qcW8', 'alt': 'json'},
    headers={'User-Agent': 'Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:77.0)', 'Accept-Encoding': 'gzip, deflate, br', 'Accept': '*/*', 'Connection': 'keep-alive',
             'X-YouTube-Client-Name': '1', 'X-YouTube-Client-Version': '2.20210223.09.00', 'Referer': 'https://www.youtube.com/'}
)
data = res.json()
items = data["contents"]["twoColumnWatchNextResults"]["secondaryResults"]["secondaryResults"]["results"]
for item in items:
    if "compactVideoRenderer" in item:
        if "videoId" in item["compactVideoRenderer"]:
            vid = item["compactVideoRenderer"]
            print(vid["videoId"])  # video id
            print(vid["title"]["simpleText"])  # video title
            break


Crash in FirebaseStorage version 10.1.0

Description
We are seeing a crash on iOS in Crashlytics ever since upgrading to FirebaseStorage version 10.x.x. The code where this crash occurs hasn't changed, so I think this is something to do with FirebaseStorage itself.
Crash trace:
Crashed: NSOperationQueue 0x107b84e00 (QOS: UNSPECIFIED)
EXC_BAD_ACCESS KERN_INVALID_ADDRESS 0x0000000000000000
0 FirebaseStorage
$s15FirebaseStorage0B0C24initFetcherServiceForApp33_A1F548C0F6900337354627997364394ALLySo010GTMSessiondE0CSo6FIRAppC_SSSo14FIRAuthInterop_pSo0m5CheckO0_ptFZTf4nnnnd_n + 236

1 FirebaseStorage
$s15FirebaseStorage0B0C16ensureConfigured33_A1F548C0F6900337354627997364394ALLyyF + 88

2 FirebaseStorage
$s15FirebaseStorage0B0C9referenceAA0B9ReferenceCyF + 24

3 FirebaseStorage
$s15FirebaseStorage0B0C9reference8withPathAA0B9ReferenceCSS_tF + 52

The specific line tied to this crash is simply creating a storage reference in Swift as follows.
let fileReference = storage.reference(withPath: "subfolder/video.mov")
Any pointers/help appreciated. Thanks.
Reproducing the issue
We haven't been able to reproduce this on our end. The crash is occurring on customer devices.
Firebase SDK Version
10.1.0
Xcode Version
14.0.1
Installation Method
CocoaPods
Firebase Product(s)
Storage
Targeted Platforms
iOS
Relevant Log Output
No response
If using Swift Package Manager, the project's Package.resolved
No response
If using CocoaPods, the project's Podfile.lock
No response

Sorry about the trouble. It's possible that the issue you're seeing is related to #10463 that was fixed in last week's 10.2.0 release.
Would you update to 10.2.0 and let us know?

@paulb777 thanks for the quick reply. I'll update this thread once we find out the result of the update.

@paulb777 sorry for delay. Yes, 10.2.0 fixed the issue! Thanks for the quick reply and followup!


Added doremon puzzle Issue: #2554

PR Description 📜
Doremon puzzle is a frontend game developed using HTML, CSS, JavaScript
Fixes #2554

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @sujanrupu ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Thank you  @sujanrupu , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Better functionality for PartitionedGraph

This PR adds improved functionality and function naming for the PartitionedGraph type.
Firstly, we add support for reverse(pe::PartitionedEdge) which previously required the user to unwrap and wrap the edge again.
Secondly, there is proper support for getting edges out of a PartitionedGraph.
New functions in this domain:

partition_edge(pg, edge) is renamed which_partitionedge(pg, edge::AbstractEdge) to be more consistent with the naming of the vertex version which_partition(pg, vertex).
which_partitionedges(pg, edges) and which_partitions(pg, vertices) have been added to support the vectorized version of those functions
partition_edges(pg, pe::PartitionEdge) is renamed edges(pg, pe::PartitionEdge) (and a vectorized version is added) to be more clear about the return type
partitionedges(pg) and partitionvertices(pg) is added to just return the lists of edges and vertices of the partitioned_graph(pg) (wrapped in the PartitionEdge and PartitionVertex type)

These functions are exported.

Thanks.
I prefer partition_edge/partition_edges over which_partitionedge/which_partitionedges and partition_vertex/partition_vertices over which_partition/which_partitions.
I agree with renaming partition_edges to just edges.
I would call them partition_edges(pg)/partition_vertices(pg) instead of partitionedges(pg)/partitionvertices(pg), since we generally are using snake casing for other functions with similar names in this package.

Yeah I am happy with that naming. The only issue is that we also have partition_vertices(pg; kwargs...) defined for when calling a partitioning backend like KaHyPar. Any thoughts on what we should rename that function to avoid the clash.

I see, yeah that is an unfortunate name clash. I don't like partition(pg) for that since it doesn't indicate that it is outputting vertices (I would think that might be outputting a graph).
The problem seems to basically be that when reading partition_vertices(...), partition could be interpreted either as a verb (as in, the function is performing a partition), or as a noun (as in, the vertices of a partitioned graph). It feels more natural to me to interpret it as a verb, and I think for that reason the names PartitionVertex and PartitionEdge have felt a bit awkward to me, but I can't think of better names for those so we are left with this conundrum.
One way out could be to call the function that partitions the vertices with KaHyPar/Metis partitioned_vertices(pg; kwargs...), to name it after what it is outputting rather than what it is doing.

Okay will go with partitioned_vertices(pg; kwargs...) for now

Codecov Report
Attention: 22 lines in your changes are missing coverage. Please review.

Comparison is base (40f0edd) 78.83% compared to head (941b4d2) 77.70%.

Files
Patch %
Lines

src/Graphs/partitionedgraphs/partitionedgraph.jl
20.00%
12 Missing :warning:

...aphs/partitionedgraphs/abstractpartitionedgraph.jl
22.22%
7 Missing :warning:

src/Graphs/partitionedgraphs/partitionedge.jl
0.00%
2 Missing :warning:

.../Graphs/partitionedgraphs/abstractpartitionedge.jl
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
- Coverage   78.83%   77.70%   -1.14%
==========================================
  Files          31       31
  Lines        1167     1184      +17
==========================================
  Hits          920      920
- Misses        247      264      +17

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

Looks good, thanks. Could you add tests for the new functions?

Something I realized is that I think the snake casing in partition_vertex/partition_vertices and partition_edge/partition_edges may not be the right way to go. My thinking is that, with the current design, we are establishing a convention that partition_vertex(...) will output a PartitionVertex, partition_vertices will output a list of PartitionVertex, etc., which I think is a good convention to establish and makes the code clearer (and makes it clearer to us what to name functions going forward).
I think removing the snake casing, i.e. changing to partitionvertex, partitionvertices, etc., could make that convention even more obvious. To me, reading those new names makes it even clearer that there is a connection between the name of the function and the type of object it is outputting (i.e., makes it clearer that in partitionvertex, it is using "partition" as a noun rather than a verb, and will output an object of type PartitionVertex).
I'm on the fence about that since we use snake casing in other functions, let me know what you think.

I think partitionvertex (as opposed to partition_vertex ) is clearer to me in terms of what it is outputting (i.e. a PartitionVertex) and so I am in favor of that to avoid ambiguity about partition being a verb here. In fact here we could argue that we are not even necessarily forgoing snake casing as we have defined a PartitionVertex type and so partitionvertex is just simply a one word noun and no spaces are required?

Yeah, I think that is a more succinct reason for using the style partitionvertex. Also, it is common practice in Julia to have two sets of constructors for types, one using camel casing and one using lower casing (for example Tuple/tuple and String/string) which have different behaviors. Often the camel cased constructor is lower level and stricter. so this is a good example of following that convention.

Okay great. I have gone with that naming convention for now! I also added further testing.

Looks good, thanks!


Added CustomID to UploadImageParams

Description
I added support for CustomID when we upload images to Cloudflare images which I hope is supported by CF API: https://developers.cloudflare.com/images/upload-images/upload-custom-path/
Please correct me if I'm wrong but Custom ID means that we can select which ID to have for the image right?
Has your change been tested?
Yes. I tested this in my own app and seems to be working
Types of changes
What sort of change does your code introduce/modify?

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

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] This change is using publicly documented in cloudflare/api-schemas
and relies on stable APIs.

The reason for the failed CI is
go: downloading golang.org/x/exp/typeparams v0.0.0-20221212164502-fae10dda9338
go: golang.org/x/tools/cmd/goimports@latest (in golang.org/x/tools@v0.25.0): go.mod:3: invalid go version '1.22.0': must match format 1.23
internal/tools/tools.go:22: running "go": exit status 1

Do you want me to fix this?

unfortunately, this isn't documented in the API schemas so it is not yet something we can add support here for - https://developers.cloudflare.com/api-next/resources/images/subresources/v1/methods/create/

unfortunately, this isn't documented in the API schemas so it is not yet something we can add support here for - https://developers.cloudflare.com/api-next/resources/images/subresources/v1/methods/create/

Got ya, Do you might know when this will be added to the docs?

i do not. that is a question for the service team or CSUP.

i do not. that is a question for the service team or CSUP.

Got ya!
Should I close this PR so we can open this if this becomes something you want in the future?

Thank you for answering btw! :D

up to you but this will go stale within 2 weeks and be closed automatically.
now that we're codegen'ing the libraries, this will automatically be picked up when it does land.

Okay, Then I close this issue. Thank you for the answers


Syntax checker doesn't support "Create index"

Create index works in neo4j. Would be great if the syntax checker didn't show an error on it.

Hi Joerg, thank you for opening this issue. I'm going to pick this up next.


Execution failed for task ':react-native-webview:compileDebugKotlin'.

Bug description:
when run app in android return this error Execution failed for task ':react-native-webview:compileDebugKotlin'.

To Reproduce:

Expected behavior:

Screenshots/Videos:

Environment:

OS:mac os
OS version:12.6
react-native version:64.0
react-native-webview version:11.26.0

Can you provide the full build log and a sample app that reproduces the issue?

I have the same problem......
Environment:
"react-native-webview": "^11.2.4"
"react-native": "0.63.4"
when run app in android return the same error:
Task :react-native-webview:compileDebugKotlin FAILED
Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
......

@czjdrj Same question as above.

Same issue also: Task :react-native-webview:compileDebugKotlin FAILED

@marwan2 Same question as above.


Prevent 'global' references from being resolved as URLs

Currently, if a global. reference is used in a scene node that represents a URL, that reference will be resolved as a relative URL and not replaced with the appropriate global value.
This change makes global. a reserved prefix that will prevent values from being treated like URLs. This means that global values will now behave as documented for values that expect URLs.
This also implies that you cannot use global.png or similar file names/paths. If you really need to call your file "global.png" then you can use the equivalent path ./global.png

What about scene import?

@bcamper Gooood question - I'm not sure how to handle that. I don't think we can allow global. references in import statements because imported paths are used before global.s can be replaced.

Oh hehe right, makes sense. Good policy to stick to then.
On Tue, Nov 15, 2016 at 10:43 AM, Matt Blair<EMAIL_ADDRESS>wrote:

@bcamper https://github.com/bcamper Gooood question - I'm not sure how
to handle that. I don't think we can allow global. references in import
statements because imported paths are used before global.s can be
replaced.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tangrams/tangram-es/pull/1089#issuecomment-260676525,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABBXYLV1vGg6xyAtRjt6uWxedrv3S_Wks5q-dMmgaJpZM4KyIE3
.

@hjanetzek Does this look alright to you?

@blair1618 can't global.png be a path unless global: {png: ... } is defined? This would be the same logic as for global textures.

@hjanetzek The trick is that we don't know the total set of values in global: {} until the merging is done, so we can't assume that a value won't be defined by a "parent" scene.

I see - so one would have to make a first pass to get all global blocks before merging imports (and resolving URLs).
I'm ok with merging this. We can consider some special cases or the import logic later.

Oh yeah that's possible - we can iterate over the DAG of imports and generate the total set of global: values before actually merging the scenes, then check potential URLs against that set before resolving them. Let's keep that in mind in case this becomes a problem in practice!


Add .cshtml and .razor files for .NET projects

Fixed via https://github.com/miguelsolorio/vscode-symbols/pull/130


Error term in existence error should be a predicate indicator:

?- catch(inex, error(E,_),true).
E = existence_error(procedure, 'inex/0').

in place of the atom 'inex/0', there should be the predicate indicator inex/0.
7.7.7 Selecting a clause for execution
Execution proceeds in a succession of steps:
...
b) If no procedure has a functor and arity agreeing
with the functor and arity of curract, then action
depends on the value of the flag unknown (<IP_ADDRESS>):
error - There shall be an error
existence_error(procedure, PF)
where PF is the predicate indicator of curract, or
...

Changed!


Sync test statuses with corresponding build statuses only if configured

Fixes https://github.com/packit/packit-service/issues/1943.
RELEASE NOTES BEGIN
Added a new configuration option sync_job_statuses that controls whether test job statuses are updated together with their corresponding build job statuses. When set to false, while tests jobs are waiting for their corresponding build jobs to finish, their statuses remain in pending state and only build job statuses are updated.
RELEASE NOTES END

recheck

regate

regate


setup AICoE-CI

Signed-off-by: Christoph Görn<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from goern by writing /assign @goern in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@goern: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

pre-commit
8a93ecfacc774e07b09b9c249dbf9057310bfdaf
link
true
/test pre-commit

Full PR test history. Your PR dashboard. Please help us and open an issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@harshad16 will the repository on quay be created by aicoe-ci?

repository is available at https://quay.io/repository/operate-first/bragi?tab=info


Fix typo in the core readme

Issue #, if available:
Description of changes:
Fix a little typo
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@alessandro-aglietti
Thanks for the README correction! Merging!


Fails to load dynamic routes when opened directly (opening from the address bar, not redirecting to it via code)

This is most probably related to the import maps.

TL;DR
A dynamic route is at /foo/bar, and the import map entries like ./vendor/... and ./src/... lead to /foo/vendor/... and /foo/src/... which will then be 404, hence return text/html, and not the wanted script.
I could only reproduce this in production build.

This is fixed in v2.1.3

@deckchairlabs Seems like the location is now fixed, but it is trying to load .tsx instead of the production .js. Should we reopen this? :)

Browser

The error
Uncaught (in promise) Error: Unsupported Content-Type "text/html; charset=utf-8" loading https://host:port/src/app.tsx imported from https://host:port/client.b3e5c79f.js. Modules must be served with a valid MIME type like application/javascript.

Yes, that <base href="/" /> fixes it.


Detect and correct overflows of the RAPL microjoule counter

Problem
The RAPL energy counter is incremented and can overflow.
Currently, this overflow is not handled.
Currently, the energy measurements are "slightly" (potentially a lot?) wrong.
Fixing that might fix other issues where the user complain about "wrong" power usage.
Solution
Instead of ignoring the value, the overflow should be corrected.
Quoting @uggla:

If previous_microjoules !=0 then we could probably do microjoule = (u64::MAX - previous_microjoules) + last_microjoules

Alternatives

Additional context
See https://github.com/powercap/powercap/issues/3#issuecomment-636256230

The LIKWID tool does take the overflows into account. Some info here: https://github.com/RRZE-HPC/likwid/issues/13

I saw this issue linked from the powercap project.  FYI I think your proposed solution won't work correctly for two reasons:

While the MSR is 64 bits, only 32 bits are used for energy values.
Those 32 bits are encoded using status units from the MSR_RAPL_POWER_UNIT register.  See Section 15.10 of the Intel Software Developer's Manual, Volume 3, March 2023 edition.  The standard configuration encodes using the formulation 1/2^ESU, but some processors are different (particularly some Intel Atom CPUs) as are some domains like DRAM and PSYS on some processors which might have fixed ESU values for those domains that differ from the unit register.

I've found that detecting overflow in RAPL can be a challenge.  At a minimum, you need to compute the actual max energy value that the MSR register can report and use that value when accounting for overflow, e.g., as done here [1] (full disclosure: my code).  I'm not entirely convinced that this always works as expected though, even if you don't "miss" an overflow---I've seen quirky behavior in the past that resulted in overestimating power consumption.  It could be that it's not really guaranteed that the register will achieve it's max logical value before it actually turns over, but this approach is at least logically correct modulo bad register behavior.  I haven't conducted a rigorous experiment in a long time though, so I'm not sure how prevalent problems might be.
[1] https://github.com/energymon/energymon/blob/38ef1e6d2d69abf1e3496832369663918d9e56d4/msr/energymon-msr.c#L174-L182
Cheers.

That's right, 64 bits is too much for the counter:

I haven't seen aberrant values when correcting the overflows just after reading the counter (before the unit conversion), I'll check that again :)


HBASE-28760 Exclude pom file of jaxws-ri in output tarball

The client integration passed after applying this patch.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m 39s

master passed

+1 :green_heart:
spotless
0m 49s

branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 17s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
xmllint
0m  0s

No new issues.

+1 :green_heart:
spotless
0m 51s

patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
0m 10s

The patch does not generate ASF License warnings.

10m 49s

Subsystem
Report/Notes

Docker
ClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6135/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6135

Optional Tests
dupname asflicense javac codespell detsecrets xmllint spotless

uname
Linux 26fb2b2409d2 5.4.0-186-generic #206-Ubuntu SMP Fri Apr 26 12:31:10 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d65982bb04b21b26363f7ea9cfd696acdc9f72e3

Default Java
Eclipse Adoptium-17.0.11+9

Max. process+thread count
80 (vs. ulimit of 30000)

modules
C: hbase-assembly U: hbase-assembly

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6135/1/console

versions
git=2.34.1 maven=3.9.8 xmllint=20913

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.


Update docs with docGen

What changes are proposed in this pull request?
Updates the Alluxio docs with new properties & metrics.
Why are the changes needed?
Keeping docs up-to-date.
Does this PR introduce any user facing changes?
Public Alluxio docs will show these new values.

alluxio-bot, merge this please


⚠ Bofrost.fr has degraded performance

In 4f07ee6, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 2996 ms

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


NAT discovery for node with autonat protocol

Before starting process for hole punching, the node need to know if it is behind NAT or not and for that we need to have autonat protocol.
We will be using pyrsia central node as boot_node and a server as well for this.

Moved into implementation phase. In fact, we are in code review. Close.


fix: make status values consistent across validation types (#377)

Schema validation outputs Pass/Fail for the status column in the result set. The other validation types output success/fail. This PR unifies status values across validation types.

fix: make status values consistent across validation types (#377)

These values should perhaps be in consts?


Is there any kind of SMT available in camel-kamelets

I am sending data from kafka in  the form of json to the sink datasource but it is not populated in the supposed database ,but array of data working fine [1,"Alice"] and its added to the rows .  I already saw an issue regarding this  in camel-kafka-connector  but the solution was to make a custom  smt  , Let me know whether there is a field available in yaml of kamelets to add custom smt .
I want to add the data in the below format to the cassandra db
[{
    "id" : "1",
    "name" : "Alice",
} ]

https://github.com/apache/camel-kafka-connector/issues/1487

No, there is not. Since Kamelets are generic and not only for Kafka Connect framework, there is no point in having a field for SMTs in their YAML. You should create your own SMT.


Simplify IsAssignableFrom() use

standardize on the Type extension method; less verbiage
ModelMetadata had a redundant IsAssignableFrom() call
ModelBindingHelper.ValidateBindingContext() over-engineered and used just once
do useful bit inline in KeyValuePairModelBinder
found while investigating #3482 but not part of a fix for that issue

@rynowak (though anyone could review this cleanup) you should be happy to see I'm killing ModelBindingHelper.ValidateBindingContext() to death.

:shipit:

f889965

Why would we add more ifdefs?


Question: Ractive ecosystem

I really like the idea of "Official" utilities like "routing" or "state management" that will propose a way and help build web apps.
Coming from the "server side rendering" world I myself struggle to make 3rd party client side libraries play hand-in-hand with Ractive for creating a large SPA.
Are there any future plans for such an "ecosystem", what are your thoughts?

Are there any future plans for such an "ecosystem", what are your thoughts?

There was a meta discussion about a monorepo so that everything gets built and tested as Ractive's version progresses. Not everyone was on the same boat and that kinda got lost in time. You could try reviving that thread to get everyone's opinion after being dormant for a year and being exposed to other tools that are doing the same. 😄
I tried building the monorepo during vacation. It's collecting dust at the moment while I learn more about best practices. It's not as easy as it sounds. 😱

Well, sounds stupid, but "we" as a community are responsible for making the "ecosystem" progress or gather dust. Not anyone else. So @kouts I'd say a good way to start is to write good tutorials on how you solved the difficulty you faced, or publish the helper libraries you made... There was also a talk that instead of a monorepo, we should just present the plugins page better, with nice examples and such.

Thanks @dangelies I will when I'm there.


Remove Logstash, update ElasticSearch models directly on learner changes

This removes the Logstash Docker container and makes the Portal::Learner class responsible for updating ElasticSearch. It also adds the field created_at to the ES models, which will be important for later sorting of learners.
There is also a new Rake task, app:report:update_elastic_search_learners, which will find all learners and push them to the database. Because the ES operations are doing using "upsert," this will create docs if they don't exist and update them if they do, which means that the task is safe to run over an existing database.
The tests broken by this took quite a while to fix, and there is still one (or more, we'll see after this finishes building) failing test, ./spec/models/dataservice/process_external_activity_data_job_spec.rb:91 # Dataservice::ProcessExternalActivityDataJob#perform perform, which I can't even work out what it does, so some advice there will be helpful.

there is still one (or more, we'll see after this finishes building) failing test

I knew I was tempting fate to say that before the build ended.
It seems that my WebMock reset may have broken a bunch of tests, which is odd because that didn't seem to happen locally, although maybe I was doing steps out of order.
I'll mark this PR as draft, although if either of you has any advice for the ./spec/models/dataservice/process_external_activity_data_job_spec.rb:91 test, that would be helpful

Other tests fixed, only one is the mystery ./spec/models/dataservice/process_external_activity_data_job_spec.rb:97 which I don't understand.


Node with DAGU, and communication over mycelium

Set up

package dagu as part of flist,
install hero crystal v etc using hero cmd in the build process of flist
Run dagu ui server and dag with cron upon init which:
uses env variable called home which is mycelium ip address (multiple allowed, comma separated)
Ping home mycelium address with "im alive",
Specify secret of dagu in env variable and use for dagu authentication,
listens to mycelium incoming messages for DAGs from home mycelium
runs and updates dags

Future considerations:

3Script listener
incremental step additions to existing dags

Fixed hero installer and added dagu installation for hero bootstrap -d:

d56cf416cb6f2bec4e2740517d8c4287030d3af6
0e9577825f39cc48ec13ff3123b82e274fe98af9
fce14c9007feabca9242ad5cca9158e03e2580d4
76032d0d467e48fa3a216f71d3f2bf8d610b2246

Dagu example to listen for incoming mycelium messages and ping home every 5 seconds with authentication:

5950bf3f4f037ed869e50ce86e2d9e0ce66b91b7

Image and flist with mycelium, hero (and crystallib, dagu, v etc installed using hero bootstrap)

first attempt without using zinit (efcaeaab017273b72be60cfb63043105efe6b3c7)
updated version using zinit copied from holochain image (7cf3724418f3f99420c9e121ae11d181fe559e05)
(flist)[https://hub.grid.tf/timur.3bot/timurgordon-dagu-latest.flist.md]


show full text when hovering over truncated field

When a text field's value doesn't fit on the screen (since the column is too narrow), it would be great if the full text could be displayed when you hover over that field.
Thanks :)

Is this supported on the native Salesforce fields (i.e. everything but picklists)?

It's not working on text fields or email fields. On lookup fields there's a tooltip which shows the record id, which is interesting but not necessarily what the user would want.
I didn't try any others, let me know if you want me to.
Thanks!

Hmm sounds like that would require reimplementing the default cell templates, which is beyond the scope of this project

From: solo-1234<EMAIL_ADDRESS>Sent: Tuesday, February 11, 2020 8:49:33 PM
To: shliachtx/lwc-listview<EMAIL_ADDRESS>Cc: shliachtx<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [shliachtx/lwc-listview] show full text when hovering over truncated field (#25)
It's not working on text fields or email fields. On lookup fields there's a tooltip which shows the record id, which is interesting but not necessarily what the user would want.
I didn't try any others, let me know if you want me to.
Thanks!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/shliachtx/lwc-listview/issues/25?email_source=notifications&email_token=AAIYCWSKET5VG3TFYWZLZ33RCNIS3A5CNFSM4KP4UA32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELPAUQQ#issuecomment-584976962, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAIYCWTG2MO3FTSUUUHSKYLRCNIS3ANCNFSM4KP4UA3Q.

Fair enough - thanks for the update!


Unexpected behaviour

Hi! I'm getting some unexpected behaviour when using this. My list seems to only render 2 items, even though there's plenty of vertical space for more. Furthermore, when I resize the window, it then loads every item which kinda defeats the purpose of the lib 😅
Any ideas? Here's a screen of the props values in the DevTools

Hi @syropian, could you please paste here CSS of your .collection-cluster element and items?
Make sure .collection-cluster has overflow and height set, items should not have any margin.

@adamcikado Hey Adam, I'll try and get this to you after I'm done work today. Cheers!

@adamcikado Hey Adam, I was able to get it working by switching the collection cluster height from using 100% height to using CSS calc() to substract its top offset from the viewport height. Thanks!


Able send messages

Able send messages of user to contacts through the view of chat

Resources:

https://github.com/slackhq/SlackTextViewController

https://github.com/lukaskubanek/LoremIpsum

Resolve in pull request: https://github.com/EstefaniaGuardado/exampleXmpp/pull/23


Prohibit http-request node from sending cookies to the different domain when redirected

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes
As we discussed, we prohibit http-request node from sending cookies to the different domain when redirected using cookiejar.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[x] I have run grunt to verify the unit tests pass
[x] I have added suitable unit tests to cover the new/changed functionality

Coverage increased (+0.02%) to 76.381% when pulling 62b2adab78b93a17acc8cf7c3f044cf0c52e5307 on node-red-hitachi:dev-redirect-bugfix into 0529eed0c9c1f9c0d60b1f2a1c88fe7c1b6e0d75 on node-red:dev.


[Feature Request]: Update Quickstart FSM

Is your feature request related to a problem? Please describe.
After #81, update the CarWashFSM example that's used in Quickstart tutorial
Describe the solution you'd like
No response
Additional context
No response

#83 Updated here.

@kanak8278 Can you update the status here?


mold: 1.1.1 -> 1.2.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for mold is: "A high performance drop-in replacement for existing unix linkers"
meta.homepage for mold is: "https://github.com/rui314/mold"
meta.changelog for mold is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/rlm5ynvn75lnjy3fgs0v27j237njh4zs-mold-1.2.0/bin/ld.mold had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/rlm5ynvn75lnjy3fgs0v27j237njh4zs-mold-1.2.0/bin/ld64.mold had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/rlm5ynvn75lnjy3fgs0v27j237njh4zs-mold-1.2.0/bin/mold had a zero exit code or showed the expected version
0 of 3 passed binary check by having a zero exit code.
0 of 3 passed binary check by having the new version present in output.
found 1.2.0 with grep in /nix/store/rlm5ynvn75lnjy3fgs0v27j237njh4zs-mold-1.2.0
found 1.2.0 in filename of file in /nix/store/rlm5ynvn75lnjy3fgs0v27j237njh4zs-mold-1.2.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
0 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
mold

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/rlm5ynvn75lnjy3fgs0v27j237njh4zs-mold-1.2.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A mold https://github.com/r-ryantm/nixpkgs/archive/6d377ae0a926121234584f9c743993710325381d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/rlm5ynvn75lnjy3fgs0v27j237njh4zs-mold-1.2.0
ls -la /nix/store/rlm5ynvn75lnjy3fgs0v27j237njh4zs-mold-1.2.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

mold

Maintainer pings
cc @nitsky for testing.

@GrahamcOfBorg eval


Virtual Terminal should stick to end UNLESS user manually moved the scrollbars

Right now if you fill a virtual terminal with text it always stays stuck at the top.

In VirtualTerminal#write...
For some reason, the vertical and horizontal scroll bars fall behind when we append text to the end of the pane, and we can't tell the difference if that happened due to us or due to the user clicking on the scrollbars.
It would be nice to handle clicking on the scrollbars explicitly with a listener...


CB-14585 ccm upgrade revert flow in case of error in freeipa

please resolve configs, then we can merge


Resource usage

I have problems understanding how to use a resource to create a new observable from scratch. The most trivial example I can come out with is:
rxcpp::observable<>::scope(
	[]() { return rxcpp::resource<int>(1); },
	[](rxcpp::resource<int> res) {
		return rxcpp::observable<>::create<int>(
			[res](rxcpp::subscriber<int> sub) {
				sub.on_next(res.get());
				sub.on_completed();
			}
		);
	}
);

In this case, I would get compilation error as resource::get is not const and my lambda is not mutable. Changing the lambda to mutable leads to #473 . Am I misusing observable::scope? I could not find many information about its correct usage...
Still, to make my code work, I could make mutable the callback in observer as proposed in #473, but I'm wondering if it does makes sense in this case. Is there a compelling reason not to make resource::get() const? By looking at the source I can see that it is just calling get on a shared_ptr (which is const already...).

yes, resource::get should be const. at some point I would like to revisit the design of scope as well


a problem of sending and receiving UDP by pkts

Why the experiment sends UDP  from a server using pktgen, and when the MAC address is deliberately miswritten, the traffic is displayed through the dump-pkts in another server？it can also be received by dump-pkts when the ip is wrong?I don't know why it happened. Is it a broadcast？

Packet dumpers usually operate in promisc mode and don't care about the mac address. The routing between your servers is unrelated.


Test against 3.4 too, since we still have 3.4 installations at some beamlines

Kick travis again now that tifffile is being installed

Apparently we are using my personal anaconda.org for some of the deps...
conda install super_state_machine cycler xray-vision lmfit -c tacaswell

Yep. That is on my list of things to fix!

restarting travis against current master

@tacaswell @danielballan this is good to go


More Makefile cleanup.

Can one of the admins verify this patch?

I am going on a Makefile cleaning spree, hence the "DO NOT MERGE" flag on that PR. I'll remove it when I'm done.

Can one of the admins verify this patch?


Wrap breaks numbers

Wrap in UI may split one number in two parts. It is better to wrap by words.

Hi @Vanav,
Thanks for reporting!  This is now fixed in v5.4.2

Wrap doesn't work at all in v5.4.2, if input is without spaces (like on my screenshot).
I think, wrap should work without spaces too, because spaces should be inserted manually, and are not required, so they are not commonly used.

Unfortunately it is not possible to wrap at specific characters such as + - / *.  So in your example, only way to prevent the wrap at a random place (in the middle of 2000) is to allow breaks only for full words or scroll if one long line.  Which is what was implemented in v5.4.2
If no spaces are provided, then line won't wrap but will be scrolled to prevent wrapping in the middle of a word or number.  If a space is added, then line will wrap at that point.
Although I don't have an immediate fix for what you are looking for, I will continue to explore ways to improve the line breaks.

Hi @Vanav,
Just released v5.5.0 with some improvements on line wrapping. Please let me know if this is now what you were expecting.

Seems you have found a perfect solution! Thank you so much.


Missing UK Scientific Dictionary?

I'll keep this brief.
I notice you have this http://jpetrie.net/wp-content/uploads/custom_scientific_US.dic
But do not appear to have the UK version http://jpetrie.net/wp-content/uploads/custom_scientific_UK.dic
Thanks,
tecosaur

Great idea.
I'll try to add it when I have time. If you would like it sooner, please feel free to do a pull request.


Fixed newly uploaded subscriber emotes not being available

Pull request checklist:

[x] CHANGELOG.md was updated, if applicable

Description
🚧 ⚠WORK IN POGRESS ⚠ 🚧

Added a handle function to GLOBALUSERSTATE irc message.
Moved initial loading of user's emotes to IrcMessageHandler::handleGlobalUserStateMessage - it's done, so that we're sure we've already have emote-sets cached in TwitchAccount::userstateEmoteSets_.
Refactored whole logic regarding (re)loading user's emotes.

Current order of handling emotes:

Load emote sets from IVR
Set which emotes failed to resolve.
Call Kraken's getUserEmotes.
Fill up TwitchAccount::emoteData with emote sets that were returned in a Kraken call, but aren't present in emoteData.

We still continue to call TwitchAccount::loadUserstateEmotes in case of follower emotes (which are not present in GLOBALUSERSTATE message) and/or newly obtained emote sets, e.g. on firsst USERSTATE message after receiving a gifted sub.
Closes #2935
Closes #3007

Apparently my login from #2856 was wrong, I've fixed it in 9564b3b and it seems to split emote sets correctly now.


Update plan-for-the-multilingual-user-interface.md

Changed the invalid link with https://docs.microsoft.com/en-us/sharepoint/add-and-edit-user-profile-properties

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@bentoncity Changes look correct. Could you please review and merge? Thanks!


Improve error message when connecting to banned peer using Swagger API

This PR provides the following error via Swagger:
{
  "errors": [
    {
      "status": 400,
      "message": "Can't perform 'addnode' for a banned peer.",
      "description": "System.InvalidOperationException: Can't perform 'addnode' for a banned peer.\r\n   at Stratis.Bitcoin.Connection.ConnectionManager.AddNodeAddress(IPEndPoint ipEndpoint) in E:\\Public\\StratisBitcoinFullNode\\src\\Stratis.Bitcoin\\Connection\\ConnectionManager.cs:line 413\r\n   at Stratis.Bitcoin.Connection.ConnectionManagerController.AddNodeRPC(String endpointStr, String command) in E:\\Public\\StratisBitcoinFullNode\\src\\Stratis.Bitcoin\\Connection\\ConnectionManagerController.cs:line 46\r\n   at Stratis.Bitcoin.Connection.ConnectionManagerController.AddNodeAPI(String endpoint, String command) in E:\\Public\\StratisBitcoinFullNode\\src\\Stratis.Bitcoin\\Connection\\ConnectionManagerController.cs:line 81"
    }
  ]
}

See https://github.com/stratisproject/StratisBitcoinFullNode/pull/3277.

Yes the API response is still the same.

@fassadlr your initial PR was actually correct 👍


KEYBOARD MANAGER - Please add "Function" key to list of combination keys

For Keyboard Manager, one can remap a keyboard key for combination of keys.
Example, one can map:
[Fn] + [Right Arrow] = End
[Fn] + [Left Arrow] = Home
Also, one can remap a shortcut using "Function" key". The possibilities are amazing and endless!!
Example: [Fn] + M = Opens Minecraft

I found that Keyboard Manager does not accept alt+left to be remapped to Home for example like osx binding. "Shortcut must start with a modifier key" error. But alt+shift+left can be remapped as shift+home without any troubles.
Need to give ability to remap 2-3 keys combo to functions like home, end, pgup, pgdown.

From what I remember the function key is often hardcoded by Laptop and keyboard manufacturers. So that might not be possible. Or it might at least not be always possible.
E.g. for my Yoga OneMix 2s, fn+Escape actually controls the fan, which has nothing to do with Windows.

Closing because this is a duplicate issue: #3219
Check it out, where we explain the difficulty of remapping the Fn key and why we don't support it. It's often a hardware key that the OS cannot access as @DavidGretzschel said
also @Zigman13 we are tracking shortcut to key here: #3290


add functionality to find an application by name

What I wanted is the functionality to find an application by name.
Let me know the right place to add the functionality if kintoneService.app is not appropriate.

I'll close this pull request because it breaks tests.


User additionalFields gives type errors because of not found

Describe the bug
AdditionalFields are no longer accessible on user via. inferAdditionalFields(). It works fine in 0.7.5.
Expected behavior
AdditionalFields should be found on the user.

Hey, when exactly is it not working? With my little tasting it's still working

Can you try with this example? My client is not finding anything (I have a workspace/monorepo). I am trying to do useSession() to get the data from user, but the fields don't show up after 0.8+.
import { inferAdditionalFields } from "better-auth/client/plugins";
import { createAuthClient } from "better-auth/react";
import type { auth } from "./auth";

export const authClient = createAuthClient({
  plugins: [inferAdditionalFields<typeof auth>()],
});

can you show me your auth.ts file?

import { admin, username } from 'better-auth/plugins';

import { betterAuth } from 'better-auth';
import { db } from '../../db';
import { drizzleAdapter } from 'better-auth/adapters/drizzle';

export const auth = betterAuth({
  appName: 'App',
  baseURL: process.env.BETTER_AUTH_URL as string,
  database: drizzleAdapter(db, { provider: 'mysql' }),
  databaseHooks: {
    user: {
      create: {
        before: async (user) => {
          return { data: { ...user, role<EMAIL_ADDRESS>? 'admin' : 'user' } };
        },
      },
    },
  },
  emailAndPassword: { enabled: true },
  plugins: [admin(), username()],
  socialProviders: {
    gitlab: {
      clientId: process.env.GITLAB_CLIENT_ID as string,
      clientSecret: process.env.GITLAB_CLIENT_SECRET as string,
      issuer: process.env.GITLAB_ISSUER as string,
    },
  },
  trustedOrigins: [],
  user: {
    changeEmail: { enabled: true },
    additionalFields: {
      ageGroup: { type: 'string', required: false },
      startDate: { type: 'date', required: false },
      endDate: { type: 'date', required: false },
      muteNotifications: { type: 'boolean' },
      timeZone: { type: 'string', required: true },
    },
  },
});


Change async_setup_entry to fix reload problem.

Fixes: #7

Merging as per:  https://github.com/raulgbcr/lednetwf_ble/issues/7#issuecomment-1836768020


No Active Account

Hello, tried to set up an account but was unable to reach IBM to activate account

@blenden Did you sign up for an IBM Cloud account and log in? You'll need to do that before provisioning a Watson Studio instance. Are you following these steps? https://github.com/Call-for-Code/Spot-Challenge-Wildfires/blob/main/watson-studio-instructions.md


Cross-build to sbt 1.0.0-RC2

should MiMa itself be built with 1.0.0-RC2? it's a bit odd to use 0.13.16-M1

No. sbt 0.13 brought support for building plugins using an older version of sbt. So you could release 0.13 version of plugins while still using 0.12. We're doing the same here.

Great! I like that you moved artifact downloads to the new 1.0 API and added compatibility support to the "old" 0.13 code.

Yeah, it's how macro-compat does its compatibility support. I kind of like it.


Eosc/eosd multi-transaction mode

push_transactions has been implemented and tested in eosc on the benchmark branch.


Support jinja2 3.10+

jinja 3.10 removed some deprecated features https://jinja.palletsprojects.com/en/3.1.x/changes/#version-3-1-0
Looks like from test fails "contextfilter and contextfunction are replaced by pass_context." is the problematic change, but there may be others.

I was just looking through all the projects I'm the maintainer on and figuring out which ones to prune because I'm overwhelmed. This is at the top of the list because I figured we (collectively) must be beyond the point where this project is needed by now.
If you want to take it over, that'd be great! If not, can you find someone else who's using it to take it over? If not, I'm going to archive it.

Per #74, we're ending Puente. Closing this out.


No prod release in 3 years?

There are many problems with this package:

Many open PRs
No typescript support
Popsicle dependency is outdated and broken
Security issues
Relying on contributions from OSS community

It was stated that there would be a focus on updating this package in August of 2023. The development branch has been maintained but there hasn't been a prod release since 2021...
I'm sure this minimally impacts Intuit's bottom line, but it would be great for other businesses looking to integrate with the Quickbooks API to be able to authenticate securely with their own Oauth SDK.

For those of you in the same boat, I've managed to successfully integrate with Quickbooks using an integration agnostic Oauth2 Client (@badgateway/oauth2-client. It has much better documentation and is written in Typescript.
You can look at the guts of the intuit-oauth package to obtain the relevant URLs and scopes needed to connect.
OAuthClient.cacheId = 'cacheID';
OAuthClient.authorizeEndpoint = 'https://appcenter.intuit.com/connect/oauth2';
OAuthClient.tokenEndpoint = 'https://oauth.platform.intuit.com/oauth2/v1/tokens/bearer';
OAuthClient.revokeEndpoint = 'https://developer.api.intuit.com/v2/oauth2/tokens/revoke';
OAuthClient.userinfo_endpoint_production =
  'https://accounts.platform.intuit.com/v1/openid_connect/userinfo';
OAuthClient.userinfo_endpoint_sandbox =
  'https://sandbox-accounts.platform.intuit.com/v1/openid_connect/userinfo';
OAuthClient.migrate_sandbox = 'https://developer-sandbox.api.intuit.com/v2/oauth2/tokens/migrate';
OAuthClient.migrate_production = 'https://developer.api.intuit.com/v2/oauth2/tokens/migrate';
OAuthClient.environment = {
  sandbox: 'https://sandbox-quickbooks.api.intuit.com/',
  production: 'https://quickbooks.api.intuit.com/',
};
OAuthClient.jwks_uri = 'https://oauth.platform.intuit.com/op/v1/jwks';
OAuthClient.scopes = {
  Accounting: 'com.intuit.quickbooks.accounting',
  Payment: 'com.intuit.quickbooks.payment',
  Payroll: 'com.intuit.quickbooks.payroll',
  TimeTracking: 'com.intuit.quickbooks.payroll.timetracking',
  Benefits: 'com.intuit.quickbooks.payroll.benefits',
  Profile: 'profile',
  Email: 'email',
  Phone: 'phone',
  Address: 'address',
  OpenId: 'openid',
  Intuit_name: 'intuit_name',
};

You can use the fetchWrapper to automatically include the token in the authorization header for requests to the Quickbooks API.

As release 4.1.0 was completed, closing this issue


Annotation vault-agent-env-variables does not work

Preflight Checklist

[X] I have searched the issue tracker for an issue that matches the one I want to file, without success.
[X] I am not looking for support or already pursued the available support channels without success.
[X] I agree to follow the Code of Conduct.

Vault Secrets Webhook Version
1.21.2
Installation Type
Official Helm chart
Bank-Vaults Version
No response
Kubernetes Version
1.29
Kubernetes Distribution/Provisioner
GKE
Expected Behavior
Hello,
As referenced in https://github.com/bank-vaults/vault-secrets-webhook/blob/main/pkg/common/common.go#L84, there is an annotation identified with "vault.security.banzaicloud.io/vault-agent-env-variables".
I tried to use it with :
      "vault.security.banzaicloud.io/vault-agent-configmap" : ...
"vault.security.banzaicloud.io/vault-agent-env-variables" : "SKIP_SETCAP=true"

But the env variable is not added in the container of Vault-Agent
Have you an idea ? Maybe it is me who do it wrong
Thanks a lot in advance,
FP
Actual Behavior
Env var is not set
Steps To Reproduce
No response
Configuration
No response
Logs
No response
Additional Information
No response

Hey @4FunAndProfit,
Thanks for using Bank-Vaults!
This annotation is used to pass env-vars to the Vault agent sidecar, and not to Vault Env.
vault.security.banzaicloud.io/vault-agent-env-variables

If you would like to pass env-vars to Vault Env, use:
vault.security.banzaicloud.io/vault-env-passthrough

@csatib02, Sorry there is a mistake on my description issue (was tired when i wrote it i think)
I said vault env but it is on my Vault Agent container which does not have the env, not vault env
So i think it is a bug ?

For information, maybe related, i use:
"vault.security.banzaicloud.io/vault-agent-configmap"
but not "vault.security.banzaicloud.io/vault-agent"

I would suggest going through our documentation on using Vault Agent with the Webhook: https://bank-vaults.dev/docs/mutating-webhook/vault-agent-templating/
If you don't find anything useful I suggest you go through some of our examples, you can find it here: https://github.com/bank-vaults/secrets-webhook/blob/main/examples/test-deploy-templating.yaml
NOTE: The previous examples are used on our new project called the Secrets-Webhook, it uses different annotations, but it's easy to figure out which annotations to use on the old Webhook.

@csatib02 Thanks for your answer,
I have read these documents and I thank you for sending them to me,
Nevertheless, for me there is always unexpected behavior!
Have a look:
This works :
     => "vault.security.banzaicloud.io/vault-agent" =  "true" <= added
      "vault.security.banzaicloud.io/vault-agent-configmap" : "my-configmap"
      "vault.security.banzaicloud.io/run-as-group"= "636"
      "vault.security.banzaicloud.io/run-as-user"= "100"
      "vault.security.banzaicloud.io/vault-agent-env-variables" = "SKIP_SETCAP=true"

This don't work :
      "vault.security.banzaicloud.io/vault-agent-configmap" : "my-configmap"
      "vault.security.banzaicloud.io/run-as-group"= "636"
      "vault.security.banzaicloud.io/run-as-user"= "100"
      "vault.security.banzaicloud.io/vault-agent-env-variables" = "SKIP_SETCAP=true"

Error on Vault agent container:
unable to set CAP_SETFCAP effective capability: Operation not permitted
=> env is not given here (same for skip chown)

I've check the code, and you always need to set vault.security.banzaicloud.io/vault-agent: "true", otherwise the Vault Agent won't be used, and the vault.security.banzaicloud.io/vault-agent-env-variables annotation will be completely ignored.
Referring to the docs:
When to use vault-agent

You have an application or tool that requires to read its configuration from a file.
You wish to have secrets that have a TTL and expire.
You have no issues with running your application with a sidecar.

Moreover, checking the code I saw that env-vars are expected to be in JSON format, I would give that a go too.

Okay thanks for all these information @csatib02
Have a good day :)


horizontal scrollbar missing

On todays master build with the filesystemevents I noticed that I am missing horizontal scrollbars on files with long lines. shift + scrollwheel does not work either.

Hi @KingEdwardI ,
Interesting - we do have word-wrap on by default - when word-wrap is on, we don't show the horizontal scrollbar (because the lines get wrapped).
Does the scrollbar come back if you set "editor.wordWrap": false in your configuration? (Or - can run :set nowrap, just to test it out).
Wondering if this is due to the changed defaults, or if there is a bug in the wrap case (not wrapping fully), or if there is a bug in the no-wrap case (not showing scrollbar correctly).
I wasn't able to reproduce when I tested on that build, though:

Just tested :set nowrap which activated a functioning horizontal scrollbar. After resetting by :set wrap the wordwrap now functions as expected even after a restart.
So for me it seems that due to some unfortunate configuration state the wordwrap feature was broken.

@bryphe after building from master with the latest fix for the file stat issue, this issue surfaces again. This is my config file:
{
  "git.ignoreLimitWarning": true,
  "ui.zoom": 1.3,
  "workbench.colorTheme": "Nord",
  "editor.detectIndentation": true,
  "editor.fontFamily": "JetBrainsMono-Regular.ttf",
  "editor.fontSize": 14,
  "editor.largeFileOptimizations": true,
  "editor.highlightActiveIndentGuide": true,
  "editor.indentSize": 4,
  "editor.insertSpaces": true,
  "editor.lineNumbers": "relative",
  "editor.matchBrackets": true,
  "editor.minimap.enabled": true,
  "editor.minimap.maxColumn": 120,
  "editor.minimap.showSlider": true,
  "editor.renderIndentGuides": true,
  "editor.renderWhitespace": "none",
  "editor.rulers": [],
  "editor.tabSize": 4,
  "editor.zenMode.hideTabs": true,
  "editor.zenMode.singleFile": true,
  "files.exclude": [ "_esy", "node_modules", ".git" ],
  "workbench.activityBar.visible": true,
  "workbench.editor.showTabs": true,
  "workbench.iconTheme": "vs-seti",
  "workbench.sideBar.location": "left",
  "workbench.sideBar.visible": true,
  "workbench.statusBar.visible": true,
  "workbench.tree.indent": 2,
  "vim.useSystemClipboard": [ "yank" ],
  "window.menuBarVisibility": "hidden",
  "files.useExperimentalFileWatcher": true
}

Is there some other place where configuration is stored where I could check. I believe I can fix this again with the steps from my previous comment but I leave it as is for now so I can debug this issue.

@bryphe I found another issue with :set nowrap. the last few characters of the line can not be seen even if I am scrolled fully to the right and some more characters are behind the minimap bar. Ideally i could scroll to the right so far that the last character of the line is not behind the minibar.
here are to screenshots with the same line, one with wrap and one with nowrap

Thanks for the screenshot @kinglouie !
Ya, this looks like a bug too - similar to #3405 , but maybe a different manifestation.
Just curious - is the indentation 'tab' characters or 'spaces'? I wonder if there is a bug with considering tab characters in the context of wrapping

Your assumption with the tabs may be correct because the file indeed has tab indentation
<?php
/*
Template name: Page - Legal Texts
*/
get_header();
$post_type = get_post_type();
require_once 'template-parts/page-nav.php';

?>

<?php do_action( 'flatsome_before_page' ); ?>

				<?php while ( have_posts() ) : the_post(); ?>
					<?php if($post_type != 'page'): ?>

						<?php echo $navigation ?>

						<?php
							$meta1 = get_post_meta($post->ID, 'ext_name_1');
							$meta2 = get_post_meta($post->ID, 'ext_name_2');
							$title1 = isset($meta1[0]) ? $meta1[0] : '';
							$title2 = isset($meta2[0]) ? '  - '.$meta2[0].'' : '';
						?>

<?php echo $title1 . $title2 ?>

					<?php endif; ?>

						<?php the_content(); ?>

					<?php echo $navigation ?>
					<?php echo do_shortcode('[gap height="50px"]') ?>

						<?php echo do_shortcode('[block id="74"]') ?>

				<?php endwhile; // end of the loop. ?>

<?php do_action( 'flatsome_after_page' ); ?>

<?php get_footer(); ?>


⚠ Asset Delivery has degraded performance

In d3c7457, Asset Delivery (https://assetdelivery.roblox.com) experienced degraded performance:

HTTP code: 404
Response time: 5057 ms

Resolved: Asset Delivery performance has improved in 16509e0 after 4 days, 2 hours, 46 minutes.


Galaxy CloudMan - Running but failing the Health Test

Hello,
I am still trying to get Galaxy on AWS. This time with CloudMan.
I find it quite peculiar that, right before clicking on the Health Test button, the instance was running ok.
Here is the message I get (it is always the same):
UNABLE TO CHECK STATUS
Health check failed: CloudBridgeBaseException: Unable to locate credentials from exception type: <class 'botocore.exceptions.NoCredentialsError'>

See https://github.com/galaxyproject/cloudlaunch/issues/212#issuecomment-672057147 . Also note Galaxy CloudMan is old and to my knowledge no longer maintained. The GVL is the newest released version of CloudMan.


Fix issue with selecting time and date inside a lobby

https://github.com/sopra-fs23-group-07/sopra-fs23-group-07-client/commit/1fbfb1ebd449117ff8a931accacc4aba9e5f994d


NPE in MetamodelGenerator with named object argument

void v() {
    value v = [].follow { object head {} };
}

error encountered [the 'MetamodelVisitor' caused an exception visiting a 'ObjectArgument' node: '"java.lang.NullPointerException"' at 'com.redhat.ceylon.compiler.loader.MetamodelGenerator.encodeObject(MetamodelGenerator.java:541)'] at 2:33-2:36 of run.ceylon

The problem is the object head {} (head is the name of follow’s parameter). Happens for any named argument invocation with a named object argument (originally discovered in ceylon.ast) if the result of the invocation is assigned to a non-toplevel value. Above example will compile if you remove value v = or move it outside of void v() (specifying an explicit type).

Putting a high-prio tag on this because this affects the tests in the language module as well which can't be compiled anymore right now.

@chochos I merged a temporary "fix" to make the NPE go away at least: 262446c , but please take a look at it because it's sure to be wrong

well, it’s enough to make my tests pass :)


Too high mob spawning rate

The way it appears to me, spawn rate seems sensible. How did the too-high-rate manifest?

Can someone actually test if mob spawning rates are similar to vanilla?

Can't reproduce for now, closing until it comes up again.

I haven't seen anyone mention it to be an issue, so it probably works correctly. Closing for now.

It doesn't seem to be an issue very often. Sometimes with Quark there are a few extra mobs, mostly guardians.
The screenshot below shows 460+ squid all spawning at once. I don't know what the usual number is in vanilla at render distance 16/16. Forge 2315 and CC build 265. None of the logs report anything other than the usual File IO Thread/DEBUG.

Vanilla also sometimes tends to spawn unreasonable amountes of squids, but this seems a bit too high.


[BUG]: Unable to save Heatmap Visualization and Query

What is the bug?
Heatmap visualization and query cannot be saved because of a toast that says Invalid Configuration Options found
How can one reproduce the bug?
Steps to reproduce the behavior:

Select any visualization or query.
If visualization selected, select heatmap from chart dropdown.
Render heatmap by selecting dimensions and series.
Try to save visualization by clicking on save.
Select panel and name for visualization and click on save, you will see error.

What is the expected behavior?
Visualization and query should be saved without getting an error if user has added all the correct data.
Do you have any screenshots?

This is fixed and merged to main.


Create a Table to showcase upcoming society events

Upcoming events include:

Hey @OliverM224  Can you please assign this issue to me


Documentation + Fixes / Cleanup

What's changed:

Added documentation to a few classes.
Rewrote vague documentation in a few instances.
Removed redundant code such as myBoolean == true and myBoolean = false in constructors.
Minor formatting and code cleanup in a few classes.

What's fixed:

Grass and dirt textures were not properly merging due to the world rewrite. Now looks as it did before the re-write.
Proper resource closure in ModLoader.
Proper resource closure in WorldProvider.
Corruption BG not reaching the bottom of the screen.
ConcurrentModificationException thrown if entity.die() was used, now fixed.
maxAI used by Enemy was static, but was set to unique values in different enemy implementation constructors.

Thank you!


Can CONFIG_SOC_WATCH-related code be removed?

Commit ac47070d10 ("tests: qmsi: remove soc watch sample") removed the CONFIG_SOC_WATCH symbol, but there's still a bunch of related code left (see git grep SOC_WATCH).
Can it be removed?

AFAIK, it is already removed.


Inline images for a blur placeholder

Hi there,
I was wondering if it’s possible to inline blur placeholders as Base64 data URLs to reduce load time?
In my current project, I display 50 images in a grid on each page. Loading 50 placeholder images via HTTP as separate files seems to create unnecessary overhead. It would be great if this plugin could support inlining these placeholders to improve performance. I couldn’t find an option for this feature in the existing implementation.
Here’s an example of the current behavior:

And here’s what I’d love to see:

Thanks for considering this feature request!

Hi @markvital,
I thought about this feature a while ago, and I don't think it's possible in the way this library works currently. The library would need to modify all occurrences of ExportedImage and change the source code itself.
The only way I can think of is, that the library spits out a JSON file of the image file names as keys and the base64 placeholder images as values.


Change how load files is handled

Currently we use the viewer utils/io.py to load files which is not great, see https://github.com/vais-ral/CILViewer/issues/162.
This should be handled internally using the resample readers in the viewer.

this is waiting for https://github.com/vais-ral/CILViewer/pull/216

Currently, the io.py has been removed from the viewer and moved to this repo.
What still needs doing is to clear the mess that that file contains ;)

the above pr addresses most of the tidying needed


Why not publish to Maven Central

It would make it easier if you published to Maven Central so we could simply reference your jar in our pom.xml. I found, after building with: mvn install clean I was able to reference locally in my ~/.m2/repository using:
    <dependency>
      <groupId>com.google.search.robotstxt</groupId>
      <artifactId>robotstxt-java</artifactId>
      <version>1.0-SNAPSHOT</version>
    </dependency>

But I need to inform others to clone and build locally if they intend to use the library.
It would also be helpful to add the build for *-jar-with-dependencies.jar by updating the pom.xml with these lines in the plugin:
      <plugin>
        <artifactId>maven-assembly-plugin</artifactId>

add these
        <executions>
          <execution>
            <id>simple-command</id>
            <phase>package</phase>
            <goals>
              <goal>attached</goal>
            </goals>
          </execution>
        </executions>

end addition
        <configuration>
          <descriptorRefs>
            <descriptorRef>jar-with-dependencies</descriptorRef>
          </descriptorRefs>
          <archive>
            <manifest>
              <mainClass>com.google.search.robotstxt.RobotsParserApp</mainClass>
            </manifest>
          </archive>
        </configuration>
      </plugin>

+1


Improve lazyloading images

New Feature for Image LazyLoad
On lazy loading a image, it is good idea to show placeholder/default image if somehow data-src or data-background  src image couldn't get loaded.

Merged, thanks


adds test example for hello_update.py

What was changed
adds test example for hello_update.py
Why?
Checklist

Closes

How was this tested:

Any docs updates needed?

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

This test looks good, thanks! But I think CI is failing for a known update bug fix we haven't released yet. In https://github.com/temporalio/sdk-python/pull/521 we made sure that workflows and updates completed in the same task are properly handled. You may be able to update to Python SDK 1.6.1 to check (update pyproject.toml to 1.6.1 then poetry update temporalio), but otherwise it may have to wait until next release.

Make sure you reformat (e.g. poe format), CI is currently failing

Yep, my bad - I had just accepted the suggestion you shared - fixed now


Broken example for NavigatorPopHandler

Steps to reproduce

Open https://api.flutter.dev/flutter/widgets/NavigatorPopHandler-class.html#widgets.NavigatorPopHandler.2 (Example 2)
Click "Go to another route ... "
Click "Go back"
Error

Expected results
Successfull back navigation

@LowLevelSubmarine I believe this has been fixed in https://github.com/flutter/flutter/pull/132249, but it is not yet in stable.
To test the change
https://main-api.flutter.dev/flutter/widgets/NavigatorPopHandler-class.html#widgets.NavigatorPopHandler.2


add possibility to echo the manipulated media

I'd like to have an option to directly echo the manipulated audio/video.
For example:
require 'vendor/autoload.php';

$ffmpeg = FFMpeg\FFMpeg::create();
$video = $ffmpeg->open('video.mpg');
$video->filters()->resize(new FFMpeg\Coordinate\Dimension(320, 240))->synchronize();

header('Content-type: video/mpeg');
echo $video->readData();

It would be awesome if there could also be a possibility to read audio/video data from a variable and not only from a file.
For example:
require 'vendor/autoload.php';

$data = file_get_contents('video.mpg');
$ffmpeg = FFMpeg\FFMpeg::create();
$video = $ffmpeg->read($data);

As a work-around, you can use a tmpfile :)


conda package?

Would it be possible to make HpBandSter available not just as a pip but also as a conda package?

Package is being created at https://github.com/conda-forge/staged-recipes/pull/13440

Awesome, thank you!


Default column length

When working with a large number of columns, setting the column length as the maximum varchar length( 65,535 characters) could cause an error:
 target_postgres.postgres.PostgresError: ('Exception writing records', QueryCanceledError('Intermediate result row exceeds database block size\n',))
This is why I suggest decreasing the default (also, this would decrease the table size, so, the cost of our Redshift cluster). Yes, this is still a limitation, but sometimes is better to start by using a smaller column length and increase it afterwards(if needed) than the opposite.

I like this approach since values would get rejected still providing feedback to users.
@mirelagrigoras what would you think about providing a default from the configuration and using that? ie, a user could specify default_column_width.
@awm33 thoughts here?

To me, this seems to be a great idea.This way, the target could be
customized by the user according to their needs.
În Sâm, 9 feb. 2019, 01:26 Alexander Mann<EMAIL_ADDRESS>a scris:

I like this approach since values would get rejected still providing
feedback to users.
@mirelagrigoras https://github.com/mirelagrigoras what would you think
about providing a default from the configuration and using that? ie, a user
could specify default_column_width.
@awm33 https://github.com/awm33 thoughts here?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/datamill-co/target-redshift/pull/9#issuecomment-461981646,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AtMRVGeedfRbEPzpzOHpfFQh6E_jSTPgks5vLgesgaJpZM4avkpL
.

I like this approach since values would get rejected still providing feedback to users.
@mirelagrigoras what would you think about providing a default from the configuration and using that? ie, a user could specify default_column_width.
@awm33 thoughts here?

I added the default length in the configuration file. Let me know what you think @awm33  @AlexanderMann
Thanks!

@mirelagrigoras This great! Thank you for your contribution! I just noticed a one line issue, if you could take a look

@mirelagrigoras Great! Approved! We'll get this merged and deployed soon

@mirelagrigoras Great! Approved! We'll get this merged and deployed soon
Actually, let me explain what was the logic when I set the value to self.MAX_LENGTH, because I think this is the correct way to handle the situation.
So, from what I understand, in the schema there might be a key "maxLength". If there is, max_length will be sent to that value, otherwise, it will be set to self.default_column_length. However, if the key exists in the dictionary and its value is greater than the default, then we should set max_length to self.MAX_VARCHAR since setting it to a lower value could cause the import of the data into the table to fail (resulting into a Unable to load data:
String  length exceeds DDL length
error).
I migth be wrong, so please, let me know if I misunderstood something.

@mirelagrigoras Ah, I see, so if maxLength is set, it overrides the default. If that's the intent, then I think I would have expected:
max_length = schema.get('maxLength', self.default_column_length)
if max_length > self.MAX_VARCHAR:
    max_length = self.MAX_VARCHAR

This way, there is a default, and it can be overridden by maxLength in the schema, but we don't necessarily set it to MAX_VARCHAR when it's greater than the default. For example if the default was 2000, and maxLength was 2500, we would set it to 64k.

That is a great catch.This should be the expected behaviour.I will change
the condition.Thank you for your time!
În Dum, 10 feb. 2019, 00:25 Andrew Madonna<EMAIL_ADDRESS>a
scris:

@mirelagrigoras https://github.com/mirelagrigoras Ah, I see, so if
maxLength is set, it overrides the default. If that's the intent, then I
think I would have expected:
max_length = schema.get('maxLength', self.default_column_length)
if max_length > self.MAX_VARCHAR:
max_length = self.MAX_VARCHAR
This way, there is a default, and it can be overridden by maxLength in
the schema, but we don't necessarily set it to MAX_VARCHAR when it's
greater than the default. For example if the default was 2000, and
maxLength was 2500, we would set it to 64k.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/datamill-co/target-redshift/pull/9#issuecomment-462084900,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AtMRVOELsZe1OhDzb8Gyla90DEPaVLy-ks5vL0q9gaJpZM4avkpL
.

@mirelagrigoras looks good. I'm going to go ahead and merge this. At the beginning of next week, if there are no other changes, I'll get a release of this ready to go!

@mirelagrigoras version 0.0.3 is now live!

Thank you both so much!
În Lun, 18 feb. 2019, 17:32 Alexander Mann<EMAIL_ADDRESS>a
scris:

@mirelagrigoras https://github.com/mirelagrigoras version 0.0.3 is now
live! https://pypi.org/project/target-redshift/0.0.3/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/datamill-co/target-redshift/pull/9#issuecomment-464776478,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AtMRVFaCx5E-BqC5T_U7CrpV0z5CeG4zks5vOsd7gaJpZM4avkpL
.


Shift-enter mappings being ignored

Make sure you have done the following

[X] I have updated to the latest version of blink.cmp
[X] I have read the README

Bug Description
Firstly, thanks for the awesome plugin!
Apologies in advance if I've misunderstood the way keybinds are managed. I've found that, in certain files (in particular markdown and typst, but possibly others), the completion window appears after typing common punctuation characters, and so I'd like to be able to press shift+enter to dismiss the window and enter a newline.
But with the configuration I have below, shift+enter behaves exactly the same as just enter by itself (i.e. it accepts the suggestion). Is there a different approach I should be using?
Thanks :grin:
Relevant configuration
{
    keymap = {
      preset = "enter",
      ['<Tab>'] = { "select_next", "fallback" },
      ['<S-Tab>'] = { "select_prev", "fallback" },
      ['<C-j>'] = { "select_next", "fallback" },
      ['<C-k>'] = { "select_prev", "fallback" },
      ['<C-c>'] = { "hide", "fallback" },
      ['<S-CR>'] = { "fallback" },
    },
    // ...
  }

neovim version
NVIM v0.11.0-dev Build type: Release LuaJIT 2.1.1713773202 Run "nvim -V1 -v" for more info
blink.cmp version: branch, tag, or commit
v0.5.1

Thanks for the kind words! :) Do you mind retesting this on the latest release (v0.6.1 as of writing), and if the bug persists, trying on neovim 0.10? The outputs of :imap <CR> and :imap <S-CR> would also be helpful

Closing for now since this seems likely be an issue with your config but lmk if you're still running into this on a fresh config

Sorry, just got back to this - I realised the whole issue was actually a tmux issue - when I was checking to see kitty could actually handle <S-CR>, I was checking in a terminal window that wasn't running tmux, but was testing blink inside tmux, which was stopping the keybinding from working at all. The filetype-specific trigger characters seems like a very useful change in the meantime though :grin: Thanks for your effort


fix: set leaderOnly parameter camelCase

What

changed leaderOnly parameter to camelCase

Why

to match JS convention

Checks

[X] I have tested this code
[X] I have reviewed my own PR
[ ] I have created an issue for this PR
[X] I have set a descriptive PR title compliant with conventional commits

Good call, got confused with the backend when prototyping I guess! LMK when you have make a new release so I can test it with the studio. @cristiam86


Observer not working when store obj update

I did same thing in my application. On click of menu icon store data is update but that time observer not called.
My component structure is,
global Component
|
Header Component      Sidebar Component      Home Component      Footer component
Global Component is parent component and these four of child component but these four component are sibling component each other.
Menu icon is placed inside header component and click event emit into global component and fire dispatch event from global component. After that nothing happen.
Kindly help me to get it out of it.
Thanks in advance..!!

I would recommend reaching out on Gitter for support and questions.


suppress_stdout_stderr failing in notebooks

Expected Behavior
Loading a model with verbose=False should not print things.
Current Behavior
verbose=False has no effect in notebooks (I tested with VS Code notebooks, but I think it is ipykernel related)
Steps to Reproduce
Just load a model with verbose=False and observe output in a notebook environment.
How to fix
The patch given in the ipykernel issue below fixes the issue for me (but requires me to do my own override of stdout and stderr since the patch also causes suppress_stdout_stderr to crash as currently written). A real fix might involve integrating this into suppress_stdout_stderr directly.
https://github.com/ipython/ipykernel/issues/795
thanks!

I got a nicer version of the patch working that I will share here is case it helps others:
class normalize_ipykernel_stdout_stderr():
    '''Remaps stdout and stderr back to their normal selves from what ipykernel did to them.

    Based on: https://github.com/ipython/ipykernel/issues/795
    '''

    def __enter__(self):
        normal_stdout = sys.__stdout__.fileno()
        self.restore_stdout = None
        if getattr(sys.stdout, "_original_stdstream_copy", normal_stdout) != normal_stdout:
            self.restore_stdout = sys.stdout._original_stdstream_copy
            sys.stdout._original_stdstream_copy = normal_stdout

        normal_stderr = sys.__stderr__.fileno()
        self.restore_stderr = None
        if getattr(sys.stderr, "_original_stdstream_copy", normal_stderr) != normal_stderr:
            self.restore_stderr = sys.stderr._original_stdstream_copy
            sys.stderr._original_stdstream_copy = normal_stderr

    def __exit__(self, exc_type, exc_value, traceback):
        if self.restore_stdout is not None:
            sys.stderr._original_stdstream_copy = self.restore_stdout
        if self.restore_stderr is not None:
            sys.stderr._original_stdstream_copy = self.restore_stderr

given the above context manager you can now wrap a model load and things work like you would expect:
with normalize_ipykernel_stdout_stderr():
    llama_cpp.Llama(model_path=model, verbose=False)

@slundberg yes been meaning to dig into this issue, thank you so much for the fix!

I haven't been able to get a version that consistently works in all environments, if anyone can give this a shot it would be much appreciated.

Have y'all been using the environment variable?
LLAMA_LOG_VERBOSITY=0

Been testing locally and this seems to work to hide the enormous preamble debugging stuff that we usually get.