Updated Fluent Button designs for macOS

Platforms Impacted

[ ] iOS
[x] macOS

Description of changes

Incorporated more recent (but still evolving) designs for Fluent button styles
Added semantic colors associated with the styles
Added properties and init parameters for button size and accent color
Added properties to customize button foreground/background/border color for any of the rest/pressed/disabled states, for a total of 9 colors
Added computed properties that will allow setting just one color (the rest color) and will automatically apply system effects to determine colors for the other states
Fine-tuned text and image alignment for good looking results in all cases
Added a property allowing a secondary button to de-highlight a linked primary button when the secondary is pressed:

Verification

Added examples of each the four standard styles and one custom style to the test app, in both large and small sizes
Included examples of each possible imagePosition
Included examples of different accent colors
Added controls to preview different background materials (including translucent effects), background colors, and NSColor system effects
Made the button pane scrollable so that it doesn't expand to arbitrarily large size with added cases

Pull request checklist
This PR has considered:

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

Microsoft Reviewers: Open in CodeFlow

I know you probably already have talked to designers / gotten requirements and such, but I wanted to ask here so it's somewhat public / I wasn't sure: Why do we want to override the rest/hover/press colors? Wouldn't that be going against the FluentUI design language if we can let clients override those? Or is it because those are design tokens that are overridable, so we need to support them?

I know you probably already have talked to designers / gotten requirements and such, but I wanted to ask here so it's somewhat public / I wasn't sure: Why do we want to override the rest/hover/press colors? Wouldn't that be going against the FluentUI design language if we can let clients override those? Or is it because those are design tokens that are overridable, so we need to support them?

Good question.  I brought this up today with Amy from the design team and she seemed to agree that keeping the configurable parameters limited to size, style and accent color would make sense from the perspective of the Fluent design language.  Might need some input from Amanda on how this affects the custom button style PR she recently completed, but I did notice that the "custom" style example in the test app could be represented equally well by a Primary style button with a different accent color, without tweaking any of the specific colors at all.

Oh yeah... I forgot about this. We do have local documentation in the form of markdown files for each of the macOS controls, here.
Those eventually make it to the FluentUI Public website by making a PR like this one.
I don't know if that's worth blocking the review, since we (surprise) haven't seemed to keep up with documentation muc, but it's worth noting, even if it's just future work for the next documentation day.

Added updated docs to the pull request.

In reply to: 741579753 [](ancestors = 741579753)


app.listen() error behavior is different in Express 4 vs. 5, if port is occupied

Hi there,
Great to see that Express 5 with async request handlers is finally available :-)
I have a question about behavior that differs in Express 4 vs. 5, if the listen port is already in use.
I wanted to know if that's on purpose, or if it's a bug.
If it's on purpose, it should probably be mentioned here? https://expressjs.com/en/guide/migrating-5.html
Thanks for having a look!
// minimal-express.mjs

import express from 'express';
import expressPkg from 'express/package.json' with { type: "json" };

const app = express();
app.use(express.json());
app.get('/api/health', (req, res) => {
	res.status(200).json({ status: 'ok' });
});

console.log('Express ', expressPkg.version);
console.log('Starting Express server on port 7072...');
const server = app.listen(7072, '<IP_ADDRESS>', () => {
	console.log('Callback');
	if (!server.address()) {
		// This line is only reachable in Express 5, but not in Express 4.
		throw new Error('Failed to start server (Express 5).');
	}
	console.log('Listening on http://localhost:7072/ - e.g. http://localhost:7072/api/health');
});

Enviroment: WSL2, Ubuntu 22.04, Node 20 or 22
PRETTY_NAME="Ubuntu 22.04.5 LTS"
Linux MYHOSTNAME <IP_ADDRESS>-microsoft-standard-WSL2 #1 SMP Tue Nov 5 00:21:55 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Node.js v20.18.1
Node.js v22.11.0 (same result)

Results with Express 4, if port is already in use:
Express  4.21.1
Starting Express server on port 7072...
node:events:497
      throw er; // Unhandled 'error' event
      ^

Error: listen EADDRINUSE: address already in use <IP_ADDRESS>:7072
    at Server.setupListenHandle [as _listen2] (node:net:1904:16)
    at listenInCluster (node:net:1961:12)
    at doListen (node:net:2135:7)
    at process.processTicksAndRejections (node:internal/process/task_queues:83:21)
Emitted 'error' event on Server instance at:
    at emitErrorNT (node:net:1940:8)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  code: 'EADDRINUSE',
  errno: -98,
  syscall: 'listen',
  address: '<IP_ADDRESS>',
  port: 7072
}

Results with Express 5, if port is already in use:
Express  5.0.1
Starting Express server on port 7072...
Callback
file:///some/folder/minimal-express.mjs:14
                throw new Error('Failed to start server (Express 5).');
                ^

Error: Failed to start server (Express 5).
    at Server.<anonymous> (file:///some/folder/minimal-express.mjs:14:9)
    at Server.f (/some/folder/node_modules/once/once.js:25:25)
    at Object.onceWrapper (node:events:634:26)
    at Server.emit (node:events:519:28)
    at emitErrorNT (node:net:1940:8)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)

Although I don't know for sure, the code indicates that this is on purpose. Comparing the implementation of the listen method in 4.x ...
https://github.com/expressjs/express/blob/59fc27028ec5d212be653d35d7e3f73a2c3ac3c0/lib/application.js#L633-L636
... with 5.x ...
https://github.com/expressjs/express/blob/344b022fc7ed95cf07b46e097935e61151fd585f/lib/application.js#L609-L617
... the code now explicitly invokes the user-provided callback (if it exists) in the event of an error. I'm guessing this is to provide the user with more control over their errors, instead of being subject to a crashing app. However, I agree with you that this transfer of error-handling responsibility to the user in the listen method would be good to capture in the migration docs. I can open a PR for that soon.

Okay probably the Express user is now required to check the 1st argument of the callback.
const server = app.listen(7072, '<IP_ADDRESS>', (error) => {
	if (error) {
		// This line is only reachable in Express 5, but not in Express 4.
		console.error("Failed to start server (Express 5)");
		throw error;
	}
	console.log('Listening on http://localhost:7072/ - e.g. http://localhost:7072/api/health');
});

Works fine:
$ node /tmp/express-test/minimal-express.mjs
Express  5.0.1
Starting Express server on port 7072...
Failed to start server (Express 5)
file:///tmp/express-test/minimal-express.mjs:16
                throw error;
                ^

Error: listen EADDRINUSE: address already in use <IP_ADDRESS>:7072
    at Server.setupListenHandle [as _listen2] (node:net:1817:16)
    at listenInCluster (node:net:1865:12)
    at doListen (node:net:2014:7)
    at process.processTicksAndRejections (node:internal/process/task_queues:83:21) {
  code: 'EADDRINUSE',
  errno: -98,
  syscall: 'listen',
  address: '<IP_ADDRESS>',
  port: 7072
}

I took a quick read here, but am a bit busy with other work, just wanted to link the original PR for context: https://github.com/expressjs/express/pull/3216


Torque minimum granularity blocked to 4 minutes

Spotted in Torque cat but also happening in the regular Torque wizard.
Ranges for the data:
max
1865-04-02T18:29:00Z
min
1865-04-02T12:00:00Z
We tried adding some years in order to maintain the epoch timestamps as positive numbers:
max
2005-04-02T18:29:00Z
min
2005-04-02T12:00:00Z
If you reduce the steps to small values, this is correct. Once you pass 100 frames it gets blocked to 4 mins no matter which big number you use there.
Sample of CartoCSS:
/** torque visualization */

Map {
-torque-frame-count:2048;
-torque-animation-duration:10;
-torque-time-attribute:"nueva_hora";
-torque-aggregation-function:"count(cartodb_id)";
-torque-resolution:2;
-torque-data-aggregation:linear;
}

#lumpkin_points_final{
  comp-op: lighter;
  marker-fill-opacity: 0.9;
  marker-line-color: #FFF;
  marker-line-width: 1.5;
  marker-line-opacity: 1;
  marker-type: ellipse;
  marker-width: 6;
  marker-fill: #FF9900;
}
#lumpkin_points_final[frame-offset=1] {
 marker-width:8;
 marker-fill-opacity:0.45;
}
#lumpkin_points_final[frame-offset=2] {
 marker-width:10;
 marker-fill-opacity:0.225;
}

CSV sample attached here. Category column should be type number and the dates columns of type date.

Closing this since it isn't a problem with granularity. If the issue can be replicated let's open a new issue regarding it.


Script Commands Store – Unable to remove inaccessible scripts

Description
If script directory is inaccessible; Raycast throws an error when you try to run or remove an installed script.
This presents a problem if your script directory is shared between Macs across a smb share, for example.
Also, if the directory is unavailable Raycast lets you install a script, reports as installed, but is no where to be found, then can't be removed. Reloading scripts directory has no effect.

Error:
Error: ENOENT: no such file or directory, stat '/Users/jxxxxxxxxx/raycast/script-commands/images/pulse-secure.png'

Object.statSync:node:fs:1536:3
__node_internal_:utils:793:8
rmSync:node:fs:1211:13
qo.deleteFile:list.js:17:10918
qo.deleteIcon:list.js:17:10866
qo.delete:list.js:17:8275
eu.deleteScriptCommand:list.js:21:3994
x:list.js:21:7149
d:list.js:21:12365

Raycast scripts directory (from above): /Users/jxxxxxxxxx/raycast/script-commands/
Actual scripts directory (configured in extensions/Script Commands): /Volumes/Shared/Temp/Sync/Raycast/script-commands/
personal info redacted
Steps To Reproduce

Install Raycast
configure custom scripts directory on network share
Install script
Unmount network share
Try to run or uninstall script

The current behavior
Unusable "installed" scripts; yet unable to remove without error.
The expected behavior
Throw error that script is unavailable; allow user to remove reference to unavailable script.
Raycast version
Version: 1.35.2

Not complete, issue still exists.


Is there any way to use Knock with route constraints?

I'm looking at authenticating Shrine presigned routes through constraints and if the Knock::Authenticatable could be leveraged in a route constraint.
Something like:
lib/constraints/authenticated_request.rb
module Constraints
  class AuthenticatedRequest
    include Knock::Authenticable

    def matches?(request)
      auth_header = request.headers.fetch('Authorization')
      return false unless auth_header
      authenticate_user
    end

  end
end

and in routes:
  mount ImageUploader.presign_endpoint(:cache) => '/images/presign', constraints: Constraints::AuthenticatedRequest.new

