x/build: gotip-windows-arm64 builders stops working

#!watchflakes
default <- builder ~ `windows-arm64` && log ~ `fatal error: out of memory`

https://ci.chromium.org/ui/p/golang/builders/luci.golang.ci/gotip-windows-arm64
seems all builders are offline.
cc @golang/release @thanm

I got an access grant, and logged into VMs to inspect them.  Both were up (not hung or dead) but the "swarming" user was completely inactive (which is not supposed to happen if the systems are healthy). I inspected the system event logs but I don't see any red flags-- last entry in the logs for anything useful done by swarming is on Apr 7th, then after that the user just vanishes.
From the bot logs I see this in the Apr 7th swarming bot log ("C:\Users\swarming.swarming\logs\bot_stdout.log.1"):
Found a previous bot, 11832 rebooting as a workaround for https://crbug.com/1061531
Sleeping for 300 secs
We have SWARMING_NEVER_REBOOT set to true for these VMs, but the code in question doesn't seem to respect that.
Of course that doesn't explain why we would have two copies of the swarming bot running at the same time in the first place. Also a mystery as to why we don't get a proper auto-logon of the swarming user after this happens (since when I do manual restarts we don't seem to have this issue). If anyone has any ideas on how to debug this let me know.
I restarted both VMs and and they seem to be processing jobs again.

I'll take a look. Wish I could figure out how to make this builder a bit more bulletproof.

OK, VMs restarted again.  VMs were in the same state as last time, e.g. the
Found a previous bot, 11832 rebooting as a workaround for https://crbug.com/1061531
problem.

Still working on trying to make our LUCI windows-arm64 builders more reliable.
The latest set of problems here seem to relate to system oversubscription. I restart the VMs, and they run for a few days or a week, then at a certain point jobs launched to them wind up failing early with "out of memory" errors.
Sometimes the problems are in the LUCI infrastructure (ex: cas_download), e.g.
runtime: VirtualAlloc of 3710976 bytes failed with errno=1455
fatal error: out of memory
...
runtime.(*mheap).alloc(0x38a000?, 0x1c5?, 0x58?)
    runtime/mheap.go:958 +0x54 fp=0x40037cca10 sp=0x40037cc9c0 pc=0x7ff6df88cad4

and sometimes the out of memory errors happen during test build:
    go_test.go:2588: go [build x] failed unexpectedly in C:\Users\swarming\.swarming\w\ir\x\w\goroot\src\cmd\go: fork/exec C:\Users\swarming\.swarming\w\ir\x\t\cmd-go-test-3753799972\tmpdir1058398312\testbin\go.exe: The paging file is too small for this operation to complete.

or this during a test run:
        fatal error: out of memory allocating heap arena map

        runtime stack:
        runtime.throw({0x7ff622467db4?, 0x0?})
            C:/Users/swarming/.swarming/w/ir/x/w/goroot/src/runtime/panic.go:1026 +0x38 fp=0x4e8f3ff4a0 sp=0x4e8f3ff470 pc=0x7ff621d45808
        runtime.(*mheap).sysAlloc(0x7ff6229de540, 0x101f?, 0x7ff6229ee980, 0x1)
            C:/Users/swarming/.swarming/w/ir/x/w/goroot/src/runtime/malloc.go:757 +0x348 fp=0x4e8f3ff560 sp=0x4e8f3ff4a0 pc=0x7ff621cdf6f8

I am not sure what could have changed with the VMs to start triggering these sorts of issues-- the swarming account logs seems to be clean for the most part, I don't see anything odd in the system event logs.
When I log into the builders and examine them, there are no tests running, but the system commit charge is at or near 100%. Pictures from process explorer:

Paging @golang/windows experts -- if anyone has debugged these sorts of issues before and might have ideas on how to proceed, let me know (I am certainly out of ideas).   My gut is that there is some sort of zombie process here, but given that I can't see any processes active from LUCI, I'm not sure how to debug this.

Sometimes the problems are in the LUCI infrastructure (ex: cas_download), e.g.
runtime: VirtualAlloc of 3710976 bytes failed with errno=1455
fatal error: out of memory
...
runtime.(*mheap).alloc(0x38a000?, 0x1c5?, 0x58?)
    runtime/mheap.go:958 +0x54 fp=0x40037cca10 sp=0x40037cc9c0 pc=0x7ff6df88cad4

and sometimes the out of memory errors happen during test build:
    go_test.go:2588: go [build x] failed unexpectedly in C:\Users\swarming\.swarming\w\ir\x\w\goroot\src\cmd\go: fork/exec C:\Users\swarming\.swarming\w\ir\x\t\cmd-go-test-3753799972\tmpdir1058398312\testbin\go.exe: The paging file is too small for this operation to complete.

errno of 1455 is also ERROR_COMMITMENT_LIMIT The paging file is too small for this operation to complete. error - so both your LUCI infra and tests encounter the same error.

I am not an expert in this area anymore. I even doubt page file still exist on modern Windows. But I agree with you that Current Commit Charge of 100% on System Information program looks bad for your workload. Maybe 16 G of memory needs to be increased. Maybe there are different ways to add virtual memory.
I googled for "page file windows task manager", and I find
https://learn.microsoft.com/en-us/troubleshoot/windows-client/performance/introduction-to-the-page-file
but I cannot find any good suggestions there.
Hopefully other Windows experts will help.
Alex

Hi there 👋🏼 I'm the Go group manager in Microsoft and I'd like to setup a call to discuss improving the stability of these builders. Who should I include? Thanks!

Hello @gdams on the goland core team side if you could please include myself (@thanm) and Dmitri (@dmitshur) that would be a good start. Thanks, Than

Recording here that there was another instance of the builders disconnecting and needing to be restarted around September 20-23:

Purple boxes are where it was missing until being restarted. The 3 failed builds before that all failed with "cannot allocate memory" errors:

https://ci.chromium.org/ui/p/golang/builders/ci/gotip-windows-arm64/b8736241631668442465/infra
https://ci.chromium.org/ui/p/golang/builders/ci/gotip-windows-arm64/b8736250382334003409/infra
https://ci.chromium.org/ui/p/golang/builders/ci/gotip-windows-arm64/b8736231583074874113/infra

Since the restart it's been working okay again. That suggests the work to have the builder fully start up after a restart is complete and working, and so it could work well to stop setting SWARMING_NEVER_REBOOT to allow LUCI restart the builders when it's deemed necessary.

Again, both windows-arm64 builders have gone offline as of around Nov 2-5, with first OOMs on Oct 29 (here):

@dr2chase and I will restart them for now.


Certificate Chain with Shield configuration

My root CA has an intermediate CA that signs certificates. I would like the option to configure the shield store. I can see two alternatives to perform this based on some light reading of the java_ks documentation:

Allow the ca certificate to be a bundle (and have the shield module process each certificate in the PEM, it currently only looks at the first one in my testing)
Allow an array to be passed in pointing to the PEM files for the certificates.
Modify the shield class to allow a already built JKS to be used.

Based upon my reading in the java_ks provider, it's already attempting to read a chain from the certificate PEM, so this may be an upstream issue if java_ks isn't reading chains correctly. Have you tried re-ording the chain of certs in the PEM file?
If the ordering doesn't fix anything, we could try exposing an additional parameter on instance.pp for chain, since that seems to be the way the module likes to handle chains, though the previously-mentioned provider is processing the chain in the same way, so that may give you similar results.
From an end-user perspective, if you'd prefer to just manage the keystore yourself, we can always modify instance.pp to only manage keystores if those parameters are passed and just let you manage those and the ssl config yourself.
Let me know what you think.

I tried different orders and they didn't seem to work. Looking at the debug statements it appears that only the first one was processed. This could be environment specific though.
I have configured things manually for now by creating a file resource for all instances and manually building the host specific keystore and copying them to the server manually. Then I apply the settings in a config statement. This will work for now, but if we wanted to set up instance specific keys this could get complicated.

As i understand you want to add "many" CA certs into keystore? I suggest it doint as i do. So in module use intermidiate Ca cert and you can add toplevel certificates after that using java_ks (copy them on server as files beforoe): eg.
java_ks { 'puppetca:keystore':
ensure       => 'latest',
certificate  => '/etc/pki/tls/certs/ca_certs/CA',
target       => '/etc/elasticsearch/master0/shield/keystore.jks',
password     => 'secret',
trustcacerts => true,
}
Hope it helps..


Inconsistent left most single ray

** Issue **
Of the left most ray (when x = 0) wall begins and ends at a different point. Which is inconsistent with the view and the rest of the map.
** Possible solution **
Not printing the left most ray

was already closed 4 days ago


Coding template for ISR on ESP32

It is not clear from the documentation how this should be done

Some background reading...

Arduino framework code for attaching ISRs to pins
Safely pre-allocating events from an ISR
Configuring and Handling ESP32 GPIO Interrupts In Arduino IDE
Working with Events -- Quantum Leaps doco
About the QP/C Port to FreeRTOS
BSP for DPP on ESP32

The following code works perfectly (but does nothing) for a button press
struct Button {
    const uint8_t PIN;
    unsigned long last_interrupt_ms;
};
Button button = {BUTTON_PIN, 0U};

void BUTTON_ISR(void) {
    BaseType_t xHigherPriorityTaskWoken = pdFALSE;
    unsigned long now_ms = millis();
    if  ((now_ms - button.last_interrupt_ms) >  BUTTON_BOUNCE_SUPPRESS_MS) {
        // not a bounce
        button.last_interrupt_ms = now_ms;
        //event publishing goes here
    } else {
            // key bounce, ignore
    }
    if(xHigherPriorityTaskWoken) {
        portYIELD_FROM_ISR();
    }
}

Don't put a Serial.println() call in the ISR, the chip will reset randomly

Very interesting comments here from Miro and Andy (the author of the QP ESP32 port)

Victor Chavez QP/C++ port to ESP32 on GitHub

Further discussion about the QP/ESP32/FreeRTOS port -- experimental only and with questionable parentage.


smart_listing_controls_for auto-fires once (and only once)

Let me start off by saying that I LOVE smart_listing and how easy it makes simple sorting and filtering through AJAX. Thanks for all the hard work and I hope I'll be able to contribute in some small way! The issue is as follows:
Sorting, filtering, etc. all are working perfectly. The issue I'm having is that when using smart_listing_controls_for with a select box (or a text box), when the input is changed, it will auto-fire and update the table with the correct filtered results. However, it only ever works the first time. On any page. Using a button to submit works as predicted, but I'd really like to ditch the button and just have it always fire on change. Checking the logs, the first change fires off the GET request as it should, but any subsequent changes do nothing. My first guess would be that it is losing it's binding somewhere along the line?
What information does anyone need from me to help fix this issue? I'm not using turbolinks in this project nor am I using bootstrap-daterangepicker-rails.
Thanks in advance, -D

Hey everyone!
Same here. Disabling Turbolinks didn't work. Going through "application.js" and deleting "requires" one by one didn't help either - same story.
Some of my code:
controller:
users_scope = User.where('id <> ?', current_user.id)
users_scope = users_scope.where('name LIKE :filter', filter: "%#{params[:filter]}%") if params[:filter]
@activity = Activity.find(params[:id])
@users = smart_listing_create :users, users_scope, partial: "activities/user_list", default_sort: { name: "asc" }
_user_list:
<%= smart_listing_controls_for(:users, { class: "form-inline" }) do %>
  <div class="filter input-append">
    <%= text_field_tag :filter, '', class: "filter-users form-control", placeholder: "Filtrer users", autocomplete: :off %>
  </div>
  <button type="submit" class="btn btn-primary disabled">Filtrer</button>
<% end %>
<% smart_listing.collection.each do |user| %>
  <%= render 'users/user', user: user %>
<% end %>

Sometimes it behaves weirdly, reloading the whole page when I type one letter. I'm sure it's something on my end, but can't figure out what exactly.
The above setup works when I type a query, e.g "Mike" and hit "Enter". But auto-refreshing doesn't work. Also, the above code generates a data-smart-listing="users", when the one on your example website is data-smart-listing="controls" (while we have mostly the exact same setup). If I write the form by hand, with data-smart-listing="controls", the auto-refresh happens every time I stop typing (as it should) - but I get "smart_listing.params is not a function" in the console, and no refresh whatsoever.
P.S: I'd love to be able to use that "like" scope you have used, but I defined mine because "like", of course, doesn't work.

+1 Looks like the search box loses its "on change" binding when it loads in the new content via ajax.
A workaround is to do the on change binding yourself to work after the content loads back in.
$(function () {
  $('body').on('change', '.smart-listing-controls input', function(e){
    e.preventDefault();
    $(this).closest('form').submit();
  });
});

Smart listing generates a remote form around your fields when you do smart_listing_controls_for which normally submits on change.
So the code just submits that form on change but the binding is attached to body so it doesn't break after the content reloads.
You can add any other events you would like - i.e. add events for click on the submit button in the control group form if you need.
I have a bit of a dig through the smart_listing JS to see where the change is triggered but couldn't find it...

I solved it thanks to @MindRave and @SixiS!! You need to take the filter (i.e. <%= smart_listing_controls_for... do %>) out of the partial you are rendering (_user_list.html.erb for @MindRave), and just place it in the file--index.html.erb I'm assuming--that contains the <%= smart_listing_render :users %> code. If the code is inside the partial, it will be re-rendered every time you search and will NOT re-bind the auto-fire AJAX submit on the filter. Once you take it out of said partial, voila, auto-refresh works every time. Hope this helps and thanks to both of you for helping me figure out what exactly was causing it and fixing this problem site-wide.
-Dave

@davidrouten, @SixiS confirming that the solution proposed by @davidrouten works!
Thank you so much for your help!


Winget reports packages in the "msstore" with available version 'Unknown'

Brief description of your issue

Some packages in the msstore repo are tagged as version 'Latest', and winget regards them as upgradable all the time. For me, it happens to package Microsoft Whiteboard (Microsoft.Whiteboard) and Python 3.8 (PythonSoftwareFoundation.Python.3.8).
> winget upgrade | findstr Latest
Microsoft Whiteboard                         Microsoft.Whiteboard                21.10823.5772.0 Latest         msstore
Python 3.8                                   PythonSoftwareFoundation.Python.3.8 3.8.2800.0      Latest         msstore

However, the latest version is actually already installed. So if I try to upgrade it, winget will check for upgrade and then do nothing, although still reports there's an upgrade available after that.
Steps to reproduce

Take Microsoft Whiteboard as an example. Run the command:
winget upgrade
or
winget upgrade --all
or
winget upgrade Microsoft.Whiteboard
Expected behavior
Microsoft Whiteboard will not show in the list since I have the latest version already installed.

Actual behavior
For winget upgrade:
> winget upgrade
Name                                         Id                                  Version         Available      Source
-----------------------------------------------------------------------------------------------------------------------
Microsoft Whiteboard                         Microsoft.Whiteboard                21.10823.5772.0 Latest         msstore

For winget upgrade -all or winget upgrade Microsoft.Whiteboard:
> winget upgrade microsoft.whiteboard
已找到 Microsoft Whiteboard [Microsoft.Whiteboard]
验证/请求程序包获取成功...
验证/请求程序包获取成功
正在启动程序包安装...
找不到适用的更新。

Sorry for the inconvenience since my system language is Chinese. The resutls simply means 'I checked the repo successfully and found no upgrade available'.
BTW, these packages were first installed via MS store rather than winget command. But I think there's no difference, isn't it?

Environment
[winget --info]
Windows Package Manager version
Windows: Windows.Desktop v10.0.19043.1165
Package: Microsoft.DesktopAppInstaller v1.12.11692.0

@littlef0x the store source doesn't currently provide the version for the package. We don't currently have a way to know if the one installed on your system is up to date or not without that information. We are currently working with the store team on a new "source". We will be changing "latest" to "unknown" until such time as the store source can provide the version so the package manager will know if it should update. If you have the Microsoft Store app, it will provide updates as they are released automatically.

Thanks for your reply. I examed all items listed by winget list and indeed, these two packages are the only two tagged as source 'msstore'. It means that most packages installed via Microsoft Store are not identified, although for many of them, Microsoft Store is the only way to install. Furthermore, I found more confusion related to msstore source.

Some packages in Microsoft Store can be listed in winget list, but cannot be found by winget search, e.g., VLC.
Some packages installed via Microsoft Store seems to be upgraded using winget source rather than msstore source, e.g., Notepads. There is a msstore version of the same software with a different ID. I don't know whether they are exactly the same and why they use different IDs (JackieLiu.NotepadsApp vs. 19282JackieLiu.Notepads-Beta), while some other packages use the same ID in different sources, e.g., Windows Terminal. This may cause severe confusion.

With the latest release, the "msstore" source displays "Unknown" for the version. We have also removed the correlation from the store source for list and export due to current matching logic.
I'm modifying the title on this Issue to match the current experience.

We've released Windows Package Manager 1.3.
The Microsoft Store still reports "Unknown" in the version. Other Issues are tracking this current state of the system, and work is in progress to begin reporting versions from the Microsoft Store.
As the winget upgrade --all experience has been improved to exclude version "Unknown" I'm going to close this bug in favor of others that are tracking progress.


Does Netty use/plan to use the LMAX Disruptor?

We use Netty in our project for our REST API handlers. From my understanding, Netty does use separate request handler threads and worker threads, i.e. a producer-consumer model.
The LMAX Disruptor library is known to be one of the fastest ways of implementing the queues that are required for communication between producers and consumers. Does Netty already use or plan to use this library internally?
I did see a similar question/suggestion, but the thread was inconclusive. It did seem like this hasn't been done already but might be considered in the future. Could I get some clarity about where things stand as of now?
Thanks,
Anurag
PS: Sorry about opening an 'issue', I wasn't sure where else to ask this.

@anuragw there is currently no plan to do so... That said if you would like to do a proof-of-concept of this and show how this improve things via a PR we are more then happy to review.

@anuragw could you please point me to the class/package that deals with off-loading tasks to various threads? Would MultithreadEventExecutorGroup be a right place to start?


Where does the "binds SPAC1F7.05.2" come from for spd1 / SPAC29B12.03?

https://www.pombase.org/gene/SPAC29B12.03
It's just "with cdc22" in the session:
https://curation.pombase.org/pombe/curs/dfd6ea81df55ce19/ro/

The "binds SPAC1F7.05.2" was generated by the code that automatically makes reciprocal IPI annotations.  It was getting things wrong because SPAC1F7.05/cdc22 has two transcripts.  I've fixed this for Wednesday night's load and I'll check on Thursday morning.
After this fix there will still be some duplicate annotations, but there's a separate issue for that: pombase/website#1789

The chado_checks log now reports about a zillion (well, 4454) duplicate GO annotations. Is that connected with this? It looks suspicious because the vast majority are 'protein binding' annotations. Or should I put it in a separate ticket?

I see what you mean.

Is that connected with this? It looks suspicious because the vast majority are 'protein binding' annotations.

Yep, I think it must be caused by my "fix".  I'll fix it better this time.

I've committed a fix for my fix.  I'll check in the morning.
As far as I can tell the annotation on the gene pages turned out OK despite all the warnings.

I've committed a fix for my fix. I'll check in the morning.

The log file is looking better so I'll be an optimist and close this issue.


Implements feature #8: add chart packaging feature

This pull request first and foremost helps me get familiar with Github's workflow (I'm used to Gitlab, which is a bit more seamless and issue-driven).  I will probably make a hash out of things as a learn, so you have been warned.
This pull request adds the notion of an AbstractChartWriter and some subclasses to enable the serialization of hapi.chart.ChartOuterClass.ChartOrBuilder instances.

