Why test_loader use batchsize=1

  if flag == 'test':         shuffle_flag = False         drop_last = True         if args.task_name == 'anomaly_detection' or args.task_name == 'classification':             batch_size = args.batch_size         else:             batch_size = 1  # bsz=1 for evaluation         freq = args.freq
I am wonder this setting

Hi, this design is to mimc the online serving scenario.
If you have a online model, you can use this configuration. But if you propose a normal forecasting model, you can just increase this number.


feat(artifacts): start of artifact docs

This kicks of the artifact documentation.
Still missing and critical:

Configuration (under setup),
Common artifact formats (gcs vs. docker vs. s3...)
Provider support

https://github.com/spinnaker/spinnaker/issues/2177

I'm gonna go ahead and merge this as-is, and do an editing run in a separate PR rather than polluting it with lots of long comments.

Thanks - I've addressed @duftler's feedback


error if i try to run

👉 Please follow one of these issue templates 👈
Uncaught Error: A dynamic link library (DLL) initialization routine failed.
\?
<EMAIL_ADDRESS>

https://github.com/FakeFullStack/QA/issues/2
here, I write chinese doc to rebuild serialport in electron.

OK, I'll find a time to translate


Added TravisCI scripts

Note: This pull request depends on #113. For reviewing purposes the commit is https://github.com/intel/haxm/commit/4f623111ffd368232fe452b8c798453236c673d7.
Based on the discussion at https://github.com/intel/haxm/pull/108#issuecomment-430210463, I've prepared a TravisCI configuration script for HAXM.
This should be extremely helpful to review pull requests. For now it only checks for successful builds, but we can extend it by checking the emulator unit tests, or even integration tests, such as launching a VM (assuming the hardware/hypervisor used in TravisCI servers allows it).
Windows support is very recent (barely two weeks!), and right now they support only Visual Studio 2017 build tools, which apparently is not enough to build HAXM. However I've tested the script locally, and it should work once they provide a proper Visual Studio installation, which is likely. If you want to remove the Windows script in the meantime, let me know (I can add it as a comment to this PR until its ready).
EDIT: It seems https://github.com/intel is already integrated with TravisCI (see badges below). That really helps, and totally makes this choice even more convenient than alternatives (e.g. AppVeyor).

Thanks a lot! Indeed this is a great help.
The code looks nice and simple. We can keep the Windows script, but how can we test it locally? It doesn't seem to be written in Batch syntax.

We can keep the Windows script, but how can we test it locally? It doesn't seem to be written in Batch syntax.

According to the TravisCI Windows reference, the Windows script is executed within Git BASH (which I am thankful for, considering how terrible cmd.exe is).
Their current environment is essentially a stock Windows Server 2016 installation, with Chocolatey and these preinstalled packages, which include Git BASH. Testing locally, using Windows 10 instead and Visual Studio 2017, that script works for me.

All done!

Cool! I can see in the Travis build log that the macOS build is working :-)


Fixed Vi mode indicator, Added nice-exit-code plugin

Changes

Fixed an issue that Vi mode won't appear before the user changes the mode. (This change shows INSERT when failed to get current Vi mode)
Added nice-exit-code plugin. This can be activated with the flag POWERLEVEL9K_STATUS_NICE_EXIT_CODE and shows a signal name of the last status code.

This is how it works.

Hey @sei40kr! Thanks so much for the bug fix and new feature!
We are pretty strict about keeping new feature merges on to the next branch for stability reasons on master. Would you mind doing one of the following?

Refile this entire PR against next
OR
Break this into two PRs, file the bugfix against master, and the new feature against next

I look forward to getting this merged! It looks :+1:

I stole your idea and implemented it in #581.  :heart:

@sei40kr - Your exit code idea is really great :) @docwhat has pulled it out into #581, separately.
So, we still need to get your bugfix for Vi-mode merged. You can either re-file this PR (keep it against master) with only 5f2e0ecb7ae, or I can cherry-pick it over onto master. Do you have a preference? You're commit will be in the tree either way, but having PRs is nice as an additional log, of sorts.

@bhilburn @docwhat
Thank you for your review.
I've edited this PR and make this contain only a commit which fixes Vi mode issue.

@sei40kr - Excellent work! Thanks so much for incorporating the review feedback. This is a great contribution to P9k.
Welcome to the project =)


WIP: Adding domain home in image tests

Adding tests for domain home in image using WLST and WDT.
For domain home in image, logs are written on PV when logHomeOnPv=true, the logs are created by root user. The current cleanup.sh and statedump.sh(which archives logs) uses job.sh which doesn't run as root and can't access these logs. So, modified these scripts to use krun.sh provided by Tom Barnes which gives the option to select the image you want to use, using jre image solves the issue as it uses root user.
Jenkins run passed - http://wls-jenkins.us.oracle.com/job/weblogic-kubernetes-operator-javatest/1163/console

@tbarnes-us please review


feat: disable default features in workspace dependencies

Disables default features in the workspace dependencies.
This is done so that features that do not compile in wasm32-wasi are not automatically enabled. This change was originally in #9430 but was seperated at the request of @mattsse.

@mattsse Noticing EF tests failed. Any ideas why compilation flags would effect this?
Will do some A/B testing with dependencies and try find source of issue.

Should be fixed now


Az WebApp Up sees LSW as Windows

In v64, when using Bash on Windows (Linux SubSystem for Windows), az webapp up identifies the platform as Windows, which results in Oryx build and deployment issues.
I expect the az webapp up command to identify LSW as Linux
jasonsha@Jason6700:~/temp/jrsdjango$ uname -a
Linux Jason6700 4.4.0-17763-Microsoft #379-Microsoft Wed Mar 06 19:16:00 PST 2019 x86_64 x86_64 x86_64 GNU/Linux
jasonsha@Jason6700:~/temp/jrsdjango$ az webapp up -n jrsDjango
Creating Resource group 'jasonsha_rg_Windows_centralus' ...
Resource group creation complete
Creating App service plan 'jasonsha_asp_Windows_centralus_0' ...
App service plan creation complete
Creating app 'jrsDjango' ...
Webapp creation complete
Configuring default logging for the app, if not already enabled
Updating app settings to enable build after deployment
All done.
You can launch the app at 'http://jrsdjango.azurewebsites.net'
To redeploy the app run the command az webapp up -n jrsDjango -l "Central US"

@JasonRShaver az webapp up doesn't default the OS based the OS its run on. Its based on the code the command detects. You need to run the up command from the directory where the code you want to deploy to app is located. Run the az webapp up command with --dryrun you will see the Src folder, the commands trys  to parse the files Ex. if it detects Node.js files or python files we use Linux, anythign else is windows. Please check the folder you are running the code from & re-run. Let me know if this helps. Thank you.

Ah, the Python logic failed because I didn't have a requirements.txt file.  Is it possible to get a parameter to set the OS or runtime for cases where detection logic doesn't have us covered?

good to know. We did think about adding a --os-type parameter but was not a high priority.

@JasonRShaver i am closing this issue please file a new issue to support the OS type if you still want this as a feature request. Thank you.


Not an issue, just wanted to say thanks!

This might be the best European keyboard layout available, thanks so much!
Actually, since it's the issues thread, if I really wanted to nitpick I could say the naming doesn't really reflect what it is, it goes well beyond just french characters, maybe the name "qwerty-eu" would be better?

Thanks for your warm comment!
With regard to the naming I agree that we should ultimately move away from qwerty-fr. I will organize a “give-your-suggestions” thread at some point.

I just want to echo a big thank you, I was struggling with the dead keys on international qwerty and this layout is so much more logical and convenient with the accents, only took like 15mns for it feel natural.

Yeah right! Still has issues between è and é because è is more common so I tend to press e to form è, but I havent been using accents a lot so far, maybe I just need to get used to it.
I'm definitely keeping the layout though!

I'm joining people to say "thank you!" too :)
Using it on a Preonic ortholinear keyboard where I had no option except to drop the azerty layout (because of keycaps availability), qwerty-fr is so much better than other alternatives!

Yeah right!
Still having trouble with "é" being on W but I have to force myself to get used to it, I can't judge so quickly.

Okay, so, I've been using this layout pretty much everyday, and I really wanted to have a long term feel about it.
I'm still really happy with it, but I definitely think in the end that é and è should be swapped, so é on E and è on W.
Might be worth creating a poll about it?

Per the documentation modifying the layout shouldn't be too hard given all the groundwork is done. On Windows at least the layout is modifiable by using the link they provide; https://www.microsoft.com/en-us/download/details.aspx?id=102134 and the "Modify an existing keyboard layout (.KLC) file and build a new layout from it" section, the KLC for qwerty-fr is right here in the github.
I don't mind giving it a try when I have some time so probably next weekend. I don't think breaking the logic of all other accents with the left -> acute and right -> grave would be worth it just for é even if I agree it feels more natural to hit the e key for it, so a personal modification or doing a fork is probably the best way to get that.

Oh yeah no, that would definitely be just for é (I just missed it again lol), more like an "exception" approach.
I know exceptions suck, most of the time, but for something as recurrent as é it is deserved in my opinion.

@Deus-nsf @adn34 It's not the right place to discuss the design of the layout. Please move to the discussions, thanks!


Dynamic version of the Error effect

Is it even needed? The implementation in Effectful.Error.Static is arguably the only one possible, given the interface.

A thrwoIO-based implementation might also be useful. (maybe even more efficient?)

Yeah, it's the one in Effectful.Error.Static that I'm talking about ;)

I think even if you don't think it is useful now, I feel like having it at least for consistency isn't a bad thing. I don't think we really lose anything by including it.

That's true, I'm just wondering how to best document it.

I think honestly saying that you couldn't think of a use for it at the time of writing would be beneficial (:


[BUG] - Wiki Docs need update

Limit cropped image result min max size
still refer to com.theartofdev.edmodo.cropper.CropImageView
rather than com.canhub.cropper.CropImageView
I am sure more changes lik this is nessesary!

Thanks @Morons was a lot of documentation and code to learn and import in the transition. Indeed we still find some bits to fix!
Already fix this one, feel free to keep pointing when you find =D
We are always improving.