I know this is an old issue but I bumped into this problem today. This is the current implementation I'm using:
##
# This constraint asserts the current user is an admin
class AdminConstraint
  def matches?(request)
    return false unless request.headers['Authorization']

    token = request.headers['Authorization'].split(' ')[1]

    user_id = Knock::AuthToken.new(token: token).payload['sub']
    user = User.find(user_id)

    user.admin?
  end
end

Be wary as I'm not familiar with the Knock APIs.


add http headers to healthcheck

What does this PR do?
Tries to fix issue #3043
Motivation
We need this feature, describe in issue
More

[ ] Added/updated tests
[ ] Added/updated documentation

Additional Notes

This is still WIP
I am adding healthchecks
        "healthcheck": {
            "path": "/",
            "interval": "5s",
            "headers": {
            	"Host": "foobar.com",
            	"header1": "test",
            	"headeR2": "test2"
            }
        },

I can see in logs DEBU[2018-03-19T14:32:53+02:00] Setting up backend health check [Headers: map[Host:foobar.com header1:test headeR2:test2] Path: / Port: 0 Interval: 5s]
However, still I am not able to see Host header in client.
ERROR:root:Host: <IP_ADDRESS>:8000
User-Agent: Go-http-client/1.1
Header1: test
Header2: test2

I am thinking might this cause this issue still https://github.com/golang/go/issues/7682

Got it working, its little bit tricky in Host header. Is these tests and docs enough or do we need more?

Thanks for your contribution!
I understand your use case with the hostname, but I'm not sure about changing headers.
Do you think you can just add "hostname option" and remove other headers option?

@Juliens IMO headers are useful for instance if the healthcheck needs authentication. So I do not see why we could not add those in same PR

@zetaab After discussion with the team, we agree to add both hostname and headers on healthcheck.
But we think we need to separate hostname from the headers.
If I take your proposal, it becomes like this:
        "healthcheck": {
            "path": "/",
            "interval": "5s",
            "hostname": "foobar.com",
            "headers": {
            	"header1": "test",
            	"headeR2": "test2"
            }
        },

Moreover, we need the functionnality to be added on every providers.
If you need help, you can share with us on our Slack workspace

@zetaab Could you also update the doc for each provider, thanks.


starter app now has a button that does something

Addresses https://github.com/flutter/flutter/issues/1152

cc @devoncarew @abarth

LGTM


Value of type 'TransitionController' has no member 'present'

Hi,
I just installed this library using CocoaPods and I have this error, but I use the exactly same code provided by demo project
        transitionController.userInfo = ["destinationIndexPath": indexPath, "initialIndexPath": indexPath]
        transitionController.present(viewController: presentedViewController, on: self, attached: presentedViewController, completion: nil)

Any idea ??
Thanks.

@aliasdoc Can you build and run demo project?
(I want to check that error immediately  but I currently can not run cocoapods in my PC :sad:)

Hi @naru-jpn, it's resolved, it was just an older version installer with cocoa pods (0.0.2), just run
pod update View2ViewTransition

to install last version (0.0.4)


Update output format for test run in circle ci

Motivation / Context

The current output format for tests run in Circle CI is short-verbose (aka testname). This format does not print sufficient info in case of test failure. For example, it does not print the content written by the program to stdout.
Because of this, reason for test getting stuck (see this CI job) cannot be figured out from test logs.
Description

Update the output format to standard-versbose, where content written by the program to stdout is also printed.
Relates to

This CI job

Closed by #178.


Update PSA example program for counters for latest psa.p4

@hanw @cc10512 Before updating all of the example PSA programs, I wanted to run these suggested updates to one example program by your first, since the rest will likely be copy and paste from this one.
In particular, look at the instantiation of PSA_Switch near the end of this proposed change to the program psa-example-counters.p4 of this PR.  This is syntactically correct according to p4test, but requires adding a 0-argument constructor for PacketReplicationEngine and BufferingQueueingEngine extern objects in psa.p4.  Is there another way that avoids the need for an explicit construct you were thinking of instead of this?

We can provide a macro in psa.p4
#define PSA_SWITCH(x, y) PSA_Switch(x, PacketReplicationEngine(), y, BufferingQueueingEngine())

and in user program they can write
PSA_SWITCH(ip, ep) main;

I could not figure out another way in p4.

@hanw @cc10512 I added the convenience macro that Han described, and used it in the example.  Calin, if you are good with this, I can update the rest of the example PSA programs using this updated psa-example-counters.p4 as a pattern.


The Big One (details to come)

What is this PR doing?
How can these changes be manually tested?
Does this PR resolve or contribute to any issues?
Checklist

[x] I have manually tested these changes
[ ] Post a link to the PR in the group chat

Guidelines

If your PR is not ready, mark it as a draft
The resolve conversation button is for reviewers, not authors

(But add a 'done' comment or similar)

@voltrevo Appreciate the gusto in this refactor, but this is going to be difficult to review. Possible it can be broken up into multiple, smaller chunks?

@voltrevo Appreciate the gusto in this refactor, but this is going to be difficult to review. Possible it can be broken up into multiple, smaller chunks?

Quite possibly. I'm going to put a significant effort into documenting these changes.

@jacque006 I said before that I'd have updates today, buuut I'm gonna defer til next week instead. Just got done with what I needed to do with my other job and I've only got Friday left now. Time to take a rest. Still happy to chat sooner if you wish.


Wrong Receiver builder type state

The receiver type state should require source to be set before it can proceed to attach to a session

#45


bytes from sync.Pool not zeroed

