Blob Storage - Remove regex pattern constraint from blob name parameter

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[x] Other, please clarify

Remove over-restrictive parameter constraint. Blob service documentation says that "A blob name can contain any combination of characters". The removed constraint restricts blob names to alphanumeric characters with some token punctuation.

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.

N/A

When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.

N/A (I think?)

If updating an existing version, please select the specific langauge SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

I'm unsure of how to answer the above question. Since this change removes a constraint, should all the SDKs be regenerated?
Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.

N/A - I do not have permissions to access the linked page.

[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

N/A - I do not have permissions to access the linked page. I assume CI will catch validation errors.

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that label “WaitForARMFeedback” will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

Hmm, I'm not sure how removing a constraint constitutes a breaking change. I cannot access any of the links in the bot's comment to the breaking change policy or anything else.
@anuchandy, can you please let me know how to proceed from here? Thank you.

I don't think I have permission to remove the no-recent-activity label 😕
@anuchandy @seanmcc-msft @kasobol-msft  Could one of you please take a look at this and let me know how to proceed? Thank you!

Commenting to mark as unstale.
@anuchandy @seanmcc-msft @kasobol-msft Could one of you please take a look at this and let me know how to proceed? Thank you in advance!

Commenting to mark as unstale.

I have to say disappointed by the complete lack of maintainer attention to a good-faith PR on this open-source repo. A little bit unprofessional :/
Oh well...

@okofish / Jesse, I am not sure if removing constraints from the blobname is the right thing to do. Can you please provide a justification for this change? @seanmcc-msft to look when he is back.

Hi @amishra-dev, thanks for your response. My justification is that the blob name constraint is too strict. The current regex admits only alphanumeric characters, and only blob names following a strict filename-like syntax. It does not match, for example, $FooBar or abc/def.tar.gz, both of which are valid blob names.
The blob service documentation says that "A blob name can contain any combination of characters".

Commenting to mark as unstale.

Commenting to mark as unstale.

Commenting to mark as unstale.

Commenting to mark as unstale.

Commenting to mark as unstale.

Commenting to mark as unstale.

Commenting to mark as unstale.

Commenting to mark as unstale.

Commenting to mark as unstale.

Commenting to mark as unstale.

Commenting to mark as unstale.

Commenting to mark as unstale.


reviewerfaq: suggested explanation

Thanks so much, this is really helpful! (I was about to do the same thing myself when I checked this PR)


Questions about Schedule object

I tried to show notification every saturday.
To test repeating notification, I scheduled notification like this(targeting time is : '2021-05-01 23:30:30+09:00')
AwesomeNotifications().createNotification(
  content: NotificationContent(
      id: 1,
      channelKey: defaultChannelKey,
      title: 'title',
      body: 'body'
  ),
  schedule: NotificationCalendar(
    weekday: 6,
    hour: 23,
    minute: 30,
    second: 30,
    repeats: true
  )
);

and it's not worked.
so i kept trying with different parameters, and i found exact parameters that work correctly
  schedule: NotificationCalendar(
    weekday: 7,
    hour: 14,
    minute: 30,
    second: 30,
    repeats: true
  )

i wonder,

those parameters are calculated as UTC+0, so i had to change hour from 23 to 14 (here is UTC+9). i hope there's a setting for timezone.
if i pass weekday: 0 it throws error. and saturday is 7, not 6. is this intended?

ps. how about changing the parameter 'weekday' to 'dayOfWeek'?

those parameters are calculated as UTC+0, so i had to change hour from 23 to 14 (here is UTC+9). i hope there's a setting for timezone.

It is a good pratice to use UTC+0 in storages and for internal procedures. Time zones are political and very complex to deal. Take a look at this video to see where the rabbit hole ends:
The Problem with Time & Timezones - Computerphile
To easily translate your local time into utc, or vice versa, use this in dart:
print(DateTime().toUtc());
print(DateTime().toLocal());

if i pass weekday: 0 it throws error. and saturday is 7, not 6. is this intended?
ps. how about changing the parameter 'weekday' to 'dayOfWeek'?

I only copied the method names from NSCalendar and Calendar, from Swift and Java libraries, respectively. Those date values usually starts from 1. But ill check if that name is better to understand and put it in the next release, if necessary.
NSCalendar Weekday

I have same issue about weekday.
Flutter dateTime has weekday that start 1 for monday. But this plugin weekday is not.
You can see a document here.

For both java and swift, the day 1 is sunday. Notifications work based on those 2 languages and will be scheduled on OS following that rule.
I gonna need to force you guys to not use dart DateTime helpers, changing the model objects to demand by code the correct use of those values.

For both java and swift, the day 1 is sunday. Notifications work based on those 2 languages and will be scheduled on OS following that rule.
I gonna need to force you guys to not use dart DateTime helpers, changing the model objects to demand by code the correct use of those values.
Or i gonna need to convert those values on native level

In my opinion, it would be better to receive input from the user according to the dart and convert it in or before passing it to the native.

I gonna need to change it in native for each platform, because the notification could arive from a push notification with the same issue.

New version released with monday = 1. Please check it for me if the problem was solved.

It works fine on Android. Thanks for your awesome plugin!

It works good! Thanks a lot!


why not indicate the version of visual studio used in this page??

I have tried vs2013 vs2015 vs2019, all is not work, you can't find Office/SharePoint node

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

ID: 3f7527e2-b81b-187a-3511-28c27db71cca
Version Independent ID: 81c544e4-3751-6bdc-40f6-6a5ba55f5e32
Content: Office project templates overview - Visual Studio
Content Source: docs/vsto/office-project-templates-overview.md
Product: visual-studio-windows
Technology: office-development
GitHub Login: @John-Hart
Microsoft Alias: johnhart

@yunfeng2003 for the best support please send feedback by accessing the Help > Send Feedback menu in Visual Studio 2019


🐛 Bug Report: the definition file return for the attributes property, a list of string for the reponse of database.listAttributes in the sdk

👟 Reproduction steps
When I
import * as sdk from "https://deno.land/x/appwrite/mod.ts";

// Init SDK
let client = new sdk.Client();

let database = new sdk.Database(client);

client
    .setEndpoint('https://[HOSTNAME_OR_IP]/v1') // Your API Endpoint
    .setProject('5df5acd0d48c2') // Your project ID
    .setKey('919c2d18fb5d4...a2ae413da83346ad2') // Your secret API key
;

let promise = database.listAttributes('[COLLECTION_ID]');

promise.then(function (response) {
    console.log(response);
}, function (error) {
    console.log(error);
});

👍 Expected behavior
I expected I have at least a object with the key in the definition
let {total, attributes } = database.listAttributes('[COLLECTION_ID]');
where
attributes => { key:string }[]
and even It should be
{ key:string
| attributeBoolean?
| attributeInteger?
| attributeFloat?
| attributeEmail?
| attributeEnum?
| attributeUrl?
| attributeIp?
| attributeString?}[]

👎 Actual Behavior
It actually
let {total, attributes } = database.listAttributes('[COLLECTION_ID]');
I have
attributes => string[]
🎲 Appwrite version
Version 0.10.x
🎲 Deno version
Version 1.14.x
💻 Operating system
MacOS
🧱 Your Environment
No response
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

PR merged, released 🥳 Feel free to re-open the issue if you still face the problem.


Printing digits twice on a single keyboard press.

Hello @DhanushNehru
On a single number key press the digit is printed twice on the screen. I have fixed the issue.
Please assign this issue to me.
Thank you 😄

@DhanushNehru , is this still open?

It is resolved @CodeVibranium

@DhanushNehru , please let me know if there are issues to fix.

You can check the issues tab @CodeVibranium


Error: ConnectFailed when attempting to communicate with a MB8600

Hi.  I have an MB8600 (<IP_ADDRESS>).  Attempts to run moto_arris_cm_stats results in a ConnectFailed error.  I've verified that I can manually log into the modem's admin UI using a web browser.
Any ideas how to proceed?

If GitHub notified me of this, then I completely missed it. Sorry! Did you manage to get this working? I know at one point, they switched my firmware to only use HTTPS. I'm wondering if perhaps yours might still be using HTTP.

Hi.  I never got your particular app working on my MB8600 but it gave me some hints about how to proceed.  Combined with some time spent with the Firefox debugger, I was eventually able to throw together a Python script that successfully logs in and extracts the stats.  My modem still uses HTTP so it's good to know that my ISP might push an update to only allow HTTPS.
One thing that's different between my code and yours is your appears to be able to access the HNAPs without logging-in.  I'm could've swore that failed when I tried something similar.

Ah interesting. I guess each ISP is going to have variations in their firmware. As for the HNAPs: I initially was passing headers for auth, but then noticed that it didn't seem to matter in my case. So, I started trimming away headers until it stopped working. It turns out that with Comcast, you seem to barely need any.
In any case, as of recently, I'm no longer using that modem at all, so I suspect that that code path may rot unless I've got some secret users out there.

I should have referenced this issue in the commit, but I've added a flag to disable SSL in: 0b4414a51857d8ef98b5522a652f9a53a9637c41


Docs build is failing because of multiple versions of bitcoin_hashes

We have this warning:
warning: output filename collision.
The lib target `bitcoin_hashes` in package `bitcoin_hashes v0.15.0 (/home/runner/work/rust-bitcoin/rust-bitcoin/hashes)` has the same output filename as the lib target `bitcoin_hashes` in package `bitcoin_hashes v0.14.0`.
Colliding filename is: /home/runner/work/rust-bitcoin/rust-bitcoin/target/doc/bitcoin_hashes/index.html
The targets should have unique names.
This is a known bug where multiple crates with the same name use
the same path; see <https://github.com/rust-lang/cargo/issues/6[313](https://github.com/rust-bitcoin/rust-bitcoin/actions/runs/11376855348/job/31650018128?pr=3467#step:6:314)>.

And this error:
error: "/home/runner/work/rust-bitcoin/rust-bitcoin/target/doc/bitcoin_hashes/all.html": No such file or directory (os error 2)

But I'm not sure why we get the error.

@tcharding we have seen this error before and the solution was to add -j1 somewhere to make sure that we were only building one doc instance at a time. Maybe we dropped that when we migrated to rust-bitcoin-maintainer-tools?
I recall this error triggered very reliably in CI, but not at all locally.

Posted elsewhere, we probably need to use cargo -j2 in run_task.

docsrs job on cecent PRs is not failing anymore, win.


Transparent Ciphertext

Hello,
I am aware there have been issues reported on transparent ciphertexts, including #179 and #137. I also get the error of libc++abi.dylib: terminating with uncaught exception of type std::logic_error: result ciphertext is transparent when I do,
double p = 0.0046;
ckks_encoder.encode(p, ptext);
evaluator.mod_switch_to_inplace(ptext, ctext.parms_id());
evaluator.multiply_plain_inplace(ctext, ptext);

To avoid, I included two ifs in SEAL/native/src/seal/evaluator.cpp file like following:
void Evaluator::multiply_plain_inplace(Ciphertext &encrypted, const Plaintext &plain, MemoryPoolHandle pool)
{
        (...)
        if (encrypted.is_transparent())
        {
            continue;
        }
        if (plain.is_transparent())
        {
            continue;
        }
        // these two ifs are what I added

        if (encrypted.is_ntt_form())
        {
            multiply_plain_ntt(encrypted, plain);
            // Could not understand what ntt is.
        }
        else
        {
            multiply_plain_normal(encrypted, plain, move(pool));
        }

// I've also tried disabling the below #ifdef part, but same error occurs.
#ifdef SEAL_THROW_ON_TRANSPARENT_CIPHERTEX
        // Transparent ciphertext output is not allowed.
        if (encrypted.is_transparent())
        {
            throw logic_error("result ciphertext is transparent");
        }
#endif
}

However, the error is not reconciled.
FYI, my SEAL version is 3.5.6 (master), Apple clang version is 11.0.3, and cmake version is 3.13.4.
Does anyone know how to fix this problem?
Thank you!
hdk

There is a bug in your code: you are calling the incorrect overload of CKKSEncoder::encode for encoding a single integer. Your double is cast to 0 so the plaintext is identically zero. multiply_plain with an identically zero plaintext doesn't make sense and is not allowed, which is what the transparent ciphertext guard correctly catches.
I've made this mistake too. It would probably be a good idea to rename the CKKSEncoder::encode overload for a single integer to something else to avoid issues like this. The difference between the single floating-point and the single integer versions is that the single-integer version doesn't require a scale to be given (it uses scale 1.0).

Thanks Kim!! This is really helpful. All solved.

Hello! I was trying to solve the exact same problem, I checked if something had changed in the ckks.h since what you said about the "calling the incorrect overload" however I found that calling CKKSEncoder::encode was still in such an overload. I was wondering if there is a way to specify which type you're using or how exactly was this solved? Initially I thought it was missing a double scale variable...  but then I tried that, ckks_encoder.encode(p, scale, ptext); to try to call the
inline void encode(
double value, double scale, Plaintext &destination, MemoryPoolHandle pool = MemoryManager::GetPool())
{
encode(value, context_.first_parms_id(), scale, destination, std::move(pool));
}
Is there something I'm missing?


Update ryujinx-dev.json - simple typo correction

Simple typo correction.

The pull request has been merged in the one by BlackPowerade - closing.


[error] decimal.DivisionByZero

Hi, thank you for this tool - using it for some months, but now I have encountered the following error:
Traceback (most recent call last):   File "/Users/silviu/apple-slicer/./slicer.py", line 268, in <module>     currency_data = parse_currency_data(args.directory + '/' + currency_data_filename)   File "/Users/silviu/apple-slicer/./slicer.py", line 143, in parse_currency_data     tax_factor = Decimal(1.0) - abs(tax / amount_pre_tax) decimal.DivisionByZero: [<class 'decimal.DivisionByZero'>]
Thank you!

You'd need to provide your input file in order for me to debug that issue. Feel free to obfuscate / change anything that could identify your app – but please leave the numerical price values intact.

Thanks for the fast reply @fedoco
Before sending you files, can the error be triggered by this line? I saw it was marked in red and maybe, maybe.

Thanks!

Yeah, it looks like it's the culprit.

"Brazil (BRL)","0","0.00","0.00","0","0","-12.74","-12.74","0.19231","-2.45","USD",

Nope.

(The number of columns is actually correct – my bad, sorry.)
Okay, it looks like the issue here is that you have to give back earnings to Apple because a Brazilian customer got a refund which didn't get balanced out by other earnings in Brazil. It's kind of amazing that this hasn't occurred in all the years the apple-slicer script exists. I'll fix it asap for you to test, probably later today. Thanks for reporting it!

Oh, and could you please tell me if there is a file ending with _BR.txt in your App Store Connect downloads for the affected month?

Yes, it is 87727887_1022_BR_WHT.txt

I'm sorry, I need the content of that file, too. Please obfuscate any identifying strings and the "Apple Identifier" (column 11) if you want to.

dld
PS: tell me when you have it. thx

"Unavailable. The owner stopped sharing, or you don't have permission to open it."

https://drive.google.com/file/d/1G8_dEyMtzq39J1jfDMEhe24YYs7yqOtB/view?usp=sharing

Peak Apple. Grabbed it, thanks!

Very odd. That isn't a typical "Payments & Financial Reports" file. Where did you download it? Was it really included with the other *.txt in the .zip?

Very odd. That isn't a typical "Payments & Financial Reports" file. Where did you download it? Was it really included with the other *.txt in the .zip?

The files in .zip:

Hmm, that is really quite strange as WHT information is normally part of what you can download with Apple's reporter Java tool or https://github.com/fedoco/itc-reporter. Anyway, I'll have a look into it and report back.

Thanks, I don't think that will be necessary. One last question, though: Can you please search for the string BRL in the file 87727887_1022_ZZ.txt and show me that whole line if there is a match?

Thanks, I don't think that will be necessary. One last question, though: Can you please search for the string BRL in the file 87727887_1022_ZZ.txt and show me that whole line if there is a match?

There is no match. Thanks!

That's a bummer because it means that there isn't enough context in the files provided by Apple in order to apply a clean fix for that issue. Nevertheless, I of course updated the script so that it doesn't crash anymore and instead issues a warning with a note on what needs to be done manually. Please give it a test run.
By the way, I am pretty sure that I was wrong in my original analysis of the problem. It is simply not recognisable from the files why Apple withheld tax for Brazil without listing associated sales.

That's a bummer because it means that there isn't enough context in the files provided by Apple in order to apply a clean fix for that issue. Nevertheless, I of course updated the script so that it doesn't crash anymore and instead issues a warning with a note on what needs to be done manually. Please give it a test run.
By the way, I am pretty sure that I was wrong in my original analysis of the problem. It is simply not recognisable from the files why Apple withheld tax for Brazil without listing associated sales.

Thanks for the quick fix @fedoco, I appreciate it!
The single issue for me now is what to do with that amount further for accountability..

Well, thank you for your help tracking down the issue!
As for the bookkeeping, the taxes reduce your revenue. Therefore I would simply include an invoice item of -2,45 € labelled "Taxes withheld" in the invoice to Apple LATAM and adjust its total accordingly.

No problem; it was my pleasure to collaborate with you and chase this edge case.
I hope to can contribute soon to the project somehow.
Also, thanks for the bookkeeping explanation; I was thinking the same but was unsure if it would be ok.


Release?

It's been some months since the last release, and some nice commits have been made since then. It would be great if we could get an updated snapshot.

A release would be very useful, as sheerun/vim-polyglot#779 points out. It imported new mainline vim files (specifically ft.vim at vim/vim@a2baa73d1d33014adea0fd9567949089ca21a782) and it fails to run in the currently released version of MacVim 8.2 (patch 3455). I have a fix in at sheerun/vim-polyglot#780, but it is not clear to me whether this will be accepted as it is changing an upstream file.

Sorry I have been away from MacVim development for a while which is not great. Let me catch up to see if there are glaring issues, and if not, make a release soon.

As I have started to use Vim9 scripts, MacVim has started to crash because of some bugs in Vim that have been fixed only recently. So, I am stuck using Vim in the terminal. I would really appreciate if a new MacVim release could be pushed forward!

It was supposed to happen over the weekend. Maybe there were issues.

@ychin is there anything we can help you with?

A bunch of runtime files were merged so maybe that is an indication a new version will drop "soon".

Vim9 is now out. Will MacVim update to it anytime soon?

I would hope that a final release of MacVim with 8.2 would come out, and then work start on Vim 9 support.

Hi, I'm so sorry for the radio silence in recent months. life has been catching up to me and I was struggling to find time for MacVim, which is something I'm trying to correct and doesn't excuse the hold on the release.
One thing I'm trying to do is to do a more thorough validation of making sure one year's worth of Vim updates didn't break anything, but that's probably misguided compared to just pushing out a release. Can people who have been syncing and building MacVim give me a basic thumbs up/down on whether they see any glaring bugs that need to be addressed before I push out a Vi 9 release?

Thanks you @ychin! I built the latest master and gave it a spin. I noticed no issues. 👍

I did a manual compile as of 286bd68d59d940dab45dbe9afe4dd77fa8810973 and so far so good. No glaring issues that I can see.

@ychin we'd be happy to test!
Manual tests seems to be working fine ;) push the pre-release if you worry about any additional issues

