struct.error: 'i' format requires -2147483648 <= number <=<PHONE_NUMBER>

I'm training a Cascade RCNN using mmdet 2.0, but after 3000 iters I got such error:
2021-08-30 00:53:10,508 - mmdet - INFO - Epoch [1][3000/4796]   lr: 0.00200, eta: 2 days, 15:57:30, time: 1.914, data_time: 0.529, memory: 29272, loss_rpn_cls: 0.0858, loss_rpn_bbox: 0.0754, s0.loss_cls: 0.6329, s0.acc: 80.7804, s0.loss_bbox: 0.2457, s0.loss_mask: 0.3179, s1.loss_cls: 0.3352, s1.acc: 78.5220, s1.loss_bbox: 0.2506, s1.loss_mask: 0.1529, s2.loss_cls: 0.1677, s2.acc: 80.6841, s2.loss_bbox: 0.1237, s2.loss_mask: 0.0698, loss: 2.4576
2021-08-30 00:54:45,441 - mmdet - INFO - Epoch [1][3050/4796]   lr: 0.00200, eta: 2 days, 15:51:02, time: 1.899, data_time: 0.503, memory: 29272, loss_rpn_cls: 0.0815, loss_rpn_bbox: 0.0734, s0.loss_cls: 0.6333, s0.acc: 80.4203, s0.loss_bbox: 0.2536, s0.loss_mask: 0.3176, s1.loss_cls: 0.3320, s1.acc: 78.2193, s1.loss_bbox: 0.2572, s1.loss_mask: 0.1527, s2.loss_cls: 0.1644, s2.acc: 80.4639, s2.loss_bbox: 0.1287, s2.loss_mask: 0.0699, loss: 2.4644
2021-08-30 00:56:33,999 - mmdet - INFO - Epoch [1][3100/4796]   lr: 0.00200, eta: 2 days, 15:52:56, time: 2.171, data_time: 0.557, memory: 29272, loss_rpn_cls: 0.0851, loss_rpn_bbox: 0.0760, s0.loss_cls: 0.6396, s0.acc: 80.6315, s0.loss_bbox: 0.2481, s0.loss_mask: 0.3193, s1.loss_cls: 0.3345, s1.acc: 78.2621, s1.loss_bbox: 0.2521, s1.loss_mask: 0.1525, s2.loss_cls: 0.1640, s2.acc: 80.5509, s2.loss_bbox: 0.1238, s2.loss_mask: 0.0693, loss: 2.4642
Traceback (most recent call last):
  File "/home/work/.conda/envs/co/lib/python3.7/multiprocessing/queues.py", line 242, in _feed
    send_bytes(obj)
  File "/home/work/.conda/envs/co/lib/python3.7/multiprocessing/connection.py", line 200, in send_bytes
    self._send_bytes(m[offset:offset + size])
  File "/home/work/.conda/envs/co/lib/python3.7/multiprocessing/connection.py", line 393, in _send_bytes
    header = struct.pack("!i", n)
struct.error: 'i' format requires -2147483648 <= number <=<PHONE_NUMBER>

How to solve it ?

It seems the code got error when packaging something for multiprocess. Could you follow the issue template to describe the detail environments, running commands, etc?


cms 165 won't generate patient data

message: Attempted to set a value for 'principalDiagnosis' which is not allowed on the model QDM::EncounterPerformed. summary: Without including Mongoid::Attributes::Dynamic in your model and the attribute does not already exist in the attributes hash, attempting to call QDM::EncounterPerformed#principalDiagnosis= for it is not allowed. This is also triggered by passing the attribute to any method that accepts an attributes hash, and is raised instead of getting a NoMethodError. resolution: You can include Mongoid::Attributes::Dynamic if you expect to be writing values for undefined fields often. on /vendor/bundle/ruby/2.6.0/gems/mongoid-6.4.5/lib/mongoid/attributes/processing.rb:99:in `process_attribute'

Brandy, are you seeing this error message on the Demoserver?


Consolidate usage fields into single bitwise field to trim memory

Don't know if we'd definitely want to do this or not, but it was quick to implement and test.  No performance regression, and probably some marginal memory savings for large builds with lots of confs at the cost of marginally more complex code.
It's here for consideration.

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test APT

I've triggered the following builds for you:

AllPerformanceTestsReadyForRelease build

Closing as not impactful enough to warrant the additional complexity.


Type='Date' not working in IE

We're currently using an editable date field in our project. The date picker pops up properly on Chrome and Firefox but noticed it doesn't work in IE (which is necessary for us). When editing the field, it acts as a regular text field in IE instead of displaying the date picker tab.
This is the field:
{ field: 'ApprovalDate',          displayName: 'Approval Date', type: 'date', cellFilter: 'date:"yyyy-MM-dd"',      width: 144 }
I searched and noticed the ones in this tutorial don't work in IE as well:
http://ui-grid.info/docs/#/tutorial/201_editable
Also, is there any way to disable the "X" and the up/down icons when editing the date field. I've attached a screenshot showing what I'm talking about.

I don't believe this is an issue with ui-grid.
The "date" type of html input tag is a new part of html5 and is not supported in IE or Firefox, currently.
If you want a consistent experience across browsers, you should consider implementing your datepicker with jQuery or Angular-ui (ui-bootstrap).
Sources:
http://www.w3schools.com/html/html_form_input_types.asp
https://stackoverflow.com/questions/12455506/html5-input-type-date-color-range-support-in-firefox-and-internet-explorer

Gotcha. I'm still fairly new to angular so I assumed when defining type="date", it was using some pre-defined cell template that was set by ui-grid. I looked into using jQuery, but the grid is in a modal, so I had some challenges. I'll look into Angular-ui datepicker.
Thanks for the reply.

Be careful here - there are a number of issues open related to this - #2279, #2321, #2196, #1785, #2290.
They all come down to that ui-grid uses native html5 editors and widgets, and not all browsers support these as yet.  What would be nice would be to implement bootstrap widgets instead (as a feature), but that is actually kinda hard, and is why people haven't done it.  Any new editor directive needs to honour the EDIT_START and other EDIT events - refer the edit tutorial that briefly mentions this, and look in the edit feature itself to see the directives that have been created for each of the edit widgets.


Unable to create a new user

I am running the latest konga Docker image as well as kong version 0.10.3. I am able to update the existing konga admin and demo users in the dashboard, which is great. But when I select Create User, enter all the information into the form, and click the Submit User button, nothing happens. The screen doesn't change except for the very brief appearance of a loading wheel on the left side of the Submit User button.
Additionally, if I input no user information into the form and click the Submit User button, the same happens as described above: no response except for the quick flash of the loading wheel. I would expect that in this case I should see error messages appear as I did not populate the required fields.
Any ideas why the Submit Button click event is not triggering the appropriate responses?
Thanks!

Hi @jamelseagraves and sorry for the late response. I was on vacation :).
What version of Konga are you using?
Everything here seems to work fine with version 0.8.3. Check out the  demo instance as well. A new user can be created normally.
If you're still having this issue when trying to create a new user,  open the browser's javascript console and post the errors here.

No problem @pantsel. Thanks for the reply.
I am using version 0.8.3 for my instance. Not sure what could be the issue as everything else seems to work fine.  Can you please expand config in this line and post it again?
{data: "forbidden", status: 403, config: {…}, statusText: "Forbidden", headers: ƒ}

Here is the config object broken out:
config:
	cached: false
	data:
		active: true
		admin: false
		email<EMAIL_ADDRESS>		firstName: "testy"
		lastName: "testerton"
		passports: {password: "testsecret", protocol: "local"}
		password_confirmation: "testsecret"
		token: "eyJhbGciOiJIUzI1NiJ9.MQ.Lu-KcR4aCeuT9hi1K474zV3s4VaopLDCcf4nZvH6DQo"
		username: "test"
		__proto__: Object
	headers: {Accept: "application/json, text/plain, */*", Content-Type: "application/json;charset=utf-8", authorization: "Bearer eyJhbGciOiJIUzI1NiJ9.MQ.Lu-KcR4aCeuT9hi1K474zV3s4VaopLDCcf4nZvH6DQo"}
	method: "POST"
	paramSerializer: ƒ ngParamSerializer(params)
	transformRequest: [ƒ]
	transformResponse: [ƒ]
	url: "auth/signup"
	__proto__: Object
	data: "forbidden"
	headers: ƒ (name)
	status: 403
	statusText: "Forbidden"

I was just going to open a ticket but found this.  If you have the setting "Allow users to sign up." disabled, (i.e. the signup tab on the login screen is not visibile) then you cannot add new users though any means. If you enable signups you can add users.  This is definitiely a bug. The creatUser function calls signup and and signup requires needsSignupEnabled : true.

@azumafuji .
Great! Thanks for the pointer! I'll get to it asap.

Thanks @azumafuji! Good catch.


[question] work with unavailable attribute

I am using @available(*, unavailable, message: "Use NewRequestListTableViewCell_v2 instead") attribute for one of class. Upon compiling the project, I get error in R.swift file because the library generates code for the unavailable class as well.
What is the right way to use unavailable attribute with R.swift?

Unfortunately, R.swift can't read your source code, so it doesn't know about the unavailable attribute.
At the moment, the only way to have R.swift not generate a reference to an unavailable class, is to not reference it from your Storyboard / Nib. Or perhaps by ignoring the file, but I haven't tried that for this use case.

I will try the 'ignore the file' option. Closing the issue now. Thanks for the response.


Use KOKKOS_CLASS_LAMBDA

Requiring C++17, we can now resolve the comments mentioning "capture *this". Also, this shows that we forgot to update the GitLab CI. I would propose to always just build Kokkos for clarity.

Does ArborX requiring C++17 mean that we require C++17 from Kokkos? I didn't think so. So I'm not sure that we can use this approach until the minimum Kokkos version we require is a one requiring C++17.

