Why can't I automatically hide the search bar? When I clicked on a search result.

From @DebugLyu on March 23, 2018 2:24
vscode 一切都好，就是这个搜索栏非常 反感，不像 sublime也行 跟 notepad++ 显示在下面 按两下esc关闭 也行啊，这个 常驻左侧，很麻烦，也不美观 还需要 ctrl + b隐藏掉，对我们这些两列编辑的 简直太难受了
Copied from original issue: Microsoft/vscode-docs#1523

I don't understand the problem. Please include some more details about what you want.

What @DebugLyu was saying is he doesn't like having the search bar on the left always (unlike Sublime which puts the search bar in the bottom) and he would expect auto hide after clicking on search results.
I think it's not valid anymore as we can have search viewlet in the panel.

@DebugLyu Please check out the search.location setting which might help. But I'm not expecting to autohide the sidebar or anything like that.


C# tutorials - serialization

Shows how to implement binary, XML, and SOAP serialization in C#.
Tutorials should include one or more code examples, each example should also show expected output and should include an explanation of the code where it isn't obvious. Examples shouldn't be dependent on a specific IDE or target platform.

Forget SOAP. Show JSON

:+1: let's not have SOAP be a part of this.

Is anybody working on this? I'm willing to do this if nobody else has already done it.

I'd love to see more people get involved in this project. Are you interested in contributing in general or is it this particular topic?

Definitely! I'm still learning C# but I'm planning on contributing as I go along. I know a good bit of Java so it shouldn't be too long.

Great. This article is a more in-depth one, so it probably isn't the best one to start with. I would recommend starting with "getting started" material and then growing from there into more complex/deep material.
We were talking about porting some "getting started" .NET Framework content to core. It both needs to be massaged into markdown and validated that it the content applies to Core. We're going to start that exercise soon, but could use some help. Interested in that?
Here's the content: https://msdn.microsoft.com/en-us/library/ms172157(v=vs.110).aspx

Rich - it sounds like there might be some bigger ideas/epics y'all want to tackle that aren't clearly reflected by the individual issues.  I'm interested in helping out.


Inverted include matcher never returns for infinite enumerables

Subject of the issue
I would like to write a spec where I assert that a value should not be included in an enumerable that is lazy and infinite. For example:
expect(fibonacci_sequence).not_to include(4)
# or
expect(1..).not_to include(0)

However, the rspec include matcher goes into an infinite loop and never returns for these specs, despite the fact that (1..).include?(0) does return false immediately (for fibonaccci_sequence it would depend on your implementation).
Your environment

Ruby version: 2.6.0
rspec-expectations version: 3.8.2

Steps to reproduce
I've created a simple test case on github here, which reproduces the issue using an infinite range. It's run on travis here
Expected behavior
If include? returns false, the expectation should fail.
Actual behavior
It goes into an infinite loop and the expectation never completes.
It looks like this is because the matcher calls any? after include? and that does never return.

but I'm unsure if the extra complication is worth it.

I don't think so

@iainbeeston You may use be_include predicate matcher that calls include? directly underneath and thus doesn't go into an endless loop.
include accepts an arbitrary number of arguments, while include? does not.
It's possible to define does_not_match? so that it passes some flag or a strategy to use for comparison of collections. However, this adds complexity.


use v-m-p for vert.x

cc @rafabene @yanaga @kameshsampath

I'm ok with these changes.

Funny thing is that I've used the generator on the vert.x website, and it uses the shade plugin instead of v-m-p.


fix: Change download and print button to be added only on first attach

Fixes #45

Fix is not working okay as download button is being shown twice. It happens because of change added in constructor. Removing that makes download button not to show correctly in simpler examples not using dialog. That's why I'm closing this in favor of https://github.com/vaadin-component-factory/vcf-pdf-viewer-flow/pull/47


Per Pixel Transparency black windows

Not sure what version , it's whatever is the current here > https://hugo.pro/projects/godot-builds/
Windows 10 1703, 1803 (Just updated it and still not working)
Ryzen 5 1600
Nvidia GTX 1060
For some reason I only get black windows, all the objects show up fine, but the transparency never seems to work even across the demos and such even though it works for everybody else

Try adding OS.window_per_pixel_transparency_enabled = true into _ready instead of setting display/window/per_pixel_transparency in Project Settings.

Nope still a black area where the alpha is ment to be

Got 2 other people to test using this, the build i compiled, they both get the black screens, so i'm assuming something possibly changed since the files were last messed with.
bin.zip

You can also run Godot from a console with godot.exe --version to get version information

the version linked in that file above is 3.1.dev.custom_build.1d84c4f

I have tested your project with both my (3.1.dev.custom_build.d603a74, debug 64-bit tools, 32-bit and 64-bit release export template) and your executable, transparency work correctly with all of them. Tested on Windows 10 (version 1803 build 17134.167).

This is very odd, i'm on the same version of windows as you and the issue still persists, do you have a discord? my user is Duskitty#4455 if you're on there

Done some additional testing (same Windows 10 version):

AMD FirePro D300, driver version 17.10.3101.1006 - OK.
Intel HD Graphics 530, driver version <IP_ADDRESS>49 - OK.
NVIDIA GTX 970M, driver version 398.36 - Black screen.

I can confirm it doesnt work on both 390.95 and 398.36 with a GTX 1060

Had somebody test it on an Intel computer vs a Nvidia one, this seems to be restricted to Nvidia cards so far

Tested on Linux with NVIDIA GTX 670MX 390.59 and it works fine.

@Duskitty please test #20245, works for me on GTX 970M.

Can confirm that this fixes it @bruvzg  @akien-mga


A little improvement of events for video analytics

I want to analyze the playback of each video, including which parts were played and which parts were skipped. But I found it a bit difficult.
The current situation is that when the user clicks on the progress bar, a 'pause' event is triggered, followed by a 'play' event. In other words, the following two cases trigger same events:

The user directly clicks the progress bar to jump to 10:00
The user normally plays to 10:00, pauses for a while, and then resumes playback.

Both of these situations trigger a pause event (at 10:00) and a play event(at 10:00).
Here's my suggestion and a small change can meet my needs:
When the user jumps from 5:00 to 10:00
Current events:                Pause (at 10:00) + Play (at 10:00)
My suggestion:    Pause (at 5:00) + Play (at 10:00)

Thanks I'll do that. I'm going to close this one 👍


Failed to delete scratch org after validation

Describe the bug
When a PR validation is failed, an error also occurs that is unable to delete the scratch org.
When PR validation is successful, the deletion of scratch org after the run is successful as well.
I have checked this on every PR starting yesterday as the unusual error started yesterday, I noticed that it only occurs when a PR validation is failed.
I have attached the full raw logs as well.
Failed PR Validation (failed to delete scratchorg after validation).txt
Successful PR Validation (successful deletion of scratchorg after validation).txt
To Reproduce
Steps to reproduce the behavior:

Run a PR validation that will success using the sfpowerscripts:orchestrator:validate
Run a PR validation that will fail using the sfpowerscripts:orchestrator:validate

Expected behavior
When a PR validation failed the deletion of scratch should not fail as well.
Screenshots
Failed PR validation:

Successful PR validation:

Platform Details (please complete the following information):

OS:
Version [e.g. CLI Version  eg: 1.6.6]: 10.2.13
Salesforce CLI(sfdx cli) Version:
CI Platform: Github

Additional context
Add any other context about the problem here.

@rubiatoEJ  It could be a erroneous log, Can you confirm whether the org is active in DevHub?

@azlam-abdulsalam  we are seeing the same thing, a scratch org taken from the pool cannot be deleted, seeing the same error here. Use latest SFDX CLI version

@rubiatoEJ It could be a erroneous log, Can you confirm whether the org is active in DevHub?

@azlam-abdulsalam I have checked a latest failed PR validation as the previous org has already expired.
this is the latest failed PR validation and failed to delete the scratch org

I have asked to check if the org is active in DevHub and it shows as active

Folks, this will need further investigation, as something with sfdx-cli 7.136.2 has changed. We advise all of you to rollback into release-dec21 docker image  or 7.129.0 until we figure out a work around.

