text:
Inference speed test difference

Hi~
I test inference speed with your scripts which use model(input_ids) for generating next token. It works fine and GPU memory cost seems nice with a high batch-size. While when I try to use model.generate(input_ids) to test end-to end inference speed, it seems much slower and memory cost increases sharply. I wonder what makes it different between such two methods?

Can you show me the code for how you use model.generate?

Can you show me the code for how you use model.generate?

Of course, here is my codes for model.generate, I test with a randomly initialized data of length 512, batch-size 32
with LLaMa2-13B，it works well with benchmark.py while failed with model.generate because of OOM on single A100
`from awq import AutoAWQForCausalLM
from transformers import AutoTokenizer
import time
#model_name_or_path = "TheBloke/CodeLlama-34B-AWQ"
model_name_or_path = "TheBloke/Llama-2-7b-Chat-AWQ"
#model_name_or_path = "casperhansen/vicuna-7b-v1.5-awq"
Load model
model = AutoAWQForCausalLM.from_quantized(
model_name_or_path,
#"awq_model_w4_g128.pt",
fuse_layers=True,
trust_remote_code=False,
safetensors=True,
max_new_tokens=4096,
)
tokenizer = AutoTokenizer.from_pretrained(model_name_or_path, trust_remote_code=False)
tokens =  torch.randint(0, tokenizer.vocab_size, (32, 512)).cuda()
Generate output
for _ in range(10):
start = time.time()
generation_output = model.generate(
**tokens,
do_sample=True,
temperature=0.2,
top_p=0.95,
top_k=0,
max_new_tokens=512,
)
elapsed = time.time() - start
count = len(generation_output[0])
print("Tokens per second:  ",(count/elapsed))
#print(tokenizer.decode(generation_output[0], skip_special_tokens=True))`

It is hard to say why this happens. It could be that model.generate() adds overhead when the batch size is large. Do you observe the same large difference in speed from benchmark to generate if you use a single batch size?

authors: casper-hansen moonlightian

repo: casper-hansen/AutoAWQ


text:
Drop effects if WASM module execution caused a revert and also add a test for this.

Drop effects if WASM module execution caused a revert and also add a test for this.

bors r+

bors r+

bors r+

source: gharchive/pull-request

repo: casper-network/casper-node

license_type: permissive

license_source: github-api


text:
Quick question on EnumerateIt

Greetings ... great gem but not clicking yet.
I've set this up and it seems to work fine as documented... except I cannot save it.
i have model

class EventType < EnumerateIt::Base
  associate_values(
      :phone   => [1, 'Phone'],
      :meeting  => [2, 'Meeting'],
      :conference    => [3, 'Conference'],
      :task => [4, 'Task'],
      :email => [4, 'Email']
  )
end

and in Events:

class Event < ActiveRecord::Base
  extend EnumerateIt
  attr_accessor :event_type
  has_enumeration_for :event_type, :with => EventType
end

My event_type column in the DB in events is type: integer
When i manually enter data in the DB and try to list the records nothing dispalys for Event_type even though i can debug it and see the value in the attributes.

--- !ruby/object:Event
raw_attributes:
  id: '1'
  name: Start up Meeting
  event_start: '2015-03-02 03:52:00'
  event_end: '2015-03-02 15:52:00'
  manager_id: '1'
  backup_id: '1'
  location: meeting room 3
  created_at: '2015-03-02 03:53:57.160064'
  updated_at: '2015-03-02 03:53:57.160064'
  description:
  eventable_id:
  eventable_type:
  event_type: '3'

I am thinking maybe I need to use another column type. What value should get saved?
:meeting or 2 ?
in the form the drop down works fine and passes the value to the console object and the request parameters

{"utf8"=>"✓", "_method"=>"patch", "authenticity_token"=>"Q5J84VkpYcr1nGSoJDmatOkXA2s2yvvcY322epuCLDJuFmzxPRrhQNNw4HANJPMSmD2rf5TMNHqypoBPmdNR4w==", "event"=>{"name"=>"Start up Meeting", "event_type"=>"4", "event_start(1i)"=>"2015", "event_start(2i)"=>"3", "event_start(3i)"=>"2", "event_start(4i)"=>"03", "event_start(5i)"=>"52", "event_end(1i)"=>"2015", "event_end(2i)"=>"3", "event_end(3i)"=>"2", "event_end(4i)"=>"15", "event_end(5i)"=>"52", "manager_id"=>"1", "backup_id"=>"1", "location"=>"meeting room 3"}, "commit"=>"Update Event", "controller"=>"events", "action"=>"update", "id"=>"1"}

but I get this error when it tries to update:

ActionView::MissingTemplate at /events/1
Missing template events/update, application/update with {:locale=>[:en], :formats=>[:html], :variants=>[], :handlers=>[:erb, :builder, :raw, :ruby, :coffee, :jbuilder]}. Searched in: * "/home/ims/railsdev/watan_0228/app/views" * "/home/ims/.rvm/gems/ruby-2.0.0-p353/gems/devise-3.4.1/app/views"

any help would be appreciated
Thanks
Steve

Also i tried another model on this (different but same code) and when I update is get

Hi,
If you have an enumeration like
class EventType
  associate_values (
    :meeting  => [2, 'Meeting'],
    :conference    => [3, 'Conference'],
    :task => [4, 'Task'],
    :email => [5, 'Email']
  )
end

The value of the "enumerated attribute" is going to be the integer number. You could have an enumeration defined like:
class EventType
  associate_values :conference, :task, :email

and in that case the value would be "conference", "task", etc. If you want to use the enumeration with an ActiveRecord model and persist it to the database, the column has to have the correct data type, but it can be a number or a string, depending on how you defined you enumeration.
Your first error is not related to the gem, it looks like you are missing a view file that maps to your update action, or you should be redirecting and you're not.
Also, when using EnumerateIt in conjunction with ActiveRecord, it will validate if the received value for the enumerated attribute exists in the enumeration. That's why you have the other error.

Thanks for the reply... yeah i understand that first error.
the event_type in the DB was integer.
I created another verison which uses a string:
Same thing... the form is submitted with the correct name=value showing up in the "puts params"
the param is not saved to the DB
I tried manually inputing "news" and ":news" in the field but it would not show up on Show.
enumerater:
The column in the DB is string "lead_source"
class LeadSource < EnumerateIt::Base
  associate_values :conference, :task, :email, :news
end

model applied to:
class Lead < ActiveRecord::Base
  extend EnumerateIt
 attr_accessor :lead_source
  has_enumeration_for :lead_source, :with => LeadSource

  has_many :opportunities, :through => :leads_opportunities
  has_many :leads_opportunities, :dependent => :destroy
  has_many :contacts
  has_many :events
end

i checked the permits they include the field
def lead_params
      params.require(:lead).permit(:name, :lead_source, :manager_id, :backup_id, :description)
    end

console:
{"utf8"=>"✓", "_method"=>"patch", "authenticity_token"=>"rr0ANA/5JU/YxhsAoH+83wYg8S2GA57Gcbe+TWFSz1mDORAka8qlxf4qn9iJYtV5dwpZOSQFUWCgbIh4YwOyiA==", "lead"=>{"name"=>"Funding for new School in Province", "lead_source"=>"news", "manager_id"=>"", "backup_id"=>"", "description"=>"sdfafsfsf"}, "commit"=>"Update Lead", "controller"=>"leads", "action"=>"update", "id"=>"1"}

Some questions:

Is the record being saved but the lead_source attribute is nil or is the record not being saved at all?
If you go to the rails console and try something like the following, does it work?

lead = Lead.first
lead.update_attributes lead_type: LeadType::TASK
lead.reload
lead.lead_type # should output "task"

I tried that ...when I updated i got:
2.0.0-p353 :012 > lead.update_attributes lead_source: LeadSource::TASK
   (0.3ms)  BEGIN
   (0.2ms)  COMMIT
 => true

and this for the event_type (integer).
2.0.0-p353 :027 >   event.update_attributes event_type:1
   (0.4ms)  BEGIN
   (0.3ms)  COMMIT
 => true

when i reload both  they remain nil
If i manually enter ":task" or "1" in the POSTGRES table it will show up on the console when loaded and will show in debug output on browser but not display using <%= @event.event_type %>
ill try and load the gem on a fresh rails install

Basically .... if I follow the directions
add the gem to the Gemfile
bundle install
rails g enumerate_it:install
create event_type model
add extension to event model.
Is it required to add that code to the application.rb file as well????
thanks

This is what happens in a basic rails save attempt (saving an integer for event _type)

I created an example Rails application showing how you can integrate EnumerateIt into your project.
https://github.com/cassiomarques/enumerate_it_rails_example

source: gharchive/issue

authors: cassiomarques mrdinghy

license: mit

license_source: bigquery


text:
enable multiple port uploads in project

retrieving the wsdl of a port additionally requires the port name in uri
the request parameter wsdl is ignored (a simple get will return the wsdl)

:zap:  this breaks backward compatibilty for WSDL retrieval. @karldahlgren would you think a simple fallback strategy to just return the first wsdl in a project if nothing could be found by uri is ok? (that is the behavior of the latest release)
fixes #100

Once again, thank you for your contribution. Before accepting the PR, all contributors needs to sign an ICLA,  Individual Contributor License Agreement. For more information, please see the following page: https://github.com/castlemock/castlemock/blob/master/CONTRIBUTING.md#copyright-and-license
Best regards,
Karl Dahlgren

Once again, thank you for your contribution. Before accepting the PR, all contributors needs to sign an ICLA,  Individual Contributor License Agreement. For more information, please see the following page: https://github.com/castlemock/castlemock/blob/master/CONTRIBUTING.md#copyright-and-license
Best regards,
Karl Dahlgren

Regarding this line:
this breaks backward compatibilty for WSDL retrieval. @karldahlgren would you think a simple fallback strategy to just return the first wsdl in a project if nothing could be found by uri is ok? (that is the behavior of the latest release)

I believe that it would be great if we could avoid breaking backward compatibility if possible. A simple fallback strategy sounds like it could work. What do you think?

Regarding this line:
this breaks backward compatibilty for WSDL retrieval. @karldahlgren would you think a simple fallback strategy to just return the first wsdl in a project if nothing could be found by uri is ok? (that is the behavior of the latest release)

I believe that it would be great if we could avoid breaking backward compatibility if possible. A simple fallback strategy sounds like it could work. What do you think?

thanks for your fast feedback @karldahlgren !
we'll fix everything but i'll take some days until we find time to do it.

Is this PR still valid? I'm currently going through some old PRs in this project. It seems that this might not be valid anymore.
Thank you for creating the PR.

repo: castlemock/castlemock


text:
Can we please create a build(more this)/deploy(less this for now) on appveyor?