This is more of a question/bug report.
From my understanding of sync.Pool(https://golang.org/pkg/sync/#Pool); once you get an item from it, you should zero it before using it.
However, in this case; caddy does not do so;
https://github.com/mholt/caddy/blob/13a54dbdda25c5da12f34d49f22b8d3eb23e3140/caddyhttp/proxy/reverseproxy.go#L63
Is there a reason why caddy doesn't zero the []byte before use, or is it a bug?
here's an example of the zeroing i'm talking about;
https://github.com/bradfitz/http2/blob/aa7658c0e9902e929a9ed0996ef949e59fc0f3ab/server.go#L1414-L1416

Please read the document of io.CopyBuffer.
https://golang.org/pkg/io/#CopyBuffer
buf is just a buffer. The content doesn't matter.

You could zero it out, but it's not the responsibility of the pool consumer to do that. The pool might zero it out if the implementation change.


Requirements to display branch name in commit details page?

What are the requirements to display branch name in commit details page, when the branch is not master?  For example, we'd like to display the release branch name in the commit details page.  Currently, no branch name is displayed for commits in the release branches.

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

ID: a6255243-5e5e-424c-5527-ec792c9b950e
Version Independent ID: 2033d250-5385-1e02-d111-c47e44fdc8ef
Content: View commit details - Azure Repos
Content Source: docs/repos/git/commit-details.md
Product: devops
Technology: devops-code-git
GitHub Login: @apawast
Microsoft Alias: sancha

I agree with @yorhpokie, we need to see the branch name, this is pretty critical to our current workflow


The type or method 'ClearLastError' cannot be used for 'Extension' development.

The header says it all, why is this method not usable in extension development?

I will be.


soft-serve: init at 0.1.0

Motivation for this change
Add soft-serve, a tasty, self-hosted Git server for the command line to the nixpkgs collection 🍦
Things done

Built on platform(s)

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

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

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

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

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

[X] Fits CONTRIBUTING.md.

@legendofmiracles Thanks for the suggestions 💪 !

Heh, seems like githubs code suggestions doesn't comply to the editorconfig settings 😅 quickly fixed that

@legendofmiracles Thanks for the suggestions 💪 !

No problem, really awesome tool! Can you squash the commit made by my changes into your other commit?

Nice to hear 😊 Yup, squashed the commits 🤙 thanks for reviewing this!


Sprint1 depoly lambda function

See the following updates:

Changed file structure within lambda to differentiate source code for local test and AWS deployment
Created new versions of dynamoDB connecting util function and lambda handler function
Adjust table writing logic to check duplicate before inserting
Update dockerfile in order to build the correct image for deployment

Looks good, merged and safely deleted branch.


Tutorial: getting started is missing the "require " line

I'm a very occasional PHP developer who codes in Java and JS most of the time. I tried using this module and followed the getting started wiki page but I got an interupted html output. It was only after I read issue #483, did I see I had not included the require 'Parsedown.php'; line.
Please add this line to both the getting started page and the README.md file.

This is already in the README.md, check the Installation section. I've explicitly added it to the wiki, too.

Thanks for the quick reply. I found the README file to be ambiguous. I did read it before starting but I interpreted the "include" text to mean you simply download (include) the file in the same directory as the php file, which I did. Maybe explicitly show code example with and without the include line, depending on whether the composer package is used(?).


A 8 pages brochure to show case how tokenization "help" with car use case

Can be used stand-alone or in the deck.
A token can be blockchain token to non-blockchain token
Take out my mobile, and tell Siri I want to buy a new car for running “uber”, based on the tokens in my wallet, Siri knows the full picture on me like age, family, income, asset, preferred colour, lifestyle and etc. Siri suggest a Honda Odessey to me, with a customised loan option. Siri scared all the new car token on market, pick the one fit me best. Then compared with all the loan options. (As they are all tokenised, in a frictionless market, anyone can access all the information in the market). I click on button, with one atomic transection, the loan contract verified my tokens which proved my ability to pay, collect my first instalment Dai, speak to the car sell contract, transfer the full amount to car sell contract, car sell contract speak to Gov reg contract, Reg contract verify my indemnity tokens then issue reg token to me, car sell contract returns delivery token, receipt token and car token, car sell contract speak to manufacture contract, manufacture contract returns with warranty token. So after this one click I transfer out 10000 Dai, and receive a car token, a receipt token, a warranty token, a delivery token, a reg token, a loan token.
My asset
My debit
Told Siri, I want my car be delivery to my home on 18:00pm Tuesday. Siri add these requirement to delivery token, offers start being sent to delivery token, pickup the best offer based on reputation and cost. Then update the delivery token again with delivery company’s info. The delivery company go to warehouse pickup the car smoothly as their info already in delivery token. Company delivery the car to my home, he scanned my delivery token and received payment from car sell contract. I check the car, all good. Using my car token to use the car. Ask Siri to get me a permission of run “uber” services from Gov, Siri paid 10dai fee to GOV contract on the same time submitted tokens to the contract to verify that I am a qualified drivers. Get the permission token associate with my car token. Now Ih ave one more button on the car token which is start “Uber”, once I click start I start receive p2p offers and marketplace engine offers, my SIRI is picking up offers form me automatically. I start doing uber business.
I’m leaving town for a while, with one click I start receiving	rent offers from p2P or marketplace offers. SIRI picks up the right ones to maximum my returns.
Car services
Car repaire
Car Collateralise
Car parking fee
Car used by my friend and family
Car resell (with loan)
Etc.

Please post the idea to https://www.tokenscript.org/c/usecase and I'll follow up from there.


Modifying profile, referral, clear sections to follow format

Summary
Resolves #297
Changes referral and profile section to follow format of the other sections, split clear into its 4 respective sections. Image removed from help as well. Can help to check if missed out anything or if something is out of place.
Detail
Pictures / Gif
Checks

[ ] Link PR to issue
[ ] Add Reviewer
[ ] Pass CI

Codecov Report

Merging #299 (4dbf7c0) into master (9fca825) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 4dbf7c0 differs from pull request most recent head 990bcfd. Consider uploading reports for the commit 990bcfd to get more accurate results

@@            Coverage Diff            @@
##             master     #299   +/-   ##
=========================================
  Coverage     72.50%   72.50%
  Complexity     1037     1037
=========================================
  Files           147      147
  Lines          3463     3463
  Branches        391      391
=========================================
  Hits           2511     2511
  Misses          809      809
  Partials        143      143

Continue to review full report at Codecov.

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

ive fixed the necessary things u can check them again @simonjulianl


Unusual assert.expect in TrailTests.js

In https://github.com/phetsims/aqua/issues/31#issuecomment-350799249 @pixelzoom said:
@jonathanolson I assume that this is also unnecessary in TrailTests.js:
680    // this may change if for some reason we end up calling more events in the future
681    assert.expect( 4 );

OK to remove it?

In TrailTests.js the assert.expect( 4 ) is checking that exactly 4 of the preceding 6 callbacks were called--it may be necessary (though I must admit I don't understand why we would only expect 4 of those 6 callbacks).

There are two callbacks that assert false, and note that they should never be called. We want to ensure that the other 4 callbacks are called once, and that the other two are never called. Is there a better way to do that?

Thanks for explaining, it makes sense to me now.  Let's run it past @pixelzoom

I added documentation similar to what @jonathanolson said in https://github.com/phetsims/scenery/issues/720.
But I'm still not clear on why the assert.expect call is necessary.  The 2 callbacks that should never be called are calling assert.ok( false,...), so wouldn't those failures be sufficient?


feat(routing): Add support for adding multiple routes for the same resource

Implementation of #584

Made requested changes -- thanks so much for the input @kgriffs, really appreciate it!

Alrighty, thanks again for your input @kgriffs and @vytas7 ! Much appreciated. Made the changes suggested, let me know if you catch anything else.

Hey everyone, how did it ended up being implemented? With the alt keyword or as another method?

are there plans to get this feature into a release?  The tutorial references issue #584, the tag  on this issue is 'merged', but it doesn't look like it's been merged into a deployed version.
thanks

@bkcsfi it has been merged onto the master branch of the repository.
Nonetheless, it won't be added before falcon 2.0 release, and won't be merged into the falcon 1.4 actually available through pip as it obviously changes the api.

thanks for the update, I'll follow the old-style shown in the docs for now..


Peek definition shows definition from .razor.g.cs instead of just .razor file

This issue has been moved from a ticket on Developer Community.

[regression] [worked-in:16.11.6]
Call Peek definition on type param T in Test.cs line 3 in attached solution
Then there are 2 definitions found, Test.cs and Test.razor.g.cs. (which is showing an error). VS2019 shows correct definitions  - Test.cs and Test.razor,

Original Comments
Feedback Bot on 11/14/2021, 06:32 PM:
We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.

Original Solutions
(no solutions)

I investigated a bit and the key difference here is in 16.X if you had the .razor file open then peek would show you the top-level Razor document. In VS2022 the tooling understands what's trying to be mapped (and it can't be) because the @typeparam directive doesn't generate real C# code. Instead of returning an invalid location in the Razor file we return the backing generated code. If the compiler generated real @typeparam code then we'd be able to map


Feat: Add bank validations

As requested on #171, this PR adds validation to the following banks:
Banco do Brasil, Santander, CEF, Bradesco, Itau, Real, HSBC, Citibank
The validations goes according to this provided documentation
http://<IP_ADDRESS>/ercompany.com.br/boleto/laravel-boleto-master/manuais/Regras Validacao Conta Corrente VI_EPS.pdf

Hello @mateusbw . We are very happy with your first contribution to our library! I gonna review your code as soon as I have time available.
Thanks!


New contact wizard: org name entered in card should perform Google Places search

When you enter a company name here and click the Lookup… button...

The company name should be transferred to the search box, the search should default to Google Places, and the search should be executed

Ready to review.


option to remove overlap duplication

there should be an option in the ocr import pipeline to detect and remove duplicates as in
https://github.com/OpenPecha/OCR-Pipelines/issues/8
This should be relatively (!) straightforward by looking at pixel coordinates within a line and either:

remove everything that has a big enough overlap (this is the best but more complex option)
remove everything that goes back horizontally

okay I will look into it


BasePlugin.GetUrl should replace query parameters, not append

It should only keep pathname of a url and replace any query parameter in current url.
In Detail pages, there is an Edit button if we use GetUrl helper. when we click on it, it will return wrong URL

@pournasserian when we want to navigate from Detail page to edit page.
Currently we don't have any detail page yet.

@pournasserian when we want to navigate from Detail page to edit page.
Currently we don't have any detail page yet.

Since we are redesigning the plugins, I am closing this issue. Don't forget to reopen it if we face it again.


CI failed and v0.5.1 was not published

From CI#217, CI failed.
v0.5.1 was not published.
I don't know if this is the cause, but on ubuntu-22.04 with image version 20240403.1.0 or later, the node v16 job is failing to find yarn.

From CI#217, CI failed.

this a lint action that was triggered by the pull request, it has nothing to do with releases

v0.5.1 was not published.

it was - https://www.npmjs.com/package/eslint-plugin-import-x/v/0.5.1

I don't know if this is the cause, but on ubuntu-22.04 with image version 20240403.1.0 or later, the node v16 job is failing to find yarn.

not related to this repo at all, see https://github.com/actions/setup-node/issues/1075

It seems that Node.js 16's corepack can't install yarn. I don't know why though, but I am going to work on it.

Node.js 16's corepack can't install yarn.

No, the latest Corepack can't fetch anything because it tries to use Node's built-in fetch, which was only added in Node v18. Why is the latest Corepack being used? Because corepack enable is run before setting up Node.

I am going to work on it.

The only solution: https://github.com/actions/setup-node/issues/1075#issuecomment-2165643488

@SunsetTechuila
Thanks for providing so many details and context!
Would you like to create a PR to help us fix the CI?

@SukkaW done

Thank you!


send_email not working in sct_utils.py

import sct_utils as sct

<EMAIL_ADDRESS><EMAIL_ADDRESS>
 "some-potential-password-like-thing",
 'smtp.gmail.com',
  587)

The email module appears to have been refactored, changing the location of some of the imports
ModuleNotFoundError: No module named 'email.MIMEMultipart'

i suggest we also move send_email() to utils.py


Helper photo in profile and helper icon

once we have #53 completed, we need to enhance the helper profile page to allow for upload of a photo.
Photos need to be adhering to a minimum pixel size in width and heigth (min 180x180 seems standard), limited in file size (1 MB?), and cropped to fit into a round circle icon.
An example of this was done in the mockup by @stephanerangaya

Helpers needs to be able to change and also remove a photo from their profile page.
If a photo is uploaded, it should be displayed in the helper icon everywhere instead of the initials.

Unclear as to how much work the whole cropping and centering of images is, hoping there are some standard libraries to help with that stuff (like this).

@kristysnet Adding this one into the mix. It would be really neat to have the helper photo finally in the platform. Maybe talk to  @phil-monroe to get a sense of how to do this best and what the effort would be. Seems like every platform has this so likely there are existing gems to handle all the imagery stuff....

Long time in the wish list - come out great! Thank you @phillipoertel !!!!

