text:
Record url is not set + Endpoint is not updated

Environment
Python version: 3.6.6
NetBox version: 2.5.10
PynetBox version: 4.0.6
Steps to reproduce :
Python 3.6.6 (default, Aug 13 2018, 18:24:23)
[GCC 4.8.5 20150623 (Red Hat 4.8.5-28)] on linux
Type "help", "copyright", "credits" or "license" for more information.

import pynetbox
api = pynetbox.api(
...     'http://YYYYYYYY',
...     token='XXXXXXXXX',
... )
db=api.dcim.device_bays.get(1)
db.url
db.endpoint.url
'http://YYYYYYYY/api/dcim/device-bays'
device=db.installed_device
device.url
'http://YYYYYYYY/api/dcim/devices/2/'
device.endpoint.url
'http://YYYYYYYY/api/dcim/device-bays'
api.dcim.device_bays.get(2).delete()
True
device.delete()
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python3.6/site-packages/pynetbox/core/response.py", line 423, in delete
return True if req.delete() else False
File "/usr/local/lib/python3.6/site-packages/pynetbox/core/query.py", line 318, in delete
raise RequestError(req)
pynetbox.core.query.RequestError: The requested url: http://YYYYYYYY/api/dcim/device-bays/2/ could not be found.

Expected Behavior

db.url returns 'http://YYYYYYYY/api/dcim/device-bays/1'
device.endpoint.url returns 'http://YYYYYYYY/api/dcim/devices/'
device.endpoint.delete() do a DELETE on http://YYYYYYYY/api/dcim/devices/2/

Observed Behavior

db.url is None
device.endpoint.url 'http://YYYYYYYY/api/dcim/device-bays' (i.e. db.endpoint.url)
seems device.delete() tries to do a DELETE using device.endpoint.url that is wrong

It seems the bug is there in response.py def _parse_values(self, values) -> def list_parser(list_item) . self.endpoint is used there. In my opinion, a new endpoint should be used:
v = lookup(v, self.api, self.endpoint)
v = self.default_ret(v, self.api, self.endpoint)

Please kindly review the pull request

Hi @louis-oui, thanks for the bug report and PR. I'm not sure I want to go down the path of making nested Records equivalent to Records returned from an Endpoint. Nested Records were really only intended for read operations. The docs, and code, should be more explicit about that.
It's just my opinion, but I feel it greatly increases the potential for less than intuitive client code and unexpected behavior. That all being said, I'm not completely opposed to it. Will this issue open for a bit to see it gets traction.
Btw, somehow PR #148 got mixed up in your PR.

Thank you for your review.
I mistakely put PR #148. Sorry for that.
In current code, Record delete method use the endpoint url to delete the record (this is what I have observed). If Record endpoint url is from a nested Record, it will try to delete another Record.
In addition to that, it could be useful to use endpoint url to know which kind of record it is

Hello @zachmoody
Any news on the issue ?
According to me, it a bug. I think it will be useful to correct the delete issue that mistakenly try to delete nested objetct with ID with the endpoint url.

Hey @louis-oui - I was just leaving it open to see if anyone else had an opinion. I agree that it's a bug, I'm just not completely sure the best way to handle it.

OK let me know how

I clicked on the wrong button.
Let me know how you want to handle the bug. Thank you

@zachmoody
kindly handle the bug. Your help is appreciated

I'm running into the same issue as well, while trying to delete Cable objects inside Interface objects. I'm working around the issue as follows:
def delete_interface_cable(cable):
    cable.endpoint.url = cable.endpoint.url.replace('interfaces', 'cables')
    cable.delete()

interface = nb.dcim.interfaces.get(device='host1.example.com', name='eth0')
delete_interface_cable(interface.cable)

I think it's useful to be able to delete cable objects that are discovered within interface objects (i.e. as opposed to having to separately search for them).

@zachmoody Because of this bug, some devices on our netbox were wrongly deleted.
What is the blocker to not merge the fix ?

What is the blocker to not merge the fix ?

Mainly keeping the API's behavior intuitive. While device > device-bay and interface > cable are great examples of where this behavior works well. The PR would allow ones that would be unintuitive, like deleting a site from an IPAddress.
Ideally, we'd allow it in the places it's logical but otherwise disallow it. Just haven't had the time to address it properly.

Thank your for your reply.
Pynetbox is an awesome project that saved me a lot of time but current behaviour does not fit my needs even for read-only operation.
In that situation, I cannot know which kind of cable termination termination_a is refering:
cable = nb.dcim.cables.get(7605)
cable.termination_a.endpoint.url
'http://XXX/api/dcim/cables'

Results is always cables url. But it could be interface, front_ports... I need this info to manage patch panel with netbox. I have not found any workaround for this.

Mainly keeping the API's behavior intuitive. While device > device-bay and interface > cable are great examples of where this behavior works well.

I would to like to contribute to the pynetbox project and to find a common solution for this. I can take more times to suggest you a new pull request on a behaviour that we both agree. However, am not sure to understand what is your concern about this. Is it the following behaviour that you want to keep ?

I'm not sure I want to go down the path of making nested Records equivalent to Records returned from an Endpoint.

What is the use case for keeping nested Records equivalent to Records returned from an Endpoint ?
May be we can create a new different endpoint attribute to store the url on which delete function will rely on to do the delete API request. Also it will be used to know which type of record a nested record is.

The PR would allow ones that would be unintuitive, like deleting a site from an IPAddress.
In that case, site deletion would imply IPAddress->Device->Site.

It seems that deletion operation is explicit in the code. It is unlikely that someone deletes an object like a site by mistake.

In that situation, I cannot know which kind of cable termination termination_a is refering:
cable = nb.dcim.cables.get(7605)
cable.termination_a.endpoint.url
'http://XXX/api/dcim/cables'

Results is always cables url. But it could be interface, front_ports... I need this info to manage patch panel with netbox. I have not found any workaround for this.

fwiw, there should be a termination_a_type attr that should tell you this on cables.

I would to like to contribute to the pynetbox project and to find a common solution for this. I can take more times to suggest you a new pull request on a behaviour that we both agree. However, am not sure to understand what is your concern about this. Is it the following behaviour that you want to keep ?

Yeah, I agree it makes sense that at least in those two examples you should be able to perform write operations on nested objects. There may be more, but they can be addressed as they come up as. I'm interested in coming up with a uniform way to handle them.

I'm not sure I want to go down the path of making nested Records equivalent to Records returned from an Endpoint.

What is the use case for keeping nested Records equivalent to Records returned from an Endpoint ?
May be we can create a new different endpoint attribute to store the url on which delete function will rely on to do the delete API request. Also it will be used to know which type of record a nested record is.

I'm beginning to feel they shouldn't be equivalent. Like, nested objects might need to be some kind of read-only record objects that have write ops raise NotImpelemented. But that's the easy part, building in the desired behavior where we want it, and sorta how that plays in with custom models, is where I have to stop thinking about it. :grimacing:

It seems that deletion operation is explicit in the code. It is unlikely that someone deletes an object like a site by mistake.

I don't disagree, but having a calamity like that just a miss-assigned variable away is something I'm leery of.

fwiw, there should be a termination_a_type attr that should tell you this on cables.

You are right. I took a wrong example. I will let you know what was my real issue.
I seems it is unecessary to implement safeguard to avoid undesired write operation because netbox application does not allow deleting objects that are used by other objects.
Let's take the example of deletion of site from IPaddresses:
>>> nb.ipam.ip_addresses.get(614)
10.10.20.1/30
>>> ip=nb.ipam.ip_addresses.get(614)
>>> intf=ip.interface
>>> device=intf.device
>>> site=device.site
>>> site.delete()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.6/site-packages/pynetbox-4.0.7.dev34+g4c1a140-py3.6.egg/pynetbox/core/response.py", line 461, in delete
    return True if req.delete() else False
  File "/usr/local/lib/python3.6/site-packages/pynetbox-4.0.7.dev34+g4c1a140-py3.6.egg/pynetbox/core/query.py", line 342, in delete
    raise RequestError(req)
pynetbox.core.query.RequestError: The request failed with code 409 Conflict: {'detail': 'Unable to delete object. The following dependent objects were found: Side A (circuits.circuittermination), Side A (circuits.circuittermination), Side Z (circuits.circuittermination), Side Z (circuits.circuittermination), Side A (circuits.circuittermination)'}

Same in webui

Yeah, that's a good point. Had forgotten about that behavior.

Was it the only reason you wanted read-only nested records ? Shall we keep them in read-write mode ?

Yeah, it was mostly that. I think the API's cleaner when you can't do write ops on nested Records, but I admit that's not a good reason to prevent it. I'll have another look at the PR when I get a chance this week.

Thank you for that. I updated the PR yesterday to resolve conflicts.

authors: kartiksubbarao louis-oui luigi1809 zachmoody

repo: digitalocean/pynetbox


text:
Can it block erotic text?

I am especially interested in furry, gay porn, nipples, feets, chest

No it does not

source: gharchive/issue

authors: dikshantrajput ivanstepanovftw

repo: dikshantrajput/noFap

license: MIT

license_type: permissive

license_source: github-api


text:
Отступ под кнопкой зарегистрироваться в материалах

Во всех местах, где выключены комменты кнопка “зарегистрироваться” уползла вверх.

Таких материалов быть не должно. Включили комменты, проблема решилась.

repo: dimabuko/lililoveme

license: mit

license_source: bigquery


text:
auto upgrade pip

I've had reports of people getting tripped up on having an old version of pip on their control env, and I've had this happen to me as well.
I think this is all we need to permanently fix it, unless anyone wants to advocate for the more conservative approach of pinning pip to a specific version, which I at this point think is probably overkill.

👍

source: gharchive/pull-request

authors: dannyroberts snopoke

repo: dimagi/commcarehq-ansible


text:
Fix shared dirs in proxy config files

@benrudolph @dannyroberts @emord

:+1: , does --check on prod return empty now?

I did not try that, but I will...

I tested on staging and zambia, which each have different values for shared_drive_enabled

Close to empty. I think fine as far as this PR is concerned. Messaging you offline with changes that I observed.

authors: benrudolph millerdev


text:
Xform engine recognize intents

Fixing 156946, where the form_validator tool was warning about "Unrecognized question type" on extensions to the xform specification, such as "intents"
This also includes a bunch of clean up and code style changes.
Examples:
The following exhibits the warnings we want to suppress (where the inputted xml is the External Callout form from corpora on HQ):
$ java -jar form_translate.jar validate external_callout_intent.xml
To suppress the warnings we extend the spec, via command-line flags, with place-holder information:
$ java -jar form_translate.jar -E http://opendatakit.org/xforms=intent -E http://www.w3.org/2002/xforms=extra -S http://opendatakit.org/xforms -I http://opendatakit.org/xforms validate ~/dimagi/xforms/external_callout_intent.xml
where:
-E says we are expecting "intent" tags at the namespace "http://opendatakit.org/xforms"
-S says suppress warnings that come from parsing expected elements in the "http://opendatakit.org/xforms" namespace
-I says continue parsing inner elements of expected tags from the "http://opendatakit.org/xforms" namespace

Cleaned up the PR and reposted here:
https://github.com/dimagi/javarosa/pull/68

repo: dimagi/javarosa


text:
Use GlobalTransform

Use GlobalTransform instead of Transform to place colliders correctly and added a test for it. I also had to add GlobalTransform to all examples.

Thank you for this PR! The transform writeback formulae is slightly incorrect (the translation part should take the parent’s rotation into account). In addition, I see that instead of querying the parent’s global transform, you recover it from the child global transform and transform. While this is a good idea to avoid a query, it will break the transform change detection we do in apply_rigid_body_user_changes.
I have a fix ready for these, it it appears that I don’t have the access right to push to your branch. Could you please enable edits from maintainers?