TEST/XCAS/IMAP/IMAP Service Status (Informational)

In https://github.com/ExchangeAnalyzer/ExchangeAnalyzer/releases/tag/v0.2.1-beta.5


Visual studio code frequently crashes since Catalina OS release

Issue Type: Bug
The crashes occcur on multiple scenarios but the most recen was when running reac-native run-ios on a fairly mature and stable react native application.  I has also happened when I attempted to run the view history
VS Code version: Code 1.27.1 (5944e81f3c46a3938a82c701f96d7a59b074cfdc, 2018-09-06T09:14:32.385Z)
OS version: Darwin x64 19.0.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4258U CPU @ 2.40GHz (4 x 2400)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
24, 14, 8

Memory (System)
8.00GB (0.52GB free)

Process Argv
/Applications/Visual Studio Code.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

Extensions (12)

Extension
Author (truncated)
Version

vscode-eslint
dba
1.6.0

githistory
don
0.4.6

gitlens
eam
8.5.6

flow-for-vscode
flo
1.0.1

lintlens
ghm
2.7.0

beautify
Hoo
1.5.0

vscode-gutter-preview
kis
0.15.3

python
ms-
2019.1.0

debugger-for-chrome
msj
4.12.1

git-merger
sha
0.4.1

markdown-preview-enhanced
shd
0.3.11

html-preview-vscode
tht
0.2.5

(1 theme extensions excluded)

@ConnectedReasoning Can you please add some steps that cause crashes for you?

I will keep this updated as the crashes occur

Ok

It just occurred. I woke up the mac, my screen was already in vscode. I moved my mouse over a folder in the explorer panel. I right clicked the folder and selected 'new file'. it happened twice in a row.

It just occurred. I woke up the mac, my screen was already in vscode. I moved my mouse over a folder in the explorer panel. I right clicked the folder and selected 'new file'. it happened twice in a row.

Can you please follow the steps in https://github.com/Microsoft/vscode/wiki/Native-Crash-Issues to get at more details around the crash and attach the result here? Thanks!


[ZEPPELIN-1251] Update interpreter artifact option list for netinst package

What is this PR for?
Add scala 2.11 artifact options for netinst package
What type of PR is it?
Documentation
What is the Jira issue?
ZEPPELIN-1251
Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

What will happen with ./bin/install-interpreter.sh --all ?
@minahlee Do you have any good idea to not install xxx_2.10 and xxx_2.11 together with --all option?

Instead of listing all xxx_2.10 and xxx_2.11 in conf/interpreter_list, i think just listing one version that is included in zeppelin-bin-all package would be fine, i think.
User can always manually install the other version of interpreter using -n and -t option of bin/install-interpreter.sh command.

Addressed your comment in 63509c0. Please review

Looks good to me

Merge if there is no more discussion

Looks great to me. :shipit:


Blackline- The titles of jobs are the same as on the company's official website

Description : After entering the company Blackline in the search bar and entered to search it, the jobs are displayed with the same title as on the company's offician website.
Precondition : The website peviitor.ro is up and running

Step 1 : Open peviitor.ro
Expected result : The website is opened and you can see the homepage with the searching bar in the middle

Step 2: Write in the searching bar the name of the company ("Blackline"). Than click enter
Expected result  : you can write the name of the company inside the searching bar and you can click enter

Step 3 : Verify if the jobs of the company are displayed.
Expected result : you can see the company's jobs, each job inside a box

Step 4 : Verify the title of the jobs.
Expected result : inside the boxes with the jobs the titles are displayed.

Step 5: Go on the company's oficial website and see if the titles of the jobs are the same with the jobs titles displayed on peviitor.ro
Expected result : the title of jobs from the oficial company website are the same with the titles of the jobs on peviitor.ro


[issue 1902] Correct NCHWc solver config validation check restrictions.

As reported in #1902 , some cases failed due to incorrect restrictions.
This PR fixed it.
cc. @atamazov

@averinevg Please review this.
@junliume Recommended labels: https://github.com/ROCmSoftwarePlatform/MIOpen/labels/correctness, https://github.com/ROCmSoftwarePlatform/MIOpen/labels/bug, https://github.com/ROCmSoftwarePlatform/MIOpen/labels/urgency_blocker (due to correctness issues).

Hi @junliume , could you please approve and merge this PR : )


CON-2180 removed programmatic scrolling at the cart page

Coverage increased (+0.001%) to 1.69% when pulling 1f56d1a5d49708fff2dde6fb0c89f576787bc033 on CON-2180-no-auto-scroll-if-you-tap-in-coupon into 6ce57cf1ef0d9665488479932e9bf00bfd6af693 on master.


Wrong Measurements from MeasureString

Hi Developers,
I have notice that the MeasureString returns a wrong size. The string "W" is used to measure the string size. The spritefont file is generated using MakeSpriteFont tool. The font setting is

FontName: Arial
FontStyle: Bold
FontSize: 185

The returned result for the width and the height are 233 and 283.642578 respectively. "W" is placed at (0,0) I have edited Image 1 to include a black dotted line and marked a red circle to indicate the position where my mouse positioned which reflects the values shown in the Window Title. Clearly, there is a big difference between the Vector2 returned and the size which I had measured.
Image 1:

In addition, when I used SpriteBatch to draw "W" on the screen (0,0) using origin (0,0), it does not draw at the correct place. Instead, there is some sort of spacing as shown in Image 2.
Image 2:

To test it further, I have used SpriteBatch to draw "W" on the screen at the bottom left (i.e. making it right justified) of the screen using the origin calculated via MeasureString, but it does not draw at the correct place as shown in Image 3. Instead, it is lifted and cut off.
Image 3:

Which version of DirectX Tool Kit are you using? There was a bug with MeasureString that was fixed in the Febraury 2016 release.

I am using:
Id: directxtk_windowsstore_8_1
Version: 2<IP_ADDRESS>
For your additional information:
I am developing Win32 application using Visual Studio 2013, but instead I am using directxtk_windows_8_1 (Version: 2<IP_ADDRESS>) as using the directxtk_desktop_2013 (Version: 2<IP_ADDRESS>) gives me error LNK1104:cannot open file 'x3daudio.lib'