In the learning process, I'm going to select the Rebase and merge option.  Everything is green so far!


Maximum speed settings do not allow limiting robot to less than full speed

The robot should be limited to the minimum of its achievable velocity and a user specified maximum which allows for slower, more refined movement that can be programmatically set during operation. This allows for demo-modes or allowing the driver more control during parts of the match.
Changes:
Instead of getting the Math.max, get the Math.min of attainable and user specified speeds. This ensures that users can not command the robot faster than achievable while allowing for slower movement in certain cases.
Renamed getMaximumChassisVelocity() to getMinimumChassisVelocity() and changed math to minimum to be consistent.
Renamed getMaximumChassisAngularVelocity() to getMinimumChassisAngularVelocity() and changed math to minimum to be consistent.
Added setMaximumAllowableSpeeds() which sets the user configured driving speeds.
Renamed setMaximumSpeeds() to setMaximumAttainableSpeeds() which sets the maximum speeds achievable by the robot.
For this to work, an attainable speed pair must be set so that they are not 0 and the robot rendered immobile by default. So, a reasonable max of 6mps and 2PI rps were set. I'm open to other suggestions for this. I considered using the max module speed functions, but they calculated much higher than expected and I'd rather have some kind of hard-coded true failsafe for inexpert users of the library to fall back on. Users should use the setMax* functions after configuring their swerve.
Note: I could see changing the functions back to being named Max instead of Min, but I felt that the naming should reflect the function to avoid confusion over what it actually does.