@sebcrozet for unknown reason I don't see such button:

Maybe you can just create another PR based on my?

@Shatur That’s weird… Ok, I created a new PR: #178 Closing this one.

authors: Shatur sebcrozet


text:
snap changed collider scales to avoid circular updates

Previously, scales were being always-applied to rolling ball colliders that do not specify a ColliderScale when the snapped value was in effect. This occurs because as the ball rolls, numerical constraints were not being checked in apply_scale, leading to always updating the collider scale, but snapping the value only on update.  On the next sync, since the Bevy change detector has gone off for the collider shape, it would then set the scale a second time.  This process would repeat as long as the ball was rolling.
A workaround to this issue is to apply a ColliderScale::Absolute to the ball so the scale never changes.
This change resolves the issue by only changing the collider when the effective change is different from the would-be snapped value along with the scaled collider.

Thanks!

authors: cscorley sebcrozet


text:
Stable?

Hi,
I'm about to use this package in a big project, as it's referred by jwt-auth that we wanted to use anyway, this package seemed very useful for us. This package is being developed since 2014, when i look to the past, there are some peaks in development, at this moment it's quit, does that mean everything runs smooth? If so, why isn't there a stable version yet? in the installation docs there's still the message: "At this time the package is still in a developmental stage and as such does not have a stable release. You may need to set your minimum-stability to dev.".
Why?
Bart

Also really interested is the answer for the same question. Hope @jasonlewis or @hskrasek will pay attention on this issue to answer it.

#1383

Not stable yet. More details here #1436

authors: BartHuis Mahmoudz from2day vinterskogen

repo: dingo/api


text:
fix: add WRAPPED_NATIVE_CURRENCY to Uniswap V3 Base and Avalanche

[x] add WRAPPED_NATIVE_CURRENCY
[x] yarn changeset

LGTM

authors: Jeff-CCH zodahu

repo: dinngo/protocolink-logics


text:
An additional step in dipdup new to choose addons

Checkboxes, resulting array stored in replay.yaml. Example of optional layers: Swarm deploy, Squid Cloud deploy. Changes in config shouldn't be breaking. If not set, use what's currently in base.

I can assign the problem, and I'll be happy to assist you with it :) @droserasprout

@Josue19-08
Hi! Thanks for your interest. OD Hack 8 will begin September 26th. After that you will need to apply via the OnlyDust platform. Please wait for announcements in the next few days.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hi
How I plan on tackling this issue
Hi

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I'm proficient in multiple programming languages, including Cairo, JavaScript, Python.
How I plan on tackling this issue
Step 1: Define the checkbox options I'll define the checkbox options for the optional layers, such as Swarm deploy and Squid Cloud deploy. These options will be used to generate the array of selected layers.
Step 2: Create a checkbox component I'll create a checkbox component that will render the defined options. This component will allow users to select one or more optional layers.
Step 3: Store the selected layers in an array I'll store the selected layers in an array, which will be updated dynamically as users check or uncheck the checkboxes.
Step 4: Write the array to replay.yaml I'll write the array of selected layers to replay.yaml when the user saves their changes. This will ensure that the selected layers are persisted across sessions.
Step 5: Implement default behavior I'll implement default behavior to use the current values in the base configuration if no optional layers are selected. This will ensure that changes to the config are not breaking.
Step 6: Test the implementation I'll test the implementation to ensure that the checkboxes work correctly, the array is stored and retrieved correctly from replay.yaml, and the default behavior is implemented correctly.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am a python dev. A new-comer here, willing and ready to contribute to solve this issue.

@droserasprout I am almost done. I would create a PR soon

authors: Josue19-08 droserasprout ooochoche petersdt tosoham

repo: dipdup-io/dipdup


text:
fix typo

fix typo

;-)

authors: dirkolbrich rubensdemelo


text:
Unreal Engine 4.15 Packaged Game Problem

Updated my code from this repo today, all compiles. But at package for x64 on Windows 7 and execute the game I get this error at start. On VS2015. The dll is on the folder at package and on the folder of the project too aswell the headers etc with it.
Unknown exception - code c06d007e (first/second chance not available)
KERNELBASE
_Win64_Shipping!__delayLoadHelper2() [f:\dd\vctools\delayimp\delayhlp.cpp:323]
_Win64_Shipping!_tailMerge_discord_rpc_dll()
_Win64_Shipping!UDiscordRpc::Initialize() [d:\users...\unreal projects...\plugins\discordrpc\source\discordrpc\private\discordrpcblueprint.cpp:92]
_Win64_Shipping!UDiscordRpc::execInitialize()
_Win64_Shipping!UFunction::Invoke() [d:\unrealengine\4_15_wind\engine\source\runtime\coreuobject\private\uobject\class.cpp:4526]
_Win64_Shipping!UObject::CallFunction() [d:\unrealengine\4_15_wind\engine\source\runtime\coreuobject\private\uobject\scriptcore.cpp:723]
_Win64_Shipping!UObject::ProcessContextOpcode() [d:\unrealengine\4_15_wind\engine\source\runtime\coreuobject\private\uobject\scriptcore.cpp:2149]
_Win64_Shipping!UObject::ProcessInternal() [d:\unrealengine\4_15_wind\engine\source\runtime\coreuobject\private\uobject\scriptcore.cpp:948]
_Win64_Shipping!UObject::CallFunction() [d:\unrealengine\4_15_wind\engine\source\runtime\coreuobject\private\uobject\scriptcore.cpp:845]
_Win64_Shipping!UObject::ProcessInternal() [d:\unrealengine\4_15_wind\engine\source\runtime\coreuobject\private\uobject\scriptcore.cpp:948]
_Win64_Shipping!UObject::CallFunction() [d:\unrealengine\4_15_wind\engine\source\runtime\coreuobject\private\uobject\scriptcore.cpp:845]
_Win64_Shipping!UObject::ProcessInternal() [d:\unrealengine\4_15_wind\engine\source\runtime\coreuobject\private\uobject\scriptcore.cpp:948]
_Win64_Shipping!UObject::CallFunction() [d:\unrealengine\4_15_wind\engine\source\runtime\coreuobject\private\uobject\scriptcore.cpp:845]
_Win64_Shipping!UObject::ProcessInternal() [d:\unrealengine\4_15_wind\engine\source\runtime\coreuobject\private\uobject\scriptcore.cpp:948]
_Win64_Shipping!UObject::CallFunction() [d:\unrealengine\4_15_wind\engine\source\runtime\coreuobject\private\uobject\scriptcore.cpp:845]
_Win64_Shipping!UObject::ProcessInternal() [d:\unrealengine\4_15_wind\engine\source\runtime\coreuobject\private\uobject\scriptcore.cpp:948]
_Win64_Shipping!UObject::CallFunction() [d:\unrealengine\4_15_wind\engine\source\runtime\coreuobject\private\uobject\scriptcore.cpp:845]
_Win64_Shipping!UObject::ProcessInternal() [d:\unrealengine\4_15_wind\engine\source\runtime\coreuobject\private\uobject\scriptcore.cpp:948]
_Win64_Shipping!UFunction::Invoke() [d:\unrealengine\4_15_wind\engine\source\runtime\coreuobject\private\uobject\class.cpp:4526]
_Win64_Shipping!UObject::ProcessEvent() [d:\unrealengine\4_15_wind\engine\source\runtime\coreuobject\private\uobject\scriptcore.cpp:1318]
_Win64_Shipping!UGameInstance::ReceiveInit()
_Win64_Shipping!UGameInstance::Init() [d:\unrealengine\4_15_wind\engine\source\runtime\engine\private\gameinstance.cpp:76]
_Win64_Shipping!UGameEngine::Init() [d:\unrealengine\4_15_wind\engine\source\runtime\engine\private\gameengine.cpp:576]
_Win64_Shipping!FEngineLoop::Init() [d:\unrealengine\4_15_wind\engine\source\runtime\launch\private\launchengineloop.cpp:2517]
_Win64_Shipping!GuardedMain() [d:\unrealengine\4_15_wind\engine\source\runtime\launch\private\launch.cpp:155]
_Win64_Shipping!GuardedMainWrapper() [d:\unrealengine\4_15_wind\engine\source\runtime\launch\private\windows\launchwindows.cpp:134]
_Win64_Shipping!WinMain() [d:\unrealengine\4_15_wind\engine\source\runtime\launch\private\windows\launchwindows.cpp:210]
_Win64_Shipping!__scrt_common_main_seh() [f:\dd\vctools\crt\vcstartup\src\startup\exe_common.inl:253]
kernel32
ntdll

Fixed! Well taking a look here https://github.com/discordapp/discord-rpc/issues/18 seems like is a problem in the project file of the plugin at "Edit the discordrpc.uplugin file in a text editor and change the type from "Developer" to "Runtime" under modules."

Fixed! Well taking a look here https://github.com/discordapp/discord-rpc/issues/18 seems like is a problem in the project file of the plugin at "Edit the discordrpc.uplugin file in a text editor and change the type from "Developer" to "Runtime" under modules."

repo: discordapp/discord-rpc


text:
Fix messageUpdate event

Please describe the changes this PR makes and why it should be merged:
Logic failure incurred receiving events. Strong case not to compare content, as modifying embes and removing images still classify as a messageUpdate and not a messageDelete
Status

[x] Code changes have been tested against the Discord API, or there are no code changes

Semantic versioning classification:

[x] This PR changes the library's interface (methods or parameters added)

[] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)

[ ] This PR only includes non-code changes, like changes to documentation, README, etc.

There are too many unnecessary commits.

There are too many unnecessary commits.

I don't think some of the changes made are related to this PR.

I don't think some of the changes made are related to this PR.

authors: OSCAR-WOS ayntee itohatweb

repo: discordeno/discordeno


text:
tdesign 组件库与包自动引入冲突

信息
nodejs 版本
18.15.0
(推荐使用 v18 以上)

包管理器及其版本
pnpm 8.13.2
(请使用 pnpm)

系统
win11
(该项目开发环境系统为 win11)

bug
描述

<template>

<t-table :columns="columns" />

</template>

老师，这段代码。我用Vitesse就没问题，但是使用咱们的报错,错误提示是:
`chunk-TBCSS62C.js?v=caf2dfd6:9  Module "util" has been externalized for browser compatibility. Cannot access "util.inspect" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
get @ browser-external:util:9
(匿名) @ loupe.js:682
(匿名) @ loupe.js:2
node_modules/.pnpm/loupe@2.3.6/node_modules/loupe/loupe.js @ loupe.js:5
__require @ chunk-TBCSS62C.js?v=caf2dfd6:9
(匿名) @ index.js:7
browser-external:util:9  Module "util" has been externalized for browser compatibility. Cannot access "util.inspect" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
get @ browser-external:util:9
(匿名) @ index.js:125
__x00__virtual:modules:19  [Vue Router warn]: Unexpected error when starting the router: Error: Vitest failed to access its internal state.
One of the following is possible:

"vitest" is imported directly without running "vitest" command

"vitest" is imported inside "globalSetup" (to fix this, use "setupFiles" instead, because "globalSetup" runs in a different context)

Otherwise, it might be a Vitest bug. Please report it to https://github.com/vitest-dev/vitest/issues
at createVitest (vendor-vi.23b98fa1.js:3129:11)
at vendor-vi.23b98fa1.js:3321:16
warn @ vue-router.mjs:35
(匿名) @ vue-router.mjs:3553
Promise.catch(异步)
install @ vue-router.mjs:3551
use @ runtime-core.esm-bundler.js:3752
(匿名) @ __x00__virtual:modules:19
useModules @ __x00__virtual:modules:6
(匿名) @ main.ts:13
setTimeout(异步)
(匿名) @ __uno.css:20
(匿名) @ __uno.css:20
Show 4 more frames`

