Lockup/timeout with large uploads over fast networks (iOS7/8)

Whenever I try to do a large file upload over fast networks (fiber, good cable, or even 4G), I get a timeout error (after a long delay). I'm using AFNetworking 2.5.2
I construct a upload operation with AFHTTPRequestOperationManager POST, building the body with appendPartWithInputStream.
The upload appears to continue: NSLog tells me the progress-block is still being called, and bytesWritten continues to count up, but the UI updates (done from within the same block) stop, and the app appears frozen.
This only happens on fast networks - on my crappy home ADSL (1mbit up on a good day), everything works smoothly.
On fast networks, the upload completes successfully if I artifically slow down the upload with [formData throttleBandwidthWithPacketSize:kAFUploadStream3GSuggestedPacketSize delay:kAFUploadStream3GSuggestedDelay / 2];
(Dividing by two was a random successful choice...)

Can you provide any other timing information around this? I.e. the upload finishes in under a second? Can you confirm the upload actually makes it to the remote server? Any other symptoms?

What size files are you talking about for "large file upload"s?

The files in quesiton are video-recordings up to 1 min long - depending on chosen quality, in the range 6 - 10mb. On a 'slow' (my crappy home adsl) or artificially slowed-down connection this translates to around 2 mins for a successful upload.
If I don't throttle the upload (& have a fast connection) the upload times out after 3-5 minutes.
I should clarify: when I wrote 'The upload appears to continue:..." - I meant that the NSLog messages from the progress-block continue, bytesWritten continue count up, right up until the timeout. Server-side, the file upload never completes. I don't know how much (if any) of the data actually makes it to the server.
The only symptom I see is that while the progress block is still being called (as evidenced by the NSLog messages), the app UI appears frozen (the progress-bar no longer moves). Only after the timeout the app becomes responsive again.

Hm. That sounds possibly similar to, but not exactly like, something I'm seeing uploading ~30MB videos. In my instance, the network operation goes on for minutes then I get a "connection not available"-family error from iOS itself. The UI never locks up.

Could this be related to #2481?

Also, do you see the problem if you use an AFHTTPSessionManager instead of an AFHTTPRequestOperationManager?

Now that 2.6.2, we are going to be shifting our focus to the 3.0.0 stream, which removes support for NSURLConnection.
We are planning on only addressing critical security and stability issues in the 2.x branch going forward. If this becomes a widespread problem, it will be addressed.
:beers:


Diagnostic Extension for VMSSs

I have had trouble answering the following questions using the current documentation on the Diagnostic Extension.  These questions relate to the Diagnostic Extension for VMSSs.

There appear to be separate Diagnostic Extensions with different extension schemas for Linux and Windows VMs.  Is the Diagnostic Extension for VMSSs distinct from both of these?

Are there different diagnostic extensions for VMSSs with a Windows VM profile and ones with a Linux VM profile?

What is the extension schema of the Diagnostic Extension(s?) for VMSSs? Does it match the schema for either the Linux or Windows VM diagnostic extension?

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

ID: 0d1ee470-5d0c-4281-edac-d3b88287caab
Version Independent ID: c3eb4829-1139-b5e8-7e18-960c2cbcc652
Content: Overview of the Azure Diagnostics extension
Content Source: articles/azure-monitor/platform/diagnostics-extension-overview.md
Service: azure-monitor
GitHub Login: @rboucher
Microsoft Alias: robb

@mhollismcgill Thanks for the Comment. We are actively investigating and will get back to you soon.

The diagnostics extensions for Linux(called "LAD") and Windows ("WAD") are completely separate.  WAD is the same for anything running on Windows -- that is, Windows VMs, VMSS, Cloud Services and Service Fabric.   LAD is for VMs running Linux. Note there is also a way to use Telegraph Agent on Linux importing the data into Azure Monitor using custom metrics (in preview). I'm looking into your other questions.

We will now close this issue. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Update to latest logos

Summary

Release Note

Documentation

DCO will need to be approved manually, its not picking up for some reason

DCO thing fixed, good to merge now


Add json5 support to apoc-core and extended

Json5 adds a number of nice features.
In jackson we can enable most of them with capabilities. Some of which we already do.
See
https://stackoverflow.com/questions/68312227/can-the-jackson-parser-be-used-to-parse-json5

Would it make more sense to have suggestions like this as Trello cards instead of issues on Github? :)

Cardified. Closing here.


The convert_3d_counts_to_cf and convert_rp_pi_counts_to_wp function don't care about the weight.

General information

Corrfunc version: 2.5.3
platform: Linux
installation method (pip/source/other?): source

Issue description
I want to calculate the correlation function from a LSS catalogue. The LSS catalogue has some weighting factor, and I want to calculate the real weighted correlation function. But I find out that if I use DDrppi_mocks and convert_rp_pi_counts_to_wp, it will give the exactly the same wp no matter I use weighting or not. Is this ignored intentionally or by mistake?
Expected behavior
Get the correct $\xi$ and $w_p$ with weighting.
Actual behavior
The code does not care about the weighting.
What have you tried so far?
I checked the result of DDrppi_mocks, and I think it calculated the weight. Then I checked the source code of convert_rp_pi_counts_to_wp, and find out that it does not contain any code about the weight. So I'm trying to write my own version, making use of DD['weightavg'].
Minimal failing example

from Corrfunc.mocks import DDrppi_mocks
from Corrfunc.utils import convert_rp_pi_counts_to_wp
# read from some catalogue to galaxy and random.
Nd = len(galaxy)
Nr = len(random)

Nbins = 15
rp_min = 3
rp_max = 100
rp_bin = np.geomspace(rp_min, rp_max, Nbins+1)
r_p = (rp_bin[:-1]*rp_bin[1:])**0.5
pimax = 100

dd = DDrppi_mocks(autocorr=True, cosmology=1, nthreads=50, pimax=pimax, binfile=rp_bin,
                   RA1=galaxy['ra'], DEC1=galaxy['dec'], CZ1=galaxy['cz'], weights1=galaxy['w'], is_comoving_dist=True, weight_type='pair_product')
rr = DDrppi_mocks(autocorr=True, cosmology=1, nthreads=50, pimax=pimax, binfile=rp_bin,
                   RA1=random['ra'], DEC1=random['dec'], CZ1=random['cz'], weights1=random['w'], is_comoving_dist=True, weight_type='pair_product')
dr = DDrppi_mocks(
    autocorr=False, cosmology=1, nthreads=50, pimax=pimax, binfile=rp_bin,
    RA1=galaxy['ra'], DEC1=galaxy['dec'], CZ1=galaxy['cz'], weights1=galaxy['w'],
    RA2=random['ra'], DEC2=random['dec'], CZ2=random['cz'], weights2=random['w'],
    is_comoving_dist=True, weight_type='pair_product')
w = convert_rp_pi_counts_to_wp(Nd, Nd, Nr, Nr, dd, dr, dr, rr, pimax=pimax, nrpbins=Nbins)
# calculate with weighting

dd = DDrppi_mocks(autocorr=True, cosmology=1, nthreads=50, pimax=pimax, binfile=rp_bin,
                   RA1=galaxy['ra'], DEC1=galaxy['dec'], CZ1=galaxy['cz'], is_comoving_dist=True)
rr = DDrppi_mocks(autocorr=True, cosmology=1, nthreads=50, pimax=pimax, binfile=rp_bin,
                   RA1=random['ra'], DEC1=random['dec'], CZ1=random['cz'], is_comoving_dist=True)