Hi Guys
Happy to help walk to us over to core, unfortunately don't have visibility of your builds. Do you guys think we could sort this?
AppVeyor is mega simples. I have implemented it here: https://github.com/cryosharp/fluentwindsor/blob/master/appveyor.yml
If you think I have checked in my passwords, you are wrong :)
Gav

I assume you are aware of our public build server (also supports guest login if you don't want to bother creating an account) which has both Windows and Linux build agents: http://builds.castleproject.org/
For Winsdor we had .NET 3.5, 4.0, 4.0 CP, 4.5 and Packaging build configurations, however I've just got rid of everything except 4.5 and Pack after merging your pull requests. We used to have a Mono one, however that got removed years ago when it wasn't maintained, we've got a Mono one for Castle Core. Especially that we are reducing the number of configurations we could merge them into one and enable builds for pull requests including the build status notification into GitHub, TeamCity has this support.
If we want to support Mono we should do it using the .NET 4.5 build and test on Mono, we should get rid of those really old MONO26/MONO28 configurations in the VS projects.

Awesome, will fix net40 build issue tomorrow.

repo: castleproject/Windsor


text:
plant_in_service_ferc1 table headers contain data

In the transform step of the plant_in_service_ferc1 table, there are a handful of cases in which the header rows contain data in the original database. This should never happen.  All instances of this phenomenon are in the years 2004-2005, which is immediately after the table underwent a revision (2003). In addition, the numbers reported in the header rows for 2004 and 2005 are almost identical to each other, so it seems like maybe someone was just re-submitting the same form in both years, ignoring the fact that the form itself had changed.
This means that all of these records are probably bad.  However, there are only a few dozen of them (out of ~30,000 total plant-years). It might be possible to recover this data, but it's probably not super
raw_dfs = {"plant_in_service_ferc1": pd.read_sql("f1_plant_in_srvce", ferc1_engine)}
transformed_dfs = {}
transformed_dfs = pudl.transform.ferc1.new_plant_in_service(raw_dfs, transformed_dfs)
pis_head = (
    transformed_dfs["plant_in_service_ferc1"]
    .set_index(["utility_id_ferc1", "report_year", "amount_type"])
    .filter(regex=".*_head$")
)
dude = pd.DataFrame()
for field in pis_head:
    bad_df = pis_head[pis_head[field].notnull()]
    dude = pd.concat([dude, bad_df])
(
    dude
    .reset_index()
    .drop_duplicates()
    .set_index(["utility_id_ferc1", "report_year", "amount_type"]).index
)

The indices of the resulting problematic records:
MultiIndex([(  7, 2004, 'starting_balance'),
            (  7, 2005, 'starting_balance'),
            ( 94, 2004, 'starting_balance'),
            ( 94, 2005, 'starting_balance'),
            (121, 2004, 'starting_balance'),
            (121, 2005, 'starting_balance'),
            (145, 2004, 'starting_balance'),
            (145, 2005, 'starting_balance'),
            (159, 2004, 'starting_balance'),
            (166, 2004, 'starting_balance'),
            (187, 2004, 'starting_balance'),
            (187, 2005, 'starting_balance'),
            (  7, 2004,   'ending_balance'),
            (  7, 2005,   'ending_balance'),
            ( 94, 2004,   'ending_balance'),
            ( 94, 2005,   'ending_balance'),
            (121, 2004,   'ending_balance'),
            (121, 2005,   'ending_balance'),
            (145, 2004,   'ending_balance'),
            (145, 2005,   'ending_balance'),
            (159, 2004,   'ending_balance'),
            (166, 2004,   'ending_balance'),
            (187, 2004,   'ending_balance'),
            (187, 2005,   'ending_balance'),
            ( 25, 2004, 'starting_balance'),
            ( 25, 2005, 'starting_balance'),
            ( 95, 2004, 'starting_balance'),
            (101, 2004, 'starting_balance'),
            (101, 2005, 'starting_balance'),
            (132, 2004, 'starting_balance'),
            (132, 2005, 'starting_balance'),
            (167, 2004, 'starting_balance'),
            (167, 2005, 'starting_balance'),
            (247, 2004, 'starting_balance'),
            ( 25, 2004,   'ending_balance'),
            ( 25, 2005,   'ending_balance'),
            ( 95, 2004,   'ending_balance'),
            (101, 2004,   'ending_balance'),
            (101, 2005,   'ending_balance'),
            (132, 2004,   'ending_balance'),
            (132, 2005,   'ending_balance'),
            (167, 2004,   'ending_balance'),
            (167, 2005,   'ending_balance'),
            (247, 2004,   'ending_balance'),
            (153, 2004, 'starting_balance'),
            (153, 2005, 'starting_balance'),
            (153, 2004,   'ending_balance'),
            (153, 2005,   'ending_balance')],
           names=['utility_id_ferc1', 'report_year', 'amount_type'])
``

A new instance of this problem shows up in the income_statement_ferc1 table in 2003. I had to add a little cleanup method in transform.ferc1.IncomeStatementFerc1TableTransformer.process_dbf. Docs from that method:

    In 2003, two rows were added to the ``f1_income_stmnt`` dbf table, which bumped
    the starting ``row_number`` of ``f1_incm_stmnt_2`` from 25 to 27. A small
    handful of respondents seem to have not gotten the memo about this this in
    2003 and have information on these row numbers that shouldn't exist at all for
    this table.

These bad rows consist of 33% (!) of the respondents for that year.

authors: cmgosnell zaneselvans

repo: catalyst-cooperative/pudl


text:
Add issue template for nightly build

Overview
Closes #3284 .
There was no action needed to resolve #3284, except for verifying that these things did indeed happen. I'm adding an issue template so we can handle these more smoothly in the future!
Testing
How did you make sure this worked? How can a reviewer verify this?
I'm not actually sure how to test if issue templates are properly formatted, but we can verify this once merged by trying to create an issue!
# To-do list
- [ ] verify post-merge

Updated this to hopefully be a bit clearer on what to do.


text:
Add optogenetic stimulation

Add optogenetic Stimulation data from TDT epochs

[x] Add ogen metadata: two different yaml files for different optogenetic stimulation protocol

Varying durations protocol: 40 Hz stimulation at various durations (i.e. 250ms, 1s and 4s)
Varying frequencies: 3 recording sessions corresponding to a fixed duration of stimulation (i.e., 250ms, 1s, and 4s). Each session lasted 8 minutes. The subject receives optogenetic stimulation at varying frequencies (5 Hz, 10 Hz , 20 Hz and 40 Hz)

[x] Construct ogen series from TDT events knowing the stimulation frequency of each stimulation interval ("onset"/"offset" epochs timestamps from TDT)
[x] Add OptogeneticStimulusSite, OptogeneticSeries but also a OptogeneticStimulusInterval with an additional column to store the stimulus frequency and the time series column to link to the OptogeneticSeries segment.

Example nwb file here.
One file for each experimental protocol

repo: catalystneuro/hnasko-lab-to-nwb

license: MIT


text:
Feature/json web signature2020

In this PR:

Package refactoring.
Unified Signing and Verification interface.
ED21559 Verification and Sigin refactored.
JSON Web Key implementation.
JSON Web Signature implementation.
More Test cases.
Improve documentation.
More Examples.

I've added some small remarks. Hope this is ok :-) All in all great PR! Glad to have you working on this.

authors: DominikPinsel koptan

repo: catenax-ng/product-lab-ssi


text:
Project name is "" in cases where trigger is based on included artifact and artifact is a ci.yaml file.

If a project uses ci.yml produced as an artifact then the relationship of the trigger should be the source project of the artifact but instead it's currently marked as ""
Steps to reproduce:
If your .gitlab-ci.yml contains following syntax:
trigger:
    include:
      - artifact: child-ci.yml
        job: my-job

In that scenario the name of project in relationship is marked "" whereas it should either be should be the source project of the artifact.

Yeah the parsing is missing logic for dynamic child pipelines, as per https://docs.gitlab.com/ee/ci/pipelines/downstream_pipelines.html#dynamic-child-pipelines. Will try to have a look the next few days.

repo: catouc/gitlab-ci-crawler


text:
.ron file support (using existing rust icon)

Ron is rust object notation.

.ron files already have the rust icon 🤔

haha oops! i gues they do. for some reason i thought they didnt :omegalul:

authors: WhyNotDogie prazdevs

repo: catppuccin/vscode-icons


text:
Verificar rodadas duplicadas

Atualmente está verificando apenas rodada única.
Supondo que determinada sequência foi sorteada 2x, irá aparecer somente uma.

Fala @wmarquardt, obrigado pela issue.
Acredito que possa ser discutido, até pra entender a probabilidade disso acontecer. Acha que estaticamente isso é possível?

Realmente @cauealves, é praticamente impossível. Acredito que seja a mesma probabilidade de alguém acertar o prêmio. 1 em 50M.

Concordo contigo. Se quiser realizar um fork e enviar uma proposta p/ outra issue, fique à vontade :)

authors: cauealves wmarquardt

repo: cauealves/mega-sena-cli


text:
[feature] Drogon installation guide to readme

Хотелось бы инструкцию по установке Drogon куда-нибудь в README.md...
Originally posted by @MarkTheHopeful in https://github.com/cavoke-project/cavoke/issues/5#issuecomment-1038285348

#35

authors: petrtsv waleko

repo: cavoke-project/cavoke


text:
My First PR commit

PR comments

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: cjxd-bot-test
If they are not already assigned, you can assign the PR to them by writing /assign @cjxd-bot-test in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

:star: PR built and available in a preview environment cb-kubecd-bdd-sbhg-1587748273-pr-1 here


text:
Better UI

Slightly better UI using Fuse's Basic Theme.
I am currently working on an even better with log in page different from Sign Up.
The Sign Up page will include picture, name and maybe more..

Hey Luis. Sorry I missed this, I need to turn the volume up on my github notifications :)
I will test this today and hopefully merge later. Thanks so much for the contribution

authors: LuisRodriguezLD cbaggers


text:
[Feature request] Allow the user to specify the winhighlight

Hi,
great idea for a plugin, just giving it a whirl but seems v. cool so far.
One option that I think it'd be great to have as a user would be the ability to change the highlights of the floating windows opened. I know you can change the border but as I have tweaked things globally I need to also adjust the appearance of the windows opened by this plugin to match those as well which unfortunately isn't currently a possibility.

Thanks for the kind words and feature request!
Window highlights are on my list of priorities. If you'd like, there is one hl group available right now which is PortalLabel (linked to Cursor by default). I plan to expand this for more customization soon.

@akinsho I started adding more highlight groups to Portal in the branch feat_winhighlights. I've added PortalBorder and PortalTitle. I don't typically customize my colours much, but is this kind of what you were looking for? Are there more hl groups that could be useful?