@thvd @rubiatoEJ  do you have the same issue with orgs fetched from developer pool? Locally everything works. :(

@mshanemc any ideas here?

@thvd @rubiatoEJ do you have the same issue with orgs fetched from developer pool? Locally everything works. :(

@azlam-abdulsalam I have tried fetching from developer pool and delete with the same command used in validate

and it shows that it is successfully deleted.

@aly76  I cant reproduce this locally, so it happens in docker image only. Probably the auth file is written to a plugin space?

Updated the sfdx-cli to the latest version and tried to delete the org again but it is still successful

$ sfdx --version
sfdx-cli/7.137.1 win32-x64 node-v16.13.0


Orphanet evidence lacks variant functional consequences

Describe the bug
variantFunctionalConsequenceId is null for all Orphanet evidence, even though a fraction of the evidence reports whether the disease causing mutation results in a loss or gain of function.
Observed behaviour
The type of the association established in the Orphanet evidence is described in their raw data. With this, we can populate the variantFunctionalConsequenceId field for those which result in a loss or gain of function:
CONSEQUENCE_MAP = {
    "Disease-causing germline mutation(s) (loss of function) in": "SO_0002054",
    "Disease-causing germline mutation(s) in": None,
    "Modifying germline mutation in": None,
    "Disease-causing germline mutation(s) (gain of function) in": "SO_0002053"
}

This is already handled in the parser, but the field is not present in the final output just because variantFunctionalConsequenceId is missing from the final select statement. I'm fixing this in the 22.04 branch of the evidence data parsers.
Expected behaviour
This info will be part of the final evidence set. This is their frequency:
+------------------------------+-----+
|variantFunctionalConsequenceId|count|
+------------------------------+-----+
|                    SO_0002053|  210|
|                    SO_0002054| 1175|
|                          null| 4981|
+------------------------------+-----+

Questions
Since this was not in the data, we weren't displaying it in the FE. Given the % of missingness, is it something that we want to add? @d0choa

When available, this information is very relevant. I would aim to include it in the FE.

Fixed in #122


Add borderCurve style prop to ViewStyle types

Summary
This PR adds the missing style prop type borderCurve introduced in 8993ffc82e

Changelog
[GENERAL] [FIXED] - Added the missing borderCurve style prop to ViewStyle

Test Plan
IPR

Is this ready for review? Can you rebase your changes to remove conflicts?

Is this ready for review? Can you rebase your changes to remove conflicts?

@javache looks like it's conflicting with an identical change, so I'm gonna close this one.


Bug: v1.1 unable to manual select prompt info. WildCards prompt

In v1.0 i was able to manually select and copy prompt or part of it.
In v1.1 this is impossible, only way is copy full prompt by "copy" icon.
Also after generation parameter, we have more extra data for Wildcards prompt, but the copy icon not exist for it.

In v1.1 this is impossible, only way is copy full prompt by "copy" icon.

Sorry about this, a bug crept in that I had a hard time finding.  It seems to be style related, which can be a real headache to fix, but I'll get on it right away.

Also after generation parameter, we have more extra data for Wildcards prompt, but the copy icon not exist for it.

Can you clarify this?


added field for specifying mime type in web console

JIRA Ticket: https://fedora-repository.atlassian.net/browse/FCREPO-3741
What does this Pull Request do?

Adds a field for specifying mime type when adding/updating binaries
Fixes bug where content type was being set to an empty string

Ideally, the type would be pre-populated with the existing mime type on update, but I don't know how to do that.
How should this be tested?
Do the following for adding and updating binaries:

Open the console
Add/update a binary without a file extension and without a mime type, and see it default to application/octet-stream
Add/update a binary with a file extension and without a mime type and see it guess the mime type
Add/update a binary without a file extension and with a mime type and see it use the specified type

Interested parties
@fcrepo/committers

@bbpennel, should the placeholder text say something like "leave blank to autodetect"? If the file extension is the only thing being used for autodetection, we could even say "leave blank to autodetect from file extension" to be more clear about what's going on... or, if we want to leave things more open-ended, we could say "specifying a MIME type is highly recommended." But maybe I'm getting a little ambitious about how much text we can fit in the box... in that case, perhaps simply "autodetect" or "autodetect (unreliable)" might do the job.

@bbpennel I updated it to automatically set the mime type in the input box when the type is known

The implemented approach seems to work well for me, I didn't encounter any situations where the javascript couldn't come up with a mimetype while the browser could. I was testing in firefox and chrome on macOS big sur. The webconsole definitely has some weird caching behaviors, I had to do forced reloads in chrome for the updates to show up, but that's unrelated to this PR.
So I'm inclined to go ahead with the application/octet-stream placeholder text since it appears to be accurate in so far as what gets submitted by the browser.


the same case different answer

Thanks for your great work.
I try to use the same case in the tabular math word problem. the  picture name in the test is 28034.png. I set the temperature=0,

but we get different result.

It seems like the input text has some difference with the original input text, which may influence the output. Can you give another try with the following input text:
{
"prompt": """Based on this table about 'Children's weights (lbs)', solve the following problem. In the end, output your final answer using the JSON format: {"answer": "<YOUR ANSWER>"}.
As part of a statistics project, a math class weighed all the children who were willing to participate. How many children weighed exactly 31 pounds? (Unit: children)"""
}

When we manually draw the ppt for the case study in the paper, some special tokens like spaces and '\n' in the input text were omitted to save spaces. Thus the rendered input text is slightly different from the original ones. You can access the whole test samples MMTab-eval_test_data_49K_llava_jsonl_format.jsonl from the HuggingFace.
But I think your discovering confirms an interesting and important defect of MLLMs, i.e., sometimes it was vulnerable to the adversarial attack of the tiny disturbance in the input text or image.

ok, I try it again with your prompt.

I'm look forward to your releasing inference and evaluation code. I will check it again.


Move the dialog css stuff in its own file

In order to share common parts between different dialogs, this patch aims to slightly refactor the css in making it more generic. This way it'll simplify adding a new dialog (we want to add a new one when leaving an unsaved document).

/botio-linux preview

Perhaps, in a follow-up, we should also use these CSS rules for the other dialogs as well (e.g. PasswordPrompt and PDFDocumentProperties)?
r=me, thanks!

Yep I thought about that too. If you want to make the follow-up, please do else I'll do it later in the week.


Update Tphi to allow either Ramsey or Hahn echo experiments

Summary
Added the boolean option t2star that allows Tphi to use either T2 or T2* upon instantiation. By default, Tphi still uses the T2* from Ramsey so that the switchover is smooth for users.

Thanks—I was thinking it would be better to keep the default state the same so the user experience is the same, but if the original Tphi value wasn't very useful, then it makes sense to make the default T2Hahn.

@yaelbh Ramsey is not useful for superconducting qubits, but it could be relevant for other architectures, so I think we should leave it in as an option. The interface isn't more complex for the user if they use default options.


feat: ios modules

[x] Add iOS Native Modules

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Failed to bind LUIS prediction resource when publishing bot from composer desktop 2.0

Describe the bug
After creating a new publish profile with new resources, then publish the bot from Composer desktop 2.0, the publish fails with the error fail to bind LUIS prediction resource to LUIS applications.
Version

Browser
No browser, just composer desktop 2.0.

[x ] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[x ] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Have a bot with LUIS built in Composer 2.0 desktop.
Choose to publish it on new resources.
Create a new publish profile with new resources and wait for the provision step to finish.
Go to publish and choose the publish profile created earlier.
Press Publish and wait for a few seconds

Actual result: the publish fails with this error

NOTE: this node "luisResource": "luis prediction resource here" is correctly populated in the publishing profile.
Expected behavior
The expected result should be: the publish should be successful.
Screenshots

Additional context
If the user manually binds the prediction resources in the LUIS.ai portal, the error still happens.

FYI custsomers are hitting this and LUIS requested that we promote handling of this. (more info coming).

Thanks @scheyal
Working with @natalgar; we could not reproduce the issue.
Looking into who should be a suitable owner to look into this.

Hi folks,
When adding a prediction key to Composer, even during development, I will populate both the authoring and endpoint keys appropriately in the advanced settings.
See below

Having an endpoint key means you won't run out of quota by using the authoring key for endpoint requests. When this happens, you get 403s constantly. By providing the endpoint key we expect the prediction resource should be assigned to that application.
When checking the LUIS portal, no prediction resource is assigned.

@srinaath , @dmvtech  - please see @hazemelh details. Looks like a bug (with a delayed repro b/c first you have to reach the quota to hit the condition).

@benbrown can you please chime in on this one.  I know that we worked through much of this when we were working on the LUIS provisioning and publishing.  I can connect with the customer to see the setup if needed.

Composer does NOT currently bind  the endpoint key to the LUIS apps when starting the bot locally. So if you specify the endpoint key in the advanced settings, you must manually bind the same key via the LUIS portal.
However, during the PUBLISH, it definitely should do that binding.  In the publishing profile, are both the endpoint and authoring key specified?

yes, both authoring and prediction keys were specified when I repro-ed this.

Status update. We are investigating internally and will update when we have more findings.

I am having this issue as well. Thanks for update @dmvtech

Another customer raised a support case where he faces the issue. Today in my bot, I was not able to repro, but the customer was.
The log he shared is:

"Accepted for publishing..."

"Abs settings incomplete, skip linking bot with webapp ..."

"appPasswordHint incomplete, return ..."

"Building the bot app..."

"Build succeeded!"

"start publish luis"

"Assigning to luis app id: 06a7b182-8966-4528-b72a-46760ba91c84"

"Assigning to luis app id: a51afb52-80ac-4179-be61-fcd81bb9e00c"

"{\n  "stack": "Error: Request failed with status code 400\n    at createError (C:\\Users\\gec\\AppData\\Local\\Programs\\Bot Framework Composer\\resources\\app.asar.unpacked\\extensions\\azurePublish\\dist\\extension.js:102540:17)\n    at settle (C:\\Users\\gec\\AppData\\Local\\Programs\\Bot Framework Composer\\resources\\app.asar.unpacked\\extensions\\azurePublish\\dist\\extension.js:102554:14)\n    at IncomingMessage.handleStreamEnd (C:\\Users\\gec\\AppData\\Local\\Programs\\Bot Framework Composer\\resources\\app.asar.unpacked\\extensions\\azurePublish\\dist\\extension.js:103498:13)\n    at IncomingMessage.emit (events.js:215:7)\n    at endReadableNT (_stream_readable.js:1183:12)\n    at processTicksAndRejections (internal/process/task_queues.js:80:21)",\n  "message": "Request failed with status code 400",\n  "config": {}\n}"

"{"stack":"Error: Failed to bind luis prediction resource to luis applications. Please check if your luisResource is set to luis prediction service name in your publish profile.\n    at publishLuisToPrediction (C:\\Users\\gec\\AppData\\Local\\Programs\\Bot Framework Composer\\resources\\app.asar.unpacked\\extensions\\azurePublish\\dist\\extension.js:269227:17)","message":"Failed to bind luis prediction resource to luis applications. Please check if your luisResource is set to luis prediction service name in your publish profile."}"

"Error - Failed to bind luis prediction resource to luis applications. Please check if your luisResource is set to luis prediction service name in your publish profile."
The strange fact for him: when we checked the LUIS portal, the prediction resource was assigned to the LUIS app, but compo was throwing error and the publish of the bot did not happen.
Needless to say, the publish profile contained all the nodes (inluding luisResource) with the correct values.
And last week, the same user managed to publish another bot from the same machine.

Is there a current workaround for getting your bot published here?

As of today we are hitting this same exact issue. Last week deploying worked with out issues.
No changes to the our local or cloud environments. Composer 2.1.1 in use.
This is quite a major letdown as this effectively blocks our ability to do client releases.

@benbrown Thanks! We have been looking into this already a bit so this might now be the time to push it through.
If for nothing else more information on the failing bits always helps.

@mhtmht are you a part of multiple Azure tenants? One possible source of this error is if the key you using is owned by a different tenant than supplied during the API call...

Sorry, I also faced the same issue, did we found the root cause or any plan to fix it?

@mhtmht are you a part of multiple Azure tenants? One possible source of this error is if the key you using is owned by a different tenant than supplied during the API call...

Yes actually I am. Weirdly this was not a problem with any of the other environments under the same multi tenant setup.
But just to report back on the issues we were having. All the problems have magically disappeared during the time away the component in question. No deployment issue anymore. Also why things now work and didn’t a week ago is shrouded in mystery. No changes to the setup or method of deployment.

For me it started working after I started and stopped the bot in Composer before publishing it.


[WIP] Github Actions for Windows CI

Will eventually close #113 , but not ready yet.

Note: they say silent install of npcap is only available in Npcap OEM.
https://github.com/microsoft/winget-pkgs/pull/468#discussion_r428362393
I'm going to use WinPcap which I've successfully installed silently before:
https://github.com/sekineh/pcap-alt/blob/315c846928a95e04e590f87ef9b5e9c6d34608e5/appveyor.yml#L42

@Wojtek242 , I think it's ready to be reviewed.


Finished implementing add comment

In this pull request

I built the add-comment section of the project
A user can add his/her own comment as well as see other users' comments

Hi @Mylo16  you did a great job implementing the comment feature, there is nothing else to say than merge it. Just fix the merging conflicts first.


converting string to datetime?

I am using spark_read_csv to read a csv file. After loading the data, I want to convert a datetime column, which is character type currently, to a timestamp type. In SparkR I can use unix_timestamp to do this. But how can I acheive the same thing in sparklyr?

All of the functions defined in https://cwiki.apache.org/confluence/display/Hive/LanguageManual+UDF are accessible in the dplyr interface, so you should be able to accomplish this with something like:
tbl %>% mutate(timestamp = unix_timestamp(column))

@kevinushey Thanks. Problem solved.


chore(docs): update readme

Description
Vite takes promise wrapped plugins as well.

Nice catch, thank you very much @benatshippabo 👍 When I release the next patch, I'll also make sure to copy that into the core packages's README (it is the one being picked by npm).

Yup love your work on this! We are very grateful for this plugin ❤


Review Vendor Prefixes

As noted in #279, we need to review the vendor prefixes we use. Ideally, we should use a tool like autoprefixer to automate this for us.

This is a duplicate of #142 and this was already fixed in #185


Grounditems: Add option to consider item value per inventory slot

Adds configuration to the Ground Items plugin to consider item value per inventory slot.
If enabled unstackable and unnoted items on the ground won't combine their value to determine hidden or highlighted status.
Enabled on default
Closes #12891

Just realized I created the wrong PR, whoops.


Target alternative pp column

Will wait for upcoming osu-web migration for this, which will make the tests work again.

This should be good to go!


Improve the diff between expected/received on mock calls

Currently, the way that a diff between the expected/received when using .toHaveBeenCalledWith is being shown is not nice. It'd be good to make it look similar to how a "normal" diff (like when using .toBe) looks.
Here is a ridiculous example, trying to stress out the difference:
https://repl.it/@gustavovnicius/jest-diff
Saving you a click:

Nice diff

Nasty diff

Printing a comparison using the same mechanism sounds good to me. PR welcome!

@SimenB #5846 :)


Adopt unified feature calculation output format (numpy arrays)

In order to avoid complex handling of the results of the feature functions I would like to suggest that we adopt a "duck" format for the output of the latter even for functions such as "get_n_segments".
For example, if all the outputs were numpy arrays (bonus if the are flattened as well) then a special handling would not be necessary and the stat extraction would apply the same for any feature function.
Do you guys think that this would cause any kind of conflict?

I think this is a reasonable suggestion. @juanchopanza @mgeplf any objections - alternative suggestions?

I'm not sure I follow. Could you be more specific about which outputs you mean? Maybe if you add an example of something that would have to change it would be clearer.

I am not aware of anything that needs to change for the moment. This refers to the outputs of functions such as neuron.get_section_lengths(), neuron.get_...() as they are in ezy module. If those functions will become accessible through a different module this decision should be followed there as well.
I guess this proposal refers to the following problem that might occur: when we extract multiple features and we need to handle them uniformly (for plots, statistics, etc) we need to have a uniform format for the output of those feature extraction functions.
Example of where this will fail if we don't use a uniform format:
test_list = [0,1,2,3]
test_arr = np.array(test_list)

test_arr.min()
Out[15]: 0

 test_list.min()
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-16-42e1cdc8fab9> in <module>()
----> 1 test.min()

AttributeError: 'list' object has no attribute 'min'

On top of that some feature funcitons produce integers. e.g.:
nrn.get_n_segments()

produces the result:
15

Instead it would be convenient if it produced the result
np.array([15])

In order to process all the feature results in the same way and not perform complex handling.

I would propose that we map features to arbitrary feature extraction code, and use the mappings where applicable. So the problem of creating numpy arrays or a given dimension would be fixed in the feature extraction code. For example,
features = { 'number_of_segments': lambda nrn: numpy.array([nrn.get_n_segments]), ...}

Or, even better,
# works for neuron, neurom population, or collection of trees
features = { 'number_of_segments': lambda x: numpy.array([segments.count(x)]), ...}

Ok, I will optimize the code for the feature extraction once we get back :)


Update to 2.0.2

Hi @kostyay and thanks for the PR!
Please correct Backbone::Paginator::Rails::VERSION by setting to '2.0.2' and fix file permissions.
Then this would be good to go :wink:

Closed in favor of #2


What format is the ini file?

It's al garbled in nano or vi and MacOS textedit saves it corrupted
https://imgur.com/a/SpXc7pE

It defaults to unicode UTF-16 to support international character sets. If you don't want to use a unicode enabled editor, you could copy the one from the main github page and edit that.

Hmm doesnt seem to work
Unhandled exception. IniParser.Exceptions.ParsingException: Unknown file format. Couldn't parse the line: '# Place as many of these entries as you'd like for your plex libraries.'. while parsing line number 0 with value '' - IniParser version: <IP_ADDRESS> while parsing line number 4 with value '# Place as many of these entries as you'd like for your plex libraries. ' - IniParser version: <IP_ADDRESS>  ---> IniParser.Exceptions.ParsingException: Unknown file format. Couldn't parse the line: '# Place as many of these entries as you'd like for your plex libraries.'. while parsing line number 0 with value '' - IniParser version: <IP_ADDRESS>
i just did
touch fingerprint.ini
nano fingerprint.ini
paste
save

Oh, sounds like the ini library I'm using doesn't understand comments then. The  version it generates doesn't have the documentation comments in it. Here, use this one:
[directories]
C:\path\this\tool\sees\to\library = C:\path\Plex\sees\to\library

[default]
useAudio = true
useVideo = false

introMatchCount = 0
creditsMatchCount = 1

introStart = 0
introEnd = 0.5
introMaxSearchPeriod = 900

creditsStart = 0.7
creditsEnd = 1.0
creditsMaxSearchPeriod = 600

shiftSegmentBySeconds = 2

minimumMatchSeconds = 20
PermittedGap = 3
PermittedGapWithMinimumEnclosure = 10

audioAccuracy = 4
stride = 1024
sampleRate = 5512
minFrequency = 100
maxFrequency = 2750

videoAccuracy = 2
videoSizeDivisor = 50
frameRate = 1

recheckUndetectedOnStartup = false
forceRedetect = false

databasePath = C:\path\to\database\dir
PlexDatabasePath = C:\path\to\com.plexapp.plugins.library.db
ffmpegPath = ffmpeg
TempDirectoryPath = C:\path\to\empty\temp\dir

There we go!
Scanning as we speak
Thanks!


Load AJAX not work on first load

I create year calendar but it's not working on first load. If I change year and data source will load completely.
<script>
        $(document).ready(function() {
            var currentYear = new Date().getFullYear();

            var calendar = $('#calendar').calendar({
                weekStart: 1,
                minDate: new Date(currentYear-1, 0, 1),
                maxDate: new Date(currentYear+1, 11, 31),
                style: 'border',
                dataSource: getMyData()
            });
            function getMyData() {
                var myData=[];
                $.ajax({
                    url: 'load_events.php',
                    type: 'POST',
                    dataType: 'json',
                    data: {'action':'getDataCalendarYear'},
                    success: function (response) {
                        for (var i = 0; i < response.length; i++) {
                            myData.push({
                                id: response[i].id,
                                title: response[i].title,
                                startDate: new Date(response[i].startDate),
                                endDate: new Date(response[i].endDate),
                                color: response[i].color
                            });
                        }
                    }
                });
                return myData;
            }
        });
    </script>

if I console.log(calendar) dataSource variable already set.
and If I write dataSource manually (without ajax) it will work perfectly
did you know what's wrong about it?

Hi, refer to #103 there's an example, use it with the event yearChanged instead of dataSource
Why? Well it's somehow better because the event yearChange will be activated in the first run, other way to do it is instead of dataSource use its respective event like:
$('#calendar').data('calendar').setDataSource(getMyData());
Note: the way of the example load data per year, and you should do it that way of filtering, because that way you won't burden all the possible event of past and future years, you'll probably think it's not much but believe after a few months client will be go crazy creating events.
Note2: yearChanged it's avalaible at this repo and not at the release.


Update libvirt-go to 7.0.0

The current version (5.1.0) does not compile on 32-bit architectures
such as the Raspberry Pi. Updating the dependency to the latest
version and tweaking build errors allows the unit tests to pass.
Please make sure you read the contributor documentation before opening a Pull Request.

thx @nnooney  can you rebase commits?

Rebase complete.

thx @nnooney for your work.
Just to update you on what is going on recently on the project.
We plan to migrate to  a new libvirt library. https://github.com/dmacvicar/terraform-provider-libvirt/pull/813
this will allow to fix other issue we had.
Now I'm not strictly against this change but we are bumping the bindings from 2 major version here from 5 to 7.
As maintainer I would like to have the certitude we don't introduce any breaking change to the existing user of the project.
Could you perhaps post and share your findings what has been changed between these version so we could evaluate that?
Thanks for help and contribution! :sunflower:

Thanks for the update about the direction terraform-provider-libvirt is heading. I definitely agree that switching libraries to allow for integration with the terraform registry is the best path forward. This PR is primarily motivated so that I can compile the provider for ARM 32bit architectures (Raspberry Pi).
Thankfully the libvirt-go library mentions that they keep their API stable across major versions (examining their README at https://gitlab.com/libvirt/libvirt-go). Unfortunately, I could not find a changelog on their project. I examined the diff between the two releases (https://gitlab.com/libvirt/libvirt-go/-/compare/v5.1.0...v7.0.0) and here's what I found:

Changes to support building containers for CI (replacing some Vagrant files)
Several flag types changed from int to uint
Added support for virConnectSetIdentity: https://libvirt.org/html/libvirt-libvirt-host.html#virConnectSetIdentity
Added support for additional Domain Stats
Added support for Domain Checkpoints
Added support for Domain Memory Failures
Added support for additional Network Port lookup functions
Removed the following functions

virDomainSnapshotGetConnectWrapper
virDomainSnapshotGetDomainWrapper
virInterfaceGetConnectWrapper
virNetworkGetConnectWrapper
virSecretGetConnectWrapper
virStoragePoolGetConnectWrapper
virStorageVolGetConnectWrapper

It seems that they've added a lot of features to support the libvirt API that could be adopted by this project if desired. Not much was changed/removed except for the Connect functions and flag types.

Bump, I also see libvirt-go failing to compile on 32-bit arches while packaging for a distro. With this patch it builds fine.

master does not depend on libvirt-go.
Please check https://github.com/dmacvicar/terraform-provider-libvirt/pull/859#issuecomment-870103094
Pre-release is available here.


Add language support for notes from different octaves

While the implementation already supports pitches of pretty much any frequency, the language currently only allows us to creates the 12 (half) notes from C4 (261.6 Hz) to B4 (493.9 Hz).
We need to think of a way to specify different octaves in the language, then adjust the NoteCompiler to correctly parse that.
We want to eventually support at least all notes that can be found on a standard piano.

Blocked by #13


Add src dim and attr when query weight format from expected_weights_desc

This PR intends to:
(1) Add src dim and attr when we do weight prepack to query the right weight format, otherwise reorder happens a lot; Meanwhile attr is added as well for future int8 path use;
(2) do_prepare in matmul primitive + post-ops has the issue that once we call post-op sum, it will not accept a second post-op, like matmul+sum+relu;

@XiaobingSuper @jgong5  Please have a review. Thanks!


Improve responsiveness of legend.finos.org website

The current version of alloy.finos.org does not degrade well, especially to mobile devices. It looks like the responsiveness that is there by default in both Docusaurus and the FINOS blueprint, may have gotten lost along the way, or just not included.
I think it's important the site render well on mobile and would suggest that work be undertaken to improve the sites responsiveness and how the site renders on the mobile devices. In particular the font gets so small on a mobile phone it's almost impossible to read the copy.

The responsiveness is improved in my latest PR. I suspect there's still room for improvement, so will leave this open, but let's retest and comment on the specific places that need fixing.

See too https://github.com/finos/alloy/issues/259 from @mindthegab

legend.finos.org is now responsive on mobile


Error starting userland proxy: listen tcp <IP_ADDRESS>:5672: bind: address already in use

Hi there,
Context:
To run this app, I have to stop any external services related to mysql and rabbitmq.
MySQL server is stopped ✅:

An instance of rabbitmq is listening at this port :

But when I'm going to stop it with the following command:

it disappears as you can see.
❌ When I run make start command, it logs this:

Then, I run again above command to know if this service is listening:

What's wrong, please?
Any help would be welcome.
Thanks.

I run the make stop to be sure any instance of mysql, rabbitmq and so forth is running.
Then, I run the make start command but again tells me the same error.
Best regards.

I run docker ps to be sure any docker process is not running.

I run sudo rabbitmqctl stop to be sure any rabbitmq instance is not running

None of them are running. So, I run make start  but again it outputs the below error:

What's wrong?
Why, and where, a rabbitmq instance runs before this point?:

😓😓😓😓

Hi again,
I reopened this issue because after fixing the last error I have another one.
Please, find out in the attached file.
log_make_start.log
Best regards.

When I run make stop to stop the running instances of these services and I run make start the first error I posted here backs to appear.
Please, find it out in the attached log.
log_make_start.log


onUpdate function available?

I have been trying without any luck to get https://github.com/react-ga/react-ga to work with react-router-redux and ConnectedRouter. It uses an onUpdate function available on the original Router, but it doesn't seem to be available here.
Is there an equivalent way for ConnectedRouter to have an onUpdate-like function?

You're looking at the wrong version or repo. Make sure you're on next and looking at this code: https://github.com/ReactTraining/react-router/tree/master/packages/react-router-redux

I'm also trying to integrate react-ga with react-router-redux and am hitting a similar issue. I never see updating router in the console. I'm still working on the issue, so I'll update when I solve the problem!
import React from 'react';
import ReactDOM from 'react-dom';
import ReactGA from 'react-ga';
import thunkMiddleware from 'redux-thunk';
import { createLogger } from 'redux-logger';
import { createStore, combineReducers, applyMiddleware } from 'redux';
import createHistory from 'history/createBrowserHistory';
import { ConnectedRouter, routerReducer, routerMiddleware } from 'react-router-redux';
import { Provider } from 'react-redux';
import { reducer as formReducer  } from 'redux-form';
...

ReactGA.initialize(window.env.gaTrackingId);

const logPageView = () => {
  const page = `${window.location.pathname}${window.location.search}`;
  console.log('logging page view', page);
  ReactGA.set({
    page,
  });
  ReactGA.pageview(page);
};

const history = createHistory();
const store = createStore(
  combineReducers({
    ...reducers,
    form: formReducer,
    router: routerReducer,
  }),
  applyMiddleware(
    thunkMiddleware,
    //createLogger(),
    routerMiddleware(history)
  )
);

// Needed for onTouchTap
// http://stackoverflow.com/a/34015469/988941
injectTapEventPlugin();

ReactDOM.render(
  <Provider store={store}>
    <ConnectedRouter history={history} onUpdate={() => {console.log('updating router');}}>
      <MuiThemeProvider muiTheme={kqMuiTheme}>
        <App />
      </MuiThemeProvider>
    </ConnectedRouter>
  </Provider>,
  document.getElementById('root')
);

Solved.
React router v4 has removed onUpdate. Here's the solution:
...

const history = createHistory();
history.listen(logPageView);

...


Minor fix related to microseconds change - timestamp in instance tabl…

…e needs to be converted back to milliseconds to display correctly

@ammendonca Could you review please? Unfortunately I missed this timestamp on the instance table when doing the previous change.

retest this please

LGTM

@ammendonca Thanks!


Use child window

@oriondean, could you have a look at this PR to see if anything should be changed? I'll update readme tomorrow.

@oriondean, I addressed your comments, thanks for having a look. Could you look through my changes again, just to make sure that i have got everything right?


chenriks/cookies

-Removes the BOM signature from clientaccesspolicy.xml
-Adds Secure attribute to ForceSSL cookie
-Adds HttpOnly attribute to TempData cookie when cleared

@blowdart ?

LGTM

anything changed in this file?

Refers to: src/NuGetGallery/Public/clientaccesspolicy.xml:1 in 059db77. [](commit_id = 059db77a378e67f1be1c0e73ccb7b11068b27dae, deletion_comment = True)

:shipit:

@skofman1 re:clientaccesspolicy.xml: change is removing the UTF-8 signature


Update rules_python dependency

#1671 is already in the latest release

Type of change

Chore (any other change that doesn't affect source or test files, such as configuration)

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


prisma init fails when the Prisma CLI was installed via Homebrew

Reproduce: Install the Prisma CLI via Homebrew:
brew tap prisma/prisma
brew install prisma

Now run prisma init hello-world.
This is the error message:
$ prisma init hello-world
 ▸    Error parsing plugin /snapshot/cli/node_modules/prisma-cli-core

 ▸    'MODULE_NOT_FOUND': Cannot find module 'copy-paste'
 ▸    1) If you want to compile the package/file into executable, please pay attention to compilation warnings and specify a
 ▸    literal in 'require' call. 2) If you don't want to compile the package/file into executable and want to 'require' it from
 ▸    filesystem (likely plugin), specify an absolute path in 'require' call using process.cwd() or process.execPath.

 ▸    Error parsing plugin /snapshot/cli/node_modules/prisma-cli-core

 ▸    'MODULE_NOT_FOUND': Cannot find module 'copy-paste'
 ▸    1) If you want to compile the package/file into executable, please pay attention to compilation warnings and specify a
 ▸    literal in 'require' call. 2) If you don't want to compile the package/file into executable and want to 'require' it from
 ▸    filesystem (likely plugin), specify an absolute path in 'require' call using process.cwd() or process.execPath.

 ▸    Error parsing plugin /snapshot/cli/node_modules/prisma-cli-core

 ▸    'MODULE_NOT_FOUND': Cannot find module 'copy-paste'
 ▸    1) If you want to compile the package/file into executable, please pay attention to compilation warnings and specify a
 ▸    literal in 'require' call. 2) If you don't want to compile the package/file into executable and want to 'require' it from
 ▸    filesystem (likely plugin), specify an absolute path in 'require' call using process.cwd() or process.execPath.

 ▸    Error parsing plugin /snapshot/cli/node_modules/prisma-cli-core

 ▸    'MODULE_NOT_FOUND': Cannot find module 'copy-paste'
 ▸    1) If you want to compile the package/file into executable, please pay attention to compilation warnings and specify a
 ▸    literal in 'require' call. 2) If you don't want to compile the package/file into executable and want to 'require' it from
 ▸    filesystem (likely plugin), specify an absolute path in 'require' call using process.cwd() or process.execPath.

 ▸    init tyest is not a prisma command.
 ▸    Perhaps you meant help
 ▸    Run prisma help for a list of available commands.

