Regex matching documentation not clear

It's not clear/I can't work out how to match the error message in the output (below) it picks up the previous output lines too and marks them all as the error -- becuase of the \n at the start of the match no doubt. I tried an anchor to start of line ^, but it does not appear to work at all in that case.
More examples would be most helpful.
"errorMatch": [
    "\n(?<file>[^(]+)\\((?<line>[0-9]+),(?<col>[0-9]+)\\):\\s+Error.*"
  ]

Executing: dub  build -q

## Warning for package gltest ##

The following compiler flags have been specified in the package description
file. They are handled by DUB and direct use in packages is discouraged.
Alternatively, you can set the DFLAGS environment variable to pass custom flags
to the compiler, or use one of the suggestions below:

unittests: Call DUB with --build=unittest

missing.d(13,3): Error: semicolon expected following function declaration

build uses XRegExp to match the content of the output.
I'm thinking you want something like: http://regexr.com/3c8b9 (with the appropriate namings (e.g ?<file>, etc) inserted.

Unfortunately the regex you showed me is more or less the same as what I wrote, but it doesn't always work. It seems to break when there is only one error output line, plus the other build output; and it works properly only when there is more than one error in the output.
It seems to be because the /m multiline flag just matches all lines and I can't think of a way to stop it at the moment.
I tried ^ at the start instead of \n, but that doesn't help either.
With this:
 "errorMatch": [
    "\n(?<file>[^(]+)\\((?<line>[0-9]+),(?<col>[0-9]+)\\):\\s+Error.*"
  ]

I get this -- notice the underline. There is nothing I can find to make it just grab the error line alone.

Error matcher is not setting the multiline option, altough maybe it should.
What about this regex: \n(?<file>.+\.d)\((?<line>\d+),(?<col>\d+)\) ?

I have found a solution it is this:
  "errorMatch": [
    "\n(?<file>[^(\n]+)\\((?<line>[0-9]+),(?<col>[0-9]+)\\):\\s+Error[^\n]+\n"
  ]

Changed the file name capture to add an additional \n preventing multi line matches:
[^(\n]+
That works. The mult-line must be on already no?

Well, /m makes ^ and $ match newlines. Without it, the whole output is considered one string.
I'm glad you got it to work.

Would you consider adding a set of pre-defined error matching filters, something like this:
"errorMatchLang" : ["dlang-dmd2"]
where dlang-dmd2 is maps to a built in regex?
Or even, this, which might be too much:
"errorMatchLang" : ["http://anywhere.com/dlang-dmd2.regex"]
I just see that a lot of people are going to go though the same thing trying to get their output to match.

I think the best solution to this would be creating a build provider which includes the regex

On 19/11/15 13:21, Alexander Olsson wrote:

I think the best solution to this would be creating a build provider
https://github.com/noseglid/atom-build#service-api-for-package-developers
which includes the regex
—
Reply to this email directly or view it on GitHub
https://github.com/noseglid/atom-build/issues/227#issuecomment-158054870.

I see yes. Thank you. I will look into that.


feat: Implement Archive Chat Button/Feature for Improved Organization and User Experience

Is your feature request related to a problem? Please describe.
Users may want to temporarily move chat conversations out of their current active chat list but not delete them entirely.
Describe the solution you'd like
Adding an Archive Chat button would allow users to easily access and review archived chats without affecting their active chat list.
Describe alternatives you've considered
None, as this feature request aims to improve the organization, user experience, and communication management for users.
Interface Design:
Create a clean and simple design for the button to match Open WebUI's existing aesthetics. The button should be easily identifiable and recognizable by its function.
Accessibility:
Ensure that the button is accessible to all users, including those with disabilities. This might involve providing text labels for the button and ensuring proper contrast and keyboard accessibility.
Implementation Details:
Database Structure:
Archived chats could be moved to a new table in the database, allowing for better organization and easier retrieval of data.
Front-end Changes:

A new archive button for each chat element on click, located to the left-hand side of the chat title for the conversation. This button would allow users to archive the chat with a single click.
A new "Archived Chats" option could be added somewhere within the Settings menu, accessible by clicking on the username in the bottom left side of the UI. This new Archived Chats option when clicked, would display a list of all archived chats, allowing users to search, filter, and manage their archived conversations.

Back-end Integration:

Admins would have the ability to control the archive expiration period, setting the duration for which chats remain archived before being automatically deleted or moved back to the active chat list.
Create a function for archiving chats: The back-end can include a function that moves chats from the active chat table to the archived chat table, updating the chat status accordingly.
Update the UI with the new status: The user interface would be updated to reflect the new chat status, displaying the appropriate visual cues and options based on whether a chat is active or archived.

Benefits to Users:

Improve organization: Provide users with more control over their chat history by allowing them to archive conversations they no longer need to see in their active chat list. This will help users maintain a clean and organized chat interface, making it easier to find and manage their conversations.
Enhance user experience: By providing an Archive Chat button, users can quickly find and revisit old conversations without having to scroll through or search through a large number of chats. This will save users time and effort, improving their overall experience with the Open WebUI platform.
Streamline communication management: Users can better manage and organize their chat history by using the archive feature instead of deleting or ignoring conversations. This will help users maintain a tidy chat interface while preserving important conversations for future reference.
Improved accessibility: The new "Archived Chats" option, accessible from the user options, ensures that users can easily find and manage their archived conversations. This will help users quickly locate and restore archived chats, improving their overall experience with the Open WebUI platform.
Admin control: Admin users will have the ability to control the archiving feature, including managing user permissions and access levels. Additionally, admins would have the ability to control the archive expiration period, setting the duration for which chats remain archived before being automatically deleted or moved back to the active chat list. This will help admins maintain a clean and organized chat environment, ensuring that users can efficiently manage their conversations.

This feature request proposes adding an Archive Chat button to allow users to temporarily move chat conversations out of their current active chat list without deleting them entirely. This will help users maintain a clean and organized chat interface, making it easier to find and manage their conversations. Additionally, an "Archived Chats" option will be added to the user options, allowing users to search, filter, and manage their archived conversations. Admins will have control over the archiving feature, including managing user permissions and access levels, as well as controlling the archive expiration period.

In the meantime, an alternative could exploit the tagging system to only show chats without a tag: #1060

An interim alternative could exploit the tagging system to only show chats without a tag: #1060

This sounds like a great suggestion that could work for now until any further changes could be done for deeper integration, if that’s a direction developers want to see taken with the WebUI!

Thanks for implementation! Very useful feature!
A related bug: when unarchiving a chat, the tags are not restored.

Thanks for implementation! Very useful feature!
A related bug: when unarchiving a chat, the tags are not restored.

No need to thank me, thank @tjbck for implementing the features. :)
But I will thank you for pointing out a related bug. It has been noted.

@mherrmann3 good catch, issue opened here: #1690


Firebase Auth not getting user email

this is how m handling my POST request.
Error i am getting is following
I/flutter ( 7426): {error: {code: 400, message: INVALID_EMAIL, errors: [{message: INVALID_EMAIL, domain: global, reason: invalid}]}}

This doesn't look Flutter related.
Looks like you are posting an invalid email address.
Please consider asking support questions in one of the other channels listed at http://flutter.io/support .

Hey, Have you solved this issue. If yes, then please help me. I am also facing the same issue.

Hey, Have you solved this issue. If yes, then please help me. I am also facing the same issue.

Hey, Have you solved this issue. If yes, then please help me. I am also facing the same issue.

same here .anyone with a solution?

Mismo problema por favor alqguien ya lo resolvio ?por favor Abdullah1353

you need to trim both email and password

You need to trim both email and password


bluebird or anypromise?

https://github.com/petkaantonov/bluebird/ is faster than q, and has some nice features.  Alternately, since this is a library that will likely be incorporated into other projects with existing promise libraries, we could use https://github.com/kevinbeaty/any-promise

I have no strong opinion.
If you have worked with any-promise and not experiences any problems it looks like a good idea.

I used q only because it was the first one I found

I'll send a PR for any-promise.


xNN renaming output neurons

Describe the bug

A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

Go to 'xNN Plugin'
Click on 'Neurnal Network NEPOnn'
Double-Click on output neuron
See error

Expected behavior

The renaming window appears "far away" from the output neuron

Screenshots

Device information

Type: laptop
OS: Max
Browser chrome