Compiled it. I haven't found any issues so far for my normal usage.

Built the current master. I confirm that I don't get the crashes I used to get with Vim 9 script. Thanks!

@ychin I know this is probably annoying, but is there any ETA on when the vim9 release will be out?
Is there anything we can help with?

Sorry I got distracted a little trying to figure out why the colors looked so weird, and turned out Vim just had a large colorscheme update a couple months ago where they changed the default colorschemes which sidetracked me a little, as I thought there was some rendering issue. One of those things where a regular cadence would have really helped identify where these issues come in instead of having to bisect around.
I just pushed a tag to https://github.com/macvim-dev/macvim/releases/tag/snapshot-173, doing some final touches and then should be able to release it in a couple hours. Don't want to beat a dead horse but I know this infrequent release cadence and lack of activity is not ideal at all. I'm trying to catch up to the other stuff like outstanding bugs and PRs after the release.

I got distracted a little trying to figure out why the colors looked so weird

In principle, the built-in color schemes should look better, especially in the GUI. If you think that they have some issue, could you please provide some feedback?
Btw, thanks for the huge effort you put in this project!

Thank you so much. I'm just a writer, not a programmer, but I live in this program all day every day. THANKS
I'll wait for the brew version.

Thank you for your work shipping new MacVim releases. Very appreciated :)


