Parquet reader is not correctly reading the map type

Bug description
We have a Parquet table that contains a map type. When using Velox to read the data, we are encountering incorrect results. The map does not have any null values, but Velox is reading several null:null key-value pairs. Additionally, there is another  pair in the incorrectly read map that has incorrect data.
optional group frequency (MAP) {
    repeated group key_value {
      required binary key (UTF8);
      optional int32 value;
    }
  }

############ Column(key) ############
name: key
path: frequency.key_value.key
max_definition_level: 2
max_repetition_level: 1
physical_type: BYTE_ARRAY
logical_type: String
converted_type (legacy): UTF8
compression: SNAPPY (space_saved: 27%)
############ Column(value) ############
name: value
path: frequency.key_value.value
max_definition_level: 3
max_repetition_level: 1
physical_type: INT32
logical_type: None
converted_type (legacy): NONE
compression: SNAPPY (space_saved: 41%)

During my debugging, I discovered that the definition level of these null  pairs is being read incorrectly. The correct definition level should be 2, but it is being read as 1 (where the maximum definition level is 2).
The results are correct when using the Parquet C++ SDK or pandas to read this Parquet file.
System information
N/A
Relevant logs
No response

cc @yingsu00

I will give an example to provide a simple explanation.
Correct results:

Key
Value

a
1

b
2

c
3

d
4

e
1

Velox results:

Key
Value

a
1

b
2

c
3

z
2

null
null

@liujiayi771 Thanks for creating the issue. This might be a metadata parsing issue. Do you think you can add a test to ParquetReaderTest with the problematic file after removing sensitive information? You can make it disabled for now. e.g. TEST(MixedResultTest, DISABLED_test)

@yingsu00 Thanks. I will try to see if I can construct a test data that can reproduce the bug.

https://github.com/liujiayi771/velox/pull/1

This bad case has been fixed.


Figure out how to deal with LDAP's  base64 encoded passwords

Task Description
In LDAP the userPassword attribute contains the hash of the password encoded as base64 and the end result in LDAP will be something like this {ALG}base64.encode(md5/sha1/bcrypt(mypassword)). The PasswordEncoder in Spring, however, expects that the part after {ALG} is actually the hash which causes login failures. If you have saved the userPassword using PasswordEncoderFactories.createDelegatingPasswordEncoder it will work just fine, but in all other cases it will fail since the the compared hashed strings won't match.
Task Relationships
This task:

Is a follow-up of: #1670, #1614

Useful Links

RFC4519#userPassword
RFC3112

Help

Our chat
Points of contact:

@carlspring
@sbespalov
@steve-todorov

@steve-todorov @anki2189 is this task relevant?

Yes it is, but I believe we can fix it in another way which we'd need to discuss first.

the suggestion is to make DelegatingPasswordEncoder  decorator that will try to base64 decode password if first match failed.

I believe we can make it more reliable by simply using LDAP authentication bind which means to attempt to bind to the LDAP server as the user/password trying to login. This way the LDAP server will do the necessary checks for us and we won't have to worry about all the possible configuration variations you might have.
Maybe @sbespalov can share some thoughts on how to do this in our current code? (which are the classes responsible for this and if we can use spring to create a LDAP connection)

@sbespalov / @steve-todorov ,
Could you guys please work on improving the desription of the task with:

An improved clearer description
Classes on interest
Task list

Thanks!

Recently we discussed that there should be both LDAP authentication types available, which is bind and manager DN. We could also have it configurable so user could make it work as required for particular case.
I think we should fix the manager DN authentication anyway. On the other hand it will take much less time then implementing bind authentication (which will also require some UI work as well).
So my suggestion is to fix base64 encoding issue for manager DN case, this will allow us to have working LDAP authentication without any overheads.

So my suggestion is to fix base64 encoding issue for manager DN case, this will allow us to have working LDAP authentication without any overheads.

I agree with @sbespalov's suggestion - it should work in the short-term. The task list should be something like this:

StrongboxSecurityConfig.java -

This class currently serves no purpose - maybe we should remove it?
The passwordEncoder part should be moved in SecurityConfig.java and refactored to use a custom version of org.springframework.security.crypto.password.DelegatingPasswordEncoder instead of the original one.

Create a custom version of org.springframework.security.crypto.password.DelegatingPasswordEncoder which will attempt to base64 decode the password and finally delegate it to the password encoder.
Write a unit test case for the CustomDelegatingPasswordEncoder to ensure it works properly{ALG}md5/sha1/bcrypt(mypassword)
{ALG}base64.encode(md5/sha1/bcrypt(mypassword))

Write an integration test which tests with unboundid (some additional user records will need to be added in the testing ldif for this purpose)

@sbespalov is there anything I'm missing in the task list?

In the long term we should promote the binding authentication method and keep the password comparision for legacy/flexibility/compatibility reasons. This will be much more reliable because the entire authentication logic is handled by the LDAP server (i.e. credential comparision, password expiration, is the user active/banned/etc)
One more important note is - in Active Directory you cannot obtain the userPassword field at all (check this, official docs 1, official docs 2) They have a custom implementation which most likely uses LM hash or NTLM hash (similar to what they use for windows in the sam file). This means login will always fail in AD with the current implementation.

@steve-todorov thanks for update!
Purposed sub-tasks seems right in general, just one thing which I would slightly adjust:

This class currently serves no purpose - maybe we should remove it?

this class purpose is for @ComponentScan  at most, so the strongbox-security-api module could scan the appropriate subpackages. Instead of removing this configuration class I would suggest to rename org.carlspring.strongbox.config.SecurityConfig to org.carlspring.strongbox.config.WevSecurityConfig to make naming more clear.
Everything else seems fine!

Thanks for clarifying! I'll add another task related to this one with the description.
In that case - it'd be better to rename it to StrongboxSecurityApiConfig (following the module name), because otherwise it will leaves the false sense that it is the WebSecurityConfig, but in reality the entire web security configuration is under strongbox-web-core in the SecurityConfig class (we should maybe rename this to WebSecurityConfig to follow Spring's convention?)

I'm closing this in favor of #1840 and #1841. Thanks for your help in clearing the specs @sbespalov ! :)


Add suport for metric aliasing

I find it necessary to group metrics by a dimension and, bar duplicating metrics, there is no way of aliasing metrics e.g.:
We have the following space:
servers.<server_name>.
Desired, separate space:
databases.<server_name>.
(where this is a subset of the servers namespace)
Obviously this would be a duplication of the servers space which is redundant, it would be much better if it were possible to submit configuration for aliasing:
databases.<server_name> -> servers.<server_name>
making the databases.<server_name> space virtual.

Is this handled by https://github.com/dgryski/carbonzipper/tree/dynaq ?

yeah, so let's close this ticket

I'm not sure it is clear from the link to carbonzipper how it addresses this problem. Can someone either spell this out a little bit better and/or update the docs at https://github.com/dgryski/carbonzipper/tree/dynaq to more clearly explain how it solves this problem? The readme there is pretty sparse and doesn't include any example config for the "json file loaded at startup"

Ah yes, I see.  DynaQ is a work in progress, where we will support this kind of metric aliasing based on arbitrary configuration/implementation.  It isn't in a usable state yet.

@drawks The idea is that carbonzipper can support "dynamic queries"; rather than having these aliases handled on the carbonserver side, we create a fake metric prefix that carbonzipper knows to send off to an external service.  The externals service parsing the query-that-looks-like-a-metric and returns a list of new metrics to retrieve.  In our current prototype (internal, but supported by public code), we're querying a json file that has been extracted from our internal server directory which knows things like the current status of a server, data center location, etc.  This is the general framework by which we would add alias support.

So if this is a work in process which isn't released yet is it not a bit premature to mark this issue closed?

I closed it because it's not going to get implemented/fixed here.  I'm also ok with keeping this one open until there is something in the zipper available, if that's more preferable.


Add branch alias

It'd be great to add this alias so we can require the package as ^1.0@dev instead of just @dev, because the former won't install 2.0 releases, while the latter does and that's very dangerous. See https://getcomposer.org/doc/articles/aliases.md#branch-alias for details
Also it'd be nice if you could tag a release every now and then :)

Hi @Seldaek, thanks for the PR! :heart:

Thanks. Do you think you could delete the v1.0alpha1 tag? It doesn't make sense to keep it IMO as it is older than the v0.1 and v0.1.1 tags.

done


Equation Rendering Failure

Description
The equation (9), (10) is not rendered well. (The positions of \hat and ^2 are too high.)
(Optional:) Please add any files, screenshots, or other information here.
No response
(Required) What is this issue most closely related to? Select one.
Choose One
Internal issue ID
d03559bb-13bd-4470-b22f-c7d6766b0473
Paper URL
https://arxiv.org/html/2402.02439
Browser
Chrome/<IP_ADDRESS>
Device Type
Desktop

Screenshot.


Uses aync library for copying of the contents of the zip folder

Provides an alternative way to solve the issue #899.

Current coverage is 34.39% (diff: 7.14%)

Merging #908 into development will increase coverage by 0.30%

@@           development       #908   diff @@
=============================================
  Files                7          7
  Lines             1015       1006     -9
  Methods            128        129     +1
  Messages             0          0
  Branches           159        156     -3
=============================================
  Hits               346        346
+ Misses             669        660     -9
  Partials             0          0

Powered by Codecov. Last update 729888e...f08779b

Ok, let's try this out and see how this goes for a while. Cheers.


Validate if exist user

Show RosterVC and validate if exist user and continue showing view of roster, but if doesn't exist user shows LoginVC and asks email and password

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


Cannot display images from a full Windows path

Running in Windows 10
Jupyter in Chrome, launched from miniconda prompt
Code is in c:/eaf llc/aa-analytics and bi/ipyplot and that is where Jupyter is launched from
There is a subdirectory "samples" in the code directory, which contains images
If I specify the path like:
images = \
['c:/eaf llc/aa-analytics and bi/ipyplot/samples/cosine_loss.jpg',
 'c:/eaf llc/aa-analytics and bi/ipyplot/samples/crane_w_bucket.jpg',
 'c:/eaf llc/aa-analytics and bi/ipyplot/samples/CRISP-DM on Trello Example.jpg',
 'c:/eaf llc/aa-analytics and bi/ipyplot/samples/gans_for_good.jpg',
 'c:/eaf llc/aa-analytics and bi/ipyplot/samples/learned_optimizer.jpg',
 'c:/eaf llc/aa-analytics and bi/ipyplot/samples/LTC_half_cheetah_model.jpg',
 'c:/eaf llc/aa-analytics and bi/ipyplot/samples/MSI_algorithm.jpg',
 'c:/eaf llc/aa-analytics and bi/ipyplot/samples/neuroadaptation.jpg',
 'c:/eaf llc/aa-analytics and bi/ipyplot/samples/parametric_UMAP.jpg']