Hi Ralf,
sehr schön, hat bei mir gleich funktioniert. Jetzt wollte ich das Bild ändern, hat auch geklappt.
Aber es zeigt auf dem Profil, wenn man auf „Bearbeiten“ klickt immer  an „keine Datei ausgewählt“, auch nach dem neu eingefügten Bild.
Von: Ralf Schroeder<EMAIL_ADDRESS>Gesendet: Freitag, 13. Januar 2017 12:33
An: lale-help/lale-help
Cc: Subscribed
Betreff: Re: [lale-help/lale-help] Helper photo in profile and helper icon (#227)
Long time in the wish list - come out great! Thank you @phillipoertel https://github.com/phillipoertel  !!!!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/lale-help/lale-help/issues/227#issuecomment-272422986 , or mute the thread https://github.com/notifications/unsubscribe-auth/AQr-4UDefi0WIZKnx0Cqti7QC8M4BWrMks5rR2DegaJpZM4HxzuA .Das Bild wurde vom Absender entfernt.


theme not working properly

any fixes?

fluent.js is likely properly installed because the sidebar navigation icons have the accent divs (which are inserted by the JS extension)
Ensure that you are using the latest version of spicetify (run spicetify upgrade in console)

fluent.js is likely properly installed because the sidebar navigation icons have the accent divs (which are inserted by the JS extension)
Ensure that you are using the latest version of spicetify (run spicetify upgrade in console)

yes its the latest version

any fixes?

its working now


[sk] Fixed datetime detection bug

Summary
Fixed bug where certain datetime formats were not being detected, such as "DD/MM/YYYY HH:MM:SS". Fixed by updating the regex string to handle this case.
Tests
Unit tests were run to confirm correctness of code, alongside testing on the dataset associated with this bug to confirm the date is now parsed.
cc:
@wangxiaoyou1993

Did you add a new unit test for this format?

I did not modify the unit tests, I can quickly add a new test


Issue: How can I load text embeddings from a local model?

Issue you'd like to raise.
I do not have access to huggingface.co in my environment, but I do have the Instructor model (hkunlp/instructor-large) saved locally. How do I utilize the langchain function HuggingFaceInstructEmbeddings to point to a local model?
I tried the below code but received an error:
from langchain.embeddings import HuggingFaceInstructEmbeddings

model_name = "<local_filepath>/hkunlp/instructor-large"
model_kwargs = {'device': 'cpu'}
encode_kwargs = {'normalize_embeddings': True}
hf = HuggingFaceInstructEmbeddings(
    model_name=model_name,
    model_kwargs=model_kwargs,
    encode_kwargs=encode_kwargs
)

embeddings = HuggingFaceInstructEmbeddings(
    query_instruction="Represent the query for retrieval: "
)

Error:
HfHubHTTPError: 403 Client Error: Forbidden for url: https://huggingface.co/api/models/hkunlp/instructor-large
Suggestion:
No response

Based on #2667 (thanks @dosu-beta[bot] 🙏), it looks like in order for it to recognize that you're using a local model, you just need to give it the local path, extension and all.

I am facing similar issue. I am using LangChain 0.15. I can't upgrade it to latest version due to compatibility issues with other libraries.
I am using following code:
Load the locally saved SentenceTransformer model
modelPath = r"******\MiniLM"
model = SentenceTransformer(modelPath)
Set the device
device = torch.device("cuda" if torch.cuda.is_available() else "cpu")
model.to(device)
Initialize the embeddings
embeddings = HuggingFaceEmbeddings(model=model, model_kwargs={'device': device})


require lodash issue

After I generate sails-api, I get an error that lodash can not be found. Is this expected behavior? Seems redundant to have to npm install it, when it's exposed as a sails global anyway.
If this is expected, then installing lodash should be added to the setup steps in the README.md.

+1

+1

+1


Brace style rule 'stroustrup' and multiline mixin parameters

After upgrade to 1.6.0 (to be honest I don't want to upgrade, but gulp wrapper use version ^1.5.1 - so I'm doomed...) I got many new errors in brace-styles rule.
I'm using 'stroustrup' and now this type of code fails tests:
 @mixin _suuuper-long-mixin-name(
    $_oversized-paramams-names-was-moved-to-new-line,
    $_because-i-want-to-keep-lines-under-80-chars
) {
    [...]
}

IMO it still fit in that rule, but tests checks just if start and end is on same line 😞

Yeah I agree - appears to be a regression - will look at fixing asap.

Thank you so much!


Allow finding Boost components before finding HPX

Attempt to fix #4816. The GitHub actions fetchcontent build should test this now. Let's see what it says. I have not tested the full build locally.

@kordejong would you mind checking that this fixes the problem on your end as well?

@kordejong would you mind checking that this fixes the problem on your end as well?

Your fix seems fine. Building HPX continues passed the point where it failed before. Great!


Default language set to Deutsch/German

When I first navigate to the app https://dmponline-dev.dcc.ac.uk the default language setting is German. After I log in it remembers my profile setting. I assume the desired behavior is for the default to be English-GB on public pages.

This is back to English now as default. Gott sei Dank
Closing out


Read os env vars

Description of the change
Read OS environment variables
Benefits
Example to add  OS environment variables as secret variables and read via Gateway context variable syntax in Policy and in pre-selected configuration fields (e.g. JDBC Connection URL)
Drawbacks

Applicable issues

fixes #

Additional information
Checklist

[] Chart version bumped in Chart.yaml according to semver.
[] Variables are documented in the README.md
[] Title of the PR starts with chart name (e.g. [charts/gateway])
[] If the chart contains a values-production.yaml apart from values.yaml, ensure that you implement the changes in both files

The repo that you are trying to merge is 258 commits behind stable.
Do we need an additional template for this? AdditionalEnv/Secrets can already be configured here - https://github.com/CAAPIM/apim-charts/blob/stable/charts/gateway/values.yaml#L453-L461
For production environments we advise against exposing secrets in values.yaml - it's best to make use of existingSecrets where applicable

@Gazza7205 , oh may be you are right, let me merge the stable to this PR and add the env to https://github.com/CAAPIM/apim-charts/blob/stable/charts/gateway/values.yaml#L453-L461

@Gazza7205 , oh may be you are right, let me merge the stable to this PR and add the env to https://github.com/CAAPIM/apim-charts/blob/stable/charts/gateway/values.yaml#L453-L461

Do we need an additional template for this given that we already have mechanisms in place?

@Gazza7205 , oh may be you are right, let me merge the stable to this PR and add the env to https://github.com/CAAPIM/apim-charts/blob/stable/charts/gateway/values.yaml#L453-L461

Do we need an additional template for this given that we already have mechanisms in place?
Sounds like we just need to update the readme.

Updated ReadMe and revert other changes.

@Gazza7205 , oh may be you are right, let me merge the stable to this PR and add the env to https://github.com/CAAPIM/apim-charts/blob/stable/charts/gateway/values.yaml#L453-L461

Do we need an additional template for this given that we already have mechanisms in place?
Sounds like we just need to update the readme.

Updated ReadMe and revert other changes.

It seems this readme is also from an old branch - could you review your changes and make sure they are against the latest stable commit?


Could I use Thinky.js with reqlite instead of a connection option ?

Something like
let reqlite = require('reqlite'),
     thinky = require('thinky')(reqlite)

Yes and no.
If you start reqlite with ./bin/reqlite, reqlite will behave like a rethinkdb server, so in this case it works.
You cannot do thinky = require('thinky')(reqlite) because reqlite doesn't provide a (public) API to send queries directly to the server without running a tcp server first.

Alright, so - trying to understand...
Why would I even need reqlite instead of just running another instance of RethinkDB?
Just for in-memory usage?
I thought of using this for testing. I guess I could add reqlite as a dev dependency and have my test script run reqlite before running the tests. Should that work?

The point of reqlite at the moment is:

For test, it's easier to spin up reqlite than rethinkdb: For example reqlite runs on Windows, doesn't require rights to access the filesystem etc.
For front-end developers, it's a way to execute queries in the browsers and then sync things (like Meteor).

In the longer term, it would be cool for testing purposes to be able to do:
var reqlite = require('reqlite');
var server = new reqlite();
var connection = server.createConnection();

// optional
// connection.mockResponse(...) // mock the next response

query.run(connection).then(...)

This requires a little more work though.

I've now completely moved my tests to use reqlite and redis-js so it should be completely independent of the system.
I've wanted to share my REQLite workflow with anyone who might want to do the same. Until reqlite can run independently inside Node.js, this is the script I wrote as a pre-test hook in my package.json file. It checks whether or not Reqlite is installed, if not - installs it and then my test scripts are using 28016 instead of 28015 for tests. Works very smoothly so far :)
Here's how it looks:

'use strict'

let	exec 	= require('child_process').exec,
	spawn 	= require('child_process').spawn,
	colors 	= require('colors')

let child 	= exec('npm list -g reqlite >& /dev/null', (err, stdout, stderr) => {
	let startR = () => {
							console.log("> " + "Starting REQLite...".magenta)

							exec('pkill -f "reqlite"', (err, stdout, stderr) => {
								let reqlite = spawn('reqlite', ['--port-offset', '1'], {detached: true})

								reqlite.stderr.on('data', function (data) {
									console.log("> " + "REQLite is running...".green)
									process.exit(0)
								});

								reqlite.on('exit', function (code) {
								  console.log('child process exited with code ' + code);
								});
							})
						}

	if(err) {
		if (err.code == 1){
			// Should install
			console.log("> " + "Installing REQLite...".magenta)

			let install = exec('npm install -g reqlite', (err, stdout, stderr) => {
				console.log("> " + "Done installing REQLite".green)
				console.log(" ")
				startR()

			})
		}else{
			throw err
		}
	}else{
		console.log("> " + "REQLite is already installed".cyan)
		startR()
	}
})


Make app id unique across apps started by dapr run

Signed-off-by: Pravin Pushkar<EMAIL_ADDRESS>Description

Make appID unique across apps started by dapr run. Not uniqueness was introduced by mistake in rel-1.6. There is a diff issues opened for supporting this correctly - #698.
dapr run -f will exit completely if AppIDs are not unique in dapr.yaml

AppID uniqueness was unintentionally removed in 1.6, and there is no complete support for it yet. Will work on https://github.com/dapr/cli/issues/698 to allow appID duplication.

one suggested improvement that can be tried for 1.11 until #698 is implemented -

You are checking based on the list of processes. It is not guaranteed but it is better than we have now. This can also be achieved by a file lock, but not sure how to do that on Windows. It can be in 1.11 since it would avoid race conditions.

@yaron2 had some concerns with this PR breaking some of the users and also that this unintentional change has been there from 1.6.
@yaron2 thoughts on this?

@yaron2 had some concerns with this PR breaking some of the users and also that this unintentional change has been there from 1.6.
@yaron2 thoughts on this?

Technically, it's not forbidden to run two instances of Dapr with the same app Id - this is what happens naturally on K8s with multiple pods all having the same app Id. So unless there is something in the local dev experience that is degraded by having this, maybe it's best to leave it as is, because:

It has been allowed in the CLI for the last 3 releases
It allows developers to easily test things like two consumers in a consumer group with pub/sub, and multiple consumers of a distributed Lock
There may have been users who use the CLI to bootstrap non containerized envs with Dapr who rely on this, although unlikely.

I feel like we can go either way, but would like to revert this based on something concrete.

@yaron2

Technically, it's not forbidden to run two instances of Dapr with the same app Id - this is what happens naturally on K8s with multiple pods all having the same app Id. So unless there is something in the local dev experience that is degraded by having this, maybe it's best to leave it as is

If we are going to have this as is then we have to correct the dapr stop behaviour.
Currently dapr stop appID will stop any one of the many instances of apps having same appID.
dapr stop iterates through the apps list obtained via dapr list and exist after getting the 1st match for the provided app ID.
And similar behaviour happen for dapr run -f.

The behavior of the following CLI commands also will be affected right? dapr stop, dapr publish, dapr invoke etc.
Since all of them use list to list the apps and based on the appID select a particular app to perform the function of invoke, publish or to stop

@dapr/maintainers-cli Based on the discussions that we have had offline, this change will not be done.


does not work with standoff 2 version 0.30.3

the decrypted metadata does not differ from the usual one

Easy can be fixed:
change valid = actual - reference - last_actual_value_found + last_reference_value_found < 1_000_000 to valid = actual - reference - last_actual_value_found + last_reference_value_found < 1_000_000 and size > 0

If you read the error message, you can see that it says that value not in range because value is less that zero.
So fix is:
change valid = actual - reference - last_actual_value_found + last_reference_value_found < 1_000_000 to valid = actual - reference - last_actual_value_found + last_reference_value_found < 1_000_000 and size > 0 and that's it.

If you read the error message, you can see that it says that value not in range because value is less that zero.
So the fix is:
change valid = actual - reference - last_actual_value_found + last_reference_value_found < 1_000_000 to valid = actual - reference - last_actual_value_found + last_reference_value_found < 1_000_000 and size > 0

If you read the error message, you can see that it says that value not in range because value is less that zero. So the fix is: change valid = actual - reference - last_actual_value_found + last_reference_value_found < 1_000_000 to valid = actual - reference - last_actual_value_found + last_reference_value_found < 1_000_000 and size > 0

the script does not produce any errors, it simply does not decrypt the metadata, leaving it as the original one

solved


unable to pull more than 1 address in the URL component

Bug Description
Hello, i'm trying to import documentation for projects i'm working on into context, from their web pages.
The Url component doesn't seem to work with more than one address, despite the description of accepting more addresses separated by commas...
-I've tried feeding it a properly formed python list.
-I've tried having a list of comma separated strings, encased within '' or "" and with neither of the two.
-I've tried tinkering with the code { str(self.url).replace('[','').replace(']','').split(,) as i'm feeding a list of addresses that i have already vetted with beatyfulsoup; removing the sqaure brackets was a "just in case" move}
None of the above attempts had any effect.
Somehow with more than one address, i just can't make it pass the "ensure_url" stage, hence the component doesn't build correctly.
Reproduction
Example: i'd like to chat with langflow's documentation stored in chromadb.
I've prepared the list of addresses for testing purposes, to spare -whomever can help me- some time.
(reformatting it to suit your needs is a matter of 1-2 lines of code):
https://docs.langflow.org,https://docs.langflow.org/whats-new-a-new-chapter-langflow,https://docs.langflow.org/getting-started-installation,https://docs.langflow.org/getting-started-quickstart,https://docs.langflow.org/getting-started-common-installation-issues,https://docs.langflow.org/starter-projects-basic-prompting,https://docs.langflow.org/starter-projects-blog-writer,https://docs.langflow.org/starter-projects-document-qa,https://docs.langflow.org/starter-projects-memory-chatbot,https://docs.langflow.org/starter-projects-vector-store-rag,https://docs.langflow.org/guides-new-to-llms,https://docs.langflow.org/guides-chat-memory,https://docs.langflow.org/guides-data-message,https://docs.langflow.org/workspace,https://docs.langflow.org/workspace-playground,https://docs.langflow.org/workspace-api,https://docs.langflow.org/workspace-logs,https://docs.langflow.org/components,https://docs.langflow.org/components-io,https://docs.langflow.org/components-prompts,https://docs.langflow.org/components-data,https://docs.langflow.org/components-helpers,https://docs.langflow.org/components-models,https://docs.langflow.org/components-embedding-models,https://docs.langflow.org/components-vector-stores,https://docs.langflow.org/components-custom-components,https://docs.langflow.org/components-rag,https://docs.langflow.org/deployment-hugging-face-spaces,https://docs.langflow.org/deployment-kubernetes,https://docs.langflow.org/deployment-docker,https://docs.langflow.org/deployment-gcp,https://docs.langflow.org/deployment-render,https://docs.langflow.org/deployment-railway,https://docs.langflow.org/settings-global-variables,https://docs.langflow.org/settings-project-general-settings,https://docs.langflow.org/configuration-authentication,https://docs.langflow.org/configuration-api-keys,https://docs.langflow.org/configuration-cli,https://docs.langflow.org/365085a8-a90a-43f9-a779-f8769ec7eca1,https://docs.langflow.org/configuration-backend-only,https://docs.langflow.org/integrations-langsmith,https://docs.langflow.org/integrations-langwatch,https://docs.langflow.org/contributing-telemetry,https://docs.langflow.org/contributing-how-to-contribute,https://docs.langflow.org/contributing-github-issues,https://docs.langflow.org/contributing-community
-i've tried pasting it directly into the text input of the url node.
-i've tried creating a csv, parsing it to text (with "  , with ', just pure addresses, also as python list) and then going through the parse data component
Once the data reaches the URL node, unless there is only a single address in the list it seems not to split correctly the addresses and then it doesn't pass the ensure_url() stage , hence the component doesn't build
Expected behavior
Being able to pass multiple urls
Who can help?
No response
Operating System
ubuntu 22.04 on wsl
Langflow Version
1.0.14
Python Version
3.12
Screenshot
No response
Flow File
No response