The naming should change back because the values are maximum ideal values not minimum speeds the robot will always go. The attainable speeds should be identical to the other one on startup at least. The javadocs should provide enough info to reduce confusion

Reverted naming and updated docs.

You said make the attainable the same at startup. So, I can make that maxChassisSpeedMPS for the translational speed, but the angular speed doesn't have a default.

While looking at it, i think its fine the way it is

How about this, setting the default controller config angular speed to 2PI and then setting the attainable in the constructor using that.
/**

Maximum chassis angular velocity in rad/s
*/
public       double     maxAngularVelocity = 2 * Math.PI;

...
// Defaulting to the allowable maximums.
setMaximumAttainableSpeeds(maxChassisSpeedMPS, this.swerveController.config.maxAngularVelocity);

Is it really necessary?

I think it's just my coding OCD wanting to do it that way, idk.


no icons in assets/css3buttons with rails 3.1.1 and 1.0.1 Gem Version

Started GET "/assets/css3buttons/css3buttons/icons.png" for <IP_ADDRESS> at 2011-10-12 16:17:38 +0800
Served asset /css3buttons/css3buttons/icons.png - 404 Not Found (50ms)
ActionController::RoutingError (No route matches [GET] "/assets/css3buttons/css3buttons/icons.png"):

I confirm it. I have the same problem.

+1

Same problem, please merge  :+1:

Same problem.

+1

@johnres: I tried your branch, but icons are still not showing, so at least it's not working for me (I am just starting out with the asset pipeline, so its very possible I am doing something wrong).
With the 1.0.1 release gem the error I see in the log is:
Started GET "/css3buttons/icons.png" for <IP_ADDRESS> at 2011-10-27 09:17:33 -0700
ActionController::RoutingError (No route matches [GET] "/css3buttons/icons.png"):

Hmm, it's working here, is this how your Gemfile looks?
gem 'css3buttons', :git => 'git://github.com/johnrees/css3buttons_rails_helpers.git'

This asset pipeline is giving me a headache!
I tried:
gem 'css3buttons', :git => 'git://github.com/johnrees/css3buttons_rails_helpers.git',
and
gem 'css3buttons', :git => 'git://github.com/johnrees/css3buttons_rails_helpers.git', :branch => 'patch-1'
And still get no icons.  I can see in the log that the asset is being compiled again:
Started GET "/assets/admin/admin.css" for <IP_ADDRESS> at 2011-10-27 09:30:04 -0700
Compiled admin/admin.css  (9ms)  (pid 79880)
Served asset /admin/admin.css - 200 OK (30ms)

However, the generated css (which includes the css3buttons css) does not correctly reference the image:
.button.icon:before {
    content: "";
    position: relative;
    top: 1px;
    float:left;
    width: 12px;
    height: 12px;
    margin: 0 0.75em 0 -0.25em;
    background: url(/css3buttons/icons.png) 0 99px no-repeat;
}

I appreciate your help in getting this working.
Thanks,
--
Ylan Segal
<EMAIL_ADDRESS>Tel: +1-858-224-7421
Fax: +1-858-876-1799
On Oct 27, 2011, at 9:22 AM, John Rees wrote:

Hmm, it's working here, is this how your Gemfile looks?
gem 'css3buttons', :git => 'git://github.com/johnrees/css3buttons_rails_helpers.git'
--
Reply to this email directly or view it on GitHub:
https://github.com/thetron/css3buttons_rails_helpers/issues/23#issuecomment-2545694

So…
No dice. I had already removed the pre-compiled assets and it's still not working, even though I see form the rails output that the style is being compiled:
Started GET "/assets/css3buttons/styles.css?body=1" for <IP_ADDRESS> at 2011-10-27 10:04:03 -0700
Compiled css3buttons/styles.css  (7ms)  (pid 81535)
Served asset /css3buttons/styles.css - 200 OK (18ms)

The suggested hack does works for now, but I can't figure out what could be causing this problem in my environment.
--
Ylan.
On Oct 27, 2011, at 9:49 AM, John Rees wrote:

Are any of your assets precompiled? Try running
rm -rf public/assets/*.css
Then look at
/assets/css3buttons/styles.css?body=1
and see if it has changed
background: url(/css3buttons/icons.png) 0 99px no-repeat;
to
background: url(/assets/css3buttons/icons.png) 0 99px no-repeat;

A nasty interim solution/hack would be to write
.button.icon:before { background-image:  url(/assets/css3buttons/icons.png) !important; }
in your application.css

You'll grow to love the asset pipeline eventually!
--
Reply to this email directly or view it on GitHub:
https://github.com/thetron/css3buttons_rails_helpers/issues/23#issuecomment-2546088

John, I still haven't found a solution to my particular problem, but I found a workaround that I like better:
I forked and instead of using asset_path, I am using asset_data_uri to embed the image in the stylesheet itself (which works fine for my purposes since all user's have modern browsers).

using
gem 'css3buttons', :git => 'git://github.com/johnrees/css3buttons_rails_helpers.git'
and afterwards clearing out tmp/cache/ and tmp/sass-cache/ solved the problem for me. Icons are showing again

Hi Nic,
I get installed Rails 3.1.0 but your gem seems not work.
When I hit the "rails g css3buttons" this is the output: "Could not find generator css3buttons."
Thanks for your work!

With Rails 3.1 you don't need run a generator because of the asset pipeline.
Please refer to 'Getting started - Rails 3.1' in the readme.md

Hi John,
you want say that I have to include gem "css3buttons" in my gemfile,
put these strings 	*= require css3buttons in application.css
*= require_self
and start to styling my links?
Thanks for your help

Ok,
for the reset.css ,
but for the "css3-gitub-buttons.css" there is a routing error: "No route matches [GET] "/assets/css3buttons/css3-github-buttons.css"


[Release-1.20] - Update bootstrap logic to print out all changed files on disk

Backport fix for Update bootstrap logic to print out all changed files on disk

#4797

N/A


shut up nobody cares

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

Seems good but why is this necessary?

This logging is kinda nice sometimes tho :(

This logging is kinda nice sometimes tho :(
Except f+ is dead, the entire check should be removed, if you do want some logging make it something like Successfully registered ASM transformers

Resolved in https://github.com/ChatTriggers/ChatTriggers/commit/4171f8f5fac81ff687ab328ee062ffc705fa43b0


How to specify optional dependencies

How do you specify optional dependencies? I've seen other issues where it was explicitly stated that pipenv is largely designed to mimic composer. Composer.json files have a section named suggests that allows you to list optional dependencies, but I haven't been able to find an equivalent for pipenv. Would someone be able to point me in the right direction?

@djmattyg007 I am not familiar with composer, could you describe more what an optional dependency would be like in the context of pipenv?   Right now there are the default and dev package groups, is it different from a dev dependency?

Category based package installs may be what you are looking for:  https://github.com/pypa/pipenv/issues/4745


Fix incorrectly converting relative to absolute of files and always show paths in traceback in case file cannot be found

Type of changes

[x] Bug fix
[x] New feature
[ ] Documentation / docstrings
[ ] Tests
[ ] Other

Checklist

[x] I've run the latest black with default args on new code.
[x] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[x] I've added tests for new code.
[x] I accept that @willmcgugan may be pedantic in the code review.

Description
Currently Rich assumes that if a path in a traceback frame is relative, that it's relative to the current working directory. This is not the case when the Python application is running from a packaged bundle like a PAR. In these cases the traceback loses the path information or the incorrect absolute path is used. This PR makes it so:

Frame.filename is not absolute if the original filename does not exist in the first place. That would indicate that it's not relative to the current working directory.
The path/filename is always shown in the traceback, not only if it exists.

I was considering replacing rich.__init__._IMPORT_CWD's value to one that is reflective of __file__ though that resulted in filenames relative to /proc. Those are not readable and this might change other expectations.
The linecache module understands the relative paths and even sometimes can give the lines inside of the PAR. (Though sometimes it cannot and I'm not sure when it can and cannot.) Regardless, this means we at least see some lines and also always see the location.

Any chance of tests? Although I can see why tests are tricky with this kind of thing.

@willmcgugan  Will have a look when I'm back in the office. Currently on long(er) holiday. Could you have a look at #1859?

I've added some tests to test the behaviour here. It's a bit annoying to test because of how types.FrameType and similar classes are not instantiatable. This might be a bit fragile and maybe mocks should be used instead of "fake" dataclasses. Though I feel like that's equally ugly and annoying to work with.


Factor JTAG chain code out into a module to use it for all JTAG based probes instead of just CMSIS-DAP

See https://github.com/probe-rs/probe-rs/pull/1462#issuecomment-1642901037

What's the definition of done on this issue? I think this was done some time ago (#1878?) and we're going further with unification now. Can I mark #2066 to close this?


AccessDenied executing Get-MgBetaIdentityConditionalAccessPolicy

🐛 Summary
Receiving an AccessDenied error when running Invoke-SCuBA -ProductNames aad
Even when configured as a Global Administrator AND Conditional Access Administrator enabled it is still returning AccessDenied when executing Get-MgBetaIdentityConditionalAccessPolicy
To reproduce

Invoke-SCuBA -ProductNames aad

Expected behavior
Produce the report for scanning Entra
Any helpful log output or screenshots
WARNING: Error running Get-MgBetaIdentityConditionalAccessPolicy. Your account does not have access to this report or data. Please contact your global administrator to request access. One of the following
roles is required: Security Reader, Company Administrator, Security Administrator, Conditional Access Administrator, Global Reader, Devices Admin, Entra Network Access Administrator.
Status: 403 (Forbidden)
ErrorCode: AccessDenied
Date: 2024-11-20T19:06:37
Headers:
Transfer-Encoding             : chunked
Vary                          : Accept-Encoding
Strict-Transport-Security     : max-age=31536000
request-id                    :  xxxxxxx
client-request-id             :  xxxxxxxx
x-ms-ags-diagnostic           : {"ServerInfo":{"DataCenter":"xxxxxx","Slice":"x","Ring":"x","ScaleUnit":"xxx","RoleInstance":"xxxxx"}}
Link                          :
https://developer.microsoft-tst.com/en-us/graph/changes?$filterby=beta,PrivatePreview:networkAccess&from=2022-02-01&to=2022-03-01;rel="deprecation";type="text/html",<https://developer.microsoft-tst.com/en-u
s/graph/changes?$filterby=beta,PrivatePreview:networkAccess&from=2022-02-01&to=2022-03-01>;rel="deprecation";type="text/html",<https://developer.microsoft-tst.com/en-us/graph/changes?$filterby=beta,PrivatePre
view:networkAccess&from=2022-02-01&to=2022-03-01>;rel="deprecation";type="text/html",<https://developer.microsoft-tst.com/en-us/graph/changes?$filterby=beta,PrivatePreview:networkAccess&from=2022-02-01&to=202
2-03-01>;rel="deprecation";type="text/html",https://developer.microsoft-tst.com/en-us/graph/changes?$filterby=beta,PrivatePreview:networkAccess&from=2022-02-01&to=2022-03-01;rel="deprecation";type="text/htm
l",https://developer.microsoft-tst.com/en-us/graph/changes?$filterby=beta,PrivatePreview:networkAccess&from=2022-02-01&to=2022-03-01;rel="deprecation";type="text/html",<https://developer.microsoft-tst.com/e
n-us/graph/changes?$filterby=beta,PrivatePreview:networkAccess&from=2022-02-01&to=2022-03-01>;rel="deprecation";type="text/html",<https://developer.microsoft-tst.com/en-us/graph/changes?$filterby=beta,Private
Preview:networkAccess&from=2022-02-01&to=2022-03-01>;rel="deprecation";type="text/html",<https://developer.microsoft-tst.com/en-us/graph/changes?$filterby=beta,PrivatePreview:networkAccess&from=2022-02-01&to=
2022-03-01>;rel="deprecation";type="text/html",https://developer.microsoft-tst.com/en-us/graph/changes?$filterby=beta,PrivatePreview:networkAccess&from=2022-02-01&to=2022-03-01;rel="deprecation";type="text/
html",https://developer.microsoft-tst.com/en-us/graph/changes?$filterby=beta,PrivatePreview:secureAppSessionMode&from=2023-07-01&to=2023-08-01;rel="deprecation";type="text/html",<https://developer.microsoft
-tst.com/en-us/graph/changes?$filterby=beta,PrivatePreview:secureAppSessionMode&from=2023-07-01&to=2023-08-01>;rel="deprecation";type="text/html"
Deprecation                   : Thu, 17 Feb 2022 23:59:59 GMT
Sunset                        : Sat, 17 Feb 2024 23:59:59 GMT
Cache-Control                 : no-cache
Date                          : Wed, 20 Nov 2024 19:06:37 GMT

Do you have a cap policy that might be blocking this access?  It's hard to know the cause without a little more information on your setup.

I will close out this issue.   It was a timing issue with PIM.  I had activated the role, but hadn't waited long enough it appears and the token that was assigned didn't provide access to the role.   Oddly enough, everything else worked repeatedly.
In any case, I re-executed today and everything was fine, so I'm going to close out this issue as a user error.


[docs] Standardize example names

This PR standardizes the formatting of example project names in the /examples/ folder. This is mainly to apply standard lib prefixes material-, base-, and joy- for better organization, in anticipation of adding more examples for Base and Joy in the coming months. Framework names drop js, css, etc. to avoid clutter. -ts is appended where applicable.
I also updated the names of projects in READMEs and indexes etc. to specify what tools are involved. Many older examples still refer to Material UI as MUI, so I changed that wherever I saw it. Otherwise I tried not to make too many content changes. Hopefully I caught all links to all examples throughout the docs!

I have cherry-picked this change to production https://github.com/mui/material-ui-docs/commits/latest as

it breaks the links in the docs, e.g. https://mui.com/material-ui/getting-started/example-projects/ https://mui-org.slack.com/archives/C041SDSF32L/p1676634426113669


A better approach to test transpilation?

In the tests for the transpiler, I've noticed that most tests just have a ! cpp and optionally a ! h annotation at the top of the file and then just plain Val code. Here's an example:
/*! h
    Int returns_value();
 */