The images are not found by iplyplot
If I use:
images = \
['samples/cosine_loss.jpg',
 'samples/crane_w_bucket.jpg',
 'samples/CRISP-DM on Trello Example.jpg',
 'samples/gans_for_good.jpg',
 'samples/learned_optimizer.jpg',
 'samples/LTC_half_cheetah_model.jpg',
 'samples/MSI_algorithm.jpg',
 'samples/neuroadaptation.jpg',
 'samples/parametric_UMAP.jpg']

it works fine

Hi @eafpres
Good catch, thanks for reporting that.
I've run some tests and investigated this a bit and it seems it's related to the fact that the browser cannot access users file system directly so absolute path like c:/your/path/img1.jpg will not work at all. Each absolute path should be converted to a relative path as you did in your example.
Another thing I realized is that starting a jupyter server starts some sort of web server which can access files stored on the same level as the jupyter server or below that level. Any files above the level of where jupyter server is running won't be accessible.
So following your example:
If you're running your jupyter server from c:/eaf llc/aa-analytics and bi/ipyplot directory you won't be able to access images located under c:/eaf llc/aa-analytics and bi directory.
3 action items I see here:

add info about relative vs absolute paths to the docs
add warning about jupyter web server start location and potential issues in displaying images from inaccessible directories
add os.path.relpath(..) to automatically convert all the absolute paths into relative paths in the internal function responsible for generating HTML img elements

3 action items I see here to fix/improve this:

add info about relative vs absolute paths to the docs
add warning about jupyter web server start location and potential issues in displaying images from inaccessible directories
add os.path.relpath(..) to automatically convert all the absolute paths into relative paths in the internal function responsible for generating HTML img elements

Thanks for this--I had first noticed that it could not access a level above, then proved it was general to specifying an absolute path. I guess those are actually slightly different.  I think adding to the docs is a great first step, to avoid others doing what I did!
Thank you again for this; once I figured out the path, it's really simple and it's cool you included the automatic breakout of classes by folder.

I confirm the following works if Jupyter is launched from the level of the images or above, with the code somewhere below that:
image_path = 'c:/eaf llc/aa-analytics and bi'
path = os.getcwd()
rel_path = os.path.relpath(image_path, path)
files = os.listdir(rel_path)
images = [rel_path + '\\' +
          files[i] for i in range(len(files))
          if re.search(".jp", files[i]) is not None]

Thanks for double checking and confirming that!
You can achieve the same thing with a 2-liner as below using glob package:
import glob
import os

abs_paths = glob.glob('c:/eaf llc/aa-analytics and bi/*.jp*')
relative_paths= [os.path.relpath(path) for path in abs_paths]


Larva - no button anymore to compare diffs in WinMerge or WinDiff

Describe the issue
I am writing a Larva test under Windows. I have a test failure with a difference in the Larva console. There used to be a button the see the difference in an external tool like WinMerge. That button is no longer in the console.
Reporter
Martijn Dirkse

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '...'
Scroll down to '...'
See error

Screenshots

Environment
FF! 8.1.0-RC2-20240508.012135: v500
Running on LAPTOP-R1SK28C using Apache Tomcat/9.0.88
Java Version: OpenJDK Runtime Environment (21.0.2+13-LTS)
Heap size: 114M, total JVM memory: 159M
Free disk space: 125GB, total disk space: 464GB
Up since: 2024-06-05 14:05:11 (5m)

Additional Context

De button is er nog wél.


Are SARIF uploads from private repos only available to enterprise users?

I'm using an open source SAST scanner that generates SARIF files and as far as I understood, I would be able to upload the SARIF files using github/codeql-action/upload-sarif without having to be an enterprise user/paying for CodeQL since I'm not using the service. Was I mistaken? Can I not upload the reports to and view them under the Security tab? I'm getting an error in my workflow saying HttpError: Resource not accessible by integration. If it's the case that it's not available for private repos except for enterprise users, that is deeply unfortunate.

From https://docs.github.com/en/code-security/code-scanning/integrating-with-code-scanning/uploading-a-sarif-file-to-github

Note: For private and internal repositories, code scanning is available when GitHub Advanced Security features are enabled for the repository. If you see the error Advanced Security must be enabled for this repository to use code scanning, check that GitHub Advanced Security is enabled. For more information, see "Managing security and analysis settings for your repository."

so yes, you would need GitHub Advanced Security to be enabled for your private repository for this to work.

I switched to Enterprise and Advanced Security...doesn't come with it? LMAO.

Correct.
from: https://docs.github.com/en/get-started/learning-about-github/about-github-advanced-security

GitHub has many features that help you improve and maintain the quality of your code. Some of these are included in all plans, such as dependency graph and Dependabot alerts. Other security features require GitHub Advanced Security to run on repositories apart from public repositories on GitHub.com.
To purchase a GitHub Advanced Security license, you must be using GitHub Enterprise. For information about upgrading to GitHub Enterprise with GitHub Advanced Security, see "GitHub's products" and "About billing for GitHub Advanced Security."

Seems to me like your question has been solved, so I'm closing this issue.


fix usage of repeatable inside inline create modal

refs: #2749
Problem: Using repeatable inside the the inline create operation would not work for two main reasons.

we didn't have the before_scripts tag in inline file so repeatable could not push to an inexistent stack.
the click in inline modal save button would not trigger the repeatable field to parse the input and store the correct information to be sent with request.

Solution: add the before_scripts stack in inline create files. Add the save-block class to the inline create save button, that way it will be picked by repeatable as it was the main form save button.

A new inspection was created.

A new inspection was created.

The inspection completed: No new issues

The inspection completed: No new issues

Closing as this had been merged into: #3313

@pxpm this is not a big deal, because you're the author of all these PRs 🤣
But in the future, to merge multiple PRs into one, you can change the branch the PR is pointing to (using Github, right under the PR title). Instead of pointing to master you can point them to the branch you're working on the merge them (ex: multiple-repeatable-fixes. This way:

the PR will show up as "merged" instead of "closed"
it will give credits to the contributor;
it will help our Release Drafter Bot know that the PR has been merged, so the PRs will show up individually in the release notes;

Again, not a big deal this time since you're the author for all of them. I bet @pxpm won't mind that @pxpm merged his PR into @pxpm 's branch without giving credits to @pxpm 🤣

Please bring this fix into backpack 5.

Will do! @pxpm can you please take a look?


Compile problem with  @angular-devkit/build-angular 0.800.4

🐞 Bug report
Description
I have a problem with @angular-devkit/build-angular 0.800.4 while I did not it with previous version of @angular-devkit/build-angular
When I want to compile my project with ng serve I got this error

ERROR in ./src/app/modules/main/change-password/change-password.component.ts 56:12-21
"export 'DialogRef' was not found in 'narik-infrastructure'

and this is related to this line of code:

@Inject(DIALOG_REF) public dialog: DialogRef

DIALOG_REF is an injectionToken and DialogRef is a interface in external angular package.
I get this error only when I use DialogRef in constructor. If I use it as an usual declaration like
dialog: DialogRef i get no error.
You can see error on this repo: https://github.com/NarikMe/narik-material-demo
if you change @angular-devkit/build-angular version to "0.800.4"

Duplicate of https://github.com/angular/angular-cli/issues/14876


enhance ip range config

change ip add range to  -  format
add a available ip address field to network inspect.
add tests around ip addr range

LGTM


TensorFlow documentation should be associated with the applicable specific versions of TensorFlow

URL(s) with the issue:
Please provide a link to the documentation entry, for example:
https://www.tensorflow.org/guide/gpu
Description of issue (what needs changing):
I was reading this documentation page https://www.tensorflow.org/guide/gpu, but it is unclear which TensorFlow version the documentation applies to. Especially now that a stable version of TF 2 has been released, it is important to clarify which TF version the documentation refers to.
On the top bar of the linked webpage, there is a menu called API, where you can select the API you are interested in. Right now, the sub-menu corresponding to the specific version of TF I am reading is not even highlighted, so I don't know if I am reading the documentation for TF 1 or 2. To confuse people even further, even though I think I am reading the documentation for TF 2, in another top bar, it is written TF 1.
Given that I am confused, another person can also be confused, so the documentation needs to be clarified. So, I suggest that every documentation page is associated (in a very clear way) with all applicable TF versions.

Closing this issue now since the TF guide is for TF2 and there is a separate TF1 tab in the navigation that redirects to the TF1 docs repo


JS: improve support for route handlers in containers

This is a proposal a step towards recognizing additional route handlers at our route setups.
A common theme in many advanced server applications is that route handlers are defined in one corner of the application, and then registered in route setups in a different corner using an intermediary container for transport. This intermediary container strains our current backwards type tracking for route handlers (See some of the many test cases added in this PR).
While the purest way of supporting this pattern is to extend our general type tracking, I think this PR is a sweet spot step that can leverage our existing route handler heuristics.
For team-context, this PR is safer than https://github.com/github/codeql-javascript-team/issues/57, and inspired by https://github.com/github/codeql-javascript-team/issues/52.
I am not very opposed to dropping this PR completely, but I think we should at least have a good discussion (somewhere) about the likely ways forward.
The performance seems unchanged on a preliminary evaluation.

(I have snuck in some additional tests I had around.)

Now that https://github.com/Semmle/ql/pull/3169 is merged, I used the Collection steps from that PR to make a slight generalization of your ContainerMap.
Commit: https://github.com/Semmle/ql/commit/15acc39612bb1accd78ea81eba066b92e5b45feb
You can merge that into this PR if you want.

Thanks. I will be able to do a query console run once the dist-upgrade completes.

This seems promising, but still a bit underwhelming. This query console run results in this spreadsheet which shows that we now identify 398 new Express::RouteHandlers in 35 of 120k projects.
I'll check the performance, and verify that the results above are TPs...

The results look good. Just waiting for the performance now.
The most common new route handler we identify is the deeply nested one:
app.get('/api/component', routes.componentRegistry.components);

One particularly nice new route handler to see in practice is this one:
var app = makeapp();
Object.keys(routes).forEach(function (route) {
  app.get(route, routes[route]);
})

Thanks for getting these numbers. 👍 Happy to see this moving forward again; let's hope performance works out.

The evaluation is benign.
I did try out https://github.com/github/codeql/commit/15acc39612bb1accd78ea81eba066b92e5b45feb, but had simplify it (see https://github.com/github/codeql/pull/2801/commits/f618d430e7274de8ddd5c0140e7009f9856a2668) since it did not scale as it also type tracked route handlers once they exited their container, and that is the responsibility of RouteSetup::getARouteHandler.

Tests are green

Ping. This is still ready for merge IMO.


WebSocket connection can not work inside Docker container

Do NOT ignore this template or your issue will have a very high chance to be closed without comment.

