text:
Custom default time for scheduling messages.

This feature allows users to set a custom default time to schedule messages at. This is different from the existing "custom time" functionality since it persists between messages. Currently, this is just one time, but could be expanded to a list in the future.
Fixes:
#28894
Instead of changing the default times already there, I decided to allow the user to define their own.

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Thanks! Please go through our guide on submitting a pull request to learn how to present your proposed changes to Zulip, so that your work can be reviewed.

Since this feature was not specced out to be worked on, you may want to start by sharing your proposal for how to implement in the Zulip development community.


text:
Update README.md

Use absolute link to examples directory in readme. The relative link is broken when rendered from the top level of the repository.

Thanks for the fix! LGTM

source: gharchive/pull-request

authors: imalsogreg zurawiki

repo: zurawiki/tiktoken-rs

license: MIT

license_type: permissive

license_source: github-api


text:
Popup bug in docs

Go to the Popup page
Click the "Open Popup" button
Click the "Notification" link in the navigational sidebar
The popup is still present and now misplaced near the upper-left of the viewport.

Tested in OS X stable Chrome.
Gratuitous screenshot:

Hi, I faced similiar issue, thats how I got it fixed:
In popup directive add another function destroy:
scope.destroy = function() {
    scope.active = false;
    if(tetherInit) {
        tether.destroy();
        $(element[0]).remove();
        tetherInit = false;
    }
};

also in postLink I have added cleanup(should be there anyway):
element.on('$destroy', function() {
    foundationApi.unsubscribe(attrs.id);
});

Now when I switch view using UI router in onExit I am publishing 'destroy' message to the popup

I have created a pull request #824 with my fix for the issue
(based on the comment by @WojciechLuczkow)

How about replacing Tether with Popper.js? In this way the tooltip elements shouldn't be child of body and everything would be easier.

these issues should be resolved as of v1.2.5 of the community fork for this repo: https://github.com/base-apps/angular-base-apps
@FezVrasta if you want to play around with popper.js integration then feel free to submit a PR against the fork above

source: gharchive/issue

repo: zurb/foundation-apps

license: mit

license_source: bigquery


text:
User Flow

Refer to #29 for an overview of all design related tasks

https://www.figma.com/file/UzdIIpsX43I1ZgMETPwLgQ/Authentication-Library?node-id=2076%3A8594 @ElSam20

authors: ti-oluwa yewandefajem


text:
Qubino (GOAP) Flush Dimmer ZMNHDA not exposing groups

Dear all,
new zwave-js user, running on Home Assistant using supervised zwavejs2mqtt docker. I have a number of Qubino Flush Dimmers (ZMNHDA) devices which do not update the controller. I think it may be due to the fact none of the association groups are recognized.
When adding/refreshing the node through the zwavejs2mqtt interface, the following appears in the log:
2021-02-09 09:05:28.917 WARN ZWAVE: Node 10 doesn't support groups associations
Whereas I need group 4 to report to the controller, as per documentation at https://opensmarthouse.org/zwavedatabase/209.
Most appreciated.

Please choose an appropriate issue template and fill it out completely when opening an issue. That helps us investigate your issue quicker because all the necessary information is there. Otherwise, the issue will go to the bottom of the pile - and that pile is growing quickly.

Node 10 doesn't support groups associations

@robertsLando that looks like a message from zwavejs2mqtt
I'm not sure where that comes from since the config file contains this:
	"associations": {
		"1": {
			"label": "multilevel",
			"maxNodes": 4
		},
		"2": {
			"label": "basic on/off triggered by the input I2 state",
			"maxNodes": 4
		},
		"3": {
			"label": "basic on/off triggered by the input I3 state",
			"maxNodes": 4
		},
		"4": {
			"label": "Controller updates",
			"maxNodes": 4,
			"isLifeline": true
		}
	},

Providing further info:
Device information
Qubino (GOAP) ZMNHDA
Node IDs 18, 54, 64 are affected
Last Known Working Configuration

[X] Previously working device (other platform)

Which platform? --> Home Assistant using OpenZWave
Have you made any recent configuration changes to the device? Describe. --> NO

Installation information
How did you install node-zwave-js?

[ X] Other: zwavejs2mqtt as a supervised add-on in Home Assistant

To Reproduce
Steps to reproduce the behavior:

Go to ZWaveJS2MQTT control panel
Click on  affected device
Click on "Groups"
Observe "Group" dropdown says "No data available"

Additional context
I've tried excluding/including device, as well as re-interviewing node multiple times
Logfile:
Attached
zwavejs_473.log

Can you re-interview one of those three nodes once and send me the log of that? Unfortunately the interview after startup does not include the info I'm looking for.

Adding another comment to this (sorry!): the WARN message in the original post is indeed from zwavejs2mqtt, and is caused by a catch from "driver.controller.getAssociationGroups(...)" calls in ZWaveClient.js

Can you re-interview one of those three nodes once and send me the log of that? Unfortunately the interview after startup does not include the info I'm looking for.

The tail of the log should be a re-interview of node 064

Huh, there it is:
18:09:53.256 CNTRLR « [Node 064] node info received
                      supported CCs:
                      · Multilevel Switch
                      · All Switch
                      · Binary Switch
                      · Meter
                      · Multilevel Sensor
                      · Multi Channel
                      · Powerlevel
                      · Association
                      · Configuration
                      · Manufacturer Specific
                      · Version
                      controlled CCs:
                      · Basic
                      · Multilevel Switch

...

18:09:53.997 SERIAL « 0x010a000400400486148500a2                                          (12 bytes)
18:09:54.001 DRIVER « [Node 064] [REQ] [ApplicationCommand]
                      └─[VersionCCCommandClassReport]
                          CC:      Association
                          version: 0
18:09:54.006 CNTRLR   [Node 064]   does NOT support CC Association (0x85)

The device contradicts itself. As a workaround we could do the same as in https://github.com/zwave-js/node-zwave-js/pull/1666.
I would advise you to report this to the Qubino support though. Last time I talked to them, they seemed to be quite interested in having their devices work with zwave-js.

Aha, good catch! So we could basically hard-code it to support CC Associations, post device interview?
If that could be provided for, that’d be great. I’ve rolled back to OZW in the meantime as any automations based on knowing the state of these devices is broken. Do I need to do anything to move this patch forward?
I’ll reach out to Qubino and see what they come back with.
Most appreciated!!

If you change the corresponding file, restarting or re-interviewing should do the trick

So I added the following to zmnhda.json in /config/0x0159/
	"compat": {
		"commandClasses": {
			"add": {
				"0x85": {
					"isSupported": true
				}
			}
		}
	}
}

Zwave2mqtt no longer throws the warning upon re-interview, but I'm still not seeing the groups become populated...
Any tips very welcome!!

New log please :)

Please find it attached. Even though the zwavejs2mqtt log no longer shows the warning, the zwavejs log (attached)
zwavejs_578.log
still shows
11:05:07.574 DRIVER « [Node 064] [REQ] [ApplicationCommand]
                      └─[VersionCCCommandClassReport]
                          CC:      Association
                          version: 0
11:05:07.578 CNTRLR   [Node 064]   does NOT support CC Association (0x85)

Can you confirm that the above patch I made to zmnhda.json is in fact what's supposed to do the trick?
thanks!!
Nik.

Yes, the query is still done as part of the Version CC interview, but the support info is overwritten afterwards. Later in the log you see:
11:00:22.982 CNTRLR « [Node 064] received information for association group #2:
                      maximum # of nodes: 16
                      currently assigned nodes:

Ok,thanks!
The plot thickens, as when I manipulate the physical switch, I see:
13:20:12.233 SERIAL « 0x0110000400400a3202213400000000000084                              (18 bytes)
13:20:12.237 CNTRLR   [Node 064] [~] [Meter] value[66049]: 10.5 => 0                    [Endpoint 0]
13:20:12.238 CNTRLR   [Node 064] [~] [Meter] deltaTime[66049]: 0 => 0                   [Endpoint 0]
13:20:12.239 SERIAL » [ACK]                                                                   (0x06)
13:20:12.241 DRIVER « [Node 064] [REQ] [ApplicationCommand]
                      └─[MeterCCReport]
                          type:       Electric
                          scale:      W
                          rate type:  Consumed
                          value:      0
                          time delta: 0 seconds
13:20:12.246 CNTRLR   [Node 064] [~] [Meter] value[66049]: 10.5 => 0                    [Endpoint 1]
13:20:12.253 CNTRLR   [Node 064] [~] [Meter] deltaTime[66049]: 0 => 0                   [Endpoint 1]

Endpoint 1 isn't advertised through zwavejs2mqtt (after re-interview) and upon first reception of these updates, entries for 64-38-1 are added. Although I see both updates for Endpoints 0 and 1 in zwavejs, the Endpoint 0 isn't updated in zwavejs2mqtt. Probably something to track further on that end...
Thanks for all your help!

repo: zwave-js/node-zwave-js


text:
[bug] Import QR opens the camera

Version
Checklist:

[x] I am not using Home Assistant. Or: a developer has told me to come here.
[x] I have checked the troubleshooting section and my problem is not described there.
[x] I have read the changelog and my problem is not mentioned there.

Build/Run method

[ ] Docker
[ ] PKG
[x] Snap package
[ ] Manually built (git clone - yarn install - yarn run build )

zwavejs2mqtt: 5.11.0.474d303
zwave-js: 8.7.5
Describe the bug
Import QR on a smart phone opens up the camera instead of the file explorer.
To Reproduce
Steps to reproduce the behavior:

Go to Manage nodes
Click on Inclusion
Scroll down to Smart Start and select «Import».
Camera opens and not the file explorer

Expected behavior
Should open the phone (default) file explorer so you can pick from the gallery.

@jmgiaever Are you on an iphone or an android phone? From iphone you should be able to press on bottom left bottom to add it from library

Android 😅

Is the icon you're referring to here? I've tried to click the "drop here" icon, but that's what opening up the camera app.

I am an iPhone user, SCAN doesn't do anything, IMPORT and then click Drop.... Turns camera on and takes a photo (not properly labeled but I like this).
**** HOWEVER! - It fails to recognize/find a QR/DSK code.

Scan should work as you like the import feature. The camera should open. Have you maybe denied access to the camera?
The first time you've visited the scan section it (z2m in your browser) will ask for permission to use your camera. If you deny access it will not load.

