Instance method '-attachToVerifier:verifier:' not found

I'm building some Kiwi tests and getting a warning that I cannot explain. I'm new to Kiwi.
I have a mock object setup:
id conversationMock = [KWMock mockForProtocol:@protocol(Conversation)];
[conversationMock stub:@selector(end)];

And in my controller, a property called "conversation":
@interface MyController ()
@property (nonatomic, assign) id<Conversation> conversation;
@end

The mock is assigned to the property, then in the spec I check for whether the method "end" is called on the conversation:
it(@"should end conversation", ^{
  [[[myController.conversation] should] receive] end];
  [myController stopTalking];
});

The compiler (LLVM 3.0) is showing a warning: "Instance method '-attachToVerifier:verifier:' not found"
What is the cause of this? Is this something I need to fix? (test runs ok, checks the method call to end works ok)
Also asking on Stack Overflow: http://stackoverflow.com/q/8524747/184700

Section 6 - https://github.com/allending/Kiwi/wiki/Guide:-Up-and-Running-with-Kiwi

Sorry, I glazed over your issue and prematurely assumed you were getting an exception.
-attachToVerifier:verifier: is defined in a category on NSObject.
id  cannot be statically resolved to an NSObject, thus the warning. It just happens to work because you happen to have an actual NSObject being pointed to.

No problem.
In the project I'll be changing the property entirely to be an NSObject*, so that will solve it w/ out the recast.
thanks

IMO, id <Conversation> is actually more idiomatic Obj-C. Can't remember the last time I saw NSObject actually being used in a project. This is more of a limitation of Kiwi than anything else and I cringe at the fact that a tool that is supposed to help you pressures you into doing something unnatural. If it were me, I would probably do something like [(id)myController.conversation should] in the test.

Ok I'll give that a try. I'm also having to cast the return value of the receive because "end" is a common method and it complains about the ambiguity.  I didn't mention that because it seems like more of an edge case for the codebase I'm testing.  It's a lot of re-casting just to get some warnings to pipe down.
But in any case I agree with you - there has to be some way to make the tests flow better for these cases.


Add the ability to set folder options per method

folder options can be passed as an object to each method and used to
set the folders used to store baseline, actual, and diff images

Going to merge it into a new branch that I will also use for the refactor of some stuff so I can release it at once


Failed to resolve import in adding a page

Is this a regression?
I don't know
Description
I am trying to implement simple documentation with this library and I used this tutorial  in medium and general documentation. Unfortunately, when I try to add a page I get this error:
Internal server error: Failed to resolve import "ng-doc/document-sample/guides/installation/module" from ".angular\vite-root\document-sample\main.js". Does the file exist?
I don't know it's a bug or no but, but I couldn't find any solution in the documentation
Angular CLI: 17.0.8
Node: 20.9.0
Package Manager: npm 10.1.0
@ng-doc/app: "17.2.1"
@ng-doc/builder: "17.2.1"
@ng-doc/core: "17.2.1"
@ng-doc/ui-kit: "17.2.1"
Please provide the exception or error you saw
No response
OS
Windows
Browser
Chrome
Node version
20.9.0
Anything else?
No response

Hey, thanks for the ticket! A lot has changed since the time of that article, so I would recommend using the documentation first and foremost, as it gets updated after each change.
It's hard to say what the reason is. I've encountered something similar in applications where there was no baseUrl: "." in the tsconfig.json. Could you create a repository where I could reproduce this?

@skoropadas Thank you for your answer. Honestly, I used documentation but I couldn't find my answer. so because of that, I used your article.
I create this repo  and I remove the ng-docs directory from .gitignore for more information. I didn't do any additional things. I just created a new app with angular/cli after that I added ng-doc with the ng add command after that made a directory for docs and generated an installation page with this command ng g @ng-doc/builder:page "Installation" in docs directory. after serving the app I got the error related to failing to load the installation module in ng-doc.routing

@amirhosseinfaraji Thank you, I've checked the project, and it seems to be an issue with the baseUrl. Just add baseUrl: "." to your tsconfig.json in the compilerOptions section, and everything should start working.
Previously, this property was added by default when creating an Angular project. Apparently, something has changed in version 17. In any case, I will fix this in future releases by replacing imports with relative paths.

@skoropadas After adding base Url, it works correctly. thanks a lot,


double_float needs to use NaN and inf

The double_float template class needs to make use of NaN and infinitiy. There are some basic behaviors needed for a Boost.Multiprecision backend. These include:

Handle NaNs
Division finite/0 gives infinity.
Support the concept of negative infinity.
Negative infinitiy should test less than 0.
Infinity should test greater than 0.
Probably some more characteristics we find/remember along the way...

Some progress on the issue is made in this commit.

Hi @sinandredemption your changes in the backend reqs branch are going green on CI (after a bit of back and forth from my side).
I will report fully in #34
For me, this means that double_float is fully viable as a Boost backend candidate --- in early stages --- but having loads of potential.
Full speed ahead on any plans you have to refine the backend reqs branch or draft new stuff for quad_float.

Please note that I commented out a few inf/NaN/construct-from-enum tests in test_arithmetic_df.cpp in the associated header file. We can refine the cpp_double_float backend relatively easily to accomidate for these as we move forward.

This is done. Some tweaking and corrections are being and probably will be found as progress in CI / use in special functions continues. These can be tracked as they are found.
Time to close this issue.


Analog Clock

I want to add a Analog Clock which will be made using Python

@TusharAMD , Kindly assign

@Mrjoy832 Assigned


Limit buffer usage in transcript hash calculations to save stack space.

In functions where a transcript hash is required, we frequently allocate more buffers than necessary, and we can save on stack space by being more prudent with how buffers are utilized. Since these functions can be the culprit of largest stack usage, lowering this stack space is valuable for running on any resource-limited platforms.
For example, in spdm_generate_key_exchange_rsp_signature, we allocate "th_curr_data", a buffer of MAX_SPDM_MESSAGE_BUFFER_SIZE. Later, we pass "th_curr_data" as the output buffer when calling the function spdm_calculate_th_for_exchange.
https://github.com/DMTF/libspdm/blob/631028abb33ab22d318e5ffcd43b0a87fcda89df/library/spdm_common_lib/crypto_service_session.c#L226-L243
The function spdm_calculate_th_for_exchange allocates its own large_managed_buffer_t, "th_curr", to hold the transcript.
https://github.com/DMTF/libspdm/blob/631028abb33ab22d318e5ffcd43b0a87fcda89df/library/spdm_common_lib/crypto_service_session.c#L31
Once the transcript is copied into the temporary "th_curr", it is copied out into "th_curr_data" at the end of the function.
https://github.com/DMTF/libspdm/blob/631028abb33ab22d318e5ffcd43b0a87fcda89df/library/spdm_common_lib/crypto_service_session.c#L82-L84
On the stack, this requires two MAX_SPDM_MESSAGE_BUFFER_SIZE byte buffers to be allocated on the stack. In this scenario, "th_curr" is redundant, and we could simply utilize "th_curr_data" directly as the output buffer in spdm_calculate_th_for_exchange, removing one of the large buffers from the stack.
This usage pattern is typical for transcript calculation functions, so we could make this improvement in multiple areas.

The main issue I could see with making this change is that we could write a partially completed transcript to the output buffer, then encounter an issue, leading to error being returned and a buffer filled with incomplete data, rather than empty.
This can be resolved by clearing the output buffer in case of failure.

Approved.

@taprinz, do you still see the issue with new LIBSPDM_RECORD_TRANSCRIPT_DATA_SUPPORT ?

ack request - I need to re-integrate latest in order to analyze stack space usage before and afterwards. might take me a sec to formally measure.
looking at the changes, though, I would feel pretty confident that this greatly addresses extraneous memory and stack usage.

Consider issue resolved by #22


feat: dependent contextual inference

Fixes #51377
Fixes #40439
This PR adds support for what I'm calling "dependent contextual inference"... Sometimes the type for a value depends on the value itself (see linked issues), in those cases the contextual type of the value becomes something like T extends F<T>. Here what you'd want to do is first check the value against the contextual type T extends F<T> (which is what already happens), let's say the type of the value was inferrred to be X, and then (this is the new part) you do a second check against the contextual type T extends F<X> and the result of that would be the final contextually inferred type of the value
Consider this PR a shabby experiment, I want to first get a green light if we want to do something like this, if yes then I can work on it further.
If ya'll prefer me opening an issue proposing this change and discussing it there first then let me know. Types like T extends F<T> pop up when you're typing DSLs and it'd be really great if we could support them better.
Also note that most of this is a sophisticated guesswork, I knew what I wanted to do but I don't know much about the codebase or compilers xD, so there might be better ways to write it (I'll leave some review comments).
Also leaving some todos:

[ ] Support all values and not just object literals
[x] Support all kinds of circular type parameters, ie T extends <any-expression-that-has-T>, not just T extends F<T>. And perhaps even <any-expression-that-has-T>, where T extends <any-other-expression-that-has-T>.

[x] T extends <any-expression-that-has-T>
[ ] <any-expression-that-has-T>, where T extends <any-other-expression-that-has-T>

[x] Multiple passes? I think in some cases two passes won't suffice, in theory you'd probably keep doing checks till you get the same type back

Just curious:
@typescript-bot perf test this faster

Btw we can make it even more faster by bailing to default checking even more earlier by first checking for node.parent.kind === SyntaxKind.CallExpression. Currently there's an overhead while checking all object literals, with this change it'll be only for object literals which are a child of a call expression.

I was waiting for a green light as I said but I got bored :P so here are some advancements...

widen support for T extends <any-expression-that-has-T>

Previously only T extends F<T> was supported, not instead of just F<T> (ie a type reference which is a type alias with one argument) any type expression that has a reference to T in it's declaration is supported. Eg T extends { x: keyof T } was previously not supported but with commit it is now supported.

support multiple passes

As I said in some cases two passes won't suffice. So we keep doing checks till we yield the same type as previous check, or we hit the limit of maximum 3 checks. If we hit the limit then we produce a error that says "Dependent contextual inference requires too many passes and possibly infinite". And perhaps this should be a warning instead of an error.
I've updated the todos in the description.


new source?

your source appears to be dead now.

oh thanks for notifying - Fortunately, I have a backup of users agents, but yea, I have to figure oute a new source.

suggestions?

no idea, I was using your list. I looked for a new regularly updated source as well but couldn't find anything.

@fenopa I'm creating my own suggestion – it will be shipped soon!


Check if config file is empty before attempting to read it.

Fixes the second issue reported in https://github.com/meskyanichi/backup/issues/198

Cheers, thanks for this!