That's good to hear @cbochs just tried it out I had actually been meaning more directly the actual winhighlight object/string but this is more how other plugins do it so is better this way. I think the groups you have are good although I'd also include PortalNormal to cover the background since the only other way to change that is to change it for all floats.
Also the highlights aren't set using :h highlight-default so they are harder to override i.e. if I set the highlights before your plugin loads they override my settings or if I link to another highlight group it also overrides my settings. using {default = true} just makes it easier for me as a user to override your/a plugin's highlights

Thanks for the feedback! I've added a PortalNormal group with links to NormalFloat and added default = true when creating the groups. They look like this now:
vim.api.nvim_set_hl(0, "PortalLabel", { link = "Search", default = true })
vim.api.nvim_set_hl(0, "PortalTitle", { link = "FloatTitle", default = true })
vim.api.nvim_set_hl(0, "PortalBorder", { link = "FloatBorder", default = true })
vim.api.nvim_set_hl(0, "PortalNormal", { link = "NormalFloat", default = true })

I created a PR. Feel free to take a look and add any more comments there 🙂

Just tried it out and it works great 🙌🏿

authors: akinsho cbochs


text:
Can XDF XML have custom fields?

My question goes towards whether or not we should just keep the whole XML header and footer as a string in the stream dictionary. This would only be helpful in case the file contains custom fields.

Yes, the XML can contain custom fields, so we should make headers and footers available somehow. In addition, the standardized part could of course be transformed into a more accessible data structure.

This issue was closed with pr #4.

authors: abcsds cbrnr


text:
Influxdb forwarder

This dumps data into an InfluxDB 2.x database, rather than using datadogs.

At a glance this all looks good to me, but I'd like to find some time this week to try it out before merging.

authors: agmlego ccbrown

repo: ccbrown/factorystatsd


text:
🛑 MRSpec Portal is down

