Gamma Beta Coincidence

Hello,
I am making the gamma-beta coincidence plot for a single subrun and I see the spectrum is not continuous at higher energies.  I was wondering to ask why that happens?
I need to mention  it does not happen to gamma single spectrum for the same subrun and the gamma single spectrum looks smooth at higher energies.

.

I'm assuming by "not continuous" you mean that you do not have counts in all bins? I would think that is just a question of statistics. When you look at singles your statistics are a bit higher, so you have less probability to have no counts in a bin.

Yes, sorry for wrong terminology. I mean there are no counts in some bins.
Thank you.

I assume that means the question is answered to your satisfaction and this issue can be closed?

Thank you, this issue is mostly solved.
I added some more conditions to my code and by adding more statistics, it’s working fine.
I have another question before closing this issue. How can I print the outputs on the screen while proofing?
I added some print commands to my code. But, it’s not printing the data on the terminal?
Should I edit the source code?

Any output that you put in the worker part of the code, e.g. the functions that create or fill histograms, will get printed into the grsisort.log file that is produced every time you run grsiproof. This file will also get overwritten whenever you run grsiproof again, so you might want to set up a script that renames the file after the proofing is done.

Thank you so much.
The grsisort.log file is generated after the run is done. Is there any way that I can access the results and print them on the screen  meanwhile it is running the code?

No, there isn't.

Ok, thank you so much for your help.
Sometimes, my grsiproof does not generate the grsisort.log file. How can I  generate it after running each selector code?

You can't. If grsiproof crashes before writing out the log file you can't recover it. There might be a chance that something gets saved in the .proof directory in your home.

Unfortunately, there is nothing saved in .proof directory either.
Here, I copy a piece of my beta-gamma coincidence selector code.
Basically, I am trying to fill a 2D beta-gamma coincidence histogram.
In this experiment betas are detected by both Sceptar and ZeroDegree. To skip overwriting, I set a condition so that it fills the beta energies from one of those detectors. I want to make sure this code is working as expected.
I really appreciate any comments about it.
  for(auto g1 = 0; g1 < fGrif->GetHighGainMultiplicity(); ++g1){
   if(fScep != nullptr){
    if(fZero != nullptr){
     auto hit=0;
     for(auto b1 = 0; b1 < fScep->GetMultiplicity(); ++b1){
      if((fGrif->GetGriffinHighGainHit(g1)->GetTime()-fScep->GetHit(b1)->GetTime() >= gbTLow && fGrif->GetGriffinHighGainHit(g1)->GetTime()-fScep->GetHit(b1)->GetTime() <= gbTHigh) && (hit==0)){

         fH2.at("gEB")->Fill (fGrif->GetGriffinHighGainHit(g1)->GetEnergy(), fScep->GetHit(b1)->GetEnergy());
         auto hit=1;
      }

      if(auto hit=0){
   for(auto z1 = 0; z1 < fZero->GetMultiplicity(); ++z1){
        if((fGrif->GetGriffinHighGainHit(g1)->GetTime() - fZero->GetHit(z1)->GetTime() >= gZDTLow) && (fGrif->GetGriffinHighGainHit(g1)->GetTime() - fZero->GetHit(z1)->GetTime() <= gZDTHigh)){
           fH2.at("gEB")->Fill (fGrif->GetGriffinHighGainHit(g1)->GetEnergy(), fZero->GetHit(z1)->GetEnergy());
        }
       }
      }
     }
    }
   }
  }

The if(auto hit=0){ line will always evaluated to false so you will never fill the spectrum with zero degree data. You are declaring a new variable (auto hit), which will shadow (overwrite for this scope) the previously declared variable auto hit=0;, then you assign it the value zero (=0) and evaluate the result as a boolean and in C/C++ zero = false, non-zero = true. You probably meant to write if(hit == 0){? This appears everywhere in your code btw. You should only declare the variable hit once, not three times (if I counted right). The auto keyword just means "determine the type of this variable automatically", so anytime you say auto hit=0 or 1 is equivalent to int hit=0 or 1.
Also it looks like you are running the whole if(auto hit=0){ ... loop over the zero degree from within the loop over the sceptar hits. This is not necessary. It won't break anything but it will slow things down. In principle you would want something like this inside the griffin loop:
bool hit = false;
for(auto b1 = 0; b1 < fScep->GetMultiplicity() && !hit; ++b1) {
   if(timing is okay) {
      fill histogram;
      hit = true;
   }
}
for(auto z1 = 0; b1 < fZero->GetMultiplicity() && !hit; ++z1) {
   if(timing is okay) {
      fill histogram;
      hit = true;
   }
}

But even with that mistake this shouldn't crash your selector. Have you narrowed the reason for the crash down to this section? If so, make sure the histogram is actually created with the same name and type (2D histogram). If you haven't narrowed it down and the log files aren't created, the fastest way to narrow it down is to comment out whole blocks of code and then comment them back in after making sure that the rest of the code runs without problem.
Also,  if you've based this selector on the ExampleEventSelector in the $GRSISYS/GRISProof directory, you do not need the checks that fScep and fZero are not null pointers. The way the branch address is set (creating a new object if setting the branch address failed) ensures that these are never null pointers. If they are new object created because the branches do not exist in the file, their multiplicities are zero, so the for loops would never be run.

Thank you. I really appreciate your comments. I will modify the code and check the results again.
I have another question to ask. For beta energies, coming either from Sceptar or ZeroDegree, I see some negative values. Can I ask what is the reason?

I'm not a hundred percent sure, but I think this is noise that has managed to get above the trigger threshold.
The way the digitizer works, a baseline (determined by the firmware) is subtracted from the ADC signal and the result is integrated. If the signal only get above the trigger threshold for a very short while and then drops below the baseline the result would be negative. It could also be that for some event the baseline hasn't been determined correctly by the firmware, but it is hard to tell why this happened from the data alone.

Gate on it. If it looks like low-statistics singles spectrum then you know it’s noise.

@VinzenzBildstein Thank you. So, do I need to do pole zero correction ?

@r3dunlop Thank you so much. I will try!

No, this is not an issue of pole zero correction (that would be in the hardware/firmware anyway and not in the analysis software). Like Ryan said, if you want to make sure that it's just noise, gate on it and compare the spectrum to a singles spectrum and the spectrum gated on the higher energies. It should look the same as the former (pay attention to the comparative size of peaks from the decay of the beam isotopes and background peaks). If you compare it to the latter you should see a significant difference in the background peaks.

Awesome. Thank you so much.


nil error on email validation

If you add an email without domain to the domain whitelist, you can get nil errors.
https://rollbar.com/carto/CartoDB/items/25073/occurrences/33493893756/
Probably caused by the recent refactor to the email validation. Should be a matter of adding a nil-guard.

cc @ramiroaznar for RT prioritization


Add option to skip hidden/filtered rows

Add a switch parameter to skip hidden rows
Filter out hidden row indices after all other conditionals (header, startrow, dataonly, etc) have been processed for simplicity

Looks like a good feature. Please could you add a test for this so that we can be sure it works, and then also have a way to make it works in the future.

Thanks for the feedback! I've done tons of Powershell over the years, but
this is my first submission to someone else's work so I'm feeling my way
through it. Thanks for the patience. Any reference information you could
direct me to regarding the tests would be appreciated.
Also, thanks for this module. Love it and have used it for ages.
Joel Phillips
On Sat, Oct 3, 2020 at 7:08 PM Doug Finke<EMAIL_ADDRESS>wrote:

Looks like a good feature. Please could you add a test for this so that we
can be sure it works, and then also have a way to make it works in the
future.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dfinke/ImportExcel/pull/928#issuecomment-703179827,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGVMBOP52NSMVWXSOJ5KURTSI64HTANCNFSM4SB2VMPA
.

Great, take your time, no rush, and ask questions. Check out https://github.com/dfinke/ImportExcel/tree/master/tests. Use Invoke-Pester to run the tests. Do a cd __tests__ in where you cloned the ImportExcel. Then add a ps1 file, follow the name pattern. Initially, you'll want to create an xlsx set up so you can test that your logic works.

Very cool thanks so much!
On Sat, Oct 3, 2020, 7:51 PM Doug Finke<EMAIL_ADDRESS>wrote:

Great, take your time, no rush, and ask questions. Check out
https://github.com/dfinke/ImportExcel/tree/master/__tests__. Use
Invoke-Pester to run the tests. Do a cd tests in where you cloned the
ImportExcel. Then add a ps1 file, follow the name pattern. Initially,
you'll want to create an xlsx set up so you can test that your logic works.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dfinke/ImportExcel/pull/928#issuecomment-703182957,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGVMBON4KZFDFUNYU74LNV3SI7BJLANCNFSM4SB2VMPA
.

Looking good. Please add one more test, to make sure the records you get back  have the correct data from the rows that are not hidden.

Thanks, looks good. Will be merging and publishing soon. Thanks again for the contribution.

Awesome! Thanks so much for your help!
On Mon, Oct 5, 2020, 8:06 PM Doug Finke<EMAIL_ADDRESS>wrote:

Thanks, looks good. Will be merging and publishing soon. Thanks again for
the contribution.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dfinke/ImportExcel/pull/928#issuecomment-703970013,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGVMBOOW7CXU3TR5CLJFXNDSJJUP7ANCNFSM4SB2VMPA
.

Noticed this not merged.  Any plans to merge?

I still use this module daily and would still like this feature to be merged, but did my part (I think) and have just left it to the process.

@dfinke , this is a valuable feature.  Can you merge?


NMS without cuda

Is there any way to use NMS for inference in a system that has no GPU?

No, currently I assume that the program is run with GPU.


End user and admin notes are missing from the "Application" model

When listing applications directly from the Okta API, the response includes admin and enduser notes, however, these values are not returned when using the list_applications function in this SDK.

@hb3b Thank you for posting this issue. Surprisingly, notes attribute is not described in our openapi spec. I'll find out and come back to your issue soon.

I can confirm that sample Bookmark app includes notes within SDK response.

I'm closing this issue as stalled. Feel free to reopen this issue or create a new one if your problem exists still.


Insert vertex and edge without overwrite

Codecov Report
Patch coverage: 32.00% and project coverage change: -0.07 :warning:

Comparison is base (1e01411) 31.38% compared to head (07cb563) 31.31%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##             master      #95      +/-   ##
============================================
- Coverage     31.38%   31.31%   -0.07%
  Complexity       79       79
============================================
  Files            30       30
  Lines          1520     1536      +16
  Branches        239      241       +2
============================================
+ Hits            477      481       +4
- Misses          973      985      +12
  Partials         70       70

Impacted Files
Coverage Δ

...la/com/vesoft/nebula/connector/NebulaOptions.scala
0.00% <0.00%> (ø)

...n/scala/com/vesoft/nebula/connector/Template.scala
0.00% <0.00%> (ø)

...esoft/nebula/connector/writer/NebulaExecutor.scala
0.00% <0.00%> (ø)

...ala/com/vesoft/nebula/connector/NebulaConfig.scala
58.49% <50.00%> (-0.52%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


email queue not working properly

hi, i am trying to use redis as a cache driver the problem is email not get sent with redis cache. however, as I use the sync queue driver. the email gets sent.
i also used to check

echo $defaultDriver = app('queue')->getDefaultDriver();

the driver's name but with queuer driver as redis always failed.
one thing I noticed that handle function not gets executed with redis queue driver.
[2018-03-15 13:22:29][20] Processing: App\Mail\updateemail
[2018-03-15 13:22:29][21] Processing: App\Mail\updateemail
[2018-03-15 13:22:29][22] Processing: App\Mail\updateemail
[2018-03-15 13:22:30][23] Processing: App\Mail\updateemail
[2018-03-15 13:22:30][24] Processing: App\Mail\updateemail
[2018-03-15 13:22:30][25] Processing: App\Mail\updateemail
[2018-03-15 13:22:30][26] Processing: App\Mail\updateemail
[2018-03-15 13:22:31][27] Processing: App\Mail\updateemail
[2018-03-15 13:22:31][28] Processing: App\Mail\updateemail
[2018-03-15 13:22:31][29] Processing: App\Mail\updateemail
[2018-03-15 13:22:31][30] Processing: App\Mail\updateemail
[2018-03-15 13:22:32][31] Processing: App\Mail\updateemail
[2018-03-15 13:22:32][32] Processing: App\Mail\updateemail
[2018-03-15 13:22:32][33] Processing: App\Mail\updateemail
[2018-03-15 13:22:32][34] Processing: App\Mail\updateemail
[2018-03-15 13:22:32][35] Processing: App\Mail\updateemail
[2018-03-15 13:22:33][36] Processing: App\Mail\updateemail
[2018-03-15 13:22:33][37] Processing: App\Mail\updateemai

This looks like the job is being retried, please make sure there are no errors in the mail class.
You can see your error log here: storage/logs/laravel.log or try to send the email with the queue driver on sync for testing.

with sync emails are getting sent but with thew redis or beanstalkd , handle() not work. And hence emails are not being sent.

Seems like the code to send the email is running, but the job is reporting a failure, maybe because of an exception or similar being thrown after the email is actually sent.
Are you using a try-catch block around the code inside your handle method? to ensure that no email is sent if any other part of the code in your job fails?
Having an example of your Job would help us help you.

no i am not using any try and catch block. the same process works well if
used sync queue but not if uses Redis.
On 16 March 2018 at 15:24, David Gray<EMAIL_ADDRESS>wrote:

Seems like the code to send the email is running, but the job is reporting
a failure, maybe because of an exception or similar being thrown after the
email is actually sent.
Are you using a try-catch block around the code inside your handle method?
to ensure that no email is sent if any other part of the code in your job
fails?
Having an example of your Job would help us help you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/laravel/framework/issues/23550#issuecomment-373660962,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL3kJo9ouYi0H6g65Zt2UWiRqNg_-YLhks5te4vugaJpZM4Srs9X
.

It's possible the mail DOES work in sync and DOESN'T work when it's queued.
For example, when in sync you CAN use Auth::user() since the mail is sent in the context of the users request.
However, when you queue the mail, Auth::user() is no longer available since the mail isn't sent from the context of a request.
So, please check your log file to see if there are any errors: storage/logs/larvel.log.
Also please close this issue since github issues are not meant for support requests.

sorry still not solved the issue but i will have to use with sync

@Sarfroz  did you find any solution for this problem ?
i am facing the same issue, mail send with queue=sync bot not sending with redis


Calligraphy with CompileSDKVersion = 29

Thanks for nice library but have we had a plan for fix this?
https://stackoverflow.com/a/57232072/4411879

Please


better order mapping

This pull request ensures:

If the order number and the amount match, the order is mapped as "certain".
If the invoice number and the amount match, the order is mapped as "certain".
If just the order number or the invoice number match, the order is mapped as "guessed".
If just the amount matches, the order is mapped as "guessed".

:+1:

Thank you for the improvements.
I am willing to accept the changes if you fix the following two points:

You removed support for Banco Monex, a bank you probably don't support. However, other users might use it. Could you re-add it?
You use spacing for indentation. Although that would have been a better solution from the start, this change is not about the indentation. Could you change it back to tabs? This way it is very confusing for me (and GitHub) to see what has actually changed.

We of course could not test it, but we kept the Banco Monex code: https://github.com/norgeindian/bank-integration/blob/d5adce50d457f7bfaf3fbf0561e32a49906b99e6/app/code/community/Kinento/Bankintegration/Model/Bankintegration.php#L1093
Commit follows.

Would you accept a PR which formats the extension in Magento standard?

The modules was created way back in 2009, so you'll have to explain me what you mean with the "Magento standard"?

Thanks for merging!
I would like to change the code style to the Magento standard, which is afaik the Zend standard.

OK, I'm still not sure what that would mean in practise. In any case, I agree that some things are not done the Magento way and many things can be improved from a software engineering perspective.
I am happy to merge your contributions. But perhaps you'd like that I transfer ownership in some way to you? As you can read in the README file, Kinento has done commercial activities from 2009 until the end of 2014. In 2015, the only contributions which can be expected by Kinento are bug fixes for customers who purchased this module last year. If you would like to maintain, extend, or even commercialise a version of this module, please let me know!


Debugging c++ in openSUSE Tumbleweed is completely unusable

Debugging c++ in openSUSE Tumbleweed is completely unusable, and every program takes half an hour to load.

VS Code Version:1.632
OS Version: openSUSE Tumbleweed

Steps to Reproduce:

Any program debugging will stop here

/extCpp


Docs deploy

This PR adds a GitHub Action for auto docs deployment using github-pages-action. A workflow file was added along with a requirements file, modifications were made to .travis.yml.
When reviewing, please carefully examine the changes made to .travis.yml to see that they are only docs-related modifications.

@bryanwweber, if you have a minute, please take a look at the error I get during deployment.
Any ideas how to resolve it? Am I missing a package?

@bryanwweber, if you have a minute, please take a look at the error I get during deployment.
Any ideas how to resolve it? Am I missing a package?

Looks like you need to install Pandoc onto the virtual machine. You should be able to use apt in the Ubuntu images on GitHub actions to do that.

Looks like you need to install Pandoc onto the virtual machine. You should be able to use apt in the Ubuntu images on GitHub actions to do that.


Migration project to TypeScript and added Dockerfile.

mind me asking:

Why use DockerFile?
why swap to ts?

Answering the Dockerfile question:
We have the ability to run multiple applications, which makes it easier for us to share our containers and speed up the maintenance and development process. This way, we can avoid the typical comment “it works on my machine” and be able to confirm that it works in a “production” environment.
Answering the TypeScript question:
In the package.json, I had the TypeScript dependency unused. Additionally, the ForgeScript project is made in TypeScript, this will help to solve possible conflicts or errors; and TypeScript is a rigid extension of JavaScript.

You can create a dev-typescript branch or another branch to save the project’s history.

About using Dockerfile:

At this moment im still updating akira from its old package to ForgeScript, once this is done i will look into using Dockerfile.

About using ts:

Personally i have no experience with TS and i feel more comfortable with using JS the reason ts was as dependency was because it was needed in a previous version of ForgeScript.

I understand, I appreciate the time you took to review the pull request.


Add double and triple click drag selection

Seems good to me; one note is that it seems to be selecting spaces one at a time on word mode, whereas xfce4-terminal
selects the whole space block, but that's a pretty minor nit.

whereas xfce4-terminal selects the whole space block

Here it selects only the single space. Maybe you're testing on a tab char.

Oh, no, you're right, sorry. Dunno what happened there.


Correct qualified name of HttpResources

I'd have extracted a method to call for each class name, but this'll do


WIP: In package manifest, list use of extensions separately from other dependencies

In the draft proposal for Extensible Build Tools, the extension dependencies are listed separately from other dependencies.  While it's not clear whether the accepted proposal will list them separately or together with other dependencies, this is a WIP implementation at doing them separately.
Motivation:
[Explain here the context, and why you're making that change. What is the problem you're trying to solve.]
Modifications:

add PackageDescription API parallel to the dependencies
add ProjectModel and ProjectLoading structures parallel to the dependencies
change the ResolvedTargetBuilder and ResolvedTarget to track extension use

Still need ResolvedProductBuilder support and look at all places where dependencies are used to also take into account extension usage.

Withdrawing this and will put up a new one based on the name changes in SE-0303.


⚠ Samsung has degraded performance

In b6eedf5, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2047 ms

Resolved: Samsung performance has improved in b0a899e.


Introduce GitHub Container Registry use

Ref KF community meeting 2024-02-27
We note the KF community feedback suggesting to make use of GitHub Container Registry (part of "GitHub Packages") for publishing Container Images using GitHub Actions. Eventually, GH CR images can be synced to DockerHub https://hub.docker.com/u/kubeflow by raising request to KF community.
This issue tracks steps required to enable Container Image production and publication, similarly to what was available in ODH+Quay.
Acceptance criteria:

[ ] ensure this GHA works as expected: https://github.com/kubeflow/model-registry/blob/main/.github/workflows/build-image-pr.yml
[ ] modify this GHA and related script to publish with GitHub Container Registry as needed: https://github.com/kubeflow/model-registry/blob/main/.github/workflows/build-image-pr.yml
(this GHA and related script at the time of writing were used to publish to Quay)
[ ] re-enable this workflow: https://github.com/kubeflow/model-registry/actions/workflows/build-and-push-image.yml
[ ] verify Container Image get published in Container Registry
[ ] ask KF community leads to enable sync from GH CR to DockerHub for model-registry

@tarilabs since we are starting out fresh, can you please ensure you cross-compile for ARM64 in addition to AMD64?
Depending on how many container images you plan to have, you may or may not need something as complex as what we do for Kubeflow Notebooks. But either way, here are some links you might find useful that show using QEMU emulation and Docker BuildX:

GitHub Actions:

GHA Template (workflow_call):

https://github.com/kubeflow/kubeflow/blob/master/.github/workflows/example_notebook_servers_publish_TEMPLATE.yaml

GHA Root:

https://github.com/kubeflow/kubeflow/blob/master/.github/workflows/example_notebook_servers_publish.yaml

Makefile:

Makefile Template:

https://github.com/kubeflow/kubeflow/blob/master/components/example-notebook-servers/common.mk

Makefile Root:

https://github.com/kubeflow/kubeflow/blob/master/components/example-notebook-servers/Makefile

@thesuperzapper that's a very good point Mathew thanks especially for sharing examples of cross-platforms container image build!
While we have an extensive Integration Test suite, e2e testing on arm will be limited atm due to the MLMD image unavailability (for arm). That accounted for, we can start early (as you pointed out) to account for cross-platform build so we can be prepared when all components are available on arm imho.

@tarilabs well... I have a Christmas present for you regarding ARM64 support in the MLMD image.
I made a PR upstream to introduce ARM64 support, and also made a fork and published an ARM image (built under GHA emulation) for my own project (deployKF) until it gets merged upstream.
For more information, please see:

My ml-metadata PR:

https://github.com/google/ml-metadata/pull/188

The associated KFP Issue:

https://github.com/kubeflow/pipelines/issues/10308

The deployKF fork of the ml-metadata repo:

https://github.com/deployKF/ml-metadata

Forked gcr.io/tfx-oss-public/ml_metadata_store_server image (with ARM/AMD):

ghcr.io/deploykf/ci/ml_metadata_store_server:sha-cad0c56

This is all assuming you are talking about ghcr.io/deploykf/ci/ml_metadata_store_serve image needing ARM64 support, and not some other part of ml-metadata (which may actually already support it).

dockerhub token should be setup for this repo, as:
DOCKERHUB_USERNAME
DOCKERHUB_TOKEN

@rchincha thank you for reaching out, I invite you to comment the above on the Model Registry proposal for Kubeflow, here: https://github.com/kubeflow/community/pull/682
There is recurrent community call for Kubeflow: https://www.kubeflow.org/docs/about/community/#kubeflow-community-call and I strongly encourage you to engage at that level to spread best awareness of your initiative.
There is also bi-weekly call for Model Registry, as you can find in the above linked calendar.
Hope this helps! :)

ensure this GHA works as expected: https://github.com/kubeflow/model-registry/blob/main/.github/workflows/build-image-pr.yml
verify Container Image get published in Container Registry

demonstrated by: https://hub.docker.com/layers/kubeflow/model-registry/main-dc872c1/images/sha256-24e934a1ab01d07862fb5c8b98154281ea2c981cbc257dbd9b0b469661dab093?context=explore
and up the tree

modify this GHA and related script to publish with _ dockerhub Container Registry as needed: https://github.com/kubeflow/model-registry/blob/main/.github/workflows/build-image-pr.yml
(this GHA and related script at the time of writing were used to publish to Quay)

was done with:

https://github.com/kubeflow/model-registry/commit/c9bdfc21ab5495270b1f4a6b144958e23a88b250
https://github.com/kubeflow/model-registry/commit/784c9fb68bd2326b6f3c1efd99bb4455bfe1be84#diff-c7cbc4a36f236b91dd212d03b3a15dd7ea80de2a647ac34aeec2ba9bf70f8fc4

re-enable this workflow: https://github.com/kubeflow/model-registry/actions/workflows/build-and-push-image.yml

as demonstrated above, is re-enabled.


Allow to configure actions

Actions are only simple function with no parameters. We can imagine to allow user to customize an action parameter to avoid action duplication like rotation. We can have only one action rotate with an angle parameter for example.

An advantage to have action duplication are to be more fast to select a desired action: just by checking a box.
We could same show the action icon near check box.
At the same time, to be able to customize action could be interesting. It's more slower because we need tu fill a text item.
But how to represent these customized actions by an icon ?
All these actions use the same functions (with parameters).
Suggestions :

to keep both kind of action via two tabs : a tab "default" and a tab "advanced actions".
or, to avoid visual of action duplication, we can create a buton in place of check box plus label, with a general category. When we click it, we show the list of actions (for each, a check box associeted with a label), as like an tree structure from left to right. For example, with push the buton "Rotation", we show list "Rotation 30°", "Rotation 45°", etc.

An advantage to have action duplication are to be more fast to select a desired action: just by checking a box.
We could same show the action icon near check box.

Indeed, that's why it bother me ;-)  but I think we need configurable actions.

At the same time, to be able to customize action could be interesting. It's more slower because we need tu fill a text item.
But how to represent these customized actions by an icon ?

We can imagine dynamics svg icons? or multiple icons for same action. However, I think sometimes it's not important to have the exact action meaning in the icon if the action is obvious for this specific item.

All these actions use the same functions (with parameters).
Suggestions :
* to keep both kind of action via two tabs : a tab "default" and a tab "advanced actions".

* or, to avoid visual of action duplication, we can create a buton in place of check box plus label, with a general category. When we click it, we show the list of actions (for each, a check box associeted with a label), as like an tree structure from left to right. For example, with push the buton "Rotation", we show list "Rotation 30°", "Rotation 45°", etc.

Yes! I like the two ideas, but I prefer the second. We can also image a that one item in the tree is "Custom rotation" and only if you select it, you show an input to select the exact value. We can add a search box too.
An example:

Hide/Reveal

[ ] Hide/Reveal
[ ] Hide/Reveal and reverse item order
[ ] Hide/Reveal for me
[ ] Hide/Reveal for other

Item Rotation

[ ] Tap / Untap
[ ] Rotation 30°
[ ] Rotation 45°
[ ] Rotation 90°
[ ] Custom rotation

Stack actions

[ ] Stack
[ ] Shuffle
[ ] Align
[ ] Align as a square

...

It's good !

Can also useful to allow action composition like stack+shuffle or shuffle and random rotate.

Hi @Ribakys,
I've added the configurable action system. "Rotate" and "Randomly rotate" actions can be configured if you want to test.
I had to make something a little different than we discuss because I also want to solve #360
You'll also notice that most of the actions are not yet configurable but it's coming ;-)

Can also useful to allow action composition like stack+shuffle or shuffle and random rotate.

This could be useful for example, for a "phase of initiation of a new turn" with several actions (always the same) to be carried out in succession.

Hi @Ribakys,
I've added the configurable action system. "Rotate" and "Randomly rotate" actions can be configured if you want to test.
I had to make something a little different than we discuss because I also want to solve #360
You'll also notice that most of the actions are not yet configurable but it's coming ;-)
It's a first version and need to be improved.

Hi @jrmi
That works well.
The "edit", "up", "down" and "delete" icons are a very good idea. I find it convenient.
This opens the way to many possibilities.

Can also useful to allow action composition like stack+shuffle or shuffle and random rotate.

This could be useful for example, for a "phase of initiation of a new turn" with several actions (always the same) to be carried out in succession.

I fully agree, I'll open another ticket for that.


Configure Inject Plugin based on Props

Now the auto provided jsonb can be configured by inject
Requires https://github.com/avaje/avaje-inject/pull/324

I'm waffling on these property names, feel free to change.

I'll also check that they default to match the defaults that we have


STDOUT not working

Hi, I've tried for the first time using this image, and I noticed that the STDOUT is not working as I expected, since I don't see any connect or transfer messages.
I'm starting the container using docker-compose, and I set the Environment Variable as documented.
However, after testing for sometime, I only saw one single log message on the console.
So I opened a terminal inside the running container, and I noticed there are 2 vsftpd.log files under /var/log, as follows:
/var/log:
[root@6b28dbacc585 log]# find . -ls | grep vsftp
1104614   16 -rw-------   1 root     root        14757 Nov  9 19:18 ./vsftpd.log
970431    4 drwxr-xr-x   2 root     root         4096 Nov  9 19:00 ./vsftpd
970441    0 lrwxrwxrwx   1 root     root           11 Nov  9 19:00 ./vsftpd/vsftpd.log -> /dev/stdout

Apparently, the location /var/log/vsftpd/vsftpd.log has been corretly redirected to /dev/stdout, but the most messages are being logged into a different location, /var/log/vsftpd.log.
I checked the configuration file being used, and it does point to the location that is redirected to stdout, but for some reason, the situation I just described is taking place.
/etc/vsftpd/vsftpd.conf:
[root@6b28dbacc585 log]# grep log /etc/vsftpd/vsftpd.conf
# Uncomment this to allow local users to log in.
## Enable logging
xferlog_enable=YES
xferlog_file=/var/log/vsftpd/vsftpd.log
xferlog_std_format=NO

Am I doing something wrong, or is there maybe a problem here?
I appreciate any support.

+1 to the above. I am facing the exact same issue.

+1 to the above. I am facing the exact same issue.

I guess this is because "xferlog_file" is only used if "xferlog_std_format=YES". Otherwise the path is driven by "vsftpd_log_file" which is not defined in this container.

Looks like vsftpd logs to /var/log/vsftpd.log and not /var/log/vsftpd/vsftpd.log so the the run-vsftpd.sh script is incorrect.

Is this repository still 'active'??
I also cannot see logs in STDOUT.

It is. Labeling this issue as look.

yeah, I followed the documentation on DockerHub https://hub.docker.com/r/fauria/vsftpd
LOG_STDOUT doesn't seem to do anything. thanks for checking into it tho :)


Update Zerocracy badge location

Job gh:dronmdf/telebot#25 is in scope.

Bug was reported: +15 points just awarded to @dronmdf, total is +405.

Job gh:dronmdf/telebot#25 assigned to @pavelonex, please go ahead (policy).

@PavelOnex Для подтверждения в Review нужно ставить статус Approved.

@0crat out

@0crat out (here)

@DronMDF @pavelonex resigned from gh:dronmdf/telebot#25, since the job is not in scope anymore.

@0crat out (here)

@DronMDF Job gh:dronmdf/telebot#25 is now out of scope.


Office 365 Detections

Story File description and narrative missing
Bhavin: Adding detection and test file for "High Number of Login Failures from a single source"

This PR was created by Automated Detection Testing Service :robot:
Tested detections:
O365 Suspicious Rights Delegation :white_check_mark:

diskUsage: 126976
runDuration: 0.548s
scanCount: 4

This PR was created by Automated Detection Testing Service :robot:
Tested detections:
O365 Suspicious Rights Delegation :white_check_mark:

diskUsage: 126976
runDuration: 0.561s
scanCount: 4

This PR was created by Automated Detection Testing Service :robot:
Tested detections:
O365 Suspicious User Email Forwarding :white_check_mark:

diskUsage: 126976
runDuration: 0.545s
scanCount: 77

This PR was created by Automated Detection Testing Service :robot:
Tested detections:
O365 Suspicious Admin Email Forwarding :white_check_mark:

diskUsage: 122880
runDuration: 0.541s
scanCount: 77

@miskoSplunk : Good eye, let me chat with @rosplk tomorrow and get this resolved.

This PR was created by Automated Detection Testing Service :robot:
Tested detections:
O365 Disable MFA :white_check_mark:

diskUsage: 118784
runDuration: 0.549s
scanCount: 1

This PR was created by Automated Detection Testing Service :robot:
Tested detections:
O365 Excessive Authentication Failures Alert :x:

diskUsage: 122880
runDuration: 0.532s
scanCount: 0

This PR was created by Automated Detection Testing Service :robot:
Tested detections:
O365 PST export alert :white_check_mark:

diskUsage: 118784
runDuration: 0.556s
scanCount: 14

This PR was created by Automated Detection Testing Service :robot:
Tested detections:
O365 Excessive Authentication Failures Alert :white_check_mark:

diskUsage: 126976
runDuration: 0.5650000000000001s
scanCount: 132

This PR was created by Automated Detection Testing Service :robot:
Tested detections:
High Number of Login Failures from a single source :x:

diskUsage: 126976
runDuration: 0.518s
scanCount: 0

To do in Word what you seem to be referring to go to Tools> Protect Document... then clear the check for Privacy - Remove personal information from this file on save. Note that this is a per-document setting, not one that is global for the program.

@Rickbale606 not sure I fully understood your comment, where you meant to put that somewhere else? :smile:

@Rickbale606 not sure I fully understood your comment, where you meant to put that somewhere else? :smile:


How to use

Hi， i am intersting to your work ,but i do the readme test zhe bench_make  , i got:
Benchmark: Betweenness Centrality
loading data...
undirected graph
== 1000 vertices  29790 edges
== time: 0.0504179 sec
computing BC for all vertices...
cannot open perf event: type-0  config-cannot open perf event: type-0  config-00
cannot open perf event: type-0  config-1
cannot open perf event: type-0  config-4
cannot open perf event: type-0  config-5
cannot open perf event: type-0  config-1
cannot open perf event: type-0  config-4
cannot open perf event: type-0  config-5
cannot open perf event: type-cannot open perf event: type-3  config-03
config-0cannot open perf event: type-3  config-65536
cannot open perf event: type-3  config-65536
== finish
== time: 0.344001 sec
PERF_COUNT_HW_CPU_CYCLES	==>	0
PERF_COUNT_HW_INSTRUCTIONS	==>	0
PERF_COUNT_HW_BRANCH_INSTRUCTIONS	==>	0
PERF_COUNT_HW_BRANCH_MISSES	==>	0
PERF_COUNT_HW_CACHE_L1D_READ_ACCESS	==>	0
PERF_COUNT_HW_CACHE_L1D_READ_MISS	==>	0
can you help me to explanation why all zeros
where have quesations

Hi, I have encountered with the same problem. Any thoughts on how to solve it?


Datepicker placeholder ondersteuning

Is your feature request related to a problem? Please describe.
We cannot currently add a placeholder to a datepicker component.
Is it possible to add support for this?
Describe the solution you'd like
I'd like to be able to add a placeholder prop to the datepicker component:
<Datepicker
    ...otherProps
    placeholder="a-placeholder"
 />

This was added in v6.4.0


Increase test coverage

https://trello.com/c/hbxd3ygI/19-increase-test-coverage

Let's worry about this later.


Upgrade Spark to 2.0.2

Current coverage is 63.59% (diff: 100%)

Merging #1789 into master will not change coverage

@@             master      #1789   diff @@
==========================================
  Files           357        357
  Lines         17083      17083
  Methods       16719      16719
  Messages          0          0
  Branches        219        219
==========================================
  Hits          10864      10864
  Misses         6219       6219
  Partials          0          0

Powered by Codecov. Last update 8c2990d...3cbcbd2

@rabbitonweb This should be a really easy review 😄

@jedesah i was about to upgrade to 2.1.0 :)  Can we change that to 2.1.0?

Since the build on this one is green, I think it would better to merge this one and then open a new PR to upgrade to 2.1.0. In this case, if the 2.1.0 PR gets in very soon, it won't change much, but I think in general it's best to try and be as incremental as possible on these things.

We should upgrade to latest Spark 2.1.0 - especially since they've added one feature we were missing on our demo environment.
@jedesah I've added you as reviewer there https://github.com/quasar-analytics/quasar/pull/1870

We should upgrade to latest Spark 2.1.0 - especially since they've added one feature we were missing on our demo environment.
@jedesah I've added you as reviewer there #1870


Snackbar notifications

Implement snackbar notifications to let user know that changes have been saved, errors, etc.

Snackbar has been implemented for page save + all resource create, edit, and delete functionalities.


chore(libzkp): fix modexp

this branch is v4.3.74 + a fix commit

Superseded by https://github.com/scroll-tech/scroll/pull/1267.


fix: sortMenusByFrontmatterOrder not been sorted in numerical order but in string order

The behavior inconsistent with documentation:

Sorts the menu items by the order property of the frontmatter. For each folder, sorts the value (number) of the order property in ascending order, or descending order if the sortMenusOrderByDescending option is true. If the value of order is non-numeric or does not exist, order is judged to be 0.

The test samples are also unreasonable. Using 1 2 3 4 as samples will not be able to detect this error.

Hello! Thank you for your contribution to the project.
As you describe, it should have been sorted a little more clearly based on the values in frontmatter, I missed this point.
I don't see any particular problem with the code, so I'll merge it and release a new version. thanks!

This change was released in vitepress-sidebar 1.27.1.


Can’t install on drive, potentially package errors?

After I tell it to format a drive it errors out with "failed to install packages to new root"
Also
"Failed to run command /root/ArchTitus/1-setup.sh"
Also
Up in the terminal is some errors about how core.db, extra.db, community.db failed to download

Fixed with new install script


Kernel creation feels slower

    My impression of the slowness that @DonJayamanne mentioned

In stable and insiders, I think the run button shows up at about the same time. Around 7-9s from "reload window" to the run button appearing, hard to tell whether stable is slightly faster.
But after it appears, and the preferred kernel has been discovered, I see "Discovering Kernels" for another 5s or more, which could lead to an impression of slowness if the user doesn't realize they can go ahead and run a cell at this point.
Originally posted by @roblourens in https://github.com/microsoft/vscode-jupyter/issues/11963#issuecomment-1331449598

When there is already a selected kernel, we should show the kernel name and a spinner if there are pending detection tasks.


siad will crash when upload more then 64 files by siac

BUG REPORT
Stack Trace or error message
fatal error: runtime: out of memory
Expected Behavior
How to reproduce it (as minimally and precisely as possible)

create one file in 1mb;
dd if=/dev/zero of=./1m.txt bs=1M count=1
siac to upload this file 64 times at the same time
siac renter upload ./1m.txt 1m_001.txt
......
siac renter upload ./1m.txt 1m_0063.txt
a few hours later, siad will crash.
please refer to attached logs
siad.txt

Environment
ubuntu 16.04
mem: 4g

Sia version:1.3.2
OS: ubuntu 16.04

Sia does not handle small files very well, each of your 1MB files is being padded to 40MB before being uploaded.  I would think Sia should be able to handle more than 64 files at 1MB but Sia is also fairly RAM intensive.  With only 4GB of RAM it looks like you're running out of usable memory.
I'm going to close this issue for a couple reasons.
1: You've run out of RAM, unless you can demonstrate a memory leak there is no action for the Sia devs to take.
2: The devs are very aware that Sia performs suboptimally with files smaller than 40MB.  Smaller file support is on the feature roadmap, and the devs are actively working on improving this limitation.


Feature: #[trace("a-span", black-box=true)]

This feature request arose in the course of iterating on an integration test for the proc-macro-attribute #[trace(...)].
use minitrace::trace;
use regex::Regex;
use test_utilities::*;

#[trace("test-span")]
fn f(a: u32) -> u32 {
    a
}

fn main() {
    let (root, collector) = minitrace::Span::root("root");
    {
        root.set_local_parent();
        f(1);
    }
    drop(root);
    let records: Vec<minitrace::collector::SpanRecord> =
        futures::executor::block_on(collector.collect());
    let expected = r#"[ ... ]"#;
    let pre = format!("{records:#?}");
    let re1 = Regex::new(r"begin_unix_time_ns: \d+,").unwrap();
    let re2 = Regex::new(r"duration_ns: \d+,").unwrap();
    let int: std::string::String = re1.replace_all(&pre, r"begin_unix_time_ns: \d+,").into();
    let actual: std::string::String = re2.replace_all(&int, r"duration_ns: \d+,").into();
    assert_eq_text!(expected, &actual);
}

The black-box feature requires nightly.
Requirements:

The macro should report the nightly requirement in any error messages related to black-box=true.
The default is black-box=false

Could you give more context about why black_box is needed?

Originally I thought issue #126 was caused by the compiler doing something funky - know we know its a bug we can eliminate the compiler as the root cause.
First, I can image this situation arising again, and it'd help to eliminate (some?) compiler optimizations as an issue.
Second, I can envision #[trace] being used with the Criterion crate - even through I've advocated minitrace using iai - wherein black-box is an option.

From my personal point of view, black_box will not affect the behavior of a program, but only the performance. And further, using black_box in benchmark code is sufficient to ensure the correctness of criterion.

The more I think about this the more I feel that by the time it lands it won't really be needed.
I may have saved some time by wrapping the fn in question in black-box but I could have done that by hand.  Hopefully the phase of many bugs will shortly be passed and this is not longer a convenience.  Which makes this a catch-22 change - it'll only land at a point when it is no longer useful - after the proc-macro test suite is complete.
Closing.


Events with no location export with location = “none” string

To reproduce:

Open http://events.olin.build/
Copy ICS link to Clipboard
Open https://calendar.google.com/
“Add by URL” the ICS link

Expected:

Events with no location will not display location text in the Google Calendar day, week, or month views; e.g. “Labor Day”

Actual:

Events with no location display “none” as the location; e.g. “Labor Day – none”. See screenshots.

Screen shots
ABE event

Google Calendar month view

Google Calendar event detail

This is a backend issue more than a frontend issue (issue opened), but I suppose we could make sure we don't send location if it's empty. Currently it looks like we'll send an empty string.

We should probably not have anything for location in the event data that goes to the server if the location field is blank. Then we'll need to make sure the backend knows not to export None (which looks like a Python artifact) when generating an ICS feed.

You'll want to use the Firefox developer tools to watch what data is being sent back and forth to the server.


Feature add logging system

Adds functionality to issue #54
Feature
This PR enables configuring multiple logging outputs and adds a helper to make logging easier.
Configuration

folder specifies a sub-folder within the storage folder where all logs get stored
prefix used as DateFormat to prepend names of log files
output defines a logging file, can be one for every log level

Usage
logTo('debug', 'MyMessage', $myContextArray);

Disclaimer
I based this PR on PR#328 in order to more easily work with config variables.
If required, I'd rewrite the code to work without the ConfigManager.

Works great!
Thanks again for your work.


Needs support for 1440p & 2160p

https://www.youtube.com/watch?v=rdQ0i3v_KVY&list=PL5BF9E09ECEC8F88F&index=5
shows auto where 1440 and 2160 are located.
My test wordpress site (ignore styling issue not related) if you want to see the issue:
http://dev.stopspazzing.com/

It has been already fixed on the master

I used composer to install this plugin, which is the one on the site, doesn't it pull from master?

video.js is on composer but I don't think videojs-youtube is. How did you get it?


Unable to apply affinity settings to knative-serving

I was trying to apply affinity settings as I thought affinity spec was now available based on this issue:https://github.com/knative/build/issues/116.  But I get an error when trying to apply affinity setting.
Following is the sh file that I am using
kind: KnativeServing
metadata:
  name: knative-serving
  namespace: knative-serving
spec:
  high-availability:
    replicas: 3
  # Set the label so the anti-affinity can pick it up
  podLabels:
    affinity_app: knative
  affinity: #This rule ensures that two replicas of the same application should not run if possible in the same Kubernetes node
    podAntiAffinity:
      preferredDuringSchedulingIgnoredDuringExecution:
      - weight: 100
        podAffinityTerm:
          labelSelector:
            matchExpressions:
            - key: affinity_app
              operator: In
              values:
              - kantive
          topologyKey: kubernetes.io/hostname

Error log
$ sudo sh knative-serving.sh
namespace/knative-serving created
peerauthentication.security.istio.io/default created
Installing knative-serving
error: error validating "STDIN": error validating data: [ValidationError(KnativeServing.spec): unknown field "affinity" in dev.knative.operator.v1alpha1.KnativeServing.spec, ValidationError(KnativeServing.spec): unknown field "podLabels" in dev.knative.operator.v1alpha1.KnativeServing.spec]; if you choose to ignore these errors, turn validation off with --validate=false

The knative serving CR does not support spec.podLabels or spec.affinity, so with operator we cannot configure the affinity with the knativeserving CR.
For something operator is unable to configure, I usually go to the https://knative.dev/docs/install/knative-with-operators/#installing-the-knative-serving-component, and check on Install Customized Knative Serving, spec.additionalManifests.


Path type

Hi @dmitrisweb,
Hope you well? Can we possibly have the Raml folder path (options.path) to support an array type (array of directiory paths) as well as the original string type.
Regards

Closed. Issue raised in Raml-mocker


Fix PPoly integrate and test for CubicSpline

Fixes which variable should become readonly.
Also fixes read only issues with PPoly.integrate @ev-br

@ev-br This might be all we can do till the fused types const fixes are made for Cython

@tylerjereddy Id add the defect label to this too; no reason PPoly shouldnt take read-only memviews

Thanks @dmadeka , it's in.


adding switzerlandwest australiasoutheast to pipelines

Which issue this PR addresses:
Part of Azure DevOps 7065216 and 7065243 (rollout of australiasoutheast and switzerlandwest regions)
What this PR does / why we need it:
Adds the 2 new regions to Prod Deploy ALL Regions, and E2E tests
Test plan for issue:
N/A
Is there any documentation that needs to be updated for this PR?
No documentation update required.

/skip

/azp run e2e


[Bug]: @rnmapbox/maps 10.1.3 not working with react native v0.64.4 in android

Mapbox Implementation
Mapbox
Mapbox Version
default
Platform
Android
@rnmapbox/maps version
10.1.3
Standalone component to reproduce
import React from 'react';
import {
  MapView,
  ShapeSource,
  LineLayer,
  Camera,
} from '@rnmapbox/maps';

const aLine = {
  type: 'LineString',
  coordinates: [
    [-74.00597, 40.71427],
    [-74.00697, 40.71527],
  ],
};

class BugReportExample extends React.Component {
  render() {
    return (
      <MapView style={{flex: 1}}>
        <Camera centerCoordinate={[-74.00597, 40.71427]} zoomLevel={14} />
        <ShapeSource id="idStreetLayer" shape={aLine}>
          <LineLayer id="idStreetLayer" />
        </ShapeSource>
      </MapView>
    );
  }
}

Observed behavior and steps to reproduce
Install @rnmapbox/maps by following complete instruction mentioned in documentation.
On creating build get error

FAILURE: Build completed with 2 failures.

1: Task failed with an exception.
-----------
* Where:
Build file<EMAIL_ADDRESS>line: 46

* What went wrong:
A problem occurred evaluating project ':rnmapbox_maps'.
> Could not set unknown property 'namespace' for extension 'android' of type com.android.build.gradle.LibraryExtension.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
==============================================================================

2: Task failed with an exception.
-----------
* What went wrong:
A problem occurred configuring project ':rnmapbox_maps'.
> compileSdkVersion is not specified. Please add it to build.gradle

My build.gradle code
buildscript {
    ext {
        buildToolsVersion = "29.0.3"
        minSdkVersion = 21
        compileSdkVersion = 29
        targetSdkVersion = 29
        ndkVersion = "20.1.5948944"
        kotlin_version = '1.6.10'
    }
   ... rest
 }

 ...rest

 allprojects {
    repositories {
        mavenLocal()
        maven {
            // All of React Native (JS, Obj-C sources, Android binaries) is installed from npm
            url("$rootDir/../node_modules/react-native/android")
        }
        maven {
            // Android JSC is installed from npm
            url("$rootDir/../node_modules/jsc-android/dist")
        }

        google()
        jcenter()
        maven { url 'https://www.jitpack.io' }
        maven {
            url 'https://api.mapbox.com/downloads/v2/releases/maven'
            authentication {
                basic(BasicAuthentication)
            }
            credentials {
                // Do not change the username below.
                // This should always be `mapbox` (not your username).
                username = 'mapbox'
                // Use the secret token you stored in gradle.properties as the password
                password = project.properties['MAPBOX_DOWNLOADS_TOKEN'] ?: ""
            }
        }
    }
}

Expected behavior
Should be able to make build
Notes / preliminary analysis
No response
Additional links and references
No response

We’re not supporting <rn 69 at the moment

Dependencies
node
npm
React Native (0.64+)
As per documentation it just mention that native version needs to be 0.64+
Although thanks for response will  revert to previous version.

Thanks will correct deps info in docs and pkg.json
FWIW the namespace could be made compatible with older RN - see
https://github.com/react-native-community/discussions-and-proposals/issues/671#issuecomment-1845138906


[BUG] Scaffolding dropdowns for Enum values

Affected Version
I'm using 4.8.0 version, but I think 4.* if affected
SS 3.* is ok
Description
CMS doesn't scaffolds dropdown field values when you use dots at Enum values
Steps to Reproduce
Following code snippet will work properly
private static $db = [
        'DollarValue' => 'Enum(["25","50","75","100"])',
];

This code will create db fields, the dropdown will be scaffolded, but the values of the dropdown are missing
private static $db = [
        'DollarValue' => 'Enum(["25.25","50.00","75.00","100.50"])',
];

Replication code:
MyDataObject.php
<?php

use SilverStripe\ORM\DataObject;

class MyDataObject extends DataObject
{
    private static $db = [
        'DollarValue' => 'Enum("25,50,75,100", "50")',
        'DollarValueDot' => 'Enum("25.25,50.00,75.00,100.50", "50.00")',
    ];
}

DataAdmin.php
<?php

use SilverStripe\Admin\ModelAdmin;

class DataAdmin extends ModelAdmin
{
    private static $managed_models = ['MyDataObject'];

    private static $url_segment = 'data';
}

When creating a new object in the model admin

Issue is here https://github.com/silverstripe/silverstripe-framework/blob/4.8/src/ORM/DataObject.php#L3153
For DollarValue, $spec is 'Enum("25,50,75,100", "50")'
For DollarValueDot, $spec is malformed as 'Enum("25' and the field doesn't construct properly

PR https://github.com/silverstripe/silverstripe-framework/pull/10005

Linked PR has been merged, closing this issue now


apps/v1beta1 is not valid

Replacing it with extensions/v1beta1 which actually works
Found this file from this document:
https://kubernetes.io/docs/concepts/workloads/controllers/deployment/

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.7 Features: set Milestone to 1.7 and Base Branch to release-1.7
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
NOTE: Please check the “Allow edits from maintainers” box below to allow
reviewers fix problems on your patch and speed up the review process.
Please delete this note before submitting the pull request.

This change is

apps/v1beta1 is correct, and newer. https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG.md#deployment-2

@foxish So can I close this PR?

When was this changed?
It still fails for me with this kubectl version:
Client Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.2", GitCommit:"477efc3cbe6a7effca06bd1452fa356e2201e1ee", GitTreeState:"clean", BuildDate:"2017-04-19T20:33:11Z", GoVersion:"go1.7.5", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"5", GitVersion:"v1.5.7", GitCommit:"8eb75a5810cba92ccad845ca360cf924f2385881", GitTreeState:"clean", BuildDate:"2017-04-27T09:42:05Z", GoVersion:"go1.7.5", Compiler:"gc", Platform:"linux/amd64"}

Trying to create a deployment using apps/v1beta1 gives:
error: error validating "kubernetes.yaml": error validating data: couldn't find type: v1beta1.Deployment; if you choose to ignore these errors, turn validation off with --validate=false

The server version needs to be 1.6 or higher for it to work. https://github.com/kubernetes/kubernetes/pull/39683 added the new apps/v1beta1.Deployment which will become the default eventually and we'll deprecate extensions/v1beta1.Deployment by the year end. See https://github.com/kubernetes/kubernetes/issues/43214

In the absence of versioned docs (as opposed to docs that reflect the latest version only), we can't make this change. cc @janetkuo @kubernetes/sig-apps-misc

Yeah, we really need versioned docs.

Agreed, versioned docs are essential here.
In the meanwhile, could we add a note about how in previous versions the value is different?
This was very confusing for me

I think a note makes total sense! @campoy, can you modify your PR to add that? Also I think you'll need to sign the CLA. It can be in manner similar to how the node controller specifies pre-1.5 behavior. Thanks!
@chenopis, In the long term, perhaps we should come up with a standard way to call out such changes in docs.

@kubernetes/sig-docs-maintainers is there any discussion somewhere re versioned docs?

PTAL

I am authorized to contribute, as a Google employee

I was thinking more along specifying the constraint in the text description surrounding the YAML. @chenopis WDYT of comments inline in the YAML itself?

Well, I don't know from how many places this specific file is embedded.
If it's only one I could easily add a note there

Yeah, I like this solution. YAML comments works for me! /lgtm

ping

Agree with @chenopis, lgtm

I saw a similar fix (closed) for this earlier, which proved that this is confusing enough for a fix

/lgtm


What to do in eclipse so it builds the proto files

Building 2.10.1 works fine from the commandline as gradle has the plugins to compile the .proto files. With Eclipse these files dont get generated even if conductor is imported as a gradle project. What is the preferred way to build the grpc modules in eclipse?

This is what I do.

run gradlew build in the root folder of conductor. This will generate all java sources from proto files.
Add source folder per the image.

Thanks for the help. Will do the same.


How to create a multi-branch-project via REST API?

Hi,
I was wondering if it is possible to create a new multi-branch-project via POSTing a config.xml to http://JENKINS_URL/createItem?name=newJobName
I tried playing around a bit, and the closest I came up with was:
Merging the config.xmls from the parent job and the master branch job (e.g: the parent config.xml did not contain any build steps, but the master branch job did).
This will create the new job, but will state "SCM not configured". If you then klick on Configure, all the Git infos are pre-polulated, a click on "save" without any changes then seems to set up the parent job correctly, but build and post-build steps are still missing.

You'll have to make two requests.  One to post to the URL you mention.  Don't try to add extra stuff, just use an XML format from another multi-branch project as an example.  This is where you will configure the SCM.
Your second request will be a POST to ${JENKINS_URL}/job/${JOB_NAME}/template/config.xml.  This is where the build steps are configured.  Make sure you do not set the SCM here.  And keep it disabled.  If you add an SCM it will break the job altogether (I might've tried to add a safeguard for that but I don't remember -- if I did your SCM block will get overwritten anyway so don't bother).  Leave it as NullSCM.  Use another template's config.xml as an example since it is different from the main job and slightly different from the sub-projects.

i never marked this one as closed, sorry for that! Works like a charm, thanks for the great plugin!


scripts: west_commands: openocd: Use 'reset init' by default

Instead of 'reset halt', default the command to reset and halt the target to 'reset init'.
OpenOCD strictly requires 'reset init' to halt and prepare the device before flash programming.
See https://openocd.org/doc/html/Flash-Commands.html chapter "12.2 Preparing a Target before Flash Programming."
For targets implementing the init hook, it will allows to perform some target specific init improvements (such as speed flash improvements). Targets not implementing this hook will end up executing the usual and previous 'halt'.
Highly suggested here by openocd maintainer (@tom-van).
Signed-off-by: Erwan Gouriou<EMAIL_ADDRESS>

👍


Clarified explanation about TODOs in README (Closes #652)

Checklist

[x] Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
[x] If this PR comes from a fork, please Allow edits from maintainers
[x] Set a speaking title. Format: {task_name} (closes #{issue_number}). For example: Use logger (closes # 41)
[x] Link your Pull Request to an issue (if applicable)
[x] Create Work In Progress [WIP] pull requests only if you need clarification or an explicit review before you can continue your work item.
[x] Make sure that your PR is not introducing unnecessary reformatting (e.g., introduced by on-save hooks in your IDE)
[x] Make sure that your PR does not contain changes in strings.xml (see issue #332)
[x] Make sure that your PR does not contain compiled sources (already set by the default .gitignore) and / or binary files

Description

This clarifies the note about TODOs and fixes a grammatical error (TODO's is not a plural but a possesive).

@marcmuschko just highlighting you as you were involved in the issue. I added a minor clarification on TODOs to denote "this is for developers" and fixed a grammatical error.

Accidental duplicate of #661


fix issue of login and signup form ui

herer i think ui of got exchanged

Do you want to fix it ?

can you assign me this issue

@MinavKaria can you assign it to me?

@MinavKaria Worked on it and fixed it , Do review and merger Accordingly


Ability to force lowercase tags. Management commands to convert mixed case to lowercase tags.

Case insensitive tags was added in https://github.com/alex/django-taggit/pull/325 and is available in the latest version of taggit (0.16.0)


Test PyGranta BoMAnalytics against 2024 R2

Once we get closer to release we should test against 2024 R2. This can be achieved by pointing the main branch at the new 'next' test VM.

Currently blocked: CR-1489


Fix SpriteBatch crash for rendering inventory

Should fix the issue described here.

Whoops. Got it!

Hmm, it seems to crash if I have the start and end added there because it's already been started. Removing the start/end fixes it, so I'm updated that for now.

@MrIbby  Hmm, when synced with this repository without this change, I'm not getting the crash you are.

Yeah, normally I don't get the crash. When I got the exception, I was clicking a lot of stuff.

Okay, it turns out I was using the inventory screen incorrectly by rendering it myself. Pushing out a PR which puts the SpriteBatch start/end back in InventoryScreen (which should fix this), and uses inventory screens properly so the chest can be closed.


Fixed broken singleAsteriskRegex

Fixes #9918
Unfortunately it's unveils other gulp related errors not related to the regexp.

Hi @Artazor, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Ooops, need to fix. I've thought jake is deprecated in favour of gulp but seems that jake is the only stable way to build TS

Now efficiency of the regexp is improved while accepting the same regular language. All tests are passed.

LGTM

I'm not seeing this change really fix the issue when manually applied (gulp clean (and all other gulp tasks) takes forever to start). While this change may be fine to make (and an improvement to our RegExp efficiency), I think there's something else going on. It looks, to me, as though ts-node is looking up way more files than it should. (Like, it's traversing our entire baselines directory looking for files...). I think I'll lock our ts-node version to 1.1 (which didn't have this problem) while I troubleshoot the issue.

I've opened #9933, which should be merged alongside this while the issue in ts-node is investigated.

@weswigham I've already traced a lot of the stuff: when ts-node  requires any ts file it firstly tries to find a project file (tsconfig), to read include/exclude info there from. And if the project config is not found (which is true for our project) then the include pattern is assumed '**/' + '.ts' and '*.d.ts' which yields traversing all the files (including baselines).

@Artazor Ack, that's quite a painful change in behavior for us - it used to just add the require'd file and anything it included. (How does that only cause a minor version bump? :tongue:)

Anyways, @RyanCavanaugh  LGTM'd this, and I :+1: it. I'll merge this.

Here is inglorious call stack.


Define lapack_int as long long for 64-bit indexing on Windows.

Windows 64-bit uses LLP64 data model.
Without this change, unexpected errors or segfaults occur when configuring with -DBINARY=64 -DINTERFACE64=1 on Windows.

Some check were failing. But I'm not sure how they can be related to the proposed change in this PR.

Thanks. Looks like I somehow missed https://github.com/Reference-LAPACK/lapack/pull/624 (which defines it as int64_t and at least to me looks to be a simpler fix (?))

I wasn't sure which is the minimum C standard version that is required to compile OpenBLAS. But if you require C99 or newer anyway, using int64_t would indeed simplify that part of the code.

Good point - we do require C99 on x86_64 but that is mainly due to feature creep in recent kernels. Pretty sure attempting to support VS2010 or earlier would be pointless (ISTR we need 2013 or newer for various reasons).
The entire lapack-netlib tree is imported from Reference-LAPACK (with some functions subsequently overridden by what is in lapack) so minimal difference would be a plus. (Perhaps long-time members of the LAPACK team recall what the "weird NEC"
macro was about - might well be specific to some particularly exotic 1990s hardware)

I cherry-picked that commit to the lapack-netlib folder.
I didn't know what to make out of the following hunk:
--- lapack-netlib/CMakeLists.txt
+++ lapack-netlib/CMakeLists.txt
@@ -53,7 +53,7 @@ if(BUILD_INDEX64)
   set(LAPACKELIB "lapacke64")
   set(TMGLIB "tmglib64")
   set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -DWeirdNEC -DLAPACK_ILP64 -DHAVE_LAPACK_CONFIG_H")
-  set(CMAKE_Fortran_FLAGS "${CMAKE_Fortran_FLAGS} -fdefault-integer-8")
+  set(FORTRAN_ILP TRUE)
 else()
   set(BLASLIB "blas")
   set(CBLASLIB "cblas")

I omitted it for now. Is that ok?

Looks to be only tangentially related - `guess the intent is to replace the compiler directive with a flag that gets translated into the appropriate compiler-specific idiom later but I´d need to check where Fortran_ILP came in (possibly another recent PR in their develop branch)

Ok, the logic for Fortran_ILP was introduced in that same PR (but I notice Reference-LAPACK merged it with an unresolved change request...) Need to make sure it does the right thing when built as part of OpenBLAS, as we override some of the  CMakeLists

IIUC (and please correct me if that is wrong), OpenBLAS doesn't use their cmake files to build lapack-netlib.
The changes regarding FORTRAN_ILP in that changeset feel kind of orthogonal to the rest.
It might make change to adopt this part of their changeset (maybe in a different PR):
diff --git a/CMAKE/CheckLAPACKCompilerFlags.cmake b/CMAKE/CheckLAPACKCompilerFlags.cmake
index d727a1592..609bd4115 100644
--- a/CMAKE/CheckLAPACKCompilerFlags.cmake
+++ b/CMAKE/CheckLAPACKCompilerFlags.cmake
@@ -14,6 +14,19 @@ macro( CheckLAPACKCompilerFlags )

 set( FPE_EXIT FALSE )

+# FORTRAN ILP default
+if ( FORTRAN_ILP )
+    if( CMAKE_Fortran_COMPILER_ID STREQUAL "Intel" )
+        if ( WIN32 )
+            set(CMAKE_Fortran_FLAGS "${CMAKE_Fortran_FLAGS} /integer-size:64")
+        else ()
+            set(CMAKE_Fortran_FLAGS "${CMAKE_Fortran_FLAGS} -integer-size 64")
+        endif()
+    else()
+        set(CMAKE_Fortran_FLAGS "${CMAKE_Fortran_FLAGS} -fdefault-integer-8")
+    endif()
+endif()
+
 # GNU Fortran
 if( CMAKE_Fortran_COMPILER_ID STREQUAL "GNU" )
   if( "${CMAKE_Fortran_FLAGS}" MATCHES "-ffpe-trap=[izoupd]")

But apply it to somewhere inside cmake/fc.cmake (and replace FORTRAN_ILP with INTERFACE64).
IIUC, that change would be necessary for support Intel compilers with 64-bit indexing builds. (Feels unrelated to this change here...)

Yes lapack and lapacke CMakeLists are replaced by respective scripts in ~/cmake but need to confirm their toplevel CMakeLists is also ignored. (And if that is the case - fairly sure but no time to check right now - it would make sense to import their complete PR instead of cherrypicking within it, to limit divergence from upstream to what is necessary. And then check if OpenBLAS needs to copy the equivalent of this Fortran_ILP thing or if it is already handled correctly elsewhere (I think it is)

I couldn't find that line in lapack-netlib/CMakeLists.txt:
set(CMAKE_Fortran_FLAGS "${CMAKE_Fortran_FLAGS} -fdefault-integer-8")
So, I don't know what to do other that omitting that part of their commit. I took the remainder (including the part that I showed in the previous comment).
Maybe OpenBLAS' copy of lapack-netlib is missing some previous changes?

And then check if OpenBLAS needs to copy the equivalent of this Fortran_ILP thing or if it is already handled correctly elsewhere (I think it is)

I think that change would need to go here:
https://github.com/xianyi/OpenBLAS/blob/develop/cmake/fc.cmake#L70

I also cherry-picked the patch those changes were based on and also ported the part about 64-bit integers with Intel Fortran compilers to OpenBLAS' build rules.
I hope this is better.
I built a version with 32-bit and another one with 64-bit indexing on mingw with gcc and gfortran (both version 11.2.0). Both versions pass ctest for me.


[Bug]: "Draw" cursor shouldn't revert back to normal after the first line

Attach (recommended) or Link to PDF file
any pdf
Web browser and its version
Firefox 131.0.3
Operating system and its version
macOS 14.6.1
PDF.js version
The one built-in in Firefox
Is the bug present in the latest PDF.js version?
Yes
Is a browser extension
No
Steps to reproduce the problem

Open any PDF, such as just the default one at https://mozilla.github.io/pdf.js/web/viewer.html.
Click on the "Draw" (Ink) annotation icon, to enable the tool
Move the cursor on the PDF: it looks like a pencil
Draw a line

Notice that after these step the drawing tool is still active (you can draw another line without having to re-select the tool).
What is the expected behavior?
After step 4, the cursor should still look like a pencil to indicate that the tool is still active and that you can still draw.
What went wrong?
After step 4, the cursor reverts back to the default "arrow".
Link to a viewer
No response
Additional context
https://github.com/user-attachments/assets/5549192c-e426-4688-9a61-b371d35db938

Interestingly, I cannot reproduce this on Windows 11.

After experimenting more with this, I noticed that if after drawing the first line you move the cursor out of the PDF page and then back in, it becomes again a pencil.
Also, I can only reproduce this problem in Firefox. In Chrome it works as expected, and in Safari it kinda works as expected bu the cursor quickly blinks between pencil and arrow while moving it.

And this is even weirder: if the window is narrow enough that I can move the cursor from the toolbar to the page without hovering outside of the page, then:

if I move the cursor left enough, it turns back from a pencil to the default
there is a point where it shows the "resize" cursor
moving back to the right doesn't turn the cursor back into a pencil
if I try to click where there is the "resize" cursor, it doesn't actually resize anything but it just draws (which is expected, since I have the ink tool enabled)

https://github.com/user-attachments/assets/7d110182-ee0e-4ec4-bd87-a0501921c980


Confusing /map parameter description

During an extraction, files are not read from inside the folder specified by the /folder parameter.

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

ID: c2076638-9361-d4a1-66a4-c08bc59e138a
Version Independent ID: 2718b0bc-77b8-d992-b26c-a0a95014d491
Content: Use the SolutionPackager tool to compress and extract a solution file (Developer Guide for Dynamics 365 Customer Engagement)
Content Source: ce/customerengagement/on-premises/developer/compress-extract-solution-file-solutionpackager.md
Service: crm-online
GitHub Login: @JimDaly
Microsoft Alias: jdaly

@KumarVivek
Please remove/redirect people to the CDS topic: https://docs.microsoft.com/en-us/powerapps/developer/common-data-service/compress-extract-solution-file-solutionpackager
At least remove me as the author for this one. I believe this would be owned by @shmcarth


Clean up and simplify Javadoc

The current RDF4J Javadoc is massive and quite hard to find your way in. We should try and do some simplifications to make it easier to browse. Things to think of:

[ ] remove AST classes from class overview
[ ] remove "internal" packages from general overview
[ ] reorganize to have user-facing interfaces/package most prominently featured on main page

Related issue: #619

Marking this as done for now - I'm sure there will be more improvements later though.


Performance

I would be interested in seeing some performance benchmarks on the client. Are their any?

Not yet. I was going to contact you when I got things ready to go to 1.0.0, to see if you had any suggestions on setup, etc.
Right now, I am trying to find the source of a suspected memory leak issue.

For me the best test would be on the same machine to run something like WRFL and RDFL from the TestClient. Then compare to the .NET client

Ok, I will follow up with some questions after this next release.

No worries. The related code is pretty basic btw:
https://github.com/EventStore/EventStore/blob/release-v3.2.0/src/EventStore.TestClient/Commands/ReadFloodProcessor.cs
https://github.com/EventStore/EventStore/blob/release-v3.2.0/src/EventStore.TestClient/Commands/WriteFloodProcessor.cs
Cheers,
Greg
On Wed, Aug 26, 2015 at 7:58 PM, Brian Mavity<EMAIL_ADDRESS>wrote:

Ok, I will follow up with some questions after this next release.
—
Reply to this email directly or view it on GitHub
https://github.com/bmavity/ges-client/issues/42#issuecomment-135107618.

--
Studying for the Turing test


Redo JSON conversion classes

The classes to parse into and from JSON should be redone. Adding new tests for it.

Working on a new set of dictionary parsers which will handle this.


tensorgram package added

I added tensorgram package in python
You can find the official page for the package
https://pypi.org/project/tensorgram/
Please kindly review & merge my PR

@weierophinney Kindly review my PR

@weierophinney Kindly review my PR

I added pyforest package in python
You can find the official page for the package
https://pypi.org/project/pyforest/0.1.1/
Please kindly review & merge my PR

I added pyforest package in python
You can find the official page for the package
https://pypi.org/project/pyforest/0.1.1/
Please kindly review & merge my PR

@ksdkamesh99 I'm not affiliated with this repo, so my review will not provide any value.

@ksdkamesh99 I'm not affiliated with this repo, so my review will not provide any value.


No Longer Connects

Windows 1.0.0 32 and 64
Stuck at Connecting in the top right corner. Worked before yesterdays release.
Troubleshooting steps. Remove App -> remove AppData-> Roaming, reboot try again etc.

@davibe do you see this too?

Confirmed. I can reproduce on win 7 32b.

we hope new version is working.


VM: Add raw.idmap support for running disk device proxy processes in user namespace

Uses forkusernsexec from @brauner in https://github.com/lxc/lxd/pull/9536 to run the VM disk proxy processes in their own user namespaces. This is to effectively disable the shift=true default behaviour for VM disk devices when using a restricted source path, in order to prevent the creation of root-owned setuid files which could then be used to gain root privileges on the host outside of the VM.
Also adds restricted.idmap.uid and restricted.idmap.gid project settings which will include a range of host uid/gid which the project can map disk devices to using raw.idmap on the instance/profile. This allows instances with restricted disk devices to "punch" holes through the instance's unprivileged idmap range, for certain allowed UID/GID ranges.

Testsuite passed

@stgraber this is working now, but need to do some thorough testing on it and add automated tests.

@stgraber tests are passing now. Thanks


Repository.Get should set mediaTypeRepositoryVisibilityPreview

Per https://developer.github.com/v3/repos/#get-a-repository, the mediaTypeRepositoryVisibilityPreview accept header needs to be set to see the Visibility flag for a repository.

Great catch!  Thank you, @propertone.  Would you like to submit a PR?
The value needs to be added here:
https://github.com/google/go-github/blob/master/github/repos.go#L417

@gmlewis could I take this issue?

Thanks, @wesleimp!  This one is yours. :smile:


[BUG] AttributeError: 'tweet' object has no attribute 'user_rt'

When I try to use Pandas I get an error with an unexpected attribute
Twint Version:
twint==<IP_ADDRESS>
Command Ran
Python 3.6.4 |Anaconda, Inc.| (default, Jan 16 2018, 12:04:33)
[GCC 4.2.1 Compatible Clang 4.0.1 (tags/RELEASE_401/final)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import twint
>>> c = twint.Config()
>>> query= 'bar'
>>> c.Search = query
>>> c.Pandas = True
>>> twint.run.Search(c)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File ".../lib/python3.6/site-packages/twint/run.py", line 201, in Search
    run(config)
  File ".../lib/python3.6/site-packages/twint/run.py", line 153, in run
    get_event_loop().run_until_complete(Twint(config).main())
  File ".../lib/python3.6/asyncio/base_events.py", line 467, in run_until_complete
    return future.result()
  File ".../lib/python3.6/site-packages/twint/run.py", line 140, in main
    await self.tweets()
  File ".../lib/python3.6/site-packages/twint/run.py", line 103, in tweets
    await output.Tweets(tweet, "", self.config, self.conn)
  File ".../lib/python3.6/site-packages/twint/output.py", line 161, in Tweets
    await checkData(tweets, location, config, conn)
  File ".../lib/python3.6/site-packages/twint/output.py", line 142, in checkData
    panda.update(tweet, config)
  File ".../lib/python3.6/site-packages/twint/storage/panda.py", line 61, in update
    "user_rt": object.user_rt,
AttributeError: 'tweet' object has no attribute 'user_rt'
>>>

Description of Issue
AttributeError: 'tweet' object has no attribute 'user_rt'
Environment Details
Mac OSX anaconda

Has this fix been included in the latest release? With version <IP_ADDRESS> still happening.

The error is fixed, try updating with pip3 install --upgrade -e git+https://github.com/twintproject/twint.git@origin/master#egg=twint


Ensure mojang's cursed/edge-case JSON codec inlining in NBT is supported

### Tasks
- [ ] Painting entity data
- [ ] Trims (yes, in 1.20.5 too)
- [ ] Goat Horns (yes, in 1.20.5 too)
- [ ] Find more :concern:

Honorable mention: jukebox song sound events. It appears that mojang hardcodes range for these, because custom sound events require an object to wrap the sound event id.
https://github.com/SpyglassMC/vanilla-mcdoc/blob/9ac13a62e49c3f79e1784815708ef0fe1e19d457/java/data/variants/jukebox_song.mcdoc#L8

Hmm, it might be helpful to have a not in the type syntax. So you could do float not 0 instead of using a really small number for the range minimum

I mean they're also not negative, so it may be more realistic to have built-in literal variables.
float @ %MIN_POS_FLOAT..

mcdoc supports exclusive ranges: float @ 0<..
https://spyglassmc.com/user/mcdoc/#number-range


Component/generate embeddings

Besides:

ignore vscode settings
mypy retry

Thanks @tillwenke. I've left a few comments.
Would you up for adding a simple test case to your component?
Based on your component code I can not see why your pipeline is failing. I think we have to look on your whole pipeline to see what is going wrong.

done


UNRECOVERABLE AUTHORIZATION ERROR: INVALID_REQUEST       # #       Entering limited interaction mode.

##################################################################
UNRECOVERABLE AUTHORIZATION ERROR: INVALID_REQUEST
Entering limited interaction mode.
##################################################################
+----------------------------------------------------------------------------+
|                          In Limited Mode:                                  |
+----------------------------------------------------------------------------+
| Status : Unrecoverable authorization failure.                              |
+----------------------------------------------------------------------------+
| Info:                                                                      |
|       Press 'i' followed by Enter at any time to see the help screen.      |
| Stop an interaction:                                                       |
|       Press 's' and Enter to stop an ongoing interaction.                  |
| Privacy mode (microphone off):                                             |
|       Press 'm' and Enter to turn on and off the microphone.               |
| Speaker Control:                                                           |
|       Press 'p' followed by Enter at any time to adjust speaker settings.  |
| Reset device:                                                              |
|       Press 'k' followed by Enter at any time to reset your device. This   |
|       will erase any data stored in the device and you will have to        |
|       re-register your device.                                             |
|       This option will also exit the application.                          |
| Quit:                                                                      |
|       Press 'q' followed by Enter at any time to quit the application.     |
+----------------------------------------------------------------------------+
2018-10-19 13:15:04.624 [  2] 5 CBLAuthDelegate:handleStopping
2018-10-19 13:15:04.624 [  f] E CapabilitiesDelegate:getAuthTokenFailed:reason=Unrecoverable error by auth delegate
2018-10-19 13:15:04.624 [  f] E CapabilitiesDelegate:publishCapabilitiesFailed:reason=getAuthTokenFailed
2018-10-19 13:15:04.624 [  f] 5 CapabilitiesDelegate:setCapabilitiesState:newCapabilitiesState=FATAL_ERROR
2018-10-19 13:15:04.624 [  f] 9 CapabilitiesDelegate:callingOnCapabilitiesStateChange:state=FATAL_ERROR,error=FORBIDDEN
2018-10-19 13:15:04.624 [  f] E CapabilitiesDelegate:capabilitiesPublishFailed:reason=unableToPublishCapabilities
###############################################################
UNRECOVERABLE CAPABILITIES API ERROR: FORBIDDEN
Entering limited interaction mode.
###############################################################
+----------------------------------------------------------------------------+
|                          In Limited Mode:                                  |
+----------------------------------------------------------------------------+
| Status : Unrecoverable Capabilities API call failure.                      |
+----------------------------------------------------------------------------+
| Info:                                                                      |
|       Press 'i' followed by Enter at any time to see the help screen.      |
| Stop an interaction:                                                       |
|       Press 's' and Enter to stop an ongoing interaction.                  |
| Privacy mode (microphone off):                                             |
|       Press 'm' and Enter to turn on and off the microphone.               |
| Speaker Control:                                                           |
|       Press 'p' followed by Enter at any time to adjust speaker settings.  |
| Reset device:                                                              |
|       Press 'k' followed by Enter at any time to reset your device. This   |
|       will erase any data stored in the device and you will have to        |
|       re-register your device.                                             |
|       This option will also exit the application.                          |
| Quit:                                                                      |
|       Press 'q' followed by Enter at any time to quit the application.     |
+----------------------------------------------------------------------------+
Briefly summarize your issue:

Not responding. Not sure how to authenticate CBL
What is the expected behavior?

ALEXA IDLE
What behavior are you observing?

Limited Mode
Provide the steps to reproduce the issue, if applicable:

sudo bash startsample.sh
Tell us about your environment:

What version of the AVS Device SDK are you using?
  <x.y.z>

AVS SDK 1.9.0
Tell us what hardware you're using:

[ ] Desktop / Laptop
[X ] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[ ] Linux
[ ] MacOS
[X] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

I'm having the exact same error/response ... its showing that I've successfully registered "Success!
Your registration is now complete." however it's not seeming to work.  Now it's just scrolling "2018-10-19 14:11:12.470 [  3] 5 InternetConnectionMonitor:testConnection
2018-10-19 14:11:12.725 [  3] 0 InProcessAttachmentReader:readFailed:reason=SDS is closed
2018-10-19 14:11:12.725 [  3] 5 InternetConnectionMonitor:updateConnectionStatus:connected=true
2018-10-19 14:11:12.725 [  3] 9 LibCurlHttpContentFetcher:~LibCurlHttpContentFetcher
2018-10-19 14:11:39.775 [ 10] I AbstractKeywordDetector:readFromStreamFailed:reason=readerTimeOut"

Hi @sansari1959 ,
Make sure the value of clientId in your AlexaClientSDKConfig.json file is the clientId found under the Other devices and platforms tab on developer.amazon.com, not the one found under the Web tab
After that remove the CBL database file you are using and rerun the SampleApp. Let me know if it works for you

Not that the above mentioned clientID is NOT the clientID you get on top of the device page on amazon. You HAVE to go to the other devices tab, and generate an ID there.

Thanks to all. Indeed, I recognized my mistake. It works perfectly now.

Thanks to all. Indeed, I recognized my mistake. It works perfectly now.
Hi
I met the same problem, could you please tell me how to solve it.
tks


use hostResolve instead /etc/resolv.conf

i think a constant variable is better than string

@pengbinbin1 needs rebase

@pengbinbin1 need you rebase, and I will merge this pr.

This issue has been resolved in PR #19
/close


Noise segments

Hi, and thanks for sharing this wonderful model.
I must say, it is probably the best model I've tried so far for sample-by-sample generation.
I do have a question, though: I constantly end up having noise-burst or noise-segments on my generated audios; sometimes that covers the whole generation, sometimes it just comes at a certain point.
This does not seem to improve with epochs (after a couple of days of training, I still have the same noise burst), and this seems to happen throughout different datasets (mostly classical music) - oddly enough, this seems to be better in noisy datasets, such as rain or water sounds :)
Does anyone have an idea of how to avoid or limit this issue?
Is there a parameter I should fine tune for this?
Thanks again,
Daniele

I also have this problem with the piano dataset. My guess is that noisy datasets help protect the network from overly confident output distributions, essentially approximating https://arxiv.org/abs/1701.06548
It might be worth trying dithering, randomly flipping the least significant bit.
I found that multitrack music and other instruments (violin) suffer from this problem far less. I also found the problem went away with absolute-max normalization (not the current scheme which essentially adds a DC offset).
It only takes a small number of poor samples to throw the model off, so I've also suggested scheduled sampling https://arxiv.org/abs/1506.03099 but it might be impractical.
If I find anything concrete I'll let you know.

Hi, and first of all thanks for answering.
Is it hard to perform the absolute-max normalization you mention? If you could drop a couple of words on how I might do that within in the current code, that'd be great...
Thanks again,
Daniele

It isn't hard at all, instead of (pseudo-code) (x - x:min()) / (x:max() - x:min()) you want to do (x / max(abs(x:min()), x:max()) + 1) / 2.
https://github.com/soroushmehr/sampleRNN_ICLR2017/blob/master/datasets/dataset.py#L53
The current method almost acts like a form of "local contrast normalization" over the dataset but also ends up shifting the mean of the signal, which actually might help. Whenever I tried abs-max normalization the samples tend towards silence or very quiet tones for the majority of the generated audio. I didn't assess this extensively though.

Oh yep, I understand. I'll give it a try, then!
Thanks again,
Daniele

It might also be interesting to see if the LSTM works better here, again something I haven't tried.

I have tried abs-max normalization, which does reduce noise but samples always go to almost-silence situations very quickly, and the quality of the generation is hence quite poor, exactly as you said.
I will try LSTM as well, with abs-max normalization.

Yes, I experience exactly this. With abs-max my thoughts are that issue may
be due to the difference in dynamic range. Trying some of the non-linear
quantization schemes might help, Mu-law/A-law etc.
On 13 April 2017 at 11:20, danieleghisi<EMAIL_ADDRESS>wrote:

I have tried abs-max normalization, which does reduce noise but samples
always go to almost-silence situations very quickly, and the quality of the
generation is hence quite poor, exactly as you said.
I will try LSTM as well, with abs-max normalization.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/soroushmehr/sampleRNN_ICLR2017/issues/17#issuecomment-293850912,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA1ALEXd4760OIto8OikFkKf1JaZY1S9ks5rvfbqgaJpZM4M3qGY
.

Do you have anything new about this issue?
I'm trying to train my model using my own piano music data(approx. 3 hours long) with default parameters but the problem OP mentioned seems to persist.
Based on the comment from both of you, what I suspect is that the noise is due to the fact that my music data has very wide dynamic range, from almost silence notes to very loud notes.
I would really appreciate it if you could share your results.

I've had some success in reducing the sampling temperature slightly.
See my torch implementation for details.
https://github.com/richardassar/SampleRNN_torch
Let me know if it helps.
On 23 May 2017 17:34, "pipoket"<EMAIL_ADDRESS>wrote:
Do you have anything new about this issue?
I'm trying to train my model using my own piano music data(approx. 3 hours
long) with default parameters but the problem OP mentioned seems to persist.
Based on the comment from both of you, what I suspect is that the noise is
due to the fact that my music data has very wide dynamic range, from almost
silence notes to very loud notes.
I would really appreciate it if you could share your results.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/soroushmehr/sampleRNN_ICLR2017/issues/17#issuecomment-303329653,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA1ALIJKyCbhJlx3_hfrPmggeNar70qaks5r8pobgaJpZM4M3qGY
.
On 23 May 2017 17:34, "pipoket"<EMAIL_ADDRESS>wrote:
Do you have anything new about this issue?
I'm trying to train my model using my own piano music data(approx. 3 hours
long) with default parameters but the problem OP mentioned seems to persist.
Based on the comment from both of you, what I suspect is that the noise is
due to the fact that my music data has very wide dynamic range, from almost
silence notes to very loud notes.
I would really appreciate it if you could share your results.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/soroushmehr/sampleRNN_ICLR2017/issues/17#issuecomment-303329653,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA1ALIJKyCbhJlx3_hfrPmggeNar70qaks5r8pobgaJpZM4M3qGY
.

@richardassar, Thanks for the heads up.
Although I tried to find the difference looking at your code and I suppose that the logic for reducing the sampling temperature resides in SampleRNN_torch/scripts/generate_dataset.lua code, I don't think I could say that I have fully understood the difference.
I think it would be better for me to train with your Torch implementation. AFAIK, if I manually generate train data again using your lua script (not using the ones generated using Python script), there is a chance of my problem being solved. Is my understanding correct?
Anyway, I'm going to try to train my model with your implementation and see whether there is a difference :)

It is trivial to add sampling temperature to the theano implementation.
Feel free to use my implementation however, the generate_dataset.lua script
is used in the create_dataset.sh scripts and is very easy to use.
You will achieve effectively similar results with my implementation, the
only difference is the lack of learn_h0 but this makes very little to no
difference to training or sampling as confirmed by Soroush Mehri in our
correspondance. I do intend to add this and I have developed the supporting
code but want to test it before integrating, at which point my
implementation will be 100% identical to the theano implementation.
Let me know how it goes. Thanks.
On 24 May 2017 15:16, "pipoket"<EMAIL_ADDRESS>wrote:

@richardassar https://github.com/richardassar, Thanks for the heads up.
Although I tried to find the difference looking at your code and I suppose
that the logic for reducing the sampling temperature resides in
SampleRNN_torch/scripts/generate_dataset.lua code, I don't think I could
say that I have fully understood the difference.
I think it would be better for me to train with your Torch implementation.
AFAIK, if I manually generate train data again using your lua script (not
using the ones generated using Python script), there is a chance of my
problem being solved. Is my understanding correct?
Anyway, I'm going to try to train my model with your implementation and
see whether there is a difference :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/soroushmehr/sampleRNN_ICLR2017/issues/17#issuecomment-303628246,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA1ALK-DnAiHoiLWHFpslILE7A3-ndxAks5r88tYgaJpZM4M3qGY
.

@richardassar Very interesting, thanks for sharing the implementation, I will test it probably next week, as soon as the GPUs I have access to are free.
Would it be possible for you to point out what should be changed to the theano implementation in order to add the sampling temperature?
Thanks again,
Daniele


Build(deps-dev): Bump eslint from 8.34.0 to 8.36.0 (#4721)

Pull request opened by the merge tool on behalf of #4721

Merged in<PHONE_NUMBER>2d8f935982b37de2da687ee06cdc64