Error writing tags to CBR on MacOS

Hi. First of all great work
I tried both versions 1.6.0-alpha.6: Hal Jordan and 1.5.5 on a Mac M1. They both work and save info on cbz files. However I'm not able to save the tags in CBR files. Installed rar using brew install rar and both rar and Unrar work from command line. I get error 9 and I'm not able to understand why. Rar version is RAR 6.23
Bellow a copy of the log
2023-09-19T12:16:55 | comictagger | INFO | ComicTagger Version: 1.6.0a7.dev0 running on: Darwin PyInstaller: Yes
2023-09-19T12:16:55 | comictagger | DEBUG | user_data_dir: /Users/jpleite/Library/Application Support/ComicTagger
2023-09-19T12:16:55 | comictagger | DEBUG | user_config_dir: /Users/jpleite/Library/Application Support/ComicTagger
2023-09-19T12:16:55 | comictagger | DEBUG | user_cache_dir: /Users/jpleite/Library/Caches/ComicTagger
2023-09-19T12:16:55 | comictagger | DEBUG | user_state_dir: /Users/jpleite/Library/Application Support/ComicTagger
2023-09-19T12:16:55 | comictagger | DEBUG | user_log_dir: /Users/jpleite/Library/Logs/ComicTagger
2023-09-19T12:16:55 | comictagger | DEBUG | Installed Packages
2023-09-19T12:16:55 | comictagger | DEBUG | comictagger	1.6.0a7.dev0
2023-09-19T12:16:55 | comictagger | DEBUG | importlib-metadata	6.8.0
2023-09-19T12:16:55 | comictagger | DEBUG | metron-talker	0.1.1
2023-09-19T12:16:55 | comictagger | DEBUG | pycountry	22.3.5
2023-09-19T12:16:55 | comictagger | DEBUG | wheel	0.41.2
2023-09-19T12:17:13 | comicapi.archivers.rar | ERROR | Error writing rar archive [exitcode: 9]: /Users/jpleite/Downloads/Ast3r1x - PT0001 - 2020.cbr :: ComicInfo.xml

This sounds like macOS is getting in the way that error code should mean that it isn't allowed to create a file in that directory.
In System Settings -> Privacy and Security -> files and folders you should see an entry that looks like this

thanks @lordwelch  it was already enabled. I even tried before with full disk access. Still the same error. Forgot to mention I'm using latest beta version of Sonoma. Any log I can send that might be useful? Thanks in advance

That's actually what I just tested on except it's an M2 air

Try running this in the terminal
xattr -d com.apple.quarantine "$(dirname $(readlink -f $(which rar)))/"*

This will remove the quarantine flag that downloaded items get from the rar program
and then can you try downloading the cbr file from here https://archive.org/download/CoryDoctorowsFuturisticTalesOfTheHereAndNow
and move it out of your downloads folder before trying to save to it

That is strange. It does not seem a file permission issue as I can write tags in cbz files. Any other suggestions?

Try running this in the terminal
xattr -d com.apple.quarantine "$(dirname $(readlink -f $(which rar)))/"*

This will remove the quarantine flag that downloaded items get from the rar program and then can you try downloading the cbr file from here https://archive.org/download/CoryDoctorowsFuturisticTalesOfTheHereAndNow and move it out of your downloads folder before trying to save to it

Thanks. Tried exactly as you described. Still the same error. RAR has quarantine removed and it's bin is under rar -> /opt/homebrew/Caskroom/rar/6.23/rar/rar installed via homebrew

lets try using rar manually
echo testing rar edit | rar c  '~/Downloads/Cory_Doctorows_Futuristic_Tales_of_the_Here_and_Now_CBR.cbr'
rar cw '~/Downloads/Cory_Doctorows_Futuristic_Tales_of_the_Here_and_Now_CBR.cbr'