i had already tested that, however i've re-tested it:
Error Building Component
Error building Component URL: Invalid URL: http://["https://docs.langflow.org","https://docs.langflow.org/whats-new-a-new-chapter-langflow","https://docs.langflow.org/getting-started-installation","https://docs.langflow.org/getting-started-quickstart","https://docs.langflow.org/getting-started-common-installation-issues","https://docs.langflow.org/starter-projects-basic-prompting","https://docs.langflow.org/starter-projects-blog-writer","https://docs.langflow.org/starter-projects-document-qa","https://docs.langflow.org/starter-projects-memory-chatbot","https://docs.langflow.org/starter-projects-vector-store-rag","https://docs.langflow.org/guides-new-to-llms","https://docs.langflow.org/guides-chat-memory","https://docs.langflow.org/guides-data-message","https://docs.langflow.org/workspace","https://docs.langflow.org/workspace-playground","https://docs.langflow.org/workspace-api","https://docs.langflow.org/workspace-logs","https://docs.langflow.org/components","https://docs.langflow.org/components-io","https://docs.langflow.org/components-prompts","https://docs.langflow.org/components-data","https://docs.langflow.org/components-helpers","https://docs.langflow.org/components-models","https://docs.langflow.org/components-embedding-models","https://docs.langflow.org/components-vector-stores","https://docs.langflow.org/components-custom-components","https://docs.langflow.org/components-rag","https://docs.langflow.org/deployment-hugging-face-spaces","https://docs.langflow.org/deployment-kubernetes","https://docs.langflow.org/deployment-docker","https://docs.langflow.org/deployment-gcp","https://docs.langflow.org/deployment-render","https://docs.langflow.org/deployment-railway","https://docs.langflow.org/settings-global-variables","https://docs.langflow.org/settings-project-general-settings","https://docs.langflow.org/configuration-authentication","https://docs.langflow.org/configuration-api-keys","https://docs.langflow.org/configuration-cli","https://docs.langflow.org/365085a8-a90a-43f9-a779-f8769ec7eca1","https://docs.langflow.org/configuration-backend-only","https://docs.langflow.org/integrations-langsmith","https://docs.langflow.org/integrations-langwatch","https://docs.langflow.org/contributing-telemetry","https://docs.langflow.org/contributing-how-to-contribute","https://docs.langflow.org/contributing-github-issues","https://docs.langflow.org/contributing-community","https://github.com/langflow-ai/langflow/blob/main/LICENSE","https://github.com/langflow-ai/langflow/stargazers","https://github.com/langflow-ai/langflow/forks","https://github.com/langflow-ai/langflow/branches","https://github.com/langflow-ai/langflow/tags","https://github.com/langflow-ai/langflow/activity","https://github.com/langflow-ai/langflow/issues","https://github.com/langflow-ai/langflow/pulls","https://github.com/langflow-ai/langflow/discussions","https://github.com/langflow-ai/langflow/actions","https://github.com/langflow-ai/langflow/projects","https://github.com/langflow-ai/langflow/security","https://github.com/langflow-ai/langflow/pulse","https://github.com/langflow-ai/langflow/commits/main/","https://github.com/langflow-ai/langflow/tree/main/.devcontainer","https://github.com/langflow-ai/langflow/tree/main/.github","https://github.com/langflow-ai/langflow/tree/main/.vscode","https://github.com/langflow-ai/langflow/tree/main/deploy","https://github.com/langflow-ai/langflow/tree/main/docker","https://github.com/langflow-ai/langflow/tree/main/docker_example","https://github.com/langflow-ai/langflow/tree/main/docs","https://github.com/langflow-ai/langflow/tree/main/scripts","https://github.com/langflow-ai/langflow/tree/main/src","https://github.com/langflow-ai/langflow/blob/main/.env.example","https://github.com/langflow-ai/langflow/blob/main/.eslintrc.json","https://github.com/langflow-ai/langflow/blob/main/.gitattributes","https://github.com/langflow-ai/langflow/blob/main/.gitignore","https://github.com/langflow-ai/langflow/blob/main/.pre-commit-config.yaml","https://github.com/langflow-ai/langflow/blob/main/CODE_OF_CONDUCT.md","https://github.com/langflow-ai/langflow/blob/main/CONTRIBUTING.md","https://github.com/langflow-ai/langflow/blob/main/Makefile","https://github.com/langflow-ai/langflow/blob/main/README.KR.md","https://github.com/langflow-ai/langflow/blob/main/README.PT.md","https://github.com/langflow-ai/langflow/blob/main/README.ja.md","https://github.com/langflow-ai/langflow/blob/main/README.md","https://github.com/langflow-ai/langflow/blob/main/README.zh_CN.md","https://github.com/langflow-ai/langflow/blob/main/eslint.config.js","https://github.com/langflow-ai/langflow/blob/main/poetry.lock","https://github.com/langflow-ai/langflow/blob/main/pyproject.toml","https://github.com/langflow-ai/langflow/blob/main/render.yaml","https://github.com/langflow-ai/langflow/blob/main/docs/static/img/langflow_basic_howto.gif","https://github.com/langflow-ai/langflow/blob/main/docs/docs/Deployment/deployment-gcp.md","https://github.com/langflow-ai/langflow/blob/main/docs/docs/Deployment/deployment-kubernetes.md","https://github.com/langflow-ai/langflow/custom-properties","https://github.com/langflow-ai/langflow/watchers","https://github.com/langflow-ai/langflow/releases","https://github.com/langflow-ai/langflow/releases/tag/v1.0.14","https://github.com/langflow-ai/langflow/network/dependents","https://github.com/langflow-ai/langflow/graphs/contributors"]

Invalid URL: http://[  <<<  this suggest that the list doesn't get split at all

that is exactly the format of input i'm using.
in order to make a step forward i had edited the component like this:
def fetch_content(self) -> list[Data]:
        adresses=str(self.urls).replace('[','').replace(']','').replace('"','').split(',')
        urls = [self.ensure_url(url.lstrip().strip()) for url in adresses]
        loader = WebBaseLoader(web_paths=urls, encoding="utf-8")
        docs = loader.load()
        data = [Data(text=doc.page_content, **doc.metadata) for doc in docs]
        self.status = data
        return data

the result is an error on the first address:
Error building Component URL: Invalid URL: http://' https://docs.langflow.org
This way it splits the addresses correctly but a leading space gets inserted.
Also:  all of this seems so suggest that a plain comma separated list of addresses as input and not a proper python list, would be the intended use
however it still adds that space and then regex doesn't find the https at the start but a space.. et voilà

ok....
i've made it.
this is the modification needed to the component:
import re

from langchain_community.document_loaders.web_base import WebBaseLoader