/*! cpp
    Int returns_value() {
    return 13;
    }
 */
public fun returns_value() -> Int {
  return 13
}

My goal with the test annotations was to get them as close as possible to the Val code under test, just like with FileCheck. However, this goal appears to have been missed if I look at the standard practice in our tests. A striking example is DoWhileStmt.val.
I think one issue is that the framework pushes you to write entire functions inside a single annotation. This is obviously not the granularity level that we'd like.
I have an idea to solve this problem. Rather than having annotations represent a single entity to check, we could concatenate their contents over multiple lines. The result would represent the string to be matched in the transpiled file.
//! h Int returns_value();
//! cpp Int returns_value() {
public fun returns_value() -> Int {
  //! cpp return 13;
  return 13
}
//! cpp }

One problem with this approach, however, is that there's no hard guarantee on the order in which declarations appear in the transpiled file. We could tackle this problem with another annotation.
//! h Int returns_value();
//! cpp-start returns_value
//! cpp Int returns_value() {
public fun returns_value() -> Int {
  //! cpp return 13;
  return 13
}
//! cpp }
//! cpp-end returns_value

I think that would work ok.
My concern is that the test becomes harder to read. I find the first code snippet more readable than the last one.
However the code snippet in the middle looks better, and I can try to write tests in this style.

We don't have to go through with this change. It's just an idea.
The main problem is the order in which declarations will show up. I can't think of a nice solution to deal with it.

It's not even clear to me whether it's a good idea to put the inputs and expected output in the same file.  Maybe we should make special line-identifying comment markers where checking a correspondence matters.  But it's probably premature to decide any of this before we have a much larger body of tests, so I think we should let this one percolate.

No longer relevant.


/admin, email drop report fails

displays "The form requested is not supported"

Hold off reports for now.


Web2py framework resources

I think this is related to Python so it should go under Python Resources, not required to create a separate section for it. What do you think @ArushiSinghal ?

@vicky002 web2py is a web-framework just like Django or Ruby on Rails. and we have the separate section for the both so I think for web2py there should be a different section.Also for web2py, there are many learning resources which I will keep on adding as I found.
Also mixing everything in python section will not do justice to both web2py and python.
@vicky002 please let me know your viewpoint.

@ArushiSinghal hmmm I agree and we have Django Resource as a separate section so this make sense.  Merging your PR and looking forward to more contributions from you. 👍

@ArushiSinghal , please add Web2py in the main README.md file so that people can easily find it. 🙂


[MM-49590] Moving header buttons for insights

Summary
Moved the header buttons to align with the content. According to our telemetry people were having trouble finding them
Ticket Link
https://mattermost.atlassian.net/browse/MM-49590
Screenshots
Before:

After:

Release Note

NONE

In cases where there are invisible buttons, it looks like the text is aligned with something below:

While the right side is perfectly aligned, the left one seems a bit off to me with this new arrangement:

With a change like this, it looks more aligned.
.header-container .left {
  margin-left: -8px;
}

What do you think?

In cases where there are invisible buttons, it looks like the text is aligned with something below:
While the right side is perfectly aligned, the left one seems a bit off to me with this new arrangement:
With a change like this, it looks more aligned.
.header-container .left {
  margin-left: -8px;
}

What do you think?

Yeah that's a good point, I wasn't 100% sure which approach to take but I think I agree with you

In cases where there are invisible buttons, it looks like the text is aligned with something below:
While the right side is perfectly aligned, the left one seems a bit off to me with this new arrangement:
With a change like this, it looks more aligned.
.header-container .left {
  margin-left: -8px;
}

What do you think?

Yeah that's a good point, I wasn't 100% sure which approach to take but I think I agree with you

I'm not actually sure how I feel about this dropdown not lining up. @annapohorielova111 Which approach do you think is best for the dropdown alignment?


Fix #262

Fix #262
We might need some test cases to make sure everything works as expected

@as3boyan, @romanmikhailov,  Hi guys,  You both have competing change/pull requests here (#264 #265).  Can you both review each other's changes and come to agreement about which is better?  Either change seems reasonable to me, but we probably don't want both.

@as3boyan Viewed your fix. It's seems to retrieve only static functions. Static fields still not displayed.

@as3boyan But my fix not cover Enum autcompletion and not contains test cases fr this issues.

@as3boyan @EBatTiVo My suggestions get both merged pull requests.

Static fields still not displayed.

@romanmikhailov Could you please describe steps how to reproduce issue you mentioned, because to me, it seems like my implementation should fix #262, and it seems to me like everything works fine.

Could you please describe steps how to reproduce issue you mentioned, because to me, it seems like my implementation should fix #262, and it seems to me like everything works fine.

Your fix works fine for case #262 (+ Enum autocompletion) but fails on next example:
class StaticMethods {
    public static function foo() {}
    public static var field:Float;
}

class Main {
    public static function main() {
        //StaticMethods.foo(); // after typing is resolved, this is good
        StaticMethods. // produce only foo. field is absent.
    }
}

Thanks for example, I was able to reproduce it
class StaticMethods {
    public var field:Float;
}

Works fine.
class StaticMethods {
    public static var field:Float;
}

Doesn't work. Item is not shown in completion, because of
namedComponent.isStatic() doesn't works for HaxeVarDeclarationPart

@as3boyan I'm close my pull requests. Please add tests for fields and enum.

@as3boyan Maybe will be better to make your check from AbstractHaxeNamedComponent.isStatic in overriden HaxeVarDeclarationPartImpl.isStatic?

There is already test for enum, I will add test for fields.

@as3boyan Maybe will be better to make your check from AbstractHaxeNamedComponent.isStatic in overriden HaxeVarDeclarationPartImpl.isStatic?

Maybe, but it seems like, HaxeVarDeclarationPartImpl is in gen folder, and it seems like it is generated from parser.

@romanmikhailov Thanks a lot for your help

@as3boyan You are welcome.

@as3boyan @romanmikhailov
Approved for merge, once the field test is added.

Test case for static field completion is added 3088efa


Augment-Api changes for Asset VC

Signed-off-by: Shreevatsa N<EMAIL_ADDRESS>

Updated the same in #194


Importing: Cut-scenes head models are stretched

Game
Resident Evil 5
Example game file names
evfpl00.arc
The heads for cut-scenes look all stretched when being imported:

Fixed in Albam Reloaded and backported to Albam.


Remove obsolete Jemalloc

Rust 1.31 is long dead.

Why delete it entirely, rather than move it to the end, where it will be out of the way of people going down the list for practical use but will still be visible to people who might be interested in what was relevant historically?

I too have some reservations about deleting it entirely.
I could accept the PR if it was changed to moving it to a new section at the very bottom titled
# Legacy Techniques

## Remove Jemalloc
...

I've thought about it, but couldn't find any reason to keep it, other than loss aversion.
This default is not in Rust any more. Literally nobody uses Rust so old to benefit from this information. It's just an irrelevant distraction, and it became wrong information diluting usefulness of the guide for everyone. It has negative value.

Literally nobody uses Rust so old to benefit from this information.

Not many people now use BSD versions from 30 years ago either, but FreeBSD's source code contains various historical files in case they might be interesting to someone.

It's just an irrelevant distraction, ... It has negative value.

It may be irrelevant and valueless to you, or to people practically shrinking modern Rust executables, but do you have arguments (rather than mere assertions) as to why keeping some record of this historical technique would be so bad? You say

it became wrong information diluting usefulness of the guide for everyone

but, if having it at the end is still too much a "distraction", it could be moved to a separate file, linked from the end of the main file.

@kornelski Thank you for bringing this up and providing stats. This has been resolved in
@8573 Thank you for providing your perspective.

#72


Error interface unification

Use ts-result for failed and successful results representation.

neverthrow is used instead of ts-result.


[Main screen] Total balance sometimes more than expected

Describe the bug
I cannot reproduce right now, but see it second time. Looks like one of coins counting twice sometimes

for me it’s still consistently reproducible with https://github.com/vultisig/vultisig-ios/issues/401#issue-2279877626

Also was able to reproduce and caused by Rune and Maya chains being activated.
When one or both chains are active it is possible to have RUNE or CACAO balance counted double:

mainly caused by adding/removing tokens of chains
Having activated tokens on MAYA and ETH
removing or adding tokens on ETH chain (higher probaility when adding multiple tokens at once)
sometimes possible to recreate when adding/removing tokens on other chains.

Also when toggeling chains or tokens, total balance is not updated automatically and need to enter detailview to update total balance.

[ ] Refresh total balance on swipe up.

We fixed that.


blockcache: test returned errors

Pull Request Test Coverage Report for Build 617953155

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.9%) to 95.832%