Thanks @lordwelch .
just had to correct your command to (remove the '):
echo testing rar edit | rar c ~/Downloads/Cory_Doctorows_Futuristic_Tales_of_the_Here_and_Now_CBR.cbr rar cw ~/Downloads/Cory_Doctorows_Futuristic_Tales_of_the_Here_and_Now_CBR.cbr
and the output is as expected

Turns out I don't log the output of the rar command, I'll have another alpha put up this weekend that logs it and we can hopefully see what's failing.

thanks @lordwelch I'll try again after that release

Hi @lordwelch did you manage to compile that build? Thanks in advance

It will be published in a few minutes

Hi @lordwelch . Just tried and have the following error at startup: Failed to load any talkers, please re-install and check the log located in '/Users/jpleite/Library/Logs/ComicTagger' for more details. The log states
2023-10-07T08:14:06 | comictagger | INFO | ComicTagger Version: 1.6.0a7 running on: Darwin PyInstaller: Yes 2023-10-07T08:14:06 | comictagger | DEBUG | user_data_dir: /Users/jpleite/Library/Application Support/ComicTagger 2023-10-07T08:14:06 | comictagger | DEBUG | user_config_dir: /Users/jpleite/Library/Application Support/ComicTagger 2023-10-07T08:14:06 | comictagger | DEBUG | user_cache_dir: /Users/jpleite/Library/Caches/ComicTagger 2023-10-07T08:14:06 | comictagger | DEBUG | user_state_dir: /Users/jpleite/Library/Application Support/ComicTagger 2023-10-07T08:14:06 | comictagger | DEBUG | user_log_dir: /Users/jpleite/Library/Logs/ComicTagger 2023-10-07T08:14:06 | comicapi.comicarchive | WARNING | Failed to load talker: rar 2023-10-07T08:14:06 | comicapi.comicarchive | WARNING | Failed to load talker: sevenzip 2023-10-07T08:14:06 | comictalker | ERROR | Failed to load talker: metron Traceback (most recent call last):   File "comictalker/__init__.py", line 28, in get_talkers   File "setuptools/_vendor/importlib_metadata/__init__.py", line 208, in load   File "importlib/__init__.py", line 127, in import_module   File "<frozen importlib._bootstrap>", line 1030, in _gcd_import   File "<frozen importlib._bootstrap>", line 1007, in _find_and_load   File "<frozen importlib._bootstrap>", line 972, in _find_and_load_unlocked   File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed   File "<frozen importlib._bootstrap>", line 1030, in _gcd_import   File "<frozen importlib._bootstrap>", line 1007, in _find_and_load   File "<frozen importlib._bootstrap>", line 984, in _find_and_load_unlocked ModuleNotFoundError: No module named 'metron_talker' 2023-10-07T08:14:06 | comictalker | ERROR | Failed to load talker: comicvine Traceback (most recent call last):   File "comictalker/__init__.py", line 28, in get_talkers   File "setuptools/_vendor/importlib_metadata/__init__.py", line 208, in load   File "importlib/__init__.py", line 127, in import_module   File "<frozen importlib._bootstrap>", line 1030, in _gcd_import   File "<frozen importlib._bootstrap>", line 1007, in _find_and_load   File "<frozen importlib._bootstrap>", line 972, in _find_and_load_unlocked   File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed   File "<frozen importlib._bootstrap>", line 1030, in _gcd_import   File "<frozen importlib._bootstrap>", line 1007, in _find_and_load   File "<frozen importlib._bootstrap>", line 984, in _find_and_load_unlocked ModuleNotFoundError: No module named 'comictalker.talkers' 2023-10-07T08:14:06 | comictagger | DEBUG | Installed Packages 2023-10-07T08:14:06 | comictagger | DEBUG | comictagger	1.6.0a7 2023-10-07T08:14:06 | comictagger | DEBUG | importlib-metadata	6.8.0 2023-10-07T08:14:06 | comictagger | DEBUG | metron-talker	0.1.1 2023-10-07T08:14:06 | comictagger | DEBUG | pycountry	22.3.5 2023-10-07T08:14:06 | comictagger | DEBUG | wheel	0.41.2

1.6.0-alpha.8 has the startup issue fixed

Thanks @lordwelch. Already tried. The log states

2023-10-09T12:02:52 | comicapi.archivers.rar | ERROR | Error writing rar archive [exitcode: 9]: /Users/jpleite/Downloads/Asterix - PT0001 - Astérix, o Gaulês (1961).cbr :: ComicInfo.xml :: b'\nCannot create __rar_99329.35340\nRead-only file system'

Can you try out PR #554 and let me know if it works?
https://github.com/comictagger/comictagger/wiki/downloading-packages-from-pull-requests

Just tried Just tried app from #554 . Still get the following error

2023-10-10T12:47:12 | comicapi.comicarchive | ERROR | comicapi/archivers/rar.py:172: Error saving CIX! for /Users/jpleite/Downloads/Asterix - PT0001 - Astérix, o Gaulês (1961).cbr: [Errno 20] Not a directory: PosixPath('/Users/jpleite/Downloads/Asterix - PT0001 - Astérix, o Gaulês (1961).cbr')

I've updated #554 can you try again?

Hi @lordwelch . I'm happy to report it did work this time. Thank you for your time and patience. I'm just curious about what was the matter?

the rar command creates a temporary file when it modifies the the cbr comic and it was trying to create it in it's current directory which was probably a temporary directory and for some reason you didn't have access to it, so I've set the working directory to be the same as the file that gets modified.


[Thaumcraft 6.1.BETA26] Arcane Workbench Won't Craft Items

Despite the circling animation appearing around the primal vis crystals in the workbench menu, all the items being in the correct spot, and the amount of vis available appearing above the output slot, I can't seem to craft a Runic Matrix. It just doesn't show up in the output.

make sure you have completed the research required to build the runic matrix

Even with the research completed, the arcane workbench wont allow for any crafting. Crucible is now generating random essentia instead of what the item is that was thrown into it. Azanor has all but abandoned this mod, and anyone still supporting him is a fool, at best

there's more recipes that's beem lost

BTW, in the Arcane Crafing terminal from Thaumic Energistcs is working

Can someone help with the research for the Arcane Crafting Terminal

Can someone help with the research for the Arcane Crafting Terminal

Again, the creator of this mod abandoned it a while ago.  You might have better luck getting some training with programming and see if you can fix it.  Maybe even update it to work with 1.15 and such

I already know some programming do you know were I find the files for thaumcraft


How to dump msdk log messages on Windows

According to this issue, https://github.com/Intel-Media-SDK/MediaSDK/issues/948#issuecomment-442511043
I could dump MSDK log messages on Linux. But I don't have an idea where to create the /.mfx_trace on Windows. When I build the msdk source code with release mode there is nothing and for debug mode I could only see MSDK_HEVCE_HW_DDI_LOG.txt which is not what I want.
It seems useless when I put the .mfx_trace under c:/Users. Is there any solution for Windows.
And I also don't know which directory the log file will be put. Thanks.

It's much easier on Windows. MediaSDK dev packages have GUI tracer app. Please downloaded the latest dev package from official Intel website.

Thanks for the reply. I am using the open source oneVPL dispatcher and the inner-source mdp-msdk-lib to build the runtime. After I downloaded the tracer.log and pressed the start button, no bytes are written into the log file. Does the runtime that I'm using have to be the msdk runtime in the dev package? I put the libmfx64-gen.dll which I built myself under C:\Windows\System32 and seems that tracer.log doesn't react to it when I run the application to call the runtime.

Hi @tong1wu ,
Please ask the question of tracer availability in https://github.com/oneapi-src/oneVPL

ok, thanks.


Setup Usage of Kyverno App in CAPZ Clusters

Motivation
In order to set policies and with that increase the security in our management clusters, we are using Kyverno throughout all providers in Giant Swarm. Team Shield already created Kyverno and Kyverno Policy Apps that in best case just need to be included to our app collection.
Todo

[ ] Introduce kyverno app to our capz app collection
[ ] Introduce the kyverno policy apps to our capz app collection
-- we at least need the kyverno-policies app, but could also need the other ones, depending on the touchpoints we already integrated.
[ ] Verify that everything works as expected

Technical

We can checkout the kyverno policy apps in capas app collection: https://github.com/giantswarm/capa-app-collection/tree/master/flux-manifests

Outcome

We have policies in place that check requests to our Management Clusters against a set of policies increasing the safety and stability of our clusters.

This is in responsibility of team shield
PodSecurity should be enough for us now as long as we use <1.25 of k8s

I just noticed in the comments that this is team-shield responsability so i am putting this back into the backlog

As discussed with Shield, we're responsible to introduce the kyverno and kyverno-policy apps into our app collection. They should work out of the box, but as soon as trouble arise, we can get in touch with them.
Changed the scope of this ticket to Management Clusters only. In a more general direction regarding Kyverno Policies on Workload Clusters we will setup a meeting with Shield in the next weeks to get an update on the roadmap for this on shield side.
So this ticket can be worked on in the above listed scope.

:+1:

looks to be working fine

I will wait for the linked PR to be approved and merged before closing this


Devtools support

Thanks for this amazing library!
Is there any plans for devtools support as in redux ?

@ruslan-byondxr for the next version 0.8 not yet, but it will be nice to do it before 1.0! Thank you! 😊
It would be nice to define what you would be interested to show in devtools. Apart from the store that each component consumes, I don't know if there are other features that could contribute.

I want dev tool as well !!
What I want is just I can see data in store, only this ! ( I also think this is the most people want)
well
really nice library that make store more simple just like useState
appreciate it
Let's have some fun and drink some tea !

You can use teaful-logger as a dev-tool for teaful.

A proof of concept of what will become Teaful DevTools. Any feedback is good. The idea will also be to add more features like being able to modify the store apart from visualizing it. I hope to have a stable version next week.

Some improvements:

Some improvements (also I added dark mode):

Another update (possible to modify the store):

Any feedback will be welcome @ruslan-byondxr @siddharthborderwala @WreewanMorhee @danielart It is likely that in a few days the first version will be available (for the moment only for Chrome).

Looks great to me! Also, I love the option to modify the store this way.

Done!!! Already available in 0.9.0.

Teaful DevTools GitHub
Blog post

Sorry but ...
Why will this happen ??
Is this normal phenomena ??
Can this be solved ??

Sorry but ... Why will this happen ?? Is this normal phenomena ?? Can this be solved ??

I've no idea, I'll look into it. Thank you for notifying. The extension has been reviewed by Chrome developers and they have accepted it. All the code is open-source: https://github.com/teafuljs/teaful-devtools/
Maybe it's because since 2016 that I didn't upload any Chrome extensions...

For new developers, it generally takes a few months to become trusted

https://support.google.com/chrome_webstore/answer/2664769?hl=en-GB#zippy=%2Cinstall-with-enhanced-safe-browsing


Installation error

Hi kevin,
Thanks for developing such a convenient package. I want to use it but I don't know how to install python. I'd appreciate it if you could give me some advice. Thanks!

Hi kevin,
Great tool, can't wait to see your article online.
I just have one question about your tool. What's python and how to install it.
Many thanks.

Please refer to https://www.anaconda.com/distribution/#download-section and download the python 2.7 version.

Hi lijxug,
Python is a programming language. It's simple and easy to learn.


Add new ebook and tweak univserities go rogue ebook.

Merged in the latest from main after merging changes from #75
Resolved the merge conflict which is due to the directory structure changing.
Updated this new book for conform to the new directory, file, and template names and added in the tags: [chapter] frontmatter.

Looking good now, have a peek: https://pr76-tszl1dpfsetbcrmzb1ftrtarg2qcwnj0.tugboatqa.com/ebooks/keeping-government-consistent/cover/


Unable to use with newly created Angular4 site using "ng new".  Issue with sourceFile['symbol'].exports

These are the steps I take to reproduce the issue.

ng new example
cd example
npm install
npm install --save angular-ssr
ng build
./node_modules/.bin/ng-render

I get the following error:
C:\tests\example\node_modules\angular-ssr\source\application\static\find.ts:16
const exports: Map<string, any> = sourceFile['symbol'].exports;
^
TypeError: Cannot read property 'exports' of undefined
at Object.exports.exportClause (C:\Users\nick\Desktop\example\example\node_modules\angular-ssr\source\application\static\find.ts:16:57)
at _loop_1 (C:\Users\nick\Desktop\example\example\node_modules\angular-ssr\source\application\static\modules\root.ts:60:29)
at Object.exports.discoverRootModule (C:\Users\nick\Desktop\example\example\node_modules\angular-ssr\build\application\static\modules\root.js:59:9)
at Object.exports.loadApplicationModule (C:\Users\nick\Desktop\example\example\node_modules\angular-ssr\source\application\compiler\options.ts:57:14)
at WebpackCompiler.compile (C:\Users\nick\Desktop\example\example\node_modules\angular-ssr\source\application\compiler\webpack\compiler.ts:30:38)
at ApplicationBuilderFromSource.build (C:\Users\nick\Desktop\example\example\node_modules\angular-ssr\source\application\builder\from-source.ts:18:29)
at Object. (C:\Users\nick\Desktop\example\example\node_modules\angular-ssr\source\bin\render.ts:31:29)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (C:\Users\nick\Desktop\example\example\node_modules\angular-ssr\bin\render.js:3:1)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.runMain (module.js:604:10)
I would like to figure out why this is happening myself, but when I go through the source, I'm having trouble figuring out the entire flow of the process.
So, my questions are if anyone knows what this issue could be, and also where to start looking to get a better feel for how this package works overall.
Thank You!
--nick

One of the first things I thought of was that it was failing because there was no routing going on, so I added a few basic routes, and I still get this same error.   I also tested this on my Mac and I get the same error as well, just in case it was a windows only issue.

I'm getting the exact same error in my project.
Do anyone has any idea on how to solve this problem?
Thanks
G.

I think it's missing a webpack configuration that causes this error.
@clbond do you have any idea how to overcome this issue?
Thanks,
G.

I managed to understand the problem and fix the issue. When you create a new project with "ng new" inside your tsconfig.app.json and tsconfig.spec.json files you will have this line:
"extends": "../tsconfig.json"
This line is causing the issue. If you delete it from both the files and import the config you need from the tsconfig.json directly inside the two files the ssr process will execute correctly.
G.

You can avoid that error just by passing --module relative/path/to/app.module and --symbol AppModule. Replace those values with the path to your root module and the name of the symbol. That should fix the problem and let you use your normal tsconfigs. I will look into a proper fix when I get a bit of free time this weekend

Manually flattening my tsconfigs works as expected, but the module/symbol workaround seems to consistently fail with this error:
[info] Rendering application from source (working path: /tmp/9c3c20b511c89)

<--- Last few GCs --->

[125:0x55cf5f8cf6f0]    95031 ms: Mark-sweep 1487.9 (1574.9) -> 1487.9 (1575.9) MB, 1192.4 / 0.0 ms  allocation failure GC in old space requested
[125:0x55cf5f8cf6f0]    96189 ms: Mark-sweep 1487.9 (1575.9) -> 1487.8 (1544.9) MB, 1156.7 / 0.0 ms  last resort
[125:0x55cf5f8cf6f0]    97359 ms: Mark-sweep 1487.8 (1544.9) -> 1487.8 (1544.9) MB, 1167.9 / 0.0 ms  last resort

<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 0x15fa9bba66a1 <JS Object>
    1: slowToString(aka slowToString) [buffer.js:473] [pc=0x3840923cc3f5](this=0x1b9ab3b02311 <undefined>,buf=0x2d7e82b9a9f9 <an Uint8Array with map 0x23d77702c579>,encoding=0x15fa9bbaf309 <String[4]: utf8>,start=0,end=13777754)
    2: toString [buffer.js:~501] [pc=0x384092348896](this=0x2d7e82b9a9f9 <an Uint8Array with map 0x23d77702c579>,encoding=0x15fa9bbaf309 <String[4]: utf8>,start=0x1b9ab...

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
 1: node::Abort() [node]
 2: 0x55cf5cf6f61e [node]
 3: v8::Utils::ReportOOMFailure(char const*, bool) [node]
 4: v8::internal::V8::FatalProcessOutOfMemory(char const*, bool) [node]
 5: v8::internal::Factory::NewRawOneByteString(int, v8::internal::PretenureFlag) [node]
 6: v8::internal::Factory::NewStringFromOneByte(v8::internal::Vector<unsigned char const>, v8::internal::PretenureFlag) [node]
 7: v8::internal::Factory::NewStringFromUtf8(v8::internal::Vector<char const>, v8::internal::PretenureFlag) [node]
 8: v8::String::NewFromUtf8(v8::Isolate*, char const*, v8::String::NewStringType, int) [node]
 9: node::StringBytes::Encode(v8::Isolate*, char const*, unsigned long, node::encoding) [node]
10: node::Buffer::Utf8Slice(v8::FunctionCallbackInfo<v8::Value> const&) [node]
11: v8::internal::FunctionCallbackArguments::Call(void (*)(v8::FunctionCallbackInfo<v8::Value> const&)) [node]
12: 0x55cf5c6c715b [node]
13: v8::internal::Builtin_HandleApiCall(int, v8::internal::Object**, v8::internal::Isolate*) [node]
14: 0x3840918843a7
/bin/bash: line 1:   125 Aborted                 ./node_modules/.bin/ng-render --module src/js/cyph.com/app.module.ts --symbol AppModule

@buu700 Are you running this in a very low-memory environment?

At the time I was running it in a Docker container with 2 GB allocated memory, but since then I've increased it to 4 GB to work around an unrelated issue with the Angular CLI (ng serve bug). I'll go ahead and test this again with the higher memory limit.

Actually, I think I have resolved the issue that was causing your bug. Can you try with version 0.1.103 please? It was accidentally including all of node_modules when compiling the TypeScript (due to ng news weird tsconfig.json structures)

Awesome, sure, I'll report back here soon.

I am not able to reproduce this issue anymore with projects created using ng new. I am pretty sure this issue is fixed but I'll wait to hear back from you before I close it

Yep, confirmed that it's no longer reproducible for me.


Mention mustRunAfter as a solution for missing task dependencies

mustRunAfter is already one solution in the problem, though it has not been mentioned in the manual.

@bot-gradle test and merge


feat(module:tree, tree-select): support customize icons

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #2890 #2503
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

抽时间考虑一下string支持问题，如果支持要么取消旋转class，要么对用户说明旋转属于设计问题，之后rebase一下master

@simplejason 移除了 string 的支持，新增了 show line 时也支持自定义图标。

@HsuanXyz @simplejason 没其他问题的话，麻烦 merge 一下吧，不然隔段时间就要 rebase


illegal date

When adding a semester, if the end date is before the start date, should I give an error?
If the date is ridiculous, such as 1900-1-1 or 3000-1-1, should I give an error?

The first one should give an Illegal Argument Error
The second one should not give error.


Move website to HTTPS

Gotta do it.

Is you know what enabled for public gh-pages repositories yet?

Is you know what enabled for public gh-pages repositories yet?

No, but we could move hosting over to Netlify 😆

I’d rather avoid additional dependencies if someday this will just work. But Netlify does look pretty rad though!

https://blog.github.com/2018-05-01-github-pages-custom-domains-https/

"Domain awaremac.com is not eligible for HTTPS at this time." 🤔 I'll try again later.

https://awaremac.com


add encryptionformat to volume reponse for admins

ISSUE TYPE

Enhancement Request

COMPONENT NAME

API/UI

CLOUDSTACK VERSION

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

On requests like listVolumes or updateVolume the output should include encryption format (at least for admins)
see the discussion in https://github.com/apache/cloudstack/pull/8643#issuecomment-2085073967
STEPS TO REPRODUCE

EXPECTED RESULTS

ACTUAL RESULTS

to be consistent with volume table, maybe better to use encryptformat ?

thanks, updated

Closing the issue as the pr is merged


Added some basic contributing guidelines

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1547/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1441/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/112/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1553/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1447/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1554/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1448/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/118/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/119/ for details.


Consider versions stored in filesystem

i've always considered committing to git in the CI process to be fraught with peril.  I would think the sensible default to be storing the version manifest on a remote filesystem.  Perhaps possible to pull the version from that remote disk as part of the push process so that a version history could be maintained?

Hi @sethphillips, thanks for your support via coffee! I'm glad Lasso works well for you. This was actually quite a challenging aspect to Lasso. I agree with you and wouldn't personally commit anything while in the CI process - this is why I added the "--no-git" flag. This creates the lasso-bundle.json file on the Filesystem rather than locally in the project, but it's challenging to keep track of versions.
Perhaps to add versioning without requiring you to commit the file, Lasso could keep track of the current commit that CI is working on, and then download the bundle relating to that. This would then help solve the rolling-back issue, and if we already have assets for a certain commit-code, we skip rebuilding and just tell CI that it's already been completed.
What do you think? Do you have any other suggestions to help with this discussion?
Thanks again for using Lasso!

im trying to fully wrap my head around the process of the rollback.
You're counting on a quick commit rollback and re-pull the previous zip from the committed bundle.json in version control?
I guess in that scenario CI doesn't come into play at all whereas in most of the systems I work on a rollback would actually entail another CI build with the committed changes.

Yes that's exactly how it works. It only checks the history on the "pull" command. I was naieve of the other approach - where CI will also be run again during a roll back. In that case, Perhaps by default, it should check the version before compiling and if it sees that a bundle has been created for that commit hash, it will just skip the build process?
I'd say that's a pretty solid improvement. We could also tie the versioning to the commit hash rather than simply the number of bundles that has been created. This would then solve the isse where you can't have versioning without Git. I love it.

Hello. I had the same kind of questions.
I would like to use lasso, but with the --no-git option.
I was thinking of a way to associate the generated zip file with a git commit number, or a git tag version.
This way, when pulling the files, it's possible to either

get the file using the git commit number
or using the last tag version (it might not be up to date, but it's better than nothing)

The lasso-bundle.json file can be suffixed with the commit number, or the tag version, so we can get it, and multiple json files can point to the same asset zip file.

Hi guys, apologies for the delay on this issue. I'm going to be spending a bit more time on Lasso over the next coming weeks as I've got a few PRs here to merge in and I have a few v1.5/2 ideas that I want to work on.
On this issue, there's a PR open that appends the git commit hash onto the lasso-bundle file to help improve Lasso in this scenario. I'm also going to be building in an option where Lasso will append the environment name onto the bundle so this is tracked better within git.


Update Encdec to use stronger crypto

This should resolve issue #9
The test_endec suite runs successfully.
Extended test suite to check new logic path (version checking)
There is an issue with the test_rest suite, looks like it is trying to reach a particular IP? Had a colleague check out master and confirm the test also failed for him.

I see the Travis job has failed. Locally everything passes except for test/test_rest.py::test_rest_uses_manifest_console_ip_when_env_var_not_set which seems to rely on a host that I assume is not reachable from the internet. Travis does not seem to print any output from the ./lint.sh && ./test.sh command so I cannot confirm which test(s) actual failed.

I updated my local urllib3 and now I have the REST test failing. It's because the URL validation has been tightened. The test (stupidly) uses a URL 9.123.321.101, and 321 is not valid in an IP address. I'll update the test on a separate pull request.


x/tools/gopls: Compiler should support : in asm go1.20

As you know, go 1.20 auto generated symbols's name is changed(go.->go: and type.->type:).
But when I reference these new symbols in asm, the compiler reported error(illegal or missing addressing mode for symbol type).
I think the compiler's behavior should be adjusted accordingly.
Thanks for all.

Closing as a duplicate of #58244.
There is nothing gopls can do to address this issue.


chore: release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2024-12-09)
Features

add a ProcessPort to connect current process (9f6eb1d)
implement port, server and client architecture (a920275)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.0
:sunflower:


feat: Move the save/export buttons to the bottom of the screen

Fixes https://github.com/excalidraw/excalidraw/issues/3378 (eventually)

I kinda like this one!


Fix bug with model fragments support

See: https://github.com/danielspaniel/ember-data-factory-guy/pull/184#issuecomment-205084674

This issue should now be fixed with version 2.5 ( which has full support for ember-data-model-fragments )


DAOSGCP-175 Updates for google-cloud-daos v0.4.0

The google-cloud-daos repository which contains the DAOS Server and Client terraform modules has been tagged v0.4.0. The new version now uses the Google HPC Rocky Linux 8 image by default.
Submission Checklist
Please take the following actions before submitting this pull request.

Fork your PR branch from the Toolkit "develop" branch (not main)
Test all changes with pre-commit in a local branch #
Confirm that "make tests" passes all tests
Add or modify unit tests to cover code changes
Ensure that unit test coverage remains above 80%
Update all applicable documentation
Follow Cloud HPC Toolkit Contribution guidelines #

/gcbrun


Indirect's move constructor should be conditionally noexcept (because of allocators)

see https://github.com/jbcoe/value_types/pull/223

Note that it is not the move-constructor that needs to be conditionally noexcept, but rather the indirect(std::allocator_arg_t, const Allocator& alloc, indirect&& other) constructor.
I believe the indirect(indirect&& other) constructor should be able to be unconditionally noexcept.


View Course handouts link updated as per VD

The UI for "View course handouts" link in Course Info has been updated as per comments from Frances.
@rohan-dhamal-clarice @aleffert - Please review
JIRA: https://openedx.atlassian.net/browse/MOB-1505

:+1:


Convert to Erlang

This PR covers the conversion to Erlang as discussed in #23. This is my first real go at Erlang, so I expect there will be some items to address, especially as it relates to documentation. 😃 I imagine using code suggestions in the reviews would be very helpful to move the process along quickly.

@tsloughter - any help on the circle config would be helpful. I tried to update to your latest orb but it seems fairly different than the one in telemetry.

@bryannaegele ok, I'll send a patch to telemetry to update the circle orb first.

@bryannaegele take a look at https://github.com/beam-telemetry/telemetry/pull/45

Ok, I think I have all the feedback addressed.
@arkgil I think we'll need to update the checks with the circle updates.

Just two minor comments, then we only need to rebase and I think we are ready to go!

@bryannaegele looks great! :tada: Now we just need to address CI and we can merge this. 18 should no longer run and both 19 and 20 are failing. :)

So it seems like I'll need to parse the OTP version in 19 and 20? I can't seem to find a way to do this with macros since it appears the -if directive is a 21+ thing.

Hi @bryannaegele! I have removed the requirement for otp-18 but otp-19 is still failing on CI. :(

Yeah. @tsloughter - seems like that snippet didn't work correctly and I'm not familiar enough with rebar to know if that config piece is correct.

@bryannaegele you just had extra tuple brackets. Hopefully works with my suggested change.

🤦♂ - thank you @tsloughter.
And thank you @josevalim for helping me through my first Erlang project 🙌  🎉

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


Compare with django-zen-queries

Django Zen Queries occupies a very similar space.
It's worth doing a comparison. If this project is not in some way an improvement over it, it should be archived. If not, it's worth adding this project to a "similar projects" section of the readme.

👋🏻 hello! Author of django-zen-queries here, just found this project.
I'd agree there's a lot of overlap between the two, and they occupy very much the same space. From a very quick glance, it looks like the differences are:
sans-db

Multiple database support (zen-queries just blocks all databases)
Template backend

zen-queries

An "escape hatch": the ability to re-enable queries temporarily inside a disabled block (queries_dangerously_enabled)
Replacement for render and TemplateResponse
REST framework integration
Part of an ecosystem, ie automatic integration with django-readers

Hi @j4mie, thanks for reaching out, and good job on finding this ticket :laughing:
It looks to me as though both projects have handy features. I'm relieved that they're different enough to keep this one going (for the moment at least), because I have a bit of a soft spot for this project. It definitely seems like it's worth me adding a link to your project in the readme though.
Thanks for the link to django-readers too, that looks like a fun idea!


Added namespacing to the schema, as well as adding it to the example …

…XML doc and mentioning it in the README.

@ldko, @damonkelly, does this look good to you guys? Just added namespacing, so now you must include namespacing in some form (either by a default namespace or explicit namespacing) with the XML doc. And it must match the target namespace given in the schema.

LGTM :+1:

:+1:


Error with generating code for Beyblade Metal Masters !!!

When I want generate code it says
''Finishes analysing ROM
No HTTPs URLs to patch were found.''
Please fix this !!!

Although the cover features the Nintendo Wi-Fi Connection logo, this game doesn't actually have any online functionality. (source) So it's not surprising that no URLs would be found.


Remove irregular whitespace

Guess these were 'non-breaking whitespace' characters.  These confuse many tools, e.g. eslint.
@rbrechter not sure how they got in from your editor.
Also enable eslint to catch these cases in the future.

Good point. I do not know how these whistespaces came in. I used VS Code as usual.


[iOS]: onDemandResources 'prefetch' type is not working on iPhone

Describe the bug
In onDemandResources, when I set type to 'prefetch'. It did not fetch the assets.
To Reproduce
Steps to reproduce the behavior:

We can upload a sample project to TestFlight to test the onDemandResources.
Open the test app on iPhone and click the 'ODR Check Folder 2 Assets' button. It will always say assets not downloaded for iPhone X or greater devices.

Target platform and where build was made:

Device: iPhoneX or greater.
OS: iOS15.5

Additional context
I tested the same code with iPhone 7 and 6s and they seem to be working fine.
Sample Project:
https://www.dropbox.com/s/i1q80fqz48ecnkg/ODRSampleTest.zip?dl=0
Video of issue:
https://user-images.githubusercontent.com/13268636/176410284-faf15fff-eb35-485e-a71c-e31808dbf0cc.mov

Hello!
Pre-fetch is not same as install. It is designed to help download faster, and iOS can start it on the background, when apps are off. You may have to still request the tag to download as well. They probably changed behaviour of pre-fetch over the time so people doesn't use it as 'install'. Try when downloading the app leave phone idling for like 20 minutes...

Thanks.
I will test it.

Hi @Shchvova I tried to fetch 'prefetch' type assets with request and they worked fine. But no matter for how much time I left my app in background they did not download.
So I guess I have to make request to download them maybe it's necessary.


Creates unnecessary output for imported methods in the module

When running with this template, in contrast to the standard output from JSDoc, I find entries in the resultant RST file for functions that have only been defined by import into the module in question.
Compare:

with the RST output:
=======================
Module: assetLoader
.. contents:: Local Navigation
:local:
Children
.. toctree::
:maxdepth: 1
Description
Asset Loader module.
Loads frontend assets (JS and CSS) asynchronously, to avoid page bloat, and improve page load time.
Function assetLoader
Take an Array of frontend asset files and asynchronously load in order.
.. js:function:: assetLoader(assets[, callback][, timeout])
:param Array assets: An array that contains the paths for each of the files to be loaded.
:param assetLoadCallback callback: A callback that gets called when fetching is complete.
:param Number timeout: A ms timeout after which to error for CSS files - this compensates for the lack of wide
support for the onerror event on 'link' tags in HTML.

Function scriptjs
.. js:function:: scriptjs()
Function loadcss
.. js:function:: loadcss()
Function onloadcss
.. js:function:: onloadcss()
Function logging
.. js:function:: logging()
Function _
.. js:function:: _()
Function timeout
.. js:function:: timeout()

@oliamb Any ideas on where to try to stop this from happening? I have tried looking at the default JSDoc template, but can't quite map how to make that functionality happen in the RST template.

The project against which I ran the doc is not using ES6 (shame), so I cannot really tell.
I just run it against this file for a test https://github.com/wwayne/react-tooltip/blob/master/src/index.js but I don't experiment the same effect. Can you point me to some code that produce this result ; and maybe some details about the command you run against it?

@rtibbles, I managed to reproduce your issue. Can you comment the pull request above?


Automatically trim spaces from column headings

Some of the SBtab examples unintentially have extra spaces in the column headings which could be automatically trimmed rather than generating errors.

Done


updated to targetSDK 25 and added new dummy activity to ask permissions to avoid crash

updated to targetSDK 25 and added new dummy activity to ask permissions to avoid crash when click on record button in main activity

Changes to gradle have already been made. See #39 . Also, permissions have been implemented. Please claim an issue before working on it. So that there are no conflicts. @getsanjeev

@samagra14 great that you updated the gradle & added maven support! Yes I see now you had raised an issue 4 days back. For there were no contributors few more days back I dint bother to look at issues having in mind that project is not that active.

@samagra14 I will resolve the conflicts in this PR and will be tuned to issues as well. Hope its alright now!

I guess an entirely new activity just for asking permissions is a huge overkill.

@samagra14 Yes I also share that thought. I did so for two purposes :

Avoid putting so much code (permissions are a little different logic) from main Activity's logic.
Restrict user to get to main Activity unless he gives the permissions. (We dont have anything in app that can be done without permissions)

Its a little unusual from what we see in tutorials, I dont see any potential threat with it. What do you think?

@samagra14 Tell me if you still want this to be in MainActivity, I will do it. Doesnt need much efforts!


Tampering timing attack with CRYPTO_memcmp usage.

even though the name is a bit misleading, closer to bcmp in fact,
also the main difference, it does not stop going through the buffer
upon the first mismatch.

Yes I ve been pushing too far for no gain. :-)


[MINOR] Fixing spotbug fail - removing unused variable.

Fixing nit in c6633a157eec1712116d294eb3785a96cba4e331.
This commit break spotbug check with the "Dead store to isFreshAssignment in org.apache.kafka.clients.consumer.internals.AbstractStickyAssignor.generalAssign(Map, Map)"
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Hello @ableegoldman
It looks like your patch breaks spot bug check.
I prepared oneliner fix for it.
Can you, please, take a look?


Add challenge readme

Related to https://github.com/Sage-Bionetworks/rocc-schemas/pull/180
Preview

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
114 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.6%) to 42.836%