Fix HorizontalExecutionMerging Complexity

Description
Fixes #599.
Requirements
Before submitting this PR, please make sure:

[x] You have run the code checks, tests and documentation build successfully
[x] All fixes and all new functionality are tested and documentation is up to date
[x] You looked at the review checklist

Additionally, if this PR contains code authored by new contributors:

[x] All the authors are covered by a valid contributor assignment agreement,
signed by the employer if needed, provided to ETH Zurich
[x] The names of all the new contributors have been added to an updated
version of the AUTHORS.rst file included in the PR

bors try

Allow me to sneak that last commit in here. The test I added should have been part of #586, but it would be broken by this PR.
Otherwise, looks good.

bors try


ModuleNotFoundError: No module named 'tensorflow.contrib' when I use Tensorflow GPU processing

System information

Have I written custom code:  No
OS Platform and Distribution: Windows 10
TensorFlow backend: yes
TensorFlow version:  2.0.0
Keras version:  2.3.1
Python version:  3.7.4
CUDA/cuDNN version: 10.1/<IP_ADDRESS>
GPU model and memory:  Nvidia GeForce GTX 1070 8 GB

I am experiencing ModuleNotFoundError: No module named 'tensorflow.contrib' while executing from tensorflow.contrib.cudnn_rnn.python.ops import cudnn_rnn_ops command in the keras\layers\cudnn_recurrent.py, line 425. This issue is specific to Tensorflow when using GPU processing. No issues at all if I do not use GPU processing.
Code to reproduce the issue
The only line changed in the code in order to benefit from GPU processing is importing keras.layers.CuDNNLSTM instead of keras.layers.LSTM. The CPU-version of the same code works as it is expected.
Stacktrace
Using TensorFlow backend.
2019-11-25 02:44:47.169716: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library cudart64_100.dll
2019-11-25 02:44:54.145607: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library nvcuda.dll
2019-11-25 02:44:54.165862: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1618] Found device 0 with properties:
name: GeForce GTX 1070 major: 6 minor: 1 memoryClockRate(GHz): 1.645
pciBusID: 0000:01:00.0
2019-11-25 02:44:54.166083: I tensorflow/stream_executor/platform/default/dlopen_checker_stub.cc:25] GPU libraries are statically linked, skip dlopen check.
2019-11-25 02:44:54.166651: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1746] Adding visible gpu devices: 0
2019-11-25 02:44:54.167105: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2
2019-11-25 02:44:54.169843: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1618] Found device 0 with properties:
name: GeForce GTX 1070 major: 6 minor: 1 memoryClockRate(GHz): 1.645
pciBusID: 0000:01:00.0
2019-11-25 02:44:54.170061: I tensorflow/stream_executor/platform/default/dlopen_checker_stub.cc:25] GPU libraries are statically linked, skip dlopen check.
2019-11-25 02:44:54.170602: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1746] Adding visible gpu devices: 0
2019-11-25 02:44:54.804769: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1159] Device interconnect StreamExecutor with strength 1 edge matrix:
2019-11-25 02:44:54.804977: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1165]      0
2019-11-25 02:44:54.805105: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1178] 0:   N
2019-11-25 02:44:54.805957: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1304] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 6372 MB memory) -> physical GPU (device: 0, name: GeForce GTX 1070, pci bus id: 0000:01:00.0, compute capability: 6.1)
Traceback (most recent call last):
  File "C:\Users\talha\AppData\Local\Programs\Python\Python37\Lib\contextlib.py", line 130, in __exit__
    self.gen.throw(type, value, traceback)
  File "D:\.virtualenvs\signal-proc-keras-gpu-8KfKslfY\lib\site-packages\tensorflow_core\python\framework\func_graph.py", line 404, in inner_cm
    yield g
  File "D:\.virtualenvs\signal-proc-keras-gpu-8KfKslfY\lib\site-packages\keras\backend\tensorflow_backend.py", line 75, in symbolic_fn_wrapper
    return func(*args, **kwargs)
  File "D:\.virtualenvs\signal-proc-keras-gpu-8KfKslfY\lib\site-packages\keras\engine\base_layer.py", line 463, in __call__
    self.build(unpack_singleton(input_shapes))
  File "D:\.virtualenvs\signal-proc-keras-gpu-8KfKslfY\lib\site-packages\keras\layers\cudnn_recurrent.py", line 425, in build
    from tensorflow.contrib.cudnn_rnn.python.ops import cudnn_rnn_ops
ModuleNotFoundError: No module named 'tensorflow.contrib'

Same issue for me. Here is a reproducing code:
import keras as ks
inp = ks.layers.Input((10, 1))
x = ks.layers.CuDNNLSTM(1)(inp)

which throws the above error.

Had the same Issue. I could solve it (for now) by using
tensorflow.compat.v1.keras.layers.CuDNNLSTM instead of keras.layers.CuDNNLSTM.

Hi,
keras.layers.CuDNNLSTM did not work out at all for me using keras 2.3.1. I did not find any way to make it work. Also the proposed solution by switching to tensorflow.compat.v1.keras.layers.CuDNNLSTM neither worked out for me.
I found the following way to solve this problem and to have my data trained with GPU-acceleration (Windows 10, CUDA 10.1):

Use at least tensorflow 2.1.0.
Change your imports from from keras import something to from tensorflow.keras import something. This seems to be the preffered way as stated in the 2.3.0 release notes [0].
As it appears, the CuDNNLSTM has been merged into LSTM in Tensorflow 2.1.0. According to the documentation [1] the GPU-accelerated version is executed automatically. Quoting from [1]:

Based on available runtime hardware and constraints, this layer will choose different implementations (cuDNN-based or pure-TensorFlow) to maximize the performance. If a GPU is available and all the arguments to the layer meet the requirement of the CuDNN kernel (see below for details), the layer will use a fast cuDNN implementation.

The requirements to use the cuDNN implementation are:

    activation == tanh
    recurrent_activation == sigmoid
    recurrent_dropout == 0
    unroll is False
    use_bias is True
    Inputs are not masked or strictly right padded.

By doing so, I can successfully train the LSTM on my GPU, as the CuDNNLSTM is automatically executed through LSTM().
Good luck to you!
[0] https://github.com/keras-team/keras/releases/tag/2.3.0
[1] https://www.tensorflow.org/api_docs/python/tf/keras/layers/LSTM

@jollyjonson's solution worked for me on keras 2.3.1 and tensorflow 2.1.0. Anyway, @jliebers's solution also works in my code and seems to be the way it should be done in keras 2.3. Thanks to both of you!

so, is there a solution how to use CuDNNLSTM?

so, is there a solution how to use CuDNNLSTM?

As stated by @jliebers:

Hi,
keras.layers.CuDNNLSTM did not work out at all for me using keras 2.3.1. I did not find any way to make it work. Also the proposed solution by switching to tensorflow.compat.v1.keras.layers.CuDNNLSTM neither worked out for me.
I found the following way to solve this problem and to have my data trained with GPU-acceleration (Windows 10, CUDA 10.1):

Use at least tensorflow 2.1.0.
Change your imports from from keras import something to from tensorflow.keras import something. This seems to be the preffered way as stated in the 2.3.0 release notes [0].
As it appears, the CuDNNLSTM has been merged into LSTM in Tensorflow 2.1.0. According to the documentation [1] the GPU-accelerated version is executed automatically. Quoting from [1]:

Based on available runtime hardware and constraints, this layer will choose different implementations (cuDNN-based or pure-TensorFlow) to maximize the performance. If a GPU is available and all the arguments to the layer meet the requirement of the CuDNN kernel (see below for details), the layer will use a fast cuDNN implementation.

The requirements to use the cuDNN implementation are:

    activation == tanh
    recurrent_activation == sigmoid
    recurrent_dropout == 0
    unroll is False
    use_bias is True
    Inputs are not masked or strictly right padded.

By doing so, I can successfully train the LSTM on my GPU, as the CuDNNLSTM is automatically executed through LSTM().
Good luck to you!
[0] https://github.com/keras-team/keras/releases/tag/2.3.0
[1] https://www.tensorflow.org/api_docs/python/tf/keras/layers/LSTM

Just use LSTM. If you tried this and found no GPU usage in your machine, check that you have tensorflow-gpu installed instead of tensorflow.

Hey,
so in the meantime I migrated all my code to use tf.keras (2.1) and not Keras (3.2.1) anymore, and I can confirm, that the GPU-accelerated LSTMs are executed correctly when sticking to the parameters I quoted from the documentation above.
But please be aware that there are massive issues when executing them on a Windows operating system (see my other issues in tensorflow). They work best on Linux by far.
Cheers,
Jonathan

I had the same problem using Keras 2.3.1.
When I switched, like it was written here, to:
from tensorflow.keras import ...
instead of:
from keras import ...
it worked using the GRU or the LSTM layers.
I used tensorflow-gpu 2.2.0


Amsolo sql docs local

Migration Assets section is updated

@ms-amsolo : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@aciortea adding Alexandra to this PR about heterogeneous migration assets.

Thanks @mokabiru for approving the changes.
@cawrites Could you please review and approve this PR?


Crash Report 283625784