In f28c092, MRSpec Portal (https://mrspec.ccm.sickkids.ca) was down:

HTTP code: 0
Response time: 0 ms

Resolved: MRSpec Portal is back up in 8b68800 after 27 minutes.

repo: ccmbioinfo/status


text:
🛑 OpenEvo is down

In f790107, OpenEvo (https://openevo.eva.mpg.de/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: OpenEvo is back up in 04914f6 after 16 minutes.

repo: ccp-eva/uptime-monitor


text:
Video issues

[x] Shadowbox swiping gets laggy
[x] There may be an audio bug? Need to re-look at how we mute vids
[x] Opening videos from thumbnail opens web

Found out that not all videos have audio tracks at 00:00, which was causing our issue (we muted it because no audio, but it appeared later)

repo: ccrama/Slide-iOS


text:
Fix Failing Deploy

Deployment fails in the CI.
Hopefully fixing the CI also fixes #105.

To the best of my understanding as of now, this PR does nothing to fix the mentioned issue or the deployment failures.

repo: cda-tum/ddvis


text:
豆瓣广播rss推送无图片

kindle截图：https://s2.ax1x.com/2019/09/08/n3zah8.png
rss地址：https://rsshub.app/douban/topic/51678
排序方式为最新
脚本中有图片地址，推送到kindle上并没有图片
脚本中加入了referrerpolicy="no-referrer"属性防止跨域，是因为kindleear触发了跨域导致图片无法加载吗？

查看了网页源代码，图片格式为webp，应该是Kindle还不支持。
如果要转换为JPG，目前暂时还不支持，要等GAE上的PIL库支持，参考 #511

repo: cdhigh/KindleEar


text:
Bug: Navbar buttons are not aligned properly

Describe the bug
The navbar buttons are not properly centered, only the Home and GitHub buttons are.
To Reproduce
Steps to reproduce the behavior:

Go to website
Check the navbar and you will see that buttons are not properly centered.

Expected behavior
Every navbar button should be centered.
Screenshots

Hello, I can fix this. Please assign me.

Pls assign
So I can start working on it !
Thx

Hey @DireWolf707, since @Eddayavuz commented first I will let Eddayavuz work on this!

Thank you!

Hello,
Please let me know if there's a problem with the PR. This is my first open source contribution and I'm very excited! :)


text:
chore(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2472

Questions ?
Please refer to the Backport tool documentation


text:
feat(serviceaccount, pod): set automountServiceAccountToken to false (#1146)

Backport
This will backport the following commits from k8s-24/main to k8s-22/main:

feat(serviceaccount, pod): set automountServiceAccountToken to false (#1146)

Questions ?
Please refer to the Backport tool documentation

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1166

Questions ?
Please refer to the Backport tool documentation


text:
chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1905

Questions ?
Please refer to the Backport tool documentation


text:
chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-27-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2610

Questions ?
Please refer to the Backport tool documentation


text:
chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-27-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3677

Questions ?
Please refer to the Backport tool documentation


text:
chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-30-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4941

Questions ?
Please refer to the Backport tool documentation


text:
Opening settings in Firefox causes typing to stop working properly elsewhere

It doesn't seem to matter how you open the settings (mouse, keyboard, refresh with it already open, tab over to it) and it doesn't seem to matter if or how you close it. Once the settings input has been focused certain keys stop working in editor windows (like Enter and the down arrow).
This happens with or without the selection clipboard enabled. No other inputs exhibit this behavior.
My current guess is that there is some focus-related event that is handled differently or improperly in Firefox (or Chrome has incorrect behavior that VS Code depends on).

Confirming this is a upstream issue, I'll mirror this over to Microsoft when able.

If this is still a problem, please feel free to reopen.

authors: code-asher nhooyr sr229

repo: cdr/code-server


text:
ubuntu dev go share mod does not share from $GOPATH

https://github.com/codercom/sail/blob/b19d719fb5c1f96600d4e6a092b03208425a220a/ubuntu-dev/ubuntu-dev-go/Dockerfile#L13
It only shares the default $GOPATH directory. I use ~/Programming/gopath and sail is not picking that up.
I'm not really sure what a good solution here is.

Going to close as it has been a while and I'm not sure if this is relevant anymore.

repo: cdr/sail


text:
Gracefully fallback if an error comes back from the Canada Post api

Summary | Résumé
If an error comes back from the Canada Post API (for example, when credits run out), just display "No results" and allow the user to continue to fill in the form manually.

Do we have any way to be alerted if this fails / lapses billing etc?

Yeah, there's a dashboard for the Production account, I'll get some accounts setup for team members who need to keep an eye on it.

repo: cds-snc/covid-alert-portal


text:
postCode feels confusing

In https://github.com/cds-snc/cra-claim-tax-benefits/blob/34435cf4b0655ffd30b3d310f4c569193f3142b6/routes/login/login.controller.js#L13
the function is called postCode, which I assume is meant to reference POSTing the access code but it feels like it's like the postal code or something.
Given that I assume we'll have many similar functions handling the posting of data, should we rename them to something more clear?
cc @pcraig3 @katedee

we could call it postLoginCode?

resolved since #9

repo: cds-snc/cra-claim-tax-benefits


text:
chore: add translations for API screens

Summary | Résumé
PR to add translations for the API screens.

@anikbrazeau starting this PR for when you have some time to start adding the strings
There's a bunch on the form-builder file
i.e.
"responseDeliveryHelpWithApi": {

Added a couple of commits to the form builder file to update the French strings and match some tweaks in English as well!

Created a first bilingual version of the READ.ME that we can tweak later as well.

authors: anikbrazeau timarney

repo: cds-snc/platform-forms-client


text:
🛑 tanecni-divadlo.cz is down

In ab61aec, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 314036f.

repo: cebreus/upptime


text:
🛑 tanecni-divadlo.cz is down

In 8fc31bd, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in eda3fdc.


text:
🛑 tanecni-divadlo.cz is down

In 31203b7, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 502
Response time: 9272 ms

Resolved: tanecni-divadlo.cz is back up in 81f994f.


text:
🛑 tanecni-divadlo.cz is down

In 1cad169, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 08e7594.


text:
🛑 tanecni-divadlo.cz is down

In bcd0acc, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 000ed46.


text:
🛑 tanecni-divadlo.cz is down

In e5b3b7e, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 5c0ff34.


text:
🛑 tanecni-divadlo.cz is down

In c6dd69b, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 336cf8d after 477 days, 15 hours, 17 minutes.


text:
🛑 tanecni-divadlo.cz is down

In 1bcf5b9, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in bc89643 after 3 minutes.


text:
🛑 tanecni-divadlo.cz is down

In d08382e, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 0fb04df.


text:
🛑 tanecni-divadlo.cz is down

In ec102c3, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 3540109.


text:
🛑 tanecni-divadlo.cz is down

In d6d062b, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 5c9aa3a.


text:
🛑 tanecni-divadlo.cz is down

In 4079448, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 6e80d77.


text:
🛑 CDN is down

In b2e1291, CDN (https://cdn.ceccun.com/cdynamic/captive) was down:

HTTP code: 522
Response time: 15365 ms

Resolved: CDN is back up in 0373f8b.

repo: ceccun/status


text:
🛑 CDN is down

In e5b32ac, CDN (https://cdn.ceccun.com/cdynamic/captive) was down:

HTTP code: 522
Response time: 15320 ms

Resolved: CDN is back up in e4adbff.


text:
🛑 CDN is down

In 10ae582, CDN (https://cdn.ceccun.com/cdynamic/captive) was down:

HTTP code: 522
Response time: 15298 ms

Resolved: CDN is back up in f414b8f.


text:
🛑 CDN is down

In 4627c56, CDN (https://cdn.ceccun.com/cdynamic/captive) was down:

HTTP code: 522
Response time: 15667 ms

Resolved: CDN is back up in eec5b77.


text:
Fixed a long standing bug in the camera projection matrix, it should …

…be updated each time the  near/far changes in the renderer. Now intersectors works well in far distances.

@hishamabdelrahman it should fix the error you experienced when trying to pick from to far away.
I was experiencing too this error.

Probably we should apply this fix in CullVisitor too. Once the setNearFar is done: https://github.com/cedricpinson/osgjs/blob/develop/sources/osg/CullVisitor.js#L348
@Kuranes thoughts?

the popCameraModelViewProjectionMatrix just above is supposed to do that already by calling the PopProjectionMatrix.
It doesn't in your case  ?

For my case (IntersectorVisitor) it inherits from NodeVisitor, so no clamp is done, it needs to get the matrix from camera, so the clamp should be done in the renderer. (If I recall well OSG does it also).

@Kuranes Ok I see now that it indeed applies the clamp, but the _computedFar value is wrong.

@Kuranes First camera never gets traversed by the CullVisitor so it never reaches that:
https://github.com/cedricpinson/osgjs/blob/08e095f7032d6c30865e6dfb6594091bed26d251/sources/osg/CullVisitor.js#L336
And the clamp is not done.
So it makes sense to do it in Renderer

Ha, yes, that first camera oddity not going through the same code as the rest. Bug prone.
Could be added on that code part, in View code
Seems osg does that too
https://github.com/openscenegraph/osg/blob/77d4847753cc28eb9c0006c4788ac6f0f7c0f273/include/osgUtil/CullVisitor#L254
Cannot find OSG renderer clamping projection matrix ?

Yep, not in Renderer. It is done in SceneView

@Jordi. ...thanks Jordi I was thinking that this bug can't be fixed so I was using polytope inter sector at far distance .but your fix is much neater way
thanks

@jtorresfabra could you add a link to the code in osg ? I dont remember this part.
thx a lot for the PR

The clamp is done in cullVisitor in popProjection and later in sceneview for OSG. SceneView is deprecated so I did not reproduce this class, instead I put the 'similar' code in Renderer.js . It seems the right way to do it @jtorresfabra
Thank you very much it's a kind of tricky :+1:

I don't know how I got here, but I want to be out of this emails.
Thanks!
On Wed, Jun 17, 2015 at 9:55 PM, Cedric Pinson notifications@github.com
wrote:

Merged #337 https://github.com/cedricpinson/osgjs/pull/337.
—
Reply to this email directly or view it on GitHub
https://github.com/cedricpinson/osgjs/pull/337#event-333193665.

Hi @Jordi,
another user ( @hishamabdelrahman ) used your username instead of mine (@jtorresfabra). You should not get more messages again as this thread is closed.
Sorry for the inconvenience.

authors: Jordi Kuranes cedricpinson hishamabdelrahman jtorresfabra

repo: cedricpinson/osgjs


text:
Build - Add Support for building with VS2015/2019

Currently only builds with VS2017

Should be resolved in https://github.com/cefsharp/CefSharp.BrowserSubProcess.NetCore/commit/81f9e01f64df4942ea4209e9ec830e1f8337313a


text:
e2e: peer discovery

Ref: #599
With current implementation decision it will be beneficial to test how light nodes will behave in cases where:

Light node only has half of max peers connected and other half is having high latency
Light node is connecting to a new set of peers after a period of time has passed with the current peers' set

this will require a separate branch of the node repo to contain tests' faster reconnection time

Closed as done in part of reconstruciton tests.
Our full nodes can find each other in the network

repo: celestiaorg/celestia-node


text:
Pass app version to CreateCommitment and NewMsgPayForBlobs

When creating PFBs, and specifically PFB commitments, we will need to pass the app version to provide enough context for the application to generate the correct commitment for that version.

Blocked on authored blobs, as that is the next change that could require this, at least for creating blobTxs, but probably not when we actually call create commitment. We might want to pass the app version anyway to avoid API breaks in the future when we inevitably do need the app version.

@cmwaters should we consider this part of authored blobs? if so, can we add this to the tracking for authored blobs?

Is this still necessary? CreateCommitment moved to the go-square repo and now accepts a param for subtreeRootThreshold which can vary based on app version.

This is correct. CreateCommitment can now handle changes to the subtreeRootThreshold that was hardcoded before. There is nothing else that is currently hardcoded. I would also refrain from adding app version to the go-square repo. I would prefer if it were agnostic to it and celestia-app would define the parameters but perhaps I could be proven wrong.

authors: cmwaters evan-forbes

repo: celestiaorg/go-square


text:
User not redirecting to sending page after scanning the contacts QR code from send secure flow

Frequency: 100%
Repro App version:  Android Valora Mainnet build v1.1.0 (1004294309), Test Flight build Valora app build  v1.1.0 (22)
Does not Repro App version: App store  Valora app build  v1.0.1, Android Play store Valora  build v1.0.1
Repro on device: Samsung Galaxy A70(9.0), iPhone 11 (14 beta 6)
Repro Steps:

Launch the Application and sign in with a test account
Tap on send button
Select contact which have 2 verified account with same number
Enter valid amount
Tap on review button
Make sure user should redirect to Send secure screen
Tap on Scan QR code
Scan contact user Qr code
Observe

Current Behavior: User not redirecting to sending page after scanning the contacts QR code from send secure flow
Expected Behavior:  User should Redirect to sending page after scanning the contacts Qr code
Investigation:
Account number confirmed message is shown after scanning the Qr code
User redirect to sending page after tapping on “X” button on scan code page
Impact: User get blocked to scan the Qr code .
Attachment: ScanQR.MP4(Bug observed at 26 sec)

i have verified this issue Android internal build v1.1.0(1004294311) & iOS test flight build v1.1.0(23) and found that issue is Fixed So, closing this ticket

repo: celo-org/celo-monorepo


text:
adds key management for multiple versioned keys

Description
Adds support for multiple versioned keys to the ODIS signer's key management code and implements the following key rotation protocol.

Signers add the new key to their key vault
The signers should update the "latest" version of the key in their env file so that this configuration goes into effect and the key is prefetched the next time they redeploy their signer.
The new keys must be named according to the patterns ${env.PHONE_NUMBER_PRIVACY_KEY_NAME_BASE}-{env.PHONE_NUMBER_PRIVACY_LATEST_KEY_VERSION} or ${env.DOMAINS_KEY_NAME_BASE}-{env.DOMAINS_LATEST_KEY_VERSION}. Note that these names are configurable via the .env file, but must be suffixed by a '-' followed by the latest key version number (key versions start at 1). If base key names are not configured, they default to 'phoneNumberPrivacy' and 'domains' respectively (ex. 'phoneNumberPrivacy-1' would be the default name for the original ODIS Key).
The combiner's config file is updated with the new latest version of the key only once all signers have confirmed that they have correctly stored the new key in their keystore.
The combiner is redeployed, and will now send requests to signers with an incremented 'keyVersion' header. Upon receiving requests for this new keyVersion, signers will fetch it from their keyvaults and begin using it to sign requests.
The combiner monitors signer responses and ensures they are signing with the correct keyVersion
Once the signers have confirmation that the system is operating properly and the key rotation went smoothly, they can delete their old key shares.

Other changes
None
Tested
Key rotation will be tested manually in staging and alfajores as part of CIP-40 release process and e2e testing
Related issues

Fixes #8660

Backwards compatibility
Yes
Documentation
Added to README

Nice work overall.
One disadvantage of this approach is that we are now requiring operators to name their keys in a specific way wheras before they could use any name they like. This is probably fine, but it would be nice to allow them to override the default key paths (derived from the key Key obejct) with a custom path in the config file.
The code itself also does not seem to support deploying a new key version (i.e. executing resharing) without code changes and redeployment of the signer. Ideally it would only require config changes, or even only placing the new key in the key store. This will be important if we want to start running key resharing proactively. We should also keep in mind that the goal is to delete the old key shares, which we would do after everyone has upgraded to the new shares.

Nice work overall.
One disadvantage of this approach is that we are now requiring operators to name their keys in a specific way wheras before they could use any name they like. This is probably fine, but it would be nice to allow them to override the default key paths (derived from the key Key obejct) with a custom path in the config file.
The code itself also does not seem to support deploying a new key version (i.e. executing resharing) without code changes and redeployment of the signer. Ideally it would only require config changes, or even only placing the new key in the key store. This will be important if we want to start running key resharing proactively. We should also keep in mind that the goal is to delete the old key shares, which we would do after everyone has upgraded to the new shares.

Agreed. Let me know if this updated implementation addresses your concerns

Important: As of right now our helm charts do not support this new secret naming convention. We'll need to fix this before deploying these code changes.

closing in favor of https://github.com/celo-org/celo-monorepo/pull/9161

authors: alecps nategraf


text:
Set middleware version in preparation for release

Sets the new middleware version, in preparation for release.

@palango its the version of this rosetta implementation. Basically allows clients to discover what version of this service they are talking to. Note its not the version of the rosetta api its the version of the implentation.

repo: celo-org/rosetta


text:
update: reset flags before and after each osd node upgrade

It might be possible at some point even with osd flags noout and
norebalance set the PGs states can change depending on the amount of data
written meantime. It means the check for PGs state will fail.
This commit changes the way we set those flags:
we set them before an OSD node upgrade and unset them before the PGs
state check so they can recover.
Fixes: #3961
Signed-off-by: Guillaume Abrioux gabrioux@redhat.com

jenkins test pipeline

jenkins test dev-centos-non_container-update

jenkins test centos-container-all_daemons

jenkins test v

jenkins test centos-non_container-collocation

jenkins test centos-container-all_daemons

repo: ceph/ceph-ansible


text:
tests: force using the more recent build

We should use  latest-master-devel for switch_to_containers job.
Otherwise it might happen we actually downgrade the ceph version when
the image used is older than the rpm initially used for installing ceph.
Signed-off-by: Guillaume Abrioux gabrioux@redhat.com

jenkins test centos-non_container-switch_to_containers


text:
ceph_cephfs

When using provider ceph_cephfs to mount a cephfs volume, there is no way to specify mount options.
Wouldn't it be wise to enable some sort of options, for instance adding "noatime"?

updated subject

repo: ceph/ceph-cookbook


text:
deploy: adjust terminationGracePeriodSeconds to help rbd-nbd

Describe what this PR does
As part of pod termination, Kubernetes initially sends SIGTERM to rbd-nbd and waits for ~30 sec i.e default terminationGracePeriodSeconds and then sends SIGKILL as part of ungraceful termination sequence. This is where some of the IO goes unhandled and mount point may turn out to read-only.
rbd-nbd expects some time for it to handle pending IO after SIGTERM is issued, currently at the rados level rados_osd_op_timeout is unlimited (wait forever), this means we never know when the IO will finish, if rbd-nbd can set the IO ops some timeout via rados_osd_op_timeout (in the rbd-nbd) then its possible for us at ceph-csi to exactly define time to wait for rbd-nbd at rbd nodeplugin.
But till we have the fix in rbd-nbd, to avoid the frequency of hitting such issue at the csi level, we should consider increasing terminationGracePeriodSeconds to a good value from the current 30 sec.
terminationGracePeriodSeconds field of a Pod spec instructs Kubernetes to wait for a given number of seconds before killing a container after termination. This leaves the processes time to free resources and flush any important data.
updates: #2204
Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>

@Madhu-1 @nixpanic IMHO, we need to have this at 3.4.0. Thanks!

/test ci/centos/mini-e2e/k8s-1.20

rbd-nbd expects some time for it to handle pending IO after SIGTERM is issued, currently at the rados level rados_osd_op_timeout is unlimited (wait forever), this means we never know when the IO will finish, if rbd-nbd can set the IO ops some timeout via rados_osd_op_timeout (in the rbd-nbd) then its possible for us at ceph-csi to exactly define time to wait for rbd-nbd at rbd nodeplugin.

If this is the issue in the test, would calling sync after writing the file be a solution?

rbd-nbd expects some time for it to handle pending IO after SIGTERM is issued, currently at the rados level rados_osd_op_timeout is unlimited (wait forever), this means we never know when the IO will finish, if rbd-nbd can set the IO ops some timeout via rados_osd_op_timeout (in the rbd-nbd) then its possible for us at ceph-csi to exactly define time to wait for rbd-nbd at rbd nodeplugin.

If this is the issue in the test, would calling sync after writing the file be a solution?

Yes, we need to ensure that we are flushing everything pending (as part of handling SIGTERM) something like what we have done before with tcmu-runner here: https://github.com/open-iscsi/tcmu-runner/blob/master/rbd.c#L1478
I have opened a tracker to double-check the code that we are not assuming anything here: https://tracker.ceph.com/issues/51820
@idryomov @trociny. might have more insight.

I really only meant sync in our e2e tests, as I expect rbd-nbd to flush its devices before exiting. #2204 is an instability in the tests that might be fixed that way.
For a real world workload, applications might continuously write, so at one point there needs to be a stop in accepting I/O, and caching it for a while before the next rbd-nbd has taken over. Device timeouts before the filesystem receives an error and goes read-only can usually be specified under /sys/block/<bdev>/device/... I think. Or maybe rbd-nbd has a parameter for it? Tuning that might be needed too?

I expect rbd-nbd to flush its devices before exiting

When rbd-nbd receives SIGTERM, it stops accepting I/O.  I think the current assumption that Prasanna is working off of is that k8s sends SIGKILL before rbd-nbd is able to process all outstanding in-flight I/O.

As part of pod termination, Kubernetes initially sends SIGTERM to rbd-nbd and waits for ~30 sec i.e default terminationGracePeriodSeconds and then sends SIGKILL as part of ungraceful termination sequence. This is where some of the IO goes unhandled and mount point may turn out to read-only.

rbd-nbd expects some time for it to handle pending IO after SIGTERM is issued, currently at the rados level rados_osd_op_timeout is unlimited (wait forever), this means we never know when the IO will finish, if rbd-nbd can set the IO ops some timeout via rados_osd_op_timeout (in the rbd-nbd) then its possible for us at ceph-csi to exactly define time to wait for rbd-nbd at rbd nodeplugin.

But till we have the fix in rbd-nbd, to avoid the frequency of hitting such issue at the csi level, we should consider increasing terminationGracePeriodSeconds to a good value from the current 30 sec.

I am sorry to say that, I am completely against this change.  That said, we should not increase the timeout to this proposed higher value. The main reason being:
*) This means, an application hang of 180 seconds+ ( 3minutes+) before kubelet can make next scheduling decision
*) The change is * NOT * NBD specific at all, we are pushing node plugin container timeout to a higer value which  affect all other scenarios even in absense of 'NBD' shares in place !
*) Also, here we are fixing a value blindly without any assurance of it can solve the issue.

As part of pod termination, Kubernetes initially sends SIGTERM to rbd-nbd and waits for ~30 sec i.e default terminationGracePeriodSeconds and then sends SIGKILL as part of ungraceful termination sequence. This is where some of the IO goes unhandled and mount point may turn out to read-only.

rbd-nbd expects some time for it to handle pending IO after SIGTERM is issued, currently at the rados level rados_osd_op_timeout is unlimited (wait forever), this means we never know when the IO will finish, if rbd-nbd can set the IO ops some timeout via rados_osd_op_timeout (in the rbd-nbd) then its possible for us at ceph-csi to exactly define time to wait for rbd-nbd at rbd nodeplugin.

But till we have the fix in rbd-nbd, to avoid the frequency of hitting such issue at the csi level, we should consider increasing terminationGracePeriodSeconds to a good value from the current 30 sec.

I am sorry to say that, I am completely against this change. That said, we should not increase the timeout to this proposed higher value. The main reason being:
*) This means, an application hang of 180 seconds+ ( 3minutes+) before kubelet can make next scheduling decision
*) The change is * NOT * NBD specific at all, we are pushing node plugin container timeout to a higer value which affect all other scenarios even in absense of 'NBD' shares in place !

I hope we are clear that the pod deletion is delayed only until the rbd-nbd process is waiting on IO, which means let's say for example if the rbd-nbd process is done (flushing the pending IO) in 2sec, then the pod can continue to terminate on the very next second.
This delay doesn't mean anything for non-rbd-nbd mounters (krbd).
So we only wanted to wait here when rbd-nbd processes are waiting for IO to finish at the OSD level. Because not waiting might cost us data loss.

*) Also, here we are fixing a value blindly without any assurance of it can solve the issue.

We are fixing the time bounded by pod eviction timeout i.e. 5m.
Yes, of course, this was just to add some time for rbd-nbd to finish its job and I never said this is the real fix, it was not even a complete workaround solution too. This was the least that we can do while our solution is still baking.
It might also be very much possible the inflight IO is still pending at the fs or block layers. To rule this out completely we are working on ways to fix it., some initial idea is updated here https://github.com/ceph/ceph-csi/pull/2313#discussion_r675575745
Fixing this is going to take some time and we shouldn't expect that to happen at 3.4.0 we can target it for the next release.
That all said, no objections from me waiting for the actual fix.

*) This means, an application hang of 180 seconds+ ( 3minutes+) before kubelet can make next scheduling decision
*) The change is * NOT * NBD specific at all, we are pushing node plugin container timeout to a higer value which affect all other >scenarios even in absense of 'NBD' shares in place !

I hope we are clear that the pod deletion is delayed only until the rbd-nbd process is waiting on IO, which means let's say for example if the rbd-nbd process is done (flushing the pending IO) in 2sec, then the pod can continue to terminate on the very next second.
This delay doesn't mean anything for non-rbd-nbd mounters (krbd).

Not true :) It is equally applicable to any process in the nodeplugin pod and its responsiveness to the outside events.

