Merge master into moravska galerie

Fixes https://jira.sng.sk/browse/WEBUMENIA-690?
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] I have updated the CHANGELOG
[ ] I have requested at least 1 reviewer for this PR
[ ] I have made corresponding changes to the documentation

well. it's huge! I reviewed just briefly. seems working.
but tests are not even starting on my machine. are tests passing for you?

Yes, tests pass.


RuntimeError: shape '[1024, 512, 3, 3]' is invalid for input of size 453966

Can you please help me out with this?
Traceback (most recent call last):   File "demo_yolo3_deepsort.py", line 95, in <module>     with Detector(args) as det:   File "demo_yolo3_deepsort.py", line 20, in __init__     self.yolo3 = YOLOv3(args.yolo_cfg, args.yolo_weights, args.yolo_names, is_xywh=True, conf_thresh=args.conf_thresh, nms_thresh=args.nms_thresh)   File "/home/aayush/new/deep_sort_pytorch/YOLOv3/detector.py", line 14, in __init__     self.net.load_weights(weightfile)   File "YOLOv3/darknet.py", line 353, in load_weights     start = load_conv_bn(buf, start, model[0], model[1])   File "YOLOv3/cfg.py", line 191, in load_conv_bn     conv_model.weight.data.copy_(torch.from_numpy(buf[start:start + num_w]).view_as(conv_model.weight.data)); start = start + num_w RuntimeError: shape '[1024, 512, 3, 3]' is invalid for input of size 453966

Seems like the weight file got corrupted because of a break in the download. Closing the issue.


Update Forge Guide summary, fix misc in deploying.md

Something like this?

I think so. Maybe remove the emoji - we shouldn't really treat it as a callout, it's more like a quick reference for the most common issues, so it should be easily linkable imo. Wdyt?

Good call. This is cleaner.

Thanks again :smile:


docs($interpolateProvider): document XSS in $interpolate

$interpolateProvider.startSymbol & friends are often used dangerously, to embed Angular templating in other templating languages. This change documents that that is a very dangerous practice.

Done.

LGTM


Automation : convenience script for generators

Hey @stylesuxx
Fantastic generator, great work!
One (small) thing I've found using developing with this is that when I run the sub-generators, I have to copy/paste their names from readme, which always feel like a bit of a hassle. So I made a simple shell script like this:
#!/usr/bin/env bash

yo react-webpack-redux:$1  $@:2

First argument is the generator name, rest are passed through.
I wonder though if this something that could be achieved with npm? So you could have something like
npm run action action_name
npm run container container_name
npm run reducer reducer_name

Happy to submit a PR if this has affected anyone else!

Hey @andytango ,
interesting idea. Technically it should be possible to add those npm actions. But to be honest, I am not sure if I want to do this. That way I am abstracting away the usage of yo, but I want the user to know, that he is using yo in case he wants to pass own args.
However - aliasing is of course a good idea, maybe this could be added in form of a wiki article and linked from the Readme.
What do you think?

@stylesuxx Sure, so provide instructions and examples on how to do this that people could paste into their package.json if they wish?


Fix distance_matrix and k_shortest_path_lengths if node holes.

Previously, distance_matrix and k_shortest_path would panic
if the input graph contained holes in node indices. This commit
fixes this in distance_matrix by mapping nodes to a compact interval
0..n where n is the number of nodes in the graph.
In k_shortest_path we should initialize a vector of node_bound size.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

17 of 17 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.0005%) to 97.53%

Totals

Change from base Build<PHONE_NUMBER>:
0.0005%

Covered Lines:
10188

Relevant Lines:
10446

💛  - Coveralls


Reduce number of warnings

What are the reasons/motivation for this change?
There are a number of easy to fix warnings:

comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int' [-Wsign-compare]

Explain how this is achieved.

Const::size() returns int, so change iterators that use it to auto instead of size_t. For cases where size is being explicitly cast to int, use the wrapper that we already have instead: Yosys::GetSize().

If applicable, please suggest to reviewers how they can test the change.
Check what/if any warnings still show up at the bottom of the "Files changed" tab and in the "Compiler testing" section of the checks

@povik Do you want me to tidy up the warnings in the aiger2 code?

Yes please


Allows to customized repository instance

We should allow to pass a custom repository instance defined by the client. For example:
describe RailsEventStoreMongoid::EventRepository do

  subject(:repository) { described_class.new(adapter: InMemoryAdapter.new) }

  it_behaves_like :event_repository, described_class

end

+1
Not all event repositories must have parameterless constructor


AtClientManager.synchService.isInSync crashes after a few calls .

Describe the bug
Crashed app when checking that status of Sync between local and remote secondaries
To Reproduce
Steps to reproduce the behavior:
When using this code
  while (true) {
    print('sync in progress');
    var a = await atClientManager.syncService.isInSync();
    print('Are we waiting ?' + a.toString());
    sleep(const Duration(seconds: 5));
  }
  print('OK Ready');

I always get a Unhandled exception after a few cycles..
 Error : FormatException: Unexpected character (at character 1)
data:[{"atKey":"_latestnotificationid.ai6bh@ai6bh","value":"{\"id\":\"50dc4...
^

Unhandled exception:
NoSuchMethodError: The method '[]' was called on null.
Receiver: null
Tried calling: [](0)
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:68:5)
#1      SyncUtil.getLatestServerCommitId (package:at_client/src/util/sync_util.dart:87:18)
<asynchronous suspension>
#2      SyncServiceImpl._getServerCommitId (package:at_client/src/service/sync_service_impl.dart:410:27)
<asynchronous suspension>
#3      SyncServiceImpl.isInSync (package:at_client/src/service/sync_service_impl.dart:392:26)
<asynchronous suspension>
#4      main (file:///C:/Users/colin/GitHub/@company/mwc_demo/dart/iot_sender/bin/iot_sender.dart:52:13)
<asynchronous suspension>
PS C:\Users\colin\GitHub\@company\mwc_demo\dart\iot_sender>

Looks like a low level protocol problem
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Were you using an @application when the bug was found?

MWC demo sender code

Additional context
Isthere or should there be a better way to check ?

@murali-shris please can you take a look since you're working on the sync progress listener at the moment

@murali-shris please can you take a look since you're working on the sync progress listener at the moment

Sure Gary.

root cause:
The call to isInSync from the main method, interferes with a sync from server. Both share the same remote secondary. Hence we get sync verb response in the place we are expecting latest server commit Id.
possible resolution:
in the call to isInSync create a new remote secondary instead of reusing the remote secondary.
To be discussed in architecture call

https://github.com/atsign-foundation/at_client_sdk/pull/405

Fixed in at_client version 3.0.10

@cconstab can you please verify the fix in latest version of at_client

pending fix verification.Moving to next sprint

@cconstab I am closing this issue.Please re-open if you face the issue again.


Create README.ptBr.md

Translate from English to Brazilian Portuguese

adding language in README.md and reviewing a typo in README.ptBr.md file

.


Truncate sub-{mili,}second durations to 1

Closes https://github.com/go-redis/redis/issues/1363

cc @vmihailenco


VisUI: Filechooser background task resets user input

If the user was typing a file name (such as for saving a file) while the list of files in the directory are being fetched the name textbox will be reset, causing them to be missing part of the file name they typed out and to be very confused.

Now it won't be cleared when it has keyboard focus.


Termination response, does not include Tus-Resumable header

Added Tus-Resumable header for both the minimal api filter and the ActionFilter attribute implementations.


feat(identify files): automatically detect files using git

Fixes #589

@Archcry I've refacted / updated all you're remarks. I will work on the last todo's tonight/this weekend. Let's keep the ball rolling!

Also, it still mentions the InputFileDescriptor which is gone, right? And there is an odd + at the end

The typescript plugin mentions it changes the files array, but I believe that has been removed from the code: https://github.com/stryker-mutator/stryker/tree/feat/identify-files/packages/stryker-typescript#typescriptconfigeditor

@simondel how do you want to merge this in master? A squash commit is not helpful I think (as we would only have 1 note in the changelog). I was thinking on rebasing it and do 1 commit per package. That way we can outline exactly what changed in that package. Most packages will have breaking changes, so we need to manually release this.

@simondel Thanks for your review comments! I've refactored the code. Could you please take another look?

@nicojs I'll take another look at it :) I'm fine with creating one merge commit. We should also consider a blog post.

I've cleanup our changelog (force push). Ready to merge. Let's wait for one last green build


build lightgbm with gcc

I did a test by slamming in the h2ogpu built from the PR inside DAI 1.6.0 on Power and it worked for me.


NPE in RibbonApacheHttpResponse.getInputStream when issuing a PATCH request to spring data rest resource

I need to be able to issue PATCH requests through a ZuulProxy and was glad it is possible since https://github.com/spring-cloud/spring-cloud-netflix/issues/412. Unfortunately I had an NPE when trying the new classes.
Setup :

a spring data rest microservice exposing a @RepositoryRestResource, for example /users
an @EnableZuulProxy microservice with the following bean :

@Bean
public HttpClientRibbonCommandFactory ribbonCommandFactory(SpringClientFactory clientFactory) {
    return new HttpClientRibbonCommandFactory(clientFactory);
}

If I issue a PATCH request directly to the data microservice everything works fine. If I issue the same PATCH request through the Zuul Proxy I get the NullPointerException :
java.lang.NullPointerException: null
run_1 | 	at org.springframework.cloud.netflix.ribbon.apache.RibbonApacheHttpResponse.getInputStream(RibbonApacheHttpResponse.java:126) ~[spring-cloud-netflix-core-1.1.0.BUILD-SNAPSHOT.jar:1.1.0.BUILD-SNAPSHOT]
run_1 | 	at org.springframework.cloud.netflix.ribbon.RibbonHttpResponse.getBody(RibbonHttpResponse.java:49) ~[spring-cloud-netflix-core-1.1.0.BUILD-SNAPSHOT.jar:1.1.0.BUILD-SNAPSHOT]
run_1 | 	at org.springframework.cloud.netflix.zuul.filters.route.RibbonRoutingFilter.setResponse(RibbonRoutingFilter.java:164) ~[spring-cloud-netflix-core-1.1.0.BUILD-SNAPSHOT.jar:1.1.0.BUILD-SNAPSHOT]
run_1 | 	at org.springframework.cloud.netflix.zuul.filters.route.RibbonRoutingFilter.run(RibbonRoutingFilter.java:77) ~[spring-cloud-netflix-core-1.1.0.BUILD-SNAPSHOT.jar:1.1.0.BUILD-SNAPSHOT]

There is a workaround with the following code in the data microservice :
public class RestConfiguration extends RepositoryRestMvcConfiguration {
    @Override
    protected void configureRepositoryRestConfiguration(RepositoryRestConfiguration config) {
        config.setReturnBodyOnUpdate(true);
    }
}

The default value for returnBodyOnUpdate is false, and when issuing a patch in this configuration there is no body that is returned, only an HTTP 204 No Content. Setting the value to true returns the modified resource.
I was expecting the same problem with a POST (which doesn't return a body by default either), but it's not the case.

I've just checked the latest build and can see no NPE on my setup anymore :smiley:
Many thanks !


Revert "Stops second invocation of createNewSession while expanding OE tree items on disconnected servers (#21437)

This reverts commit 8c8a7859a02e8907af183f60a6478895123a5cdc.  Removing commits in release branch that possibly are related to https://github.com/microsoft/azuredatastudio/issues/21578.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 7 (14.29%)  changed or added relevant lines in 3 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.008%) to 42.05%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/services/objectExplorer/browser/objectExplorerService.ts
1
2
50.0%

src/sql/workbench/services/objectExplorer/browser/treeUpdateUtils.ts
0
1
0.0%

src/sql/workbench/services/objectExplorer/browser/treeSelectionHandler.ts
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/sql/workbench/services/objectExplorer/browser/treeSelectionHandler.ts
2
2.59%

Totals

Change from base Build<PHONE_NUMBER>:
0.008%

Covered Lines:
28598

Relevant Lines:
63167

💛  - Coveralls

Based on testing, this PR doesn't appear related to https://github.com/microsoft/azuredatastudio/issues/21578, so closing.


Plot not compatible with Bokeh 0.9.1

Sad but true, see also the discussion in #11. @mdda , if you feel you can tackle this, that would be great, since I am by no means a Bokeh guru.

Ok - I'm game for having a look at this.  My original problem with fixing #11 is that it seems to work (for me) against v0.9.0 from PyPI (and the bokeh changelog doesn't include any obvious smoking gun).  Let me play around with Plot against bokeh:master over the weekend and see where I get...
For now, my acceptance test will be whether the sample iPython notebook ( https://github.com/mdda/pycon.sg-2015_deep-learning/blob/master/ipynb/1-LivePlotting.ipynb ) works.

Bokeh 0.9.0 works for me as well.  But now 0.9.1 is also on PyPI. And indeed the changelog does not give a cue.

This seems broken (bokeh 0.9.3), any chance of having it fixed ?

This is a bug in Bokeh, see bokeh/bokeh#2787. Apparently the index page for the Bokeh server was kind of an afterthought, and nobody noticed when it broke following a change to the jQuery settings about 5 months ago ( bokeh/bokeh@607c920 ). I don't really know JS that well or CoffeeScript at all, but if you do, feel free to give it a shot. (I just bisected it yesterday, so there's a chance one of the Continuum guys will have some insight how to fix it in the coming week.)

I've managed to fix it, pending the pull request's acceptance. See bokeh/bokeh#2792.

That sounds great!
On 1 September 2015 at 00:48, David Warde-Farley<EMAIL_ADDRESS>wrote:

I've managed to fix it, pending the pull request's acceptance. See
bokeh/bokeh#2792 https://github.com/bokeh/bokeh/pull/2792.
—
Reply to this email directly or view it on GitHub
https://github.com/mila-udem/blocks-extras/issues/15#issuecomment-136585246
.

The relevant PR has been merged. There should be a development build of Bokeh out by the end of the week, or latest, early next week. In the mean time, you can take advantage of it by building from master (although building Bokeh is somewhat involved, it is relatively painless with Conda in your arsenal).

You can use the release here, or install with conda:
conda install bokeh==0.10.0dev.78671086 -c bokeh/channel/dev

which contains the fix.


Add testing support

Integration testing
Perf testing
Unit testing
Keep azure-core-test transitive/test dependency?

So far we have added unit testing for most of our classes and duplicated a few components from azure-core-test to help with HttpClient testing.


Update DefaultSettings.lua

what?

I don't think the DefaultSettings.lua should be on because it means it would apply for old loaders as well which couldn't disable them.
They should only be on for new loaders aka Settings.lua

Not that the PR is good, but surely that doesn't matter a whole deal. You're talking about loaders from 0000AD during the birth of Jesus Christ. It's safe to assume that the effects are negligable (If it wasn't really a bad PR in the first place).

@ccuser44

My thoughts on leaving anti-exploit enabled by default can be seen within other merge comments and on the Discord. Due to the variety of games Adonis is used in, enabling detections by default always leads to enough false positives that it's better to just leave them off by default and avoid the issue entirely. If someone is having an exploiter issue and wants to opt-in to game-related anti-exploit features (that is, something that exist to protect the game, not Adonis itself) then they can choose to do so.


AVRO-1717: Support for optional fields when converting json to avro

Currently there is an issue when we want to convert json object to avro using schema that allows optional fields (fields of type 'null'). When json object does not explicitly have such field with 'null' value then is treated as not conforming to schema. I added few test cases that illustrate this problem.

+1 for this. We are also facing the same issue.

Can someone look at this?

I have the same problem, +1 for merge this

+1  same issue here and this patch seems to solve it.

I have the same problem, +1 for merge this

This is a major blocker for us to adopt Avro. Please merge.

@horyna @erikvanoosten @maasg you can use our converter to tackle this issue https://github.com/allegro/json-avro-converter

Thanks. Unfortunately we already tried. We encountered https://github.com/allegro/json-avro-converter/issues/3.

I'm an original author of this library. I will look into this issue. Don't worry :-)