Link:
https://wowsims.github.io/wotlk/deathknight/#eJztVM9rE0EY3fe5JtOxpaOnaWjKungokcDsjgEFYVf/goCXHlukItRDLIK0IEShUBS0B6lpFK+WnvxFjaFCIR7MQc0h7UE8SEFJ0WBuFj3Y2U1b8CaoN9/hm5nvve99w8ww/CgjQcMYQYFmgUXgIagCrAKvgAbQArZADaI5wmnhkEJoxKMoxuJDVmpm/zUk8hfGpsYnWVLw1DPwfrFUI7mZdhPLNXrzmniP2HpP7r6VCvFe0V4h+SHdXSVFtUryW9qUfDLppwNu4n6dIoaLR8skZ4ZcWLxPtAx5e9C1axWyjMV6lfaE/eKLIb+m9xJJcdN4zg9yW9x4TiZuPoniajWKn99Gggdme0spM/nxmORiyuRvGUfGSPXl7RE7M0fWf/yC6QSDpFF8x0k+VAIYFPK42CU7wVQ8ztbCSvwm5MFdwkHmAPu5C1xHVnlZ7Svl65zSx3xfaeX5Wimltac8nfOyZmV4rXImbECyu00Sd5ok2x/JKTVpuLRGamGNWrA7SJZhvYNVJsttU0/cszgR7nQ/Eg6UFyLUgxPdTCM4fC7GeuC/oMK9K9O9+bFLZ89fHp+ccPycc0ayf333navh37Ap1l/+iY86nimc+l1xfHjF+Y1wp6IVmJ9iGxJfk0Y=
RNG Seed:<PHONE_NUMBER>
you can't unh sub without ERW in the opener...yet
Stack Trace:
goroutine 63 [running]:
runtime/debug.Stack()
/opt/hostedtoolcache/go/1.18.8/x64/src/runtime/debug/stack.go:24 +0x6
github.com/wowsims/wotlk/sim/core.runSim.func1()
/home/runner/work/wotlk/wotlk/sim/core/sim.go:59 +0x10
panic({0xbb100, 0x2c2d60})
/opt/hostedtoolcache/go/1.18.8/x64/src/runtime/panic.go:838 +0x29
github.com/wowsims/wotlk/sim/deathknight/dps.(*DpsDeathknight).SetupRotations(0x1498b40)
/home/runner/work/wotlk/wotlk/sim/deathknight/dps/dps_deathknight.go:163 +0x98
github.com/wowsims/wotlk/sim/deathknight/dps.(*DpsDeathknight).Reset(0x1498b40, 0x1581ef0)
/home/runner/work/wotlk/wotlk/sim/deathknight/dps/dps_deathknight.go:300 +0x20
github.com/wowsims/wotlk/sim/core.(*Character).reset(0x17e6a80, 0x1581ef0, {0x2c7d10, 0x1498b40})
/home/runner/work/wotlk/wotlk/sim/core/character.go:359 +0xf

the error message says "you can't unh sub without ERW in the opener...yet"
because there's no code for non ERW opener for frost sub unholy yet.


Code "gselog" does not prompt menu

Typing to code word "gselog" does not open the logging menu, is there another way to access the menu? I'm using the Crome extension on a Mac running OS X 10.9.2.

Same Prob..

Need to implement a better way of toggling that menu for sure. It's running on web form semantics circa 2012, so I doubt the logging is very accurate these days either. Along with some really significant security concerns that need to be addressed ASAP (credentials just stored in plain text at the moment, so a lot of sensitive information can easily be accessed without so much as the passphrase), a better mechanism for prompting the menu will be packaged with a Chrome Web Store update soon, probably this weekend.

Hello I have same problem. I typed gselog facebook or google search bar but it never shows log management menu. Is there any other way to open it?

yes...i think u can go here and get what u need .
chrome-extension://mekpfngodchodemgmkhinohkfjefjeea/html/manage.html

Thanks alot...Diozz... the link worked..
chrome-extension://mekpfngodchodemgmkhinohkfjefjeea/html/manage.html
it works.

what I've noticed, you need to just type "gselog" in a password submission form and it seems to have worked. It even worked when I typed it in this comment box

thanks it working

gselog

gselog

Mute the thread
On Dec 21, 2016 5:14 AM, "tommy pickles"<EMAIL_ADDRESS>wrote:

gselog
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ajartille/gumshoe/issues/7#issuecomment-268520192,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIdhbDG7YLrSK9kcjMrdfjiVbgwOZjLdks5rKSZAgaJpZM4Bu3jU
.

great it works. but its a bug in gumshoe.
chrome-extension://mekpfngodchodemgmkhinohkfjefjeea/html/manage.html


Regression analysis (Linear regression, Logistics regression)

Description 📜
Name: Jay Gohil
Serial No: 124
Batch: 4
Issue Number: 5848
Type: Documentation
Created a detailed Jupyter Notebook with code as well as explanation of Linear and Logistic regression evaluation (and how to analyze and compare them) that provides all necessary details pertaining to regression analysis in a concise manner.
Fixes #5848

Type of change 📝

[ ] Audio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Video (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[x] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

Greetings mam @prathimacode-hub, I have made all the necessary changes asked in the previous review. Please check the same. Thanks!


::after pseudo element as a tooltip

Hello, thank you for an awesome library.
To show the tooltip you need to attach event listeners for in and out. If your app has a lot of buttons - it causes performance issues. Is it possible to do something like:
  function setTooltipsListener() {
    document.querySelectorAll("[data-tooltip]").forEach((el) => {
      var tooltip = window.getComputedStyle(el, ":after")
      computePosition(el, tooltip)
    })
  }
  addEventListener('scroll, resize', setTooltipsListener);

Maybe the CSS hover event could be more performant than 2 JS event listeners. This snippet could utilize the following styles:
[data-tooltip] {
  position: relative;

  &::after {
    content: attr(data-tooltip);
    position: fixed;
}}

Thank you

I'm not sure I understand your question. Adding mouseenter and mouseleave events don't cause perf issues with lots of elements (which are equivalent to "hover" in CSS). scroll and resize however will, if element is not currently visible, is that your issue? You just need to make sure you aren't listening while the floating element is not visible.


OWLS-95596: Add Cluster watcher

Add Cluster watcher to handle processing of Cluster Resource events.

The watcher obviously isn't going to allow us to have much make-right behavior if the domain processing doesn't do anything with the cluster resource. Are you intending to defer that to a new PR?

The watcher obviously isn't going to allow us to have much make-right behavior if the domain processing doesn't do anything with the cluster resource. Are you intending to defer that to a new PR?

The idea was to handle Cluster Resource events similar to Domain events.  How we handle the Cluster Resource events is intended for the next PR that we pair together on.


fix: website accessibility fixes

update image labels for app-layout images
fixes for icon buttons
properly label getting started intro for supported browsers image

Signed-off-by: Matt Hippely<EMAIL_ADDRESS>PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] If applicable, have a visual design approval

PR Type
What kind of change does this PR introduce?
Adds more accessible behavior to website content.

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

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Adds more accessible behavior to website content.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

@hippee-lee I have some code for solving other issues, so would you like to push my code into your branch and get all the fixes we have at once?

Seems also like the file we had with the issue is not much of a great way to track what we do because we did a double work here

@hippee-lee I have some code for solving other issues, so would you like to push my code into your branch and get all the fixes we have at once?

Let's keep the changes separate. If the code is changes in a similar way it will just rebase overtop but if they are different we can discuss that on the new PR.
Sorry about the confusion - perhaps the spreadsheet is not updating?


Quest/ANC - Toggle filter for Child clients

Marked as blocked for now as this view has neither been discussed with the design team or groomed as a required feature


Proofreading feedback

Description
I noticed on the foreword that you indicated proofreading was something you would seek out from others. I noticed at least one typo, and wanted to reach out and ask if I could be of service in helping proofread further.
Steps to reproduce
The bottom of the front page uses available twice inappropriately. It's an easy oversight to make when building something so large.
Expected result
n/a
Actual result
n/a
Environment
n/a

hey. I'll migrate the content to another platform once it's ready and there are people that will proofread what is on devpill. Thanks for your offer :)


Integration tests: VAnchor withdraw with a default UTXO commitment not inserted on-chain

@AhmedKorim you left the .only in the tests.


Feature Request: fileReplacements for assets

Bug Report or Feature Request (mark with an x)
- [ ] bug report -> please search issues before submitting
- [x] feature request

Area
- [x] devkit
- [ ] schematics

Desired functionality

Currently, fileReplacements option supports only for TypeScript module resolution.
I'd love to use it for assets.
Use cases

replace manifest.json
replace images with same URL

Mention any other details that might be useful

@sebek64 Could you provide more informations how to archive that, or at least the according pull request?
I would like to build the same app multiple times with different assets.

Yes, the corresponding issue is for example https://github.com/angular/angular-cli/issues/10992.

Hi all. thanks for this feature request. But I don't think that we want to continue expanding the scope of fileReplacements.
Most likely in the future we'll provide some sort of token based replacement, but until then replacing non TS files is out of scope.


docs: auto docs are not available anymore

Description
Directory was removed in 3b917646315f4cc81c49749ad64979b17ffe0720
Migrations required
NO
Verification
Not applicable.
Documentation
We use pre-commit to update the Terraform inputs and outputs in the documentation via terraform-docs. Ensure you have installed those components.

👍 for your contribution, will check asap

Thanks!


MDL menu component bug when using ASP.NET web form

I am trying to implement a drop down menu component described on the following page: http://www.getmdl.io/components/#menus-section. It works fine in a regular html page but on a web form something is occuring where the menu doesn't show.

Please use stackoverflow for integration support. We can only help with specific issues with what is provided.


Clean up UI for new design

This PR:

replaces the use of a separate style for "inactive" by using iPlug2's IControl::SetDisabled() on param change. This means the UI state will also change when automated/controlled externally, it wouldn't before
removes old resources
removes doubled up knobs in c673b561f513f7ac4cb4633192a772868fea9700, replaces with a new control NamKnobControl
replaces Bitmap switches in c673b561f513f7ac4cb4633192a772868fea9700 with a new NamSlideSwitchControl - the bitmaps here need cleaning up a bit (also did not try on windows or low-DPI screen yet), also layout is somewhat hacked

I believe i have fixed those two things but I don't have an appropriate .nam to test the disabling of the normalization switch. I also removed the ITextControl "Normalization' which didn't fit so nicely with the UI. I think an icon + tooltip might be a good solution there.

I don't have an appropriate .nam to test the disabling of the normalization switch.

Same model with loudness metadata in case you get to it before me :)


Android Compose Generics Objects and Extensions

Name of the Codelab or Codelab URL
https://developer.android.com/codelabs/basic-android-kotlin-compose-generics?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-3-pathway-1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-generics#5
Describe the problem
"Error extension property, step 3. It gives an error.
Solution
println(Quiz.progressText)
->
println(Quiz.StudentProgress.progressText)
In which lesson and step of the codelab can this issue be found?
6, step 3 of extension property
How to reproduce?
execute:      println(Quiz.progressText)
Versions

What version of Android Studio are you using? flamingo (lastest)
What API level are you targeting? 21