dr = DDrppi_mocks(
    autocorr=False, cosmology=1, nthreads=50, pimax=pimax, binfile=rp_bin,
    RA1=galaxy['ra'], DEC1=galaxy['dec'], CZ1=galaxy['cz'],
    RA2=random['ra'], DEC2=random['dec'], CZ2=random['cz'],
    is_comoving_dist=True)
w_noweight = convert_rp_pi_counts_to_wp(Nd, Nd, Nr, Nr, dd, dr, dr, rr, pimax=pimax, nrpbins=Nbins)
# calculate without weighting

assert np.isclose(w, w_noweight).all()

Thanks @MeloDi-23 for using Corrfunc and opening the issue. The utility uses only the raw pair counts and assumes weights are not involved. If you need a weighted correlation function, unfortunately, you will have to account for the semantics of the weights and the weighting scheme you have used and then normalise accordingly to get the correct correlation function.

Hmm - I suppose that's fair enough. Since you found the wording unclear - do you mind sending through a pull request with a proposed fix?


[Mention The Feature]: User interaction tools

Is your feature request related to a problem? Please describe.
The site could get benefit by adding tools such as likes rating and comments for scholarship and jobs to increase user engagement
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context.
No response
Show us the magic with screenshots
No response
Checklist

[X] I have checked the Existing Issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

sure go ahead
Make sure to join the discord server for further communication
Don't forget to star the repository
Complete the task in next 2 days and kindly mention your issue # in your pr like this:
Closes #IssueNumber
Note: the level of the issue may change based on the quality of the PR cod


Lower the supported VSCode Version

Is your feature request related to a problem? Please describe.
We have a really restricted env and we cannot use the latest VSCode versions. Current version we have is 1.44. Will you extension functionally work on 1.44? Could you please lower the supported version?
Describe the solution you'd like
Lower supported VSCode version to 1.44

Hey! The latest version of the extension v0.5.1 supports vscode v1.44.0 onwards.

Hey! The latest version of the extension v0.5.1 supports vscode v1.44.0 onwards.

Thanks!

Thanks!


--save and --bail don't combine

The combination would be particularly useful. Running all tests with --bail will make them go faster, quitting and writing the failing test file names to the save file immediately, and then  allowing the failing tests to be looked at in more depth.

Hm.  This could be tricky, right?  I guess if a save file is set, then a bailout should mean that all subsequent tests get written to the file?  That would make these two things work well together, I suppose.


tailscale: support split DNS endpoint

Add support for /api/v2/tailnet/{tailnetID}/dns/split-dns endpoint for updating split DNS settings for a given tailnet.
Updates https://github.com/tailscale/corp/issues/19483

TODO:

[ ] wait for endpoint to be deployed


flutter连接mysql时执行查询语句报错

[ERROR:flutter/lib/ui/ui_dart_state.cc(209)] Unhandled Exception: SocketException: Socket has been closed
添加查询语句报错：await conn.query(
'CREATE TABLE users (id int NOT NULL AUTO_INCREMENT PRIMARY KEY, name varchar(255), email varchar(255), age int)');

Hi @notture, this seems like an issue with the mysql1 package and not an issue with flutter.
Kindly file the issue over on their own issue tracker over here.
There even seems to be a similar issue in https://github.com/adamlofts/mysql1_dart/issues/112.
Closing from here as this is an issue with a third-party package.
Thank you


Create New Index Points study tree command fails to open dialog (or tab).

Build: CI-CD #41 (Merge pull request #354), dated 2022-05-20
From the Study Tree the command to create new index points fails to open a dialog (nothing happens).

Good catch. This functionality does not yet exist. Nor will the hdf hydraulics imports, nor graphical frequency functions.

This is no longer applicable.


Fix localised content in pod.doc

Using !IfLocale within yaml doesn't get evaluated if used within pod.doc.
 {% set partial = pod.doc('/content/partials/footer.yaml').fields %}
 {{renderPartial.renderPartial(partial)}}

I think the issue here is we don't have any implicit localization available since pod.doc is just being called directly; the immediate fix is to pass in the current document's locale to the call to pod.doc:
 {% set partial = pod.doc('/content/partials/footer.yaml', doc.locale).fields %}
 {{renderPartial.renderPartial(partial)}}

We can leave this open in the meantime, to see if we can make calls to pod functions context-aware, or if we should close this out as working as intended or infeasible.
Note: Regardless, we should likely update the starter with this improvement.


[next][Switch/Checkbox/Radio] consistent implementation of disabled across selection controls

disabled was not implemented on selection control labels and switch root.

[ ] PR has tests / docs demo, and is linted.
[ ] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[ ] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

Still have test warning...looking at that.

Aside from the needed regression baseline, the switch disablement with label still needs work (it is clickable).  Still looking at it.