from langflow.custom import Component
from langflow.io import MessageTextInput, Output
from langflow.schema import Data

class URLComponent(Component):
    display_name = "URL"
    description = "Fetch content from one or more URLs."
    icon = "layout-template"
    name = "URL"

    inputs = [
        MessageTextInput(
            name="urls",
            display_name="URLs",
            info="Enter one or more URLs, separated by commas.",
            is_list=True,
        ),
    ]

    outputs = [
        Output(display_name="Data", name="data", method="fetch_content"),
    ]

    def ensure_url(self, string: str) -> str:
        firstH=string.find('h')
        firstW=string.find('w')
        index=min(firstH,firstW)
        if index!=-1:
            trimmed = string[index:]
        # if not string.startswith(("http://", "https://")):
        #     string = "http://" + string

        # Basic URL validation regex
        url_regex = re.compile(
            r"^(https?:\/\/)?"  # optional protocol
            r"(www\.)?"  # optional www
            r"([a-zA-Z0-9.-]+)"  # domain
            r"(\.[a-zA-Z]{2,})?"  # top-level domain
            r"(:\d+)?"  # optional port
            r"(\/[^\s]*)?$",  # optional path
            re.IGNORECASE,
        )

        if not url_regex.match(trimmed):
            raise ValueError(f"Invalid URL: {string}")

        return string

    def fetch_content(self) -> list[Data]:
        adresses=str(self.urls).replace('[','').replace(']','').replace('"','').replace("'","").split(',')
        urls = [self.ensure_url(url) for url in adresses]
        loader = WebBaseLoader(web_paths=urls, encoding="utf-8")
        docs = loader.load()
        data = [Data(text=doc.page_content, **doc.metadata) for doc in docs]
        self.status = data
        return data

