proposed changes to configuration format

Changes:

version listed at top
removed  top level camp-data
every reaction not involving aerosols has products and reactants keys. The type determines how they are used
extraneous information not related to mechanisms can be added with two underscores __, like the absolute tolerance which is a solver parameter
Species don't need to specify which phase they are in
every reaction can now have an optional label called name
Time units can no longer be specified
now we have explicit arrays for species, reactions, phases, meaning we no longer need TYPE keys for anything except reaction types
Removed Ternary Activation in favor of Troe; the only difference was the presence of M, which can be added
Used full names for exp (exponential) and coeff (coefficient)
Renamed Wennburg_NO_RO2 and Wennburg_Tunneling to Branched_NO_RO2 and Tunneling

@jcurtis2 @mwest1066, if you get a chance, I'd appreciate your feedback on these proposed changes.
And as an FYI, this repository will also publish a tool that parses these files and returns all of the data in arrays of structs for solvers to do what they will with


Self-deleting Arrow of Deletion doesn't respawn

Connect the arrow of deletion back on itself, start the loop, and loose your ability to destroy. This doesn't chuck a backup arrow to you like using the arrow to delete itself "by hand".
Edit: press "d". still a bug

steps:

make a loop of 4 nodes, one of them being AoD
start loop by deleting something out of loop using AoD


[todo-mvp-clean] Applying Dependency rule

I noticed the sample is not following the dependency rule proposed by the linked article Clean Architecture.I.e. GetTasks uses the Task Repository, which belongs to the data package. I would argue it is a style common when writing a 3-layer architecture where the TaskRepository is used as a Port to access the data layer.
In clean architecture the depencendy rule would force us to have a interface inside the domain layer which is implemented by some class in the data layer. As it is very common to have some kind of persistence, we could say we break the rule for convinience? How do we apply the dependency rule to other cases where domain logic wants to talk to the outside world i.e. Messaging or Sharing information.
What do you think? How did you manage this in practice ?

https://github.com/android10/Android-CleanArchitecture/issues/273 is tackling the same problem.

I very much agree with your point @mjohenneken .
I've expanded the topic to more aspects of Clean and summed them up here if you're interested:
https://github.com/android/architecture-samples/issues/695


fix: Return undefined for nonexistent action headers

This issue occurs when users switch the datasource of an API action, and the actionConfiguration headers is empty. This PR adds conditional checks to prevent an error from happening.
Fixes #18682
Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Bug fix (non-breaking change which fixes an issue)

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=b64ff45


office 2010 test is failed

Hi unamer
Thank you for seeing this issue。
test env：
OS: Windows2008 R2
Office：Microsoft Office Professional Plus 2010

generate vuln.rtf
python CVE-2017-11882.py -c "cmd.exe /c ipconfig >c:\windows\temp\1.txt" -o vuln.rtf

test is failed after open vuln.rtf ，no c:\windows\temp\1.txt
but i test https://github.com/Ridter/CVE-2017-11882/blob/master/Command43b_CVE-2017-11882.py is ok。
expecting your reply!
Thanks regards

Are you confirm that you cloned the newest update?
I fix a bug in command execution in last commit.
Now your command works in my meachine

yeah, i used newest update failed。
your test env is some？

No.
I'm using Office Pro plus 2013 in Win10.

python CVE-2017-11882.py -c "cmd.exe /c ipconfig >c:\windows\temp\1.txt" -o vuln.rtf

my exploit url：
https://mega.nz/#!trxWWLjA!OtZHHN-X9UjfZr0-pH0eF_KvU3ni7-rtE0Li8afvhHM

Well. That exploit you uploaded actually works on my machine.
Maybe I will test Office 2010 later.

I have tested Office 2010 Pro Plus in Win7 and my exploit works well.
I pop a calc.exe by rtf created by python CVE-2017-11882.py -c calc.exe -o test.rtf

thx for your reply,unamer
same commands generate calc poc url:
https://mega.nz/#!4rpgmJyQ!LPkCqJ2aEsXzDCNwL1J1rXnHvKfmxp0PShTe088EeSI
test is failed ,if file is ok,maybe i have other problems.
No matter what，i'm very thx for your help。😄

Well. Can you debug the shellcode?

Only 43 bytes can be used.

@roy-lion  What's your environment?

Office Pro 2007 in Win10.
The test is unsuccessful.


feat(@angular-devkit/build-angular): differential loading

This PR adds differential loading to the browser builder. First, it checks if differential loading is needed. This is the case if the compilation target is ES2015 while the browserslist points to ES5 browsers.
For providing differential loading, it calls the methods for creating the webpack config for each compilation target (e. g. ES5 and ES2015). The needed differences between those configurations are defined using parameters also added by this PR. Then it calls webpack for each of them, merges the results and writes the index.html.
This feature is currently hidden behind a flag. To activate it, set the ENABLE_DIFFERENTIAL_LOADING flag in packages/angular_devkit/build_angular/src/browser/index.ts.

@mgechev we are chatting about diff-loading in the triage meetings. Some ideas to make this more grop effort:

Could we reduce the scope for this PR to just produce the two different bundles into the dist/ directory?
@clydin can help with writing the index next, and figure out the right script ordering
Let's land it with a private flag that disables the feature to start with so we can get this in without breaking the next release

Some more comments above, also since we have a redundant css build. IE, we are building css twice. I am not seeing any logic to remove the CSS files from one of the builds.

Let's do this in another PR regarding optimizations. Is this ok?

@manfredsteyer, my point was more because in the html you have duplicate css tags. But yeah let’s do the index generation in a separate PR.
For the time being let’s focus on:

Emitting all the needed files and with the correct content.
Add a couple of test for the above.
Make this green.
Disable differential loading feature with a flag in code

Ah, now I see. The generate-index-html function uses the following code that was originally part of the IndexHtmlPlugin:
for (const file of fileNames) {

    if (existingFiles.has(file)) {
      continue;
    }

    [...]
}

This filters out duplicate files before css and script references are written.


audioRecorderPlayer.stopRecorder() not functioning properly

Please fill the template to help you out. Also, please try the Example project compare before submiting the issue when you have certain issue with your project setup.

Version of react-native-audio-recorder-player = 3.6.7
Version of React Native = 0.73.0
Platforms you faced the error (IOS or Android or both?) = both
Expected behavior = should have stopped the recording
Actual behavior = it showed that the recording has stopped but it kept on recording in the background
Steps to reproduce the behavior

Please fill the template to help you out. Also, please try the Example project compare before submiting the issue when you have certain issue with your project setup.

Version of react-native-audio-recorder-player = 3.6.7
Version of React Native = 0.73.0
Platforms you faced the error (IOS or Android or both?) = both
Expected behavior = should have stopped the recording
Actual behavior = it showed that the recording has stopped but it kept on recording in the background
Steps to reproduce the behavior

i  have solved it
i manage to solve it by going to the "react-native-audio-recorder-player" folder in my dependencies folder (node_modules for npm) and locating the index.ts file in it, then i looked for the stopRecorder function and made the following changes
BEFORE
`  stopRecorder = async (): Promise => {
if (this._isRecording) {
this._isRecording = false;
this._hasPausedRecord = false;
  return RNAudioRecorderPlayer.stopRecorder();
}

return 'Already stopped';

};
`
AFTER
`
stopRecorder = async (): Promise => {
if (this._isRecording) {
this._isRecording = false;
this._hasPausedRecord = false;
  return RNAudioRecorderPlayer.stopRecorder();
}

return RNAudioRecorderPlayer.stopRecorder();

};
`
i replaced the "return 'Already stopped';  " with the " return RNAudioRecorderPlayer.stopRecorder(); " so now it works properly.  NOTE: this wont cause any error for my project cause i am also rendering it with a condition.
so from the above i think it was not working properly initially because the "this._isRecording" was false so it couldn't return the RNAudioRecorderPlayer.stopRecorder(), and it should have been false.. so i think they need to look into it.

Update!!!!
i now realize where the main issue itself is coming from.
i should have defined the constant of audio player outside the component.
like this >>
import....
const audioRecorderPlayer = new AudioRecorderPlayer();
const App()=>{}


Improve detect-secrets parser

Motivation
The detect-secrets parser has not some basic information (like description or CWE code). This information can be useful to users in order to have a better UX and filter by more fields (like CWE).
What do I contribute?

[x] A description with some secret information
[x] The CWE code

If someone detects more information to add, we can use this Pull Request to include it 😄

Hi @valentijnscholten & @Maffooch. Thanks for your review! I don't remember to adapt the unit tests and verify it, so I include new changes since the approvals.


Java Script issue

problem when formatting js code where strict comparison "===" is used.
This does not happen with all strict comparison characters "==="
p.s.
it would be more convenient if you didn't have to highlight code for formatting. following the example of Prettier...
You are great, thanks for the extension.
Before format

After format

Duplicate with #44


Make maxWaitTimeSeconds configurable on AmazonSQSRequesterClientBuilder

Provide a way to shorten the default 20 second timeout waiting for a message.
Allows quick shutdown of client (e.g. in automated tests) when it is known that no more messages will arrive.

AWS CodeBuild CI Report

CodeBuild project: Java-Temporary-Client-CI
Commit ID: 179163e9e47524fa8ec735a7dba8b72b5db979d1
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


fix(parser): factory with prop access should be like css

my bad I misunderstood how the the styled.div({ ... }) usage, it should be like css not like cva
fixes this one https://github.com/chakra-ui/panda/pull/787

@astahmer there's no changeset

@astahmer there's no changeset

i'll update this one https://github.com/chakra-ui/panda/blob/30f41e01153c01cd69c27e10a27a5a226cf9eb56/.changeset/famous-students-reflect.md from the previous PR when we have a final decision regarding that API

closing this one as i'll treat it in this one https://github.com/chakra-ui/panda/pull/786 with the others new use cases


Remove problematic ciriquant-yml process

The process in question creates a YML file with absolute paths to working directories of other. I think this could have problematic effects with caching. Also, I made a change that makes the process more robust to problems as described in #109.

As this pipeline still has not had its 1.0 release, we haven't cared too much about a changelog yet. About official modules: There are several modules in this pipeline that could qualify to be added to the official modules, I am currently getting into module contributions and there will be PRs soonish.
Thanks for the review!


feat(clevercloud): add clevercloud secrets

I did add the clevercloud_sso_salt secrets across the deployment chain.
Related to https://github.com/octopize/avatar/pull/1368

Hello, i did take all your comments into account. I did also rebase.


lib license?

just wondering the license for the library, is GPL, MIT, something else?

This is WTFPL: https://github.com/PSP-Archive/LibPspExploit/blob/main/LICENSE.txt


Avoid localstorage