Get in touch if you need help: https://www.prisma.io/forum/
To get more detailed output, run $ set -x DEBUG "*"

Prisma version:
$ prisma -v
prisma/1.22.0 (darwin-x64) node-v10.4.1

This should be fixed now, please try brew upgrade prisma


Transfer to ethersphere organization

Basically replace all references to filoozom/swarm-actions to ethersphere/swarm-actions.
I guess there's the licensing stuff to figure out too.

Yeah please tackle the license as well.


📣 README.md translation: looking for contributors (Chinese, Japanese, French, Portuguese, etc.)

The DLE development team is looking for more contributors to translate README.md and help decrease the entry barriers for new users.
You can help! 🤟
If you're ready to help:

First, please see the original README.md and current translation efforts in the ./translations directory.
Learn how to contribute – CONTRIBUTING.md has useful tips (pay attention to the "Translation" section)
We consider both GitHub PRs and GitLab MRs.

It is fine to use Google Translate as a basis for translation – but we want the final quality of the document to be good, so you have to know both English and {{Language}} to contribute. 🎏
All contributions will be recognized, mentioned in various places, and respected. 🥇
Please help us improve this repository and built up a great open-source community! 💯

I would love to contribute.

@GAURAV1-ui sounds great! let me know if you have questions


Has been marked unsafe by the Chrome Web Store.