@wojtkiewicz  Great! Thanks :)


Goodwe 6000-DT,  E_total does not indicate the correct value.

In home assistant "sensor.goodwe_e_total" gives a value of 240,8 kWh but according to the semsportal it should be 13.348,10 kWh.
The value of "sensor.goodwe_e_day" seems good. There is no e_month sensor.
I cannot trace the value of e_total stated in Home assistant to something that is stated in the semsportal.

I've also installed the addon "goodwe Sems api to MQTT" and it gives me the following values:
Solar Goodwe produced 13.348,1 kWh
sems_battery_soc 200,0 %
sems_battery_soh -0,100000000000364 %
sems_grid_current 1,8 IAC
sems_grid_frequency 50,01 Hz
sems_grid_voltage 226,4 VAC
sems_inverter_status Online
sems_inverter_temperature 40,8 °C
sems_inverter_type GW6000-DT
sems_produced_this_month 704,6 kWh
sems_produced_today 4,1 kWh
sems_solar_power 1.213,0 W
sems_str_1_current 2,1 VDC
sems_str_1_voltage 322,8 VDC
sems_str_2_current 2,2 VDC
sems_str_2_voltage 327,2 VDC

How can i help you ?
Values of sensor.goodwe_inverter:
timestamp: '2021-08-31T11:12:32'
vpv1: 310.8
ipv1: 3.3
ppv1: 1026
vpv2: 317.2
ipv2: 3.4
ppv2: 1078
xx14: -1
xx16: -1
xx18: -1
xx20: -1
xx22: -1
xx24: -1
xx26: -1
xx28: -1
vline1: -0.1
vline2: -0.1
vline3: -0.1
vgrid1: 230.4
vgrid2: 228.2
vgrid3: 231
igrid1: 2.8
igrid2: 2.7
igrid3: 2.8
fgrid1: 49.99
fgrid2: 50.01
fgrid3: 49.99
pgrid1: 645
pgrid2: 616
pgrid3: 647
xx54: 0
ppv: 1910
work_mode: 1
work_mode_label: Normal
xx60: 0
xx62: 0
xx64: 0
xx66: -1
xx68: -1
xx70: -1
xx72: -1
xx74: -1
xx76: -1
xx78: 0
xx80: -1
temperature: 40.8
xx84: -1
xx86: -1
e_day: 4.8
xx90: 2
e_total: 241.8
xx94: 0
h_total: 8451
safety_country: 20
safety_country_label: Holland
xx100: 0
xx102: -1
xx104: 0
xx106: -1
xx108: 0
xx110: -1
xx112: 0
xx114: -1
xx116: -1
xx118: -1
xx120: -1
xx122: -1
funbit: 0
vbus: 602.9
vnbus: 306.1
xx130: -1
xx132: -1
xx134: 207
xx136: 351
xx138: 752 (This one seems the monthly produced, e_month?)
xx140: 0
xx142: 0
xx144: 100
model: GW6000-DT
serial_number: ***censored***
software_version: 22.22.10
unit_of_measurement: W
friendly_name: PV Inverter
icon: mdi:solar-power

xx138: 752 (This one seems the monthly produced, e_month?)

These two sems/mqtt values do not sound very reliable :-D
sems_battery_soc 200,0 %
sems_battery_soh -0,100000000000364 %
I'll check the code, problem might be we read just 2 bytes instead of 4 in e_total.

These two sems/mqtt values do not sound very reliable :-D
sems_battery_soc 200,0 %
sems_battery_soh -0,100000000000364 %

No, you are right, :-) I've no battery :-)

Indeed, if xx90 is included into e_total as upper two bytes, the e_total is 13349.0 kWh.
But is it realistic ? e_total/h_total = cca 1.8kW average permanent output ?
Maybe yes, although on sample data from XS inverter e_total will get from 201kWh to 13 198 950kWh.
That's odd. Either the sample data are wrong or DT uses 4 bytes and XS only 2 ??

Actually it would help if you can provide the raw hex value of the inverter data response (from the debug output of the plugin, see troubleshooting in readme how to enable it). I'll then include it to unit tests to ensure future correctness of the code ...

You mean this code?
2021-08-31 12:03:04 DEBUG (MainThread) [custom_components.goodwe.goodwe.protocol] Connection made to address ('<IP_ADDRESS>', 8899)
2021-08-31 12:03:04 DEBUG (MainThread) [custom_components.goodwe.goodwe.protocol] Sent: 7f0375940049d5c2
2021-08-31 12:03:05 DEBUG (MainThread) [custom_components.goodwe.goodwe.protocol] Received: aa557f039215081f0c03020c88001f0ca90020ffffffffffffffffffffffffffffffffffffffffffff08d008f90906001b001a001b1386138613860000072b0001000000000000ffffffffffffffffffffffff0000ffff019dffffffff003c0002097e0000210300140000ffff0000ffff0000ffff0000ffffffffffffffffffff0000177c0beeffffffff00cf016302f00000000000649f03

e_total by Sems portal = 13350.30 kWh
e_day by Sems portal = 6,10 kWh
ppv = 1.955kW

Exactly, that's what will help.
I'll fix the code, but I need to make XS specific tweak back to 2 bytes, so it I'll do it later evening...

Glad I could help. Thank you.

DT's e_total was fixed in v0.8.5


Ensure getParsedBody() return null if body is not parsed

PSR-7 states that getParsedBody() must return null, an array or an object. If we can't parse the body as there's no media type parser available, then return null.
Fixes #1807

:+1: looks good.

:+1:


How to use restoreStateFunction/restoreStateIdentifier

Hi, thanks for your awesome library, it's very useful in my project.
I am working on reconnecting device after the application has relaunched. Following the document, I see that we have 2 properties: restoreStateFunction/restoreStateIdentifier.
The snippet code:
  bleManager = new BleManager({
    restoreStateIdentifier: 'bleManagerRestoredState',
    restoreStateFunction: (bleRestoredState: BleRestoredState) => {
      if (bleRestoredState == null) {
        // BleManager was constructed for the first time.
      } else {
        // BleManager was restored. Check `bleRestoredState.connectedPeripherals` property.
      }
    },
  });

In the document, I didn't see how to use this value. I log the value and see that it is an array and contains device information from the last connected. However, how can we reconnect with this information?
Do we have another way to reconnect the device instead of rescanning and connecting as usual?

When you get the information in restored state that an interesting peripheral is connected by the OS the only thing you need is to connect it via the library. This call should resolve very quickly. No scanning is needed in this scenario.
Does it answer your question?

@dariuszseweryn
Thank you for your quick response.
Could you help me clarify a bit about "connect it via the library"?
I see that the library provides 2 ways to connect a device:

1/ using method from BleManager: connectToDevice (https://polidea.github.io/react-native-ble-plx/#blemanagerconnecttodevice)
2/ using method from Device: connect (https://polidea.github.io/react-native-ble-plx/#deviceconnect)
When using these methods, do we need to store the instance of BleManager?

To use the library one needs a BleManager instance. In the restore state you should get information which identifiers connected peripherals have. Then you call the adequate function on BleManager to connect the peripheral device.
I am not sure what is the problem you face

My goals: automatically reconnecting the device in case the application has already connected the device before
Current behavior: sometimes bleRestoredState is null or contains connectedPeripherals. It is unstable.
Also, in the document, I didn't see how to use bleRestoredState.connectedPeripherals value to reconnect the device.
Using bleRestoredState.connectedPeripherals[0].connect() or bleManagerInstance.connectToDevice(bleRestoredState.connectedPeripherals[0].id)?
That's why I asked this question.
Current my solution: stored the last connected device information, when app is relaunched, we have to scan and connect again based on that info.
@dariuszseweryn

I think that both calls should work.

When using these methods, do we need to reuse the old instance of BleManager which is created before the app is killed? (It means we have to store BleManager instance somewhere)
Or we simply use the new BleManager instance which returns the bleRestoredState.connectedPeripherals?

If this method is called your application has to have a BleManager instance created already (because otherwise you would not register the listener). Use the instance you have. An instance of BleManager should be created only once for each JS application start.

Thanks for your explanation @dariuszseweryn
Because after app is killed, relaunching the app, I think BleManager instance is gone and has to create again for each JS app.
So call these functions will not work as I think.
I will close this question here.

I think you see a chicken and an egg problem where there is none. Since restoreStateFunction is a JS function and the app was killed then BleManager has to be created again so the function can be called from the native part of the library. There is no magic there.
If the BleManager would be gone then the function would also be not attached (or non-existent) and thus- not callable.

@dariuszseweryn Great. I think I understand better now.

@nucleusdeveloper did you manager to get your devices connected with BleManager again? If yes, could you share you solution please?


Upgrade autoloader to PSR-4 standards

With PSR-0 standard being marked as 'deprecated' in favour of PSR-4, upgrade the autoloader file to PSR-4 compatability

Merged into master


[release-1.1] Manual backport of address incompatibility between IOThreads and unsupported buses

What this PR does
This is a manual cherry-pick of https://github.com/kubevirt/kubevirt/pull/11219
Libvirt only supports IOThreads in disks with virtio bus (see this). This behavior was unaddressed in Kubevirt and could lead to several unhandled cases, mainly, enabling dedicated IOThreads on a single, valid virtio disks would cause other non-virtio disks to error.

Fixes # https://issues.redhat.com/browse/CNV-34292
Why we need it and why it was done in this way
This Pull Request introduces changes such as:

Webhook rejection of disks with non-virtio bus explicitly setting dedicated IOThreads.

Using dedicated IOThreads in a single disk would mean enabling them for all remaining ones independently of their bus type. Now, only virtio disks will use IO threads when another disk enables them.

Improves handling of all SCSI disks with IOThreads, not only hotplugged volumes (see https://github.com/kubevirt/kubevirt/pull/6425).

While SCSI buses can still benefit from enabling a IOThread policy, USB and SATA have been left unhandled as their controllers don't support iothreads.
Release note

Bugfix: Improve handling of IOThreads with incompatible buses

/retest-required

/retest-required

/test pull-kubevirt-e2e-arm64-1.1
/test pull-kubevirt-goveralls-1.1

/test pull-kubevirt-goveralls-1.1

Wondering about the goveralls lane failure, the error The Table Body function expected 7 parameters but you passed in 6 doesn't make sense in 1.1 since we used 6 parameters in all entries for this unit tests. It changes to 7 in 1.2.

/lgtm
/approve

/hold
Unit tests and goveralls failing for some reason. I'll take a look.

/unhold

/retest

/test pull-kubevirt-e2e-k8s-1.27-sig-storage-1.1

/lgtm


Don't include default port on https

In https://github.com/pac4j/pac4j/blob/master/pac4j-core/src/main/java/org/pac4j/core/http/RelativeCallbackUrlResolver.java#L36 the port number will be inserted into the URL even if it is the default HTTPS port.  This logic should really be of the form:
If ((http AND port != DEFAULT_PORT) OR (https AND port != DEFAULT_HTTP_PORT))
The corresponding DEFAULT_HTTP_PORT constant would also need to be created in https://github.com/pac4j/pac4j/blob/master/pac4j-core/src/main/java/org/pac4j/core/context/HttpConstants.java

You're right. I just saw your PR: https://github.com/pac4j/pac4j/pull/481 which I'm going to review carefully.

Your PR targets the master. I think it would be great to also fix this for the 1.8.x branch (in another pull request), so that we can include this fix in the 1.8.6 release.
In any case, many thanks for your help!

Sorry, new to this.  To also fix 1.8.x branch do I branch the 1.8.x repo and manually re-apply the same changes there or is there a more correct way starting from this bugfix branch that is targeting master?

Generally, you can cherrypick (git command) the appropriate commit. That said, it might be tricky and we expect a slightly different solution in both branches. So the easiest way for you is to re-apply and update changes.

I didn't notice the need for a different solution between the branches (unless I missed something) but I still re-applied the changes manually since there are just a few.

I was wrong indeed, same solution for both branches. Thanks.
It will go in the 1.8.6 release.


Documentation website not found

Hi Pollen Robotics.
I just wanted to let you know that the documentation link on your Readme file leads to a 404 "There isn't a GitHub Pages site here. ".
"
For more information on how to get started: https://pollen-robotics.github.io/reachy-docs/
"
Is it supposed to go to the 2021 documentation page?
https://docs.pollen-robotics.com/
Or the 2019 documentation page?
https://pollen-robotics.github.io/reachy-2019-docs/

Thanks! I've fixed the link.


[SYSTEMML-2233]cleanup redundant error message logging

https://issues.apache.org/jira/browse/SYSTEMML-2233
This task aims to cleanup redundant and thus unnecessary error logging in the package org.apache.sysml.parser. Specifically, there are many instances where the same (copied) error message is logged via LOG.error and thrown as an exception, which eventually ends up in the log as well. We should delete the redundant logging via LOG.error for all instances that are immediately followed by a thrown exception. As a byproduct this cleanup also increases maintainability because these redundant messages would otherwise require consistent updates.

LGTM - thanks @Matsunaga-D. I'll merge it in once the tests on the PR builder passed.


Contact initialized

initialized Contact variable to empty object

@piyushcoader think I fixed this. Did you get it working?

yeah, i saw the code you changed... deleting the branch.. that should work i think..

yeah i saw the change in code... the fork was behind 4 commits


Build issues on ROCm with random_ltd extension

I saw this reported before, I am trying to install deepspeed suing the git repo
DS_BUILD_UTILS=1 DS_BUILD_CPU_ADAGRAD=1 DS_BUILD_RANDOM_LTD=1 DS_BUILD_CPU_ADAM=1 DS_BUILD_FUSED_ADAM=1 DS_BUILD_FUSED_LAMB=1 DS_BUILD_CCL_COMM=1 python setup.py build_ext -j80 bdist_wheel
I am hitting this error:
In file included from csrc/random_ltd/token_sort.hip:9:
In file included from csrc/includes/custom_hip_layers.h:17:
In file included from csrc/includes/context_hip.h:17:
csrc/includes/gemm_test_hip.h:312:13: warning: ignoring return value of function declared with 'nodiscard' attribute [-Wunused-result]
            hipDeviceSynchronize();
            ^~~~~~~~~~~~~~~~~~~~
csrc/includes/gemm_test_hip.h:306:13: warning: ignoring return value of function declared with 'nodiscard' attribute [-Wunused-result]
            hipDeviceSynchronize();
            ^~~~~~~~~~~~~~~~~~~~
csrc/includes/gemm_test_hip.h:222:24: note: in instantiation of function template specialization 'StridedGemmTest<__half>::Run<(lambda at csrc/includes/gemm_test_hip.h:222:35)>' requested here
        int algo_bw1 = Run(loops, [=](int algo) {
                       ^
csrc/includes/context_hip.h:160:53: note: in instantiation of member function 'StridedGemmTest<__half>::TestAlgo' requested here
            _gemm_algos.push_back(test_attn_scores->TestAlgo(100));
                                                    ^
In file included from csrc/random_ltd/token_sort.hip:9:
In file included from csrc/includes/custom_hip_layers.h:17:
In file included from csrc/includes/context_hip.h:17:
csrc/includes/gemm_test_hip.h:312:13: warning: ignoring return value of function declared with 'nodiscard' attribute [-Wunused-result]
            hipDeviceSynchronize();
            ^~~~~~~~~~~~~~~~~~~~
csrc/includes/gemm_test_hip.h:306:13: warning: ignoring return value of function declared with 'nodiscard' attribute [-Wunused-result]
            hipDeviceSynchronize();
            ^~~~~~~~~~~~~~~~~~~~
csrc/includes/gemm_test_hip.h:256:24: note: in instantiation of function template specialization 'StridedGemmTest<__half>::Run<(lambda at csrc/includes/gemm_test_hip.h:256:35)>' requested here
        int algo_bw2 = Run(loops, [=](int algo) {
                       ^
csrc/includes/context_hip.h:160:53: note: in instantiation of member function 'StridedGemmTest<__half>::TestAlgo' requested here
            _gemm_algos.push_back(test_attn_scores->TestAlgo(100));
                                                    ^
In file included from csrc/random_ltd/token_sort.hip:9:
In file included from csrc/includes/custom_hip_layers.h:17:
In file included from csrc/includes/context_hip.h:17:
csrc/includes/gemm_test_hip.h:312:13: warning: ignoring return value of function declared with 'nodiscard' attribute [-Wunused-result]
            hipDeviceSynchronize();
            ^~~~~~~~~~~~~~~~~~~~
17 warnings and 2 errors generated when compiling for gfx1030.
error: command '/opt/rocm-5.7.0/bin/hipcc' failed with exit code 1

Some info about my torch:
$ pip show torch
Name: torch
Version: 2.2.0.dev20231128+rocm5.7
Summary: Tensors and Dynamic neural networks in Python with strong GPU acceleration
Home-page: https://pytorch.org/
Author: PyTorch Team
Author-email<EMAIL_ADDRESS>License: BSD-3
Location: /env/rocm_57_p39_V2/lib/python3.9/site-packages
Requires: filelock, fsspec, jinja2, networkx, pytorch-triton-rocm, sympy, typing-extensions
Required-by: torchaudio, torchvision

FYI - @rraminen and @jithunnair-amd.
@Hobbes-Le-Chat - could you let us know what version of DeepSpeed you are using/share your ds_report?

@Hobbes-Le-Chat I don't think you actually captured the error snippet, all we see are warnings and then:
17 warnings and 2 errors generated when compiling for gfx1030.
error: command '/opt/rocm-5.7.0/bin/hipcc' failed with exit code 1

Can you please provide a snippet of the actual error in your log?

FYI - @rraminen and @jithunnair-amd.
@Hobbes-Le-Chat - could you let us know what version of DeepSpeed you are using/share your ds_report?

after removing DS_BUILD_RANDOM_LTD=1
$ DS_BUILD_UTILS=1 DS_BUILD_CPU_ADAGRAD=1 DS_BUILD_CPU_ADAM=1 DS_BUILD_FUSED_ADAM=1 DS_BUILD_FUSED_LAMB=1 DS_BUILD_CCL_COMM=1 python setup.py build_ext -j80 bdist_wheel

$ ds_report
[2023-11-29 18:25:08,359] [INFO] [real_accelerator.py:161:get_accelerator] Setting ds_accelerator to cuda (auto detect)
--------------------------------------------------
DeepSpeed C++/CUDA extension op report
--------------------------------------------------
NOTE: Ops not installed will be just-in-time (JIT) compiled at
      runtime if needed. Op compatibility means that your system
      meet the required dependencies to JIT install the op.
--------------------------------------------------
JIT compiled ops requires ninja
ninja .................. [OKAY]
--------------------------------------------------
op name ................ installed .. compatible
--------------------------------------------------
 [WARNING]  async_io requires the dev libaio .so object and headers but these were not found.
 [WARNING]  If libaio is already installed (perhaps from source), try setting the CFLAGS and LDFLAGS environment variables to where it can be found.
async_io ............... [NO] ....... [NO]
fused_adam ............. [YES] ...... [OKAY]
cpu_adam ............... [YES] ...... [OKAY]
cpu_adagrad ............ [YES] ...... [OKAY]
cpu_lion ............... [NO] ....... [OKAY]
 [WARNING]  Please specify the CUTLASS repo directory as environment variable $CUTLASS_PATH
evoformer_attn ......... [NO] ....... [NO]
fused_lamb ............. [YES] ...... [OKAY]
fused_lion ............. [NO] ....... [OKAY]
inference_core_ops ..... [NO] ....... [OKAY]
cutlass_ops ............ [NO] ....... [OKAY]
quantizer .............. [NO] ....... [OKAY]
ragged_device_ops ...... [NO] ....... [OKAY]
ragged_ops ............. [NO] ....... [OKAY]
random_ltd ............. [NO] ....... [OKAY]
 [WARNING]  sparse_attn is not compatible with ROCM
sparse_attn ............ [NO] ....... [NO]
spatial_inference ...... [NO] ....... [OKAY]
transformer ............ [NO] ....... [OKAY]
stochastic_transformer . [NO] ....... [OKAY]
transformer_inference .. [NO] ....... [OKAY]
--------------------------------------------------
DeepSpeed general environment info:
torch install path ............... ['/rocm_57_p39_V2/lib/python3.9/site-packages/torch']
torch version .................... 2.2.0.dev20231128+rocm5.7
deepspeed install path ........... ['/rocm_57_p39_V2/lib/python3.9/site-packages/deepspeed']
deepspeed info ................... 0.12.4+8640b8e5, 8640b8e5, master
torch cuda version ............... None
torch hip version ................ 5.7.31921-d1770ee1b
nvcc version ..................... None
deepspeed wheel compiled w. ...... torch 2.2, hip 5.7
shared memory (/dev/shm) size .... 251.35 GB

@Hobbes-Le-Chat I don't think you actually captured the error snippet, all we see are warnings and then:
17 warnings and 2 errors generated when compiling for gfx1030.
error: command '/opt/rocm-5.7.0/bin/hipcc' failed with exit code 1

Can you please provide a snippet of the actual error in your log?

Where can I find the log file?

@Hobbes-Le-Chat - does this mean that you do not see the error when removing the DS_BUILD_RANDOM_LTD=1?
And if you can just copy/paste the whole output from your console, that should have the error.

@Hobbes-Le-Chat -
That's better, @jithunnair-amd these look to be the errors:
/opt/rocm-5.7.0/include/hip/amd_detail/amd_hip_cooperative_groups.h:638:3: error: static assertion failed due to requirement 'integral_constant<bool, false>::value': Tile size is either not a power of 2 or greater than the wavefront size

/opt/rocm-5.7.0/include/hip/amd_detail/amd_hip_cooperative_groups.h:723:18: error: no member named 'sync' in 'cooperative_groups::thread_block_tile_base<64>'

I'm not sure if we've tested everything on our side with ROCm 5.7 or not yet though.

This is a known issue that we are looking into and as a work around comment the asserts in /opt/rocm-5.7.0/include/hip/amd_detail/amd_hip_cooperative_groups.h like below:
637 template <unsigned int size> class thread_block_tile_base : public tile_base<size> {
638   //static_assert(is_valid_tile_size<size>::value,
639   //              "Tile size is either not a power of 2 or greater than the wavefront size");
640   using tile_base<size>::numThreads;

836 __CG_QUALIFIER__ thread_block_tile<size, ParentCGTy> tiled_partition(const ParentCGTy& g) {
837   //static_assert(is_valid_tile_size<size>::value,
838   //              "Tiled partition with size > wavefront size. Currently not supported ");
839   return impl::tiled_partition_internal<size, ParentCGTy>(g);
840 }

@Hobbes-Le-Chat Thanks, the log file helped!
Btw, I think you should update the title of this issue to "Build issues on ROCm with random_ltd extension" or something, since I don't think HIP_PLATFORM_AMD change has anything to do with the issues you're seeing.

@Hobbes-Le-Chat - I believe the other workaround would be to use a different version of ROCm if that's possible in this setup?

@Hobbes-Le-Chat - I believe the other workaround would be to use a different version of ROCm if that's possible in this setup?

A different version as a newer ROCm version or older?

I believe older - though do you need the random_ltd op for what you are doing?

I believe older - though do you need the random_ltd op for what you are doing?

No, I don't need the random_ltd for I am doing. I was testing that most of the options worked for future users that may do AI applications on the cluster.

I see - I suspect then that you are better off either disabling the DS_BUILD_RANDOM_LTD op for now, and users will at least see it as unavailable in their ds_report, or waiting for an updated ROCm package.

@Hobbes-Le-Chat - anything else we can try to help on this issue?  Or should we close it for now?

@Hobbes-Le-Chat - anything else we can try to help on this issue? Or should we close it for now?

Well, I think we can close it. are there any plans to support DS_BUILD_RANDOM_LTD for rocm?

That's a question for @rraminen and @jithunnair-amd, but I would guess it just needs a newer version of ROCm.  If any DeepSpeed features are needed, we will update those too, so yes, there will be support, it just isn't there yet.

@Hobbes-Le-Chat , yes, we will be working on enabling DS_BUILD_RANDOM_LTD on ROCm.

@rraminen and @jithunnair-amd - any ideas if this is fixed with ROCm 6.0.0?


nested tactics are broken after the lean-v4.9.0 upgrade

For example, the output of lake exe Animate Input/Examples.lean foo2 is wrong now.

Fixed in 04f473f74dd4c232835efea52ad6f4e3b7963882.
The spans are still a little bit imprecise on have := by ... .


Clang warning: possible misuse of comma operator here [-Wcomma]

This line issues a clang warning when compiling with -Wcomma: https://github.com/skypjack/entt/blob/master/src/entt/meta/meta.hpp#L1803C32-L1803C32
meta_iterator operator++(int) noexcept {
    meta_iterator orig = *this;
    return ++(*this), orig;
}

I'm assuming this is fixed by replacing the increment with vtable(operation::incr, handle, nullptr); or similar.

What version of clang is it? What version of EnTT? What flags are you using?
The library is widely tested on the CI and I never hit this warning to be honest.

I'm using the most recent version and clang 16 with C++20. I did try to reproduce this in compiler explorer with no luck. I'm also using the single include file. For the flags, we do use -Weverything without some stuff:
-Werror -Weverything -Wno-abstract-vbase-init -Wno-c++20-compat -Wno-c++98-compat-pedantic -Wno-covered-switch-default -Wno-ctad-maybe-unsupported -Wno-date-time -Wno-deprecated-copy-with-dtor -Wno-deprecated-copy-with-user-provided-dtor -Wno-disabled-macro-expansion -Wno-double-promotion -Wno-exit-time-destructors -Wno-extra-semi-stmt -Wno-float-conversion -Wno-float-equal -Wno-global-constructors -Wno-gnu-line-marker -Wno-implicit-int-conversion -Wno-implicit-int-float-conversion -Wno-invalid-noreturn -Wno-language-extension-token -Wno-missing-noreturn -Wno-missing-prototypes -Wno-padded -Wno-parentheses -Wno-range-loop-bind-reference -Wno-reserved-identifier -Wno-reserved-macro-identifier -Wno-shadow-all -Wno-shorten-64-to-32 -Wno-sign-compare -Wno-sign-conversion -Wno-switch-enum -Wno-undef -Wno-undefined-func-template -Wno-undefined-reinterpret-cast -Wno-unused-template -Wno-unsafe-buffer-usage -Wno-weak-template-vtables -Wno-weak-vtables

What I don't get is why it returns this error while all other uses of the comma operator are fine.
 Do you get a warning only in this case?

I do hit other -Wcomma warnings, as well as some other stuff. Because we're using warnings as errors, I'm assuming not every warning is being reported, it stops with just a few warnings. Since I wasn't the one actually utilizing the lib, I was a bit short on the context. In the meantime, I compiled the project and these were the warnings emitted:

So, there are also -Wdocumentation warnings in the mix. But probably these aren't the only ones. It just happens to stop compiling at this point.
These are all simple to fix though, I wouldn't mind submitting a PR if you agree it should be fixed.

What version of EnTT is it? I have -Wdocumentation enabled and I can't find these errors anywhere.
Before submitting a PR, you should probably update to the latest version at least.

What version of EnTT is it?

We're using the single_include file (from master) because it was easier to integrate with our project. I was under the assumption that the file was updated to the latest version, but looking at the commit I was wrong. It matches the date from v3.11.0 release (and taking a quick look inside the file it is indeed v3.11.0).  So, it is an outdated version. Looking at some of these documentation issues in the most recent version, it seems they have been fixed, such as:
https://github.com/skypjack/entt/blob/cb974bf5671c757cbce2e0248787bad616b06bf9/src/entt/core/memory.hpp#L107-L112

Yup, I've tried with the single_include file from v3.12.2 and I had no documentation warnings.

Btw, for a quick patch in 3.12.2 should we do
meta_iterator operator++(int) noexcept {
      meta_iterator orig = *this;
      vtable(operation::incr, handle, nullptr);
      return orig;
  }

The fix in wip https://github.com/skypjack/entt/commit/d08a07044cd2eb9f0e38952dde573a745f083c0d is not applicable since the signature is different, so I'd like to confirm what the equivalent in 3.12.2 is.

Technically speaking, it's not a bug nor a blocking issue. I don't cut releases in this cases. However, I'm not opposed to doing it if you need it.
Note that you can regenerate the single include file on your own if that's what you need though. See the scripts dir. 👍

Ok, for the time being, we can just do the patch locally. Thank you for your help with this :slightly_smiling_face:

You're welcome. Thanks for pointing this out. 👍


Radio cannot be without the selected value

observation: by default(when a radio group is dropped and no changes are made) accessibility is fine but when I am creating custom options like the number options in the radio group accessibility seems to be lost.
cc: @KelvinOm @jsartisan

This is fixed here - https://github.com/appsmithorg/appsmith/pull/31382


Porto Alegre, Brazil

Address ranges for the city of Porto Alegre are available here (see here for more details, in Portuguese).
The most relevant metadata fields are the following:

NOME_LOG street name
INIC initial number in the segment
FIN final number in the segment

There are separate lines for the even and odd sides of the street. There is no zipcode data.
Also, street names contain many abbreviation, have all diacritis supressed, and are all caps ("ESTR     JOAO ANTONIO SILVEIRA" instead of "Estrada João Antônio Silveira"). I know this is kind of bad, but I wanted to remark that similar data quality issues are very likely to be present as other Brazilian cities release their data.  Also, I wouldn't expect individual addressing data to be available, only ranges.  So, if openaddresses plans to support Brazil at all, these are issues that will need to be dealt with...
Data is provided by the municipal government under the ODbL.

Address ranges no longer interesting


Why are responses that ChatGPT refuses to answer considered as hallucinations?

Dear authors,
I found that in some cases where ChatGPT refused to answer, you were labeled as hallucinated.
The following is a case from the General QA subset.
{"ID": "815", "user_query": "Describe the painting using 3 adjectives.\n[painting.jpg]", "chatgpt_response": "As an AI language model, I do not have access to the picture of your choice, please provide me with a description of the painting so that I can offer 3 adjectives that match your description.", "hallucination": "yes", "hallucination_spans": ["As an AI language model, I do not have access to the picture of your choice, please provide me with a description of the painting so that I can offer 3 adjectives that match your description."]}
Would you please describe why these cases are considered as hallucination?
Thanks.

We are sorry for that. General QA dataset is labeled by humans, so there might be few little mistakes which are not checked by us. You can delete these cases from the datasets.


'Jobs' object has no attribute 'wait_for_job'

Describe the bug
Hi,
I got the error 【'Jobs' object has no attribute 'wait_for_job'】 when I ran my code,would you tell me how to solve it?
Versions
Details of your environment, including:

Tableau Server version (or note if using Tableau Online)

Tableau Online

Python version

Python 3.7

TSC library version

3.13(I also tried 3.1 and 3.5 but got the same error)

To Reproduce
Steps to reproduce the behavior. Please include a code snippet where possible.
    tableau_auth = TSC.TableauAuth(user_name, password, site_id=site_id)
    server = TSC.Server(site_url)

    server.version = '3.13'
    server.auth.sign_in(tableau_auth)

    get_info_for_project_id = [proj for proj in TSC.Pager(server.projects) if proj.name == 'lt_bigquery_test']
    project_id = get_info_for_project_id[0].id

    with server.auth.sign_in(tableau_auth):
        for datasource in TSC.Pager(server.datasources.get):
            if datasource.project_name == 'lt_bigquery_test':
                datasource = server.datasources.get_by_id(datasource.id)
                refreshed_datasource = server.datasources.refresh(datasource)
                print("Waiting for job...")
                job = server.jobs.wait_for_job(refreshed_datasource)
                print("Job finished succesfully")

Results
What are the results or error messages received?
Waiting for job...
Traceback (most recent call last):
  File "/Users/reika.yo/dev/lt-datainfra-etl/workflows/ltsf2BQ/refresh_tableau_datasource_new.py", line 49, in <module>
    refresh_tableau_datasource()
  File "/Users/reika.yo/dev/lt-datainfra-etl/workflows/ltsf2BQ/refresh_tableau_datasource_new.py", line 45, in refresh_tableau_datasource
    job = server.jobs.wait_for_job(refreshed_datasource)

NOTE: Be careful not to post user names, passwords, auth tokens or any other private or sensitive information.

TSC version is not the same as Tableau Server version. The latest TSC version is 0.17. wait_for_job is only available in 0.17 and later. Fix the issue by running pip install --upgrade tableauserverclient.

You also want to consider using server = TSC.Server(site_url, True) so that TSC automatically fetches and sets the API version, rather than you manually setting it.


[ECS] FinderSanitizer bug on single file

Expected

should process the file

Workaround
vendor/bin/ecs check ./false-positives.php

Bug from version 4.8, alraedy fixed with decoupled FinderSanitizer


No location updates after first one

I'm only able to get the first location from the listener in ANDROID only
I/flutter (19669): locationUpdate: {lat: 32.2333, lon: -5.53321, alt: 0.0, vertical_accuracy: 0.5, horizontal_accuracy: 5.0, course: 36.11000061035156, course_accuracy: 30.0, speed: 23.113969802856445, speed_accuracy: 0.5, logging_enabled: true} I/flutter (19669): Location Update: Lat: 35.84597 Lon: -5.54432 D/BackgroundManager(19669): Got success, destroy engine!
And after the Got success, no location is received
I tried it on your example code without changing anything and same result I tried updating Google Play service location to version 21 and still the same thing

UPDATE:
On an emulator running SDK 33 I've got this and I think it comes from Geolocator dependency. It's some sort of conflict between both packages
I/flutter (13881): locationUpdate: {lat: 35.8527348, lon: -5.5378524, alt: 0.0, vertical_accuracy: 0.5, horizontal_accuracy: 5.0, course: 38.32645034790039, course_accuracy: 30.0, speed: 21.961938858032227, speed_accuracy: 0.5, logging_enabled: true}
D/BackgroundManager(13881): Got success, destroy engine!
D/FlutterGeolocator(13881): Disposing Geolocator services
E/FlutterGeolocator(13881): Geolocator position updates stopped
D/FlutterGeolocator(13881): Stopping location service.

There is another issue related to the geolocator package, maybe they are related?
https://github.com/icapps/flutter-background-location-tracker/issues/43

Unfortunately, no the app isn't crashing due to a conflict of the google play service, it's working but when running alongside Geolocator it seems that one is trying to over-run the other

@oussamaniba  Can you please check https://github.com/icapps/flutter-background-location-tracker/issues/43 as well, It would be great if this package can support latest  play-services:location version

I'm having the same issue. In Android no location updates after the first one... In iOS everything works.
Is there any workaround to solve this?

https://github.com/Yukams/background_locator_fixed/issues/64#issuecomment-1459845605
I have put a solution here and I forgot to add here maybe or my comment was deleted idk
Anyway this is a working solution for me and I think for you too

@oussamaniba Are you still having these issues? There was indeed an issue with the play services, that has been fixed

Yeah it was solved, using two packages one for each platform, but thanks anyways!


Create 404 page

Overview
Build a 404 page as shown in Figma.
Action Items

[ ] Create the 404 or Not Found page using our Material-UI best practices. Probably use import ErrorOutlineRoundedIcon from '@material-ui/icons/ErrorOutlineRounded';. Maybe base it off the Placeholder page.
[ ] Add it into the site in App.js.

Resources/Instructions
https://www.figma.com/file/EFoSuj1b9G4aZ7bN8OX8tf/?node-id=4304%3A1

Progress: 90% complete. Error behavior is in.
Blocks: Familiarity with javascript and how to override font size and background color. See below for current appearance at max width.
ETA: 3/11 once blocks resolved during meeting.


[WIP] Allow interaction with ray client service via Route from outside of OCP cluster

How to test:
# clone the branch
git clone --single-branch --branch ray-loca-interaction https://github.com/tedhtchang/codeflare-sdk.git
cd codeflare-sdk/

# create python virtual env with [pyenv](https://github.com/pyenv/pyenv#automatic-installer)
pyenv install 3.8.13
pyenv virtualenv 3.8.13 localinter
pyenv local localinter

# Install python packages
pip install -r requirements-dev.txt
pip install jupyterlab
pip install -e .

# Run a sameple noteook and then Run -> Run all cells
jupyter lab demo-notebooks/interactive/local_hf_interactive.ipynb

# clean up
pyenv uninstall localinter
# delete the cloned repo

Expect something

@MichaelClifford I passed all the tests. looks good. I also renamed the example notebook.

Thanks for adding more tests @tedhtchang
LGTM

@tedhtchang, with the latest code with the Ray Operator v0.5.0, the worker is now able to start all 3 init containers and that's working ok.  But when I try to do the import ray step I'm getting:
File ~/.pyenv/versions/3.8.13/envs/localinter/lib/python3.8/site-packages/ray/util/client/worker.py:260, in Worker._connect_channel(self, reconnecting)
    252 if log_once("ray_client_security_groups"):
    253     warnings.warn(
    254         "Ray Client connection timed out. Ensure that "
    255         "the Ray Client port on the head node is reachable "
   (...)
    258         "more information."
    259     )
--> 260 raise ConnectionError("ray client connection timeout")

ConnectionError: ray client connection timeout

Tested on my own mac and another Ubuntu VM from scratch. Both env seems to work. @MichaelClifford @Maxusmusti Could you test again?

I tried one last time last night and local interaction worked on my cluster to default namespace from my Mac.  I'm not certain why it worked this time, but perhaps a new cluster and/or some local changes on my Mac... not sure.
Was also trying to test from an Ubuntu image, but I ran out of time trying to setup my Ubuntu with the proper prereqs.  Last thing Ted wanted me to try was a different namespace other than default.   Will post update here once my cluster rebuilds and I can test.

OK - tried on the IBM network with both default and jim namespaces - looked good!


Is there a way to enter login credentials in version 0.36.0 (windows x64)?

Hello,
I am trying to connect to a Minecraft Server v(1.11.2) in a local Docker container, but am getting this error :
[14:40:08] [MCO Compatability Checker #1/INFO]: Could not authorize you against Realms server: Invalid session id
[14:40:08] [MCO Compatability Checker #1/ERROR]: Couldn't connect to realms:
[14:50:26] [Client thread/INFO]: Connecting to <IP_ADDRESS>, 25565
[14:50:44] [Client thread/INFO]: Connecting to <IP_ADDRESS>, 25565
[14:53:59] [Thread-12/INFO]: [STDOUT]: INFO: ->mcp(0) Socket exception - usually caused by ServerSocket being closed under our feet (normal for stopping polling): java.net.SocketException: socket closed
I am assuming that I need a valid Minecraft account to do so, but see nowhere to enter credentials. There was a previous issue (https://github.com/microsoft/malmo/issues/408) that was resolved by saying that one could enter credentials in to the Malmo Mod UI screen, but there are no login fields there or in the associated config.

Is the only way to use the multi-agent script?
I also see that one can simply place the Malmo Mod into a valid standard distribution Minecraft Client as in this issue https://github.com/microsoft/malmo/issues/737. Perhaps that is the appropriate method?
thanks

I ended up getting a real Minecraft ID and launching from installer after downloading Forge, this allowed me to connect to a local docker container running Minecraft Server ...

@Bulbousonions13  Hi, were you able to run the multi-agent examples?


Need to create a SiteMap

I saw your website, it's fabulous. And as it is big, it should have the SiteMap.
I will make a sitemap.xml and sitemap.html for the website.

Can you please assign this issue to me as a part of GSSOC.

@chaitanyashimpi  yes you can work on it. We just need an idea like how you will proceed. As we are redisigning everything so everything will be changes and so will the site structure. We just need to know how you are going to implement it. Prepare a prototype of code and send it within this week. Best of luck!

@himanshu007-creator @divyashukla23 I have made a Pull request, check it out. And I have tried my best to explain the process of how you can update it in the future.

@chaitanyashimpi i will check it and update soon.

@chaitanyashimpi , please wait for some time. Sitemap will be meaningful once the frontend work is over. Then you can help yourself and send a new PR

Same issue as  #262  !  @himanshu007-creator


remove old url dependency

i come across some hard-coded urls that contain old PATRIC urls.
We need to replace with new url pattern.
$ grep -r "/portal/portal/patric" *
app/p3app.js:			xhr.post("/portal/portal/patric/GenomicFeature/GenomicFeatureWindow?action=b&cacheability=PAGE", {
app/p3app.js:				document.location = "/portal/portal/patric/GlobalSearch?cType=taxon&cId=131567&dm=&pk=" + results;
widget/WorkspaceBrowser.js:				xhr.post("/portal/portal/patric/FIGfam/FIGfamWindow?action=b&cacheability=PAGE", {
widget/WorkspaceBrowser.js:					Topic.publish("/navigate", {href: "/portal/portal/patric/MSA?cType=&cId=&pk=" + results});
widget/WorkspaceBrowser.js:				xhr.post("/portal/portal/patric/IDMapping/IDMappingWindow?action=b&cacheability=PAGE", {
widget/WorkspaceBrowser.js:					Topic.publish("/navigate", {href: "/portal/portal/patric/IDMapping?cType=taxon&cId=131567&dm=result&pk=" + results});
widget/WorkspaceBrowser.js:				xhr.post("/portal/portal/patric/TranscriptomicsEnrichment/TranscriptomicsEnrichmentWindow?action=b&cacheability=PAGE", {
widget/WorkspaceBrowser.js:					Topic.publish("/navigate", {href: "/portal/portal/patric/TranscriptomicsEnrichment?cType=taxon&cId=131567&pk=" + results});

harry, i don't have knowledge on what should be used to replace those hard-coded urls, can you give me an example or more details? i know this might take you longer time than fixing it directly, but i wanted to learn and go through the files if you don't mind. thanks.

each url is mapped to a tab or service

/portal/portal/patric/GenomicFeature -> features tab in genome or taxon
/portal/portal/patric/GlobalSearch -> global search
/portal/portal/patric/FIGfam/ -> protein families tab in genome or taxon
/portal/portal/patric/MSA -> MSA
/portal/portal/patric/IDMapping/ -> services -> IDMapping
/portal/portal/patric/TranscriptomicsEnrichment/ -> transcriptomics tab (?)

before we mechanically change this, I suggest to track down where we are using these, and if we don't use it, i would like to remove them.
@dawenx would you track down this ?

I will look into it.

harry, i made change in app/p3app.js and pushed my change to my fork.
i have hard time understanding what xhr.post() was doing in WorkspaceBrowser.js, i also don't have much testing data in my workspace to even see MSA, Pathway Summary and IDMapping action icons, can you take over fixing WorkspaceBrowser.js? thanks!


Combined PR

Combined PR to check for interactions.
https://github.com/wildfly/wildfly-core/pull/4196
https://github.com/wildfly/wildfly-core/pull/4024

One PR merged, the other will be merged independently.


Dcopeland feature terrain builder

Fixed my merge conflicts. I lost some changes when I pulled as my original rebase was incorrect. So I still have more changes to restore.


Enable Multicore for other request types

At present only the DataProcessing Request type includes support for Multicore controls, exposing the "Multicore" parameter.
This should be selectively enabled for other Request types that need to use this mode of operation as it is now supported in StdBase.
This should just be a case of supporting the setting in the Request Type, the underlying code will then set up all processing steps as Multicore jobs with the setting provided.
At present, we distiguish Processing from MultiProcessing in the Task type, so some adjustment may be needed.  (Eg would ReReco become MultiReReco etc)
Input needed on:

Which Requests types to enable Multicore for
Wether this impacts the task type for those Requests

Irrelevant now

One for Throwback Thursday...


fpga: add prefix to FPGA resource name

Added mode ("af" or "region") prefix to the resource name to
distingush between announced functions and regions, e.g.
intel.com/fpga-af-f7df405cbd7acf7222f144b0b93acd18
intel.com/fpga-region-ce48969398f05f33946d560708be108a

I don't like hardcoded string in multiple places. we should be able to re-use same string constants for flag values validation and in resource name


Step 1: Create an application in Azure Active Directory (Azure AD) - needs rewriting due to portal makeover

This step does not reflect current UI in AAD portal anymore. I tried to figure out the right steps myself but subsequent steps fail and now i'm not sure if i've done anything wrong :)
https://docs.microsoft.com/nl-nl/windows/security/threat-protection/microsoft-defender-atp/configure-mssp-support#step-1-create-an-application-in-azure-active-directory-azure-ad

Documentdetails
⚠ Dit gedeelte niet bewerken. Het is vereist om problemen te koppelen tussen docs.microsoft.com en GitHub.

ID: 79be1cc1-eaf6-15b7-d2fa-696fc4c5a709
Version Independent ID: 66a6394c-a351-dee8-6f7d-81cf2634d1ef
Content: Configure managed security service provider support
Content Source: windows/security/threat-protection/microsoft-defender-atp/configure-mssp-support.md
Product: w10
Technology: windows
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@officedocsbot assign @e0i

Hello, @rwx-mrx
The content is being reviewed in the light of your feedback and we'll keep you posted on updates.
Thank you for providing feedback.

@rwx-mrx
Upon your feedback, We have updated the content with relevant changes accordingly. Thanks.

@officedocsbot close


[WIP] Traces: Use OpenTelemetry

What is OpenTelemetry?

open standard
sandbox CNCF project.
formed through a merger of the OpenTracing and OpenCensus.

OpenCensus and OpenTracing have merged to form OpenTelemetry, which serves as the next major version of OpenCensus and OpenTracing. OpenTelemetry will offer backwards compatibility with existing OpenCensus integrations, and we will continue to make security patches to existing OpenCensus libraries for two years.

There is a lot of outdated documentation as a result of this merger, and we think adopting the latest standard and libraries will benefit the project.

@labkode @ishank011 does it make sense I update all traces occurrences that use the go.opencensus.io lib? I see the EOS storage provider is rich with traces and metrics, we could tackle that in this PR or in a follow-up one. How does that sound?
There are still a few technicalities to solve, but I'd say this PR is in good track.

Hi @refs, yes we can take the EOS part later. I think we should go for OT as default.

@labkode I think the current state should be ready for review. However I still need to wrap my head around the metrics instrumentation.


[Question] World colour name

I have the end dimension and idk what the colour name is for &5...
What is the colour name?
Example: &c = RED

For mv colour codes: https://minecraft.tools/en/color-code.php
To set worldname alias of different colours: https://github.com/Multiverse/Multiverse-Core/wiki/World-properties#alias

Thanks!


[Bug] : Remove onRowUpdate event in table docs

We don't support this as of now and this gives a wrong impression to the end user that we do.
https://docs.appsmith.com/widget-reference/table#events

Docs updated.


Support passing in --watcher to pass to ember-cli

Update app.rb to support passing in --watcher THEPOLLER
For example for those of us developing in Vagrant setting this to polling as detailed here http://www.ember-cli.com/#watched-files will really help the app to see the file changes made

We're sorry that this issue has sat for so long.
The project is now maintained by thoughtbot, and we're declaring an issue bankruptcy of sorts.
We're closing this issue. If what you've described is still a problem, please comment on this issue and we'll reopen.
Thanks for your patience.


Firefox supports CSS Containment behind preference

Firefox supports CSS Containment behind "css.layout.contain.enabled" true or false since Firefox 47.
I tried to edit it, but I guess it failed because I am not authenticated to do that. (I'm new to Github tho, so something else could be the error.)
When I edited it, I put:

      "47":"n d #2",
      "48":"n d #2",
      "49":"n d #2",
      "50":"n d #2",

and created a note:

"2":"Can be enabled through the \"layout.css.contain.enabled\" preference in about:config"

I believe this issue is obsolete, given the age but also because https://caniuse.com/css-containment by now shows what you requested:

Closing as per above.


Crashing on startup on 2017 iMac macOS 11.0 Beta (20A5354i)

terminal output when running the executable directly:
dyld: Symbol not found: _$s7SwiftUI12ProgressViewVyxGAA0D0AAMc
  Referenced from: /Applications/Curiosity.app/Contents/MacOS/./Curiosity
  Expected in: /System/Library/Frameworks/SwiftUI.framework/Versions/A/SwiftUI
 in /Applications/Curiosity.app/Contents/MacOS/./Curiosity
zsh: abort      ./Curiosity

Nevermind, didn't have XCode 12 beta installed, will likely work after I did (12GB to download wth)

Installed XCode Beta 12, still getting the same error:
Termination Reason:    DYLD, [0x4] Symbol missing
Application Specific Information:
dyld: launch, loading dependent libraries
Dyld Error Message:
Symbol not found: _$s7SwiftUI12ProgressViewVyxGAA0D0AAMc
Referenced from: /Applications/Curiosity.app/Contents/MacOS/Curiosity
Expected in: /System/Library/Frameworks/SwiftUI.framework/Versions/A/SwiftUI
in /Applications/Curiosity.app/Contents/MacOS/Curiosity

I ran into this issue today. This maybe related to an issue mentioned in the macOS Big Sur 11 Beta 6 Release Notes:
ProgressView generic type signature has changed, adding the currentValueLabel generic parameter. This change doesn’t require any source changes, but will cause apps compiled using an earlier beta SDK to quit unexpectedly. (63580200)

Workaround: Re-compile apps for macOS Big Sur 11 Beta 6 or later to resolve this issue.

BTW, I got tons of compilation errors trying to build this project in xcode 12.0 beta3:

Fixed with the new 0.1 release and in Big Sur beta 20A5374i


Only one level of includes is allowed

Describe the bug
With latest version (0.53.0) it seems that terragrunt only accept one level of includes.
We have terragrunt.hcl in root and in each module subfolder. It seems when we run a terragrunt run-all plan the first run works as intended and generate cache. If we want to run terragrunt run-all plan again the terragrunt.hcl present in cache call the terragrunt.hcl in module subfolder and this one call the terragrunt.hcl in root but it fail as only one level of includes is allowed so we can not run multiple terragrunt run-all plan without cleaning cache.
'[...]/deployment/terragrunt/bootstrap/agents/.terragrunt-cache/[unique-cache-id]/[cache-subfolder]/terragrunt.hcl'. How this module was found: Terragrunt config file found in a subdirectory of [...]/deployment/terragrunt/bootstrap. Underlying error: [...]/deployment/terragrunt/bootstrap/agents/.terragrunt-cache/[unique-cache-id]/[cache-subfolder]/terragrunt.hcl includes [...]/deployment/terragrunt/bootstrap/agents/terragrunt.hcl, which itself includes [...]/deployment/terragrunt/bootstrap/agents/terragrunt.hcl. Only one level of includes is allowed.
To Reproduce
We use in our terragrunt.hcl located in our module subfolder.
include "root" {
path = find_in_parent_folders()
}
FYI when we use the version 0.50.14 we can run multiple plan without any issue, i do not know exactly from which version our issue appear but it persist on the latest 0.53.0 (and i do not see anything in changelog related to that)
// paste code snippets here

Expected behavior
Accept multiple include (i guess it is the case for the version 0.50.14 as it works fine)
Nice to have

[ ] Terminal output
[ ] Screenshots

Versions

Terragrunt version: 0.53.0
Terraform version: 1.5.7
Environment details (Ubuntu 20.04, Windows 10, etc.): Win10

Additional context
Add any other context about the problem here.

@denis256 Do you have an idea about the issue ?


Cannot connect to postgres without mapping port

Howdy, I'm a novice Docker user kludging my way through learning. Was very excited to discover this repo that features a clean implementation of Rails + Docker w/ best practices; thank you!
One issue I encountered what that I could not access/connect to my running Postgres container using the standard config in your docker-compose.yml file. However, once I manually mapped the port, everything worked fine.
Specifically, I had to add the following under the postgres service in docker-compose.yml:
ports:
  - 5432:5432

I assume your implementation works great for you out of the box yet not on my workstation (2021 M1 MacBook Pro). Any thoughts as to why I had to manually map the port in the config?

Hi,
This issue was covered in https://github.com/nickjj/docker-rails-example/issues/42.


Remove special field from song model

Move solo/feature for SongInPlaylist model and use that to override any time limit

closed by pull request #63


Latest commit crashes

Seems like layersurfaces, much like the infamous 2-week crash before.
Hyprland log: https://linxer.org/uploads/view.php?name=23r89io
Coredump: https://linxer.org/uploads/view.php?name=3908rqji

try with another latest

Looks to be fixed, ty


File indexing for search so that contents of the files can be searched to like with Spotlight Search on Mac. Alongside an inbuilt Dictionary

File indexing for search so that contents of the files can be searched to like with Spotlight Search on Mac. Alongside an inbuilt Dictionary
Originally posted by @Yates2099 in https://github.com/Awesome-Windows/Awesome/issues/390

Originally posted by @zaasgd in https://github.com/zaasgd/add/issues/1#issuecomment-1182479760

#``

[ ] #71


v0.8.5_02: “YEAR” behind the “Mountain Chart” is truncated.

SEVERITY:  Major
MODULE:  Mountain Chart
Platform:  Windows 7 / Chrome 48.0
RELEASE: v0.8.5
STEPS TO REPRODUCE:

Navigate to the URL :  http://static.gapminderdev.org/vizabi/release/v0.8.5/preview/mountainchart.htm
“Mountain Chart” page will be loaded.
Click “landscape” view button on the preview page.
Note that “YEAR” behind the “Mountain Chart” is truncated.

ACTUAL RESULTS:
“YEAR” behind the “Mountain Chart” is truncated.
EXPECTED RESULTS:
“YEAR” behind the “Mountain Chart” should not truncate.
SCREENSHOT:

fixed in 8d31a216abacf5f6b648d61e00f97208968f2ece


On-premise Marketplace

For organizations that would like to develop internal extensions, or organizations with VSC installed on machines in an offline/air-gaped environment, is there any way there could be an on-premise marketplace for extensions? Then there could be a configuration setting that allowed for directing VSC to search this on-premise marketplace in addition to, or even instead of, the existing online extension marketplace.

Duplicate of #21839


Unknown column 'event_name' in 'field list'

I run umami on a koyeb instance using the docker image. As a database I use MySQL on a remote host.
It runs normally but in the logs I see the following error quite often:
PrismaClientKnownRequestError:
Invalid `prisma.event.findMany()` invocation:
The column `XXXXXXX.event.event_name` does not exist in the current database.
at RequestHandler.handleRequestError (/app/node_modules/@prisma/client/runtime/index.js:30873:13)
at RequestHandler.request (/app/node_modules/@prisma/client/runtime/index.js:30856:12)
at async PrismaClient._request (/app/node_modules/@prisma/client/runtime/index.js:31836:16)
at async Promise.all (index 2)
at async getRealtimeData (/app/.next/server/chunks/9447.js:2036:43)
at async __WEBPACK_DEFAULT_EXPORT__ (/app/.next/server/pages/api/realtime/init.js:160:22)
at async Object.apiResolver (/app/node_modules/next/dist/server/api-utils/node.js:366:9)
at async NextNodeServer.runApi (/app/node_modules/next/dist/server/next-server.js:481:9)
at async Object.fn (/app/node_modules/next/dist/server/next-server.js:735:37)
at async Router.execute (/app/node_modules/next/dist/server/router.js:247:36) {
code: 'P2022',
clientVersion: '4.5.0',
meta: { column: 'XXXXXX.event.event_name' }
}

Migrations are all done:
Running update...
Prisma schema loaded from prisma/schema.prisma
Datasource "db": MySQL database "XXXXXXX" at "XXXXXXXXX"
4 migrations found in prisma/migrations
Applying migration `04_add_uuid`
The following migration have been applied:
migrations/
└─ 04_add_uuid/
└─ migration.sql
All migrations have been successfully applied.
✓ Database is up to date.

Running it locally (with the same db) gives me the same error in the console.
In the UI no events are shown.

I do have the event_data table, but no event_name field in the event table.


Test GH build with JDK 17

Quick and dirty test to see if the app builds with JDK 17.

Build's successful, we no longer need that PR.


FieldArray doesn't reflect data order.

I just initialized redux-form with:
[
  {
    id: 1,
    title: 'something',
    children: [...Objects],
  },
  {
    id: 2,
    title: 'blahblah',
    children: [...Objects],
  },
]

And redux-form reinitialized with some actions such as up and down:
[
  {
    id: 2,
    title: 'blahblah',
    children: [...Objects],
  },
  {
    id: 1,
    title: 'something',
    children: [...Objects],
  },
]

The form values are changed at redux-form/INITIALIZE action but the components(title) are not changed.
Only children fields are able to be changed when I expand tree before they are rendered.

It is problem of redux component.


Rename KubeVirtDeprecatedAPIRequested to singular

Renaming KubeVirtDeprecatedAPIsRequested to KubeVirtDeprecatedAPIRequested avoids confusion about plural/singular use.
See https://github.com/kubevirt/monitoring/pull/172#discussion_r1219942486

/cc @apinnick
/cc @sradco
Can you please take a look at this?

/lgtm


Fleet request: more real time visibility

Add ForceDirectedTreeChart component to project
Render ForceDirectedTreeChart component on dedicated Graph tab in Fleet Git Repo details view
Add svgs to project to be used as icons in the ForceDirectedTreeChart component in one file module to be exported as a string (workaround to fix issue with icons not appearing in Firefox. Long term solution should be the usage of icons fonts with ligatures, for which setting up requires some time to do it properly)
Change tab logic to incorporate a dedicated Graph tab to be displayed only in Fleet Git Repo details view
Add CompoundStatusBadge component to project
Add compoundIcons to state resource class
Change logic to improve icon assignment of CompoundStatusBadge in Fleet Dashboard
Other changes

@richard-cox superseded by https://github.com/rancher/dashboard/pull/5938 in order to resolve conflicts (had to squash first). I'll close this. Thanks


enhancement: multi-resolution stream MD5 value calculation

we should support multi-resolution stream MD5 value calculation

AFAICT, ffmpeg and gstreamer decode will automatically scale each frame to the designated output resolution.  Thus, if input has 100 frames in multiple resolutions and output is specified as NxM resolution, then all 100 frames in the output will be NxM resolution.
Can you describe a case where we need multi-resolution support?

same as https://github.com/intel/vaapi-fits/issues/15#issuecomment-467272942

https://github.com/intel/vaapi-fits/pull/24 fix this issue

now gstreamer add parse "videoscale" to make the output as single resolution, so no need multi-res ssim support, close it


add lineage example and relations to catalog

Coverage remained the same when pulling 947058d19c16715d239fe5347b58c9fc42df15c4 on add-lineage-example into 30c8aac58b1a08c22b39a1d8d4ac279b06baadea on master.


where is ul_math_bcd.h

when make compliance，
ul_s.c:9:25: fatal error: ul_math_bcd.h: No such file or directory

You can remove this include. In most cases it not used.

added ul_math_bcd


UI thinks the user is logged in, but that's not the case

We've received many tickets from users who are confused because the overall UI shows they're logged in, but when navigating to pages that do backend checks whether they have permissions to edit something, they can't because they are not in fact logged in.
Example:

The root cause of this seems to be that some browsers periodically clear cookies set through the document.cookie DOM API.
Possible solutions:

periodically refresh the cookie, to avoid the timeout
set the cookie from the backend, not the frontend (that seems to be the case though :confused: )
refresh the login state more often (expensive, would need to happen on every request to be 100% accurate)
have the UI watch the status of the session cookie
...

It seems to happen not only with Safari, see https://inspirehep.zulipchat.com/#narrow/stream/122202-devel/topic/inconsistent.20login/near/237466200.


change PHPUnit namespace, set coverage setting

As title.

@peter279k is this still required? Since we did some changes to the CI pipeline recently already.

Hi @pimjansen, thanks for your reply.
After looking at the .travis.yml setting on master, it seems that we don't have to let all PHP versions be precise dist.
I think we can consider using the matrix setting (it's like my approach on this PR) to dispatch different dist for different PHP versions.

@peter279k i dont really see a problem why they cant all be at precise though?

@peter279k i dont really see a problem why they cant all be at precise though?

As far as I know, the pre-built PHP versions will be different on Travis CI build.
For example, the precise includes the php-5.3.
And the trusty includes the php-5.4 to php-7.3 possibly.
I'm not sure that the internal Travis CI build for PHP versions will be changed and now it's fine for this Travis CI setting at this moment.
Maybe Travis CI will drop the old dist (e.g. precise) in the future.

@peter279k agree but i dont think we will much longer support those older versions where the builds probably will be executed in isolated docker containers anyways.
5.6 is already EOL and no longer supported so for future versions this will probably be dropped completely

If we drop unsupported PHP versions, we will consider using the xenial dist in the future.

@peter279k well nothing is final actually so i would like to leave that as is for now where the builds run fine.

@pimjansen, thanks for your reply. I think it's time to close this PR until the Travis CI build is failed :).

@peter279k no worries! Thanks for contributing and thinking with us


Parse errors with JSONDecoder

Parse errors with JSONDecoder instead of JSONWrapper, and remove JSONWrapper. In relation to this issue: https://github.ibm.com/arf/planning-sdk-squad/issues/525

All Travis failures appear to be timeouts or server-side errors, so I think we should consider the tests to be passing.


A minute to connect with errors ([ERROR][com.winpr.sspi.Kerberos] - [kerberos_AcquireCredentialsHandleA]: krb5glue_get_init_creds (Cannot find KDC for realm "YY" [-1765328230]))

Describe the bug
Ever since Windows 10 LTSC has received a terminal services upgrade 10.0.19041.4355, xfreerdp takes a minute to connect and spits out errors:
xfreerdp /u:user /d:yy /drive:tmp,/tmp /sound /compression-level:2 /v:<IP_ADDRESS> +clipboard
[06:59:26:816] [1076502:00106d17] [INFO][com.freerdp.client.x11] - [xf_setup_x11]: Enabling X11 debug mode.
[06:59:27:767] [1076502:00106d17] [WARN][com.freerdp.crypto] - [verify_cb]: Certificate verification failure 'self-signed certificate (18)' at stack position 0
[06:59:27:768] [1076502:00106d17] [WARN][com.freerdp.crypto] - [verify_cb]: CN = HOST
Password:
[07:00:01:911] [1076502:00106d17] [ERROR][com.winpr.sspi.Kerberos] - [kerberos_AcquireCredentialsHandleA]: krb5glue_get_init_creds (Cannot find KDC for realm "YY" [-1765328230])
[07:00:31:941] [1076502:00106d17] [ERROR][com.winpr.sspi.Kerberos] - [kerberos_AcquireCredentialsHandleA]: krb5glue_get_init_creds (Cannot find KDC for realm "YY" [-1765328230])
[07:00:33:583] [1076502:00106d17] [INFO][com.winpr.timezone] - [winpr_detect_windows_time_zone]: tzid: zoneinfo/UTC
[07:00:33:584] [1076502:00106d17] [ERROR][com.winpr.timezone] - [winpr_detect_windows_time_zone]: Unable to find a match for unix timezone: zoneinfo/UTC
[07:00:33:730] [1076502:00106d17] [WARN][com.freerdp.core.license] - [license_read_binary_blob_data]: license binary blob::type BB_ERROR_BLOB, length=0, skipping.
[07:00:33:844] [1076502:00106d17] [INFO][com.freerdp.gdi] - [gdi_init_ex]: Local framebuffer format  PIXEL_FORMAT_BGRX32
[07:00:33:844] [1076502:00106d17] [INFO][com.freerdp.gdi] - [gdi_init_ex]: Remote framebuffer format PIXEL_FORMAT_BGRA32
[07:00:33:849] [1076502:00106f4e] [INFO][com.freerdp.channels.rdpdr.client] - [devman_load_device_service]: Loading device service drive [tmp] (static)
[07:00:33:853] [1076502:00106d17] [INFO][com.freerdp.channels.rdpsnd.client] - [rdpsnd_load_device_plugin]: [static] Loaded pulse backend for rdpsnd
[07:00:33:853] [1076502:00106d17] [INFO][com.freerdp.channels.drdynvc.client] - [dvcman_load_addin]: Loading Dynamic Virtual Channel rdpsnd
[07:00:33:853] [1076502:00106d17] [INFO][com.freerdp.channels.drdynvc.client] - [dvcman_load_addin]: Loading Dynamic Virtual Channel ainput
[07:00:33:853] [1076502:00106d17] [INFO][com.freerdp.channels.drdynvc.client] - [dvcman_load_addin]: Loading Dynamic Virtual Channel rdpgfx
[07:00:34:684] [1076502:00106f54] [INFO][com.freerdp.channels.rdpsnd.client] - [rdpsnd_load_device_plugin]: [dynamic] Loaded pulse backend for rdpsnd

Application details

FreeRDP version (xfreerdp /version): 3.5.1 from Fedora 40
Output of xfreerdp /buildconfig:

This is FreeRDP version 3.5.1 (n/a)
Build configuration: BUILD_TESTING=OFF WINPR_HAVE_AIO_H=1 WINPR_HAVE_EXECINFO_BACKTRACE=1 WINPR_HAVE_EXECINFO_BACKTRACE_SYMBOLS=1 WINPR_HAVE_EXECINFO_BACKTRACE_SYMBOLS_FD=1 WINPR_HAVE_EXECINFO_HEADER=1 WINPR_HAVE_FCNTL_H=1 WINPR_HAVE_GETLOGIN_R=1 WINPR_HAVE_GETPWUID_R=1 WINPR_HAVE_INTTYPES_H=1 WINPR_HAVE_POLL_H=1 WINPR_HAVE_PTHREAD_MUTEX_TIMEDLOCK_LIB=1 WINPR_HAVE_PTHREAD_MUTEX_TIMEDLOCK_LIBS= WINPR_HAVE_PTHREAD_MUTEX_TIMEDLOCK_SYMBOL=1 WINPR_HAVE_STDBOOL_H=1 WINPR_HAVE_STDINT_H=1 WINPR_HAVE_STRNDUP=1 WINPR_HAVE_SYSLOG_H=1 WINPR_HAVE_SYS_EVENTFD_H=1 WINPR_HAVE_SYS_FILIO_H= WINPR_HAVE_SYS_SELECT_H=1 WINPR_HAVE_SYS_SOCKIO_H= WINPR_HAVE_SYS_TIMERFD_H=1 WINPR_HAVE_TM_GMTOFF=1 WINPR_HAVE_UNISTD_H=1 WINPR_HAVE_UNWIND_H=1 WITH_AAD=ON WITH_ABSOLUTE_PLUGIN_LOAD_PATHS=ON WITH_ADD_PLUGIN_TO_RPATH=OFF WITH_ALSA=ON WITH_BINARY_VERSIONING=OFF WITH_CAIRO=ON WITH_CCACHE=ON WITH_CHANNELS=ON WITH_CLANG_FORMAT=ON WITH_CLIENT=ON WITH_CLIENT_AVAILABLE=1 WITH_CLIENT_CHANNELS=ON WITH_CLIENT_CHANNELS_AVAILABLE=1 WITH_CLIENT_COMMON=ON WITH_CLIENT_INTERFACE=OFF WITH_CLIENT_SDL=ON WITH_CLIENT_SDL_AVAILABLE=1 WITH_CUPS=ON WITH_DEBUG_ALL=OFF WITH_DEBUG_CAPABILITIES=OFF WITH_DEBUG_CERTIFICATE=OFF WITH_DEBUG_CHANNELS=OFF WITH_DEBUG_CLIPRDR=OFF WITH_DEBUG_CODECS=OFF WITH_DEBUG_DVC=OFF WITH_DEBUG_EVENTS=OFF WITH_DEBUG_KBD=OFF WITH_DEBUG_LICENSE=OFF WITH_DEBUG_MUTEX=OFF WITH_DEBUG_NEGO=OFF WITH_DEBUG_NLA=OFF WITH_DEBUG_NTLM=OFF WITH_DEBUG_RAIL=OFF WITH_DEBUG_RDP=OFF WITH_DEBUG_RDPDR=OFF WITH_DEBUG_RDPEI=OFF WITH_DEBUG_RDPGFX=OFF WITH_DEBUG_REDIR=OFF WITH_DEBUG_RFX=OFF WITH_DEBUG_RINGBUFFER=OFF WITH_DEBUG_SCARD=OFF WITH_DEBUG_SCHANNEL=OFF WITH_DEBUG_SDL_EVENTS=OFF WITH_DEBUG_SDL_KBD_EVENTS=OFF WITH_DEBUG_SND=OFF WITH_DEBUG_SVC=OFF WITH_DEBUG_SYMBOLS=OFF WITH_DEBUG_THREADS=OFF WITH_DEBUG_TIMEZONE=OFF WITH_DEBUG_TRANSPORT=OFF WITH_DEBUG_TSG=OFF WITH_DEBUG_TSMF=OFF WITH_DEBUG_TSMF_AVAILABLE=0 WITH_DEBUG_URBDRC=OFF WITH_DEBUG_WND=OFF WITH_DEBUG_X11=OFF WITH_DEBUG_X11_LOCAL_MOVESIZE=OFF WITH_DEBUG_XV=OFF WITH_DIRECTFB=OFF WITH_DSP_EXPERIMENTAL=OFF WITH_DSP_FFMPEG=OFF WITH_DSP_FFMPEG=OFF WITH_DSP_FFMPEG_AVAILABLE=0 WITH_EVENTFD_READ_WRITE=1 WITH_FAAC=OFF WITH_FAAD2=OFF WITH_FFMPEG=OFF WITH_FREERDP_DEPRECATED=OFF WITH_FREERDP_DEPRECATED_COMMANDLINE=OFF WITH_FUSE=ON WITH_GFX_H264=OFF WITH_GPROF=OFF WITH_GSM=ON WITH_ICU=ON WITH_INTERNAL_MD4=OFF WITH_INTERNAL_MD5=OFF WITH_INTERNAL_RC4=OFF WITH_IPP=OFF WITH_JPEG=ON WITH_KRB5=ON WITH_KRB5_NO_NTLM_FALLBACK=OFF WITH_LAME=ON WITH_LIBRARY_VERSIONING=ON WITH_LIBRESSL=OFF WITH_LODEPNG=OFF WITH_MACAUDIO=OFF WITH_MACAUDIO_AVAILABLE=0 WITH_MANPAGES=ON WITH_MBEDTLS=OFF WITH_NATIVE_SSPI=OFF WITH_NEON=OFF WITH_OPENCL=OFF WITH_OPENH264=OFF WITH_OPENSC_PKCS11_LINKED=OFF WITH_OPENSSL=ON WITH_OPUS=ON WITH_OSS=ON WITH_PCSC=ON WITH_PKCS11=ON WITH_PLATFORM_SERVER=ON WITH_POLL=ON WITH_PROFILER=OFF WITH_PROXY=ON WITH_PROXY_APP=ON WITH_PROXY_EMULATE_SMARTCARD=OFF WITH_PROXY_MODULES=ON WITH_PULSE=ON WITH_RDTK=ON WITH_SAMPLE=OFF WITH_SANITIZE_ADDRESS=OFF WITH_SANITIZE_ADDRESS_AVAILABLE=1 WITH_SANITIZE_MEMORY=OFF WITH_SANITIZE_MEMORY_AVAILABLE=1 WITH_SANITIZE_THREAD=OFF WITH_SANITIZE_THREAD_AVAILABLE=1 WITH_SDL_IMAGE_DIALOGS=OFF WITH_SDL_LINK_SHARED=ON WITH_SERVER=ON WITH_SERVER_CHANNELS=ON WITH_SERVER_INTERFACE=ON WITH_SHADOW=ON WITH_SHADOW_MAC=ON WITH_SHADOW_X11=ON WITH_SMARTCARD_EMULATE=ON WITH_SMARTCARD_INSPECT=OFF WITH_SMARTCARD_PCSC=ON WITH_SOXR=ON WITH_SSE2=ON WITH_SWSCALE=OFF WITH_SYSTEMD=ON WITH_THIRD_PARTY=OFF WITH_UNICODE_BUILTIN=OFF WITH_URIPARSER=ON WITH_VAAPI=OFF WITH_VAAPI=OFF WITH_VAAPI_AVAILABLE=0 WITH_VALGRIND_MEMCHECK=OFF WITH_VALGRIND_MEMCHECK_AVAILABLE=1 WITH_VERBOSE_WINPR_ASSERT=ON WITH_VIDEO_FFMPEG=OFF WITH_VIDEO_FFMPEG=OFF WITH_VIDEO_FFMPEG_AVAILABLE=0 WITH_WAYLAND=ON WITH_WEBVIEW=OFF WITH_WINPR_DEPRECATED=OFF WITH_WINPR_TOOLS=ON WITH_WIN_CONSOLE=ON WITH_X11=ON WITH_XCURSOR=ON WITH_XEXT=ON WITH_XFIXES=ON WITH_XI=ON WITH_XINERAMA=ON WITH_XKBFILE=ON WITH_XRANDR=ON WITH_XRENDER=ON WITH_XTEST=ON WITH_XV=ON WITH_ZLIB=ON
Build type:
CFLAGS:              -O2 -flto=auto -ffat-lto-objects -fexceptions -g -grecord-gcc-switches -pipe -Wall -Werror=format-security -Wno-error=incompatible-pointer-types -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1  -m64 -march=x86-64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer -Wall -Wpedantic -Wno-padded -Wno-cast-align -Wno-declaration-after-statement -fPIC -Wall -fvisibility=hidden -Wimplicit-function-declaration -Wredundant-decls -g -fno-omit-frame-pointer
Compiler:            GNU, 14.0.1
Target architecture: x64

OS version connecting to (server side): Windows 10 Enterprise LTSC 21H2 Build 19044.4412

Environment (please complete the following information):

OS: [e.g. Linux/Windows/Android/..]: Fedora 40 AMD64

Debug log: log.txt.gz

I've found a workaround: https://github.com/FreeRDP/FreeRDP/issues/10138#issuecomment-2096488234
Once I disabled dns_lookup_kdc it all works as expected (fast).

@birdie-github check your /etc/krb5.conf, most likely there are non reachable KDC entries (that explains the delay)
will create an FAQ entry

which os do you use?

Fedora 40 as indicated.
I've not changed anything vs the default krb5 configuration. I'm not using kerberos for anything at all:
rpm -qa | grep krb
krb5-libs-1.21.2-5.fc40.x86_64

Just the library installed, that's it.


Expose "Open Gist in GitHub" command in the command palette

atm its only possible from the view menu

Thanks for reporting this! Are you primarily interested in this for playgrounds? Or for any gist? I just ask since we have commands for opening a Gist in GitHub, CodePen, NbViewer and GistLog. I’d love to make sure I know which one you’re interested in 👍

yeah sure, for any gist.
also it will work for playground as it have the same gist id.
this is simply a replication of the view command but in the command palette for quicker access

Agreed. I just wanted to confirm that the exact behavior you want is to open the current gist in GitHub.com. It sounds like that’s what you want, so I’ll add that behavior 👍

I’d prefer not to add any more commands to the command palette, particularly since the GistPad tree view has become very complex, and therefore, is required for performing a lot of different operations. Since I haven’t heard any other feedback for this, I’m going to close it for now. Please reopen it if you feel like this is important enough to become prioritized over other work. Thanks!


Add a noninteractive authenticator

This pair of commits adds another authenticator, which is guaranteed to be non-interactive and just uses stored refresh tokens obtained from a previous authenticator.  I want to use this in an offline job that I run periodically, which I build once using a helper binary to generate the NoninteractiveTokens, and then can use repeatedly.  This authenticator is simple and guaranteed to just error out if it can't provide tokens, instead of trying to do something interactive.

Hi, may I know if there is any follow up on this feature, or at least a temporary workaround?

Hi, may I know if there is any follow up on this feature, or at least a temporary workaround?

Good point, I'm sorry for not having come back to this. It looks like a useful addition, and I'd merge it once the conflicts are resolved!

Is there a difference between this NonInteractive authenticator and an InstalledFlowAuthenticator with a InstalledFlowDelegate that always returns an error in InstalledFlowDelegate::present_user_url?

Is there a difference between this NonInteractive authenticator and an InstalledFlowAuthenticator with a InstalledFlowDelegate that always returns an error in InstalledFlowDelegate::present_user_url?

I think in principle it would have the same effect. I find this flow slightly more transparent and could see some use cases for this, even though the effect would be the same when using an appropriately implemented InstalledFlowDelegate.


Support a new TSDB TickTock

4 DB_SWITCHES added for TickTock:

TickTock-HTTP_LINE: TickTock with HTTP writes using Influxdb line protocol;
TickTock-TCP_LINE: TickTock with TCP writes using Influxdb line protocol;
TickTock-HTTP_PLAIN: TickTock with HTTP writes using OpenTSDB plain put protocol;

TickTock-TCP_PLAIN: TickTock with TCP writes using OpenTSDB plain put protocol;

Connection pool with 10k connections

I can't build it. Lots of coding style issues...


Use project-wide properties

Hello, Thanks for the pull request !
Can you update the samples/LocationEnable project, to give an example to how use the project-wide properties plz ?

Thanks!


Adding volume visualization

I actually found that there is a function exposed in app.ts for displaying surfaces. I used the following code and was able to get something. The loadVolumeFromUrl could probably be adapted in the same way as you do for the other ones.
test = html`<div id="test"></div>`
molstar.Viewer.create(test, {layoutIsExpanded: false, layoutShowControls: false}).then(viewer => {
viewer.loadStructureFromUrl("./traj.xyz", "xyz")
viewer.loadVolumeFromUrl(
    {url: './density.cube',
    format: 'cube',
    isBinary: false},
    [{type: "absolute",
     alpha: 1,
     value: 0.001,
        }
        ]
    )
});

Resolution is not great, but it seems to work, and the visualization can be adjusted later in the viewer

Archive.zip

It was quite straightforward indeed, so I just added it. Let me know if you have further suggestions or PRs :)

That's really cool. I can confirm it works on my end as well. However, I think there are some things which are not ideal at the moment.

The type, alpha and value parameters are hardcoded, but they should be possible to set by the user as different types of surfaces have different requirements.
https://github.com/jmbuhr/quarto-molstar/blob/7192d5d1725bf84ec807dd13e1280d87854e7982/_extensions/molstar/molstar.lua#L170-L172

isBinary is hardcoded to be false but will likely have to be true for some file formats.

https://github.com/jmbuhr/quarto-molstar/blob/7192d5d1725bf84ec807dd13e1280d87854e7982/_extensions/molstar/molstar.lua#L169
I'm no expert on different volume formats, but it seems to be part of the examples so allowing it to be set would be very desirable.

In the original app you can give a list of isosurfaces.

https://github.com/jmbuhr/quarto-molstar/blob/7192d5d1725bf84ec807dd13e1280d87854e7982/_extensions/molstar/molstar.lua#L170-L174
The purpose of this is to give the possibility for example for positive and negative values. You can use this to display orbitals like this. I'm not sure how that could be done in your current input format, but it would be great to have it.
molstar.Viewer.create(app, {layoutIsExpanded: false, layoutShowControls: false}).then(viewer => {
viewer.loadStructureFromUrl("./traj.xyz", "xyz")
viewer.loadVolumeFromUrl(
    {url: './mo_15.cube',
    format: 'cube',
    isBinary: false},
    [{type: "absolute",
     alpha: 0.75,
     value: 0.04,
     color: 0xFF0000,
        },
    {type: "absolute",
     alpha: 0.75,
     value: -0.04,
     color: 0x0000FF,
        }
        ]
    )
});

mo_15.cube.zip

I suppose have to think about passing additional arguments to the shortcode in general. At the moment the different shortcodes take either one or two positional arguments and the keyword arguments are assumed to be molstar options for the viewer creation. But we need some way of passing arguments to the respective loading functions as well. Maybe as a keyword followed by a json string? Do you have some thoughts on a sensible interface?

Ideally we keep it simple for a basic use case, but extensible enough for this. Though at some point you would be better off just writing the html div and a js script tag yourself, so we don't want to be more convoluted than that.

Json string sounds reasonable and I think it's quite similar as to how Google's Fire implements dictionary parsing from the command line. Another option would be to have a delimiter token, for example |.
As you say, in the end you can also use Observable JS to create your own visualization like I did in the blog post, but supporting most functionality in app.ts seems like a reasonable goal.