Totals

Change from base Build 617768844:
0.9%

Covered Lines:
2437

Relevant Lines:
2543

💛  - Coveralls


[1pt] Switch badges for Rockets - Conditional components

Rockets that have already been reserved should show a "Reserved" badge and "Cancel reservation" button instead of the default "Reserve rocket" (as per design)
Use the React conditional rendering syntax:
{rocket.reserved && (
    // render Cancel Rocket button
)}

Hi @Kidd254 I kindly ask you to review my last PR: https://github.com/NitBravoA92/space-travelers-hub/pull/40
Thanks.


Document all the supported directives

The usage mentions only <ng-fb-(like|share|comment)-count>. What other social networks are supported?

:+1: All supported networks should be listed. My understanding of this codebase so far is that it's purely for facebook. If this is the only network supported the repo should be renamed to 'angular-facebook-count' or something like that.

added documentation & support for Facebook, Twitter, Pinterest, Linkedin.
Cheers !


Sending client id and client secret as basic authentication when using client credentials flow

According to OAuth2 standards documentation (see https://datatracker.ietf.org/doc/html/rfc6749#section-2.3.1), sending client id and client secret as basic authentication is supported in the client credentials flow (and actually recommended over sending the credentials in the request body). This method is not documented in the Azure documentation, however when testing this with Postman against Azure AD this works just fine. Could you please update the documentation to clearly indicate that this is supported?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d32f55ac-acbc-292b-d1b7-ffdd257b11e1
Version Independent ID: ae2a266c-051b-7307-93fd-74bc521c93bf
Content: OAuth 2.0 client credentials flow on the Microsoft identity platform
Content Source: articles/active-directory/develop/v2-oauth2-client-creds-grant-flow.md
Service: active-directory
Sub-service: develop
GitHub Login: @hpsin
Microsoft Alias: hirsin

@blahbap
Thanks for your feedback! We will investigate and update as appropriate.

@hpsin Reassigning this issue to you so you can evaluate and provide input. Let me know how you'd like to proceed if/when this needs doc work. Thanks!
#reassign:hpsin

We do support it per standards requirement, but don't use that in our standard flows, nor do we especially recommend it. We can add a note to the doc saying that we support basic auth for client credentials pursuant to OAuth 2.3.1


Write a step-by-step aircraft design example

(@whoburg): Educational example that steps through aircraft design models, building up from very simple to increasingly complex.

Hmm, @bqpd does the SimPleAC qualify? I'm hoping to convert it to sizing/performance model based model incrementally as part of my thesis anyways.

It totes will. I'll close this; it's covered in the new documentation plan.


Add Japan country map

Add Japan country map
Fix obsolete paths in the document about adding new country map

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6368   +/-   ##
=======================================
  Coverage   77.33%   77.33%
=======================================
  Files          67       67
  Lines        9578     9578
=======================================
  Hits         7407     7407
  Misses       2171     2171

Continue to review full report at Codecov.

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


D2.4 - Graphs

Long-Term Plan:

Complete this week's work
Week of 3/10:

Sibbons: Issue #837

Week of 3/17:

Sibbons: Issue #818


Benchmarks

For a comparison table:

[ ] Boost.Lexical Cast
[ ] Boost.Spirit Qi
[ ] STL
[ ] Additionally MSVC STL from_chars to show if it's worth the MSVC team pursuing Lemire's implementation.  (https://github.com/microsoft/STL/issues/1610)
[ ] printf / strtod

With the fixes from #47 and #49 we can now successfully run all the benchmarks.
 274.4 ns | Boost.lexical_cast  float
 439.7 ns | Boost.lexical_cast double

 300.1 ns | Boost.spirit.karma  float
 307.4 ns | Boost.spirit.karma double

 175.1 ns | std::snprintf  float general  9
 342.3 ns | std::snprintf double general 17

  33.6 ns | std::to_chars  float general shortest
  38.3 ns | std::to_chars double general shortest

  33.6 ns | Boost.Charconv::to_chars  float general shortest
  38.3 ns | Boost.Charconv::to_chars double general shortest
----------
  79.3 ns | std::strtof  float scientific
 134.7 ns | std::strtod double scientific

  24.0 ns | std::from_chars  float scientific
  23.5 ns | std::from_chars double scientific

 105.3 ns | Boost.Charconv from_chars  float scientific
2064.1 ns | Boost.Charconv from_chars double scientific

Ouch. Why is our from_chars so much slower?

Ouch. Why is our from_chars so much slower?

That is what I am trying to figure out. Even if we fell back to std::strtod every single time the double performance shouldn't be nearly that bad.

Here is the most recent which hits all of the requests from the ML:
GCC 13 x64 + libstdc++
 202.8 ns | Boost.lexical_cast float
 365.1 ns | Boost.lexical_cast double
  61.8 ns | Boost.spirit.karma float
  64.4 ns | Boost.spirit.karma double
 159.6 ns | std::sprintf float scientific 8
 319.8 ns | std::sprintf double scientific 16
  32.6 ns | std::to_chars float plain shortest
  34.7 ns | std::to_chars double plain shortest
  30.5 ns | Boost.Charconv::to_chars float plain shortest
  34.4 ns | Boost.Charconv::to_chars double plain shortest
----------
  66.9 ns | std::strtof float scientific
 124.0 ns | std::strtod double scientific
  23.7 ns | std::from_chars float scientific
  22.0 ns | std::from_chars double scientific
  20.9 ns | Boost.Charconv::from_chars float scientific
  21.1 ns | Boost.Charconv::from_chars double scientific
  26.5 ns | Boost.Spirit.Qi float scientific
  37.2 ns | Boost.Spirit.Qi double scientific
  12.9 ns | std::from_chars uint64_t
  14.0 ns | Boost.Charconv::from_chars uint64_t


Error: ToSql not implemented

Hey, I just tried out this crate.
Initial setup works fine, but I'm getting this error when inserting:
diesel::insert_into(db_schema::contacts::table).values(&contact).execute(&*db);

error[E0277]: the trait bound `db::types::Gender: diesel::serialize::ToSql<diesel::sql_types::Nullable<db::types::pg_enum_impl_Gender::GenderMapping>, diesel::pg::Pg>` is not satisfied
  --> src/app/users/mod.rs:55:78
   |
55 |             diesel::insert_into(db_schema::contacts::table).values(&contact).execute(&*db);
   |                                                                              ^^^^^^^ the trait `diesel::serialize::ToSql<diesel::sql_types::Nullable<db::types::pg_enum_impl_Gender::GenderMapping>, diesel::pg::Pg>` is not implemented for `db::types::Gender`
   |

Relevant types:

#[derive(Serialize, Deserialize, Identifiable, Queryable, Insertable, Clone, Debug)]
#[table_name = "contacts"]
pub struct Contact{
    pub id: Uuid,
    pub gender: Option<Gender>,
    pub given_name: Option<String>,
    pub family_name: Option<String>,
    pub company_name: Option<String>,
    pub title: Option<String>,
    pub phone_landline: Option<String>,
    pub phone_mobile: Option<String>,
    pub email: Option<String>,
    pub birthday: Option<NaiveDate>,
    pub address_id: Option<Uuid>,
    pub created_at: DateTime<Utc>,
    pub updated_at: DateTime<Utc>,
}

#[derive(PgEnum, Serialize, Deserialize, Clone, Copy, PartialEq, Eq, Debug)]
pub enum Gender {
    Male,
    Female,
    Other,
}

#[derive(Serialize, Deserialize, Identifiable, Queryable, Insertable, Clone, Debug)]
#[table_name = "contacts"]
pub struct Contact{
    pub id: Uuid,
    pub gender: Option<Gender>,
    pub given_name: Option<String>,
    pub family_name: Option<String>,
    pub company_name: Option<String>,
    pub title: Option<String>,
    pub phone_landline: Option<String>,
    pub phone_mobile: Option<String>,
    pub email: Option<String>,
    pub birthday: Option<NaiveDate>,
    pub address_id: Option<Uuid>,
    pub created_at: DateTime<Utc>,
    pub updated_at: DateTime<Utc>,
}

table! {
    use diesel::pg::types::sql_types::*;
    use diesel::sql_types::*;
    use db::types::GenderMapping;

    contacts (id) {
        id -> Uuid,
        gender -> Nullable<GenderMapping>,
        given_name -> Nullable<Text>,
        family_name -> Nullable<Text>,
        company_name -> Nullable<Text>,
        title -> Nullable<Text>,
        phone_landline -> Nullable<Text>,
        phone_mobile -> Nullable<Text>,
        email -> Nullable<Text>,
        birthday -> Nullable<Date>,
        address_id -> Nullable<Uuid>,
        created_at -> Timestamptz,
        updated_at -> Timestamptz,
    }
}

diesel: 1.1
diesel-derive-enum: 0.3.0

Thanks for the bug report. Fix is in 0.4.1

@adwhit I've tried with 0.4.1.
Still not working.

Hmm have you set features = ["postgres"]?

Yep.

Sorry, can't reproduce this. Chance you could post a snippet? (SQL and Rust)

Will do.

I'm actually having a similar issue on queries, where multiple trait bounds fail:
models::foo::Bar: diesel::Queryable<models::foo::db_enum_impl_Bar::BarMapping, diesel::pg::Pg>
and
models::foo::Bar: diesel::sql_types::HasSqlType<models::foo::db_enum_impl_Bar::BarMapping>
in my case.
In case it helps, in src/models/foo.rs:
#[derive(DbEnum)]
pub enum Bar {
    A,
    B,
    C,
}

table! {
    use diesel::sql_types::*;
    use super::BarMapping;

    foos (id) {
        id -> Integer
        // *snip*
        bar -> BarMapping
    }
}

#[derive(Indentifiable, Queryable)]
#[table_name = "foos"]
pub struct Foo {
    // *snip*
    pub bar: Bar,
}

and in migrations/blah/up.sql:
CREATE TYPE bar AS ENUM (
    'a',
    'b',
    'c'
);

CREATE TABLE foos (
    -- *snip*
    bar bar NOT NULL
);

Lastly, the call that fails:
foos::table
    .limit(5)
    .load::<Foo>(&pg_connection)
    .expect("Boom 💥");

@berwyn thanks, I'm looking into it

I think I got burned by this exact issue as well. Can't get everything to completely compile given your exact instructions.
error[E0277]: the trait bound `diesel::pg::Pg: diesel::sql_types::HasSqlType<models::transaction_card::db_enum_impl_CardNetwork::Card_network>` is not satisfied
   |
38 |         .execute(&*conn)?;
   |          ^^^^^^^ the trait `diesel::sql_types::HasSqlType<models::transaction_card::db_enum_impl_CardNetwork::Card_network>` is not implemented for `diesel::pg::Pg`

Thanks for bug reports. I have yanked 0.4.1, fixed in 0.4.2

Confirmed, my case compiles now. Thanks for the quick turnaround!


remove within.countries

this property is not supported.

Remove it from readme too.

already removed

Remove it from readme too.

already removed


终端无法输入

所需的数据太大，无法直接发送。我们已经将其写入剪贴板，请粘贴。

Hi,
Would you mind sending us images or videos to reproduce this issue in details?
You can use http://www.srecorder.com/home to make a video to reproduce this issue step by step for being analyzed....


docs: Fix typos in mobile-push-notifications.

(Noticed these while working through setting up a development workflow for push notifications.)

Merged, thanks for fixing these @jackrzhang!


Move ExampleKey constant to 'official' package.

This useful constant currently lives in a testing package, which leads to production code importing test packages. That feels kinda awkward.

/approve


README's download link broken

README's "Download a recent stable build" link is broken.  It leads to
http://central.maven.org/maven2/org/jmxtrans/jmxtrans/
Which gets no where.  I can still use the SNAPSHOT link though.

Fixed by  #771

Fixed by  #771


Update floating menu styles.

@keybase/react-hackers, this is ready for review. It left-aligns menu items and adds an icon in FloatingMenus (on mobile).

@keybase/react-hackers, bump.

@keybase/react-hackers, bump.

@keybase/react-hackers, bump.


[bug] Breaking change for dependent targets in v1.30.0

Describe the bug
The behavior of moon ci triggering of affected targets was changed in v1.30.0.
There are two issues here:

While this is partially a desired change in my view as it resolves https://github.com/moonrepo/moon/issues/1624, it is nonetheless a breaking change and should have been introduced behind a flag or mentioned in the release notes.
It introduced a side effect of affected dependencies of a target triggering their own dependents, which is undesired in my view.

Steps to reproduce
Full reproduction is available in this branch

Use moon v1.29.4
Make a change to apps/backend/app-01/package.json
Commit
Run moon ci --base HEAD~1:

$ moon --version
moon 1.29.4

$ moon ci --base HEAD~1
▪▪▪▪ Gathering touched files
  apps/backend/app-01/package.json
▪▪▪▪ Gathering potential targets
  app-01:install
  app-02:install
▪▪▪▪ Generating action graph
Target count: 2
Action count: 6
▪▪▪▪ Running tasks
installing
▪▪▪▪ root:install
▪▪▪▪ root:install (11ms)
app-01:install | test
▪▪▪▪ app-01:install (cached, 5aba2e13)

 SUMMARY

pass SyncWorkspace (46ms)
skip SetupToolchain(system) (skipped, 5ms)
pass SyncProject(system, root) (5ms)
pass SyncProject(system, app-01) (5ms)
pass RunTask(root:install) (15ms)
pass RunTask(app-01:install) (cached, 40ms, 5aba2e13)

 STATS

Actions: 5 completed (1 cached), 1 skipped
   Time: 113ms

Switch to moon v1.30.0
Run moon ci --base HEAD~1:

$ moon ci --base HEAD~1
▪▪▪▪ Gathering touched files
  apps/backend/app-01/package.json
▪▪▪▪ Gathering potential targets
  app-01:install
  app-02:install
▪▪▪▪ Generating action graph
Target count: 2
Action count: 8
▪▪▪▪ Running tasks
▪▪▪▪ root:install
  root:install | installing
▪▪▪▪ root:install (9ms)
app-01:install | test
▪▪▪▪ app-01:install (cached, 5aba2e13)
app-02:install | test
▪▪▪▪ app-02:install (cached, 009f8d7a)

 SUMMARY

pass SyncWorkspace (33ms)
skip SetupToolchain(system) (skipped)
pass SyncProject(system, root)
pass SyncProject(system, app-02)
pass SyncProject(system, app-01)
pass RunTask(root:install) (11ms)
pass RunTask(app-01:install) (cached, 30ms, 5aba2e13)
pass RunTask(app-02:install) (cached, 39ms, 009f8d7a)

 STATS

Actions: 7 completed (2 cached), 1 skipped
   Time: 86ms

As you can see in the second run with moon v1.30.0, the root:install task which was triggered because it's a dependency of the app-01:install task, triggered its own dependents - app-02:install.
Expected behavior
I would want a way to control if a dependency triggered by upstream tasks wii trigger its own dependents.

@milesj after upgrading to v1.30.3 this has been resolved.
Most likely fixed by https://github.com/moonrepo/moon/pull/1746
Perhaps some regression tests could be added to make sure behaviors don't change from version to version unless they are marked as breaking changes?
This will help improve the stability of moon for production use.

Cool, luckily that was the fix. I'll look into adding more tests.


Polkadot launch instructions/refrence added

Checkout the src/spec file for generating keys without building polkadot with the chainspec file (saves a lot of time)

add how-to on recipes / workshop for using this tool

at the end of the workshop to shorten for future testing after you do this manually to learn.
add to the docker section back in to shorten workshop(?)
- first there needs to be a config file for this in the parachain template. in root dir.

closing in favor of #74


feat(service/storage context): Globals APIs

Description
Fixes https://github.com/jerryjliu/llama_index/issues/6722
TLDR:
# Initialize the global contexts
llm = OpenAI(temperature=0, model="text-davinci-002")
ServiceContext.from_defaults(llm=llm).set_global()
StorageContext.from_defaults(..).set_global()

# Application code - uses the global choices
documents = SimpleDirectoryReader('../paul_graham_essay/data').load_data()
index = KeywordTableIndex.from_documents(documents=documents)
engine = ReActChatEngine.from_defaults(
  query_engine_tools=[QueryEngineTool.from_defaults(index.as_query_engine())]
)

# now want to use a local model
ServiceContext.get_global().embedding = CustomEmbedding()

Type of Change
Please delete options that are not relevant.

[X] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[X] Added new unit/integration tests
[X] I stared at the code and made sure it makes sense

Suggested Checklist:

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

Next steps:

[ ] Update all notebooks to use set_global() where appropriate
[ ] Remove unnecessary mock_service_context to majority of tests, replacing with either default (set_to_global_default) or global (set_global) mocked service contexts

Hey @jon-chuang after looking at this a bit more, while I super appreciate the effort that went into this, I don't think the UX change here is enough to justify the merge.
The change compared to current global service context is pretty minor, and a global storage context doesn't seem useful enough right (since most users will have a different storage context object for each index, i.e. a different storage context for each of a users weaviate indexes)
Happy to keep this branch though and revisit it when it makes more sense to do so! I think other ongoing high-impact projects like async callback manager take a higher priority right now, as well as just promoting our current global service context functionality more in our docs.

Hey @logan-markewich , makes sense. Yep, le'ts start with using set_global_service_context


Fix get_protonym for missing intermediate ancestor

nil values for authorship details in the where hash were breaking the search.
This would cause the "Create if Missing" strategy to duplicate existing protonyms if it had authorship info while the new record didn't (likely because it was an intermediate name).

@LordFlashmeow would be possible for you to resolve the merge conflicts?

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

Comparison is base (6e0b9da) 85.83% compared to head (e765dd2) 85.83%.

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

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@               Coverage Diff               @@
##           development    #3657      +/-   ##
===============================================
- Coverage        85.83%   85.83%   -0.01%
===============================================
  Files             2020     2020
  Lines            74419    74336      -83
===============================================
- Hits             63881    63808      -73
+ Misses           10538    10528      -10

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


Suggested Actions don't im-ing back (Card Action's type is "imBack")

Tested in Emulator. The code copy-pasted from documentation and it doesn't put selected action to the conversation:
// Create the activity and add suggested actions.
var activity = MessageFactory.SuggestedActions(new CardAction[] {
new CardAction(type: "imBack", text: "red", title: "red"),
new CardAction(type: "imBack", text: "green", title: "green"),
new CardAction(type: "imBack", text: "blue", title: "blue")
});
// Send the activity as a reply to the user.
context.Reply(activity);

You're not seeing the actions at all? That looks correct

I've test this code:
        protected override async Task OnReceiveActivity(IBotContext context)
        {

            if (context.Request.Type == ActivityTypes.Message)
            {
                if (context.Request.AsMessageActivity().Text == "foo")
                {
                    var activity = MessageFactory.SuggestedActions(new CardAction[] {
                        new CardAction(type: "imBack", text: "red", title: "red"),
                        new CardAction(type: "imBack", text: "green", title: "green"),
                        new CardAction(type: "imBack", text: "blue", title: "blue")
                        });
                    context.Reply(activity);
                }
            }

            return;

I get back this.

Note: I confirmed that ACtivity sent is correct and that this is a WebChat bug. The work-around is to send text, which is then rendered:
  var activity = MessageFactory.SuggestedActions(new CardAction[] {
                        new CardAction(type: "imBack", text: "red", title: "red"),
                        new CardAction(type: "imBack", text: "green", title: "green"),
                        new CardAction(type: "imBack", text: "blue", title: "blue")
                        }, "test test test")

@cleemullins your work around doesn't work for me (sending text).
I am not saying that Card Actions are not properly showing up, I am saying that clicking on action doesn't do anything ( it doesn't put action that was selected to conversation).  And I think that the issue should still be open.
The workaround I found is to specify value (instead of text):
var activity = MessageFactory.SuggestedActions(new CardAction[] {
new CardAction(type: "imBack", text: "red", title: "red", value:"red"),
new CardAction(type: "imBack", text: "green", title: "green", value:"green"),
new CardAction(type: "imBack", text: "blue", title: "blue", value:"blue")
}, "test test test");
                // Send the activity as a reply to the user.
                context.Reply(activity);


clientv3: fix disconnect breakage

the etcd-runner lock tester seems to work fine now

Minor issues. Looks good to me in general.

squash and merge?

ok will do


array_max_ext (and similar for other array functions)

The function array_max (already implemented) returns the maximum value in the array.
It would make sense to offer array_max_ext which does the exact same thing,
but returns an array containing the maximum value as first entry and the index of this value as second entry.
A similar extension is possible for array_min, array_mean, array_sum and array_median.
Here is a possible implementation:
`/// @function array_max_ext(array)
/// @param {array}	array	Input array
/// @return [max_value, index]
function array_max_ext(_array)
{
if is_array(_array)
{
max_val = -infinity;
index = -1;
for(var i = 0; i<array_length(_array); i++)
{
if (_array[i] > max_val)
{
max_val = _array[i];
index = -1;
}
}
return [max_val, index];
}
else
{
return [undefined, undefined];
}
}`

I'd be wary of returning a new array for each call, as it might clutter lots of memory pretty quick, depending on the usage.
However, I wouldn't mind a function like array_max_index and its kin, which would return the first index of the max value, and could then be used to quickly retrieve the actual max value. If someone's so inclined, they could make the array_max_ext function like yours using array_max_index then.

I think I too prefer an array_max_index if anything.
I actually wonder how fast it'd be if we cobble it with array_max + array_get_index. 🤔

I forgot about memory issue, thanks for that. But a version just returning the index would be awesome :)