Describe the bug
I use Docker container to develop a Vite app.
I use the default port 3000 for app, and expose 8080 from Docker container.
Normally I can open the app inside Docker container from host machine's browser with <IP_ADDRESS>:8080.
The index.html page shown correctly, but the console of browser throw an error of WebSocket connection and infinitely refresh page:

[vite] connecting...
VM1722 vue.js:8446 You are running a development build of Vue.
Make sure to use the production build (*.prod.js) when deploying for production.
VM1719 client:25 WebSocket connection to 'ws://<IP_ADDRESS>:3000/' failed: Error in connection establishment: net::ERR_CONNECTION_REFUSED
(匿名) @ VM1719 client:25
VM1719 client:125 [vite] server connection lost. polling for restart...

The WebSocket connection port is the same as default port(or port set in vite.config.ts), and didn't use the port 8080.

Reproduction
Create a app inside a Docker container, expose a different port from Vite config, run.
After search in Vite's code, I find the WebSocket port is set in https://github.com/vitejs/vite/blob/master/src/client/client.ts#L35 and comes from the config file vite.config.ts.
To make it work, I modified the node_modules/vite/dist/client/client.js#L23, manually set the expose port 8080 and it works. The error gone.
I think the problem is caused by the script run in client, it get port from config in app, and can't get the port in browser.
If we could add a config item socketPort or is there a method to automatically get the right port in browser?
Many Thanks!
System Info

required vite version: 1.0.0-rc.4
required Operating System: debian 10 with Docker image node:12.18.3-alpine
required Node version: 12.18.3
Optional:

npm/yarn version: 1.22.0
Installed vue version (from yarn.lock or package-lock.json<EMAIL_ADDRESS>Installed @vue/compiler-sfc version<EMAIL_ADDRESS>

Logs (Optional if provided reproduction)

Run vite or vite build with the --debug flag.
Provide the error log here.

Hi @angela-1 and @underfin ,
I am having issues setting up a basic vite project in docker. I documented the steps in this repo and readme.
This was the only docker-related issue I found in Vite gh and it seems like you got further than me. I tried looking in your latest vue project but didn't see any docker files. Do either of you have a basic config that works that you can share?
Thanks,
Carlos

@shortpoet
The issues from a privite project, and I have uploaded a new public project to show the structure.
You can see https://github.com/angela-1/azalea
You should read the README.md to make it work.

Wow, that is awesome! Thank you very much Angela!! 😊 千万谢谢你!!

@shortpoet
The issues from a privite project, and I have uploaded a new public project to show the structure.
You can see https://github.com/angela-1/azalea
You should read the README.md to make it work.

thanks, guy

Hey any news on merging this issue ? Would love to use vite on github codespace.


Remove unused test case

Recover failing CI build

Coverage remained the same at 62.699% when pulling 43a7935edaf84e163ac1b0b9570d4df3bce6e3f6 on valvijo:master into cd1b9738af616b232b8c9ad3817369815cb536f1 on esaude:master.


🚀 Feature: Support more advanced entity card filtering in new frontend system

🔖 Feature description
Support multiple kind and type filter combinations for entity cards in config.
An example is to show an entity card on a Component entity page but only for the Website type but also show the same card on System entity pages regardless of type.
🎤 Context
Currently, entity card filter config does not allow for filtering the type for one specific kind when you specify multiple kind values e.g.
    - entity-card:catalog/depends-on-resources:
        config:
          filter: kind:component,system type:website

In the above example, System entity pages will not show the card because the type filter applies to both component and system but there is no website type for a System.
✌ Possible Implementation
An example of how this could be expressed in config could be something like:
filter: kind:component&type:website kind:system

Open to better ideas though.
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

Thanks for writing up this issue.
Another possible string format, would be to be even LESS structured than it is today, and letting <kind.type> be a shorthand that always has an implicit "OR" operator between them.
Example:
component.website component.service system is:orphan
would effectively mean
(
  (kind:component & type:website)
  OR
  (kind:component & type:service)
  OR
  (kind:system)
)
AND
(
  is:orphan
)


Handle other display sizes

The driver is currently hardcoded for 128x32 displays, though the controller supports up to 128x64, and other sizes are available. Implement a mechanism for runtime or compile-time display size selection.

Initial support for 64-line displays. Switching between is currently a manual process - will be runtime-changeable.

This is now runtime-settable using either the Start or Startx method in the driver. The display's width and height are passed as the first two parameters.

Should've been closed a long time ago...


New var in constructor

Added new constructor variable to choose between https and http protocol, by default http

LGTM, thanks!


Proxy added

fixed issue #118

Ok, my bad.
On Fri, Jul 14, 2023, 5:51 PM Ammoniya @.***> wrote:

Merged #120 https://github.com/c2siorg/Scan8/pull/120 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/c2siorg/Scan8/pull/120#event-9821372115, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AN34FSLNOBSDRQADN5DTYPTXQE2VBANCNFSM6AAAAAA2I2JAFQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Reference dynamic route variables in Legacy Collection examples in v5.mdx

Description (required)
As dynamic routes are also allowed to use variables, it's easy to overlook when following directions specific to file content. This PR explicitly calls out changing any references to slug used in dynamic routes to id.

Related issues & labels (optional)

Closes #
Suggested label: improve documentation, hacktoberfest-accepted

If you have additional questions, I'm also cdvillard in the Discord server.

Hehe, interesting idea! :)
We're currently not able to show a before/after diff of the file name inside an open file tab.
I really like the FileTree idea, but unfortunately the component doesn't seem to support diffs either.
I therefore think we'll need to resort to regular text to explain the necessary changes for now as you suggested.

@sarah11918 Took your recommendation into account, though I also renamed the file as well to reflect the change as well. Would that work, instead of worry about illustrating it via a diff?


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v4.12 to before v5.10.99",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=ded845a781a578dfb0b5b2c138e5a067aa3b1242",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=78b3f20c17cbcb7645bfa63f2ca0e11b53c09d56"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "ded845a781a578dfb0b5b2c138e5a067aa3b1242",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "78b3f20c17cbcb7645bfa63f2ca0e11b53c09d56",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "net: ieee802154: ca8210: Stop leaking skb's\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.10.99 by commit 78b3f20c17cbcb7645bfa63f2ca0e11b53c09d56, it was introduced in version v4.12 by commit ded845a781a578dfb0b5b2c138e5a067aa3b1242. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000226


DESIGN_1332: fix Close (confirmation) button after requesting to join a team

Change impacts the Challenge Portal only


MaterialProductGroupReport's modal shows only "Left" and "Right". Should the "Environment" value associated

like this


Notification when status is ALERT or OK

Hi guys,
Like many notification systems, I think sentinl should have an option to execute when the value is below threshold, sending an alert when over threshold then sending an ok when it back to normal. And slack action should allow customize template with color for ALERT(red) or OK(green)
Thanks

This will be possible with action conditions, discussed in https://github.com/sirensolutions/sentinl/issues/234


[Bug]: Sandbox image build failed on eval

Is there an existing issue for the same bug?

[X] I have checked the troubleshooting document at https://docs.all-hands.dev/modules/usage/troubleshooting
[X] I have checked the existing issues.

Describe the bug
I ran the following on the eval/24-sep branch:
$ ./evaluation/swe_bench/scripts/run_infer.sh llm.mini HEAD CodeActAgent 2 30 1 princeton-nlp/SWE-bench_Lite test &> ~/sand/eval_log.log

But I'm getting issues pulling the docker image:
20:44:21 - openhands:ERROR: runtime_build.py:386 - Sandbox image build failed: 404 Client Error for http+docker://localhost/v1.46/images/ghcr.io/all-hands-ai/runtime:d123cc51fdfb0f69ea7a3c8bdda940d7/json: Not Found ("No such image: ghcr.io/all-hands-ai/runtime:d123cc51fdfb0f69ea7a3c8bdda940d7")

Current OpenHands version
eval/24-sep

Installation and Configuration
git clone

Model and Agent
No response
Operating System
No response
Reproduction Steps
No response
Logs, Errors, Screenshots, and Additional Context
No response

Thanks a bunch @xingyaoww ! But it seems like maybe it's still broken? Here's a log. I also deleted all my docker images to be sure it wasn't something residual.
eval_log.log

23:05:08 - openhands:INFO: docker.py:31 - ---> [Warning] The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested
ahh i think it is probably because SWE-Bench doesn't officially support arm64 evaluation i believe? So maybe it is related to trying to build amd64 image on arm64?

Ah, I see... Maybe I'd have to do it on a VM.

Hmm @xingyaoww , it seems that it's also failing on a computer where I have x86_64
$ arch
x86_64

Here is the log:
eval_log.log


[BUG] 2020.3.25f1 fail to compile The name 'NuGet' does not exist in the current context.

After installing the package:

The NuGet file was fully ignored due to the macro #IF ZIP_AVAILABLE.
So what control the zip available?

Thank you for your report.
This package uses the ZipFile class to unpack the NuGet package, which is not available in Unity by default. Therefore, I enclose the affected code with ZIP_AVAILABLE to avoid errors. Next, I make the ZipFile class available, and define ZIP_AVAILABLE to make this work.
This bug is completely my mistake. This bug is caused by not correctly enclosing the affected code with ZIP_AVAILABLE.
I will fix it right now.

Thanks! Can't wait to use your importer~~

I fixed this as v2.0.1, and it will work. If this still doesn't work, please feel free to comment.

Hi, it still does not work. I saw you wrap all the code with #if ZIP_AVAILABLE, so now the whole package - all the code have been ignored cause it seems the macro is not activated in my project.  I wonder where you #define the ZIP_AVAILABLE macro or how can I trigger it?

OK, so it now works. The reason cause it was I
accidentally change the
-r:System.IO.Compression.FileSystem.dll in csc.rsp to
-r: System.IO.Compression.FileSystem.dll,
which cause the compile arg ignored maybe.


Kirby 3.6 - Internal error in the Versions plugin (error code user-not-logged-in)

Describe the bug
After upgrading to the latest version of Kirby (<IP_ADDRESS>) and the latest version of this plugin (1.0.1) , it is impossible to open the panel as a logged out user.
No exception is thrown for a user who is already authenticated.
Steps to reproduce

Go to https://example.com/panel
See the following error:

Kirby\Exception\LogicException thrown with message "Internal error in the Versions plugin (error code user-not-logged-in)"