The different NuGet packages for DirectX Tool Kit are built for specific platforms, and make different assumptions. If you are writing a "classic" Win32 desktop application, you should not use the directxtk_windowsstore_8_1 NuGet package.
For Windows Store 8.1, the application requires Windows 8.1 or later, and so directxtk_windowsstore_8_1 makes use of XAudio 2.8 in the Windows  8.1 SDK. Similarly for UWP, the application requires Windows 10, and so directxtk_uwp makes use of XAudio 2.9 in the Windows 10 SDK. If you are making a Win32 classic desktop application, then I assume you are wanting Windows 7 compatibility. Therefore, if you use the ```directxtk_desktop_201x`` NuGet package, any use of DirectX Tool Kit for Audio requires the legacy DirectX SDK. If you want specifically to write a Win32 classic desktop application that requires Windows 8 or later and thus uses XAudio 2.8, you can do so but you should use project-to-project references--this is a rare scenario, so I don't have a NuGet package specifically for this one. This is covered in detail on the wiki.

In any case I'm able to repro your issue and I'm investigating.

I think this was a bad fix, but I need to go back and figure out why I needed this change to make other things work.

Chuck Walbourn, thanks for the explanations. Actually, I am shifting from DirectX9 to DirectX11 and happened to found DirectXTK. At the same time, I am not going to use the legacy SDK (June 2010) to support my game engine even though it is only used for building very simple 2D games. Using DirectXTK makes the conversion easier as I am not very technical or familiar with the Pipeline related knowledge/experience.
I do have the intention of converting it to Windows Store App style instead of Win32. However, based on my research, I found that I may also have to pick up C++/CX. Currently, I decided to convert it from DirectX9 (Win32) to DirectX11 (Win32) using DirectXTK first before I further my research on Windows Store App and the Pipeline. This may explains why my 2D Game Engine (Win32) uses directxtk_windowsstore_8_1 instead of directxtk_desktop_2013 for DirectXTK's Audio as it works for this code snippet.
Another question aside from Audio and Spritefont is: Do you have the intention of making DirectXTK's documentation similar to the MSDN style or is it advisablefor us to cross reference DirectXTK with XNA?
As for the SpriteFont, I will wait for you to fix it.

Hi Chuck,
Has the said issue been resolve?

I'm wrapping a robust fix this weekend. You can download the library and make the source edit yourself as well.

Ok noted, in that case, i will wait for you to release all the fix at one go. Thanks,

Sync to the master branch to pick up the fixes. Reverted MeasureString to old behavior, and added MeasureDrawString.
RECT MeasureDrawBounds(_In_z_ wchar_t const* text, XMFLOAT2 const& position) const;
RECT XM_CALLCONV MeasureDrawBounds(_In_z_ wchar_t const* text, FXMVECTOR position) const;

Can you roughly estimate when will this fix be pushed to the NuGet Package?

I generally wait until enough work has accumulated to justify a full release, but I can accelerate a publish to NuGet if that's required. I'll look at doing one this week.

alright, noted with thanks.

reminder for NuGet release. Thanks.

Now that June 2016 is done, I'll be doing all the NuGet publishing for it tomorrow.

Published to NuGet:
This version is also available on NuGet as version 2<IP_ADDRESS>

Universal Windows Platform apps using Windows Tools 1.4 (10586)
Windows desktop app using VS 2013 Update 5
Windows desktop app using VS 2015 Update 3
Windows 8.1 Store
Windows phone 8.1

yes thanks!

Hi Chuck,
I am getting this error message: "This project references NuGet package(s) that are missing on this computer. Enable NuGet Package Restore to download them.  For more information, see http://go.microsoft.com/fwlink/?LinkID=322105. The missing file is packages\directxtk_windowsstore_8_1.2<IP_ADDRESS>\build\native\directxtk_windowsstore_8_1.targets."
The solution I had discovered is to remove the lines that contains the April NuGet release. This is what is being removed:

<Import Project="packages\directxtk_windowsstore_8_1.2<IP_ADDRESS>\build\native\directxtk_windowsstore_8_1.targets" Condition="Exists('packages\directxtk_windowsstore_8_1.2<IP_ADDRESS>\build\native\directxtk_windowsstore_8_1.targets')" />
<Error Condition="!Exists('packages\directxtk_windowsstore_8_1.2<IP_ADDRESS>\build\native\directxtk_windowsstore_8_1.props')" Text="$([System.String]::Format('$(ErrorText)', 'packages\directxtk_windowsstore_8_1.2<IP_ADDRESS>\build\native\directxtk_windowsstore_8_1.props'))" />
<Error Condition="!Exists('packages\directxtk_windowsstore_8_1.2<IP_ADDRESS>\build\native\directxtk_windowsstore_8_1.targets')" Text="$([System.String]::Format('$(ErrorText)', 'packages\directxtk_windowsstore_8_1.2<IP_ADDRESS>\build\native\directxtk_windowsstore_8_1.targets'))" />

Is there a better way to do it rather than editing it manually in the *****.vcxproj?

As I said earlier in this thread, you shouldn't be using directxtk_windowsstore_8_1 for a Windows desktop project. In my experience, the NuGet Manager is able to do the Update as long you have restored the old package.


WIP: Add buffer to outgoing framed transport writer

Highly WIP, don't mind me messing around here

Is this for perf (gathering writes) or some other reason?
Maybe we could chat a bit about the whole peer networking at some point. I'm clueless about it, and we had a conv. with kitman about it https://discord.com/channels/990354215060795454/1052585969745936444.

Is this for perf (gathering writes) or some other reason?

Nope, not for perf at all. And after I wrote some unit tests it turned out that the BufWriter from tokio doesn't even do what I want and I implemented my own, just didn't push it yesterday after getting distracted by life. Sorry for the confusion.
My goal is to make send always return immediately, so that the sending messages in the IO thread never blocks:
https://github.com/fedimint/fedimint/blob/86ee3a748f758593e5bc03d1ae0cd308d6cb829d/fedimint-server/src/net/peers.rs#L394-L413
To make this send call on a framed transport non-blocking the underlying stream has to be non-blocking.
My current approach with my custom BufWriter is to just write to the buffer and then send as much as possible, but never block. Then there's a separate async fn that can drive forward sending out the buffer while we aren't trying to send anything in particular (can just always be selected in parallel to the other tasks).
To avoid the buffer becoming too big if the connection is dead I'm planning on implementing a check before each send operation that closes the connection and clear all buffers above a certain threshold. This will prevent peers from stalling us if they aren't reading from the socket (which is a problem we found but postponed shortly after introducing the network stack).

To make this send call on a framed transport non-blocking the underlying stream has to be non-blocking.

@elsirion
All the IO in async rust is already non-blocking.
It seems to me what you want to achieve is best done with a timeout.

Sure, async io doesn't block the thread, it can still block a task. That's what I'm trying to avoid, maybe in the wrong way.
Timing out isn't what we want here. A peer can block for a few s and become responsive again. But during these few seconds we couldn't send anything to other peers. So sending should happen asynchronously.
Maybe the right way to achieve that would be having a to-be-sent message buffer in the io thread from which we constantly try to send messages over the socket. I was a bit scared of cancellation safety with this option, not sure if it could lead to half-sent messages when sending is aborted in a select that had another future resolve early (but now I doubt that's a problem).

Sure, async io doesn't block the thread, it can still block a task.

It doesn't have to. One task can potentially drive multiple futures at the same time:
https://docs.rs/tokio/latest/tokio/macro.join.html#examples
https://docs.rs/tokio/latest/tokio/macro.select.html#examples
https://docs.rs/tokio/latest/tokio/task/struct.JoinSet.html#examples
I imagine when doing consensus we want to send bunch of data to all peers at the same time, so I guess JoinSet could do.

Timing out isn't what we want here. A

We would map over all peers, where sending to each peer is wrapped in some deadline for the whole round. At the end we would join on all and see which one were OK, and which one timed out.
Though it occured to me now that this is how  if we were not using these IO threads at all. I'm not sure what I described is what you need.

I was a bit scared of cancellation safety with this option, not sure if it could lead to half-sent messages when sending is aborted in a select that had another future resolve early (but now I doubt that's a problem).

Cancelation safety shouldn't be a problem with network IO. In principle the tcp connection can get abruptly terminated at any point anyway. Then dropping a future or not is a matter if select or join is being used.

I looked again at the code, but I'm fuzzy on the overall picture of the consensus network IO. We should talk about it tomorrow. :)

This is probably the wrong approach, I'll try to implement message buffering instead.


brew doctor/cleanup taps homebrew/core when using API mode

brew doctor output
$ brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Tapping homebrew/core is no longer typically necessary.
Add --force if you are sure you need one.
==> Tapping homebrew/core
Cloning into '/opt/homebrew/Library/Taps/homebrew/homebrew-core'...
remote: Enumerating objects: 1545603, done.
...
Tapped 3 commands and 6757 formulae (7,127 files, 467.2MB).

Warning: You have an unnecessary local Core tap!
This can cause problems installing up-to-date formulae.
Please remove it by running:
 brew untap homebrew/core

Verification

[X] My "brew doctor output" above says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update twice and am still able to reproduce my issue.
[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

brew config output
$ brew config
HOMEBREW_VERSION: 4.0.28
ORIGIN: https://github.com/Homebrew/brew
HEAD: 2b4cfb033757b63ab0b128d01c58db76480956f9
Last commit: 7 days ago
Core tap origin: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 90d3e99e5d7e721ada4b06943d528bffb45c0908
Core tap last commit: 21 minutes ago
Core tap branch: master
Core tap JSON: 13 Jul 12:46 UTC
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_EDITOR: vim
HOMEBREW_MAKE_JOBS: 8
Homebrew Ruby: 2.6.10 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: octa-core 64-bit arm_firestorm_icestorm
Clang: 14.0.3 build 1403
Git: 2.41.0 => /opt/homebrew/bin/git
Curl: 7.88.1 => /usr/bin/curl
macOS: 13.4.1-arm64
CLT: <IP_ADDRESS>.1.1683849156
Xcode: 14.3.1
Rosetta 2: false

What were you trying to do (and why)?
This morning I ran brew update which told me tapping homebrew/core was not necessary anymore. I untapped it but then found other brew commands will tap it back
What happened (include all command output)?
$ brew update
Installing from the API is now the default behaviour!
You can save space and time by running:
  brew untap homebrew/core

$ brew untap homebrew/core
Untapping homebrew/core...
Untapped 3 commands and 6728 formulae (7,106 files, 477.7MB).

$ brew cleanup
Warning: Tapping homebrew/core is no longer typically necessary.
Add --force if you are sure you need one.
==> Tapping homebrew/core
... proceeds to clone homebrew/core ...

$ brew untap homebrew/core
Untapping homebrew/core...
Untapped 3 commands and 6728 formulae (7,106 files, 477.7MB).

$ brew doctor
Warning: Tapping homebrew/core is no longer typically necessary.
Add --force if you are sure you need one.
==> Tapping homebrew/core
... proceeds to clone homebrew/core ...

What did you expect to happen?
Commands should agree on keeping or removing homebrew/core?
Step-by-step reproduction instructions (by running brew commands)
See commands above

Can you show the output of
brew untap homebrew/core
brew cleanup --debug --verbose

$ brew untap homebrew/core

Untapping homebrew/core...
Untapped 3 commands and 6757 formulae (7,127 files, 467.2MB).

$ brew cleanup --debug --verbose

/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libtermkey from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libtool from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading coreutils from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading gmp from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading asdf from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading pinentry from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading msgpack from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading tree-sitter from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libidn2 from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libksba from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading docker from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading skopeo from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading gpgme from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading go from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading postgresql@14 from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libpng from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading ncurses from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::TapLoader): loading /opt/homebrew/Library/Taps/cblecker/homebrew-tap/Formula/ocm-backplane.rb
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading unibilium from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading nettle from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading pixman from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading mpdecimal from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading pycparser from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libuv from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading kube-ps1 from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading docutils from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading yt-dlp from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading rtmpdump from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading dive from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading starship from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libunistring from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading terragrunt from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libyaml from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading brotli from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libslirp from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading icu4c from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading fd from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading cffi from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading glib from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading readline from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading hyperfine from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading lz4 from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading awscli from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading stow from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libassuan from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading sqlite from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libgpg-error from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading xz from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading unixodbc from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libnghttp2 from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading vde from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading grep from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading bash from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libgcrypt from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading goreleaser from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading curl from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading capstone from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading krb5 from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading snappy from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading age from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading openssl@3 from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading ca-certificates from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading gettext from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libssh2 from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading qemu from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading zstd from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading autoconf from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading m4 from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libtasn1 from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading docker-buildx from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading oniguruma from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libssh from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading pcre2 from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libevent from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading<EMAIL_ADDRESS>from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading bat from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading pcre from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading difftastic from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading luajit from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading kubernetes-cli from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading helix from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading six from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading unbound from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading ssh-copy-id from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading zsh from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading jq from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading neovim from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading openldap from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libusb from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading<EMAIL_ADDRESS>from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading ripgrep from API
/usr/bin/env /opt/homebrew/Library/Homebrew/shims/shared/git --version
Warning: Tapping homebrew/core is no longer typically necessary.
Add --force if you are sure you need one.
==> Tapping homebrew/core
git clone https://github.com/Homebrew/homebrew-core /opt/homebrew/Library/Taps/homebrew/homebrew-core --origin=origin --template= --config core.fsmonitor=false
Cloning into '/opt/homebrew/Library/Taps/homebrew/homebrew-core'...
remote: Enumerating objects: 1545647, done.
remote: Counting objects: 100% (53058/53058), done.
remote: Compressing objects: 100% (534/534), done.
remote: Total 1545647 (delta 52689), reused 52818 (delta 52524), pack-reused 1492589
Receiving objects: 100% (1545647/1545647), 397.46 MiB | 22.54 MiB/s, done.
Resolving deltas: 100% (1085292/1085292), done.
Tapped 3 commands and 6757 formulae (7,127 files, 467.2MB).
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::TapLoader): loading /opt/homebrew/Library/Taps/homebrew/homebrew-core/Formula/ocm.rb
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading findutils from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading helm from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading gnupg from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading pinentry-mac from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading p11-kit from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading cocoapods from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading luv from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading ruby from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading fzf from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading automake from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading git from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading lzo from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading jpeg-turbo from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading gnutls from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libvterm from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading npth from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading openjpeg from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading direnv from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading pygments from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading fish from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading go-bindata from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading ldns from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading tfenv from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading colima from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading podman from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libfido2 from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading magic-wormhole from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading dooit from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading gnu-time from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libsodium from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading netpbm from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading hut from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading bore-cli from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading libcbor from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading lima from API
/opt/homebrew/Library/Homebrew/brew.rb (Formulary::FormulaAPILoader): loading openssh from API
/opt/homebrew/Library/Homebrew/brew.rb (Cask::CaskLoader::FromAPILoader): loading android-sdk
/opt/homebrew/Library/Homebrew/brew.rb (Cask::CaskLoader::FromAPILoader): loading flutter
/opt/homebrew/Library/Homebrew/brew.rb (Cask::CaskLoader::FromAPILoader): loading easy-move-plus-resize

I tried untapping the one non-homebrew tap I have and then the other commands were happy and stopped tapping homebrew/core
$ brew untap cblecker/tap
Untapping cblecker/tap...
Untapped 6 formulae (40 files, 449KB)

$ brew untap homebrew/core
Untapping homebrew/core...
Untapped 3 commands and 6757 formulae (7,127 files, 467.2MB).

$ brew cleanup
(no output)

Two things:

It is not immediately obvious why this is happening and what I should do as an end user
It is not clear what kind of bug report I should submit to the tap's repo/owner

I happen to be working with the maintainer of cblecker/tap so I can help fix this one but I suspect there will be more occurences of this case?

I think this is because of the tap's tap_migrations.json. To be sure, this is definitely a bug in brew, and this should be fixed so that there are no changes required to external taps.


centos 5 builders seem a little broken

The linux nightly tarballs are having some strange issues, I guess since the power outage? Did they need to recompile the source build of gcc they've been using from scratch or something?

I sure hope not.  I nuked the build directories, we'll see if something strange happened that botched that, but if that doesn't work, I'll restore from one of my saved images.

If you do need to rebuild gcc, probably worth using 4.9 or 5.1 so things are copacetic on bleeding-edge ubuntu: https://github.com/JuliaLang/ZMQ.jl/issues/81#issuecomment-103662471

I'm building 5.1 now to see if that helps.  Also, working on a set of Ansible scripts that should make this provisioning easier, with way less investment necessary as compared to the vagrant/packer approach I had earlier.

Does your ansible stuff upgrade the cygwin packages on the windows buildbots? I posted by powershell update snippet in a few other places, can dig it up again if you can't find it.

alright, I think I've got the centos 5 guys up and running.  You know, I looked around for the update script 'cause I know you posted it, but I can't find it.  :(  If you could post it here again, that'd be great.

In powershell, the cygwin-package-updating bit is this:
mkdir -Force C:\cygwin | Out-Null
(new-object net.webclient).DownloadFile(
  "http://cygwin.com/setup-x86_64.exe", "C:\cygwin\setup-x86_64.exe")
foreach ($pkg in @("git,make,curl,patch,python,gcc-g++,m4,cmake,p7zip",
    "mingw64-i686-gcc-g++,mingw64-i686-gcc-fortran",
    "mingw64-x86_64-gcc-g++,mingw64-x86_64-gcc-fortran")) {
  C:\cygwin\setup-x86_64.exe -q -n -R C:\cygwin -l C:\cygwin\packages `
    -s http://mirrors.chauf.net/cygwin -g -P $pkg | Where-Object `
    -FilterScript {$_ -notlike "Installing file *"} | Write-Output
}