ofc it can be cleaned by a more advanced coder.
by inputting the addresses separated by commas as in my first post, it works.
adding http or https in case it was missing was for some reason breaking it (adding it also when it wasn't needed)


how get  variables defined in the scss

js file：
import styles from './var.scss';

scss file：
$themeColor: #49bc85;

styles there is no enumerated attribute

This loader has nothing to do with Sass variables. Please ask your question on StackOverflow if you're still having this issue, but be sure to provide more information than this.


Bug: it returns 500 if package not found

From the log
HTTPError: 404 Client Error: NOT FOUND for url: http://localhost:5000/api/v1/packages/devoxx/mysql/pull

but the client receive a 500

fixed with CNR api


recycle bin not working

Description
recycle bin not working
Steps To Reproduce
recycle bin not working
Requirements
recycle bin not working
Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.3527
Log File

Thanks for the report, Please try reinstalling Files.
If that doesn't work, can you answer the following

What isn't working about it, other than the text on the sidebar
Can you please upload (Do not copy and paste content) the debug.log file directly to GitHub (email attachments don't work) . You can get it in Files settings > About > Open log folder.

when i try to pin recycle bin or network to explorer, it gives error. i upload another screenshot.
i add debug.log file too.
debug.log

Did you try reinstalling Files?

yes. it solved problem. thanks.

Thanks for letting us know!


Audio cutting out (0.0.9)

I'm getting daily reports from users of my app (getlucid.com) that the audio cuts out intermittently and may or may not work after an app restart.
Has anyone experienced anything similar? Is this library being maintained or is there a different/better RN audio streaming solution out there?
Thanks!

Hey @stirman I have been using react-native-audio-streaming but it does not have events, so you do not when audio finishes playing


⚠ gorzdrav.org has degraded performance

In ff7fb45, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 6109 ms

Resolved: gorzdrav.org performance has improved in 1ecaec2.


throw new BridgeException(Attempt to access '${stack.join('.')}' failed.)

throw new BridgeException(Attempt to access '${stack.join('.')}' failed.)

For further investigation could you provide your code where you use the module and your device's OS?

No response, unknown issue


[Update]final version non-ddp MaskDiT

this version may still encounter recursive errors if the RAM is not large enough, only a GPU with more than 24GB vram would secure no risk with running with the current version.

pass, more work need to be done upon this weird model.


WIP: notifications: Disable default permission pop up.

The issue here was that if the user has not allowed desktop
notifications, the browser would give its own pop up requesting for
permission, this would happen concurrently with zulip's custom green bar
that requests for permission, we would prefer that users interact with
our green bar and so this commit disables the pop up.
THIS IS STILL WIP.
I'd like to upload a gif to make sure this is working exactly as we want it to.

This fixes #11504 by removing the cause of the problem.
Testing Plan:
Open firefox, navigate to local env, log in as user 1.
Open chrome, navigate to local env, log in as user 2.
click "i" icon and reset notification permissions to default (ask), allow a reload, confirm that green bar is present. [on chrome]
send message to user 2 from user 1. [from firefox to chrome]

prior to this, you would see a browser pop up
after this, that pop up no longer triggers.

repeat in opposite order and confirm everything to be same.
(will upload a gif of this)
node tests:
I've just restored our previous test cases, but we could possibly increase coverage here by writing more tests, but maybe that would be a good follow up?

Chrome->Firefox is failing to notify. I guess there's something in the webkit paths that needs to be cleared up. Looking into that now.

This is great, merged as 93696729c6b51e1c86c95ad39ceac8ac3cde8a05, thanks @YashRE42!  I'd absolutely love to have better test coverage for this module; it's a very old module written long before we had the node test suite, so test coverage isn't great.
One note: I used an edited version of the text and GIT_AUTHOR_* data from @snickell's original PR description, because I thought the description was cleaner and in any case that seems like the most correct attribution.
Thanks for getting this finished @YashRE42!


label-important has been replaced by label-danger

In the upgrade from bootstrap 2 to 3, the class names for label-important
changed to label-danger. Looks like this one was missed, so this is the fix
Before:

After

:+1:


Update list installed versions

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

Project coverage is 99.66%. Comparing base (edf3e9d) to head (0a43d63).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #173      +/-   ##
==========================================
- Coverage   99.66%   99.66%   -0.01%
==========================================
  Files           8        8
  Lines         600      599       -1
==========================================
- Hits          598      597       -1
  Misses          2        2

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


dolibarr: php version too high

Describe the bug
Trying to activate the dolibarr service, in the installation phase, dolibarr complains that the PHP version (8.2.13) is too high, requests version 8.1.0 or lower...
I guess we could add a phpPackage option to change the version of php used ?
Steps To Reproduce
on nixos 23.11 for a minimal installation add
services.dolibarr = {
  enable = true;
  nginx = {
    rejectSSL = true;
    forceSSL = false;
    enableACME = false;
  };
};

to your config, go to http://localhost/install/check.php and see the error message about the php version.
Notify maintainers
@RaitoBezarius
Metadata
[user@system:~]# nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.64, NixOS, 23.11 (Tapir), 23.11.1209.933d7dc15509`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - channels(smyds): `""`
 - channels(root): `"nixos-23.11"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I'm not maintainer anymore of Dolibarr (this was not updated on NixOS stable yet).

@smyds are you able to make a PR with the fix?


Autocad Color Indexes

I've been searching around the internet again for a definitive color index. Once again, there are lots of lists, which don't match.
However, I ran into a discussion on the Autocad support blogs. And there was a very nice gentlemen that created color indexes for Autocad 2018 and Autocad 2017. I attached the spread sheet for your use.
colorcomp.xlsx
I suspect that the color index changes whenever Autocad wants. Which is why DXF is such a huge mess.
If you have someone that has Autocad installed, they could verify the color index fairly quickly.

Thanks, like I said, I use http://gohtx.com/acadcolors.php, and it's close enough to the spreadsheet you sent.
I agree DXF is a huge mess.


[Bug]: cloudformation_stacks_termination_protection_enabled is not performed / sent to security hub

Steps to Reproduce

deploy cloudformation template into aws account
run ./prowler aws --security-hub -c cloudformation
check security hub

Expected behavior
cloudformation stacks with termination protection deactivated should appear in the list of security hub findings as 'failed'
Actual Result with Screenshots or Logs
no findings make it to the security hub
How did you install Prowler?
From pip package (pip install prowler)
Environment Resource
EC2 instance
OS used
Amazon Linux 2023
Prowler version
Prowler 3.15.0
Pip version
pip 21.3.1
Context
when running: ./prowler aws --security-hub
progress bar indicates that 304/304 scans were performed and 35 findings made it into security hub but not cloudformation ones. i specifically deployed stacks with termination protection deactivated to test reporting by Prowler and remediation within security hub

role assigned to Prowler instance didn't have permissions to scan cloudformation


[bug] SystemTray shows the menu on LeftClick, it should be on RightClick only

Describe the bug
default SystemTrayEvent::LeftClick will show Tray Menu, I not found API or setting to  disable the behavior.
https://tauri.studio/docs/guides/system-tray
use tauri::{CustomMenuItem, SystemTray, SystemTrayMenu};
use tauri::Manager;

fn main() {
  let tray_menu = SystemTrayMenu::new(); // insert the menu items here
  tauri::Builder::default()
    .system_tray(SystemTray::new().with_menu(tray_menu))
    .on_system_tray_event(|app, event| match event {
      SystemTrayEvent::LeftClick {
        position: _,
        size: _,
        ..
      } => {
        // the will auto pop Tray Menu
        println!("system tray received a left click");
      }
      SystemTrayEvent::RightClick {
        position: _,
        size: _,
        ..
      } => {
        println!("system tray received a right click");
      }
      SystemTrayEvent::DoubleClick {
        position: _,
        size: _,
        ..
      } => {
        println!("system tray received a double click");
      }
      SystemTrayEvent::MenuItemClick { id, .. } => {
        match id.as_str() {
          "quit" => {
            std::process::exit(0);
          }
          "hide" => {
            let window = app.get_window("main").unwrap();
            window.hide().unwrap();
          }
          _ => {}
        }
      }
      _ => {}
    })
    .run(tauri::generate_context!())
    .expect("error while running tauri application");
}

Reproduction
No response
Expected behavior
No response
Platform and versions
Operating System - Mac OS, version 12.2.1 X64

Node.js environment
  Node.js - 16.13.2
  @tauri-apps/cli - 1.0.0-rc.2
  @tauri-apps/api - 1.0.0-rc.0

Global packages
  npm - 8.1.2
  pnpm - 6.30.0
  yarn - 1.22.17

Rust environment
  rustc - 1.58.1
  cargo - 1.58.0

Rust environment
  rustup - 1.24.3
  rustc - 1.58.1
  cargo - 1.58.0
  toolchain - stable-x86_64-apple-darwin

App directory structure
/dist
/node_modules
/src-tauri
/src

App
  tauri.rs - 1.0.0-rc.0
  build-type - bundle
  CSP - default-src 'self'
  distDir - ../dist
  devPath - http://localhost:3000/
  framework - React

Stack trace
No response
Additional context
No response

Ideally we want right click to show the menu without the left click.

Great!

All applications I used will trigger the try both right and left clicks.
Telegram, Wireguard, and even system IME do this behavior.

It is good to follow the OS convention, but I remember @lemarier showed a demo of 1password app where LeftClick opens a window instead of a menu.

Alright, I will close this since it is the OS default behavior but If more people ask for this, we will reconsider.


add tag navigation (#5)

I thought it'd be a hard thing to do knowing well I haven't tried Astro :laughing:
Also, I think it would be good to change the selection border on post cards because it's barely visible, nice website btw keep it up!
Closes #5 once and for all.

thank you!


Intern DataSegments in SQLMetadataSegmentManager

Helps with heap pressure on coordinator

Related to https://github.com/druid-io/druid/pull/3238

👍

👍


Export csv feature

Related Issue
Closes #24
Type of Change
Put x inside the square bracket to specify what type of change your PR is:

[x] New Feature

Description of Change
Now user can export the tasks data to a csv file at a directory location of their choice.
Demo

@pranavkonidena Merge conflicts resolved, please review.


Fixup function_pointers2

This fixes the merge conflicts I introduced (mentioned at https://github.com/solson/miri/pull/21#issuecomment-225388541).
Merging this should automatically update your PR against my repo.

cool, thanks!


issue 12: fixed tooltip flicker; prevent tooltip run off screen

The heart of this change was implementing custom OverlayLayout to size and position tooltips without waiting an extra frame (if I understood the root cause correctly).  I haven't witnessed any flicker in the test app, nor in my app, since this change.
I added more tooltips to the demo app.  I can remove them if needed.
OverlayLayout: I added a parameter gapTooltipScreen to control how close tooltips can approach left and right side of screen, but I didn't expose this in calling composables.  I can do so if you want.
It's still possible to specify tooltips so that they run off the top or bottom of the screen.  This is a harder than preventing left and right "overrun"; I mentioned it briefly in the layout code.

Hey thanks for raising the pr, sure will check this

Hey, I am still seeing the issue
https://github.com/pseudoankit/coachmark/assets/54987308/03b55c2d-49b3-4f8f-b3e9-25af0b35362b

I added a parameter gapTooltipScreen to control how close tooltips can approach left and right side of screen, but I didn't expose this in calling composables
please expose it let's give client flexibility

Somehow I can't reply here on the comment you made re: the enum.  I replied on it in the code listing.

Thanks for the pr 🙏 , merged


Alert experimental

To address the other half of #192 , the sister PR of #211
Set up an alert on uses of STM_thread.Make and explicitly disable it on uses in src/neg_tests.

I just pushed commit 4aff086 to elaborate a bit on the warning description

Dang, the same thing happened again :grimacing:
There were merge conflicts with this PR because 15d850c and 04df79b were also adding -experimental warnings on Lin_thread.Make_internal and Lin_effect.Make_internal covered by #211. I therefore locally made commits to undo these changes, squashed these into the original commit, and merged. Oh well.


Replacement for RDFAlchemy?

The RDFAlchemy project's repo is archived and inactive - https://github.com/gjhiggins/RDFAlchemy and it doesn't look like there is a clear replacement. Perhaps https://github.com/cosminbasca/surfrdf or https://pypi.org/project/Owlready2/ ?

Hi Joe,
I used RDFLib recently.
Please take a look: https://github.com/RDFLib/rdflib

Great thank you! Are you planning on opening a PR that replaces RDFAlchemy with RDFLib?

Hi I haven't worked on OLGA to replace it yet.

@joephayes is this still of any interest?

@charbull - Yes it is!

@joephayes I am trying to do some refactoring to make it easier to add this.
Take a look at this PR: #22

Hi @charbull - I went through and was able to build the Python library.  When I go to import specific things, I am thrown the following:
>>> from digitalbuildings.DigitalBuildings.Rdf.Model import EntityType
>>> from digitalbuildings.DigitalBuildings.Rdf.Model import Equipment
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python2.7/site-packages/digitalbuildings/DigitalBuildings/Rdf/Model/Equipment.py", line 5, in <module>
    from digitalbuildings.www.google.com.digitalbuildings._0_0_1.EntityType import EntityType
ImportError: No module named www.google.com.digitalbuildings._0_0_1.EntityType

I'm not positive if this is a related issue or if will get taken care of when upgraded to RDFLib.  If so, seems like that is making progress and can wait until then - thanks!

Hi @corymosiman12, please add -preserve option for the code generation since there are similar class names but different IRI there is a clash.
Thanks

Had the same error here: https://github.com/EcoStruxure/OLGA/issues/17
Will find some time to fix that.

ah ok - will do, thanks!

@corymosiman12 I found a bug and for now fixed manually the code generation for Java.
I put a detailed steps here and the generated source code:https://github.com/google/digitalbuildings/tree/master/ontology/rdf/example

Thanks!


add cert rotate upgrade prevention

We named this "unsupported-cert-rotation-config".  This adds logic to give that some teeth.
/assign @enj

@enj now with working e2e.  just gofmt remaining.

LGTM minus formatting.  Feel free to tag when ready.


As an open source community member, I want to know what Justice40-specific terms and acronyms mean.

Description
We had a glossary in our open source kickoff and someone suggested putting it in our Github repo so that it lives on and is discoverable. This PR adds the initial version of the glossary including ways to update or submit new terms to the glossary.

Oh, good points! I will add EVCM to this too.


⚠ Carbubu has degraded performance

In e019cae, Carbubu (https://carbubu.fr) experienced degraded performance:

HTTP code: 200
Response time: 10184 ms

Resolved: Carbubu performance has improved in 31df241.


Swap links for buttons

Clicking on links scrolls to the top of the page. Besides that, buttons is more semantic for this component.

Fixed by: https://github.com/metal/metal-tabs/pull/8


re #889 Update Naming for Blog Posts

@EricWittmann - This PR is not ready to be merged in yet.
Changes:

Added a block of HTML with an 'if else' statement to the post.html layout to conditionally check whether or not a newUrl or oldUrl exists for the Jekyll post you are currently viewing, and display a different section of text depending on which exists, if any. cc @msavy - This is mostly to reuse that same HTML and keep things dry instead of having to copy and paste it for each post. It should also make it a little easier to swap out the dates from the URLs if we do end up doing that. Let me know what you think.
Created redux files for the 6 most recent posts, but only 2 of those have had the naming updated (the most recent one was also skipped since it didn't apply in this case). It should take me about 30-60 minutes to do the remaining posts and test them once I get back online. I actually had 4 done on Friday but lost some of my work and it was just easier to re-do it.
Updated gitignore to ignore .idea files generated by WebStorm.

To Do:

Finish renaming 3/6 posts. Can open a separate PR for other posts outside of these 6 to keep it easy to review.
Update images for some blog posts to reflect new naming. This is a quick thing, should not take long at all, and can help avoid confusion.

Please resubmit against the 1.2.x branch.


Description of Lambda function is hard coded

This ought to be a setting you can set locally

Done.


Add run compiler command

See #466.
This is fairly simplistic and doesn't implement all the ideas from the proposal but I think it'll cover the vast majority of use cases right now. Summary of the features below.
A little messy, I have some visions for how I'd like to see the compiler cli but that's another issue. Also untested on Windows but provided I implemented the os details without compiler errors they should do the job.
I've skimped on tests for the moment.
Here is a simple test program that can be used to verify arguments (modify the shebang location to suit).
#!/home/me/local/bin/zig run

const std = @import("std");
const os = std.os;

const allocator = std.debug.global_allocator;

pub fn main() void {
    var args_it = os.args();

    while (args_it.next(allocator)) |arg_or_err| {
        std.debug.warn("{}\n", arg_or_err);
    }
}

zig run file.zig builds a file and stores the artifacts in the global
cache. On successful compilation the binary is executed.
zig run file.zig -- a b c does the same, but passes the arguments a,
b and c as runtime arguments to the program. Everything after an -- are
treated as runtime arguments.
If the file was not modified or renamed and the compile arguments have not
changed (order cannot change either) then a cached version may be used
instead of recompiling. Runtime arguments do not trigger a rebuild.
On a posix system, a shebang can be used to run a zig file directly. An
example shebang would be #!/usr/bin/zig run. You may not be able pass
extra compile arguments currently as part of the shebang. Linux for example
treats all arguments after the first as a single argument which will result
in an 'invalid command'.
Currently there is no customisability for the cache path as a compile
argument. For a posix system you can use TMPDIR=. zig run file.zig to
override, in this case using the current directory for the run cache.
Closes #466.

There would be an issue if you wanted to pass flags to the program. Right now we allow flags in any position, even after a positional argument. We would need to disallow flags after positionals for this case so we could treat all trailing arguments, positional or flags, as runtime arguments.
I think being able to add flags after a command is still too useful. For example, if you have forgotten one or just want to a previous command from your history with an added flag.
Unless, you mean when omitting the run command entirely all arguments are treated as runtime arguments, with the first being the input file?

Unless, you mean when omitting the run command entirely all arguments are treated as runtime arguments, with the first being the input file?

Yep that's what I meant. The only problems I can foresee are things like, wanting to pass include directories or link libraries.
But maybe for zig run we shouldn't support that, since it's really meant to be a convenience thing. If a user needs more complicated compiler invocation commands, they can use the normal compilation process or zig build.
I do think that it would make sense to be able to depend on external zig packages though for a run script. We'll have to revisit that idea once we have a package manager.

Normally I'm in favor of incremental improvements - but there's one thing where I think we should have a 100% correctly working implementation, or not at all - and that is for caching. I've already noticed that people don't trust zig-cache/ and delete it before building, even though we don't do any caching yet! Years of programming experience has taught us to never trust caches.
Let's enumerate an exhaustive list of things that need to go into the cache hash. I'll check the ones that it looks like you handled in this PR:

[x] whether the root source file changed (in name or contents)
[x] the args to the compiler
[ ] whether any of the files that are depended on via @import changed
[ ] whether any of the files that are depended on via @embedFile changed
[ ] whether zig binary changed
[ ] whether any of the dynamic libraries that zig binary depends on changed, and their dynamic library dependencies, recursively
[ ] whether any of the above things changed, but for the builtin.o target
[ ] whether any of the above things changed, but for the compiler_rt.o target
[ ] whether any of the --object or --library files changed, and recursively for dynamic libraries.
[ ] other command line args that reference files, such as linker scripts

That's all I can think of. Can you think of any other ways to fool the cache?
One other reason to use the less powerful zig foo.zig syntax is that not supporting zig compiler arguments would be an easier caching problem.
It's a bit of work to have a foolproof cache. It's why I've been putting off this issue and the other caching issues. But I really think we should not merge a cache that is not perfect into master.

Fair points. That is a good list and highlights the many different things we need to consider. Since the scope of this is much larger, how about I remove the caching for this current PR and we think about it further and implement a robust version with caching in the self-hosted version?
I'll still use a global cache for run, but it'll be removed/replaced on every invocation as is. Sound good? I can make a new issue and put these points in and we can get some discussion going there.


[Enhancement]: Support for Custom Keyboard Buttons

I'm trying to a build a keyboard that will make it as easy as possible to enter a web address. The Google Chrome app for iOS uses a customized keyboard to provide a lot of the special characters that are required in a web address on a secondary row at the top of the keyboard so that the user doesn't need to dig through the keyboard layers for these required characters.

Such customization would be amazing if it could be brought into React Native! Any ideas on how we can achieve this?

You could build a keyboard module and publish it to npm so others can use it.
This issue looks like a question that would be best asked on StackOverflow.
StackOverflow is amazing for Q&A: it has a reputation system, voting, the ability to mark a question as answered. Because of the reputation system it is likely the community will see and answer your question there. This also helps us use the GitHub bug tracker for bugs only.
Will close this as this is really a question that should be asked on StackOverflow.


PacketLib should expose the functionality to fetch the remote user's IP address

I'm currently working on a project with PacketLib (and more specifically, MCProtocolLib).
While working on IP rate-limiting functionality, I noticed that while the host name is available, server names are not. This is bad because it means you can get around the rate limit by merely changing your reverse DNS.
For now, I use this reflection-based work around.
public class SessionUtil {
    private static final Field CHANNEL_FIELD;
    static {
        try {
            CHANNEL_FIELD = TcpSession.class.getDeclaredField("channel");
            CHANNEL_FIELD.setAccessible(true);
        } catch (NoSuchFieldException e) {
            throw new AssertionError(e);
        }
    }

    public static InetAddress attemptPryingRemoteAddress(Session session) throws IllegalAccessException {
        if (session instanceof TcpSession) {
            Channel channel = (Channel) CHANNEL_FIELD.get(session);
            if (channel != null) {
                return ((InetSocketAddress) channel.remoteAddress()).getAddress();
            } else {
                return null;
            }
        }

        return null;
    }
}

Added.


depth byte binary text and image from Realsense D415, how do i change it to a mesh/3d-model for display in website?

Required Info

Camera Model
{  D415 }

Language
{C/C#/labview/nodejs/opencv/pcl/python/unity }

Segment
Smartphone/Web

Issue Description
Hi i have the depth byte binary text file and picture from the D415 realsense camera,
How do i use this to create a mesh or 3d-model, it doesnt have to be 360 degrees, i just have to make it 3D from the front for example? Can i do this with these output files?

Hi @elyazinho   Could you confirm which file format you want to convert please (ply, raw, bin, csv, etc).  Thanks!

@MartyG-RealSense, is gITF possible? if not we could work with ply as well. The files from the realsense camera are depth byte binary text file and image , do you have a step-by step instruction on how to convert to such formats? we can do it with python or Node.js
Do you need the sample binary text file and it's accompanying image?

Hi @elyazinho    I am not aware of past cases of librealsense data being used with gITF unfortunately.
If at all possible, I would strongly recommend using ply files as the basis of your conversion to mesh or 3D model, as such a conversion is well supported by librealsense and by 3D modelling software..
The librealsense SDK offers the ability to configure ply-file export to generate the ply with on / off settings such as normals, color and binary   A Python example of this export mechanism is in the link below.
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/python/examples/export_ply_example.py#L40
There are other Python ply export refererences, such as this one:
https://github.com/IntelRealSense/librealsense/issues/3009
If you are adding the file to the website at a later time after it has been captured though, you may find it easier to just use the built-in 3D point cloud mode of the RealSense Viewer program, which has these same ply file export options integrated into it.

Hi @elyazinho    Do you require further assistance with this case, please?   Thanks!

Case closed due to no further comments received.


Finance and Insurance Services

Due to a lack of interoperability between Point of Service (PoS) systems of health care providers (e.g. Hospital Information Systems) and information systems of health care payers (e.g. Health Financing and Insurance System) there is a lack of automated payment processes for health services.
A PoS needs to communicate at several stages of the health care process with the payer of the service: a) to verify that occurred costs are covered for a patient (and to which extent) and b) claim the incurred costs for a delivered health care service from the payer. Depending on the number of patient visits and the nature of the coverage, this can create high data volumes at all process stages, e.g Nepal ca. 20 thousand hospital claims to the national health insurance  per day. Manual entry and transfer of data results in costs for duplicated data entry and errors.

https://github.com/IHE/IT-Infrastructure/blob/master/Proposals/2022-11-10.IHE_Profile_Proposal_FIS-Brief.DOCX

Discussed ITI-Planning November 15, 2022
Actions: John

Need to outreach to find more that might be interested. Use existing mailing lists,
create a new topic-specific-mailing list to gather these interested parties.
reach out to HL7 Finance workgroup, and their projects

https://guides.ohie.org/arch-spec/introduction/openhie-finance-and-insurance-services-workflows


Fix syntax like arr[0] < num highlight error

Syntax like arr[0] < num was not highlight correctly. This pr fixed this by remove \] in the start regexp.
But there still some rare edge case was not highlight correctly:

[

]

You're always so reliable 🤗


monaco.editor.setTheme

error: monaco.editor.setTheme is not a function version: 0.10.1

Solved.  One of my projects had an older version. 0.10.1 works fine.


style.css is not fully compiled in lib folder

Hello! I just updated to 5.0 and have an issue with styles.
The bundled lib/style.css is not fully compiled, referring to relative imports like @import url(~react-flexview/lib/flexView.css); and @import url(./icons.css);, and thus giving me errors when I import it with webpack.

thank you for reporting this @ingro
We'll fix it asap

Thanks for the fast fix @FrancescoCioria !


Add Readme.md file the project.

maybe you can consider a standard format to follow and also use markdown language in readme file.
Project Title
One Paragraph of project description goes here
Table of contents
 Adding the list of contents it will easy of user.

Getting Started
These instructions will get you a copy of the project up and run on your local machine for development and testing purposes. See deployment for notes on how to deploy the project on a live system.
Running the tests
Explain how to run the automated tests for this system
Break down into end to end tests
Explain what these tests test and why
Give an example

License
This project is licensed under the MIT License - see the LICENSE.md file for details
Acknowledgments

Hat tip to anyone who's code was used
Inspiration
etc

And also you can add contribute.md file which is written in markdown language.
Contributing to the project
Please take a moment to review this document in order to make the contribution
process easy and effective for everyone involved.
Following these guidelines helps to communicate that you respect the time of
the developers managing and developing this open source project. In return,
they should reciprocate that respect in addressing your issue, assessing
changes, and helping you finalize your pull requests.
As for everything else in the project, the contributions to this project are governed by our team.
Bug reports
A bug is a demonstrable problem that is caused by the code in the repository.
Good bug reports are extremely helpful - thank you!
Guidelines for bug reports:

Use the GitHub issue search — check if the issue has already been
reported.

Check if the issue has been fixed — try to reproduce it using the
latest master or next branch in the repository.

Isolate the problem — ideally, create a reduced test case.

A good bug report shouldn't leave others needing to chase you up for more
information. Please try to be as detailed as possible in your report. What is
your environment? What steps will reproduce the issue? What OS experiences the
problem? What would you expect to be the outcome? All these details will help
people to fix any potential bugs.

Any other information you want to share that is relevant to the issue being
reported. This might include the lines of code that you have identified as
causing the bug, and potential solutions (and your opinions on their
merits).

Feature requests
Feature requests are welcome. But take a moment to find out whether your idea
fits with the scope and aims of the project. It's up to you to make a strong
case to convince the project's developers of the merits of this feature. Please
provide as much detail and context as possible.
Pull requests
Good pull requests - patches, improvements, new features - are a fantastic
help. They should remain focused in scope and avoid containing unrelated
commits.
Please ask first before embarking on any significant pull request (e.g.
implementing features, refactoring code), otherwise you risk spending a lot of
time working on something that the project's developers might not want to merge
into the project.
For new Contributors
If you never created a pull request before, welcome: tada: : smile: Here is a great tutorial
on how to send one :)

Fork the project, clone your fork,
and configure the remotes:
# Clone your fork of the repo into the current directory
git clone https://github.com/<your-username>/<repo-name>
# Navigate to the newly cloned directory
cd <repo-name>
# Assign the original repo to a remote called "upstream"
git remote add upstream https://github.com/this projecthq/<repo-name>

If you cloned a while ago, get the latest changes from upstream:
git checkout master
git pull upstream master

Create a new topic branch (off the main project development branch) to
contain your feature, change, or fix:
git checkout -b <topic-branch-name>

Make sure to update, or add to the tests when appropriate. Patches and
features will not be accepted without tests. Run npm test to check that all
tests pass after you've made changes. Look for a Testing section in the
project’s README for more information.

If you added or changed a feature, make sure to document it accordingly in
the README.md file.

Push your topic branch up to your fork:
git push origin <topic-branch-name>

Open a Pull Request
with a clear title and description.

For Members of the this project Contributors Team

Clone the repo and create a branch
git clone https://github.com/this projecthq/<repo-name>
cd <repo-name>
git checkout -b <topic-branch-name>

Make sure to update, or add to the tests when appropriate. Patches and
features will not be accepted without tests. Run npm test to check that all tests
pass after you've made changes. Look for a Testing section in
the project’s README for more information.

If you added or changed a feature, make sure to document it accordingly in
the README.md file.

Push your topic branch up to our repo
git push origin <topic-branch-name>

Open a Pull Request using your branch with a clear title and description.

Optionally, you can help us with these things. But don’t worry if they are too
complicated, we can help you out and teach you as we go :)

Update your branch to the latest changes in the upstream master branch. You
can do that locally with
git pull --rebase upstream master

Afterward, force push your changes to your remote feature branch.

Once a pull request is good to go, you can tidy up your commit messages using
Git's interactive rebase.
Please follow our commit message conventions shown below, as they are used by
semantic-release to automatically
determine the new version and release to npm. In a nutshell:

Issues
Issue open :
It is not just fun.If there is really the bug or issue or suggestion the create an issue or make a pull request.

Fixed in #57.