Stacktrace:
#12 Kirby\Exception\LogicException in /Users/raphaelsaunier/code/project-name/site/plugins/versions/src/classes/Plugin.php:180
#11 LukasBestle\Versions\Plugin:hasPermission in /Users/raphaelsaunier/code/project-name/site/plugins/versions/src/config/areas.php:7
#10 Kirby\Filesystem\F:{closure} in /Users/raphaelsaunier/code/project-name/vendor/getkirby/cms/src/Cms/Loader.php:170
#9 Kirby\Cms\Loader:resolve in /Users/raphaelsaunier/code/project-name/vendor/getkirby/cms/src/Cms/Loader.php:92
#8 Kirby\Cms\Loader:areas in /Users/raphaelsaunier/code/project-name/vendor/getkirby/cms/src/Panel/Panel.php:61
#7 Kirby\Panel\Panel:areas in /Users/raphaelsaunier/code/project-name/vendor/getkirby/cms/src/Panel/Panel.php:302
#6 Kirby\Panel\Panel:router in /Users/raphaelsaunier/code/project-name/vendor/getkirby/cms/config/routes.php:101
#5 Kirby\Http\Route:{closure} in [internal]:0
#4 Closure:call in /Users/raphaelsaunier/code/project-name/vendor/getkirby/cms/src/Http/Router.php:111
#3 Kirby\Http\Router:call in /Users/raphaelsaunier/code/project-name/vendor/getkirby/cms/src/Cms/App.php:345
#2 Kirby\Cms\App:call in /Users/raphaelsaunier/code/project-name/vendor/getkirby/cms/src/Cms/App.php:1086
#1 Kirby\Cms\App:render in /Users/raphaelsaunier/code/project-name/index.php:9
#0 require in /Users/raphaelsaunier/.composer/vendor/laravel/valet/server.php:211

Expected behavior

I should be able to at least log in. 😅
Additional context

Technical details

Plugin version: 1.0.1
Kirby version:
Browser: not relevant
Server operating system: macOS
git --version on the server: 2.30.1 (Apple Git-130)

Sorry, that's a pretty obvious bug. 😅
I'll fix it ASAP.

No worries man! Thanks for all the time you put into this useful plugin. 🙏
After using it for a couple of months I noticed some other rough edges; if I find the time I'll try to contribute or at least open a couple of issues.

Fixed in a new plugin release 1.0.2, which also comes with some other minor enhancements. :)


esp_http_client https handshake takes over 15s and really slow in async mode idf 4.3 (latest) (IDFGH-5829)

idf 4.3 esp_http_client https became slow and unstable. 4.1.2 works fine
    esp_http_client_config_t config = {
            .url = url,
            //.cert_pem = cert_store_pem_start,
            //.timeout_ms = 10000,
            .event_handler = http_event_handler,
            .user_data = context,
            .is_async = true,
            .use_global_ca_store = true,
            //.keep_alive_enable = true,
    };

    esp_http_client_handle_t client = esp_http_client_init(&config);
    esp_err_t err;
    while (1) {
        err = esp_http_client_perform(client);
        if (err != ESP_ERR_HTTP_EAGAIN) {
            break;
        }
        vTaskDelay(2000 / portTICK_RATE_MS);
        ESP_LOGI(TAG, "esp_http_client_perform delay 2s");

    }

    if (err == ESP_OK) {
        ESP_LOGI(TAG, "HTTPS Status = %d, content_length = %d",
                 esp_http_client_get_status_code(client),
                 esp_http_client_get_content_length(client));
    } else {
        ESP_LOGE(TAG, "Error perform http request %s", esp_err_to_name(err));
    }

Please, refer https://github.com/rucebee/esp-telegram-bot/blob/main/main/app.cpp for full code and https://github.com/rucebee/esp-telegram-bot/blob/main/sdkconfig for config
esp_http_client http sync work fine, btw
Async esp_http_client_*
I (2533) app: https://httpbin.org/get?test=1
D (2533) HTTP_CLIENT: Begin connect to: https://httpbin.org:443
D (2533) esp-tls: host:httpbin.org: strlen 11
D (2563) esp-tls: connecting...
D (2763) esp-tls: handshake in progress...
D (2773) HTTP_CLIENT: Connection not yet established
I (4773) app: esp_http_client_perform delay 2s
D (4773) HTTP_CLIENT: Begin connect to: https://httpbin.org:443
D (4773) esp-tls: handshake in progress...
D (4793) HTTP_CLIENT: Connection not yet established
I (6793) app: esp_http_client_perform delay 2s
D (6793) HTTP_CLIENT: Begin connect to: https://httpbin.org:443
D (6793) esp-tls: handshake in progress...
D (6803) HTTP_CLIENT: Connection not yet established
I (8803) app: esp_http_client_perform delay 2s
D (8803) HTTP_CLIENT: Begin connect to: https://httpbin.org:443
D (8803) esp-tls: handshake in progress...
D (8813) HTTP_CLIENT: Connection not yet established
I (10813) app: esp_http_client_perform delay 2s
D (10813) HTTP_CLIENT: Begin connect to: https://httpbin.org:443
D (10813) esp-tls: handshake in progress...
D (11453) HTTP_CLIENT: Connection not yet established
I (13453) app: esp_http_client_perform delay 2s
D (13453) HTTP_CLIENT: Begin connect to: https://httpbin.org:443
D (13453) esp-tls: handshake in progress...
D (13463) HTTP_CLIENT: Connection not yet established
I (15463) app: esp_http_client_perform delay 2s
D (15463) HTTP_CLIENT: Begin connect to: https://httpbin.org:443
D (15463) esp-tls: handshake in progress...
D (15473) HTTP_CLIENT: Connection not yet established
I (17473) app: esp_http_client_perform delay 2s
D (17473) HTTP_CLIENT: Begin connect to: https://httpbin.org:443
D (17473) esp-tls: handshake in progress...

W (77033) esp-tls: Failed to open new connection in specified timeout
E (77033) TRANS_SSL: Failed to open a new connection
E (77033) HTTP_CLIENT: Connection failed, sock < 0
E (77033) app: Error perform http request ESP_ERR_HTTP_CONNECT
D (77053) app: HTTP_EVENT_DISCONNECTED
I (77053) app: Last esp error code: 0x8012
I (77063) app: Last mbedtls failure: 0x0
I (80063) app: https://httpbin.org/get?test=1
D (80063) HTTP_CLIENT: Begin connect to: https://httpbin.org:443
D (80063) esp-tls: host:httpbin.org: strlen 11
D (80233) esp-tls: handshake in progress...

D (120773) app: HTTP_EVENT_ON_CONNECTED
D (120783) HTTP_CLIENT: Write header[3]: GET /get?test=1 HTTP/1.1
User-Agent: ESP32 HTTP Client/1.0
Host: httpbin.org
Content-Length: 0

D (120793) app: HTTP_EVENT_HEADER_SENT
D (124063) HTTP_CLIENT: on_message_begin
D (124063) HTTP_CLIENT: HEADER=Date:Tue, 07 Sep 2021 11:47:05 GMT
D (124063) app: HTTP_EVENT_ON_HEADER, key=Date, value=Tue, 07 Sep 2021 11:47:05 GMT
D (124073) HTTP_CLIENT: HEADER=Content-Type:application/json
D (124073) app: HTTP_EVENT_ON_HEADER, key=Content-Type, value=application/json
D (124083) HTTP_CLIENT: HEADER=Content-Length:299
D (124083) app: HTTP_EVENT_ON_HEADER, key=Content-Length, value=299
D (124093) HTTP_CLIENT: HEADER=Connection:keep-alive
D (124093) app: HTTP_EVENT_ON_HEADER, key=Connection, value=keep-alive
D (124113) HTTP_CLIENT: HEADER=Server:gunicorn/19.9.0
D (124113) app: HTTP_EVENT_ON_HEADER, key=Server, value=gunicorn/19.9.0
D (124123) HTTP_CLIENT: HEADER=Access-Control-Allow-Origin:*
D (124123) app: HTTP_EVENT_ON_HEADER, key=Access-Control-Allow-Origin, value=*
D (124133) HTTP_CLIENT: HEADER=Access-Control-Allow-Credentials:true
D (124133) app: HTTP_EVENT_ON_HEADER, key=Access-Control-Allow-Credentials, value=true
D (124143) HTTP_CLIENT: http_on_headers_complete, status=200, offset=230, nread=230
D (124153) HTTP_CLIENT: http_on_body 282
D (124163) app: HTTP_EVENT_ON_DATA, len=282
D (124163) app: HTTP_EVENT_ON_DATA, waiting 17 bytes more
D (124163) HTTP_CLIENT: content_length = 299
D (124173) HTTP_CLIENT: data_process=282, content_length=299
D (124173) TRANS_SSL: remain data in cache, need to read again
D (124183) HTTP_CLIENT: http_on_body 17
D (124183) app: HTTP_EVENT_ON_DATA, len=17
E (124183) app: not parsed {
  "args": {
    "test": "1"
  },
  "headers": {
    "Content-Length": "0",
    "Host": "httpbin.org",
    "User-Agent": "ESP32 HTTP Client/1.0",
    "X-Amzn-Trace-Id": "Root=1-61375139-1f1a213671c0d43447347658"
  },
  "origin": "<IP_ADDRESS>",
  "url": "https://httpbin.org/get?test=1"
}

D (124223) HTTP_CLIENT: http_on_message_complete, parser=3ffb9c40
D (124223) app: HTTP_EVENT_ON_FINISH
I (124233) app: HTTPS Status = 200, content_length = 299
D (124233) app: HTTP_EVENT_DISCONNECTED

Hi @rucebee, thanks for reporting the issue.
Can you reproduce same issue with esp_http_client example provided in IDF?
If not, please share a small example so we can reproduce this issue at our end.

Hi @rucebee, thanks for reporting the issue.
Can you reproduce same issue with esp_http_client example provided in IDF?
If not, please share a small example so we can reproduce this issue at our end.

Thank you for reply,
The problem is not in IDF itself. I installed the latest from github and now it works fine.
So, the steps a got into the trouble:

Install fresh Visual Studio Code
Install IDF plugin
Let IDF plugin to download the latest version of IDF while configuring
And then you get IDF configured for self debugging and as a result working terribly slow

Now, I have the latest IDF installed manually from github and all is fine
BTW, "a small example" is in the first post and the further link to repo

@rucebee Great to hear that you are no longer observing this issue. Feel free to reopen.


can not show show multi-line and multiple aligned equations correctly

I am aware that the theme enables MathJax support. But it seems that right now, multi-line equations and multiple aligned equations are not shown correctly.
For multi-line equations
$$\begin{equation}
\begin{aligned}
a &= b + c \\
  &= d + e + f + g \\
  &= h + i
\end{aligned}
\end{equation}$$

It is rendered as

There are no line breaks between different lines.
For multiple aligned equations
$$\begin{align}
a &= b + c \label{eq3} \\
x &= yz \label{eq4}\\
l &= m - n \label{eq5}
\end{align}$$

It can not be rendered at all. The output is

More failed examples can be seen here.
How to make it work?

see https://discourse.gohugo.io/search?q=mathjax escape

@jdhao You should use the mmark processor.
Just add markup: mmark at the top of your markdown file.

thanks, I also found that we can use pandoc as external render. we need to change the extention of markdown file to pdc  and install pandoc.


tabela responsiva com opcao de traducao em portugues + opcao de adici…