The mirror choice (mirrors.chauf.net/cygwin above) may need to be tweaked occasionally, I've yet to figure out the absolute most reliable long-term choice.
CentOS 5 builders do appear to be back in business though, so closing this issue. Thanks!

As far as mirrors for this go, there's an easy no-brainer: http://mirrors.mit.edu/cygwin/

Indeed. I'm blind apparently.

cc1: error: -fno-gnu89-inline is not supported http://buildbot.e.ip.saba.us:8010/builders/package_tarball32/builds/1003/steps/make binary-dist/logs/stdio ?

I'm gonna take a wild guess and say that the buildbot was refreshed but the new gcc wasn't rebuilt yet?

It looks like the PATH wasn't set right after installing the new stuff; retrying.

Better, thanks. Looks like I need another libgit2 patch for old openssl. https://github.com/JuliaLang/julia/commit/e87bf5325db494aed02b2efa12a67a8f27aa679a

What did I manage to break this time? libunwind error: 'REG_FS' undeclared ?
http://buildbot.e.ip.saba.us:8010/builders/package_tarball32/builds/1007/steps/make binary-dist/logs/stdio

aha. your CLOEXEC thing clobbered _GNU_SOURCE - I'm going to comment it out to just see if we can get a green nightly first, then we'll keep experimenting with where to put it to make lldb happy

how did it clobber it?

Guessing, but it probably gets added somewhere in libunwind's configure unless CPPFLAGS is set externally. Maybe we could do CPPFLAGS="-D_GNU_SOURCE -DO_CLOEXEC=0" but who knows how many other secretly necessary flags are getting set like that throughout all the deps

Am I running afoul of python syntax, or does the buildbot configuration take some time/manual intervention to update?

Unfortunately, there is a manual command that I typically run.  I should add that to the rundeck thingy.

Buttons are good. Looks like my manual build with one more libgit2 patch fixed things, though the auto-nightlies are stalled because of 12 pending builds on the ubuntu14.04-x86 builder?

Indeed.    I'm pretty sure it would have timed out eventually, but I have added a "kick_buildslave" job on Rundeck that allows you to stop and then start the buildslave process on any of the buildbots.

Looks like we're back in business, thanks as always

I found the "build gcc" button on rundeck, but looks like it's missing the "add to path" step since I still occasionally get -fno-gnu89-inline is not supported

There's a small bug in one of the scripts where it starts the buildslave process with the wrong environment variables.  It should be running properly now.


HTMLElement.shadowRoot is deprecated.

The contents of atom-text-editor elements are no longer encapsulated
within a shadow DOM boundary. Please, stop using shadowRoot and access
the editor contents directly instead.
HTMLElement.shadowRoot (E:\Atom\Atom x64\resources\app.asar\src\text-editor-element.js:13:10)
MarkdownScrlSync.getVisTopHgtBot (C:\Users\lyf21\.atom\packages\markdown-scroll-sync\lib\utils.coffee:20:34)
MarkdownScrlSync.setMap (C:\Users\lyf21\.atom\packages\markdown-scroll-sync\lib\map.coffee:25:14)
MarkdownScrlSync.startTracking (C:\Users\lyf21\.atom\packages\markdown-scroll-sync\lib\main.coffee:75:12)
<unknown> (C:\Users\lyf21\.atom\packages\markdown-scroll-sync\lib\main.coffee:58:23)
Function.simpleDispatch (<embedded>:11:1172354)

Thanks for contributing!
It appears that this is being caused by the markdown-scroll-sync package, not Atom itself. It may be that there is still a bug in Atom. If so, we'll need the maintainer of the package that is causing the problem to help track that down. You should file this issue on their repository so they can either fix the problem or tell us where the problem is coming from. The Atom FAQ has information on how to contact the maintainers of any Atom community package or theme.


EntityFramework migration to PostGreSql does not create auto-incrementing ids

I switched from MSSql to PostGreSql and let Entity Framework Core create the database using the migrations with /seed argument from the Sample Quickstart 8_EntityFrameworkStorage.
Under MSsql it worked fine but using a PostGreSql Db it has issues with the auto-incrementing ids (e.g. table Clients).
Inspecting the generated PostGreSql DB with pgadmin it shows that the default value is not set.

Do you have a suggestion for how to fix the problem?

Is there a way to do this in your migration creation, so we don't have to add code inside the core ef module for DB specifics?

First link you posted told you how to do this.
In your DbContext impl:
protected override void OnModelCreating(ModelBuilder modelBuilder)
{
    //...

    // Do this for all IdSrv entitites which should auto increment
    modelBuilder.Entity<Client>()
        .Property(e => e.Id)
        .ValueGeneratedOnAdd();
}

Yeah, already forgot about that post. Thank you for your hints. For myself I fixed it by writing a create script for PostGre SQL and scaffolding the db context from that database. Within the Identity Server I still use the default provided EF db context for reading from the db.
This issue was more pointing out (for you or other users) that the provided entiy framework core db context only works with MS SQL databases. It supports at least reading from a PostGre SQL database (since the missing sequences do not influence reading). With the links given I wanted to provide some hints (for other users targeting PostGre SQL) how to get it done.
But this also means using migrations for creating the database with the given db context against other databases than MS SQL won't work out of the box.
Next week (and the week after) I have some days off where I plan to give myself some learning time. One thing I want to learn is how to support multiple databases with one EF Core database context. If I find the time I will learn that by experimenting with the EF core db context of the Identity Server. I hope I get it to work to support MS SQL, PostGre SQL and MySQL.
If you are interested in the results I could share my experiences.

@MontyGvMC one thing we've started using to manage multiple database (instead of EF migrations) is DbUp: https://dbup.readthedocs.io/. I use migrations to get my starter SQL, then hand work it as needed. DbUp then helps apply is to the DB and deal with versioning over time.

If this question is into my direction: I am sorry I don't understand.