So we only wanted to wait here when rbd-nbd processes are waiting for IO to finish at the OSD level. Because not waiting might cost us data loss.

The dataloss and RO mount is a concern.

I hope we are clear that the pod deletion is delayed only until the rbd-nbd process is waiting on IO, which means let's say for example if the rbd-nbd process is done (flushing the pending IO) in 2sec, then the pod can continue to terminate on the very next second.
This delay doesn't mean anything for non-rbd-nbd mounters (krbd).

Not true :) It is equally applicable to any process in the nodeplugin pod and its responsiveness to the outside events.

I think there is a difference in understanding my statement. Try to answer this, and I hope you will get it. What process can be in stuck/hung in the state for more than ~30sec in a non-rbd-nbd mounter scenario?

I think there is a difference in understanding my statement. Try to answer this, and I hope you will get it. What process can be in stuck/hung in the state for more than ~30sec in a non-rbd-nbd mounter scenario?

@pkalever  we never know and thats why we have that timeout in first place even before nbd. Increasing that is risky as kublet get stalled during this period for next action.

I think one solution we could think of here is possibilities of prestop hook.  If nodeplugin want to do any 'bookkeeping' kind of work, the right approach would be looking at getting it done by the hook mechanism. The hook  is called immediately before a container is terminated due to an API request or management event such as a liveness/startup probe failure, preemption, resource contention and others. From there the grace termination period count down starts. Thoughts ?

authors: humblec idryomov nixpanic pkalever

repo: ceph/ceph-csi


text:
makefile: add a flag to run go commands via a container

These changes convert instances of the go and gofmt commands in the makefile to variables.
Then we add a new variable "flag" USE_GOCO=1 to enable running go commands through a container image. This is inspired by recent work     to enable development containers for VSCode, but is more aimed at     vim/cli users like myself who want to compile the code via makefile     rules and make use of vim's quickfix feature without having to have ceph     libs or, more importantly, the right version of the ceph libs, locally.
Full example: make build test-bins USE_CACHE=1 USE_GOCO=1

@mergifyio rebase

@mergifyio rebase

repo: ceph/go-ceph


text:
Reinstate change to LongByReference for listChildren

Hello @wido ,
Pull requests https://github.com/ceph/rados-java/pull/9 and https://github.com/ceph/rados-java/pull/11 by my colleague @nightfurys changed how children of snapshots are listed, to use LongByReference. This resolved a seg fault due to a mismatch between int sizes from Java to C++.
In Sep 2015, you changed this back to using IntByReference. Due to other changes in rados-java or ceph, the original symptom did not re-appear, but the problem now exists again in a more hidden fashion.
This pull request fixes the issue.
To reproduce the problem, increase the childCount from 3 to 82 in TestRbd#testListChildren. With the existing code, you'll get a coredump. With these fixes, you won't. I have not made this change to the unit test since it lengthens the test scope and runtime, but it could be changed if desired.
Note that if you change the childCount to 80 or 81, you will see errors about childImage9 not listed as a child, a symptom of verging on the threshold of failure. This threshold depends on (name length * number of children). For example, changing the name to "childImagechildImagechildImagechildImagechildImage" (50 chars) causes it to fail with 20 children.
The imageSize does not matter.
In addition:

a correction to a mvn command in README.md
a unit test change to clean up objects after tests complete, and objects that may be left over from previous runs, before tests start. This allows repeated runs of unit tests to pass, which they were not.

Thanks in advance. I'll watch for any comments here.

I see the Travis CI build failed, in the same test I am seeing failure, unrelated to the code changes in this PR. My errors look like:
	testCreateFormatOne(com.ceph.rbd.TestRbd)  Time elapsed: 0.247 sec  <<< ERROR!
	com.ceph.rbd.RbdException: Failed to remove image imageformat1
	        at com.ceph.rbd.TestRbd.cleanupImage(TestRbd.java:68)
	        at com.ceph.rbd.TestRbd.testCreateFormatOne(TestRbd.java:158)

TestRbd.java:158 is:
            cleanupImage(rados, ioctx, imageName);