Files with Coverage Reduction
New Missed Lines
%

openapi_server/models/challenge_platform_all_of.py
13
0%

openapi_server/controllers/challenge_controller.py
101
13.44%

Totals

Change from base Build<PHONE_NUMBER>:
-0.6%

Covered Lines:
1423

Relevant Lines:
3322

💛  - Coveralls


Illusion Effect Reference

Where could I find the papers for the implemented illusion effect? I am specifically looking for the Ponzo and the Simultaneous Contrast illusion. (like which paper states that inclined lines would give a stronger effect than vertical lines in the Ponzo illusion)
Thank you for the awesome package!

We are just in the process of finalizing a validation of the illusions, that we hope to be able to put-up as a preprint within a few weeks :)
Here's a preliminary results figure, where you can see the the effect of illusion strength on error rate:

Dear Dominique,
Thank you for your prompt response! This looks amazing. For each illusion, you have two pairs of illusions. May I ask why is that? Is that another variation of the illusion and which pair are those graphs for?
I'm a Machine Learning researcher at MIT CSAIL that is conducting research in illusion and big models. And, this library has helped me a lot! However, I am having so problems with illusion in general because I'm coming from a computer science background. I just wonder if you have a paper list for that illusion strength distortion available somewhere that I could refer to? I could send you an email if it is better!
Thank you a lot!