Avoid mixing. Use the same compiler and the. same cxx standard.

@masterleinad Can you please check whether xSDK builds Trilinos with C++17, or whether there are plans to do that?

@masterleinad Can you please check whether xSDK builds Trilinos with C++17, or whether there are plans to do that?

https://github.com/xsdk-project/spack-xsdk/blob/develop/var/spack/repos/builtin/packages/xsdk/package.py currently shows that they are building with C++14 support. For the next release, they will likely go with whatever the bundled Kokkos requires at that point. Hence, it depends probably on the date Kokkos 4.0 is released (which would require C++17).

Alternatively, put into kokkos_ext header.

Build fails with error: function-like macro 'KOKKOS_VERSION_GREATER_EQUAL' is not defined


Test cases on sphere with Earth topography

@akshaysridhar #1202 doesn't close this, right? will there be a PR for this?

@szy21 Yes, there will be a separate PR to introduce the Earth cases into the buildkite setup.

Moist Baroclinic Wave Test Case: Earth Orography with Gaussian Smoothing (Raw -> Lat Long Spline Interpolation) + Laplacian Smoothing (On nodal points, with spectral div/grad operators)
h_elem = 16
n_poly = 3
z_elem = 25
z_stretch = false
kappa_4 = 8e16
alpha_rayleigh_w = 1.0
alpha_rayleigh_uh = 1e-4
use_reference_state = false
t_end = 30days

Moist Baroclinic Wave Case Summary:

dt
kappa_4
z_elem
z_stretch
h_elem
n_poly
t_end (days)
use_ref_state
P/F

25
8e16
25
false
16
3
30
false
P:day30 (sim end time)

35
8e16
25
false
16
3
30
false
P:day30 (sim end time)

50
8e16
25
false
16
3
30
false
P:day15 (in progress)

100
8e16
45
false
16
3
30
false
F: sat_adjust -> T = NaN

100
1e16
45
true
16
3
30
false
F: sat_adjust -> T = NaN

Dry Baroclinic Wave Test Case Summary:

dt
kappa_4
z_elem
z_stretch
h_elem
n_poly
t_end (days)
use_ref_state
P/F

25
8e16
25
false
16
3
30
false
P:day30 (sim end time)

50
8e16
25
false
16
3
30
false
P:day30 (sim end time)

75
8e16
25
false
16
3
30
false
P:day10+ (in progress)

100
1e16
45
false
16
3
30
false
F: NaN (< day 2)

150
1e16
45
true
16
3
30
false
F: NaN (< day 2)

summary_1_day30.pdf
summary_10_day30.pdf

Dry HS: Day 400 (simulation end time)
[day = 400, dt = 300s, kappa = 1e16, Gauss-smoothing = 4, Laplace-smooth = 100R_e, 100 passes]
Level 1 Summary
summary_dry_hs_l1_day400_dt300_1e16_smooth4.pdf
Level 10 Summary
summary_dry_hs_l10_day400_dt300_1e16_smooth4.pdf
Equil-Moist HS: Day 125 (simulation end time 400 days, in progress)
[day = 125, dt = 100s, kappa = 1e16, Gauss-smoothing = 4, Laplace-smooth = 100R_e, 100 passes]
Level 1 Summary
summary_emhs_l1_day125_dt100_4e16_smooth5.pdf
Level 10 Summary
summary_emhs_l10_day125_dt100_4e16_smooth5.pdf


Update index.js

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

Hi @Bennebotix, this PR doesn't really make sense, and you haven't explained what you're trying to do, so I'm going to close it. I don't think this code should be made more complicated like this - the idea is it's a super simple generic base that people can extend, not a complex example.
There are more complex examples listed in the readme. If you think there's something missing that might be useful there, feel free to open a PR against that file.


Task is not detected if gulp installed after 'Run Task' command was executed

Testing #29442.

Start with fresh workspace
Create gulpfile.js as mentioned in instructions.
Execute Tasks: Run Task command. -> No tasks found.
npm install gulp in workspace directory
Execution of the command again does not detect any tasks until you reload VS Code.

It would be great if it can pick up tasks without reloading after freshly installating gulp.

We could watch the node_modules/gulp folder. However watching the global install folder might not be a good idea. In general this is a trade off between performance and reload. We could always try to detect the tasks when needed however this will make the UI slower. But at the end this is up to the extension implementor.

Instead of watching, can we just fs.stat node_modules/gulp to see if something changed since last time the command was executed, or this is also considered slow?
Otherwise, there should be an option to go around this, e.g. special on-demand command to refresh this list that always appears at the bottom of the list. As a user I do not see a reason why would I reload VS Code when I perform npm install gulp, these are implementation details on the editor side.

@michaelchiche nice idea.

With multi root and global installed gulp I am not sure if we should implement this. Keeping as a feature request.


[W6.1f][T15-B4]Azizi Azfar

removed "throw new UnsupportedOperationException("This method should be implement in child classes");"

@aziziazfar
Nothing much to review here. Hence closing with minimal review


about the train data

First. thanks your code. I can't down the data in China . because I cannot visit the  google driver. can you share it by Baidu disk? . also I try to down it from the http://roboimagedata.compute.dtu.dk/ but the problem is that I can't find the right version. as your paper said : As it provides the ground truth point cloud with normal information, we use the screened Poisson surface reconstruction (SPSR) [16] to generate the mesh surface, and then render the mesh to each viewpoint to generate the depth maps for our training I think is should be a point cloud data. But I'm not sure which one should I download. Besides. I  want try to classify the object with different size .  so I want to classify with the depth image. I'm not sure if it can work. do you have any good idea about the classification? many thanks

thanks very much!!!

hi YoYo000. sorry to bother you again. I want to produce the extrinsic and intrinsic by a series images from two fixed cameras. I use the opensfm but it produce like this :


Add Events Dynamically and Refreshed CalenderView?

Hi, Thanks for the great library.
As title clarifies that i need to add an event, when i select a date a pop up will appear (already done) and upon save i need to add and show that event on calendar view?
second i am getting events data from server so how can i refresh calendar view to show updated events?
I am using Example 5 for my project.

Example 5 is the perfect guide which you seem to have followed. To refresh a date, just call calendarView.notifyDateChanged() and pass in the date you want to update. This causes the view for that date to be rebound, allowing you to update the view to show that it has an event.

The CalendarView does not store your events, you just have to update the UI for dates with an event to display how you prefer in the provided DayBinder.

Thanks


ARROW-3607: [Java] delete() method via JNI for plasma

Exposed delete() method

@ShubhamChaurasia looks pretty good to me though note that there are some Java linting errors.

LGTM. Thanks for the contribution.

Thanks @ShubhamChaurasia! I'm merge this once the linting errors are fixed.

Codecov Report

Merging #2829 into master will increase coverage by 1.34%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #2829      +/-   ##
=========================================
+ Coverage   86.65%     88%   +1.34%
=========================================
  Files         493     425      -68
  Lines       69675   64594    -5081
=========================================
- Hits        60375   56844    -3531
+ Misses       9204    7750    -1454
+ Partials       96       0      -96

Impacted Files
Coverage Δ

cpp/src/arrow/util/parsing.h
95.6% <0%> (-4.4%)
:arrow_down:

cpp/src/arrow/builder.cc
78.3% <0%> (-3.93%)
:arrow_down:

cpp/src/arrow/buffer.h
94.05% <0%> (-1.99%)
:arrow_down:

cpp/src/arrow/array.h
98.37% <0%> (-1.63%)
:arrow_down:

cpp/src/arrow/builder.h
97.06% <0%> (-0.7%)
:arrow_down:

cpp/src/arrow/compute/kernels/hash.cc
96.07% <0%> (-0.21%)
:arrow_down:

cpp/src/arrow/test-util.h
97.97% <0%> (-0.03%)
:arrow_down:

cpp/src/plasma/client.cc
84.54% <0%> (ø)
:arrow_up:

cpp/src/arrow/compute/kernels/util-internal.h
100% <0%> (ø)
:arrow_up:

cpp/src/arrow/csv/csv-converter-test.cc
100% <0%> (ø)
:arrow_up:

... and 87 more

Continue to review full report at Codecov.

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


Sync PyLint config with Python judge

https://github.com/dodona-edu/judge-pythia/pull/56

While we’re at it, we might consider ruff as a more performant replacement for pylint


Rename QueryIdCachingProxyHandler to HttpUtils

Description
Supersedes #504
Fixes https://github.com/trinodb/trino-gateway/issues/480
Release notes
(x) This is not user-visible or is docs only, and no release notes are required.

Merging as https://github.com/trinodb/trino-gateway/pull/504 is already approved.

👍


Setup depth limit accept function

Description
In short, I was thinking about how to achieve auto-depth-limit effect based on the width of the current window to make use of the WinBar as much as possible. I believe this will be a good enhancement for anyone.
Setup
With this (partial) config:
config = function ()
  local navic = require('nvim-navic')
  navic.setup{
    depth_limit = function (context)
      return vim.api.nvim_win_get_width(context.winid)
        - string.format('%s%s  ', vim.fn.fnamemodify(context.path, ':t'), context.mod and '*' or ''):len()
    end,
  }

where I define the context to have these data that a user cannot get itself in the depth_limit function:
{
  winid = vim.api.nvim_get_current_win(), -- the current window id.
  path = vim.fn.expand('%:p'), -- full path of the current file.
  mod = vim.bo.mod, -- whether or not the current file is modified.
}

and autocmd like this:
vim.api.nvim_create_autocmd({ 'CursorHold', 'InsertEnter' }, {
  callback = function()
    if should_exclude[vim.bo.filetype] or should_exclude[vim.bo.buftype] then return end
    local winbar_prefix = string.format('%s%s  ', vim.fn.expand('%:t'), vim.bo.mod and '*' or '')
    local winbar_content = navic.is_available and navic.get_location() or ''
    vim.wo.winbar = winbar_prefix .. winbar_content
  end,
})