Additional information
Add any other context about the problem here.
codelab: Generics, objects, and extensions

Hi @tibolanneau
Thank you for writing in with this issue.
The lesson has been updated to address it and will be republished in the near future.
Hope you are enjoying the course!
Regards.


Added TestRunInteractive

Added quit() to interactive

Codecov Report

Merging #215 into master will increase coverage by 1.56%.
The diff coverage is 45%.

@@            Coverage Diff             @@
##           master     #215      +/-   ##
==========================================
+ Coverage   80.78%   82.35%   +1.56%
==========================================
  Files          44       44
  Lines        3029     3015      -14
==========================================
+ Hits         2447     2483      +36
+ Misses        435      378      -57
- Partials      147      154       +7

Impacted Files
Coverage Δ

anko.go
59.59% <45%> (+34.81%)
:arrow_up:

colortext.go
25.92% <0%> (+18.51%)
:arrow_up:

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 f825696...daf8026. Read the comment docs.

@mattn
Seems like colortext is not needed as much any more, what do you think about removing that dependency?

Sounds good to me. How is this PR?

Thank you


Bugfix/crt static in config

I ran into an issue when compiling xz2 crate under Windows, where .cargo/config RUSTFLAGS does not set the correct env vars in the build (full details can be found at
https://github.com/alexcrichton/xz2-rs/issues/46).
The problem seems to be that an empty env var will take precedence over the config file.
Cargo version is:
cargo 1.34.0-nightly (5c6aa46e6 2019-02-22)
I added a test for this and a small fix.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Thanks for this @ohadravid!
I'm gonna run this by others on @rust-lang/cargo since this is a beahvioral change, but I think this is a reasonable semantics for Cargo to implement. Cargo doesn't necessarily uniformly handle empty env vars though, so I'd be curious about others' thoughts on that
@rfcbot fcp merge

So how do you override a config file and use the defaults after this change?

I can change the code so it's possible to differentiate between an empty string and some whitespace and treat whitespace as the 'default', but this would be very confusing for users (breaks the common expectations around environment variables).
Also as a user, I think of whatever's in the config file is the 'default'.  For example I have a crate which is built against a static lzma.so file, and trying a 'default' (==empty) flags will break the build.
It's worth noting that the current behavior for the 'build.rs' file is inconsistent: cfg(crt-static) is enabled (respects the config file), but CARGO_CFG.. is not set.

@Eh2406 @alexcrichton something wasn't quite right, since the bug I saw didn't actually allow for the default flags to be passed.
I created a test which reproduces this (but which is NOT fixed by my change).
This actually has nothing to do with empty env args.
If you run it, it will actually fail in the second assert.
fn main() {
    // The crt-static feature should be enabled.
    #[cfg(not(target_feature = "crt-static"))]
    assert!(false);

    assert!(std::env::var("CARGO_CFG_TARGET_FEATURE").unwrap().contains("crt-static"));
}

(Changing the [target(cfg...)] to [target] will not trigger the bug).
I'm not really sure what to do, since requested_target in build_context/mod.rs:env_args seems to be set, but this seems like a contradiction (either both feature and env var should be set, or both shouldn't).

Sorry I don't know off the top of my head what's going on, so I'd have to dig in, but I don't have time right now to do so

:umbrella: The latest upstream changes (presumably #6867) made this pull request unmergeable. Please resolve the merge conflicts.


Regular view changes

Rebased and fixed lint

Rebased on persistent storage changes

Fixed failing unit test, changed how idle timeout is started.


⚠ Palworld Tunnel (Connection URL) has degraded performance

In a88514d, Palworld Tunnel (Connection URL) ($PG_PANEL/tstat.txt) experienced degraded performance:

HTTP code: 200
Response time: 78 ms

Resolved: Palworld Tunnel (Connection URL) performance has improved in b88ec52 after 37 minutes.


BZ1277556: Project selector Dropdown becomes useless with the right(wong) OU/Repository/Project name lengths

See https://bugzilla.redhat.com/show_bug.cgi?id=1277556
This only affects 6.3.x (more by luck). The "Project" CustomDropdown is aligned along the right-hand edge (compared to OU and Repository dropdowns, that are aligned left). This can lead to the scenario described in the BZ. This fix simpy re-positions the drop-down if clipped.
See https://issues.jboss.org/browse/UF-246 too (as it's related).

@cristianonicolai Hi, since you've become the resident UI-whizz-kid do you mind taking a look at this?

@Rikkola Hi... I'd hoped @cristianonicolai may have reviewed this in time for todays deadline.. would you mind taking a look please, so it can be merged today? Thansks.

ok to test

+1
This is a rendering issue so hard to unit test. There would have to be a Selenium test for this.

The CI failure relates to https://github.com/droolsjbpm/jbpm/commit/509b56fa47f6013cd771295797180a2d963f3269

merged into 6.3.x with https://github.com/droolsjbpm/kie-wb-common/commit/98b2c47ba3525fb75e5d6a05a5d20121f80e9dbf

@manstis really sorry, I somehow missed it. :(


Update commands in quickstart and in readme.

add ocdev url to quickstart
change ocdev component create to ocdev create

LGTM


Cannot implement LoRA on a custom model containing transformer encoder from pytorch

I am using LoRA for my custom model. Within the model i have trasnformer encoder block and a series of linear layers. I want to implement LoRA in the custom model specifically targeting the q,k and v projection weights on the self attention block. However, in the transformer encoder layer in pytorch, I cannot find any module names corresponding to the q, k and v projection.
class CustomModel(nn.Module):
   def __init__():
      self.embedding = nn.Linear(100,512)
      self.encoder_layer_1 = nn.TransformerEncoderLayer(d_model=self.model_dim, nhead=16)
      self.transformers_1 = torch.nn.TransformerEncoder(self.encoder_layer_1 , num_layers = 24)
      self.output = nn.Linear(200 * 512,200)

     def forward():

The code above represents the architecture of my custom model.
Has anybody implemented LoRA on the nn.Transformer on the q,k and v projection weights before? If yes, what is the module name for those weights?

I meet the same problem, I think first, nn.TransformerEncoderLayer uses MultiheadAttention, and MultiheadAttention uses
self.in_proj_weight = Parameter(torch.empty((3 * embed_dim, embed_dim), **factory_kwargs))
self.register_parameter('q_proj_weight', None)
self.register_parameter('k_proj_weight', None)
self.register_parameter('v_proj_weight', None)
If u use PEFT to find qkv, it only finds nn.Module(like nn.linear, nn.conv1d, etc), not Parameter, so even u print the model, u can not find in_proj_weight, u can only find out_proj that uses
self.out_proj = NonDynamicallyQuantizableLinear(embed_dim, embed_dim, bias=bias, **factory_kwargs).
Second, if u separate qkv and register them by nn.linear, it uses F.multi_head_attention_forward and in this function, it uses
q, k, v = _in_projection(query, key, value, q_proj_weight, k_proj_weight, v_proj_weight, b_q, b_k, b_v)
and then in _in_projection it
return linear(q, w_q, b_q), linear(k, w_k, b_k), linear(v, w_v, b_v)
so it goes wrong too. Even if u set in_proj_weight=None and use_separate_proj_weight=False and there's
if not use_separate_proj_weight:
assert in_proj_weight is not None, "use_separate_proj_weight is False but in_proj_weight is None"
so I think u can only write transformerencoder by urself, or find other repository.
I wish this can help u


Firefox raises InvalidStateError when trying to destroy player

See jsfiddle here:
https://jsfiddle.net/d4hwuzwm/
(this loads a video, waits for it to start playing, then calls destroy 5 seconds later)
Calling destroy on a player instance in Firefox raises InvalidStateError: An attempt was made to use an object that is not, or is no longer, usable
This works fine in Chrome/Safari.

It looks like if you wait for the playback to stop, it won't raise the exception, but unfortunately, I don't have that luxury because I need to destroy the player immediately.
      console.log('Attempting to destroy player...');
      player.stop();
      player.core.getCurrentPlayback().once('playback:stop', function() {
 	     player.destroy();
      });

reproduced here


Session set_flashdata() - Error '502 Bad Gateway'

Hello,
i dont know if it is a bug:
I got an 502 Bad Gateway-Error after an Ajax-Request.
In the target controller/method i have setted 3 flashdata-session-items:
$this->session->set_flashdata("success","true");
$this->session->set_flashdata("success_to_user_id","1");
$this->session->set_flashdata("success_something_else","xyz");
The request successfully finished. But then, when i refresh the site
(window.location.reload or F5) i got an '502 Bad Gateway' Error.
After playing around I have fixed that error:
I reduced the 3 set_flashdata() calls to 1. Then it works, no more 502.
So:
Less or equal 2x $this->session->set_flashdata(); = OK, NO ERROR
Greater or equal 3x $this->session->set_flashdata(); = ERROR 502 BAD GATEWAY
Is this a bug ?
Greetings,
Julian Fuerderer

Fixed by #3073.

@AdenFraser thank you.
in Nginx, you must set conf with:
    location ~ \.php$ {
    	try_files $uri =404;
        fastcgi_split_path_info ^(.+\.php)(/.+)$;
        fastcgi_pass php:9000;
        fastcgi_index index.php;
        include fastcgi_params;
        fastcgi_param SCRIPT_FILENAME $document_root$fastcgi_script_name;
        fastcgi_param PATH_INFO $fastcgi_path_info;
        fastcgi_buffer_size 128k;
        fastcgi_buffers 4 256k;
        fastcgi_busy_buffers_size 256k;
        fastcgi_temp_file_write_size 256k;
        fastcgi_intercept_errors on;
    }


fixing a race condition with hitting a rate from multiple threads

Fixes #485 .  This prevents a race condition that can occur when a rate is incremented from multiple threads at the same time, and the rate had never previously been hit before.  Hits from multiple threads afterwards work correctly, so generally at most only 1 hit was lost.

Current coverage is 83.70% (diff: 100%)

Merging #488 into master will not change coverage

@@             master       #488   diff @@
==========================================
  Files            79         79
  Lines          3940       3940
  Methods        3611       3611
  Messages          0          0
  Branches        329        329
==========================================
  Hits           3298       3298
  Misses          642        642
  Partials          0          0

Powered by Codecov. Last update 73c30f2...b10f679


Updating Openni to be Apache2.0

Type: Missing
Summary:
Updating Openni to be Apache2.0
Details:
Looked at the repo on Github and saw that the declared License is Apache2.0, but CDL doesn't have an associated license.
Resolution:
https://github.com/OpenNI/OpenNI/blob/master/LICENSE
Affected definitions:

openni d50d0e3fb0a5913185ea03c5fd5268cb4472a2a4

Welcome to the ClearlyDefined community, @akurus. Thank you for your contribution!

@akurus, the license provided should be what was at the time of this commit, not master since master represents the latest commit.
When I look at the repo for this commit, it has GPL-3.0 and LGPL-3.0. Going through the history, the switch to the Apache license didn't happen until 2013.
Please let me know if you agree that the declared license should be GPL-3.0 and LGPL-3.0.

@capfei Apologies for the error on my part. I agree with your assessment that it should be GPL-3.0 & LGPL-3.0


It can not be compiled with purs 0.15

Error found:
at .spago/spec-mocha/v4.0.0/src/Test/Spec/Mocha.purs:1:1 - 57:44 (line 1, column 1 - line 57, column 44)

  A CommonJS foreign module implementation was provided for module Test.Spec.Mocha:

    .spago/spec-mocha/v4.0.0/src/Test/Spec/Mocha.js

  CommonJS foreign modules are no longer supported. Use native JavaScript/ECMAScript module syntax instead.

This has been resolved.

Thank you!


CIP-???? | CBOR Tag definition for CIP-0003 ED25519-BIP32 Keys

This CIP is both desired by Project Catalyst to help define CBOR data structures, and serves also as an example implementation for https://github.com/cardano-foundation/CIPs/pull/752.
Note, the example registry in https://github.com/cardano-foundation/CIPs/pull/752 has matching definitions for these tags, typically a PR would include changes for that file once its been merged.
This is the third of three PRs that should be considered together for context.
See also:

#751
#752

Rendered proposal on author's fork

Should this be its own CIP or could this just be added as a document inside of https://github.com/cardano-foundation/CIPs/pull/752?
Just thinking if it is easier to manage if contained in one CIP?

Should this be its own CIP or could this just be added as a document inside of #752?
Just thinking if it is easier to manage if contained in one CIP?

I didn't think it was normal to keep editing CIPs once they are accepted.
There are a bunch of candidates for tags, this is just a really easy one I need.
CIP-19 jumpts to mind as well.
Following the example of the metadata registry CIP it just explains how, and new metadata CIPs are supposed to include a patch to its registry file.  All metadata isn't defined in CIP-10

@Ryun1 @stevenj adding CIPs for tag definitions would be consistent with our precedent for CIP-0068 - for which the metadata and motivation / rationale would be hopelessly more complicated than we could ever simply tabulate in the originally conceived CIP-0067 table to store those asset labels.
Since we've told new asset label (token type) creators to submit new CIPs for any new definitions beyond the first four, it would make sense to do so here... and even more sense if @stevenj your planned introduction of these key/tag definitions can be broken into groups as large as possible, as you already appear to have done here.
On the other end of the complexity spectrum: we were able to "get away with" using a simple JSON array in CIP-0010 because the information stored was simpler, and the details of each entry were colloquial and could be challenged in the PR if necessary (cc @KtorZ).  But the information associated with each of these CBOR tag/keys needs more authentication that needs to be recorded more officially: so requiring new CIPs does make sense.
@stevenj one thing we need to stipulate for both CIP-0013 and CIP-0068 (cc @Crypto2099)... to make both of them "extensible"... is to say very clearly that a certain number of initial definitions were supported in the original CIP and then others beyond would follow a specific form.  You may already have done this but please keep this "plug-in" convenience as a goal when refining your draft for #752.
... and therefore @Ryun1 that would indeed allow a number of initial definitions to included as suggested in https://github.com/cardano-foundation/CIPs/pull/753#issuecomment-1911992973.

@stevenj @Ryun1 - separate question from the last: I'm avoiding tagging this Metadata or Tools until we settle the question asked earlier in https://github.com/cardano-foundation/CIPs/pull/752#discussion_r1466557385.

Agreed @stevenj @rphair
lets do new CIPs!

@Ryun1 it does seem like it would be cleaner to have 1 CIP for the framework and 1 CIP for each new set of related keys.  We were kind of stuck with CIP-0068 because we had to represent the state of the art until that point... but here we can start fresh, so there's no reason the framework CIP would have to include a "base" set of keys (I'd be happy to leave that to @stevenj this time).
This can be confirmed during Triage at the next meeting where it would be up for introduction anyway (I've put it on the agenda): https://hackmd.io/@cip-editors/81

@Ryun1 it does seem like it would be cleaner to have 1 CIP for the framework and 1 CIP for each new set of related keys. We were kind of stuck with CIP-0068 because we had to represent the state of the art until that point... but here we can start fresh, so there's no reason the framework CIP would have to include a "base" set of keys (I'd be happy to leave that to @stevenj this time).

I agree!

Going to echo some of the sentiment from @rphair and @Ryun1. I love how these three PRs have been drafted to work in unison and provide a "clean" template for how these "registry" types of CIPs/CPSs could be defined in the future. This seems much cleaner than what we have learned through past experience w/ CIP-10, 68, etc.


[google_maps_flutter] Cannot run on iOS simulator

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Attempt to run an app using the google_maps_flutter plugin on an iOS simulator

Expected results
It runs
Actual results
It doesn't run
It used to run on simulators. Not sure if it was an Xcode, macOS, iOS, etc update that broke it, but it doesn't work anymore.
Xcode can run it on a simulator but it has a huge popup about needing to run it with rosetta. I guess Flutter can't handle doing that itself? Either way this is probably a config issue with the google maps plugin.
Code sample
Unnecessary. The plugin itself doesn't build.
Screenshots or Video
No response
Logs
logs
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.13.2, on macOS 13.5.1 22G90 darwin-arm64, locale en-US)
    • Flutter version 3.13.2 on channel stable at /Users/rexios/fvm/versions/3.13.2
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ff5b5b5fa6 (8 days ago), 2023-08-24 08:12:28 -0500
    • Engine revision b20183e040
    • Dart version 3.1.0
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0-rc3)
    • Android SDK at /Users/rexios/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0-rc3
    • Java binary at: /Applications/Android Studio Preview.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (version unknown)
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.81.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.70.0