This new changes avoid the use of localstorage and remove the fake disconnect function for Metamask.
Added documentation and now the context handles the provider too.


Accept @vocab as part of the context

If we specify @vocab as part of the json-ld context, querying for bare property names should then be interpreted as belonging to the @vocab.
e.g. if the context is
{
    "@vocab": "http://www.example.org#"
}

then a query for "name @value" should be interpreted as targetting:
{
    "http://www.example.org#name": [ { "@value": "gabrisha" } ]
}

Fixed with 81be975d9286b42c56262d29e9a4e015cd7e9382


pass through LF compiler option

for the tsc option:
--newLine NEWLINE                  Specifies the end of line sequence to be used when emitting files: 'CRLF' (dos) or 'LF' (unix).

I added this to my tsconfig.json:
"compilerOptions": {
    "newLine": "LF"

however i still see lots of this on each commit of generated files:
warning: CRLF will be replaced by LF in bot1/src/mongo/MongoWrap-spec.js.
The file will have its original line endings in your working directory.

do these compiler options have to be explicitly passed through to tsc?
I added "foo": "bar" within atom, and got no errors, but running tsc at the command line did give:
error TS5023: Unknown compiler option 'foo'.

so I assume they are NOT being passed from atom and have to be added to atom-typescript somewhere...?

they are NOT being passed from atom and have to be added to atom-typescript somewhere...

Possible bug. Relevant code : https://github.com/TypeStrong/atom-typescript/blob/9a7680146e6b6ece5edaba3aebeeb9baa7b8a8cd/lib/main/tsconfig/tsconfig.ts#L266
up for grabs

Sadly I'm suddenly getting CRLF in my transpiled files and I don't know why.
I guess it started when I added some typings containing CRLF (borisyankov/DefinitelyTyped#5881) but now that I cleaned them, it keeps using CRLF.
I've tried setting the compilerOptions.newLine to "LF" with no luck either :(

Any news? I don't wanna see that CRs :(

seems that adding lineFeed: LF to tsconfig.js is being ignored by atom-typescript. I've confirmed the format of tsconfig.ts is correct by running tsc which correctly emits LF, but compiling via atom-typescript emits CRLF

bugs me, too.

I'm being stung by this too. :-(  Atom TypeScript apparently ignores the newLine option in tsconfig while tsc works as expected. This bug creates confusion about whether output files have really changed. However, the code pointed to above by @basarat does seem to do something with newLine which is in typescriptEnumMap, so it may not be something as simple as just an option not being copied over?

A dirty fix for 8.4.0 to use 'LF' is adding following this.getNewLine to file ~/.atom/packages/atom-typescript/dist/main/lang/core/languageServiceHost2.js around line 126
function LanguageServiceHost(config) {
    this.getNewLine = function() {return '\n';};
    var _this = this;

Then compiled files will end line with '\n'.

@liukun Your workaround suggestion is working for me. Thanks!


feat: support UDP in TURN

Add UDP option to TURN
GetLocalIPAddress default to IPv4

Nice have fun

99e9997d0afe87e3ee63cd7f70476e02e92bf6b6

Add UDP option to TURN
Works as STUN if server only opens port 443 but client's network allows hole punching
Expose non-443 port (less-overhead alternative with LB)
GetLocalIPAddress default to IPv4

@davidzhao Unlike TLS, can't run UDP port behind UDP proxy/LB. And network LB can't change UDP port, so the commit is updated to reflect this limit.

https://streamyard.com/4wb9z7pn7c


fix urllib3 version in cit to fix api 1.1 post_nodes as a temp solution

In Jenkins, "POST:/nodes/" of api 1.1 in CIT always fails with the error message "connection abort" in these two days.
It is found that urllib3 used by on_http_1.1 python client releases a new version yesterday (see https://pypi.python.org/pypi/urllib3). Fix the version used for cit to the older one solves the problem.
It can be a temp solution until we verify the newer version of urllib3 works for our system.

I know that this is a smoke/CI blocking issue, need more time to root cause. a story is created to root cause https://rackhd.atlassian.net/browse/RAC-4014
so the temporary solution is 👍  from me so that not block other PR smoke/CIT/FIT test.

BUILD RackHD #1187 : UNSTABLE

** BUILD build-config-setup  #1007: SUCCESS
** BUILD CIT  #591: UNSTABLE
** BUILD FIT  #459: SUCCESS

BUILD CIT  #591  Error Logs  ▼Test Name: patch_lookup
Error Details: ('Connection aborted.', BadStatusLine("''",))
Stack Trace: Traceback (most recent call last):
  File "/usr/lib/python2.7/unittest/case.py", line 331, in run
    testMethod()
  File "/usr/lib/python2.7/unittest/case.py", line 1043, in runTest
    self._testFunc()
  File "/home/jenkins/workspace/RackHD/build/test/.venv/on-build-config/local/lib/python2.7/site-packages/proboscis/case.py", line 296, in testng_method_mistake_capture_func
    compatability.capture_type_error(s_func)
  File "/home/jenkins/workspace/RackHD/build/test/.venv/on-build-config/local/lib/python2.7/site-packages/proboscis/compatability/exceptions_2_6.py", line 27, in capture_type_error
    func()
  File "/home/jenkins/workspace/RackHD/build/test/.venv/on-build-config/local/lib/python2.7/site-packages/proboscis/case.py", line 350, in func
    func(test_case.state.get_state())
  File "/home/jenkins/workspace/RackHD/build/test/tests/api/v1_1/lookups_tests.py", line 113, in patch_lookup
    Lookups().lookups_id_patch(self.id,body=self.patchedNode)
  File "/home/jenkins/workspace/RackHD/build/test/.venv/on-build-config/local/lib/python2.7/site-packages/on_http_api1_1/apis/lookups_api.py", line 528, in lookups_id_patch
    callback=params.get('callback'))
  File "/home/jenkins/workspace/RackHD/build/test/.venv/on-build-config/local/lib/python2.7/site-packages/on_http_api1_1/api_client.py", line 322, in call_api
    response_type, auth_settings, callback)
  File "/home/jenkins/workspace/RackHD/build/test/.venv/on-build-config/local/lib/python2.7/site-packages/on_http_api1_1/api_client.py", line 149, in __call_api
    post_params=post_params, body=body)
  File "/home/jenkins/workspace/RackHD/build/test/.venv/on-build-config/local/lib/python2.7/site-packages/on_http_api1_1/api_client.py", line 370, in request
    body=body)
  File "/home/jenkins/workspace/RackHD/build/test/.venv/on-build-config/local/lib/python2.7/site-packages/on_http_api1_1/rest.py", line 222, in PATCH
    body=body)
  File "/home/jenkins/workspace/RackHD/build/test/.venv/on-build-config/local/lib/python2.7/site-packages/on_http_api1_1/rest.py", line 143, in request
    headers=headers)
  File "/home/jenkins/workspace/RackHD/build/test/.venv/on-build-config/local/lib/python2.7/site-packages/urllib3/request.py", line 70, in request
    **urlopen_kw)
  File "/home/jenkins/workspace/RackHD/build/test/.venv/on-build-config/local/lib/python2.7/site-packages/urllib3/request.py", line 148, in request_encode_body
    return self.urlopen(method, url, **extra_kw)
  File "/home/jenkins/workspace/RackHD/build/test/.venv/on-build-config/local/lib/python2.7/site-packages/urllib3/poolmanager.py", line 244, in urlopen
    response = conn.urlopen(method, u.request_uri, **kw)
  File "/home/jenkins/workspace/RackHD/build/test/.venv/on-build-config/local/lib/python2.7/site-packages/urllib3/connectionpool.py", line 649, in urlopen
    _stacktrace=sys.exc_info()[2])
  File "/home/jenkins/workspace/RackHD/build/test/.venv/on-build-config/local/lib/python2.7/site-packages/urllib3/util/retry.py", line 347, in increment
    raise six.reraise(type(error), error, _stacktrace)
  File "/home/jenkins/workspace/RackHD/build/test/.venv/on-build-config/local/lib/python2.7/site-packages/urllib3/connectionpool.py", line 600, in urlopen
    chunked=chunked)
  File "/home/jenkins/workspace/RackHD/build/test/.venv/on-build-config/local/lib/python2.7/site-packages/urllib3/connectionpool.py", line 379, in _make_request
    httplib_response = conn.getresponse(buffering=True)
  File "/usr/lib/python2.7/httplib.py", line 1051, in getresponse
    response.begin()
  File "/usr/lib/python2.7/httplib.py", line 415, in begin
    version, status, reason = self._read_status()
  File "/usr/lib/python2.7/httplib.py", line 379, in _read_status
    raise BadStatusLine(line)
ProtocolError: ('Connection aborted.', BadStatusLine("''",))

test this please

BUILD RackHD #1188 : ABORTED

** BUILD build-config-setup  #1008: SUCCESS
** BUILD CIT  #594: ABORTED
** BUILD FIT  #462: ABORTED

Good Catch! Thanks @iceiilin  quickly fix for blocking smoke/CI issue

+1


Enable scanning all other supported files for trivy

For now, we are limiting trivy to scan only the terraform files. See here: https://github.com/coopnorge/engineering-docker-images/pull/2080
It would make sense to enable all other supported file types, but before that we need to resolve the following to make sure we comply or ignore the issues:
### Tasks
- [ ] AVD-DS-0026: Add HEALTHCHECK instruction in your Dockerfile
- [ ] AVD-DS-0002: Specify at least 1 USER command in Dockerfile with non-root user as argument
- [ ] AVD-DS-0013: MEDIUM: RUN should not be used to change directory (Dockerfile)
- [ ] AVD-DS-0029: ‘apt-get’ missing ‘–no-install-recommends’

The above are some of the common issues. There are many more in various repos.

We have hadolint and kubeconform for other things.


NavigationStack(path:root:destination:) not rendering root view when scoping the store into a new one.

Description
Hi all, first of all, great work on this library!
I wanted to implement navigation within the app I'm working on to learn/practice more about TCA, I implemented a new feature RootFeature that will hold the state of the stack, and added there the initial state of my top most feature to then scope it out and have it as the root of the NavigationStack(path:root:destination) view.
But when building the main feature using the RootFeature I only get a white screen, that does loads and sends the initial .onTask action but never updates anything else, even when printing the changes on the reducer I can see that there is no change on the state and no other action is sent even tho the .onTask sends an array of data.
However, if I create a new store and pass it to the MainView inside the root parameter of the NavigationStack I do get the screen displaying and all the functionality that I expect, but I haven't found a way of comunicating to the parent RootFeature and get the actions that then will trigger the navigation I have defined on the RootFeature.
I am unsure if this is something that I should be doing or not or instead focus on having all the navigation logic directly inside the feature that will be the "true" root feature, but I like having my features a bit more isolated, thats why having the RootFeature take care of all the details of the other screens/features makes sense to me, it is also how I understood the docs since the  examples use a simple NavigationLink as root (which to my app does not make sense to do that).
Checklist