fixed highlight and made it dynamic based on url

Fix #5
🔗 Test URLs:

Before: https://main--caesars-careers--hlxsites.hlx.page/careers
After: https://issue-5-header-fixes--caesars-careers--hlxsites.hlx.page/careers

📝 Description:
Highlight menu items based on current URL and fixed the animation on the highlights

Code LGTM - Do you want to look at the logo misalignment too in this PR?

I fixed that in the CSS.. which misalignment you talking about?

If you look at https://www.caesars.com/careers it's the same. the logo is not misaligned, it is intentionally bigger than the rest of the nav and stands out and overflows down the nav a bit overlaying the hero.


Add OperatorCondition class

Signed-off-by: Erkan Erol<EMAIL_ADDRESS>Short description:
Operator Condition was added into operator-sdk with this PR https://github.com/operator-framework/operator-lifecycle-manager/pull/1875 and it is used by OLM now.
hco-operator started to use it with this PR https://github.com/kubevirt/hyperconverged-cluster-operator/pull/1414
What this PR does / why we need it:
We need this class in upgrade tests.

/approve

/verified


Map view friendly Data format

Hi Jose, thanks for great Work.
I use Maps view plugin to graphically display my addresses on the map. Would it be possible to generate the geo coordinates in the options like Maps view can create a property:

Instead of 4.75443,12.87666
That would be very helpful and the plugins would work hand in hand.
Thanks
Frank

Thanks for the kind words! This should be an easy change. Just to verify, you want the exact format from the screenshot?

Hey, thanks for your answer. Exact in this format:

Thanks!
Nice Day
Frank
Am Fr., 23. Feb. 2024 um 20:35 Uhr schrieb Jose Alvarez <
@.***>:

Thanks for the kind words! This should be an easy change. Just to verify,
you want the exact format from the screenshot?
—
Reply to this email directly, view it on GitHub
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/issues/5#issuecomment-1961882686,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFFXYJZYDKYARWYYGEHDTYTYVDVPLAVCNFSM6AAAAABDUPCI4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRRHA4DENRYGY
.
You are receiving this because you authored the thread.Message ID:
<jose-elias-alvarez/obsidian-geocoding-properties/issues/5/1961882686@
github.com>