This extension has been marked unsafe by the Chrome Web Store.

And the direct link to the DEV Tool is not working anymore - https://chrome.google.com/webstore/detail/threejs-developer-tools/ebpnegggocnnhleeicgljbedjkganaek

/ping @jsantell

Any update on this?

Any updates? @

Just pinging you guys, that there's a fork ongoing here: https://github.com/oslabs-beta/BACE


Auto format feature not configured for st.css files

Trying to auto format (alt+shift+f, for me) any .st.css files returns the following error:

Sorry, but there is no formatter for 'stylable'-files installed.

Implemented formatting functionality, closing as resolved.


The support of screen sharing

Really glad that LiveKit uses Rust, this opens so many possibilities! 🙂 Having an excellent and easy-to-use library for real-time communication is really cool!
I've shortly gone through the example code and published tracks to see if there is a simple and straightforward way to add screen-sharing (something as easy to use as in the JS/TS SDK). From what I've understood so far, it seems like currently the only way to have screen sharing is to capture the frames in the user code and then convert them to YUV with one of the helpers, then pass it to the capture_frame which will encode the shelf and send it down the line to the RTP engine, etc.
I.e. is my understanding correct that at the moment there is no simple way to e.g. use the screen capturing engine from libwebrtc, so the user is expected to capture the frames by other means and supply them to the tracks?

Hey, you're correct, at the moment there is no simple way to do screen capturing.
But exposing the native libwebrtc capture engine is planned (as well as microphones and webcams).


Bug: Crash when extracting a folder to a OneDrive folder (Business or personal)

Description
The app crash systematically when extracting a folder from download to a folder in a OneDrive folder (personal or Business)
Steps To Reproduce

Open files app
extract a full folder that you downloaded in a OneDrive folder
See the crash