[✓] Connected device (4 available)
    • Aaron’s iPhone (mobile)    • 00008120-001C0CA201DB401E            • ios            • iOS 16.6 20G75
    • iPhone 14 Pro Max (mobile) • FF9097BB-27DB-4F50-A9AC-DA77CA19DB71 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)            • macos                                • darwin-arm64   • macOS 13.5.1 22G90 darwin-arm64
    • Chrome (web)               • chrome                               • web-javascript • Google Chrome 116.0.5845.110
    ! Error: Aaron’s iPhone is busy: Waiting to connect and unlock the device. Xcode will continue when Aaron’s iPhone is finished. (code -10)

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

warning: ignoring file [...]/build/ios/Debug-development-iphonesimulator/google_maps_flutter_ios/google_maps_flutter_ios.framework/google_maps_flutter_ios,
building for iOS Simulator-arm64 but attempting to link with file built for iOS Simulator-x86_64

Are you doing something in this app that would force an ARM simulator? Apps that use google_maps_flutter should be running under the Rosetta simulator.

Either way this is probably a config issue with the google maps plugin.

The requirement to use Rosetta is discussed in https://github.com/flutter/flutter/issues/94491

What would force the app to use an ARM simulator?
Xcode seems to run the app in the rosetta simulator fine (with an initial information popup), but the flutter tool itself doesn't do that for some reason.
We've been running this app in the simulator since we started using the google maps plugin and only just now started having the issue. The issue came up without updating the google maps plugin, so it must be something external that broke it but I'm not sure what it would be.

I think I fixed it with the following pieced together from the issue you linked:
post_install do |installer|
  installer.pods_project.targets.each do |target|
    flutter_additional_ios_build_settings(target)
    target.build_configurations.each do |config|
      # We already had this
      config.build_settings['IPHONEOS_DEPLOYMENT_TARGET'] = '13.0'
      # I added this
      config.build_settings['EXCLUDED_ARCHS[sdk=iphonesimulator*]'] = ''
    end
  end
end

Not sure why it suddenly broke but thanks for the insight

The exception I was getting is:
Error (Xcode): Undefined symbol: _OBJC_CLASS_$_FLTGoogleMapsPlugin

So this issue is searchable in the future


[BEAM-10145] Delete persistent disks after every KafkaIO performance test run

Persistent volumes for KafkaIO test inherit the reclaim policy, specified in the StorageClass, which was set to 'Reclaim'. For automatic reclamation of the resource, the 'Delete' policy should be used instead.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Run Java KafkaIO Performance Test

Run Java KafkaIO Performance Test

R: @udim

I ran the test and all persistent volumes and GKE disks were automatically removed. I also deleted all disks that were created in the past.


killing pragma: no cover statements

Added:
"except Exception as msg",
"except KeyError as msg",
"except IntegrityError as msg",

to [tool.coverage.report.exclude_lines] so that these patterns get automatically excluded from coverage.
Note that doing except Exception as X where X is anything but msg will not be automatically excluded.
This gets rid of about half of the pragma no cover statements and draws out the ones that we might want to think more about.

Note that line exclusions can be regexps -- if you did want to cover more than as msg, you could say as .* there instead


Some other issues

@caspahouzer
Please see this:
https://github.com/ricardoalcocer/jssql/blob/master/app/lib/com.alcoapps.dbhelper.js#L54 ,  This error should be a callback that the user can react to.
Also
https://github.com/ricardoalcocer/jssql/blob/master/app/lib/com.alcoapps.dbhelper.js#L207 and https://github.com/ricardoalcocer/jssql/blob/master/app/lib/com.alcoapps.dbhelper.js#L268 , this broke after getData was implemented...I'm assuming it's because that by the time getLastInsertedRecordId is called, getData already closed the DB connection.
Lastly, run the app.  I started providing a best test app, but stopped after I found those issues.
Any thoughts?

Pull this new version...has a couple changes on top of your recent PR.

