dill does not work in standalone mode, despite dill-compat (unable to save functions)

Nuitka version, full Python version, flavor, OS, etc. as output by this
command.

1.5.6
Commercial: None
Python: 3.9.16 (main, Mar  8 2023, 10:39:24) [MSC v.1916 64 bit (AMD64)]
Flavor: Anaconda Python
Executable: c:\users\alexw\ANACON~1\envs\NUITKA~1\python.exe
OS: Windows
Arch: x86_64
WindowsRelease: 10
Version C compiler: ~\AppData\Local\Nuitka\Nuitka\Cache\downloads\gcc\x86_64\11.3.0-14.0.3-10.0.0-msvcrt-r3\mingw64\bin\gcc.exe (gcc).

How did you install Nuitka and Python

pip, on Windows

The specific PyPI names and versions

dill==0.3.6
Dill does not appear to be working properly when compiled as --standalone or --onefile, when using dill-compat. If a function is serialized using dill, then it fails and crashes.
Here's a minimal test script which fails:
import dill

def testFunction():
	pass

with open("dillTest.txt", "wb") as f:
	dill.dump(testFunction, f)

(compiled using python -m nuitka dillTest.py --standalone --enable-plugin=dill-compat)
When the compiled exe is run, it fails with the following error:
Traceback (most recent call last):
  File "C:\DILLTE~1.DIS\dillTest.py", line 14, in <module>
  File "C:\DILLTE~1.DIS\dill\_dill.py", line 235, in dump
  File "C:\DILLTE~1.DIS\dill\_dill.py", line 394, in dump
  File "C:\DILLTE~1.DIS\pickle.py", line 487, in dump
  File "C:\DILLTE~1.DIS\dill\_dill.py", line 388, in save
  File "C:\DILLTE~1.DIS\pickle.py", line 560, in save
  File "C:\DILLTE~1.DIS\dill-postLoad.py", line 44, in save_compiled_function
AttributeError: module 'dill._dill' has no attribute 'stack'

I seem to recall, that in __main__ it may not have worked, but everywhere else?

This also doesn't work:

def dillTest():

	def testFunction():
		pass

	with open("dillTest.txt", "wb") as f:
		dill.dump(testFunction, f)

dillTest()

It might be necessary for the code to be in a separate module, and not the main program. But I am in no way sure of that and it's not the issue here.
The error you are facing is from different code. The post load Python code in createPostModuleLoadCode of NuitkaPluginDillWorkarounds seems to not work for latest dill. It provides a save_compiled_function that seems to want to use a stack from dill._dill and that seems to no longer match the internals of dill.
I kind of added a plan to revisit the dill plugin and add coverage for it. But for now, this will have to wait for others to pick it up (cannot be that hard if you poke dill internals how to save a standard function) and wait for me to get to it.

Thanks as always for the rapid response.
I thought it might help to try an earlier version of dill. Indeed it seems that this issue appeared in dill v3.5, as v3.4 and earlier don't have the same problem.
Unfortunately a different failure occurred (with dill v3.4, python 3.9, nuitka 1.5.6):
Traceback (most recent call last):
  File "C:\DILLTE~1.DIS\dillTest.py", line 3, in <module>
  File "C:\DILLTE~1.DIS\_dillModule.py", line 9, in dillTest
  File "C:\DILLTE~1.DIS\dill\_dill.py", line 276, in dump
  File "C:\DILLTE~1.DIS\dill\_dill.py", line 498, in dump
  File "C:\DILLTE~1.DIS\pickle.py", line 487, in dump
  File "C:\DILLTE~1.DIS\pickle.py", line 560, in save
  File "C:\DILLTE~1.DIS\dill-postLoad.py", line 69, in save_compiled_function

It occurred to me that this might be the main issue you were referring to, so I tried moving the dill code to a separate module, but this made no difference. In case it helps, here is the full code:
dillTest.py:
from dillModule import dillTest

dillTest()

dillModule.py:
import dill

def dillTest():

	def testFunction():
		pass

	with open("dillTest.txt", "wb") as f:
		dill.dump(testFunction, f)

It may also have been limited with nested functions. Maybe if the __qualname__ was used rather than the __name__ the lookup in the table would work for it. If you move it to the top level, I would expect it to maybe work, but then also closure variables would have to be done, etc.
I didn't know it works with files. I think I originally added it for usage with some sort of multiprocessing usage.

Hmm, you are correct that moving it to the top level works.
Unfortunately this doesn't work for my case where I am serializing quite a complex object which includes a lot of functions.
Is there any way I can make use of the qualname idea, or does that require making changes to the dill plugin source code? (I'd be happy to do that in principle, but not remotely qualified!)

I didn't know it works with files. I think I originally added it for usage with some sort of multiprocessing usage.

Yes, dill is amazing for saving the state of a program. It can save the state of pretty much all python objects, which makes for a fantastic save/load feature.

the MRE works nowadays, but i'm not sure if this issue should be closed, up to you.

Yes, lets close it then it then.


Cycle detection in undirected graph [Graph][C++]

I would like to add the program to cycle if present in an undirected graph.
Please assign this task to me @AdarshAddee

I would like to know more about this before assigning this task to you. Kindly explain me, I wanna know about as you know about this you can explain me better than others. I'm waiting for your response!!!
Thank you for your interest.

Sorry for the typo in the above request.
I will be adding a program to detect whether a cycle is present in an undirected graph or not. I will be using the DFS technique and for each node will check if it is already visited and if the particular node is not the parent of the current node, a cycle is present and will return true;


0.1.20

Codecov Report
Attention: 22 lines in your changes are missing coverage. Please review.

Comparison is base (05b59c1) 82.14% compared to head (0875be3) 82.05%.

Files
Patch %
Lines

easychart/colormaps/__init__.py
23.07%
10 Missing :warning:

easychart/__init__.py
60.00%
6 Missing :warning:

easychart/extensions/racechart/__init__.py
62.50%
6 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           latest       #7      +/-   ##
==========================================
- Coverage   82.14%   82.05%   -0.09%
==========================================
  Files          74       77       +3
  Lines        1417     1505      +88
==========================================
+ Hits         1164     1235      +71
- Misses        253      270      +17

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Breakpoints do not work with a react native setup in Visual Studio Code

Issue

Breakpoints don't work in VSCode. If you use a debugger statement, it will break at the correct place, and show the correct sources, so the sourcemaps are being loaded and parsed.

On every breakpoint VSCode reports:
breakpoint not ignored because generated code not found (source map problem?)
This issue is related to #209 - it is the same setup that works in webstorm.
Here are my config files.
launch.json
{
    "version": "0.2.0",
    "configurations": [
        {
            "type": "node",
            "request": "launch",
            "name": "Debug Jest",
            "program": "${workspaceRoot}/node_modules/jest/bin/jest.js",
            "args": ["--runInBand", "--no-cache"],
            "cwd": "${workspaceRoot}",
            "console": "integratedTerminal",
            "sourceMaps": true,
        }
    ]
}

Tsconfig:
{
    "compilerOptions": {
        "jsx": "react",
        "allowSyntheticDefaultImports": true,
        "target": "es2015",
        "moduleResolution": "node",
        "inlineSourceMap": true
    },
    "exclude": [
        "node_modules"
    ]
}

package.json>jest
	"jest": {
		"preset": "react-native",
		"transform": {
			"^.+\\.js$": "<rootDir>/node_modules/babel-jest",
			".(ts|tsx)": "<rootDir>/node_modules/ts-jest/preprocessor.js"
		},
		"testRegex": "(/__tests__/.*|\\.(test|spec))\\.(ts|tsx)$",
		"mapCoverage": true,
		"moduleFileExtensions": [
			"ts",
			"tsx",
			"js",
			"json"
		],
		"globals": {
			"__DEV__": true
		}
	}

.babelrc
{
	"presets": [
		"react-native"
	],
	"sourceMaps": "inline"
}

Expected behavior

I expect it to break at the correct places.

Link to a minimal repo that reproduces this issue

https://github.com/GeeWee/rn-typescript-example/tree/ts-loader-bug
Here, simply run the debug configuration. It will stop at the debugger; statement but will ignore any breakpoints.

 That's magical! Perhaps it has something to do with the new debugger protocol in node 8.
If so, we should definitely update the readme.

lol .. yeah. I was just setting it up to see if #306 fixed this but it seemed to work just fine even with the current version. If you can confirm this works for you as well, it will close one major issue

Yeah it works ! Damn. I wonder if this only works in node 8. Either way we should update the readme - this is awesome.

That should be simple - just gotta take one section out :)

@GeeWee Every issue will be solved by itself... It's just the matter of time.


The web wechat will be closed recently

Is there a plan for the PC version?

Yes, there has an unscheduled plan.
Please See #603 #400


IsNameValue Intention

What was the original intent for the IsNameValue property on IHubSpotModel? I couldn't find anywhere that is was being used.

It was used in the RequestDataConverter when mapping out the data properties:
foreach (var prop in allProps)
            {
                if (prop.HasIgnoreDataMemberAttribute()) { continue; }

                var propSerializedName = prop.GetPropSerializedName();
                if (prop.Name.Equals("RouteBasePath") || prop.Name.Equals("IsNameValue")) { continue; }

                // IF we have an complex type on the entity that we are trying to convert, let's NOT get the
                // string value of it, but simply pass the object along - it will be serialized later as JSON...
                var propValue = prop.GetValue(entity);
                var value = propValue.IsComplexType() ? propValue : propValue?.ToString();
                var item = new HubspotDataEntityProp
                {
                    Property = propSerializedName,
                    Value = value
                };

                if (entity.IsNameValue)
                {
                    item.Property = null;
                    item.Name = propSerializedName;
                }
                if (item.Value == null) { continue; }

                mapped.Properties.Add(item);

                if (batchMode && prop.GetPropSerializedName() == "email")
                {
                    mapped.email = value;
                }
            }

I think there are some properties that take that format but they might be legacy - I'd have to do a bit more digging to find out any more!

Okay, that's what I was thinking. I couldn't find any other references to it.


No easy way to reach Export profile button

When exporting a profile, a SR user has to navigate the whole list to reach the Export button. We should either move the button to the top or have a keybinding/command which does this.
cc @jooyoungseo

When exporting a profile, a SR user has to navigate the whole list to reach the Export button. We should either move the button to the top or have a keybinding/command which does this.

In my opinion a keybinding/command is good idea because I can activate from any place in the list.

@meganrogge Is not the tab should take to Export button when the complete tree is focused instead of focusing the check box?

@sandy081 I'm not following

correct - pressing tab when my screen looks like the above causes focus to be lost rather than going to the button.