@oliviertassinari @nathanmarks - I could use some implementation help here. As-is, disabled needs to be implemented with opacity as color is unknown.  For example, <Switch checked /> is accent pink, so the ``IconButtonicon cannot usepalette.text.disabledand instead must useopacity.  This is most prominent where SelectionLabelwraps theSwitchBase`.
I've added examples and the problem is visible with the Switch section.
See the following:

Icon Button (good) - http://localhost:3000/#/component-demos/buttons
http://localhost:3000/#/component-demos/selection-controls

Checkbox (not so good, opacity builds on the box icon)
Switch (bad)
Switch w/label (worse - building transparency)

I'm going to abandon this PR.  I have other priorities and don't want to let it sit open.  I'll leave the branch in case someone else wants to pick this up.

@rosskevin I'm gonna pick it

I have forgotten to close this PR. Thanks. Here is a second iteration #5725.


Annotation for splicing site.

Hi, I am confused of the pathogenicity annotation of splice_site mutation.
In two articles recently published in NEJM, one classified splice variants affecting penultimate and the last exon as VUS or B/LB, and the other only keep splice sites (+/- 1 or 2). I dont know if InterVar also annotates the splice variants in a similar way.

ANNOVAR annotates canonical splice site which is within 2bp from
exon/intron boundary.
On Sun, Jul 11, 2021 at 10:38 PM Caijin Lin @.***>
wrote:

Hi, I am confused of the pathogenicity annotation of splice_site mutation.
In two articles recently published in NEJM, one classified splice variants
affecting penultimate and the last exon as VUS or B/LB, and the other only
keep splice sites (+/- 1 or 2). I dont know if InterVar also annotates the
splice variants in a similar way.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/WGLab/InterVar/issues/66, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNG3OAHJICRZQMLPH6I52TTXJITVANCNFSM5AF5W74Q
.

Thanks for your reply.
I dont know if it is possible to identify variants located in the last exon or the last 50bp of the penultimate exon from the ANNOVAR output.


[v1.2][WEEK 9][UI][LOGIC] Status bar UI enhancement + New Backup command

Backup command in progress.

Changes Unknown when pulling 3be877cc49c6df32f6e9580fcee6db17684af4e8 on charlesgoh:master into ** on CS2103AUG2017-W09-B2:master**.


EHPR-97 | Optionally apply fields in constructor

https://freshworks.atlassian.net/browse/EHPR-97
It worked!

10/10 Great thinking

Credit to whoever suggested it this morning 😄


New version: Nushell.Nushell version 0.92.2

Pull request has been created with WinGet Releaser v2 :rocket:
Microsoft Reviewers: Open in CodeFlow

@stephengillie can you look at this please? i recently restarted it by closing and reopening, which didn't quite work right. then i modified one of the files to add an extra line and that restarted it. I had to restart because the last validation run was over 13 hours long. This one is now over 1.5 hours and we'd like to get this version published since it's in response to a CVE.

@wingetbot run

We have recently had a few PRs that had unusual failed runs that provided no logs or labels - possibly froma temporary pipeline issue. The first run or two might have been more of these. This run I recently started appears to still be validating - about 4.5 hours into what I thought was a 3 hour timeout. Unfortunately, I don't currently have any info on why this is occurring.

@fdncred,
Thank you for your extended patience.

Thanks. So, what was the deal? Should we expect this from now on?


Writing cost function in Python

Thanks for this awesome wrapping library for ceres.
In https://github.com/cvg/pyceres/issues/20, you said:

We don't provide any way to write cost functions in Python.

However, I have noticed that PyCostFunction already provides the corresponding trampoline functionality. The only thing that needs to be added is the encapsulation of set_num_residuals(). Which is done as follows:
      .def("set_num_residuals", &PyCostFunction::set_num_residuals)

After adding it, I tested the helloworld CostFunction in Python. It seems ok.
class HelloworldCost(pyceres.CostFunction):
    def __init__(self):
        pyceres.CostFunction.__init__(self)
        self.set_num_residuals(1)
        self.set_parameter_block_sizes([1])

    def Evaluate(self, parameters, residuals, jacobians):
        # ref: examples/helloworld_analytic_diff.cc
        x = parameters[0][0]
        residuals[0] = 10. - x
        if jacobians is not None:
            jacobians[0][0] = -1.
        return True

def test_python_cost():
    x = np.array(5.0)
    prob = pyceres.Problem()
    cost = HelloworldCost()
    loss = pyceres.TrivialLoss()
    prob.add_residual_block(cost, loss, [x])

    opts = pyceres.SolverOptions()
    opts.minimizer_progress_to_stdout = True
    summary = pyceres.SolverSummary()
    pyceres.solve(opts, prob, summary)
    print(summary.BriefReport())
    print(x)

The output is:
iter      cost      cost_change  |gradient|   |step|    tr_ratio  tr_radius  ls_iter  iter_time  total_time
   0  1.250000e+01    0.00e+00    5.00e+00   0.00e+00   0.00e+00  1.00e+04        0    4.41e-05    1.08e-04
   1  1.249750e-07    1.25e+01    5.00e-04   5.00e+00   1.00e+00  3.00e+04        1    6.51e-05    2.06e-04
   2  1.388518e-16    1.25e-07    1.67e-08   5.00e-04   1.00e+00  9.00e+04        1    2.88e-05    2.45e-04
Ceres Solver Report: Iterations: 3, Initial cost: 1.250000e+01, Final cost: 1.388518e-16, Termination: CONVERGENCE
9.999999983335556

I am not sure if this approach has any issues or if it can be scaled to larger problems.
Looking forward to your comments.

We actually haven't tried this out - very nice! This will indeed be much slower than C++ cost functions for large problems because the GIL prevents evaluating costs in parallel, but this is nonetheless very useful for debugging! Would you mind submitting a PR with set_num_residuals and a simple test script? Thank you.

Sure, submited.


Sharepoint Office Library Templates cant be chosen in Office Programs

Newly created Library wont show up anywhere, not ont he PPT StartPage, not in the New Menu and not in the Open-->Sites area.

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

ID: 5bacce83-65f6-b862-e413-09122aee356d
Version Independent ID: 5c4e1169-1d5d-6e0d-d574-f3aaba0a5d92
Content: Create an organization assets library - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/organization-assets-library.md
Service: sharepoint-online
GitHub Login: @kaarins
Microsoft Alias: kaarins

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Option to set game idling order

I'd like to be able to idle games with the fewest number of card drops first and save the games with the most card drops for last. This is useful because it's often more affordable to buy just two or three cards to complete while sets with many cards usually aren't as affordable. Ergo small sets are more useful as a short-term priority over large sets which are more often just sold at some point in the future. Furthermore, by starting with small sets, the idler can get through a larger percentage of total games more quickly.

Ergo small sets are more useful as a short-term priority over large sets which are more often just sold at some point in the future.
Only if you plan to craft badges. You'll be surprised, but many people don't. This feature would make ASF slower (because if parsing cards quantity and sorting game list), and I'm not sure how useful it would be for the majority of users.

I feel exactly the same as @Ryzhehvost but I didn't drop this suggestion right away, as it could be beneficial and I could tune ASF code-wise to spend extra time only if user wants that, as in - if appropriate config property (which will be named PreferredFarmingOrder or similar) is set.
Wishlist for now.

I also think it would be useful for badge crafters: sometimes I get cards from my secondary bots with only 1 or 2 to go for the badge but still have 1000+ games to farm on my main account so I have to keep track manually on which one to farm first via the !play command without forgetting to !resume or !play another game when all cards dropped.
The PreferredFarmingOrder parameter would be an elegant way to do so :)

I would also love feature like this.
Also parsing card price data from Steam or 3rd party site and you could have option like farm most valuable cards first.

Also parsing card price data from Steam or 3rd party site and you could have option like farm most valuable cards first.

Parsing card price data from Steam is too expensive to even consider adding to ASF, and I won't include any mandatory dependency such as third-party Enhanced Steam API that Idle Master uses only to add such extra optional feature.

I think it will be a useful option)

This is now done starting in https://github.com/JustArchi/ArchiSteamFarm/releases/tag/<IP_ADDRESS> thanks to @stackia
As always wiki page will come later - FarmingOrder bot config property with byte value.
0 - Unordered,
1 - MostCardDropRemainingFirst,
2 - FewestCardDropRemainingFirst

Notice: actual values might change as I'll be probably adding more options later, so until stable release keep an eye on updates.

FarmingOrder is in wiki now - https://github.com/JustArchi/ArchiSteamFarm/wiki/Configuration - it affects latest version (<IP_ADDRESS>) as of now.

Downloaded <IP_ADDRESS> (still says <IP_ADDRESS> in the logs), added "FarmingOrder": 3 to my bot's json and it started farming with: 2016-07-30 22:00:13|INFO|darkmos1|ShouldFarm() Status for 375460: 1 cards remaining
So seems it's working as foreseen but maybe it's resuming where he stopped, I'll monitor which game he takes after this :)

Oops, fixed links for <IP_ADDRESS>. Thanks!

Seems it farmed all my games where 1 or 2 cards were still pending: http://pastebin.com/HKdK19gg
Thanks to both of you and @stackia for this update ;)
Concerning <IP_ADDRESS> no regression detected so far but didn't test the new redeem behaviour as I have no spare keys laying around. I like the card count in !status answer in chat: Bot darkmos1 is farming appIDs: 231740 and has a total of 853 games left, 3195 cards drop remaining to farm. :D

Final version for testing is https://github.com/JustArchi/ArchiSteamFarm/releases/tag/<IP_ADDRESS> - stable candidate.
This issue will get closed with stable release, so it's final time to verify that new feature works OK, I couldn't find any bugs myself. Thanks!

I consider this issue implemented and tested, but let me know if you spot anything weird in terms of farming order. Thanks for feedback!


How to personally edit mods to work with save

My friend and I have been playing together, and we noticed the series of Dragonball mods do not save progress with their levels. Can I modify those files myself to work with ProperSave?

Technically, yes, not sure about the amount of work you would have to do.

Thanks. I'll try to contact the mod creators.

Well, it will be much easier for the mod creator.

I tried contacting the mod creator and they seem to be opposed to fixing the issue globally. Is there anyway I could contact you directly to talk about modifying the code myself? I am a bit inexperienced with RoR2 modding, and your knowledge would be useful.

My discord is KingEnderBrine#3624

Thank you. I am DirtyDeedsDoneDirtCheap


docs.rs broken

https://docs.rs/crate/opencv/0.42.0/builds
I'm assuming you're trying to runtime buildtime-generation on docs.rs, which then fails.

Thanks for the catch! I’ll fix that

Fixed


Added QtCore.Framework

Without QtCore, meson can't compile a code. Had an error 'QThread' missing file.
MacOS 10.15.1

Thanks 👍


comparison experiments

Hello, the kuaishou dataset has not been published yet. Apart from Taobao dataset,are there any other datasets available for comparison experiments in this code? Any suggestions?Thanks!

Hello, you can use the provided data preprocess codes to generate training and test data for your custom datasets. The data preprocess codes can be found at here. Most of the datasets adopted in sequential recommendation can be used, such as Amazon, Yelp, etc. Good Luck!

Thank you very much. When processing the yelp dataset, I noticed that in the sequential_review.py, load_data in the yelp_main uses categories.json, as follows:
“
with open(os.path.join(dirs, 'categories.json'), 'r') as f:
category = json.load(f)
”
When I downloaded the yelp dataset, I found no categories.json.Is the yelp dataset not completely downloaded?
(My download address:https://www.yelp.com/dataset/download）

You can find the category file of the Yelp dataset at here.


[DOCS] Improve Database configuration in the README file

Description
In my opinion, we could improve the README about the DB connection. Just saw that a couple of other folks (yes, me too :rofl: ) had a couple of issues connecting and running all the setup. I think we just need a very simple guide on how to setup MongoDB Atlas. Or at least just write down that we suggest the use of it.

Yes, strongly agree with you. 😄

@matmont i've assigned the issue to you


Environment variable(s) for Steward config

Rather than requiring users to pass their config toml as an argument to every single steward command, it would be good for there to be a STEWARD_CONFIG_PATH environment variable that is checked on startup and used if present. This also prevents the strange behavior of using unexpected default config values when you forget to use the -c parameter, such as when creating a key and it appearing in /tmp/keystore by mistake.
Not sure if Abscissa actually supports this currently, it may require a PR there.

I think I've seen something like this in clap. If clap has it, Abscissa definitely will. So, I'm picking this up since I have the better experience here.


Solidarity failures cause confusing error on Ignite new

While debugging #1912, I noticed that any solidarity failures (like incorrect cocoapods version for example) cause Ignite new to fail very vaguely.
Steps to reproduce

Set cocoapods version to 1.10
Run npx ignite-cli new Pizza App

You can loosen the solidarity rules, or even remove solidarity completely imo.

It's still nice to have to ensure folks have the right environment, I just need to figure out why the actual error isn't bubbling up through Gluegun.

@jamonholmgren since Solidarity is removed, this issue could be closed?


V3 commands return 422 status code instead of 400 for malformed requests

Issue
Many V3 endpoints return 422 Unprocessable instead of 400 Bad Request when the request message is malformed.
Context
In pattern of using message objects to represent the body of requests made to V3 endpoints, message validation is generally restricted to syntactic rather than semantic validation. In our view, a 400 status code would be more appropriate than a 422 when this validation fails, since a 422 indicates a semantically invalid request.
We searched for unprocessable! (message.errors.full_messages) unless message.valid? in the cloud controller code and found many hits in the v3 controllers. Since it's so widespread, we thought it'd be best to raise this as an issue instead of PR immediately to see if there's some context we're missing.
→ ag 'unprocessable!\(message.errors.full_messages\) unless message.valid?'
app/controllers/internal/packages_controller.rb
13:        unprocessable!(message.errors.full_messages) unless message.valid?

app/controllers/v3/app_features_controller.rb
38:    unprocessable!(message.errors.full_messages) unless message.valid?

app/controllers/v3/apps_controller.rb
49:    unprocessable!(message.errors.full_messages) unless message.valid?
70:    unprocessable!(message.errors.full_messages) unless message.valid?
184:    unprocessable!(message.errors.full_messages) unless message.valid?

app/controllers/v3/builds_controller.rb
8:    unprocessable!(message.errors.full_messages) unless message.valid?

app/controllers/v3/droplets_controller.rb
58:    unprocessable!(message.errors.full_messages) unless message.valid?

app/controllers/v3/isolation_segments_controller.rb
24:    unprocessable!(message.errors.full_messages) unless message.valid?
77:    unprocessable!(message.errors.full_messages) unless message.valid?
158:    unprocessable!(message.errors.full_messages) unless message.valid?

app/controllers/v3/organizations_controller.rb
41:    unprocessable!(message.errors.full_messages) unless message.valid?
69:    unprocessable!(message.errors.full_messages) unless message.valid?

app/controllers/v3/packages_controller.rb
38:    unprocessable!(message.errors.full_messages) unless message.valid?
109:    unprocessable!(message.errors.full_messages) unless message.valid?

app/controllers/v3/processes_controller.rb
46:    unprocessable!(message.errors.full_messages) unless message.valid?

app/controllers/v3/route_mappings_controller.rb
36:    unprocessable!(message.errors.full_messages) unless message.valid?

app/controllers/v3/service_bindings_controller.rb
15:    unprocessable!(message.errors.full_messages) unless message.valid?

app/controllers/v3/service_instances_controller.rb
37:    unprocessable!(message.errors.full_messages) unless message.valid?

app/controllers/v3/spaces_controller.rb
25:    unprocessable!(message.errors.full_messages) unless message.valid?
55:    unprocessable!(message.errors.full_messages) unless message.valid?

app/controllers/v3/tasks_controller.rb
45:    unprocessable!(message.errors.full_messages) unless message.valid?

Steps to Reproduce

Run cf curl /v3/organizations -X POST -d "{}"
Observe the following error (status code is 422):

{
   "errors": [
      {
         "detail": "Name can't be blank",
         "title": "CF-UnprocessableEntity",
         "code": 10008
      }
   ]
}

Expected result
{
   "errors": [
      {
         "detail": "Name can't be blank",
         "title": "CF-InvalidRequest",
         "code": 10008
      }
   ]
}

(Status code would be 400)
Possible Fix
Change unprocessable!(message.errors.full_messages) unless message.valid? to invalid_request!(message.errors.full_messages) unless message.valid? (and look for other similar lines of code)
Thanks,
SAPI (Jen and @Samze)

Hi @jenspinney and @Samze,
Thanks for raising this. You have some good points. We'll escalate to @zrob for what he thinks since this sounds like a feature request, and we don't have the context to make this call.
Thanks,
CAPI Community Pair (Annie and @ericpromislow)

Thanks @anniesing and @ericpromislow! It seems more like a bug than a feature request to me. There are also many places in the v3 API where 400 is being correctly returned. If you ag for "unless message.valid?" in the cloud controller codebase, you'll see a lot of inconsistencies (sometimes 400, sometimes 422).

The expected error codes when dealing with 400/422 across v3 are:

the message fails to be parseable (it isn't json), then return 400
the message is paresable (it is json), but we won't accept it b/c it lacks required information or it would result in failing business rules (exceed quota), then return 422

Essentially always return a 422 with an actionable error message.  Return a 400 if the request is totally unrecognizable.
How does that line up with what you're seeing across the code base?

Thanks @zrob (also CC @Gerg). This doesn't seem to totally line up with the Style Guide.
The Style Guide says re: Bad Request:
This status MUST be returned for requests that provide malformed or invalid data. Examples: malformed request body, unexpected query parameters, or invalid request fields.

Specifically the invalid request fields would fall under 422 in your description. Does the style guide need updated, or has the world moved on in uhhh.. 18 months. Thanks!

There are some places where we intentionally return 400s on v3: https://github.com/cloudfoundry/cloud_controller_ng/blob/4aad75b928a075bb7c936fa62b8f381f6b302896/app/controllers/v3/application_controller.rb#L5-L11
Not sure if usage of those methods line up with the style guide.


Demo Server down

please check the demo server availibility!
`