An assertion is failing because the image is being created as format two instead of format one.
I see that the default for image creation was changed from one to two (documented in https://github.com/ceph/ceph/pull/10732). You can still create format one on the CLI as in:
	$ rbd create -s 20 --image-format 1 rados_junit/lptfmt1
	rbd: image format 1 is deprecated

	$ rbd info rados_junit/lptfmt1
	rbd image 'lptfmt1':
	        size 20480 kB in 5 objects
	        order 22 (4096 kB objects)
	        block_name_prefix: rb.0.2332d.74b0dc51
	        format: 1

But I'm not sure this accounts for the failure here, tracing how rados-java calls the rados C++ code. It seems to create a format one image.
A topic for another forum?

Thanks for the PR! I'll take a look, but at first glance it looks very nice.
I'm swamped with all kind of work, so PRs are very welcome!
Regarding the format 1 images, that's not rados-java, but your local librados could still default to format 1 images.
The rbd_create3() function for example creates format 2 images. But you can also set the rbd_default_format option to 2.

Hello @wido ,i add two methods in interface Rados,here are they:
int rados_ioctx_snap_rollback(Pointer ioctx, String oid,String snapname);
int rados_ioctx_selfmanaged_snap_rollback(Pointer ioctx, String oid,long snapid);
and i also add two methods in class ioctx to invoke those methods above,in order to realise rollback .

authors: daien0418 lincolnthomas wido

repo: ceph/rados-java


text:
task.internal.archive: gzip archived file size > 128MB

misc: add an optional write_to argument to misc.pull_directory()
so the caller can optionally specify the function to write
to local file.
task/internal: add a global option "log-compress-min-size" which
defaults to "128MB". if the size of a file pulled from remote
host is greater or equal to the specified size, it will be
compressed with gzip with the extension of ".gz" before
stored in the archive directory.

Signed-off-by: Kefu Chai kchai@redhat.com

tested at https://pulpito.ceph.com/kchai-2020-11-30_17:04:28-rados-wip-jan-testing-2020-11-30-1049-distro-basic-smithi/

https://pulpito.ceph.com/kchai-2020-12-02_05:39:45-rados-wip-jan-testing-2020-11-30-1049-distro-basic-smithi/

repo: ceph/teuthology


text:
Gozfs in Cerana

Issues affected/resolved
(See https://github.com/blog/1506-closing-issues-via-pull-requests)
Resolves #85
Description:
Bring mistifyio/gozfs into cerana as cerana/cerana/zfs. This includes full history. Commits starting from April 05 "Prepare to merge into cerana" are specific to this PR.

This change is

Reviewed 65 of 66 files at r1, 5 of 5 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

Merged via cli.

Fixed in 41db25e

authors: TauZero mmlb

repo: cerana/cerana


text:
removed lodash dep

We actually do not need lodash here :)

merge please :smiley:

authors: christianalfoni fmal

repo: cerebral/cerebral-operators


text:
docker run gives me "no such file or directory" for the config file

Hi there,
if I try to run the docker container it always exits with the following error. I've tried many things, changed the directory, tried the default file path, used different releases.
Failed to load config: could not read /root/reg_auth/auth_server.yml: open /root/reg_auth/auth_server.yml: no such file or directory

Hello, thank you for opening this issue. Could you share the docker run command (or docker-compose file) that you used to run the service?

Here you go :)
"docker run --rm --name docker_auth -p 5001:5001 --network registry_network -v /root/reg_auth:/config:ro     cesanta/docker_auth:1 /root/reg_auth/auth_server.yml"

I have some good news!
-v /root/reg_auth:/config:ro says that the /root/reg_auth directory will be found as /config inside the docker container. Which means you'll need to change /root/reg_auth/auth_server.yml to /config/auth_server.yml so the service inside the docker container can pick up the config :)

My mistake, sorry

@DL3KB all good, glad we were able to solve it :)


text:
Revisão do código de Jhonatan Castro

Nota

Escrita dos Testes 6.6/7.0

Qualidade dos testes

Isolamento 1.0/1.0
Nomenclatura do teste 0.6/1.0

Correta utilização de mocks, stubs, spies, fake e dummy 1.0/1.0

Cobertura 2.7/3.0

88.4%

Extra

GitHub Actions +0.5
Sonar +0.5

Nota Final: 10

@jhonatanscastro sua nota na disciplina Testes Unitários da Especialização em testes Ágeis é 10.

repo: cesar-school/unit-testing-project


text:
Feat/navigator 2.0

celkem improvizace, tak budu rád za jakýkoli feedback:) @ryzizub @killalad
Screenu s povolením notifikací jsem zatím skipl - tam je potřeba mít nějaký if statementy jestli ty notifikace už povolené jsou (na iOS), což zatím nemáme

ještě je nefunkční Skip routa "Nevím" u date pickerů - ta nemá skipovat na Create account screen ale na dalšího doktora (gynekolog / zubař např.). Ještě přemýšlím jak nejlíp na to (asi nějaký enum zase u OnboardingState provideru)

Ja mam komentar ke stabni kulture commitu.

Prvni pismeno velke - "Address review" :+1: ; "address review" :-1:
Pouzivej imperativni slovesa - "Add route guard" :+1: ; "added transition animations" :-1:
Prvni dva body nejsou dogma! Je to obecne doporuceni, nekdy dava smysl to formatovat jinak.

Ja mam komentar ke stabni kulture commitu.

Prvni pismeno velke - "Address review" 👍 ; "address review" 👎
Pouzivej imperativni slovesa - "Add route guard" 👍 ; "added transition animations" 👎
Prvni dva body nejsou dogma! Je to obecne doporuceni, nekdy dava smysl to formatovat jinak.
Edit: Deskriptivni title - vidim commit s nazvem "update". Update ceho? "Update dependencies" by pravdepodobne davalo vetsi smysl

good points 👍
ad 4. PRko bylo ve stavu draft, dělal jsem na více věcích zároveň + potřeboval jsem sync mezi laptopem a PC 😅

@ryzizub je třeba ještě něco adressnout?

authors: mzdm okarmazin

repo: cesko-digital/loono


text:
[Feature Request] - Enum <-> String

In the case of attributes with the same name and one is enum while the other one is string, use the enum tryparse and enum description function to do 2 way transform.

Hi
What is enum description function ? Could you provide sample mapping code that you expecting in described situation?

for example.

using System;

[Flags] enum Colors { None=0, Red = 1, Green = 2, Blue = 4 };

public class Example
{
   public static void Main()
   {
      string[] colorStrings = { "0", "2", "8", "blue", "Blue", "Yellow", "Red, Green" };
      foreach (string colorString in colorStrings)
      {
         try {
            Colors colorValue = (Colors) Enum.Parse(typeof(Colors), colorString, true);
            if (Enum.IsDefined(typeof(Colors), colorValue) | colorValue.ToString().Contains(","))
               Console.WriteLine("Converted '{0}' to {1}.", colorString, colorValue.ToString());
            else
               Console.WriteLine("{0} is not an underlying value of the Colors enumeration.", colorString);
         }
         catch (ArgumentException) {
            Console.WriteLine("{0} is not a member of the Colors enumeration.", colorString);
         }
      }
   }
}
// The example displays the following output:
//       Converted '0' to None.
//       Converted '2' to Green.
//       8 is not an underlying value of the Colors enumeration.
//       Converted 'blue' to Blue.
//       Converted 'Blue' to Blue.
//       Yellow is not a member of the Colors enumeration.
//       Converted 'Red, Green' to Red, Green.

So for string to enum if we can generate the following
Colors colorValue = (Colors) Enum.Parse(typeof(Colors), colorString, true);
ref: https://docs.microsoft.com/en-us/dotnet/api/system.enum.parse?view=netframework-4.7.2#System_Enum_Parse_System_Type_System_String_

Ok, and how to handle Enum -> String conversion? Just simply invoking ToString() method?

Yes.
Ref; https://docs.microsoft.com/en-us/dotnet/api/system.enum.tostring?view=netframework-4.7.2#System_Enum_ToString_System_String_

ok, I will try to make this kind of adjustment to the mapping rules

Thanks a lot for your work.

@bseng  here's a version with this feature, would you mind to test it? Any feedback is welcome.
https://ci.appveyor.com/api/buildjobs/xognjdrwd41ohp85/artifacts/MappingGenerator%2FMappingGenerator%2FMappingGenerator.Vsix%2Fbin%2FRelease%2FMappingGenerator.vsix

Hi @bseng, did you have an occasion to test this new feature?

I haven't got a chance yet.

It works great thanks!

Thanks for your time and the feedback, I really appreciate it.

authors: bseng cezarypiatek

repo: cezarypiatek/MappingGenerator


text:
Support Microsoft Git Provider

Any chance you could add support for the Microsoft Git Provider built into VS2013?

Microsoft Git Provider should be already supported and you can verify the switch is correctly done by checking "Options -> Source Control". I just tried it: differently from others provider the switch doesn't pop-up related windows. In case it doesn't work for you please tell me what happens when you:

Switch manually to a diffent provider (ex. Team Foundation);
Initialize a empty git repository on any project;
Close and reload the solution;
Go to "Options -> Source Control" and discover the loaded provider.

I'm not sure what I did wrong the first time. I retried using the process you posted above, and it's been working great ever since. Thanks for a great plugin!

authors: ceztko prodrive

repo: ceztko/SccAutoSwitcher


text:
Documentation typo

Found on the advanced components page, under getters:

Fixed, thanks

authors: ThatTonybo cferdinandi

repo: cferdinandi/reef


text:
Change the video on the Bureau page

One line fix to switch the video on the Bureau Page. Part of DFJR-1117.
Testing

Visit http://localhost:8000/about-us/the-bureau/ and play the video

Review

@jimmynotjim
@anselmbradford
@sebworks

:+1:

:+1:

authors: KimberlyMunoz jimmynotjim sebworks

repo: cfpb/cfgov-refresh


text:
Enable output of Resource snippet thumbnails in Snippet Lists

Per https://[GHE]/CFPB/hubcap/issues/54, we want to enable the dormant thumbnail field of Resource snippets for output in Snippet List modules.
Additions

A boolean option to show thumbnails for all snippets in a Snippet List
Template code to output snippet thumbnails in a Snippet List if the above option is selected
Some necessary styling to enforce the thumbnail width of 150px

Changes

Fixes the part where I was stupidly overriding Wagtail's native image template tag
Fixes a few whitespace mistakes (both in rendered styles and in code)

Testing

Pull branch
Migrate
Run server
Add a thumbnail to a Resource snippet
Set up a Snippet List that will pull in that Resource
Check out the rendered thumbnail in the Snippet List

Checklist