Tabela responsiva com opcao de traducao em portugues + opcao de adicionar e remover colunas

Please translate in English... @MatheusLima7

Translation done with Google Translate:
Responsive table with option of traducao in Portuguese + option to add and remove columns
I think we can all understand what he's trying to say :)

Add and Remove Collumn. It's practical.
$('#sample_2_column_toggler input[type="checkbox"]').change(function(){
console.log("Entrou");
/* Get the DataTables object again - this is not a recreation, just a get
of the object */
var iCol = parseInt($(this).attr("data-column"));
var bVis = addCollumnTable.fnSettings().aoColumns[iCol].bVisible;
addCollumnTable.fnSetColumnVis(iCol, (bVis ? false : true));
});
2017-07-11 9:24 GMT-03:00 William<EMAIL_ADDRESS>
@William-H-M commented on this pull request.
In src/js/custom.js
https://github.com/puikinsh/gentelella/pull/529#discussion_r126672407:

+$(document).ready(function(){

addCollumnTable = $("#datatable-responsive-collumn").dataTable( {

   "aoColumnDefs": [

       { "bVisible": false, "aTargets": [ 7, 8 ] }

   ],

   "aaSorting": [[1, 'asc']],

       "aLengthMenu": [

       [5, 15, 20, -1],

       [5, 15, 20, "Todos"] // change per page values here

   ],

   // set the initial value

   "iDisplayLength": 10,

   "oLanguage": {

           "sProcessing": '<i class="fa fa-coffee"></i> Aguarde...',

           "sLengthMenu": "_MENU_ Quantidade",

           "sInfo": "Total de _TOTAL_ registros - de _START_ até _END_",

I think this shouldn't be include as DataTable has a full translation of
these option on multiple languages on json format here:
https://datatables.net/plug-ins/i18n/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/puikinsh/gentelella/pull/529#discussion_r126672407,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APOKToNwdrC4C39fx3GKIOZAIMQhCTcBks5sM2mRgaJpZM4OTseA
.

--

Hi, certainly column visible property is useful, but this more a plus of DataTables extensions, that has been now moved to the buttons section, the idea of the template is to show the basic of the work for basic examples of use, for more advanced use of the different plugins of the template user should go to the respective vendor of the plugin

I understand. I tried to contribute.
2017-07-11 9:58 GMT-03:00 William<EMAIL_ADDRESS>
Hi, certainly column visible property is useful, but this more a plus of
DataTables extensions, that has been now moved to the buttons section
https://datatables.net/extensions/colvis/, the idea of the template is
to show the basic of the work for basic examples of use, for more advanced
use of the different plugins of the template user should go to the
respective vendor of the plugin
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/puikinsh/gentelella/pull/529#issuecomment-314435045,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APOKTk0nH45uGbE7h8GvMBmJ5CwGp89Gks5sM3FpgaJpZM4OTseA
.

--

Hi, @MatheusLima7 yes we understand that, but as there're too many examples of it, it's not necessary, at least for now or until I know, but thank you very much for your contribution, keep helping us maybe a PR of yours in the future could help somebody :)

:)
2017-07-11 10:51 GMT-03:00 William<EMAIL_ADDRESS>
Hi, @MatheusLima7 https://github.com/matheuslima7 yes we understand
that, but as there're too many examples of it, it's not necessary, at least
for now or until I know, but thank you very much for your contribution,
keep helping us maybe a PR of yours in the future could help somebody :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/puikinsh/gentelella/pull/529#issuecomment-314449889,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APOKTsEV1Sgk6aIwkV4dv_CNBJA_NZgvks5sM33rgaJpZM4OTseA
.

--


Tags should be formatted in the way they are specified

Is your feature request related to a problem? Please describe.
Currently the template formats the tags differently to the way I write them.
Examples:

I have the tag LaTeX, but the tag can be found at /tags/latex (not /tags/LaTeX) and the headline says Latex (not LaTeX).
I have the tag cost-efficiency analysis. It can be found at tags/cost-efficiency-analysis (which is fine), but the heading says Cost-efficiency-analysis (first letter upper case and space replaced by a dash).

Describe the solution you'd like
The tag should be formatted everywhere, i.e., on the pages /blog,  /blog/the-post-with-the-tag,  /tags, /tags/the-tag,  in the exact form as it was specified.
Describe alternatives you've considered
I can fix this for my blogs but I thought others might like the change as well.
So I'm opening a feature request for a discussion :)

There are a few parts of the blog that refer to specific tags. E.g. "the tag" gets rendered in:

The URL path - /tags/the-tag
The tag component itself - THE-TAG
The tag header - The-tag

The URL path has to be specially formatted e.g. by removing whitespace and special characters - we use github-slugger for that.
My approach, was to then make 2 and 3 follow the URL path with some styling changes e.g. all caps for the tag component and capitalizing the first letter of the first word for 3, just because I think that makes it look better 😄. I find white spaces in the tag to look pretty odd as well and might be mistaken as a different tag. A side benefit is that it combines tags that are likely to be similar e.g. Cost-efficiency and cost efficiency and Cost Efficiency.
I can see the rationale for keeping 2 and 3 to follow the way they are specified especially for something like LaTeX, but that seems more of an edge case scenario and I believe most people would prefer the current defaults.
Happy to hear other thoughts on this issue.

Related Hugo discussion - https://discourse.gohugo.io/t/would-be-great-if-frontmatter-variables-didnt-have-to-be-all-lowercased/1608
At least with our situation, you can easily change it to suit your own preferences (as you mentioned) instead of having to make a change to the entire framework.

I implemented this for my website. I'm happy to write a short description what files need to be changed if someone is interested. Closing.


Kali in Windows 11 Problem

I tried to instal Kali Linux in my Windows 11, but I'm having a lot of issues HELP!!!!
Failed to attach disk 'C:\Users\Lopez\AppData\Local\Packages\KaliLinux.54290C8133FEE_ey8k8hqnwqnmg\LocalState\ext4.vhdx' to WSL2: The system cannot find the file specified.
Error code: Wsl/Service/CreateInstance/MountVhd/ERROR_FILE_NOT_FOUND

That's because Windows Defender has quarantined several of the files.
My suggestion is to download Kali from their own repository, disable Defender, go offline and then install the image.
You will probably have to exclude \wsl.localhost\Kali (or wherever you've got it located) from Defender, since it regards every tool Kali uses as a security risk (which, in the wrong hands, it is).


Adding code lever to list

Thanks for maintaining this!

Awesome, thanks for the addition @nickveys :+1:


MWAA and Environment Variables in profiles.yml

Hello! I found your package tonight and am currently trying to get this set up. After finding workarounds for the below by essentially using the /tmp directory on MWAA, I believe I am on my last issue before successful deploy.
modules-path
target_path
log-path

I am trying to get secrets read in through my profiles.yml file. I have tried entries like the below but to no avail.
host: "{{ env_var('AIRFLOW__DB_ENDPOINT') }}"
host: "{{ env_var('AIRFLOW_DB_ENDPOINT') }}"
host: "{{ env_var('DB_ENDPOINT') }}"
host: "{{ env_var('AIRFLOW__DB_ENDPOINT', 'not_set') }}"

The last option let me bypass the error temporarily before it failed when building the model. I know this may not be entirely specific to your package, but noticed that you have helped people out before with issues trying to get MWAA running.
https://docs.aws.amazon.com/mwaa/latest/userguide/configuring-env-variables.html - I was using this somewhat but have not yet actually added anything under custom options. As you can see above, I tried adding AIRFLOW_ and AIRFLOW__ to the beginning but no luck.
Thought I would reach out and see if you have had luck with something similar. We just switched over to using AWS's secret manager in MWAA and its been working like a charm in our Python tools and it still uses Variable.get() (from Airflows library).
The typical error I get is:
Env var required but not provided: 'DB_ENDPOINT'
but have also gotten this:
  Env var required but not provided: 'DB_ENDPOINT'. This can happen when calling a macro that does not exist. Check for typos and/or install package dependencies with "dbt deps".
Any help would be appreciated if you have the time!

@lukedappleton would you mind sharing the plugin? I’m solving the same problem

@dnascimento It would just be something like this.
from airflow.plugins_manager import AirflowPlugin
from airflow.models import Variable
#Define the plugin class
class MyPlugins(AirflowPlugin):
    #Name your AirflowPlugin
    name = "my_plugin"
    os.environ.get(variable_name) = Variable.get(variable_name)

then just save it in your plugins folder and Airflow will initialize it automatically.

Hey! I have the same issue. There's still no better way that creating a special plugin for this?
Thanks!


Add a property on a part to indicate whether a track is audio only

Choir leads may want to upload a backing track that their choirs are familiar with already, and that may take the form of an MP3 (or similar) file. While the rendering pipeline in its current iteration should treat the source media the same as it would a video, the front end has no way of knowing whether a piece is a video or an audio file, which presents UX problems.
If we add a property (something like audio : true || false default false) that the front end can set through the API when creating a part record, it would enable the front end to change how it responds to the type of media that has been posted by the user. It also opens up additional possibilities in future products if we ever want to offer something focussed solely on audio.

Implemented in #52


How to use GPIO_digitalWrite_branching ?

I think I need to use GPIO_digitalWrite_branching() because I need to write a variable to a pin.
I tried multiple ways to call GPIO_digitalWrite_branching
GPIO_digitalWrite_branching(GPIOv_from_PORT_PIN(GPIO_port_D, 0), 1);
GPIO_digitalWrite_branching(GPIOv_from_PORT_PIN(GPIO_port_D, 0), high);

but they all return errors:
In file included from examples\GPIO\GPIO.c:17:
examples\GPIO\GPIO.c: In function 'main':
extralibs/ch32v003_GPIO_branchless.h:331:72: error: lvalue required as left operand of assignment
 #define GPIO_digitalWrite_lo(GPIOv)     GPIOv_to_GPIObase(GPIOv)->BSHR = (1 << (16 + GPIOv_to_PIN(GPIOv)))
                                                                        ^
extralibs/ch32v003_GPIO_branchless.h:341:96: note: in expansion of macro 'GPIO_digitalWrite_lo'
 #define GPIO_digitalWrite_branching(GPIOv, lowhigh)   (lowhigh ? GPIO_digitalWrite_hi(GPIOv) : GPIO_digitalWrite_lo(GPIOv))
                                                                                ^~~~~~~~~~~~~~~~~~~~
examples\GPIO\GPIO.c:103:3: note: in expansion of macro 'GPIO_digitalWrite_branching'
   GPIO_digitalWrite_branching(GPIOv_from_PORT_PIN(GPIO_port_D, 0), 1);

What am I doing wrong?

Use funDigitalWrite()/funDigitalRead() and funPinMode() instead.

Exactly what I was looking for.
Thanks!


External tags support

I`d like to suggest the addition of new feature - possibility to tag metrics using predefined tags.

Thanks for feedback, I modified it a bit as you`ve suggested.

Awesome! Thanks. One last thing: the README needs to be updated now since the params are now host and port instead of statsd_host and statsd_port.

0.2.0 is now released. Thanks a lot for the PR!


User listen stats are wrong

@sudara is this still an issue?
I've checked your page and it seems to be showing correct-ish number of Listens: 39174
I've checked locally for number per day and it shows it correctly but I'm assuming it's because I'm not caching anything locally. Would you please check your account if you are still seeing this issue?

@innatewonderer
I think maybe it's just the per-day value that's wrong?

We should just remove that number anyway, it's sorta meaningless to have a daily average. We'll get a chart in here soon which will be more useful.


bug: ~=, ~~ and ~* these three operation expressions need to be improved

When the operator is ~= and the request parameter name is missing

=== TEST 30: the request parameter `name` is missing and the operator is `~=`

--- config
    location /t {
        content_by_lua_block {
            local expr = require("resty.expr.v1")
            local ex = expr.new({
                {"arg_name", "~=", "jack"}
            })

            ngx.say(ex:eval(ngx.var))
        }
    }
--- request
GET /t
--- response_body
true

In this test case, due to the lack of the name parameter in the acquisition, the expected result should be false, and the actual result is true.

When the operator is ~~ and the request parameter name is missing

=== TEST 31: the request parameter `name` is missing and the operator is `~~`
--- config
    location /t {
        content_by_lua_block {
            local expr = require("resty.expr.v1")
            local ex = expr.new({
                {"arg_name", "~~", "[a-z]{1,4}"}
            })

            ngx.say(ex:eval(ngx.var))
        }
    }
--- request
GET /t
--- response_body
true

In this test case, due to the lack of the name parameter in the acquisition, the expected result should be false, and the actual result is true.

When the operator is ~* and the request parameter name is missing

=== TEST 32: the request parameter `name` is missing and the operator is `~*`
--- config
    location /t {
        content_by_lua_block {
            local expr = require("resty.expr.v1")
            local ex = expr.new({
                {"arg_name", "~*", "[a-z]{1,4}"}
            })

            ngx.say(ex:eval(ngx.var))
        }
    }
--- request
GET /t
--- response_body
true

In this test case, due to the lack of the name parameter in the acquisition, the expected result should be false, and the actual result is true.

Therefore, when the operator is in these three cases, we should deal with the case where the value of the obtained variable is nil.

I agree with you except the first case: should not nil ~= "jack" be true?

I agree with you except the first case: should not nil ~= "jack" be true?

Yes, the result I got is true. I think nil should not be compared.

I agree with you except the first case: should not nil ~= "jack" be true?

Yes, I thought about it. It is also a good choice to perform the ~= operation when getting the variable value to be nil. I will submit a PR later to fix the two cases of ~~ and ~*.


fix: ci

fix ci

Codecov Report

Merging #38 (2bad687) into main (b05f612) will increase coverage by 1.15%.
Report is 2 commits behind head on main.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main      #38      +/-   ##
==========================================
+ Coverage   70.04%   71.19%   +1.15%
==========================================
  Files          27       27
  Lines         247      243       -4
==========================================
  Hits          173      173
+ Misses         74       70       -4

see 5 files with indirect coverage changes


confuse of clearExtraData in ApplySnapshot

I found a helper function engine_util.ExceedEndKey():https://github.com/tidb-incubator/tinykv/blob/b4b7dca34ff1ebefa0a5844f08424224c888b620/kv/util/engine_util/util.go#L105-L110

When endKey equals to "", it stands for +infinity, So we need check it first, then use byte.Compare to compare them.
But in https://github.com/tidb-incubator/tinykv/blob/b4b7dca34ff1ebefa0a5844f08424224c888b620/kv/raftstore/peer_storage.go#L264
it compare them directly. Is it a bug?

When I running test, i met a situation:

If there are two store:

nodeId
store
region
state
range

1
1
1
Leader
["1", "2"]

2
2
1
follower
["1", "4"]

7
2
6
leader
["2", "4"]

region 1 split a new region 7 and split key is "2". Then node 1 send snapshot to 2, it update new region info and call clearExtraData to clean data range ["2", "4"] in store 2. But node 7 also in store 2 and it's a leader. Then it execute client command like "Get", it will not found it's data in the store. So why we need call clearExtraData in ApplySnapshot?

Node 7 should be in store 1,  because it split from node 1, I think.
Did you mean region 1 current leader 1 split firstly, and region 1 follower 2 had not split yet,  then 1 sent a snapshot to  2， which made 2 clearExtraData clean the data range ?
In this case, when 2 process the split command, and split a new node in region 6 at store 2, the new node will receive its leader's snapshot,  and get the data in range ["2", "4"] again. Perhaps this leader is node 7 in region1 at store 1.

@Whorra Thank you, maybe I was mistaken.

You are welcome.😊 And I met the same problem in clearExtraData as you describe.
I think it would go wrong if the old end key is "", and new end key is not "". For example, if the old end key is "", and new end key is "3". After comparing, clear would not happen because "3" > "".
Replace like that maybe help.
if bytes.Compare(newEndKey, oldEndKey) < 0 || len(oldEndKey) == 0

Node 7 should be in store 1, because it split from node 1, I think. Did you mean region 1 current leader 1 split firstly, and region 1 follower 2 had not split yet, then 1 sent a snapshot to 2， which made 2 clearExtraData clean the data range ? In this case, when 2 process the split command, and split a new node in region 6 at store 2, the new node will receive its leader's snapshot, and get the data in range ["2", "4"] again. Perhaps this leader is node 7 in region1 at store 1.
I have some question about clearExtraData:

We cannot guarantee that the leader is node 7 in region1 at store 1, other nodes can also be elected as leader.
Assume node 7 in region1 at store 1 is leader after split, it may keeps synchronous with the follower on store 2, so it needn't send snapshot. But the Key-Value of the follower on store 2 was cleared by node 2 on store 2 due to node 2 receives a snapshot from node 1(Leader).


Beta-test feedback: commas in prices

Can a comma break be added to average prices, the figure reads easier.

Added number formatting to front-page "current index", and to graph mouseovers

Comma included on average prices on graph and front page, happy to close


Modified the previous validtion function for checking domain protocol

Here is change:

I modified the stripProtocol() function with preg_replace() and modified the const values.

I am looking forward some feedback .

Aside from the commit comments. What's the benefit here?

@prio101 Like @jasonlewis said, what exactly does this PR aim to do?

Closing this as no feedbacks from  @prio101


Support tectonic tex engine

Description of proposed feature
Add support for using tectonic tex engine
Per project page, Tectonic is derived from Xelatex. It downloads the required files on demand, so the installation size is much smaller than a full Latex system.
For example, a typical Texlive installation on Linux is about 2.5 GB, whereas Tectonic is a single binary of about 20 Mb, and 50 Mb of files need to be downloaded to work with manim.
How can the new feature be used?
config.tex_template = TexTemplate(tex_compiler="tectonic", output_format=".pdf")

Additional comments
With this modification, I was able to get Manim to use Teconic.
diff --git a/manim/utils/tex_file_writing.py b/manim/utils/tex_file_writing.py
index 45e84d4..0393551 100644
--- a/manim/utils/tex_file_writing.py
+++ b/manim/utils/tex_file_writing.py
@@ -160,6 +160,18 @@ def make_tex_compilation_command(
             f"-output-directory={tex_dir.as_posix()}",
             f"{tex_file.as_posix()}",
         ]
+    elif tex_compiler == "tectonic":
+        if output_format == ".xdv":
+            outflag = ["--outfmt", "xdv"]
+        elif output_format == ".pdf":
+            outflag = []
+        command = [
+            "tectonic",
+            *outflag,
+            "-o",
+            tex_dir.as_posix(),
+            tex_file.as_posix(),
+        ]
     else:
         raise ValueError(f"Tex compiler {tex_compiler} unknown.")
     return command

However, with this configuration I had some problems rendering the fraction lines, which disappear during the animation. So there may be some incompatibilities.
For example
from manim import *

config.tex_template = TexTemplate(tex_compiler="tectonic", output_format=".pdf")
config.verbosity = "WARNING"
class CauchyIntegralFormula(Scene):
    def construct(self):
        formula = MathTex(r"[z^n]f(z) = \frac{1}{2\pi i}\oint_{\gamma} \frac{f(z)}{z^{n+1}}~dz")
        self.play(Write(formula), run_time=3)
        self.wait()

%manim -qm CauchyIntegralFormula

https://github.com/user-attachments/assets/db88a0a2-5a2c-4b65-9be4-bdd44d71ba80

Very good idea, thanks for the suggestion!
I've seen the issue with the fraction lines in the context of xelatex-rendered files before; there might even be some issues about it. It might justify spending some time to see whether we can get this fixed.
I suppose it would also be sufficient if we implemented some fairly general and flexible customization options for setting the TeX compiler.

The disappearing fraction lines are caused when the resulting SVG file contains strokes instead of filled areas - as for regular rendered text and equations. A general solution would be to not generally set stroke_width=0 on the imported SVG, but to try to parse the SVG file more detailed - which also would solve a lot of Tikz related issues.


Update resolution / aspect ratio in settings

The theme looks bad if the resolution and aspect ratio in Stepmania are not the same as the real values. For example, if our display is 10:16 1440p, and Stepmania thinks it's 3:4 and 960x1280, all UI elements are way, way off.
That said, we don't provide the options for 9:16 and 10:16 and the corresponding resolutions in settings.

Research if it's possible to change the values available in settings, and provide a sensible default
If not possible, write a readme section on how to change the values in Preferences.ini
(doesn't Preferences.ini affect all themes? can we set display options per theme?)

Added instructions into readme — can't find an easy way to add an option to the system options row. It goes metrics.ini -> _fallback/metrics.ini, and into "conf" where the rows are read from, but what is conf? Where is conf?..

SL-OperatorMenuOptions.lua might be relevant actually

My Graphics/Sound Options screen shows 9:16 as an available aspect ratio - among many other weird ones, actually. I'm using SM 5.1-beta2 and this theme's dev branch. Maybe upgrading SM would be a good solution?


Replace 'tracked' by 'planned' for future dates in the tooltips.

I guess the title says it all.

That's awesome! Thanks Thomas

Bear in mind I just patched the .ts source so you might run your compiler to update dist as well ;-)

Thanks again! Published a new version now.
v0.2.3


how to add multi select input box in admin bro please help with this issue

I spent alot of time trying to figure this out myself, and finally found out that Admin Bro uses react-select. Here is the link fro your reference
https://react-select.com/home

thank you can share sample code how you have implemented the functionality it will help me a lot


[BUG] When loading this plugin, I receive a ClassNotFoundException.

OS: Macos 13
java version "11.0.18" 2023-01-17 LTS
Burp Suite version: 2022.3.9 Professional

My Burp is giving me the following notification, and I have followed the steps for compilation.
java.lang.ClassNotFoundException: burp.BurpExtender
at java.base/java.net.URLClassLoader.findClass(URLClassLoader.java:445)
at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:587)
at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:520)
at java.base/java.lang.Class.forName0(Native Method)
at java.base/java.lang.Class.forName(Class.java:467)
at burp.cs1.a(Unknown Source)
at burp.cs1.(Unknown Source)
at burp.a6l.a(Unknown Source)
at burp.c7v.lambda$panelLoaded$0(Unknown Source)
at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539)
at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
at java.base/java.lang.Thread.run(Thread.java:833)

+1

+1

+1

+1

+1

The latest version of burpsuite is required. I tried the 2023 2 version and it didn’t work.

But when I load this plugin, the settings interface for GPT still cannot pop up.

I don't know if your burpsuit is the 2023 3 version, but I imported it successfully in the burpsuit 2023 3 version and it shows up in the settings window

https://github.com/aress31/burpgpt/issues/5


Feature: User can view annual leave

User story
As an employee of GDS
So that I can manage my remaining annual leave
I want to be able to see how many days of annual leave I have remaining this year.

What

Adds a UserData class which is used to store data about a User's remaining annual leave. This is a one-to-one relationship.
Adds a data migration to add the user_data table with a foreign key referencing the user table.
Implements the feature, with associated feature test

Why
Screenshots
Before

After

Design specification

I sketched out a class diagram for this portion of work, thinking about what classes I would need for upcoming work. I will expand on this in the future and it may change slightly, but for the time being this is how I envision some of the classes and their associations.


"Round when copying" is inverted

It rounds if set to No and doesn't if set to Yes
https://github.com/Boxadactle/coordinates-display/assets/63524957/aa0e1165-62d4-4430-aeda-65f24c65c3ba

I've fixed the language on the button.


segmentations

https://github.com/Daniil-Osokin/lightweight-human-pose-estimation.pytorch/blob/1590929b601535def07ead5522f05e5096c1b6ac/datasets/coco.py#L39
If using custom data, I have to label segmentations polygon ?
Can I only label keypoint and person bounding box?

Hi! Yes, you can just comment out this line. Check #127, #207 for more info about mask usage.

Some images in COCO has unlabeled areas, mask is used to compute loss for labeled areas only.
But I only train my data 10000 images and I don't label segmentations polygon for all.
Does your method will not train my data (loss is always 0)? Because all of the data don't label segmentations.

Please, check the previous line of code. It creates a mask and fills it with ones, so loss will be computed over all image. This line excludes from loss unlabeled areas, so you can comment it out.

Hope, it is clear now.

Hi,
I have a few questions after reading the tips and issues pointed out here.
I am currently using Labelme to annotate the keypoints and would convert them to the COCO format later on.

What if my custom dataset only has keypoints annotated?

If i have the 18 keypoints like shown in the image below,  where can i change the keypoint definitions in this repo to match my usage?

And for Validation of the model, are other metrics available like PCK, PCP (more info)? Would i be able to use your validation metrics with just keypoints in my GTs?

Thanks!


Add tracking numbers for orders

It should be fairly straightforward to provide a tracking number when dispatching a delivery group. The number (and URL if possible) should be included in the email the customer receives.

Re: #377


bug: scp does not use ssh_opts

scp config currently hardcoded, need to use ssh_opts

Fixed in https://review.openstack.org/379539


Notification Shows 3 Times on iOS

Hi,
Working on an Xamarin Forms app and when I show a notification using this library on an iOS device, the notification appears 3 times... 3 toasts appear, and then the notification is duplicated in the action centre 3 times.
The same behaviour does not occur on Android running the same code, and I have added an alert as well as some logging immediately before showing the notification, and the call to show the notification is only executed once. It seems the tripling up is happening inside the library?
Any advice would be appreciated.
Thanks.

Ok, please ignore this (at least for now).
I was seeing 3 notifications. I am now only seeing 2, no idea why the 3rd one has disappeared or why it happened in the first place.
The remaining 'duplicate' is coming from iOS - I'm using this library when receiving a push notification and it appears despite me thinking I'm sending a 'silent' notification the OS is still showing it's own toast (which looks exactly like the one I show using this lib). Haven't figured out why yet, but either way not a problem with this library.


Reduce noise of AmbiguousArtifactVariantsException logs even more

Follow-up to #955 and #956
Summary
Even after the the latest changes in #956, our code base is still flooded/spammed by the AmbiguousArtifactVariantsException logs.
We have a quite large amount of flavor dimensions with 4 different flavors. This many flavors probably causes more cluttering logs than in many other code bases. So we are especially affect.
Therefore I'd like to propose to reduce the log noise even more.
Background
AmbiguousArtifactVariantsException Insights
The content of the AmbiguousArtifactVariantsException message is the following:
The consumer was configured to find a library for use during runtime, preferably optimized for Android, as well as attribute 'com.android.build.api.attributes.AgpVersionAttr' with value '8.4.0', attribute 'com.android.build.api.attributes.BuildTypeAttr' with value 'debug', attribute 'com.android.build.api.attributes.ProductFlavor:appType' with value 'standalone', attribute 'com.android.build.api.attributes.ProductFlavor:endpoint' with value 'dev', attribute 'com.android.build.api.attributes.ProductFlavor:platform' with value 'androidtv', attribute 'com.android.build.api.attributes.ProductFlavor:tenant' [...]
  [...]
  - Configuration
  [...]
  - Configuration ':core:tenantADebugRuntimeElements' variant supported-locale-list declares a library for use during runtime, preferably optimized for Android, as well as attribute 'com.android.build.api.attributes.AgpVersionAttr' with value '8.4.0', attribute 'com.android.build.api.attributes.BuildTypeAttr' with value 'debug', attribute 'com.android.build.api.attributes.ProductFlavor:tenant' with value 'tenantA', attribute 'org.jetbrains.kotlin.platform.type' with value 'androidJvm':
      - Unmatched attributes:
          - Doesn't say anything about com.android.build.api.attributes.ProductFlavor:appType (required 'standalone')
          - Doesn't say anything about com.android.build.api.attributes.ProductFlavor:endpoint (required 'dev')
          - Doesn't say anything about com.android.build.api.attributes.ProductFlavor:platform (required 'androidtv')
          - Provides attribute 'artifactType' with value 'supported-locale-list' but the consumer didn't ask for it
          - Provides attribute 'com.android.build.gradle.internal.attributes.VariantAttr' with value 'tenantADebug' but the consumer didn't ask for it
          - Provides attribute 'tenant' with value 'tenantA' but the consumer didn't ask for it

So printing "only" the message of this AmbiguousArtifactVariantsException exception is already adding a lot of noise to the Gradle logs.
The message alone can take up to 157 lines when using four Android flavor dimensions. And this exception is thrown multiple times. In the case of four flavor dimensions 30 times (30*157=4710 lines).
The current isDebugEnabled condition is seemingly inverted
Printing the exception stacktrace for non-debug logs is likely unwanted and "only" the message for debug log.
The isDebugEnabled  else condition is currently always printed
LOGGER.warn is always printed (because isWarnEnabled is always true). Therefore the (expected) AmbiguousArtifactVariantsException exception is always logged.
Plugin Users Point Of View
AmbiguousArtifactVariantsException is for the plugin consumers not actionable at the moment. So any details about it is not adding value, but only clutters the users log output.
Pull Request Changes
With the insights from above this Change Request disables by default any output of the AmbiguousArtifactVariantsException exception. It is not actionable for the plugin users and therefore should not be shown by default.
Slightly more information can be requested via the Gradle --info CLI parameter (isInfoEnabled) where only the an hint is printed, that there's an ambiguity in the resolved config, and no further details.
Debugging information can be requsted via Gradle --debug CLI parameter (isDebugEnabled) which provides the complete exception with all the details.

Thank you so much for the PR. The behavior was intentional before, with the major difference being the log to contain the stacktrace vs not.
However, I can see how that might still cause a lot of logs, especially for significantly more complex projects.
I'll try to come to review this one as soon as possible.

@mikepenz Thanks for the approval and merge 🙏
Do you have an estimate when then next release will happen?

I might be able to get out a new build sometimes this week.

Can you please provide more details @androidacy-user

which log messages do you see
what's your log level (logs will show for info and debug log levels)
how do you run the gradle task

@androidacy-user Did you set org.gradle.logging.level in the gradle.properties file, either the one in the Gradle project or the one in your Gradle user home directory (e.g. $HOME/.gradle/gradle.properties)? And if so what is its value?
And can you please post the output of ./gradlew buildEnvironment?

@androidacy-user you are using an old version (11.1.3) of the AboutLibraries plugin. Please update to the latest version 11.2.1.
11.2.1 is the first version where this pull request is applied.


Update zio, zio-streams to 1.0.6

Updates

dev.zio:zio
dev.zio:zio-streams

from 1.0.5 to 1.0.6.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

labels: library-update, semver-patch

Superseded by #2125.


cargo build cache is invalidated.

Problem
In a Windows environment, using Docker for cross-compilation can cause some of the compilation cache to become invalid.
Steps

run command like this: docker run --rm -it -v D:\git\rathole:/root/src  rust_aarch64-gcc_openssl:1.80.1_9.4.0_1.1.0l         sh -c "cargo build --bin rathole --release -v "
run command again.It will result in logs similar to the following:

Dirty libz-sys v1.1.12: the dependency build_script_build was rebuilt<PHONE_NUMBER>.427356500s, 427356500ns after last build at<PHONE_NUMBER>.000000000s)
Dirty rustversion v1.0.14: the file `target/release/build/rustversion-0bf5fffcec081f72/out/version.expr` has changed<PHONE_NUMBER>.366127800s, 366127800ns after last build at<PHONE_NUMBER>.000000000s)