Requirements

Fix crash that occurs when extracting a folder to a folder in OneDrive

Files Version
<IP_ADDRESS>
Windows Version
26120.751 (24H2)
Log File
debug.log

@yaira2 nope, just tested with everything being the same, it's all good

System.InvalidCastException: Unable to cast object of type 'Files.App.Utils.Storage.SystemStorageFolder' to type 'Windows.Storage.StorageFolder'.

I think this is related to the changes in #15341. @hishitetsu didn't you recently fix a similar issue?

@hishitetsu didn't you recently fix a similar issue?

I can't remember.

@CAPSLOOCK Thank you for bringing this to our attention, a fix will be included in the next preview build.


istio install file parity

right now we have
istio.yaml  # everything, no auth
istio-auth.yaml # same file with mtls on

we discussed that
istio.yaml -> istio-no-auth.yaml

and
istio-auth.yaml -> istio-with-auth.yaml

would be more clear and make it more likely a user would understand the choice
cc @rshriram @andraxylia
only downside is there is a lot of references to istio.yaml and istio-auth.yaml from 0.1 and tests that would need update. a less risky alternative may be to only change it on the release tgz and the website
or just leave with it until 0.3 / installer....
putting 0.2 for now as a proposal for tweaking this last minute (on the other hand the last PR setting up those files was merged only this morning so it's not like it's been "stable")

I propose we wait until auth will be on by default, and we can toggle auth on and off. At that point, there will be just istio.yaml.

I would prefer to leave as it is today.

sorry i was not super clear, that was a statement for 0.2.   because we have a lot of work right now for 0.2 and istio.yaml generates no confusion for me... plus like you said a lot of references.

I would opt for istio-noauth and istio-auth.yaml.. The reason is because, once defaults go out, its going to be hard to change perceptions and systems that rely on really not having auth or some features, will break when they install an istio.yaml which has auth turned on by default. It also makes the two options very clear in the docs.

There will be just istio with auth pretty soon, and a single istio.yaml file, with the option of toggling encryption on/off via some config/configMap.  The current model helps with the transition. A better change would be to have istio and istio-noauth, and get rid of istio-noauth in 0.3.

defer this to 0.3 as it is not critical for 0.2...let me know if anyone strongly disagrees :)

sgtm

We are moving to helm template, where each feature can be individually configured.
We'll still generate istio and istio-auth, for backward compat - but expectation is that each user will set the features however they want.
We will have some pre-baked sets of defaults - 'values-demo' with all on, 'values-auth', etc

A good start would be to have istio-noauth.yaml and istio-auth.yaml
generated from whichever is the source of truth for an upcoming release

AFAIK the plan is to switch to helm/istioctl, and keep the 2 configs only for migration/legacy.
In helm/istioctl each feature is independently enabled/disabled.


Updated linting and added spellchecking

Resolves #61.  Added spellchecking using retext-spell with a custom ember-dictionary for correctness and consistency in the guides. Initial word list is based on the new Ember CLI guide.
All available with npm run lint:md

corrected spellings and word inconstancies across files
local dictionary for guide specific words .local.dic
Added additional listing:

catch repeated words
catch inappropriate contractions
check indefinite articles
catch dead URLs
spellchecking

.remarkignore to ignore files during linting
.remarkrc.js for all remark and retext configuration, removed remark config from package.json

I had to refactor the dictionary for node 6 and drop linting for dead links to get tests to pass.  Can add back dead link linting when we move to node v8

@maxwondercorn thank you! I will open a new PR for updating Travis to Node 8, and an issue for adding in correct links.


set Access-Control-Allow-Origin header according to origin header

Feature
What did you expect to see?
It seems that traefik only support set Access-Control-Allow-Origin to a fixed value such as "Access-Control-Allow-Origin: *", "Access-Control-Allow-Origin: https://google.com" via traefik.frontend.headers.customResponseHeaders.
It would be better if we can set Access-Control-Allow-Origin to origin if origin satisfy requirements.
for example:
origin: http://domain1.google.com  -> Access-Control-Allow-Origin: http://domain1.google.com
origin: http://domain2.google.com  -> Access-Control-Allow-Origin: http://domain2.google.com

#1292

@irgb I have a branch where I am working on creating CORS headers, but there is a fair bit of logic involved. Should have something up in the next few days.

Closed by #3809.


Introduce async/await style tests

Seems good except for having to disable hinting.

@opsb if you're fine with #108, please merge and then re-enable hinting in this PR. I think a rebase may be needed as well.

Thanks @opsb


Update thrift gem dependency to 0.8.0

If this could get applied and a new thrift_client pushed, that'd be very helpful.  thrift < 0.8 won't build on Ubuntu 11.10.

+1. Thrift 0.7 won't compile under Ruby 1.9.

+1. The upstream issue that is fixed in 0.8 is Thrift-1382.
Anyone running Ubuntu 11.10 will not be able to install the Cassandra gem without manual intervention.

+1, Thrift 0.7 won't build on ubuntu 11.10

+1. Thrift 0.7 does not seem to work under ruby 1.9.3 and passenger 3.0.11.

Can you update this pull request?

@ryanking
It looks like master already has the 0.8 dependency.  Is this pull request even needed any longer?


west debug: openocd: console garbled with --tui option

Describe the bug
gdb option --tui to split screen and display source along with gdb console is available as an openocd runner option.
Though, when used with west debug, it provides a garbled output:

When using --tui gdb arg outside of west debug, it works as expected

with following commands run in 2 different windows:
$ ~/zephyr-sdk-0.17.0/sysroots/x86_64-pokysdk-linux/usr/bin/openocd -s ~/zephyrproject/zephyr/boards/st/nucleo_l152re/support -s ~/zephyr-sdk-0.17.0/sysroots/x86_64-pokysdk-linux/usr/share/openocd/scripts -f ~/zephyrproject/zephyr/boards/st/nucleo_l152re/support/openocd.cfg -c 'tcl_port 6333' -c 'telnet_port 4444' -c 'gdb_port 3333' '-c init' '-c targets' '-c halt'

$ ~/zephyr-sdk-0.17.0/arm-zephyr-eabi/bin/arm-zephyr-eabi-gdb-py -ex 'target extended-remote :3333' build/nucleo_l152re/zephyr/zephyr.elf -tui -ex load

To Reproduce
Steps to reproduce the behavior:

west build -b nucleo_f401re sample/hello_word
west debug -d build --tui
See error

Expected behavior
--tui should help debug experience, not garble it.
Impact
west debug not usable with --tui option
Logs and console output

Environment (please complete the following information):

OS: (e.g. Linux, MacOS, Windows): Ubuntu
Toolchain (e.g Zephyr SDK, ...): zephyr-sdk-0.17.0
Commit SHA or Version used: v4.0.0.rc3

Additional context
I suspect a timing issue between gdb server and client command Doesn't seem to be the case.

@pdgendt thoughts on this one?

I don't have something at hand to test the openocd runner, what's the output of west debug without the --tui option?
My guess would be that the gdb server is also started as a child proces for west?

what's the output of west debug without the --tui option?

Works as expected.

Works as expected.

Yes, I didn't doubt that, but from the garbled image there's a print Listening on por in between all the output. Is this also printed when not passing --tui? This to confirm my suspicion of having the server output interfering with the client output.

Does the following naive fix help?
diff --git a/scripts/west_commands/runners/openocd.py b/scripts/west_commands/runners/openocd.py
index 9b39791e1ee..0e477135687 100644
--- a/scripts/west_commands/runners/openocd.py
+++ b/scripts/west_commands/runners/openocd.py
@@ -396,7 +396,7 @@ class OpenOcdBinaryRunner(ZephyrBinaryRunner):
         self.print_gdbserver_message()

         if command in ('attach', 'debug'):
-            self.run_server_and_client(server_cmd, gdb_cmd)
+            self.run_server_and_client(server_cmd, gdb_cmd, stderr=subprocess.DEVNULL)
         elif command == 'rtt':
             self.print_rttserver_message()
             server_proc = self.popen_ignore_int(server_cmd)

Does the following naive fix help?

Nope
-- runners.openocd: OpenOCD GDB server running on port 3333; no thread info available
FATAL ERROR: command exited with status 1: ~/zephyr-sdk-0.17.0/arm-zephyr-eabi/bin/arm-zephyr-eabi-gdb-py -tui -ex 'target extended-remote :3333' build/disco_l475_iot1/zephyr/zephyr.elf -ex load

@erwango I am not familiar with TUI but it looks like west has supposedly included support for it since almost forever. Just curious if you were using it successfully before and it broke at some point?

New attempt:
diff --git a/scripts/west_commands/runners/openocd.py b/scripts/west_commands/runners/openocd.py
index 9b39791e1ee..d97cec1fe1e 100644
--- a/scripts/west_commands/runners/openocd.py
+++ b/scripts/west_commands/runners/openocd.py
@@ -396,7 +396,12 @@ class OpenOcdBinaryRunner(ZephyrBinaryRunner):
         self.print_gdbserver_message()

         if command in ('attach', 'debug'):
-            self.run_server_and_client(server_cmd, gdb_cmd)
+            server_proc = self.popen_ignore_int(server_cmd, stderr=subprocess.DEVNULL)
+            try:
+                self.run_client(gdb_cmd)
+            finally:
+                server_proc.terminate()
+                server_proc.wait()
         elif command == 'rtt':
             self.print_rttserver_message()
             server_proc = self.popen_ignore_int(server_cmd)

New attempt:

and this will be the last... It works like a charm, now. Thanks !

Confirmed!
It does look like this will need to be applied (factorized?) to other runners too. I can confirm that a similar fix seems to be needed for e.g. pyocd runner, so I wouldn't be surprized if it's needed for jlink too... maybe others?

Maybe some option like suppress_server_output could be introduced for all runners?

@pdgendt @erwango sounds like you have a workaround, are any of you planning to open a PR for this?

I'll open a PR with the current suggestion, I don't have the bandwidth for a more generic solution.


Different spacing found between "add event" and "no events"

Description
I went through all the sections, and I found an unusual spacing in the "Events" section. While other sections have no spacing between "add" and "no" containers. If it's a valid issue, then please assign it to me I'll fix this.
Screenshots
-> In events section :

-> In other section

Additional information
No response

Thank you for raising, it is a valid issue but we should solve it as part of an existing issue https://github.com/EddieHubCommunity/LinkFree/issues/7945 - so I will close this


Remove adam-cli jar from classpath during adam-submit

The ./bin/compute-adam-jars.sh script does not remote the adam-cli JAR from the classpath. This exposes SPARK-1921 when running on YARN.

So this undoes the "fix" #669 of #663. Can you say more about how this exposes SPARK-1921 @fnothaft?