Please make it as an separate properties...
i mean your version:  Property1: 49.39211515x 7.00510515
and the new: Property2:
Am Fr., 23. Feb. 2024 um 20:49 Uhr schrieb Frank Groß <
@.***>:

Hey, thanks for your answer. Exact in this format:

Thanks!
Nice Day
Frank
Am Fr., 23. Feb. 2024 um 20:35 Uhr schrieb Jose Alvarez <
@.***>:

Thanks for the kind words! This should be an easy change. Just to verify,
you want the exact format from the screenshot?
—
Reply to this email directly, view it on GitHub
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/issues/5#issuecomment-1961882686,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFFXYJZYDKYARWYYGEHDTYTYVDVPLAVCNFSM6AAAAABDUPCI4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRRHA4DENRYGY
.
You are receiving this because you authored the thread.Message ID:
<jose-elias-alvarez/obsidian-geocoding-properties/issues/5/1961882686@
github.com>

Closed by abc5b2a and published in 1.1.0.

Hi, Jose,thanks for updating, but i can`t see any changes... lg frank Am So., 25. Feb. 2024 um 16:39 Uhr schrieb Jose Alvarez < @.***>:
…
Closed by abc5b2a <abc5b2a> and published in 1.1.0. — Reply to this email directly, view it on GitHub <#5 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/BFFXYJZV7NDVVQPBQX6DOWLYVNLL3AVCNFSM6AAAAABDUPCI4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRSHE3TQMBQGY . You are receiving this because you authored the thread.Message ID: <jose-elias-alvarez/obsidian-geocoding-properties/issues/5/1962978006@ github.com>

It will take a while for Obsidian's repository to pick up the new version (at least a few days, I think).

The new version was updated...
Jose Alvarez @.***> schrieb am So., 25. Feb. 2024,
19:41:

Hi, Jose,thanks for updating, but i can`t see any changes... lg frank Am
So., 25. Feb. 2024 um 16:39 Uhr schrieb Jose Alvarez < @.***>:
… <#m_-1104269976499283334_m_-5487525830962571826_>
Closed by abc5b2a
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/commit/abc5b2a39c06d54c3fabe551801c0e0c1a250b06
<abc5b2a
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/commit/abc5b2a39c06d54c3fabe551801c0e0c1a250b06>
and published in 1.1.0. — Reply to this email directly, view it on GitHub <#5
(comment)
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/issues/5#issuecomment-1962978006>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFFXYJZV7NDVVQPBQX6DOWLYVNLL3AVCNFSM6AAAAABDUPCI4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRSHE3TQMBQGY
. You are receiving this because you authored the thread.Message ID: <
/issues/5
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/issues/5
/1962978006@ github.com>
It will take a while for Obsidian's repository to pick up the new version
(at least a few days, I think).
—
Reply to this email directly, view it on GitHub
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/issues/5#issuecomment-1963025330,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFFXYJ6X27C2Y4RY7DFJERLYVOAXPAVCNFSM6AAAAABDUPCI4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRTGAZDKMZTGA
.
You are receiving this because you authored the thread.Message ID:
<jose-elias-alvarez/obsidian-geocoding-properties/issues/5/1963025330@
github.com>

The last release might have gotten messed up, could you try the latest one? It adds a new map_view_link property with the format you requested.

Hello, thank you for your effort...I'm sorry, I can't see any change. Am I
doing something wrong? Am I too stupid?
LG Frank
Am So., 25. Feb. 2024 um 21:49 Uhr schrieb Jose Alvarez <
@.***>:

The last release might have gotten messed up, could you try the latest
one? It adds a new map_view_link property with the format you requested.
—
Reply to this email directly, view it on GitHub
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/issues/5#issuecomment-1963057016,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFFXYJYCCKAPBJYW3LBEQRLYVOPV3AVCNFSM6AAAAABDUPCI4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRTGA2TOMBRGY
.
You are receiving this because you authored the thread.Message ID:
<jose-elias-alvarez/obsidian-geocoding-properties/issues/5/1963057016@
github.com>

Hello, thank you for your effort...I'm sorry, I can't see any change. Am I doing something wrong? Am I too stupid? LG Frank Am So., 25. Feb. 2024 um 21:49 Uhr schrieb Jose Alvarez < @.***>:
…
The last release might have gotten messed up, could you try the latest one? It adds a new map_view_link property with the format you requested. — Reply to this email directly, view it on GitHub <#5 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/BFFXYJYCCKAPBJYW3LBEQRLYVOPV3AVCNFSM6AAAAABDUPCI4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRTGA2TOMBRGY . You are receiving this because you authored the thread.Message ID: <jose-elias-alvarez/obsidian-geocoding-properties/issues/5/1963057016@ github.com>

Not at all, did you check for updates in the Obsidian UI? The app doesn't update plugins automatically so you'll have to check by going to Settings -> Community Plugins -> Check for Updates.

Version 1.1.1 is installed
Jose Alvarez @.***> schrieb am Mo., 26. Feb. 2024,
18:26:

Hello, thank you for your effort...I'm sorry, I can't see any change. Am I
doing something wrong? Am I too stupid? LG Frank Am So., 25. Feb. 2024 um
21:49 Uhr schrieb Jose Alvarez < @.***>:
… <#m_-2075334327378134969_>
The last release might have gotten messed up, could you try the latest
one? It adds a new map_view_link property with the format you requested. —
Reply to this email directly, view it on GitHub <#5 (comment)
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/issues/5#issuecomment-1963057016>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFFXYJYCCKAPBJYW3LBEQRLYVOPV3AVCNFSM6AAAAABDUPCI4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRTGA2TOMBRGY
. You are receiving this because you authored the thread.Message ID: <
/issues/5
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/issues/5
/1963057016@ github.com>
Not at all, did you check for updates in the Obsidian UI? The app doesn't
update plugins automatically so you'll have to check by going to Settings
-> Community Plugins -> Check for Updates.
—
Reply to this email directly, view it on GitHub
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/issues/5#issuecomment-1964706683,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFFXYJ6CNZQTUGVRLHEHOBTYVTAUNAVCNFSM6AAAAABDUPCI4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRUG4YDMNRYGM
.
You are receiving this because you authored the thread.Message ID:
<jose-elias-alvarez/obsidian-geocoding-properties/issues/5/1964706683@
github.com>

Frank Groß @.***> schrieb am Mo., 26. Feb. 2024, 19:03:

Version 1.1.1 is installed
Jose Alvarez @.***> schrieb am Mo., 26. Feb. 2024,
18:26:

Hello, thank you for your effort...I'm sorry, I can't see any change. Am
I doing something wrong? Am I too stupid? LG Frank Am So., 25. Feb. 2024 um
21:49 Uhr schrieb Jose Alvarez < @.***>:
… <#m_-6322842491702218338_m_-2075334327378134969_>
The last release might have gotten messed up, could you try the latest
one? It adds a new map_view_link property with the format you requested. —
Reply to this email directly, view it on GitHub <#5 (comment)
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/issues/5#issuecomment-1963057016>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFFXYJYCCKAPBJYW3LBEQRLYVOPV3AVCNFSM6AAAAABDUPCI4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRTGA2TOMBRGY
. You are receiving this because you authored the thread.Message ID: <
/issues/5
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/issues/5
/1963057016@ github.com>
Not at all, did you check for updates in the Obsidian UI? The app doesn't
update plugins automatically so you'll have to check by going to Settings
-> Community Plugins -> Check for Updates.
—
Reply to this email directly, view it on GitHub
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/issues/5#issuecomment-1964706683,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFFXYJ6CNZQTUGVRLHEHOBTYVTAUNAVCNFSM6AAAAABDUPCI4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRUG4YDMNRYGM
.
You are receiving this because you authored the thread.Message ID:
<jose-elias-alvarez/obsidian-geocoding-properties/issues/5/1964706683@
github.com>

I will try a new Installation
Jose Alvarez @.***> schrieb am Mo., 26. Feb. 2024,
20:05:

Hmm not sure what to say then, I just tried installing the plugin on a
fresh vault and I see the new property. Maybe try uninstalling and
reinstalling? Beyond that I'm not sure I can help.
—
Reply to this email directly, view it on GitHub
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/issues/5#issuecomment-1964962292,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFFXYJ37F5HU5MVKATDVGR3YVTMH5AVCNFSM6AAAAABDUPCI4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRUHE3DEMRZGI
.
You are receiving this because you authored the thread.Message ID:
<jose-elias-alvarez/obsidian-geocoding-properties/issues/5/1964962292@
github.com>

Okay thats it.
Thank you very much.
Frank Groß @.***> schrieb am Mo., 26. Feb. 2024, 20:13:

I will try a new Installation
Jose Alvarez @.***> schrieb am Mo., 26. Feb. 2024,
20:05:

Hmm not sure what to say then, I just tried installing the plugin on a
fresh vault and I see the new property. Maybe try uninstalling and
reinstalling? Beyond that I'm not sure I can help.
—
Reply to this email directly, view it on GitHub
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/issues/5#issuecomment-1964962292,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFFXYJ37F5HU5MVKATDVGR3YVTMH5AVCNFSM6AAAAABDUPCI4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRUHE3DEMRZGI
.
You are receiving this because you authored the thread.Message ID:
<jose-elias-alvarez/obsidian-geocoding-properties/issues/5/1964962292@
github.com>

Thanks
Am Mo., 26. Feb. 2024 um 20:19 Uhr schrieb Frank Groß <
@.***>:

Okay thats it.
Thank you very much.
Frank Groß @.***> schrieb am Mo., 26. Feb. 2024, 20:13:

I will try a new Installation
Jose Alvarez @.***> schrieb am Mo., 26. Feb. 2024,
20:05:

Hmm not sure what to say then, I just tried installing the plugin on a
fresh vault and I see the new property. Maybe try uninstalling and
reinstalling? Beyond that I'm not sure I can help.
—
Reply to this email directly, view it on GitHub
https://github.com/jose-elias-alvarez/obsidian-geocoding-properties/issues/5#issuecomment-1964962292,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFFXYJ37F5HU5MVKATDVGR3YVTMH5AVCNFSM6AAAAABDUPCI4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRUHE3DEMRZGI
.
You are receiving this because you authored the thread.Message ID:
<jose-elias-alvarez/obsidian-geocoding-properties/issues/5/1964962292@
github.com>

FYI, there was an issue with the plugin that made the new property invisible. The issue should now be resolved for any future additions, but I apologize for the trouble!


start Building server... runs forever

Environment
Nuxt CLI v3.0.0-rc.3                                                                                10:44:31
RootDir: /Users/david/projects/finder/churchtools-finder                                                                                10:44:33
Nuxt project info:                                                                                10:44:33

Operating System: Darwin
Node Version:     v16.13.2
Nuxt Version:     3.0.0-rc.3
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      runtimeConfig, css, build, buildModules, intlify, serverMiddleware, render
Runtime Modules:  -
Build Modules<EMAIL_ADDRESS>

👉 Report an issue: https://github.com/nuxt/framework/issues/new                                                                                10:44:33
👉 Suggest an improvement: https://github.com/nuxt/framework/discussions/new
👉 Read documentation: https://v3.nuxtjs.org
Describe the bug
We upgraded to nuxt rc3. First it worked. But now 2 days later without changing anything running nuxi build does not terminate. It just hangs at start Building server...
Here is how it looks:
npm run build

> ChurchTools<EMAIL_ADDRESS>build
> nuxi build

Nuxt CLI v3.0.0-rc.3                                                                                10:41:34
ℹ Vite client warmed up in 2191ms                                                                                10:41:41

 WARN  [vite:css] Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated.                                                             10:41:43
651|  @media print {
652|  	/* Prevent printers from removing background-images of controls. */
653|  	.leaflet-control {
   |       ^
654|  		-webkit-print-color-adjust: exact;
655|  		color-adjust: exact;

 WARN  [vite:css] Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated.                                                             10:41:45
651|  @media print {
652|  	/* Prevent printers from removing background-images of controls. */
653|  	.leaflet-control {
   |       ^
654|  		-webkit-print-color-adjust: exact;
655|  		color-adjust: exact;

ℹ Client built in 10043ms                                                                                10:41:49
ℹ Building server...                                                                                10:41:49

 WARN  @fortawesome/fontawesome-free doesn't appear to be written in CJS, but also doesn't appear to be a valid ES module (i.e. it doesn't have "type": "module" or an .mjs extension for the entry point). Please contact the package author to fix.

 WARN  reflect-metadata doesn't appear to be written in CJS, but also doesn't appear to be a valid ES module (i.e. it doesn't have "type": "module" or an .mjs extension for the entry point). Please contact the package author to fix.

 WARN  [vite:css] Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated.                                                             10:41:50
651|  @media print {
652|  	/* Prevent printers from removing background-images of controls. */
653|  	.leaflet-control {
   |       ^
654|  		-webkit-print-color-adjust: exact;
655|  		color-adjust: exact;

✔ Server built in 2511ms                                                                                10:41:51
✔ Generated public .output/public                                                                                nitro 10:41:51
start Building server...

I can see that esbuild is running and not terminating. Is there a debug flag where i can see more infos?

Still having this issue
Repo: https://github.com/AlejandroAkbal/Image-to-Text-OCR/tree/b05032c0e4242cfcc8da9b0da3e670db5844633d

@AlejandroAkbal This is probably a different issue. See https://github.com/nuxt/cli/issues/169.


docs(code): optimize docs

Checklist
List of tasks you have already done and plan to do.

[ ] Fix linting errors
[ ] Tests have been added / updated (or snapshots)

Change information
Describe your modifications here.
Issues
The issues you want to close, formatted as close #1.
Related Links
Links related to this pr.

@cszhjh @haoziqaq  I added missing word-wrap prop in docs. please review it. Thanks!


Degrade 3DES to MEDIUM in SSL2

The SWEET32 fix moved 3DES from HIGH to MEDIUM, but omitted SSL2.

Added "Need CLA" label...unless we consider this change trivial? Also should this go to 1.0.1 as well?

Oh...sorry. I see you added the 1.0.1 label already.

I have no problem looking at this as trivial.

Looks trivial to me as well.

Ok. Agreed - trivial. Removing "Need CLA" label.

OpenSSL_1_0_1-stable 9d9e053, OpenSSL_1_0_2-stable 6d69dc5
Thanks!


Remove redundant documentation base-typography.md / typography_base.md

It seems that base-typography has been renamed into typography-base some time ago and then the two files existed in parallel.

I don't think there is anything wrong with this PR. The Travis error is The command "npm install" failed and exited with 1 during .


2nd printed grid in console is staggered

it prints it out of its original format, because it is in the wrong style? object/number/string etc. should we stringify?

Used .join function - solved


Exception in pub build

@MikeMitterer
@munificent
@nex3
Mike, I noticed that there was an exception on pub build  execution before pub get.
Natalie would like to get more information about this problem.

Can you move this to the pub repo?

How to move issues?

I think there's a tool for it, but it's probably easier to just close this and open a new one.

I'm fine with having it the SDK repo.

That's not really the question. Pub issues should be filed against pub, not against the SDK.

Oh, sorry.
It looked as a question to me.
https://github.com/dart-lang/pub/issues/1362

Moved to https://github.com/dart-lang/pub/issues/1362


Vars are not appended to the request body when using POST

During POST, the variables should be URL encoded information placed in the request body, not appended to the URL. It also requires specific headers:
"Content-type": "application/x-www-form-urlencoded
"Content-length": ...
Additional details.
http://www.openjs.com/articles/ajax_xmlhttp_using_post.php

Reading the spec it seems that the headers should be set automatically? - https://fetch.spec.whatwg.org/#concept-bodyinit-extract
I've already cooked up a solution for attaching POST variables however, so that much is done.

You are correct that it will default to application/x-www-form-urlencoded. There are other encodings which you may want/need to support, though they are less common in AJAX. Would you want multi-part for file uploads?
Off the top of my head, I don't recall the details for all the other methods.
You linked the spec for fetch rather than XHR.
https://dvcs.w3.org/hg/xhr/raw-file/tip/Overview.html
https://developer.mozilla.org/en-US/docs/Web/API/XMLHttpRequest

IIRC the "living standard" is now specifying XMLHttpRequest in terms of fetch, which explains how I got there. That being said it's probably better to stick closer to XHR.
It should be possible to support multi-part/formdata/blobs without too much issue, but I'll probably have to add a method or two - I'd like to keep the library from "guessing" what you're doing, but some data inference may be necessary.

So I did some testing and reading up - Any URL can include paramaters (url.com?a=1&b=2), but POST can include data in the request body as well (for forms etc). The solution I have arrived at is adding a .data() method that allows setting the request body. At the moment it just supports form-urlencoded type data, but multipart and formdata should be trivial to implement.

Fix for 1.0.1 is in 72121099e48539c37dd527667d743c995193375f
I'll target additional functionality for 1.1


[AutoPR resources/resource-manager] Fix the ARM R2062 error for Microsoft.Resources

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4600

This PR has been merged into https://github.com/Azure/azure-sdk-for-js/pull/504


Update pci-req7.md

Changed "etc." to "and so on."
Removed period from second bullet under 7.1.1.

@EdPrice-MSFT,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/compliance/pci-req7.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[0.9.5] automatic_control.py can not run

hello, everyone.
In Carla0.9.5,After I run the script automatic_control.py. It shows the error as below, anyone can help me? thanks!

pygame 1.9.4
Hello from the pygame community. https://www.pygame.org/contribute.html
Traceback (most recent call last):
File "automatic_control.py", line 88, in
from agents.navigation.roaming_agent import RoamingAgent
ImportError: No module named agents.navigation.roaming_agent

You need to add carla to your PYTHONPATH.
Example:
export PYTHONPATH=$PYTHONPATH:carla-0.9.5/PythonAPI/carla/

You need to add carla to your PYTHONPATH.
Example:
export PYTHONPATH=$PYTHONPATH:carla-0.9.5/PythonAPI/carla/

thans very much! your way helps me a lot.

For windows this will help.
install package using pip utility as the following
go to directory PythonAPI/carla/agents
and create setup.py file and write the following code in that file
from distutils.core import setup
setup(name='agents',
        version='0.9.5', #doesn't matter I guess
        py_modules=['agents'],
        )

Now go to anaconda terminal and do cd to directory PythonAPI/carla
and apply following command to install agents
pip install -e agents
This will install agents and resolve the agents.navigation not found issue


Fix build error with the default buildMajor and buildMinor

In the current build pipeline template, we use empty string as the default value of PRODUCT_MAJOR, PRODUCT_MINOR, BUILD_MAJOR, BUILD_MINOR and pass all these properties to msbuild.
In our verisons.props, we have algorithms to generate BUILD_MAJOR and BUILD_MINOR based on time stamp if they are not present or if they are empty strings. So the original expectation was the below command would work with BUILD_MAJOR, BUILD_MINOR auto generated based on the time stamp:
msbuild compression.sln /p:platform="x86" /p:configuration="Release" /p:PRODUCT_MAJOR=1 /p:PRODUCT_MINOR=0 /p:BUILD_MAJOR= /p:BUILD_MINOR=
However, the build actually fails with empty BUILD_MAJOR and BUILD_MINOR.
The root cause is that the command line passed-in properties are always set AFTER the property setting defined in versions.props and hence the former always wins.
See the exact same issue discussed here:
https://social.msdn.microsoft.com/Forums/vstudio/en-US/4e777a1e-2627-40cf-b624-88f8f6dd7e52/how-do-i-test-for-empty-properties-set-on-the-command-line?forum=msbuild
So the workaround of using the auto-generated BUILD_MAJOR and BUILD_MINOR is NOT passing them to msbuild:
msbuild compression.sln /p:platform="x86" /p:configuration="Release" /p:PRODUCT_MAJOR=1 /p:PRODUCT_MINOR=0
To implement the workaround, I added a Powershell task to parse the msbuild-related variables and to generate a new MsbuildArgs pipeline variable for the following msbuild tasks.
There might be a better way to directly consume template parameters in Powershell task. But for now, I use pipeline variables instead for simplicity. We can improve it later if necessary.

@bariscaglar ping on this PR


Regression/bug: esbonio support broken since 4.0.0

We're using https://github.com/swyddfa/esbonio for linting, preview etc, as are probably many others. This combination does no longer work since sphinx-needs 4.0.0. It works fine in sphinx-needs 2.1.0.
Observation
sphinx-needs throws https://github.com/useblocks/sphinx-needs/blob/6c97c47002a09c197da1a4969bded6028a5fb337/sphinx_needs/data.py#L781
Reason GUESS
Esbonio keeps sphinx "open" and runs build() on changes. I'm guessing this is in conflict with marking needs "post_processed" as they are revisted within the same sphinx process again.

It seems we found a quick fix:
def setup(app: Sphinx):
    app.connect("env-before-read-docs", fix_sphinx_needs_restart_behavior)
    checks.add_warnings(app)

def fix_sphinx_needs_restart_behavior(_app: Sphinx, env, _documents):
    """ See https://github.com/useblocks/sphinx-needs/issues/1350 """
    # sphinx-needs 2.1.0
    if hasattr(env, "needs_warnings_executed"):
        env.needs_warnings_executed = False

    # sphinx-needs 4.1.0
    if hasattr(env, "_needs_warnings_executed"):
        env._needs_warnings_executed = False

    # sphinx-needs 4.1.0
    if hasattr(env, "_needs_is_post_processed"):
        env._needs_is_post_processed = False


Invalid spork address specified

I am trying to frok Dash,
But after I change the address prefix, dash-qt shows this message to me.
How can I generate new customized spork addresses?

Hi,
You need to generate a new keypair (masternode genkey), and replace the spork address in chainparams.cpp. Or you could re-serialize the current spork address with your new address prefix, but you wouldn't have the corresponding spork private key if you did that.
Good luck!

Hi @nmarley, thank you
1- The masternode genkey is not available in my fork cause I don't have valid spork address,
2- when I use it with dash itself (not my fork) it gives me a private key, how can I get address from private key?

OK my problem sloved,
I disabled spork address checking at init.cpp then from dash-qt Debug Console I created key-pairs using masternode genkey, importprivkey THEKEY "name", getaddressesbyaccount "name"`.

You need to generate a new keypair (masternode genkey), and replace the spork address in chainparams.cpp.

How is this done?  And how many spork addresses need to be created, just for main net, test net, reg net?  Does this depend on address prefixes as well, or will it work by generating keys with openssl and somehow getting a private key in WIF and spork address from them?
Thanks!

OK my problem sloved,
I disabled spork address checking at init.cpp then from dash-qt Debug Console I created key-pairs using masternode genkey, importprivkey THEKEY "name", getaddressesbyaccount "name".

This worked for me too, thank you!


LatestTransactions Component - Table rows don't update on first pagination click.

Broken Pagination in LatestTransactions Component
Problem
First click of right pagination arrow updates title, but not records in data grid.
Reproduce

Load the page
View the first 25 records.
Click ➡
Title updates from "25 transactions" to "25-50 transactions"
Still viewing first 25 records though
Click ⬅, Title updates correctly, records stay the same
Click ➡, Title + Records update properly
Click ➡, Title + Records update properly
Click ⬅, Title + Records update properly
Click ⬅, Title + Records update properly

Resolved by https://github.com/sortxyz/react-components/pull/24


Service Request by Megan Donato

[ ] Qualtrics Service Request Form was received from User
[ ] e-mail acknowledging that User's Qualtrics Service Request Form was sent
[ ] Problem NOT RESOLVED
[ ] Problem PARTIALLY RESOLVED
[ ] Problem FULLY RESOLVED and Issue is Closed
[ ] Qualtrics Post-Service Request Survey was sent to User

Use comments feature (below) to document:

A clear and concise description of the solution.

A clear and concise description of any alternative solutions or features you've considered.

Any additional context, such as screenshots about the problem.

Hi Jeff
We have to document every request for internal data analytic purposes.
This was not done to poke the bear.
In fact, I BCC'd you on the email to Megan alerting her that you've begun working on this already.
Given that Megan filled out a Qualtrics Service Request Form. I am obliged to create the service request.
My actions are all about documentation, accountability, and creating a paper trail,...

Ok. Thanks. Let’s skip assigning the issue to me if it is a duplicate.
From: donriepurcell @.***
Sent: Thursday, October 7, 2021 11:54 AM
To: dlounsbu/NYHEALing-IPAtracker-support @.>
Cc: Jeff Bazinet @.>; Assign @.***>
Subject: Re: [dlounsbu/NYHEALing-IPAtracker-support] Service Request by Megan Donato (#54)
This Message is being sent from outside of FDRHPO or NCI.  Please do not click on links or download attachments without properly verifying them.
Hi Jeff
We have to document every request for internal data analytic purposes.
This was not done to poke the bear.
In fact, I BCC'd you on the email to Megan alerting her that you've begun working on this already.
Given that Megan filled out a Qualtrics Service Request Form. I am obliged to create the service request.
My actions are all about documentation, accountability, and creating a paper trail,...
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/dlounsbu/NYHEALing-IPAtracker-support/issues/54#issuecomment-937925387, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVIFNMAI3SSKPXUOZZ6YBULUFW7APANCNFSM5FRSULFQ.
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 electronic message is intended to be for the use only of the named recipient, and may contain information from the North Country Initiative that is confidential or privileged. If you are not the intended recipient, you are hereby notified that any disclosure, copying, distribution or use of the contents of this message is strictly prohibited. If you have received this message in error or are not the named recipient, please notify us immediately, either by contacting the sender at the electronic mail address noted above or by calling the North Country Initiative at<PHONE_NUMBER> x 23, and delete and destroy all copies of this message.


Delete double 'Config' section

Checklist

[x] I have made a material change to the repo (functionality, testing, spelling, grammar)
[x] I have read the [Contribution Guide] and my PR follows them.
[x] I updated my branch with the master branch.
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation about the functionality in the appropriate .md file
[x] I have added in line documentation to the code I modified

Short description of what this PR does:

Delete double 'Config' section in text.

If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.

 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.

Some checks were not successful


[SPARK-16813][SQL] Remove private[sql] and private[spark] from catalyst package

What changes were proposed in this pull request?
The catalyst package is meant to be internal, and as a result it does not make sense to mark things as private[sql] or private[spark]. It simply makes debugging harder when Spark developers need to inspect the plans at runtime.
This patch removes all private[sql] and private[spark] visibility modifiers in org.apache.spark.sql.catalyst.
How was this patch tested?
N/A - just visibility changes.

Test build #63037 has started for PR 14418 at commit 5fa81a9.

Test build #63037 has finished for PR 14418 at commit 5fa81a9.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class MonotonicallyIncreasingID() extends LeafExpression with Nondeterministic
case class SparkPartitionID() extends LeafExpression with Nondeterministic
case class AggregateExpression(
case class CurrentDatabase() extends LeafExpression with Unevaluable
class GenericInternalRow(val values: Array[Any]) extends BaseGenericInternalRow
class AbstractScalaRowIterator[T] extends Iterator[T]

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63037/
Test PASSed.

Merged build finished. Test PASSed.

cc @cloud-fan

LGTM, merging to master!

I've also backported this into branch-2.0 to avoid future merge conflicts.


[VL] Support IsNull filter pushdown

What changes were proposed in this pull request?
Previously, is_null filter was separated into remaining filter. With this PR, it can be pushed down into subfield filter.
How was this patch tested?
Verified with TPC-DS q76.

Thanks, @rui-mo , with this fix, we should not need to change velox, is this right?

@zhli1142015 That's true. But considering there's a similar function in Velox to extract conditions from remaining filter to subfield filter, maybe we can shift to that one when it covers enough cases in the long term. As for now, it looks all right to continue to improve Velox's implementation.

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_3791_time.csv
log/native_master_11_27_2023_08cdbee7e_time.csv
difference
percentage

q1
31.34
33.96
2.618
108.35%

q2
24.92
24.50
-0.422
98.31%

q3
36.03
36.70
0.673
101.87%

q4
38.03
36.44
-1.592
95.81%

q5
71.99
70.50
-1.490
97.93%

q6
6.80
7.29
0.490
107.20%

q7
86.12
83.48
-2.639
96.94%

q8
86.45
88.10
1.649
101.91%

q9
121.17
122.04
0.866
100.71%

q10
46.01
46.95
0.939
102.04%

q11
19.84
19.82
-0.023
99.88%

q12
25.38
26.08
0.694
102.73%

q13
46.48
48.07
1.590
103.42%

q14
16.85
15.26
-1.584
90.60%

q15
28.27
28.91
0.640
102.26%

q16
15.73
15.89
0.160
101.02%

q17
102.94
103.54
0.597
100.58%

q18
150.18
151.52
1.339
100.89%

q19
12.83
12.96
0.127
100.99%

q20
27.99
28.70
0.711
102.54%

q21
223.94
226.06
2.121
100.95%

q22
13.32
13.19
-0.123
99.07%

total
1232.62
1239.96
7.340
100.60%


Empty code

Please, add some code

Will do :+1:

Ok, I'll close it when I'll se the code.

I don't like the label, please, fix it too.

Can u fix this plz? My team is blocked bc this


Don't include LGPL docs in the docs if both embed-lgpl-docs and purge…

…-lgpl-docs features are selected
This makes usage of RLS/rust-analyzer on the repository much faster and
less annoying as the docs don't have to be included and removed on every
change.

CC @GuillaumeGomez @EPashkin

Thanks!


JSBIN 502 Gateway

Need this fixed asap

what is happening here? anybody know this?

See @js_bin on Twitter. Fixed now.


Rework current DNS logic for Agent

This PR repurposes the defunct DNS logic for use in Istio agent.
Note that this functionality is not active, as there is nothing requesting the DNS resource type yet.
Keeping the PRs small for easy reviewing.
I explicitly chose to use a custom proto (nds.proto) instead of individual service entries
as the overhead of marshalling and unmarshalling individual service entries was huge. Compared to that,
the DNS table that we ship is much more lightweight on CPU and memory - one map with all the required data,
that is convenient for istiod to produce and agent to consume.
Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[x] Does not have any changes that may affect Istio users.

@rshriram: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
ef777101c71586d49c950733e427f4ea1c8098ba
link
/test gencheck_istio

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

@rshriram: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
ef777101c71586d49c950733e427f4ea1c8098ba
link
/test gencheck_istio

release-notes_istio
ef777101c71586d49c950733e427f4ea1c8098ba
link
/test release-notes_istio

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

@rshriram: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
ef777101c71586d49c950733e427f4ea1c8098ba
link
/test gencheck_istio

release-notes_istio
ef777101c71586d49c950733e427f4ea1c8098ba
link
/test release-notes_istio

lint_istio
ef777101c71586d49c950733e427f4ea1c8098ba
link
/test lint_istio

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

@rshriram: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
ef777101c71586d49c950733e427f4ea1c8098ba
link
/test gencheck_istio

lint_istio
ef777101c71586d49c950733e427f4ea1c8098ba
link
/test lint_istio

release-notes_istio
2461c09a913d2f60ba70746cf396f38c55b16c50
link
/test release-notes_istio

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

@rshriram: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
ef777101c71586d49c950733e427f4ea1c8098ba
link
/test gencheck_istio

release-notes_istio
2461c09a913d2f60ba70746cf396f38c55b16c50
link
/test release-notes_istio

lint_istio
2461c09a913d2f60ba70746cf396f38c55b16c50
link
/test lint_istio

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

LGTM.

I am not sure this is a good idea:

We are now locked into our own API. I was thinking we could use envoys v3.DnsTable. This way, our implementation is pretty much transparent and we can swap out for envoy easily.
If we don't want to do that, this must be in istio/api with proper review...

We have the google CA proto in our repo - its undocumented anywhere else in the web. So there is precedent. We also had the Istio CA proto for the longest time.
Secondly, the DNS table in envoy is not sufficient for our needs as we need to express a bunch of metadata related to each entry (the k8s name/namespace so that the variations can be auto computed). The DNS table forces us to only use the envoy data format - which requires us to enumerate all possible entries. Also keep in mind that the consumer on the agent side is Istio agent (coredns/mkieg/dns code and not envoy. So it seems futile to use envoy dns table when its not going to be used on agent side). I briefly debated using service entries - but then we would be stuck with marshalling and unmarshalling each and every one of them - in a system with 1000 entries, it was contributing ot a lot of cpu and memory. The simpleton table seemed like the best fit.
Put another way, if we had our own debug channel between the agent and istiod (like the one proposed in a recent design doc (https://docs.google.com/document/d/11P6OXlITExJvFqzsWfFu9WhFLjY6uqOcCUKqPVgSSoc/edit#), we cannot continue to rely on envoy. DNS is on the same boat. Nor does this need to be publicly documented until there is an external consumer wanting to intercept - that is not the goal here either.


Enable capability to use >1 sector per cluster in ghostfat

@kaetemi ... maybe I'm missing something, but I think there needs to be a way to enable the use of the new >1 sector per cluster support.
For example, shouldn't the following:
https://github.com/adafruit/tinyuf2/blob/15fa707341df119583e5241e3acf91320859be82/src/uf2.h#L45-L47
Be changed to:
#ifndef CFG_UF2_FLASH_SIZE
    #define CFG_UF2_FLASH_SIZE          (4*1024*1024)    // TODO absolute max flash size
#endif
#ifndef CFG_UF2_NUM_BLOCKS
    #define CFG_UF2_NUM_BLOCKS          (0x10109)        // just under 32MB
#endif
#ifndef CFG_UF2_SECTORS_PER_CLUSTER
    #define CFG_UF2_SECTORS_PER_CLUSTER (1)
#endif

If so, I'll mark PR #90 as ready....

Yeah, that would be more practical. :)

Can we include the tested large value combination in the comments?

Yeha, there is still lots of TODO, we will move it to per ports/boards later on. But this look good

Can we include the tested large value combination in the comments?

Yeah, I think having a working/tested value combination would be great. It is removed recently (probably mistaken as obsolete value). Would you mind added it, maybe along with a bit of detail on your setup e.g like this combination appear as 1.5 GBs disk and tested on abc hardware would be helpful as well.

#define CFG_UF2_FLASH_SIZE          (256*1024*1024)
#define CFG_UF2_NUM_BLOCKS          (0x300000)       // 1.5 GB
#define CFG_UF2_SECTORS_PER_CLUSTER (64)

That's for 256MB flash, so 2x a 512MB UF2 file, and plenty for space remaining for the two text files (or for the remainder of a UF2 file with non-flash content).
Tested on RP2040, for flashing BT815-series graphics chip. (Maybe add a note on the memory required for the buffer that keeps the written block flags... CFG_UF2_FLASH_SIZE / 256 / 8 iirc? That's the bottleneck, it seems.)

#define CFG_UF2_FLASH_SIZE          (256*1024*1024)
#define CFG_UF2_NUM_BLOCKS          (0x300000)       // 1.5 GB
#define CFG_UF2_SECTORS_PER_CLUSTER (64)

That's for 256MB flash, so 2x a 512MB UF2 file, and plenty for space remaining for the two text files (or for the remainder of a UF2 file with non-flash content).
Tested on RP2040, for flashing BT815-series graphics chip. (Maybe add a note on the memory required for the buffer that keeps the written block flags... CFG_UF2_FLASH_SIZE / 256 / 8 iirc? That's the bottleneck, it seems.)

Great, would you make an PR to add these comment along with the note. CFG_UF2_FLASH_SIZE is from original uf2 bootloader for nrf52, there is still work to generalize this project. It is still in early stage, so yeah, we will revise it and update/remove it when needed.

@kaetemi -  It was my PR that removed the large commented-out values.  Sorry!
@hathach -  Having the CFG_* values is important ... I am close to having self-validation code working, at least to the point I must hand it to you for proper integration.  I will open a separate issue/PR for that. :)

@henrygab ah ok, actually, I have no plan/time to remove it anytime soon. But rather will evolve slowly when needed :) I am looking forward to your PR.

Mostly done, except for a couple build/CI steps I need help with.  See PR #94.


Propagate API models not having type unions

Previous core models had Union(str, List[str]) (e.g. for image representation uris), which were changed in the api to just List[str] to avoid simplify use/avoid ifs. Update downstream code to reflect the change.

In some cases (e.g. queries, programmatically interacting with multiple things with no extra knowledge of what the type is), easier to know how to interact with something if it has a single non-null type.
When populating (especially) an object, useful to allow type options at different granularities.


Custom Strategy to sell any ALTS based on Bitcoin price breaking support

Hi ,
I downloaded gekko but i cannot trade based on price of btc.
I wanted to create this simple strategy  but i don't know whether this is possible in gekko as gekko trades using signals and not based on price of btc.
Here is what i need .
Set a Bitcoin support price example 5000$
if price of btc goes below 5000$ sell an Altcoin.
If btc goes back above 5000$ buy the sold altcoin.
I think this will be helpful as all altcoins dump during btc breaking major support. I would have saved 20% today if this was possible.
I am sure others are looking for the same. Anyone can comeup with this.
Thanks.

your logic is apposite..you sell low and buy high ;D

I can't help you code this, but I maybe could advise that you should probably take this kind of inquiry to the forum ( https://forum.gekko.wizb.it/ ) instead of using the bug and issue tracker.

@Mytechshout  You want to buy/sell an altcoin taking the signals from bitcoin?

as @151henry151 said, please ask this on the forum. This is the bug tracker of the core software.


Add Transform widget to the Widget Catalog

Transform Widget needs to be implemented in the Widget Catalog. Make sure to include the description of the widget in the Description tab and add an argument to include link to the official docs of the widget (Refer to any example in homepage.dart for better understanding. Showcasing the various implementations of the widget including rotate, translate, scale, matrix, skew, 3D, etc is a must.
Please attach a screen recording of the same once done in your PR.

Hey i can work on this one too , can you pls assign this to me

Hey i can work on this one too , can you pls assign this to me

I am sorry but working on only one issue at a time is allowed. If the issue has not been assigned to anyone by the time you are done with yours, you can tag me to get assigned to this issue. Hope you understand.

Sure no problemo

Hey @AryanKhubchandani, can I work on this issue?

Hey @AryanKhubchandani, can I work on this issue?

Sure, I will assign it to you. Good Luck!


Look into Windows support

It seems discord.py for one had libopus DLLs directly in their package. A primary goal would be to mimic this.

[ ] check copyright and source of the libopus DLL binaries in the discord.py repo -- https://github.com/Rapptz/discord.py/tree/master/discord/bin
[ ] patch the opus package to include these DLLs, or alternatively add them in this repository
[ ] check that bindings in the opus package reliably works both on Linux and Windows. If the above step concluded that the DLLs are to be in this repository, make sure discord-haskell-voice reliably selects local bindings or the opus package bindings, using OS conditionals.

Looks like a tough issue. I'm leaving it on hold since I can imagine a bit of work might be needed for this, and my Windows machine isn't in the best condition (disk-space wise) to try install stack and develop.

Windows builds are currently being done in V2.4.0 pipelines. https://github.com/yutotakano/discord-haskell-voice/pull/16


Authenticator's Restore method being called on every paging request

I just  noticed this when looking at my chrome inspector but my custom authenticator's restore method is being called every-time I increment through pages on a table on my route.
This seems very wasteful and doesn't happen anywhere else in my application.  My only theory is that it has something to do with the way query params reload the route as I'm using query params to implement paging.
Has anyone seen this behavior before?  I'm not sure if this is some strange edge case that is causing ember-simple-auth to behave non-optimally or if my code is just wrong.
Happy to provide more information if someone has an idea of what to look for or look at.

Just isolated the cause down to calling refresh on the router from the controller.
So the following line in the controller:
this.get('target.router').refresh();

Causes the restore method to be called.  Any ideas why?  Does refresh bubble up all the way to the application route and trigger refreshes on everything?
(sorry in advance if this is not ember simple auth related.  I'm happy to move this to stack overflow if you think it's not related).
Thanks!

The session is restored in the application route's beforeModel method. So if the application route is refreshed I guess the beforeModel route runs again and thus the session is restored.

Doesn't seem ember-simple-auth related and I posted an issue over on the ember repo here:
https://github.com/emberjs/ember.js/issues/12707
For whatever reason refreshing a child route is causing all parent routes (including the application route) to reload which triggers ember-simple-auth to restore the session.
Closing this issue here.  Thanks!


Implement large responses

Currently, a request is passed as a buffer which means, it has to be in memory as a whole. The same goes for the response. Cause of protobuf restrictions, there is currently probably a limit of 64kb (has to be tested). It might be better to change the interface to use std::io::{Read, Write} and transfer everything in smaller chunks. This should avoid also some memory copying.

Uff! That was a hard one. The tests are running locally, but I'm not convinced that this is stable. Some refactoring will follow. HTTP responses should now never be completely in memory and streamed through the delix network. HTTP requests are still read completely - will create a separate issue.