@MontyGvMC I was looking for help in solving this within the core EF code we provide without making it provider specific. Otherwise, your fix is to do something in your migrations code, which is beyond this repo thus I can close the issue.


TypeError: write() argument 1 must be unicode, not str

i am getting the following error, please help me to resolve. i am running in python2.7.
Number of parameters is 17926152
Building model...
WARNING (theano.tensor.blas): We did not found a dynamic library into the library_dir of the library we use for blas. If you use ATLAS, make sure to compile it with dynamics library.
Traceback (most recent call last):
File "punctuator.py", line 176, in
restore(output_file, text, word_vocabulary, reverse_punctuation_vocabulary, predict)
File "punctuator.py", line 112, in restore
f_out.write(punctuations[j] if punctuations[j] != data.SPACE else " ")
TypeError: write() argument 1 must be unicode, not str

the issue resolved after running in python 3. very good utility.


Updated Vault to latest 1.14.5 version

The commit message says 1.14.15 but it's actually 1.14.5 in the code and that is the latest version available from https://releases.hashicorp.com/vault


Failed to find match for field 'ring'.

@bierschi, Thanks for your great project!
I use this repository to make NCLT bag files.
Using this repository, I created a 2013-01-10 bagfile and execute the faster-lio algorithm.
But I got a weird result :cry:
The trajectory diverges.

The output of the terminal is below.

In faster-lio repository,
They provide NCLT dataset example in bag file format.
I execute the same sequence(2013-01-10) and the same config file (just change the topic name).
I got a great result.

The output of the terminal is below.

I think the difference between two terminals are
failed to find match for field 'time'
failed to find match for field 'ring'

How could I make ring and time when I make bag file?
Any idea?
Thanks,

Hi, @Taeyoung96
Has the problem been solved?
I want to convert a bag with imu, odometry and velodyne, but the velodyne data failed to find match for field 'time' ,failed to find match for field 'ring'.

@chengwei0427
Hi, I used another python script to generate bag file.
In FAST-LIO, you could use this script and it works fine for me.
Thanks,

Hi， @Taeyoung96
Thanks for your reply.
I want to test my LIO algorithm with encoder/wheel info, so I need  a bag with odometry info.
Do your have any advice? Or some dataset with encoder/wheel info?
Thanks


the cloneType and chiselCloneType hot mess 🔥

Addresses #419
Overview:

cloneType is now marked (through comments only) as an internal API.
chiselCloneType deprecated (and changed to cloneTypeFull internally, analogous to cloneTypeWidth).
chiselTypeOf(data) introduced as the external API to get a chisel type from a hardware object
Intended usage: cloning is an implementation detail, and chisel types and hardware objects both should act as immutable types, with operations like Input(...), Reg(...), etc returning a copy and leaving the original unchanged. Hence, the clone operations are all deprecated.
Deletes what appears to be an unused Bundle companion object.
Input(...), Output(...), Flipped(...) require the object to be unbound

Remaining issues:

Several uses of chiselCloneType remain in utils and tests for compatibility reasons, chiselCloneType does not check that the input is hardware bound while chiselTypeOf(...) does check. This goes into strictness issues with #635 and needs more discussion.
It would be nice for cloneType to return type Data, and have cloneTypeFull / etc fix up the types, because this.type is a nasty hack (see #193) and to avoid the .asInstanceOf[this.type] boilerplate needed in every cloneType implementation. It seems rocket-chip still has things that call cloneType directly so this causes compile errors. Only a small handful, though, so maybe this is something we could do? Otherwise, Scala does not seem to try implicit conversions on the wrong return type, so PML isn't a solution here.
cloneType is not deprecate-able because it is simultaneously an internal and external API. Ideally, if the usages in rocket-chip are few, those could be fixed (with either chiselCloneType or chisel3-semantics chiselTypeOf) and the function (possibly could be?) made protected. (note, this will protect things within our control like Data.cloneType, but not user defined functions that don't explicitly protect their cloneType functions, especially Bundle sub-types)
Cases where the user needs to define cloneType is where it get hairy, because that's where the intended abstraction does break down and users have to think about object uniqueness. Prime examples: ComplexAssign and RecordSpec test cases. Bundle annotations (or more advanced reflection-based cloning) might address some (perhaps even most) cases. Record especially might be difficult, because the elements can be defined however. This will probably be the biggest issue, especially since Bundle constructors can be deceptively wrong (what if you put a chisel type argument directly as a Bundle element and don't redefine the Bundle clone so the elements get cloned?)

This should not be merged yet, this is just a starting point to think about these issues.

Should note (at least for now) the necessity of overriding cloneType for user-defined types (e.g. subclasses of Bundle).

Resolution:

For internal APIs that need to be exposed to Chisel._ compatibility (like chiselCloneType), qualify them into some namespace to prevent people from just using them.
No resolution for the cloneType/chiselCloneType mess when the nice "all chisel types are immutable and you don't need to worry about object reference" abstraction breaks, because we don't know how to fix it. Goal is to reduce the number of cases where the user needs to override cloneType by incremental whackamole.

Alright I've merged master into this (@ducky64 if you could check my work that would be great because it was nontrivial). Master is now bumped on rocket-chip for the first time in months so we can more easily check this against big code bases!

@jackkoenig changes look good.

Okay so the only problem I see with this is pretty minor but I want to see if there's a better solution. Basically, this PR makes it to where if you are using chiselCloneType (be it in compatibility or regular chisel3 code) you get inferred existential types. This is just an annoying warning, but I don't understand why it now comes up but didn't before. It's also problematic if you default to erroring on warnings (you know, good practice).
An example that illustrates the problem:
import chisel3._

class MyTestSubModule extends Module {
  val io = IO(new Bundle {
    val in = Input(UInt(32.W))
    val out = Output(UInt(32.W))
  })
  io.out := io.in
}

class MyTestModule extends Module {
  val child = Module(new MyTestSubModule)
  val io = IO(child.io.chiselCloneType)
  io <> child.io
}

object Test extends App {
  chisel3.Driver.execute(args, () => new MyTestModule)
}

This simply passes through the IO of a child module. On master, there is no warning, on this branch we get the existential type warning for the io of the parent module (MyTestModule).
Weirdly, if we change the io of the parent module to flip the child io type twice, the warning goes away:
class MyTestModule extends Module {
  val child = Module(new MyTestSubModule)
  val io = IO(Flipped(Flipped(child.io.chiselCloneType)))
  io <> child.io
}

Is there any way we can remove this warning without forcing users to change their code or build configuration?

Strange... chiselCloneType was moved to compatibility using PML, so instead of returning this.type, it's now:
implicit class cloneTypeable[T<:Data](val target: T) extends AnyVal {
   def chiselCloneType: target.type = {
     DataMirror.internal.chiselTypeClone(target).asInstanceOf[target.type]
   }
 }

Also interestingly, IO is defined as
protected def IO[T<:Data](iodef: T): iodef.type = {

while Flipped is defined as
def apply[T<:Data](source: T): T = {

Does redefining IO to return T (instead of iodef.type) help? Or does that also break things? I'm not sure why IO is defined that way...

Changing IO to return T instead of iodef.type helps for this specific case, but not for a minorly modified version:
class MyTestModule extends Module {
  val child = Module(new MyTestSubModule)
  val io = IO(new Bundle {
    val port = child.io.chiselCloneType
  })
  io.port <> child.io
}

However, changing chiselCloneType to return T instead of target.type did fix it. The question is why do they return the structural type instead of just the type parameter type?

I'm not sure why IO returns a structural type, but if it works with parameter types, we should change it to that.
I think the reason chiselCloneType returns a structural type is because the old style (this.cloneType) returns this.type, and this was the minimal change. But again, I think the type parameter type should be preferred if it works. A long time ago, the uses of this.type was flagged as an issue which we're slowly solving.


Error executing verb "orderby" in create_base_text_units, Error running pipeline!

Describe the issue
Just tried to load in a simple txt file.
In the terminal is goes until :
⠼ GraphRAG Indexer
├── Loading Input (text) - 1 files loaded (0 filtered) ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 100% 0:00:00 0:00:00
├── create_base_text_units
├── create_base_extracted_entities
├── create_summarized_entities
└── create_base_entity_graph
❌ Errors occurred during the pipeline run, see logs for more details.
Steps to reproduce
No response
GraphRAG Config Used
encoding_model: cl100k_base
skip_workflows: []
llm:
api_key: ${GRAPHRAG_API_KEY}
type: openai_chat # or azure_openai_chat
model: gpt-4-turbo-preview
model_supports_json: true # recommended if this is available for your model.
max_tokens: 4000
request_timeout: 180.0
api_base: https://.openai.azure.com
api_version: 2024-02-15-preview
organization: <organization_id>
deployment_name: <azure_model_deployment_name>
tokens_per_minute: 150_000 # set a leaky bucket throttle
requests_per_minute: 10_000 # set a leaky bucket throttle
max_retries: 10
max_retry_wait: 10.0
sleep_on_rate_limit_recommendation: true # whether to sleep when azure suggests wait-times
concurrent_requests: 25 # the number of parallel inflight requests that may be made
temperature: 0 # temperature for sampling
top_p: 1 # top-p sampling
n: 1 # Number of completions to generate
parallelization:
stagger: 0.3
num_threads: 50 # the number of threads to use for parallel processing
async_mode: threaded # or asyncio
embeddings:
parallelization: override the global parallelization settings for embeddings
async_mode: threaded # or asyncio
llm:
api_key: ${GRAPHRAG_API_KEY}
type: openai_embedding # or azure_openai_embedding
model: text-embedding-3-small
# api_base: https://.openai.azure.com
# api_version: 2024-02-15-preview
# organization: <organization_id>
# deployment_name: <azure_model_deployment_name>
# tokens_per_minute: 150_000 # set a leaky bucket throttle
# requests_per_minute: 10_000 # set a leaky bucket throttle
# max_retries: 10
# max_retry_wait: 10.0
# sleep_on_rate_limit_recommendation: true # whether to sleep when azure suggests wait-times
# concurrent_requests: 25 # the number of parallel inflight requests that may be made
# batch_size: 16 # the number of documents to send in a single request
# batch_max_tokens: 8191 # the maximum number of tokens to send in a single request
# target: required # or optional
chunks:
size: 1200
overlap: 100
group_by_columns: [id] # by default, we don't allow chunks to cross documents
input:
type: file # or blob
file_type: text # or csv
base_dir: "input"
file_encoding: utf-8
file_pattern: ".*\.txt$"
cache:
type: file # or blob
base_dir: "cache"
connection_string: <azure_blob_storage_connection_string>
container_name: <azure_blob_storage_container_name>
storage:
type: file # or blob
base_dir: "output/${timestamp}/artifacts"
connection_string: <azure_blob_storage_connection_string>
container_name: <azure_blob_storage_container_name>
reporting:
type: file # or console, blob
base_dir: "output/${timestamp}/reports"
connection_string: <azure_blob_storage_connection_string>
container_name: <azure_blob_storage_container_name>
entity_extraction:
llm: override the global llm settings for this task
parallelization: override the global parallelization settings for this task
async_mode: override the global async_mode settings for this task
prompt: "prompts/entity_extraction.txt"
entity_types: [organization,person,geo,event]
max_gleanings: 1
summarize_descriptions:
llm: override the global llm settings for this task
parallelization: override the global parallelization settings for this task
async_mode: override the global async_mode settings for this task
prompt: "prompts/summarize_descriptions.txt"
max_length: 500
claim_extraction:
llm: override the global llm settings for this task
parallelization: override the global parallelization settings for this task
async_mode: override the global async_mode settings for this task
enabled: true
prompt: "prompts/claim_extraction.txt"
description: "Any claims or facts that could be relevant to information discovery."
max_gleanings: 1
community_reports:
llm: override the global llm settings for this task
parallelization: override the global parallelization settings for this task
async_mode: override the global async_mode settings for this task
prompt: "prompts/community_report.txt"
max_length: 2000
max_input_length: 8000
cluster_graph:
max_cluster_size: 10
embed_graph:
enabled: false # if true, will generate node2vec embeddings for nodes
num_walks: 10
walk_length: 40
window_size: 2
iterations: 3
random_seed: 597832
umap:
enabled: false # if true, will generate UMAP embeddings for nodes
snapshots:
graphml: false
raw_entities: false
top_level_nodes: false
local_search:
text_unit_prop: 0.5
community_prop: 0.1
conversation_history_max_turns: 5
top_k_mapped_entities: 10
top_k_relationships: 10
llm_temperature: 0 # temperature for sampling
llm_top_p: 1 # top-p sampling
llm_n: 1 # Number of completions to generate
max_tokens: 12000
global_search:
llm_temperature: 0 # temperature for sampling
llm_top_p: 1 # top-p sampling
llm_n: 1 # Number of completions to generate
max_tokens: 12000
data_max_tokens: 12000
map_max_tokens: 1000
reduce_max_tokens: 2000
concurrency: 32
Logs and screenshots
{"type": "error", "data": "Error executing verb "orderby" in create_base_text_units: 'id'", "stack": "Traceback (most recent call last):\n  File "C:\Users\\u00c1ronHet\u00e9nyi\graphrag\.venv\lib\site-packages\datashaper\workflow\workflow.py", line 410, in _execute_verb\n    result = node.verb.func(**verb_args)\n  File "C:\Users\\u00c1ronHet\u00e9nyi\graphrag\.venv\lib\site-packages\datashaper\engine\verbs\orderby.py", line 32, in orderby\n    output = input_table.sort_values(by=columns, ascending=ascending)\n  File "C:\Users\\u00c1ronHet\u00e9nyi\graphrag\.venv\lib\site-packages\pandas\core\frame.py", line 7189, in sort_values\n    k = self._get_label_or_level_values(by[0], axis=axis)\n  File "C:\Users\\u00c1ronHet\u00e9nyi\graphrag\.venv\lib\site-packages\pandas\core\generic.py", line 1911, in _get_label_or_level_values\n    raise KeyError(key)\nKeyError: 'id'\n", "source": "'id'", "details": null}
{"type": "error", "data": "Error running pipeline!", "stack": "Traceback (most recent call last):\n  File "C:\Users\\u00c1ronHet\u00e9nyi\graphrag\graphrag\index\run.py", line 323, in run_pipeline\n    result = await workflow.run(context, callbacks)\n  File "C:\Users\\u00c1ronHet\u00e9nyi\graphrag\.venv\lib\site-packages\datashaper\workflow\workflow.py", line 369, in run\n    timing = await self._execute_verb(node, context, callbacks)\n  File "C:\Users\\u00c1ronHet\u00e9nyi\graphrag\.venv\lib\site-packages\datashaper\workflow\workflow.py", line 410, in _execute_verb\n    result = node.verb.func(**verb_args)\n  File "C:\Users\\u00c1ronHet\u00e9nyi\graphrag\.venv\lib\site-packages\datashaper\engine\verbs\orderby.py", line 32, in orderby\n    output = input_table.sort_values(by=columns, ascending=ascending)\n  File "C:\Users\\u00c1ronHet\u00e9nyi\graphrag\.venv\lib\site-packages\pandas\core\frame.py", line 7189, in sort_values\n    k = self._get_label_or_level_values(by[0], axis=axis)\n  File "C:\Users\\u00c1ronHet\u00e9nyi\graphrag\.venv\lib\site-packages\pandas\core\generic.py", line 1911, in _get_label_or_level_values\n    raise KeyError(key)\nKeyError: 'id'\n", "source": "'id'", "details": null}
Additional Information

GraphRAG Version:
Operating System:
Python Version:
Related Issues:

请问您是如何解决这个问题的

Hi! Solution?

hi？

same

Instead of using > operator, that includes everything returned from the website, -o should be used for downloading the file. Changing to -o attribute fixes the error.
Change this -
curl https://www.gutenberg.org/cache/epub/24022/pg24022.txt > ./ragtest/input/book.txt
To this -
curl https://www.gutenberg.org/cache/epub/24022/pg24022.txt -o ./ragtest/input/book.txt

Instead of using > operator, that includes everything returned from the website, -o should be used for downloading the file. Changing to -o attribute fixes the error.
Change this -
curl https://www.gutenberg.org/cache/epub/24022/pg24022.txt > ./ragtest/input/book.txt
To this -
curl https://www.gutenberg.org/cache/epub/24022/pg24022.txt -o ./ragtest/input/book.txt

The.txt set format to UTF-8.

This fixed the issue for me. Thank you.
How to change format> --> just open the file in a notpad and save it again, change the format to UTF-8.

Instead of using > operator, that includes everything returned from the website, -o should be used for downloading the file. Changing to -o attribute fixes the error.
Change this - curl https://www.gutenberg.org/cache/epub/24022/pg24022.txt > ./ragtest/input/book.txt To this - curl https://www.gutenberg.org/cache/epub/24022/pg24022.txt -o ./ragtest/input/book.txt

thank you this fixed my problem


Move and_bloq.py and multi_control_multi_target.py to new mcmt/ folder

Part of a larger effort to organize bloqs/

one tiny note with the "multi control multi target" thing: the current module made me think there was a bloq in it that would do multi-control multi-target cnot. But it's really "multi control" or "multi target". Making it a (sub)package instead of a module might actually improve things, but I wanted to highlight this potential confusion.

Making it a (sub)package instead of a module might actually improve things

Yeah, splitting multi_control_multi_target.py into smaller files is part of the plan. I didn't want this PR to blow up even more so I'll do that as a follow-up.


sokol_gfx.h: minor doc fixes (typo, copy-paste)

Fixed minor typos in sokol_gfx.h doc: metal was used instead of d3d11 or wgpu.

Much appreciated, thanks!


VSIX Format Change for Snippets

VSCode Version: 1.8.1 and earlier
OS Version: Win 10, Win 7

Steps to Reproduce:

create a VSIX for snippets
install as extension
call snippet
some characters are not escaped correctly
exact same VSIX works in versions prior to 1.8.1

Looks like {{ is not escaped correctly and $ now needs to be escaped. Is this new?

@billjwolff Please attach a sample snippet or the extension in question. Snippets contributed via extensions, were always using a subset of the TextMate snippet syntax and still do: https://code.visualstudio.com/docs/customization/userdefinedsnippets#_snippet-syntax.
For 1.8 we have rewritten our snippet parser to be more correct and to allow for new feature. That also removed internal snippet syntax we were supporting there by accident (not by spec). What you refer to {{ seems like old/internal/undocumented syntax


[otbn] elf loading non-functional for standalone OTBN sim

When building a binary using otbn-as directly supplying the elf file as initial memory contents for the IMem in the standalone simulation doesn't seem to work. You need to produce a .vmem from the .elf with objcopy and srec_cat.

estimate 4

Yep, we definitely need to get this working! :-) I think the easiest approach might be a simple-minded Python tool that gets called as a subprocess. Then we can use pyelftools to do the heavy lifting, and can probably re-use the result for the final DV environment too (calling the subprocess via some DPI wrapper).

Given it works fine with the same code in the ibex simple_system environment and the OT earlgrey I'd hope we wouldn't need to do anything that involved as the work is already done. Mostly raised the issue so I don't forget about it, wanting to concentrate on non OTBN things for the rest of the day.
I've switched this to a 'bug' from 'enhancement' as it technically should work already.

After looking into this .elf loading is perfectly functional but you need to be supplying it with an appropriate .elf (I had initially used the a.out produced by otbn-as but as this lacked a program header it didn't load).
I've written a short shell script that will build a single .S link it (with otbn-ld) then use objcopy to get seperate imem and dmem elfs. We could alter the Verilator infrastructure to make this a bit easier (take a single .elf then load it into seperate dmem and imem memories) but this was the simplest thing to do for now.
I'll close this issue as I've marked it as a bug but it's perfectly functional after all. We may consider improvements to the verilator infrastructure but that will be a separate issue.

Well, long term, we should definitely consume a single ELF for IMEM and DMEM. But I'm happy to track that separately.

Yup agreed. For now I'm trying to get smoke test going in the simplest way possible (as it will be retired when we have more DV infrastructure). Doing this properly may involve some changes to the shared verilator infrastructure or spinning up something new which I felt fit better with the overall DV infrastructure work.


Hardware keyboard auto blur issue on keyboard hide

When a hardware keyboard is connected to the device, if you click on close toolbar (arrow-down) icon on bottom right corner, the input is not focusable anymore. It loses focus immediately at the time of touching on it.
I noticed that native.keyboardshow and native.keyboardhide events fired multiple times, the state of the keyboard of being showed or not is not stable therefore ionic-angular calls blur on input and it loses the focus immediately.

I'm seeing the same issue. Will try out the pull request tomorrow. Thanks!

We have tried out the fix in the pull request, and it works well! Thanks, onderceylan, for reporting and making the fix!

Glad to see it validated @uniphonic, hope it will be merged soon.

@tlancina Do you think that we can have this feature merged to the code base?


Complete upgrade to Cypress 13.13.2

See https://github.com/plone/volto/pull/6241

Thanks!

Thank you for cleaning up my mess. One of these days, I will become a JavaScript developer.


Fix fixed-size Array visualization

Don't assume that has_maximum_size means that a maximum_size field
exists on the object. In the case of an Array object, has_maximum_size
is True, but there is no 'maximum_size' field, only 'size'.
Fixes: #77
Signed-off-by: Michael Jeronimo<EMAIL_ADDRESS>

Linux
Linux-aarch64
Windows


remove comlib

Original author: Jonathan Lifflander
Original issue: https://charm.cs.illinois.edu/redmine/issues/322

No body.

Original date: 2015-03-21 00:57:00

Progress noted in dupe.


nightly 2018/03/19: change of position of arguments passed to cargo subcommands

Hello,
This nightly just broke my crate:
https://travis-ci.org/rust-fuzz/honggfuzz-rs/builds
It looks like it there has been a change in how cargo passes arguments to subcommands.
I guess we have a typical case of:

Here is the piece of code that worked before and now doesn't: https://github.com/rust-fuzz/honggfuzz-rs/blob/master/src/bin/cargo-hfuzz.rs#L169-L171
    let mut args = env::args().skip(1);
    if args.next() != Some("hfuzz".to_string()) {
        eprintln!("please launch as a cargo subcommand: \"cargo hfuzz ...\"");
        process::exit(1);
    }

Just to be clear, this is absolutely no big deal for me and I don't care adapting my code. I just want to be sure, the change in behavior is known.

cc @matklad

@PaulGrandperrin if you've already investigated it, could you tell how the args looks on stable and on nightly?

Figured it out myself:
Before
["/home/matklad/.cargo/bin/cargo-hfuzz", "hfuzz", "clean"]

After
["/home/matklad/.cargo/bin/cargo-hfuzz", "clean"]

I can't believe we don't have a test for it...

Arg, you were fast!
Yes, that's why I felt it would be sensible to report it :-)

@PaulGrandperrin thank you so much for the report! We've recently switched from docopt to clap, so I've expected regressions for some obscure corner cases, but this one is as far from the spacebar heater as it gets...

Glad it helped!
And it's always fun to be able to mention XKCD strips in bug reports 😉
Thanks for your work!

Hiya! I know the title says nightly 2018/03/19, but I'm wondering if this is the same issue that's causing this to happen on 1.26.0-nightly (d6c3983fe 2018-03-16):
$ cargo audit
thread 'main' panicked at 'cargo-audit is intended to be invoked as a cargo subcommand', src/main.rs:53:13
note: Run with `RUST_BACKTRACE=1` for a backtrace.

$ cargo audit audit
    Fetching advisories `https://raw.githubusercontent.com/RustSec/advisory-db/master/Advisories.toml`
    Scanning 238 crates for vulnerabilities (5 advisories in database)
     Success No vulnerable packages found

$ cargo --version --verbose
cargo 1.26.0-nightly (d6c3983fe 2018-03-16)
release: 1.26.0
commit-hash: d6c3983fe3bd8fa06b54712e53fb23645598188b
commit-date: 2018-03-16

If it's different I can raise another issue, but the test code looks like it's this one.

@azriel91 yep, this looks like the same issue! Hopefully the Cargo fix will to the nightlies soon


Error in documentation on the main page

Please note that the commands in section "Add/Update the RegEx filters to Pi-hole" are in reversed order. The first command shouldn't install the filters, but uninstall them. Uninstall the filters first, then install them.

Fixed. Thanks.


Add ice40 and riscv toolchain support to the conda env;

add Fomu GHA that uses new conda-provided gcc.
Signed-off-by: Tim Callahan<EMAIL_ADDRESS>

@alanvgreen PTAL.   This doesn't make the final jump to Conda but it does use Conda to acquire Ice40 tools for running in CI.


[Feature] [SwaggerV2] Create Folders from implicit defined tags

Swagger Tags can be defined in the document root
...
tags:
  - name: tagname
    description: tagdescription
...

or implicit in every method
...
get:
  operationId: "getsomething"
  tags:
    - tagname
...

The implicit variant can not define an description.
With this little change the implicit tags get imported as folders as well

Yes, i can add a few test fixtures. The tests didn't run correctly on my side and always returned "passed". Not ideal to write tests.


Invalid call on onThemeChanged

The onThemeChanged is called even though the current theme is the same.

Issue Description Update Due to Repository Changes :construction:
Description :pencil2::
During the beta phase of development, this issue was created to address specific functionality within the repository structure. However, since then, there have been significant changes to the repository's architecture and organization. As a result, the context and relevance of this issue have evolved.
Given the updated repository structure, the original problem or feature addressed by this issue may no longer be applicable or aligned with the current development goals. :triangular_flag_on_post: Consequently, the issue needs to be reviewed and potentially redefined to match the revised project scope.
To maintain a clear and accurate issue tracking process, it is recommended to reevaluate the objectives and requirements associated with this issue in light of the new repository structure. :mag_right: If the issue is no longer relevant or can be resolved through the updated architecture, it should be closed with appropriate comments explaining the decision.
The development team is encouraged to collaborate and communicate effectively during this evaluation process to ensure that all changes are documented, understood, and aligned with the project's goals.
Thank you for your understanding and cooperation in maintaining an organized and up-to-date repository. :raised_hands:

This issue will be closed.


Switch the format of the click data input to the new source

Goal
We need to get the Thompson Sampling click data differently than we are doing it right now. This is the fix!
Tickets
BACK-680

Chelsea and I tried for a little bit to fix the test, but we couldn't quite figure out where it was failing. I created a fresh PR with just the clickdata refactor, without removing the old explore topics logic, and it passes tests. Not quite sure why, but we needed to split this PR up anyway to get it out Tuesday morning: https://github.com/Pocket/recommendation-api/pull/225


Add external munin roles and idr munin playbook

Adds munin (server) and munin-node (client) roles as submodules in ansible/vendor/, and an example playbook for setting them up on a suite of servers.

Empty PR description. Please add a short summary of the PR scope and some testing instructions.

Empty PR description. Please add a short summary of the PR scope and some testing instructions.

Empty PR description. Please add a short summary of the PR scope and some testing instructions.

Empty PR description. Please add a short summary of the PR scope and some testing instructions.

Mentioned to @manics, I'd probably err on the side of requirements.yml rather than a submodule, but could use other's opinions: @kennethgillen @aleksandra-tarkowska @sbesson

Two quick comments:

this effectively introduces a second nested layer of submodules (as infrastructure is included as a submodule of management_tools). As far as I know, this is something we have never dealt with this and it is likely this might require work on some of the tooling/workflow/training.
generally, I don't think we want to point to a personal repository from an organization repository. Is there a plan to migrate the munin roles to the organization at some point?

generally, I don't think we want to point to a personal repository from an organization repository. Is there a plan to migrate the munin roles to the organization at some point?

Yes, it's a fork of another repo so I had to fork it into my own user.

Is this off-hold now? i.e. we can make decisions and get it in?

Force pushed, added https://github.com/manics/ansible-role-munin.git and https://github.com/manics/ansible-role-munin-node.git using git read-treeForce pushed, added https://github.com/manics/ansible-role-munin.git and https://github.com/manics/ansible-role-munin-node.git using git read-tree

Probably needs a general agreement from @openmicroscopy/devops

--exclude

I've force pushed using a new strategy described in the description.

A playbook requiring these nodes is in https://github.com/openmicroscopy/infrastructure/pull/187

Force pushed again, this time using requirements.yml

I've switched to using https://github.com/openmicroscopy/ansible-role-munin{,-node}.git
We could leave the tagging discussion for later?

Merging to make progress on the other currently open PRs. Would be good to get tags in place for the overall 0.3.0 tag though.


Why variables only support numbers and no strings

great tool and some suggest
I think you can add a variable support string and suggest that possible values for a given argument can be entered instead of not returning directly.

I don't get it. You're saying that you can type "hello" for , for example? If so, that's a bug.
Did you install the tool via brew or git? Could you run "navi --version"? I believe you're using a broken version

my version is 0.8.1 , installed by git.
echo <x> <y>
$ x: echo -e '1'
$ y: echo -e "$((x))"

it's fine , `y=1'
but if use
echo <x> <y>
$ x: echo -e 'a'
$ y: echo -e "$((x))"

y is noting, it's not work.

In your particular example, "$((x))" is a math operation, and won't work with non-numerical x
This should work:
echo <x> <y>
$ x: echo -e 'a'
$ y: echo -e "$x"


BUG: Wasserstein Distance

Describe your issue.
The EMD for the following case is expected to be non-zero, but Scipy gives an answer of 0.
Reproducing Code Example
if __name__ == '__main__':
    from scipy.stats import wasserstein_distance
    print(wasserstein_distance([1/2, 1/2, 0, 0], [0, 0, 1/2, 1/2]))

Error message
No.

SciPy/NumPy/Python version information
1.5.2 1.19.2 sys.version_info(major=3, minor=6, micro=13, releaselevel='final', serial=0)

Thanks @xingyuuchen for reporting. This is a statistical distance comparing the "distance" between probability measures, since you have the same elements in both arrays I don't see why it should be different than 0. Why did you expect something different?

Thanks @xingyuuchen for reporting. This is a statistical distance comparing the "distance" between probability measures, since you have the same elements in both arrays I don't see why it should be different than 0. Why did you expect something different?
I am closing the issue since I am confident this is the expected result.

The Wasserstein distance is also known as the earth mover’s distance, since it can be seen as the minimum amount of “work” required to transform  u into v, where “work” is measured as the amount of distribution weight that must be moved, multiplied by the distance it has to be moved.
Obviously the two distributions are not equal, so the "work" to transport u into v is a positive number, isn't it?

Obviously the two distributions are not equal, so the "work" to transport u into v is a positive number, isn't it?

Why would that be? When you would calculate the empirical CDF of both vectors, they would be exactly the same. And Wasserstein is just |CDF1 - CDF2|. The order of the elements does not matter for PDF/CDF, only the frequency does.

Obviously the two distributions are not equal, so the "work" to transport u into v is a positive number, isn't it?

Why would that be? When you would calculate the empirical CDF of both vectors, they would be exactly the same. And Wasserstein is just |CDF1 - CDF2|. The order of the elements does not matter for PDF/CDF, only the frequency does. To convince yourself, just take a 1D array of size 1000 and plot the histogram, shuffle the array and plot again the histogram. They will be strictly the same.

Thanks, I see what you mean. But in some cases the order of elements matters. e.g.,
distribution1: the animal has a 50% chance to be a cat, 50% to be a dog, 0% tiger, 0% lion;
distribution2: the animal has a 0% chance to be a cat, 0% to be a dog, 50% tiger, 50% lion,
in which case, these two differs.

You are not using the right tool for that. For it to work with this distance: since you have a 3-dimensional distribution, you have to either assume no correlation and do 3 analysis (and possibly combining them together in a single metric) or you need to work on the 3-dimensional distribution at once (not supported by SciPy). As it is, you described it as a 1-dimensional distribution which is wrong.

So how can I measure the EMD between my example (distributions of the animal prediction), is there an API?

I cannot say more than above. I am sorry but this is not a forum and we don't have much bandwidth. I will redirect you to StackOverflow or https://stats.stackexchange.com in your case might be appropriate.

Ah....
I checked the documentation again, it says the input vectors are observed values in the distribution, not the probability of each observed value. This would be the exact reason that my example is not appropriate.
Thanks again for your patience.

Ah.... I checked the documentation again, it says the input vectors are observed values in the distribution, not the probability of each observed value. This would be the exact reason that my example is not appropriate.

Yes, u and v are samples from 1-d distributions.

Thanks again for your patience.

Pleasure.


Nested Navigation back not working in web in Beamer version 0.13.0

Refer the attached video for steps.
https://user-images.githubusercontent.com/16254421/119208859-85223c80-bac1-11eb-9189-108e6edb41b0.mov

Thanks for creating an issue @hamirshekhawat
I can confirm this is also present on master. It seems that the problem is somewhat intermittent, it sometimes rebuilds the app after browser back/forward and sometimes not

I am wodin from your discord server btw. I can test this in prod cuz mostly such issues are resolved after doing a 'build web' command. I have been able to reproduce it always in debug build. Especially between the first two pages in a flow. Once you navigate to a 3rd page and come back to second and first, the pages change respectively. But if you only navigate between the first and second, the defect is reproduced consistently.


Design patterns

I just want to discuss a little bit about design patterns for the server signals.
Leptos promotes a software design, which is very modular. You have different components, which are normally isolated from each other and which are combined to get a full application. The signal system pays in into this modular design, since every component can have its own private signals and there is no global instance managing those signals. Another element to that are server-functions, little pieces of code, which are running on the server, but they are closely implemented together with the components.
The wasm side of the server signals fits perfectly to this vision: It is easy to create server signals for every component, so that the functionality is isolated from other components. But the backend side (axum for example) is breaking this design pattern. Since the websocket is shared for all server signals and there can only be one handler for it. This means, that this single handler has to manage all server signals, regardless in which component they are used. Which is in contract to the modular design of the leptos components.
So how to solve this? How to disangle the server signals on backend from each other to keep the backend implementation for all server signals close to the component, where they are used?
I'm currently thinking of the following design on backend:

There is a single handler, which is a little bit like a wrapper around the set of handlers for different server signals
Inside the handler the websocket is put into an Arc<Mutex<...>>
There is a set of server signal specific handlers, which are all awaitable

Every handler gets a clone of the Arc<Mutex<Websocket>> object
When a signal update should be updated, we lock the mutex and get a dereferenced mutable object from the websocket to send the update

There is a join operation to join the futures of all handlers and the joined future is then awaited

This solution is far more complicated than implementing just some server functions for a component. If, for example, a server signal should be updated from a server function, we need to connect the server function with the handler over something like a message queue. But in the end it allows to implement a dedicated handler for the server signals used by a single component and to register this handler on backend side for the usage inside the wrapper around all handlers.
What are your design patterns around server signals?

Hi sorry for the late response.
I agree with your issue, in that it would be ideal for server signal code to be in same area as components. Ideally, there would be a solution pretty much mimicking server functions in leptos, where a macro is provided, and you annotate a function with this macro and write your websocket code in there.
I wrote a basic example in this issue: https://github.com/tqwewe/leptos_server_signal/issues/6
The only reason this hasn't been implemented is simply because of the extra work involved in this, and I haven't had the time or need to go this far yet. However this would be a great API design for this library, and would contribute to solving client writable signals too.

Yes you are right, this would be a really nice API. It would be cool to see something like that in future.


Add more amazing people to the names generator

Signed-off-by: Tobias Klauser<EMAIL_ADDRESS>

LGTM

LGTM

LGTM


naming munging isn't universally applied

Some places like case condition-bodies don't have name munging applied:
#...
| cnd {
    name-token = ...
}
#...

isn't munged: we get raw name-token in the results, but not in function calls. Need to check all writers to see if they're using name munging.

Should be fixed with https://github.com/lojikil/coastML/commit/fbfc4fbbea0cef662888d0ee59a69437c9c6d4b0


Symbol Rename does NOT rename all instances thus breaking code

VSCode Version:
latest 1.12.1

OS Version:
Windows 7

Steps to Reproduce:

In ionic2 development we typically have a view (eg. HomePage.html) and its associated "controller" eg. HomePage.ts or even perhaps an injected provider/service class eg. HomePageProvider.ts

When renaming in typescript code, the rename process fails to catch  angular2 embedded references to objects/props/methods,  For example if I renamed prop HomePageProvider.Users to UsersNamesList it will only update the .ts files in the project.

However, in pages as in the code below Symbol Rename fails to detect instances like .Users below which will require manually finding and fixing.
HomePage.html
<ion-list>
        <ion-item-sliding *ngFor="let level of HomePageProvider.Users">

I bet you those angular2 folks did not see it coming when they embedded code inside comments :)
like old SQL.
Thank you for your help and/or providing alternative solutions.

We don't support angular out of the box. Try out the angular language service extension: https://marketplace.visualstudio.com/items?itemName=Angular.ng-template I'm not sure if that extension supports rename fully, but the angular team has been working to integrate better with TypeScript for scenarios like this


Update aws-java-sdk-dynamodb to 1.12.533

About this PR
📦 Updates com.amazonaws:aws-java-sdk-dynamodb from 1.11.1034 to 1.12.533
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-dynamodb" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-dynamodb" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #485.


View's touch nativeEvent doesn't provide correct pageY value with respect to the window/screen area

Environment
react-native -v: 9.2.1
npm ls react-native-macos: 0.68.58
node -v: 16.17.1
npm -v: 8.6.0
yarn --version: 1.22.18
xcodebuild -version: Xcode 14.0, Build version 14A309

Steps to reproduce the bug
I recently tried a animation sample on rn-macos, and noticed a unexpected behaviour in compare to other 4 platforms I ran the implementation on, it is related to the measured events we get for a view with respect to the rendered screen area/window, when it is first rendered and one from the touch event on the view, as they are comparably different.
Expected Behavior
I expect it to provide the data in similar way as I get on other platforms, especially rn-windows desktop app, as it is much closer to macos compared to mobile and web.
It should not consider the title and tab bar in pageY value in case of touch events, though I'm not really sure if this is a expected behaviour here, but even if it is, is there any way to get title bar height in rn-macos, or also to know if "show tab bar" is enabled, to workaround this?
Actual Behavior
The above mentioned example here is this one, where I'm doing a gesture based toolbar animation, and for that:

I'm getting the view's offset from the top using view ref's measure function here, which is giving me pageY value for 40, which is correct as this is the margin top I've applied here
Using touch events, onTouch(Start/Move/End) here, when user touch the view/screen, using the same property pageY from nativeEvent, I'm comparing the two and activating the toolbar's button based on that.

Now on the first case I'm getting the supposedly expected value, but in the second, in touch events, I'm getting pageY value as the margin top (40) + title bar height, which is then around ~70.
see the image below

and this is causing inconsistency with the animation, as compared to other platforms, even on rn-windows.
Now if you enable to tab bar option from menu bar -> view -> 'show tab bar'

it's gonna also consider this tab bar's height in the pageY value I get from the touch events, which is then gonna be somewhere around ~97.
Let me know if any more details is required.
Reproducible Demo
for repro you can try the sample here, as this is where I faced this problem, the project also contains other platform modules, so you can try them out and compare the results.
Additional context
I searched the existing issue to see anything related, and the PR #1288 seems somewhat related, though I'm not sure if the mentioned issue is really fixed here in case of touch events, as for some reason I'm getting the location(X/Y) and page(X/Y) values same, as you can see in the screenshot

Hello! Is this still an issue you're seeing? I don't think we've changed this code path recently but I wanted to check again since it's been a while.

Hi, I gave the example a try in 0.71, and it seems to be working fine, both for pageY value and the mentioned PR point.