Please give a try to https://github.com/zwave-js/zwavejs2mqtt/pull/1956
BTW Are you running it using https?

Me or @doug62 ? I'm running it behind a reverse proxy. Z2m itself is http, but I always request it over the reverse proxy since I've blocked every other ports on this device.

It was for @doug62
BTW the PR attached should fix all the issues expect the **** HOWEVER! - It fails to recognize/find a QR/DSK code. , I have no clue about that as all my qr codes are correctly parsed. I would need the qr code to test

@robertsLando I attached the file to an earlier bug #1958 but attached here as well, I have tried 5 different codes, my phone parses them but zwavejs2mqtt doesn't.  Thanks for your awesome work, I'm hankering to contribute when i get more up to speed w HA

@doug62 In fact I'm able to add it when scanning from camera but not when adding it as an image (weird). Let me investigate

@doug62 I have a feel the problem is that the QR code is too big in the image and the detect fails, just to make a double check could you try to make a new picture of it?

Just tried to use this (simply reduced browser zoom and made a screenshot and it is working:

authors: doug62 jmgiaever robertsLando


text:
[question] Network Graph behavior - should it show lines between all neighbors?

In 6.5.2, I have noticed that the graph does not render unless you click the reload button and then it only draws lines showing the path back to the controller for each node. Is it supposed to show lines for all neighbor realtionships?

Is it supposed to show lines for all neighbor realtionships

Thre is a dedicated option to show all relationships, check 'relevant neighbors' option

@robertsLando::  Where is this relevant neighbors option? I am sorry but I do not see it.

Sorry @nmeachen I forgot I removed that setting in latest release and it was making only confusion

I could restore it if it's so important

That would be great! I would like that as you can't hover over the entries in a mobile view, so the lines where the only real indication.

authors: nmeachen robertsLando


text:
RuntimeError

Hi, thanks for your great work.
When I use the config configs/clipvis_ConvNeXt-B.yaml, there is an error below.
RuntimeError: Pretrained weights (laion2b_s29b_b131k_ft_soup) not found for model convnext_base_w_320.
Available pretrained tags (['laion_aesthetic_s13b_b82k', 'laion_aesthetic_s13b_b82k_augreg'].

And I check on the openclip website(https://github.com/mlfoundations/open_clip/blob/main/docs/openclip_results.csv), it seems the laion2b_s29b_b131k_ft_soup is not valid for the convnext_base_w_320.
Besides, is the checkpoint for the setting trained on COCO and YTVIS2019 available?

I'm sorry for this mistake. The pretrained weights is laion_aesthetic_s13b_b82k.

Thanks for your reply. Besides, is the checkpoint for the setting trained on COCO and YTVIS2019 available?

Hi, @SuleBai
I have update the Readme file, please check it for checkpoint.

Thanks for your reply.


text:
Uncaught exception when decoding particular MicroQR

When decoding a particular MicroQR, CodecModeForBits() throws std::invalid_argument which is then not caught by QRCode::DecodeBitStream(). An android app decodes "barcode xpress" in there, but I can't claim the barcode is 100% valid as other readers fail as well.
Is it expected that MultiFormatReader::read() throws?
Please note also the typo in the name of the 2nd argument of CodecModeForBits() - isMirco

Not directly solving your problem, but I generated a MicroQR code encoding "barcode xpress" with Zint:

Is it expected that MultiFormatReader::read() throws?

Not in this case. See fix.
The symbol itself is violating the specification, though. (As far as I can tell).

Thanks for the fix, it is certainly correct. However, I am still wondering why other reader(s) can read the barcode. So I did some digging.
The capacity of 17x17 L microQR is 16 bytes. The encoded string occupies exactly 15 bytes. Then the Terminator should follow - nine 0 bits. The QR ISO 18004-2006 specification says “The end of the data in the complete symbol is indicated by a Terminator consisting of between 3 and 9 zero bits (see Table 2), which is omitted or abbreviated if the remaining symbol capacity after the data bit stream is less than 4 bits.”. So, we have 8 bits remaining, but the generator did not fill them with zeros, but put 0xEC there. Which seems to be half of the padding double-word: 0xEC11. So, the generator followed the “omitted” part and not  the “abbreviated” part because strictly speaking there are more than 4 bits remaining. So, literally, the barcode follows the specification. Logically, it does not, because the specification IMHO is wrong, it should consider the fact the Terminator is 3, 5, 7, 9 bits long for MicroQR and 4 for QR. So, what can fit into the 8 bits other than incomplete Terminator? Actually nothing. Mode has 3 bits, followed by a counter having at least 3 bits and then not even a single character can fit. Therefore I suggest to change the condition "bitsAvailable == 0" in IsEndOfStream() to "bitsAvailable < bitsRequired".

I appreciate your analysis and your attention to detail. I could not make any sense of your citation "if the remaining symbol capacity after the data bit stream is less than 4 bits". So I double checked in the spec. I only have the more recent ISO 18004-2015 and there it reads in 7.4.9: "The Terminator shall be omitted if the data bit stream completely fills the capacity of the symbol, or abbreviated if the remaining capacity of the symbol is less than the required bit length of Terminator." I hope you can agree that this makes perfectly clear that your symbol is violating the specification, since those remaining 8 bits have to be 0. If you can, do you also concur that the code as is correctly implements the specification?
As to your suggested code modification: I can follow your thought for this particular situation (M4, bits.available() == 8). But I did not go through all the other possible circumstances where bitsAvailable < bitsRequired to convince myself that this change does not break other (potentially valid) symbols. While that may be, my 'default' stance on the matter is that people with buggy encoders should fix their code instead of make all the decoders work around those bugs.
Do you have any idea where this image came from? Can you get in touch with the people responsible for it?

Your citation from the new spec shows that they fixed the wording. So, yes, I do agree the barcode violates the 18004-2015 spec. The barcode content helped to find the likely creator and I found the barcode at https://help.accusoft.com/BarcodeXpress/v13.5/BxNet/micro_qr.html. I have contacted them.
I think it is a not easy decision for you: do you want to strictly follow the specifications or go beyond? Some other vendors call their ability to decode barcodes others can't decode a competitive advantage. In my opinion, the latter is better, whenever there is no ambiguity, like in this case.

Thanks for helping, making the world a better (barcode) place :).

whenever there is no ambiguity, like in this case.

As explained above, I am personally not convinced yet about the absence of ambiguity in all possible cases.

I have contacted them.

@pictview I'm curious: did you get any reaction from them?

I have contacted them.

@pictview I'm curious: did you get any reaction from them?

Not at all :-(

Ah, commercial vendors.... they have their priorities straight ;).
If you still care about this particular issue, please feel free to help me get convinced about the "absence of ambiguity in all possible cases" so we might change the code such that your 'broken' image works.

authors: axxel duanqn pictview

repo: zxing-cpp/zxing-cpp


text:
Deterioration in QR decoding after updating zxing

Hello zxing team!
We are using your library to decode QR codes and upon updating the version, I noticed that the reading of QR codes we use as a reference to check the changes in our development has worsened.
So far, we were using zxing-cpp-wrapper==0.2.0 and we want to update to zxing-cpp==2.2.0. Previously, there were some images with QR codes that we couldn't decode, and we read the following:

There were about 9 images out of 2250. However, with other QR codes, we were able to decode the information correctly. Now, with the same images, there are about 7 more images where the information is read incorrectly, going from an incorrect reading rate of 0.4% to 0.7%. Do you know what could be causing this? Could it be a configuration issue due to some default parameter changes?
Thanks for your help!

So far, we were using zxing-cpp-wrapper==0.2.0 and we want to update to zxing-cpp==2.2.0.

This information is not enough to understand what you are talking about. Based on the format of the string zxing-cpp==2.2.0 I assume you are talking about the pypi.org python package. But there is no zxing-cpp-wrapper package that I know of and there never has been a zxing-cpp==0.2.0 package.
Independent of that, I'd need a sample image that used to work and now does not anymore. Also, it sounds like zxing-cpp==2.2.0 does find the QRCode for you but returns the "wrong" result?

Without further feedback from you, I'll close this issue.


text:
chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

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

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (66a94f4) 95.69% compared to head (1d72625) 95.69%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #817   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17

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

authors: codecov-commenter zxkane

repo: zxkane/snat


text:
Release 0.9.0

[ ] allPass
[ ] anyPass
[ ] call
[ ] dec
[ ] dropWhile
[ ] evolve
[ ] inc
[ ] init
[ ] insertAll
[ ] invert
[ ] invertObj
[ ] mapObjIndexed
[ ] negate
[ ] nthArg
[ ] toLower
[ ] toUpper

@all-contributors please add @zydmayday  for code doc


text:
Add option toggle command

Description of the problem or steps to reproduce
It would be useful to have a toggle command, next to set. This way, we can toggle a setting on and off by hotkey, without the need for a seperate action to be implemented for every option.
For example, toggle softwrap: Instead of having to write lua code (as suggested by zyedidia), we could simply bind Alt-z command:"toggle softwrap".
Specifications
Version: 2.0.8
Commit hash: cfcb2e45
Compiled on February 21, 2021

OS: All
Terminal: All

It can be done in Lua. I have this in my init.lua:
local micro = import("micro")
local config = import("micro/config")

function toggleOption(bp, args)
    if #args < 1 then
        micro.InfoBar():Error("Not enough arguments")
        return
    end
    local val = config.GetGlobalOption(args[1])
    if type(val) ~= "boolean" then
        micro.InfoBar():Error("Non-boolean option ", args[1])
        return
    end
    config.SetGlobalOptionNative(args[1], not val)
end

function toggleLocalOption(bp, args)
    if #args < 1 then
        micro.InfoBar():Error("Not enough arguments")
        return
    end
    local val = bp.Buf.Settings[args[1]]
    if type(val) ~= "boolean" then
        micro.InfoBar():Error("Non-boolean option ", args[1])
        return
    end
    bp.Buf:SetOptionNative(args[1], not val)
end

function init()
    config.MakeCommand("toggle", toggleOption, config.OptionComplete)
    config.MakeCommand("togglelocal", toggleLocalOption, config.OptionComplete)
end

Though I agree it would be nice if micro provided it out of the box.

repo: zyedidia/micro


text:
[Feature Request] Suspend to background

Current Behaviour
If I want to run a shell comand, I have to type CtrlB and then type out the command in full.
Desired Behaviour

Hit a key sequence to run a Suspend command
micro process is sent to the background, dropping me back to shell
Do all my shell tasks with tab completion etc
Run fg
micro screen returns; resume my editing session.

This behaviour is seen in vi and clones (press ^Z to suspend) and many emacsen (on GNU Emacs it's M-x suspend-emacs)

I'm not sure how this could be done or if it is possible. I still have to look into a more.
For a temporary solution you can always press Ctrl B and then execute bash. To run a shell.

I'm not sure if it's portable to micro (since my editor uses termbox) and it's Linux only (possibly other unix-likes as well? I don't have any lying around to try) but this implementation, based on the one in godit, Works On My Machine™.
https://github.com/japanoise/gomacs/commit/ee95d6fcd6700773a83b3ae941a2891afb548e74

This functionality has been added for Linux only. It may be possible to implement on unix but the syscalls used in godit/gomacs are only available on linux.

FWIW I find this to be pretty buggy - if I suspend in bash, then reopen the program with fg, then suspend again, the display doesn't refresh properly, and I get some pretty weird and bizarre visual glitches. If you suspend, then un-suspend, then re-suspend, does anyone else get this problem?

I use xonsh as my main terminal btw, and suspend doesn't work there at all, but rather crashes everything.

authors: battaglia01 japanoise zyedidia


text:
Return shader " " after calling optimizer multiple time

I dont know exactly how, but after optimizing multiple shaders, the optimizer return " " without error.
you can reproduce the issue with this url http://osgjs.org/examples/pbr/?GLSLOptimizer=1 There are extra info in the console.
It happens on some examples on my repo https://github.com/cedricpinson/osgjs but if I extract the shader that does not optimize and put it in your web interface it works. So it looks that after different call, something happens inside the optimizer.
Any clue ?

Hi @cedricpinson, it's been awhile since I've used this so I don't remember why that happens off from my head, although I'm guessing it could be a memory issue or probably some bugs that yield strange behaviour.
Anyways, I tried loading http://osgjs.org/examples/pbr/?GLSLOptimizer=1&log=info but I don't seem to see the error that you're describing?

Hu @zz85, it happens when a shader contains some errors. Instead of showing them (sometime it shows the error), it returns unreadable Chinese symbols.

Just checking to see if there's been any updates on this in the last few months? Seeing this as well.


text:
stront40 firmware

Hi There:
First I want to say Thank you for create stront. It is really amazing.
I notice you have create new stront40. I can't wait to try it.
May I ask do you have plan to release firmware or it just same as stront?
Thanks.

Hey! Thanks for following my work :) stront40 is work in progress, I created and ordered it but did not assemble and test it yet. There's no firmware, it will be just a bit different from regular stront.
I should be able to properly test PCB and firmware in a week or two but the case will take longer.

thank you. I will wait for your release.

authors: tangbonze zzeneg

repo: zzeneg/stront


text:
会经常出现提示“添加已读取失败”

大概率事件

能详细描叙下吗？我记得我代码里并没有添加已读取失败这种字样，是不是你自己项目的问题？

authors: zjyhll zzhoujay

repo: zzhoujay/RichText


text:
Dynamic linq not correctly apply on list of DynamicClass

I have following code. From external source I'm getting field.Name = "firstName" and field.Value = "firstValue".
var dynamicIndexes = new List<DynamicClass>();

var props = new DynamicProperty[]
{
   new DynamicProperty(field.Name, typeof(string))
};

Type type = DynamicClassFactory.CreateType(props);

var dynamicClass = Activator.CreateInstance(type) as DynamicClass;
dynamicClass.SetDynamicPropertyValue<string>(field.Name, field.Value);

dynamicIndexes.Add(dynamicClass);

var query = dynamicIndexes.AsQueryable();
bool isValid = query.Any("firstName eq \"firstValue\"");

But I'm getting isValid as false.
I did following tests to understand the issue:
added following code line at the beginning, to overwrite the value and then I'm getting isValid as true.
       field.Value = "firstValue";

but if I change that as follows then I'm getting isValid as false.
       byte[] utfBytes = Encoding.Default.GetBytes("firstValue");
       field.Value = Encoding.Default.GetString(utfBytes);

I'm guessing dynamic linq not working with string that has different encodings.

It's a bug, as far as i can tell, but it is not related to encodings.
The bug rather seems to be related to strings being interned (i.e., strings not being allocated on the heap, like string literals defined in the source code of the program) vs. strings living on the heap.
For reasons i don't know, the expression parser/language of dynamic linq does not like strings that live on the heap (i.e., strings that are dynamically/programmatically created). And it so happens that Encoding.Default.GetString(utfBytes) creates a new string that lives on the heap.
You should be able to reproduce the same bug effect without involving anything encoding-related at all by replacing Encoding.Default.GetBytes/Encoding.Default.GetString with something like:
var cc = "firstValue".ToCharArray();
field.Value = new string(cc);

Some further insights about what's going on here...
The System.Linq.Dynamic.Core library is building an System.Expressions.Expression tree from the string expression in query.Any("firstName eq \"firstValue\""). The left-hand expression is the identifier firstName. So far so good.
The System.Linq.Dynamic.Core library tries to figure out the type of the left-hand expression (which is being just the identifier firstName. Since query is a List<DynamicClass>, it will reflect upon the type DynamicClass to find a field or property with the name firstName to figure out its type. But, of course, DynamicClass itself does not feature any firstName member, so System.Linq.Dynamic.Core is treating firstName as being of type object.
And treating firstName as being of type object is the crux of the problem.
The behavior emanates from the System.Linq.Expressions classes (provided by the .NET base class library). It can be reproduced with some simple System.Linq.Expressions.Expression setup without even involving System.Linq.Dynamic.Core:
// lets first create two non-interned strings (i.e., not going to use string literals)
// Both strings have the same value, but since we don't use string literals, each string is a separate instance.

var s1 = new string(new char[] { 'f', 'i', 'r', 's', 't', 'V', 'a', 'l', 'u', 'e' });
var s2 = new string(new char[] { 'f', 'i', 'r', 's', 't', 'V', 'a', 'l', 'u', 'e' });

// Create left-hand and right-hand constant expressions, with the values of the strings s1 and s2.
// However, the left-hand expression will be of type object, whereas the right-hand expression will be of type string.

var left = System.Linq.Expressions.Expression.Constant(s1, typeof(object));
var right = System.Linq.Expressions.Expression.Constant(s2, typeof(string));

// now create the equality expression 'left equals right'

var eq = System.Linq.Expressions.Expression.Equal(left, right);

// and evaluate the equality expression

var lam = System.Linq.Expressions.Expression.Lambda(eq);
var fn = (Func<bool>) lam.Compile();
bool result = fn();
Console.WriteLine(result);

Dotnetfiddle: https://dotnetfiddle.net/IArvJA
This will result in the expression evaluating to false, even though both the left-hand and right-hand expression having exactly the same value. It's just that the left-hand expression has been typed as object, not as string. Kablooey!
(It really is the left-hand expression being typed as object. If it were typed as string, the result would be, as expected, true. https://dotnetfiddle.net/5VhuVF)

Considering that this is a behavior baked into the System.Linq.Expressions classes provided by the .NET base class library itself, i am not certain that this should be considered a bug in the System.Linq.Dynamic.Core library itself. But i still do find this to be an unexpected, surprising and quite frankly absolutely undesirable behavior, one that incurs a high risk of leading developers to write brittle code that is just waiting to break at a moments notice. And as such i wish System.Linq.Dynamic.Core could find a solution or workaround for mitigating this behavior.

Knowing this, a workaround is possible:
bool isValid = query.Any("firstName.ToString() eq \"firstValue\"");

Since basically any type derives from System.Object (except interfaces and some such, perhaps, i don't know really...), the ToString method (declared by System.Object) can be used to make the type of the left-hand expression here to be string (which is practical in this scenario here, since firstName contains a string value anyways), thus avoiding the issue. But i admit myself, it's not a pretty workaround...

Knowing this, a workaround is possible:
bool isValid = query.Any("firstName.ToString() eq \"firstValue\"");

Since basically any type derives from System.Object (except interfaces and some such, perhaps, i don't know really...), the ToString method (declared by System.Object) can be used to make the type of the left-hand expression here to be string (which is practical in this scenario here, since firstName contains a string value anyways), thus avoiding the issue. But i admit myself, it's not a pretty workaround...

Thank you very much for your clear explanation. but this workaround not work for me because query expression ("firstName eq "firstValue"") capturing from external source and that can be more complex than the example. I have to use some tokenizer for identify properties and then I can use simple linq query without using dynamic linq.

Thanks for the explanation. but @elgonzo when I overwrite field.Value = "firstValue";  at the beginning, this is work correctly. how you can explain that?

Some further insights about what's going on here...
The System.Linq.Dynamic.Core library is building an System.Expressions.Expression tree from the string expression in query.Any("firstName eq \"firstValue\""). The left-hand expression is the identifier firstName. So far so good.
The System.Linq.Dynamic.Core library tries to figure out the type of the left-hand expression (which is being just the identifier firstName. Since query is a List<DynamicClass>, it will reflect upon the type DynamicClass to find a field or property with the name firstName to figure out its type. But, of course, DynamicClass itself does not feature any firstName member, so System.Linq.Dynamic.Core is treating firstName as being of type object.
And treating firstName as being of type object is the crux of the problem.
The behavior emanates from the System.Linq.Expressions classes (provided by the .NET base class library). It can be reproduced with some simple System.Linq.Expressions.Expression setup without even involving System.Linq.Dynamic.Core:
// lets first create two non-interned strings (i.e., not going to use string literals)
// Both strings have the same value, but since we don't use string literals, each string is a separate instance.

var s1 = new string(new char[] { 'f', 'i', 'r', 's', 't', 'V', 'a', 'l', 'u', 'e' });
var s2 = new string(new char[] { 'f', 'i', 'r', 's', 't', 'V', 'a', 'l', 'u', 'e' });

// Create left-hand and right-hand constant expressions, with the values of the strings s1 and s2.
// However, the left-hand expression will be of type object, whereas the right-hand expression will be of type string.

var left = System.Linq.Expressions.Expression.Constant(s1, typeof(object));
var right = System.Linq.Expressions.Expression.Constant(s2, typeof(string));

// now create the equality expression 'left equals right'

var eq = System.Linq.Expressions.Expression.Equal(left, right);

// and evaluate the equality expression

var lam = System.Linq.Expressions.Expression.Lambda(eq);
var fn = (Func<bool>) lam.Compile();
bool result = fn();
Console.WriteLine(result);

Dotnetfiddle: https://dotnetfiddle.net/IArvJA
This will result in the expression evaluating to false, even though both the left-hand and right-hand expression having exactly the same value. It's just that the left-hand expression has been typed as object, not as string. Kablooey!
(It really is the left-hand expression being typed as object. If it were typed as string, the result would be, as expected, true. https://dotnetfiddle.net/5VhuVF)

Thanks for the explanation. but @elgonzo when I overwrite field.Value = "firstValue"; at the beginning, this is work correctly. how you can explain that?

Thanks for the explanation. but @elgonzo when I overwrite field.Value = "firstValue"; at the beginning, this is work correctly. how you can explain that?

Because you are using a string literal here. String literals are by default interned.

@anc-dev-lk
Is your question answered?

@StefH
It is a bug. do you have any plan to fix that?

@StefH

I tried to reproduce your issue with this unit test, but I could not reproduce it.

That is because the left-hand side of the query (the firstName identifier) in the test method uses an interned string as its value, hence why it fails to reproduce the issue. The value is is an interned string in the test because the value specified is a string literal (Value = "firstValue").
To reproduce the issue, you will need to avoid using string literals here, as string literals are interned by default. One way to get around this for the test is using one of the string constructors to dynamically create a string, or use any other string function that produces a new string instance dynamically (such as string.Concat, string.Substring, etc...).
So, instead of Value = "firstValue", try doing either of this:
    // Arrange
    var cc = "firstValue".ToCharArray();
    var field = new
    {
        Name = "firstName",
        Value =  new string(cc)
    };

or
    // Arrange
    var cc = "firstValue".ToCharArray();
    var field = new
    {
        Name = "firstName",
        Value =  "first".Concat("Name")
    };

or any other code construct that avoids creating an interned string...
dotnetfiddle showing the test failing when using a non-interned string for field.Value: https://dotnetfiddle.net/DHvT1f

Hello @anc-dev-lk and @elgonzo ,
I can reproduce the issue using that example code and you analysis is correct.
Another part of the issue is this code:
var dynamicClass = (DynamicClass) Activator.CreateInstance(type);

Because the result from CreateInstance (which is actually another type : <>f__AnonymousType0`1[System.String]), the information about the real properties, a string property, is lost.

If your use case is this:
I have following code. From external source I'm getting field.Name = "firstName" and field.Value = "firstValue".
The solution for your issue is just passing the data (field), create a list of field entities and query on the "Value", like this:
        var cc = "firstValue".ToCharArray();
        var field = new
        {
            Name = "firstName",
            Value = new string(cc)
        };

        var array = new[] { field }.AsQueryable();

        var isValid = array.Select("it").Any("Value eq \"firstValue\"");

        isValid.Should().BeTrue();

https://github.com/zzzprojects/System.Linq.Dynamic.Core/pull/679

Another part of the issue is this code:
var dynamicClass = (DynamicClass) Activator.CreateInstance(type);
Because the result from CreateInstance (which is actually another type : <>f__AnonymousType0`1[System.String]), the information about the real properties, a string property, is lost.

No, that's incorrect. The type information of the firstName property is not lost in the created instance of the anonymous type. The created anonymous type has exactly one firstName property of type string, which can also be reflected upon. Dotnetfiddle: https://dotnetfiddle.net/mhSo9U.

As i mentioned in my comment above the problem is related to using the type List<DynamicClass> as queryable source. To demonstrate and prove this, look at this dotnetfiddle: https://dotnetfiddle.net/dKC5XH. It will yield true for isValid. It still uses var dynamicClass = (DynamicClass) Activator.CreateInstance(type) as DynamicClass;, so this line itself is not an issue.
Rather, instead of using a List<DynamicClass>, it creates a List<T> of the type of the dynamic instance and adds this instance to the list. (It's not pretty because it required reflection to create such a list, though). As the dotnetfiddle demonstrates, avoiding List<DynamicClass> avoids the problem. (That said, i do not recommend this reflection-based approach, the dotnetfiddle is just to prove that the problem is not with var dynamicClass = (DynamicClass) Activator.CreateInstance(type);)
But, of course, var dynamicClass = (DynamicClass) Activator.CreateInstance(type) as DynamicClass; is practically a stepping stone leading to the problemtic code using List<DynamicClass>, because how could one create a queryable of the type of the dynamically created object without resorting to "insane" reflection shenanigans...

@elgonzo
Thank you for the explanation.
The problem is this example for 'Any' (using List) is that the Type, in the Any method, is retrieved using the ElementType which is DynamicClass.
A solution for this is that the real type is provided as parameter to the Any method.
Or getting the first element from the IQuerable and call GetType(). However I am not sure this can work will a real IQueryable which accesses a DB.
However the question is if this a real problem in an application or a more technical -what if- issue.

Hello @anc-dev-lk and @elgonzo,
There are several solutions for this issue:

Passing the data (field), create a list of field entities and query on the "Value".

Use a list from the real type:

var customListType = typeof(List<>).MakeGenericType(dynamicInstance.GetType());

Use ToString()

var isValid = query.Any("firstName.ToString() eq \"firstValue\"");

Use a cast to string:

var isValid = query.Any("string(firstName) eq \"firstValue\"");

authors: StefH anc-dev-lk elgonzo


text:
Dev

注： publish 改为 PUBLISH

sql语句中的注释不对啊，请改成正确的注释或者去除

my fault, 应该用idea编辑的

authors: GPSGH zzzzbw

repo: zzzzbw/Fame


text:
[Bug - Settings] Switching Skins during Song Enumeration process will stop enumerating songs

Fixed by #413


text:
feat: use title in streaming from layout api

This is desktop, height for the input has decreased

On mweb the emojis are open by default

The sheet content does not take full width

There is a gap above the first chat

I have fixed the chat footer in a different PR.


text:
控制台使用更好看的字体

0.8版本


text:
Simple Router Unit Tests in Python

@Lastdayends @854768750 @soneryaldiz I've created some very simple routing tests.
Here is one that I think should pass but doesn't currently:

There is space to extend the fourth pin on each side up so that another M2 can be used to connect them.
To run these tests try:
export ALIGN_HOME=`pwd`
mkdir -p work
export ALIGN_WORK_DIR=$ALIGN_HOME/work
pytest -- tests/pnr/test_router_unit.py

Then
cd $ALIGN_WORK_DIR/four_horizontal_wires_extend_routing_unit_tests
schematic2layout.py anything_can_be_here -s FOUR_HORIZONTAL_WIRES_EXTEND --flow_start 3_pnr --skipGDS --viewer

(A viewer will be available to see the results.)

@stevenmburns , this looks great. The boiler-plate code could be separated to a common utilities script.
Could the test case definition also be simplified to a single canvas? The instance and the corresponding formal actual map could be auto-generated from the connected components in the canvas - which is can be identified by the checker.

Example:
from align.pdk.finfet import CanvasPDK

def test_case_101()
  nm = get_test_id()
  cv = CanvasPDK()
  # Add bunch of named terminals to the canvas
  cv.addWire()
  cv.addVia()
  generate_run(cv)

def generate_run(cv):
  cv.remove_duplicates()
  new_terminals = list()
  for opens in cv.rd.opens:
     # add a new formal actual pair for the connected terminals
     # rename these terminals and append to new_terminals

@Lastdayends @854768750 @soneryaldiz I've created some very simple routing tests. Here is one that I think should pass but doesn't currently:
There is space to extend the fourth pin on each side up so that another M2 can be used to connect them. To run these tests try:
export ALIGN_HOME=`pwd`
mkdir -p work
export ALIGN_WORK_DIR=$ALIGN_HOME/work
pytest -- tests/pnr/test_router_unit.py

Then
cd $ALIGN_WORK_DIR/four_horizontal_wires_extend_routing_unit_tests
schematic2layout.py anything_can_be_here -s FOUR_HORIZONTAL_WIRES_EXTEND --flow_start 3_pnr --skipGDS --viewer

(A viewer will be available to see the results.)

@stevenmburns
I can see there are only 3 m2 tracks for the pin connection. And all those 3 tracks are used. Can you show me your expectation for the forth pin here? Manual draw is fine for me to understand what extension means.

@Lastdayends @854768750 It seems to work okay if the terminals are positioned a little differently.  Here the pins on the right start at the top and go down. The pins are the left are the same. It works in this case, extending a pin on the left upward and a pin on the right downward. But in the previous case it didn't extend pins on both sides upwards.

@stevenmburns 'Extend pins', do you mean the pin extension function of P&R?

Yes, the pins in the input to P&R are all the same length. It the second case (left pins aligned closer to the bottom, right pins aligned closer to the top), the P&R engine extended the pins so that four nets could be routed. That didn't happen in the first case (both the left and right groups are aligned on the bottom.)

I want this:

I want this:

It seems the pin extension functions are removed. Let me try to recover it.

@Lastdayends How are the two cases different from each other? There is something that is extending pins in the second case above.

@stevenmburns
In the second case, I guess the router find a path for the 4-th pin. Finding a path is not extending the 4-th pin.
We used to have a function called pin_extension, which is going to extend pins after reading lef file but before router. Now this function is no longer there.
Do you want to solve the problem by recovering the pin_extension function or debug the router for the first case about why a path is not find?

It is up to you how you fix it. It seems that we will want the router to sometimes find paths that include wrong way routes. There may be other situations where they are required. I'll try to generate a new testcase that requires one.

@stevenmburns
I see.

Should be ready to check in.

authors: Lastdayends soneryaldiz stevenmburns

repo: ALIGN-analoglayout/ALIGN-public


text:
#17: Proper scaling factor in zoom slider

About Index page, seems some css bring to parent component instead of own component, no major changes. I think, it's better to maintain clean code on top level component. Attached both code structure as screenshot. What do you think @kirillt ? Also, we can hide zoom bar for gallery view by a simple login with galleryStore.galleryView store.

There is not really a need to maintain "clean-code" at top level component. Later, we want to remove the explicit check
class:hidden={!$galleryStore.galleryView}
and handle it with sveltekit rounding, but we can do that in a sepeate pr
In sveltekit, usually any CSS styles that are related to the page layout, (meaning margins, paddings, flex ect) should be in the layout component. This allows for the components of the page to be more pure and more reusable.

@alvinosh now it feels working to me, thanks.
@prokawsar does it look good enough for you? I have no idea about JS and Svelte code, but if something looks suboptimal but not critical we can merge and raise this question on dev sync.

authors: alvinosh kirillt

repo: ARK-Builders/ARK-Gallery


text:
update ppp

LGTM


text:
More bug fixes and list command modification

Codecov Report
Patch coverage: 50.00% and project coverage change: +0.76 :tada:

Comparison is base (c323a84) 24.07% compared to head (55b2740) 24.84%.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #115      +/-   ##
============================================
+ Coverage     24.07%   24.84%   +0.76%
- Complexity      500      512      +12
============================================
  Files           220      220
  Lines          5200     5208       +8
  Branches        622      622
============================================
+ Hits           1252     1294      +42
+ Misses         3882     3843      -39
- Partials         66       71       +5

Impacted Files
Coverage Δ

...s/logic/commands/RemovePartFromServiceCommand.java
0.00% <0.00%> (ø)

...mmands/RemoveTechnicianFromAppointmentCommand.java
0.00% <0.00%> (ø)

...c/commands/RemoveTechnicianFromServiceCommand.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
25.86% <0.00%> (-0.46%)
:arrow_down:

.../RemoveTechnicianFromAppointmentCommandParser.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
31.96% <ø> (+9.49%)
:arrow_up:

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

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

authors: codecov-commenter ginloy


text:
List expense by category Closes #49

Tag this to the relevant issue(s) by using the term closing #issue_number

@jinbesan

closes #49


text:
Add Appendix: Instructions for Manual Testing into DG

Completed in #229


text:
Update UG ClearCommand

LGTM

authors: glenngnng syamfarh


text:
fix: move echo statements refactor: merge workflows

:tada: This PR is included in version 1.6.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

authors: adaction-cicd dakotawashok

repo: AdGem/adgem-sdk-unity-package


text:
[anthropic] add to condaforge

blocked by https://github.com/Arize-ai/openinference/issues/953

repo: Arize-ai/openinference


text:
Check we are using sign-in not log-in

@libbyv This looks to be a duplicate of #200?

https://github.com/govau/content-guide/issues/33

authors: joolswood klepas libbyv

repo: AusDTO/gov-au-content-guide


text:
Implement the HR plan screens

HR Plan is implemented at UI level and code committed to repo over the weekend.

repo: Avinya-Foundation/SMS-HRMS-API


text:
Prepare Model-Adapter-Controller architecture

Codecov Report
Base: 70.08% // Head: 70.08% // No change to project coverage :thumbsup:

Coverage data is based on head (c0e6888) compared to base (bd19297).
Patch coverage: 57.14% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##           master       #6   +/-   ##
=======================================
  Coverage   70.08%   70.08%
=======================================
  Files           5        5
  Lines         224      224
  Branches       28       28
=======================================
  Hits          157      157
  Misses         43       43
  Partials       24       24

Impacted Files
Coverage Δ

src/setup-renpy.ts
0.00% <0.00%> (ø)

src/utils.ts
76.92% <ø> (ø)

src/adapter/parameters.ts
89.47% <100.00%> (ø)

src/controller/executor.ts
86.95% <100.00%> (ø)

src/controller/installer.ts
73.61% <100.00%> (ø)

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

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

authors: Ayowel codecov-commenter

repo: Ayowel/renpy-setup-action


text:
Prepare OTel exporter release beta.23

API change check
API changes are not detected in this pull request.

authors: azure-sdk jeanbisutti

repo: Azure/azure-sdk-for-java


text:
AutoPr-Storage-zfchen95-REST Spec PrNumber 5559

Closing since Rest Spec PR is closed

authors: adxsdknet dsgouda

repo: Azure/azure-sdk-for-net


text:
Parse SAS connection string correctly for DFS endpoint

I'm not sure if this is going to work - https://docs.microsoft.com/en-us/rest/api/storageservices/naming-and-referencing-containers--blobs--and-metadata#blob-names
"A blob name can contain any combination of characters."
I think this means someone could name their blob "my.blob.3"

these aren't blob names - its the endpoint

repo: Azure/azure-sdk-for-python


text:
[SB] update stress tests -- cleanup

API change check
API changes are not detected in this pull request.


text:
upgraded to Node16

#47

repo: Azure/setup-kubectl


text:
Update SRI hashes for msal@1.4.2 and browser@2.4.0

Coverage decreased (-0.08%) to 82.901% when pulling 2edf8ba703ad49c78f548e17ace9ee494abf30f1 on browser-core-1021-sri-hashes into 98a31a04ad907976aeba96108c7c844890fcfcd1 on dev.

authors: coveralls jasonnutter


text:
Update app initialization doc

A couple suggestions to add clarity. Would also be good to start a migration doc where we can enumerate all the changes between v2 and v3.

Good idea. I will add a migration doc in a separate PR. Thanks for review!


text:
The top N populated countries in the world where N is provided by the user.

this wont get done, prioritizing other areas of the project.


text:
AdminListeUtilisateur, AdminVoirUtilisateur, AdminModifierUtilisateur, AdminSupprimerUtilisateur

stylé

authors: arthur-mdn mathis-lambert


text:
Add solution for Leetcode problem 206. Reverse Linked List

Added the PR #76


text:
a readme file is required.

@wordssaysalot  can you work on it

sure, i will work on it

great


text:
Doc tweaks

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (adc97c1) 95.79% compared to head (ffa3fc6) 95.67%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #72      +/-   ##
==========================================
- Coverage   95.79%   95.67%   -0.13%
==========================================
  Files          16       16
  Lines        1664     1664
  Branches      414      414
==========================================
- Hits         1594     1592       -2
- Misses         34       35       +1
- Partials       36       37       +1

Flag
Coverage Δ

unittests
95.67% <ø> (-0.07%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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

authors: BrianPugh codecov-commenter

repo: BrianPugh/cyclopts


text:
Add update geocode page

We should be able to adapt the current geocode creation interface to use for updating as well.

But surely they won't change all details more just available and hints. I don't think we should allow them to change locations. Was just going to make a pop up or modal on the profile page

Good point. Since the geocode profile page uses the new modular geocode details component, we can just add a hide-able button to it (same way the QR code button is currently implemented) to determine if an edit button should be visible. Then if it is clicked, bring up a modal for editing

authors: JennaLynGallagher robotic-coder


text:
Development

Refactor Map to load data from a file and add Manifold method for collision detection

repo: Cairhiin/TheDarkBelow


text:
CC-BY licensing of this repo

here's the CFF generator: https://citation-file-format.github.io/cff-initializer-javascript/#/

Now we also have a name: the teachingRSE project . We can also this in our license file.

Any updates over here?

Re-checked and removed the CFF file which will be a separate PR addressing #253. I guess you can merge that now, @CaptainSifff.

authors: CaptainSifff hvwaldow


text:
chore: add MIT license

@ctjlewis Thank you!

authors: ChangoMan ctjlewis


text:
Update pipeline.yml

bors r+


text:
Add a save button and functionality to the profile page to save the user details.

Right now every update saves the profile automatically

I think automatic update will be better

repo: ClioDynamics/cojourney


text:
V1.9

fixed donation list showing old items now as well

repo: CodyWallbridge/DirectDeposit


text:
Make sure we do not ignore repring failures caused by incorrect computation

@mrocklin are you ok with this? we only want to try to repr things as tables that have all these properties

one dimensional
only record or scalar measure

but we never want to fail, because we'll always show something rather than an error message, which makes it really hard to know what is going wrong when things inevitably do go wrong

Yeah, this seems like a good change to me

merging on pass

authors: cpcloud mrocklin

repo: ContinuumIO/blaze


text:
remove trailing space\

We decided trim will not work here

authors: tcosentino zaher-aa

repo: Contuit/Acromyrmex


text:
Minor README.md fixes

@isimluk looks like there is an issue with pylint.

I noticed. Thanks God we have him.

authors: isimluk redhatrises

repo: CrowdStrike/falcon-integration-gateway

license: Unlicense


text:
fix: Add video length field

:tada: This PR is included in version 1.32.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

authors: cairnsj rgparkins

repo: DFE-Digital/childrens-social-care-cpd


text:
integrate frankencoin in a loyaltyprogram with userbase >100k users

Interested in finding out more.

Happy to have a chat
Toni caradonna
Operal.solutions
Von: TaprootFreak @.>
Gesendet: Dienstag, 12. März 2024 15:05
An: DFXswiss/frankencoin @.>
Cc: OperalAG @.>; Author @.>
Betreff: Re: [DFXswiss/frankencoin] integrate frankencoin in a loyaltyprogram with userbase >100k users (Issue #5)
Interested in finding out more.
—
Reply to this email directly, view it on GitHub https://github.com/DFXswiss/frankencoin/issues/5#issuecomment-1991730089 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ATHOYFKX4U6YSZFHAC2Z7ADYX4DPVAVCNFSM6AAAAABESJE4KWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOJRG4ZTAMBYHE .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/ATHOYFPLKCGDRWRMCRJD3ZLYX4DPVA5CNFSM6AAAAABESJE4KWWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTWW5R2S.gif Message ID: @.*** @.***> >

authors: TaprootFreak operalag

repo: DFXswiss/frankencoin


text:
Local Account Token Filter Policy

@dtcjoe  can you add details to your pull request? Why do you want this added? What does LocalAccountTokenFilterPolicy do?

I have added a script to msft-windows that creates a dword 32 registry entry at  HKLM:\SOFTWARE\Microsoft\Windows\CurrentVersion\Policies\System name LocalAccountTokenFilterPolicy with a value of 1. This disables UAC remote restrictions which often causes issues when attempting to perform backups.

I have added a script to msft-windows that creates a dword 32 registry entry at HKLM:\SOFTWARE\Microsoft\Windows\CurrentVersion\Policies\System name LocalAccountTokenFilterPolicy with a value of 1. This disables UAC remote restrictions which often causes issues when attempting to perform backups.

Are the backups your performing outside of a domain environment because this change can pose a security risk to the environment. Can you include how your organization is compensating by enabling this?

To add to Joe, the backups in question are being performed on Hyper-V hosts which are outside of the domain environment. This is done for security where the security isn't enforced by Policy. The passwords on the Hyper-V hosts are randomly generated. In the future there are plans to rotate them with LAPS.
We're using Veeam to its full potential.

To add to Joe, the backups in question are being performed on Hyper-V hosts which are outside of the domain environment. This is done for security where the security isn't enforced by Policy. The passwords on the Hyper-V hosts are randomly generated. In the future there are plans to rotate them with LAPS.
We're using Veeam to its full potential.

Perfect. Approving and merging request.

authors: B-Moriik dtcjoe nettts

repo: DTC-Inc/msp-script-library


text:
JS: Dialog laten openen en closen

done


text:
[WIP] Update docker runner tag

Closing due to #295 and stale.

authors: duncanista maxday

repo: DataDog/datadog-lambda-extension


text:
v

c

repo: DataRohit/Stockastic


text:
(comment-newimpl): Updating Option to Option

Can this be separated from the work on package parameters? That would make a nice clean diff.

Yes of course, I'm closing this PR and will create another clean PR checked out from origin/master.

authors: DaveMcEwan vasilissoti

repo: DaveMcEwan/svdata


text:
Fixed news API bug

Im testing out the headlines tab on the website, but im getting an error when I click on the tab.

There are two thing that may be causing this.

Make sure you perform a flask db upgrade before testing it out.
Go to the news settings page before going to the news page. The news settings page attaches a news db table to the current user and I'm realizing now that the news table may be queried before going to the news settings page.

The second option is another bug I'll need to fix before merging.

I fixed the bug.

should this be expected?

I updated the env file with the new keys and updated the database so idk if this is normal for now

Yes and no. I've never seen a 'no key given message' before. But you should get either an 'invalid key' or 'call limit reached'.
I'm guessing that may have something to do with your .env file or config.py. When you updated your .env file, did you download it and not add the '.' before the file name? or did you copy and paste contentions into your existing file?
Try this one.
SECRET_KEY=c7cb4c6a10694d9287ab7b5f232959d1
NEWSCATCHER_API_KEY=6-w_rxdCWCIiAqtMQsRQb1BfTt2DdggZu8Bs7h2qUPk
SENDGRID_API_KEY=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
MAIL_DEFAULT_SENDER='Jerry.Aragon@student.csulb.edu'
ALPACA_API_KEY = "PK7HV61VZ6DSUZQW2JGO"
ALPACA_SECRET_KEY = "FKKC0fWQe9NrgrmHbgApvuBPUkU39pFUfaUja0JW"

I should work just fine, just don't go to the news page too many time, because the call limit for that key is limited.
Here are all of the keys for news catcher that I've gotten.

The first one is the original key that no longer is valid.
The second is the one that has its limit reached that is published on discord.
The third I haven't really published since its has limited call and I'm saving it for demo days.

I just tried removing the '.' in the .env file name and got the same error message.

I think you just need to fix the .env file name.

authors: J-Kid-Beast tmsoc


text:
add in-sar-cubes.c-scale.eu project

Thanks!
Enrollment URL for fedcloudops account:
https://perun.egi.eu/egi/registrar/?vo=in-sar-cubes.c-scale.eu

Pending #232

authors: enolfc jiri256 sebastian-luna-valero


text:
Add book Symfony: Fast Track

remark-lint

repo: EbookFoundation/free-programming-books


text:
Add ORKO06

@ORKO06 Thanks for contributing to EddieHub Community 🚀. @schmelto this PR can be merged (:

@schmelto, I don't wanna sound rude or anything, but why did you merge this PR? I think the order is wrong, as B goes first, and K goes after B. I can make a PR to fix that if you want. 🙂

Heetesh comes before Hemant


text:
Wrapped TNumberBox Component

Thanks @shaunroselt

authors: lmbelo shaunroselt


text:
Add Enabled manifest placeholder

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#231  👈
#230
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rbro112 and the rest of your teammates on  Graphite

Closing in favor of #232

repo: EmergeTools/emerge-android


text:
Change Phoenix to Mosaic

No longer applicable

repo: Energinet-DataHub/opengeh-arch-diagrams


text:
Audio player Feature/lmsdemo 115

use VideoPlayer from components it is based on https://github.com/cookpete/react-player and does player video with tracks
you can rename this components to AudioVideoPlayer

authors: qunabu victazzz

repo: EscolaLMS/Components


text:
Fix tcp_connect() for non-Linux systems

This PR is simply wrong. Declining it

repo: F-Secure/async


text:
finish

Codecov Report

Merging #590 (9c36692) into main (79d9c1a) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #590      +/-   ##
==========================================
- Coverage   69.77%   69.77%   -0.01%
==========================================
  Files          97       97
  Lines       10423    10422       -1
  Branches     1243     1243
==========================================
- Hits         7273     7272       -1
  Misses       2865     2865
  Partials      285      285

Flag
Coverage Δ

unittests
69.77% <100.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ion_tools/migration_tasks/authority_transformer.py
61.90% <100.00%> (-0.89%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


text:
Autonomous Builder

https://github.com/FRC1360/Stronghold2016/pull/11
See Autonomous Branch for more info

authors: JamieSinn Megapead


text:
Fix connectivity error

In the future, you should also author release notes for connectivity browser, for better transparency on what changed.

Sure, I'll take it in mind.


text:
Inventory Item functionality (dropping)

Completed on items branch

repo: FailedIndustry/Industrial-Failure


text:
refactor(routes/pdf/txt): remove workaround for conditional ocr prop

Pull Request Test Coverage Report for Build 1707649636

-2 of 5 (60.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 99.614%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/pdf/txt/schema.js
1
3
33.33%

Totals

Change from base Build 1683314575:
-0.4%

Covered Lines:
566

Relevant Lines:
568

💛  - Coveralls

Pull Request Test Coverage Report for Build 1707649636

3 of 5 (60.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 99.614%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/pdf/txt/schema.js
1
3
33.33%

Totals

Change from base Build 1683314575:
-0.4%

Covered Lines:
566

Relevant Lines:
568

💛  - Coveralls

authors: Fdawgs coveralls

repo: Fdawgs/docsmith


text:
chore: remove dry_run from release workflow

This PR is included in version 0.2.1 :tada:

repo: FidelusAleksander/gh-action-regex


text:
Increase Trait value field length

This is currently documented as being set to 2kb - I think that is sufficient.

repo: Flagsmith/flagsmith


text:
Merge update

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
LevanisART, @radonn24, @dulumao, @devArno88, @wisdomabioye, @MarshallChang, @onepisYa, @py7hon, @4pmtong, @chapter802, @real-lazy-coder, @haythemsellami, @emg110, @AdemKao, @agnescameron, @Nach0Git, @lyknice23, @0xYakuza, @lichenysq, @gHashTag

authors: FlyingShuriken Talkingbaby


text:
Add files via upload

p;l[


text:
[core,nla] print name of error code in log

@freerdp-bot test properly!

repo: FreeRDP/FreeRDP


text:
Define 'withExercise' config inside Constants.ts together with all other lesson definitions

Done!


text:
Use asyncio instead of threads

Nice :ok_hand:

LGTM

[ ] Manual tests

authors: Fruktus kjarosh

repo: Fruktus/QuadradiusPreservationProject


text:
fix(file): Support Nest 9

Pull Request Test Coverage Report for Build 6c78ec10-c69a-433c-8fab-47086903f753

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

Totals

Change from base Build fb410655-2b23-44d8-ad2f-a49b69d52535:
0.0%

Covered Lines:
44

Relevant Lines:
45

💛  - Coveralls

:tada: This PR is included in version 1.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: Fuse-Autotech/nest-timeout


text:
Multiple Selection

It should not be possible to select any location more than once.

Will hold back on this at least until after OpenLayers port.

This should be fixed.
Location.select in the listview checks for an existing record with the same layer and id.
If found the record is cleared and the select function returns.

authors: RobAndrewHurst dbauszus-glx

repo: GEOLYTIX/xyz


text:
Create license

0

repo: GabrielleKw/gabekw


text:
Created HTML

This is the uploaded HTML from Visual Code

repo: Gheost/SurveyForm


text:
Added image format/compression options to geopackage adapter - #126

Coverage increased (+1.57%) to 57.41% when pulling 5331b63fb775724211899ab98d7b0f3b3029d9fe on fix-UI-126 into 939d15421113f47fedaeb1a826588a9f4b865ffd on master.

Looks good!

authors: Jal3k5 LLambertRGi coveralls

repo: GitHubRGI/swagd


text:
Export icons

:tada: This PR is included in version 1.8.14 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: Giveth/ui-design-system


text:
chore: enable secret_scanning

bumped github provider in lock file to resolve API errors

applied/no-op


text:
Chanjin

:tada: This PR is included in version 1.3.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: Gosrock/Gosrock-storybook


text:
chore: add the owner of "/tensorbay/conftest.py" in CODEOWNERS

Pull Request Test Coverage Report for Build 1410769140

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

Totals

Change from base Build 1407647658:
0.0%

Covered Lines:
6937

Relevant Lines:
8178

💛  - Coveralls

authors: coveralls linjiX

repo: Graviti-AI/tensorbay-python-sdk


text:
Rerun last experiment with 90 epoch cosine lr decay

Done /close

repo: Grutschus/human-fall-detection


text:
add file api design(add to wishlist)

Thank you @obeidahr for your PR
good start,
There are some notes to take care of:

you don't to create new file, move your code to apis.yaml and delete [API Design] add to wishlist.yml file.
it would be great to re-read API Spec's contributing guideline and confirm your code.
you should use /wishlist/{productId} endpoint, with POST method for add
you only need to define productId parameter, in path, as int, no need for requestBody
your 200 response does not have to return content (there is nothing to return except status code 200)

Please replay if you see any point is not logical, you can also discuss it in the group chat.

Thank you @obeidahr
you are getting better 👍
however, we still need to add productId in path as /wishlist/{productId}
and use the method POST (for adding) instead of put (for update)
and use tags: wishlist instead of user
and add parameters like in delete from wisthlist
and finally, move components: section to the left one step (two spaces)

authors: haithamalnaeb obeidahr

repo: HibatApp/hibat-backend


text:
Delete old formats and make SystemFormat the only format exist

Note the test fail with import error on SystemFormat.

authors: elronbandel yoavkatz

repo: IBM/unitxt


text:
The swinb checkpoint is downloaded slowly,can you upload a HFlink?

A HF link has been updated: https://huggingface.co/ShilongLiu/GroundingDINO/resolve/main/groundingdino_swinb_cogcoor.pth

authors: Rickustc SlongLiu

repo: IDEA-Research/GroundingDINO


text:
Remove Column Data Types todos and export false cause the sample exports just fine

please do not change.remove "export" field in xplat samples

authors: HUSSAR-mtrela mddragnev

repo: IgniteUI/igniteui-xplat-examples


text:
Test opening an issue for usnistgov/OSCAL#1936 (test reopening too)

This seems to work.


text:
Input require a property placeholder making some pages return a null error

Closing this issue since it same like  #15

repo: Internity-dev/Internity


text:
Profile page tab not working

This issue is fixed now.


text:
H4T2: Implementación de funciones para hallar rutas

Se complementará la codificación durante la fase de revisión del hito final


text:
Add son.txt

@Alligato08 son.txt I am sorry, I have to close the request without merging, because you made a mistake in a request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!

repo: JetBrains/swot


text:
Uoh

@farrukh1212cs I am sorry, I have to close the request without merging, because you made a mistake in a request and requested to merge too many files, most of them do not relate to the organization which you would like to add to this repository. Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


text:
Add test user to database

Completed


text:
Send logging messages to terminal instead of file

Done: https://github.com/KBNLresearch/pdfquad/commit/9c10ec2ebfa6df8276517435234c5144b4799baa and https://github.com/KBNLresearch/pdfquad/commit/cd38b6816a3ceca38053ed914157cff083eb6807

repo: KBNLresearch/pdfquad


text:
Machine Learning

Machine Learning


text:
在4096x4096的情况下精度对不上

Thank you, i will check soon.
Today, i have no time to deal with, it might take more than a day.

good

i just made a commit, could you check is it work?

thank you，you are very good.give me big help,next step, i want to try int8

repo: Kitsunetic/GEGLU-triton


text:
chore(ci): add basic CI structure

do we plan to work again on this or we should close it?

repo: Kong/gojira


text:
Verificar a possibilidade das requisições das imagens passar pelo servidor primeiramente.

Foi aplicado uma verificação das imagens como previsto em
https://github.com/KoreaComK/visao-libertaria-site/commit/b841addcd6b63e331bc74090c7a676e5b27c8427
Foi revertido devido ao tempo de carregamento da página aumentar exponencialmente conforme a quantidade de imagens que ela possui, inviabilizando a verificação da imagem. Reversões feitas em
https://github.com/KoreaComK/visao-libertaria-site/commit/eff4de76a910bdeb06bf1a8bd0b24df5bcab4192
https://github.com/KoreaComK/visao-libertaria-site/commit/4ed5de9bfb22c9a5d631fb214e52c31325540431

repo: KoreaComK/visao-libertaria-site


text:
final draft for login

ayus bro! 💯

To follow link to login page from nav.

authors: Kristoff-Villanueva jryambao

repo: Kristoff-Villanueva/first-ecommerce


text:
Added React Project that searches Github users

Thanks for your valuable contribution.
Don't forget to give a star to this repo.

sure.


text:
split_#7 #1050_yaswanth.html

Heartfelt Thanks for Your Contribution to Our Open Source Project!
Dear @ymulakala27 (Yaswanth Mulakala),
I trust this message finds you in good health and high spirits. I am reaching out with a heart full of gratitude for your recent contribution to our open source project. Your dedication has not only elevated the functionality of our project but has also exemplified the collaborative spirit that lies at the core of thriving open source communities.
A heartfelt congratulations are in order for your exceptional work! Your commitment and expertise have played a pivotal role in the success of our project, and we consider ourselves truly fortunate to count you among our community members.
I must extend my sincere apologies for the delay in responding to your pull request. Your patience and understanding during this time are immensely appreciated. As our project continues to grow, we acknowledge the imperative need to enhance the quality and response time for contributions. To address this, we've kickstarted a call for maintainers and reviewers who can help us uphold the high standards of our project and ensure swift responses to pull requests.
Your involvement and insights would be invaluable in this initiative. If you, or someone you know, is interested in taking on a more active role in the project, please don't hesitate to reach out. We firmly believe that with a robust team of maintainers and reviewers, we can elevate both the quality of the project and the speed of our responses.
On an exciting note, I am thrilled to share that we have activated the sponsorship button for our project. This development allows supporters like yourself to contribute to the sustainability and growth of the project. Your ongoing support, whether through code contributions or sponsorship, is genuinely appreciated.
Once again, congratulations on your outstanding contribution, and thank you for being an indispensable part of our open source community. We eagerly anticipate your continued collaboration and shared success.
Best regards,
Ntem Kenyor K.
Lead
Kwickerhub

repo: KwickerHub/WebCraftifyAI


text:
Test old vs new version of quadratic point is left

3x to 30 speedup


text:
Added tasks 121-146

👍

authors: ThanhNIT javadev

repo: LeetCode-in-C/LeetCode-in-C


text:
1.8.6 patch

sexy lol, found LA ISSUE

😒

authors: JasonLovesDoggo LevaniVashadze

repo: LewisProjects/Ogiroid


text:
Fix error in documentation (closes !148)

This looks fine to me based on what you wrote in the issue. I have not (and don't plan to) test it in any way.

repo: LibreTexts/ngshare


text:
Fix deserialization to work with meta-circular language definitions

What is a meta-circular language definition?

E.g. a language definition that defines an annotation which is used to annotate the language itself.

authors: dslmeinte joswarmer

repo: LionWeb-io/lionweb-typescript


text:
After downloading and converting the file from Listenbrainz to a .mp3, it doesnt seem to delete the .webm file it downloads

Hey, could you provide some more info, such as:
What operating system are you running?
Are you running docker or the binary?
Does it fail to delete every .webm file or just a few?
If you're running docker can you provide a output of docker logs explo?

Been trying to test this for you, but havent actually gotten it to finish downloading a whole playlist. Keeps giving me a 403 error when trying to find songs.

I assume this is for Youtube.
This can happen for 3 reasons:

Your Youtube API Key is wrong.
If you make too many requests in a short period of time, Youtube can rate-limit you.
You have no Youtube API credits left (you can make 100 total requests per day).

Test your API key validity run this CURL command command:
curl "https://youtube.googleapis.com/youtube/v3/search?part=snippet&q=test&type=video&videoCategoryId=10&key=YOUR_API_KEY"
Don't forget to replace YOUR_API_KEY with your actual key, if you get a json response you should be good
Explo already makes 50 requests (by default) in a short period of time, if you have other tools that also access Youtube, you could get rate-limited. You can check your API usage in the Google Cloud admin page

Also in the newest version the .webm file will get deleted right after ffmpeg converts it to .mp3 (previously it deleted extra files if all songs had been downloaded)

hmm, that seems to be working. haven't been able to get it to add the virtual playlist to emby though. does it support emby?

Unfortunately Explo doesn't support Emby yet, but if I have the time I'll for sure add support for it

repo: LumePart/Explo


text:
Testing GROUPs

Codecov Report

Merging #2 (596f1f5) into main (98da756) will decrease coverage by 0.91%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main       #2      +/-   ##
==========================================
- Coverage   62.53%   61.62%   -0.91%
==========================================
  Files          13       13
  Lines         347      344       -3
==========================================
- Hits          217      212       -5
- Misses        130      132       +2

see 4 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

authors: avik-pal codecov-commenter


text:
Implement Chinese Traditional to MCSManager 10

冒昧請問一下，您是？

代码是否已经经过测试证实可以运行并且使用？

代碼已經過測試證實可以運行並且使用 繁體中文是使用Google翻譯的

代码是否已经经过测试证实可以运行并且使用？你的繁体中文是工具一键转换的吗？速度这么快？

這樣顯得人工沒優點

代码是否已经经过测试证实可以运行并且使用？你的繁体中文是工具一键转换的吗？速度这么快？

~這樣顯得人工沒優點~

我先合并他的PR，如果你想矫正翻译，可以在此基础上修改。

代码是否已经经过测试证实可以运行并且使用？你的繁体中文是工具一键转换的吗？速度这么快？

~這樣顯得人工沒優點~

我先合并他的PR，如果你想矫正翻译，可以在此基础上修改。

Okok

repo: MCSManager/MCSManager


text:
Add verbose (or logging) option for debugging

Working with: e0d8b1c8e7bc1db08f95ff2fad30782cae5263b1

Tested and working on Mac, WSL and Ubuntu with 6ddf5eac1ed01dcf29df194acb750ac5dcb63e74


text:
add docs for HPO on SLURM

@EthanMarx heard that you are in transit. I'll pull this in. And in fact it might be easier to read the rendered docs.


text:
Fuck you codacy

Test this at WireGuard-KT-63-2cbda4f06c946c3a21c193a823e45c6c4e736a71.apk.
This is an automatically posted message from Travis-CI

Test this at WireGuard-KT-63-543d51b80b4109988facb1f0d078700ee2081188.apk.
This is an automatically posted message from Travis-CI

Test this at WireGuard-KT-63-924354395cfd47a55b6481494bf0c898b1311026.apk.
This is an automatically posted message from Travis-CI

Test this at WireGuard-KT-63-d2e470876e49f3a7e5f92badd850ffb5c0046714.apk.
This is an automatically posted message from Travis-CI

repo: MSF-Jarvis/wireguard-kt


text:
Initial commit

Good job!

authors: MatejMecka niojam


text:
Only save guest_id to comments model if user is not authenticated

Looks good to me. Will put this in the next release

authors: BataBoom Matildevoldsen

repo: Matildevoldsen/wire-comments


text:
From private repo

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

repo: MicrosoftDocs/azure-stack-docs


text:
Build pipeline

commit


text:
Bugfix/home page typo

@microsoft-github-policy-service agree


text:
[Doc Improvement][SEO1 Metatitle Update][3860121]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/app-icon-badging-for-your-app.md
90
5
link
:white_check_mark:

msteams-platform/apps-in-teams-meetings/build-extensible-conversation-for-meeting-chat.md
90
7
link
:white_check_mark:

msteams-platform/apps-in-teams-meetings/in-meeting-notification-for-meeting.md
92
7
link
:white_check_mark:

msteams-platform/apps-in-teams-meetings/teams-live-share-how-to/how-to-custom-azure-fluid-relay.md
91
9
link
:white_check_mark:

msteams-platform/bots/calls-and-meetings/debugging-local-testing-calling-meeting-bots.md
89
14
link
:white_check_mark:

msteams-platform/bots/calls-and-meetings/real-time-media-concepts.md
95
4
link
:white_check_mark:

msteams-platform/bots/calls-and-meetings/registering-calling-bot.md
88
20
link
:white_check_mark:

msteams-platform/bots/calls-and-meetings/requirements-considerations-application-hosted-media-bots.md
91
9
link
:white_check_mark:

msteams-platform/bots/how-to/authentication/bot-sso-code.md
86
21
link
:white_check_mark:

msteams-platform/bots/how-to/authentication/bot-sso-graph-api.md
88
5
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/app-icon-badging-for-your-app.md
90
5
link
:white_check_mark:

msteams-platform/apps-in-teams-meetings/build-extensible-conversation-for-meeting-chat.md
90
7
link
:white_check_mark:

msteams-platform/apps-in-teams-meetings/in-meeting-notification-for-meeting.md
92
7
link
:white_check_mark:

msteams-platform/apps-in-teams-meetings/teams-live-share-how-to/how-to-custom-azure-fluid-relay.md
91
9
link
:white_check_mark:

msteams-platform/bots/calls-and-meetings/debugging-local-testing-calling-meeting-bots.md
89
14
link
:white_check_mark:

msteams-platform/bots/calls-and-meetings/real-time-media-concepts.md
95
4
link
:white_check_mark:

msteams-platform/bots/calls-and-meetings/registering-calling-bot.md
88
20
link
:white_check_mark:

msteams-platform/bots/calls-and-meetings/requirements-considerations-application-hosted-media-bots.md
91
9
link
:white_check_mark:

msteams-platform/bots/how-to/authentication/bot-sso-code.md
86
21
link
:white_check_mark:

msteams-platform/bots/how-to/authentication/bot-sso-graph-api.md
88
5
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/app-icon-badging-for-your-app.md
90
5
link
:white_check_mark:

msteams-platform/apps-in-teams-meetings/build-extensible-conversation-for-meeting-chat.md
90
7
link
:white_check_mark:

msteams-platform/apps-in-teams-meetings/in-meeting-notification-for-meeting.md
92
7
link
:white_check_mark:

msteams-platform/apps-in-teams-meetings/teams-live-share-how-to/how-to-custom-azure-fluid-relay.md
91
9
link
:white_check_mark:

msteams-platform/bots/calls-and-meetings/debugging-local-testing-calling-meeting-bots.md
89
14
link
:white_check_mark:

msteams-platform/bots/calls-and-meetings/real-time-media-concepts.md
95
4
link
:white_check_mark:

msteams-platform/bots/calls-and-meetings/registering-calling-bot.md
88
20
link
:white_check_mark:

msteams-platform/bots/calls-and-meetings/requirements-considerations-application-hosted-media-bots.md
91
9
link
:white_check_mark:

msteams-platform/bots/how-to/authentication/bot-sso-code.md
86
21
link
:white_check_mark:

msteams-platform/bots/how-to/authentication/bot-sso-graph-api.md
88
5
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

repo: MicrosoftDocs/msteams-docs


text:
spec runtime lookup tables

Can we close this issue?

Don't forget to add an estimate to it before closing.

repo: MinaProtocol/mina


text:
Get notarization files for two APIs

@xiangxiecrypto can we close this one?

I think so.

authors: joseandro xiangxiecrypto


text:
龍魚ドリルの範囲の関数を立方根にする

クールタイムが伸びるようになったことで調整


text:
Wikiseite Fahrzeugverwaltung

Sah fertig aus, wird geschlossen.

repo: MultiKnacker/Projekt-WIN


text:
Handle malformed RGB HEX values

Hi @raxbg,
This has been fixed by #485 with an almost identacal code change. Additional tests were added with #359.
I'm going through the open PRs starting with those more recent, so got to #485 before this. It's reassuring to see that the change you proposed matches the change agreed upon there.
I think this can be closed as a duplicate, unless you think including the additional test here would add extra value - ?
Thanks.

Awesome! Closing this one.

repo: MyIntervals/PHP-CSS-Parser


text:
a couple of tweaks

make it an issue not on a previous merged PR

it's honestly quite likely that it's just not possible in the first place
there's a lot of things I have wanted to implement, but found that JCEF is just incapable of


text:
Интерфейс. Клиент. Отправка пакетов

При загрузке поля создаём клиент, получаем информацию  заполненности лобби
Ждём подключения второго клиента для игры


text:
add DB index on skills.storage.model.UserPoints#contributesToSkillsGroup, set skill_definition.enabled to true when null

PR: #952

repo: NationalSecurityAgency/skills-service


text:
add missing #[cfg(feature = "log")]

Thanks!

repo: Nazariglez/notan


text:
💤

desde cuándo estaba esto?


text:
Change all validation classes to internal

Done


text:
Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit b2d89f8:
:clock10: Full build: the average full build time is 6 min(s) 6 sec(s), this is based on the last 415 build(s) for this repository.

Docs Build status updates of commit b2d89f8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

authors: OPSTestPPE luyajun0205 v-susu


text:
Merge prTesting_202007231925 branch to master branch

Docs Build status updates of commit c42b9d3:
:warning: Validation status: warnings

File
Status
Preview URL
Details

split/updateTestCon.md
:warning:Warning

Details

split/updateTestCon.md

Line 0: [Warning-h1-not-first] Markdown content is not allowed before H1.
Line 1: [Suggestion-h1-missing] H1 is required. Use a single hash (#) followed by a space to create your top-level heading. NOTE: This Suggestion will become a Warning around July 2020.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


text:
03/27/2018 19:08 : Create Pull Request for branch prTesting_2018032719 -> master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


text:
Merge prTesting_202002061919 branch to master branch

Docs Build status updates of commit 8b38d42:
:warning: Validation status: warnings

File
Status
Preview URL
Details

split/updateTestRef.md
:warning:Warning

Details

split/updateTestRef.md

[Warning] Missing attribute: author. Add the current author's GitHub ID.
[Warning] Missing attribute: title. Add a title string to show in search engine results.
Line 1: [Suggestion] H1 is required. Use a single hash (#) followed by a space to create your top-level heading. NOTE: This Suggestion will become a Warning on 02/03/2020.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


text:
SMHE-71 Add keypair

Retest this please

Retest this please

Retest this please

Retest this please

Retest this please

Retest this please

Does the file firmwarefile0903.ota belong with this pull request?

No certainly not. It was used for some test. I've removed it.

repo: OSGP/open-smart-grid-platform


text:
fix emoji picker & beatmap packs i thikn

lmaoo

repo: Osekai/osekai-wip


text:
lint to make all composable functions internal

not important enough

repo: Otarium/App


text:
【Do not Merge】Add TripletAngularMarginLoss_XBM

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: HydrogenSulfate:x: rootroot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant HydrogenSulfate

repo: PaddlePaddle/PaddleClas


text:
[pipeline-docs]: update  regular expression；add llustration of s3 storage when cache is enable

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: HaozhengAN:x: anhaozhenganhaozheng seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant HaozhengAN

repo: PaddlePaddle/PaddleFlow


text:
pp-tracking里的模型是否可以用paddleserving进行服务器部署

就想请问一下pp-tracking里的模型可不可以用paddleserving进行部署

就想请问一下pp-tracking里的模型可不可以用paddleserving进行部署

可以的。Paddle Serving是服务化部署框架，常规模型都可以部署的。 （个别不常见的模型，存在算子不支持的情况）

repo: PaddlePaddle/Serving


text:
add: retro2

Codecov Report
Base: 90.39% // Head: 90.39% // No change to project coverage :thumbsup:

Coverage data is based on head (94f9194) compared to base (42d1718).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #29   +/-   ##
=======================================
  Coverage   90.39%   90.39%
=======================================
  Files           8        8
  Lines         333      333
  Branches       42       42
=======================================
  Hits          301      301
  Misses         25       25
  Partials        7        7

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

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

authors: codecov-commenter ronituohino

repo: Perttu-Kangas/ohtu-mini


text:
Document (and test) various versioning concepts

Various versions

messages
DSU files structure & format

repo: PharmaLedger-IMI/epi-workspace


text:
Feature/acs 8005 jdbc golden merging

Not sure how that test ever passed.

repo: Populus-Global-Solution/fhir-data-pipes


text:
#96 router doesn't work in IE 11

Coverage remained the same at 93.556% when pulling 00dcecc54919c6c2428b039c10b6070a9568b0c2 on IE into 2d7afc18dff7db2ab31c15f2f8eae3ed2a1c6969 on master.

Coverage remained the same at 93.556% when pulling 815ba4f6737207ea1541fc4f6875ec5bc6c8303d on IE into 2d7afc18dff7db2ab31c15f2f8eae3ed2a1c6969 on master.

authors: barsh coveralls

repo: Profiscience/ko-component-router

license: WTFPL


text:
Add .env to .gitignore

.env already added in a separate PR

authors: Gerv-G ShindouMihou

repo: ProgrammingPhilippines/progphil-bot


text:
Finish writing initial version of project plan

Initial version (3) has been finished.

Chapter 5 still needs fleshing out

Initial version (4) has been finished and sent off to relevant people.

authors: j-nikki tatuhtuni

repo: Project-C-SQL/postgresql-for-novices


text:
Add feynman cluster to dev branch

I am ok to merge this if @nelsonand confirms that it works


text:
New configuration example for the K8s Prometheus Operator

approved.

authors: chrroberts-pure genegr


text:
Execute with another engine e.g. basexclient

Hi @jakobjanot, thanks for this. Hopefully I can take a look at merging this next week.

repo: Quodatum/vscode-basex


text:
added back CUCount property in gfx942 80cu BBS/HHS TN GridBased logic

This was caused by previous PR#796.

repo: ROCm/hipBLASLt