Tab at this state is taking focus to the checkbox in the tree. IMO checkbox should be focused when the row is focused and you tab then.
@alexr00 Is it possible to change this behaviour?


fix(schemas): harden version_override definition

@mason-org/triage notify(Reason::NewPullRequest)
This comment ensures a notification is delivered to maintainers.
 🤖 This is an automated comment.    📖 Source code


new vulkan extensions added to latest windows amd drivers but not loaded by dxvk

Hi, mostly as the title says I have just installed latest windows amd drivers 24.2.1 as of Feb 26 and they include the vulkan extension VK_EXT_graphics_pipeline_library among others, after installing it and trying Aliens Fireteam Elite with dxvk 2.3 the log shows:
info:  VK_EXT_graphics_pipeline_library
info:    graphicsPipelineLibrary                : 0
any idea?
OS: Windows 11
DXVK: 2.3

AMD disables the extension specifically for DXVK because their implementation is broken at the moment.

This is now enabled in the 24.6.1 driver

OMG! Gonna test ASAP! thanks!


Add vJunos-router support

Modified the vJunos-switch template for vJunos-router

Modified Makefile to support vJunos-router regex.
Modified launch.py for smbios string for VMX type and lab license.
Modified launch.py class name for vjunos-router instead of vjunos-switch.

Builds successfully with 23.2R1.15. Can be used in Containerlab with kind vjunos-switch and built image.

thanks @tomncarter !


Correct filename case in #include directive

Incorrect capitalization of Arduino.h causes compilation to fail on filename case-sensitive operating systems like Linux.

I've edited SpeedyStepper.h line 37 to #include <Arduino.h> on my computer, and it now compiles on Raspberry Pi OS. Not tested further yet, or tried on another Linux distro.

I can confirm that the change described by JohnInWales also fixes a compile error on my system, Arduino IDE 1.8.15 on Linux Mint 20.1.

Sorry for the bad place to ask this, don't know how to ask the autors about an equation:
For calculating the next time step, an equation is used, this one: ramp_NextStepPeriod_InUS = ramp_NextStepPeriod_InUS *     (1.0 - acceleration_InStepsPerUSPerUS * ramp_NextStepPeriod_InUS * ramp_NextStepPeriod_InUS)
at the line 978-980 in src/SpeedyStepper.cpp, where does this equation come from? is it an approximation? It's for education purpose.
Best regards,
happylulux

It is corrected in:
https://github.com/Stan-Reifel/SpeedyStepper/commit/89e4b960792956fbe33a6141b8dabb352ca33e88
But for Arduino to notice the update to the library, the new version needs to be tagged.
Arduino's library system currently sees only the 1.0.0 release form before that commit.
https://github.com/Stan-Reifel/SpeedyStepper/releases/tag/1.0.0

Closing because @Stan-Reifel duplicated my work in https://github.com/Stan-Reifel/SpeedyStepper/commit/89e4b960792956fbe33a6141b8dabb352ca33e88
@Stan-Reifel it is frustrating to me that I took the time to submit this important fix to your project, but you then completely ignored it. I very well understand that we can't always find time to review pull requests in our projects, but that was clearly not the case here because you found the time to duplicate my efforts when you could have instead just merged my pull request. I recommend you make a stronger effort to monitor the contributions submitted by the community to your GitHub repositories. If you are not already watching the repository, you can get notifications for any pull requests received by clicking the "Watch" button on the repository home page:
https://github.com/Stan-Reifel/SpeedyStepper

Closing because @Stan-Reifel duplicated my work in 89e4b96
@Stan-Reifel it is frustrating to me that I took the time to submit this important fix to your project, but you then completely ignored it. I very well understand that we can't always find time to review pull requests in our projects, but that was clearly not the case here because you found the time to duplicate my efforts when you could have instead just merged my pull request. I recommend you make a stronger effort to monitor the contributions submitted by the community to your GitHub repositories. If you are not already watching the repository, you can get notifications for any pull requests received by clicking the "Watch" button on the repository home page:
https://github.com/Stan-Reifel/SpeedyStepper

It seems one can't raise an issue about the out-of-date release.  I didn't know you could disable issues.


xilem_html: Add an (ordered) VecMap for small n

A few benchmarks have shown, that for small n (roughly n < ~20, with small types < 50 bytes) a Vec is faster than a BTreeMap.
This seems to be the average case for html attributes.
Since it was a low hanging fruit to boldly copy the docs from the std::collections::BTreeMap, I have done so, it also implicitly adds doc tests for this (I guess it makes sense to add cargo test to the CI).
The VecMap is now used instead of the BTreeMap for the element attributes

I have rebased this, it's ready for review/merge.
One thing that I'm not super happy with, is that doc tests force pub for the used types in the crate (currently xilem_html::vecmap::VecMap and xilem_html::diff::Diff). I don't think this is too bad for the necessary exposed types (and they might even be helpful in other crates).

What if you rename diff_tree_maps to diff_kv_iterables and then make that function look like this:
pub fn diff_tree_maps<'a, II, K, V>(
    prev: II,
    next: II,
) -> impl Iterator<Item = Diff<&'a K, &'a V>> + 'a
where
    K: Ord + 'a,
    V: PartialEq + 'a,
    II: IntoIterator<Item = (&'a K, &'a V)> + 'a,
{
    DiffMapIterator {
        prev: prev.into_iter().peekable(),
        next: next.into_iter().peekable(),
    }
}

And then impl IntoIterator for VecMap?

What if you rename diff_tree_maps to diff_kv_iterables ... And then impl IntoIterator for VecMap?