a minor comment: please remove always the template text (as 'A clear and concise description of what the bug is.' resp. replace it.
Further: I expect, that the expected result is, that the rename window is close to the neuron.

getting the d3 relative mouse position behaves different for click and double click

It seems to work fine with Input-Neurons


[FLINK-25796][network] Avoid record copy for result partition of sort-shuffle if there are enough buffers for better performance

What is the purpose of the change
Currently, for result partition of sort-shuffle, there is extra record copy overhead Introduced by clustering records by subpartition index. For small records, this overhead can cause even 20% performance regression. This ticket aims to solve the problem.
In fact, the hash-based implementation is a nature way to achieve the goal of sorting records by partition index. However, it incurs some serious weaknesses. For example, when there is no enough buffers or there is data skew, it can waste buffers and influence compression efficiency which can cause performance regression.
This ticket tries to solve the issue by dynamically switching between the two implementations, that is, if there are enough buffers, the hash-based implementation will be used and if there is no enough buffers, the sort-based implementation will be used.
Brief change log

Dynamically switching between the two implementations, that is, if there are enough buffers, the hash-based implementation will be used and if there is no enough buffers, the sort-based implementation will be used.

Verifying this change
This change added tests and existing tests can also help to verify the change.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Very thanks @wsry for the PR! I have left some comments.
For the long run, I tend to we move the difference implementation to the implementation class, like the strategy to split write buffers and sort buffers and how they write the buffers to the files. Perhaps we could create a new issue for the future refactor?
Also if possible I tend to we rename the classes to DataBuffer, HashBasedDataBuffer and SortBasedDataBuffer and also the variables to avoid reuse the word sort.

@gaoyunhaii Thanks for the review and comments. I agree that we can rename the sort buffer class in this PR. As for other refactor, I will create a new ticket to do it latter. I will update the PR soon.

@gaoyunhaii I have updated the PR.

@flinkbot run azure


eureka client not register  it self ofter a long time network connection problem

after a long time network connection problem
client instance is removed from eureka server
renew thread try to send heart beat but eureka server return 404
sendHeartBeat throw TransportException with 404 status code
in the code below, try to re-register client but cause step 4 throw exception, so the re-register logic not executed

boolean renew() {
        EurekaHttpResponse<InstanceInfo> httpResponse;
        try {
            httpResponse = eurekaTransport.registrationClient.sendHeartBeat(instanceInfo.getAppName(), instanceInfo.getId(), instanceInfo, null);
            logger.debug(PREFIX + "{} - Heartbeat status: {}", appPathIdentifier, httpResponse.getStatusCode());
            if (httpResponse.getStatusCode() == Status.NOT_FOUND.getStatusCode()) {
                REREGISTER_COUNTER.increment();
                logger.info(PREFIX + "{} - Re-registering apps/{}", appPathIdentifier, instanceInfo.getAppName());
                long timestamp = instanceInfo.setIsDirtyWithTime();
                boolean success = register();
                if (success) {
                    instanceInfo.unsetIsDirty(timestamp);
                }
                return success;
            }
            return httpResponse.getStatusCode() == Status.OK.getStatusCode();
        } catch (Throwable e) {
            logger.error(PREFIX + "{} - was unable to send heartbeat!", appPathIdentifier, e);
            return false;
        }
    }

Eureka client always retry to send heartbeat with no re-register

It's pretty ridiculous a 404 response from the server would be a TransportException as the transport is perfectly fine and this is an application protocol code but if this is the case then it's a bug as the code you included clearly assumes it will not result in an exception. Open to PRs.


.Net: ONNX connector requires MaxTokens is set

Describe the bug
ONNX connector doesn't work unless you indeed specify MaxTokens on OnnxRuntimeGenAIPromptExecutionSettings
... if you don't (which the API pattern allows not using execution settings), the ONNX impl w/ Phi-3 at least fails with a very obscure message.

Created this issue so we investigate what is happening here and what the correct behaviour should be. If the expectation is that we include default values in the request for all models we may have to set a default in our prompt execution settings.

According to the ONNX Runtime team, by default the MaxTokens if not provided is set to the current model maximum capacity.
I'm closing this thread as any investigation on the details of the error is being taken care offline.


string_params_for should cast embeddings into maps

Lets say you have a Car that embeds_one Engine.
def car_factory do
  %Car{
    engine: build(:engine)
  }
end

if you call string_params_for, the nesting will still have a struct typed Engine instead of a plain map. i.e:
%{
  "engine" => %Engine{ ... }
}

not
%{
  "engine" => %{ ... }
}

Hi @Sitch, I find that a little counterintuitive since string_params_for(:car) doesn't give me %Car{} with string keys but %{} with string keys. So if that's the behavior at the top level, I would think we would keep that behavior consistent when dealing with nested structures. What do you think?

@germsvel the use case for this post was getting a semi-deep nested json payload from a 3rd party provider.  For me, it was useful to not have to write the same factory in two different ways (one version with typed nested structures and one version that represented the json decoding with string keys).  Additionally, the rational in the docs for string_params_for is phoenix controllers, which would not implicitly type cast those nested arguments

@Sitch I'm sorry. I misread your original post. I thought you wanted string_params_for/2 to give,
%{
  "engine" => %Engine{ ... }
}

I have now read it correctly. You're saying that's the current behavior, but that instead it should give you something like this,
 %{
  "engine" => %{ ... }
}

I agree. Sorry for the confusion.

@germsvel no worries -- gotta work on my verbal clarity 👓.  Let me know if you want help with the PR

@Sitch thinking about it, I actually think https://github.com/thoughtbot/ex_machina/pull/224 will fix this issue.

I believe #224 closes this. If not, please re-open. Thank you for reporting this!


[WIP] Redesign Power Down Screen

What does this PR?
This PR redesigns powerdown screen similar to delegate screen.

Reused beneficiary selection module from post options modal. Currently it is placed separately in components and beneficiary in post options modal isn't replaced with this one but can be replaced later.
redesigned slider and value input similar to delegate screen.

Todos

text has to be replaced with suitable text and titles
multi step has to be done,
currently OptionsModal has been used for confirmation, may be need to be replaced with more intuitive actions sheet with some extra info of the operation.
code cleanup is also remaining as for now commented the old code, maybe we need to reuse some info from there as well like info related to incoming funds , still needs to decide place for incoming funds and then we can place them accordingly.

Issue number
fixes #2246
Screenshots/Video
https://user-images.githubusercontent.com/48380998/163926249-6c2bfa81-7131-441e-a675-8f3a2f3c37c5.mp4

I have not reviewed code in detail yet but beneficiary content can be generalised rather than having different reading mechanism for power down and beneficiary.
functionality wise it is working well... check the screenshot below for changes required
Auto Vests label can be multiline perhaps

yeah that can be generalised, the reason I didn't replaced the beneficiary selection module in post options modal with this one is that we can do that in separate PR after this one is done, I am just cautious and dont want to break things

Fine tuned power down screen UI...


Fix 'gridlabd help' command output

Fix output from gridlabd help command so it's clean.
Note, this is a repeat PR due to mismanagement of conflicts in previous PR#1238

Closing and reverting to original PR #1238 until develop is fixed


[Question] How does the S3 package share the credentials between client and server?

I was playing around with the AWS SDK in MeteorJS and had some issues accessing config credentials in the client if I stored the credentials in the server. But when I used your package it makes it possible to have the config setup in the server side and have client side upload and delete function possible.
Just looking into working with AWS SDK and any help would be appreciated in this regard.

There isn't really a good way of interfacing with AWS SDK on the client (because it's not meant to be on the client, if it was you'd expose your private keys). AWS SDK assumes your infrastructure has server-side rendering (which Meteor does not have out of the box) so that you can build forms with the required params on the server and pass them directly to the client. This package uses Meteor.methods to interface with server functions and get info like this which in turn produces a virtual form to make the upload work.

Much appreciated and thank you for the insight


First Release

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


After publish, does Exponent run a diff on the code and only download the delta?

When publishing an app, the new script is downloaded to the app.
Does it only download the code changes or is it a full download every time?

No we don't use a delta right now but are planning on using a delta or something similar in the future.

Is there a way to calculate or find the size of this download?
I'm busy producing an app for a country with bad internet connections and so it is quite important to understand the impact of a publish.
Also, please let me know if I should be asking these questions somewhere else.

This is a fine place for questions! We're usually more responsive on http://slack.getexponent.com/ if you want to have faster conversations.
After you publish you get a url that looks like https://exp.host/@jesse/native-component-list. If you change the url to https://exp.host/@jesse/native-component-list/index.exp?sdkVersion=14.0.0 you can see the manifest. That has a field named bundleUrl that links to the file we download on the client.

Hi! I'm moving this to a feature request on Canny so we can track it with other enhancements we're planning. Thanks!


Add FAQ

Added Frequently Asked Questions

added a notebook that goes through solutions for FAQs.
Has the following questions answered:

EntitySet
  - How do I get a list of variable (column) names, and types in an `EntitySet`?"
  - What is the difference between **copy_variables** and **additional_variables**?"
  - Why did variable type change to Id, Index, or datetime_time_index?"
  - How do I combine two or more interesting values?"

DFS
  - Why is DFS not creating aggregation features?"
  - How do I speed up the runtime of DFS?"
  - How do I include only certain features when running DFS?"
  - If I didn't specify the **cutoff_time**, what date will be used for the feature calculations?"
  - How do select a certain amount of past data when calculating features?"
  - How do I apply DFS to a single table?"
  - How do I prevent label leakage with DFS?"
  - What is the difference between passing a primitive object versus a string to DFS?"

Features
  - How can I select features with specific strings after DFS? What about features that return certain types? Features at certain depth?"
  - How do I create **where** features?"

Primitives
  - What is the difference between the primitive types (Transform, GroupBy Transform, & Aggregation)?"
  - How do I get a list of all Aggregation and Transform primitives?"
  - How do I change the units for a TimeSince primitive?"

Errors/Warnings
  - Why am I getting this error 'Index is not unique on dataframe'?"
  - Why I am getting the following warning 'Using training_window but last_time_index is not set'?"
  - Why am I getting errors with Featuretools on [Google Colab](https://colab.research.google.com/)?"

2 comments:

Near code snippet 32/in section "How do I prevent label leakage with DFS":
What about if the user uses train test split to split the training and testing data? This might be a better example, because there are a few things that must be done to ensure accuracy with this method (ie matching the labels after making the features). I also don't really understand fully how the training and testing data is generated/divided in this example.
Also, how are categorical variables encoded when splitting training and testing data? What happens if the categorical variable has a value in the testing data but doesn't have it in the training data (this might be an uninformed question)

Below code snippet 64/in section "Why am I getting...?"
Would it be possible to further clarify the difference between last time index and time index/clarify what it does with the excluded rows? Are these rows used in testing, or how are they factored in (if at all)? Again, not sure how well informed this question is.

These two questions might not be valid questions, in which case they can be ignored.


fix(transformer/react): should not collect it if it is a nested member expression use hook

close: #6139

#6143  👈
main

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

Join @Dunqing and the rest of your teammates on  Graphite


clean up and finalize symmetric crypto functionality

[x] Ensure we use common code paths for deriving symmetric keys for memo, note, and swap encryption (I believe we do for memo and notes): https://github.com/penumbra-zone/penumbra/issues/1265
[x] Add Ock type: OutgoingCipherKey: https://github.com/penumbra-zone/penumbra/issues/1267
[x] Generate a better name for the return value of Note::encrypt_key and add that struct
[x] Name the derived symmetric key e,g, PayloadKey (line 138 of note.rs)
[x] Provide some sort of interface to get the PayloadKey - this is a more fine grained functionality than the viewing functionality since it would allow viewing a single action
[x] Collecting the nonces together in one place (for swap, memo, note)
[x] https://github.com/penumbra-zone/penumbra/issues/1268
[x] Consider encrypting the shared secret directly instead of the transmission key and ephemeral secret
[ ] Document how the transaction key hierarchy works in the spec

Note that this is closely related to the memo change (#1222)

The SharedSecret used to derive the PayloadKey (with the ephemeral public key) can be derived from the transmission key and the ephemeral private key. The OvkWrappedKey saves the transmission key (32 bytes) and the esk (32 bytes). We could instead encrypt directly (as the OvkWrappedKey) either the SharedSecret (32 bytes) or the PayloadKey (32 bytes), either option saves 32 bytes and a DH key exchange. I’ve tacked that change onto PR #1284.


Wrong type for return value of mmap leads to UB

From the VERSIONS section in mmap man page:

If the MAP_FIXED flag is specified, and addr is 0 (NULL), then the mapped address will be 0 (NULL).

In this case, the kernel does allocate the requested memory at zero address (and dereferencing the NULL pointer will be OK since then). However, the return type of mmap and mmap_anonymous is designed as NonNull, which will then become UB for evident reasons.
The documentation of such functions refers SAFATY as to "See the mmap(2) man page for detailed requirements", while such man page does not impose any safety requirements on the above case.
So there are two ways to solve this:

Change the return type of mmap and mmap_anonymous from NonNull<c_void> to usize (we cannot use *mut c_void since Rust requires pointers to be non-null), which would however lose the provenance.
Add documentation to mmap and mmap_anonymous to add the above usage as a UB.

Hi, thanks for catching this!
I tried to reproduce this but got EPERM, could you please provide an example code that can reproduce the issue, not necessarily in Rust, C is ok. And I kinda guess this behavior is Linux-only?

Thank you for your quick response :)
#include <sys/mman.h>
#include <stdio.h>

int main() {
	void *res = mmap(0, 4096, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANON | MAP_FIXED, -1, 0);
	printf("res is %p\n", res);
	return 0;
}

If EPERM is returned, just sudo it :)

Hi, I actually tried sudo and run the binary with the root account, still got EPERM:
$ bat repr.c
───────┬──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
       │ File: repr.c
───────┼──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
   1   │ #include <sys/mman.h>
   2   │ #include <stdio.h>
   3   │
   4   │ int main() {
   5   │     void *res = mmap(0, 4096, PROT_READ | PROT_WRITE, MAP_PRIVATE | MAP_ANON | MAP_FIXED, -1, 0);
   6   │     printf("res is %p\n", res);
   7   │     return 0;
   8   │ }
───────┴──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────

$ gcc repr.c -o repr && ./repr
res is 0xffffffffffffffff

$ vim repr.c # check res and call perror()

$ gcc repr.c -o repr && ./repr
mmap: Operation not permitted

$ sudo ./repr
[sudo] password for steve:
mmap: Permission denied

$ sudo su -
root@fedora ~# cd workspace
root@fedora workspace# ./repr
mmap: Permission denied

Hmmmm, I tested it in Ubuntu 24.04 and it returns nil indeed.
My env:
$ uname -r
6.8.0-47-generic

$ clang --version
Ubuntu clang version 14.0.0-1ubuntu1.1
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin

As a supplement, some real-world programs do use this feature. For example, this paper (which is in IEEE S&P '20, which is one of the top conferences in security area) uses this feature to avoid crashes in forced execution.


Error executing

Hi all,
first of all I want to thank you for your apportation to the community. We have an ASA5520 and when we try to execute the script we have this error:
$ python getusers.py -f <IP_ADDRESS> -c SNMPPublic -o test.txt
Traceback (most recent call last):
File "getusers.py", line 107, in
main()
File "getusers.py", line 98, in main
users = asa.get_connected_users()
File "getusers.py", line 16, in get_connected_users
output = self._get_ascii_codes(output)
File "getusers.py", line 36, in _get_ascii_codes
output = [[x[0].replace(string_to_remove,'').strip(' '), x[1]] for x in output]
IndexError: list index out of range
Do you know what kind of problem could be?
Thank you for your support :)

Hmmm.. its been a long time since I wrote this script.
Can you put a "print output" after the output = self._commands_output_to_list(output) in the get_connected_users function?

Thank you for your fast answer, here is the output:
$  python getusers.py -f <IP_ADDRESS> -c SNMPPublic -o test.txt
Traceback (most recent call last):
File "getusers.py", line 108, in
main()
File "getusers.py", line 99, in main
users = asa.get_connected_users()
File "getusers.py", line 17, in get_connected_users
output = self._get_ascii_codes(output)
File "getusers.py", line 37, in _get_ascii_codes
output = [[x[0].replace(string_to_remove,'').strip(' '), x[1]] for x in output]
IndexError: list index out of range

_argument('-o', '--output', action='store', default='json', help='Output type, option are json and text. Defaults to json')
Your issue appears to be -o takes one of two arguments "json" or "text".  You're trying to give it a file name.  You need to pipe the output to a file yourself.
python getusers.py -f <IP_ADDRESS> -c SNMPPublic -o text >> test.txt

Dammit.. I didn't even spot that! Thanks :)

Sure, now it works! Thank ou mdiorio and dizzy!


Need an RPC command to get a RAW transaction minimum fee

Need an RPC command to get the minimum fee required to create a raw transaction
this way, using daemon would be easier to generate transaction without putting insufficient (or too many) fees
i think min fee should be available as RPC command because it's a daemon responsability to know that
P.S.
don't know if it's already available, but sendrawtransaction should check if the minimum fee has been used in the transaction

What are you trying to do?  The raw transaction interface is designed to be "raw", i. e., sendrawtransaction should just send the transaction and not do any higher-level "bitching around about the fee".  (If the fee is too low to be relayed at all, it won't accept the tx anyway.)
For sending transactions and having the daemon handle the internals (which includes determining an appropriate fee), you should use the higher-level interface like sendtoaddress and name_update.

problem is that i would like to create a transaction on behalf of some address i don't have in the wallet (but i know priv key, just not taken from wallet)
i want to create a raw transaction, sign it then send it from an hot wallet/daemon that doesn't hold such address
Try to explain better: think about my bounty system i talked about on forum, what i want is a service that connect to a running daemon, and has a wallet with a readonly address (loaded using importwallet) that is the address i use to get bounty request people send
I want to use the same service, to pay back to bounty killers when conditions are satisfied, so i need to generate a transaction using that readonly address, then sign using the private key i get from other sources (not wallet), maybe even on another pc, and than use the running daemon to send the signed transation to peers
so basically i can't use sendtoaddress because i need to pick the transaction inputs from the outputs sent to the bounty address and that wallet don't hold anyway the private key so it can't sign the transaction, so i have to build it, but then the problem is that to build such transaction without knowing the fee isn't easy (afaik)
Let me know If i haven't explained thing properly

Ok, the usecase is clear now.  How do Bitcoin applications solve this?  It actually shouldn't be too hard to compute the fee.  Or just attach, say, 0.05 HUC to be on the safe side (which isn't really anything at all).  If sendrawtransaction fails, you will know that you have to increase the fee.


AbstractItem error

I'm using your simple implementation, but when I rename the class I get this error on bindView method.
'bindView(ViewHolder)' in 'com.test.orion.sample.HistoryItem' clashes with 'bindView(VH)' in
'com.mikepenz.fastadapter.items.AbstractItem'; both methods have same erasure yet neither overrides the other. How can I solve this?

@EpsilonOrionis could you please provide some code? Which version are you using?
https://github.com/mikepenz/FastAdapter/blob/develop/app/src/main/java/com/mikepenz/fastadapter/app/items/SampleItem.java#L93
The bindView() takes the ViewHolder type which was defined for the AbstractItem<YourItem, YourViewHolder>
https://github.com/mikepenz/FastAdapter/blob/develop/app/src/main/java/com/mikepenz/fastadapter/app/items/SampleItem.java#L27
So you don't have to cast the RecyclerView.ViewHolder to the specific ViewHolder used for this item

I'm using your sample, still that message...

@EpsilonOrionis the readme is not updated to version 0.9.0 . Should be @Override     public void bindView(final ViewHolder holder) { instead of @Override     public void bindView(final RecyclerView.ViewHolder holder) {

It works, thank u :+1:


Does it support websocket?

I tried to switch the url and it showed:
http://ws://<IP_ADDRESS>:4337

NO, currently not supported

ok


curl: option -sfLo: is badly used here

Describe the bug
I guess curl options have some issue in the env vars, when I tried switching curl to wget (by editing the script), wget showed an error saying "No such file or directory"
Reproduction steps

Open your respective terminal
Enter the installation command: curl -sSL mangal.metafates.one/install | sh
observe the error

Expected behavior
It was supposed to install
Additional context
Using ZSH and Fedora Workstation 38
Operating System
Linux

Did you updated your system? Cause I am not facing any issue like that?
Also this is a copy of this problem #174


Tangent vector generation for vertexes

In addition to normal generation, we need tangent vector generation to allow for tangent-space effects like height maps and normal maps. There is already an implementation of this in stardazed/stardazed, but it needs porting and verification.

Implemented and already obsoleted.


[WIP] Update/correct logD analysis, correct experimental values

Previously, aspects of the logD analysis were incorrect/needed updating. This PR is to address that. In particular, it:

[x] Corrects the expression used for deriving logD from pKa and logP was incorrect and is now corrected. (This was not used in computing the experimental logD values for comparison; rather, the reported logD values were used. However, this affected the analysis which computed predicted logD from predicted pKa and logD)
[x] Provides derivation of the correct expression relating logD and pKa/logP
[ ] Updates the analysis using the correct expression to estimate logD from pKa and logP
[x] Updates the repository to provide the full experimental data needed for logD analysis, and scripts for using that data
[ ] Corrects the experimental logD values (the Ballatore lab is submitting an erratum, and we will also need to update our overview paper accordingly) based on an error in their provided values that we noticed in the process of this work
[ ] Update main README.md changelog to reflect the above changes

Once the above is done I will also need to get out an e-mail blast. I should probably do a new point release before merging this, as well, so people can easily/clearly indicate which version of the repo they are referring to.

I believe @dhimanray is updating the analysis based on the changes here; @dhimanray , we will need to work out who is doing the "Corrects the experimental logD values..." here which should be done in connection with that. Let's talk tomorrow.

@dhimanray It's this file which needs to get updated/corrected: https://github.com/samplchallenges/SAMPL7/blob/master/physical_property/experimental_data/Experimental_Properties_of_SAMPL7_Compounds.csv
Then we just re-run the analysis scripts in this directory -- https://github.com/samplchallenges/SAMPL7/tree/master/physical_property/logD -- though if you just get the values corrected I can re-run the scripts and update the README.md files.

@dhimanray It's this file which needs to get updated/corrected: https://github.com/samplchallenges/SAMPL7/blob/master/physical_property/experimental_data/Experimental_Properties_of_SAMPL7_Compounds.csv
Then we just re-run the analysis scripts in this directory -- https://github.com/samplchallenges/SAMPL7/tree/master/physical_property/logD -- though if you just get the values corrected I can re-run the scripts and update the README.md files.

I created pull request #137 for adding the correct experimental data in the CSV file. But I could not update the pdf file.


Blank window at first run

Hi there, I just found that the blank window appear at first run
Describe the bug
empty window appear at first run
To Reproduce
Steps to reproduce the behavior:

Close and reopen Antares
See error

Expected behavior
I expect that the window is appear smoothly
Screenshots

Application (please complete the following information):

Version 0.4.2 and the previous version
Distribution: Windows

Desktop (please complete the following information):

OS: Windows
Version Windows 10 build 18363

Additional context

Hello @greveill,
Is your installation from Microsoft Store or .exe?

I install it from .exe

Hi @Fabio286 , this version is from the in app update, but I have this issue from the previous version. This blank window is about a few second. I will try to fresh install and let you know soon (it's in my office computer)

Thank you for collaboration, no need to try a fresh install. I thought in your case Antares was stuck at that screen.
Sadly it's a known issue of latest versions of electron (14, 15, 16, and 17-beta).
I'm considering a downgrade to electron 13.6.6 until they release a fix.

it's happen also for the fresh install. I hope downgrading electron will be a good solution. thanks for your attention, have a good day

I found a temporary workaround without downgrade electron, to hide the window before its fully loaded.
This should work until electron team fixes this issue.

Sadly from latest Windows 11 update this problem is come back. It occurs on VSCode too.
We have to wait for an Electron (or Windows) fix.

I found a temporary workaround without downgrade electron, hiding the window before its fully loaded. This should work until electron team fixes this issue.

I am forced to introduce the temporary workaround adopted months ago, at least until the bug is fixed.


fix: sync when forkchoiceUpdated hash is unknown

Attempts to get us to sync when we are given an unknown head hash from engine_forkchoiceUpdatedV1. We can't perform other engine checks if we don't have the full block.
To do this, PipelineTarget::Hash(hash) is added to the possible pipeline targets, and we run the pipeline until we reach the unknown hash.

Adds contains_block to the BlockchainTree

Codecov Report

Merging #2133 (b6551a5) into main (f4fc9b5) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #2133      +/-   ##
==========================================
- Coverage   44.83%   44.82%   -0.01%
==========================================
  Files         440      440
  Lines       53758    53780      +22
==========================================
+ Hits        24101    24109       +8
- Misses      29657    29671      +14

Flag
Coverage Δ

integration-tests
19.30% <0.00%> (-0.02%)
:arrow_down:

unit-tests
35.50% <100.00%> (+0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

crates/interfaces/src/blockchain_tree.rs
80.00% <ø> (ø)

crates/consensus/beacon/src/engine/mod.rs
57.55% <100.00%> (+0.73%)
:arrow_up:

...ates/executor/src/blockchain_tree/block_indices.rs
38.46% <100.00%> (+0.84%)
:arrow_up:

crates/executor/src/blockchain_tree/mod.rs
19.03% <100.00%> (+0.40%)
:arrow_up:

crates/executor/src/blockchain_tree/shareable.rs
37.50% <100.00%> (+3.53%)
:arrow_up:

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

on second thought, geth notes that this is an unexpected case:
https://github.com/ethereum/go-ethereum/blob/2adce0b06640aa665706d014a92cd06f0720dcab/eth/catalyst/api.go#L225-L233
	// If the head hash is unknown (was not given to us in a newPayload request),
	// we cannot resolve the header, so not much to do. This could be extended in
	// the future to resolve from the `eth` network, but it's an unexpected case
	// that should be fixed, not papered over.

So maybe we should instead just return Syncing here and not run the pipeline?

Closing this PR in favor of the above logic ^


[core] Fix eslint-plugin-react-compiler issues - usePagination.test.js file

Fixed eslint issue with usePagination.test.js file by using React.useEffect hook to update hook after execution, error: Writing to a variable defined outside a component or hook is not allowed. Consider using an effecteslint(react-compiler/react-compiler).
Ran the test file and had 16 tests passed, so should work :)
Fix for the  packages/mui-material/src/usePagination/usePagination.test.js file which the box was unchecked from issue #42564 , where issue #43117 still contained the eslint error.

[X] I have followed (at least) the PR section of the contributing guide.

Hello @aarongarciah !
Im new to opensource and want to do my first contribution to MUI!
I saw an older thread from june (#42564), where I fixed an Eslint issue with usePagination.test.js file and did this PR. Uploaded the yaml file to fix circleci: checkout with pnpm deduplicate accoarding to CONTRIBUTE.md file. Is the PR ready for review or is there additional things to add?
Thank you in advance! 😃


Support of ImageOverlay ?

Hi,
Could you please add support of image overlay inside this flutter_map library ?
Is there any workaround solution to be able to add an image on the map ?
Thanks

You can use the MarkerLayerOptions with Marker. Marker uses a builder where you can return an Image Widget.

@wawanopoulos can you describe more what you mean by overlay? Are you talking about a custom tile layer or a marker, or something else?

@johnpryan I would like to insert an image in overlay on the Map at a specific position. Like this using Leaflet on the web: https://leafletjs.com/reference-1.5.0.html#imageoverlay

Sounds like OverlayImage? https://github.com/johnpryan/flutter_map/blob/master/example/lib/pages/overlay_image.dart

@ay8s is correct, it sounds like OverlayImage is what you're looking for. Feel free to re-open or file an issue if it's not what you need.


FancyZones Reverts to Default Layout Every 24 Hours - Should Use Active Layout Until Told Otherwise

Microsoft PowerToys version
v0.51.1
Running as admin

[X] Yes

Area(s) with issue?
FancyZones, FancyZones Editor
Steps to reproduce
First, recall that FancyZones has a preset layout called PriorityGrid.
Next, create custom layout (doesn't matter what it looks like). Let's call this FancyZoneBad.
Finally, create a second custom layout. Let's call this FancyZoneGood.
Next, make FancyZoneGood your active layout in FancyZones
Wait 24 hours (also let PC go to sleep and require user password to log back into Windows).
Issue - FancyZoneGood should remain the active layout. However, the active layout reverts to one of your presets (ex. PriorityGrid) or the first custom layout that was created (FancyZoneBad). At no point did you change the active layout, so FancyZoneGood should remain active.
✔ Expected Behavior
FancyZoneGood should remain the active layout.
❌ Actual Behavior
However, the active layout reverts to one of your presets (ex. PriorityGrid) or the first custom layout that was created (FancyZoneBad). At no point did you change the active layout, so FancyZoneGood should remain active.
Other Software
FancyZones Only

@SeraphimaZ Still seemss to be an issue. Do you know why?

@SeraphimaZ Still seemss to be an issue. Do you know why?

No, I don't. I need to look at a bug report at least to find it out.

/bugreport

Same issue here, bug report taken directly after wake from sleep.
PowerToysReport_2021-12-16-14-09-10.zip

I noticed here a resolution change and probably wrong monitor id "device-id": "...UID0_3440_1440_{E4C50100-4F8F-4112-9F20-0E9F9E2DB493}"
Resolution reference: https://github.com/microsoft/PowerToys/issues/14261
Let's not close as a duplicate since I need to check the monitor id retrieving:
https://github.com/microsoft/PowerToys/blob/3805348afdb216f2dea8f693b2554bdba2caba7e/src/modules/fancyzones/FancyZonesLib/util.cpp#L88-L114

Related to #13069 ?

@hkcomori #13069 was a different issue

Let's close this issue against https://github.com/microsoft/PowerToys/issues/16370.


Give each howto its own page

Preview of what docs look like: https://dsuo-flax.readthedocs.io/en/dsuo-howto-individual-pages
FYI for @rolandgvc.
Some notes:

I added a check to make sure the howto author adds a documentation file. It's not a perfect check, but hopefully it reminds reviewers/authors that there is some plumbing to add for documentation
Right now, the individual pages just show the diff and nothing else. We will add some context to these in the future

Codecov Report

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

@@           Coverage Diff           @@
##           master     #313   +/-   ##
=======================================
  Coverage   79.33%   79.33%
=======================================
  Files          34       34
  Lines        2255     2255
=======================================
  Hits         1789     1789
  Misses        466      466

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 5493b91...7d5fdc3. Read the comment docs.

Thank you Daniel!


Update workflow_job webhook language

On September 20th, we at Github rolled out the --ephemeral flag to self-hosted runners, and went live with documentation for the workflow_job webhook. Since it's no longer a forthcoming feature, I took the liberty of changing the language in the README that described them as experimental, or forthcoming. They're a first-class citizen now!
Also:

I deleted the warning in the Autoscaling section because the team shipped a fix to enterprise job assignment in August which I believe addresses this issue completely.

Changed wording about workflow_job webhooks that refers to them as unfinished, since they're live and documented now! (Except for GHES: they won't fully land there until 3.3, which is mentioned.)

I hope this is helpful! (And as always, thanks for all your work maintaining this great project!)

Ty for PR, there were additional changes to make to the docs related to this release as well as quality of life changes, that's merged now. If you spot any mistakes / typoes please do raise a PR for fix them :D


Added integration tests for product eligibility

Attention
Sponsored Product Test PR must go in before this one can be merged, because I branched off of sponsored product tests.
Changes
Please include a summary of the changes completed.
Github Issue
Please link to the github issue.
Testing
Please describe the tests that you ran to verify your changes. If possible please also provide steps to reproduce and screenshots of test evidence.
These tests should be live, not unit/integration tests.
Reviewer Checklist
Reviewer: please make sure that this has been filled out before approving

[ ] Changes are appropriate for the ticket
[ ] Github Issue is linked to the PR and the PR title includes the ticket ID
[ ] Evidence of live testing has been provided by the PR author
[ ] Reviewer has done live verification/testing and evidence has been provided as a comment below

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

@wbmgaye - Please fix the conflicts.


MS Teams online meeting link is enabled by DEFAULT - shouldn't the default be disabled?

https://github.com/PlaceOS/calendar/blob/ab8babafca01954545694da5cc116108a7af4eb3/src/office365.cr#L3-L9
Currently the above means that placeos/calendar clients defined like this will always have the MS Teams link in ALL events, with no way to disable/enable it on per event basis AFAIK.
I believe a better default would be to have the MS Teams link DISABLED, and then the placeos/calendar create_event() method should have an optional param for allowing an OnlineMeetingLink to be added, and another optional param for specifying which onlinemeeting TYPE (should use default if none is specified).

Technically you can provide the conference type via the configuration
https://github.com/PlaceOS/calendar/blob/master/src/place_calendar.cr#L37
So we just need to update this driver, for instance and same with the Staff API to support the conference type param
https://github.com/PlaceOS/drivers/blob/master/drivers/place/calendar.cr#L41-L53
but yeah, currently no way to disable on a per-event level

fixed in 0ecd9dedeca9f4b57fdd645898e9d58b39953b4a


[Unity] Popping when playing audio source for the first time

When playing an audio source for the first time, there's a weird, loud and really annoying sound artifact.
It seems similar to https://github.com/ValveSoftware/steam-audio/issues/22, but it doesn't only occur on specific DSP buffer settings.
Instead it only occurs when playing an audio source for the first time. I tried several workarounds like "initializing" the sounds by playing them with low Listener volume, but since the effect even occurs when spatializing is turned off for a specific audio source, the only 'solution' that works is to deactivate the Steam Audio Spatializer completely...

@Luux - let us know if this is still an issue. Get the latest version of Steam Audio Unity integration here.

@Luux I'm closing this issue out; let us know if you continue to face this issue in the latest release of Steam Audio, and we'll reopen it.


Workspace Symbols not including packages in vendor directory

OS: Windows 10
VS Code: 1.15.1 x64
I have a Laravel 5.5 app, and when I hit CTRL+T for Workspace Symbol search it doesn't include any files from the vendor directory except Composers own autoloader-files.
Here's some info from the Output->PHP Language Server tab
[Info  - 22:55:32] 74 Packages
[Info  - 22:55:32] Indexing laravel/framework for definitions and static
..........
[Warn - 22:56:05] Could not compute cache key for laravel/framework
[Info  - 22:56:05] Restored nesbot/carbon:1.22.1 from cache
..........
[Info  - 22:56:07] All 4962 PHP files parsed in 36 seconds. 282 MiB allocated.

Here's without any search input. As you can see it loads my own files (in ./app and root directory) just fine.

Here's when I search for Laravels Illuminate packages.

Here's when I search for Carbon.

Here's when I search for Composer.

This is by design. workspace/symbol is designed to return the symbols defined in your own workspace. It would also be very slow to return all matches from dependencies.

This is by design. workspace/symbol is designed to return the symbols defined in your own workspace. It would also be very slow to return all matches from dependencies.

That's too bad 😕 Sublime handles this really well with no performance issues, would be awesome to have support for it in Code!
Thanks for all your hard work on these packages regardless, incredibly helpful stuff 👍

excuse me, but my project.
If "new Class()" inside one file it working good, but if  "new Class()" in other file after include it not working definition.


[feature request] Slot filter with Pincode

It would be nice if you can make available this Pincode filter and users can enable notifications for any Pincode they prefer.

This feature has been added in the latest release.
Thanks for the feedback & suggestion! 👍


rx-nostr adapter

https://github.com/penpenpng/rx-nostr

this feature is needed: https://github.com/penpenpng/rx-nostr/issues/37


Fixed missing javah in openJDK 10+

This is a change that fixes issue of missing javah in openJDK 10+
I know that it is not compatible with earlier Java versions.

When might this (and perhaps other changes for newer Java distributions) be merged into master?  Trying to build tinyb on Raspbian Buster with OpenJDK 11 fails due to Java issues.

@d-f-clark
It seems that there aren't any changes in this project anymore. I suggest you to fork this project and fix the issues / applying PR in your fork.
BTW: Actually I also wanted to implement my bluetooth projects with Java, e.g. MipowPlaybulbTinyb but realized that it has not the expected stability. May be you take an alternative like bluez-dbus into account. However, I haven't validated this alternative yet.

Thanks for the response.  I actually used tinyb in a project back in 2017 and it worked fairly well, with some help from Petre at the time.  Had hoped to use it again on a Raspberry Pi Zero project, but build issues have gotten in the way.
I do find it odd that in this day of IoT BlueZ is so far behind the BLE curve.

This worked for me :)


feat: UI can allow relative URL links (#13479)

Added the ability for users to supply relative links within the UI for the links feature so that buttons and links do not have to have the absolute URL baked into each link. This will allow for easily adding buttons for different filters on the workflow list page for example. fixes #13479 ref #10447

@jessesuen this should address your ask in #10447


build: update glslang known-good to fix cmake compatibility

Change-Id: Ie3f955af4c3e981c4931d8acd0fef000bca862ac

CI Vulkan-ExtensionLayer build queued with queue ID 3241.

CI Vulkan-ExtensionLayer build # 11 failed.

Manual CI jobs 86, 88 and 89 show that the failing builds in the webhook CI tests aren't due to this change.


Potentially rename terminology in declarative schema definitions

TableType --> ?
Model --> Class
Attribute --> ClassAttribute

Renaming plan

Schema

Model --> Class

Table metadata (also simplifies the key names and aligns it with the comparatively simpler keys Description and Date)

ModelId --> Id
ModelName --> Name
TableType --> Type

Renaming implemented in obj_tables.
Todo: update dependent repos.

Done


"Smeared" surface at the bottom when scrolling down

When I scroll down in my game there's a smear effect at the bottom of window.

There should be a black line at the bottom of the wall, as you can see some pixels above.

Here you can see the effect even better.
It wouldn't be that bad if I use pixel perfect scrolling, but my character moves in a smooth grid, so it's very visible while playing.
Any way to prevent this smear effect? I probably could overlay it with my GUI, but I'd rather have the effect gone.

seems and sounds like a bug, will fix it when i can

https://github.com/jack27121/STANNcam/releases/tag/v2.3.1 now released @oliverhermanni


Modify tests and instructions to accept span for icon tag

Pre-Submission Checklist

[X] Your pull request targets the staging branch of FreeCodeCamp.
[X] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[X] You have only one commit (if not, squash them into one commit).
[X] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[X] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[X] Tested changes locally.
[X] Closes currently open issue (replace XXXX with an issue no): Closes #9464

Description

Added tests to accept span tags in addition to i tags on Font Awesome challenges.
Added brief explanation that span tags are supported for use with icons.
Added test to accept i tags for emphasis on jQuery .html() challenge.
Added brief explanation to jQuery .html() challenge to explain that while traditionally i tags indicated emphasis, the em tag is now widely accepted.

LGTM, any regex experts want to give this a look?

@dhcodes thanks.


README and information on how to gather logs and errors

There is no information on how to gather logs or to trace errors.
To gather some further information about the DRM segfault I'm getting on my aarch64 devices, I've built the drm backend with an additional parameter so that lldb and gdb can provide more detailed feedback:
cmake -DUSER_PROJECT_PATH=examples/flutter-drm-backend -DCMAKE_BUILD_TYPE=Debug ..
What would be the preferred method of capturing/tracing errors with the embedder?

Thank you for your report. I'll update README.

I've added the debug mode. You can use cmake -DUSER_PROJECT_PATH=examples/flutter-drm-backend -DCMAKE_BUILD_TYPE=Debug .. to get gather logs and so on.

Checking DEBUGGING.md.
@HidenoriMatsubayashi , please correct me if I'm wrong.
For the Building with debug mode section - when building the embedder with -DCMAKE_BUILD_TYPE=Debug this would generate the debug symbols for the embedder itself, right? It won't have anything to do with the Flutter app build type. Probably it would be better to separate this section.
Note:
To further double-check my understanding, I've already tried and I was able to open the Dart debugger when the embedder has been built without specifying the debug flag. Debugging the Flutter app went as normal.
On the other hand, inserting break points inside the embedder itself or analysing an unexpected behaviour, can produce more information through gdb / lldb if the embedder is built with DCMAKE_BUILD_TYPE.
For debugging the Flutter app and the Command line, maybe it will be good to add the note that the command has to be executed from the root folder of the Flutter project? ie:
$ cd <path_to_flutter_project>/
$ flutter attach --device-id=flutter-tester --debug-uri http://<IP_ADDRESS>:40409/k8IUol2dnPI=/

when building the embedder with -DCMAKE_BUILD_TYPE=Debug this would generate the debug symbols for the embedder itself, right?

You are right.

maybe it will be good to add the note that the command has to be executed from the root folder of the Flutter project?

Exactly.
I'll update READMEs again.

Looks good! Thank you, @HidenoriMatsubayashi , @Mai-Matsuura !
Will close this ticket now.


Update passMoodleRelogin.js

I like your rewrite of the passSience.js, however I don't quite understand what this rewrite fixes or improves, could you please elaborate your thought process quickly?
Thank you

okay here are some of the improvements

Instead of directly modifying the innerHTML of the document.body, the
code now creates and appends elements to improve performance and avoid
potential security risks.
The code now uses optional chaining (?.) when accessing the accessButton
element to prevent errors if it doesn't exist.

Thank you
On Mon, Jun 26, 2023 at 9:37 AM Jonathan Costa @.***>
wrote:

I like your rewrite of the passSience.js, however I don't quite understand
what this rewrite fixes or improves, could you please elaborate your
thought process quickly?
Thank you
—
Reply to this email directly, view it on GitHub
https://github.com/costa-jonathan/passepartum-chrome/pull/4#issuecomment-1606748015,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXVOHKDZAZG5S4BQFC4RIN3XNEU4HANCNFSM6AAAAAAZTV4BVM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thank you for your comment. However from my understanding the code does not actually modify any innerHTML and also doesn't append any new element, it just creades a variable, which is not needed here.
Additionally, why exactly do you think the event listener is necessary?
Best,
Jonathan


Add support for Expect scripts

Closes #6613
Description
Expect is an extension to the Tcl scripting language, so it can be categorized as Tcl. See #6613
Checklist:

[x] I am adding a new extension to a language.

[x] The new extension is used in hundreds of repositories on GitHub.com

Search results for each extension:

https://github.com/search?type=code&q=NOT+is%3Afork+path%3A*.exp+proc

[x] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

https://github.com/gcc-mirror/gcc/blob/master/gcc/testsuite/lib/compat.exp

Sample license(s):

GPLv3+

[x] I have included a change to the heuristics to distinguish my language from others using the same extension.

OK I think this is ready for review now; I don't think a new heuristic is necessary, since I don't see anything else using the .exp extension that would need to be distinguished from.

I don't think a new heuristic is necessary, since I don't see anything else using the .exp extension that would need to be distinguished from.

I just remembered that .exp is sometimes used in test-suites to contain the expected output of a test run. It's usually paired with a file ending in .act (to denote the actual output from a test), which is compared with the .exp file using the diff(1) utility.
There also appear to be roughly ~203k search results for the .exp extension that doesn't include lines beginning with set or expect. They don't appear to be fixtures for a test-suite either, so this might warrant a more in-depth investation.

Hm, so maybe just a really simple heuristic then? e.g. '^\s*(?:set|expect|proc)\s' or something... (I'm not very good at regexes, so please correct me if I got it wrong)

Hm, so maybe just a really simple heuristic then?

Actually, I was thinking .exp might need to be registered as a generic extension. This is a more esoteric feature of Linguist that's called upon when an extension has widespread usage in unrelated contexts, but none of them have enough adoption individually to merit their own entries in languages.yml. The original use-case was to support Solidity's .sol extension without misclassifying the numerous repositories that used .sol as an abbreviated suffix for "solution".
However, generic extensions have the caveat that an accurate, airtight heuristic is needed, preferably one that matches a pattern or string that's all but guaranteed to appear in the files that we're interested in supporting. The regex ^\s*(?:set|expect|proc)\s will match any line that happens to begin with the word "set", "expect" or "proc", followed by a whitespace character (including a newline). Needless to say, that's very, very error-prone; even if we weren't looking to add a generic extension, I still wouldn't recommend such an open-ended heuristic.
I'm happy to help you with penning the regex, but that assumes there even is a predictable and guaranteed pattern unique to Expect scripts.


Asset 362: wrong answer ID for ACE inhibitors

Asset 362 says the expected output name is ACE inhibitors. However, the output ID PUBCHEM.COMPOUND:37056 is Ethyl4-ethoxy-6-methylnicotinate (Dev NodeNorm, PubChem).
Instead, BTE does return an Angiotensin-Converting Enzyme Inhibitors (UMLS:C0003015), although not in the top 10%.
If this test is kept, I imagine the expected output ID needs to be changed.
But I wonder if this test falls under the "too general" issue brought up in https://github.com/NCATSTranslator/Tests/issues/64.

I'm not the best person to comment on generality, but Name Resolver returns UMLS:C3653768.

Wikipedia includes the ID MESH:D000806. This is in NodeNorm Dev too - but its primary ID CHEBI:35457 appears to be a role, not really an entity...

I'm not sure who the original asset owner is.

I am not the owner of this test but Angiotensin-converting-enzyme inhibitors (ACE) inhibitors is a whole class of compounds. The user expect the tool to answer a class of compounds which is not the question asked and the Translator response should be a given compound. Once getting the results, the user can select certain class of compounds so their need will be satisfied through UI interaction. For me this result is out of scope for this MVP.


[eu] Update Basque translation

eskerrik asko

Kaixo,
can you help us to translate the json file ?
You have the english version in script/en/en.json. And you create an eu.json in json directory.
Thanks in advance

Sure! I will try to get it done this weekend.


Petrakeas/cli push options

This is how strings are displayed in verbose mode. In the second screenshots, tags have been appended.


Fixes netcdf example

Fixes #499

Looks good! Wonder if there's a way to automatically close NetCDF files like with a destructor or finalizer though.

Would this work?
Base.close(ow:JLD2OutputWriter) = nothing
Base.close(ow:NetCDFOutputWriter) = close(ow.dataset)

for ow in model.output_writers
    finalize(ow, close)
end


Remove all limitations when lobby got empty

title tells all

To clarify, this is related to things like rating limitations.

To clarify, this is related to things like rating limitations.

Yes, the story was:
Joining an empty battleroom and can't get player because min rating was higher than mine.


DO NOT MERGE! Fast Forward stable to 1bb187ed8285409793b2dc9509715be2a8230eb0.

This PR will be merged automatically once checks are successful.

This change is

Jenkins job mixer/postsubmit passed

Jenkins job mixer/e2e-suite passed


wiki: npm global install requires sudo on *nix

The instructions for installing dependencies should make use of sudo for npm install as global packages on Linux and Mac.
diff --git a/How-to-Contribute.md b/How-to-Contribute.md
index ab42f72..42d4228 100644
--- a/How-to-Contribute.md
+++ b/How-to-Contribute.md
@@ -32,7 +32,7 @@ After you have these tools installed, run the following commands to check out Co
 **OS X**

        git clone https://github.com/microsoft/vscode
-       cd vscode && npm install -g mocha gulp
+       cd vscode && sudo npm install -g mocha gulp
        ./scripts/npm.sh install

 **Windows**
@@ -45,7 +45,7 @@ After you have these tools installed, run the following commands to check out Co
 **Linux**

        git clone https://github.com/microsoft/vscode
-       cd vscode && npm install -g mocha gulp
+       cd vscode && sudo npm install -g mocha gulp
        ./scripts/npm.sh install --arch=x64
        # for 32bit Linux
        #./scripts/npm.sh install --arch=ia32
@@ -120,4 +120,4 @@ We're also interested in your feedback for the future of Code. You can submit a

 ## Discussion Etiquette

-In order to keep the conversation clear and transparent, please limit discussion to English and keep things on topic with the issue. Be considerate to others and try to be courteous and professional at all times.
\ No newline at end of file
+In order to keep the conversation clear and transparent, please limit discussion to English and keep things on topic with the issue. Be considerate to others and try to be courteous and professional at all times.

That's a setup problem. Please refer to this guide: http://www.johnpapa.net/how-to-use-npm-global-without-sudo-on-osx/

We could add a link to that info on the instructions, though.

Feel free to make a PR

Couldn't figure out how to make a PR against the Wiki. I edited it directly, I hope that's fine?

I use nvm on Linux and sudo is not required. It's not a certainty that sudo is required and I feel like someone who needs sudo for a global npm install already knows that.

@mackenza Agreed. I'm always suspicious of wikis, tutorials, etc that put sudo in sample code. Copying and pasting commands into the console without double checking might turn a good day into a bad one.

I think this can be closed, the wiki was modified to add this line:

Note: For more information on how to install NPM modules globally on UNIX systems without resorting to sudo, refer to this guide .


Angular 12 upgradation issue related to css-what version inside package  @angular-devkit/builder-angular

🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑
Please read https://angular.io/guide/security#report-issues on how to disclose security related issues.
🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑🛑
I am on the way to upgrade my Angular project. I have followed all the steps mentioned here https://update.angular.io/. After that am able to resolve all the issues except one high related to @angular-devkit/builder-angular.
This is issue
High            Denial of Service
Package         css-what
Patched in      >=5.0.1
Dependency of   @angular-devkit/build-angular [dev]
Path            @angular-devkit/build-angular > css-minimizer-webpack-plugin
> cssnano > cssnano-preset-default > postcss-svgo > svgo >
css-select > css-what
found 3 vulnerabilities (2 moderate, 1 high) in 1651 scanned packages
3 vulnerabilities require manual review. See the full report for details.
I have tried updating using all the version @angular-devkit/builder-angular available, but no luck. You can see some of the version which i have tried below.
--<EMAIL_ADDRESS> --<EMAIL_ADDRESS>--<EMAIL_ADDRESS>     --<EMAIL_ADDRESS>--<EMAIL_ADDRESS>         --<EMAIL_ADDRESS>--<EMAIL_ADDRESS>             --<EMAIL_ADDRESS>--<EMAIL_ADDRESS> --<EMAIL_ADDRESS>--<EMAIL_ADDRESS>     --<EMAIL_ADDRESS>--<EMAIL_ADDRESS>         --<EMAIL_ADDRESS>--<EMAIL_ADDRESS>             --<EMAIL_ADDRESS>--<EMAIL_ADDRESS> --<EMAIL_ADDRESS>--<EMAIL_ADDRESS>     --<EMAIL_ADDRESS>--<EMAIL_ADDRESS>         --<EMAIL_ADDRESS>--<EMAIL_ADDRESS>             --<EMAIL_ADDRESS>In above @angular-devkit/build-angular is excepting css-what version should be >= 5.0.1 which I am not able to find.
Which version of @angular-devkit/builder-angular@? i need to install so that css-what >= 5.0.1 . Then only that high get resolve.
FYI
Node version - v12.18.2
npm - 6.14.5
"dependencies": {
"@angular-devkit/core": "^12.0.2",
"@angular/animations": "^12.0.2",
"@angular/common": "^12.0.2",
"@angular/compiler": "^12.0.2",
"@angular/core": "^12.0.2",
"@angular/forms": "^12.0.2",
"@angular/localize": "^12.0.2",
"@angular/platform-browser": "^12.0.2",
"@angular/platform-browser-dynamic": "^12.0.2",
"@angular/platform-server": "^12.0.2",
"@angular/router": "^12.0.2",
"@angular/service-worker": "^12.0.2",
"@fortawesome/fontawesome-free": "^5.15.3",
"@ng-bootstrap/ng-bootstrap": "^9.1.2",
"@ng-select/ng-select": "^5.1.0",
"@ngrx/store-devtools": "^6.1.0",
"ag-grid": "^18.1.2",
"ag-grid-angular": "^18.1.0",
"ag-grid-community": "^19.0.0",
"angular2-text-mask": "^9.0.0",
"autoprefixer": "^10.2.6",
"classlist.js": "^1.1.20150312",
"compass-mixins": "^0.12.10",
"core-js": "^2.6.12",
"jquery": "^3.6.0",
"moment": "^2.29.1",
"ng6-toastr": "^6.0.0",
"ngx-bootstrap": "^6.2.0",
"ngx-moment": "^3.2.0",
"ngx-spinner": "^6.1.2",
"ngx-toastr": "^14.0.0",
"postcss-scss": "^3.0.5",
"rxjs": "^6.6.7",
"rxjs-compat": "^6.6.7",
"zone.js": "^0.11.4"
},
"devDependencies": {
"@angular-devkit/build-angular": "^12.1.0-next.6",
"@angular/cli": "^12.0.2",
"@angular/compiler-cli": "^12.0.2",
"@angular/language-service": "^12.0.2",
"@ngrx/store": "^12.1.0",
"@types/jasmine": "^3.7.6",
"@types/jasminewd2": "^2.0.9",
"@types/node": "^15.6.1",
"codelyzer": "^6.0.2",
"jasmine-core": "^3.7.1",
"jasmine-spec-reporter": "^7.0.0",
"jspm": "^0.16.53",
"karma": "^6.3.3",
"karma-chrome-launcher": "^3.1.0",
"karma-coverage-istanbul-reporter": "^3.0.3",
"karma-jasmine": "^4.0.1",
"karma-jasmine-html-reporter": "^1.6.0",
"protractor": "^7.0.0",
"ts-node": "^8.3.0",
"tslint": "^6.1.0",
"typescript": "^4.2.4"
},

This is not actionable from our end.
Please follow https://github.com/svg/svgo/issues/1488.

Hi @alan-agius4  this seems to be resolved now on svgo's side, the relevant bug is now on cssnano (already merged on master and only needs tagging), relevant information is here
I hope you can update your dependencies when it's tagged.
Regards

@aseques Based on the existing SemVer range for svgo in cssnano of ^2.3.0, the corrected version should already be available for use.  A newly generated 12.1.0 CLI project, for instance, no longer contains the audit vulnerability.  Existing projects should be able to use npm audit fix.  If that does not solve the issue, removing both the node_modules directory and the package lock file followed by a fresh install, should resolve the problem.


mipsel_24kc support (Onion omega2, others)

This actually runs on the onion omega2 by manually extracting the ramips_24kec.ipk and putting the files onto the filesystem manually.
This package doesn't auto install because ramips_24kec is missing as an architecture on the Onion Omega2.
Would it make sense to build for mipsel_24kc as well as ramips_24kec?  It's the same toolchain triplet, just different ipk architecture identifiers.

Closing per #339


Solver fixes

In addition to what was done by @MLanghof in #266:

Disabled check- and cross-tagging of ascendancy nodes.
Fixed Skill Tree Generator crashing on window opening.
Renamed crit multi pseudo attributes to the new naming scheme.

Bugs I know of but don't have time to fix atm (probably won't get to them over the next few days):

Ascendant additional start nodes are not used for the solver (even if the "Path of the ..." node is skilled). This is because the connections between the start nodes and the Ascendant nodes are not bidirectional so the start nodes are thrown out of the search space because they don't have the "Path of the ..." node as a neighbor.
Ascendancy nodes count to the node limit in the Solvers.

So pull this, close the other?
As for the start nodes not being bi-directional, I didn't know of a better way to prevent the distance algorithm to ignore the ascendancy tree.

MLanghof's commits from the other are included here. So, yes, I guess.
Yea, it's ok. The solver was just not setup for directed edges. It needs more time than I have atm but it shouldn't be too bad. And it's not a big problem anyway, one only needs to skill the actual start node and it will work.

Alright, cool. Thanks for helping with this.

Oh, that is weird haha it merged both requests. That is pretty cool.


eslint 1.2.* breaks lint

no-duplicate-case:24:26 - infinite recursion

Related to eslint/eslint#3440

eslint/eslint#3440 is fixed and merged to 1.2.1 of Eslint. Now we can test.


undefined: C in C.PCAP_NETMASK_UNKNOWN

I'm seeing the below error at travis-ci once it tries to build myLG app w/ latest Go version but w/ Go version 1.7 it's fine.
go version devel +7ea5829
The command "eval go get -t -v ./..." failed.  github.com/google/gopacket/pcap ../../google/gopacket/pcap/pcap.go:594: undefined: C in C.PCAP_NETMASK_UNKNOWN

Hmm, we dealt with PCAP_NETMASK_UNKNOWN before here:   https://github.com/google/gopacket/issues/14
Maybe this is a regression of some sort?

Actually, that's saying that "C" itself is undefined, which is REALLY weird... there's definitely a big 'import "C"' at the top of that file.

I have yet to confirm this, but if confirmed, I'd say a post to golang-nuts is probably the next logical step.  If you want to short-circuit things, feel free to do that right away.

I tried to post there but as I haven't posted there maybe it takes 24 hours to show up.

Yes, it seems to be a regression in Go itself. Finding the faulty commit now.

Regression by commit a16954b8a7d66169760fb60dd7f3d4e400a5e98c. Not sure what a simpler reproduction is, but I have an idea...

Nope, can't figure out what's causing it to miscompile, sorry. But we can now report it to the Go team.

And it's fixed: https://github.com/golang/go/commit/b7c7949817e5bee801dfb333f483a080fa1f29e7

Confirmed, Thanks!

Also fwiw yes your original message did show up on the mailing list, but I guess we all just decided to silently ignore it.

Have you reported, or would you like me to?  Either way works fine for me.
On Fri, Oct 21, 2016 at 8:05 AM, Pietro Gagliardi<EMAIL_ADDRESS>wrote:

Nope, can't figure out what's causing it to miscompile, sorry. But we can
now report it to the Go team.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/gopacket/issues/244#issuecomment-255386104,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJ9ttFBl_2LSxbvCJXadV_BsQ1_aspRks5q2MbDgaJpZM4KctJh
.

A bit late, @gconnell; everything's been fixed now.

That's weird, I sent that email 4 days ago :-/


need some conversions

string -> stringarray[1]
double -> doublearray[1]
etc.

Create new object, add copy constructor, add test to see that object is reclaimed when no longer needed.

Did this a different way. For a scalar VRDatum type f, such as an int or string, you can do either:
VRInt d = f.getValue()
or
VRIntArray d = g.getValue()
in which case you get an array of length 1.


[TS React Apollo] Variables should be optional for Query components

Describe the bug
The generated types for React Apollo <Query> components explicitly override the "variables" prop and makes it required:
export type NavHierarchyComponentProps = Omit<
  Omit<ReactApollo.QueryProps<NavHierarchyQuery, NavHierarchyQueryVariables>, 'query'>,
  'variables'
> & { variables: NavHierarchyQueryVariables }

It looks as though this was done intentionally in https://github.com/dotansimha/graphql-code-generator/pull/1687, and then undone for mutations in https://github.com/dotansimha/graphql-code-generator/issues/1741, so I'm probably missing some context as to why that was desirable.
Our use-case is that we use the skip prop to conditionally execute the query only once the variables are available. Something like this:
<NavHierarchyComponent
  variables={someContext ? { foo: someContext.bar } : undefined}
  skip={someContext === undefined}
/>

This fails with the current typings since it expects variables to be explicitly non-optional.
Expected behavior
Either use the built-in React Apollo types for variables, or add some kind of clever typing that allows the variables to be optional if the skip prop is provided. I believe simply using the typings from react-apollo would solve my use-case, but I'm sure there is a reason it was overridden in the first place.
Edit: It looks as though this was done since the react-apollo typings were wrapping the variables in Partial, meaning the types were not as restrictive as they could be (https://github.com/dotansimha/graphql-code-generator/issues/1626).
In my opinion it's best to lean on the Apollo typings as much as possible, so I would suggest removing the override (Omit + & { variables ... }) altogether. If they wrap the types in Partial (incorrectly?), that should be dealt with on their end. Anyway, as far as I can tell, this is no longer the case. These are the typings from the latest version of react-apollo:
export interface QueryOpts<TGraphQLVariables = OperationVariables> {
    variables?: TGraphQLVariables;
}

...Which looks correct to me. Variables can be omitted altogether, but if they are present they must exactly match the typings.

If at least one of the variables is required, variables property would be required; because this means that query is unable to work without variables. So, codegen generates the component of that query with a required variables property.

Thanks @ardatan, the issue is related to the skip prop which tells react-apollo to not execute the query at all. In that case it's valid to not pass variables.

@ardatan I think the variables should be the same as they declared in React-Apollo. if they are optional there, let's do the same

I discussed it with @ardatan , we should allow variables to be VarType | null and it will make it easier to use, but still force variables to be there when it's needed.
What do you think @cecchi ?

@dotansimha @ardatan, sure, that works -- I'm not positive what assumptions react-apollo makes under the hood but if null is equivalent to undefined then that suits my use-case.

I created a PR to make variables optional if skip: true is provided instead of adding | null; because it didn't look good to me.
@dotansimha What do you say?

Available in 1.2.1


Add printf and use it instead of com_ functions where appropriate.

Also stop pagefault-looping if we get a pagefault before running_tcb is set to a sane value.

Two higher level design comments:

It's not clear to me that this should be called printf - the puts and putc functions output to a different output (the VGA screen) than the printf function does, which strikes me as wholly confusing. I think a good standard would be to not overload standard c function names in the kernel that don't do the same thing (e.g. memcpy and memcopy are fine). The naming issue should be fixed before merging.

IMO varargs are bad, and should not be used in the kernel. The ABI for them is complex and bypasses the C type system entirely. It's a recipe for creating lots of bugs and could be nipped in the butt now.

Also, it would seem like a good idea to well-document the subset of printf formatting that is (and is not!!) supported by this implementation.

This should be more mergeable now. Implementation is cleaner, has unit tests.

Looks pretty good.  Tiny comments only


Make Upload Image button optional

Hi. Thanks for this addon, we use it regularly.
Could you please make the new Upload Image button optional, though. I don't need it and I keep pressing it when I want to press the chat trashcan instead.

Thanks. :-)


Why are sync/async errors/warnings being cleared on fields with registration count > 0?

I'm currently working on a form where I am doubling up on some fields, for example node.properties.text. I am attempting to delete the duplicated field called node.properties.text and move it to node.properties.original_text
In the current code (https://github.com/erikras/redux-form/blob/master/src/createReducer.js#L418) there is
      if (count <= 0 && destroyOnUnmount) {
        // Note: Cannot use deleteWithCleanUp here because of the flat nature of registeredFields
        result = deleteIn(result, key)
        if (deepEqual(getIn(result, 'registeredFields'), empty)) {
          result = deleteIn(result, 'registeredFields')
        }
      } else {
        field = setIn(field, 'count', count)
        result = setIn(result, key, field)
      }
      if (destroyOnUnmount) {
        result = deleteInWithCleanUp(result, `syncErrors.${name}`)
        result = deleteInWithCleanUp(result, `submitErrors.${name}`)
        result = deleteInWithCleanUp(result, `asyncErrors.${name}`)
        result = deleteInWithCleanUp(result, `syncWarnings.${name}`)
      }
      return result

What is the intention on leaving count <= 0 out of the second if condition? If the field is not being deleted surely any current error conditions are still valid?
This is an issue for me currently because (and I'm not filing this as my bug because I suspect I'm using the library incorrectly) result = deleteInWithCleanUp(result, syncErrors.${name}) is returning Uncaught TypeError: Invalid keyPath: Value at ["syncErrors"] does not have a .get() method: [object Object]. I am using the immutable library, and anything returning a validationError is at least returning an empty Map(), but getIn from https://github.com/erikras/redux-form/blob/master/src/deleteInWithCleanUp.js#L20 is failing looking for anything more nested than syncErrors, and getIn(result, "syncErrors") returns an object with no getIn function (an ArrayMapNode?), like in https://github.com/erikras/redux-form/issues/1269

The first half of what I'm saying is duplicated by https://github.com/erikras/redux-form/issues/2979

Because this issue saw almost no activity for a few months, I'm closing it.


Make runtime ssr ready

Sunmao runtime is going to support ssr. The popular ssr frameworks like remix and nextjs don't support esm packges. So this pr replace or remove these esm packages.
This pr makes sure that the runtime is ssr ready. However, arco-lib and chakra-ui-lib are not, because they have some other packages which does not support ssr well.
Changes

edit dependencies and peerDependencies
remove esm packages, like lodash-es...

how about the bundle size


Support passing reactions through the bridge

Some xmpp clients have started implementing reactions (dino, movim for example) and it'd be good to pass reactions through the bridge both ways.

this is https://xmpp.org/extensions/xep-0444.html

I tried implementing this and found out it's not possible. Since XEP-0444 uses a refreshing list of reactions per user, and Matrix uses a deletable one message per reaction approach, the two aren't compatible, and porting can't be done without being able to read other messages, which isn't available for Bifröst to do on either end.


Invalid reference in fabric.mod.json

In the fabric.mod.json file, fabriccontrolling.server.json is referenced for servers, but doesn't actually exist.

Fixed in version 1.0.3, now available on Curse


job-builder: remove toybox from the base job image

Why
We want to secure our base image by deleting the toybox because of its rich capabilities:
acpi base64 basename blkid blockdev bunzip2 bzcat cal cat catv chattr chgrp chmod chown chroot chvt cksum
cmp comm count cp cpio cut date devmem df dirname dmesg dos2unix du echo egrep eject env expand factor fallocate
false fgrep file find flock free freeramdisk fsfreeze fstype fsync ftpget ftpput grep groups gunzip gzip head
help hexedit hostname hwclock id ifconfig inotifyd insmod install ionice iorenice iotop kill killall killall5
link ln login logname losetup ls lsattr lsmod lspci lsusb makedevs md5sum microcom mix mkdir mkfifo mknod mkpasswd
mkswap mktemp modinfo mount mountpoint mv nbd-client nc netcat netstat nice nl nohup nproc nsenter od oneit
partprobe passwd paste patch pgrep pidof ping ping6 pivot_root pkill pmap printenv printf ps pwd pwdx readahead
readlink realpath renice rev rfkill rm rmdir rmmod sed seq setsid sha1sum sha224sum sha256sum sha384sum sha512sum
shred sleep sort split stat stty su swapoff swapon switch_root sync sysctl tac tail tar taskset tee test time
timeout top touch true truncate tty tunctl umount uname uniq unix2dos unlink uptime uudecode uuencode vconfig
vmstat w wc which who whoami xargs xxd yes zcat

What
Deleted toybox from the base job image.
Testing done
Locally against Kind cluster.
Signed-off-by: Miroslav Ivanov<EMAIL_ADDRESS>

How are the new base or job images being tested (in automated way) ? @mivanov1988
Our CICD tests with only  1 type of image ?
I think we need some Testing design here ... Have you planned something ?


I added get users endpoint files

Description
Endpoint to retrieve users - GET api/users
Fixes #31
How to test?
first run: npm run dev and then go to http://localhost:3000/api/documentation
Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] This PR is ready to be merged and not breaking any other functionality

Hello Nashmil, please rename the PR with a more descriptive name and a reference to a userstory.

Hello Nashmil, please rename the PR with a more descriptive name and a reference to a userstory.

Hello ditlev. I made changes. Hope it works fine now :)

I did resolve the conflict and now it's ready to be merged.


Fix QDac for qcodes v0.8.0: inconsistent values for vrange

This addresses an issues that came to light with the improvement of the cache in qcodes v0.8.0. The 'vrange' parameter was used inconsistently. (So was 'irange')
A lot of refactoring was necessary in order to find the bug, but it is a good time to do the refactoring since the new driver version needs to be tested on a real device anyway.
Ready for review, but not merging, since the hardware tests are missing.

This code has now been tested including the actual voltage output:
ch = qdac.ch41

def assert_equal(v):
    assert np.isclose(ch.v.cache(), v, rtol=1e-2)
    assert np.isclose(ch.v(), v, rtol=1e-2)
    assert np.isclose(ch.v(), dmm.volt(), rtol=1e-2)

ch.vrange(0)
ch.v(1)
assert_equal(1)
ch.vrange(1)
assert_equal(0.1)
ch.vrange(0)
assert_equal(1)
ch.vrange(0)
ch.v(0.5)
assert_equal(0.5)
ch.vrange(1)
assert_equal(0.05)


[Issue] Problem loading mod?

Hi! I want to use this mod but I would like to configure its features for my modpack. The thing is... the mod is not appearing in ModMenu o_o And also, inside config folder, the file is empty. I'm using MC 1.19.2 with Fabric 0.14.13.
Im sending you the config and the log to see if you can tell what could be the possible error. Thank you! : )
davespotioneering.json5
latest.log

Mod version?

fixed


update libgit2 to 0.26

It builds fine against libgit2-sys 0.6.12 with no changes required, I was just able to produce a cargo 0.20 Debian package from it with automatically-generated Depends: libgit2-26 (>= 0.26.0).

This has actually already been done, just hasn't been published yet, I'll do so.


Extract the current examples from the main project

Create an examples-package that will be required in composer.json and leave the project clean.

Problem 1
    - Installation request for laravel/horizon v1.3.1 -> satisfiable by laravel/horizon[v1.3.1].
    - laravel/horizon v1.3.1 requires ext-pcntl * -> the requested PHP extension pcntl is missing from your system.

  To enable extensions, verify that they are enabled in your .ini files:
    - C:\xampp\php\php.ini
  You can also run `php --ini` inside terminal to see which files are used by PHP in CLI mode.`

I got this error when i install composer composer install


Remove superfluous pragma directives

This PR removes superfluous pragma directives for DOTNETCORE and tidies up some superfluous method calls and unused methods.
With the move to NetStandard 2.0, many of the directives that were needed for NetStandard 1.3 are no longer required, because the API surface in NetStandard 2.0 is much larger.

porting to 7.x and master


Service Discovery API refactoring

Our requirements on Service Discovery have evolved since we first created it, eg we added more semantics to the input string. There were also some vestigial types, members, and patterns to clean up. API review also revealed some things which could be simplified or changed to align with prevailing patterns.
This PR addresses these points by refactoring the API:

Move types in the *.Abstractions assembly out of the .Abstractions namespace, and clean up namespaces in general
Hide types & members which do not need to be exposed. Delete unused types (eg, unused IServiceEndPointSelector impls).
Seal most types
Simplify the resolver API by removing the ResolutionStatus type
Rename IHttpClientBuilder extension from UseServiceDiscovery to AddServiceDiscovery to align with existing patterns. The existing method is kept and marked as [Obsolete] for now.
Add a ServiceEndPointQuery type to communicate the parsed input from the core resolver to the providers
Resolve IServiceEndPointSelector from DI instead of using a factory (and register it as transient).
Add missing overloads to hosting methods. Specifically, overloads which accept configuration delegates.
General code style cleanup (primary constructors, etc)
Rename ServiceEndPointCollectionSource to ServiceEndPointBuilder, make internal, and extract public interface: IServiceEndPointBuilder
Rename ServiceEndPointCollection to ServiceEndPointSource to match existing ASP.NET pattern

Microsoft Reviewers: Open in CodeFlow

We should merge this for this fact alone

/azp run


Update azure-stack-diagnostic-test.md

Made edits to the BackupInfra access check commands

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

#sign-off


[Expression] Add array return type

close: #3615
Add Array return type to make array type errors be discovered during the static check period.
For example, before :
var expr = Expression.Parse("skip(1, 1)"); // it's ok
var result = expr.TryEvaluate(null); // throw runtime exception, skip can not applied on int

After:
var expr = Expression.Parse("skip(1, 1)"); // 1 is not a valid array

remark
why does Expression.Parse("skip(1, 1)") not throw exception before.
If the method accepts Object type, e.g. count method accepts one parameter, which could be string or array. If the param is a string, return string length, and if the param is an array, return the array size.
So, we can only mark the param object, any return type here is ok during the static checker period.
Same logic with skip(1, 1), skip accepts an object as the first parameters, and all the input types here are treated as valid types in the static checker.
The root cause is that we treat Object as the base type (or any type). So, extract Array type may help us get array related errors in advance.
Additional feature:
Support mixed return type in the static check.
Before, if the expression needs one param that maybe a string or a number. we can only mark it as Object return type, but now, we can use ReturnType.Number | ReturnType.String to mark it.
This can make type matching more precise.

Looks good to me, but i can't tell all cases are covered, can you add test cases in both "Good" and "Bad" expression tests, to make things clear.

public enum ReturnType

Should tag with flags, i.e. https://docs.microsoft.com/en-us/dotnet/api/system.flagsattribute?view=netframework-4.8

Refers to: libraries/AdaptiveExpressions/Expression.cs:21 in 66e3e1e. [](commit_id = 66e3e1ef6ade3a4f67b457daf26814442416519c, deletion_comment = False)

Indentation.

Refers to: libraries/AdaptiveExpressions/ExpressionFunctions.cs:3049 in 66e3e1e. [](commit_id = 66e3e1ef6ade3a4f67b457daf26814442416519c, deletion_comment = False)