[X] I have determined whether this bug is also reproducible in a vanilla SwiftUI project.
[X] If possible, I've reproduced the issue using the main branch of this package.
[X] This issue hasn't been addressed in an existing GitHub issue or discussion.

Expected behavior
The NavigationStack's root view can be created using a scoped version of the RootFeature's store and renders and works with all the defined actions.
Actual behavior
The NavigationStack's root view is not displayed and the console prints out no changes to the sate even when they are hard-coded as when the view loads.
Steps to reproduce
I have created a brand new project and recreated the issue there.
The Composable Architecture version information
1.9.2
Destination operating system
iOS 17
Xcode version information
15.0
Swift Compiler version information
swift-driver version: 1.82.2 Apple Swift version 5.9 (swiftlang-<IP_ADDRESS>.10 clang-15<IP_ADDRESS>)
Target: arm64-apple-macosx13.0

Hi @danielsan22, the problem is in the body of the RootFeature:
https://github.com/danielsan22/NavStackReproduction/blob/973142a5ef7a4e7153d7fbe2a6ef97bf7d315256/NavStackReproduction/RootView.swift#L19-L31
You aren't composing the MainFeature reducer into this at all. It should be more like this:
 var body: some ReducerOf<Self> {
+    Scope(state: \.cardsSets, action: \.cardsSets) {
+      MainFeature()
+    }
     Reduce { _, action in
         switch action {
         case .path:
             return .none
         case .cardSets:
             return .none
         }
     }
     .forEach(\.path, action: \.path) {
         Path()
     }
 }

Since this isn't an issue with the library I am going to convert it to a discussion. Please feel free to continue to discuss over there.


Use maps instead of lists in why command

fixes #2844

from 48s down to 7s on my PC (while dottrace attached)


Check for cache directory exists before write

awscli_bastion version: 0.8.3
Python version: 3.6.9
Operating System: Ubuntu 18.04

Description
Just installed and configured, when first login, the command fails:
Enter MFA code for arn:aws:iam::xxx:mfa/xx:

Error when retrieving credentials from custom-process: Traceback (most recent call last):
  File "/home/okelet/.local/bin/bastion", line 11, in <module>
    sys.exit(main())
  File "/home/okelet/.local/lib/python3.6/site-packages/click/core.py", line 764, in __call__
    return self.main(*args, **kwargs)
  File "/home/okelet/.local/lib/python3.6/site-packages/click/core.py", line 717, in main
    rv = self.invoke(ctx)
  File "/home/okelet/.local/lib/python3.6/site-packages/click/core.py", line 1137, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/okelet/.local/lib/python3.6/site-packages/click/core.py", line 956, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/okelet/.local/lib/python3.6/site-packages/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/home/okelet/.local/lib/python3.6/site-packages/awscli_bastion/cli.py", line 43, in get_session_token
    sts_creds = sts.get_session_token(mfa_code=mfa_code, mfa_serial=mfa_serial, duration_seconds=duration_seconds)
  File "/home/okelet/.local/lib/python3.6/site-packages/awscli_bastion/sts.py", line 85, in get_session_token
    self.cache.write(sts_creds)
  File "/home/okelet/.local/lib/python3.6/site-packages/awscli_bastion/cache.py", line 72, in write
    with open(self.bastion_sts_cache_path, 'w+') as f:
FileNotFoundError: [Errno 2] No such file or directory: '/home/okelet/.aws/cli/cache/bastion-sts.json'

What I Did
aws sts get-caller-identity --profile xxx

So that is a FileNotFoundError. Can you check that the /home/okelet/.aws/cli/cache directory exists? That should have been created for you.

Fix:
Create the missing directory: /home/okelet/.aws/cli/cache
Explanation:
So the cache directory is actually created and managed by the awscli when you assume a role. awscli-bastion just so happens to uses this directory but assumes that the awscli has already created it... but it probably wouldn't hurt to just create the cache directory if it doesn't already exist.
More Context:
When you specify a profile that has IAM role configuration, the AWS CLI will make an AssumeRole call to retrieve temporary credentials. These credentials are then stored (in ~/.aws/cli/cache). Subsequent AWS CLI commands will use the cached temporary credentials until they expire, in which case the AWS CLI will automatically refresh credentials.

https://docs.aws.amazon.com/cli/latest/topic/config-vars.html

Fix:
Create the missing directory: /home/okelet/.aws/cli/cache
Explanation:
So the cache directory is actually created and managed by the awscli when you assume a role. awscli-bastion just so happens to uses this directory but assumes that the awscli has already created it... but it probably wouldn't hurt to just create the cache directory if it doesn't already exist.
More Context:
"When you specify a profile that has IAM role configuration, the AWS CLI will make an AssumeRole call to retrieve temporary credentials. These credentials are then stored (in ~/.aws/cli/cache). Subsequent AWS CLI commands will use the cached temporary credentials until they expire, in which case the AWS CLI will automatically refresh credentials." and awscli-bastion works the same way.
https://docs.aws.amazon.com/cli/latest/topic/config-vars.html

Hi @aidanmelen , the directory didn't exist, I had to create it manually.


Add Infectious ( Original Mix ) by Tobu, 7obu

Song Name
Infectious ( Original Mix )
Artist Name
Tobu, 7obu
Source
Youtube
Youtube Link
qlinKNqHCyg
Direct File Link
No response
Song ID
593702
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 4363865

accept


Support for path-based environments

It seems like the path-based environments (i.e. -p) are not supported, are there any plans to support them? (if any, any name-based environment can be used in conda as a path-based one, but not vice versa).
It would be nice to at least be able to use "Conda Packages" functionality for the current conda environment, regardless of whether it's name or path based.

This is an interesting request... my first reaction would be how to "codify" the name of that path env... because the very same path thing is probably really long to be the name... @bollwyvl thoughts?

Hm... putting cross-platform file paths into URLs is always peril-fraught... though conda env names can be just about anything you want (spaces, unicode... just no os path specials) so i guess there's nothing too different there.
BUT this little extension doesn't want to be in the business of discovering non-$CONDA_PREFIX/envs-homed envs... if you want them found, the overhead of symlinking them to where conda looks for them is probably more useful than nb_conda maintaining some extra system for locating and referring to them.
That aside, the whole URL scheme would have to change to allow for it:

GET /environments/<env>/packages/ -> GET /environments/packages/<path>/<to>/<env>

Which is pretty gross. Further, since presumably the envs could be ANYWHERE, this sounds like we'd have to be careful about path traversal, etc. So I don't think we'd want to take that on right now.
The handling of this current env right here is possible, as I suppose one could exploit several of the above no-no characters, i.e. /environments/::current;;/, or use a convention like $CONDA_DEFAULT_ENV to ensure a semantic name no sane person would name a folder. We'd have to rethink how the things like listing worked, as it definitely uses conda env list under the hood, which wouldn't pick up /some/env even if it was active, I'm thinking. But perhaps worth exploring!

@bollwyvl Yea, supporting arbitrary path-based environments sounds like an overkill, but the kernel environment could indeed be referred by something like /environments/^__^::current::^__^/ as you suggested :+1:
The listing (in the tree plugin) wouldn't change though because path-based envs won't be listed as available kernels? It only becomes relevant when you're already running the kernel inside some conda env (regardless of how exactly you started it or whether it was name- or path-based).

Just a minor correction:

but the current kernel's server's environment could indeed be

This is the one that is available as Python [default]...

Thanks @bollwyvl for your feedback... in my first reaction I did not think about all the complexities with paths.

Hi everyone, any progress on this? We'd like to keep the conda-env location packed within the project directory to keep everything nice and together, rather than using the default environment locations.
What about escaping the slashes of the prefix path in the URL? / > %2F. I could try it out and submit a merge request next month if desired.


Fluent model interface getter impl should also look for non-flattened property

Try generating from Logic api-vrsion 2018-07-01_preview

Closing out old issues - feel free to reopen if it's still an issue.


python: update release instructions & automation

Rationale for this change
Automate and document the PyPI release process.
closes https://github.com/apache/arrow-datafusion/issues/1298.
What changes are included in this PR?

Python artifact automation: dev/release/download-python-wheels.py
Docs for PyPI release

Are there any user-facing changes?
no

👍


Update nvm.sh

Add prompt and return non-zero when there is no binary for node version greater than v1.0.0
Currently, on FreeBSD, if we try nvm install 4.2, we will only get:

Installing node v1.0 and greater from source is not currently supported

In fact I had no idea what's wrong because I didn't use -s parameter for it,
and then I found that there are two reasons:

There was no pre-built nodejs binary from official for FreeBSD, which means we need to build from source.
nvm doesn't support build for v1.0 and greater version from source yet.

So I think there should be a prompt message to tell the user about the reason why nvm will go to build from source, no matter it will succeed or fail.

This seems like a good approach.

Updated!

Your description says "return nonzero" but this PR doesn't achieve that - I think just the error message is sufficient tho. Do you agree?

@ljharb my fault, that's the previous version of that commit before review, thanks for your prompt reply.

@ljharb is this good to merge? Thanks.

The tests were failing because of an SSL issue with iojs.org, but I've now disabled them in latest master. Can you rebase, and once the tests pass, I'll be happy to merge it?

Of course, give me few mins.

@ljharb Updated!


chore: upgrade cli

What does this change?

Multiline card title (https://github.com/reg-viz/reg-cli-report-ui/pull/30)

:dog2: :dash:
References

https://github.com/reg-viz/reg-cli-report-ui/pull/30

Screenshots
Nothing.
What can I check for bug fixes?
Nothing.

Thx


Consider adding CurSearch highlight

Hi Folke, CurSearch can highlight search match under the cursor.
:h 'hlsearch':
	When there is a previous search pattern, highlight all its matches.
	The |hl-Search| highlight group determines the highlighting for all
	matches not under the cursor while the |hl-CurSearch| highlight group
	(if defined) determines the highlighting for the match under the
	cursor. If |hl-CurSearch| is not defined, then |hl-Search| is used for
	both. Note that only the matching text is highlighted, any offsets
	are not applied.

I think it could be as simple as linking it to IncSearch which was considered as a default by upstream I believe (might be recalling wrong now though).
Would you want me to create PR for that?

Didn;t know about that one. Fixed!

It's pretty new. Thank you, that was fast!


improve calendar layout

close #472
I decided to go for a GCal-styled view:

On mobile it still looks quite shitty:

I think we should also GCal here, in the app they use the color as the background color for the whole div and don't display the indicator and the start time. @felixrindt could you help a bit there? not quite sure how to achieve that

Coverage: 87.961% (+0.01%) from 87.95% when pulling 302a25ce0386abec21afb9275ab088bb96825ed0 on calendar-layout into bac42f65edfc5d58779c5a658cc6604978916d9b on main.


Refactor and update input configuration.

Updated the input configuration. Configuration is initially set through the editor. Once the node is deployed, configuration can be set dynamically from input. If any parameters from the input message are empty, the previously set configurations are used.
Refactored the code.

Updated content, and tested inputs and outputs.


Added docs about granting initial permissions

Relates to #628
Added a description of use case with granting system permissions while starting the app (before runApp/pumpWidgetAndSettle)

Hello @bartekpacia! Let me know if it looks fine :)