However, they shouldn't actually be considered changes.

Possible Solution(s)
No response
Notes
No response
Version
cargo 1.80.1 (376290515 2024-07-16)
release: 1.80.1
commit-hash:<PHONE_NUMBER>8c3df9ac2c1744589362a02ecafa99
commit-date: 2024-07-16
host: x86_64-unknown-linux-gnu
libgit2: 1.7.2 (sys:0.18.3 vendored)
libcurl: 8.6.0-DEV (sys:0.4.72+curl-8.6.0 vendored ssl:OpenSSL/1.1.1w)
ssl: OpenSSL 1.1.1w  11 Sep 2023
os: Ubuntu 16.4.0 [64-bit]

Thanks for the issue report. This is a known issue: https://github.com/rust-lang/cargo/issues/12060, and we have an open PR #13955. Close in favor of those.


How can i use Common Table Expressions (CTE)?

"with" queries are not supported at this time. I hope to add support for this at some point.
We do support select ... from (select ...) which is similar in some cases.


Cannot read properties of undefined (reading 'install')

Running into the following error when trying to run your project, installed via npm version 0.0.5 https://www.npmjs.com/package/nestjs-temporal
It seems that it thinks the core package from @temporal/worker is undefined
/node_modules/nestjs-temporal/dist/temporal.explorer.js:55
                yield worker_1.Core.install(coreConfig);
                                    ^