I have used the possibility to add a member method to VecMap directly (since it's not possible without extension traits for BTreeMap). But as you say it certainly makes sense to generalize this function and reuse it for the member method (which is done by the latest commit).

I have used the possibility to add a member method to VecMap directly (since it's not possible without extension traits for BTreeMap). But as you say it certainly makes sense to generalize this function and reuse it for the member method (which is done by the latest commit).

I've got a headache and don't feel great, so I might be misunderstanding things ... but my function works with BTreeMap being passed in with no extension stuff needed. To pass in a VecMap, the only extra thing to implement would be IntoIterator for it which should be possible without much pain.
But either way ... I was just looking at how pluggable / extensible it could be so that changing types in the future or using the diff code elsewhere would be easier.

With a function pointer (fn(&'a (K, V)) -> (&'a K, &'a V)) the IntoIterator for &'a VecMap is possible (as the closure doesn't catch any other vars), not sure whether the optimizer is smart enough to optimize it similarly as with the iter() method (as this is a coercion, but I guess it is).
I've extracted the examples as tests, and removed the pub modifiers, this adds (unfortunately) another 100 LOC (but easy to review, since they're just the doc tests).

Will review within 8 hours, hopefully.


SMART on FHIR docs update

While using this tutorial we noticed it's missing information about the Admin Consent process.
This updates the gap, explains what is an admin consent, when to expect it, and how to approve it, in order for this tutorial to work.

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

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

@ginalee-dotcom I've applied edits for Microsoft style before merging. Could you check my revision and let me know if the PR is good to merge? Thanks.
https://github.com/MicrosoftDocs/azure-docs/pull/77760/commits/3c3d5d30fd9749e3285b46c7fa6b32c7995db5f5

@zxue @ginalee-dotcom See my proposed style revisions: https://github.com/MicrosoftDocs/azure-docs/pull/77760/commits/3c3d5d30fd9749e3285b46c7fa6b32c7995db5f5

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


script name

In {{cookiecutter.directory_name}}/README.md:
* run `python dump_data.py`
instead of
* run `download_and_transform.py`

@fsanzl Please feel free to add your code changes via pull requests

Changed and merged


Cast getSize() to string for Content-Length header

This probably fixes the issues with Zend/Diactoros mentioned here https://github.com/reactphp/http/issues/28#issuecomment-290763516. The exception is throwed since getSize() returns int instead of string (diactoros is very strict about this).
This fixed my issues with Diactoros responses.

LGTM, can you add a test for this? :+1:

As discussed shortly with @clue on IRC, we may not need tests for this, so can be merged if no other issues are raised.

Awesome, thank you 👍


Support for 16x2 and 20x4 LCD displays

HD44780U Data Sheet: http://www.adafruit.com/datasheets/HD44780.pdf

@rm-hull sent one your way: http://www.ebay.com/itm/191623809073

I've got a load of code for HD44780 LCDs at  https://github.com/DougieLawson/RaspberryPi/tree/master/Unified_LCD

@DougieLawson - the bare metal stuff looks interesting in that repo, does it boot up ok? A wee while ago I had the idea of writing a bare metal forth machine for the RPi (actually got quite far making it work on i386, but I ran out of steam, so its mainly just sat on a shelf bit-rotting these days)
@thijstriemstra - So looking at that HD44780 datasheet and the pic on ebay, looks like there are two main challenges here:

Implementing a parallel interface with software bitbanging, else using something like an MCP23017 chip like Dougie's implementation
Translating the Pillow ImageDraw canvas draw method for 'custom' graphics, and an overlay for text (similar to how the max7219 device works with its sevensegment wrapper)

@rm-hull I'm currently using the https://github.com/dbrgn/RPLCD library for this, but I'd like to use a single luma.core API. There are probably some clues in that library.

@rm-hull used to boot OK on a RPi1B, so it would probably run on a Zero. I haven't looked at that code since Jul 2014. There's lots of modern bare metal stuff (probably better than my hack) on https://github.com/ultibohub/Core & https://github.com/rsta2/circle

@thijstriemstra http://www.ebay.com/itm/191623809073 <- arrived

Cool!

just a thought, maybe it makes sense to draw a distinction between character-based LCDs (HT1621, HD44780, HPDL1414) and graphical LCDs in the codebase. They seem very different under the hood.

Yes HD44780 is possibly a step too far in its default character mode, but think they do have some GDRAM that could be equated to a drawable canvas. The HT1621 is only usable by wrapping it with the sevensegment class.

oh huh! AFAIK hd44780 only has 8 customizable characters. Maybe with some trickery, like changing them partway through a line. Like in the "racing the beam" days. ha!

@rm-hull so this is blocked by https://github.com/rm-hull/luma.core/issues/41?

Yes, would need to implement that in core first I think

I'm a bit late reaching out but wanted to share that I'm close to finishing the implementation of a solution to this as well as rm-hull/luma.core#41.  As soon as I finish cleaning up the code, documentation, and writing the test cases, I should be able to request a pull.
I've added the following classes...
luma.core.interface.serial:
parallel(pulse_time, gpio, RS,  E, PINS)# yes I appreciate the irony in the name
- Implements the bit-banging functionality that a RPi/HD44780 requires if using a bus style interface (supports 4 and 8 bit modes)
pcf8574(bus, port, address, pulse_time, backlight_enabled) # subclass of i2c
- Implements the bitbanging style communications needed for the PCF8574 i2c backpack
luma.core.virtual:
character(device, font, undefined) # implements a text interface for character displays similar to the sevensegment class
luma.lcd.device:
hd44780(serial_interface, width, height, framebuffer, default_table, undefined, exec_time, bitmode) # subclass of device

Implements a graphics style interface into the hd44780 (limited by the amount of CGRAM available which is typically 8 characters)
This can be surprisingly useful with careful screen design.  To leverage successfully...

Design your screens so that only eight unique 5x8 images are displayed at one time
Fill remaining screen regions with 5x8 images that come from the embedded font table
Make sure that alignment of the regions adheres to the 5x8 character structure of the display

Includes embedded font tables that replicate the A00 and A02 font tables that are common on HD44780 devices
Maps unicode values to the appropriate glyphs in each of the tables
Auto-assembles the font tables into a master font which allows all of the glyphs across the two font tables to be leveraged
Includes a text interface (implemented using the character class)

ws0010(serial_interface, width, height, rotate, framebuffer, exec_time, bitmode) # subclass of device

This is the controller that the Winstar WEH and WEG displays leverage which supports the HD44780 command-set but...
Can be driven as a full graphical display.  This class leverages that capability
Includes the four ws0010's font tables (English_Japanese, Western_European_I,  English_Russian, Western_European_II)
Maps unicode values to the appropriate glyphs in each of the tables
Auto-assembles the font tables into a master font which allows all of the glyphs across the four font tables to be leveraged

weh001602a(serial_interface, width, height, rotate, framebuffer, undefined, font, default_table, char_font, exec_time, bitmode)

The weh displays are intended for character displays
This wrapper class to ws0010 adds a text interface (using the character class)

If I'm heading off in the wrong direction let me know.  Otherwise, I'll try to get this wrapped up in the next week or so.

Actually, that sounds fantastic. My first question was going to be about doing something like the sevenseg interface, but you already pre-empted that!

Ok, I'll try to get it finished then!

As soon as I finish cleaning up the code, documentation, and writing the test cases, I should be able to request a pull.

 Don't hesitate to open a pull request early so we can provide feedback and/or contribute.

I've got a module placement question for you.  The code I'm working on supports the HD44780 compatible devices (LCD) as well as the Winstar ws0010 devices (OLED) which are largely HD44780 compatible but have enough differences to make them their own class but also leverage a couple of shared classes.
So the question is, does it make sense from a LUMA perspective for the Winstars to be part of the luma.lcd project?  That's where I've currently placed them so that I don't have to create dependencies between the luma.lcd and luma.oled projects.  Do you support that alignment or would you prefer the winstar devices be placed in the luma.oled project?

LUMA perspective for the Winstars to be part of the luma.lcd project?

Yes, +1

That's where I've currently placed them so that I don't have to create dependencies between the luma.lcd and luma.oled project

If there is any danger of creating dependencies between sub-projects: put it in luma.core.

If you look at https://luma-oled.readthedocs.io/en/latest/api-documentation.html you can see this hierarchy diagram:

Perhaps like @thijstriemstra says, put the base class in luma.core, maybe called something like luma.core.device.character_device ?
And then in luma.oled, luma.oled.device.ws0010 extends luma.core.device.character_device.
For luma.lcd, luma.lcd.device.hd44780 will need to inherit from both luma.core.device.character_device and luma.core.device.backlit_device.

I now regret not creating a single repo with all drivers regardless of type, not least because we could've just had a simpler namespace and less tangled releases, but then someone else came along and nicked the luma package name in pypi (https://pypi.org/project/luma/)
Hey ho...

but then someone else came along and nicked the luma package name in pypi (https://pypi.org/project/luma/)

Yea what a douchebag.

Ok, I'll restructure accordingly.

TLDR; Can I add pypi:cbor as a dependency to lcd.core?
One of the classes that my two drivers use is an embedded_font class.  Right now it is VERY specific to the two drivers so I am looking to generalize it a bit since it will be living in core now.  It requires that I store the binary font data for the displays which I am placing in const.py.  I am currently using the PIL font format which is limited to 256 glyphs per font so there are some ugly workarounds that I've leveraged to extend beyond this.  So, I'm creating my own font format that largely follows the PIL font but removes the limitation.  To do that, it would be good to have an easy way to store binary data and I came across the CBOR protocol which seems perfect for the purpose.  So the question is whether you would object to adding a dependency from luma.core to pypi:cbor.  Let me know if this is ok, otherwise, I'll just continue to use the PIL format.  Alternatively, if you have a recommendation on a way to convert bitmap data into Truetype fonts that would be another way to go.

Can I add pypi:cbor as a dependency to luma.core?

The last release for cbor was in 2016 and there are quite some issues: https://bitbucket.org/bodhisnarkva/cbor/issues?status=new&status=open so it seems unmaintained to me. I'm +0 on this..

Check out cbor2

cbor2 looks much better, even has a C extension and the maintainer is someone I worked with before.

This is fixed now.


Update css of reward.html

fixes issue : #1373

@Viveksati5143 attached with the issue, please check #1373

add a ss

Before change :

After change :


Specify report_status values

Asynchronous reports API does not have a specification of values that report_stats can be.
https://developers.pinterest.com/docs/api/v5/#operation/analytics/get_report
https://github.com/pinterest/api-quickstart/blob/main/python/src/async_report.py

Good point! I'll ask the Pinterest Development Platform technical documentation expert for their advice on the best way to address this need.

I discussed with the documentation expert and the development team, and we agree that we should provide a specification for the values of report_status. I opened a task for the team to do this work.
In the meantime, I found the complete list of status values in our API code...
class JobStatusValues(Enum):
    # All possible status values for bulk reporting jobs stored here
    DOES_NOT_EXIST = 'DOES_NOT_EXIST'
    FINISHED = 'FINISHED'
    IN_PROGRESS = 'IN_PROGRESS'
    EXPIRED = 'EXPIRED'
    FAILED = 'FAILED'
    CANCELLED = 'CANCELLED'

It will probably take us some time to grep the code to make sure that we understand the exact meaning of these values and release this information as part of the API specification.

Thanks a lot!


Space before first array/object member

When using comma first notation, it is nice to have the option of aligning all keys in array/object maps. This is easy to do by adding a token space and a space.
I have a prototype patch I would be interested in perhaps committing, however it likely needs cleanup/an additional command line flag.

I'd be happy to take a look at it.
You should feel free to start a PR.  At minimum before submitting for initial review, commit your change, and run make from the root of the project - it should report success.
For you change to be merged to master, you'll need to add tests, and port your change to to python.

I would love to see this since its the whole reason I use comma-first (https://gist.github.com/isaacs/357981). I would be grateful to just have the ability to exclude a newline after the opening brace or bracket of object literal / array.
Commented on an issue over on codeformatter sublime 3 plugin that I think will require support from upstream js-beautify -> https://github.com/akalongman/sublimetext-codeformatter/issues/160
Here is an example of the style I prefer.
var join = require("path").join
  , webpack = require("webpack")
  , ExtractTextPlugin = require("extract-text-webpack-plugin")

module.exports = function(options) {
  var loaders = [ { test: /\.json$/, loader: "json-loader" }
                , { test: /\.css$/, loader: ExtractTextPlugin.extract("style-loader", "css-loader") }
                , { test: /\.less$/, loader: ExtractTextPlugin.extract("style-loader", "css-loader!less-loader") }
                , { test: /\.woff(2)?(\?v=[0-9]\.[0-9]\.[0-9])?$/, loader: "url-loader?limit=10000&minetype=application/font-woff" }
                , { test: /\.(ttf|eot|svg)(\?v=[0-9]\.[0-9]\.[0-9])?$/, loader: "file-loader" }
                ]
  var plugins = [ new webpack.ProvidePlugin({ $: "jquery"
                                            , jQuery: "jquery"
                                            , "window.jQuery": "jquery"
                                            })
                , new webpack.IgnorePlugin(/dtrace-provider/i)
                , new ExtractTextPlugin("tix.css", { allChunks: true })
                ]

  if(options.minifyJs) {
    plugins.push(new webpack.optimize.DedupePlugin())
    plugins.push(new webpack.optimize.UglifyJsPlugin())
  }

  return  { cache: true
          , watch: options.watch
          , entry: [
            './client',
            './client/index.less'
          ]
          , output: { path: join(__dirname, 'srv', 'assets')
                    , publicPath: '/assets/'
                    , filename: 'tix.js'
                    /*, chunkFilename: "[chunkhash].js"*/
                    }
          , module: { loaders: loaders }
          , plugins: plugins
          }
}

I add my vote for this.
I just spent 10 minutes trying to fiddle with brace style options, thinking the existence of the comma-first setting meant it was possible to have some Haskell-style objects in my code. Sadly no...
Falling back to collapse, preserve-inline for now.


'The request requires user authentication'

Just gone to run a load of mobile device name updates and they all fail with a 401 error. MUT.log says 'The request requires user authentication'. Username and password for the Jamf Cloud instance are accepted at the initial login window for MUT.

I fixed it by allowing basic authentication in the JSS settings. MUT hasn't been updated to use bearer tokens for authentication, I guess?

Confirmed issue here as well,    but because of our required security,  we cannot allow basic authentication as a workaround.
i hope this gets resolved,   but i see MUT hasnt been updated in some time so growing less than hopeful.

Hi, getting the same issue here and we have disabled basic auth ready for when Jamf eventually turn it off.
Any further update for this ?

Hi all, please see Issue 82, version 6.2 of the MUT will use bearer tokens.


Chat upsert does not provide sent/received message data

I've done console.log but the sent/received message data doesn't appear
Code:
socket.ev.on('chats.upsert', (data) => {
    console.log(JSON.stringify(["UPSERT", data]))
})

Response:
["UPSERT",[{"id":"628xx@s.whatsapp.net","conversationTimestamp":167xxx,"unreadCount":1}]]

It's the default behavior of baileys event emitter and i couldn't do anything about it


'Respect EXIF orientation' option at source level

The changes in issues #107, #332, and #333 would ensure that we consistently ignore image EXIF orientation everywhere. That's certainly respectable, but then we might ask: would any CoralNet users want to display their benthic images with EXIF orientation respected? After all, browsers are making EXIF-respecting the default behavior for webpage images, and surely they must have a decent reason to do so.
So that might point to a source-level 'Respect EXIF orientation' option, which can be turned on or off. For simplicity, I think such an option should deal purely with how things are displayed, without changing how essential data (original image files and point positions) is stored.
The option would affect:

The annotation tool image and points display. This means the image would either be rotated or not, and the point positions have to be 'rotated' or not as well. The point-rotation logic can either happen on the Python side or the Javascript side. Since the image has to be downloaded on the Javascript side anyway, it would probably be more performant to let the Javascript detect the EXIF and do the point rotation.

The annotation area edit UI, where you manually edit the annotation area for a specific image. The image would either be rotated or not, and the draggable area box would either be rotated or not.

Thumbnail displays everywhere else in the source, including Browse Images and perhaps even Browse Patches. This can either be done by deleting all generated thumbnails when the source-level option is changed, or by encoding EXIF orientation metadata into the thumbnails (which feels icky to me for some reason, but I guess it would be valid).

Points and annotation areas would still be stored in unrotated coordinates.
It gets trickier when we display annotation areas as text, or export point positions. The simplest, most obvious thing to do is to just ignore EXIF orientation for those purposes. But I can't help but wonder if the dissonance is just too much when the annotation area edit page has fields labeled "Left boundary X" and "Top boundary Y", and visually they are really "Bottom boundary Y" and "Left boundary X" respectively since the image is EXIF-rotated 90 degrees.
Suppose we just changed the field names on that annotation area edit page, and left everything else alone. Then you set the area as X 100-900 and Y 300-700, but come back to the Image Details page and it says that you set the area as X 300-700 and Y 100-900. Okay, so let's display that Image Details text in rotated coordinates too. But the Image Details page also shows the source-default annotation area as X 30-70% and Y 10-90%, and it's strange to display this source default as different coordinates for different images... you get the idea. If we accommodate one thing, then the dissonance just seems to move somewhere else.
Overall, unless we come up with a very well thought out scheme, I might argue that the confusion about which way is X, which way is Y, which ways are rows/columns going, etc. would make turning on the EXIF orientation option more trouble than it's worth. But I could be missing something. I guess we'll see if people request this and give good reasons for it. Until then, it seems like low priority to me.

For future reference, here is a sample image with non-default EXIF (90 degree rotation): https://coralnet.ucsd.edu/image/12849/view/


Remove: Discord.Discord.Canary version 0.0.298

## .\Tools\YamlCreate.ps1 -SkipPRCheck -Mode 5 -PackageIdentifier Discord.Discord.Canary -PackageVersion 0.0.298

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.0 schema?

Microsoft Reviewers: Open in CodeFlow

/azp run


abolished mitotic spindle elongation in anaphase B

PMID:25375240
def
A cellular process phenotype  in which mitotic spindle elongation  (spindle extension) does not occur in anaphase B. This  is indicated by the presence of the short 2 micron spindle (blob spindle).
Parents
FYPO:0003829 | abolished mitotic spindle elongation
FYPO:0000732 | short mitotic spindle
note
FYPO:0003829 | abolished mitotic spindle elongation should have a soft check for "do not annotate" since we should be abe to say when (i.e which spindle elongation, assembly or extension)

abolished mitotic spindle elongation during prophase FYPO:0006173
abolished mitotic spindle elongation during anaphase B FYPO:0006174
(you'll need both if you want to disallow annotation to FYPO:0003829)
edit file: 8aa8e2d9f153e0a57b0d4d62cdeebcfbe68024a7
release: 36712382f085093b31d1d1e73ad8414c05c15b7b


increased label.value field size

https://github.com/rancher/rancher/issues/8168
@ibuildthecloud @LLParse apparently k8s puts a lot of data in stateful set pod container's label. Increased the label.value size in cattle db to mediumtext to fix it.

64Kb should certainly be enough... @vincent99 is there any problem with doing this?

mediumtext is 64mb

Whoops, was looking at mysql text (64kb). mysql mediumtext appears to be 16mb

Just going w/ truncating in https://github.com/rancher/cattle/pull/2668.  The main reason for these tables is indexing the values.  So we really aren't too concerned with search for labels longer than 4k.


new search form ui

created the search form ui

Is there a way for me to view this in the browser?

deployed to go/BAITproject


Document how to scope providers

Hi @rrousselGit , is there a way to scope a provider for all "upcoming" widgets in the widget tree?
I tried that using ProviderScope and overriding the provider, but it only worked for one widget, and the screen after that was not reading the overridden value.
By "upcoming" I meant widgets/screens that are pushed on top of the current one.

In 2.0, there's a "parent" parameter on ProviderScope
When pushing new routes, make sure that the new route uses a ProviderScope that specifies "parent", where "parent" is the route that pushes this new route.

There is technically already a page on the topic of scoping


Feature request: Per room language settings.

'In reply to' is correctly translated to German 'Als Antwort auf' but jumping between the languages is disturbing during reading the messages in this room. It would be great if the language of the notifications is selectable per room.
From RW in #riot-android:matrix.org

Hi @krombel , I do not really understand the reason why you're asking this

As I mentioned that got requested in #riot-android. I just forwarded it.


Make sure JSMock uses right casing, e.g. JsRuntimeMock to JSRuntimeMock

To align with Blazor and IJSRuntime case naming, all classes and methods needs to be updated to match the casing in Blazor, i.e. using a capital S in JS.

I'd like to contribute.
Since you tagged this as a good first issue and seems quite straight forward i'd like to try if i may?
now you mention all class and method names, what about variables? eg.
var mockJs = Services.AddMockJsRuntime();

to
var mockJS = Services.AddMockJSRuntime();

My first thought was it should but i just want to verify whether you agree
Hope i can be of use.

Awesome @yourilima, thanks. And yes, do change variables where it makes sense as well.

Due to the way case sensitivity is handled differently across platforms and systems i first removed the files that required renaming and then added them again with the refactorization.
I'll try to test this on different systems first before i create a pull request.
I also updated the comments and strings according to the new naming.
What's left to do is updating the documentation. But here i think we could end up either with some conflicts or quite a bit is not present in the dev branch and would not come in this change.
What is the strategy here?
Is this a different pull request or should i take that in too and you merge the master branch to the dev branch before i can continue there?

Due to the way case sensitivity is handled differently across platforms and systems i first removed the files that required renaming and then added them again with the refactorization.
I'll try to test this on different systems first before i create a pull request.

Sounds like a good approach.

What is the strategy here?
Is this a different pull request or should i take that in too and you merge the master branch to the dev branch before i can continue there?

Just skip the documentation alone. When I get a little extra time, I want to get all the sections in the docs done, including those on JSInterop.


404 error page when connecting to AWS ec2

Expected Behaviour

After configuration end start up the docker. The first page should be where you have to log
Current Behaviour

Page 404 error
Possible Solution

I cut the section in cvat_proxy/conf.d/cvat.conf
to the form:
server {
    listen       80;
    server_name  ${CVAT_HOST};

    proxy_pass_header       X-CSRFToken;
    proxy_set_header        Host $http_host;
    proxy_pass_header       Set-Cookie;

    location ~* /api/.*|git/.*|opencv/.*|analytics/.*|static/.*|admin(?:/(.*))?.*|documentation/.*|django-rq(?:/(.*))? {
        proxy_pass              http://cvat:8080;
    }

    location / {
        proxy_pass              http://cvat_ui;
    }
}

Steps to Reproduce (for bugs)

all points from the installation guide with AWS extension(docker-compose.overrife.yml)
Your Environment

AWS EC2 t2.medium instance

First of all make sure that you configured CVAT_HOST as described here https://openvinotoolkit.github.io/cvat/docs/for-users/installation/#advanced-settings
If it will not help please attach as files logs from containers: docker logs cvat_proxy > proxy.log and docker logs cvat > cvat.log

@azhavoro I am sure that, the configuration was good. I wrote the AWS IP public address to the CVAT_HOST section configuration.
The first main page which was showing up was the 404 error.
After solution described above, I was able to get the right view after connection.

What did you write in CVAT_HOST instead of AWS IP public address, I have installed cvat version 1.4, and I'm still not able to connect to the port 8080, please help

What did you write in CVAT_HOST instead of AWS IP public address, I have installed cvat version 1.4, and I'm still not able to connect to the port 8080, please help

Most likely port 8080 is closed for inbound connections, you should open it in the AWS console.

@azhavoro I am sure that, the configuration was good. I wrote the AWS IP public address to the CVAT_HOST section configuration.
The first main page which was showing up was the 404 error.
After solution described above, I was able to get the right view after connection.

So, I close the issue


Log more information about unhandled incoming RPCs

@keybase/react-hackers
This change would make these two bugs trivial to triage:
https://keybase.atlassian.net/browse/DESKTOP-1408
https://keybase.atlassian.net/browse/DESKTOP-1476

:mag_right: The commits 5832a68ae3d5...2b3bf5f85b1c introduced some visual changes on win32:

Changed: unlock_folders-Single (before) (after)

Changed: Profile-Changed_Scrolled (before) (after)

👍


SOLR-17146: Add DelegatingBackupRepository and alternative checksum verification

Description at https://issues.apache.org/jira/browse/SOLR-17146
1- Add an optional initialization argument to disable the checksum verification (enabled by default) in each BackupRepository implementation.
2- Introduce an abstract DelegatingBackupDirectory to extend when we need to perform different/additional logic on top of a delegate BackupDirectory. For example, verify the checksum differently. Also, the DelegatingBackupDirectory could inject some initialization arguments when the BackupRepositoryFactory creates its delegate.
Also make the existing TrackingBackupRepository (for tests) extend DelegatingBackupDirectory.
3- Add a test case to AbstractBackupRepositoryTest to ensure all implementations support disabling the checksum verification. And add a test for LocalFileSystemDirectory (not tested currently) that also extends AbstractBackupRepositoryTest.

Thanks @gerlowskija for the review!

Hi there! I'm wondering how this feature can be disabled in a collection managed by the Solr operator @HoustonPutman? Since the current Solr operator does not have this initialization parameter in SolrCloud.spec.backupRepositories, my first guess is to rely on a solr.xml file with the necessary backup definitions and use IAM in the SolrCloud object, right?


Expose optional "anisotropic" parameter to the texture funtions in OSL.

This gives explicit control to the amount of anisotropy.  This consol
existed in the underlying OpenImageIO texture lookup options, but never
was exposed to direct control by the OSL shaders.

I think this one slipped through the cracks?
LGTM :)

Is this still something we want to integrate? Or should we drop it?


Improve css preprocessing

Followup to #2610, #2700, #2733
@flaiker very welcome to review :)
Goals:

make css preprocessing more abstract, less explicit
improve debugging (copy/paste HTML from system required non-trivial css links fixing )

Solution:

introducing zif_abapgit_gui_html_processor and "default" zcl_abapgit_gui_html_processor which is

triggered prior to cache_asset in gui->render
finds css links in html head
comments them out
collects urls from them
runs css processor over them
integrates the result into html

the instance of processor is created in factory, external to GUI, and, in particular, populated with css links to preserve unchanged
thus pages should not care about proper preprocess css naming as well as GUI itself
gui->cache_asset separated into an interface, the code didn't change - it is SE80 reorderings. There are some ideas how to reuse the interface in future + to isolate critical methods like free ...
minor

css processor reuse get_text_asset. Otherwise the code didn't change - it is SE80 reorderings. But some changes to UTs: for get_text_asset and also removed asset_man double - the existing class is abstract enough to be used in tests
improvements to asset_man UTs
there are some indent alignments I didn't do to make the commit clearer, can do before the merge or separately

P.S. will probably add some doc in a separate PR

result sample
  <head>
    <meta http-equiv="content-type" content="text/html; charset=utf-8">
    <meta http-equiv="X-UA-Compatible" content="IE=11,10,9,8" />
    <title>abapGit</title>
    <link rel="stylesheet" type="text/css" href="css/common.css">
    <link rel="stylesheet" type="text/css" href="css/ag-icons.css">
    <!-- by AG HTML preprocessor <link rel="stylesheet" type="text/css" href="css/theme-default.css">-->
    <!-- by AG HTML preprocessor <link rel="stylesheet" type="text/css" href="css/theme-belize-blue.css">-->
    <script type="text/javascript" src="js/common.js"></script>
    <style>.icon.large { font-size: inherit }</style>
    <!-- by AG HTML preprocessor --><link rel="stylesheet" type="text/css" href="css/bundle.css">
  </head>

How exactly does this improve debugging?

Well, not that much, but ... the point is that the original CSS links remain in the html commented (above was an example). So it is easy to comment the "bundle" and uncomment "originals" and it works. No need to copy/paste or remember names. I prefer to save all the files to the PC and edit them in a good editor and view in chrome (although IEChooser is cool indeed). I have a tool which simplifies file update so no manual SMW0 :)

CSS preprocessing unnecessarily happened on every render

Yeap, it is the same. Good point, didn't think about this honestly. But imho it is not very big issue. It does not affect performance a lot from what I see, besides, what if specific page will need some specific additional css? Option is: some kind of global variable to hold the last set of css urls built in the bundle ... or ... maybe create the name of the bundle in some predictable and unique way like concatenation of css urls or some hash ( 'css/theme.css/css/theme-belize.css' or 'css/{ sha1('css/theme.css/css/theme-belize.css') }' ) and then remembering all the cached assets in GUI itself (and automatically prevent duplicates to load). What do you think ?


Issue in Add inventory schema

add inventory schema needs to be fixed .
Error: ADD_INVENTORY_HISTORY not found

Added inventory history schema reducer to combined reducers.
Please refer to: https://github.com/prathyu99/CSC510_Group25_Project1/commit/d685e6d7d99268b57fa94a10e1415ab6cb568011
Issued fixed successfully


Fix translations

Building breaks because of missing and/or extra translations:
:app:lintVitalRelease/home/krt/src/fdroiddata/build/com.majeur.applicationsinfo/app/src/main/res/values/strings.xml:33: Error: "declared_premission" is not translated in "pt-BR" (Portuguese: Brazil) [MissingTranslation]
    <string name="declared_premission">Permissions</string>
            ~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/krt/src/fdroiddata/build/com.majeur.applicationsinfo/app/src/main/res/values/strings.xml:73: Error: "sort" is not translated in "pt-BR" (Portuguese: Brazil) [MissingTranslation]
    <string name="sort">Sort</string>
            ~~~~~~~~~~~

   Explanation for issues of type "MissingTranslation":
   If an application has more than one locale, then all the strings declared
   in one language should also be translated in all other languages.

   If the string should not be translated, you can add the attribute
   translatable="false" on the <string> element, or you can define all your
   non-translatable strings in a resource file called donottranslate.xml. Or,
   you can ignore the issue with a tools:ignore="MissingTranslation"
   attribute.

   By default this detector allows regions of a language to just provide a
   subset of the strings and fall back to the standard language strings. You
   can require all regions to provide a full translation by setting the
   environment variable ANDROID_LINT_COMPLETE_REGIONS.

   You can tell lint (and other tools) which language is the default language
   in your res/values/ folder by specifying tools:locale="languageCode" for
   the root <resources> element in your resource file. (The tools prefix
   refers to the namespace declaration http://schemas.android.com/tools.)

/home/krt/src/fdroiddata/build/com.majeur.applicationsinfo/app/src/main/res/values-pt-rBR/strings.xml:33: Error: "declared_permission" is translated here but not found in default locale [ExtraTranslation]
    <string name="declared_permission">Permissões</string>
            ~~~~~~~~~~~~~~~~~~~~~~~~~~

   Explanation for issues of type "ExtraTranslation":
   If a string appears in a specific language translation file, but there is
   no corresponding string in the default locale, then this string is probably
   unused. (It's technically possible that your application is only intended
   to run in a specific locale, but it's still a good idea to provide a
   fallback.).

   Note that these strings can lead to crashes if the string is looked up on
   any locale not providing a translation, so it's important to clean them
   up.

3 errors, 0 warnings
 FAILED

Build passed for me, I released it on Playstore. I'll look into this as soon as possible.  Thanks

Since this issue seems stalled, I fixed the build on our side for now:
echo -e "android { lintOptions { disable 'MissingTranslation'\n\ndisable 'ExtraTranslation' }}" >> app/build.gradle

However, lint issues should always be resolved (or disabled) by upstream: you know best when its okay to disable warnings, most of the time they are there for a reason.


Added the Code and Image for Chessboard Pattern

Added the Code and Image for Chessboard Pattern in CPP

@Crazycoder122 I'm closing the PR as it is not fulfilling the requirement...you can send a PR again after maintaining all the steps and eligibility criteria for a valid PR...


Perform fire on reset after doing no-ops, make fire_on_reset configurable

Fire on reset should probably happen after doing the no-ops, not before. Firing and then doing no-ops for 30 steps is probably harmful in some cases. OpenAI baselines follow this order as well: https://github.com/openai/baselines/blob/8c2aea2addc9f3ba36d4a0c937e6a2d09830afc7/baselines/ppo1/run_atari.py
I have also made fire_on_reset configurable since repositories that reproduce DeepMind numbers don't use it (dopamine and @Kaixhin's rainbow). Setting it to False is probably a better default but I didn't change that yet.
Edit: Switched to False by default at Kai's suggestion, my own experiments with Data efficient rainbow also suggest that using FireOnReset hurts.

Codecov Report

Merging #158 into master will decrease coverage by 0.14%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #158      +/-   ##
==========================================
- Coverage   22.53%   22.38%   -0.15%
==========================================
  Files         128      128
  Lines        8002     8005       +3
==========================================
- Hits         1803     1792      -11
- Misses       6199     6213      +14

Flag
Coverage Δ

#unittests
22.38% <100.00%> (-0.15%)
:arrow_down:

Impacted Files
Coverage Δ

rlpyt/envs/atari/atari_env.py
83.87% <100.00%> (+0.39%)
:arrow_up:

rlpyt/utils/buffer.py
50.00% <0.00%> (-10.00%)
:arrow_down:

rlpyt/runners/minibatch_rl.py
42.78% <0.00%> (-1.00%)
:arrow_down:

Continue to review full report at Codecov.

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

Check this https://github.com/openai/baselines/issues/240 - I confirmed with DM that they don't use it.

Thanks a lot Kai, my own experiments with DER suggest not using fire on reset is better as well! While we have you here, could you clarify the following settings as well?

Did you find clipping the grad norm not useful? I saw your repo doesn't use it.
What's the correct exploration strategy to use during eval with NoisyNets? Does DM still use a small epsilon?
When prioriotizing with C51, is it useful to use cross-entropy over KL? I saw both dopamine and your repo use CE, but the Rainbow paper mentions KL.

I think it's not mentioned in the Rainbow paper so presume unused?
Evaluation is ε-greedy with ε = 0.001, but no noise in the noisy layers.
The original distributional RL paper's algorithm specifies the cross-entropy loss.

Thanks, super helpful! For 1, I was just looking at the DER paper, so maybe it only helps there.

Oh - that specifies that it was used in both. I found the DER paper was useful for specifying hyperparameters, so I'll add it in to my repo...

Good catch! If you want to be super consistent, both version don't seem to be using Dueling as well.

Dueling is one of the main components of Rainbow (both)?

I don't think so.
From the Rainbow paper:

In aggregate, we did not observe a significant difference when removing the dueling network from the full Rainbow.

And, the DER paper mentions both variants:

Update: Distributional Double Q

which I assume implies not using dueling

I see, that might be the case!

Confirmed with Matteo Hessel that gradient clipping is used in Rainbow.

OK looks good, thanks for digging into the details!
I thought fire was needed in some games to start the action after a lost life?  I guess you're right tho on Breakout, if you press fire, then wait up to 30 steps, the ball might have already gone too low to reach....could be why I'm getting low scores in that game sometimes hahah hmmm...

@astooke and everyone else here, I am just curious, but are there benchmarks on Breakout with and without this change, to see if reward is better? I also notice (at least anecdotally) that before this pull request, my Breakout scores are OK sometimes but can be lower than expected. I have not tried with this change.

@Kaixhin
From your comment above

I think it's not mentioned in the Rainbow paper so presume unused?
Evaluation is ε-greedy with ε = 0.001, but no noise in the noisy layers.
The original distributional RL paper's algorithm specifies the cross-entropy loss.

Can you please share the source of _2. Evaluation is ε-greedy with ε = 0.001, but no noise in the noisy layers.
I thought the Rainbow Paper (https://arxiv.org/pdf/1710.02298.pdf) on page 4 says to set ε = 0.

@rfali the part you highlighted from the Rainbow paper seems to refer to training settings, rather than evaluation settings. Unfortunately this bit of info was a bit hard to find, and I'm afraid I can't remember where I picked it up (it's been several years since I worked on Atari), so if you do find a primary source that says otherwise you should trust that instead.


Location RealTime not work

Description
[Description of the bug, When Issue Happens]
Environment

Wppconnect-server version(s): 2.0.0
OS: Windows 10
Node version: 19.9
WhatsApp version: ?
MultiDevice (BETA): yes
config.json:
repository default!

Steps to Reproduce

[First Step]
Connect in a session and ask a user to send a location in REAL TIME!

[Second Step]
Accompanying the console it registers the onLiveLocation which should send the location information in real time! This information ends up going only from the first location through "onMenssage" and in some cases comes 1 new webhook with "location" type! However, the location is not updated at any other time!

Log Output
Console Log

WebHook 1

This resource was dropped by WhatsApp Web MultiDevice, but I am studing a way to do that, but without support


Added a link to help learn what confidential and Public clients are

Added a link to help learn what confidential and Public clients are from developer feedback

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

@kalyankrishna1 One benefit of working in the private repo is the PRMerger bot that would have auto-merged this PR after your #sign-off.
:star: Auto-merge in private repo - no waiting for PR reviewer or doc author to review and approve if PR meets certain criteria (% changed lines, no images, ...):
PR criteria required for auto-merge by PRMerger (MSFT-internal)


old

[ ] Extend the country list in the JS file
[ ] Add this functionality into the front end
[ ] Ensure it has integrated properly
[ ] Ensure the currency conversions are correct

This isnt needed anymore. Mark as closed


access rights to my OS

should I be worried that there is a ⚠ where my Library is?
 aichat --info
model               openai:gpt-3.5-turbo
temperature         0.2
dry_run             false
save                true
highlight           true
light_theme         false
wrap                no
wrap_code           false
auto_copy           false
keybindings         emacs
prelude             -
config_file         /Users/owner/Library/Application Support/aichat/config.yaml
roles_file          /Users/owner/Library/Application Support/aichat/roles.yaml
messages_file       /Users/owner/Library/Application Support/aichat/messages.md
sessions_dir        /Users/owner/Library/Application Support/aichat/sessions ⚠

~/Documents/GitHub/intelligence-record main !1                                               09:47:30 AM
❯

❯ aichat --version
aichat 0.11.0
❯ aichat -h
Use GPT-4(V), LocalAI and other LLMs in the terminal.

Usage: aichat [OPTIONS] [TEXT]...

Arguments:
  [TEXT]...  Input text

Options:
  -m, --model <MODEL>        Choose a LLM model
  -r, --role <ROLE>          Choose a role
  -s, --session [<SESSION>]  Create or reuse a session
  -f, --file <FILE>...       Attach files to the message to be sent
  -H, --no-highlight         Disable syntax highlighting
  -S, --no-stream            No stream output
  -w, --wrap <WRAP>          Specify the text-wrapping mode (no, auto, <max-width>)
      --light-theme          Use light theme
      --dry-run              Run in dry run mode
      --info                 Print related information
      --list-models          List all available models
      --list-roles           List all available roles
      --list-sessions        List all available sessions
  -h, --help                 Print help
  -V, --version              Print version
❯ aichat --info
model               openai:gpt-3.5-turbo
temperature         0.2
dry_run             false
save                true
highlight           true
light_theme         false
wrap                no
wrap_code           false
auto_copy           false
keybindings         emacs
prelude             -
config_file         /Users/owner/Library/Application Support/aichat/config.yaml
roles_file          /Users/owner/Library/Application Support/aichat/roles.yaml
messages_file       /Users/owner/Library/Application Support/aichat/messages.md
sessions_dir        /Users/owner/Library/Application Support/aichat/sessions ⚠

~/Documents/GitHub/intelligence-record main !1                                               09:47:30 AM
❯

Your issue may already be reported!
Please search on the issue tracker before creating one.
Describe the bug
A clear and concise description of what the bug is.
If the problem is related to dynamic response data, you should see if it can be reproduced through --dry-run.
To Reproduce
Steps to reproduce the behavior, including any relevant code snippets.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots/Logs
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

os version: [e.g. Ubuntu 20.04]
aichat version: [e.g. 0.9.0]
terminal version: [e.g. GNOME Terminal 3.44.0]

Additional context
Add any other context about the problem here.

⚠ indicates that the folder currently does not exist.
After you create and save a session, ⚠ will disappear.


No API docs for nursery.

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Please tag this pr with at least one of these labels to make our release process faster: BREAKING, new feature, bug fix, other change, dev setup

🚀 bad boy nursery, no api docs for you


How to create/removal specify custom host without docker run container

I need to gen and auto renew ssl certificate for some domains. How to create/removal and force specify custom host without docker run container(not use LETSENCRYPT_HOST env).

Hi. You can achieve this using the standalone certificates feature.

Hi. You can achieve this using the standalone certificates feature.

Oh, thanks for your answer. But sorry but i don't know how to use it.
$ docker run --detach \     --name nginx-proxy-acme \     --volumes-from nginx-proxy \     --volume /var/run/docker.sock:/var/run/docker.sock:ro \     --volume acme:/etc/acme.sh \     --volume /path/to/your/config_file:/app/letsencrypt_user_data:ro \     nginxproxy/acme-companion
What's configuration file(file extension)? Can you explain for configuration file and how to create it.

Basically you have to create a configuration file that mimic the structure of the internal letsencrypt_data file and mount it inside your container as /app/letsencrypt_user_data.
Let say you want to get a certificate for example.yourdomain.tld

Choose a unique identifier for this certificate, let's say certificate_example_yourdomain. This identifier will be repeated on the variables names contained in the user configuration file to let the service bash script know they're to be used together. Letters, numbers and underscore only, no space and no hyphen.
Create your config file on your local host, let's say it's is named /home/quangnv13/letsencrypt_user_data. The minimal amount of info it should contain is :

# bash array containing the unique identifier(s) you chose
LETSENCRYPT_STANDALONE_CERTS=('certificate_example_yourdomain')
# bash array containing the domains to be covered your certificate identified by certificate_example_yourdomain
LETSENCRYPT_certificate_example_yourdomain_HOST=('example.yourdomain.tld')

Create the acme-companion container with /home/quangnv13/letsencrypt_user_data mounted to /app/letsencrypt_user_data:

$ docker run --detach \
--name nginx-proxy-acme \
--volumes-from nginx-proxy \
--volume /var/run/docker.sock:/var/run/docker.sock:ro \
--volume acme:/etc/acme.sh \
--volume /home/quangnv13/letsencrypt_user_data:/app/letsencrypt_user_data:ro \
nginxproxy/acme-companion

And you're done, if example.yourdomain.tld correctly resolve to your host and everything else is correctly configured, you should obtain your certificate and it will be available along the others in /etc/nginx/certs
To understand the unique identifier stuff, let's say you now want to get another multi domain certificate that covers 1.yourdomain.tld and 2.yourdomain.tld.
The minimal configuration inside /home/quangnv13/letsencrypt_user_data will look like this:
# bash array containing the unique identifier(s) you chose
LETSENCRYPT_STANDALONE_CERTS=('certificate_example_yourdomain' 'someuniqueidentifieryouchose')
# bash array containing the domains to be covered your certificate identified by certificate_example_yourdomain
LETSENCRYPT_certificate_example_yourdomain_HOST=('example.yourdomain.tld')
# bash array containing the domains to be covered your certificate identified by someuniqueidentifieryouchose
LETSENCRYPT_someuniqueidentifieryouchose_HOST=('1.yourdomain.tld' '2.yourdomain.tld')

The unique identifier can be anything you want, it just has to be unique in the context of this configuration file you're creating and be valid as part of a bash variable name. Again letters, numbers and underscore only.

Basically you have to create a configuration file that mimic the structure of the internal letsencrypt_data file and mount it inside your container as /app/letsencrypt_user_data.
Let say you want to get a certificate for example.yourdomain.tld
1. Choose a unique identifier for this certificate, let's say `certificate_example_yourdomain`. This identifier will be repeated on the variables names contained in the user configuration file to let the service bash script know they're to be used together. Letters, numbers and underscore only, no space and no hyphen.

2. Create your config file on your local host, let's say it's is named `/home/quangnv13/letsencrypt_user_data`. The minimal amount of info it should contain is :

# bash array containing the unique identifier(s) you chose
LETSENCRYPT_STANDALONE_CERTS=('certificate_example_yourdomain')
# bash array containing the domain(s) to be covered by your certificate identified by certificate_example_yourdomain
LETSENCRYPT_certificate_example_yourdomain_HOST=('example.yourdomain.tld')

The comments are obviously just for explanation purpose and not required.
1. Create the **acme-companion** container with `/home/quangnv13/letsencrypt_user_data` mounted to `/app/letsencrypt_user_data`:

$ docker run --detach \
--name nginx-proxy-acme \
--volumes-from nginx-proxy \
--volume /var/run/docker.sock:/var/run/docker.sock:ro \
--volume acme:/etc/acme.sh \
--volume /home/quangnv13/letsencrypt_user_data:/app/letsencrypt_user_data:ro \
nginxproxy/acme-companion

And you're done, if example.yourdomain.tld correctly resolve to your host and everything else is correctly configured, you should obtain your certificate and it will be available along the others in /etc/nginx/certs
To understand the unique identifier stuff, let's say you now want to get another multi domain certificate that covers 1.yourdomain.tld and 2.yourdomain.tld.
The minimal configuration inside /home/quangnv13/letsencrypt_user_data will look like this:
# bash array containing the unique identifier(s) you chose
LETSENCRYPT_STANDALONE_CERTS=('certificate_example_yourdomain' 'someuniqueidentifieryouchose')
# bash array containing the domain(s) to be covered by your certificate identified by certificate_example_yourdomain
LETSENCRYPT_certificate_example_yourdomain_HOST=('example.yourdomain.tld')
# bash array containing the domain(s) to be covered by your certificate identified by someuniqueidentifieryouchose
LETSENCRYPT_someuniqueidentifieryouchose_HOST=('1.yourdomain.tld' '2.yourdomain.tld')

The unique identifier can be anything you want, it just has to be unique in the context of this configuration file you're creating and be valid as part of a bash variable name. Again letters, numbers and underscore only.
edit : the file extension does not matter, what matter is that its is mounted as /app/letsencrypt_user_data inside the container and that the content are valid bash variables.

Now i'm using image: jrcs/letsencrypt-nginx-proxy-companion:2.0.2. If i resetup my container use --volume acme:/etc/acme.sh \ --volume /home/quangnv13/letsencrypt_user_data:/app/letsencrypt_user_data:ro \ is it ok?

Now i'm using image: jrcs/letsencrypt-nginx-proxy-companion:2.0.2. If i resetup my container use --volume acme:/etc/acme.sh \ --volume /home/quangnv13/letsencrypt_user_data:/app/letsencrypt_user_data:ro \ is it ok?

As I don't know your original setup I have no way to tell 😃

Now i'm using image: jrcs/letsencrypt-nginx-proxy-companion:2.0.2. If i resetup my container use --volume acme:/etc/acme.sh \ --volume /home/quangnv13/letsencrypt_user_data:/app/letsencrypt_user_data:ro \ is it ok?

As I don't know your original setup I have no way to tell 😃

version: '3'
services:
nginx-proxy:
image: nginx:1.19.9
container_name: nginx-proxy
labels:
com.github.jrcs.letsencrypt_nginx_proxy_companion.nginx_proxy: "true"
restart: always
ports:
- 80:80
- 443:443
volumes:
- ./conf.d:/etc/nginx/conf.d
- ./vhost.d:/etc/nginx/vhost.d
- ./html:/usr/share/nginx/html
- ./certs:/etc/nginx/certs:ro
docker-gen:
image: jwilder/docker-gen:0.7.0
container_name: nginx-proxy-gen
command: -notify-sighup nginx-proxy -watch -wait 5s:30s /etc/docker-gen/templates/nginx.tmpl /etc/nginx/conf.d/default.conf
labels:
- "com.github.jrcs.letsencrypt_nginx_proxy_companion.docker_gen"
depends_on:
- nginx-proxy
volumes:
- ./conf.d:/etc/nginx/conf.d
- ./vhost.d:/etc/nginx/vhost.d
- ./html:/usr/share/nginx/html
- ./certs:/etc/nginx/certs:ro
- ./nginx.tmpl:/etc/docker-gen/templates/nginx.tmpl:ro
- /var/run/docker.sock:/tmp/docker.sock:ro
letsencrypt:
image: jrcs/letsencrypt-nginx-proxy-companion:2.0.2
container_name: nginx-proxy-les
depends_on:
- nginx-proxy
- docker-gen
volumes:
- ./conf.d:/etc/nginx/conf.d
- ./vhost.d:/etc/nginx/vhost.d
- ./html:/usr/share/nginx/html
- ./certs:/etc/nginx/certs:rw
- /var/run/docker.sock:/var/run/docker.sock:ro
networks:
default:
external:
name: nginx-proxy
Here is my config! Can you check it?

I have letsencrypt_service_data config file like:
LETSENCRYPT_CONTAINERS=(
'19420333155d'
'99179a37c995'
'7a21b77c3145'
'57f08f766dd1'
'05acfadf3339'
'6cd2d090fcc6'
)
LETSENCRYPT_19420333155d_HOST=('adminer.quangnguyen.info' )
LETSENCRYPT_19420333155d_KEYSIZE=""
LETSENCRYPT_19420333155d_TEST=""
LETSENCRYPT_19420333155d_EMAIL=""
ACME_19420333155d_CA_URI=""
ACME_19420333155d_PREFERRED_CHAIN=""
ACME_19420333155d_EAB_KID=""
ACME_19420333155d_EAB_HMAC_KEY=""
ZEROSSL_19420333155d_API_KEY=""
LETSENCRYPT_19420333155d_RESTART_CONTAINER=""
LETSENCRYPT_99179a37c995_HOST=('chatbox.quangnguyen.info' )
LETSENCRYPT_99179a37c995_KEYSIZE=""
LETSENCRYPT_99179a37c995_TEST=""
LETSENCRYPT_99179a37c995_EMAIL=""
ACME_99179a37c995_CA_URI=""
ACME_99179a37c995_PREFERRED_CHAIN=""
ACME_99179a37c995_EAB_KID=""
ACME_99179a37c995_EAB_HMAC_KEY=""
ZEROSSL_99179a37c995_API_KEY=""
LETSENCRYPT_99179a37c995_RESTART_CONTAINER=""
LETSENCRYPT_7a21b77c3145_HOST=('cloudcmd.quangnguyen.info' )
LETSENCRYPT_7a21b77c3145_KEYSIZE=""
LETSENCRYPT_7a21b77c3145_TEST=""
LETSENCRYPT_7a21b77c3145_EMAIL=""
ACME_7a21b77c3145_CA_URI=""
ACME_7a21b77c3145_PREFERRED_CHAIN=""
ACME_7a21b77c3145_EAB_KID=""
ACME_7a21b77c3145_EAB_HMAC_KEY=""
ZEROSSL_7a21b77c3145_API_KEY=""
LETSENCRYPT_7a21b77c3145_RESTART_CONTAINER=""
LETSENCRYPT_57f08f766dd1_HOST=('monitor.quangnguyen.info' )
LETSENCRYPT_57f08f766dd1_KEYSIZE=""
LETSENCRYPT_57f08f766dd1_TEST=""
LETSENCRYPT_57f08f766dd1_EMAIL=""
ACME_57f08f766dd1_CA_URI=""
ACME_57f08f766dd1_PREFERRED_CHAIN=""
ACME_57f08f766dd1_EAB_KID=""
ACME_57f08f766dd1_EAB_HMAC_KEY=""
ZEROSSL_57f08f766dd1_API_KEY=""
LETSENCRYPT_57f08f766dd1_RESTART_CONTAINER=""
LETSENCRYPT_05acfadf3339_HOST=('ols.quangnguyen.info' )
LETSENCRYPT_05acfadf3339_KEYSIZE=""


MM-17519: do not stringify booleans for interactive dialogs

Summary
Match the webapp behaviour so as to avoid stringifying booleans in the interactive dialog payloads.
Ticket Link
Fixes: https://mattermost.atlassian.net/browse/MM-17519
Checklist

[x] Added or updated unit tests (required for all new features)

Device Information
This PR was tested on: iOS mobile simulator
Screenshots
Using the demo plugin's /dialog trigger, the behaviour before:

and after:

@lieut-data is this a regression? does every server version expects the same boolean or was this change done recently?

@enahum, unfortunately, it's not a regression: the webapp and mobile implementations diverged on this point when the feature was first implemented. It was actually reported in the right timeframe, but we couldn't reproduce... but I think that was before understanding that webapp and mobile had separate code flows here. A community member reported it specifically for mobile, and it all "clicked".

@lieut-data
Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Fetching origin
+++ Updating remotes done...
!!! 'upstream/release-1.27' not found. The second argument should be something like upstream/release-0.21.
    (In particular, it needs to be a valid, existing remote branch that I can 'git checkout'.)

This is scheduled for v1.27, so the release branch just isn't created yet. Will cherry-pick when available.

/cherry-pick release-1.27


Dockerfile improvements

Hello,
Thanks for putting out these samples and for the documentation.
I wanted to make some changes to the Dockerfiles but thought to open an issue first. The proposed changes are:

To not run the containers as root but as a separate user (maybe aeron)
Decrease the repetitiveness of COPY... statements by using .dockerignore files
Replace wget .... in favor of ADD... commands.

Let me know if these sound reasonable and I'll be happy to open a PR.

I've made some initial changes around the first point - the applications in the containers no longer run under root but now aeron, as suggested.


Publish samples

There's a bunch of samples here: https://github.com/chartjs/Chart.js/tree/master/samples
It'd be really nice if these were hosted somewhere users could browse such as in the docs

@benmccann it's planned for 2.6 (see #4043)

awesome. thanks!


Uncaught Error: imports[1] for react-transform-catch-errors does not look like a React component.

I encountered the error Uncaught Error: imports[1] for react-transform-catch-errors does not look like a React component.
I resolved the issue by removing the ^ from "redbox-react": "^1.0.4", in package.json, as mentioned in the SO post.
http://stackoverflow.com/questions/37895944/react-transform-catch-errors-does-not-look-like-a-react-component#
Shall I issue a PR? Or is there a better solution?

same

I think it's the right way to fix this problem. please submit a pull request.

I had to fix this on my fork as well so I just submitted #30

Is it resolved?


Register custom ONNX operators as dynamic operator

According to this comment:
https://github.com/onnx/onnx/blob/5972eed96752551a0ea8b932904fedd568c48d3b/onnx/defs/schema.h#L853-L872
closes #220

Oops I commented on issue instead of this PR. Heh, I didn't expect the solution is this easy. Thanks for the nice finding!


AGX Orin support on Kirkstone (copy)

Test PR for Jenkins

Can one of the admins verify this patch?

@resin-jenkins test this please

Copy PR for Jenkins testing, not needed


Move BCO to separate BCOTrainer with fixes

This is a further improvement of #1803:
As requested, the BCO training is now fully moved to a new BCOTrainer with respective BCOConfig.
This PR includes the following changes from #1803:

When using the BCO loss type there is no need for the KL dataset anymore. By skipping the creation of the KL dataset we save time and memory. -> Fixed with separate trainer now
Do not assert both desired and undesired data in each per-device mini batch. With this it was impossible to train large models that only allow a per-device batch size of 1. Also KTO and BCO are supposed to work with unpaired preference data and different amounts of desired or undesired examples. In my experiments BCO proofed to work well  without this assumption and also I cannot find it mentioned anywhere in the paper.
When checkpointing, also save the RunningMoments object which is used in BCO to calculate the $\delta$ mean reward value. If this is not saved, it will corrupt the whole resumed training process when restarting from a checkpoint.

Additional issue that is fixed only in this PR:

UDM supports non interleaved datasets now. The previously implemented underlying distribution matching assumed the same shape of prompt_embeddings across all devices in a distributed setting. The beginning of the train_dataset was interleaved, but the eval_dataset and the end of the training dataset not, causing this to crash in the current main version. As for KTO, I assume that perfectly interleaved datasets should not be a requirement of BCO to work.

@kashif, @lewtun and @seanexp as original author of BCO

very cool @claralp checking!

@claralp  the failing tests are not related to this PR

@claralp can you add the new bco_trainer.mdx to the docs toc

Sorry for late reply @claralp
The code looks mostly good!

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

and thanks for the additional fixes/improvements 👍 just checked them, they all look reasonable to me

@kashif @seanexp fixed a few more issues when running on multiple GPUs after the lastest changes.
Is this ready to merge from your side or any more open points that you discovered?


Undefined symbols for architecture arm64:

@joel-bitar this issue is coming when i update my xcode to v 16
"OBJC_CLASS$_ADClient", referenced from:
in AppleAdsAttribution.o
ld: symbol(s) not found for architecture arm6

+1


Update for Swift 2.2

The current code base uses C-style for loops, the i++ operator, and var in function parameter declarations. These all now create compiler deprecation warnings.

Will be addressed in #35.


Pubsub emulator with v1 doesn't work

Environment details

OS: MacOS
Node.js version: 12
npm version: Latest yarn 1.x
@google-cloud/pubsub version: Latest

Steps to reproduce
Using the synchronous pull example, the PusSub emulator doesn't seem to be working. It's as the env variable PUBSUB_EMULATOR_HOST is being ignored.
Also,
with the default emulator local host (localhost:8085), if I do:
const subClient = new v1.SubscriberClient({
  apiEndpoint: 'localhost:8085',
});

I am getting the error:
Error: Failed to parse target localhost:8085:443
        at Function.getDefaultAuthority (/node_modules/@grpc/grpc-js/src/resolver-dns.ts:313:11)
        at Object.getDefaultAuthority (/node_modules/@grpc/grpc-js/src/resolver.ts:134:28)
        at new ChannelImplementation (/node_modules/@grpc/grpc-js/src/channel.ts:199:31)
        at new Client (/node_modules/@grpc/grpc-js/src/client.ts:82:30)
        at new ServiceClientImpl (/node_modules/@grpc/grpc-js/src/make-client.ts:104:3)
        at GrpcClient.createStub (/node_modules/google-gax/src/grpc.ts:306:18)
        at processTicksAndRejections (internal/process/task_queues.js:93:5)

Adding this to my look-at-shortly queue.

Can confirm the same exact behaviour in a similar environment.

Any update on this?

@oande Sorry for the delay, I've still got this on my list.

Any update on this? We got the exact same problem in a similar environment.
This fix used to work but the issue (TypeError: Channel credentials must be a ChannelCredentials object) is happening again since a few days...

@feywind this is also preventing my team from working with the Pub/Sub emulator locally, which makes development difficult. The workarounds proposed in https://github.com/googleapis/nodejs-pubsub/issues/346 don't work.

@rlucioni I thought I had a suggestion for you, but digging a bit deeper suggests it didn't fix the issue.
The slightly longer answer to the original question is that the environment variable is being interpreted by the veneer client, so if you use v1 directly, it won't see it. That also seems suboptimal. There's been some talk about normalizing the environment variables in the GCP client libraries on this point, but that's still in progress.
It seems to me that the generated gapic clients should interpret that environment variable, but I think the reason they don't is that it hasn't been standardized yet. 😬
There might also be a workaround here by just pulling the info from a PubSub object. I'm going to test that and get back to you.

Here are some ways to work around this, based on my testing:
The simplest, but probably also the most prone to breakage over time (these aren't the same types, but they do technically contain the same things):
const pubsubClient = new PubSub();
const subClient = new v1.SubscriberClient(pubsubClient.options);

That will actually pick up the emulator environment variable in the normal way, since it's just using the PubSub class' init.
The specific members could be copied like so, to be a bit less breakable:
const pubsubClient = new PubSub();
const subClient = new v1.SubscriberClient({
  apiEndpoint: pubsubClient.options.apiEndpoint,
  port: pubsubClient.options.port,
  sslCreds: pubsubClient.options.sslCreds
});

You could also manually init the options, though I understand that wasn't working for some:
const grpc = require('@grpc/grpc-js');

const subClient = new v1.SubscriberClient({
  apiEndpoint: 'localhost',
  port: '8085',
  sslCreds: grpc.credentials.createInsecure()
});

(And of course requires app logic, which may not be great.)
I also feel obligated to note that digging into PubSub.options isn't really a public interface, so that could change whenever. I think probably the good long term solution for this is still the unified environment variables plus gapic client support for it. If it looks like the timeline is long on that, then I suppose another option would be to expose a "give me a client" method on PubSub.
Let me know if anything in there seems like it works, or if you have any ideas about making an interface that works better for you.

Passing options from an instance of the PubSub client worked for us! Thank you!

I am not getting any errors using the example of passing options from a PubSub instance.  But it just spins until a timeout is reached.
Any idea or suggestions?

Added the "external" label since this is an issue w/ the emulator (we think) rather than the client library. @feywind Is that correct? Otherwise, apologies in advance if I am wrong. :)

I am not getting any errors using the example of passing options from a PubSub instance. But it just spins until a timeout is reached.
Any idea or suggestions?

@sdcaulley
I had this exact problem and finally got it working. The issue for me was because the option attribute is servicePath on the pubSub client, not apiEndpoint.
So this worked for me:
const pubSub = new PubSub({
    projectId
});
const options = {
  apiEndpoint: pubSub.options.servicePath,
  port: pubSub.options.port,
  sslCreds: pubSub.options.sslCreds
};
const subClient = new v1.SubscriberClient(options);

Thanks for the solution, @feywind !

apiEndpoint should theoretically work on the PubSub object as well as v1 clients, so I'm not completely sure why it wasn't working.
Yeah, I can't say the workaround of pulling options out of the PubSub object is a good long term practice, but I'm going to make a new issue that's targeted at just figuring out what our official way forward is for v1 and alternate endpoints (and close this one). For now I'd say to consider it a temporary workaround until there's something more structured in place.


Add PyPI package

The name ertk is currently blocked per https://github.com/pypa/warehouse/issues/10810. I've put in a request at https://github.com/pypa/pypi-support/issues/1711 for it to be reassigned to me. After that's gone through I will upload the package to PyPI.

The ertk package is now available at https://pypi.org/project/ertk/


Add a little red dot

Can you add a little red dot on a Button Bar or Segmented?

I also need that. I want to add a round circle filled with red color in button bar is that possible?
Did you find any solution for that?


Guidance:  httpChecksumRequired trait implementation and test

This trait doesn't carry with it any information as to how to implement it .
How is a code generator meant to interpret it ?
There is a default set to MD5 , this test fails if the default is not passed in (meaning there is no Content-MD5 header)
How would one signal the choice to not use the defaul

There is currently no way to select a checksum algorithm other than the default without another trait. For AWS, we have the aws.protocols#httpChecksum trait. This more complex trait and behavior specification overlaps with httpChecksumRequired, but is AWS-specific until there is a proposal to update the httpChecksumRequired trait or standardize on the AWS behavior.

Thank you
So currently the httpChecksumRequired trait is equivalent to requiring an MD5 hash

That's correct. Closing as question is answered.


Add option to override server timezone for MySQL datasources

Provides a way to resolve the error reported on #589.


Move script reloading to the JSHint watch task

All other tasks separate their core functionality and browser reloading by placing the reload method in the watch task. Patterns aid readability / grokkability, so let's normalize this for JS.
It seems like the options once and stream aren't in the BrowserSync API docs anymore. Tests for them still exist in the BrowserSync repo, so not sure if they're being deprecated. This PR removes them either way.
I'm not entirely sure if placing the reload logic in the JSHint task was solving a specific problem, but in my project these changes have been working well.

Thanks for the PR!

Tests for them still exist in the BrowserSync repo, so not sure if they're being deprecated.

@shakyShane do you have any insights on this one?

using reload({stream: true, once: true}) is deprecated (but not removed until a major version bump)
There is a dedicated .stream() method to replace it, but I'd agree this PR is a better solution anyway and will yield identical results :+1:

Thanks for the input and confirmation on on when that would be removed @shakyShane :sparkles:. PR LGTM2. Let's land it.

Yussss! Thanks.
On Tuesday, September 22, 2015, Addy Osmani<EMAIL_ADDRESS>wrote:

Merged #754 https://github.com/google/web-starter-kit/pull/754.
—
Reply to this email directly or view it on GitHub
https://github.com/google/web-starter-kit/pull/754#event-415807165.

--
Rob Wierzbowski
@robwierzbowski http://twitter.com/#!/robwierzbowski
http://github.com/robwierzbowski
http://robwierzbowski.com


Switch to coursier 1.1.0-M13-1

(Waiting for it to be released and sync-ed to Maven Central.)

ivyCache is removed as the new high level API of coursier now has also has such a cache (see withFetchCache in the changes here).

Going to test that a bit more locally, then merge I guess.

Start-up time was hit when replacing the ivymap by the fetch cache from coursier. Running
$ mill 'amm[2.12.8].assembly'
$ cat foo.sc
import $ivy.`org.apache.spark::spark-sql:2.4.0`
import org.apache.spark.sql._
println(SparkSession)
$ time java -jar out/amm/2.12.8/assembly/dest/out.jar foo.sc

was consistently resulting in around 1500 ms with the coursier fetch cache, and around 1400 ms without, and with Ammonite 1.6.4. I'll investigate later… (the call to coursier itself didn't seem to be the culprit… maybe it's just because there are more classes to load)


Experimental functional reactive state production

An experimental branch for functional reactive state production based on the stateflow branch.
It offers feature parity with the existing branch and tries to establish some core FRP state production concepts:

Idiomatically functional: All inputs are eventually reduced into the UI state
No side effects: all changes to state must be modeled in the stream
Restricted UI state visibility: The state production pipeline cannot read the UI state until is ready to make changes to it
Well defined inputs: All information required to process an action must be present at the input of the action

MVI? Who's MVI? I'm just a functional reactive state production approach!


Add support for getting contract IDs that would be assigned to deployments

We need to add support for getting contract IDs that would be assigned to deployments.
We have stub functions already in the code for this capability, but they aren't filled in because the host functions don't exist for them to work. For example:
https://github.com/stellar/rs-soroban-sdk/blob/5a6429c297e9a9135dc797029ddc7a79ca3abee6/soroban-sdk/src/deploy.rs#L113-L117
The reason to provide this functionality is two fold:

So that in tests developers can pre-generate a contract ID that they expect will be used later in the test.
So that contracts can store the contract ID of a contract given inputs.

Dependent on https://github.com/stellar/rs-soroban-env/issues/538
Blocking https://github.com/stellar/soroban-examples/issues/152

@leighmcculloch , Can this be closed now that we have get_contract_id host fn?

The host fn was implemented, but the sdk work to use it has not been completed, which is what this issue is for.
For example:
https://github.com/stellar/rs-soroban-sdk/blob/f743d6f9e49caa08924318907cd0588b60d7f187/soroban-sdk/src/deploy.rs#L152-L157
https://github.com/stellar/rs-soroban-sdk/blob/f743d6f9e49caa08924318907cd0588b60d7f187/soroban-sdk/src/deploy.rs#L184-L188
Also interesting note those two functions used to be named the same, and should still be named the same. Strange that one has been renamed without the other.

Closed by #1086


how about the result you got about coco?

Hi, Wonder,
currently we achieve 36.5 for ResNet-101 with 1.5x training strategy and
single scale train, but recently I do not have time to complete this,
thanks for your attention.
Wonder<EMAIL_ADDRESS>于2019年5月13日周一 上午9:46写道：

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DetectionTeamUCAS/FCOS_GluonCV/issues/1, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AIHPPCCPQS4SS7VYKDIHETLPVDB77ANCNFSM4HMLYHHQ
.


Exponentation bug

Running it in playground:
https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=0d25f92071a744a754c9cc583fe8ee6b
I get the wrong answer raising any number to the power of any number.
Am I using the crate incorrectly?

That's not exponentiation, it's the bitwise XOR operator. Rust doesn't have an operator for exponentiation, but you can use Pow.

I am sorry. My mistake. Closing ticket.


Select subsets of multitask learning tasks

Add functionality to select a subset of the tasks in a multitask learning task.

this is addressed in v0.4.0


Fix non-exact version specs

@jonashaag could you merge/rebase with the latest develop and add a description?
Thanks!