[ISSUE #8145] Optimize some code style in client module

Which Issue(s) This PR Fixes

Fixes #8145
Brief Description

How Did You Test This Change?

Codecov Report
Attention: Patch coverage is 61.90476% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 42.91%. Comparing base (03b16aa) to head (21fcf9a).
Report is 1 commits behind head on develop.

Files
Patch %
Lines

...mq/client/impl/producer/DefaultMQProducerImpl.java
57.14%
2 Missing and 4 partials :warning:

...rocketmq/client/impl/factory/MQClientInstance.java
66.66%
1 Missing :warning:

...he/rocketmq/client/trace/AsyncTraceDispatcher.java
66.66%
1 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #8146      +/-   ##
=============================================
- Coverage      42.93%   42.91%   -0.03%
+ Complexity     10386    10377       -9
=============================================
  Files           1270     1270
  Lines          88693    88689       -4
  Branches       11401    11402       +1
=============================================
- Hits           38083    38060      -23
- Misses         45920    45931      +11
- Partials        4690     4698       +8

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


source-port and destination-port options not enabled in security-list module

Majority of the cases, the traffic is restricted by port numbers like 22.
There is no way mention this in the current version. The module requires modifications.

Hey @shravaniakkineni  we're tracking this issue under LANZ-1756. We will keep you updated with progress.


Performance with SVG as ReactComponent and eslint with React.memo missing display name

I'm using this component in a create-react-app app:
import { ReactComponent as ProfileIcon } from "./icons/profile.svg";

...
render(){
  <ProfileIcon {...props}/>
}

Using also why-did-you-render (https://github.com/welldone-software/why-did-you-render) I got this warning:
SvgProfileIcon
whyDidYouRender.min.js:1191 {SvgProfileIcon: ƒ} "Re-rendered because the props object itself changed but it's values are all equal." "This could of been avoided by making the component pure, or by preventing it's father from re-rendering." "more info at http://bit.ly/wdyr02"
whyDidYouRender.min.js:1191 prev props: {svgRef: null, className: "icon", height: "24", width: "24"} !== {svgRef: null, className: "icon", height: "24", width: "24"} :next props

So I made a custom PureComponent like this:
import React from "react";

export default WrappedComponent =>
  React.memo(props => <WrappedComponent {...props} />, () => true);

FIRST QUESTION: Is this performances-correct?
I'm using it like this:
import { ReactComponent as ProfileIcon } from "./icons/profile.svg";

import PureComponent from "./PureComponent";

const PureProfileIcon = PureComponent(ProfileIcon);

...
render(){
  <PureProfileIcon {...props}/>
}

SECOND QUESTION: Can I avoid this component at all using React.memo (or something else) differently?
Now eslint is complaining about:
Component definition is missing display name eslint(react/display-name)

THIRD QUESTION: How can I fix this?

No, stale.

I thought we merged a PR that added display name to the SVG component. I'll have to try to find that.


Using save rather than copy to handle images that are only in memory

I've found that when I import and fbx with embedded textures the texture source path is not valid despite the texture being loaded into memory. This results in an error while trying to copy the texture. As a work around I use the "save" method rather than a copy.
I've included the traceback I was seeing and attached the FBX with an embedded texture.
Traceback (most recent call last):
  File "/Applications/blender/blender.app/Contents/Resources/2.79/scripts/addons/io_scene_gltf2/__init__.py", line 366, in execute
    return gltf2_export.save(self, context, export_settings)
  File "/Applications/blender/blender.app/Contents/Resources/2.79/scripts/addons/io_scene_gltf2/gltf2_export.py", line 84, in save
    generate_glTF(operator, context, export_settings, glTF)
  File "/Applications/blender/blender.app/Contents/Resources/2.79/scripts/addons/io_scene_gltf2/gltf2_generate.py", line 2750, in generate_glTF
    generate_images(operator, context, export_settings, glTF)
  File "/Applications/blender/blender.app/Contents/Resources/2.79/scripts/addons/io_scene_gltf2/gltf2_generate.py", line 2036, in generate_images
    create_image_file(context, blender_image, path, file_format)
  File "/Applications/blender/blender.app/Contents/Resources/2.79/scripts/addons/io_scene_gltf2/gltf2_create.py", line 338, in create_image_file
    copyfile(src_path, dst_path)
  File "/Applications/blender/blender.app/Contents/Resources/2.79/python/lib/python3.5/shutil.py", line 120, in copyfile
    with open(src, 'rb') as fsrc:
FileNotFoundError: [Errno 2] No such file or directory: '/tmp/chars/../../Pictures/wildtextures-seamless-paper-texture.jpg'

lowpolyPeopleCasualMan31_lowpolyPeopleCasualMan.fbx.zip

Ugh, I seem to have messed up the history here while trying to fix the author email to sign the CLA. I'll try again tomorrow.

@FreakTheMighty Thanks for this.  But, we want to leave the copy in place for images such as JPG that would go through a decode/re-encode cycle if we did a save here.
Is there some way to detect in-memory images and save only them, while copying the ones from disk?
I'm going to close this PR due to the history issues, but please do make a new one if you get this working with both in-memory and pre-existing images.  Thanks!

@emackey I can't remember why I thought this, but I was thinking [save](https://github.com/KhronosGroup/glTF-Blender-Exporter/pull/237/files#diff-d5139f2a7d35ee2dee8f86eae18eea8dR337] just saved the blender Image bytes to disk, where save_render re-encodes it. If that's the case, then saving shouldn't be destructive.
https://github.com/dfelinto/blender/blob/9f8bb5813bbbd2fc8a6a174b8ff4d86529bef94a/source/blender/makesrna/intern/rna_image_api.c#L67-L117


UART Serial read / read_until doesn't work as expected

I'm trying to read data in a python ScriptNode via UART, and the read and read_line functions return immediately rather than blocking until the configured timeout is reached. By default, the timeout is None, which means that those pySerial functions should block until the requested number of bytes are available / until a line feed is received. I have tested this on both an OAK-FFC 4p and OAK-PoE 1p using the default UART settings on both the transmitter and receiver (i.e. 1 stop bit, no flow control or parity bits). To reproduce, you can run the following script:
    import serial

    ser = serial.Serial("/dev/ttyS0", baudrate=115200, timeout=None)
    while True:
        result = ser.read_until()
        node.warn(f"received {len(result)} bytes: {str(result.decode(encoding='ascii',errors='ignore'))}")

If nothing is plugged into the UART pins you will see the script print 1000s of messages per second, and always 0 bytes read. If you actually send a line feed terminated message over UART you will see that your message is broken over several log lines rather than just the expected single line - and of course there will still be 1000s of 0 byte messages in between.
Note that if you set timeout=1 (1 second according to the pySerial docs), the behaviour is approximately the same as above, however if you set much larger values, then the results are strange: with timeout=1000 the call blocks for ~1.6 seconds, and for timeout=100 it blocks for 23 seconds 🤷♂
I also noticed that on two occasions that the very first call to read_until would block indefinitely even when the TX side was sending data.
PS: the docs don't say that it is pySerial that is provided by the script node environment, or which version of the library is being used. I'm just assuming this is the case based on the module and method names.

@RichardHMaxwell
Its using VTime impl, based off of 3.0.1 pyserial.
Its odd that read_until does not function properly. What does read return when no characters are sent? Just a None?
Given the implementation of read_until it might be best if it was redone on your end as such:
def read_until(self, terminator=LF, size=None, timeout=None):
    """\
    Read until a termination sequence is found ('\n' by default), the size
    is exceeded or until timeout occurs.
    """
    tstart = time.time()
    lenterm = len(terminator)
    line = bytearray()
    while True:
        c = self.read(1)
        if c:
            line += c
            if line[-lenterm:] == terminator:
                break
            if size is not None and len(line) >= size:
                break
        if timeout is not None and time.time() - tstart > timeout:
             break
    return bytes(line)

This should loop until the actual read_until gets fully complete.
There are 2 options left that you may test:

Setting timeout to timeout=-1
Setting timeout to timeout=-1/1000

Unfortunatelly actual impl. on device will not likely change

@themarpe

What does read return when no characters are sent? Just a None?

It returns an empty bytes object.

There are 2 options left that you may test:
Setting timeout to timeout=-1
Setting timeout to timeout=-1/1000

These throw a ValueError because they are negative.
Thanks for providing an implementation of read_until. I suppose the main issue would be high CPU usage, but you can work around that by sleeping for a few millis on ser.inWaiting() < 1 , although this may slightly lower throughput.


Use Sentry instead of Errbit

Instructions for upgrading are available at https://docs.publishing.service.gov.uk/manual/upgrade-to-sentry.html.

This should probably wait for Use Sidekiq instead of Delayed Job, because it requires Sidekiq to be setup.

This should probably wait for Use Sidekiq instead of Delayed Job, because it requires Sidekiq to be setup.

I don't think it does. If you are looking at the bit about govuk_sidekiq, I think that is only relevant if you are using govuk_sidekiq already.

@cbaines: Ah, OK. Thanks for pointing that out! 😄


PHP Port =D

function spark($numbers,$delimiter = ',') {
	$ticks = array(▁,▂,▃,▄,▅,▆,▇);
	$sorted = explode($delimiter,$numbers);
	sort($sorted);
	foreach (explode($delimiter,$numbers) as $i => $tick)
		$out .= $ticks[round(($tick / ($sorted[count($sorted)-1] == 0 ? 1 : $sorted[count($sorted)-1])) * (count($ticks)-1))];
	return $out;
}

Thought it'd be fun to port to PHP so I did!
It's not so much a port as it is a recreation, tell me what you think.

Looks sharp; I've seen a lot of Python and Ruby samples floating around too. Goes to show that a higher-level language can be much more terse. :)


Pcb layout

Hi, can you give the pcb layout?
Or when are you planning to build other board?

If you are willing to share PCB layout files, I would also very much appreciate that:)

Hi @sqfmi! We would love to fork your hardware design of badgy and would highly appreciate to get access to the PCB layout (and the rev 2B schematics)! Thanks!!


Borken build: cuda validation tests

The refactoring of sampling interfaces has broken the cuda validation tests:
[ 36%] Building NVCC (Device) object tests/unit/CMakeFiles/test_cuda.exe.dir/test_cuda.exe_generated_test_mc_cell_group.cu.o
/home/bcumming/software/github/nestmc-proto/tests/unit/test_mc_cell_group.cu:9:36: fatal error: ../test_common_cells.hpp: No such file or directory
compilation terminated.
CMake Error at test_cuda.exe_generated_test_mc_cell_group.cu.o.cmake:222 (message):
  Error generating
  /home/bcumming/software/github/nestmc-proto/build/tests/unit/CMakeFiles/test_cuda.exe.dir//./test_cuda.exe_generated_test_mc_cell_group.cu.o

NB: it was the CUDA unit test rather than validation test that broke.


"scoop install ?" triggers unexpected installation

Out of curiosity I typed in the PowerShell console
scoop install -?

The result was an error
parsing "?:?" - Quantifier {x,y} following nothing.
At ...\scoop\apps\scoop\current\lib\getopt.ps1:49 char:8
+                 if($shortopts -match "$letter`:?") {

Fine, this form of getting help is not supported.
The next command I tried was
scoop install ?

In this case scoop started to install things that I did not expect.
It managed to install innounp then failed:
installing innounp (0.40)
downloading http://heanet.dl.sourceforge.net/project/innounp/innounp/innounp%200.40/innounp040.rar...done
checking hash...ok
extracting...done
creating shim for innounp
innounp (0.40) was installed successfully!
installing ? (3.1.1)
new-item : Illegal characters in path.
At ...\scoop\apps\scoop\current\lib\core.ps1:212 char:14
+     'mkdir' = { new-item -type directory @args }

This is all my fault, I was experimenting with unusual commands, I got unusual
results. I am just wondering if this is really by design, what scoop does in
these cases.

This cracked me up. It shouldn't work I don't think.

Well this is interesting. It appears that Powershell automatically filters paths with ? in them such that the ? matches any character. So, get-childitem ?.json' from the bucket directory matches R.json` for me.
get-childitem '.\bucket\??.json'

    Directory: W:\scoop-stuff\scoop\bucket

Mode                LastWriteTime     Length Name
----                -------------     ------ ----
-a---        27/06/2015   1:54 PM        605 go.json
-a---        27/06/2015   1:54 PM        412 jq.json
-a---        27/06/2015   1:54 PM        266 ln.json


add container volumes test case

part of #18
@feiskyer @Random-Liu Please review it, thanks.
Signed-off-by: Helen-Xie<EMAIL_ADDRESS>

Update. Please review it again. @Random-Liu @xlgao-zju

Update, and I am sorry for my carelessness.


Update PR label notifier

I found it hard to recall which labels are primary and which are secondary. This PR changes the bot to include that in the message.
% python .github/process_commit.py cbf267c357a7574d9382ffcace75089dc2798295
Hi @mthrok

You merged this PR, but the category labels are missing.
Please add a primary label (`bug fix`, `example`, `other`, `prototype`, `new feature`, `BC-breaking`, `deprecation`, `improvement`) and a secondary label (`module: ops`, `style`, `module: tests`, `other`, `build`, `module: docs`, `perf`, `module: models`, `module: datasets`, `module: pipelines`, `module: I/O`).

% python .github/process_commit.py 8c78273be11f05b8b77ba24f0764829b7b404b7b

The message suggestion is because it is possible for one label to be added but one to be missing, so this sounds more correct to me but feel free to play around with the message.

Sounds good.

Another possibility if you're interested is to have separate cases for missing primary and secondary labels, where the message only reminds of the missing label category(ies) instead of listing both all the time.

I considered that but labels can be removed as well, so I thought it would not not hurt to list all of them all the time. Let's see how we actually feel about that with the new one and follow up.

Okay, the multiple lines are not working. I reverted this in #1965. Will retry.


Cannot include empty folders

Python's zipfile module supports writing empty folders to zip archive.
when trying with zipfly i get:
File "/.../zipfly.py", line 211, in generator
    with open( path[self.filesystem], 'rb' ) as e:
IsADirectoryError: [Errno 21] Is a directory: '/path/to/dir'

Is there an undocumented way for writing empty folders?

Hello,
It doesn't make any sense to include an option to add empty folders in zipfly.
You always can open a pr :)

Well, there are valid usecases. This is no question about "sense".


It takes too long to create "isolate"

When I execute the following code, I find that waiting for success takes about 1 second or more.
Also, when I need to create "isolation" multiple times in my application, I usually wait a long time.
When waiting for a long time, I use "CALL STACK" in vscode and use pause> start to operate a single "isolate",
Then "isolate" will execute correctly and give a return value, otherwise it will wait forever.
int begin = DateTime.now().millisecondsSinceEpoch;
    Isolate isolate = await Isolate.spawn(command.params['method'], argparams,
        debugName: command.params['token'],
        errorsAreFatal: true,
        onError: _onError);
    print(
        "Isolate.spawn ${command.params['token']} complete:cost time:${DateTime.now().millisecondsSinceEpoch - begin}");

What can I do to get the return value of "isolate.spwan" quickly?
[✓] Flutter (Channel stable, v1.12.13+hotfix.8, on Mac OS X 10.15.2 19C57, locale zh-Hans-CN)
    • Flutter version 1.12.13+hotfix.8 at /Users/XXXX/development/flutter
    • Framework revision 0b8abb4724 (7 weeks ago), 2020-02-11 11:44:36 -0800
    • Engine revision e1e6ced81d
    • Dart version 2.7.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/XXXX/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.1, Build version 11A1027
    • CocoaPods version 1.9.0

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 43.0.2
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] VS Code (version 1.43.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.8.1

[✓] Connected device (2 available)
    • MI PAD 4                     • 69e3c4be      • android-arm64 • Android 8.1.0 (API 27)
    • Android SDK built for x86 64 • emulator-5554 • android-x64   • Android 10 (API 29) (emulator)

Hi @govanyjj
This is more of a Dart issue
Please raise your issue on their GitHub
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


distro: drop the second dash

Currently we are seeing the error:
SDK_VENDOR should be of the form '-foosdk' with a single dash
This is showing up now due to oe-core commit b0efd8d4d0db [sanity:
check the format of SDK_VENDOR] added to fix [ YOCTO #13573 ] which
basically enforces the use of a single dash in SDK_VENDOR. We follow
the guidance and drop the second dash.
Signed-off-by: Mark Asselstine<EMAIL_ADDRESS>

merged


{AzureMaintenance} fixes Azure/azure-rest-api-specs#22872 Fix for incorrect rebootSettings Enums

fixes https://github.com/Azure/azure-rest-api-specs/issues/22872
Fix for incorrect rebootSettings Enums.
While invoking the Maintenance Configurations - Get REST API the portal returns the below values for the rebootSettings.
AlwaysReboot
RebootIfRequired
NeverReboot
So fixing the enums values to the same to retain the consistency.
ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

@paul-selvaraj Could you please review this PR once you get a chance ? Thanks in advance.

@navba-MSFT,
We cannot make changes in swagger, swagger has correct Enum values,
Looks like backend accept both values  (old + new) . This need to fix from backend instead of swagger change.
Change in Enum is not allowed and introduce breaking changes,  Event required rebuild all cli, sdk and powershell.
Need to investigate and require fix from backend, if backend make translation for Enum values then fix sould go to backend.

Thanks @paul-selvaraj and @KalpeshChavan12 . I will let the customer in https://github.com/Azure/azure-rest-api-specs/issues/22872 know about this.
Any ETA that could be shared on when the fix would be applied to the backend service / any workitem that I can keep track of ?

Thanks @paul-selvaraj and @KalpeshChavan12 . I will let the customer in #22872 know about this. Any ETA that could be shared on when the fix would be applied to the backend service / any workitem that I can keep track of ?

We have to investigate issue, We will provide ETA ASAP based on investigation

Thanks @paul-selvaraj and @KalpeshChavan12 . I will let the customer in #22872 know about this. Any ETA that could be shared on when the fix would be applied to the backend service / any workitem that I can keep track of ?

We have to investigate issue, We will provide ETA ASAP based on investigation

@KalpeshChavan12 Thanks for your reply. Please let me know if you had any tentative ETA for the fix that could be shared.

Thanks @paul-selvaraj and @KalpeshChavan12 . I will let the customer in #22872 know about this. Any ETA that could be shared on when the fix would be applied to the backend service / any workitem that I can keep track of ?

We have to investigate issue, We will provide ETA ASAP based on investigation

@KalpeshChavan12 @paul-selvaraj Could you please let me know if you had any updates on this ? Awaiting your reply.


1b03

CENSE Site: http://dais-ita.org/node/5459

doc-6119 created as already had SL ID on Cense - but need paper uploading to box at                  - WIll leave this bit to @davebraines


merge package template from Kronos-Tools/npm-package-template

package.json

chore(devDependencies): add<EMAIL_ADDRESS>from template
chore(devDependencies): add<EMAIL_ADDRESS>from template
chore(devDependencies): update<EMAIL_ADDRESS>from template
docs(package): remove keyword null

Codecov Report

Merging #175 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #175   +/-   ##
=======================================
  Coverage   97.52%   97.52%
=======================================
  Files           1        1
  Lines         121      121
=======================================
  Hits          118      118
  Misses          3        3

Continue to review full report at Codecov.

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


refactor: package name changed from ruleengine to ruleenginecore

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 93.333%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
434

Relevant Lines:
465

💛  - Coveralls


Closes: https://github.com/swaywm/sway/issues/3625

What's the reason this is marked as "do not merge"?
Because it depends on a new wayland feature or because it needs more testing?

Blocked on a new Wayland release.

Can we make a more generalized helper for delaying global destruction for any kind of global?

Wayland 1.18 has been released, so we'll soon be able to (finally!) land this. This PR is ready for review.

Can we make a more generalized helper for delaying global destruction for any kind of global?

Yeah that's a good idea. Pushed a commit doing that a while ago.

Thanks!

I had I hopes for this PR to fix firefox crashes on reload. Sadly, my firefox still crashes on swaymsg reload. Is this related ?

Hard to say without a WAYLAND_DEBUG=client log.


net: lwm2m: Registration retry behavior on timeout

There are some confusing and in my opinion redundant LwM2M retry settings in the LwM2M client code.
First, there is the CONFIG_COAP_MAX_RETRANSMIT Kconfig setting which controls the number of retransmissions for CON transactions like the registration message. This works as expected. I set  CONFIG_COAP_MAX_RETRANSMIT=2 and the client does only two retransmissions before starting over with a new transaction.
On the other hand there is the CONFIG_LWM2M_RD_CLIENT_MAX_RETRIES Kconfig entry for which according to the description, should do the same as CONFIG_COAP_MAX_RETRANSMIT but only for registration messages? After tests with the current code, I can't figure out what CONFIG_LWM2M_RD_CLIENT_MAX_RETRIES is intended to do, it seems to be ignored.
Timeouts do not result in ENGINE_NETWORK_ERROR as indicated in the CONFIG_LWM2M_RD_CLIENT_MAX_RETRIES help text:

Specify maximum number of registration retries, before the application
is notified about the network failure. Once application is notified,
it's up to the application to handle this situation in a way
appropriate for the specific use-case (for instance by waiting for
LTE link to be re-established).

On reaching the CONFIG_COAP_MAX_RETRANSMIT limit, two events are processed: LWM2M_RD_CLIENT_EVENT_REG_TIMEOUT and LWM2M_RD_CLIENT_EVENT_DISCONNECT.
[00:00:59.860,000] <wrn> net_lwm2m_rd_client: Registration Timeout
[00:00:59.860,000] <dbg> net_lwm2m_client_app: rd_client_event: Disconnected
[00:00:59.860,000] <dbg> net_lwm2m_client_app: rd_client_event: Registration timeout!

Steps to reproduce the behavior:
Build with different settings of CONFIG_COAP_MAX_RETRANSMIT and CONFIG_LWM2M_RD_CLIENT_MAX_RETRIES and observe the behavior:
west build -p auto -b native_posix -d build-native zephyr/samples/net/lwm2m_client -- -DCONFIG_NATIVE_UART_0_ON_STDINOUT=y -DCONFIG_COAP_MAX_RETRANSMIT=2 -DCONFIG_LWM2M_RD_CLIENT_MAX_RETRIES=1

Tested with release 3.2.0 and current main.
So to summarize, the question would be, whats does, or should CONFIG_LWM2M_RD_CLIENT_MAX_RETRIES do and how does it fit together with CONFIG_COAP_MAX_RETRANSMIT?

Thank you for the question @mlasch
If you use non-secure LwM2M, these two values seem to do almost the same.
However, they do have different purposes and it is visible when you use DTLS.
CoAP retry can handle single packets dropped from the application layer traffic. If that can recover, then there is no network failures assumed.
But when network NAT timeout have happened, CoAP retry can not recover. But if there is still lifetime left, recovering the DTLS session might recover the LwM2M connection.
So the difference between CoAP retries and RD_CLIENT_MAX_RETRIES is that socket is closed in between RD client retries.
Here is a relevant piece from the state machine diagram:

Here is a Wireshark capture of the effect. Here CoAP retries fail to recover the connection, so we restart the DTLS and it recovers


Bug?: Token Hotbar does not clear when no token is selected

Test Info:
Foundry 0.62, D&D 0.92, Token Hotbar 1.2.3, Electron app as GM, only module installed in test world. All tokens are unlinked SRD monsters. Same behavior happened with linked tokens.
Test Steps:

Click a token that has a token hotbar defined with at least one macro on it
Deselect the token by selecting no tokens

Current behavior: Macro remains on token hotbar
Expected behavior: Token hotbar is cleared (or maybe reverts to a set of macros, like hotbars 1-4) until another token with a defined hotbar is selected.

I'm thinking maybe it should revert to the last hotbar active before the token hotbar was active instead?

I think both solutions are great. (y)

Show the regular 'user' hotbar with macros that were put there when no tokens were selected.
Switch to the last active hotbar. This would make more sense with relation to #3 and also has the benefit that I can warn when no tokens are selected when the user puts a macro on the token hotbar. I think I'll go with this option.

Marked as duplicate, as the solution is also proposed in #3 (Auto-Hotbar toggling).


Javascript XMLHttpRequest error

Hi, i'm trying to call an api and i get this error:

Need assistance to try to solve this.
Thank you.

solution: http.send();


VS Live Share not showing login status bar on Ubuntu Linux 16.04

Issue Type
Bug
Description
Co-worker on MaxOS has his live share connected and sent me a link.  I've installed Live Share on VS Code in Linux, but the link doesn't work.   Also doesn't show the login button on the command line.
Tried on my linux laptop as well with the same result.
VS Code Info
VS Code version: Code 1.20.1 (f88bbf9137d24d36d968ea6b2911786bfe103002, 2018-02-13T15:31:21.019Z)
OS version: Linux x64 4.4.0-87-generic

System Info

Item
Value

CPUs
Intel(R) Xeon(R) CPU E5-1650 v4 @ 3.60GHz (12 x 1696)

Load (avg)
1, 1, 1

Memory (System)
31.31GB (9.80GB free)

Process Argv
/usr/share/code/code --unity-launch

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

code-settings-sync
Sha
2.8.7

npm-intellisense
chr
1.3.0

githistory
don
0.4.0

es7-react-js-snippets
dsz
1.3.3

prettier-vscode
esb
1.1.3

githd
hui
1.1.1

vscode-scss
mrm
0.6.2

node-debug2
ms-
1.21.1

vsliveshare
ms-
0.2.146

vscode-react-native
vsm
0.6.0

Reproduces without extensions

Please report this on Live Share: https://github.com/MicrosoftDocs/live-share


Added docs for unreceived_acknowledgements and unreceived_packets

This answers a user's question from Discord.
I originally wanted to document these details in the guide, but opted for code-level docs since this seems more appropriate.
Tagged @seanchen1991 as reviewer since we're both monitoring Discord and he has the most context.

PR author checklist:

[ ] Added changelog entry, using unclog.
[ ] Added tests: integration (for Hermes) or unit/mock tests (for modules).
[ ] Linked to GitHub issue.
[ ] Updated code comments and documentation (e.g., docs/).

Reviewer checklist:

[ ] Reviewed Files changed in the GitHub PR explorer.
[ ] Manually tested (in case integration/unit/mock tests are absent).

Thanks @mzabaluev and @seanchen1991 !
@seanchen1991 Please re-review and merge if you agree.


ansible: remove hostname/group conflict

virtualbox-iso:  [WARNING]: Found both group and host with same name: localhost

not using the group name for anything here, so just renaming the group from localhost to local


Delete from Solr records marked as deleted in MARC files

First pass at deleting records from Solr that have been marked as deleted in MARC files.
The delete code is inspired in the code that Jonathan Rochkind posted but adapted for our needs and version of Traject.
Fixes #824

Coverage decreased (-0.4%) to 82.529% when pulling d206b3c63b6a7c84bd221bc1b68b4bd2e88c9f98 on issue_824 into afd685ba4397530724d48c2364053a6f3c608584 on alma.


delete-package-versions supports delete-only-untagged-versions

for your information
https://github.com/actions/delete-package-versions
Support from v4.1.0

yes i know. thank you


Support for lein-ring plugin (run with Run button)

'Run' button is now able to start ring server for projects using lein-ring plugin;

I haven't had a chance to test it, but I wonder if it may be problematic for projects like the clojurescript template which have a :ring declaration in the project.clj but still rely on the normal lein run command to run. Maybe they don't even need that declaration, come to think of it...

I think they can use both for two separate ways to run: if you start it with lein run, it uses :main and if lein ring server it uses handler from :ring hashmap to start ring plugin. So, I think that necessity of :main depends on the application code itself (if it relies on it's own main function or not).

I've added (nil? :main) check to ring-project? function.

Thanks very much!


Removing references to yahoo inside callback url and others

Not sure when Yahoo updated this, but you cannot copy/paste the example to get it working, you'll run into this error when setting up your Y! Application.

Nevermind, I got ahead of myself.


Make it easy to add system packages

Currently there is no easy way to add system packages, the ones that are not available via conda. It is a very fundamental thing to be able to do.
It will also avoid requests like these: https://github.com/Quansight/qhub/issues/147

One potential solution is using a K8s daemonset, similar to the way we do it for installing nvidia-drivers on node. I have not done a poc for this proposal, so we need to do some exploration to explore this possibility.

Ping @verhulstm any thoughts on this?

@aktech Any updates on this?


Bluetooth: SSP: No MITM if required level is less than L3

Regardless IO capabilities, clear MITM flag for pairing initiator if the required security level is less than BT_SECURITY_L3.

@lylezhu2012 please rebase

@lylezhu2012 please rebase

Yes. It is done.


Switch license to MIT

Most of our projects are using MIT license, so we tend to switch the lisence on this repo too.
Comments?

This should propably a major bump

It uses the same ISC license as https://github.com/renovatebot/renovate-approve-bot. The Github description of the license says:

The ISC license is functionally equivalent to the BSD 2-Clause and MIT licenses, removing some language that is no longer necessary.

@rarkins  WDYT? Should we stay on MIT and switch remaining too, or schould we switch all to ISC? 🤔


Open URLs in background when command-clicking

On OS versions that support openURL:configuration:completionHandler:, allow command-click to open URLs without activating the destination application.

Much appreciated, thank you!

Released in v1.7.0


Fix "Getting started" links

Right now all three links point to the same URL

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

Thank you @ranjan-purbey!


Consume expression to indicate how many characters to look ahead

Where possible consumeExpression should be provided with the maximum number of characters the expression could be made of. This allows for more efficiently peeking for a few characters rather than the entire remaining character map.
This issue was discovered at Moodle when compiling with FontAwesome which uses unicode more heavily. https://tracker.moodle.org/browse/MDL-58646

Travis may fail, but here is my local PHP Unit output:
PHP 7.0.15-1ubuntu4 (cli) (built: Feb 28 2017 21:33:59) ( NTS )

Copyright (c) 1997-2017 The PHP Group
Zend Engine v3.0.0, Copyright (c) 1998-2017 Zend Technologies
    with Zend OPcache v7.0.15-1ubuntu4, Copyright (c) 1999-2017, by Zend Technologies
PHPUnit 5.4.6 by Sebastian Bergmann and contributors.

...............................................................  63 / 108 ( 58%)
.............................................                   108 / 108 (100%)

Time: 102 ms, Memory: 6,00MB

OK (108 tests, 212 assertions)

Good idea, thanks!
(I’ll fix travis later)

Fastest merge ever! W00t!

FMCorz you beautiful man!


[Refactoring] Build script

How about refactoring build.sh file a little bit shorter?
[ ] Remove PKG_LIST and use a relative path to run test scenarios.
[ ] Streamline code in terms of BUILD_TAGS such as secure, mnedc.
[ ] Delete duplicated code that can be eliminated

Good idea, we can remove duplicated code and check on refactoring the file. Check for all the functions and might be make separate .sh for functions. Just my thought, need to investigate how much it would add value. Lets check on refactoring to make build script simple and understandable.

I think that BUILD_TAGS: secure should be enabled by default (and I will try to do it soon). But for ease of development, I plan to add a BUILD_TAGS:unsecure, but only as a tag for the build!

I think that BUILD_TAGS: secure should be enabled by default (and I will try to do it soon). But for ease of development, I plan to add a BUILD_TAGS:unsecure, but only as a tag for the build!

@tdrozdovsky Thank you for sharing your idea. I hope to set secure as a default.
In order to do so, it seems that the usability of Security Manager should be improved.
Let's talk about security components like Verifier and Authenticator by opening a new issue.


So what is causing the problem? The ppd is created when adding the printer with lpadmin and "-m everywhere". Is it the printers fault??

So what is causing the problem? The ppd is created when adding the printer with lpadmin and "-m everywhere". Is it the printers fault??
Originally posted by @chalmerssys in https://github.com/OpenPrinting/cups/issues/20#issuecomment-724522339

I don't think it's got anything to do with papercut, the ipp backend did the same thing.

I'll make a change for it to use the first supported media-source value instead. This just means that the printers "media-col-default" doesn't specify a default tray...

[master be8848be1] Don't use "unknown" default InputSlot value for IPP Everywhere PPDs (Issue #44)


coin changes

To do:

[x] Remove coin field from entries collection.
[x] Add coin field to addresses collection.
[ ] Handle adding on github url pull for the new cointype handling.
[ ] Update api endpoints to handle new structure:

addresses: {
ETH: [ ],
BTC: [ ],
LTC: [ ],
}

instead of:
addresses: [ ]

To merge: https://github.com/CryptoScamDB/blacklist/pull/146


Issue #184 - New page for job type error metrics

This needs to be tested against a live API.

This is on the right track of the print out I saw, here are some suggestions.

[ ] Change the colors to the new scheme from issue #182  (you already mentioned it)
[ ] I'm not too keen on the name "Performance", as that implies to me how fast it runs or much it uses in terms of resources. I think something like "Error Rates" would be better. Also move this tab to the last position after Jobs because it makes a bit more logical flow left to right.
[ ] I'm seeing an extra vertical scroll bar at the page level on my screen.
[ ] The date columns should be more like 24 hours, 48 hours, 72 hours, 1 week rather than consecutive days. I realize this requires multiple queries to the backend for the moment. Also the time should be newest to oldest from left to right to emphasize more recent changes.
[ ] Add a total error count at the beginning of each table cell breakdown.
[ ] Having color on every single value I think will make it a bit hard to recognize important trends. I think we should consider some sort of scheme that introduces more variation. For example, only color the values with the highest count within a table cell or turn it into more of a progress bar where the width of the color is proportional to the % of the errors.
[ ] Add a tool-tip over each value that shows the value as a % and the error category label.

I fixed a few of the basic things you mentioned, but I think in the spirit of iteration we should merge this in and then move on from there.

I'm going to merge this in to finish up the sprint. We'll gather more feedback and do additional changes.


Error when running rails new

I can't generate a new rails app. When running the rails news test command, the image message is always returned.

MacBook with the latest version of Macos
I don't have rails installed
Even running the command docker run --rm -it -v ${PWD}:/rails -v ruby-bundle-cache:/bundle ghcr.io/rails/cli new test message is returned

Fixed with latest image 👍


Code error when using not yet treated as controls

When I estimate the aggregated ATE using employment rate data and change the control group to notyettreated, the code gives an error:
import pandas as pd
from csdid.att_gt import ATTgt

 data = pd.read_csv("https://raw.githubusercontent.com/d2cml-ai/csdid/function-aggte/data/mpdta.csv")
 out = ATTgt(yname = "lemp",
              gname = "first.treat",
              idname = "countyreal",
              tname = "year",
              xformla = f"lemp~1",
              control_group = "notyettreated",
              data = data,
              ).fit(est_method = 'dr')
out.aggte(typec='simple')

Error:
compute_att_gt.py", line 83, in compute_att_gt
    n3 = np.where(data[gname] != glist[g], True, False)
IndexError: index 2004 is out of bounds for axis 0 with size 3`

Hi @dadepro! Thanks a lot for posting this. There was an issue with the indexation for treatment groups when control_group = "notyettreated"
`  # for _, g, in enumerate(glist):
for g_index, g in enumerate(glist):
# g = glist[1]
G_main = (data[gname] == g)
data = data.assign(G_m = 1 * G_main)

for t_i in range(tlist_len):
  pret = t_i
  tn = tlist[t_i + tfac]
  if base_period == 'universal' or g < tn:
    try:
      pret = np.where(tlist + anticipation < g)[0][-1]
    except:
      raise f"There are no pre-treatment periods for the group first treated at {g}\nUnits from this group are dropped"
      # break

  if base_period == 'universal':
    if pret == tn:
      add_att_data()

  if not never_treated:
    n1 = data[gname] == 0
    n2 = (data[gname] > (tlist[np.max([t_i, pret]) + tfac]) + anticipation)
    #n3 = np.where(data[gname] != glist[g], True, False)
    n3 = np.where(data[gname] != glist[g_index], True, False)`

I have run the equation with the updated package
`import pandas as pd
from csdid.att_gt import ATTgt
data = pd.read_csv("https://raw.githubusercontent.com/d2cml-ai/csdid/function-aggte/data/mpdta.csv")
out = ATTgt(yname = "lemp",
gname = "first.treat",
idname = "countyreal",
tname = "year",
xformla = f"lemp~1",
control_group = "notyettreated",
data = data,
).fit(est_method = 'dr')
out.summ_attgt().summary2`

exactly the same as the R output.

Please instal the github version to test this out :)

@pedrohcgs I think we can close this issue :)


fatal: not a git repository (or any parent up to mount point /tf) Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set). Error /tf/rover/functions.sh on or near line landingzone folder not setup properly. Fix and restart.; exiting with status 1[bug]

I need to run CAF terraform landing zone for my production environment & testing the given script however unable to run & getting below error while running. If the steps below are wrong, can you guide me on how to run or resolve it? Also, as per the video (https://www.youtube.com/watch?v=w0W90bqZaWA&list=PLU8D1GcHfTRoYHqVUV856zZ4q3ZgxWiKg&index=3) landing zone folder also not available. I think that is 2 years OLD so might be some changes. I also tried via (rover --walkthrough) however that also failed.  I can share screenshot if you provide any mail ID?
rover -lz /tf/caf/caf_launchpad
-launchpad
-var-folder /tf/caf/caf_launchpad/scenario/100
-env test
-a plan
fatal: not a git repository (or any parent up to mount point /tf)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
Error /tf/rover/functions.sh on or near line landingzone folder not setup properly. Fix and restart.; exiting with status 1
No terraform cloud runs found.
@calling clean_up_variables
cleanup variables
clean_up backend_files
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Configuration (please complete the following information):

OS and version: [e.g. Windows 10 1909]
Version of the rover[e.g. 22]
Version of the landing zone[e.g. 11]

Additional context
Add any other context about the problem here.

Plan: 18 to add, 0 to change, 0 to destroy.
Changes to Outputs:
~ global_settings = (sensitive value)
╷
│ Warning: Argument is deprecated
│
│   with module.launchpad.module.api_management_custom_domain.azurerm_api_management_custom_domain.apim,
│   on /home/vscode/.terraform.cache/sandpit/modules/launchpad/modules/apim/api_management_custom_domain/module.tf line 1, in resource "azurerm_api_management_custom_domain" "apim":
│    1: resource "azurerm_api_management_custom_domain" "apim" {
│
│ proxy is deprecated and will be renamed to gateway in version 3.0 of the AzureRM provider
╵
╷
│ Error: building AzureRM Client: please ensure you have installed Azure CLI version 2.0.79 or newer. Error parsing json result from the Azure CLI: waiting for the Azure CLI: exit status 1: Could not import runpy module
│ Traceback (most recent call last):
│   File "", line 1027, in _find_and_load
│   File "", line 1002, in _find_and_load_unlocked
│   File "", line 945, in _find_spec
│   File "", line 1439, in find_spec
│   File "", line 1411, in _get_spec
│   File "", line 1548, in find_spec
│   File "", line 1591, in _fill_cache
│ OSError: [Errno 12] Cannot allocate memory: '/tf/caf/caf_launchpad'.
│
│   with provider["registry.terraform.io/hashicorp/azurerm"],
│   on main.tf line 30, in provider "azurerm":
│   30: provider "azurerm" {
│
╵
╷
│ Error: Building AzureAD Client
│
│   with provider["registry.terraform.io/hashicorp/azuread"],
│   on main.tf line 84, in provider "azuread":
│   84: provider "azuread" {
│
│ Please ensure you have installed Azure CLI version 2.0.81 or newer. Error parsing json result from the Azure CLI: Error waiting for the Azure CLI:
│ exit status 1: Could not import runpy module
│ Traceback (most recent call last):
│   File "", line 1027, in _find_and_load
│   File "", line 1002, in _find_and_load_unlocked
│   File "", line 945, in _find_spec
│   File "", line 1439, in find_spec
│   File "", line 1411, in _get_spec
│   File "", line 1548, in find_spec
│   File "", line 1591, in _fill_cache
│ OSError: [Errno 12] Cannot allocate memory: '/tf/caf/caf_launchpad'.
╵
Terraform plan return code: 1
Error 1 on or near line 57: Error running terraform plan; exiting with status 1
@calling clean_up_variables
cleanup variables

Via /tf/caf/caf_launchpad/scenario/100 \ & /tf/caf/caf_launchpad/scenario/200, after running the script, I can see Storage
Key vault, Manage identity deployment successfully. I would like to know if ONLY these resources should deploy or just because the landing zone folder is missing, other resources were not deployed.


Hyundai: Ioniq 6 is not a radar ACC platform

All dongles have ADAS ECU in carFw, not sure why it was added

KONA_EV_2ND_GEN also has "ADAS ECU" in the car firmware.
Is this different from Ioniq 6 ?

@sunnyhaibin any issues with the ADAS ECU disable? or should we add to unsupported long for now?

All dongles have ADAS ECU in carFw, not sure why it was added

I drove the KONA_EV_2ND_GEN vehicle excluding the "CANFD_RADAR_SCC_CARC" list and it operates very well without any abnormalities.
However, I would appreciate it if you could check the status.
drive route is " 1618132d68afc876|2023-10-12--13-05-32--16 "
All routes (1618132d68afc876|2023-10-12--13-05-32) have the same conditions. (excluded CANFD_RADAR_SCC_CAR list)

@haram-KONA Can you try OP long? If it doesn't work we'll need to add to the unsupported long list

@haram-KONA Can you try OP long? If it doesn't work we'll need to add to the unsupported long list

Setting OP long will result in a "Hardware Fault" warning.

Do you have a route?

Do you have a route?

1618132d68afc876|2023-10-13--13-42-36--0

@haram-KONA try the branch now with your car removed

Changed this to be a simple refactor. Splits some diff from https://github.com/commaai/openpilot/pull/30033


Enhancement to Semantic Scholar Loader - full text reader

Added document loading option for full text extracted from downloaded open access pdfs.
As shown below, just marking full_text flag as True will download all the relevant open access pdfs and extract full text from the pdf and load it.
documents = s2reader.load_data(query_space, total_papers, full_text=True)

[x] Added demo notebook
[x] Updated the README
[x] Added pypdf and arxiv to requirements.txt

Attaching a screenshot of the loader in action!

sorry for the delay! merged, thanks @shauryr


pull request for e2e test at 12/05/2016 06:26:04

:white_check_mark: Validation status: passed
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.


tensorflow/core/common_runtime/bfc_allocator.cc:246] tried to allocate 0 bytes

I'm training some yolo models using

qqwweee/keras-yolo3

When I trained with 50 epochs, the test can be done although the results is bad.
So I trained with 150 epochs, and when I want to test with this model, I got the following ... (it's not an error but like an error):
Input image filename:eagle.jpg
(416, 416, 3)
2019-04-20 12:33:44.137176: E tensorflow/core/common_runtime/bfc_allocator.cc:246] tried to allocate 0 bytes
2019-04-20 12:33:44.140305: W tensorflow/core/common_runtime/allocator_retry.cc:32] Request to allocate 0 bytes
2019-04-20 12:33:44.143018: E tensorflow/core/common_runtime/bfc_allocator.cc:246] tried to allocate 0 bytes
2019-04-20 12:33:44.145718: W tensorflow/core/common_runtime/allocator_retry.cc:32] Request to allocate 0 bytes
2019-04-20 12:33:44.149139: E tensorflow/core/common_runtime/bfc_allocator.cc:246] tried to allocate 0 bytes
2019-04-20 12:33:44.151644: W tensorflow/core/common_runtime/allocator_retry.cc:32] Request to allocate 0 bytes
2019-04-20 12:33:44.154246: E tensorflow/core/common_runtime/bfc_allocator.cc:381] tried to deallocate nullptr
2019-04-20 12:33:44.157496: E tensorflow/core/common_runtime/bfc_allocator.cc:381] tried to deallocate nullptr
2019-04-20 12:33:44.160937: E tensorflow/core/common_runtime/bfc_allocator.cc:381] tried to deallocate nullptr
Found 0 boxes for img
6.07202067316063
I think it's something wrong in TF.
environment: Anaconda python(3.6.8), keras 2.1.5, tensorflow (cpu) 1.10.0.
I use "qqwweee/keras-yolo3" and made a train data as its requirement like follows :

Generate your own annotation file and class names file.
One row for one image;
Row format: image_file_path box1 box2 ... boxN;
Box format: x_min,y_min,x_max,y_max,class_id (no space).
For VOC dataset, try python voc_annotation.py
Here is an example:
path/to/img1.jpg 50,100,150,200,0 30,50,200,120,3
path/to/img2.jpg 120,300,250,600,2
...

Please fill in the information of your environment so we could know how to reproduce this error.
Thanks.

Please fill in the information of your environment so we could know how to reproduce this error.
Thanks.

I updated the problem with environment.

Could you try to install the latest tf-nightly package? I believe this issue has been fixed.

But if I use TF 1.13.1,  there will be another problem. See #27906

Could you try tf-nightly? It helps us to reproduce your problem.
We cannot fix your issue if we cannot reproduce it.

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


test_drake_ros: libdrake.so cannot be found if using source ./install/setup.bash, but can run otherwise

Trying to fix CI for https://github.com/RobotLocomotion/drake-ros/pull/153
CI Failure
Looking at this failure:
https://github.com/RobotLocomotion/drake-ros/actions/runs/3280593631/jobs/5401564148
   6:  - /__w/drake-ros/drake-ros/ros_ws/build/drake_ros_core/test_drake_ros --gtest_output=xml:/__w/drake-ros/drake-ros/ros_ws/build/drake_ros_core/test_results/drake_ros_core/test_drake_ros.gtest.xml
  6: /__w/drake-ros/drake-ros/ros_ws/build/drake_ros_core/test_drake_ros: error while loading shared libraries: libdrake.so: cannot open shared object file: No such file or directory
  6: -- run_test.py: return code 127
  ...
  6/7 Test #6: test_drake_ros ...................***Failed    0.07 sec

Local Repro
I am able to reproduce this locally on drake-ros at 0c1768d (from #153)
$ export PYTHONPATH=/opt/drake/lib/python3.10/site-packages
$ source /opt/ros/humble/setup.bash

$ cd ros_ws
$ rm -rf build/ install/ log/
$ ls
src
$ ls src/
drake-ros

$ colcon build --packages-up-to drake_ros_core

$ colcon test --packages-up-to drake_ros_core
# Fails as it does on CI.

$ bash-isolate -c 'source /opt/ros/humble/setup.bash && ./build/drake_ros_core/test_drake_ros'
# Passes.

$ bash-isolate -c 'source ./install/setup.bash && ./build/drake_ros_core/test_drake_ros'
# Fails.

Reproduced w/ Ubuntu 22.04 Jammy (via Apptainer on Ubuntu 20.04 Focal).
Aliase bash-isolate can be found here:
https://gist.github.com/EricCousineau-TRI/6870970d68f75d23061177a52932a5d1#file-bash_env_isolate-sh
Questions
Why does it pass without workspace setup, but not with the workspace setup?
I assume it is related to #143
\cc @IanTheEngineer @cottsay @gbiggs

This does not happen on current main at 32c174b.
I believe that #153 needs to be rebased to incorporate #143, and it should fix acute error.
However, I am still concerned that, somehow, adding entries to LD_LIBRARY_PATH, can somehow cause libdrake.so to not be found.


get rid of in-tree standalone

This is a suggestion in order to ease future development and get rid of bloated git history containing the compiled result.
Please have a look at https://github.com/jsarenik/bip39/commit/5215311ccdc0fe5b161dc82df1c46664f736a075
lomidrevo:~/Downloads$ ls -lh bip39-*
-rw-r--r-- 1 jasan jasan 1.2M Jul  9 12:36 bip39-jasan-rewritten.zip
-rw-r--r-- 1 jasan jasan 2.4M Jul  9 12:36 bip39-master.zip

It might not be so bad yet, but I have seen repositories with hundreds of GBs of generated blob.
For reference: https://github.com/samhocevar/wincompose/issues/16
bip39 is a wonderful piece of software. Thank you!

Good suggestion. Now that releases are being made it's a good idea to remove it from the repo. Thanks for the suggestion.

You are welcome. Good job. Respect.


ci: add cargo bloat ci function

Codecov Report

Merging #243 (66d0ba7) into master (9fb13f7) will decrease coverage by 2.86%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #243      +/-   ##
==========================================
- Coverage   45.36%   42.50%   -2.87%
==========================================
  Files          19       19
  Lines        1327     1327
==========================================
- Hits          602      564      -38
- Misses        725      763      +38

Impacted Files
Coverage Δ

server/src/lib.rs
100.00% <100.00%> (ø)

server/src/error.rs
0.00% <0.00%> (-100.00%)
:arrow_down:

server/src/id_manager.rs
69.23% <0.00%> (-15.39%)
:arrow_down:

server/src/connection_list.rs
42.85% <0.00%> (-12.50%)
:arrow_down:

server/src/tcp.rs
54.02% <0.00%> (-9.20%)
:arrow_down:

server/src/connection.rs
32.17% <0.00%> (-2.72%)
:arrow_down:

server/src/server.rs
87.09% <0.00%> (-0.54%)
:arrow_down:

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


ftdv-ilb is failing in Azure ARM deployment

Hi.
When trying to deploy the AutoScaler FTDv för Azure via ARM I'm getting the following:
Worth mentioning is that all resources exept ftdv-ilb is working and can't se any difference between ftdv-ilb and ftdv-elb.
{
    "status": "Failed",
    "error": {
        "code": "InvalidResourceReference",
        "message": "Resource /subscriptions/08b5eae9-f8bc-418e-ab44-a4ce25959426/resourceGroups/CISCO-VIRTUALNET-RG/providers/Microsoft.Network/virtualNetworks/CISCO-VIRTUALNET referenced by resource /subscriptions/08b5eae9-f8bc-418e-ab44-a4ce25959426/resourceGroups/FirePowerDelete/providers/Microsoft.Network/loadBalancers/ftdv-ilb was not found. Please make sure that the referenced resource exists, and that both resources are in the same region.",
        "details": [
            {
                "code": "NotFound",
                "message": "Resource /subscriptions/08b5eae9-f8bc-418e-ab44-a4ce25959426/resourceGroups/CISCO-VIRTUALNET-RG/providers/Microsoft.Network/virtualNetworks/CISCO-VIRTUALNET not found."
            }
        ]
    }
}

I'm using the .json at https://github.com/CiscoDevNet/cisco-ftdv/tree/master/autoscale/azure/ARM Template

I am sure you are populating 'virtualNetworkRg' this variable with wrong value (its using default value)  Or you dont have Vnet  'CISCO-VIRTUALNET' in this Resource Group (even this is default value)

If I create all the mentioned vNets and subnets in the ARM template it is working 👍


WIP: added js files

Files added to src/maps/mapfiles. Verified that files are moved over via install_server.sh
Will work on unit test

Issue number?

Addresses issue #590


⚠ Minecraft has degraded performance

In af727bd, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1363 ms

Resolved: Minecraft performance has improved in f548ffb.


Feedback

Hi there,
I made a public repository where I will publish feedback every week. Each release on this repository is the feedback for each iteration.
Here is the first release:
https://github.com/ValMai/cp2016-mm-feedback/releases/tag/iteration1
If you have any questions, don't hesitate to contact me on Slack.
Regards,
Valentine

Thanks, we will process this feedback in the final versions


print pretty JSON

@rultor release, tag is 0.9.0

@rultor release, tag is 0.9.0

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.9.0

@yegor256 Done! FYI, the full log is here (took me 5min)


Base setting is changed to pydantic_settings

Describe the bug:
Base setting is changed to pydantic_settings instead of pydantic.
Expected behaviour:
Base setting is changed to pydantic_settings to new module
Steps to reproduce the issue:
Go to file
Technical details:

Host Machine OS (Windows/Linux/Mac):
Browser (Chrome/Firefox/Safari):

Possible Fix:

Additional context:

duplicate of #38


Update windows dependencies

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[x] Other, please describe: update dependencies

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[ ] No

Checklist

[x] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[ ] I have added a convincing reason for adding this feature, if necessary

Other information

Thank you but, newer versions of windows-rs require MSRV of 1.64 and we want to maintain MSRV of 1.61


fe-homework-10

Screenshot

Drift video

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
0

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


Add Mark

Added my website to the list.
The Polyring banner will be added under https://markc.su/polyring/

Awesome :100:


Add documentation github app ssl check

if user has self certified cluster and doesn't check github app ssl check it would fail, we need to let the user knows

this is now detected automaticallly and warn user if the ssl is a self encrypted one


minimalistic config files for all workflows

Is your feature request related to a problem? Please describe.
The current /oktoberfest/example_config.json needs to be heavily modified when wanting to run a different workflow.
Describe the solution you would like
I think it would be useful to provide example config files for all three workflows:

CE Calibration
SpectralLibraryGeneration
Rescoring

In addition one might think about incl. some minimalistic input  files and expected results. This would also allow implementing a unit test for all three workflows.
For 2. this could be a very small fasta file for instance:
>sp|ADD000|iRTKit iRT-Kit_WR_fusion
LGGNEQVTRYILAGVENSKGTFIIDPGGVIRGTFIIDPAAVIRGAGSSEPVTGLDAKTPVISGGPYEYRVEATFGVDESNAKTPVITGAPYEYRDGLDAASYYAPVRADVTPADFSEWSKLFLQFGAQGSPFLK

This protein would be digested into the iRT11 peptide set, using trypsin. All of these peptides have known HCD fragmentation patterns, RTs and inverse mobilities.
Additional context

We already have minimal config files for all three workflows in our documentation: https://oktoberfest.readthedocs.io/en/latest/jobs.html
Concerning your second point, yes most definitely a good idea. I will add individual tutorial notebooks using the new API once it is released.

Hi @picciama,
it is good to have those files in the docu, but I would also include them in the package, so they get installed on the local system of the user...

Valid point, I will include this in the next release.

👍


example/image: Lens properly

I am not sure why no-op map is here and make_height was using AppState::width
Fixes #1909 . But there is bigger problem:
https://github.com/linebender/druid/blob/049747a5fd9dd26cd82a1e01f932b40759b31ef6/druid/src/widget/value_textbox.rs#L373-L381

looks like an issue with ValueTextBox


How to only do testing using pre-trained model

Dear Authors,
Sorry for the dumb question since I'm a beginner. I only want to use the pre-trained weight that you already provided for testing. However, I do not know how to load the weights in the model using this detectron2. Could you please show me the step from loading the weight (say VOS on ResNet) and evaluating it on the dataset? (ID: VOC, OOD: OpenImages). Thanks!

hi, @mcairlangga2 , please follow the instructions in the readme file. the inference-only command line is already specified.


MAINT: store/load state_dict of model/optimizer into/from files

state_dict is a mutable object (literally, it a dict), so that
it can be changed in runtime.
In the original version, we just hold a reference of state_dict,
not a copied instance. So that it won't work correctly if we want
to reset the state after running LRFinder.range_test().
In order to keep the original state of model and optimizer, we have
to:

make a deep copy of state_dict
save state_dict into file, then load it back later

We choose the second method. Although it take a little more time on
reading and writing file, it avoids consuming extra memory. So that
it won't lead to an out-of-memory issue.

First of all thanks for the contribution!
Your description makes sense to me, given how dictionaries behave in Python. Though after running some tests it seems to behave as expected; the model and optimizer are both reset to their initial states after LRFinder.reset().
To test the current behavior I printed the states of the model and optimizer before and after running LRFinder.range_test(), and then after running LRFinder.reset().
self.model.state_dict() and self.optimizer.state_dict() both change during the test but self.model_state and self.optimizer_state do not, so when LRFinder.reset() is called the model and optimizer state are loaded from them as expected.
Can you post some code where LRFinder.reset() fails?

Sorry for the late reply and the buggy commit in previous submission...
I've submit a new commit to fix those buggy code.
Besides, the code snippet below shows the difference between using LRFinder.reset() without/with storing state_dict to files and reload them back:
import torch
import torch.nn as nn
import torch.nn.functional as F
import torch.optim as optim
from torch.utils.data import Subset
from torchvision import datasets, transforms

class ConvNet(nn.Module):
    def __init__(self):
        super(ConvNet, self).__init__()
        # 1,28x28
        self.conv1 = nn.Conv2d(1, 10, 5)
        self.conv2 = nn.Conv2d(10, 20, 3)
        self.fc1 = nn.Linear(20 * 10 * 10, 500)
        self.fc2 = nn.Linear(500, 10)

    def forward(self, x):
        in_size = x.size(0)
        out = self.conv1(x)
        out = F.relu(out)
        out = F.max_pool2d(out, 2, 2)
        out = self.conv2(out)
        out = F.relu(out)
        out = out.view(in_size, -1)
        out = self.fc1(out)
        out = F.relu(out)
        out = self.fc2(out)
        out = F.log_softmax(out, dim=1)
        return out

def test_run_1():
    from lr_finder import LRFinder

    device = torch.device('cuda')
    model = ConvNet()
    model = model.to(device)
    optimizer = optim.Adam(model.parameters())
    criterion = nn.NLLLoss()

    data_folder = './data'
    train_data = datasets.MNIST(data_folder, train=True, download=True,
        transform=transforms.Compose([
            transforms.ToTensor(),
            transforms.Normalize((0.1307,), (0.3081,))
    ]))
    train_data = Subset(train_data, range(1000))
    train_loader = torch.utils.data.DataLoader(
        train_data, batch_size=64, shuffle=True
    )

    print(model.state_dict()['conv1.weight'][0])
    lr_finder = LRFinder(model, optimizer, criterion, device=device)
    lr_finder.range_test(train_loader, num_iter=20)
    lr_finder.reset()
    print(model.state_dict()['conv1.weight'][0])

def test_run_2():
    # I just rename `lr_finder.py` of my version to `lr_finder2.py` for a quick demo
    from lr_finder2 import LRFinder

    device = torch.device('cuda')
    model = ConvNet()
    model = model.to(device)
    optimizer = optim.Adam(model.parameters())
    criterion = nn.NLLLoss()

    data_folder = './data'
    train_data = datasets.MNIST(data_folder, train=True, download=True,
        transform=transforms.Compose([
            transforms.ToTensor(),
            transforms.Normalize((0.1307,), (0.3081,))
    ]))
    train_data = Subset(train_data, range(1000))
    train_loader = torch.utils.data.DataLoader(
        train_data, batch_size=64, shuffle=True
    )

    print(model.state_dict()['conv1.weight'][0])
    lr_finder = LRFinder(model, optimizer, criterion, device=device)
    lr_finder.range_test(train_loader, num_iter=20)
    lr_finder.reset()
    print(model.state_dict()['conv1.weight'][0])

if __name__ == '__main__':
    # In this case, you can see that 'conv1.weight' is not restored to the original state
    test_run_1()
    # In this case, 'conv1.weight' will be restored to the original state
    test_run_2()

Thanks for the sample code; I was able to reproduce the issue with it.
When I tested it before I always had the model in the CPU and the device was cuda; with that configuration, everything works because the model_state is in the CPU but the model is in the GPU, so reset() loads the CPU model and overwrites the GPU model. Whenever the model and the model_state are in the same device reset() fails.
I personally prefer to not write things to disk, so I think the deep copy option is more interesting but at the same time it does use memory so having the save to file option is also good.
My suggestion would be to add a 'memory_cache' boolean input argument that by default is set to True and would make a deep copy of the state_dict. When set to False it would save to a file exactly as you have done already. What do you think?

I get it, and I think that your suggestion is a much more proper approach for this issue!
Using memory for caching state_dict is usually faster then reloading from a file, and it also reduce the chance of frequently reading and writing on disk. For the case of running range_test with a large model, users are also able to switch the strategy of loading state_dict by changing the value of argument memory_cache.
I am willing to implement this improvement later.

Nicely done. Merged, thanks for the PR!

Cheers!


Question : how to get orderbook from a websocket Client object?

https://github.com/bitfinexcom/bitfinex-api-go/blob/286a8310973258e9b33e66475f38697f83e813c0/v2/websocket/client.go#L116
Sorry, but I did not found the clue how to get the order book even if I have set ManageOrderbook to true.
Thank so much!!!

Hi @gpmn currently the local orderbook is not exposed, it is just used to ensure that the websocket stream is in sync. Bitfinex generates a crc32 checksum of the top 25 trades in its book and passes that down the websocket every few seconds. The manage orderbook flag tells the go lib to keep a local copy of the all the trades coming through the websocket and generates its own crc32 checksum using the same method as Bitfinex. If the crc32 checksum is not equal to the bitfinex crc32 checksum then we are not in sync anymore so we reconnect to the websocket and reset the orderbook.
But saying that I can definitely see the benefits of being able to have quick access the local orderbook so Ill try and put something together. Thanks for the suggestion

Thank you, it should be a great improvement!

Just giving you an update on this @gpmn. I've created this pull request (which will be merged hopefully tomorrow) https://github.com/bitfinexcom/bitfinex-api-go/pull/156/files which exposes the order book to the consumer.
You can use it like this:
// Load the latest orderbook
ob, _ := wsClient.GetOrderbook(bitfinex.TradingPrefix+bitfinex.BTCUSD)
if ob != nil {
    log.Printf("Orderbook asks: %s", ob.Asks())
    log.Printf("Orderbook asks: %s", ob.Bids())
}


Git label commenter

Signed-off-by: Monalisha Ojha<EMAIL_ADDRESS>Description
This PR fixes #20
Notes for Reviewers
Added configuration for:

issue/remind
issue/dco
duplicate
invalid
proposal
wontfix

Signed commits

[X] Yes, I signed my commits.

hey @monalisha31 , looks like you forgot to Sign-off your commit, you could follow: https://github.com/layer5io/linkerd-service-mesh-workshop/pull/21/checks?check_run_id=3661063206 or if you don't have a sign-off set, you could give this a read https://github.com/layer5io/layer5/blob/master/CONTRIBUTING.md#signing-off-on-commits-developer-certificate-of-origin

@warunicorn19 Thanks for letting me know. There was some error before while signing off the commits. Now, the checks are passed.

@warunicorn19: We need to create a label for issue/dco. :rocket:

@amino19 done!


ampere support

based on this post :
https://www.techpowerup.com/312631/nvidia-bios-signature-lock-broken-vbios-modding-and-crossflash-enabled-by-groundbreaking-new-tools
is there any progress in supporting ampere architecture  GPUs?

There is no progress as Ampere GPUs that officially support vGPU use SR-IOV. As far as I know at the time of writing this response, there is no current VBIOS tweak or hardware modification to enable SR-IOV capabilities on consumer Ampere and newer cards.


test: fix ssr-search-parameter-manager e2e tests fails

https://coveord.atlassian.net/browse/KIT-2782
The tests were failing both in dev and prod locally.

If the test is now correctly working/no flakiness, then we should mark it as blocking for a valid build.
This is done in .github/worflows/prbot.yml , making it a "required-job"

The test works without flakiness in CI as it runs against next.js in prod mode. It fails only when next.js is run in dev mode.


[4.1] DynamicCasts: Emit a direct copy if soure and target SIL types match

DynamicCasts: Emit a direct copy if soure and target SIL types match
unconditional_checked_cast_addr ImplicitlyUnwrappedOptional in %136 : *Optional to *Optional in %135

Description: Casts involving IOUs crashed the compiler.

public class  Crasher {
public func crash() -> Dictionary<String, AnyObject?> {
let world = Dictionary<String, AnyObject!>()
return world
}
}

Origin: We must have changed how we treat IOUs recently

Scope: Target fix in the cast optimization

Risk: Low. Targeted fix of code that without fix would have cause a compiler crash.

Testing: A CI test was added.

rdar://35987160

@swift-ci Please test

@swift-ci Please test source compatibility suite

@slavapestov You reviewed this change on trunk. Could you give me your okay for swift-4.1-branch?

@swift-ci please nominate

@swift-ci Please test source compatibility


##[error]The variable '$RootFolder' cannot be retrieved because it has not been set.

Describe the bug
Hello, when using "ADF deployment from ARM Template files" component I get an error:
2022-12-22T10:43:41.4337812Z STEP: Reading Azure Data Factory from ARM Template files...
2022-12-22T10:43:41.5633948Z ##[error]The variable '$RootFolder' cannot be retrieved because it has not been set.
2022-12-22T10:43:41.6005998Z ##[section]Finishing: ADF deployment from ARM Template files
To Reproduce
YAML:
steps:

task: SQLPlayer.DataFactoryTools.DeployADFFromArm.DeployAdfFromArmTask@0
displayName: 'ADF deployment from ARM Template files'
inputs:
azureSubscription: '$(subscription)'
ResourceGroupName: '$(resource_group_name)'
DataFactoryName: '$(adf_name)'
location: 'West Europe'
csmFile: '$(System.DefaultWorkingDirectory)/($path_to_artifacts)/ARMTemplateForFactory.json'
CreateNewInstance: false
StopStartTriggers: true

Expected behaviour
There is nowhere to set RootFolder variable. I have tried to set environment variables in the component and pipeline variables, but the error remains.
Screenshots

Version of PowerShell module of:

ver.........

Provide all files to reproduce the issue

Attach at least log file (you can wipe out any potentially sensitive info)
Attach files of objects which are involved in the process

This issue should be fixed and released yesterday in version 1.23.


Use str IDs for RGCN notebook, not rdflib types

Previously the notebook wouldn't run because numpy was getting confused by the rdflib term types that were being used as the node IDs before; in particular, rdflib.term.URIRef and rdflib.term.Literal are subclasses of str, so np.array(some_ids) gets treated as an array of strings. However, the types get preserved as-is elsewhere (in particular, in a pandas Series or Index) and so there's a mismatch between rdflib.term.URIRef and str when doing node look-ups later.
Arguably, just converting these types to proper strings isn't the correct fix, and instead we should extend StellarGraph to support "weird" ID types like this, but that seems somewhat subtle. Thoughts?
See: #911, #972

I opened https://github.com/stellargraph/stellargraph/issues/988 with a minimal reproducer, and a bit of a discussion about what we might do.


dbt: Add DREMIO to supported dbt profile types

Problem
The dbt provider has a hardcoded set of supported dbt connection types. This adds dremio to the list.
Closes: https://github.com/OpenLineage/OpenLineage/issues/2668
Solution
Add static DREMIO to the Adapter(Enum)
Add an if clause for DREMIO to DbtArtifactProcessor.extract_namespace

[ ] Your change modifies the core OpenLineage model
[ ] Your change modifies one or more OpenLineage facets

One-line summary:
Add support for dbt-dremio, solving https://github.com/OpenLineage/OpenLineage/issues/2668
Checklist

[x] You've signed-off your work
[x] Your pull request title follows our guidelines
[ ] Your changes are accompanied by tests (if relevant)
[ ] Your change contains a small diff and is self-contained
[ ] You've updated any relevant documentation (if relevant)
[x] Your comment includes a one-liner for the changelog about the specific purpose of the change (if necessary)
[ ] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
[ ] You've added a header to source files (if relevant)

SPDX-License-Identifier: Apache-2.0
Copyright 2018-2023 contributors to the OpenLineage project

Will add sign-off and initiate it again


The request contained an unknown or invalid Host header OpenVAS error

Hi, I have followed the docker installation and configured the OpenVAS in it. Now when i try to access the console through browser, i am getting the error 'The request contained an unknown or invalid Host header. If you are trying to access GSA via its hostname or a proxy, make sure GSA is set up to allow it.'
I have also tried re-installing the OS but still the issue remains the same. Help is needed bit urgent. Please help me in this regard.
OS used:
CentOS 7
Ubuntu 18.10

try to add
PUBLIC_HOSTNAME:
in docker-compose openvas environment


SUBMARINE-281. Remove Chinese characters

What is this PR for?
Remove Chinese characters from source code
What type of PR is it?
[Improvement]
Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/SUBMARINE-281
How should this be tested?
https://travis-ci.com/cchung100m/submarine/builds/149643127
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@cchung100m Thank you for contribution.
Will merge if no more comments.

Hi @liuxunorg
Thanks for review.
Hi @jiwq
Sure, any suggestions/guidelines for more description info  are welcome. :)


Correct versioning numbers for macOS download

Description
Describe your changes in detail
Correct versioning numbers for macOS download

Motivation and Context
Why is this change required? What problem does it solve?
Versioning numbers for macOS download  were always 0.1.0_pre

How Has This Been Tested?

[ ] Please describe how you tested your changes.
[ ] see how your change affects other areas of the code, etc.

Checklist:
Go over all the following points, and put an x in all the boxes that apply.
If you're unsure about any of these, don't hesitate to ask. We're here to help!

[x] I have read the CONTRIBUTING document in my spoken language, and understand the terms
[ ] I have updated (or added) the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] I have gone through all the steps, and have thoroughly read the instructions

Very nice. Many thanks.

As far as i can I see a few jobs are failing for unrelated reasons (one related to clang formating and another one related to FreeBSD).

Yeah I did that from outside


 I noticed that they all have their own launch file. Do these launch with Movo1 launch?

I mainly see a bunch of config files and they seem to be reasonable. I noticed that they all have their own launch file. Do these launch with Movo1 launch?

so each launch is for a specific use case. ill explain more in the readme. these nodes are purely information processing so they can run on any machine. they get all their data from ros topics and pub to ros topics (no IO)

younes todo move base rviz requires bas frame be map and that the goal pose publish to move_base_navi special topic not the default rviz has

Here's a video showing MoveIt usage on the movo in simulation
https://user-images.githubusercontent.com/52610118/208826921-003b0b03-64c4-415b-ba60-e17a69df410b.mp4

Looks good! Good work!


Finished instances starting with a "b" and added SpecialMesh

12 Instances added to the documents which I may say is VERY cool.

Hey, thanks for your contribution. Before we can merge I noticed one issue. You put the long descriptions in the metadata description, you want to put the short ones there, and put the long ones under the main description section. Other than that, great work and I’m looking forward to merging your contribution.


fixes the initialization of icache

Before: the tag data in the MSBs was set to 0 always
Now: we set the tag data do the MSBs of the pc (we should have been doing this the whole time).

there's no issue with the code as written - I had been confusing myself with a stale binary.

Stale manycore binary?

Yeah - I was seeing icache misses that I didn't think should be there but it turned out that when you run make <test>.log from bsg_replicant it doesn't automatically rebuild main.riscv

Yeah, this is a known issue. I've tried to solve it several times unsuccessfully.

(fwiw, solved in baseline)

not trying to point fingers - just explaining why I closed the PR for posterity.

No worries, didn't feel like you were. Just trying to understand what stale binary meant.
It's been a frustration for me too, obviously.


mTLS definition

We should have a definition for mTLS. What it is and what it does.

Hey,
Can you provide more info about this issue,
and where it's located.
Mutual Transport Layer Security (mTLS)

Hi @ShivankPro, we need a new definition for mTLS that follows the style guide.  You'd be creating a new file — we don't have anything on mTLS yet. Does that make sense?

Hi @CathPag , I am just trying to start with some Beginner-friendly issues,
Therefore trying to know, How should I get started.

Does this help?

Yes, it is,
Thanks a lot!

Hey @Alt-Shivam I set you as the assignee, please reach out to me on slack if you need any help. Here's a definition we have for it in the linkerd docs: https://linkerd.io/service-mesh-glossary/#mtls
Thanks!

Yeah, Sure


Unexpected Error: Non-relative paths are not allowed when 'baseUrl' is not set. Did you forget a leading './'?

https://github.com/vitejs/vite/runs/7047103190?check_suite_focus=true#step:10:93
reproduction: https://github.com/JounQin/vite/tree/chore/tsx_esno
run pnpm i && pnpm build
Maybe related to #16

Please provide a minimal reproduction and explain why this is a bug.
pnpm run build runs so many commands I have to investigate which one is causing it and which tsconfig.json is problematic.

@privatenumber Please help to review #18

:tada: This issue has been resolved in version 4.0.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat(concealer, traffic_sim): ss2/AWSIM connection: concealer split

Types of PR

[ ] New Features
[x] Upgrade of existing features
[ ] Bugfix

Link to the issue
https://tier4.atlassian.net/browse/AJD-660
Description
This PR is step 2 to implement architecture two discussed in the issue.
The following changes were introduced:

simplified the concealer's threading configuration
Added stopAndJoint method to TaskQueue and fixed concealer's deinitialization
Replaced publishers and subscribes macros with a generic class
Split concealer into "User" and vehicle interface parts

More detailed explanations and reasoning behind the changes can be found in this document
How to review this PR.
The changes in the concealer are rather extensive so in the case of autoware.* and autoware_universe.*, the recommended review way would be to review the whole files rather than diffs.
Others
n/a

@piotr-zyskowski-rai
I will start the review.
We intend to review carefully, so replies may be delayed.


Useless changes

This is a really useful mod.

Code completely rewritten
Code cleanup
Optimization on_step
Slight texture compression

Thanks for your contribution :slightly_smiling_face:
I merged this manually in https://github.com/minetest-mods/gauges/commit/f9471512bd4607465d7d5a6234aed0925056dcde, keeping the original author name. Please respect their work.


Please support the Whisper models. It's a real hassle to run them locally.

Please include support for the Whisper speech-to-text models. It's a real hassle to run them locally.

https://github.com/ollama/ollama/issues/1168
In particular, https://github.com/ollama/ollama/pull/6241 indicates new PRs for voice are incoming.


Bugs and Future Expandability

Bugs

Removed outline that affects Mac Browsers. (images at the end)
The message text area is not only vertically resizable. Horiz resize would break the send button.
align-items value was misspelled as strech.  changed it to stretch.

Future Expandability

Changed li's hover backgorund-color to a .2 opacity of black. Changing nav's background color will be responded automatically.
Removed padding and nav li a. New li item will take it's own space w/o breaking the nav; and the whole li is still clickable.

Misc.
Updated .gitignore for Mac and Prepros.

Images of Outline

EDIT (11/8)

Container-all won't overflow-x when nav-is-open
Nav was too wide when viewport was too wide; fixed max-width to 800px
Updated the container-all to fit the max-width: 800px for the nav

Thanks a ton! Great ideas :)