Ah! Good catch @ryan-williams. You all are running Spark-on-YARN on a CDH5 setup at Mt. Sinai, right? Perhaps the behavior is different between our clusters (which wouldn't be surprising, I think the bug is caused by a race condition), but we are in the middle of bringing up a CDH5 cluster at Berkeley and saw the exact behavior described in SPARK-1921 (JAR timestamps differ). By removing the adam-cli jar from the JARS list when calling spark-submit, you don't copy the JAR twice, and thus don't have the timestamp race. It should be easy to make this work for both adam-submit and adam-shell; I'll fix this up.

@ryan-williams just pushed a new commit to address your comment. Doing things correctly actually makes the adam-submit code a bit cleaner, for certain definitions of clean (specifically, we get rid of the brittle way of determining the CLI jar). Let me know what you think, and I'll squash down and rebase.

this lgtm btw, unless I've persuaded you here to push CLI-jar detection and removal into compute-adam-jars.sh

@fnothaft Do you plan to move cli-jar detection tin compute-adam-jars.sh or is this ready to be merged?
@ryan-williams If this is ready to be merged, feel free to hit the button and merge it.

This isn't ready to merge yet; I still need to make fixes.

@ryan-williams just pushed a new round of fixes. Let me know what you think of these.

Builds failed with
hudson.plugins.git.GitException: Could not checkout null with start point 49a466a4d424b6acd1730c5daafe12f8e40d26b4

idk what that's about. Looking at the code now

Jenkins, retest this please.
@ryan-williams there's a bug in the Jenkins Git plugin where fetching force-pushes fails occasionally.

 This should be good to merge.

lgtm

+1
(Feel free to merge @ryan-williams)

the latest diff looks like it adds back the $ADDL_JARS stuff -- was this intentional?
https://github.com/fnothaft/adam/blob/fix-compute-jars/bin/adam-submit#L54

@heuermh thanks for the catch; not intentional, rather this comes from a rebase gone wrong... I will fix.

Fixed. Sorry; the rebase on #754 was kind of nasty; lots of messy conflicts.

want to retest this @fnothaft ? looks like git force-push problem again

good catch @heuermh btw! I looked at it and just assumed it was intentional, woops

Jenkins, test this please.

This is passing now; can I get a merge?


Add warning for large files in validate/conversion/convert

#306

@goneall Let me know what to change the file size limit to and warning msg

@kestewart This is the PR we talked about on the tech call that provides better feedback for large files.
I'll merge the PR, but I'm going to wait to deploy to see if there are other tooling fixes based on the NTIA plugfest.


InvalidPasswordException is never thrown

When I try to detect if a pdf has password with
private void isPDFProtected(String destinationPath,String password) throws IOException {         PDDocument pdfDocument = PDDocument.load(new File(destinationPath), password);         pdfDocument.setAllSecurityToBeRemoved(true);         pdfDocument.save(new File(destinationPath));     }
and I catch with
 catch (IOException e) {             e.printStackTrace();             //Open Alert Dialog             uiThread.post(new Runnable() {                 @Override                 public void run() {                     openAlertForPassword(destinationPath,url);                 }             });         }
As per docs the InvalidPasswordException should be thrown. I looked at the source and all load methods are throwing IOException. Is there something I am missing?

InvalidPasswordException extends IOException. This is a javadoc issue, which is fixed in Apache PDFBox, where the InvalidPasswordException is mentioned separately.

Ohh okay, thanks for replying, Can you tell a workaround for this? Where I can get this Exception ??

Try catching the exception like this:
try
{
     doc = PDDocument.load(....);
}
catch (InvalidPasswordException ipe)
{
    // wrong password
}
catch (IOException ioe)
{
    // other problem
}

I tried this , I think studio was showing me an error when I added InvalidPasswordException. I will definitely try this out again. Thanks

Either you didn't import it, or I'm wrong and in that case you'd have to analyze the text of the IOException that you get. But InvalidPasswordException should be thrown, I looked in StandardSecurityHandler.java and it is thrown.

Thanks its working !!! 😄


Potential Security Issue

I intend to contact you for a possible security vulnerability. I first wanted to open a ticket but then I thought that I should first try to directly reach out to you. Please describe your security policy in the repo.

You can reach out to @uranusjr @frostming and myself via email for now if you like, and we can determine how to proceed from there. We will likely try to get  a fix out if necessary before the next release.
Thanks for reaching out

I believe this has been addressed offline, and ultimately is not a major issue.
What is an issue is our lack of documented security policy, so we'd be happy to address that one for sure. Thanks for raising your concerns and reporting responsibly!


Run rules linter in master and PRs

Create a CI GitHub action with a rule_linter job similar to the one in capa.

GitHub Actions are not enabled in this repo 😭 😭 😭 @williballenthin @mr-tz can you please enable it? 🙏

The linter fails because of the bug being solved in https://github.com/fireeye/capa/pull/166

The syncronization of the submodule seems to be working properly: https://github.com/fireeye/capa/commit/96453db3be05c784f047492eeefd6f3ee299e917 🥳


How to loop

I want play music with loop.
How to do that?

I thinkwhile true is better than setInterval
while(true){
   await sound.play(filename)
}

Is there any reasons for while over setInterval?


new to merge

layouts/index.html
post/list.html
post/single.html

before:
https://github.com/Jyny/blog-theme/commit/5fefd49f62ef1e851728ada4835ca92f4247f472
after:
https://github.com/Jyny/blog-theme/commit/9943ab407d75f500ab2a71fc7d20245de79542ae


[Snyk] Security upgrade axios from 0.19.0 to 0.21.1

Snyk has created this PR to fix one or more vulnerable packages in the `yarn` dependencies of this project.

Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
yarn.lock

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

616/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.9
Server-Side Request Forgery (SSRF) SNYK-JS-AXIOS-1038255
No
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

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


Tweaks to compile with GraalVM

I was able to get HumbleUI app compiled using GraalVM native image (demo repo).
JWM and Skija work as-is only needing to specify some native image configuration flags. For HumbleUI I had to make some changes.
The main issue is that in order to make native image work I had to make classes that call into native libraries get initialized at runtime (using the trick to extract the lib into temp and load from there). However, this causes two categories of problems:

def bindings (which needed to initialize at build time) that use native classes are problematic. In my demo I encountered the problem forTimer, Shaper, BlendMode. The solution I came up with is to wrap them in delay and deref on use. For shaper which is used in other namespaces I wrapped it in a function which looks a bit weird, but it is so that consumers don't need to type hint it like ^Shaper @core/*shaper and can just use (core/shaper).
(defmethod print-method that dispatch on native classes. There are two classes Font and Typeface. To make it work I just commented these out. Are these used for some functionality or are these more for nicer debugging experience? If the latter they could perhaps be just deleted. Or as an alternative maybe implementing toString on Skija classes could work?

I opened an issue to discuss whether you are up to incorporate some of these changes first, and to find out if there is a better way to work around these issue, but am also happy to create a PR if we settle on a good solution.

Fantastic! Will try to run it tomorrow.
I’m curious why e.g. blend-modes don’t work? It’s a simple def that defines a map of values from static field from java classes. Seems like a perfectly valid use-case. Any idea why Graal has hard time with them?
print-methods are just for better readability, can be safely removed

I’m curious why e.g. blend-modes don’t work? It’s a simple def that defines a map of values from static field from java classes. Seems like a perfectly valid use-case. Any idea why Graal has hard time with them?

 But it looks like BlendMode does not call native, so it will probably be possible to tighten the config and revert the change. I will look into that.

print-methods are just for better readability, can be safely removed

:+1:

I was able to make it work without the BlendMode change and pushed a revert.

re: macOS. I was able to run JWM and Skija examples (which is amazing!), but humble does not even open a window :(
re: blend modes, what is the problem with top-level defs in general? Can it be solved by e.g. whitelisting all the classes somewhere?

re: macOS. I was able to run JWM and Skija examples (which is amazing!), but humble does not even open a window :(

Sweet, since both JWM and Skija work that means the native libraries work which is the main challenge. If it compiled but fails to run that's already a half-won battle :) I am not that familiar with mac, but if you post the error trace I might be able to help if it is similar to some issue I had to overcome.

re: blend modes, what is the problem with top-level defs in general? Can it be solved by e.g. whitelisting all the classes somewhere?

On one side to make clojure classes work they need to be initialized at build time. Any nested classes also need to initialize at build time. Since clojure namespaces get compiled into classes then anything inside def needs to be initializable at build time.
On the other side JNI works by dynamically loading the shared library (dylib/dll/so) at runtime using System/loadLibray. So if we try to initialize classes with calls to native method at build time, it fails with unsatisfied link error because we did not load the native libraries at that time yet. So we mark these classes to delay initialization to runtime.
The issue is where these two sides meet, when a build time initialized def encounters a runtime initialized class it fails to compile due to an initialization error.
There is a undocumented unstable feature to link JNI statically which perhaps could enable to initialize the native classes at build time. I will probably look into it at some point in the future because I am curious, but it is very uncertain whether it will work.

if you post the error trace

No error, just nothing happens

hmm, that's strange, even when I got segfaults it printed error report :thinking:


No feature is working

Issue Type: Bug
Most of the extensions are not working. VS code is very unresponsive. It's working the same as a notepad that is just writing code and nothing else. It delays in responding sometimes it does not respond at all. Had to wait until it finishes saving the file or some other task. Please look into the issue.

More details about bisect: https://code.visualstudio.com/blogs/2021/02/16/extension-bisect
Also: https://github.com/microsoft/vscode/wiki/Performance-Issues

thanks for reply will check it out
On Thu, 8 Jul 2021 at 12:50, João Moreno @.***> wrote:

More details about bisect:
https://code.visualstudio.com/blogs/2021/02/16/extension-bisect
Also: https://github.com/microsoft/vscode/wiki/Performance-Issues
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/128143#issuecomment-876196604,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKDYEYWVKBMDFKWWNOGBBTTTWVGT5ANCNFSM47676ETA
.

Let me know how it goes.


Error deploying example-etcd-cluster-with-backup-pvc

Not sure How to resolve the "gc deployment failed" error, the pvc does not resolve after 15 tries.
Am I missing some steps?
2017-05-23T19:51:44.458385071Z time="2017-05-23T19:51:44Z" level=error msg="gc deployments failed: the server could not find the requested resource" pkg=gc
2017-05-23T19:52:40.469359097Z time="2017-05-23T19:52:40Z" level=error msg="cluster failed to setup: fail to wait PVC (example-etcd-cluster-with-backup-pvc) '(Pending)/Bound': still failing after 15 retries" cluster-name=example-etcd-cluster-with-backup pkg=cluster
2017-05-23T19:52:40.472023433Z time="2017-05-23T19:52:40Z" level=warning msg="fail to handle event: ignore failed cluster (example-etcd-cluster-with-backup). Please delete its TPR" pkg=controller

Can yo provide k8s versions, operator versions, cluster spec, and steps to reproduce it?

Could be same issue as https://github.com/coreos/etcd-operator/issues/1122#issuecomment-303541843 .

-Kubernetes v1.5.4 (CentOS 7.3)
-etcd-operator Version: 0.3.0
-creating cluster with Spec (spec.ClusterSpec{Size:3, Version:"3.1.8"
(Note: The example-etcd-cluster-with-backup-pvc never shows up in the dashboard while the pvc was pending)

Steps to Repoduce:

kubectl create -f deployment.yaml
kubectl create -f example-etcd-cluster-with-backup.yaml

Kubernetes v1.5.4

Right. See my above comment. Could you try again with k8s 1.6 +?

I don't have a 1.6 env. at the moment.

Question about etcd-operator.
If I already have a etcd cluster deployment.  Can I configure the etcd-operator to the existing etcd cluster deployment?
Thanks.

It will discover if in the same namespace.

@bamb00
Do you have any list of requirements for production?

Not at the moment.  Thanks.


how to use Spring cloud gateway to support wss?

I use netty to request this url : wss://ykccdpz.huobicloud.com:443/-/s/pro/ws it can work well, when I use this SC-gateway like this:

id: wss
uri: wss://ykccdpz.huobicloud.com:443/-/s/pro/ws
predicates:
- Path=/ws
I get a result like this:
reactor.ipc.netty.http.client.HttpClientException: HTTP request failed with code: 403.
Failing URI: /-/s/pro/ws

2018-08-30 15:36:19.879 ERROR 1719 --- [ctor-http-nio-3] o.s.w.s.adapter.HttpWebHandlerAdapter    : Failed to handle request [GET http://localhost:8000/ws]
java.lang.IllegalStateException: Status and headers already sent

Are You sure You are not using SockJS on the frontend side ?

是的是的

Closing this due to inactivity. Please re-open if there's more to discuss.


Clarify privacy text

I agree that this is an important change. Merged request and pushed new app version.

I don't think you synced all changes before pushing. The photos are still
busted on prod but work ok for me locally.
On Mon, Aug 31, 2015 at 10:33 PM guberti<EMAIL_ADDRESS>wrote:

Merged #106 https://github.com/guberti/epschedule/pull/106.
—
Reply to this email directly or view it on GitHub
https://github.com/guberti/epschedule/pull/106#event-397410428.


Supporting A-Frame 1.6.0 and newer versions of three

Do you want to request a feature or report a bug?
Bug. Haven't had much time for AR.js of late, and unlikely to have much time in the near future, but using a few days' break in my schedule to investigate a few things that are perhaps easily fixed.

What is the current behavior?
AR.js 3.4.5 does not support A-Frame 1.6.0 or latest versions of three.js e.g. 0.164.0.
If the current behavior is a bug, please provide the steps to reproduce.
Modify A-Frame dependency to 1.6.0 when building.

Please mention other relevant information such as the browser version, Operating System and Device Name
What is the expected behavior?
Should work, but location-based examples fail with:
PlaneBufferGeometry is not a constructor.
This is easily fixed; use PlaneGeometry instead, e.g. see: https://stackoverflow.com/questions/77130600/error-three-planebuffergeometry-is-not-a-constructor
When this is fixed, the new-location-based A-Frame examples work.
If this is a feature request, what is motivation or use case for changing the behavior?
Able to support latest A-Frame.
@kalwalt not sure if you have any time to spend on AR.js these days (no worries if not... I don't have much time either, sadly!) but can you foresee any problems with upgrading A-Frame to 1.6.0 (and three to 0.164.0) on the marker and NFT side? Location-based side seems to work with the fix above.
Also wondering whether we need to include three as a dependency in package.json given it's already a dependency of A-Frame (see #504)

@nickw1 sadly I have no time to look into this. Did you try upgrading three.js also to the latest? I think it is not compatible with AR.js at this time. We should check upgrading three.js dependencies before upgrading A.frame? What do you think?

@kalwalt makes sense, yes. I'm not familiar with the marker-based/NFT side of things though, so all I can really do is test the location-based side (which seems to work if the above fix is applied).
In fact (as I messaged you a while back) I'm thinking of separating out the location-based AR into its own project within the AR.js organisation. I feel that this will result in faster bugfixes and improved compatibility with latest three or A-Frame as I will be able to do this without worrying about marker-based or NFT. Any thoughts?

Update for anyone reading this: there's now a new standalone location-based project here:
https://github.org/AR-js-org/locar.js
which aims (amongst other things) to keep up to date with latest versions of three.js.

Great @nickw1! i will try to upgrade A-frame and three.js in the main repository too, i have more time now.

@kalwalt just an update: I have made live LocAR.js examples (only two so far) available via GitHub Pages https://ar-js-org.github.io/. A while ago I think you mentioned the idea of making more examples available live, do we want to add some for other parts of AR.js?

@kalwalt just an update: I have made live LocAR.js examples (only two so far) available via GitHub Pages https://ar-js-org.github.io/. A while ago I think you mentioned the idea of making more examples available live, do we want to add some for other parts of AR.js?

Nice job @nickw1! maybe we can think in a near future to buy a web domain and have a simple web page of the github organization.
As we talked in the past we should have more live examples, we can start with the simple pattern markers and so continue with the others.

In regards of this issue i will start to upgrade AR.js to a more recent three.js version, I will start  this week with a draft PR.

@kalwalt Thanks! Sounds good.

Im working on this issue in draft PR #615. As mentioned in the PR it seems that it solves the issue #504 about the multiple Three.js instances. I think we need a .module version for all the build libs...

I merged PR #615 in the dev branch anyone interested can test it and report here any issue (about this issue of course...) In few days i will make a new release if nothing against.

@kalwalt great!

I'm opening a new PR to enhance the readme with the new changes and to make last small fixes before the release.

The code from #615 and #625 is merged in master! 🎉


Make issue in commit graph "clickable"

Create clickable links to internal/external issue tracker from commit graph UI.
Before/Turned off issue tracker:

After (turned on issue tracker):

LGTM

LGTM

LGTM


Fixed Prototype Pollution in keypather

📊 Metadata *
Fixed Prototype Pollution in keypather.
Bounty URL: https://www.huntr.dev/bounties/1-npm-keypather/
⚙ Description *
keypather is vulnerable to Prototype Pollution.
💻 Technical Description *
The package is vulnerable to Prototype pollution in the set function. This vulnerability is fixed by filtering the keywords that causes pollution from the object.
🐛 Proof of Concept (PoC) *
// poc.js
const set = require('keypather/set')
var obj = {}
console.log("Before : " + {}.polluted);
set({}, '__proto__.polluted', 'Yes! Its Polluted');
console.log("After : " + {}.polluted);

Before : undefined
After : Yes! Its Polluted

🔥 Proof of Fix (PoF) *

👍 User Acceptance Testing (UAT)
All Ok.

@d3m0n-r00t - can you remove the package-lock.json from the diff?

@d3m0n-r00t - can you remove the package-lock.json from the diff?

@JamieSlome I have removed the diff from package-lock.json.

@JamieSlome I have removed the diff from package-lock.json.


Include DAA evaluation

Adaption and integration of the source code from cakedefi-review for DAA evaluation

daa working with URL
integration into sidebar needed


Node v4 x64 profiling not displaying function names

Starting with a basic express application, launching profiling and loading some of the pages yields:

We have a similar issue (#299), where the function names were being displayed, but function code view was not visible. In this case, however, the function names aren’t being displayed at all – it just reports [Unknown]

duplicate of #362

Are you sure you duped this to the right one? They don't seem related.

@billti oops, thx for catching that :smiley:

From the manual test results:

0.10.33 x86

PASS - Win 8.1, Dev 14
PASS - Win 7, Dev 12
PASS - Win 10, Dev 12

0.10.33 x64

PASS - Win 8.1, Dev 11
PASS – Win 10, Dev 14 [my box]

0.12.7 x86

PASS - Win 10, Dev 14

0.12.7.x64

FAIL - Win 7, Dev 14
PASS - Win 8.1, Dev 12

4.1.0 x86

PASS - Win 7, Dev 11

4.1.0 x64

FAIL - Win 8.1, Dev 14

Also worth noting that 0.10.33 x64 still works because it uses the 32-bit address space for profiling logs.


Subpasses in MSL and HLSL

Sorry for all the issues 😅 Given the following GLSL:
#version 450 core
#extension GL_ARB_separate_shader_objects : enable

layout(input_attachment_index = 0, set = 0, binding = 0) uniform subpassInput test;
layout(location = 0) out vec4 out_color;

void main() {
    out_color = subpassLoad(test);
}

../glslang-build/standalone/glslangvalidator test.frag -V; and ../spirv-cross/spirv-cross frag.spv --msl creates the following MSL (silently fails with unknown_texture_type):
#include <metal_stdlib>
#include <simd/simd.h>

using namespace metal;

struct main0_out
{
    float4 out_color [[color(0)]];
};

fragment main0_out main0(unknown_texture_type<float> test [[texture(0)]])
{
    main0_out out = {};
    out.out_color = test.read(int2(0));
    return out;
}

--hlsl with a high enough shader model will throw an exception.
I realize this is  because DimSubpassData is not handled at the moment for MSL and HLSL. I was wondering whether implementing this in MSL should be as straightforward as the HLSL comment suggests, or if you have any thoughts about how it should be handled.

Neither are supported currently, but shouldn't be that hard to add.


enhanced BaseListView summary display with no result

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
yes

Tests pass?
yes

Fixed issues
comma-separated list of tickets # fixed by the PR, if any

if the dataProvider has no result, the summary div will be hidden. the outer list view div will move up.

this text is already provided by the $emptyText property. With your change the text would appear twice.

the $emptyText is not effective on summary div

and it should not as the text is already displayed below the summary, why do you want to display it twice?

As we konw, the summary-div is above of list-view div. If the dataProvider has no result, the summary-div will be hidden. the list-view div will up to the position of summary-div. To avoid going up, we can fill in the several words in the summary-div.


node/KeepUp: don't check TTL synced if node is not synced back past start of active era

It is possible that if a node just caught up, its highest orphaned block is within the current active era. We need to let the node sync back past the start of the active era and only after that we can determine if it is synced to TTL.
Fixes: https://github.com/casper-network/casper-node/issues/4160

closed in favor of https://github.com/casper-network/casper-node/pull/4195


scrollTopOffset being applied on state-change with same route

Stencil version:
<EMAIL_ADDRESS><EMAIL_ADDRESS>
I'm submitting a ...  (check one with "x")
[ x ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or https://stencil-worldwide.slack.com
Current behavior:

When setting a value for scrollTopOffset on stencil-route-switch, the route will apply the offset when simply updating the state of a matched component.
As far as I can tell it is caused by the following line in route.tsx
https://github.com/ionic-team/stencil-router/blob/228e1f3888759d651aa436ea3bfe5201e5872803/packages/router/src/components/route/route.tsx#L42
Which says that as long as a route is defined as part of a switch/group, isGrouped will always be true
This in turn makes the following code always break the function:
   if (!newLocation || isGrouped) {
      return;
    }

And therefore the previousMatch variable is never set. This leads to matchesAreEqual always failing, as it will compare against an undefined value, triggering the scrolling to top
https://github.com/ionic-team/stencil-router/blob/228e1f3888759d651aa436ea3bfe5201e5872803/packages/router/src/components/route/route.tsx#L76
Expected behavior:

When not changing the active route node, the scrollTopOffset should not be applied and snap back to the top of the window
Steps to reproduce:

You can use the default example and add some @State() variables to the landing page:
<stencil-router>
  <stencil-route-switch scrollTopOffset={1}>
    <stencil-route url="/" component="landing-page" exact={true} />
    <stencil-route url="/demos" component="demos-page" />
    <stencil-route url="/other" component="other-page" />
    <stencil-route component="page-not-found" />
  </stencil-route-switch>
</stencil-router>

When changing the state it will scroll back top without having changed the route
Note: scrollTopOffset is 1, because 0 is falsy and makes the router think the value hasn't been set per the following line:
https://github.com/ionic-team/stencil-router/blob/228e1f3888759d651aa436ea3bfe5201e5872803/packages/router/src/components/switch/switch.tsx#L87
Other information:

As a current hack, disabling line 44 to 46 of route.tsx seems to fix the issue, without any visible side effects, but I might be overlooking something

+1

I don't know if the root cause & stuff are legit, but the bug report sure as hell is.

@rayniervandam : Thanks for the hack. It works for my case as well.


Setting own API Key should not need modifying your package

thanks for the suggestion. it should already be fixed in https://github.com/ridvanbaluyos/face/releases/tag/v1.1


[BEAM-4461] Introduce Group transform.

run Java Precommit

retest this please

run java precommit

Can you split this PR? E.g. infrastructure work into one, SchemaAggregateFn into another, and the Group into a separate one? Otherwise it is hard to review all at once

Several parts are now split into separate PRs.
On Thu, Aug 30, 2018 at 11:04 PM Anton Kedin<EMAIL_ADDRESS>wrote:

Can you split this PR? E.g. infrastructure work into one,
SchemaAggregateFn into another, and the Group into a separate one?
Otherwise it is hard to review all at once
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/6298#issuecomment-417561294, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AUGE1clhRBReJcZGlxTKQdasH750LutMks5uWNHdgaJpZM4WSM0f
.

Thank you, looking at those

@akedin  @apilloud most of the files are now reverted, as they are submitted in other PRs. There's a little more I can split off into separate PRs, if you think necessary.

@akedin comments addressed

@akedin any more comments?


EC2 run instances with --iam-instance-profile strange error

I encounter strange behavior with aws ec2 run-instances which includes --iam-instance-profile flag.
With simple
aws ec2 run-instances --image-id ami-example1234567890

it works fine, instance is created. I get 200 response in log:
<IP_ADDRESS> - - [11/Oct/2023 11:52:26] "POST / HTTP/1.1" 200 -

In case I include --iam-instance-profile flag like:
aws ec2 run-instances --image-id ami-example1234567890 --iam-instance-profile Name="arn:aws:iam::123456789012:instance-profile/admin-role"

or any Name content like:
aws ec2 run-instances --image-id ami-example1234567890 --iam-instance-profile Name="test"

I get error An error occurred (NoSuchEntity) when calling the RunInstances operation: Instance profile arn:aws:iam::123456789012:instance-profile/admin-role not found.
With log:
<IP_ADDRESS> - - [11/Oct/2023 11:52:39] "POST / HTTP/1.1" 404 -

But even with 404 error new instance is created.
I'm not against creating instance without validating such rare flag, but it fails to return correct 200 response.
I use docker image with version 4.2.5.
All aws client commands I run against container like:
aws --endpoint-url=http://localhost:5000 ec2 run-instances

Thanks for raising this @romanilchyshynb - that's a bug indeed. I'll submit a PR shortly to make sure this validation occurs before actually creating the instance.


Missing LET Commands

Some of the lines don't have LET commands but need that functionality so add them and it will make it a lot easier to execute them

It wasn't very difficult and now it's much cleaner


Language pack breaks settings menu

Issue
Language pack breaks settings menu labels / values. (Only tested with EN_GB)
Actual result
Some settings menu options/values display '[native code]' instead of expected label.
Expected result
Correct settings menu labels displayed in English.
Notes
Uninstall language app and menu back to normal

Hi,
can you please add pictures to fix this - thanks.

@MaBecker

Thanks - just fixed this with ea32401ba6fc944825c94dad69e27a5da61159c0
I'd spotted it once before but assumed I was using a broken dev firmware :)


[BUTTON IDEA]: Star Button

1. Name of button: Star Button
2. Description: A star button that is filled when active
3. Button Type (Animated, Special, etc...): Icon button
4. Will you work on it?: Yes

Nice idea @vitoriapena I'll assign you


Error loading webview: Error: Could not register service worker: InvalidStateError: Failed to register a ServiceWorker: The document is in an invalid state..

Error loading webview: Error: Could not register service worker: InvalidStateError: Failed to register a ServiceWorker: The document is in an invalid state..

Not sure what else I can provide here. I was trying to generate a call graph in the vs code extension

Restarting VS Code after installing the extension solved this issue for me

Yep, same here. Well, I restarted my whole computer but it works again

I haven't seen this error, but seems it can be solved by restarting VS Code (and some cleaning maybe).
Closing this issue as it seems not a problem of the extension.


Replace upbound/upjet to crossplane/upjet

Description of your changes
This PR replaces the github.com/upbound/upjet Go mod dependency with github.com/crossplane/upjet.
I have:

[ ] Run make reviewable to ensure this PR is ready for review.

How has this code been tested

/test-examples="examples/applications/application.yaml"


Setup Json & Run the program

Good Reviewer

On exit from this app saved all the data (books, people, and rentals) into JSON files.
On startup, this app loaded all data from files.

All suggestions are welcome!

Thank you for the review @Ugboaja-Uchechi 😘😘


iOS: Image View Tweaks

Make sure to set the initial content mode
Make the clipsToBounds flag available. Pretty much any time you're using .ScaleAspectFill you want this set. It matters.

Great finds :+1:


From private repo

@PRMerger15 : Thanks for your contribution! The author, @aczechowski, has been notified to review your proposed change.


Discuss: how to configure admin_key for multi-APISIX cluster.

Related to #33

For now, apisix-ingress-controller do not support the config admin_key from Apache APISIX.
When using  apisix-ingress-controller with APISIX, it is recommended to remove the admin_key setting. You can use allow_admin to restrict the access scope of the APISIX control plane to ensure security.
Now we can discuss how to support the admin_key in apisix-ingress-controller.
In different APISIX clusters, there are different admin_keys. Usually we manage admin_key through secret resource. But it is not easy to sync with CRDs in apisix-ingress-controller, for secret may in the different namespace.
We can use ingress.class to specify APISIX cluster, and also can define an annotation for admin_key, such as ingress.admin_key, But it will need to be updated manually in the future, if admin_key need to be changed.
So anyone has a good idea, can discuss it here. Thanks.

@tokers what do you think about this ?

Secret is the most appropriate way, we can watch all namespaces except some unexpected like kube-system, istio-system and etc.

The multiple APISIX cluster support should be implemented by Manager API. we as the invoker of Manager API, only need to pass cluster flag to it.
@gxthrj @membphis

for the CP API, need to support mTLS way. The Admin API has supportted this feature. The Manage API, need @nic-chen 's confirming.
if the ingress-controller needs to connect multiple APISIX clusters, I think using mTLS is a better way than the token or JWT method.

The multiple APISIX cluster support should be implemented by Manager API. We as the invoker of Manager API, only need to pass cluster flag to it.
@gxthrj @membphis

This sounds good.
But there is another problem. If we  need to specify / add  an APISIX Cluster in CRD,  the manager api need to config too.
The configuration link has become too long and uncontrollable


wasnt quite working for correctly in certain cases!

Types of change(s)
Was working in all cases - somehow.
This is part of what's needed (will be needed) for mkiau to handle cubed-background files when running mkiau.x and cubed analysis files (JEDI) when available.
Now all works yet.
Checklist
Description
Related Issue
FOR NOW THIS IS A DRAFT

This should be a DRAFT ...

Closing as this was taken care of in other PRs to develop and to an older tag.


Hide Dropmessage

is there anyway we can hide drop message display?

Did you try to set an empty string in [dropBoxMessage]?

yeah. but the default value will display. i did try to hide but I had to override the css, with display:none. but it works anyway. :) should i closed this one? thanks


Configure & Deploy BlockRewardContract for PoA Bridge

Description
As a user I want to be able to
Transfer ERC20 Tokens to the Trustlines Sidechain and generate native Coins on the Sidechain.
Inside the PoA Network Bridge Test Environment
So that I can
Transfer Trustlines Tokens and evaluate the PoA Bridge
Acceptance Criteria

[ ] PoA Validator Deployed
[ ] BlockRewardContract configured to use PoA Addresses
[ ] BlockRewardContract Deployed
[ ] BlockRewardContract Activated via chainspec

Related story / issue

https://github.com/trustlines-protocol/blockchain/issues/9#issuecomment-484578874 for RewardContract Details (3rd paragraph)

Reviewer

The original POA RewardByBlock contract depends on complex proxy storage, key manager and eternal storage dependencies. This is part of the big poa-network-consensus-contracts repo and not the bridge itself. We only need parts of that which are much more connected to other components we are not using or have our own one (like validator set contract). The deployment of this is a huge script also containing more than we need. The deployment of the reward contract is related to their developer set contract, the storage proxies are filled etc.
To keep it easy on tests, a derived contract has been written which only needs a single init call for all necessary information. It implements the interface for Parity as well as the bridge. The contract has been verified to work with Parity as reward contract. The bridge functionality has been tested via unit tests, but not with a deployed bridge, which is the next step now.

The original reward contract with slight modifications has been deployed and is working

We still consider to use an own implementation, since we will need to extract their contract anyways. Therefore we can also simplify their Solidity implementation to get rid of the unnecessary dependencies.

@jannikluhn the first to requirements can be stroked out, since you modified the reward contract to not check for validators, correct?

I don't understand criteria 1 and 2. 3 and 4 is done.

I agree they are not well understandable. Unfortunately I was on vacation during the refinement of this.
So I understood it as we deploy the ValidatorSet contract and connect it to the bridge. Doens't make sense I guess...?

@andfletcher could you clarify the mentioned criteria?

We clarified the open questions during sprint meeting and figured out that we have done them already.