[x] PR has an informative and human-readable title
[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[x] Code follows the standards laid out in the development playbook
[x] Passes all existing automated tests
[x] New functions include new tests
[x] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged
[x] Visually tested in supported browsers and devices
[ ] Project documentation has been updated
[x] Reviewers requested with the Assignee tool :arrow_right:

@richaagarwal Thanks for the review! Yes, I will add tests for that.

Looks good to me.

@richaagarwal I've pushed up a test for the show_thumbnail logic (and basic snippet list output). It follows the model of existing organism tests in test_organisms.py, but I'm not sure if this is an outdated model we're trying to move away from. Please let me know what you think!

(I'll rebase just prior to merging.)

authors: Scotchester sebworks


text:
Add skip-nav styles to project

Adds the styles from this PR
https://github.com/cfpb/cfgov-refresh/pull/1923 to the project.

Checks out locally! :+1:

so, will downstream projects get this automagically, or do they need package.json updates?

They need the package.json update. I will update them this morning.

authors: KimberlyMunoz Scotchester rosskarchner

repo: cfpb/cfgov-sheer-templates


text:
wip fix: [AlertFieldLevel] margin 0.5em, icon vertical centering issue, TextInput changes, Checkbox changes

closes #276
TODO

[ ] Change Margin between icon and text to 5px in AlertFieldLevel
[ ] fix the centering icon in Alert (not fieldlevel) (Delay LOW PRIORITY: [context](figma comment that we will revisit this change later))
[ ] Error dotted line around checkbox/textinput
[ ] Center Checkbox, red outline, get rid of hover blue, focus color should be same as border (e.g. error red)

Changes

Sets text to inline instead of block
Vertically aligns the icon with the text
Sets the margin between the icon and the text to be 8px (0.5em)

Screenshots

Before
After

Heyo @shindigira! Is this one ready for a review or is still WIP?

authors: billhimmelsbach shindigira

repo: cfpb/design-system-react


text:
[native_dio_adapter] Windows build breaks when adding package

Package
native_dio_adapter
Version
1.3.0
Operating-System
Windows
Adapter
NativeAdapter
Output of flutter doctor -v
[√] Flutter (Channel stable, 3.19.0, on Microsoft Windows [Version 10.0.19045.4291], locale en-CA)
    • Flutter version 3.19.0 on channel stable at F:\Dev\tools\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision bae5e49bc2 (2 months ago), 2024-02-13 17:46:18 -0800
    • Engine revision 04817c99c9
    • Dart version 3.3.0
    • DevTools version 2.31.1

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\Users\Hakim\AppData\Local\Android\Sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = C:\Users\Hakim\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Build Tools 2022 17.7.4)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools
    • Visual Studio Build Tools 2022 version 17.7.34031.279
    • Windows 10 SDK version 10.0.22000.0

[√] Android Studio (version 2023.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)

[√] VS Code (version 1.88.1)
    • VS Code at C:\Users\Hakim\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.86.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19045.4291]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 124.0.6367.62
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 122.0.2365.80

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

• No issues found!

Dart Version
3.3.0
Steps to Reproduce

Create a flutter app
Add native_dio_adapter
Try to build for Windows

Expected Result
It should build properly
Actual Result
Building for Windows is broken for me, even though we don't want to use native_dio_adapter on Windows (we only need it for Android and iOS). Building for Android from the same Windows machine works.
This seems to be a probelm from the jni dependency. I've posted an issue here .
The only workaround I have found until now is to disable the Windows plugin for jni with a fork (since we don't need it). But would appreciate any other ways to prevent this issue.
CMake Error at C:/Program Files (x86)/Microsoft Visual Studio/2022/BuildTools/Common7/IDE/CommonExtensions/Microsoft/CMake/CMake/share/cmake-3.26/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find JNI (missing: JAVA_INCLUDE_PATH JAVA_INCLUDE_PATH2 AWT)
Call Stack (most recent call first):
  C:/Program Files (x86)/Microsoft Visual Studio/2022/BuildTools/Common7/IDE/CommonExtensions/Microsoft/CMake/CMake/share/cmake-3.26/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
  C:/Program Files (x86)/Microsoft Visual Studio/2022/BuildTools/Common7/IDE/CommonExtensions/Microsoft/CMake/CMake/share/cmake-3.26/Modules/FindJNI.cmake:569 (find_package_handle_standard_args)
  flutter/ephemeral/.plugin_symlinks/jni/src/CMakeLists.txt:32 (find_package)

Error: Unable to generate build files

Thank you!

This is unlikely to be solved from our side. Please follow up on the issue you've created with the dart/native.

repo: cfug/dio


text:
Add a login link in menu and relative login page (and password change page)

Using django.contrib.auth.views

password change link is available in the admin.

added.

authors: capooti zhangmingda

repo: cga-harvard/HHypermap


text:
OperationalError: database is locked

Hi,
I'm using a pipeline implemented by Ruffus that calls several other programs. I'm getting an error when Ruffus calls "bamM make".
This error only occurs when the data is stored on some parts of the server but not others. Any troubleshooting help would be greatly appreciated.
The error message is as follows:
Original exception:
Exception #1
  '<class 'ruffus.ruffus_exceptions.RethrownJobError'>

    Exceptions generating parameters for

    task = 'transcriptm.pipeline.QC_output'

Original exception:

    Exception #1
      'sqlite3.OperationalError(database is locked)' raised in ...
       Task = def transcriptm.pipeline.QC_output(...):

    Traceback (most recent call last):
      File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/task.py", line 5118, in parameter_generator
        job_history, verbose_abbreviated_path):
      File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/task.py", line 4923, in job_needs_to_run
        verbose_abbreviated_path=verbose_abbreviated_path)
      File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/file_name_parameters.py", line 548, in needs_update_check_modify_time
        if os.path.relpath(p) not in job_history and p not in set_incomplete_files:
      File "/srv/sw/python/2.7.4/lib/python2.7/_abcoll.py", line 348, in __contains__
        self[key]
      File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/dbdict.py", line 174, in __getitem__
        (key, )).fetchone()
    OperationalError: database is locked

    ' raised in ...
Traceback (most recent call last):
  File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/task.py", line 5887, in pipeline_run
    verbose)
  File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/task.py", line 5283, in fill_queue_with_job_parameters
    for params in job_parameters:
  File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/task.py", line 5215, in parameter_generator
    raise errt
RethrownJobError:

    Exceptions generating parameters for

    task = 'transcriptm.pipeline.QC_output'

Original exception:

    Exception #1
      'sqlite3.OperationalError(database is locked)' raised in ...
       Task = def transcriptm.pipeline.QC_output(...):

    Traceback (most recent call last):
      File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/task.py", line 5118, in parameter_generator
        job_history, verbose_abbreviated_path):
      File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/task.py", line 4923, in job_needs_to_run
        verbose_abbreviated_path=verbose_abbreviated_path)
      File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/file_name_parameters.py", line 548, in needs_update_check_modify_time
        if os.path.relpath(p) not in job_history and p not in set_incomplete_files:
      File "/srv/sw/python/2.7.4/lib/python2.7/_abcoll.py", line 348, in __contains__
        self[key]
      File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/dbdict.py", line 174, in __getitem__
        (key, )).fetchone()
    OperationalError: database is locked

I did a quick google and found this:
http://stackoverflow.com/questions/2740806/python-sqlite-database-is-locked
The checkpointing uses an SQLite database file (defaults to
.ruffus_history.sqlite) that requires file locking semantics to ensure
database integrity. The error message is telling you this has gone wrong.
Here are some suggestions:

Your underlying file system does not provide file locking semantics.
The stack overflow article says this can fail if your directory is samba
enabled....?
Run sqlite manually on the checkpoint database
$ sqlite3 .ruffus_history.sqlite
Check you don't have a hung pipeline process sitting on the file.
$ fuser .ruffus_history.sqlite
Check you don't have multiple pipeline running instances using the
same database file. (This is not sensible on so many levels...)
Check there isn't a crashed session that has not been cleaned up
properly
$ ls .ruffus_history.sqlite-journal
Check that the database is not corrupt
$ sqlite3 .ruffus_history.sqlite "pragma integrity_check;"
Try renaming the database
$ mv .ruffus_history.sqlite .ruffus_history.sqlite.backup

Please tell us if any of this works.
If your file system does not support placing the checkpoint database in
your working directory, you can always put it elsewhere
Leo
On 17 February 2016 at 05:43, AHCChan notifications@github.com wrote:

Hi,
I'm using a pipeline implemented by Ruffus that calls several other
programs. I'm getting an error when Ruffus calls "bamM make".
This error only occurs when the data is stored on some parts of the server
but not others. Any troubleshooting help would be greatly appreciated.
The error message is as follows:
Original exception:
Exception #1
'<class 'ruffus.ruffus_exceptions.RethrownJobError'>
Exceptions generating parameters for

task = 'transcriptm.pipeline.QC_output'

Original exception:
Exception #1
  'sqlite3.OperationalError(database is locked)' raised in ...
   Task = def transcriptm.pipeline.QC_output(...):

Traceback (most recent call last):
  File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/task.py", line 5118, in parameter_generator
    job_history, verbose_abbreviated_path):
  File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/task.py", line 4923, in job_needs_to_run
    verbose_abbreviated_path=verbose_abbreviated_path)
  File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/file_name_parameters.py", line 548, in needs_update_check_modify_time
    if os.path.relpath(p) not in job_history and p not in set_incomplete_files:
  File "/srv/sw/python/2.7.4/lib/python2.7/_abcoll.py", line 348, in __contains__
    self[key]
  File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/dbdict.py", line 174, in __getitem__
    (key, )).fetchone()
OperationalError: database is locked

' raised in ...

Traceback (most recent call last):
File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/task.py", line 5887, in pipeline_run
verbose)
File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/task.py", line 5283, in fill_queue_with_job_parameters
for params in job_parameters:
File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/task.py", line 5215, in parameter_generator
raise errt
RethrownJobError:
Exceptions generating parameters for

task = 'transcriptm.pipeline.QC_output'

Original exception:
Exception #1
  'sqlite3.OperationalError(database is locked)' raised in ...
   Task = def transcriptm.pipeline.QC_output(...):

Traceback (most recent call last):
  File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/task.py", line 5118, in parameter_generator
    job_history, verbose_abbreviated_path):
  File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/task.py", line 4923, in job_needs_to_run
    verbose_abbreviated_path=verbose_abbreviated_path)
  File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/file_name_parameters.py", line 548, in needs_update_check_modify_time
    if os.path.relpath(p) not in job_history and p not in set_incomplete_files:
  File "/srv/sw/python/2.7.4/lib/python2.7/_abcoll.py", line 348, in __contains__
    self[key]
  File "/srv/sw/ruffus/2.6.3/lib/python2.7/site-packages/ruffus/dbdict.py", line 174, in __getitem__
    (key, )).fetchone()
OperationalError: database is locked

—
Reply to this email directly or view it on GitHub
https://github.com/bunbun/ruffus/issues/63.

A solution is to use something like:

authors: AHCChan AndreasHeger bunbun

repo: cgat-developers/ruffus


text:
Adding a 'focused' assertion property

I initially tried flag(this, 'object').is(':focus') but that was not passing the test.  Using document.activeElement seemed to fix that.  Used http://api.jquery.com/focus/ as a reference.