Then we can achieve the effect as DEMO:

If the content of WinBar from nvim-navic exceeds the return value of depth_limit, the depth_limit_indicator will be used to fold the content in a leave-less-space manner. (The old behavior: a fixed depth is used. It's very likely to waste the space of WinBar.)

If not, then the full content of WinBar from nvim-navic will be used instead, without the depth_limit_indicator. (In the DEMO, I use my own plugin to create a floating window with larger width for the current buffer my cursor is on.)

https://user-images.githubusercontent.com/24765272/211439858-a962289c-2995-4af8-976d-993e8cd66bdb.mov

Notes:

This PR doesn't include any modification of the docs. I need your help on this.
While this change is backward-compatible, I also added some fixes to the current implementation so that it's more logically correct, i.e. the two start_index > 1 checks to make sure that depth_limit_indicator will only be shown when it should.
You might have a better idea of variable naming.

Thanks for your reading.

FYI: 73f57e2 can be better implemented if it's possible to get_location() without those highlight annotations. Maybe providing something like get_location(no_hl=true) would help.

While this rarely happens: you might need to add some padding/offset for the value that is returned by depth_limit. Otherwise, you might still have a chance to see the built-in < sign instead of the separator you defined. This is caused by a miscalculation of the limit.
local padding = 10
config = function ()
  local navic = require('nvim-navic')
  navic.setup{
    depth_limit = function (context)
      return vim.api.nvim_win_get_width(context.winid)
        - string.format('%s%s  ', vim.fn.fnamemodify(context.path, ':t'), context.mod and '*' or ''):len()
        - padding
    end,
  }

Hi @nyngwang !
Thanks for such a detailed PR, I appreciate it very much 😄
Since get_location provides ability to override the depth_limit option when its called by passing the opts table to it, does this approach of accepting a depth function have any advantage over the user themselves making the calculations before calling the get_location function.
For example, If someone is using lualine they could just do something like below code to achieve similar results right?
require("lualine").setup({
    sections = {
        lualine_c = {
            { function()
                   -- Some maths to determing depth value
                   return navic.get_location({ depth_limit = <calculated value here>})
              end,
              cond = navic.is_available },
        }
    }
})

The implementation can be improved a lot if you could provide a polymorphic function get_location(no_hl=true) to get rid of those highlight annotations. (This should increase the readability a lot.)

get_location can be called with highlights=false like so
get_location({highlight = false})

Is this what you were looking for?

Hi, @SmiteshP:

Since get_location provides ability to override the depth_limit option when its called by passing the opts table to it, does this approach of accepting a depth function have any advantage over the user themselves making the calculations before calling the get_location function.

My original idea is to help users to get an adaptive depth without any calculation themselves since it's very reasonable to have a no-off-screen winbar by default. (I did read your document and did try to externalize the while loop before. And at that moment I decided to create a PR because I thought it will be more efficient since I might have access to some internal data.)

get_location({highlight = false}) Is this what you were looking for?

I will revisit my implementation after rebasing all my stuff onto the current master. it's my bad that I didn't notice the option at that time. Apologies.

Can you briefly explain whats happening here

With highlight = false, maybe I can remove them all.

get_location({ highlight = false })

I just tried. It is what I was looking for, but the output is inaccurate. Say it output the following string:
 return   use   config   navic.setup

The length(by calling :len())  of it is 57, but it's actually 43. (I guess this might be related to those symbol?)
01:46:22 msg_show   print(require('nvim-navic').get_location({ highlight=false }):len()) 57
01:46:37 msg_show   print(type(require('nvim-navic').get_location({ highlight=false }))) string
01:46:45 msg_show   print(require('nvim-navic').get_location({ highlight=false }))  return   use   config   navic.setup

Closing cus I think it will be more efficient if I just create a new PR.


Port all campfire tools to a Bazel-only world [phab:D254]

Created by schroederc at 2015-05-14 20:19:34:
This moves some tools completely:

buildtools/linter.sh
buildtools/module_versions.sh

And forks the rest:

buildtools/{check,update}_modules.sh

Everything else is updated to reflect these changes and the
non-existance of campfire.

schroederc wrote at 2015-05-15 16:38:48:
Fix update.sh to use clang++/g++

schroederc wrote at 2015-05-15 17:03:48:
Fix cpp config

schroederc wrote at 2015-05-15 17:25:23:
Setup CROSSTOOL toolchain.compiler setting

zarko commented inline (context lost during migration) at 2015-05-15 18:01:13:
Meganit and this looks like my fault: "dependent" doesn't really work here; maybe "required"?

zarko commented inline (context lost during migration) at 2015-05-15 18:01:13:
Maybe add a call to update.sh in this file?

zarko accepted this revision at 2015-05-15 18:01:13

schroederc wrote at 2015-05-15 18:03:46:
review change

schroederc commented inline (context lost during migration) at 2015-05-15 18:03:55:
I'd like to keep that separate.  This is just for Bazel.

schroederc commented inline (context lost during migration) at 2015-05-15 18:03:55:
Done

zarko wrote at 2015-05-15 18:11:41:
SGTM, still good to land

schroederc closed via commit cc154417123e5cbc7ede111abeaa0d41d4f59ac7 at 2015-05-15 18:12:23


how do i translate the text and contribute it

question but not for bugs
i want to translate the text into Traditional Chinese(zh-TW), and i found that there are many files about translation in i18n
i think these json files may change anytime when they are updated (like the bluefeed feature that was added yesterday), so i would like to know how to maintain these translation files
or is there any guideline for translations and contributions?
thanks!

@yotv2000tw
Thank you for your interest in translations!
Basically, there is no problem if you pull request the ch-zw.json file that you are currently creating.
Of course, the number of lines will increase as features are added, but untranslated portions will fall back to English.
The languageMap directory is for translations related to language names, the labeling directory is for translations related to moderation labels, and everything else is consolidated in json files in the locales directory.

ok, thanks!


Remove map module from la-arena public API

It’s unlikely that more items will be added to the module, so it’s simpler for users if ArenaMap is re-exported and the module made private.
This doesn’t compile for the same reason that #7275 doesn’t:

This pull request doesn’t compile because dependencies on la-arena go through crates.io, so existing dependencies on the crate are referencing an old version. As such, this PR will only compile once a new la-arena version has been published.

urgh, I guess we need to spell deps in Cargo.toml as { version = "0.2", path = "../../../../lib/arena"} in the end? Could you update the tomls then(/

urgh, I guess we need to spell deps in Cargo.toml as { version = "0.2", path = "../../../../lib/arena"} in the end? Could you update the tomls then(/

Should we move it to a different repo?

Should we move it to a different repo?

Maybe! But I'd love to experiment with the lib folder a bit at first. Abstractly, keeping all independent crates.io crates (like rowan or text-size) in a monorepo in a lib dir seems proper to me, but I'd like to see the amount of accidental complexity we get here...
The crate's on crates.io thoug, there's no need to download the repo

Maybe! But I'd love to experiment with the lib folder a bit at first. Abstractly, keeping all independent crates.io crates (like rowan or text-size) in a monorepo in a lib dir seems proper to me, but I'd like to see the amount of accidental complexity we get here...
The crate's on crates.io thoug, there's no need to download the repo

The crate's on crates.io thoug, there's no need to download the repo

TIL. I thought the crate archive would include the whole repo, it looks like it doesn't.

The crate's on crates.io thoug, there's no need to download the repo

TIL. I thought the crate archive would include the whole repo, it looks like it doesn't.

Abstractly, keeping all independent crates.io crates (like rowan or text-size) in a monorepo in a lib dir seems proper to me, but I'd like to see the amount of accidental complexity we get here...

I think a monorepo for these projects is a bad idea, since people who are interested in them and want to, for example, watch the repo, aren’t necessarily interested in activity on rust-analyzer (apart from rowan maybe). This is doubly true in the case of la-arena, which is just a generic data structure that isn’t obviously related to a language server implementation.

Abstractly, keeping all independent crates.io crates (like rowan or text-size) in a monorepo in a lib dir seems proper to me, but I'd like to see the amount of accidental complexity we get here...

I think a monorepo for these projects is a bad idea, since people who are interested in them and want to, for example, watch the repo, aren’t necessarily interested in activity on rust-analyzer (apart from rowan maybe). This is doubly true in the case of la-arena, which is just a generic data structure that isn’t obviously related to a language server implementation.

Should I update the Cargo.tomls or wait for the next version of la-arena to be published?

Should I update the Cargo.tomls or wait for the next version of la-arena to be published?

update the tomls to include both the path and the version fields
On Fri, 15 Jan 2021 at 15:08, Aramis Razzaghipour<EMAIL_ADDRESS>wrote:

Should I update the Cargo.tomls or wait for the next version of la-arena
to be published?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rust-analyzer/rust-analyzer/pull/7276#issuecomment-760904684,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANB3M5FSCKHO2ZJPHOYQZDS2AV2VANCNFSM4WDIXSRA
.

update the tomls to include both the path and the version fields
On Fri, 15 Jan 2021 at 15:08, Aramis Razzaghipour<EMAIL_ADDRESS>wrote:

Should I update the Cargo.tomls or wait for the next version of la-arena
to be published?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rust-analyzer/rust-analyzer/pull/7276#issuecomment-760904684,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANB3M5FSCKHO2ZJPHOYQZDS2AV2VANCNFSM4WDIXSRA
.

bors try

bors try

Oops 😅

Oops 😅

bors try

bors try

Let's wait for the other PR.

Let's wait for the other PR.

bors r+

bors r+


German Alternative Translation

I found an alternative German translation but it has a lot of differences some translations aren't as good and some are better.
Too many differences for one person to manage.
global_ger_alternative.txt

Oha where did you find the really is not bad but it would be a lot to check. :(

Somebody uploaded it to a file host page and linked it on Spectrum without any source

I found another really active Repo for a German Translation. I asked them if they would like to Collaborate.
https://github.com/rjcncpt/StarCitizen-Deutsch-INI

I don't know what their problem is. They seem to want to do their own thing... They're just typical Knebel people. xD

I don't know what their problem is. They seem to want to do their own thing... They're just typical Knebel people. xD

Yes, okay then not.  I just find it unnecessary if everyone cooks their own soup.  It's more work for everyone.  😪😮💨

I agree with you on that.


nginxMainline: 1.25.2 -> 1.25.3

Changes: https://nginx.org/en/CHANGES
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 263304 run on x86_64-linux 1

4 packages built:

nginxMainline
nginxMainline.doc
nginxQuic
nginxQuic.doc


Github emoji not working (anymore?)

Thought this was supported?

Bug Report Dump (Auto-generated)

Version 1.26.0<PHONE_NUMBER>)
Device: iPhone X (iOS 12.1)
TestFlight: true

Was also about to post this as I saw labels for :bug: bug (assuming that’s related)

I don’t think we had those in labels before so I am not sure if they were supported in the first place, but yes.
Sent with GitHawk

🚫
Sent with GitHawk

Oh I bet it’s from moving the string replace into markdown text instead of the whole string. Needs to find/replace HTML too.
Sent with GitHawk

Nope we’ve never supported emoji in labels
Sent with GitHawk


Is that possible to recover the original transcript from the .wav stimulus, including the tone and the punctuation?

Hello,
There is no "original" transcript and .wav files are just audio files they do not contain by themselves any other information. All of the stories were originally spoken stories and as part of The Moth they were not allowed to prepare a written story in advance. The syntax of spoken stories do not fall cleanly onto patterns of punctuation and syntax found in written text. Thus it would be non trivial and likely a manual process if you wanted to approximate punctuation.
Additionally we do not currently have any kind of tone labeling system. But this is likely possible to do from the .wav files yourself by developing a feature space if that is something you are interested in. I would recommend searching out some NLP tone models and working off of that to develop a tone feature space.
Good luck!
Amanda


Fix garnix ci

According to the failling CI stderr output, flake input 'flake-parts'
error: 'flake:flake-parts' is an indirect flake reference, but registry
lookups are not allowed. Hope adding flake-parts input fix this.

Update lock broken daed due to https://github.com/NixOS/nixpkgs/pull/351836#issuecomment-2442975047
Recalculating deps hash.


[tests] More granular tests for faster builds

There are a few validation tests that take a very long time to execute. Say, there are 2 tests that take 1000 seconds each, and the rest are single seconds. It is possible that a single executor will get both of those long tests, while all the other fast ones land on other executors. So the test time would be 2000 seconds, and roughly half of this time will be spent in just one thread.
This change splits the long running tests into (even) smaller pieces, thus halving the maximum duration of a single test file execution.

@swift-ci Please test

For comparison: the most recent successful build for the PR took 1 hr 33 min on macOS-33.
Let's wait for this one.

/cc @shahmishal

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 0913206f29972e6d86a53aae03d1fdbd9eb39037
Test requested by - @moiseev

@swift-ci Please test

Took 1 hr 36 min on macOS-30
Hmm... Something's not right.

@swift-ci Please test

@swift-ci Please test

@swift-ci Please clean test macOS


"Version-free" method to get container id inside container

Inspired by comments of this answer: http://stackoverflow.com/a/25729598
Problem: get container ID inside of this container
It would be nice use the same method across different docker server versions.
My current solution is:
cat /proc/self/cgroup | grep docker | grep -o -E '[0-9a-f]{64}' | head -n 1

I created this issue to find more "versionless" method

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Hi @viatcheslavmogilevsky  Not quite understand your needs. Do you want feature added in docker ?
Or just ask what related shells magic or linux magic can be used to get container ID ?

@HackToday it's about shell magic to get container ID. If someone know solution better than proposed - I would be glad to see here.

There are several proposals for this type of feature. See https://github.com/docker/docker/issues?utf8=✓&q=is%3Aissue+is%3Aopen+introspection

@viatcheslavmogilevsky see the issues that @phemmer referred to for alternative proposals.
However, the issue tracker is not the best location for asking other uses to show there approach. Could you ask/discuss this question on one of the below;

the forums: https://forums.docker.com/
the docker-user mailing list https://groups.google.com/forum/#!forum/docker-user
the #docker IRC channel on Freenode.

I'll close this issue here, but feel free to comment here after I've closed


Typing Game

Project title
Typing Game
Project Description
To check your real time typing speed, i developed a project for you. Here you can test you tyoing speed by typing some random words that pop up on your screen
Stack:

[x] Html
[x] CSS
[x] JavaScript

@khushi-purwar Please asssign this to me

/assign


fix: subscription source should be observability-operator #156

closes #156

I left some comments on how i tested in the operator in this issue. I can add more to this PR if you all would like. The comment shows the whole flow of installation
https://github.com/rhobs/observability-operator/issues/156

Thank you for the patch. I used the UI to install :)

Thank you for the patch. I used the UI to install :)

Thank you for the merge! v0.0.10 is great!


is there any  ablation study of TDB reference feature size?

Thanks for your great work. did you do any ablation study of the TDB reference feature size, for example with 64x64, 32x32, 16x16, 8x8?

We don't have the ablation for feature size with 64x64. One reason is that doing the same attention operation on 64x64 features may not fit in the RAM.


AssertionError: Pisces supports tumor-only variant calling

Brad,
thanks for the tremendous work on benchmarking somatic callers. I wanted to try Pisces on a tumor only sample and got an error message (with development version 520fad7). Does Pisces support only paired tumor-normal samples?

Ben;
Thanks for the question and for trying out Pisces. In addition to the error you're getting, it does require manual installation of the dotnet runtime to use, as that's not currently packaged into the conda installs.
Pisces only supports tumor-only variant calling, so the code has a check for this and is reporting the error you're seeing. It should error out if your sample either has a normal associated with it, or if the tumor only input isn't labelled as phenotype: tumor.  Do either of those cases apply to your configuration? Hope this helps get it running for you.

Brad,
ok, dotnet is certainly not installed, hence the error. I will check how to install dotnet by hand.
Thanks for the hint!

Ben;
I added a link to the .NET install from the documentation on optional requirements: https://bcbio-nextgen.readthedocs.io/en/latest/contents/installation.html#automated
To be clear, this isn't the source of your issue, which has something to do with how your tumor-only sample is organized, but I just wanted to avoid you hitting another error immediately after fixing that.
Thanks for exploring Pisces and hope this helps

Brad,
you are right, I have a number of other issues with this sample that are not related to bcbio at all. Thanks for the link. It should be easy to bake this into the docker image.


Modify preprocessing script

Description
This PR modifies the preprocessing to:

Add CSA computation of soft and hard GT
Create disc labels, crop and add them to data_processed_clean
Check if there are NaN values in generated softseg
Thresholding softsegs between 0 and 1

An example of results can be found under duke/projects/contrast-agnostic-seg/contrast-agnostic-test-2022-08-09-all

I think this is ready to merge, since the naming convention of MTS will not be addressed here (a least yet)


Fix angular universal server side rendering issue #77

What kind of change does this PR introduce? (check one with "x")
[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)
document is not defined in angular universal server side
What is the new behavior?
no errors.
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

Hi, any updates on when this will be merged?

Any updates?

Has this PR been forgotten?
Any updates on if this bug will be fixed?

SSR support was added in PR #159.


[UI/UX] Add Icon to Profile and Overflow menu

Description
Add Icon to Profile and Overflow menu
Screenshot

TODO

[ ] - Update the user profiles user details card vie to be able to have a placeholder for an image
[ ] - Get the image from the assets folder and update the compose to support it (dimensions)
[ ] - Configure the rule to show and hide the icon based on the user details provided interpreted  by the rules engine

@HenryRae Kindly provide an SVG image for this

Can you share a sample screen with an image?

It's possible to use configurations to provide the images for the overflow menu items.

I'd need to confirm the position.

Icon config works but need to check wether their is property to move is to the right and not left
"icon": {
                "type": "local",
                "reference": "ic_household"
            },

This class can be used to create a config to add image on profile https://github.com/opensrp/fhircore/blob/main/android/quest/src/main/java/org/smartregister/fhircore/quest/ui/shared/components/Image.kt

is the issue that you can't specify which side of the text to add the image?


new DomSnapshot();   not defined

Uncaught ReferenceError: DomSnapshot is not defined
html include dom-snapshot

Hi @QGB thank you for report, will check it

fixed in https://github.com/lifeart/dom-snapshot/commit/20fa2b2fb0de5d65edb43a2064d61d9e1a5060a9


Force Click for definition

Check for existing issues

[X] Completed

Describe the feature
This is a bit of an "out there" suggestion but it just occurred to me.
~All modern MacBooks have a Force Touch trackpad with which you can press firmly for extra actions on most words, eg in TextEdit or Safari.
As well as Cmd+Click, it would be badass if you could press extra firmly on the trackpad to jump to the definition of a token.
If applicable, add mockups / screenshots to help present your vision of the feature
No response

It is possible we could bind force click to command+click similar to the solution we added for control+clicking (cc @ForLoveOfCats!)


Ros2 modue not found

im using the publisher example qml file with ros humble on ubuntu 22.04 and try to run the app through qmlscene as told but the Ros2 Qml Plugin is not installed, how can i install it?
i already have installed the qml Ros2 Plugin through apt

I found the Module in the lib folder but now how do i make qmlscene find it?


Can add multiple (duplicate or conflicting) conditions

https://github.com/adamchainz/gargoyle/blob/9d699ae94deccedd35b90b0ce93f48fcb503f9dc/gargoyle/models.py#L141
It seems like this line of code is supposed to prevent adding conditions that already exist, however because self.value[namespace][field_name] is a list of tuples with the second item as the condition, the condition not in will always be True.
Seems like there are two solutions, either just prevent duplicates:
condition_tuple = (exclude and EXCLUDE or INCLUDE, condition)
if condition_tuple not in self.value[namespace][field_name]:
    self.value[namespace][field_name].append(condition_tuple)

or prevent adding conflicting or duplicate conditions (seems like this would probably make more sense, however probably shouldn't just silently fail, should probably let the user know to remove the other condition first):
if condition not in (c for _, c in self.value[namespace][field_name]):

I might be able to find some time to work on a patch if nobody else wants to quickly throw something together, but would like some feedback first.

Thanks for your in-depth analysis. It looks right to me. It also seems you are half-way to a patch so it would be easiest for you to do it, if you can find the time.

Conflicting would be more useful, but perhaps preventing duplicates would be a good first step.

Fixed in #212


Transparent pixels

When using aarty on PNGs with a transparent background the program panics because it tried to divide by zero.
thread 'main' panicked at 'attempt to calculate the remainder with a divisor of zero'

at line
aarty-0.4.9/src/ascii_processor.rs:18:16

it only happens in the colored mode

when I clone it and run it there it doesn't happen so i guess it was fixed
sry for the unnecessary issue

when I clone it and run it there it doesn't happen so i guess it was fixed sry for the unnecessary issue

actually, the issue is still there but in some way, it doesn't happen all the time even with the same inputs, idk way but this it, but it works most of the time, and this's what's important :D. 'cause if I try to figure out the issue and fix it I'll probably wanna delete all spaghetti that I make in this project and rewrite it, and I don't have a time for this right now


Create HTML report even if FakeNet closed improperly

Currently, FakeNet will only generate its new HTML report/output if closed properly (using Ctrl+C and not forcing it to quit).
I think we can create a way to store the data required to make this HTML report in a local database that is not in memory so that if a user quits FakeNet forcefully (or if it closes improperly for other reasons), we would still be able to create it.
This would likely require a separate script to perform a conversion of the database to HTML report, but I think that code would mostly be a copy of what already exists inside FakeNet with its most recent update.

After discussion, a good path to take for this may be to add a flag (or possibly flags) for fakenet to be run in a way that is sort of like a 'recovery mode' where it can process the information needed to generate the html report in the event of either an error/crash occurs or in the event that a user closes FakeNet Improperly (such as pressing Ctrl+C too many times or closing via the X at the top)
This will require continuously writing pertinent information out to disk so that when this event occurs, we will have the necessary information required to generate the report.
A likely thing we can do is to use pickle to serialize to disk and deserialize information that we need back into memory so that we can keep most of the logic simple and clean (in regards to not creating additional logic for data formatting to write to disk and back into memory).

I'd like this functionality too! currently i'm using fakenet as a service through NSSM, that (since version 2.17) includes a control-C simulation to shutdown the service. Just play around with wait time a bit


vibe.core.concurrency: Update references to Isolated template

At least it seems like it refers to the same thing.
Not sure how to do the linking correctly here.

Thanks. The reference should work (just "Isolated" in backticks works, too, because it's in the same scope).


[FEATURE REQ]

What & why
Is your feature request related to a problem, or a new addition to the guide? Provide a clear and concise description of what you'd like added to the RaspiBolt guide. If possible, please phrase it as a user story:

As a [description of user], I want [functionality] so that [benefit].

How
Describe the solution you're thinking of. Provide a clear and concise description of what you want to happen. Did you consider alternative options?
Additional context
Add any other context or screenshots about the feature request here.

@Smithjuanita this is just the standard template, without any custom content. I'll close this, feel free to open a new (proper) feature request with a description of your own.


Update README.md - Pycharm instructions not working for files path containing white spaces

Currently, If there is a space in the file path of the python file in PyCharm then an error is generated
Error: Invalid value for "[SRC]...": Path .... does not exist.
It stops the file path at the space.
By enclosing the Arguments in quotes it fixes this.
Also a small issue in that the instructions for how to access settings were incorrect on the Mac

Pull Request Test Coverage Report for Build 853

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 91.614%

Totals

Change from base Build 851:
0.0%

Covered Lines:
2775

Relevant Lines:
3029

💛  - Coveralls

Pull Request Test Coverage Report for Build 853

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 91.614%

Totals

Change from base Build 851:
0.0%

Covered Lines:
2775

Relevant Lines:
3029

💛  - Coveralls

Thanks for your PR! As with #680 and #681, I'm not comfortable merging this without hearing from more people who actually use PyCharm. @Heaford and @gsganden, maybe you could review each other's PRs?


Segfaults due to protobuf -std=c++11 mismatch

I was seeing segfaults in std::unordered_map, and the symptoms looked like potential memory corruption.  Took a fair amount of digging in gdb to figure out that it wasn't that, but was just a build/library version issue:

tensorflow's embedded protobuf is built with -std=c++11
a default protobuf build is built without -std=c++11

I'm not super familar with the details of dlopen and symbol resolution, but the RTLD_GLOBAL when it loads _pywrap_tensorflow.so might be causing it to shadow the symbols in libprotobuf.so (for the cpp-enabled protobuf python package).  The C++11 vs non-C++11 libraries use different headers for std::unordered_map (in bits vs tr1), and things blow up when they start using the wrong library's version of the code.  (Correct me if my understanding of RTLD_GLOBAL is wrong and that's irrelevant).
In any case, building protobuf with C++11 fixed the segfaults I was seeing.
Now as for solutions, here are some ideas:

Allow tensorflow to link to an external libprotobuf.so.  This is probably the best solution, since then only one protobuf library is loaded at runtime, and a version check is probably simple to add.
If RTLD_GLOBAL is a relevant factor, maybe _pywrap_tensorflow.so can be split up and only the parts that actually need to be global can be global.  Or maybe some gcc attributes to control visibility can do the trick.
Maybe we can shove the embedded tensorflow into a namespace so the symbols don't collide

Possibly useful info to reproduce:

gcc-4.8.x
libstdc++.so.6.0.19
r0.8 branch

I've seen some other people post about similar segfault issues and I suspect many of them might be related to this.
Also now that I've found the issue, it seems obvious in retrospect, and a related issue on the protobuf issues is https://github.com/google/protobuf/issues/839.

I should add that I know nothing about bazel so I will need someone involved in the project to help.

Seems like #2034 had similar issues with symbol visibility related to RTLD_GLOBAL, maybe that can be revisited and tightened up a bit?  We really only want it to make certain things global, and we should be careful what's exposed.

Yeah I believe sizeof(std::unordered_map) is different between the c++0x (tr1 version of std library) and --std=c++11. If protobuf and tensorflow are not communicating unordered_maps amongst each other and protobuf functions are not inlined into tensorflow translation units it can be perfectly safe. However, if some of the inlined functions are not inlined and instead turned into exported symbols implicitly, the dso loader will attempt to share one symbol with both libraries which will not work (it will assume the wrong structure layout).  You could attempt to work around this by adding
-fvisibility-inlines-hidden to tensorflow which will force tensorflow to use its own copies of all inlines that are spilled to function calls. OTOH, if this is due to inlines using a std::unordered_map in a tensorflow translation unit that was allocated in a protobuf translation unit, this will not solve the problem.
-fvisibility-inlines-hidden

@aselle: To pull an offline comment into the thread, is there a way we could detect the size mismatch (possibly at runtime) and bail with an explanatory error?

Hiding inlines might solve one class of problems, but non-inlined functions could also be clobbered.  We really need to either make the two libraries hidden from each other (i.e. remove/restrict RTLD_GLOBAL-ness), or share the same libprotobuf.so.
By far the best solution would be to dynamically link to an existing protobuf library and sprinkle in one or two version checks to make sure the right one is loaded.  Is there any reason not to do this other than the work involved to put it in the bazel build configs?

@girving a size check would be a good first line of defense, and would certainly be a good addition until this is ironed out, but the struct could also have different members/layout with the same size so it should only be a temporary solution.

I think this may impact the other libraries that are embedded in tensorflow.  Suppose I do this:

Import tensorflow, it loads _pywrap_tensorflow.so with RTLD_GLOBAL, libpng symbols are loaded with global visibility
Import matplotlib.pyplot, it loads _png.so, which depends on libpng15.so, but when it's linked some of the symbols it needs are globally visible via _pywrap_tensorflow.so so it uses those instead
Potential problems?  On my box, I have libpng-1.5.x, but tensorflow embeds libpng-1.2.x.  I don't know the details of the implementation but there could very well be struct changes between those versions.

In an ideal world we'd link to existing libraries whenever possible.  If I had to guess, we are embedding these libraries so we don't need to support the entire universe of different versions people might have installed.  That's fine while TF is still young, but if we're going to do that we really can't have those embedded libraries loaded with global visibility or they will cause problems with libraries loaded the normal way.

@llchan With respect to libpng and other libraries embedded in _pywrap_tensorflow.so, look here. The linker scripts strictly control the symbols exported by _pywrap_tensorflow.so. Even though the library is loaded with RTLD_GLOBAL, only tensorflow related symbols are available in the global address space.

Ah my bad, I was building TF 0.8 so I had the r0.8 branch checked out.  Should have checked out master, looks like the exported symbols list was added after r0.8.  I'll try a r0.9 build.

Sorry for the late reply. It's difficult to do this in general, because you'd need to put something in a translation unit for every project that might have possibly been compiled differently that you can query later. I.e. something already has to be compiled that remembers the size or that it is c++11. I don't really know enough about the toolchain to know if the compiler dumps something we could reliably use to determine if a c++11 or c++0x standard library was used. We could roll our own, but it seems difficult to control. It is best to avoid std::unordered_map being part of a systems API to avoid most times you would hit this, at least until c++0x tr stuff is put to bed.

@keveman 0.9 is much, much better about symbol visibility, the libpng symbols I saw before in 0.8 are no longer global, and poking around in gdb, I now see symbols coming from the right place.  Thanks, for pointing me to the right place to look!
@aselle Yeah, youre right, it sounds like it'd be a pain to check c++11 vs c++0x for every dependency, and it's unclear if theres enough information as-is to determine that at runtime.  I think the stuff about restricting global symbols should address my concerns though.
For anyone else reading, try r0.9 released a day or two ago, the issues related to shared libraries may not exist anymore.

@llchan Thanks for your comments. I am closing this issue as there seems to be no outstanding action items. Feel free to open the issue if you feel that it has not been sufficiently addressed.


How does limiting the constraint generation work?

Description
I am not sure how the constraint creation works. Indeed if I limit the number of constraints, will the Supervised class remove examples from similar pairs, from negative pairs or will it arbitrarily cut the part of the data that comes after num_constraints?

The pairs generation process is as follows: we first sample one point x from the dataset X, then we sample another point of X from the same class as x (with the same y) (for sampling similar pairs) or from a different class (for negative pairs), and repeat until we have reached the number of constraints needed. Note that for now, for Supervised classes we sample n_constraints positive pairs, and n_constraints negative pairs.

Thanks for your reply. So, now the upper limit for the number of constraints is:
2 * min(n_positive_examples, n_negative_examples).  with the number of positive and negative examples being equal?

In fact I didn't say it but the _pairs function ensures that no duplicated pairs (pair with same order) is returned. It does this by not adding duplicated pairs to the result, doing at most max_iter passes through X to try to find n_constrained not duplicated pairs. But if even after max_iter passes, there are no n_constraints pairs to return, it will return them with a warning.
In this case, the "same length" argument allows to force positive_negative_pairs to return the same number of samples.
So to sum up if you see no warnings thrown, Constraints.positive_negative_pairs has returned n_constraints positive pairs and n_constraints negative pairs.
But if a warning is thrown, either you have set the flag same_length=True and the method will return min(positive_pairs_built, negative_pairs_built) positive pairs and min(positive_pairs_built, negative_pairs_built) negative pairs , or if same_length=False the method has returned positive_pairs_built positive pairs and negative_pairs_built negative pairs, with positive_pairs_built and negative_pairs_built potentially different

Thanks for your clarifications. I am using the MMC_Supervised class at this point and I do not believe there is a way to set the same_length argument, is there (it is False by default)?

Indeed, MMC_Supervised calls Constraints.positive_negative_pairs with its default argument: same_length=False, so there is no way to set it to True from the MMC_Supervised interface for now.
Does MMC_Supervised throw you a warning though ? Because if not, this means that the number of positive pairs and negative pairs built are the same and equal to n_constraints.

Yes, I am using the class with different datasets in a loop. I put the num_constraints to be the maximum I can handle given the amount of RAM I have. For larger datasets, this is not a problem but for some smaller datasets, it is throwing a warning.

I see, it makes sense indeed, since for small datasets the algorithm cannot create a too big number of constraints without duplicates... If you want to have the same number of constraints for negative and positive constraints, I guess for now maybe you could modify the default with something like this, overwriting the method:
import copy
new_func_bis = copy.copy(Constraints.positive_negative_pairs)
def new_func(self, num_constraints, random_state=np.random):
    return new_func_bis(self, num_constraints=num_constraints, same_length=True, random_state=random_state)
Constraints.positive_negative_pairs = new_func

But this is kind of hacky..., or you could fork/clone the repo and change the default value of Constraints.positive_negative_pairs to True
Let's let this issue open to remember that in the future it could be good to allow setting same_length=True when creating a metric learner.

Thanks for exposing the alternatives. I feel that, in the long run, forking the repo will be the most sensible thing to do.


NoSuchMethodError: The method 'fold' was called on null.

When running a simple unit test, I´m getting the error mentioned on the title with the following log:

NoSuchMethodError: The method 'fold' was called on null.
Receiver: null
Tried calling: fold(0, Closure: (int, dynamic) => int)
dart:core                                                                               Object.noSuchMethod
package:equatable/src/equatable_utils.dart 4:19                                         mapPropsToHashCode
package:equatable/src/equatable.dart 35:46                                              Equatable.hashCode
package:dartz/src/either.dart 94:36                                                     Right.hashCode
dart:collection                                                                         _CompactLinkedHashSet.contains
package:matcher/src/pretty_print.dart 28:14                                             prettyPrint._prettyPrint
package:matcher/src/pretty_print.dart 119:22                                            prettyPrint
package:matcher/src/description.dart 49:11                                              StringDescription.addDescriptionOf
package:test_api                                                                        expect
package:flutter_test/src/widget_tester.dart 234:3                                       expect
test\features\number_trivia\domain\use_cases\get_concrete_number_trivia_test.dart 33:7  main.

Entity classes:
class NumberTrivia extends Equatable {
  final String text;
  final int number;

  NumberTrivia({
    @required this.text,
    @required this.number,
  });

  @override
  // TODO: implement props
  List<Object> get props => null;
}

abstract class Failure extends Equatable {
  Failure([List properties = const<dynamic>[]]);
}

This is my use case class, the Either class corresponds to the dartz package:
class GetConcreteNumberTrivia {
  final NumberTriviaRespository repository;

  GetConcreteNumberTrivia(this.repository);

  Future<Either<Failure, NumberTrivia>> execute({@required int number}) async {
    return await repository.getConcreteNumberTrivia(number);
  }
}

My repository:
abstract class NumberTriviaRespository {
  Future<Either<Failure, NumberTrivia>> getConcreteNumberTrivia(int number);
  Future<Either<Failure, NumberTrivia>> getRandomNumberTrivia();
}

And my test, the Right class you see corresponds to the dartz package:
class MockNumberTriviaRepository extends Mock
    implements NumberTriviaRespository {}

void main() {
  GetConcreteNumberTrivia useCase;
  MockNumberTriviaRepository mockNumberTriviaRepository;

  setUp(() {
    mockNumberTriviaRepository = MockNumberTriviaRepository();
    useCase = GetConcreteNumberTrivia(mockNumberTriviaRepository);
  });

  final tNumber = 1;
  final tNumberTrivia = NumberTrivia(number: tNumber, text: 'test');
  test(
    'should get trivia from the repository',
    () async {
      //arrange
      when(mockNumberTriviaRepository.getConcreteNumberTrivia(any))
      .thenAnswer((_) async => Right(tNumberTrivia));
      //act
      final result = await useCase.execute(number: tNumber);
      //assert
      expect(result, Right(tNumber));
      verify(mockNumberTriviaRepository.getConcreteNumberTrivia(tNumber));
      verifyNoMoreInteractions(mockNumberTriviaRepository);
    },
  );
}

The error jumps in the expect(result, Right(tNumber)); line, and I have no idea what to do with this method fold, all I know is that it´s in the dartz package. I´ve searched through docs and nothing, I don´t know what I´m doing wrong.
Anyone know how to fix it? Much appreciated!

@IgnacioCentola were you able to fix this issue? I am facing the same problem

@IgnacioCentola were you able to fix this issue? I am facing the same problem

Yes, the issue lies in this line :
List<Object> get props => null;
When using Equatable, you must pass a list with the properties you define in your classs contructor with the propspropertie, as follows:List get props => [text, number];`


Include different sources by variable in Multidimensional Poverty Index

@Marigold I am experimenting with two things:

Call values to fill three different source definitions that share parameters (in source_common)
Call the sources structure for each indicator (like in cme_mpi_national or cme_mpi_urban)

Is it possible to make placeholders work for (1) and (2)?
For (1) I thought I was doing right, but I have YAML warnings (strangely not for hot (?)):

I tried to use placeholders for (2), but I couldn't. For now, I was trying to use anchors, as in here, but we are trying to avoid that.

Now I have fixed the string warnings, but not some date warnings:

Thank you @Marigold! The dates are working now. Is there a way to include different sources with placeholders? I tried anchors, which are not working properly, and we want to get rid of them.

Cool! Thank you very much, @Marigold! Yeah, now it's working as expected. This is a very specific use case, where the authors want to see the three papers related to their corresponding indicator, but they are, at the same time, the same data product.


Make certain security features customizable for better UX

Starting this issue to collect pain-points related to Ledger integration with wallets and act on them:

public key export call always triggers a prompt on the device. This is a complication for BIP44 wallets like Adalite where a sequential account discovery requires loading one public key at a time and bulk export prompts to eager load multiple keys result in unintuitive UI and complex caching/eager loading logic.

Some wallets may use withdrawals as replacements for unspent outputs in which case prompting the user to confirm them may result in an unintuitive UX for them (TBD though it is desired from an accounting perspective as staking rewards may have a different tax burden than the standard "capital gains")

Mostly resolved, also by introducing expert mode.


Conform dataset ref to Nat SciData format

@NickRaymondKurtansky Should this be merged?

Hi Brian,
No because Dan already merged one of these pull requests I put in at
https://challenge2020.isic-archive.com/ . It was for the Nature Scientific
Data reference update. I was hoping to make the change at
https://challenge.isic-archive.com/data next.
--
Nicholas Kurtansky
On Tue, Mar 2, 2021 at 1:37 PM Brian Helba<EMAIL_ADDRESS>wrote:

@NickRaymondKurtansky https://github.com/NickRaymondKurtansky Should
this be merged?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ImageMarkup/isic-challenge-2020/pull/14#issuecomment-789123744,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIBTACWHFH3PNDRLLMVGZCDTBUWABANCNFSM4SM4IFAQ
.

@NickRaymondKurtansky You should now have permission to merge this yourself. Feel free to do so whenever you're satisfied with the content.


Change address book to sort contacts by their class timing, any clash…

… in class timing is left to user discretion, schedule command is more functional and neat

Codecov Report

Merging #71 (c766f86) into master (9ecde55) will increase coverage by 0.57%.
The diff coverage is 93.33%.

@@             Coverage Diff              @@
##             master      #71      +/-   ##
============================================
+ Coverage     71.16%   71.73%   +0.57%
- Complexity      491      508      +17
============================================
  Files            84       84
  Lines          1602     1645      +43
  Branches        182      187       +5
============================================
+ Hits           1140     1180      +40
- Misses          414      416       +2
- Partials         48       49       +1

Impacted Files
Coverage Δ

.../java/seedu/address/logic/commands/AddCommand.java
100.00% <ø> (ø)

...n/java/seedu/address/model/person/ClassTiming.java
89.79% <91.89%> (+5.18%)
:arrow_up:

...a/seedu/address/model/person/UniquePersonList.java
93.75% <100.00%> (+1.06%)
:arrow_up:

Continue to review full report at Codecov.

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


Update tutorial FAQ with CHM13

Added info about CHM13/T2T reference to FAQ. Fixed order of parents in example_cohort.yaml. Removed redundant variant consequence filter in slivar.

Resolves #84 #85


Link not working

File: _samples/library/alpine/index.md
The following link from the above page does not seem to be working:
This image is well documented. Check out the documentation at Viewdocs.
http://gliderlabs.viewdocs.io/docker-alpine
Just gives:

<title>503 Backend unavailable, connection timeout</title>

Error 503 Backend unavailable, connection timeout
Backend unavailable, connection timeout
Guru Mediation:
Details: cache-bwi5121-BWI<PHONE_NUMBER><PHONE_NUMBER>

Varnish cache server

@spectrumcomputing - Hi - I've tried this link in both FF and Chrome and it's working for me. Can you clear your browser cache and try the link again? Thank you in advance!

Hi, thanks for coming back to me so quickly.  It must have been a temporary issue with the server as I'm all up again.  Thanks again.

@spectrumcomputing You are very welcome - glad everything is OK now! Please let us know if you experience any additional issues - we value your feedback!


implement infinite-scroll-disabled and infinite-scroll-listen-for-event

I would love to help implementing this but I'm opening the issue as a reminder.
Thanks!

hi @marcalj ,
These seems to be a logic oriented features that should be handled in services.

Makes sense, thanks.

Hmmm, not so quickly.
I was using infinite-scroll-listen-for-event to say "hey, if the DOM element is at the bottom, trigger the next page event". So, it's useful because this directive knows if it should trigger the event or not.

Also I was using infinite-scroll-disabled to gain performance and disable checks when no one is listening. I think it worth it.

still - it looks like it belongs in logics.
lets wait for others to vote for it.

I would be interested in infinite-scroll-disabled so I can stop the evaluation if I'm already loading the next set of data.

@zoehneto ,
i'm just taking a note for the implementation - as now, the window scroll event is implemented with rxjs functions. it sounds like it can be integrated to this flow, though another handler should be implemented if the @HostListener is invoked on the actual DOM element rather than the window.

@orizens How can I check if the infinite scroll event should be triggered outside the component? I think I can't access this information and I cannot implemented in my business code. Thanks!

hi @marcalj
I havn't tried it - however, you can access the directive instance with @ViewChildren or @ViewChild
and then, trigger on the scroller property the "clean()" function for removing the scroll. for re-attaching the behavior, use the "defineContainer()" function.
This should work for a "window" listener only.


fetchCargoVendor not working for rustdesk-flutter

Describe the bug
Wanted to switch the rustdesk-flutter build to the new cargo fetcher. Doesn't work.
Traceback (most recent call last):
  File "/nix/store/wadx2h6yard87zgvddshm3mx2nimagwc-fetch-cargo-vendor-util/bin/fetch-cargo-vendor-util", line 293, in <module>
    main()
  File "/nix/store/wadx2h6yard87zgvddshm3mx2nimagwc-fetch-cargo-vendor-util/bin/fetch-cargo-vendor-util", line 289, in main
    subcommand_func()
  File "/nix/store/wadx2h6yard87zgvddshm3mx2nimagwc-fetch-cargo-vendor-util/bin/fetch-cargo-vendor-util", line 281, in <lambda>
    "create-vendor": lambda: create_vendor(vendor_staging_dir=Path(sys.argv[2]), out_dir=Path(sys.argv[3]))
                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/wadx2h6yard87zgvddshm3mx2nimagwc-fetch-cargo-vendor-util/bin/fetch-cargo-vendor-util", line 238, in create_vendor
    copy_and_patch_git_crate_subtree(git_tree, pkg["name"], crate_out_dir)
  File "/nix/store/wadx2h6yard87zgvddshm3mx2nimagwc-fetch-cargo-vendor-util/bin/fetch-cargo-vendor-util", line 179, in copy_and_patch_git_crate_subtree
    shutil.copytree(crate_tree, crate_out_dir)
  File "/nix/store/px2nj16i5gc3d4mnw5l1nclfdxhry61p-python3-3.12.7/lib/python3.12/shutil.py", line 600, in copytree
    return _copytree(entries=entries, src=src, dst=dst, symlinks=symlinks,
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/px2nj16i5gc3d4mnw5l1nclfdxhry61p-python3-3.12.7/lib/python3.12/shutil.py", line 498, in _copytree
    os.makedirs(dst, exist_ok=dirs_exist_ok)
  File "<frozen os>", line 225, in makedirs
FileExistsError: [Errno 17] File exists: '/nix/store/0zn7mrd3dld2dmj2fxgpjb8skx45dx1w-cargo-deps-vendor/core-foundation-sys-0.8.6'

Steps To Reproduce
Steps to reproduce the behavior:

cargoDeps = rustPlatform.fetchCargoVendor {
  inherit src; # hash fits for 1.3.3 but the problem happens on every release
  hash = "sha256-K9E2x01Y2WU0DewwKWXtnE92Jw2+uJOBUopHLvNs8Ew=";
};

nix-build

Expected behavior
It works
Screenshots
Pretty much the text from above but rendered in my terminal
Additional context
Don't know how the clownish developers were able to misuse their cargo so much but they somehow created a lock file with duplicate entries:

https://github.com/rustdesk/rustdesk/blob/1.3.3/Cargo.lock#L1195
https://github.com/rustdesk/rustdesk/blob/1.3.3/Cargo.lock#L1201

They don't seem to be aware of that and cargo seems to be fine with it.
Metadata

Notify maintainers

@TomaSajt @Bot-wxt1221

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

What's funny is that not even cargo vendor (the official vendor implementation) can handle this:
error: failed to sync

Caused by:
  found duplicate version of package `core-foundation-sys v0.8.6` vendored from two sources:

  	source 1: registry `crates-io`
  	source 2: https://github.com/madsmtm/core-foundation-rs.git?rev=7d593d016175755e492a92ef89edca68ac3bd5cd#7d593d01


Request to include H5MD package in Deepmd Lammps library

Hello Deepmd Team,
I am running lammps simulation using the deepmd kit. However, during the final collection of my data in h5md format, it gave me an error stating "ERROR: Unrecognized dump style 'h5md' is part of the H5MD package which is not enabled in this LAMMPS binary. (src/output.cpp:771)"
Therefore, in order to use the h5md package, we need to add h5md package in the current deepmd lammps library. Could you please add the H5MD package in the deepmd LAMMPS library?
Thank you!

Hello Deepmd-Kit-Recipes/Lammps-Feedstock Team,
Thank you so much for addressing this issue.
Regards,
Sandip Thakur
On Thu, Jun 1, 2023 at 10:44 AM Jinzhe Zeng @.***>
wrote:

Closed #55
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_deepmd-2Dkit-2Drecipes_lammps-2Dfeedstock_issues_55&d=DwMCaQ&c=dWz0sRZOjEnYSN4E4J0dug&r=dtUZWSndR-7LqwRauoGykTngvus9vG7uX_4tdpSjsxs&m=ADy00P6mN6zdIs5z-k6dO_aie_mq5E9GVan4HFN-KJ6r50t67UiKTBJrYjtbBt1m&s=a-UBvIte_xUGwcBHXSIbwX_iJrtywK9p6SZwCW1tWKY&e=
as completed via #56
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_deepmd-2Dkit-2Drecipes_lammps-2Dfeedstock_pull_56&d=DwMCaQ&c=dWz0sRZOjEnYSN4E4J0dug&r=dtUZWSndR-7LqwRauoGykTngvus9vG7uX_4tdpSjsxs&m=ADy00P6mN6zdIs5z-k6dO_aie_mq5E9GVan4HFN-KJ6r50t67UiKTBJrYjtbBt1m&s=NWvDLOERVwWtW9_AaKVYI7e505Av6oLADHEG4VO2Kr4&e=
.
—
Reply to this email directly, view it on GitHub
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_deepmd-2Dkit-2Drecipes_lammps-2Dfeedstock_issues_55-23event-2D9404377364&d=DwMCaQ&c=dWz0sRZOjEnYSN4E4J0dug&r=dtUZWSndR-7LqwRauoGykTngvus9vG7uX_4tdpSjsxs&m=ADy00P6mN6zdIs5z-k6dO_aie_mq5E9GVan4HFN-KJ6r50t67UiKTBJrYjtbBt1m&s=pUZM-SZPvRdHcbtVC0XamDjNLX9ec2ZPWmN2iKG_8yI&e=,
or unsubscribe
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_A5QUR5YOKJA32WXT7GE2JQTXJCTFTANCNFSM6AAAAAAYUYH4R4&d=DwMCaQ&c=dWz0sRZOjEnYSN4E4J0dug&r=dtUZWSndR-7LqwRauoGykTngvus9vG7uX_4tdpSjsxs&m=ADy00P6mN6zdIs5z-k6dO_aie_mq5E9GVan4HFN-KJ6r50t67UiKTBJrYjtbBt1m&s=lhxLkP7hpfPx4iE7Zg1KaaATDTsH22ozsNSCXEh2w9M&e=
.
You are receiving this because you authored the thread.Message ID:
<deepmd-kit-recipes/lammps-feedstock/issue/55/issue_event/9404377364@
github.com>


Web Server: 404 File Not Found.

When connecting to the server, it's not seeing any of the web files. This is just a reminder issue as we have already talked about this @DeathCradle.

Does this issue occur with the latest files still?
I'd like to resolve this. Also make sure you're using my permissions.xml file from the Binaries\Data directory or at least the web section from it. In addition remove Data\web.logins from the tdsm directory so the test web logins are restored (in the case I've changed them)

This has now been replaced with OWIN, http://:8448/web should do the trick now. However instead of being directly in TDSM it will be in a separate OTA plugin located here: https://github.com/DeathCradle/Terraria-s-Dedicated-Server-Mod/tree/master/WebInterface
I'll close this issue off as it's separate, and not yet fully under way. When the time comes (and if there are issues) we can simple make a new one for the RAWS.


⚠ Beulah Revival Network has degraded performance

In 6bd628b, Beulah Revival Network (https://beulah.clicki.digital) experienced degraded performance:

HTTP code: 200
Response time: 5122 ms

Resolved: Beulah Revival Network performance has improved in 7b03107.


GH-371 Fix zoom factor connection knot rendering

Fixes #371

Applied to main (=2.1) and branch 2.0.


Replace #ifdef and such for _WIN32 with

#include <osvr/Util/PlatformConfig.h>
#if defined(OSVR_WINDOWS)

Actually, let's rather not - ifdef _WIN32 is standard and universal, while the other is a platform-dependent generated header.

So... then why do we have the other one?  I assumed that we should be using OSVR-defined things within OSVR.  Glad to switch things back to _WIN32.

Question: Should we modify the standard OSVR_{platform} macros to operate at compile-time instead of configure-time?

I don't remember why you added that header, Kevin. Perhaps git can tell us.
If we keep it a shipped, public header, it's preferable to be compile time rather than configure time, yes.

Put them back to _WIN32 in dcc4f222b6413379b18d8f6361d92792aff45e1d


TypeError: NavCalendar is not a constructor

Hello, all options give errors. How to solve the problem?

const NavCalendar = require('telegram-inline-calendar')
ReferenceError: require is not defined in ES module scope, you can use import instead

import { NavCalendar } from 'telegram-inline-calendar'
SyntaxError: Named export 'NavCalendar' not found. The requested module 'telegram-inline-calendar' is a CommonJS module, which may not support all module.exports as named exports. CommonJS modules can always be imported via the default export, for example using:
import pkg from 'telegram-inline-calendar';
const { NavCalendar } = pkg;

import pkg from 'telegram-inline-calendar'
const { NavCalendar } = pkg
console.log(pkg)
const calendar = new NavCalendar(bot, {
date_format: 'MMM D, YYYY h:mm A',
language: 'de',
start_week_day: 1,
bot_api: "telebot",
time_selector_mod: true,
time_range: "08:00-15:59",
time_step: "15m"
});
[class Calendar]
TypeError: NavCalendar is not a constructor

import * as pkg from 'telegram-inline-calendar'
const { NavCalendar } = pkg
console.log(pkg)
[Module: null prototype] { default: [class Calendar] }
TypeError: NavCalendar is not a constructor

Hello,
The problem is related to incorrect interaction with TypeScript.
I'll fix it in a few hours.

@kpripper, hello,
At the moment, the package does not support ES module, I will release an update with support in the near future.
Now, to use the package, you need to remove the line type: module from package.json.
I apologize for the inconvenience.

@kpripper, can you try using the following version: npm i telegram-inline-calendar@ecmascript?

@kpripper, If you managed to make everything work, please consider closing this issue.


[JK] Portable/Relocatable Python Linking

What's the Issue?
Python binary is compiled with rpath that's /opt/hostedtoolcache/Python. Now, latest github runners  define RUNNER_TOOL_CACHE/AGENT_TOOLSDIRECTORY   differently than  /opt/hostedtoolcache and that installs python at /home/runner/_work/_tool/Python/.  So, with this, there are 2 issues.

sudo python --version doesn't work  ( See Error section) because most systems's doesn't allow passing LD_LIBRARY_PATH due to security issues.
python --version doesn't work without setting environment variable LD_LIBRARY_PATH

output of ldd :
runner@arss-runner-xxxx-runner-5jvhz:~/_work/_tool/Python/3.12.3/x64/bin$ ldd python3
	linux-vdso.so.1 (0x00007ffceb776000)
	libpython3.12.so.1.0 => not found
	libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f7ccc75c000)
	/lib64/ld-linux-x86-64.so.2 (0x00007f7ccc991000)

Error:
python: error while loading shared libraries: libpython3.9.so.1.0:
cannot open shared object file: No such file or directory

Solution:
$ORIGIN is used to as reference to the binary path.  So, we can use rpath that references relative path instead of absolute compile time path. With this relative path, Python binaries become relocatable.
Change:

--rpath changed to -rpath as it's the standard documented one.
Use $ORIGIN instead of absolute path

How this is tested?

I've also written a dockerfile to test this solution. Dockerfile moves the python installation from one location to another, and verifies it.
New Unit test / PR checks added.

Why bother about this issue?
LD_LIBRARY_PATH can't be used with sudo python due to security concerns. But, more details are here https://github.com/actions/setup-python/issues/871

Any update on this PR?

Just adding a +1 here. This is a real issue for many users, @jaswanthikolla has put in the work to resolve it for many use cases, and it's been going stale for 5 months.

Also echoing the need for this fix to be merged.


FlowRouter.current().route.group.name

I am trying to get group name but it always return 'undefined'.
Codes are as below
onboardingLayout.js
Template.onboardingLayout.helpers({
	title: function () {
		var groupName = FlowRouter.current().route.group.name;
		console.log(groupName);
	}
})

Route.js
var contributorSection = FlowRouter.group({
    prefix: "/contributor"
});

// Contributor onboarding routes
var contributorOnboarding = contributorSection.group({
  prefix: "/onboarding",
  triggersEnter: [AccountsTemplates.ensureSignedIn]
});

contributorOnboarding.route("/profile", {
  name : "contributorOnboardProfile",
  action: function(params) {
    BlazeLayout.render("onboardLayout", {navBar:"headerUser", navOnBoard:"contributorOnboardNav", main: "contributorProfile"});
  }
});

group name is always undefined. Seems group nam is not working.

hi @ashishapy i tried this one. But could not reproduce the issue. may be you are using a older version of flow-router which doesnt include this  (https://github.com/kadirahq/flow-router/pull/290) fix

Thanks @pahans for your response.
I am using latest FlowRouter also I don't need my help to be reactive.
I just updated my code to use FlowRouter.current().route.group.name;  still it is not working :(

@ashishapy can you build a sample repo with your code? I will look into it. thanks

I am able to solve by changes code as below.
var contributorSection = FlowRouter.group({
    name: "contributor",
    prefix: "/contributor"
});

&
var groupName = FlowRouter.current().route.group.parent.name;

so basically I added "name" property in group
& instead of getting "group.name", I changed it to "group.parent.name"


Markets page should update list of orders when order is created

Hi @yivo. I was checking that issue, after creating order, markets page make updating the list of orders. It's ok for me. I don't see anything problem here. You can check my screencast: https://drive.google.com/open?id=1a4ddoOwZee0jpP_KH0z_1OLOhLZxePnd

Hello @amir-budaychiev
Thank you for taking a part in it!
Can you find a way to refresh list of orders without manually reloading page?

ping @mkuvshynnikov

@yivo Sure. I can do that via realtime feature(gem pusher). What you think about it?

@amir-budaychiev Can you please check if this feature is already implemented? I think it is but it is broken or disabled.

@yivo all clear!

Hello @yivo. I did double check. I think it works fine. Let's check again my screencast please: https://drive.google.com/open?id=133SUoMjxsKlE4YRC1LEif3VFmVPJ5FMC

@yivo UPD I've fixed it, also it works fine from UI: https://drive.google.com/open?id=1m0wIqhbLMOWQC4XKDETn2gHQkfluX_Ts

@mkuvshynnikov Can you check what @amir-budaychiev prepared?

Closed. See https://github.com/rubykube/peatio/pull/455#issuecomment-363463859 and https://github.com/rubykube/peatio/pull/455#issuecomment-363740795.


[74] Custom types as Dictionary keys serialise as the string "[object…

Fixes #74

The issue mentioned above affects any dictionary (not limited to custom ones), so it was logged under #76.


Work around emscripten bug

Thanks @nikic and @bvinc for tracking this down.
cc emscripten-core/emscripten-fastcomp#258

I did more tests and decided that the only line that's needed is the first one, above capacity_to_buckets.  The others aren't necessary.

bors r+


Server under heavy load?

The server is taking from multiple seconds up to a minute to send pushes, what is the problem? Is there anything we can do to help?

Ah, it's not load. Calls just seem to time out under IPv6. Is nginx not listening on the IPv6 interfaces?

Actually, the server doesn't even respond to ping over IPv6. Is the hostname correct?

I had ipv6 disabled for quite a while due to issues with malicious hosts. Currently updating my DNS records and adding my ipv6 range to my server.

Amazing, thank you! I'll try again soon.

Is it working? My home connection does not support IPv6 at the moment.

Yes it does, thanks!