预期结果
TODO

最小测试仓库
TODO

link: https://github.com/dishait/vite-auto-import-resolvers/issues/9

@zanderzhg 靓仔你好，该问题已解决

@zanderzhg 靓仔你好，该问题已解决，请重新下载，或 pull 拉取模板，并重新执行 pnpm install 即可

谢谢

我把vitest 删除就没问题了，

我把vitest 删除就没问题了，

也可以这样做 👍

authors: markthree zanderzhg

repo: dishait/tov-template


text:
Create additional info blocks

You can make your resume less official by adding more entertaining info, like your hobbies and interest, or what do you like, some interactive content or joke, easter egg, etc.

hello binary studio academy

repo: dispache/homepage


text:
🛑 DSAT is down

In b29372d, DSAT (https://www.cptec.inpe.br/dsat) was down:

HTTP code: 0
Response time: 0 ms

Resolved: DSAT is back up in f1d1c9c.

repo: dissm-inpe/dsat-upptime


text:
DITA 1.1 DTDs should point to each other using 1.1 specific Public IDs

Let's say my DITA topic has a DOCTYPE like:
<!DOCTYPE glossentry PUBLIC "-//OASIS//DTD DITA 1.1 Glossary//EN" "glossary.dtd">

which is usually mapped to this DTD file:
 DITA_OT_DIR\plugins\org.dita.specialization.dita11\dtd\glossary.dtd

which is OK. But if you open that, it refers to all modules like:
<!ENTITY % topic-type   PUBLIC
"-//OASIS//ELEMENTS DITA Topic//EN"
 "topic.mod"                                                          >
 %topic-type;

Notice that the public IDs are not 1.1 specific. Because of this, they
will be mapped through the XML catalog support to the default 1.2 public
IDs.
All these inner-reference PUBLIC IDs between the DTDs (including 1.2
DTDs) should have been modified to refer directly using the proper version.

Do you have interest in sending a pull request for this?

Not really, it was just something one of our integrators stumbled into.
Also this looks like it needs some time and manual labor to do.

I think this may be a task that could be done using regexp search and replace. Just add version number to every public ID in .dtd, .mod, and .ent file.

Fortunately the DTDs for 1.2 like:
DITA-OT\dtd\technicalContent\dtd\glossentry.dtd

properly refer explicitly to 1.2 public IDs.
So we will not have the same issue when the DITA 1.3 standard is released in the DITA OT and people will want to use explicitly the DITA 1.2 public IDs.

I realize this may be obsolete, but I'm looking to close out simple old issues where I can and this seemed like a nice cleanup item. I tested it by switching everything in the samples directory to use 1.1 specific DTDs, then built hierarchy.ditamap and taskbook.ditamap, which hits almost every domain and topic type.

Well, that failed the Travis build, but I haven't checked why yet - will look into it.

Ah. Our test cases actually contain files that are technically invalid. First one I checked has this:
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE topic PUBLIC "-//OASIS//DTD DITA 1.1 Topic//EN"
 "file:/C:/DITA-OT1.4/dtd/topic.dtd">
 <topic id="critdates_replace" xml:lang="en-us">
     <title>TestCritdates</title>
     <prolog>
         <source href="http://www.ibm.com" format="html" scope="external">somewhere,someplace</source>
     </prolog>

asbsdf

 </topic>

The <source> element did not have the @format attribute in DITA 1.1, so the test case was correct to fail (that attribute was added in DITA 1.2). Need to update this test case and potentially others in order for this to pass.

@robander is this fixed now?

authors: jelovirt raducoravu robander

repo: dita-ot/dita-ot


text:
In case of chunking, new topic file name should be created using the title.

When DITA-OT chunks dita file having multiple topics, it generates the names of the chunk using the id attribute of the topic element. This happens in case of chunking "by-topic". It should rather generate the filenames on the basis of the title of that dita topic.

See DITA 1.3 Using the @chunk attribute. The spec uses SHOULD for the naming scheme, so an implementation is allowed to use an alternative scheme, but the default behavior of DITA-OT is to follow the DITA 1.3 specification.

authors: jelovirt riteshksriv


text:
dita2markdown Tables are malformatted when cells contain multiple paragraphs or lists

Expected Behavior
Tables rendered in a dita2markdown conversion should be well formatted regardless of whether cells contain paragraphs or lists.
Actual Behavior
Tables with even relatively simple content are rendered in markdown output poorly and often don't conform to general markdown requirements.
For example, the generated markdown output for a table that contains a list, might appear as follows:
|Title|Description|
|-------------------------|-----------|
|tools
|Provides a set of useful tools. Useful files under the directory include:

 -   **`wibble`**
...

This example illustrates one of many issues with the table formatting that results in a dita2markdown conversion.
Possible Solution
Perhaps the easiest solution to this problem is to provide an option to render tables in HTML format, which most markdown renderers and parsers respect.
Table formats are notoriously problematic in Markdown, particularly for more complex tables, so providing an option to output tables in HTML format could help work around these problems.
Environment

DITA-OT version: 3.6.1 and 4.1.2
Operating system and version:
Linux
How did you run DITA-OT?
dita command
Transformation type:
markdown_github markdown_gitbook markdown

This is a known issue and will likely be implemented in the next LwDITA plug-in version. See https://github.com/jelovirt/org.lwdita/pull/200

authors: jelovirt ol-roboro


text:
Templatetag "render_model_add" using properties from instance

It would be cool to be able to create a new instance with some default property values in order to have these pre-filled in the popup dialog in the frontend.
# views.py
context['new_person`] = Person(category='Some category based on current view')

# template.html
{% render_model_add new_person %}

What do you guys think?

I would like to see this too!
@yakky, did I hear you mention the other day that you're working on this?

@mkoistinen right: during the weekend I'll post a PR and will take care of this

@mkoistinen @helderco the linked PR should tackle the issue.
Setting instance.pk = 0 looks a little hackish, but it's ok since the instance it's never really touched

@helderco does #3888 fixes the issue for you?

Hey @yakky thanks!
It fixes my original idea, but this issue is actually about something else. It's about having the form pre-populated with some values by instantiating an object with those values and using it in render_model_add.

@helderco sorry, I read the original text and missed the update.
Postponing this after 3.1, then

@yakky, I thought no one would want to refactor to use the class name because you can simply create a new empty object in the view, that's why I updated this issue to mean a different situation.
Since you've done it, I want to re-edit this issue to be what it was and create a new one with what this has now, but I don't have my original description.
Have you got some sort of a notification when the issue was created?

@helderco ok, I'll edit the description  with the oriiginal idea

@helderco actually it makes sense to leave this as is as a proposed enhancement, and just let the PR fix the original idea

Hmm... I just created a new issue. What do you want to do? Remove the one I just created or restore this one?

@helderco race condition! :D no worries, I'll restore this to its original content and will close with the above PR

Can't we apply this to 3.0.x? I mean, this is how it was supposed to work anyway.

Fixed in #3899

authors: helderco mkoistinen yakky

repo: divio/django-cms


text:
Can't install - Why do Django CMS want to include MySQL-python with Python 3.4.3?

Trying to install Django CMS 3.2 with Django 1.9 and Python 3.4.3 and when it's time to install dependencies it gives me an error:
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/n9/t1wyb5y171df9032_fykvr8h0000gn/T/pip-build-3fncvfa2/MySQL-python
Why do Django CMS even want to include the module MySQL-python with Python 3.4.3 as it is not supported? mysqlclient is the recommended one at the moment.
How can I come around this? Currently the installer won't put any files in the folder I specified.

Django 1.9 is currently not supported. django CMS does not depend on Mysql and allows to use any mysql driver

OK. Thank you for the quick reply. When do you guys think you will ship out a new release that supports 1.9?

No official ETA, but the end of the year is a sensible timeframe

authors: yakky zruu


text:
Djangocmstutorial

Description

Related resources

#...
#...

Checklist

[ ] I have opened this pull request against develop
[ ] I have updated the CHANGELOG.rst
[ ] I have added or modified the tests when changing logic

Coverage remained the same at 78.215% when pulling b6dc3dfe61ae7617d351806be3c19056d1490a6f on mebdivio:djangocmstutorial into 251879ec6f6312ced985cc0296800bd6dd363f94 on divio:develop.

authors: coveralls mebdivio


text:
🛑 Arcadja is down

In e7af732, Arcadja (https://www.arcadja.com/auctions/en/) was down:

HTTP code: 302
Response time: 1677 ms

Resolved: Arcadja is back up in f01f64a after 16 minutes.

repo: divtiply/artupptime


text:
🛑 China Guardian is down

In dde960f, China Guardian (https://www.cguardian.com/en/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: China Guardian is back up in a11f6f2 after 4 minutes.


text:
🛑 Radio France Open API is down

In b0a7acd, Radio France Open API ($SECRET_SITE) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Radio France Open API is back up in 8d7c4cf.


text:
🛑 Dixneuf19 website is down

In ecbd596, Dixneuf19 website (https://www.dixneuf19.me) was down:

HTTP code: 504
Response time: 15426 ms

Resolved: Dixneuf19 website is back up in db46832.


text:
TODO:

[x] #7
[ ] #8
[ ] #9
[ ] #10
[ ] #11

added new task

CODE REFACTOR

repo: diyarfaraj/AppBlockr


text:
Wrong result if program reads no input.

If tested program reads no input (but the testcase has input), it can termiante before we write the input to the stdin and it will cause IOStreamException Broken Pipe or Stream Closed. Then the result will be "aborted" while it should process the testcase normally comparing output with expected output.

When is such a scenario expected to arise?

Not too often, for sure. Although it did come up for Krig as you can see :)

OK. I tried a couple of problems from codeforces to get the situation (mentioned above).
But in all the cases hightail seems to be working fine.
Please correct me if my understanding of the issue is incorrect:
If we have a problem which takes no input, and in hightail we have specified some input, then since the program will terminate before anything would be written to stdin, and hence IOStreamException is expected to be thrown. Right?
@Krigpl Can you please share a link to the problem, while testing which you encountered this exception?
PFA the screenshot:

It was quite a long time ago and I don't recall any details. But if it
seems to be working now I would assume the bug is gone.
2015-07-26 21:21 GMT-07:00 ASTGS notifications@github.com:

OK. I tried a couple of problems from codeforces to get the situation
(mentioned above).
But in all the cases hightail seems to be working fine.
Please correct me if my understanding of the issue is incorrect:
If we have a problem which takes no input, and in hightail we have
specified some input, then since the program will terminate before anything
would be written to stdin, and hence IOStreamException is expected to be
thrown. Right?
@Krigpl https://github.com/Krigpl Can you please share a link to the
problem, while testing which you encountered this exception?
PFA the screenshot:
[image: hightail_bug_1]
https://cloud.githubusercontent.com/assets/9794969/8898626/f8c6bfbc-3444-11e5-9c60-9f0d3b179e54.jpg
—
Reply to this email directly or view it on GitHub
https://github.com/dj3500/hightail/issues/42#issuecomment-125081799.

authors: ASTGS Krigpl dj3500


text:
Fix glyphicon display for generic-lookup

This fixes issue #214

Merged with the the th-list icon. Thanks a lot!

authors: andrewyager xrmx


text:
Items and help needed for a new release?

I wondered about the plans for a new release and what help might be needed to make it happen? There was some talk about beefing up the documentation for the next release, but I'm unsure what needs to be done since I found everything I needed in the docs.

Hey thanks for the comment and appologies for the slow uptake.
There's larger work being done on crispy-forms core but I think we can go ahead and get a release of this template pack out now.
Good to hear you found everything in the docs 👍


text:
How to search for immediate children node?

I want to search like this  'Searching for all employee and sub-department in a department' , or 'Searching for all immediate children nodes by a certain DN'.
I tried to code like this but failed
rootDN = request.GET.get('rootDN')
CommonLdapObject.rootDN = rootDN
resultList = CommonLdapObject.objects.all()
Is there any way to search like that by django-ldapdb?

I tried code below, it seems to be OK
rootDN = request.GET.get('rootDN')
 newSearchClass = CommonLdapObject.scoped(rootDN)
 newSearchClass.search_scope = ldap.SCOPE_ONELEVEL
resultList = newSearchClass.objects.all()

Yeah, the design of django-ldapdb is to have models mapping objectclasses; thus, dumping the whole directory is not exactly a primary goal.
It looks like your work-around suits you, so I'm closing this; feel free to re-open if you need more help!

authors: rbarrois zhuaiji

repo: django-ldapdb/django-ldapdb


text:
running ./manage.py test runs dist-package/nose/case.py

running ./manage.py test fails on these given tests. Does anyone know if this is because I am importing these tests somewhere and it is running them? Any explanations as to why these tests are running and/or failing would be helpful. thanks.
======================================================================
ERROR: Construct a test case with the specified label. Label should be of the
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/nose/case.py", line 197, in runTest
self.test(*self.arg)
File "/usr/local/lib/python2.7/dist-packages/nose/util.py", line 619, in newfunc
return func(*arg, **kw)
TypeError: build_test() takes exactly 1 argument (0 given)
======================================================================
ERROR: Creates a test database, prompting the user for confirmation if the
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/nose/case.py", line 197, in runTest
self.test(*self.arg)
TypeError: create_test_db() takes at least 1 argument (0 given)

Hi, I don't know if this is information is still useful but it turns out we we running django_nose tests because we subclassed NoseTestSuiteRunner in our TestSuiteRunner
We had the following:

class TestSuiteRunner(NoseTestSuiteRunner):
    blah

Don't subclass and issue goes away.

Smells similar to #209.

repo: django-nose/django-nose


text:
Fixed #29995 -- Used higher contrast colors on debug page.

In our technical error page in debug mode, some of the text/background
colors were not high contrast enough to meet the WCAG standards for
accessiblity (https://www.w3.org/TR/UNDERSTANDING-WCAG20/Overview.html).
This change addresses those low contrast color combinations.

Hey @zachgarwood, thanks for submitting this PR.
Would it be possible to provide before/after screenshots of the debug page?

Looks identical to me. You probably need to render the template with Django rather than viewing the file.

@timgraham We're dealing with changing shades of grey to slightly darker shades of grey; the difference it's not readily apparent. To you or I viewing this page normally, there will not be much difference, but to a developer using an assistive tool like high contrast mode, these subtle differences may be more apparent.

Okay. I think it's fine to keep it as is given the reporter's conclusion on the ticket.

I believe it is an error to close this issue without merging in the changes, despite the fact that these changes did not directly address the issue expressed by the issue reporter. As it stands, the technical error page does not meet the W3C's WCAG's standards for accessibility. I consider a non-compliant, accessibility-unfriendly page in the codebase a bug. These changes should be merged in to correct that bug.

If you can find another person to review the changes and confirm that they're useful, I am not completely opposed. I'm reluctant to make unnecessary changes that might cause a regression.

I’m happy to review.
@zachgarwood I don’t know much: is there anything beyond the links here and on the ticket that you think is important/relevant?

My sources:

https://www.w3.org/TR/UNDERSTANDING-WCAG20/visual-audio-contrast7.html
https://medium.com/@matuzo/writing-css-with-accessibility-in-mind-8514a0007939
https://webaim.org/articles/visual/lowvision#highcontrast
https://usabilla.com/blog/how-to-design-for-color-blindness/

And this is the tool I used to determine compliance and get recommendations for change (but there are other recommended tools):

http://accessible-colors.com/

As a point of order, I think dismissing accessibility issues out of hand as "unnecessary changes" is (unconsciously) ableist and is a disappointing direction for the Django code base to go in. I think we can do better.

What I meant is that it would be nice if these improvements were suggested and/or confirmed as useful by users of accessibility tools. Since no one has raised the issue in 10 years of Django, I don't know.

I entirely support any improvement that can ease the experience of users with accessibility issues. I think this patch is very constrained and has very few chances to cause regressions. A big +1 from me.

@timgraham, investigating which existing styles are / should be used is beyond the scope of this PR. I recommend opening a separate ticket for your concerns.

authors: carltongibson charettes claudep timgraham zachgarwood

repo: django/django


text:
Fixed #30056 -- Added StdDev() and Variance() support for SQLite.

Ticket #30056.

For the future, please submit cleanups as separate PRs so they can be merged more easily.


text:
Backport: Fixed #29182 -- Fixed schema table alteration on SQLite 3.26+.

SQLite 3.26 repoints foreign key constraints on table renames even when
foreign_keys pragma is off which breaks every operation that requires
a table rebuild to simulate unsupported ALTER TABLE statements.
The newly introduced legacy_alter_table pragma disables this behavior
and restores the previous schema editor assumptions.
Thanks Florian Apolloner, Christoph Trassl, Chris Lamb for the report and
troubleshooting assistance.
Backport of c8ffdbe514b55ff5c9a2b8cb8bbdf2d3978c188f from master.

This patch doesn't qualify for a backport. Please check also the discussion starting at https://github.com/django/django/pull/10733#issuecomment-456731434.

FWIW, this issue makes it impossible to test libraries supporting both Django 1.11 and Django 2.2 on a single Centos server:

Centos 7 supports sqlite3 3.7.17 (see EL-7 here)
Centos 8 supports sqlite3 3.26.0 (see EL-8 here

Our team runs internal libraries on Centos servers using tox to test against various versions of dependencies. The current version incompatibilities mean that Django 1.11 tests of a library must run on Centos 7 and tests for Django 2.2 must run on Centos 8.

Applied at phpMyAdmin on our GitHub actions CI !
Thank you so much for the patch !
See: https://github.com/phpmyadmin/website/commit/644136cf6675bd9cf03ef3756c4bcfe4ed7dd1ac
Logs: https://github.com/phpmyadmin/website/runs/1835844702#step:5:16

authors: bennofs felixxm tonysyu williamdes


text:
Fix commas every three digits.

This change replaces the decimal separator according to the "settings.USE_L10N" variable.
For exemple, 2000 becomes 2,000 or 2.000

intcomma already respects format localization.

desculpe, mas não está funcionando...

Please take a look at this test, it depends on your THOUSAND_SEPARATOR.

I already did this test and it still displays, because in the code it will always change to "," as in the attached image

If it will always change to comma then test_i18n_intcomma() won't work. intcomma doesn't reach this code when USE_L10N is enabled, it uses number_format(). Please use one of support channels, GitHub is not one of them.

thank you

authors: felixxm lucassobfer


text:
Fixed #31404 -- Changed selector-chosen's multiple attribute to HTML5 boolean syntax.

…boolean attribute syntax.

@zriv Thanks for this patch :+1: Welcome aboard :boat:
I don't think that release notes are required in this case, so I removed them.

buildbot, test on selenium.

@felixxm Thank you for the review, edit, and merge.  I'm glad I got the opportunity to give a little something back, small though it might be. :smile:

authors: felixxm zriv


text:
Fixed #21661 -- Improved Authentication Views documentation

Ticket:
https://code.djangoproject.com/ticket/21661

Merged in eb9b7abb833a6d317e355265552d47e0d8f24af8, thanks!

authors: erikr gitaarik


text:
Recommend relative imports within Django components.

django-developers thread:
https://groups.google.com/d/msg/django-developers/11XvmVdx58w/sFrF0pL8LTgJ

Looks good.

authors: aaugustin timgraham


text:
Fixed #26765 - Removed ETag when Cache-Control: no-store

Removed ETag when Cache-Control contains no-store.
See https://code.djangoproject.com/ticket/26765 for details.

buildbot, test this please

It doesn't look like I was able to get the buildbot to test this, but the tests do pass with sqlite and postgres locally.
I don't think there's a need to document this since no changes should be apparent to the end-user.
I confirmed that reverting the change breaks the test.

👍 from me

The main question I had was how this might interact with https://github.com/django/django/pull/6393. The author of the other PR didn't reply yet.

buildbot, add to whitelist.

Ah sorry, missed that. Since that one already has conflicts to resolve and hasn't been touched in a couple months I don't see any harm in merging this, if you don't hear back soon at least. Your call.
I believe this is still a valid test that should be duplicated in #6393 (?).

@tobiasmcnulty @timgraham I don't see any problem with #6393 but I guess it's better to wait for it to be finished and after this add @ignore_warnings(category=RemovedInDjango20Warning) to test I've added

Looks good to me, pending a decision about #6393.

merged in 20d39325ca1da57a709f3ba38299dc7b0fc4bdfb, thanks!

authors: andrewnester marfire timgraham tobiasmcnulty


text:
localhost/past-events/

[ ] use category_list template page, new view, separate URL to share all Events that have datetime > today

All events will stay on the calendar after theyve passed so they are accessible through that.

repo: djangonaut-space/wagtail-indymeet


text:
Retry bad nonce errors with a new nonce

I'm putting this up as a possible solution for handling bad nonce errors. The new BytesBody implementation feels a bit weird but I couldn't think of a better way that wouldn't have wide reaching interface changes. I think it makes sense to do the retry within Client so that everything benefits from it. The main thing this is missing is some backoff, though that may not even be necessary because we're not retrying due to a network or server side error.
Fixes: https://github.com/djc/instant-acme/issues/62

Just realised that I don't need to do the clone of BytesResponse.parts at all. I'll squash the commits into one much more descriptive one if you approve this!

This is great, thanks!

Sorry, I didn't realise you were actively reviewing and force pushed a change to use the Replay-Nonce header.

No problem!

Thanks!

repo: djc/instant-acme


text:
Make all models methods ignore database disabled models

Fixes #223

@djezzzl All good. I added some tests as well. Let me know if something is missing or I can better refactor the tests. Thanks for your time


text:
onCancel not working.

Hi there! I have a component that will render the alert.
If I click outside the alert, I got this error:
"TypeError: Cannot read property 'onCancel' of undefined"
The code for the alert:
    <SweetAlert
      title=""
      onConfirm=""
      showConfirm="false"
      showCloseButton="true"
      onCancel={this.onCancel}
      customButtons={
        <React.Fragment>
          <input
            onClick={() => window.location.reload(false)}
            value="Add"
            type="submit"
            disabled="true"
            className="hidden bg-gray-900 text-white flex items-center content-center justify-center rounded-md p-2 w-32 mb-1 cursor-pointer"
          />
        </React.Fragment>
      }
    >

{projectForm}

    </SweetAlert>

This code is inserted on a function component, inside the return method.
I also tried:

onCancel={() => this.onCancel}
onCancel={() => onCancel()}
onCancel={onCancel}
onCancel={onCancel()}

Thanks in advance!

Not an issue.


text:
🛑 Writeguard is down

In 01b8da7, Writeguard (https://www.writeguard.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Writeguard is back up in 66c475c.


text:
🛑 Writeguard is down

In 2b9b5bc, Writeguard (https://www.writeguard.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Writeguard is back up in 0c2eca2.


text:
🛑 Writeguard is down

In c3da0a1, Writeguard (https://www.writeguard.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Writeguard is back up in 00728bc.


text:
KeyError: 'css_files'

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/frc-pdr/envs/latest/local/lib/python2.7/site-packages/sphinx/cmdline.py", line 296, in main
    app.build(opts.force_all, filenames)
  File "/home/docs/checkouts/readthedocs.org/user_builds/frc-pdr/envs/latest/local/lib/python2.7/site-packages/sphinx/application.py", line 333, in build
    self.builder.build_update()
  File "/home/docs/checkouts/readthedocs.org/user_builds/frc-pdr/envs/latest/local/lib/python2.7/site-packages/sphinx/builders/__init__.py", line 251, in build_update
    'out of date' % len(to_build))
  File "/home/docs/checkouts/readthedocs.org/user_builds/frc-pdr/envs/latest/local/lib/python2.7/site-packages/sphinx/builders/__init__.py", line 322, in build
    self.write(docnames, list(updated_docnames), method)
  File "/home/docs/checkouts/readthedocs.org/user_builds/frc-pdr/envs/latest/local/lib/python2.7/site-packages/sphinx/builders/__init__.py", line 360, in write
    self._write_serial(sorted(docnames), warnings)
  File "/home/docs/checkouts/readthedocs.org/user_builds/frc-pdr/envs/latest/local/lib/python2.7/site-packages/sphinx/builders/__init__.py", line 368, in _write_serial
    self.write_doc(docname, doctree)
  File "/home/docs/checkouts/readthedocs.org/user_builds/frc-pdr/envs/latest/local/lib/python2.7/site-packages/sphinx/builders/html.py", line 461, in write_doc
    self.handle_page(docname, ctx, event_arg=doctree)
  File "/home/docs/checkouts/readthedocs.org/user_builds/frc-pdr/envs/latest/local/lib/python2.7/site-packages/sphinx/builders/html.py", line 1139, in handle_page
    self.app.emit('html-page-context', pagename, templatename, ctx, event_arg)
  File "/home/docs/checkouts/readthedocs.org/user_builds/frc-pdr/envs/latest/local/lib/python2.7/site-packages/sphinx/application.py", line 589, in emit
    results.append(callback(self, *args))
  File "/home/docs/checkouts/readthedocs.org/user_builds/frc-pdr/envs/latest/local/lib/python2.7/site-packages/sphinx_tabs/tabs.py", line 215, in add_assets
    if path in context['css_files']:
KeyError: 'css_files'

Simple if 'css_files' not in context: continue should fix, but it seems you're trying to remove the path from context['css_files'], but that key is only populated if there are css files in that one page, yet the path includes all pages

Thanks for reporting this! I've committed some changes that should fix this.

When will we see them in a release?

I will do it now ;)

It's up on pypi

repo: djungelorm/sphinx-tabs


text:
Sort cmis result

Is it to possible to sort cmis result natively in your php libraries ?
eg: I want to sort result by name or CreationDate and if possible show folder first
Best regard

I would say this depends on the exact use case. If you are pulling all objects without pagination you can always use usort with a custom sorting function to achieve the result you want. If you need to use pagination then you would be limited to whichever sorting functions are provided by the CMIS service you connected - I would expect any such service to support sorting by creation date, but sorting folders on top may not be universally supported.

authors: NamelessCoder wallon-ines

repo: dkd/php-cmis-client


text:
Spam

:| I mean sure it only takes a few seconds to close or merge the PR, but the differences in the capitalization of cocoapods

do not introduce findability issues
do not cause installation issues
do not make it difficult to understand how to install the dependencies

and so IMO this bot creates a lot of noise without providing any value.

Sorry that it is not helpful to you, feel free to close the issue(s) :bow:

cc @orta

no worries, just my 2c on this

This bot just created a PR to our repo that not only replaces uses of "Cocoapods" with "CocoaPods" but radically changes our README.md file to include documentation about some other project:
See this link for an example:
https://github.com/Medable/iOS-SDK/pull/4
Note: I would have merged a simple PR just changing the use of that word.

Hey that was a bug sorry

No problem, closed the PR and fixed the docs. At least it brought awareness to an issue with our documentation. So thanks for that.

Thanks for fixing the spelling

authors: billybiset dkhamsing sweetmandm

repo: dkhamsing/cocoapods-readme


text:
Set custom file permissions

Hello, great work with this extension.
Back in sublime text Advanced New File lets you create files with custom file permissions using: "file_permissions": "777" in the settings. I have to run python and pass the script as argument, instead of the preferred way, running the script directly because of not having set the execution bit.
Could you implement this feature?

Hey @dann1,
I don't have any resources to implement this right now. So if anyone wants to take this over until I get to it, you are more than welcome to :)
Cheers,
Dominik

This should be very easy to do. There is a simple option available in fs.appendFile (https://nodejs.org/api/fs.html#fs_fs_appendfile_path_data_options_callback)

authors: Telokis dann1 dkundel

repo: dkundel/vscode-new-file


text:
Update obsolete varargs example that uses __va_argsave

The portable syntax has been supported on the main platforms since 2.067 or earlier.

Go go go?!

OK :)

Thanks!

authors: CyberShadow yebblies


text:
Allow to select an example on the front page

Marking this as "needs work" as I will to rebase this once we agree on #1775

Alrighty, finally thanks to the minimal solution in #1775 this should be ready.

First: Diff looks great! Thanks for not giving up.
Dropdown looks a bit weird for me, any idea why?

Okay, I think we can fix this by providing our own selector symbol from FontAwesome.
However, there are still UI differences:
Chrom(ium):

Firefox:

Alright, but why is that necessary? Why isn't the browser rendering the arrow?

Alright, but why is that necessary? Why isn't the browser rendering the arrow?

Without it looked really ugly in my Firefox:

Without it looked really ugly in my Firefox:

That's because we're setting the background color. If we exclude that block from having its color and background color set, it should use the native styling... with the downside that it will look a little weird to people who have their browsers set to have a light-on-dark color theme by default. I don't think we need to worry about that, as long as the website is not unusable to them.

I guess either way works, but if possible, I think it would be better to not rely on font-awesome for the front page.

If we exclude that block from having its color and background color set, it should use the native styling... with the downside that it will look a little weird to people who have their browsers set to have a light-on-dark color theme by default. I don't think we need to worry about that, as long as the website is not unusable to them.

That wouldn't look good:

I actually tried to be close to styling of the "Install X" boxes.

I guess either way works, but if possible, I think it would be better to not rely on font-awesome for the front page.

FWIW we already rely on it:

AFAICT it's the only way to ensure a consistent experience across different browsers.

That wouldn't look good:

No, that's not right. You're trying to undo the background rule with another background rule, right? That's not going to work. There must be no background rule in effect at all.

FWIW we already rely on it:

Ah, that's right.

Seb do you want to try the no-CSS approach or should we go forward with the current FontAwesome version?

Seb do you want to try the no-CSS approach or should we go forward with the current FontAwesome version?

Hmm, the no-CSS approach would look like this:

or should we go forward with the current FontAwesome version?

I am fine with both ways (slight preference on FontAwesome because it keeps the styling consistent).

Needs some tweaking for mobile:

Needs some tweaking for mobile:

Thanks. I came to the conclusion that tip box hasn't been used much over the last years and especially users on their phone most likely won't post an example, so imho the best solution is also the simplest here: hiding the tip box on devices with a narrow layout.

Still broken for very narrow widths:

Still broken for very narrow widths:

I can't really reproduce this in Chrome or FF, but I added max-width: 100% which should fix things even for these tiny layouts:

@CyberShadow does it work for you now?

@CyberShadow does it work for you now?

Looks perfect, thanks!

tip box hasn't been used much over the last years

Forgot to post this earlier, but I think you simply found a bug in the forum search (or SQLite indexing). I know I fixed one yesterday (regarding deleted posts), but that search and its variations is producing inconsistent results with some variations of it (for one, the query needs to search the subject, not body).

authors: CyberShadow wilzbach


text:
Habit logging

Implement feature to log habits that are being tracked with habi log <habit>

Done in b55a02c

repo: dlarocque/habi


text:
🛑 CBU Online Website is down

In 95464db, CBU Online Website (https://cbuonline.edu) was down:

HTTP code: 0
Response time: 0 ms

Resolved: CBU Online Website is back up in a0028b5 after 186 days, 10 hours, 56 minutes.

repo: dlee-calbaptist/monitor


text:
gulp-sass discards changes from earlier in the pipeline when generating source maps

I believe there's a serious flaw in this plugin with the logic around file.sourceMap and opts.sourceComments. It results in node-sass attemping to re-read the file from disk, which discards changes from earlier in the gulp pipeline, and if an earlier change is a rename (e.g. with gulp-rename) it results in node-sass attemping to read a file which either doesn't exist or is the wrong file. Here are a couple of simple examples:
Given:
Gulp 3.8.10, gulp-sass 1.2.4, Ubuntu 12.
var
  gulp = require('gulp'),
  sourcemaps = require('gulp-sourcemaps'),
  rename = require('gulp-rename'),
  sass = require('gulp-sass'),
  through = require('through2');

This (rename):
gulp.task('default', function () {
  return gulp.src('./src/x.scss')
    .pipe(rename('y.scss'))
    .pipe(sourcemaps.init())
      .pipe(sass())
    .pipe(sourcemaps.write())
    .pipe(gulp.dest('./dist'));
});

Results in:
stream.js:94
      throw er; // Unhandled stream error in pipe.
            ^
Error: Error: File to read not found or unreadable: src/y.scss

And here is an example with a simple plugin that changes file contents (prepends a comment):
var prefix = function () {
  var prefix = new Buffer("\n/* Added dynamically */\n", 'utf-8');

  return through.obj(function (file, enc, cb) {
    file.contents = Buffer.concat([prefix, file.contents]);
    this.push(file);
    cb();
  })
};

gulp.task('default', function () {
  return gulp.src('./src/x.scss')
    .pipe(prefix())
    .pipe(sourcemaps.init())
      .pipe(sass())
    .pipe(sourcemaps.write())
    .pipe(gulp.dest('./dist'));
});

This results in the changes from the prefix plugin being discarded.
This means that the changes proposed in Consistent file handling #66 should not be accepted -- things need to go the other way.
If we're on the same page about this, I may work on a pull request.

@dlmanning @Keats So I've tracked down this problem. We're passing the file name to node-sass for libsass to compile instead of passing along the data stream and we should probably be passing along the data stream. The problem, however, with passing along the data stream is that we then need to pass along all of the includePaths for that file which would require us (or node-sass, or libsass if we can convince either of them) to build a graph the includePaths.
@jmm, your rename problem can be resolved by renaming it after compiling your file. The prefix problem persists until we determine what we're going to do about the data issue

@Snugug

your rename problem can be resolved by renaming it after compiling your file

That example is just to illustrate one of the failure modes. But, doing the rename after isn't equivalent, because then the original name winds up in the sources array of the source map.

@jmm Understood, see the rest of my comment about the underlying issue.

@Snugug Yup, I read it all, but even the rename case isn't fully solved until the underlying issue is corrected.

Right, I'm looking into a solution now.

Great, thanks!

@jmm Give this a test:
npm install snugug/gulp-sass#2.x-filedata --save-dev

I think I've fixed this to work as you're expecting. The two examples you've provided now work.

Thanks @Snugug. I tried it and it does indeed work as it should now for the the test cases I posted.

authors: Snugug jmm

repo: dlmanning/gulp-sass


text:
WIP: Initial prototype of a XML transformation hook based on XSLT

Sometimes the provider does not allow to set some libvirt attributes. Good examples are #350 (the provider does not allow to change the NIC model)  #419 (one needs to add a touch device).
This PR explores the idea of allowing for a transformation hook before creating the domain. The hook is in the form of a XSLT stylesheet. This allow for almost unlimited capabilities when "editing" the generated XML.
provider "libvirt" {
  uri = "qemu:///system"
}

resource "libvirt_domain" "xslt-demo-domain" {
  name = "xslt-demo-domain"
  memory = "512"

  network_interface {
    network_name = "default"
  }

  xml {
    xslt = "${file("nicmodel.xsl")}"
  }
}

nicmodel.xsl:
<?xml version="1.0" ?>
<xsl:stylesheet version="1.0"
                xmlns:xsl="http://www.w3.org/1999/XSL/Transform">
  <xsl:output omit-xml-declaration="yes" indent="yes"/>
  <xsl:template match="node()|@*">
     <xsl:copy>
       <xsl:apply-templates select="node()|@*"/>
     </xsl:copy>
  </xsl:template>

  <xsl:template match="/domain/devices/interface[@type='network']/model/@type">
    <xsl:attribute name="type">
      <xsl:value-of select="'e1000'"/>
    </xsl:attribute>
  </xsl:template>

</xsl:stylesheet>

Some notes about the implementation:

The current implementation depends on xsltproc to do the processing
It works well for properties that are not supported by the provider. Otherwise the provider will insist in changing them back. I have an acceptance test for both cases: changing the nic model and changing the network name.
I am not sure if it is worth to have the xsltproperty nested into an xml element.
Only implemented for domain resources, but 95% of the code is generic. A few lines are needed in every resource creation and schema to add support for it.

The idea is of this PR is to gather some feedback and foresee problems I may not be seeing yet.
//cc @ncsurfus @unsubtleguy
TODO

[ ]  Gather feedback
[ ] Decide on the element name
[ ] Write some docs
[ ] Add missing resources

i will have a look more in details on this during the week.
Generally i think is a good idea to have this kind of custom parser for modify the resource.
This will give User the possibility to implement their own Custom Resource if not supported, and also can give us the possibility to improve and focus on others core zone of the provider, instead of corner cases. :+1: :heart: :cupid:
A part of this, i think as reminder we should keep in mind take care also in the mechanism following:

updating resource.
In the cloudinit (in the past) and also currently in networking resource, the update of resource is not rockstable.

In this case we should keep in mind that an user can modify the created initial xlst with other parameters in the file and this imho should force the recreation of the domain or other resource.
Also as corner case , we should not have a Diff if the xlst has spaces/tab etc, so we don't force recreation of resource for nothing ( see past cloudinit bug).
I will have look more and think about it more with calm :smiley:

That is a great idea, and I think I might need that feature very soon! :heart: :heart: :heart:

@dmacvicar What about skipping the transformation if xslt.(string) is empty (for example)?
Rationale: make transformation optional based on some other boolean. For example, in my use case, I need to leave the XML unchanged if there is no need to change the declared "hardware type".

We can make it a no-op if xslt.(string) is empty. But then you would end with:
xslt = "${var.hardware_type ? file(var.hardware_type) : ""}"

Is that ok?

@dmacvicar yes, that was exactly the idea.
That would spare writing the noop.xslboilerplate file.

@MalloZup I am using this feature for the test suite, so I would need it packaged.
For me it is good enough in this current state, and it just works smoothly.

I still need to do a last pass before merging. Hook the diff function and add the transformation to the other resources.

https://github.com/dmacvicar/terraform-provider-libvirt/blob/master/libvirt/resource_libvirt_cloud_init_test.go#L72
You can test in testacc that we do not have diffs see the example above

i will revert the PR today

I think I have addressed all points. If you are fine with the nested xml.xslt attribute, this should be ready for review or merge.
This is a very advanced feature that should not be for day to day use, but to address shortcomings of the schema and corner cases, like our teams running testsuites with it.
//cc @Bischoff

Thanks @dmacvicar !!!

authors: Bischoff MalloZup dmacvicar

repo: dmacvicar/terraform-provider-libvirt


text:
BoundsError inside LinearAlgebra.Adjoint with crossed random slope

Hello,
I am trying to fit a mixed-effects logistic regression model. My data can be obtained here: https://surfdrive.surf.nl/files/index.php/s/y8N0zrTWaj1tGAf (will not be around forever). The formula that goes along with it is: phonemeDecBin ~ dutch + step + dutch & step + l + dutch & l + step & l + &(dutch, step, l) + ((1 + step + l + stepL_by_lTRUE + stepQ_by_lTRUE + stepC_by_lTRUE) | ppn) + ((1 + step) | primetarget). (The weird 'X_by_X' terms are manually-created interaction terms to work around https://discourse.julialang.org/t/help-with-error-in-mixedmodels-jl/10374/3)
This is the code that constructs the model from the provided data file:
using RCall, MixedModels, Distributions
R"load('data_MixedModels.RData')"
part = rcopy(R"part")
form = rcopy(R"form")
m = GeneralizedLinearMixedModel(form,part,Distributions.Bernoulli())

This gives the following error message:
ERROR: BoundsError: attempt to access 0-element UnitRange{Int64} at index [1]
Stacktrace:
 [1] getindex at ./range.jl:597 [inlined]
 [2] *(::LinearAlgebra.Adjoint{Float64,VectorFactorReTerm{Float64,UInt8,8}}, ::VectorFactorReTerm{Float64,UInt16,4}) at /home/cesko/.julia/packages/MixedModels/dn0WY/src/modelterms.jl:528
 [3] LinearMixedModel(::StatsModels.Formula, ::Array{MixedModels.AbstractTerm{Float64},1}, ::Array{Float64,1}) at /home/cesko/.julia/packages/MixedModels/dn0WY/src/pls.jl:37
 [4] #LinearMixedModel#12(::Array{Any,1}, ::Dict{Any,Any}, ::Bernoulli{Float64}, ::Type, ::StatsModels.Formula, ::DataFrames.DataFrame) at /home/cesko/.julia/packages/MixedModels/dn0WY/src/pls.jl:106
 [5] Type at ./none:0 [inlined]
 [6] #GeneralizedLinearMixedModel#39(::Array{Any,1}, ::Array{Any,1}, ::Dict{Any,Any}, ::Type, ::StatsModels.Formula, ::DataFrames.DataFrame, ::Bernoulli{Float64}, ::LogitLink) at /home/cesko/.julia/packages/MixedModels/dn0WY/src/PIRLS.jl:23
 [7] #GeneralizedLinearMixedModel#42 at ./none:0 [inlined]
 [8] GeneralizedLinearMixedModel(::StatsModels.Formula, ::DataFrames.DataFrame, ::Bernoulli{Float64}) at /home/cesko/.julia/packages/MixedModels/dn0WY/src/PIRLS.jl:45
 [9] top-level scope at none:0

It works after removing the second random slope, reducing the formula to: phonemeDecBin ~ dutch + step + dutch & step + l + dutch & l + step & l + &(dutch, step, l) + ((1 + step + l + stepL_by_lTRUE + stepQ_by_lTRUE + stepC_by_lTRUE) | ppn) + (1 | primetarget). I assume this is a bug, or am I doing something wrong? The problem, but not the error, seems similar to #127

Thank you for a thorough bug report.  The data frame part in the .RData file that you made available has duplicate names of stepL, stepQ and stepC.
R> names(part)
 [1] "phonemeDecBin"  "Cond"           "dutch"          "step"
 [5] "l"              "stepL"          "stepQ"          "stepC"
 [9] "lTRUE"          "stepL_by_lTRUE" "stepQ_by_lTRUE" "stepC_by_lTRUE"
[13] "ppn"            "stepL"          "stepQ"          "stepC"
[17] "primetarget"

Could you correct that or tell me which versions I should use?
I believe the problem is due to improper expansion of the step term in the expression (1+step|primetarget).  @kleinschmidt is working on a rewrite of the StatsModels package at
https://github.com/JuliaStats/StatsModels.jl/pull/71 to address that type of problem.
While waiting for that you could try rewriting the random effects term for primetarget as (1+stepL+stepQ+stepC|primetarget).  Even for that I think there is another method that I need to write.

It appears that those columns have just been duplicated.

R> all(part[[6]] == part[[14]])
[1] TRUE

R> all(part[[7]] == part[[15]])
[1] TRUE

R> all(part[[8]] == part[[16]])
[1] TRUE

so you can get around the problem by
R> part = part[, c(1:13,17)]

Apologies for taking a while to get back to this. You are correct that the duplicated columns were an oversight on my part, but did you manage to fit the model after removing them? I've tried doing this:
using RCall, MixedModels, Distributions
R"load('data_MixedModels.RData')"
part = rcopy(R"part[,-c(14:16)]")
form = rcopy(R"form")
m = GeneralizedLinearMixedModel(form,part,Distributions.Bernoulli())

and I also tried additionally changing the form line to
form = @formula(phonemeDecBin ~ dutch + step + dutch & step + l + dutch & l + step & l + dutch & step & l + (1 + stepL + stepQ + stepC + l + stepL_by_lTRUE + stepQ_by_lTRUE + stepC_by_lTRUE | ppn) + (1 + stepL + stepQ + stepC | primetarget))

in case the step in the random-effects parts was the problem as you suspected, but on my machine (64-bit Arch Linux running Julia 1.0.1) I still get the same error, and it only goes away if I remove the random slopes over primetarget.

This should work in the current release.

authors: cvoeten dmbates palday


text:
Feature request: expose configurable hashing function?

I love jdenticon!
I'm working on a project now that could use it in an unconventional way. I want to show a table of webhooks my service receives, and visually represent similar requests. I was thinking of using jdenticon with a hash function designed to optimize collisions as opposed to avoid them (for example the Nilsimsa hash). To do that, I would need to be able to pass jdenticon a hash function. How would you all feel about exposing a configurable hash? I'd be happy to put up a PR for it.

I believe I've found that you kind of already do this. Am I correct in saying that if I pass a valid hexadecimal string, it will not be hashed, and it will be rendered directly?

Yes, you're correct.

authors: dmester zekenie

repo: dmester/jdenticon


text:
dist_async slower than dist_sync

Hi, I tested the speed of multi-machine training with the following command:
../../tools/launch.py -n 2 -H hosts python train_imagenet.py --data-dir /mnt/ficushaiti/zzhang/data/mxnet/imagenet/rec/ --gpus 0,1,2,3,4,5 --batch-size 960 --kv-store dist_(a)sync

And I found that with "dist_sync", the total speed is 1070 samples/s, but with "dist_async" the speed is 860 samples/s.
Then I tried to fix the problem. I changed the code in src/kvstore/kvstore_dist_server.h from
      } else {
        // async push
        exec_.Exec([this, key, &recved, &stored](){
            CHECK(updater_);
            updater_(key, recved, &stored);
          });
        server->Response(req_meta);
        stored.WaitToRead();
      }
    } else {
      // pull

to
        } else {
          NDArray temp_data = recved.Copy(Context());
          // async push
          exec_.Exec([this, key, temp_data, &stored](){
              CHECK(updater_);
              updater_(key, temp_data, &stored);
            });
          server->Response(req_meta);
          stored.WaitToRead();
        }
      } else {
        // pull

then the problem was solved ( 'dist_async' gets the same speed as 'dist_sync' ).
So I wonder
(1) why is 'dist_async' slower than 'dist_sync'?
(2) Is my change right?
The environment:
2 Server with Ubuntu 12.04, each with 6 TitanX
Network is 1Gbps

Could you propose your change as a PR and we'll review it.
Thanks.
@mli

i think your codes are right if Copy do the actually memory copy, rather than just pass the pointer.
the reason might due to the updater's overhead is not small. in sync_dist, there is only one update for each layer in a minibatch, while for async, there are 12 updates. copying recved decouples the updating on the same key and therefore improve the performance.
however, your method might only improve the performance if that layer size is small, e.g. convolution layer. while for big layers such as fully connected, the extra memory copy might be pretty big.
a possible solution is, copy only if the recved size is < bigarray_bound_, which is defined at https://github.com/dmlc/mxnet/blob/master/src/kvstore/kvstore_local.h#L28
could you please do the following two things

confirm that .Copy will do the actual memory copy
propose a PR to enable your trick if recved.size < bigarray_bound_

thanks

authors: lispc mli piiswrong

repo: dmlc/mxnet


text:
Error - cifar10-recipe notebook

When I tried to run cifar10-recipe in iPython, I got below error. Would you please help to solve? Thanks. (Ubuntu 16.04LTS)
MXNetError                                Traceback (most recent call last)
 in ()
2           eval_data=test_dataiter,
3           eval_metric="accuracy",
----> 4           batch_end_callback=mx.callback.Speedometer(batch_size))
5
6 # if we want to save model after every epoch, we can add check_point call back
/home/dc/mxnet/python/mxnet/model.pyc in fit(self, X, y, eval_data, eval_metric, epoch_end_callback, batch_end_callback, kvstore, logger, work_load_list, monitor, eval_batch_end_callback)
742
743         arg_names, param_names, aux_names =
--> 744                 self._init_params(dict(data.provide_data+data.provide_label))
745
746         # setup metric
/home/dc/mxnet/python/mxnet/model.pyc in _init_params(self, input_shapes, overwrite)
482     def _init_params(self, input_shapes, overwrite=False):
483         """Initialize weight parameters and auxiliary states"""
--> 484         arg_shapes, _, aux_shapes = self.symbol.infer_shape(**input_shapes)
485         assert(arg_shapes is not None)
486
/home/dc/mxnet/python/mxnet/symbol.pyc in infer_shape(self, *args, **kwargs)
456             The order is in the same order as list_auxiliary()
457         """
--> 458         return self._infer_shape_impl(False, *args, **kwargs)
459
460     def infer_shape_partial(self, *args, **kwargs):
/home/dc/mxnet/python/mxnet/symbol.pyc in _infer_shape_impl(self, partial, *args, **kwargs)
516             ctypes.byref(aux_shape_ndim),
517             ctypes.byref(aux_shape_data),
--> 518             ctypes.byref(complete)))
519         if complete.value != 0:
520             arg_shapes = [
/home/dc/mxnet/python/mxnet/base.pyc in check_call(ret)
75     """
76     if ret != 0:
---> 77         raise MXNetError(py_str(_LIB.MXGetLastError()))
78
79 if sys.version_info[0] < 3:
MXNetError: InferShape Error in concat2: [02:25:49] src/operator/./concat-inl.h:152: Check failed: (dshape[j]) == (tmp[j]) Incorrect shape[1]: (128,80,13,13). (first input shape: (128,160,14,14))

@DCYN should be fixed with https://github.com/dmlc/mxnet/pull/3599

Many thanks! It works!

authors: DCYN tornadomeet


text:
vta_config problem!

when I change the vta_config.json configuration，it will display that A fallback configuration is used, which may bring great performance regression. and the amount of  loaded data and weight is huge. How to configure the correct parameters?

Thanks for reporting the problem. Please use https://discuss.tvm.ai/ to involve more community members

repo: dmlc/tvm


text:
Tutorial: update Building a Graph Convolutional Network tutorial

Few updates on the Building a Graph Convolutional Network tutorial

Add bias in the GCN layer
Load pretrained model exported from DGL and verify model accuracy
use time_evaluator to measure the runtime

Please review
@ZihengJiang, @tmoreau89

Adding @yuluny2 to also review the PR.

lgtm

Thank @tmoreau89 for the comments.
There is no stable GPU scheduling yet, so we can't show the GPU version.

@tmoreau89
In my personal testing, TVM runs faster than MxNet but has similar speed with PyTorch.
Also, the inference time is very short (under 30ms) for each iteration.
I'm not sure whether it's fair to show the runtime comparison.

@cylinbao better to change the previous performance claim

@cylinbao thanks. If you can just submit a correctness check that will ensure that our computation is correct. Let's leave GPU support and performance comparison to a future PR.

@tmoreau89
Add the checking with DGL-PyTorch.
@ZihengJiang @yuluny2
You may want to review again. Thanks!

Thanks looks good, please address the comments above.

I think all the comments have been addressed!

Thanks, indeed they've been addressed (I resolved them). Will wait on checks to pass before merging it in!

Looks like the CIs failed, can you investigate?

It seems like the DGL version is too old in the CL.
Is there a way we can update the DGL version to 0.4?

Add one commit to handle the DGL version issue in the code.

Thank you for the update @cylinbao !


text:
XGBClassifier always stopping after 99 rounds

Issue
While training a model using XGBClassifier with early stopping in Python, the model always stops after 99 training iterations unless early stopping is set to something less than that. I'd like to  train this model for many more iterations than just 99, but no matter what I set early_stopping_rounds to it stops after 99.
Source
downloaded through Anaconda
Code
eval_set = [(X_test, y_test)]
print('Training')
xgb = XGBClassifier(max_depth=2, learning_rate=0.01, objective='binary:logistic', random_state=seed, verbose=2)

xgb.fit(X_train, y_train, early_stopping_rounds=200, eval_metric='auc', eval_set=eval_set, verbose=True)

How does the AUC metric change over time? Maybe early stopping is working as intended?

repo: dmlc/xgboost


text:
[jvm-packages] Rework the train pipeline

Add PreXGBoost to build RDD[Watches] from Dataset
Feed RDD[Watches] built from PreXGBoost to XGBoost to train

This is the first PR of https://github.com/dmlc/xgboost/pull/7361

@trivialfis Could you help to review it?


text:
update view right away

This will fix the problem not updating the page right away when request mgr link is clicked (approved, assigned)

@vkuznet, Valentin, please review. You can test it reqmgr2-dev.cern.ch instance.

Seangchan, yes I tested the proposed behavior and it works fine. Please go ahead and merge. One question though. When I created my new request I didn't see pop-up showing new request name, see issue https://github.com/dmwm/WMCore/issues/5875 and PR https://github.com/dmwm/WMCore/pull/5877 I just wonder if you put this code into reqmgr2-dev or not (since PR is merged).

Hmm, that should be updated as well

Hi Valentin, I just check. Your codes are upto date in reqmgr2-dev

Ok, I'll check then why it is not shown.
On  0, ticoann notifications@github.com wrote:

Hi Valentin, I just check. Your codes are upto date in reqmgr2-dev

Reply to this email directly or view it on GitHub:
https://github.com/dmwm/WMCore/pull/5900#issuecomment-101748969

authors: ticoann vkuznet

repo: dmwm/WMCore


text:
Use half of the resources for pulling work from GQ

wmagent branch version of #8248

 Fix #8249 - Use half of the resources for pulling work from GQ

 Fix #8249 - Use half of the resources for pulling work from GQ

@ticoann I think this one can be merged


text:
Fix archiving workflows

Introduction of https://github.com/dnanexus/dx-cwl/pull/2 introduced a regression where archiving old workflows no longer works.

Addressed by 601528c6d4667dfac6dc6784c682dadf5c533cbf

repo: dnanexus/dx-cwl


text:
API 키 생성 이슈

현재는 저희 API서버를 아무나 인증 없이 호출이 가능합니다.
적어도 사용자가 브라우저를 통해 저희 API서버를 접속할 경우 내용을 보여주지 않고, 저희가 임의로 만든 API키를 http request 헤더에 넣어야만 호출에 대한 응답을 할 수 있도록 하는것이 좋을 것 같습니다.

사실 이렇게 해도 사용자가 콘솔을 확인해 인증 키를 확인하고 그 키를 기반으로 API서버에 접속 할 수 있지만 그래도 없는것보단 나을 것 같습니다.

지금 계속 하는중인데 시간이 좀 필요 할 것 같아서 다음 미팅까지 로그인 없이 가능할지 한 번 해볼게요!

넵!! 감사합니다!
다시 한번 정리드리자면 !!
세션이나 쿠키 처리같은 내용은 아니고!!
프론트랑 백엔드랑 임의의 문자열을 Key로서 들고서, 그걸 api-key 와 같은 헤더로 주고받으면서 간단한 인증을 하는 컨셉이에요!!
저희는 프론트 요청에서 api-key가 없으면 미들웨어에서 리젝해버리고, 인증관련 코드5xx대 리턴해주면 될거같아요 !!


text:
minecraft crashes when i try to run it with ubuntu 14 on my chromebook flip

i tried to install and run minecraft and everything was working perfectly until i hit play on the launcher screen and it crashed with this crash report:
[00:00:38 INFO]: Launching game
[00:00:38 INFO]: Unpacking natives to /home/andysam151/.minecraft/versions/1.9.4/1.9.4-natives-2772957460612
[00:00:39 INFO]: Launching in /home/andysam151/.minecraft
[00:00:39 INFO]: Half command: /usr/lib/jvm/java-7-openjdk-armhf/jre/bin/java -Xmx1G -XX:+UseConcMarkSweepGC -XX:+CMSIncrementalMode -XX:-UseAdaptiveSizePolicy -Xmn128M -Djava.library.path=/home/andysam151/.minecraft/versions/1.9.4/1.9.4-natives-2772957460612 -cp /home/andysam151/.minecraft/libraries/com/mojang/netty/1.6/netty-1.6.jar:/home/andysam151/.minecraft/libraries/oshi-project/oshi-core/1.1/oshi-core-1.1.jar:/home/andysam151/.minecraft/libraries/net/java/dev/jna/jna/3.4.0/jna-3.4.0.jar:/home/andysam151/.minecraft/libraries/net/java/dev/jna/platform/3.4.0/platform-3.4.0.jar:/home/andysam151/.minecraft/libraries/com/ibm/icu/icu4j-core-mojang/51.2/icu4j-core-mojang-51.2.jar:/home/andysam151/.minecraft/libraries/net/sf/jopt-simple/jopt-simple/4.6/jopt-simple-4.6.jar:/home/andysam151/.minecraft/libraries/com/paulscode/codecjorbis/20101023/codecjorbis-20101023.jar:/home/andysam151/.minecraft/libraries/com/paulscode/codecwav/20101023/codecwav-20101023.jar:/home/andysam151/.minecraft/libraries/com/paulscode/libraryjavasound/20101123/libraryjavasound-20101123.jar:/home/andysam151/.minecraft/libraries/com/paulscode/librarylwjglopenal/20100824/librarylwjglopenal-20100824.jar:/home/andysam151/.minecraft/libraries/com/paulscode/soundsystem/20120107/soundsystem-20120107.jar:/home/andysam151/.minecraft/libraries/io/netty/netty-all/4.0.23.Final/netty-all-4.0.23.Final.jar:/home/andysam151/.minecraft/libraries/com/google/guava/guava/17.0/guava-17.0.jar:/home/andysam151/.minecraft/libraries/org/apache/commons/commons-lang3/3.3.2/commons-lang3-3.3.2.jar:/home/andysam151/.minecraft/libraries/commons-io/commons-io/2.4/commons-io-2.4.jar:/home/andysam151/.minecraft/libraries/commons-codec/commons-codec/1.9/commons-codec-1.9.jar:/home/andysam151/.minecraft/libraries/net/java/jinput/jinput/2.0.5/jinput-2.0.5.jar:/home/andysam151/.minecraft/libraries/net/java/jutils/jutils/1.0.0/jutils-1.0.0.jar:/home/andysam151/.minecraft/libraries/com/google/code/gson/gson/2.2.4/gson-2.2.4.jar:/home/andysam151/.minecraft/libraries/com/mojang/authlib/1.5.22/authlib-1.5.22.jar:/home/andysam151/.minecraft/libraries/com/mojang/realms/1.8.19/realms-1.8.19.jar:/home/andysam151/.minecraft/libraries/org/apache/commons/commons-compress/1.8.1/commons-compress-1.8.1.jar:/home/andysam151/.minecraft/libraries/org/apache/httpcomponents/httpclient/4.3.3/httpclient-4.3.3.jar:/home/andysam151/.minecraft/libraries/commons-logging/commons-logging/1.1.3/commons-logging-1.1.3.jar:/home/andysam151/.minecraft/libraries/org/apache/httpcomponents/httpcore/4.3.2/httpcore-4.3.2.jar:/home/andysam151/.minecraft/libraries/it/unimi/dsi/fastutil/7.0.12_mojang/fastutil-7.0.12_mojang.jar:/home/andysam151/.minecraft/libraries/org/apache/logging/log4j/log4j-api/2.0-beta9/log4j-api-2.0-beta9.jar:/home/andysam151/.minecraft/libraries/org/apache/logging/log4j/log4j-core/2.0-beta9/log4j-core-2.0-beta9.jar:/home/andysam151/.minecraft/libraries/org/lwjgl/lwjgl/lwjgl/2.9.4-nightly-20150209/lwjgl-2.9.4-nightly-20150209.jar:/home/andysam151/.minecraft/libraries/org/lwjgl/lwjgl/lwjgl_util/2.9.4-nightly-20150209/lwjgl_util-2.9.4-nightly-20150209.jar:/home/andysam151/.minecraft/versions/1.9.4/1.9.4.jar net.minecraft.client.main.Main
[00:00:39 INFO]: Looking for orphaned versions to clean up...
[00:00:39 INFO]: Looking for old natives & assets to clean up...
[00:01:06 ERROR]: Game ended with bad state (exit code 1)
[00:01:06 INFO]: Ignoring visibility rule and showing launcher due to a game crash
[00:01:06 INFO]: Deleting /home/andysam151/.minecraft/versions/1.9.4/1.9.4-natives-2772957460612
please help i love minecraft

This has nothing to do with crouton... I've never known this issue with Minecraft. How did you install Minecraft?

This has nothing to do with crouton, only crouton issues are supposed to be brought up here.

@andysam151 You should try somewhere that discusses Linux based Minecraft issues.

repo: dnschneid/crouton


text:
🛑 Site Histovec is down

In 7ed0faf, Site Histovec (https://histovec.interieur.gouv.fr/histovec) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Site Histovec is back up in 3fab209.

repo: dnum-mi/stats-sites-api


text:
support loading documents with from_files signature

Is your feature request related to a problem? Please describe.
In docarray v1, we support:
da = DocumentArray.from_files('imgs/*.jpg')

While it is missing in the current v2. Could be very useful for us to loading the documents from path.

Hey @samsja i would like to pick up this one as well, if it's ok to have multiple issue assigned.

@JohannesMessner may you give a bit more context here ?

If you are not working on this @agaraman0 , can I take this issue up @samsja ?

Hey @samsja , I would like to take upon this issue if its open !

repo: docarray/docarray


text:
Feat: Add not found page for iframe

fixes: #701

One issue left, the page now 404s when editing the url directly.

One issue left, the page now 404s when editing the url directly.

Could not reproduce, seems to work pretty well.

repo: docat-org/docat


text:
More resilient gpg getting

move to "high-availability" subset.

related: https://github.com/docker-library/php/pull/92

LGTM

authors: tianon yosifkit

repo: docker-library/percona


text:
PostgreSQL Database directory appears to contain a database; Skipping initialization

I get this in the docker logs when doing docker up.
/usr/local/bin/docker-entrypoint.sh: running /docker-entrypoint-initdb.d/01-init.sh

PostgreSQL Database directory appears to contain a database; Skipping initialization

Looking at docker-entrypoint.sh it looks like it's because PG_VERSION exists in the postgres data dir.
https://github.com/docker-library/postgres/blob/master/13/alpine/docker-entrypoint.sh#L227

Can you give all the commands you ran and any relevant files or logs for reproducing the issue

repo: docker-library/postgres


text:
Tomcat welcome page is not coming up after using tomcat container

root@instance-1:~/Projects# docker ps
CONTAINER ID        IMAGE               COMMAND             CREATED             STATUS              PORTS                     NAMES
08c8566704c8        tomcat              "catalina.sh run"   33 minutes ago      Up 33 minutes       0.0.0.0:15001->8080/tcp   mytomcat2
10b65dceb21f        tomcat              "catalina.sh run"   42 minutes ago      Up 42 minutes       0.0.0.0:9100->8080/tcp    mytomcat
root@instance-1:~/Projects# netstat -anp | grep 9100
tcp6       0      0 :::9100                 :::*                    LISTEN      30541/docker-proxy
root@instance-1:~/Projects# netstat -anp | grep 9100 15001
grep: 15001: No such file or directory
root@instance-1:~/Projects# netstat -anp | grep 9100
tcp6       0      0 :::9100                 :::*                    LISTEN      30541/docker-proxy
root@instance-1:~/Projects# netstat -anp | grep 15001
tcp6       0      0 :::15001                :::*                    LISTEN      30845/docker-proxy

when i try to open tomcat on by browser below error is thrown
HTTP Status 404 – Not Found
Type Status Report
Message Not found
Description The origin server did not find a current representation for the target resource or is not willing to disclose that one exists.
Apache Tomcat/8.5.50

The default webapps aren't enabled by default anymore, but they are in webapps.dist if you want to re-enable them
https://github.com/docker-library/docs/tree/master/tomcat#how-to-use-this-image
See also https://github.com/docker-library/tomcat/issues/184

Thank you so much..!! the rename did the tric.

repo: docker-library/tomcat


text:
Re-enable scanning for SBOM records in Docker image

In a slightly dated Docker Blog post, it was highlighted that you could add SBOM details for any binaries added manually to a Docker image:
https://www.docker.com/blog/generate-sboms-with-buildkit/
This was broken some time back apparently and the blog also references a snippet that does not work (need to be a full SBOM package record).
Syft maintainers suggested a minor fix to re-enable this feature:
https://github.com/docker/buildkit-syft-scanner/blob/f22f9866c24554ba087f5798d03a41c39f3e5f7b/internal/target.go#L49
syft.CreateSBOM(context.Background(), nil, syft.DefaultCreateSBOMConfig().
WithCatalogerSelection(pkgcataloging.NewSelectionRequest().
WithDefaults(pkgcataloging.ImageTag).

  	WithAdditions("sbom-cataloger"),

),
)

Full discussion with Syft is here:
https://github.com/anchore/syft/issues/3490#issuecomment-2511860821

Hm, we should also have had tests for this case.

Tried to create a PR but hit a ton of issues (as I'm not a GoLang expert) with Syft and case of "microsoft" vs "Microsoft" in Syft dependencies. Since this project pulls in both github.com/Microsoft and github.com/microsoft, it picks capitalized version and fails on syft dependencies.

Similar issue to https://github.com/anchore/syft/issues/2026

This is now fixed in master via #115.

authors: cdupuis jedevc nycnewman

repo: docker/buildkit-syft-scanner


text:
[dumb-init] docker-bench-security.sh: No such file or directory

Hello everyone,
First off, apologies if this is the wrong place to ask this. I am new to Github.
I have tried to run this tool on Docker Desktop (running on Windows 10 pro) by following the directions for building and running the tool and Docker-Compose found on this project's page. In both instances, I get the following error:
[dumb-init] docker-bench-security.sh: No such file or directory
I have verified that this file is present in the directory, so I am not sure what the issue could be. Any guidance would be appreciated.
Thanks,

Hello ! I'd try to do the same this morning and I have the same error... I need help too !
Thanks :)

Hi @Dave-0-0 and @Starbeuck, I have no experience when it comes too running docker-bench-security on anything else than a Linux system and there haven't been any form of testing on anything other than on Linux.
Could you include build and and run logs?

Hi @konstruktoid, I would be happy to do that, but I am not sure where they are located. Can you point me in that direction?
Thanks

There's actually no log files, it's the messages you get when you use docker build and docker run.

Docker build needs to know where to find the Dockerfile. It looks like you are in the directory with it, so you just need to tell it current directory '.'

Closing due to inactivity.

authors: Dave-0-0 Starbeuck jeff-cook konstruktoid

repo: docker/docker-bench-security


text:
Update supported versions of Fedora

17.12 won't support F24 or F25, but will support F27. 🎉
This work will need to be done in vnext-engine.

Duplicate of #5554


text:
Hugo: Early warning about pulling out the relative link support

I'm @bep, the lead maintainer of Hugo.
See https://github.com/spf13/hugo/search?q=SourceRelativeLinksEval&type=Issues&utf8=✓
This query illustrates the approximate situation.
This feature was implemented and merged to support Docker by now two former Docker employees,  @SvenDowideit and @Spf13.
No one is actively maintaining this feature.
So it is left to me.
I have plenty of other issues that are way plenty higher up the ladder, and, to be honest, it isn't the cleanest code I have seen, so unless someone from Docker is coming forward and claiming ownership of that code, I am marking it for deletion in Hugo 0.18.

Yup, we are now using Jekyll. So feel free to deprecate the feature.

@SvenDowideit @bep Very kind of you guys to check in on us. It's true, we are on Jekyll now. Thanks so much for looking in. :)

authors: bep joaofnfernandes johndmulhausen


text:
Added a new remove unofficial docker step

I found an error when trying to install the official docker-engine, as there a second package that conflicts with the new package.  Added instructions on how to remove it.

Proposed changes

Unreleased project version (optional)

Related issues (optional)

These look great, if you add Fedora, it will be good to go. Don't forget, that Fedora uses dnf instead of yum.

authors: mstanleyjones tradfor


text:
Trivial typo fix, but it stood out

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Thanks for helping keep the docs updated 🚀


text:
Adding wsl2 tech preview content

Adding docs for WSL 2 Tech Preview

@jcfong @Butler59 could you PTAL?

Deploy preview for docsdocker ready!
Built with commit 8cdeb6edb6ef4a7e12a6f2039f29d7e1cb2e444b
https://deploy-preview-9177--docsdocker.netlify.com

per Usha this pr was reviewed and approved by Ben and David Butler.


text:
TestPullVerified fails on Docker master branch

Running the TestPullVerified test (as part of 'make test-integration-cli') fails on the current master branch because the expected string "The image you are pulling has been verified" does not appear.

This looks like it's an issue with running the test more than once.  Since change https://github.com/docker/docker/pull/10594 added the logic to only output the verification message if the image was actually downloaded, the test will fail the second time since the image already exists and is not downloaded again.

Here is the output I'm seeing (note the image is already present, so verified message is not shown):
--- FAIL: TestPullVerified (0.81s)
docker_cli_pull_test.go:71: pulling a verified image failed. expected: The image you are pulling has been verified
got: latest: Pulling from hello-world
511136ea3c5a: Already exists
31cbccb51277: Already exists
e45a5af57b00: Already exists
Status: Image is up to date for hello-world:latest
,
FAIL

This was caused by #11265, I will see if we can come up with an alternative way to test pulling verified images. hello-world is normally used for pull tests since it is the smallest image.

Would it be possible to rework the test such that it checks for the presence of the hello-world image and deletes it if it is already there?

My first attempt to fix is by changing the tag for the frozen image such that hello-world:latest is always pulled. I am uncomfortable with removing a local image which was placed there by the frozen script since any subsequent tests are making an assumption about its presence and version.

authors: dmcgowan swernli

repo: docker/docker