TypeError: Cannot read properties of undefined (reading 'install')
    at TemporalExplorer.<anonymous> (/node_modules/nestjs-temporal/dist/temporal.explorer.js:55:37)
    at Generator.next (<anonymous>)
    at fulfilled (/node_modules/nestjs-temporal/dist/temporal.explorer.js:14:58)

Also, how does this package work, does it boot up a worker service that listens on the same process as the controllers? Trying to implement this in a monolith where there are two separate apps, one that deals with API endpoints (controller), and the worker that solely runs workflows and activities

I have the same problem

The problem was that this repository relied on @temporalio/worker version 0.18.0, while I installed the latest one...


Undeclaring paths alias causes a incorrect match result

Problem
I don't declare a path alias link '@libs/*', and when I use a matcher created by createPathsMatcher, it results
@libs/constants → /cwd/@libs/constants

It doesn't make sense and should return undefined.

Expected behavior
It should return null if it matches nothing.
Minimal reproduction URL
https://stackblitz.com/edit/vitejs-vite-2pqdvg?file=index.mjs,tsconfig.json,package.json&terminal=dev
Version
4.7.2
Node.js version
18
Package manager
npm
Operating system
macOS
Contributions

[ ] I plan to open a pull request for this issue
[ ] I plan to make a financial contribution to this project

Thanks for the bug report! Happy to accept a PR

I made some changes, please check it out 🫶


Custom white box (representing the zoom area)

Hi,
I'd like to know if I can customize the white box (representing the zoom area) within the small preview image.
Thanks.

Right now the only customization available is changing the opacity of that box. However, I can probably add in some more options in the next update. What were you looking to customize on it?

I wanted something like amazon: a div with a background image instead of the white box.

Yeah, I should be able to add that feature into the next update. Should be done within the next day or two. I've got a couple of other things I want to include.

OK. Thanks. If you could look at my previous issue too, It would be great.

It wouldn't feel right to only notify you on one issue. :) So, I just published the update to NPM (version 1.3.0). You can now give the preview box a background image on all applicable components. The overlay color and box border color can also be changed now. So, if you only want to use a background image, you can set the overlayOpacity to zero and set the overlayBoxColor to transparent.

After update, I got exactly what I wanted.
Thanks again.


Unexpected manner of bytes.ReplaceAll

What version of Go are you using (go version)?

$ go version
go version go1.14.4 linux/amd64

Does this issue reproduce with the latest release?
Yes
What did you do?
package main

import (
	"bytes"
	"fmt"
)

func main() {
	old := []byte("")
	b := bytes.ReplaceAll(old, []byte("oink"), []byte("moo"))
	fmt.Println(old == nil)
	fmt.Println(b == nil)
}

What did you expect to see?
false
true

the call of bytes.ReplaceAll on an empty slice should also return an empty slice. However, it returns nil
What did you see instead?
false
false

Can I ask why you want them to be equally nil or not nil? As far as I know, that's not a guarantee of any of Go's APIs by default.

Can I ask why you want them to be equally nil or not nil? As far as I know, that's not a guarantee of any of Go's APIs by default.

The inconsistent type of slice (empty slice or nil slice) looks confusing. To me: Empty slice in, empty slice out. Nil slice in, nil slice out.

FWIW, I think generally the pattern here is to check length instead of nil-ness. The length of a nil slice is also zero.
CC @ianlancetaylor via https://dev.golang.org/owners

The bytes package makes no promise to preserve the nil status of an argument slice.  The general guideline of Go style is to treat nil slices and empty slices the same, which means to always use len(s) == 0 rather than s == nil.  While there are packages that intentionally distinguish an empty slice and a nil slice, those packages are always clearly documented.  The bytes package is not one of them.

The documentation clearly says "ReplaceAll returns a copy of the slice..."
If the slice is empty it should return the empty slice as it was exactly before without any changes.
Because it breaks things otherwise, and introduces inconsistency.

The byte.Replace function has worked as it does today since the Go 1.0 release, and the bytes.ReplaceAll function has worked this way since it was introduced in the Go 1.12 release.  The behavior is not buggy and it matches the documentation.  We aren't going to change it now.  Sorry.


Add contract class cache stats

Add contract class cache stats
Description
Add a counter to count the number of miss and hit when reaching the cache
Checklist

[x] Linked to Github Issue
[x] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.
[ ] Documentation has been added/updated.

Codecov Report

Merging #958 (489c93d) into main (a7d6b65) will increase coverage by 0.01%.
Report is 4 commits behind head on main.
The diff coverage is 82.35%.

@@            Coverage Diff             @@
##             main     #958      +/-   ##
==========================================
+ Coverage   90.56%   90.57%   +0.01%
==========================================
  Files          54       54
  Lines       13797    13849      +52
==========================================
+ Hits        12495    12544      +49
- Misses       1302     1305       +3

Files Changed
Coverage Δ

src/state/cached_state.rs
91.97% <82.35%> (+0.26%)
:arrow_up:

... and 3 files with indirect coverage changes
:loudspeaker: Have feedback on the report? Share it here.


Question about available lists of Swift.

I run swiftenv install --list, but I can't find the latest swift version.
Here is the output.
2.2-dev
2.2
2.2.1
3.0-dev
3.0
3.0.1
3.0.2
3.1-dev
3.1
3.1.1
4.0-dev
4.0
4.0.2
4.0.3
4.1-dev
4.1
4.2-dev
5.0-dev

By default, the swiftenv install command returns the versions of Swift that swiftenv knows how to build which does not include the newer versions.
The --list-snapshots option allows listing all the binary versions of Swift available for your platform, this will include snapshots. The command invocation will make a request to download the list of binary versions of Swift  from an API which will work on your platform and thus it is not a local/fast operation. --list is used for completions as it does not hit the network.
There is also --list --no-build to list only versions that can be installed without a building Swift (thus the locally known binaries). This may not be the latest if a new version comes out.


Organization page rethink

Overview
We need to check every action and the pages expected behavior and annotate the figma as appropriate.
History
We have been reviewing the page design and been making changes.  Everything is not yet worked out, so we need to do another full review.
Action Items

[ ] Change the items we talked about at the last meeting
[ ] Identify new inconsistencies to review.
[ ] Discuss at meeting (bonnie, gio, emerson)

Resources/Instructions
Figma of organizations page

Hi @giosce , I was reviewing the document with Maria today.
I did move the first page the left so that the comments that were overlapping got fixed, but there is still quite a bit that is not consistent from page the page (e.g., the number of results showing up for affiliate and non affiliated.
Also, it would be good if you changed the shape for the badge to be a circle or star because it being a inverted triangle is confusing, its too much like a drop down.

Hi @giosce , I was reviewing the document with Maria today.
I did move the first page the left so that the comments that were overlapping got fixed, but there is still quite a bit that is not consistent from page the page (e.g., the number of results showing up for affiliate and non affiliated.
Also, it would be good if you changed the shape for the badge to be a circle or star because it being a inverted triangle is confusing, its too much like a drop down.

I changed the badge icon.
Reviewed the numbers of contributors, I think it is consistent now.
Let's review next Sunday.

I changed the badge icon.
Reviewed the numbers of contributors, I think it is consistent now.
Let's review next Sunday.

Organization page flow has been been confirmed.


Fetch Selected Option

I have a select element where user would have chosen a particular option. When I do
read<EMAIL_ADDRESS>to x
echo x
Every option is printed out. Is it possible to fetch only the selected option?

Hi,
Usually for a select HTML element you will see something like
<select ...>

When you do a read step on the select, it will also read the text content for the child options. If you want to read only a specific one, then you would want to also make use of the option element in the xpath.
E.g. read<EMAIL_ADDRESS>to x

Hi,
Unfortunately its usually not possible to tell the current option being selected from the HTML document. Instead, you probably will need to use javascript in the webpage DOM.
This may provide a reference on how to do it - https://stackoverflow.com/questions/1085801/get-selected-value-in-dropdown-list-using-javascript , I am not so well versed in manipulating webpage DOM myself.


Is that possible to add vertical label along y axis?

for a chart containing two series, since those two y axis are different data, is that possible to add a vertical label along the y axis? for example, on the left y axis, the label is "Price", on the right y axis, the label is "Position"?

I'm assuming you're talking about v1, right ?

no, I am talking about v2.
I know I can do it through css. Just out of curiosity, if n3-charts already provides this feature, then I can directly use :-)

Nope, use CSS transform. Let me know if you need help on this


CVSL-206: Stamp the full name of the approver on the licence.

PLEASE DO NOT MERGE - requires table drop in DEV first.
This PR stamps the name of the approver into the licence table on APPROVAL so that it can be used for printing in the licence document as part of the signature page.

Codecov Report

Merging #36 (d1a76fa) into main (31e782c) will increase coverage by 0.26%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main      #36      +/-   ##
============================================
+ Coverage     78.34%   78.61%   +0.26%
- Complexity      266      270       +4
============================================
  Files            42       42
  Lines           882      893      +11
  Branches         12       12
============================================
+ Hits            691      702      +11
  Misses          181      181
  Partials         10       10

Impacted Files
Coverage Δ

...l/hmpps/createandvaryalicenceapi/entity/Licence.kt
98.14% <100.00%> (+0.03%)
:arrow_up:

...al/hmpps/createandvaryalicenceapi/model/Licence.kt
98.92% <100.00%> (+0.02%)
:arrow_up:

...ateandvaryalicenceapi/model/StatusUpdateRequest.kt
100.00% <100.00%> (ø)

...createandvaryalicenceapi/service/LicenceService.kt
91.05% <100.00%> (+0.30%)
:arrow_up:

...teandvaryalicenceapi/service/TransformFunctions.kt
85.71% <100.00%> (+0.10%)
:arrow_up:


I use the .inputmask({ alias: "hh:mm t" }); and it will not stop invalid key press

I have a textbox time field, I want it to show something like 03:03 pm.  I apply the alias inputmask({ alias: "hh:mm t" });  and I am allowed to physically enter characters in the number portion and the the mask is ignored, so I can type values that look like asdfsajdfasdfjaslkdj.  Am I doing something wrong?

@cfitzh2o ,
Which js files did you load?  You need to include the date extensions or use the bundle.

I installed the entire package with the NuGet package manager in a Visual Studio 2012 project so that file is installed.  I am using an include tag for the main library at the top of the page.  Confusing isn't it?

@cfitzh2o ,
You alse need to include the date extensions.