HTTP/1.1 500 Internal Server Error
Date: Mon, 24 Oct 2016 09:54:23 GMT
Server: Apache/2.2.15 (Red Hat)
Content-Type: text/html; charset=iso-8859-1
Content-Length: 623
Accept-Ranges: none
Connection: keep-alive

`
thanks
Yassine

https://github.com/quokkaproject/quokka/issues/386

the server is down ...

Internal Server Error
The server encountered an internal error or misconfiguration and was unable to complete your request.
Please contact the server administrator, root@localhost and inform them of the time the error occurred, and anything you might have done that may have caused the error.
More information about this error may be available in the server error log.
Apache/2.2.15 (Red Hat) Server at demo.quokkaproject.org Port 80


ideep commit broken

I'm getting errors while trying to checkout the repo
git submodule update --init --recursive
error: no such remote ref fb1adc449de61b56e92f8a81e02b91c068209f47
Fetched in submodule path 'pytorch/third_party/ideep', but it did not contain fb1adc449de61b56e92f8a81e02b91c068209f47. Direct fetching of that commit failed.
Failed to recurse into submodule path 'pytorch'

It looks like we have a submodule reference to a fork of PyTorch prior to this getting merged.  What do you think is the best way to resolve this?

We're merging this into PyTorch! The API is a bit in flux due to some discussions of what the API should look like in PyTorch, but I'm on commit https://github.com/ebetica/pytorch/tree/9e3f5bb5fd7e418172eb81f90a0a9e59aa8b0222.
If you want to keep using the current version, simply update the iDeep inside PyTorch to point to pytorch/ideep, that should solve it.


Cant find my profile

I just downloaded the tool and i cant find my profile it shows a bunch of others but mine.

look on your profile settings that need to be the steam cloud off


xml onClick does not work on some platform versions

It's fixed in support library when using the AppCompat's inflater (see https://code.google.com/p/android/issues/detail?id=174871). The fix is to extend ImageView from the support library instead.
Also a workaround is to set the onClick listener programmatically instead.
Just an FYI.

Thanks for the heads-up! Extending AppCompatImageView is not really an option as that would make CircleImageView depend on the appcompat library and that would needlessly bloat this library.
I'll close this as it's an upstream bug - should be fixed in M.


playground: "Press Esc to move out of the Editor."

When I use play.golang.org today, clicking anywhere in the text box makes a notice box appear at the top that says "Press Esc to move out of the Editor." If I click on the bar itself, it goes away. If I click on a different browser window, the bar goes away. When I click back to that window, the bar comes back. This is incredibly distracting.
I assume the bar has been added as part of accessibility work, but it seems like this specific implementation cannot possibly be required to be accessible. I have never seen a bar like that in any other text box I have ever used on any web site, including for example Gmail, and I am sure plenty of accessibility effort has gone into Gmail.
We should revert the CL that introduced this flickery bar and find a better way.
/cc @jamalc

Something extra is needed for text boxes that override the default behavior of the tab key for moving focus out of the text box.
Having a notice before you enter the textbox lets keyboard/screen reader users know that a special method is required to escape the text box.
A lot of similar components use an aria live region for this so a screen reader will say something but this is incorrect as sighted users who do not use the mouse, for whatever reason, will not be informed.
In this case, the flicker could be avoided by using visibility: hidden on the message so that it takes up the same amount of vertical space whether it's displayed or not. That would help keyboard users. But, for low vision or blind screen reader users, the message won't be delivered to the screen reader until after the virtual cursor moves past where the message is so they'll never get the message. Similarly, there's a screen reader only copy of the message after the textarea which the virtual cursor won't visit until after they've figured out how to escape on their own. So a live region would need to be used in tandem with the banner.
However, this is all very complex with a lot of moving parts when there is a gordian knot cutting solution: just always show the banner before the textarea. No javascript necessary, everyone wins.

For what it's worth, the notice is not "before you enter the textbox". It appears above the box after you enter the box. It's not even clear to me whether screen readers would read this text since it appears before the current cursor. Perhaps they still will.
I agree that a simple, small unobtrusive note above the textbox in ordinary text (on the plain white background, not in its own box) that is always displayed would be preferable to the flickery box.

on mobile, the banner seems to always be there taking up screen estate with no way to dismiss it, even though there's no esc key available to press


how to calculate individual descriptors; supplying descriptors.xml file

Hi, I have run into a situation where a single 2d descriptor fails which makes padelpy hang / timeout.  I proved this by opening the java version and removing the descriptor which makes the calculation run fine.  So i would like to specify which descriptors to calculate in padelpy.  I think this should be achieved by passing the padeldescriptor(descriptortype='\path\to\descriptortypes') argument which points to the descriptors.xml file (originally found in the site-packages\padelpy\PaDEL-Descriptor location).  I tried modifying this file turning all the descriptors to false except 1 or 2 trying to get it to calculation just those descriptors.  i have attached the xml file as txt.  However even with this it just calculates all 2d descriptors for a test mol file, so i'm not doing something correctly or there is a bug in how this information is passed.
Any help would be appreciated.
descriptors.txt

not sure what i did but i eventually got it working using the logic outlined above.  I added the option to add the descriptortype to the from_mdl functions which made it easy to work with.


Payment Methods e2e specs failing even after Cart Refactor (for mobile and desktop)

Requests made during those tests sometime cause the test to fail. Need to make sure request are done before moving onto the next commands in cypress.
cx-submit-review cannot be found in payment-method.ts , must make cypress wait for page to load before asserting.
projects/storefrontapp-e2e-cypress/cypress/integration/regression/my-account/payment-methods.e2e-spec.ts
Logs here: https://jkmaster.test.c3po.b2c.ydev.hybris.com/job/spartacus-regression-tests/560/consoleFull

Maybe the cancelled delete request has something to do with this failure?

Such e2e tests are working randomly, sometimes they pass however sometimes following error appears: "RegionModel with country, isocode 'US, US-undefined' not found!".
The key here might be a timeout and for purpose of this fix was increased and changed to constant variable.
It's for sure not a best and elegant solution, we have to back and investigate this problem in a future.

Error for this test seen in dev-com6 develop branch dec_10_2019.
payment-methods.e2e-spec.ts.mp4.zip

After conversation with @qjimmy we decided to not merge PR, because tests are now passing. However let's keep this ticket and think about better (not hardcoded) solution in case of errors in the future.


Fix paths for "proper" Haiku installation

Description
Adds support for neofetch when installed on Haiku with PREFIX=/boot/home/config/non-packaged make install (as is convention)
Features
neofetch now properly loads ASCII art and default configs when installed "properly" on Haiku (without being packaged).
Issues
Would probably be better to have a $PREFIX variable defined somewhere, to be pre-populated by running PREFIX=/foo/bar/baz make install.  Not entirely sure how feasible or desirable that would be versus this sort of one-off case.
TODO
See "Issues".

We can detect OS via makefile, I'll look into this.
Anyway, the proper make command will be make PREFIX=/usr/foo/bar install.

We could also possibly modify the Makefile to store the $PREFIX value in a file. Then when sourcing the default config and ascii art we simply look at the saved prefix value. This way we don't hard-code a list of install locations for everything.
There's probably a better way of doing this though.

I made https://github.com/konimex/neofetch/commit/cb48262f25bff395291f1a920904c9db4e867d81, but this currently only works with GNU make.

Are there any other projects that have a similar installation process? We could have a loot at other projects to see how they've solved this.

cfg2html uses separate makefiles for separate OS.

That seems overly complicated (at least for us), cfg2html already has separate scripts for each OS so it makes sense there.

Due to the lack of standardized makefile formatting, the best we can do is add a note for Haiku, I've added it in the Installation Wiki.

Thanks for the PR!


Add "keys" to config_players sent via BCP, add commands to clear keys on show end

Plugin config players need a way to be able to stop whatever they've started when shows end.
Proposed solution is an extension to the MPF side of the plugin player so that any play commands that are sent via BCP also include a "key" setting, which will be a GUID (based on the show name and an ever-increasing int, such as attract_dmd_loop.1)
When a show ends, a new BCP trigger will be sent like
trigger&name=clear_config_players&key=attract_dmd_loop.1
Handling that will be handled in the base MC config player class, and it will call each config player will need to implement a clear() method, passing the key.
Each config player will need to save the key when they play their config in whatever way makes sense for them, such that they can find whatever is playing with that key and stop it.
It will be up to each config player to stop what they're doing in their own way. For example, the slide_player could choose to implement the slide's configured transition out.

done


Hi Professor, I am not very familiar with c++, I want to run this project, but there is an error when i run "python3 build.py -DVOLE_PSI_ENABLE_BOOST=ON". I can't solve it.

This appears to be a bug on my side. Let me look into it.

not sure, you might need to update your compiler but thats a total guess.


Prettier check in CI

So that @artob can stop leaving comments to people telling them to run prettier (most recent example).

Thanks, @birchmd!


Add Quantu to sites.yml

Hi! We're working on a rather large open source application that uses sveltekit. I thought this might be useful for the community as a real-world project to learn from.

Thanks for the suggestion! Is it possible for first time visitors to see/do 1 or 2 of the courses before signing up? I think that would improve your CTA and also make it faster for Svelte developers to see if QuantU implements any techniques they want to learn.

Hey, yes. Something like that is definitely planned, we're just trying to focus on building out course interactive features like auto-generated quizzes and various types of symbolic math input/evaluation stuff right now, so it might be a few months until we get around to it.


Consider executeCommandProvider for multi server

Thank you for the giving awesome samples.
How can we implement multi server with executeCommandProvider?
I met command already exists error.
https://github.com/Microsoft/vscode/blob/8e0baeddeb7ba1b563233335bc28e37163669747/src/vs/workbench/api/node/extHostCommands.ts#L65

@jrieken Isn't this explanation enough?

@jrieken I guess you're busy though, could you have a look at this?
I think this is a critical problem to provide commands/actions from multi-server.

I think this should be related to this one.
What I've ended up doing is using a global clients: Map<string, LanguageClient> (as showed in the multi-server lsp example) and then I locate the correct client for when I have to launch commands. Something like,
async function registerHiePointCommand(name: string,
                                       command: string,
                                       context: ExtensionContext) {
  const editorCmd = commands.registerTextEditorCommand(name, (editor, edit) => {
    const cmd = {
      command,
      arguments: [
        {
          file: editor.document.uri.toString(),
          pos: editor.selections[0].active,
        },
      ],
    };
    // Get the current file and workspace folder.
    const uri = editor.document.uri;
    const folder = workspace.getWorkspaceFolder(uri);
    // If there is a client registered for this workspace, use that client.
    if (clients.has(folder.uri.toString())) {
      const client = clients.get(folder.uri.toString());
      client.sendRequest('workspace/executeCommand', cmd).then(hints => {
        return true;
      }, e => {
        console.error(e);
      });
    }
  });
  context.subscriptions.push(editorCmd);
}

It gets the current file, then looks up the workspace folder. It uses this folder.uri.toString() as the key for the language server.
Then to get rid of the command already exists, I simply check if commands have been registered, else I register them. E.g. a global let hieCommandsRegistered: boolean = false;, and then check if (!hieCommandsRegistered) { ...; hieCommandsRegistered = true; }.


Encode \b and \f in JSON string as themselves

This PR fixes a test failure by adapting AppendJSONString to follow the new behavior of encoding/json in Go 1.22. The corresponding test case is appended by a check of the \b and \f characters.
Since Go 1.22 the encoding/json library encodes the characters \b and \f as themselves. Before 1.22 they were encoded as \u0008 and \u000c. The new behavior fails TestJSONString:
--- FAIL: TestJSONString (0.00s)
    gjson_test.go:2560: "1&$'\f\x16\xfa\xbah\xc9zz\x89DR\xf3"
                "1\u0026$'\u000c\u0016\ufffd\ufffdh\ufffdzz\ufffdDR\ufffd"
                "1\u0026$'\f\u0016\ufffd\ufffdh\ufffdzz\ufffdDR\ufffd"
                <<< MISMATCH >>>

Go 1.22 release notes: https://tip.golang.org/doc/go1.22#minor_library_changes

Merged. I had to modify your test to exclude older Go versions ... in case 1.21 users want to pass the tests.

and thanks for the PR :)


Sections should not immediately reorganize their contents when items are removed

Consider this usecase:
I have some items in a section, and I want to sell the first 6 of them to a vendor:

The fastest way for me to do this is to right-click each of the first 6 items, moving my mouse to each item. But after clicking the first couple of them the section will reflow, and now I've lost my place and am vendoring the wrong things. I think it would be better if item positions remained static when transferring items out of your bags (vendoring, moving to bank, equipping, etc).

That's bit me too.  I think re-flowing as items are added is OK, since you have to put them somewhre, but I don't think it's necessary when items are removed.

Instead of reflowing on remove, just enable the "New" button so we can force a reflow when we're done. (Or it can reflow when the bag is closed and reopened.)

This is solid feedback. I'll evaluate how to stop items from re-sorting if they are removed, such that they will just leave gaps, and resort on next bag open.

It doesn't just do it when selling, does it when you equipping too
(caused my pally to use cloth hands for about an hour)

I just pushed a change for this in grid mode, where items won't reshuffle if you move items out of your bag. They will reshuffle in a few cases though:

If you completely empty out a section
If you add any item to your bag
If you close and reopen the bag

I think this is a solid experience. Note, this only happens in grid mode. I likely won't implement this in list mode, as it doesn't make much sense. I'll probably implement this in one bag view mode after I make sure there are no bugs.
Please test it and let me know what you think! Thanks!

ok this seems to stop rearranging inside the groups but the groups move all over after selling one thing from the group

ok this seems to stop rearranging inside the groups but the groups move all over after selling one thing from the group

Yes, I noticed that too last night -- I'll be working on that as well.

This has been fixed and released for both bank and bags. Both items, and sections, will not reorder when removing items from the bags.
Thanks again for the feedback!

This reordering happens each time you withdraw something from your bank. The bag and bank gets reshuffled. Any way you can simply add a toggle to disable this experience, or make it only reorg when opening the bank/bags, or clicking the reorganize button?

The bags will need to be reorganized if there's insufficient space when an item moves. Was there enough space?

The bags will need to be reorganized if there's insufficient space when an item moves. Was there enough space?

Both reagent bank, and bags have space. Moving an item from bags to reagent bank -- no reorg. Moving an item from reagent bank to the bags -- reorg with each item moved.

I'm actively working on this for the bank. I have a branch going -- just need to fix a few last bugs. Sit tight.

Specific experience example from Remix:

I click a stack of prismatic gems, wanting to do this 3 times in a row to upgrade 2 steps
after the first click, there's a new gem item in the category which has now jumped to a completely different spot on my screen
I now have to hunt down the category and find the correct gem stack again
This is very annoying.

As of 9/6/24 with latest version, this is still happening for me.  I was moving things from my bags to the reagent bank and the warband bank and the section kept jumping from top to bottom with each item.  I move an item in, it moves to the top.  I move another item in, it goes back to where it was.  Repeated over and over.  Makes it really annoying to move things.


scalajs-library, ... 1.3.1 (was 0.6.33)

Updates

org.scala-js:scalajs-library
org.scala-js:scalajs-test-bridge

from 0.6.33 to 1.3.1.
GitHub Release Notes - Version Diff
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!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-js" } ]

labels: library-update, semver-major

superseded by #391

superseded by #393


Allow custom config files

Add all-in-one rake task, which runs ruby, coffee, scss code style checks.
Allow custom configuration file, which could be used in a Rails application.
Check new and modified files by default. Check-all feature can be enabled in the lint config file.
Do not abort when code style does not match style rules - by defaut. Fail-when-offence feature can be enabled in the lint config file.
Add Hound config files, which makes possible to run popular service (Hound by thoughtbot)[https://houndci.com/] checks locally.

@matthiassiegel to check it
Add to your Gemfile: gem 'lint', github: 'momelnyk/lint'.
Run bundle.
Check code style by running: rake lint.
To use Hound code style rules, add lint config file to config/lint.rb, with following content.
config_file:
  scss: config/hound/scss.yml
  ruby: config/hound/ruby.yml
  coffeescript: config/hound/coffeescript.json

# Uncomment to check all files. By default lint checks modified and new files.
# check_all_files: true

# Uncomment to fail if code style does not match style rules. By default lint
# does not fail.
# fail_when_offence: true

Fixing bugs.


After the update UmbNav doesn't work

Which exact Umbraco version are you using? For example: 8.7.0 - don't just write v8 or v9
10.3.2
Bug summary
I made the latest update of "UmbNav" to version 2.0.3 and in the "Content Section" I no longer see the items I had uploaded. I cannot add or delete entries.
The naming of my menu items has changed to a generic "item".
For example: "legislation" has become "item 1".
In the frontend the entries are displayed correctly

Specifics
No response
Steps to reproduce
Create a menu and then update Umbnav to the version 2.0.3
Expected result / actual result
be able to edit, add, view items as before the upgrade

There are no differences between 2.0.2 and 2.0.3 other than a tag being added for the new Umbraco Marketplace.
What version did you upgrade from?
Have you tried clearing the temp folder and your browser caches?
Are there any errors in the browser console?
Can you please give me a step-by-step way to replicate this.
Thanks!

Thank you for your answer.
There are no errors in the browser console.
I cleared the cache but no change.
Before I had installed version 2.0.2.
It occurs to me now that I've also switched to .Net 7.0. I tried to go back to version 2.0.2 and .Net 6.0, but the anomaly remained.
The strangest thing is that I no longer see the "edit" and "delete" buttons.
Thanx

I haven't tested it with NET 7.0 or Umbraco 11 (yet)
As you can see here there is only 1 change between the 2 versions:
https://github.com/AaronSadlerUK/Our.Umbraco.UmbNav/compare/2.0.2...2.0.3
Are there any errors in the log?
I would need something to go on to be able to replicate this and investigate, if you're on Slack we could discuss this privately if you wanted to share any screenshots / code?

Thanx.
I contact you by Slack

This has been fixed in version 2.0.4 which is now available on NuGet
https://www.nuget.org/packages/Our.Umbraco.UmbNav.Web/2.0.4


extend with new currency (crypto)

Is it possible to extend the number of currency supported via code?

You mean you want to add a missing currency? If so, we gladly accept PR's to currencies.json, and localeFormats.json :smile:

I would like to add missing currency for cryptos. Is there a way to add it
only for me? I don't think everyone would be interested.
On Fri, Apr 13, 2018 at 5:27 PM Edo Rivai<EMAIL_ADDRESS>wrote:

You mean you want to add a missing currency? If so, we gladly accept PR's
to currencies.json
https://github.com/smirzaei/currency-formatter/blob/master/currencies.json,
and localeFormats.json
https://github.com/smirzaei/currency-formatter/blob/master/localeFormats.json
😄
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/smirzaei/currency-formatter/issues/56#issuecomment-381171592,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAI69oTbuLawHD7Gpkm3bSlq9impqcH5ks5toMPMgaJpZM4TTXU4
.


Save and Load a Keras NLP Tuned model

Hi,
I was following this tutorial
Link : https://colab.research.google.com/github/google/generative-ai-docs/blob/main/site/en/gemma/docs/lora_tuning.ipynb
Could please tell me as to how to save a fine tuned model to disk and load it back from Keras NLP?

gemma_lm.save_weights("/dir/model.weights.h5")
gemma_lm.preprocessor.tokenizer.save_assets("dir")


Actionable Message in Draft mode not working

Hello,
I am trying to create draft email messages having Actionable Message card as content, using Graph Api Client.
However the card content is not visible in draft message.
It is only visible when the email is actually sent.
Is this scenario not supported. Please guide.

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

ID: e36738b7-29cb-00c2-30e3-6065addee4c1
Version Independent ID: 57a934d1-7e50-a144-d461-fc55175acfb8
Content: Get started with actionable messages via email
Content Source: docs/actionable-messages/actionable-messages-via-email.md
Product: outlook
GitHub Login: @jasonjoh
Microsoft Alias: jasonjoh

@sonalinoolkar I believe that this is expected behavior. However, I'll check with the dev team and clarify.

I just got confirmation that cards do not render in draft messages by design. You can preview cards in the Message Card Playground site or Adaptive Card Designer site, or you can simply send the card to yourself and view it in your inbox.

@jasonjoh
The actionable message cards not even visible in Outlook Client 2016 as well. They are visible only in OWA.
Can you please confirm this as well?
Thanks,
Sonali


PublishPipelineArtifact and 'publish' keyword in YAML

Type: Question
Enter Task Name: PublishPipelineArtifact
Environment

Server - Azure Pipelines

Agent - Hosted MacOS (Mojave)

Issue Description
Documentation (https://docs.microsoft.com/en-us/azure/devops/pipelines/artifacts/pipeline-artifacts) says we can use 'publish' keyword in YAML as a shortcut for the task. When I tried that however, I was getting the following error when triggering the build:
Unexpected value 'publish'
Is documentation ahead of actual feature being deployed to Azure Pipelines?

Perhaps - it does roll out across rings.  Does it work for you now?

It works now, thanks


Typescript version available

Hi,
Thanks for your great contrib.
Just so you know, I created a typescript version of your lib.
Also added support for typescript Map objects.
Also added a "quickdiff" treshold feature for comparing large objects.
https://github.com/anwfr/object-difference-ts
Best regards

Thanks! I'll add a link in the README. And sorry I took so long to answer, I was on vacations.
I haven’t updated this project in a while since I feel that I hit a hard wall with the problem of notifying a difference in the order of the items in an array. The idea with arrays was to approximate React’s reconciliation heuristics, which is why the key property is needed for objects in arrays, but I have no idea how to reliably and easily reflect order changes in the result. Did that problem arise in your use cases?

hi, you're welcome.
I didn't need to detect reordering the items in an array for my use case, so I didn't look at the problem, but it's an interesting question


Unable to get latest version of cli (or uninstall current one)

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
6.14.13
Amplify CLI Version
4.43 in use as current version / 5.2.0 installed
What operating system are you using?
Mac
Amplify Categories
Not applicable
Amplify Commands
pull
Describe the bug
When creating a react app with AWS amplify using tutorial (https://aws.amazon.com/getting-started/hands-on/build-react-app-amplify-graphql/module-two/), I discovered I was unable to update older version of the amplify CLI installed through npm as part of a 3rd party tutorial in March 2021.
I tried both npm i -g @aws-amplify/cli and then npm i -g @aws-amplify/cli@latest. The install process showed as successful for v5.2.0, but when querying local version, this is still showing as v4.4.3. I also tried npm uninstall -g @aws-amplify/cli to reset, but this did not work to remove my current instance (i.e. 'amplify -v' shows 4.4.3, and amplify configure' executes even after supposed uninstall).
Appreciate guidance on reset and update of current CLI version.
Expected behavior
amplify -v shows current version (v5.x) after current version has been installed via npm i -g @aws-amplify/cli@latest
npm uninstall -g @aws-amplify/cli results in amplify no longer being installed on desktop
Reproduction steps
1a. npm i -g @aws-amplify/cli@latest
1b. amplify -v
2a. npm uninstall -g @aws-amplify/cli
2b. amplify -v
GraphQL schema(s)

# Put schemas below this line

Log output

Successfully installed the Amplify CLI
....
Learn more - https://docs.amplify.aws/cli/reference/usage-data

<EMAIL_ADDRESS>updated 18 packages in 35.63s

amplify -v
4.43.0
Additional information
No response

Interesting - this always works for me.
What is the result of running which amplify and npm root -g?

which amplify
/usr/local/bin/amplify
npm root -g
/Users//.nvm/versions/node/v12.22.3/lib/node_modules

That would be why. When you reinstall amplify via npm, it's installing the and updating binary in your npm root -g directory. Unless /usr/local/bin/amplify is a properly configured symlink, your npm commands will have no effect.
You could try removing /usr/local/bin/amplify and making sure your npm root is in your $PATH

Got it, I am set with latest version now. Thanks @johnpc, this was extremely helpful!


Plugin execution order

It would appear that global plugins are not called in an expected order.
For example;
bottle.install(DatabasePlugin())
bottle.install(PluginThatNeedsDatabase())

The above would first call PluginThatNeedsDatabase()and then DatabasePlugin().
This goes against the advice seen in documentation;

The order of installed plugins is significant, though. If a plugin requires a database connection, you need to install the database plugin first.

As you can see from our example though, the database plugin cannot be installed first, otherwise dependant plugins will not work correctly.
You can see a full demonstration of this in action here.
I would argue that the execution order of app plugins and view plugins should be the same, but as you can see from this line, this is not the case. There are also backwards compatibility concerns with such a fix, given that it will almost certainly break every app in existence with plugin dependancies.
Thoughts @defnull ?

Another idea would be to add the concept of "plugin dependancies", which is then used to determine the preferred execution order of plugins. This would allow plugins to satisfy their dependancies, without breaking backwards compatibility.
For example;
class Plugin1(object):
    name = 'plugin1'

class Plugin2(object):
    name = 'plugin2'

class Plugin3(object);
    name = 'plugin3'
    depends = ['plugin1', 'plugin2']

app.install(Plugin3())
app.install(Plugin1())
app.install(Plugin2())

Execution chain would be Plugin1(), Plugin2(), Plugin3().
I still feel that the execution order should be consistent, however this might be one of those situations where a "bug" becomes a feature.

After much trial and error, it would seem that most simple option for now is to provide an optional flag that  changes the plugin execution order. The default option could keep current functionality, with a view towards changing the default on a later revision (assuming that this issue is accepted).
Something like;
app = Bottle(plugin_order_reversed=True)

Naturally it will need a better attribute name. Thoughts?

In the mean time, here's a small mixin that implements this functionality. Happy to throw together a proper PR if this is accepted;
class RouteMixin(object):
    """Allow plugins to be executed with dependancy chain"""

    def all_plugins(self):
        plugins = list(super(RouteMixin, self).all_plugins())
        if not self.app.plugins_reversed:
            app_plugins = list(reversed([ p for p in plugins if p in self.app.plugins ]))
            view_plugins = [ p for p in plugins if p in self.plugins ]
            plugins = app_plugins + view_plugins
        return plugins

class RouterMixin(object):
    def match(self, *args, **kwargs):
        route, args = super(RouterMixin, self).match(*args, **kwargs)
        extend_instance(route, RouteMixin)
        return route, args

class BottleReversePluginMixin(object):
    def __init__(self, *args, **kwargs):
        super(BottleReversePluginMixin, self).__init__(*args, **kwargs)
        self.plugins_reversed = kwargs.pop('plugins_reversed', False)

class Bottle(BottleReversePluginMixin, bottle.Bottle):
    pass

Also there's another problem, application wide plugins are executed /after/ view specific plugins. Again this is surprising behaviour if you have a plugin on the view which relies on a plugin in the app.

After even more trial and error, I was able to resolve my original problem by refactoring my plugins. I'm going to close this out, but if anyone else feels it needs attention then feel free to re-open.

Plugin order should work like this: If you install two Plugins P1 and P2 to the application and two Plugins P3 and P4 to the route (in that order), the flow graph for each request looks like this:
request  -> P1 -> P2 -> P3 -> P4 \
                                  handler()
response <- P1 <- P2 <- P3 <- P4 /

Each plugin can do stuff before it calls into the next plugin (or the handler in case of P4) and after it gets control back from the next plugin. If P1 is a database plugin, it creates the database connection before it calls into P2 and closes it after P2 returns control.
So, if it does not work like this, it is indeed a bug. But it is tested here: https://github.com/bottlepy/bottle/blob/master/test/test_plugins.py#L88

Thanks for the clarification @defnull. I can confirm that this is indeed working, and after refactoring my plugins I was able to get around the problem.
I'm wondering if it's worth putting your above comment into the plugin docs page, as that made it quite clear, thoughts?


Image is loaded twice

Reproduction
https://stackblitz.com/edit/nuxt-3-xqdc5j?file=app.vue
Describe the bug
Hey,
I maybe found another bug, but I am not sure why it is happening. Is hapens with src and srcset attribute. Also tested it on localhost and on a nuxt build. The image is loaded twice. Also the image is only shown, when the image is loaded the second time.
You can test this, when you go into the network panel and selected the img filter.

Thanks for your help in advance again.
Best regards
Chris
Additional context
No response
Logs
No response

Of course this is only happening, when you disable the cache. When you load it completely fresh withouth the disable cache function activated, I think it eliminates the second load, because it is already in memory cache. So maybe this isn't a bug and has no effect on the loading time of the page/images.

Thanks for the investigation.
Correct, this is the intended behavior. The image placeholder should be displayed as long as possible. The target image is loaded with new Image() to listen for the load event. Only then will the placeholder image be replaced with the actual image.
This prevents empty (image) areas from being displayed while an image is being loaded.


Operating System - add rename file keyword

New keyword with arguments current_filename, and new_filename
Could also create an additional keyword to allow renaming files using regexp

How would this be different than the current Move File?

My apologies.  I just verified that Move File does the same as rename.  Closing