Looks great! Thanks a lot for your contribution :)


Capacitor.js support

Test if the plugin works in Capacitor.js https://capacitorjs.com/
If the plugin is not working, make it work
Declare as Capacitor.js ready
????
Profit

After some consideration, Capacitor.js support is not needed at the moment (on the javascript layer), and if there will be such a request in the future, it will need to be solved in a separate plugin.

Reopening as it surfaced again - will need to deal with this.
https://capacitorjs.com/docs/cordova

Capacitor.js is not compatible due to: https://capacitorjs.com/docs/plugins/cordova#variables-and-hooks

@alesmraz Yes, we currently do not support Capacitor. 😞 We will have a look at it, but it will take us a bit longer to address, and we will unlikely make it this year. I would recommend looking at the native SDKs instead, since Capacitor is more friendly towards native modules when compared to Cordova.


Sysctl idempotency check needs to consider the existence of the sysctl.d file, not just the value

Description
The idempotency check in the sysctl resource only compares the existing value of the sysctl with the desired value.  If they match, it does nothing.  However, if the sysctl has also been changed outside of chef-client then the values may match, the /etc/sysctl.d file will not be written, and on a reboot the value will revert to whatever it was before.
If the /etc/sysctl.d file does not exist, then that should trigger the resource to fire and setup the /etc/sysctl.d file, even though the running value will not change.
Chef Version
chef-client 14.1.12
Platform Version
RHEL 7.5
Replication Case

Assume there is a sysctl parameter "foo" with a default value of "0"
Set foo to "1" outside of Chef using sysctl foo=1
Run a cookbook that contains the resource sysctl('foo') { value(1) }
Reboot the system.

When the system comes back up, I expect that the sysctl parameter foo will retain the value configured by Chef (foo=1), however, it actually will change to its default value (foo=0).
I originally reported this issue against the sysctl cookbook.  I am re-reporting it here now that the sysctl resource is part of core Chef.

@Poohblah Thanks for the report. The fix for this will go out in Chef 14.4.

Thank you!


Update to 2.13.6

Updating the version to use 2.13.6

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

@just-boris can we look at this?

I do not have access to Jenkins infra. Need @baev help

Published the latest 2.13.8
Sorry for the delay!


Fix type error in python_config.sh

In the getsitepackages code path,
library_paths is a list.
Make the distutils code path a list as well.
Remove extraneous whitespace while we're here.
Without this, running ./configure:
$ ./configure
Please specify the location of python. [Default is /Users/josh/.virtualenvs/tf/bin/python]:
Do you wish to build TensorFlow with Google Cloud Platform support? [y/N] n
No Google Cloud Platform support will be enabled for TensorFlow
Traceback (most recent call last):
File "", line 18, in
TypeError: can only concatenate list (not "str") to list
Found possible Python library paths:
Please input the desired Python library path to use.  Default is []
ln: util/python/python_lib: Invalid argument

Can one of the admins verify this patch?

@itsmeolivia PTAL

LGTM

@tensorflow-jenkins Test this please


Long Parameter list

It’s hard to understand long list of parameters, which become contradictory and hard to use as they grow longer. It is better to "Introduce a Parameter object".
https://github.com/yahya-abdul-majeed/CodeRefactoring2023/blob/13fd912b300904b6ff8bfb51da2be73ad1126bca/CLabManager-Web/Areas/Admin/Controllers/IssuesController.cs#L21

0,2


Update regularizers.md

keras.regularizers.l1_l2() takes two arguments. The doc was misleading.

Fixed, @fchollet.


EMF file stream does not convert properly

I have a word document that has an EMF image file in it. I have to read that image file and save it in the system. Following is the code that I am using:
  var lstImages = selectedImage.Descendants < DocumentFormat.OpenXml.Drawing.Pictures.Picture > ();

  if (lstImages != null && lstImages.Any())
  {
    foreach(var image in lstImages)
   {
      string imageFileName = image.NonVisualPictureProperties?.NonVisualDrawingProperties?.Name;
      string blip = image.BlipFill.Blip.Embed.Value;
      ImagePart img = (ImagePart) wordDoc.MainDocumentPart.GetPartById(blip);
      GetExtension(ref fileName, imageFileName, img);
      string binaryDateTime = _dateTimeHelper.GetDateTimeUtcNow().ToBinary().ToString();

      var stream = img.GetStream();
      var byteStream = new byte[stream.Length];
      int length = (int) stream.Length;
      stream.Read(byteStream, 0, length);
      stream.Dispose();
      var uploadPath = "D://ImageFile//"

      if (!_directoryManager.Exists(uploadPath)) {
        _directoryManager.CreateDirectory(uploadPath);
      }

      fileName = string.Concat(binaryDateTime, "_", fileName);

      string completePath = string.Concat(uploadPath, fileName);

      using(var fileStream = new FileStream(completePath, FileMode.OpenOrCreate)) {

        // Write bytestream to disk
        fileStream.Write(byteStream, 0, length);
        fileStream.Close();
      }
    }
  }

The saved file when being open says the file is damaged. Can you guide what is it that I am doing wrong or is there an easier way to save or convert EMF file as png?

Hi @SanaWaseem ,
The SDK does not save files itself, that's up to the implementation, so I recommend posting this to stackoverflow.
That said it might be simpler to use the Stream.CopyTo method instead of creating a byte array. Something like this:
using (var fileStream = new FileStream(completePath, FileMode.OpenOrCreate))
{
    stream.Seek(0, SeekOrigin.Begin);
    stream.CopyTo(fileStream);
    fileStream.Close();
}

Hi
I am implementing a similar thing that saves embedded images as TIFF.
My implementation uses the Magick.Net nuget package to convert image to new file format.
Here's link to github:
https://github.com/dlemstra/Magick.NET
I suggest you add the same package to perform your conversion from .emf to .png before saving the stream to a file.
Feel free to reclose this issue, I just wanted to add my experiences.


Save button is missing Settings page (0.45)

Describe the bug
The Save , Back and Clear snapshot history buttons are missing in the Settings page.
Version
v0.45
To Reproduce

Install v0.45
Go to Settings page

Expected behavior
Previous versions had the button
Screenshots

Additional context
Found when updating the Cloudron package to latest release

aaaah thanks for this, there was a hanging/open div - building new release now 0.45.1

Thanks for the super fast reaction, the button is back and our package tests succeed again now.

Hello, I just installed this program and I was confused why it wasn't saving the settings :D
How can I update it if I installed it via Python Pip on Windows? I ran that upgrade command in the bottom of the wiki page, but it still says v0.45.
(I'm not a programmer so sorry if it's something obvious :) )

@gramakri 0.45.1 now on docker hub and GH, can you confirm it works?

@dgtlmoon yes, works great. This issue can be closed. (@nebulade and I are together in cloudron)


Implement gelu (and other elementwise fusions) recomputation during backward

🚀 Feature

Motivation
import torch
import thunder

def f(x):
    x = x @ x
    x = torch.nn.functional.gelu(x, approximate="none")
    x = x @ x
    return x

x = torch.randn(1000, 1000, device="cuda", requires_grad=True)
jf = thunder.jit(f)
y = jf(x)

print([t.shape for t in y.grad_fn.saved_tensors])
# This will print:
# [torch.Size([1000, 1000]), torch.Size([1000, 1000]), torch.Size([1000, 1000])]

print(thunder.core.vjp_utils.get_saved_for_backward_tensors(thunder.last_traces(jf)[-1]))
# This will print:
# (<TensorProxy(name="t0", dtype=thunder.dtypes.float32, shape=(1000, 1000))>,
#  <TensorProxy(name="t5", dtype=thunder.dtypes.float32, shape=(1000, 1000))>,
#  <TensorProxy(name="x", dtype=thunder.dtypes.float32, shape=(1000, 1000))>)

In the above snippet t5 is the output of the gelu function and the request is to implement a pass that forces recomputation of the gelu function in the backward pass instead of saving this intermediate tensor.
Ongoing PR: https://github.com/Lightning-AI/lightning-thunder/pull/1003.
Implementing gelu recomputation would resolve the OOM error seen in https://github.com/Lightning-AI/lightning-thunder/issues/246.

Sounds resonable! Does this imply that recomputing the other tensors in that particular model configuration does not help?My guess would be that it doesn't but then my second question is: why the gelu in particular?

Another potential fix for this might be the use of Liger kernels for rope. I think that's worth trying but it requires creating a quick and dirty executor. Let's proceed with the remat for now and later I'll come back to try this out


How to modify Chinese characters font?

I can modify the number and English words font, but can't modify Chinese characters, can you help me?

Unless you have a font that supports those characters - I don't actually know what happens!


The counter documentation does not help with multi-level counters

Description
I needed a multi-level counter, but I found that the counter documentation only really discusses single-level counter, which made it much harder than it should have been.
I wanted to index "questions" with numbers of the form S.C, where S is the section counter and C is a question-specific counter. (For example: 1.1, 1.2, then 2.1). Here is the code that I ended up writing:
#let question_counter = counter("question")
#question_counter.update((0, 0))
#show heading: it => {
  if it.level == 1 {
    question_counter.update((s, c) => (s+1, 0))
  }
}