Started working and fixing the issues by closing the db connection manually and not in getData. Also reimplements getLastInsertedId and affected rows on update and delete

Great, thanks.  I'll hold of the sample app until I get an update.  I appreciate the help.

Honor for me. Thanks

Thank you.  Those changes fixed the last index issue.  I'll continue to work on a better sample app.


Text overflows out of button / button does not expand on homepage

Describe the bug
When accessing freecodecamp.rocks via mobile, the 'Start coding(it's free)' button does not expand with respect to text.
As a result, the  text overflows the button area.
To Reproduce
Steps to reproduce the behavior:

Go to freecodecamp.rocks via mobile
Scroll down to '....'
See error

Expected behavior
The button should expand appropriately and text should not overflow
Screenshots

Smartphone (please complete the following information):

OS: Android
Browser Chrome
Version 72.0.3626.105

I am currently looking into this issue. I will keep you posted

I've found that there is one more place where this issue happens, on welcome page:

These are the only places issue occurs, so I gonna apply an ultimate fix for such cases using a mobile first media query.

PR is ready for review https://github.com/freeCodeCamp/freeCodeCamp/pull/35378


fix: ensure parsing pretty printed json is working

Always got this error when using the lib because the lib is expecting a oneline response but openai returns pretty printed json :
com.google.gson.JsonSyntaxException: java.io.EOFException: End of input at line 1 column 2 path $.
	at com.google.gson.internal.Streams.parse(Streams.java:59)
	at com.google.gson.JsonParser.parseReader(JsonParser.java:102)
	at com.google.gson.JsonParser.parseReader(JsonParser.java:70)
	at com.google.gson.JsonParser.parseString(JsonParser.java:51)
	at com.cjcrafter.openai.OpenAICallback.handleStream(OpenAICallback.kt:66)
	at com.cjcrafter.openai.OpenAICallback.onResponse(OpenAICallback.kt:30)
	at com.cjcrafter.openai.OpenAI.createChatCompletion(OpenAI.kt:252)

related: https://github.com/CJCrafter/ChatGPT-Java-API/issues/25

Sorry for the delay and thankyou for contributing!
I seem to remember having issues with that method (readUtf8)... I believe for when you aren't streaming the response. Let me run some testing to make sure this PR doesn't affect any other methods and we'll get it merged.

Doesn't work. For example, when using stream=true & async=false, We get a json string that looks like this:
{"id":"chatcmpl-7vT7LNizNrVuZS8Yv8ZCMcKglZn8i","object":"chat.completion.chunk","created":1693930979,"model":"gpt-3.5-turbo-0613","choices":[{"index":0,"delta":{"role":"assistant","content":""},"finish_reason":null}]}

data: {"id":"chatcmpl-7vT7LNizNrVuZS8Yv8ZCMcKglZn8i","object":"chat.completion.chunk","created":1693930979,"model":"gpt-3.5-turbo-0613","choices":[{"index":0,"delta":{"content":"Hello"},"finish_reason":null}]}

GitHub highlights that second line in red since it isn't valid JSON. The simplest way to solve this would be splitting at every instance of "data: " in the string (though we would have to make sure we aren't currently inside a json block {} because then we would malform the json ourselves. )

Issue was that non-stream requests were being handled using the stream method.

Issue was that non-stream requests were being handled using the stream method.


Fix manual queries losing parameters when called

Closes #2019
Visual demonstration

Before
https://github.com/mui/mui-toolpad/assets/19550456/33d7e121-e372-4ab5-8b01-c33096ac8e1c
After
https://github.com/mui/mui-toolpad/assets/19550456/89d0a8ea-1331-4ff3-8b81-cb3d46528f05

Please follow the the contributing guide on how to create great pull requests.

Can you add a test case to either https://github.com/mui/mui-toolpad/blob/master/test/integration/backend-basic/index.spec.ts o  https://github.com/mui/mui-toolpad/blob/master/test/integration/rest-basic/index.spec.ts ?
Just a button that triggers a query with bound parameters and displaying the output


edencommon: patch to increase test discovery timeout

Description of changes
For some reason, this build is incredibly flaky on x86_64-darwin, possibly after the recent staging-next merge, failing with a test discovery timeout error both on Hydra and locally. The default discovery timeout is apparently 5 seconds, so let’s try giving it 5 times as long and see if that helps.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Made conditional on platform to avoid this having to go to staging, since it seems to only be flaky on one platform for now anyway.


After project write, menu option git commit is disabled

This is in Oscar-<IP_ADDRESS>-prerelease2.
If I've done a write, my changes are written to disk, but not committed to git.  But after a write, the commit to git button is disabled. So I can't commit to git until I make more changes.
This is probably related to the attempt to make it harder to encounter bug #317.
Minor.

I don't see any Rowan api's available that can identify a project that has uncommitted changes on disk and in the image.
One possible way to fix this issue in Jadeite without Rowan changes would be to

Never disable the Commit to Git ... project menu option
Wrap the menu item's action in an error handler so a dialog could be raised instead of a walkback.

This doesn't seem like a great solution, though.  It would be better to have a Rowan api that identified uncommitted changes in a project either on disk or in the image.
Thoughts, @LisaAlmarode ?


Fix param_type override

fixes #145

Needs a spec and a CHANGELOG entry, please.

@dblock spec and CHANGELOG added.

I made some minor comments, please fix the build and squash your commits.

Updated for your comments. But the CI job is failing for complexity check:
Cyclomatic complexity for create_documentation_class is too high. [100/99] and Assignment Branch Condition size for create_documentation_class is too high. [349.9/347].
I have no idea how to fix these.  I can't just simply simplify the class.

This is just a rubocop violation, run rubocop --auto-gen-config to update the values for this violation.

Squash everything please.

Merged via https://github.com/tim-vandecasteele/grape-swagger/commit/daaad4c47592c4c6bbc8ac8850284a030c1dd8e0. Thank you.

not work with nested param, still form
optional :images, type: Array, documentation: { param_type: 'body' } do
    requires :image
 end

@TangMonk Are you doing this on HEAD? This fix went into code that hasn't been released yet.

@dbblock, yes, I am sure I clone this repo

Can you try to write a spec that reproduces the issue?


Update reporting record limit tip

Description
There's an outdated tip on the reporting page related to https://github.com/opensearch-project/dashboards-reporting/issues/328 that we weren't aware existed when the feature was implemented. This PR updates the tip: there's no longer a hardcoded 10k record limit.
Issues Resolved
N/A
Version
2.17 and above.
Frontend features
A new "Record limit" field is available, which shows a notice if the limit is configured above the default.

Checklist

[X] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@kolchfa-aws at your convenience, I know things are hectic right now with 2.18.


[Feature request]: Programmatically add access to lock

Describe the solution you'd like
I own and manage an Airbnb and use the Eufy smart lock for access to our condo. Would it be possible using this library to programmatically add new people to the access list for the smart lock? I'm more than happy to actually implement the changes necessary but would need to be pointed in the right direction. I don't currently see this as a supported feature.
Edit: more specifically, I’m not wanting to add other eufy users but add codes programmatically. It’s functionality that exists in the current app. I don’t want to share access. This is how I’m currently doing it but it’s a manual process.

Hi @bbedwell
just a user, but giving my 2 cents here:
I don't think eufy devices can be shared the way you describe. At least your guests would need at least an eufy account and the app to use the lock. And the methods to invite users are not implemented in this library I think.
What would be theoretically possible, would be to build your own server with user accounts that deals with the lock communication in the background via this library (e.g. wrapping the eufy functionality up into your server).
But coding the server would be up to you, since this is out of scope of this project.
Also I'd be aware of the security implications this might have. Since a poorly written server might expose your lock in a way that you definitely do not want.

If this functionality is included in the app, I can also add it to the library. But for that I need access to the device.

If you would like to share the device with me:

<EMAIL_ADDRESS>(for EU)
<EMAIL_ADDRESS>(for US)

Thanks!

@bropat Did someone already share access with you? If not, let me know and I can share mine.

@fabianluque
No one yet.

@bropat Just shared my T8520 lock with you. Let me know when you’re done.

@bropat sorry I am unable to share with you as the location of the lock is in a ver

Small update:
Thanks to @fabianluque, almost all the desired functionalities have now been roughly implemented. These still need to be refined and tested.

@bropat Thank you! You still need access for a few more days?

@fabianluque yes, please.
I will let you know as soon as I no longer need it.

@fabianluque
I no longer need access. Thx!

@bropat Thank you!


fix deepspeed special test test_deepspeed_stage_3_save_warning

🐛 Bug

To Reproduce
Run on current master branch the following command:
SPECIAL_PATTERN=test_deepspeed_stage_3_save_warning bash tests/special_tests.sh
Error:

______________________________ test_deepspeed_stage_3_save_warning _______________________________

tmpdir = local('/tmp/pytest-of-adrian/pytest-1376/test_deepspeed_stage_3_save_wa0')

    @RunIf(min_gpus=2, deepspeed=True, special=True)
    def test_deepspeed_stage_3_save_warning(tmpdir):
        """
        Test to ensure that DeepSpeed Stage 3 gives a warning when saving.
        """
        model = BoringModel()
        trainer = Trainer(
            default_root_dir=tmpdir, plugins=[DeepSpeedPlugin(stage=3)], gpus=2, fast_dev_run=True, precision=16
        )
        trainer.fit(model)
        checkpoint_path = os.path.join(tmpdir, "model.pt")
        with pytest.warns(UserWarning, match="each worker will save a shard of the checkpoint within a directory."):
>           trainer.save_checkpoint(checkpoint_path)
E           Failed: DID NOT WARN. No warnings of type (<class 'UserWarning'>,) was emitted. The list of emitted warnings is: [].

tests/plugins/test_deepspeed_plugin.py:432: Failed

Expected behavior

Environment

CUDA:
- GPU:
- GeForce RTX 3090
- GeForce RTX 3090
- GeForce RTX 3090
- GeForce RTX 3090
- GeForce RTX 3090
- GeForce RTX 3090
- GeForce RTX 3090
- GeForce RTX 3090
- available:         True
- version:           11.1
Packages:
- numpy:             1.20.2
- pyTorch_debug:     False
- pyTorch_version:   1.9.0+cu111
- pytorch-lightning: 1.5.0dev
- tqdm:              4.60.0
System:
- OS:                Linux
- architecture:
- 64bit
- ELF
- processor:         x86_64
- python:            3.8.8
- version:           #64-Ubuntu SMP Wed Dec 9 08:16:25 UTC 2020

Additional context
In our GPU CI as well, check section for "special tests"

TODO: git bisect

TODO: git bisect

It's possible that the test itself failed in the original PR that added it (https://github.com/PyTorchLightning/pytorch-lightning/pull/8397). We don't know because the build log is not available.
The error code does not seem to get propagated, so special tests don't fail.


Fix: URI path element replace issue due to elements name overlap

Issue #, if available:
fixes #552
Description of changes:
Old serialization code only indexes first occurrence of target element via prefix without considering duplication cases, this pr fixes that by directly searching index of the whole element with the format of {element} or {element+}
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Old serialization code only indexes first occurrence of target element via prefix without considering duplication cases,

true, but to expand on it. The old code only looked for {key instead of {key} (unsure if optimization or off-by-one error on original implementation). Because of this, {name and {namespace} both matched {name}


pachctl: Add a command to decode protos

Sometimes you see marshaled protos floating around, like in the database.  Now pachctl can decode them.
$ pachctl misc decode-proto pfs_v2.BranchInfo 0x0A1B0A120A03666F6F1204757365721A050A036261721205656467657312530A1B0A120A03666F6F1204757365721A050A0362617212056564676573122064326362653833333161616634623634396532363430643566306662336165311A120A03666F6F1204757365721A050A03626172
{
  "branch": {
    "repo": {
      "name": "foo",
      "type": "user",
      "project": {
        "name": "bar"
      }
    },
    "name": "edges"
  },
  "head": {
    "repo": {
      "name": "foo",
      "type": "user",
      "project": {
        "name": "bar"
      }
    },
    "id": "d2cbe8331aaf4b649e2640d5f0fb3ae1",
    "branch": {
      "repo": {
        "name": "foo",
        "type": "user",
        "project": {
          "name": "bar"
        }
      },
      "name": "edges"
    }
  }
}

(No it's not messed up, I have a branch named "edges" in my instance and happened to pick that.)
It understands hex (with or without a 0x prefix), base64, and raw binary.  You can put the data on the cmdline, or you can pipe it in from stdout.  Without args, it prints all message types it knows about.
This change broke the linter (it doesn't think min is a builtin), so I upgraded it.  The linter upgrade unfortunately upgrades depguard, whose config format changed. I fixed it (and made sure it actually worked).

Yeah there were some bugs there; raw didn't work.  I also found it annoying to paste something in and have the \n at the end of the paste fail decoding, so I remove spaces from hex and base64 now.  Much better!


Opera browser don't  work with pufferpanel login

I tried for curiosity to use opera with browser, and i noticed all account that none of accounts work at login...

Which OS/version are you using?

windows 10


Not in the Menu list

I have installed the online version.But not in the player menu list .

Post here or send me via email - stream URL or details on how can I reproduce it. Some of the installed media players probably can display url/media details. The current list of supported types is intentionally small.
https://github.com/moneytoo/Player/blob/f4fe270b10dd5073949ca9912aff9d0836d92a26/app/src/online/AndroidManifest.xml#L33-L40

The Url is

https://vod4.buycar5.cn/20210305/1mtjz3nf/index.m3u8

Try the latest pre-release of v0.24: https://github.com/moneytoo/Player/releases

Thanks.
Work perfect.
But I find other problem.
I have set up the volume maximum,but when I open a new online video,the volume is small,I have to press the volume up button and the volume just   become normal (my original setup).
Please fix that.

When I first run the online version,the button has just like mouse over color,but when I play a online. video,and has no the button mouse over color.I am very hard to control the fuction with the remit controller,because I don’t know where am I .

I have set up the volume maximum,but when I open a new online video,the volume is small,I have to press the volume up button and the volume just become normal (my original setup).

The extra volume boost is intentionally not persistable. It should stay active for a while if you don't close the app or the system doesn't unload the app. Launching already loud video with headphones and not remembering it's on max could be quite unpleasant.

When I first run the online version,the button has just like mouse over color,but when I play a online. video,and has no the button mouse over color.I am very hard to control the fuction with the remote controller,because I don’t know where am I .

Yes, the button highlight on focus can be sometimes visible quite poorly. I'm already experimenting with alternatives (fa4c4b26d72fc41afee770eb1e9f2113b7432eda).

But I use it on TV. Not use headphones.


Proposal: Add State and status to Nodes

@raskle commented on Thu Feb 16 2017
Proposal: Add State and status to Nodes
Author: Michael Baird
Last updated: 2017/02/16
Abstract
Pilosa nodes are going to need various operational states other than up/down.  These states are necessary are we prepare Pilosa to dynamically grow, shrink, and replace nodes.
Background
As Pilosa goes into production we will need the ability to make changes to the cluster while it is running.  To facilitate this we will need to define a set of states of behavior for a node.  These sets of behavior will define the life cycle of a node from: birth, growth, reproduction, and death.  For a new node to join an existing cluster it will need different set of behaviors during this phase and will need to communicate this state to the rest of the cluster
Proposal

Each node in a pilosa cluster needs to maintain a state and return its current status
A pilosa node can be one of the following states

Normal: connected to the rest of the cluster and serving all requests
Leaving: In the process of leaving the cluster, no longer responding to SetBit requests.  Still responding to Sync endpoints to transfer data
Joining: In the process of joining the cluster, not responding to Bitmap queries but can respond the SetBit.
Moving: In the process of redistributing Slice Fragments across the cluster, but this node will remain with new hash assignments

Define a list of operations allowed by each state

Normal:
-all
Leaving:

/export
/slices/max
/fragment/nodes
/fragment/data
/fragment/blocks
/fragment/block/data
/nodes
/version
/status

Joining

/schema
/import
/export
/slices/max
/db
/db/time_quantum
/db/attr/diff
/frame
/frame/time_quantum
/frame/attr/diff
/fragment/nodes
/fragment/data
/fragment/blocks
/fragment/block/data
/frame/restore
/nodes
/version
/status

Moving

I believe this can respond to all requests, but we may need to limited access

Compatibility
Node state maintenance is in preperation for new features to support a dynamic cluster.
That being said internode communication is a core feature of Pilosa operating normally
We will need to ensure that cluster can respond to queries and data ingestion while some nodes are in the Leaving/Joining.  This will require that a minimum replication factor of 2 is in force.
If not we may want to consider any state other than normal "under maintenance" and return a 503 status code on affected endpoints.
Node Health is another resource we should cover in separate proposal.
Implementation
We will need to create a new /status API endpoint
This we be able to return the current node's status or the status of all nodes in the cluster
Through periodic Gossip communication across the cluster each node should maintain a status of other nodes it attempts to communicate with
We will need to maintain an API permission matrix for each state.
Proper status codes will be returned for unavailable resources

required for #737

I'm going to close this as it's part of the work going into #737


Enhancement - Encryption / Password Protection

Add capability to encrypt and password protect engagement files.

Good idea, will put it on the roadmap. any suggestions on the algo?

Not much of a crypto person, AES-256?


Update README.md

Console output on copy+paste and while using the clipboard feature (appears when hovering over the code block):

The dollar sign is to let you know to type it into your shell. :)

@GrahamCampbell sure, it does. Thank you for the explanation.  👍
Maybe this should be addressed directly to github. It makes less sense to copy the shell indicator, especially if the code block is using the language identifier shell.
I had a short look before opening this (just to understand the thoughts i had before opening this huge pull request ^^);

The shell indicator can't be copied here (https://symfony.com/doc/current/components/yaml.html#installation). I like that approach.

(He's not using it)
The two big ones using opposite approaches. Obviously a matter of taste. But what is not a matter of taste is the fact, that it leads to an unexpected outcome. The user who needs this indicator to understand that this is a shell input, will fail or at least struggle because of command not found: $ End of story. *hehe
If i'm writing to you anyway, i want to take the chance right away to say thank you for all your work: Thanks for all of your nice codes and for sharing it. :-) I used it pretty often and i loved it really much.


Modded Blocks on Datapack Furnace

Hi! I tested it using Synnitra on Forge, so if this is not on your end it would be nice to know so I can report under the Connector guys :D (but I posted it here first because I don't think the block registration change between loaders)
I noticed that the mod don't recognize the Create Modded Blocks when adding them into "additional_materials" for Furnace Datapack.
Didn't tested with the main "material" but would guess the same behavior.
Here you can check the file, just in case
deepslate_bricks_forge.json

This is just a link to the default file. What did you actually add to it?

I added the Cracked version (Minecraft) and the Cut and Small Deepslate versions from Create

I think I am going to close this issue for now, as it seems very related to datapacking. There are many other resources online on how to make one, and we provide documentation on how to use our format here:
https://docs.wispforest.io/alloy-forgery/home/


authentication token

View set authentication token

I won't be merging this for two reasons:

The token auth in DRF is actually very poor and often overridden by different token auth systems
Without any documentation or anything, it's just noise.


Updates the Regex to Be More Restrictive

To be safe, I'm using the same regex for the column headers as the column values.
The partition column entries are executed directly on the command line and need to be very restricted:
"^[a-zA-Z0-9_.-]+$" -> only alphanumeric, _, ., and -.
The other metadata columns are trying to balance safety with the reality that people use a lot of different characters in metadata:
"^[^\\n\\t\\\"\\'\\\\|;><]+$" -> ^[^\n\t\"\'\\|;><]+$ -> not newline, tab, ", ', \, |, ;, >, or <.

Thanks so much Eric. This is great.
Could you add in some test cases to verify that the regexes you've added properly error on certain metadata characters?

Added in bca08e0002c1976fd991d6f2152bf1647c68078f

Since these changes affect the input file by a user. I think you should update the description of the input criteria in the README.md, to at least avoid some of the issues coming up. In a similar vein, might be useful to also modify error messages so that they mention that certain characters are not accepted.

Updated the messages in: bca08e0002c1976fd991d6f2152bf1647c68078f

Added additional information about the metadata restrictions in the README in 331dfe2917bf0a928efcb24022f92caa8d85251b

Thanks for the update! I think that'll help with users following the regex rules. Looks good!