Sorry for forgetting about this pull =(.  I pushed up the README changes in 5484664.

This is a useful feature. +1

Hey, I've just merged #73 which implements this feature.
Thanks for all of your hard work here everyone. Sorry @golmansax that this one didn't get merged - I hope this doesn't put you off making future code contributions to this project or others - your input was definitely valued!
I'll release the contents of #73 soon.

authors: golmansax jvennix-rmn keithamus

repo: chaijs/chai-jquery


text:
Add chainerx.cumsum

Add chainerx.cumsum.

I got rid of the chainerx._testing travis failure. The native implementation of cumsum still doesn't work as expected. Will make the fixes ASAP.

Fixed the implementation (Seems to work correctly when I tested locally). Only tests are remaining.

The tests are failing for float16 and bool inputs.

numpy.cumsum returns int64 ndarray when bool or integer ndarray has taken as an input. chainerx.cumsum should also follow this dtype rule.

Have made the suggested changes.

@asi1024 I have added the backward implementation. The backward tests pass but the double backward tests fail. Could you have a look at the implementation please?
I suspect the problem might be with the usage of Cumsum itself in the backward implementation.

Have rebased with master to include bug fix in Flip from #7727 . The tests pass now.

I have added a TODO to improve backward implementation.

Is this ready to merge @asi1024 ? Can I go ahead with cumprod and nonzero ?

Jenkins, test this please.

Jenkins CI test (for commit 227778f71f4f3cc4590566178a7443d7490a6c42, target branch master) succeeded!

LGTM. Thank you for the PR!

repo: chainer/chainer


text:
public API: Should /token be under /api/v1?

Currently, the /token route is at the root of the endpoint, and not under the /api/v1. Should it be? See what the best practices for public API say about authentication routes.

If we keep the idea of using the bearer token standart, I think it define the path the /token directly.
But otherwise, I'm at ease that the login related path to be outside the versionned API path. I see that as kind of 2 different things.

authors: greenscientist tahini

repo: chairemobilite/transition


text:
Default image for posts

Since there is no form validation for add posts at present and if the user doesn't add a blog post image, it creates a bad UI experience.
Add a default image for all blog posts. This will be shown if no image is uploaded to the post.

Hi, can I contact you via gmail, please? I have a few questions.

yes, you can contact me at chaitakgorai@gmail.com

Hello, I contacted you, but you didn't respond. Did you receive the email ? Maybe it went straight to the spam folder, so could you check it please?

Hi, can you check the pull request please ? :)

repo: chaitak-gorai/blogbook


text:
docs: advanced formik integration

Closes #407
📝 Description
I've created a recipe for advanced Formik integration.
🚀 New behavior
Now the users can see a more advanced example of how they can use ChakraUI with Formik, especially with Chakra components such as the radio button as an image. On top of that, I've created a SandPack since there were multiple files and I think that would be much better.
💣 Is this a breaking change (Yes/No):
No.

@nikolovlazar, should I add this page on the resources sidebar as well?

@aacevski we definitely should. I think it's better if we do the following changes:

Create a new group "Recipes"
Put the "Advanced Formik Integration" in the new Recipes group
Move the "Atlaskit editable" into the new Recipes group
Put the Recipes group between the "Advanced" and "Integrations" groups

Here's an idea of how it should look like:

I've done just that, thanks for the input!

@nikolovlazar this is ready to be reviewed again, I've changed the wording! 🚀

authors: aacevski nikolovlazar

repo: chakra-ui/chakra-ui-docs


text:
Why not Svelte?

🚀 Feature request
Vue, React, Solid, why not Svelte?

Hi @frederikhors,
The only thing holding us back from support Svelte is https://github.com/sveltejs/svelte/issues/5112
An important aspect of Zag is that we manage the event handlers internally and provide a way for users to simply spread it unto the elements. Svelte doesn't support events in spread props just yet.
When it does, we'll add support for it.

Can we leave this open?

Can we leave this open?

Yeah sure

Nope

authors: anubra266 frederikhors segunadebayo

repo: chakra-ui/zag


text:
Rails 5 + Mongoid 6

Are there any plans to update this project for Rails 5 and Mongoid 6?
Thanks.

The thing is that this gem depends on rails-observers, and rails-observers don't release new version that support rails 5.

Unfortunate that they haven't updated their gem. However, the master branch of rails-observers does seem to support Rails 5.

Yeah, the master branch of rails-observers does support rails 5. The thing is that I can't put it as dependency on the gemspec. The user needs to reference manually, in this case.

Thanks for getting started on this. However, I'm running into some problems.
First, I tried it with:
gem 'mongoid-observers'
gem 'rails-observers', github: 'rails/rails-observers'

The app started successfully but when I made changes to a record, the observer ended up inserting hundreds of entries instead of one. It threw an exception reentered error in  ActiveSupport::Notifications::Instrumenter#instrument.
My second attempt was with:
gem 'mongoid-observers', github: 'chamnap/mongoid-observers'
gem 'rails-observers', github: 'rails/rails-observers'

Unfortunately, the app doesn't even start. Here's the error:
~/app/.bundle/gems/ruby/2.3.0/bundler/gems/mongoid-observers-b5c5361e194a/lib/mongoid/observers/railtie.rb:22:in `block (2 levels) in <class:Railtie>': undefined method `to_prepare' for "expression":String (NoMethodError)

Both scenarios are running Rails 5.0.0.1. Hope this helps.

@pbougie, you can try again with master branch. I'm fixing the builds now.

Let's close this issue. I just released a new version. Let's me know if there is any problems.

authors: chamnap pbougie

repo: chamnap/mongoid-observers


text:
feat: implement custom tags input component

Just like the example (inspiration from YouTube)

Tasks

[x] create a very basic UI to add and remove tag
[x] add copy and clear all buttons
[ ] add custom delimiter support (in YouTube it's ,)
[ ] implement logic to count no. of characters
[ ] restrict the user to enter a certain amount of characters (maybe per tag or overall)

Task 3

Instead of adding as plain text, parse the pasted string and tokenize the tags. Use , as default delimiter.

repo: chankruze/react-components


text:
Remove multidict requirement

This is no longer necessary after #302.

@r0fls done.

repo: channelcat/sanic


text:
Fixing the unittests

Recently due to some decisions made in aiohttp's recent 0.2.0 release some features in sanic's testing suite have broken down. For now, until we fix what's broken or find an alternative, sanic's testing infrastructure will require aiohttp versions less than 0.2.0.
This PR also adds support for running our unittests in a Docker container so that we can have consistent environments in terms of running the unittests (Had some differences between my local OSX box and the Travis Jobs so this a nice happy medium).
To actually run them it's now:
make test
Other than that this PR also includes some cleaning of requirements files, re-organizing some tests, as well as fixing some random PEP8 errors.
Hopefully this will start turning the builds green... 🤞

Can we get these merged and a release made to pypi. This is starting to drag on and on.

@seemethere I agree we should do a pypi release.

authors: jkbbwr r0fls seemethere


text:
How to scale down working

This is my configure
{
  "ReadCapacity": {
    "Min": 2,
    "Max": 20,
    "Increment": {
      "When": {
        "UtilisationIsAbovePercent": 90
      },
      "By": {
        "Units": 2
      },
      "To": {
        "ConsumedPercent": 110
      }
    },
    "Decrement": {
      "When": {
        "UtilisationIsBelowPercent": 30,
        "AfterLastIncrementMinutes": 60,
        "AfterLastDecrementMinutes": 60,
        "UnitAdjustmentGreaterThan": 2
      },
      "To": {
        "ConsumedPercent": 50
      }
    }
  }
}

When I set table in Dynamodb is 25. It will scale down 20. But from 20 it doesn't scale down to 2. I want to autoscale to min value if the table in Dynamo doesn't read any units. How to configure?

Downscaling is rate limited by

the parameters you have set here, so 60 mins after the last increment and decrement, plus utilisation and min adjustment values as defined
a maximum of 4 decrements per day (imposed by aws), so it splits the remaining day into chunks so you don't use up your quota too soon and be stuck on a high rate.

I suspect its one of these but to work out exactly why it didnt drop you may need to debug through it, unfortunately the logging is not enough in all cases.  Closing for now but reopen if you feel there is certainly a bug.

repo: channl/dynamodb-lambda-autoscale


text:
Yf/lineage consolidation

Test plan:

run docker-compose run web "rake load_lineage_db"
check the taxon_lineages table with all names populated.

yes. great catch.

repo: chanzuckerberg/idseq-web


text:
feat(schema 5.1.0): update validation for obsm X_ embeddings

Reason for Change

#805

Changes

cache "is_spatial" attribute on Validator, based on which assay_ontology_term_ids are set
use "self._check_is_spatial" in validate_obsm to determine whether we error on missing X embeddings (we only error if dataset is NOT spatial)
error on X embedding if suffix is "spatial", in all scenarios

Testing

unit testing + Curator testing

Notes for Reviewer

Make sure you get @brian-mott to sign off as well.

I'd like to wait on @MillenniumFalconMechanic / @NoopDog to take a look first, to make sure we're tackling the epic in compatible ways (more context in Slack, but I am open to moving some of the logic in the validation yaml if that's how their implementation is shaping up)

@brian-mott ready for your QA now whenever you're ready!

repo: chanzuckerberg/single-cell-curation


text:
Update tooltip to match design

DoD:

Match Figma design
Modify frontend/src/views/WheresMyGene/components/HeatMap/utils.ts createChartOptions()'s tooltip object
Tissue Composition is cell_count / sum of cell_count for all cells in the tissue

Working on this along with https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/chanzuckerberg/single-cell-data-portal/1852

repo: chanzuckerberg/single-cell-data-portal


text:
FE Integration Tests

[x] Create a collection
[x] Delete a collection
[x] Publish a collection
[ ] upload a dataset
[ ] delete a dataset
[ ] delete a pending dataset
[ ] upload a bad dataset
[ ] delete a bad dataset
[ ] cannot retrieve a tombstoned collection
[ ] cannot retrieve a tombstoned dataset
[x] login
[ ] logout
[ ] list owners collections
[ ] only owner can delete and upload datasets to a collection

Definition of Done

[ ] Write an integration test that run on a deployed environment.
[ ] Ensure that cleanup happens after the test.

We should fill out the rest of the tests, since they are still relevant and will improve product stability

upload dataset test ticket: https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/chanzuckerberg/single-cell-data-portal/1349

@maniarathi @dsadgat Kuni suggested this as possible work for TestCo when they return to Single Cell.

Closing since curators now use the Discover API.

authors: atolopko-czi metakuni tihuan


text:
color by test in e2e isn't actually asserting anything

          Maybe worth a ticket to capture this, but this and the original test don't seem to assert after clicking the element it does color the umap 😆

Originally posted by @tihuan in https://github.com/chanzuckerberg/single-cell-explorer/pull/714#discussion_r1465390366

Thanks Seve! I added #717 as a blocker 👍

authors: seve tihuan

repo: chanzuckerberg/single-cell-explorer