#let question(body) = {
 question_counter.update((s, c) => (s, c+1))
 strong([ Question #context{ question_counter.display() } ])
 body
}

= Example usage
#question[Foo]
#question[Bar]

= More example
#question[Baz]

This required too much trial-and-error, I believe that the counter documentation should be improved to make this easy. For example:

There are no examples of update with multi-level counters.
The behavior of step on multi-level counters is not explained.

Does the indexing start at 0 or 1?
Answer: it appears to start at 1.
is question_counter.step(level: 1) equivalent to question_counter.update((s, c) => (s+1, c)) or to question_counter.update((s, c) => (s+1, 0))?
Answer: in my tests, neither, it appears to be unusable with 2-level counters.

I get weird errors if I remove the #question_counter.update((0, 0)) initialization, why? Is there a way to indicate that a counter is multi-level at creation time?

My understanding is that the currently intended specification for the step functions on multi-level counters is given by those tests:
https://github.com/jbirnick/typst/blob/d4cd60a70d3eb7601622aafffb0479ca07947e66/tests/suite/introspection/counter.typ#L148-L158
In particular, note that if you have a two-level counter (s,c), the count.step(level: 1) results in the counter (s+1,) and not (s+1, 0) as one might expect. In other words, the number of levels will change dynamically during execution. I am not sure how user are expected to conveniently program with this specification.

In other words, the number of levels will change dynamically during execution.

The reason for this is heading numberings such as 1, 1.1, 1.2, 2, 2.1 have varying number of levels.

@gasche You can just search on Typst Universe for "counter", you will find that rich-counters and headcount do exactly what you are looking for, hopefully this makes it simpler for you.

Thanks for the pointers! My main point is that the documentation should be clarified because currently the behavior is under-specified and this makes scripting un-necessarily hard.

As Laurenz said, the reason for that is that you want e.g. to display something like "Chapter 2", for which you need a counter value of (2,); having a counter value (2,0,) would display "Chapter 2.0", meh.

... or you could decide that .display() ignores trailing zeroes.
I'm not arguing strongly in favor of one or the other behavior (I just remark that the current one makes it hard to script with counters), but I think that whatever is the intended semantics should be documented.
Also, in general I suspect that forcing oneself to include short script snippets would help improve the API, by nudging the design in directions that are easier to script.

if you want to change [the documentation], it would be helpful if you make concrete suggestions

My opening message has three specific questions (in fact four, as one has two sub-questions), and my concrete suggestion is to update the documentation so that it contains answers to these questions.

How counter works is supposed to help you scripting by incorporating the "step algorithm", so that you only have to call step() and don't need to do manual coding and case distinctions. If you want something which you can use fully "manually" then it's state. Counter is in some sense just a little extended/modified state, if you don't like the abstraction that's fine, you can just use state then and you could even publish a package that makes counters that behave the way you want!
I can try to improve the documentation soon but I don't have time these days.


Wcostream is not working

Steps to reproduce
1.trying to uninstall the app
Expected behavior
video list is working
Actual behavior
It's keeping sayvideo list empty error?
Crash logs
No response
Animiru version
App version: <IP_ADDRESS>
Android version
Motorola
Device
Motorola
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

If this is an issue with an extension, I should be opening an issue in the extensions repository.

You didn't read this, did you?


fix: add owner field if owner auth is included in auth rules

Issue #, if available:
https://github.com/aws-amplify/amplify-js/issues/5687
Description of changes:
Add owner field to generated schema if it does not exist in the original schema but owner auth is included in auth rules.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Working as expected with a simple owner auth schema:
type Post @model @auth(rules: [{ allow: owner }]) {
  id: ID!
  title: String!
}

However, subscriptions are failing with multiple owner auth rules, e.g.
type Post
@model
@auth(
  rules: [
    { allow: owner, ownerField: "owner" }
    { allow: owner, ownerField: "editors" }
  ]
) {
  id: ID!
  title: String!
  owner: String
  editors: [String]
}

They're failing with the current CLI release as well though (not just this PR). @yuth when you have a moment, let's discuss what we're expecting the generated subscriptions to look like for the 2nd schema. Maybe the bug for this is on the library side.


"Gfk" company- the scraper does not work

URL: https://admin.peviitor.ro/
Browser: Chrome
Device: Laptop
OS: Windows 11
STEPS TO REPRODUCE:

Open the URL in the browser
Search for company "Gfk"
Click on "Vizualizeaza joburi"

EXPECTED RESULTS:
The scraper should run without issues.
ACTUAL RESULTS:
The scraper is no longer working since August 18.

compania a fost stearsa din validator si nu a mai reaparut ....


Can't resolve 'react-bootstrap/lib/FormControl'

Seems like it doesn't work with React v16.3.2

You need to install peer dependency ‘react-bootstrap’

Yeah, that works. Thanks!

this is not working again with the react-bootstrap dependency

@alyssa1995 I'll update the package once react-bootstrap 1.0.0 is released. I am not supporting alpha and beta


chore(*) release 2.7.0

What this PR does / why we need it:
Commits 2.7 changes to main.
Which issue this PR fixes:
Part of #2973

Conformance-tests failed 🤔

I've re run the tests and now it works.


Include line diagnostics without a range in cursor diagnostics

I've found with some LSP servers they return diagnostics without a range. Sometimes the start and end char are 0 and in other cases they are the same value.
I have these reports in the gutter so I can see the line has an error however I have no way to view it (without using show_line_diagnostics).
I'm proposing to just include single line diagnostics with a 0 character range in all cursor diagnostics for that line.

can you update with the new diagnostic api in neovim 0.7?

@tapayne88 can you post a min code that can reproduce this error

With this minimal config you should see the above error. You might need to open neovim with it (nvim -u /path/to/minimal.lua) and wait for the install to happen, then close and reopen.
-- To make use of this file run the following
-- nvim -u ~/.config/nvim/minimal.lua ...
vim.cmd [[set runtimepath=$VIMRUNTIME]]
vim.cmd [[set packpath=/tmp/nvim/site]]

local package_root = '/tmp/nvim/site/pack'
local install_path = package_root .. '/packer/start/packer.nvim'
local function load_plugins()
  require('packer').startup {
    {
      'wbthomason/packer.nvim',
      {
        'williamboman/nvim-lsp-installer',
        requires = { 'neovim/nvim-lspconfig' },
        config = function()
          require('nvim-lsp-installer').setup {
            ensure_installed = {<EMAIL_ADDRESS>},
            automatic_installation = true,
            install_root_dir = '/tmp/nvim/lsp_servers',
          }
          local lspconfig = require 'lspconfig'
          lspconfig.sumneko_lua.setup {}
        end,
      },
      {
        'tapayne88/lspsaga.nvim',
        branch = 'line-diags-without-ranges',
        config = function()
          require('lspsaga').init_lsp_saga()
        end,
      },
    },
    config = {
      package_root = package_root,
      compile_path = install_path .. '/plugin/packer_compiled.lua',
      display = { non_interactive = true },
    },
  }
end

_G.load_config = function()
  require('nvim-lsp-installer').setup {}
end

if vim.fn.isdirectory(install_path) == 0 then
  print 'Installing packer packages...'
  vim.fn.system {
    'git',
    'clone',
    '--depth=1',
    'https://github.com/wbthomason/packer.nvim',
    install_path,
  }
end

load_plugins()
require('packer').sync()
vim.cmd [[autocmd User PackerComplete ++once echo "Ready!" | lua load_config()]]

If you open the diagnostic.lua file, wait for the language server to start and run :Lspsaga show_cursor_diagnostics on line 2 you should see the error.

Hope this helps!

yup


mtlnovel chapters paragraph order

Source information
mtlnovel
Steps to reproduce
read chapter
Expected behavior
paragraphs are displayed in normal order
Actual behavior
mtlnovel detects that you are using a plugin and makes the order of the paragraphs incorrect, making it poorly readable and out of order

LNReader version
2.0.0
Android version
13
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 1.1.19.
[X] I have updated all installed extensions.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

It's just a message to scare you to use their website, it's baked into the chapter text so I don't think removing them could be possible

Organized?


Don't get a lock when querying the parent reseed_count (alternative version)

This is an alternative fix to #20944.
This enables the parent reseed_count to be queried without a lock and only requires changes to our own DRBG implementations. It makes no change to the EVP_RAND layer and nor does it add or amend the libcrypto->provider interface. So, there are no API or behaviour changes required for this.
Previously the EVP layer would call lock and unlock functions on the underlying DRBG implementation to say when a lock should be acquired and released. This gives the DRBG implementation no say as to what kind of lock should obtained (e.g. read/write) or even whether a lock is actually needed or not.
In reality we know whether a DRBG is supposed to be in locking mode or not because the "enable_locking()" function will have been called if locks should be used. Therefore we re-interpret the lock and unlock functions as "hints" from the EVP layer which we ignore. Instead we acquire locks only when we need them. By knowing the context we can obtain either a read or a write lock as appropriate.
This may mean that in some rare cases we acquire the locks more than once for a single EVP call, if the EVP call makes several calls to the underlying DRBG. But in practice almost all EVP calls only make one such call. EVP_RAND_generate() is an example of a call where multiple DRBG calls may be made. One of these gets the "max_request" parameter (which is constant for all of our own DRBGs) and it may make several calls to the DRBG generate call - but only if the requested size is very large which will rarely be the case.
Even if a DRBG has locking enabled on it, there are certain parameters which are still safe to obtain even without a lock. The max_request value is constant for all our DRBGs. The reseed_counter does not matter if we get it wrong - so it is safe to avoid the lock. So if all we are reading are those parameters then we take no lock at all.
Partially fixes https://github.com/openssl/openssl/issues/20286

Needs some docs I think.

Also adding an OTC hold here: are we willing to accept non-thread safe access to the single primary DRBG instance?

I've added some performance figures for this PR here

OTC: master only as there is some risk involved and the patch is invasive.

Can the OTC explain the risk?

Is it reasonable to no longer export the lock and unlock RAND calls?
i.e. remove them from the dispatch table.
Callers should be null checking these before calling them.
We should, at the minimum, deprecate them.

I don't think we can remove them any time soon. DRBGs may already have been written that rely on this. We're probably stuck with them for now. We could possibly deprecate them with a view to removing them in the future. On the other hand they are harmless. Another PR in any case.

Ping for second review

Is it reasonable to no longer export the lock and unlock RAND calls?
i.e. remove them from the dispatch table.
Callers should be null checking these before calling them.
We should, at the minimum, deprecate them.

I don't think we can remove them any time soon. DRBGs may already have been written that rely on this. We're probably stuck with them for now. We could possibly deprecate them with a view to removing them in the future. On the other hand they are harmless. Another PR in any case.

I think we need some kind of policy here.
We can't get rid of having these APis defined and making an attempt to call them if they are non-NULL.  That would be a breaking change and would have to go via the deprecation process.
Callers really should be null checking all provider API function pointers and not calling them if they are NULL.  If this is being done, removal of these from the dispatch table is harmless.

and merged!

@paulidale @t8m
Will this commit be picked into the 3.1 or 3.0 branch?

OpenSSL Technical Committee (OTC) discussed this and decided that it should be for master only and not backported:
https://github.com/openssl/openssl/pull/20970#issuecomment-1568066797


logcheck: 1.3.17 -> 1.3.18

Motivation for this change
Update
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

The corresponding module seems to be broken:
nixos/modules/services/logging/logcheck.nix
This module also still uses cron instead of systemd timer.

Hmm that is weird but ok.
Sent from my iphone
On Mar 2, 2017, 3:20 PM -0600, Bjørn Forsman<EMAIL_ADDRESS>wrote:

@bjornfor commented on this pull request.
In pkgs/tools/system/logcheck/default.nix (https://github.com/NixOS/nixpkgs/pull/23317#discussion_r104032555):

@@ -1,13 +1,12 @@ { stdenv, fetchurl, lockfileProgs, perl, mimeConstruct }: stdenv.mkDerivation rec { - _name = "logcheck"; - version = "1.3.17"; - name = "${name}-${version}"; + version = "1.3.18"; + name = "logcheck${version}";

The hyphen is needed for Nix to know what part of "name" is the package name and what is the version number.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://github.com/NixOS/nixpkgs/pull/23317#discussion_r104032555), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAHL77yn7aS_V_7gw6RMqVanyYiO8CANks5rhzKGgaJpZM4MPIfJ).

@Mic92 I will try to look at the module too. Even though just trying to update packages to latestbin repo 👅

@Mic92 I don't think I can fix the logcheck module, as I don't really understand it greatly


[THORN-2162] Dependencies and version updates to improve PME results

commons-configuration 1.10 is productized, 1.8 not, minor version update, I saw no issues with the MP FT tests

Failing test on Windows (org.eclipse.microprofile.config.tck.ConfigProviderTest.testEnvironmentConfigSource) is unrelated and happens for a while now, merging.


Github Issue

We are facing problems while installing the packages.

You can use npm install @sse-auth/github --force to install the packages. We will fix this issue in the newer version.
Thankyou for informing it.


fix: use correct paths for pyproject and related files

Python bundling for sst v3 fails because the target folders for copying files are wrong. They append the absolute path to the target directory. Something like:
/Users/myuser/my/project/path/.sst/artifacts/ChatApiRouteWwasrbHandler-src/Users/myuser/my/project/path/filename

I cannot get the simple examples/aws-python-container to work given this MR.
Error:
container/.sst/artifacts/MyPythonFunction-src/Dockerfile: no such file or directory
I see in fact that the Dockerfile is at MyPythonFunction-src/src/Dockerfile not MyPythonFunction-src/Dockerfile.
Does work on v3.22.5

This is also causing issues for us, same error as above "Dockerfile: no such file or directory"

Open a new issue with your project structure?


Enable more flexible killing of jobs

Problem Statement
Killing all running jobs of a certain user or all jobs that match a certain pattern (like all SELECT statements) is currently not easily possible.  One needs to manually gather all job ids and send a sequence of KILL statements.
Possible Solutions
Support a function (like PostgreSQL's pg_cancel_backend) so you can do something like:
SELECT kill(id)
FROM sys.jobs
WHERE stmt LIKE '%my_table%'
  AND username = 'u';

Considered Alternatives
No response

Another alternative solution could be to support a generic function to run arbitrary commands against a foreign server, we could then configure a foreign server pointing to the same CrateDB instance and use it to run the dynamically generated KILL statements.

Could you elaborate a bit more on why there's a need to kill jobs like that?

One example is the abortion of a benchmark run consisting of many queries, and you want to kill those before starting another run. The benchmark tool itself didn't manage to kill them itself. KILL ALL cannot be used, as there is also parallel ingestion which should remain running. As the benchmark is using a particular user, it is handy to kill all queries of that user.

I'm only guessing here, but I could see it being handy for a CrateDB user that connects to the DB with different users, one doing insertion, one doing dashboard queries, one doing heavy analytic queries, etc. and you want to kill for example all heavy analytic queries that are saturating the cluster under some heavy ingest load peak.
Could also be that you want to do something similar based on sql statements filtering, if for example someone is experimenting (on a prod cluster) with new analytic queries that are long-running/resource-consuming, and you want to kill them.


Running the app gives error Warning: Error in loadNamespace: there is no package called ‘nhsrtheme’ in the chart

I searched for references to {nhsrtheme}, and found only these 2. Does {nhsrtheme} need added to the imports or depends perhaps?
EDIT: I see that having {nhsrtheme} specified in the remotes is supposed to handle importing it. So I am not sure what is causing the error after all.

Not sure how to get around this one other than devtools::install_github("nhs-r-community/nhsrtheme")... if you can figure it out please create a PR as it's a bit frustrating!!

I think I have just about figured this issue out. Firstly, can use {drat} to create a private repo with the zipped packages in it. Then using the additional_repositories option in DESCRIPTION file you just give the URL for that drat repo.
For actually making sure any non-CRAN dependencies are pulled in when deploying to shinyapps.io, you HAVE to do the devtools::install_github()– turns out rsconnect works out how to get non-CRAN packages by how you install them locally. But it does not recognise when drat has been used!
I got it all working last night, so will have a look into a PR 😊
[Description: NHSBSA Email signature (V1) 08]
Stella House, Goldcrest Way, Newburn Riverside, Newcastle upon Tyne, NE15 8NY
Please read our email disclaimer online at: http://www.nhsbsa.nhs.uk/email.
To reduce our environmental footprint, please only print when necessary.
From: adamivisonnhs @.>
Sent: 04 November 2021 09:36
To: nhsbsa-data-analytics/nhsbsaR @.>
Cc: MCPHERSON, Mark (NHS BUSINESS SERVICES AUTHORITY) @.>; Author @.>
Subject: Re: [nhsbsa-data-analytics/nhsbsaR] Running the app gives error Warning: Error in loadNamespace: there is no package called ‘nhsrtheme’ in the chart (Issue #7)
Not sure how to get around this one other than devtools::install_github("nhs-r-community/nhsrtheme")... if you can figure it out please create a PR as it's a bit frustrating!!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/nhsbsa-data-analytics/nhsbsaR/issues/7#issuecomment-960592420, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVX2E2XYLSQKH2S3CC4FU2LUKJHXNANCNFSM5HCSTZ3Q.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
This message originated from outside of NHSmail. Please do not click links or open attachments unless you recognise the sender and know the content is safe.

This message may contain confidential information. If you are not the intended recipient please:
i) inform the sender that you have received the message in error before deleting it; and
ii) do not disclose, copy or distribute information in this e-mail or take any action in relation to its content (to do so is strictly prohibited and may be unlawful).
Thank you for your co-operation.
NHSmail is the secure email, collaboration and directory service available for all NHS staff in England. NHSmail is approved for exchanging patient data and other sensitive information with NHSmail and other accredited email services.
For more information and to find out how you can switch visit Joining NHSmail – NHSmail Supporthttps://support.nhs.net/article-categories/joining-nhsmail/

Think I have found maybe a simpler solution? I think that the package needs specifying in Imports as well as Remotes... let me check now
https://cran.r-project.org/web/packages/devtools/vignettes/dependencies.html
This is a common theme with other packages / apps in development using {nhsbsaR} so hope this is the solution!

Try this @MarkMc1089
detach("package:nhsbsaR", unload = TRUE)
detach("package:nhsrtheme", unload = TRUE)
devtools::install_github(repo = "nhsbsa-data-analytics/nhsbsaR", ref = "7-fix-non-cran-dependency-issue")

Unfortunately didn't work:
`> remove.packages("nhsbsaR")
Removing package from ‘C:/Users/Mark/Documents/R/win-library/4.1’
(as ‘lib’ is unspecified)

remove.packages("nhsrtheme")
Removing package from ‘C:/Users/Mark/Documents/R/win-library/4.1’
(as ‘lib’ is unspecified)
detach("package:nhsbsaR", unload = TRUE)
Error in detach("package:nhsbsaR", unload = TRUE) :
invalid 'name' argument
detach("package:nhsrtheme", unload = TRUE)
Error in detach("package:nhsrtheme", unload = TRUE) :
invalid 'name' argument
devtools::install_github(repo = "nhsbsa-data-analytics/nhsbsaR", ref = "7-fix-non-cran-dependency-issue")
Downloading GitHub repo nhsbsa-data-analytics/nhsbsaR@7-fix-non-cran-dependency-issue
These packages have more recent versions available.
It is recommended to update all of them.
Which would you like to update?

1: All
2: CRAN packages only
3: None
4: cpp11 (0.4.0 -> 0.4.1 ) [CRAN]
5: cli   (3.0.1 -> 3.1.0 ) [CRAN]
6: broom (0.7.9 -> 0.7.10) [CRAN]
Enter one or more numbers, or an empty line to skip updates:
√  checking for file 'C:\Users\Mark\AppData\Local\Temp\RtmpeodwNO\remotes2ab462f21dc3\nhsbsa-data-analytics-nhsbsaR-56ab594/DESCRIPTION' ...

preparing 'nhsbsaR':
√  checking DESCRIPTION meta-information ...
checking for LF line-endings in source and make files and shell scripts
checking for empty or unneeded directories
Omitted 'LazyData' from DESCRIPTION
building 'nhsbsaR_<IP_ADDRESS>00.tar.gz'

Installing package into ‘C:/Users/Mark/Documents/R/win-library/4.1’
(as ‘lib’ is unspecified)
ERROR: dependency 'nhsrtheme' is not available for package 'nhsbsaR'

removing 'C:/Users/Mark/Documents/R/win-library/4.1/nhsbsaR'
Warning message:
In i.p(...) :
installation of package ‘C:/Users/Mark/AppData/Local/Temp/RtmpeodwNO/file2ab42a5d2911/nhsbsaR_<IP_ADDRESS>00.tar.gz’ had non-zero exit status`


Clean up some logs, old unused code, and embarassing mistakes

lol, I meant my embarrassing mistakes - that exports was all me, I'm pretty sure.


Fix Copybara import conflicts

@qstanczyk
Hi Piotr! I've reverted some changes that were made with the recent commit and updated Changelog and vcpkg baselines.
.github directory and requirements.txt are moved from the gfootball back to the root of the repository. .github has to be located in the root for CI to work. And I also decided to put requirements.txt next to setup.py, because they have to be updated at the same time. It also might be very handy for many IDEs: they automatically install the dependencies for the project into the virtual environment if they find requirements.txt in the root. If there is any specific reason, why it shouldn't be in the root directory - it's fine, but we need to update line 38 in the workflow file.
I also updated hash values for rendering tests. x86 and x64 builds have different values, and because our current CI pipeline uses windows 2016 with VS Studio 2017's SDK, we temporarily have to check for those values too.
Once you merge the PR, please publish a new release v2.10.1 to trigger the CI workflow.
By the way, since you have already incorporated the changes, the PR #287 can be closed now. (Closes #287)

Thanks a lot Victor. I have merged this PR.

... and build the release.


RFC Layer Update With Index Update

For now, I just did the StaticImageLayer to give a sense of what this sort of thing would look like.  I can do VivViewerLayer next once we settle on nomenclature, I think.  It might be a litttle more complicated though since so much happens in deck.gl that is out of our control.  This hopefully will take care of the layers' side of things in #109.
Thoughts?  We can also skip VivViewerLayer for now just to get a MVP out the door for you.

The solution will be to add a copying paradigm to the calling application a la:
const handleChannelChange = v => {
    setLoader(prevLoader => {
      const nextLoader = Object.assign(
        Object.create(Object.getPrototypeOf(prevLoader)),
        prevLoader
      );
      nextLoader._channelSelections = [[Number(v), 0, 0]];
      return nextLoader;
    });
  };


Fix panic whilst loading UASTC encoded ktx2 textures

Objective
Fixes #9121
Context:

ImageTextureLoader depends on RenderDevice to work out which compressed image formats it can support
RenderDevice is initialised by RenderPlugin
https://github.com/bevyengine/bevy/pull/8336 made RenderPlugin initialisation async
This caused RenderDevice to be missing at the time of ImageTextureLoader initialisation, which in turn meant UASTC encoded ktx2 textures were being converted to unsupported formats, and thus caused panics

Solution

Delay ImageTextureLoader initialisation

Changelog

Moved ImageTextureLoader initialisation from ImagePlugin::build() to ImagePlugin::finish()
Default to CompressedImageFormats::NONE if RenderDevice resource is missing

This doesn't fix it for me. I tested for a file that works on 5da8af7d3 (the commit before #8336). The file panics both on main (05a35f6) and this PR

Maybe the GLTF loader also needs to be adjusted?

Thanks for flagging @nicopap!
I've applied the same change to the GltfLoader as well - could you check if your file loads successfully now?


provide option to only show OLD icon

Closing as the extension isn't useful anymore with the new dashboard.


Support CSS Modules out of the box

Now that create-react-app supports CSS Modules out of the box, it would be nice if TypeScript-React-Starter did as well.
Ideally it would work without having to eject.

Hi!
It seems like TypeScript-React-Starter is no longer supported, but you can try this boilerplate:
https://github.com/NickMaev/react-core-boilerplate
It's new, production-ready and free.

That is a lot heavier than what I need and this package seems well supported. Thanks.


[BUG]: Infobox text to be updated

Describe the bug
Status / Act: Negotiable Instrument Act to be changed to Negotiable Instruments Act
Steps to reproduce the behavior: Click on File a Case

@rajeshcherukumalli  this issue is fixed. please test it in UAT.

Hi @devarajd94,
I have verified this ticket in the UAT solution, and the changes have been updated as expected.
Negotiable Instrument Act to be changed to Negotiable Instruments Act
Citizen URL: https://dristi-kerala-uat.pucar.org/digit-ui/citizen/dristi/home/login
Reference Link: https://jam.dev/c/d0ac4590-7b83-46a7-9e07-496112394da4
litigant no. :7876876676
advocate no. :8798789788

Verified


Fix getSuggestedFontNameValues()

Following the discussion in https://github.com/googlefonts/fontbakery/pull/745 it seems getSuggestedFontNameValues() is badly broken.
We should just remove this method and anywhere it is called, replace it with a very simple check that just tests if the last characters in the filename match the list of 18 acceptable strings for subfamily names, and the first characters of the filename before the - are all the same, and if not, abort the test or build saying that the filename isn't following the canonical naming patterns which are used as a source of truth.

Fixed with 094571f


Inability to insert zero-length blob

OK, the code change for this fix was fairly localized.
Storing an empty blob in SQLite is an odd thing.  Basically, inside the ISQLite3Provider implementation(s), if the span for the blob is zero-length, the fixed block will yield a null pointer from that span, which will get passed to sqlite3_bind_blob(), which will change the value to a SQL NULL.  To avoid this, for the case when the span is zero-length, we need a non-null pointer (which SQLite apparently doesn't really use), so I contrive one, and explicitly pass zero for the length.
This change causes the new failing test case to pass, and appears to cause no other regressions (per my tests).  I tend to think the fix is safe, but I'm at maybe 95% confidence, not 100%.
I'm not yet sure what SQLitePCLRaw 1.x (which did not use spans) did with a zero-length byte array.  If it was inserting a zero-length blob, then this fix certainly seems like it should happen.  Need to check that.
Originally posted by @ericsink in https://github.com/aspnet/EntityFrameworkCore/issues/17494#issuecomment-526253452

This should be fixed in 2.0.1, which has been pushed up to nuget.org.


Feature Request: Support for Generic Git Server Type and Flexible Repository URL Suffix in ArgoCD Autopilot

Hi!
I'm currently facing a limitation with argocd-autopilot as it does not support certain git server implementations. To address this, I suggest introducing a feature that allows the use of a 'generic' git server type. This would include options to bypass the API, enabling manual configuration for essential parameters like the master branch name.
Ideally, argocd-autopilot would support every git server, but I understand that's an unrealistic expectation. Therefore, a generic option seems like a practical compromise.
Additionally, some git servers, such as onedev, do not append a .git suffix to their repository paths. In these cases, I'm compelled to name repositories with a .git suffix to ensure compatibility. I recommend that the generic implementation should not automatically add a .git suffix to the repository URL, to accommodate such variations.

just to understand the issue -
at the moment, autopilot does not do anything provider-specific in most cases, unless the user supplies a repoURL for a repository that does not exist (only in repo bootstrap command). in that case, there is a provider-specific api call to create that repository.
another edge case - if running inside a docker container (or anywhere else without a valid .gitconfig that defines the username and email of the committing user. autopilot will try to get the user info by their token - another provider-specific api call.
but your issue is in a different case - autopilot currently automatically adds .git to the repoURL (between the url and the path), and this is not compatible with ondev (for example)?

just to understand the issue - at the moment, autopilot does not do anything provider-specific in most cases, unless the user supplies a repoURL for a repository that does not exist (only in repo bootstrap command). in that case, there is a provider-specific api call to create that repository.
Yes, this is correct - I was  thinking there should be a 'generic' provider, that implies there's no API action going under the hood.

but your issue is in a different case - autopilot currently automatically adds .git to the repoURL (between the url and the path), and this is not compatible with ondev (for example)?

Yes, this is the main issue I have, actually. With all providers, autopilot forces the .git suffix to the repoURL (and this, combined with my previous thought, it may be solved by having a 'generic' provider, without any API magic or repoURL manipulation)


ModulNotFoundError

Is there already an issue for your problem?

[X] I have checked older issues, open and closed

Bug Description
Traceback (most recent call last):
File "C:\Users\Narki\AppData\Local\Programs\Python\Python312\Lib\site-packages\IMDBTraktSyncer\IMDBTraktSyncer.py", line 15, in
from IMDBTraktSyncer import checkChromedriver
File "C:\Users\Narki\AppData\Local\Programs\Python\Python312\Lib\site-packages\IMDBTraktSyncer\checkChromedriver.py", line 3, in
import pkg_resources
ModuleNotFoundError: No module named 'pkg_resources'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in run_code
File "C:\Users\Narki\AppData\Local\Programs\Python\Python312\Scripts\IMDBTraktSyncer.exe_main.py", line 4, in
File "C:\Users\Narki\AppData\Local\Programs\Python\Python312\Lib\site-packages\IMDBTraktSyncer\IMDBTraktSyncer.py", line 22, in
import checkChromedriver
ModuleNotFoundError: No module named 'checkChromedriver'
Environment
Windows 11
I entered all my credentials to verifyCredentials
Screenshots

Hi thanks for reporting this. Looks like there is a required package that isn't being installed during setup. I see what the issue is.
To install the missing package run the following in command prompt:
python -m pip install setuptools

After doing that run the script again and see if that resolves it. I will implement a fix for this shortly.

This should be fixed now in v1.8.6.
You can update using the following:
python -m pip install IMDBTraktSyncer

Let me know if that fixes it for you. If you are still having issues let me know and I'll take another look.

Glad to hear the original error is fixed.
As for the IMDB captcha issue, the only solution is to login/logout on IMDB from your browser on the same device. If you run the script too many times in a short period then the captcha can show up more frequently. This is usually only temporary. If you tried logging in/out several times already on your browser and still get that error then the only other thing you can do is wait a couple hours or try again the next day. Usually this issue will resolve on its own.


Display error message for bad address entry

I want to receive an error message if I enter an incomplete or incorrect address so that I know if there is an issue with the address and that I know whether the point returned is for the full address that I am trying to search.

@jmarin @wpears @hkeeler
What are the minimum requirements for the address in the API query?

@awolfe76 Not much at this point, it takes a single string and tries to match the search against the backend. This basically means you can start to get a list of candidates with a very short string being passed in. I think this requirement will probably need an address parser, which is planned for a later milestone. I'd pass this one to M2 for now


Upgraded All packages to latest version

Hello Friends. I have been busy at my office work so for and restarting my contributions again from today and I have learn so many things . Hope I'll contribute my best again thank you

Test this PR and tell me for changes I have started working on making the app responsive


Skipping a module doesn't work?

I have a codegenerated module src/rules.rs which I want to skip in my output.
I tried placing #[cfg_attr(tarpaulin, skip)] at the top of the rules.rs and the result still incorporates rules.rs - https://coveralls.io/builds/18202798
Then I tried to place this line above mod rules; in src/lib.rs and the output still incorporates them: https://coveralls.io/builds/18203157
What am I doing wrong?

Currently, #[cfg_attr(tarpaulin, skip)] only works on blocks so mod x {}. Issue #138 raises this and handling mod x; will be supported after it's implemented. So nothing wrong but above mod rules; will be the correct way once it's implemented!

Thanks!
Closing as dupe of #138.

Commented this on #138 but in case you didn't spot it.
"So I'm working on this now. But I did remember tarpaulin has an --exclude-files command for the #[cfg_attr(tarpaulin, skip)] on a mod x; use-case.
Still working on this but it could help you out in the meantime."


Write first two Aurelia-KendoUI controls

Using the AutoComplete Implementation Alternatives wiki page based approach,  write the code that implements the following two controls:
1. KendoUI AutoComplete control
This control is a member of "Input Control Family" so its implementation based on Aurelia Custom Attributes
2. KendoUI Grid control
This control is a typical example of KendoUI control that us not a member of "Input Control Family"  so it should be implemented as Aurelia Custom Element.
These two controls should be written as carefully as possible, so that just the comments that need to be present can be used as a "developer's manual". - so that anyone joining after these two components are fully written, could pick any other KendoUI control and work on it without having to ask how does this or that work
Important note
Please finish the Development Protocol article (starting here before / in parallel with coding these first controls, so that you can make references from that article into relevant code sections (to be used as examples of a specific rule in the Development Protocol

Closed this issue as it is not relevant any more


"How to use" Example Fails with No Feedback

I have been trying to get the Sparkplug B portion of the "how to use" code working without success.  I have a project targeting .NET Framework 4.7.2.  The code was copy/pasted from the "how to use" page, with a couple fixes for callback functions that appear to have been renamed.
When I run, it makes it to node.Start(), then just exits.  No logging via Serilog, nothing to the console, nothing to the debug output window in VS.
I dropped the SparkplugNet NuGet package, and added the source directly to allow debugging.   Start() calls ConnectInternal() calls this.Client.ConnectAsync().  That last call is part of MQTTNet, which is another black box.  I tried adding MQTTNet source directly, but that is a can of worms.
Am I missing some easy way of getting feedback on what is going wrong?  I did add SeriLog packages for console & file sinks, and this code before calling node.Start():
            Log.Logger = new LoggerConfiguration()
                .WriteTo.Console()
                .WriteTo.File("logs/JoesLog.txt", rollingInterval: RollingInterval.Day)
                .MinimumLevel.Verbose()
                .CreateLogger();

I can log something myself and see it in both console & file, but neither SparkplugNet nor MQTTNet log anything.
Update: MQTTNet appears to be using .NET 4.6.1 code, which results in a crash.  Detailed here.

Strange behaviour. I had something similar with a Serilog sink I provide on my own, but the issue is still unresolved as well... I'll test this soon and see if I find a solution. Sounds like dll hell once more :/

This is not relevant anymore since the library only supports Net6, 7 and 8 at the moment...


Fetch a list of tags instead of releases

In order to validate Rasa SDK version, we should fetch a list of tags instead of releases
Remove useless env variable - REQUIREMENTS_FILE
Don't use the set-env workflow command - https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/

@wochinge or @melindaloubser1 Can you 👀  if you have a minute and approve? 😄 🙌


different separator in subtree

Describe the issue
When taking a subtree, the separator of the subtree is different from the separator of the original tree.
This feels like a very unexpected behavior.
Environment

Platform: Windows
Python version: 3.12
bigtree version: 0.19.3

To Reproduce
from bigtree import Node, get_subtree

tree = Node("a", sep='.')
b = Node("b", parent=tree)
c = Node("c", parent=b)
d = Node("d", parent=b)
e = Node("e", parent=tree)
print(tree.path_name)

>> .a

subtree = get_subtree(tree, '.a.b')
print(subtree.path_name)

>> /b

Expected behaviour
check sep of original root node and propagate to root node of subtree accordingly
Screenshots
No response
Additional context
No response
Final Check

[X] I will watch this issue to stay updated with any comments or requests for additional information. I understand that timely responses will help in resolving the issue more quickly.

[X] I have thoroughly searched the documentation, and I have also checked for existing solutions or similar issues before submitting this issue.

exemplary fix:
patch get_subtree to assign original sep to new root node before returning
subtree = get_subtree(tree, '.a.b')
subtree.sep = tree.sep
print(subtree.path_name)

>> .b

Hello, thanks for noticing this! sep was purposely set to only modify the root sep so the sep is supposed to "propagate" to every descendant node instead of changing every single node's sep, hence inevitably leading to discrepancy which is expected. But I did not consider this for get_subtree, good catch :) Will fix it


Vue2.x Support needed

Hello,
Can, we expect datatables with the Vue 2.x support.
Thanx.

no vue2 support actually?

Thanx for confirming @diego-lipinski-castro

Vue 2.x support？？？？？

I'm try to do it work with in my Vue 2 app, but I'm doing some implementations to it be working soon.
When I solve those issues I'll commit and waiting for owner of this repository check and approve my request.


make sure oneway: overrides standard oneway tag

this commit makes the oneway tests pass.
it just reorganized the tests in the extractor, so that oneway:bicycle tag is checked before the normal oneway tag.

you can test this with "cucumber --tag @oneway". there's still one scenario failing, due to one way 'traps', but that's another issue.
also note that you might need to accept the 'test_grid_size' pull request to get test to run - it seems a small part of that commit somehow made it to the main repo, so the master currrently has a problem running tests. it's related to the @zoom variable.

github is messing with words that start with @. it's supposed to be oneway, all lowercase

any thoughts on this one?


Two panel view in editor mode is gone for markdown files

Gitea version (or commit ref):
Git version: 1.12.0
Operating system: Linux Ubuntu 18.04
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
On try.gitea.io:
https://try.gitea.io/tompos/aaa/_edit/master/a.md
The two panel editing option is gone. There is no more way to edit a markdown file and see the rendered changes in real time.
...
Screenshots

I wasn't aware of the existance of such a mode but this is intended because markdown files now edit like any other file and it was done in foresight for the SimpleMDE removal. It could of course be introduced again.

Yes, please it was an extremely nice editor, actually better the most other editors.

I just realized, not only the two panel mode, but the formatting toolbar is also gone.
This is very bad. The user experience is now way worse than before. Why was the change actually? What problem required this solution?

Also headings are hard to be distinguished, the font sizes are very close to each other.

We want to remove the bloated, buggy and unmaintained markdown editor, see https://github.com/go-gitea/gitea/issues/10729. Generally I'm not sure if we want to treat editing markdown files differently, GitHub also doesn't.

Yes, GitHub also does not provide realtime preview for the .md files in the editor.
So Gitea's editor was way better so far. It's a huge drawback now and it would be a huge improvement and delighter to bring it back.

@tomposmiko This changed because we want to edit codes on the web. Maybe we should have a markdown priview plugin on monaco editor.

And adding toolbar back would also be probably doable

I have a hard time understanding how this has worked before (I've never seen it). Was it a side-by-side editor? I guess if the screen is of sufficient width, we could just put the content of the preview tab at the side of the editor instead.

you could still see on the wiki editor since it is still SimpleMDE -- just edit a page and click the 2nd to last button 'toggle side by side'

I see, so it's fullscreening the editor and preview. Certainly possible but I think we should first replace SimpleMDE, add that new editing toolbar and then make a shared abstraction over both textarea-based and monaco-based editor that adds features like this.

I don't personally miss the full screen mode, but the real time visualization.
Thanks guys!
I also have concerns about the style (font size?).
Should I report it in a different issue?

Well, can we expected any change about this issue?
If not, we can close it.
If yes, that would be awesome!

I will like these feature. Everyone who has the same opinion can react with :thumbsup: emoji.

I would love to have markdown editor when I edit a markdown file.
Currently, the default editor is apply for markdown file.
Is that possible for this one

I was the original developer of the editor and making sure we had a toolbar for properly writing Markdown files, as not everyone knows markdown. I too noticed this missing and found this issue. It would be nice if there was a least a way to turn on or somehow install your own WYSWIG editor.

Why is it the Wiki page for when you can make a new file has a toolbar for Markdown, but not editing a regular .md file in the repo?

Please turn this feature back on. The Markdown tool bar would be a great success to be used in Markdown documents using the Github UI.
Contributors that way do not need to install additional software of use other sites to do it.

We are stuck on version < 1.12.0 until the ability to edit Markdown (with editor toolbar/preview) is re-enabled.
Our use case - we have a large number of users that provide simple edits to MD documentation in a repo.  They are not technical, and cannot handle MD. The toolbar/ preview capability allows them to edit the documentation.

with editor toolbar/preview

The Markdown preview tab is still there, there's just no side-by-side mode currently.
As for toolbar, I'm thinking about integrating https://github.com/github/markdown-toolbar-element but it may not be easy because the module expects a textarea to work with, so some hackery will probably be required to make it work with monaco.

I would like side-by-side mode.


waiting for get_by_role("checkbox", name="I'm not a robot")

testing recaptcha v2 and getting 'waiting for get_by_role("checkbox", name="I'm not a robot")' errors
Is there a solution for this available?

Test this branch to see if this has fixed the issue:
pip install git+https://github.com/Xewdy444/Playwright-reCAPTCHA.git@6-waiting-for-get_by_rolecheckbox-name=im-not-a-robot

Well, unsure if it is working actually
Got the same error when testing against https://votreassistantprive.fr/forum/login

It looks like this website is using reCAPTCHA v2 invisible for the "Learn More" > "Contact Us" form. In order to use this solver with this page, you must first fill out the information on the form and click the "Send" button in order for the reCAPTCHA to show. Then, you can call the solve_recaptcha() method to solve this reCAPTCHA.

@Xewdy444 OK thank you so much for that.
BTW can I pass a proxy IP in some way to be used by the solver?

Yes, you can pass a proxy to be used by the browser like this:
With Authentication
browser = playwright.chromium.launch(
    {
        "server": "http://<IP_ADDRESS>:3128",
        "username": "username",
        "password": "password",
    }
)

Without Authentication
browser = playwright.chromium.launch({"server": "http://<IP_ADDRESS>:3128"})

@Xewdy444 TypeError: launch() takes 1 positional argument but 2 were given

Sorry about that. I initially forgot to include proxy= in my example.


Add warning when collection rules are configured but not running in Listen mode

Currently, if a user is running in Connect mode and has collection rules configured, we don't give a warning indicating the issue. Now, a Warning logger message gives a brief description of what to do to resolve the issue. The check also occurs in two separate places since changes to the diagnostic port aren't respected once dotnet monitor is running - the first check will occur during validation for the diagnostic port, and then a check will occur every time the collection rule configuration changes. This will not fail validation, as that would be a breaking change.
Closes #1774

There might be benefit to add a unit test (or functional test, if that is too hard) that validates that this message is logged under these circumstances. It will help remember to remove the warning if we relax the 'listen' mode requirement.

/backport to release/6.x


[WIP] Switch to go modules for dependency management

Not sure if this is wanted at some point, but seeing also that upstream libbuildpack is moving towards Go modules, I thought that this might be of interest, since this should become the official way sooner or later.
This PR will make an hard requirement of Go >=1.11 during build (Maybe we should bail-out also when <go-1.11 is used ? ).

We intend to keep revising this support, while preserving compatibility,
until it can be declared official (no longer preliminary),
and then at a later point we may remove support for work
in GOPATH and the old 'go get' command.

With this change, when building from a GOPATH it is required to enable GO111MODULE environment variable ( which becomes redundant if GO111MODULE=on becomes default in future Go versions )
GO111MODULE=on go build
Otherwise no change is required ( go build should now just work outside of GOPATH ).
The build process now fetches automatically the required dependencies - in case of an offline scneario it's possible to generate the vendor directory by go mod vendor and reusing it later by forcing -mod=vendor while running go commands ( e.g.  go build -mod=vendor ) .
It should not affect pipelines as long as they are not building from a GOPATH directory.
WIP: /bin/gen-kube and /bin/gen-fakes needs to be adapted still

Getting rid of vendor in the repo would be great. I just bumped controller-runtime and it's a lot of files and dependencies :+1: