Generated "bibtex" keys in "Extra" field not consistent with values in "Citation Key" field

Hello, I've just filed an error report # JW7F3246.
My Citation key format is [authorsAlpha:lower][year]
More than a year ago, I reported the issue https://github.com/retorquere/zotero-better-bibtex/issues/415 regarding "Duplicate citekeys generated in chronological order w.r.t. Date Added". You released a fix and things have worked since then.
The current issue is I've recently noticed the (occasional) inconsistency between the generated bibtex keys (by right click --> Generate BibTeX key) in the Extra field and the ones displayed in the Citation Key field of Zotero's right pane.
For example, I have a reference with bibtex key generated correctly as "lh2015" as displayed in the Extra field, while its Citation Key field displays "lh2015i" in ITALIC! Similarly for the reference with bibtex key of "lh2015a" being displayed in the Citation Key field as "lh2015e".
(Of course in the library I am having many references rendered "lh2015..." --- they're simply multiple chapters of a book of the same authors which were added to Zotero at the same Hour and Minute but at different Second, so according to the fix for issue #415, there should not be a problem here.)
Since the Citation Key field locates near the top of the right panel which is very convenient to read and copy, I hope you can make an effort to fix this issue.
Thank you for your excellent work!
Tod

Does this problem remain if you select any other reference and then select back the misbehaving one? That would tell me whether it's a display issue or a refresh issue.

The problem does remain, even if I restart Zotero. Let me know if you need further information.

This should be fixed now in the Z5 port. If it reappears, please open a new issue.


Improve SQL logging readability

This pull request improves the readability of SQL logs by formatting them in a more readable way. It also adds unit tests to ensure the correctness of the formatting function.

I have incorporated few more changes from my other PR, all are log improvements.


Autonomous improvements for states

Upper arm at a lower position relative to backdrop will reduce the bounce and thus increase the probability of scoring 20pts. Since Upper Arm is at a lower position, the angle to the backdrop is such that we need to drive closer to the backdrop.
Delay before crossing field from wing side is increased to 15s to avoid hitting RoboWolves during scrimmage at CERC
Before issuing a pull request, please see the contributing page.

Upper arm at a lower position relative to backdrop will reduce the bounce and thus increase the probability of scoring 20pts. Since Upper Arm is at a lower position, the angle to the backdrop is such that we need to drive closer to the backdrop.
Delay before crossing field from wing side is increased to 15s to avoid hitting RoboWolves during scrimmage at CERC
Before issuing a pull request, please see the contributing page.

These changes were merged into another branch and don't show here anymore


Update to verbiage

#7734 issue in sp-dev-docs mentions a typo to this. However, we should remove the SharePoint On-Premises description as PnP.PowerShell is not supported on-premises, only the older PnP Powershell Modules are.

Docs Build status updates of commit f6340e8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-pnp/sharepoint-pnp-cmdlets.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@JohanFreelancer9

@dariomws @tseward @get-itips

Docs Build status updates of commit d106aca:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: PullRequestMergeConflict] Cannot sync git repo to specified state. Please make sure the pull request is mergeable.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@dariomws

@tiburd Please merge this PR. Thanks

Docs Build status updates of commit 24590c2:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: PullRequestMergeConflict] Cannot sync git repo to specified state. Please make sure the pull request is mergeable.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit f8b751f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-pnp/sharepoint-pnp-cmdlets.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


OkHttp support

OkHttp is a very popular http client, especially on Android.
I'd like to use Betamax to record/replay network interactions that go through OkHttp for integration tests that run on an Android device or emulator so they are more consistent and repeatable.
I understand that Betamax as is doesn't work with OkHttp so I've forked Betamax and started working on the changes to make it possible.
It would basically work as a Interceptor that records/plays from the Tape using the same rules that exist on Betamax today.
Is this something you guys would consider taking as an open source contribution? Since there are quite a lot of differences, i'd keep it as a separate repository/artifact, maybe something like betamax-interceptor. How does that sound?
Thank for sharing this awesome tool!

I'd love to see that in Betamax, I started using OkHttp + Betamax quite recently as well, recording tapes in alright, but you ran into issues when you need to play back a recorded body.
So your solutions sounds nice :)

I have the work underway in a branch, working on fixing the failing unit tests now (thanks a lot for writing them 😄 ), will share when I have it in a working state

The code is now (temporarily) lives here https://github.com/felipecsl/walkman
It's currently alpha stability but feel free to use and send feedback 😃

Ok this has been officially open sourced as https://github.com/airbnb/okreplay


Simplifies PNG export, removes ChunkyPNG resizing

I've noticed that when specifying the resize_exactly_to option to as_png the resulting image would often be slightly distorted. For example, the following
image = RQRCode::QRCode.new("http://facebook.com", level: :l).as_png(resize_exactly_to: 80)

would produce the image

Notice the varying thickness around the position squares.
I've noticed that other QRCode generators (e.g. Google's now deprecated Charts API) don't have this issue so this is my attempt at fixing it.
This pull request removes the resize_exactly_to option in favour of size to make it more consistent with the other exporters. It also removes the call to ChunkyPNG's resize and instead accomplishes exact sizing by varying the amount of border space around the data portion of the code (similar to how Google's Chart API does it). So now, the following
image = RQRCode::QRCode.new("http://facebook.com", level: :l).as_png(size: 80)

will produce the image

Looks good =).
Could change so that resize_exactly_to is a alias for size so that we don't break any clients currently using it?

Good idea. I've added backwards compatibility with 92126faf9d4c60ebda8bc85f238bbb67f44f8ccf. Thanks.

Sorry that i missed it before, but do you think you could bring back the options :resize_gte_to and :module_px_size also?
Without them we are still not backwards compatible.

Hey sorry about that, I removed those in my version and I guess I forgot to add them back before making the pull-request. Those two options have been added back in and they retain the same functionality as before. Thanks for catching that.

Sorry for being so slow.
I have taken your code and made some changes. Have a look and tell me what you think?
https://github.com/bjornblomqvist/rqrcode/tree/googleis-png-export

No worries - I'm good with the changes you made, I tested them locally and everything worked as it should. I think you have a redundant module_px_size defined on L58 but other than that I think we're good. Thanks again for looking into this and your work with the project.

New version out with your improvement.


FEAT: UI for text2image

As we support stable diffusion model for text2image, a simple UI is needed to let users generate images on web pages. Like chat UI, we can launch a gradio page for a quick implementation.

@Bojun-Feng Hi, any interest? Or any suggestions we can discuss. :)

@aresnow1  Sounds like an interesting feature! However, I am worried about locally testing the UI on my computer, as my laptop has no GPU support and has only 8GB of RAM. If there are any small text2image or image2image models I can run on my laptop, I would be more than happy to implement the UI for it!

As for suggestions, we can probably look at this open source repo.
I envision the end product to look something like this Hugging Face space, except with bigger input text box.

Hi, could you try this one(https://huggingface.co/stabilityai/sd-turbo ) on your laptop?

I will create a PR to add sd turbo support.

https://github.com/xorbitsai/inference/pull/797


[zh]Sync content/zh/docs/setup/production-environment/tools/kubeadm/troubleshooting-kubeadm.md

Please help to review this PR. @tengqm

Please squeeze commits and resynchronize file content/zh/docs/setup/production-environment/tools/kubeadm/troubleshooting-kubeadm.md.

Rather than having contributors randomly pick a page to sync, please check the issues #33474, #33475, #33476, #33477, #33478, #33479 and sign on tasks there. It would help save your time and avoid conflicts when working on different pages.

Please don't mix two languages in one PR.

Please don't mix two languages in one PR.

Ok, I've split up the other languages pr

The troubleshooting-kubeadm.md file has more changes to be sync'ed than the ones carried by this PR.

The troubleshooting-kubeadm.md file has more changes to be sync'ed than the ones carried by this PR.

Ok, I will resubmit the PR


Support Cognito Auth Provider

Description
Support Cognito on front-end level.
Use case/motivation
No response
Related issues
No response
Confidential information

[X] I confirm that do not share any confidential information

Documented:

Docs
Core


Get video converter working

Converter must be able to:

[ ] Fetch video data from the database
[x] Download file from raw S3 store
[x] Convert raw video into chunking-supporting format
[x] Upload file back to converted S3 store

File format specs:

H.264/ MPEG-4 AVC seems to be most popular
H.265 has started replacing H.264 so might play around with that
source: https://imagekit.io/blog/video-file-formats/

Decided against fetching video data from the database and any operation involving the database will be done in the backend.


Newbie issue

hi
running on bookwork debian raspbian
ran bash setup.sh, and set up account info
now whilst in the folder under user i enter bash noip-renew-(MY debian username) or (noip username)
tried both and i get file not found
tried running python script but get permission denied
can you please help with this step
thankyou

update tried:
theskid@raspberrypi:~$ cd /usr/local/bin theskid@raspberrypi:/usr/local/bin $ bash noip-renew-theskid.sh noip-renew-theskid.sh: line 10: ./noip-renew.py: Permission denied theskid@raspberrypi:/usr/local/bin $ sudo bash noip-renew-theskid.sh noip-renew-theskid.sh: line 10: ./noip-renew.py: Permission denied theskid@raspberrypi:/usr/local/bin $ su Password:  su: Authentication failure theskid@raspberrypi:/usr/local/bin $ su Password:  root@raspberrypi:/usr/local/bin# bash noip-renew-theskid.sh noip-renew-theskid.sh: line 10: ./noip-renew.py: Permission denied root@raspberrypi:/usr/local/bin#


RSX parser suggestion: syn-rsx

Hey, your macro is probably feature complete and there's no interest in changing it – but in case you have to expand functionality: I just wanted to leave an hint for my syn-rsx crate here. I think it'd be great to combine efforts for that basic building block, and others like sauron and leptos are successfully using it.
Sorry for the shameless plug and obviously feel free to close this issue.

This would be amazing! I've just started using render and the fact that you have to quote every string is a bit annoying because it causes quite a bit of noise.


CustomEnchantment needs to provide an override for hashCode to not break java spec.

equals and hashCode must be deterministic. This fixes an issue with hashmaps.

Latest code re-write includes hashCode


A question about this action

Hello,
I am a little confused about what this action does. Does it explicitly set the status to the value provided in the state parameter? Sorry for the confusion, any help would be appreciated thank you.

I now understand what this action is doing. Sorry the confusion.

@shanebenlolo thanks a lot for bringing up the question, tbf I am also not sure what this actions is doing for us and I really appreciate it if you could shed some light on this topic since it is not quite clear to me just by reading the README.md.
@Sibz could you please add more info so that newbies like me do not have to spend too much time just to understand what it does?
Many thanks

@shanebenlolo thanks a lot for bringing up the question, tbf I am also not sure what this action is doing for us and I really appreciate it if you could shed some light on this topic since it is not quite clear to me just by reading the README.md.
@Sibz could you please add more info so that newbies like me do not have to spend too much time just to understand what it does?
Many thanks

Hey unfortunately I cannot recall what this was used for as it was many years ago. I should have documented my findings, sorry about that. Best of luck.

Found it @shanebenlolo, it is useful in scenarios that you wanna specify the state of a commit. It's easier to explain it with this example;

I wanna run my e2e tests, so first I need to specify the state of the commit (last commit that we pushed, assuming the action gets executed on push) to pending.
Then I run e2e tests
Now I wanna change the state of the commit to success or failure or error based on the second step

Now it is documented :grin:


File downloading appears to be broken

I tried versions 0.12, 0.12.1, 12.2 on Windows and all of them seem to be ignoring attachments without any error
EDIT: I just tried a different creator and it's working. Maybe the issue is videos? The creator not working is oda1ga.

@matrix07012
Thank you for your bug reporting!
This bug has been fixed, and we released a new version. Please check it out.
https://github.com/hareku/fanbox-dl/releases/tag/v0.13.0


Types for values

How would this feature be useful?

This feature can be useful in preventing the entry wrong type of values.
Describe the solution you'd like

Maybe, we can define a schema with the keys and the expected types .
db.set_schema({"name":str, "age":int, "email":str, "bio":Optional[str]})

Describe alternatives you've considered
¯_(ツ)_/¯

Lets say you are continuously the add function to add values to the DB. With the way PysonDB is currently designed It will take more that 4 mins to add 5000 values that has 3 keys. Now if we implement type checking it will take even more time.
Onstro db - type checker <- this was a proof of concept project that i did that implements type checking, even with optimisation and using pandas the max throughput is 10_000 datapoints a second.
Summary:
It is not feasible with the current way pysondb works. (For condition where you have large data coming in really fast,)

More over data needs to be validated during adding and updating values. If in the future if we implement this  48. It will even more difficult;t to manage types.

An easier way to do this will be to check the data before you add it. At the end of the day Its your DB and you have to mange it. 😂😂

ah hmm. That was just a suggestion. I thought that adding types would be better for the developer as they don't want to check types manually. Should I close the issue or ?


[Bug]: When apply continue_final_message for OpenAI server, the "echo":false is ignored

FIX: #10111

Test 1:
# cat test.py
from openai import OpenAI

client = OpenAI(
    base_url="http://localhost:8000/v1",
    api_key="token-abc123",
)

completion = client.chat.completions.create(
    model="meta-llama/Llama-3.2-3B-Instruct",
    messages=[{
        "role": "user",
        "content": "tell me a common saying"
    }, {
        "role":
        "assistant",
        "content":
        "Here is a common saying about apple. An apple a day, keeps"
    }],
    extra_body={
        "continue_final_message": True,
        "add_generation_prompt": False
    })

print(completion.choices[0].message)

# python test.py
ChatCompletionMessage(content=' the doctor away.', refusal=None, role='assistant', audio=None, function_call=None, tool_calls=[])

Test2
# cat test.py
from openai import OpenAI

client = OpenAI(
    base_url="http://localhost:8000/v1",
    api_key="token-abc123",
)

completion = client.chat.completions.create(
    model="meta-llama/Llama-3.2-3B-Instruct",
    messages=[{
        "role": "user",
        "content": "tell me a common saying"
    }, {
        "role":
        "assistant",
        "content":
        "Here is a common saying about apple. An apple a day, keeps"
    }],
    stream=True,  # this time, we set stream=True
    extra_body={

        "continue_final_message": True,
        "add_generation_prompt": False
    })

for chunk in completion:
    # print(chunk)
    print(chunk.choices[0].delta.content)
    print("****************")

# python test.py

****************
 the
****************
 doctor
****************
 away
****************
.
****************

****************

# cat test.py

from openai import OpenAI

client = OpenAI(
    base_url="http://localhost:8000/v1",
    api_key="token-abc123",
)

completion = client.chat.completions.create(
    model="meta-llama/Llama-3.2-3B-Instruct",
    messages=[{
        "role": "user",
        "content": "tell me a common saying"
    }, {
        "role":
        "assistant",
        "content":
        "Here is a common saying about apple. An apple a day, keeps"
    }],
    stream=True,  # this time, we set stream=True
    extra_body={
        "echo": True,
        "continue_final_message": True,
        "add_generation_prompt": False
    })

for chunk in completion:
    print(chunk.choices[0].delta.content)
    print("****************")

#  python test.py

****************
Here is a common saying about apple. An apple a day, keeps
****************
 the
****************
 doctor
****************
 away
****************
.
****************

****************

Thanks for fixing this!

The test looks good, thanks again!


XcodeVersion interprets version 10.0 as version 1.0

Platform: MacOS Mojave
Compiler: Apple LLVM version 10.0.0 (clang-1<IP_ADDRESS>)
I was attempting to reinstall node after updating my Mac to the new developer release, and I kept getting an issue that my compiler was out of date. After digging for a while, I began to play around with the functions in gyp, and I noticed that the issue resides in pylib/xcode_emulation.py.
In said file, on line 1273, the script attempts to take the output of xcodebuild -version and translate that into a numerical value that the program can use. To test my theory, I printed the version of before and after that statement. The output was as follows:
>>> import gyp.xcode_emulation
>>> gyp.xcode_emulation.XcodeVersion()
Xcode 10.0
100
('0100', '10L176w')

As you can see, the system is interpreting version 10.0 the same as it would version 1.0, causing anything that is based off of this to think that the system is out of date.
The line currently exists as version = version.split()[-1].replace('.', '') I can rewrite this and make a pull request on this repo. I just wanted to make sure there was an issue in place before I make any changes.

See https://github.com/nodejs/node-gyp/issues/1454. This was fixed by https://github.com/nodejs/node-gyp/pull/1455 and released in node-gyp 3.6.3.


Broken dependency

I get this error on installation:
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/vitepress
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/vitepress-plugin-mermaid
npm ERR!   dev vitepress-plugin-mermaid@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

Try deleting package-lock.json and npm_modules folder.
then
npm install -g npm
npm i

remove vitepress from package.json since it will be installed as a peer dependency
delete node_modules and package-lock..
npm i
it worked for me.

And for me! Thanks @emersonbottero Great work.


Eq method should not escape characters

The eq method is escaping special characters, with the result in that case return ever false

@fabrizio-cannizzaro can you post a small reproducer? Thanks

Here a snippet:
SearchStream searchStream = entityStream.of(MeetingDetailDocument.class)
if (organizingEntity != null) {
stream.filter(MeetingDetailDocument$.ORGANIZING_ENTITY.eq(organizingEntity));
}
organizing entity is a String.
if we pass"S&D" to eq method, it will be escaped to "S&D" and return false, because in Redis the value is not escaped.

I believe this may have something to do with the tokenization rules in RediSearch https://redis.io/docs/interact/search-and-query/advanced-concepts/escaping/

You are right, the problem was caused by the index: if they already and are added new fields the index is not recreated at startup


Please, add book "Python for OSINT. 21 day course for beginners"

https://github.com/cipher387/python-for-OSINT-21-days (link to PDF in the repo)

All correct. Thank you.

Perfect!
I will add it as soon as I can 😃

Here I am!
Before making a pull request, I want to be sure that I have put in your right name 😺
Below you can see a screenshot of the temporary changes I made on my forked repo:

Is everything fine or do you want to correct something? Maybe a capital C in the cyb_detective?

@eshellman
Maybe, I don't know if it's correct!, is it better to add the link to the GitHub repo of the course in the English course list and the pdf in the English book list?

GitHub repo to English course list

Perfect. These changes could be fine.
Let's see what @eshellman thinks! :)


Add an index to the "Installation guide"

Feature Description
Add an index to the "Installation guide" https://qgis.org/resources/installation-guide/.
Reference to Feature
No response

This one should now be closed, right?


Create a hook for sending a toast

Is your feature request related to a problem? Please describe.
Currently, there's no way to send a toast.
Describe the solution you'd like
A hook that includes that gives you a function to send a toast when called. I believe that using a library such as
https://react-hot-toast.com/ would be beneficial.
Additional context
Probably use the library, and style the toast to the one we currently have

@kharann you showed me how to code, now I hate myself.


How to check code inside php files?

I found the hint about eslint-plugin-html in the readme, however I would like to check code inside some php files and did not find anything similar - using eslint-plugin-html does not work for php files, it unfortunately produces parsing errors.
Yes, I know JS in PHP files is a very bad practice - this is a legacy code base and that is why I would love to fix all the JS inside all files with standard --fix *.php.

Not sure, but I would broaden the search to see if anyone has gotten eslint looking inside PHP files.

Yes, I see there are overlapping responsibilities with this use case and it is not totally clear where it should be handled - why should a JS linter take care of PHP tags? Also PHP tags in JS are simply syntax errors, so eslint does the right thing to report them as syntax errors.
But fixing things is sometimes done in situations where people did not care about standards or even correct syntax (unfortunately), so I feel like a solution for this kind of problem might have a place here, but I am not sure - I am just asking, not arguing.
I just found eslint-plugin-exclude-php-tags and will try to use this with standard - I will report back if it helps me in my specific case which involves fixing that bad practice of 'PHP tags inside JS inside PHP files with HTML and JS' - a horrible scenario, believe me! But ignoring PHP tags is not what I really want - I would like to have it fixed.
Otherwise I will try some sed to fix that mess, but it would be really nice to have stabdard handle it .

Unfortunately you might have to put a custom eslint stack together, using the shareable configs directly.  But yeah, if you find a way to do it we could add it to the FAQ.

I think @bcomnes is right; This will need to be done with eslint directly (with eslint-config-standard) since its such a particular use case.
I'm closing this issue for now. If you have questions on or need further advice please feel free to reply to the thread!


JSON encoder is necessary

The pusher_client_python had a configuration to a custom json encoder. This new module does not provide this feature.

Your PR has been merged and is now in version 1.1.0 :)


telemetry for tde user actions

This PR

Fixes the tde Teleemtry view name.
expand on the tde telemetry to add the user actions on the UX.

Views/Actions
TdeConfigurationDialog

TdeConfigurationUseADS
TdeConfigurationIgnoreADS

TdeMigrationDialog

TdeMigrationSuccess
TdeMigrationClientException
TdeMigrationFailures

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
7 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.01%) to 42.213%

Files with Coverage Reduction
New Missed Lines
%

extensions/sql-database-projects/src/common/httpClient.ts
1
66.18%

extensions/sql-database-projects/src/dialogs/publishDatabaseDialog.ts
2
52.88%

extensions/notebook/src/jupyter/serverInstance.ts
4
75.5%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
30867

Relevant Lines:
67866

💛  - Coveralls

@junierch the issue was fixed by Alex in this commit: https://github.com/microsoft/azuredatastudio/commit/fd8d8b0a272e3247554f154a7f37de4158e3f103
so no action needed from your side.


I can't import Unity SDK. The Unity IDE doesn't seem to recognize the SDK.

无法在Unity IDE中导入这个Unity SDK，怎么解决呢，多谢


Add datasets checksum

Hey ,Regarding the issue ( #492 ) .
I have added a new folder checksums to datasets folder.
I have modified all custom datasets classes to fetch corresponding checksum for url  from checksums folder.
Regarding checksums folder

One file for every dataset
File name as "dataset class name" + ".txt"
Each file has url and hash_value separated by space for every url of that dataset

MD5 hash taken for  below datasets from  #492

VCTK
Speech commands
YesNo

SHA256 hash taken for below datasets from  tensorflow datasets repo

LIBRISPEECH
LJSPEECH

Couldn't find hash values for COMMONVOICE online.
For now COMMONVOICE will not perform the checksum validation.But it will function as it is before.
Tested if downloads are starting  for all datasets . All downloads are starting.
For YesNo dataset i have completed downloaded  and Verified if checksum is working by altering a checksum character.
@vincentqb  could you review these changes

Changes completed . @vincentqb could you review these changes

Thanks for working on this! Overall, the code looks good to me. I see the checksums for VCTK, SpeechCommands, and YesNo match what's listed in #492.
I'll merge this PR after the upcoming release so that we have some time to double check the checksums.

Given that we have URLs and Checksums for datasets it might make sense, at some point, to introduce a separate .yaml or .json config file to store this information instead of including it into the implementation.

Given that we have URLs and Checksums for datasets it might make sense, at some point, to introduce a separate .yaml or .json config file to store this information instead of including it into the implementation.

As discussed above, since most of the datasets only have 1-2 URLs supported, we decided to keep the checksums in the implementation.


Use of undeclared identifier 'RLMPropertyTypeArray' error

Hello!
I have just update my pods and with the new version of Realm (3.0.0) this identifier is not available, so RLMObeject+Copying.m file is not compiling... any help?

I had same issue and found on their release page https://github.com/realm/realm-cocoa/releases that they deprecated this enum. You can now just replace
if (property.type == RLMPropertyTypeArray)
to
if(property.array)

Thanks! It solves the problem

thanks!


AOMAnalyzer doesn't work with the latest version of AOM

I build a correct JS decoder with the latest version of AOM. It build it, but it doesn't work either on windows and Linux.
I build the decoder with emscripten version 1.36.13 (recent version of emscripten compiles a decoder version that doesn't work with AOMAnlyser).
Here it what the Analyzer gives me:

I found a solution. If we use the latest aom version (not the 1.0.0 tagged) and build the JavaScript decoder it will work (and it is compatible with a 1.0 aom encoded video)


Setup: Install RVM and Bundler

@jimconner opened #5. He found the version of ruby that he had installed on his machine was not the same version as we have been using on this project. To avoid confusion we have agreed that a particular version of ruby and bundler should be installed on the behalf of users.
Acceptance Criteria:
as part of the setup script install:

Install RVM
Ruby 2.4.3
bundler

Now that the courseware is containerised, there is no need to do this any more.


Alias and Import

In my local project when i use 'import' and 'alias' with some module, the auto complete for they doesn't works
Ex:
defmodule MyProject.MyModule do
   import MyProject.AnotherModule
   alias MyProject.AliasModule, as: FastAccess

   def my_func do
      FastAccess.func() # here
      func_by_another_module() # and here, the auto complete doesn't works
   end
end

Obs: I use Atom in Windows 8.1

Yes. But that's a problem in Elixir itself. I can't do anything about it except for writing the whole thing myself
Check this out
https://github.com/elixir-lang/elixir/issues/2519

wow...
Ok, Thank you :)

No problem :)
Glad You're using my extension
Cheers

Hi everyone, the problem with aliases is already fixed there: https://github.com/elixir-lang/elixir/pull/3280. :bowtie:

Hmm.. even though I am running Elixir 1.2.2, with the alias stuff fixed and all, the autocomplete doesn't seem to work.


Remove the install files from the docker image

👍

Closing this ticket. We have another ticket that captures the updated approach to solving this problem.

Closing this ticket. We have another ticket that captures the updated approach to solving this problem.


Starting zookeeper ... FAILED TO START

Hi,
Thanks a lot for the great image.
However, I found some issues during scale up/down # of replicas.

scale-up works every time, but scale-down and then scale-up will see new replicas failed to start.

It gets:
"No quorum of new config is connected and up-to-date with the leader of last commmitted config - try invoking reconfiguration after new servers are connected and synced"
when:
zkServer-initialize.sh --force --myid=$ZOO_MY_ID
zkServer.sh start
zkCli.sh -server $ZOO_SERVER:2181 reconfig -add "server.$ZOO_MY_ID=$ZOO_MY_IP:2888:3888;$ZOO_PORT"
zkServer.sh stop
It seems the new replica cannot reconfig -add itself, when leader still has config that contains server.n for removed replicas. The new replica will eventually fail to start: "java.lang.RuntimeException: My id 184 not in the peer list" because it didn't successfully run reconfig -add itself.

Always sees Starting zookeeper ... FAILED TO START when followings are run:

zkServer-initialize.sh --force --myid=$ZOO_MY_ID
zkServer.sh start
zkCli.sh -server $ZOO_SERVER:2181 reconfig -add "server.$ZOO_MY_ID=$ZOO_MY_IP:2888:3888;$ZOO_PORT"
zkServer.sh stop
Although I have also seen followings, even with Starting zookeeper ... FAILED TO START. So scale-up somehow works.
Committed new configuration:
server.153=<IP_ADDRESS>:2888:3888:participant;<IP_ADDRESS>:2181
server.154=<IP_ADDRESS>:2888:3888:participant;<IP_ADDRESS>:2181
server.10=<IP_ADDRESS>:2888:3888:participant;<IP_ADDRESS>:2181
server.11=<IP_ADDRESS>:2888:3888:participant;<IP_ADDRESS>:2181
server.12=<IP_ADDRESS>:2888:3888:participant;<IP_ADDRESS>:2181
version=100000006
ZooKeeper JMX enabled by default
Using config: /conf/zoo.cfg
Stopping zookeeper ... STOPPED

Hi @ming2377 ,
thanks for your detailed feedback. We will look into it.

Hi @ming2377 ,
your issues were caused by invalid re-configuration of the cluster. We refactored the way we handle re-configuration in case of scale up/down or failed containers. Now, only the Leader of the cluster is responsible to reconfigure it, while before every new container was trying to do the job.

Thanks very much for the refactoring. You contribution is huge and highly appreciated.
I pulled latest from master branch and "docker build" an image to test-out.
Although, I am able to scale up replicas, say from 3 to 5, but replica 4 and replica 5 are both not joining the cluster.
I found that leader container /var/log has logs contain the same error.
Node is leader.
Getting and saving /zookeeper/config to /usr/local/bin/configuration.
server.32=<IP_ADDRESS>:2888:3888:participant;<IP_ADDRESS>:2181
server.30=<IP_ADDRESS>:2888:3888:participant;<IP_ADDRESS>:2181
server.31=<IP_ADDRESS>:2888:3888:participant;<IP_ADDRESS>:2181
Error. Exiting.

replica 4 and replica 5 container /var/log has logs contain the followings.
Node is not a member of a cluster.
Total execution time: 0s.

It appears the 5 seconds cronjob running zookeeper-cleanup.sh failed on leader container. Therefore, the reconfig did not happen, thus, replica 4 and 5 are running, but not part of cluster.
I try troubleshoot a bit more, I don't see any bad usage of calculateTime that can cause the "Error. Exiting." case. So I am puzzled.
Thanks again.

@ming2377
We fixed the issue that caused the unexpected exit of the re-configuration job and also did some enhancements in the logs.
If you continue having issues please provide:

the docker service create command
the docker service scale command
logs from the zookeeper leader located at /var/log

Thanks.

Closing the issue at seems to be resolved.


Dependabot config 20

This should fix #19051
I've gotten one warning that more than 250 folders is not supported. That could be a problem for us.
It this works, I'll make similar updates in dotnet-api-docs and samples

/cc @mairaw  Have you seen any syntax to specific multiple folders for *.*proj files?


Launch play.spotify.com / Soundbounce when the extension button is clicked

For now it's a functional message which is displayed to tell the user to open play.spotify.com and log in first, but it would be much better if when you clicked the button, it opened spotify for you.
Even better if it could open spotify for you, wait for you to be logged in and everything loaded, then launch Soundbounce!
Even more better if it could open both in a new window, or in a window which isn't just a normal browser window but has a Soundbounce title and such (I believe this kind of thing is actually not too hard from a chrome extension, but I need to do more digging)

Extensions window API: https://developer.chrome.com/extensions/windows
However, I reckon it might be nicer to make SB into a Chrome "app" which would give it more control;

Yep all good ideas, thanks Andrew. I actually quite SB in a normal tab, so this should be an option for users if we enable launching in a sep window.


Separate matched number args causes weird behavior

I made a simple command shown below
import {Command} from 'discord-akairo';
import {Message} from 'discord.js';

export default class BruhCommand extends Command {
	constructor() {
		super('bruh', {
			aliases: ['bruh'],
			description: {content: 'Get the mean of a set of numbers.'},
			category: 'utility',
			args: [
				{
					id: 'numbers',
					match: 'separate',
					type: 'number',
					prompt: {start: 'What numbers should be averaged?', retry: 'Invalid numbers provided, please try again', optional: false}
				}
			]
		});
	}

	async exec(message: Message, {numbers}: {numbers: number[]}): Promise<Message | undefined> {
		console.debug(numbers);
		if (numbers.length > 0) {
			return message.util?.send((numbers.reduce((a, b) => a + b) / numbers.length).toLocaleString());
		}
	}
}

This simply collects an array of numbers and returns the mean of them.
When the command is run with valid numbers, it works as expected.
When the command is run with invalid input (ex. a b c) it does the following

Prompts for valid input several times (sometimes once, sometimes 3 times)
Runs the command anyway, except the number array is an array of Flags

// Sample log output of `args`
[ Flag { type: 'cancel' } ]

The number of Flags passed to args is equal to the number of times correct input was prompted from a retry.

Fixed in #139.


CoreProtect Stats?

Hello,
could you maybe add a plugin to generate Stats using the core protect plugin? So that when using it there is no extra stats plugin needed.
Thx
Timo :)

This is on my todo list and have already started somewhat on this. First I'm revamping the plugins system to allow for more dynamic mysql database layouts. The main problem though is the speed. CoreProtect is designed to log changes, not stats.

I agree with @robinp7720, My CoreProtect database is currently about 5Gb. It would take ages to index all a users data, and make stats out of that. It may be a one- time conversion thing, but that might be better of to the developer of the stats plugin.


Invalid credentials provided. Missing API key or an invalid API key was provided

Description
I'm attempting to use this role to install the New Relic infrastructure agent, and I have supplied my API key and my account ID, but I'm getting Invalid credentials provided. Missing API key or an invalid API key was provided when it gets to the Run CLI install task.
Steps to Reproduce
- name: Run the newrelic.newrelic_install role to install the newrelic agent
  # https://docs.newrelic.com/docs/infrastructure/install-infrastructure-agent/config-management-tools/configure-infrastructure-agent-using-ansible/
  include_role:
    name: newrelic.newrelic_install
    apply:
      environment:
        NEW_RELIC_API_KEY: "{{ tableau_newrelic_license_key }}"
        NEW_RELIC_ACCOUNT_ID: 123456
        NEW_RELIC_REGION: US
  vars:
    targets:
      - infrastructure
      - logs
    tags:
      environment: sandbox
      name: "sandbox-tableau"
      system: tableau

Expected Behavior
The Ansible role should install the agent without error.
Relevant Logs / Console output
fatal: [localhost]: FAILED! => {
    "ansible_job_id": "959606008271.92330",
    "changed": true,
    "cmd": "/usr/local/bin/newrelic install -y -n infrastructure-agent-installer,logs-integration --tag nr_deployed_by:ansible-install,environment:sandbox,name:sandbox-tableau,system:tableau",
    "delta": "0:04:58.576067",
    "end": "2023-08-18 19:16:05.419307",
    "finished": 1,
    "invocation": {
        "module_args": {
            "_raw_params": "/usr/local/bin/newrelic install -y -n infrastructure-agent-installer,logs-integration --tag nr_deployed_by:ansible-install,environment:sandbox,name:sandbox-tableau,system:tableau",
            "_uses_shell": true,
            "argv": null,
            "chdir": null,
            "creates": null,
            "executable": null,
            "removes": null,
            "stdin": null,
            "stdin_add_newline": true,
            "strip_empty_ends": true,
            "warn": false
        }
    },
    "msg": "non-zero return code",
    "rc": 1,
    "results_file": "/root/.ansible_async/959606008271.92330",
    "start": "2023-08-18 19:11:06.843240",
    "started": 1,
    "stderr": "level=fatal msg=\"could not fetch license key for account 123456:, license key:  401 response returned: Invalid credentials provided. Missing API key or an invalid API key was provided.\"",
    "stderr_lines": [
        "level=fatal msg=\"could not fetch license key for account 123456:, license key:  401 response returned: Invalid credentials provided. Missing API key or an invalid API key was provided.\""
    ],
    "stdout": "",
    "stdout_lines": []
}

Your Environment

Ubuntu 20.04 on EC2 on AWS