Is there a difference result between use_cat ?

Hi,
Just wondering are there any differences on the result between model with use_cat True and use_cat False?

When you use resnets, the statistics of batchnorm will be different. When you use transformers, I think there is no difference.

so basically, use_cat True will yield better results for vision tasks?

so basically, use_cat True will yield better results for vision tasks?

On classic vision tasks, as the backbone is resnet-based and has batch norm layers, you should always set use_cat as True.
On USB vision tasks where we use vision transformers, the result would be same.

Thank you, that makes sense.


Issue 334: Clarify docs on positional arguments

This would resolve https://github.com/jarro2783/cxxopts/issues/334.

This change is

That is much clearer, thanks for putting the time into to improve the documentation.


[+0-normal-args] Be sure if we are forwarding arguments into memory, …

…that we do it at +1.
This can only happen when +0 is enabled. The reason why is that guaranteed self
is immutable so can not be passed inout without going through a var. There are
no other cases I can think of where an ArgumentSource will put self into
memory.
This is tested by the updated SILGen +0 tests.
rdar://34222540

@swift-ci smoke test and merge


Subdomain takeover using https://tilda.cc/

Service name
https://tilda.cc/
Proof
https://hackerone.com/reports/894657
Documentation
Subdomains which are pointing to tilda.cc,and has a unclaimed DNS record are vulnerable for subdomain-takeover.
Reference
https://help.tilda.ws/customdomain#:~:text=Navigate to the Site Settings,in the right upper corner.

I just took over one Tilda domain.
This is the error message

I found one with another error message
"Please renew your subscription". In this case is not possible to take over the subdomain.