ansible --version
ansible [core 2.13.11]
  config file = /etc/ansible/ansible.cfg
  configured module search path = ['/root/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/lib/python3.8/dist-packages/ansible
  ansible collection location = /root/.ansible/collections:/usr/share/ansible/collections
  executable location = /usr/local/bin/ansible
  python version = 3.8.10 (default, May 26 2023, 14:05:08) [GCC 9.4.0]
  jinja version = 3.1.2
  libyaml = True

Additional context
When I run this role in verbose mode using ansible-playbook -vvv, I can see in the verbose output that account ID and the API key are both being passed as environment variables and they appear to be the correct values. Eg:
ASYNC POLL on localhost: jid=959606008271.92330 started=1 finished=0
<<IP_ADDRESS>> EXEC /bin/sh -c 'echo ~root && sleep 0'
Using module file /usr/local/lib/python3.8/dist-packages/ansible/modules/async_status.py
<<IP_ADDRESS>> PUT /root/.ansible/tmp/ansible-local-76989117yt828/tmpigrobw28 TO /root/.ansible/tmp/ansible-tmp-1692385876.7790117-92311-55812131229027/AnsiballZ_async_status.py
<<IP_ADDRESS>> EXEC /bin/sh -c 'chmod u+x /root/.ansible/tmp/ansible-tmp-1692385876.7790117-92311-55812131229027/ /root/.ansible/tmp/ansible-tmp-1692385876.7790117-92311-55812131229027/AnsiballZ_async_status.py && sleep 0'
<<IP_ADDRESS>> EXEC /bin/sh -c 'NEW_RELIC_API_KEY=redactedapikeyishere NEW_RELIC_ACCOUNT_ID=123456 NEW_RELIC_REGION=US NEW_RELIC_CLI_SKIP_CORE=1 /usr/bin/python3 /root/.ansible/tmp/ansible-tmp-1692385876.7790117-92311-55812131229027/AnsiballZ_async_status.py && sleep 0'

I have obviously redacted my account number and API key in this issue. But both match what is displayed in https://one.newrelic.com/admin-portal/api-keys/home. The API key type is INGEST - LICENSE.
How can I troubleshoot this?
Thanks.

Nevermind, I'm unblocked.
Apparently the API key needs to be a User key, not a LICENSE - INGEST key. User keys apparently will have a NRAK prefix.

But... why?
Key documentation clearly states that user keys are not for that.

What madness is this?


clipboard dependency fixed

This closes #81

Thanks for PR!
New version published on npm.

Thanks for your quick update.


can't compile on arm architecture

What type of installation are you running?
Server
Which Linux OS are you using?
debian 11.7
Which version of LNXLink has the issue?
latest
The problem
Installation on arm64 fail with this error:
`      pyamdgpuinfo/_pyamdgpuinfo.c:1195:10: fatal error: xf86drm.h: File o direc                                                                          tory non esistente
1195 | #include "xf86drm.h"
|          ^~~~~~~~~~~
compilation terminated.
error: command '/usr/bin/aarch64-linux-gnu-gcc' failed with exit code 1
[end of output]
note: This error originates from a subprocess, and is likely not a problem wit                                                                          h pip.
ERROR: Failed building wheel for pyamdgpuinfo
Failed to build pyamdgpuinfo
ERROR: Could not build wheels for pyamdgpuinfo, which is required to install pyp                                                                          roject.toml-based projects
`
will be possible to exclude this module from dependencies ?
Thank you
Logs
No response

I'll have to rethink how the dependencies are installed, but for now you could install LNXlink using the development method and remove line on pyproject.toml that uses the dependency pyamdgpuinfo.

I've pushed an update on dev installation that imports the libraries automatically, so if you exclude the GPU on your config, it should not try to install it.

Yes ! It works.
Thank you


Separate icons for insiders

Trying this again. Manually validated the following scenarios on both stable and insiders builds to validate icons are correct and installers work as expected
Windows User
Windows System
Windows Zip
Mac OS
Linux Debian
(other linux packages were not tested, just verified that they exist in the drop)
Verified that the builds generated from this branch have all the expected artifacts as well.
See https://github.com/microsoft/azuredatastudio/pull/8078 for examples of what the new icon looks like.

@kburtram At this point it might be worth not doing this and just using mixin's since its all setup.

@anthonydresser  What's the process for getting that working?

if @kburtram agrees. just do a straight replacement of the icons (so in master we have the icons we want in insiders) then I'll update the mixin repo to have the correct icons for release.

and by do a straight replacement I mean don't do any of the code changes where you have branching behavior for stable vs insiders, so this pr should just be icon replacements.

What repo is this? Would we have to change our stable release builds to be from there instead then?

I am handling integrating the mixin into our build process. The resources are ready, I just need to add an additional step in our build process but I'm waiting on confirmation from @kburtram before I do this.

For what it's worth, I'd love to see us adopt mixins for more things. If we can go with that approach it's worth a small delay.

@anthonydresser if this is simple to do in the mixins I'd recommend doing it there.  No concerns on from me with updating the insiders icon.

@anthonydresser Alright, I updated this PR to just switch all the icons over to their insiders versions. Once you get your changes done for the mixin stuff either let me know or just merge it in yourself.

I think you can merge this in, I updated the resources in distro already, but had to stop my changes to the pipelines to do the mixin because of the hotfix. I'll get the mixin stuff in tomorrow or monday as my dri work.
But since we don't plan on releasing a stable release before then, this shouldn't have any impact, except to change the icons for insiders (which is desired). As a side note, are we going to have separate icons for dev builds? Or which icon's do we want to show up for dev builds out of master?

There's no plans for separate icons for dev builds at this point. The insiders icons was mostly for external users to be able to easily tell the difference when running side by side (which is less likely to be something people using dev builds are doing)


start dev script

Provided a convenience script for targeting the dev instance of Koku

Deploy preview for koku ready!
Built with commit bcc995fc83f4536a36dc99737109df4d38e37777
https://deploy-preview-57--koku.netlify.com

Codecov Report

Merging #57 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #57   +/-   ##
=======================================
  Coverage   89.05%   89.05%
=======================================
  Files          70       70
  Lines         530      530
  Branches       35       35
=======================================
  Hits          472      472
  Misses         56       56
  Partials        2        2

Continue to review full report at Codecov.

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


Return parents when searching for child metadata.

Closes #93

This solr query might not be very performant, but I couldn't get anything else to work

I guess the question is whether we'd rather have bad search-time performance (doing the join) or bad index-time performance (loading the children and indexing their values in the parent record).

I wish I could get it to work with a filter query

@escowles Yeah. I suspect since this is an ingest tool that we're less worried about query performance we can't even measure yet. @jpstroop?

This breaks the show view for scanned resources with parents. I'll work on that after lunch.

Fixed, ready for review


[Suggestion] Predicition: Night Battle HP Update

As for now, when entering the night battle, the one that being updated is the final HP on the below part of the kanmusu HP.
Why not we do this: update the actual HP (on the upper part) using the final value from the day battle, and the prediction below stay as is. This way we can track and guess the final HP based on the difference between the end of day battle and the prediction for night battle one?
Maybe @ReiFan49 can help explain the basic idea?

how about 3 parts?

api_data on night battle,
nowhps on each ship is based afterHp of day battle.
so the afterHp that reflects the night battle result, the currentHp reflects the day battle result.

yep @ReiFan49 is right. though I'm not sure where this falls, if still under @Javran's script, or the calculation just needs to be re-triggered when entering night phase

Night battle has its own "nowhps" (example), my script uses that field for both normal and night battles, accumulating damage through aerial combat, shelling, torpedo stages and remove them from nowhps.

hm wait, I remember a recent pull that deals with this or am I confusing it with something else


[Bug] Comment is not refocused on closing the comment thread and clicking on it again

Description

Add a comment at a position such that we need to scroll
Click on the comment card link from the sidebar
Close the comment thread, scroll away and click on the same card again
The comment card is not refocused

The issue is not fixed. When i followed the above-mentioned steps they could reproduce the issue. so reopening the issue

@rishabhsaxena, the comment is not refocused, correctly  when the user clicks on the comment card link from the sidebar screenshot:


Delayed self transition on parallel state is always treated as internal transition.

Description
Delayed self transition on parallel state is treated as internal transition even with internal: false. In effect it doesn't reenter itself and doesn't trigger entry/exit actions.
Expected Result
I expect an event log to be fired every second.
Actual Result
Event log is fired only once, when the machine initializes.
Reproduction
https://xstate.js.org/viz/?gist=d1ffc68cc1d29a9eb24e14b53be365b3
Additional context
XState 4.8.0

As far as I can tell by checking this quickly - it works correctly: https://codesandbox.io/s/quizzical-field-f45v6?file=/src/index.js . There might be some bug in the visualizer though.

The visualizer is using an older version of XState (4.6).

Thanks, indeed it works correctly on version 4.8, I tested it only on the visualizer.


Make sure stylus-library wysiwyg styles are applied

Like even though we're adding wysiwyg class, we need to make sure those styles are rendered.

Eh, I'm thinking it's ok to just expect the styling to come from nuxt global styles.  Or I have this package emit a CSS file that includes it.

Yeah, gonna leave this as the concern of bukwild-stylus-library/boilerplate


Race condition in netgraph

When I build netgraph with -race enabled:
go build -race github.com/ga0/netgraph

And then run on a .pcap file e.g.:
./netgraph -input-pcap trace_indexes_from_client_go.pcap

I get the following race condition report:
WARNING: DATA RACE
Read at 0x00c4200ae508 by goroutine 20:
  github.com/ga0/netgraph/ngnet.(*HTTPStreamFactory).runStreamPair()
      /Users/kjk/src/go/src/github.com/ga0/netgraph/ngnet/httpstreamfactory.go:44 +0x84

Previous write at 0x00c4200ae508 by goroutine 10:
  github.com/ga0/netgraph/ngnet.(*HTTPStreamFactory).runStreamPair()
      /Users/kjk/src/go/src/github.com/ga0/netgraph/ngnet/httpstreamfactory.go:44 +0x9a

Goroutine 20 (running) created at:
  github.com/ga0/netgraph/ngnet.HTTPStreamFactory.New()
      /Users/kjk/src/go/src/github.com/ga0/netgraph/ngnet/httpstreamfactory.go:71 +0x108c
  github.com/ga0/netgraph/ngnet.(*HTTPStreamFactory).New()
      <autogenerated>:1 +0xf9
  github.com/google/gopacket/tcpassembly.(*StreamPool).getConnection()
      /Users/kjk/src/go/src/github.com/google/gopacket/tcpassembly/assembly.go:502 +0x1f6
  github.com/google/gopacket/tcpassembly.(*Assembler).AssembleWithTimestamp()
      /Users/kjk/src/go/src/github.com/google/gopacket/tcpassembly/assembly.go:550 +0x23f
  main.runNGNet()
      /Users/kjk/src/go/src/github.com/ga0/netgraph/ng.go:175 +0x757

Goroutine 10 (running) created at:
  github.com/ga0/netgraph/ngnet.HTTPStreamFactory.New()
      /Users/kjk/src/go/src/github.com/ga0/netgraph/ngnet/httpstreamfactory.go:71 +0x108c
  github.com/ga0/netgraph/ngnet.(*HTTPStreamFactory).New()
      <autogenerated>:1 +0xf9
  github.com/google/gopacket/tcpassembly.(*StreamPool).getConnection()
      /Users/kjk/src/go/src/github.com/google/gopacket/tcpassembly/assembly.go:502 +0x1f6
  github.com/google/gopacket/tcpassembly.(*Assembler).AssembleWithTimestamp()
      /Users/kjk/src/go/src/github.com/google/gopacket/tcpassembly/assembly.go:550 +0x23f
  main.runNGNet()
      /Users/kjk/src/go/src/github.com/ga0/netgraph/ng.go:175 +0x757

This probably could be fixed by using atomic.AddInt32 for HTTPStreamFactory.seq
and:
WARNING: DATA RACE
Write at 0x00c4200c2230 by main goroutine:
  internal/race.Write()
      /usr/local/Cellar/go/1.10.3/libexec/src/internal/race/race.go:41 +0x38
  sync.(*WaitGroup).Wait()
      /usr/local/Cellar/go/1.10.3/libexec/src/sync/waitgroup.go:127 +0xf3
  main.(*NGServer).Wait()
      /Users/kjk/src/go/src/github.com/ga0/netgraph/ngserver.go:98 +0x3e
  main.runEventHandler()
      /Users/kjk/src/go/src/github.com/ga0/netgraph/ng.go:273 +0x14a
  main.main()
      /Users/kjk/src/go/src/github.com/ga0/netgraph/ng.go:286 +0x94

Previous read at 0x00c4200c2230 by goroutine 6:
  internal/race.Read()
      /usr/local/Cellar/go/1.10.3/libexec/src/internal/race/race.go:37 +0x38
  sync.(*WaitGroup).Add()
      /usr/local/Cellar/go/1.10.3/libexec/src/sync/waitgroup.go:70 +0x16e
  main.(*NGServer).Serve()
      /Users/kjk/src/go/src/github.com/ga0/netgraph/ngserver.go:144 +0x199

Goroutine 6 (running) created at:
  main.initEventHandlers()
      /Users/kjk/src/go/src/github.com/ga0/netgraph/ng.go:60 +0x404
  main.main()
      /Users/kjk/src/go/src/github.com/ga0/netgraph/ng.go:283 +0x33

This probably would require stopping listening for new requests before calling NGServer.Wait.

As the WaitGroup says:

... The main goroutine calls Add to set the number of goroutines to wait for. Then each of the goroutines runs and calls Done when finished. ...

This is the main cause.
Thank your for testing this program so patiently!


Added height to coinbase as per BIP34

As per BIP34, the first item in the coinbase transaction's scriptSig must be the height of the block mined, otherwise the block will be rejected.

Thanks, squashed and merged into d273fe51b866e5a0b41cf4f7096c7f7f9e8f74cf.


set precise window level for every app - or at least give an option for always on top

Summary of the new feature/enhancement
a feature that allows to control the window level(s) of (every) app(s) window(s)...
Or at least the option to make one window stay on top of others or all
Wasnt that promised or already there?

@patarok
as far as I know, Windows doesn't allow to set a fix level for a window, beside temporarily move them in a relative position.
Far always on top the issue is https://github.com/microsoft/PowerToys/issues/13
Closing this as duplicate
Thanks.


Settings sync is disabled because the current version (1.50.1, d2e414d9e4239a252d1ab117bd7067f125afd80a) is not compatible with the sync service

This error suddenly popped-up a few minutes ago and I cannot enable sync again on Stable (Insiders seems to be working fine for now). I cannot find any extension or VSCode update at this time

Version: 1.50.1 (user setup)
Commit: d2e414d9e4239a252d1ab117bd7067f125afd80a
Date: 2020-10-13T15:06:15.712Z
Electron: 9.2.1
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042

Steps to Reproduce:

Try to turn on Settings Sync

Error:
Settings sync is disabled because the current version (1.50.1, d2e414d9e4239a252d1ab117bd7067f125afd80a) is not compatible with the sync service. Please update before turning on sync.

Settings Sync log:
[...]
[2020-10-27 09:11:21.497] [userDataSync] [info] Auto Sync: Triggered by Activity
[2020-10-27 09:11:21.661] [userDataSync] [info] Settings: No changes found during synchronizing settings.
[2020-10-27 09:11:21.666] [userDataSync] [info] Keybindings: No changes found during synchronizing keybindings.
[2020-10-27 09:11:21.673] [userDataSync] [info] Snippets: No changes found during synchronizing snippets.
[2020-10-27 09:11:21.694] [userDataSync] [info] GlobalState: No changes found during synchronizing ui state.
[2020-10-27 09:11:21.960] [userDataSync] [info] Extensions: No changes found during synchronizing extensions.
[2020-10-27 09:11:21.962] [userDataSync] [info] Sync done. Took 371ms
[2020-10-27 09:11:49.466] [userDataSync] [info] Auto Sync: Triggered by Activity
[2020-10-27 09:11:49.571] [userDataSync] [info] Settings: No changes found during synchronizing settings.
[2020-10-27 09:11:49.577] [userDataSync] [info] Keybindings: No changes found during synchronizing keybindings.
[2020-10-27 09:11:49.585] [userDataSync] [info] Snippets: No changes found during synchronizing snippets.
[2020-10-27 09:11:49.610] [userDataSync] [info] GlobalState: No changes found during synchronizing ui state.
[2020-10-27 09:11:49.826] [userDataSync] [info] Extensions: No changes found during synchronizing extensions.
[2020-10-27 09:11:49.827] [userDataSync] [info] Sync done. Took 334ms
[2020-10-27 09:12:05.042] [userDataSync] [info] Auto Sync: Triggered by Activity
[2020-10-27 09:12:05.183] [userDataSync] [info] Settings: No changes found during synchronizing settings.
[2020-10-27 09:12:05.188] [userDataSync] [info] Keybindings: No changes found during synchronizing keybindings.
[2020-10-27 09:12:05.197] [userDataSync] [info] Snippets: No changes found during synchronizing snippets.
[2020-10-27 09:12:05.306] [userDataSync] [info] GlobalState: Updated global state ["workbench.activity.pinnedViewlets2"]
[2020-10-27 09:12:05.306] [userDataSync] [info] GlobalState: Updated local ui state
[2020-10-27 09:12:05.317] [userDataSync] [info] Deleting from backup /C:/Users/carloc/AppData/Roaming/Code/User/sync/globalState/20200927T085259.json
[2020-10-27 09:12:05.317] [userDataSync] [info] Deleting from backup /C:/Users/carloc/AppData/Roaming/Code/User/sync/globalState/20200927T085759.json
[2020-10-27 09:12:05.317] [userDataSync] [info] Deleting from backup /C:/Users/carloc/AppData/Roaming/Code/User/sync/globalState/20200927T085945.json
[2020-10-27 09:12:05.317] [userDataSync] [info] Deleting from backup /C:/Users/carloc/AppData/Roaming/Code/User/sync/globalState/20200927T090948.json
[2020-10-27 09:12:05.390] [userDataSync] [info] GlobalState: Updated last synchronized ui state
[2020-10-27 09:12:05.440] [userDataSync] [error] IncompatibleLocalContent (UserDataSyncError) syncResource:extensions operationId:unknown: Cannot sync extensions as its local version 4 is not compatible with its remote version 5
    at e.performSync (file:///C:/Users/carloc/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/code/electron-browser/sharedProcess/sharedProcessMain.js:442:788)
    at e._sync (file:///C:/Users/carloc/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/code/electron-browser/sharedProcess/sharedProcessMain.js:441:39)
    at async e.sync (file:///C:/Users/carloc/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/code/electron-browser/sharedProcess/sharedProcessMain.js:439:905)
    at async C.sync (file:///C:/Users/carloc/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/code/electron-browser/sharedProcess/sharedProcessMain.js:545:238)
[2020-10-27 09:12:05.562] [userDataSync] [info] Auto Sync: Disabled.
[2020-10-27 09:12:05.562] [userDataSync] [info] Auto Sync: Stopped
[2020-10-27 09:12:05.569] [userDataSync] [info] Did reset the local sync state.
[2020-10-27 09:12:05.569] [userDataSync] [info] Auto Sync: Turned off sync because server has extensions content with newer version than of client. Requires client upgrade.
[2020-10-27 09:13:00.994] [userDataSync] [info] Did reset the local sync state.
[2020-10-27 09:13:01.294] [userDataSync] [info] Settings: Stopped synchronizing settings.
[2020-10-27 09:13:01.295] [userDataSync] [info] Keybindings: Stopped synchronizing keybindings.
[2020-10-27 09:13:01.296] [userDataSync] [info] Snippets: Stopped synchronizing snippets.
[2020-10-27 09:13:01.299] [userDataSync] [info] GlobalState: Stopped synchronizing globalstate.

/duplicate
https://github.com/microsoft/vscode/issues/109470
Yeah this is expected because your insiders is now incompatible with your stable version. You can sync with stable again when stable is updated to 1.51


Handling of the New Auto Token Rotation Feature

Describe the solution you'd like
The new runner authentication method includes an auto-rotation feature (disabled by default). For this feature to function, the expiration interval must be configured within GitLab. Once the interval is set and the time is reached, the Gitlab runner automatically initiate a process to reset the token, with the new token stored in the configuration file. The SSM 'token counter' method will be removed in the upcoming version 8.0.0 of this module (As noted
 here  ). I am proposing that that implementation is not removed and instead is re-used to handle the new auto-rotation feature. Otherwise this module will not be compatible with Gitlab instances that have the auto-rotation feature enabled.
Describe alternatives you've considered
None
Suggest a solution
There are two possible solutions:
First Solution (easiest):
Mark the handling of token rotation as out of scope and add a disclaimer in the documentation.
Second Solution:
Do not remove the "usage counter" SSM parameter and use that along with the reset-token command. The process would be as follows:

SED config.toml for token_expires_at if it exists move to step 2 otherwise skip.
Check the usage counter, if it is 0 then continue to step 2
Run the reset-token command - https://docs.gitlab.com/runner/commands/#gitlab-runner-reset-token
SED config.toml for Token value
Update token value in SSM Parameter

Additional context
The token rotation handling event should most likely occur during the termination of the runners. In my use case I rotate out the runners every week which results in the latest AMI version being used but that would not be suitable for everyone. Handling a race condition properly is hard.

The first solution is not an option as rotating the tokens adds security.
Solution two sounds feasible.
We have to make sure that older versions of the Runner are still working. Not sure when the reset-token commabd was added.

The first solution is not an option as rotating the tokens adds security.
Solution two sounds feasible.
We have to make sure that older versions of the Runner are still working. Not sure when the reset-token command was added.

Looks like it first appeared in v15.10.0
Sources:
MR: https://gitlab.com/gitlab-org/gitlab-runner/-/merge_requests/3209
Commit: https://gitlab.com/gitlab-org/gitlab-runner/-/commit/5e8770d2cb1606f7db37d2ae706c8ad099e57295

Post to keep alive


Detecting background with more than one color.

Codecov Report

Merging #40 (27eaa48) into main (00d2af5) will decrease coverage by 0.15%.
The diff coverage is 90.00%.

:exclamation: Current head 27eaa48 differs from pull request most recent head d5ba22c. Consider uploading reports for the commit d5ba22c to get more accurate results

@@            Coverage Diff             @@
##             main      #40      +/-   ##
==========================================
- Coverage   84.38%   84.24%   -0.15%
==========================================
  Files           5        5
  Lines         506      533      +27
==========================================
+ Hits          427      449      +22
- Misses         44       47       +3
- Partials       35       37       +2

Impacted Files
Coverage Δ

img/processor/imagemagick.go
79.45% <90.00%> (+0.20%)
:arrow_up:


Consider adding support for use latest

I have a feature suggestion I'd like to open for discussion: to enable on to use nvm use latest similarly to how install can accept latest in place of a version name.
The use case here is that when switching between Node versions because that one project just requires 9 but all your other projects keep up with latest, you have to remember what is the latest version you have every time when switching from 9 or other old version you need to keep using.
I think use latest could work in two ways: either pick the latest version of the ones you have locally installed (which would probably be very simple to implement and IMO still semantically valid and useful) or install latest immediately followed by use in one command. I think the former is much simpler and personally it's all I need but I am not sure if the use of the word latest there might be deemed too confusing/surprising?
Thank you for considering this feature request.

I too was caught out by this..
> nvm install latest
Downloading node.js version 16.1.0 (64-bit)...
Complete

>nvm use latest
latest.0.0
node vlatest.0.0 (64-bit) is not installed.
FAILED!!!

This has been, but the keyword is newest.
The following commands are now supported:

nvm use newest uses the newest version currently installed.
nvm use latest attempts to use the Latest release (as found on nodejs.org)
nvm use lts attemtps to use the LTS release (as found on nodejs.org)

This will be available in v1.1.8.


Add support for metric timer resolution

Currently, timer metrics default to millisecond resolution. With this change, timer metrics will now also support microsecond and nanosecond resolution.

Why import a whole enum third-party package rather than just having a normal class with attributes?

@andrewgodwin, enum is a builtin in Python 3 with a backport for Python 2. An enum is a better semantical representation of what this is. Your point, however, makes me realize that this should be a Python 2-only requirement.


Delete postcss.config.js

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

Project coverage is 99.53%. Comparing base (5f14b34) to head (d73542b).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #308   +/-   ##
=======================================
  Coverage   99.53%   99.53%
=======================================
  Files          23       23
  Lines         214      214
  Branches       65       65
=======================================
  Hits          213      213
  Misses          1        1

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


pkgs: init go-ethereum at 1.1.3

@edolstra @wkennington where i should put this? I think it should be separate package, but it breaks current design. Do we have an agreement what to do with go packages?

I'm doing the revert so you can just put it wherever in pkgs applications. I'll be opening a PR for redesigning go-packages so don't worry about this.

Ok

@wkennington you have a sneak peek? or can you CC me when you open the PR? I'm curious

@offlinehacker sorry it didn't get merged earlier. Can you put it somewhere in pkgs/applications/networking or electrum (bitcoin wallet) is in pkgs/applications/misc but I'm not sure it's the right categorization.


It will destroy map app

Device Model
iPhone 14 PM
iOS Version
17.0
Bootstrap Version
1.2.1
Serotonin Version
none
What happened
My map app will prompt for abnormal satellite positioning signals.and it will get stuck and not move.
this problem arises whether or not I inject map app.

Log output
No response
Acknowledgement of README and FAQ

[X] I have read both the README and the FAQ.

Can you upload the photo again? Also, which app is affected?

@dleovl

Hi, are you still encountering this issue?


Print functions as javascript

https://github.com/facebook/reason/issues/1442
/* before */
let foo (bar) = 1;
let myObj = {pub p () = 10};

/* after */
let foo = (bar) => 1;
let myObj = {pub p = () => 10};

Should we do the same for functors?

Yes, we should, since they're "functions" over modules.
Still struggling with a idempotency problem at the moment with functors.
Iwans-MBP:reason Iwan$ cat functor.ml
module Increment (M : X_int) : X_int = struct
    let x = M.x + 1
  end;;
Iwans-MBP:reason Iwan$ ./refmt_impl.native --parse ml --print re functor.ml
module Increment = (M: X_int) : X_int => {
  let x = M.x + 1;
};
Iwans-MBP:reason Iwan$ ./refmt_impl.native --parse ml --print re functor.ml | ./refmt_impl.native --parse re --print re
File "", line 1, characters 30-31:
Error: 2149: <UNKNOWN SYNTAX ERROR>

/* valid parse */
module Increment = (M: X_int) => {
  let x = M.x + 1;
};

/* invalid parse */
module Increment = (M: X_int) : X_int => {
  let x = M.x + 1;
};

Aaah, is this related to https://github.com/facebook/reason/pull/1409?

it might be, parser looks pretty ok...
/* valid parse... */
module Increment  (M: X_int) :X_int {
  let x = M.x + 1;
};

Time to call the cavalry?

cc @let-def
I'm merging this as-is for now then. Let's fix the functor printing and parsing #1472.
Thanks!


Add whylogs

Adding whylogs, an OSS data profiling tool which can be used to create data logs for a monitoring system

Thanks @dleybz!


[carthage] Add no-checkout option

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context

Latest (ver: 0.33.0) carthage bootstrap command has --no-checkout option.
But, carthage action isn't implemented the option.
So, I will add no_checkout option to carthage action.
related PR
I also updated docs.
https://github.com/fastlane/docs/pull/872
carthage bootstrap option
❯ carthage version
0.33.0
❯ carthage help bootstrap
Check out and build the project's dependencies

[--no-checkout]
	skip the checking out of dependencies after updating

[--no-build]
	skip the building of dependencies after updating
	(ignored if --no-checkout option is present)

[--verbose]
	print xcodebuild output inline (ignored if --no-build option is present)

[--log-path (string)]
	path to the xcode build output. A temporary file is used by default

[--new-resolver]
	use the new resolver codeline when calculating dependencies. Default is false

[--configuration (string)]
	the Xcode configuration to build
	(ignored if --no-build option is present)

[--platform (platform)]
	the platforms to build for (one of 'all', 'macOS', 'iOS', 'watchOS', 'tvOS', or comma-separated values of the formers except for 'all')
	(ignored if --no-build option is present)

[--toolchain (string)]
	the toolchain to build with

[--derived-data (string)]
	path to the custom derived data folder

[--cache-builds]
	use cached builds when possible

[--no-use-binaries]
	don't use downloaded binaries when possible

[--use-ssh]
	use SSH for downloading GitHub repositories

[--use-submodules]
	add dependencies as Git submodules

[--color (color)]
	whether to apply color and terminal formatting (one of 'auto', 'always', or 'never')

[--project-directory (string)]
	the directory containing the Carthage project

[<dependency names…>]
	the dependency names to update, checkout and build

Thanks for this PR!
Re the CI changes, same as https://github.com/fastlane/fastlane/pull/15480#issuecomment-541336544 please. Thanks.

@googlebot I consent.


Support RelativeTo namespace

Closing. Is a dupe of https://github.com/unisonweb/codebase-ui/issues/78


Please Help me on this

Sorry to create file like this. This is blocking me. Please help me on this issue.

Thank you for contributing to openstack-infra/jenkins-job-builder!
openstack-infra/jenkins-job-builder uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
http://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.

@ranjitkumar518 the error `File "/usr/local/lib/python2.7/site-packages/jenkins_jobs/modules/builders.py", line 2019, in gen_xml

for builder in data[alias]:suggests its coming from yourbuilder:` script, not this publisher. i tested this publisher and it threw a different error for me

    kind = cdata['condition-kind']
TypeError: string indices must be integers, not str

Hi @hayderimran7 ,
I am sorry. I am also getting the same error(mistakenly posted wrong error). my intention is to create  a maven job with a post build step(not post build action) to trigger a job after maven build with conditional step. Here is my updated code. It is working fine but it is triggering a job in post build action(difference is block down stream option is not available in post build action). can you review my code and suggest some changes or example to trigger a job in post build step with conditional step as always success and block the triggered job. I also tried below commented code. Please give inputs to trigger a job in post-build-step.

job:
name: 'maven_job'
project-type: maven
description: "Modified job"
parameters:
- string:
name: branch
default: master
description: "Branch Selector"
scm:
- git:
url: https://github.com/ranjitkumar518/myfirstrepo.git
branches:
- '${branch}'
maven:
maven-name: maven-3.2.1
root-pom: pom.xml
goals: clean package
publishers:
- conditional-publisher:
- condition-kind: current-status
condition-worst: SUCCESS
condition-best: SUCCESS
on-evaluation-failure: fail
action:
- trigger-parameterized-builds:
- project: test_job
current-parameters: true
trigger-with-no-params: false
builders:
- conditional-step:
condition-kind: current-status
#    condition-worst: SUCCESS
 #   condition-best: SUCCESS
  #  steps:
   #     - shell: "whoami"

@ranjitkumar518  ah i found the issue - you were missing a - before the condition-kind and also your alignment is messed up. the following worked me 👍
- publisher:
    name: test
    publishers:
      - conditional-publisher:
          - condition-kind: current-status
            condition-worst: FAILURE
            condition-best: SUCCESS
            action:
               - trigger-parameterized-builds:
                   - project: default_job
                     current-parameters: true
                     trigger-with-no-params: false

again please be very mindful of yaml alignment , hope it works !

use the above as a publisher in your job configuration as :
- job:
     name: 'maven_job'
     project-type: maven
     description: "Modified job"
    ....
.....
....
     publishers:
        - test

Hi @hayderimran7,
Thanks for looking in my issue.
This code is triggering a job in post-build action. I want to trigger job in post-build step after maven build. The difference between post-build action and post-build step is we cant block the down-stream job. please find the attached screenshot for your reference.

do you wana use parameterized trigger or not? in the original posted, you have trigger-parameterized and thats the same i provided you.  and now you say you wana use build other projects - two different things. which one you want? im confused -

I want to trigger a build with parameters. I tried trigger-builds option(where we can pass predefined-parameters) option also. But internal to the publisher it is not allowing me only this trigger-parameterized-build option.


Font on button with stylebox override no longer centered

Godot version
v4.0.beta.custom_build [eaf306e0b]
System information
Ubuntu 22.10 LTS, Vulkan
Issue description
Something happened with latest commits, buttons text no longer centered
with stylebox texture

without

Steps to reproduce

add stylebox texture
remove stylebox texture

Minimal reproduction project
New Game Project.zip

I'd suspect #71686

@KoBeWi yes
git bisect start
# status: waiting for both good and bad commits
# bad: [eaf306e0b15e4283883f06bf903b05711a4dbfe8] Merge pull request #71864 from KoBeWi/drag_and_fail
git bisect bad eaf306e0b15e4283883f06bf903b05711a4dbfe8
# status: waiting for good commit(s), bad commit known
# good: [28a24639c3c6a95b5b9828f5f02bf0dc2f5ce54b] Merge pull request #48570 from Calinou/animation-editors-new-name
git bisect good 28a24639c3c6a95b5b9828f5f02bf0dc2f5ce54b
# good: [c02572d3d05fc59f20252bc77c1b8b002540f156] Merge pull request #71683 from reduz/fast-script-class-inheritance-check
git bisect good c02572d3d05fc59f20252bc77c1b8b002540f156
# good: [c3539b4561f9b4d7dc4ba1c5859217e7fbf9c6fe] Merge pull request #70773 from KoBeWi/lector
git bisect good c3539b4561f9b4d7dc4ba1c5859217e7fbf9c6fe
# good: [84aaa1e90eab3c40ff0769cc70a52e2b52291bc2] Merge pull request #71849 from EricEzaM/fix-run-specific-scene-shortcut
git bisect good 84aaa1e90eab3c40ff0769cc70a52e2b52291bc2
# good: [78d45fc51c40421683e623ccf8ecce9be9143631] Merge pull request #71855 from reduz/audio-stream-polyphonic
git bisect good 78d45fc51c40421683e623ccf8ecce9be9143631
# good: [5432bad36f789118fe69ae733427d0c7aebd7dcb] Merge pull request #69619 from m4gr3d/address_android_editor_crashes_main
git bisect good 5432bad36f789118fe69ae733427d0c7aebd7dcb
# bad: [5068d823745ad05d7bdaeb596526f7ca196124d3] Merge pull request #71686 from YuriSizov/stylebox-min-size-but-better
git bisect bad 5068d823745ad05d7bdaeb596526f7ca196124d3


Update 07-bootstrapping-etcd.md

Bump to latest patch version of etcd

Hi @xtreme-sameer-vohra
The lab is in the process of being updated. Newer version of etcd is part of these changes and will be applied when the update is complete.
Thanks.


feat: make react/react-dom peer dependencies

Description
docz has a direct dependency on react/react-dom which very often conflicts with other dependencies in apps that also have direct dependencies on react/react-dom.

In this PR, we make react & react-dom peer dependencies.
Review

[ ] Check it's all still working.

Pre-merge checklist

[ ] ???

Screenshots

Before
After

Image
Image

But don't you need to remove the direct dependencies? Check core/docz/package.json. It has react and react-dom under both dependencies and peerDependencies. Same goes for docz-theme-default. gatsby-theme-docz has them under dependencies, but no peerDependencies. I'd guess you needed to remove the direct dependencies for these subpackages.
Also, you should update the installation instructions in README.md, adding a mention that react and react-dom has to be installed explicitly.

@draperunner true. Will update.

Already merged into feat/gatsby.


questions about loss

Hi I noticed that in textgrad/tasks/__init__.py, there are many eval functions: MultiFieldTokenParsedEvaluation StringBasedFunction, also in textgrad/loss.py, there are other eval functions: TextLoss MultiFieldEvaluation MultiChoiceTestTime
I am a little confused about what eval function I should use, are there some detailed information about how to chose which function to use in detail? Why BBH uses MultiFieldTokenParsedEvaluation while GSM8K uses StringBasedFunction?

By the way, If I want the result to include a certain string(like [TOKEN]), which loss function should I use? Or how should I modify the code? Can you give me some advice? Thank you very much!

Cool! So if you look at object counting (here), we use this string-based evaluation: https://github.com/zou-group/textgrad/blob/main/textgrad/tasks/init.py#L42 .
This basically looks at the last numerical value, and checks if it matches the ground truth answer.
More generally for BBH tasks such as word sorting, we do not have numerical values to compare. E.g., in word sorting we need to compare two lists. Since this is much more tricky to do in general, we do this by making another language model call and asking it to compare two lists -- and use it as the loss function. That's what MultiFieldEvaluation does at a high level. MultiFieldTokenParsedEvaluation basically only adds parsing functionality (i.e., asks the evaluation to be returned within two HTML tags.
If you want the result to include a certain string, i think you can leverage constraints. e.g., you can use something like
tg.TGD(parameters=parameters, constraints=["The output should end with <TOKEN>"]). Basically, this lets the optimizer see the natural language constraints you specify through the constraints argument.
Hope this helps!

Below is my understanding of TextGrad, I am not sure if it is correct.
There is an inference model and an evaluation model (backward engine). The inference model answers questions, and then there is a loss function that calls the model from the backward engine to analyze the response. Subsequently, the optimizer will call the model from the backward engine to optimize the prompt based on this analysis.
However, if I need the analysis from the loss function to not be the model's inference analysis, for example, I want to manually write a judgment of right and wrong, manually write an analysis for the correct cases and incorrect cases, these two analyses are given, and I only want the model to do the optimizer's optimization work, combining the given analyses with the question and the answer to optimize the prompt, how should I modify it? All of the current loss classes seems to require calling the model for inference.


Fix scale size

Hi, there
I found wrong calculation when cropping image

I would also recommend approving this.  As of right now images are cropped to the wrong area if the crop size is square

this pull reques fix my question, thanks.
https://github.com/gekitz/GKImagePicker/pull/43/commits/6adf647ba031e3656eaa2aac62bc0e60629166f0


TP5: Container hangs itself up after adding NetworkController feature & get-windowsfeature

Related to https://github.com/Microsoft/Virtualization-Documentation/issues/155, but this is not about the documentation/clarification, but about the behaviour itself.
This seems to have been a bug in TP4, as Neil Peterson confirmed via Twitter (http://superuser.com/a/1048289/46725), but it still exists in TP5 as well with slightly different behaviour. Here is a screencast where the container hangs himself up once trying to add the NetworkController windows feature.
The entire container hangs, not even possible to stop it via docker stop anymore.
Here is a screencast: https://screenrecordings.tinytake.com/sf/NjY5MDI3XzMyMTExMDE
How to reproduce it:

Start container:
docker run --name="wincore" -it windowsservercore powershell

Add Network Controller
add-windowsfeature networkcontroller
It tells you that you need to restart the server (assuming it's meant to restart the container. This should be clarified in the documentation as mentioned in https://github.com/Microsoft/Virtualization-Documentation/issues/155).

On the container host, run:
docker restart wincore
docker attach wincore

Inside the container, run:
get-windowsfeature *net*
It will hang endlessly at 57%, neverending.

Try to stop the container from outside:
docker stop wincore
Hangs endlessly as well. Only way to resolve it is to actually reboot the whole container host machine.

After the reboot of the container host, run again:
docker start wincore
docker attach wincore

Inside the container:
get-windowsfeature *net*

Now, this time the call does not hang anymore, but unfortunately the NetworkController is still not installed and the status is back to "Available".
PS C:\Windows\system32> docker version
Client:
 Version:      1.12.0-dev
 API version:  1.24
 Go version:   go1.5.3
 Git commit:   2b97201
 Built:        Tue Apr 26 23:51:36 2016
 OS/Arch:      windows/amd64

Server:
 Version:      1.12.0-dev
 API version:  1.24
 Go version:   go1.5.3
 Git commit:   2b97201
 Built:        Tue Apr 26 23:51:36 2016
 OS/Arch:      windows/amd64

Windows Server 2016 Datacenter Technical Preview 5
Build 14300.rs1_release_svc.160324-1723)

Hi Mathias -
May I ask why you are trying to run Network Controller from within a container? This is not a supported configuration at this time.
Jason

@JMesser81 I was having problems with a HttpListener in my .net application and thought it was related to the NetworkController, but I meanwhile figured out that it's not related. Closing this ticket.


fix for multivalidator mandatory set_uncles transaction failure

Is this merge only to fork the authorship pallet?
And also shouldn't the pallet-authorship now be configured as a local dependency, in modules that require it as a dependency, such as the staking pallet and the runtime.

Is this merge only to fork the authorship pallet?
And also shouldn't the pallet-authorship now be configured as a local dependency, in modules that require it as a dependency, such as the staking pallet and the runtime.

There is a one-line change in authorship/src/lib.rs:line213 under // FIXME comment.
The set_uncles in no longer mandatory, and DispatchClass dependency removed as no longer used.
pallet-authorship is configured as a local dependency using patch mechanism in Cargo.toml. It rewrites the pallet-authorship dependency for every crate which requires it.


BUG: Restrict rv_generic._argcheck() and its overrides from setting members.

Address incorrect probabilities, broadcasting issues and thread-safety
related to distributions setting member variables inside _argcheck().
Addresses gh-9896, gh-2002, gh-1320, gh-7933.
Add method _get_support(*args) to rv_generic (the default behaviour is to
return sef.a, self.b.)  Distributions which have support depending on parameters
need to override this method.
Add overrides for genextreme, kappa4, reciprocal, truncexpon, truncnorm,
binom, bernoulli, hypergeom, boltzmann, randint, rv_sample.
Move code which sets members variables from _argcheck() into _get_support()
and discard any code which actually sets member data.
As most code previously called self._argcheck() anyway before calling
_cdf()/_pdf()/_sf()/..., the impact on performance is perhaps minimal.
This fixes threading concerns, because the (global) distribution is no longer
being changed every call.
Fixes broadcasting issues because the _cdf(x, *args, **kwargs)/... calls now
actually use *args, which has the correct size.

Looks good on a quick browse.
Performance is probably not a big concern indeed, the overhead is huge already, it's unlikely to get significantly worse.
Note that the framework won't be fully thread-safe anyway, because of the self._size dance in rvs.
Could you expand a bit: you mention this addresses several issues, what is their status with this change? Are they fixed? In that case, it'd be helpful to add specific regression tests, with references to GH issues.
If cooking up a threading regression test is not too difficult, it'd be nice, too.

To be explicit, this PR fixes:

gh-9896 (broadcasting, threading)
gh-7933 (broadcasting)
gh-2002 (broadcasting)
It fixes the first example in gh-1320 (broadcasting)
c=2.62;stats.genextreme.ppf(0.5,np.array([[c],[c+0.5]]))
The later examples such as
c=2.62;stats.genextreme.moment(5,np.array([c,c+0.5]))
are not addressed.
-It appears that some examples in gh-1320 which did work in 2013 (such as
c=2.62;stats.genextreme.moment(2,np.array([c,c+0.5])))
stopped working some time ago (E.g. Py2.7, Scipy 1.13.0), generating errors such as
ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()

The most recent pushes added named tests for the 4 Issues. An errorful threading situation is simulated in one of these tests.  It fails prior to gh-9990, passes with it.

The test failures appear to be unrelated.

The ci/circleci: pypy3 test failed immediately with a 404 error getting the Debian security updates.
The ci/circleci: build_docs target built the html and latex  but failed the pdflatex with Invalid fontname [FreeSerif.otf]/OT', contains '['`

build_docs should go away if you rebase it

Rebase fixed the build_docs and the pypy3 target also succeeded.

Am adding the 1.3.0 milestone, if only to indicate that I'm going to try to give it a final review before branching 1.3.0. The decision is RM's, of course.

Was able to look at this a bit last night, here are my findings:
confirmed on python 3.6 on my box that this fixes part of gh-1320  and all of gh-7933.
Here is session for gh-7933:
Python 3.6.7 |Anaconda, Inc.| (default, Oct 23 2018, 14:01:38)
[GCC 4.2.1 Compatible Clang 4.0.1 (tags/RELEASE_401/final)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import numpy as np
>>> from scipy.stats import truncnorm
>>> truncnorm.logpdf(
...     np.array([3.0, 2.0, 1.0]),
...     a=(1.5-np.array([6.0, 5.0, 4.0]))/3.0,
...     b=np.inf,
...     loc=np.array([6.0, 5.0, 4.0]),
...     scale=3.0
... )
array([-2.44840737, -2.38781507,        -inf])

and here is the session for gh-1320
(scipy-dev) Lucass-MacBook:scipy rlucas$ python
Python 3.6.7 |Anaconda, Inc.| (default, Oct 23 2018, 14:01:38)
[GCC 4.2.1 Compatible Clang 4.0.1 (tags/RELEASE_401/final)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import scipy.stats as ss
>>> c=2.62
>>> c
2.62
>>> ss.genextreme.ppf(0.5,np.array([[c],[c+0.5]]))
>>> import numpy as np
>>> ss.genextreme.ppf(0.5,np.array([[c],[c+0.5]]))
array([[0.23557562],
       [0.21836694]])
>>> ss.genextreme._ppf(0.5,np.array([c,c+0.5]))
array([0.23557562, 0.21836694])
>>> ss.genextreme._mom0_sc(2,np.array([c,c+0.5]))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/rlucas/scipy-dev/scipy/scipy/stats/_distn_infrastructure.py", line 1660, in _mom0_sc
    args=(m,)+args)[0]
  File "/Users/rlucas/scipy-dev/scipy/scipy/integrate/quadpack.py", line 337, in quad
    flip, a, b = b < a, min(a, b), max(a, b)
ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()
>>> c
2.62
>>> ss.genextreme.moment(2,np.array([c,c+0.5]))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/rlucas/scipy-dev/scipy/scipy/stats/_distn_infrastructure.py", line 1160, in moment
    if not (self._argcheck(*args) and (scale > 0)):
ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()
>>> ss.genextreme.moment(5,np.array([c,c+0.5]))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/rlucas/scipy-dev/scipy/scipy/stats/_distn_infrastructure.py", line 1160, in moment
    if not (self._argcheck(*args) and (scale > 0)):
ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()
>>>
(scipy-dev) Lucass-MacBook:scipy rlucas$

So far so good, will keep looking to see what I find on other addressed issues.

I have no time to look in detail before branching; I suggest to wait for @ev-br's final review for now and if he doesn't get to it then merge as is, since it's a clear improvement and all open comments have been addressed.

Don't wait for me, there's no point holding this PR.

Okay, in it goes!
Thanks @pvanmulbregt and all reviewers!

I went through and manually closed the issues that were claimed to be addressed in the initial PR description, and added 1.3.0 milestones to those issues. For one of them, based on the extended discussion above, it looks like this PR only provides a partial fix, so maybe another issue should be opened to split off residual problems.

Added crude "release note" to end of other changes section. Feel free to improve or just remove if not really appropriate there.

I went through and manually closed the issues that were claimed to be addressed in the initial PR description, and added 1.3.0 milestones to those issues. For one of them, based on the extended discussion above, it looks like this PR only provides a partial fix, so maybe another issue should be opened to split off residual problems.

To be explicit (and save me having to think too hard some time in the future), I presume you are referring to gh-1320. If so, (see https://github.com/scipy/scipy/pull/9900#issuecomment-473592676 this PR (and its patches) indeed only fixes the first example; the second example is still broken, and the third example broke in the intervening years.

@pvanmulbregt correct


allow effort 1,2 in encode api

Fixes https://github.com/libjxl/libjxl/issues/522

Ugh, we actually tested that effort 2 has to fail. Updated the test.


[SMALLFIX] Null check on worker address

It's possible that the block location policy cannot find any available worker.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16959/
Test PASSed.

@aaudiber not that straightforward...
I view this fix as a good practice since the return value of getWorker can be null.
Added Nullable annotation.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16960/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16961/
Test PASSed.


help not displayed if required field has not been set

Hi there
Firstly, thank you for this very helpful library.
I found that help was not being outputted if a required argument was not specified on the cmd line. Is this the desired behaviour?
I personally did not want this behaviour so have changed it so that help is always displayed (if -h/--help is specified) but a required argument is not.
I would be happy to create a PR if you wish to view my changes. I just swapped the order in which you check for unhandled required arguments and parse the handled ones in "run". This may screw it up elsewhere as I have not run the tests, but it works for me!
Cheers

Yes, this is not the desired behavior. I thought I fixed it in #4, but as you can read in that issue the bug still exists. I did not have time for the changes mentioned in the conversation from #4.
A PR would be much welcome! Thanks already!

you can simply do what i suggested in #6 evaluate if -h/--help has been called and it it is then stop evaluation of required fields:
`bool run(std::ostream& output, std::ostream& error) {
bool help =false;  // add this
if (_arguments.size() > 0) {
auto current = find_default();
/*  Add this line*/
if ( std::find(_arguments.begin(), _arguments.end(), "--help") != _arguments.end() ||
std::find(_arguments.begin(), _arguments.end(), "-h") != _arguments.end())
help = true;
...
		for (auto command : _commands) {
			if (command->required && !command->handled && help == false) { // here
				error << howto_required(command);
				return false;
			}
		}

...`
but this is just a quick "fix" for help only. in this case every time you press - od --help it prints usage no matter what other options are set or not set

This was finally fixed with PR #8.


chap: retrieve chap credentials from configMap

current implementation read chap user statically from iscsid.conf
This was not scaleable for node scale up/down scenarios
read chap credentials from configMap and update the hpenodeinfo

Signed-off-by: Raunak Kumar<EMAIL_ADDRESS>

update: made the changes to comply with environment variables, eliminating iscsid.conf reference, not involving tunelinux and setting the password after controller publish response. So either way works well. Will update the PR once configMap vs env variable is finalized.