www playbook: consume new openmicroscopy.ssl-certificate role

See https://github.com/openmicroscopy/ansible-role-ssl-certificate/pull/1 and companion IDR PR https://github.com/IDR/deployment/pull/97
This PR makes the required adjustements to the www playbook to consume the proposed new role.
Changes

set jekyll_build_force_rebuild to False and re-include molecule idempotence tests in Travis
update requirements.yml to consume proposed openmicroscopy.self-certificate role
update www-deploy.yml to consume the new role with the corresponding variable changes (no more pre tasks should be required).

Testing

Travis should pass. The www component of the matrix should run the full molecule test sequence include the idempotence and  test the self-signed certificate
the playbook should be run against the ome-www-dev server

Latest version of this PR with the new Galaxy role now deployed against ome-www-dev. Leaving @kennethgillen and/or @joshmoore to give a final round of code review, then I propose we deploy it on the production www and create an issue for propagating this new role across the other playbooks.

With the last commit, this was successfully run against ome-ww-dev. Proposing to get this merged and deploy to the production server alongside https://github.com/IDR/deployment/pull/97#issuecomment-382009005


OnnxConversion support for gpu export

OnnxConversion support for gpu export
Describe your changes
Move the model and the data to the device based on accelerator spec instead of forcing it to cpu.
Moved a function from PyTorchEvaluator to common utils so it's in shared location.
Checklist before requesting a review

[ ] Add unit tests for this change.
[ ] Make sure all tests can pass.
[ ] Update documents if necessary.
[ ] Format your code by running pre-commit run --all-files
[ ] Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

Not related to this PR, but we should also add gpu export for optimum conversion.

offline sync with @shaahji, this PR will be closed.

Closing in favor of https://github.com/microsoft/Olive/pull/482


Error with parsing CQ

I tried the Continuous Query example from the docs in the CLI:
CREATE CONTINUOUS QUERY 10m_event_count ON db_name BEGIN SELECT count(value) INTO 10m.events FROM events GROUP BY time(10m) END

But I get the following error:
ERR: error parsing query: found 10m, expected identifier at line 1, char 25

That's because the example is wrong. Names with digits in them need to be double quoted. Fixed in #2049


Usable API?

The guys who wrote the Camera2 api should REALLY read the google PDF
How to Design a Good API and Why it Matters https://static.googleusercontent.com/media/research.google.com/en//pubs/archive/32713.pdf

This sample has been migrated to a new location where we can accept Pull Requests (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


CSV sniffer needs more data

Hello- I have a TSV file with line character counts as follows (the first line is the header)
155
130
656
416
707
950
526
753
186
731
...
csv-reconcile init gives me the following error:
$ poetry run csv-reconcile init test7.tsv col1_name col2_name
...
File "/home/me/src/csv-reconcile/csv_reconcile/initdb.py", line 88, in init_db
searchidx = header.index(searchcol)
^^^^^^^^^^^^^^^^^^^^^^^
ValueError: 'col2_name' is not in list
The error is fixed if I change the amount of data being fed to the sniffer on this line
dialect = csv.Sniffer().sniff(csvfile.read(10240))
where I changed the previous value of 1024 to be 10240.

Thanks.  I’ll have a look at making this configurable.  Would it be possible to upload that file so I can test it on your data?  Definitely not necessary but it would be nice to be able to confirm.

Here is a file that fails with sniffer parameter 1024, and succeeds with 10240
test-not-working3.tsv.gz
$ poetry run csv-reconcile init --scorer=dice test-not-working3.tsv public_identifier match_string


transition-mode="out-in" causing issue with directive.

I set up a demo of the issue. Not sure if it has to do with vue-router or not so I posted it here.
http://jsbin.com/nugojawine/1/edit?html,js,output
Basically, if I use "out-in" the directive doesn't seem to wait to apply the same way it does with other transition modes. Not sure if this is an issue with my approach or the directives are being applied too early. Either way it seems inconsistent. Any light on the issue or my technique would be awesome.

This is because Masonry relies on the element to be already inserted so that it can read its width and height. What you can do is to wait until the vm is attached in the masonry directive:
Vue.directive('masonry', {
  update: function (items) {
    var self = this;
    this.vm.$once('hook:attached', function () {
      var masonry = new Masonry(self.el, {
        percentPosition: true
      });
    })
  },
  unbind: function () {
    // ...
  }
});

Thanks a lot for that.
I was triggering a window resize my actual component on attached to get around this issue but this feels better.
This also helps with how to keep the unbind on the directive from happening too quickly and getting weird right before the transition to the other component comes in.


Canada/Quebec Corrections

Which country is affected?
Canada

There are two correction that should be made for the Quebec entries in  in CanadaProvider.cs:
Include CA-QC for Easter Monday:
items.Add(new PublicHoliday(easterSunday.AddDays(1), "Easter Monday", "Easter Monday", countryCode, null, new string[] { "CA-AB", "CA-PE" }));
should be
items.Add(new PublicHoliday(easterSunday.AddDays(1), "Easter Monday", "Easter Monday", countryCode, null, new string[] { "CA-AB", "CA-PE", "CA-QC" }));
Add St. Jean Baptist CA-QC holiday:
items.Add(new PublicHoliday(year, 6, 24, "Discovery Day", "Discovery Day", countryCode, null, new string[] { "CA-NL", "CA-QC", "CA-YT" }));
Should be:
items.Add(new PublicHoliday(year, 6, 24, "Discovery Day", "Discovery Day", countryCode, null, new string[] { "CA-NL", "CA-YT" }));
items.Add(new PublicHoliday(year, 6, 24, "Fête de la Saint-Jean-Baptiste", "Fête de la Saint-Jean-Baptiste", countryCode, null, new string[] { "CA-QC" }));

Is the information on wikipedia wrong?
https://en.wikipedia.org/wiki/Public_holidays_in_Canada

Yes and no. The problem is that Quebec gives companies the option of giving their employees either Good Friday OR Easter Monday off. So either can be a holiday depending on the company you work for.

A company option is not a public holiday in my opinion. A public holiday is free for all, if the company decides it is a bonus.


Cirrus should try to merge in tip of tree master during setup

Currently on pre-submit, Cirrus runs the test suite on feature branches as-is, rather than trying to rebase on upstream master ToT first, then running the tests. Since most contributors believe it works like the latter, let's just do it. If there is a merge conflict, just abort and run as is--github will force the developer to resolve the conflict anyway.
Caveat once this runs, there could be subsequent merges to upstream that conflict/break the code in the PR.
Use case:
D     Merge C to master, post-submit tests fail because the Bar implementation doesn't match Foo API
|\
|  C  Create implementation of Foo called Bar, pre-submit tests pass
|  |
B  |  Upstream change to API of interface Foo
| /
A     Create interface Foo

In the above example, with the current setup, opening a PR with commit C branching off of commit A will pass fine on CI, even though upstream the interface of Foo changed and once merging the branch to master, CI will fail post-submit.

Is it about the auto merger?

Is it about the auto merger?

Not sure what you mean by "auto merger". I'll update the OP to try to clarify.

Here's an example PR that passed pre-submit but failed post-submit

Obsolete. The autosubmit rebases to ToT before merging.

This was specifically about pre-submit tests, however that is now being tracked in the luci bug: https://github.com/flutter/flutter/issues/68668


Docker for Mac 2.0 does not need additional configuration for PVs

Thanks David. You are right. I'm using 18.06, a little bit version.


HTML

HTML
User Story
As a bootcamp student
I want the prework notes to be structured on a webpage
So that I can easily read and find information.
Acceptance criteria
GIVEN a Prework Study Guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CSS, Git, and JavaScript with associated notes listed

Added HTML to project


Fix ggshield secret scan trying to scan too large documents

Description
Scannable.is_longer_than() currently compares the size passed to it to the decoded size, but what we actually want to compare it to is the UTF-8 encoded size.
To implement this without reading the content multiple times, Scannable implementations now store the UTF-8 encoded size, and Scannable helper methods (_is_file_longer_than() and _decode_bytes()) return the UTF-8 encoded size in addition to their current return value.
When the content they work on is already UTF-8, the helpers try to avoid reading it until necessary.
Issue: #561

Codecov Report

Merging #631 (db208b9) into main (e631154) will increase coverage by 0.05%.
The diff coverage is 91.89%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #631      +/-   ##
==========================================
+ Coverage   93.98%   94.03%   +0.05%
==========================================
  Files         103      103
  Lines        4804     4812       +8
==========================================
+ Hits         4515     4525      +10
+ Misses        289      287       -2

Flag
Coverage Δ

unittests
94.03% <91.89%> (+0.05%)
:arrow_up:

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

Impacted Files
Coverage Δ

ggshield/secret/docker.py
90.90% <50.00%> (+1.07%)
:arrow_up:

ggshield/scan/file.py
94.33% <100.00%> (-0.11%)
:arrow_down:

ggshield/scan/scannable.py
97.84% <100.00%> (+0.25%)
:arrow_up:


Failed to retrieve necessary information for download

Error: Failed to retrieve necessary information for download
messageId: 149167ca-2d70-4be2-b3ca-a15109a2aad5
content.js:79 conversationId: 66f7131d-58b4-8006-ab14-27571b041540
content.js:80 accessToken: null

Important Note: Using a VPN or other browser extensions may cause unexpected issues. To accurately diagnose the problem, we recommend testing our extension in a clean browser profile (with other extensions disabled and VPN turned off).

I encounter the same error. No VPN is used and all other extensions are disabled. Also restarted chrome, but the same error

I've just released a new update to fix the token authentication issues.


sudo: sorry, you must have a tty to run sudo

I followed the tips on the official website, but I didn't find anything related to "defaults requiretty" under /etc/sudoers, then I looked for another way to solve the problem, I added something like "defaults !thx

I have already licensed the folder to the installer, and the shell script I created in the installation directory will not have this problem.

Great, can I close this issue now?

I mean I don't have this problem with scripts I create myself, but DS does have this problem all the time

I mean I don't have this problem with scripts I create myself, but DS does have this problem all the time

Sorry , what`s the os and os version?

Tencent provides cloud hosting.
The server version is. centos 7.2 64bit
dolphinschedler version is : 1.2.0

This is my executive order.

I mean I don't have this problem with scripts I create myself, but DS does have this problem all the time

Sorry , what`s the os and os version?

Tencent provides cloud hosting.
The server version is. centos 7.2 64bit
dolphinschedler version is : 1.2.0

Can you edit the /etc/sudoers file and comment out the Defaults   !visiblepw ?  Then you should restart the DS worker server.

Can you edit the /etc/sudoers file and comment out the Defaults !visiblepw ? Then you should restart the DS worker server.

I annotated them all and rerun the ds cluster in the master node using the start-all.sh script, but it didn't seem to work!

Can you edit the /etc/sudoers file and comment out the Defaults !visiblepw ? Then you should restart the DS worker server.

I annotated them all and rerun the ds cluster in the master node using the start-all.sh script, but it didn't seem to work!

在集群里面创建的用户组是hdfs

Try to add this line in /etc/sudoers file
Defaults:%hdfs !requiretty

Defaults:%hdfs !requiretty

Following your suggestion, I've added this paragraph, but it's still not working

Defaults:%hdfs !requiretty

Following your suggestion, I've added this paragraph, but it's still not working

Should I also add sudo-free for the user groups I create?

Defaults:%hdfs !requiretty

Following your suggestion, I've added this paragraph, but it's still not working

This question is very strange, you can see if this blog can solve your problem, especially
Defaults visiblepw

https://cloud.tencent.com/developer/article/1433543

Defaults:%hdfs !requiretty

Following your suggestion, I've added this paragraph, but it's still not working

This question is very strange, you can see if this blog can solve your problem, especially
Defaults visiblepw

https://cloud.tencent.com/developer/article/1433543

I've also tried this method, doing the same operation on every machine and it still doesn't work, but I can execute it successfully when I take this command out of the log and run it, I'm not sure why I'm having this problem when it's executed inside the DS

Defaults:%hdfs !requiretty

Following your suggestion, I've added this paragraph, but it's still not working

This question is very strange, you can see if this blog can solve your problem, especially
Defaults visiblepw

https://cloud.tencent.com/developer/article/1433543

I've also tried this method, doing the same operation on every machine and it still doesn't work, but I can execute it successfully when I take this command out of the log and run it, I'm not sure why I'm having this problem when it's executed inside the DS

Because DS use java Runtime.exec to run the script , Is not the same as executing the script directly in the terminal

Defaults:%hdfs !requiretty

Following your suggestion, I've added this paragraph, but it's still not working

This question is very strange, you can see if this blog can solve your problem, especially
Defaults visiblepw

https://cloud.tencent.com/developer/article/1433543

I've also tried this method, doing the same operation on every machine and it still doesn't work, but I can execute it successfully when I take this command out of the log and run it, I'm not sure why I'm having this problem when it's executed inside the DS

Because DS use java Runtime.exec to run the script , Is not the same as executing the script directly in the terminal

Well, I really don't know what to do with this problem, I just rebooted the server and I found that I can't get into the admin interface now, how can I uninstall this? I think I might need to try the installation again.

this is my sudoers file:

Defaults    always_set_home
Defaults    match_group_by_gid

Defaults    always_query_group_plugin

Defaults    env_reset
Defaults    env_keep =  "COLORS DISPLAY HOSTNAME HISTSIZE KDEDIR LS_COLORS"
Defaults    env_keep += "MAIL PS1 PS2 QTDIR USERNAME LANG LC_ADDRESS LC_CTYPE"
Defaults    env_keep += "LC_COLLATE LC_IDENTIFICATION LC_MEASUREMENT LC_MESSAGES"
Defaults    env_keep += "LC_MONETARY LC_NAME LC_NUMERIC LC_PAPER LC_TELEPHONE"
Defaults    env_keep += "LC_TIME LC_ALL LANGUAGE LINGUAS _XKB_CHARSET XAUTHORITY"

Defaults    secure_path = /sbin:/bin:/usr/sbin:/usr/bin

root    ALL=(ALL)       ALL

%wheel  ALL=(ALL)       ALL

you can update your /etc/sudoers file and copy my content in the file. In my case, its ok.
Another thing you need to confirm is whether your sudo has been alisa. You can run alias command to show it.

this is my sudoers file:

Defaults    always_set_home
Defaults    match_group_by_gid

Defaults    always_query_group_plugin

Defaults    env_reset
Defaults    env_keep =  "COLORS DISPLAY HOSTNAME HISTSIZE KDEDIR LS_COLORS"
Defaults    env_keep += "MAIL PS1 PS2 QTDIR USERNAME LANG LC_ADDRESS LC_CTYPE"
Defaults    env_keep += "LC_COLLATE LC_IDENTIFICATION LC_MEASUREMENT LC_MESSAGES"
Defaults    env_keep += "LC_MONETARY LC_NAME LC_NUMERIC LC_PAPER LC_TELEPHONE"
Defaults    env_keep += "LC_TIME LC_ALL LANGUAGE LINGUAS _XKB_CHARSET XAUTHORITY"

Defaults    secure_path = /sbin:/bin:/usr/sbin:/usr/bin

root    ALL=(ALL)       ALL

%wheel  ALL=(ALL)       ALL

you can update your /etc/sudoers file and copy my content in the file. In my case, its ok.
Another thing you need to confirm is whether your sudo has been alisa. You can run alias command to show it.

I posted the file you sent to my own server and executed my script, but I found out that my service had been running, just not committing, I went to check my service status and all my services were running and not stopping!

this is my sudoers file:

Defaults    always_set_home
Defaults    match_group_by_gid

Defaults    always_query_group_plugin

Defaults    env_reset
Defaults    env_keep =  "COLORS DISPLAY HOSTNAME HISTSIZE KDEDIR LS_COLORS"
Defaults    env_keep += "MAIL PS1 PS2 QTDIR USERNAME LANG LC_ADDRESS LC_CTYPE"
Defaults    env_keep += "LC_COLLATE LC_IDENTIFICATION LC_MEASUREMENT LC_MESSAGES"
Defaults    env_keep += "LC_MONETARY LC_NAME LC_NUMERIC LC_PAPER LC_TELEPHONE"
Defaults    env_keep += "LC_TIME LC_ALL LANGUAGE LINGUAS _XKB_CHARSET XAUTHORITY"

Defaults    secure_path = /sbin:/bin:/usr/sbin:/usr/bin

root    ALL=(ALL)       ALL

%wheel  ALL=(ALL)       ALL

you can update your /etc/sudoers file and copy my content in the file. In my case, its ok.
Another thing you need to confirm is whether your sudo has been alisa. You can run alias command to show it.

运行结束了，失败了，还是同样的问题

@qiaozhanwei  Can you help him ?

@qiaozhanwei Can you help him ?

Or can you send me the installation documentation for the docker method?

if  centos 6 , please
vi /etc/sudoers
root ALL=(ALL)       NOPASSWD: NOPASSWD: ALL
Comment out Default requiretty  row
#Default requiretty

root ALL=(ALL) NOPASSWD: NOPASSWD: ALL

I don't have that line in my file.

add me weixin qiao825193156，Private talk


feat(app): Add React Query Devtools

Overview
Adds React Query Devtools to the desktop app & ODD, hidden behind a FF.
Lazy loading is required to properly render the devtools on the physical ODD. Because it is its own UI element (and because they are accessible in prod builds), a dev feature flag is included.
Note that this requires QueryClientProvider context to function, which is why the tools are placed where they are in the app/ODD render tree. See setup docs here.

Desktop
https://github.com/user-attachments/assets/6a411196-d69b-4db7-aff2-d6381c3682ad
ODD
https://github.com/user-attachments/assets/0d185dcb-28ff-48e4-a2ca-0cd22a84082b

Test Plan and Hands on Testing

See videos

Review requests

I created a new tools dir for this. Is this an ok place for devtools like this?

Risk assessment
low

when we open the dev tool via remove dev tools, does it show the same result as well as ODD?

Yes, they appear to work the same!

@sfoster1 technically this won't work in deployment environments that specifically exclude devDependencies (or installed with npm i --production). I don't think this is actually an issue given our CI workflows, but I'm happy to adjust this if this could be problematic for any reason.


[BUG]when one of the target node is down and httpeventWaitResponse=true, eg throughput falls down

Describe the bug
set outputMode=httpevent and httpeventWaitResponse=true, configure a list of indexers to receive hec
when one of indexers is down,
if self.config.httpeventWaitResponse:
for session in self.active_sessions:
response = session.result()
session.result() takes lots of time to get 'Connection timeout'  in each flush() cycle, and slow down the throughput(in my test, 90% of throughput)
To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
A clear and concise description of what you expected to happen.
Actual behavior
A clear and concise description of what happens after doing the reproduce steps.
Screenshots
If applicable, add screenshots to help explain your problem.
Sample files and eventgen.conf file
Please attach your sample files and eventgen conf file
Do you run eventgen with SA-eventgen?
Yes/No(No means you run eventgen with pip module mode)
If you are using SA-Eventgen with Splunk (please complete the following information):

OS: [e.g. Windows/Mac OS 10.14]
Browser [e.g. chrome, safari]
Eventgen Version [e.g. 22]
Splunk Version[e.g. 7.1.3]
What other apps you have installed in Splunk etc/apps?

If you are using eventgen with pip module mode (please complete the following information):

python version: [e.g. 2.6]
OS: [e.g. Windows10]
Virtual Env is used: Yes/No
Eventgen Version [e.g. 6.3.2]

Additional context
Add any other context about the problem here.

The feature seems to work as expected then, unless I am mistaken. If we want to wait for the responses, we're always going to block the outputter when we flush. We can use a default timeout value or allow the conf files to specify a timeout to use, but I think the easiest fix is simply not waiting for those responses (httpeventWaitResponse=false).
Is there some expected behavior here that I am missing?

@jmeixensperger i agree it is expected.
But my point is that, slow respond in one node blocks the throughput in all the nodes (self.active_sessions keeps hec requests to different idxs)
it's really annoying in a splunk deployment with hundreds of idxs because one host is down is a very normal scenario.
The more friendly behavior i think might be:
if an indexer is not available, just remove it from server_list (or do nothing if this indexer is selected in output stage to make data load per idx stable)
just catch Connection Refused or Timeout exception in wait response stage, and exclude the host?

Makes sense. I will add the functionality to remove the host that is down. Also, if there is no active host, then we will raise an error.

PR pending.


adding multiple headers

Environment info:
Version: 2.6.0
Linux 6.7.7-1-MANJARO x86_64
I understand we can add headers or modify existing ones by using header.Modifier.
https://www.krakend.io/docs/backends/martian/#header-modifier
Is there a mechanism to add multiple headers using this interface, or do we need to use Lua or custom go plugins?

Thank you very much for the snippet and the explanation. Much appreciated.


Metric for number of remote_read queries in-flight

We have a metric that shows the number in-flight queries on the promql engine: prometheus_engine_queries, with the introduction of the concurrency limits #4656, a similar metric should be added for in-flight remote-read queries.

I can take this up

Hi @gouthamve . To continue the discussion, I had created the other PR based on how other metrics were tracked in prometheus.
If I understand correctly, We want a remote read metric at the api module level regardless of the underlying remote storage implementation used. Please correct me If I am wrong.

@tariq1890 this metric applies to the /api/v1/read endpoint which can be used by external systems to read samples from Prometheus. One use case is described here: https://www.robustperception.io/accessing-data-from-prometheus-1-x-in-prometheus-2-0


'New' error for pychromecast

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.64
Python release (python3 --version):
3.6.4
Component/platform:
chromecast
Description of problem:
Whole new error, not seen before
Expected:
Dunno
Traceback (if applicable):

2018-02-26 12:45:07 ERROR (Thread-9) [pychromecast.socket_client] Failed to connect: [Errno 111] Connection refused. aborting due to stop signal.

Additional info:
Not sure that this is a bug tbh, this is just for @OttoWinter  's reference.  Feel free to close if you're already aware.
I know you've been working hard on the pychromecast thing, and I still have the usual random erros and warnings creeping in, but this error (above) is a completely new one I haven't seen before and I can't find any reference to it being mentioned elsewhere, so I thought you should be aware of it.
I can't say whther the error is actually affecting anything, because I don't know what was/wasn't playing/not playing when the errors occured, I've only just spotted them.
Like I say, just for information of the 'new' error at this time.
Thanks.

I believe that error is expected and shouldn't really be a problem. For context, I think it happens when a cast group changes its IP address and we disconnect the old group "leader" by setting the "stop signal".
What I'd be more interested in is whether the cast platform now works better for you. Apart from the logs, do your cast groups work correctly now? Anything else not working?

I'll have to let you know how reliable it is now, only upgraded this morning 👍
In fairness, mine has been pretty good for a while anyway (touch wood). There's been errors in the logs, but it was pretty rare that anything didn't work.


Discord Channel Logs Improvements

Note: This issue description is a brief and will be expanded in the near future with full details.
Current Setup
We use discord channels for providing logs of certain events, such as moderative actions, member updates, guild updates, message updates.
Log output is done through individual webhooks, to avoid contributing to the bot rate limits unnecessarily, and as it is an appropriate choice for a send-and-forget log system.
Sending messages to the logs requires hooking into the loaded Cog and using a helper function to send the appropriate information.
Events that should be skipped from logging are done by sending the event type and corresponding ID value (member ID for member updates, channel ID for channel updates, message ID for message updates).
There's not a lot of flexibility currently, and the API is a bit messy as slowly we've add more logs, destinations and behaviours contributing to complexity.
Purpose
This issue is to begin outlining our current and any possible future requirements for our discord channel logging system, any nice-to-haves that we might be able to think of, and to establish a preferred new API that will make development, configuration and future extension of the feature easier than it currently is.
Current Ideas

Have an event-based model for dispatching logs (can consider utilising the existing bot dispatch and listening system).
Ignore/filter specific events through the usage of a context manager and a function decorator.

How is this different than #303? I know you were talking to me while creating this issue but IIRC I was distracted by something else at the moment, sorry.

Tbh I was tired and had already forgotten the existence of 303 also, but the scope of this is definitely wider than the original.

I don't think there's mutch actionable here at the moment, so i'll close this issue.


Add support for Cisco Mobility Express

Description:
This platform adds device tracker support for Cisco Mobility Express wireless controllers.

Pull request in home-assistant.io with documentation: home-assistant/home-assistant.io#8777
Example entry for configuration.yaml (if applicable):
 # Example configuration.yaml entry
 device_tracker:
   - platform: cisco_mobility_express
     host: CONTROLLER_IP_ADDRESS
     username: YOUR_ADMIN_USERNAME
     password: YOUR_ADMIN_PASSWORD

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

@MartinHjelmare this PR is very very similar to the Luci device tracker you reviewed for me in the past. It’s actually even more straightforward.

Thank you @MartinHjelmare Let me know if I haven't understood you correctly. I think all your concerns are now addressed. 👍

Can be merged when build passes.

Thanks again @MartinHjelmare I am fine with it being merged 👍👍

I got the following error while trying to use this module? what could be the cause?
019-03-21 15:08:59 ERROR (MainThread) [homeassistant.components.device_tracker] Error setting up platform cisco_mobility_express
Traceback (most recent call last):
File "/usr/src/app/homeassistant/components/device_tracker/init.py", line 177, in async_setup_platform
platform.get_scanner, hass, {DOMAIN: p_config})
File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run
result = self.fn(*self.args, **self.kwargs)
File "/usr/src/app/homeassistant/components/cisco_mobility_express/device_tracker.py", line 40, in get_scanner
config.get(CONF_VERIFY_SSL))
File "/usr/local/lib/python3.7/site-packages/ciscomobilityexpress/ciscome.py", line 38, in init
self.get_system_info()
File "/usr/local/lib/python3.7/site-packages/ciscomobilityexpress/ciscome.py", line 43, in get_system_info
json_data = self._call_api(url)
File "/usr/local/lib/python3.7/site-packages/ciscomobilityexpress/ciscome.py", line 94, in _call_api
"from %s", url)
TypeError: init() takes 2 positional arguments but 3 were given

@teleions which version of Cisco Mobility Express are you running? It looks like you are getting a 404 not found during get_system_info()

Please continue troubleshooting in an issue. See below.
Please open an issue if you suspect a bug. If you need help please use our help channels:
https://home-assistant.io/help/#communication-channels
Merged PRs should not be used for support or bug reports. Thanks!


Move Explanation to top in Api docs

This is really just a suggestion based on my experience in the docs. I've found that as I look through the API docs, I frequently scroll down to the Explanation heading to see what an action or schema does, which slows me down as I'm scanning for a solution. It would be faster for me if the explanation was right under the title.

Thank you very much for your feedback! I will wait to see if more people encounter this problem. In the next few weeks, I plan to add a "on this page" menu on the right side of the page to improve navigation in this case.


Interface tab list broken

Description
Under the h2 "Worker Service template" the tab list showing the options [Visual Studio], [Visual Studio for Mac] and [.NET Core CLI] seems to break when selecting [Visual Studio for Mac]. When it is selected, switching back to [Visual Studio] doesn't seem to work, and selecting [.NET Core CLI] results in the first and second option to be highlighted.
Tested in Edge and Chrome.

Page URL
https://learn.microsoft.com/en-us/aspnet/core/fundamentals/host/hosted-services?view=aspnetcore-8.0&tabs=visual-studio
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/fundamentals/host/hosted-services.md
Document ID
b2dece1f-15b8-03af-6d52-7cc070432b4b
Article author
@tdykstra

Associated WorkItem - 253575

@themarkx11 Thanks for reporting that!


Disable kfctl e2e test because current manifests repo refactoring

Since there will be large refactoring work in kubeflow/manifests repo, results in master branch's manifest fail to work, thus, disable kfctl e2e test to avoid PR blockage.
More details can be found https://github.com/kubeflow/manifests/pull/1719#issuecomment-763947044
/cc @yanniszark

@PatrickXYS thanks for opening the PR!
Should we merge now or wait for a blockage to present and then merge?

@PatrickXYS thanks for opening the PR!
Should we merge now or wait for a blockage to present and then merge?

@yanniszark I think we can merge it as of now, because we've already predict it will happen for sure.

@yanniszark I think we can merge it as of now, because we've already predict it will happen for sure.

@PatrickXYS sounds good.
It would be great if you could comment out the presubmit (instead of deleting) and add a comment with the reason why it is disabled.
/lgtm
/hold
Adding a hold in case you want to add the change above. If you think it's ok, then you can choose to remove the hold.

@PatrickXYS sounds good.
It would be great if you could comment out the presubmit (instead of deleting) and add a comment with the reason why it is disabled.
/lgtm
/hold
Adding a hold in case you want to add the change above. If you think it's ok, then you can choose to remove the hold.

@yanniszark Sounds good, added comment to explain the reason.
/unhold

@yanniszark Sounds good, added comment to explain the reason.
/unhold

@yanniszark Seems like you need to lgtm again

@yanniszark Seems like you need to lgtm again


Engaging other teams update

@Joelkw in follow-up to our Slack chat, I added what we talked about in step 3 of the engaging other teams file and as a note in step 8 in the workflow file.
There are other changes in the engaging other teams doc that is for another purpose, so feel free to ignore those.

@Joelkw accepting your changes! Though, we do just say "engineering" to mean the team that also includes a PM ... we do that since 90% of the time, we work with an engineer when we engage for help. That said, fair to add product to that line given the context, so let's do it! But also want to explain why I won't make that change everywhere.


Update hardware content, add guides menu

@lorforlinux I have added guides menu to the header, updated graphics and content for Muscle BioAmp BisCute & Candy. Please review and merge.

@bhawnasehgal Thank you so much for the contribution, I have provided some feedback on improving the text, please check it once and resolve the comments whenever you get a chance.

Thanks for the feedback @lorforlinux, I have resolved most of the suggestions and created issues for the rest.


Does sd.wait() block all other threads?

I'm working on a multithreaded application. One thread puts wav binaries into a queue, and my playback thread plays them:
def start_sound_thread(sound_q, sound_semaphore):
    """
    Wait for sound binary in queue, then play it through the speaker.
    """

    # Thread main loop
    while True:

        # Get message from queue
        while not sound_q.empty():
            wav = sound_q.get()

            # Notify main thread that we're done
            if wav is None:
                sound_semaphore.release()
                continue

            if DEBUG:
                print("Starting sound play")

            # Play the sound
            sd.play(wav, samplerate=AUDIO_OUTPUT_SAMPLE_RATE, device=AUDIO_OUTPUT_INDEX)
            sd.wait()

            if DEBUG:
                print("Done sound play")

        # Let the thread rest
        time.sleep(0.1)

Does sd.wait() hold onto the global interpreter lock (i.e. preventing other threads from running), or does it use a sleep call to let other threads run? If the former, is there a way to make this thread play nicely with others? For example, I would love to have something like:
            # Play the sound
            sd.play(wav, samplerate=AUDIO_OUTPUT_SAMPLE_RATE, device=AUDIO_OUTPUT_INDEX)
            if not sd.is_playing():
                time.sleep(0.1)

A little more digging, and I would have answered my own question. It looks like sd.wait() is based on Event.wait(), which only blocks the current thread (allowing others to run as needed).


Update install.md to add --cluster-version=latest

Changes
added flag --cluster-version=latest so that people following the tutorial will not get the error that the latest version of Tekton is not compatible with whatever version of Kubernetes (for me it was v1.14 that was installed) that is installed by Default
Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
Describe any user facing changes here, or delete this block.

Examples of user facing changes:
- API changes
- Bug fixes
- Any changes in behavior

cc: @jlewi

/assign @sbwsg

/ok-to-test

@afrittoli  thanks!  I made the changes and tested that it works 🙇

/lgtm


Support adding new partitions into existing Hive table

It would be really helpful for periodically generated data like daily aggregate reports data. We can use Presto to generate daily reports and add it into existing Hive table.

This feature has been implemented and is awaiting review: https://github.com/facebook/presto/pull/3377


[SC] Internal Creates in smart contracts don't add to internaltransferlist.

The UTXOset will never be updated correctly. See Create method in InternalTransactionExecutor
Acceptance:

New integration test with a create and a value sent should also check UTXOset for address balance.
Internal creates can be made with an amount sent and a generated transaction will be created dishing out funds correctly.

Add:

Internal creates that fail should not update the internal transfer list


[json] Hover/completion not working for scoped packages

VSCode Version: Version 1.5.0-insider (1.5.0-insider)
OS Version: OSX 10.11.6

Feature Request:
Can you guys please add the ability to see the latest version of a package that starts with an @ symbol? Currently, if I hover any package in my package.json I can see what is the latest version, but with stuff like @types and @angular and others, I'm not able to see it so I'd have to manually search for what is the latest version of their packages.

@aeschli Verification tips?

In your package.json, add a dependency to a scoped package, e.g. @types/babel-types.
Check that the hover as well as version code completion works.

Pretty cool, didn't even know about that.


Climate intro: adding speaker notes for text-to-speech video

for slides introducing Climate data analysis

awesome @annefou! I'll start the video generation manually so that you can give it a listen and see if you want to change anything

Thank you so much!


Chapter 6

This change is

Reviewed 26 of 26 files at r1.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

mb/ch06/Token.hpp, line 9 at r1 (raw file):

    Token(char, double);

remove extra newline

mb/ch06/TokenStream.hpp, line 20 at r1 (raw file):

};

class TokenStream : public ITokenStream

consider splitting classes into separate files

mb/common/Common.cpp, line 13 at r1 (raw file):

                     const double& epsilon)
{
    if (std::fabs(first - second) < epsilon)

if(true) return true?
return std::fabs(...) < epsilon; will work just as fine

Comments from Reviewable

Review status: 21 of 27 files reviewed at latest revision, 3 unresolved discussions, some commit checks failed.

mb/ch06/Token.hpp, line 9 at r1 (raw file):
Previously, lukasz-m-maciejewski (Łukasz Maciejewski) wrote…
> remove extra newline

Done

mb/ch06/TokenStream.hpp, line 20 at r1 (raw file):
Previously, lukasz-m-maciejewski (Łukasz Maciejewski) wrote…
> consider splitting classes into separate files

Splitted

mb/common/Common.cpp, line 13 at r1 (raw file):
Previously, lukasz-m-maciejewski (Łukasz Maciejewski) wrote…
> if(true) return true?
>
> return std::fabs(...) < epsilon; will work just as fine

Reworked

Comments from Reviewable

Reviewed 6 of 6 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


add --upgrade-all flag to refresh any stale models

This change allows you to run ollama pull --upgrade-all which will check each of your local models and upgrade any that are out of date. It uses Etags to check if there is a newer manifest, and then pulls that model if it has been updated.

Are updates run synchronously or asynchronously? I've found that updating models in parallel is doable on a reasonably strong connection.

@ThatOneCalculator each model is pulled synchronously, however, the "chunks" of that model are actually pulled asynchronously, which is how we get fast pull times.

Ah. Would you consider adding multiple downloads at once? That might help speed up things even more.

@ThatOneCalculator Given how we're already pulling things, I'm not sure that would help a lot. What speeds are you seeing when pulling right now?

This would be an immensely useful feature but even better with functionality inside api/client.go maybe something like /api/upgrade/model and /api/upgrade/all?

What is blocking this PR from getting merged?

@KhimairaCrypto the etags implementation. It needs to be revamped.


trim is not enabled on SSDs

I solved this for VaporOS by packaging the scripts used for trim in Ubuntu. You can find the package source here: https://github.com/sharkwouter/vaporos-brewmaster/blob/master/sources/vaporos-trim-support2_2.1.tar.gz
It is GPL licensed, but since config files are just plain text files that shouldn't be an issue.

This is addressed by pull request #31

Nice job @sharkwouter, thanks!


⚠ ClamAV has degraded performance

In 5c6d5d1, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 75 ms

Resolved: ClamAV performance has improved in d5b6800.


Volumes UI: "View" is unclear

The volumes UI looks amazing. But I initially was confused by the View button -- since I expected to get the data of the volume -- but instead I was redirected to the unikernel that is using the block device.
Since the "get" and "set" operations are to be implemented (see #32), we should find some good names for the buttons :)
I also like the pie chart a lot. Thanks so much @PizieDust :)

Yes that's right. "View" is not very explanatory.
I was thinking of a way to display the data. For instance if we read data from a block device, do we just display this as text? What is the data corresponds to images?
Also, the way I understand the "get" operation, we have to give it an offset so it reads to that offset?

Yes that's right. "View" is not very explanatory. I was thinking of a way to display the data. For instance if we read data from a block device, do we just display this as text? What is the data corresponds to images?

I'd have a "download" button, and allow the data to be downloaded as application/octet-stream.

Also, the way I understand the "get" operation, we have to give it an offset so it reads to that offset?

The int provided to `Block_dump is the compression level to use. 0 for none, 9 for highest (I guess). It uses https://github.com/mirage/decompress.

Oh I see. Thank you. I think I understand it now.
On Wed, 13 Nov 2024, 16:36 Hannes Mehnert, @.***> wrote:

Yes that's right. "View" is not very explanatory. I was thinking of a way
to display the data. For instance if we read data from a block device, do
we just display this as text? What is the data corresponds to images?
I'd have a "download" button, and allow the data to be downloaded as
application/octet-stream.
Also, the way I understand the "get" operation, we have to give it an
offset so it reads to that offset?
The int provided to `Block_dump is the compression level to use. 0 for
none, 9 for highest (I guess). It uses
https://github.com/mirage/decompress.
—
Reply to this email directly, view it on GitHub
https://github.com/robur-coop/mollymawk/issues/92#issuecomment-2473968862,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2VKJERWV7HSFQVBNOS3QN32ANWWBAVCNFSM6AAAAABRWR5PZKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZTHE3DQOBWGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

#95 addresses this


Question: is membership in the 'video' group mandatory?

As stated in the subject, I would like to know if it is an absolute requirement that every users that try to access and run
his/her workload on the AMD GPUs must be an explicit member of the video group.
With our current setup this would be quite complicated to implement:

there is no direct login involved (nor via GDM/KDM or SSH), so the methods based on pam_group.so and /etc/security/groups.conf are not an option.
modifications on our current centralized LDAP service are tricky to perform as well, since the video group is a system group and by internal policy we do not export them.
list explicitly all the potential users under /etc/group on dozens of machines is somehow doable via configuration management but it is at best cumbersome and it is clear it does not scale when there are more than 15 users and many more may be added in the future.

Other than these three possibilities, there is no easy way to transparently add a supplementary group to every possible users as soon as  a job is started (not interactively but as batch) via our job management system (Slurm).
The solution provided in the official documentation:
https://rocmdocs.amd.com/en/latest/InstallGuide.html#setting-permissions-for-groups
is definitely not feasible for us.
Details of our configuration:

AMD Radeon MI100
ROCm framework version 4.3.1

In addition: we do not use Docker to run our GPU related workload, so its command line option like --group-add video
is definitely not a possibility for us.
Thanks in advance!

Hi @mtds
Thanks for reaching out.
Yes, user should be part of 'video' or 'render' group as part of DRM access.
Its mandatory.
Thank you.

@mtds have we resolved your issue? Can we please close it?  Thanks!

@abhimeda Sure I will close it, thanks.


Unexpected initial value with jit

Reporting a bug

[x] I have tried using the latest released version of Numba (most recent is
visible in the release notes
(https://numba.readthedocs.io/en/stable/release-notes-overview.html).
[x] I have included a self contained code sample to reproduce the problem.
i.e. it's possible to run as 'python bug.py'.

I think this maybe a bug.
Though cpython throws an exception when flag is False, according to https://github.com/numba/numba/issues/9647#issue-2400041502, x should be allocated before executing and init with zero, is also acceptable.
But actually, the following code returns 2 when flag is False.
from numba import jit

@jit
def f(flag):
    if flag:
        x = 2
    return x

r = f(False)
print(r) # expect Exception (or 0) but got 2

It's not exactly well-signposted in the docs, but variable liveness is not checked by Numba: https://numba.readthedocs.io/en/stable/reference/pysemantics.html#zero-initialization-of-variables
The docs could do with an edit / update to make this deviation a little more prominent / clear.

Yeah, this looks like a bug. My understanding is that the above code should work like:
from numba import jit

@jit
def f(flag):
    x = 0
    if flag:
        x = 2
    return x

print(f(False))
print(f(True))

Think this is a duplicate of #7338, essentially, type inference propagates 2 as the value for x across the phi node without acknowledging the undefined nature of one of the incoming values.


Upgrade Client

This is a task branch for https://github.com/Hashnode/mern-starter/issues/344
What does this PR add?

New simpler build pipeline with Webpack 3 and babel-preset-env
React 16 and React-Router v4
Revamped client folder structure
Linting hooks
Demo blog app with React-Redux

@mattlubner you might have a lot of what you're looking for in this PR for the React update.


Unable to Create backup on Android TV OS

Steps to reproduce

Go to "Backup and restore → Create backup"
Click "Create"

Expected behavior
I want to create backup to transfer all my watch list to the beta version. After clicking Create, a backup window should open to ask the user on where to save the backup file.
Actual behavior
Unable to create backup. After clicking the Create button, an error text showing "You don't have an app that can do this" will pop up.
This also happened when you clicked on Backup location.
Crash logs
No response
Aniyomi version
<IP_ADDRESS>
Android version
Android 12 (Android TV OS)
Device
Mecool KM2 (LEAP-S1)
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

This happens because Android TVs do not have the Storage Access Framework (SAF) implemented. Since Aniyomi uses that Framework it cannot access folders (which is the first thing for file creation)

https://github.com/aniyomiorg/aniyomi/issues/162


Component chunk is listed twice in react-loadable manifest

Bug report
There are duplicate chunks for the robot-bundle component. The component is lazy loaded with SSR: false.
This is the section of the react-loadable-manifest.json. You can see that 0zX2, 3at2 are duplicated.
Is that correct or what's the reason for this? There are also two chunks for the robot-bundle component on client side even though it's exactly the same component.
  "../../components/robot-bundle": [
    {
      "id": "",
      "name": null,
      "file": "static/chunks/styles.7bcfa32fbdfd22ad0da9.js",
      "publicPath": "static/chunks/styles.7bcfa32fbdfd22ad0da9.js"
    },
    {
      "id": "0zX2",
      "name": "../../node_modules/axios/lib/helpers/isURLSameOrigin.js",
      "file": "static/chunks/9c13.c708e85ffab23caa575e.js",
      "publicPath": "static/chunks/9c13.c708e85ffab23caa575e.js"
    },
    {
      "id": "3at2",
      "name": "../../node_modules/react-feather/dist/icons/eye.js",
      "file": "static/chunks/9a93.ce32de670ed7dbc20850.js",
      "publicPath": "static/chunks/9a93.ce32de670ed7dbc20850.js"
    },
    {
      "id": "3at2",
      "name": "../../node_modules/react-feather/dist/icons/eye.js",
      "file": "static/chunks/0430.1e2bbca5ac8a0d8676d8.js",
      "publicPath": "static/chunks/0430.1e2bbca5ac8a0d8676d8.js"
    },
    {
      "id": "0zX2",
      "name": "../../node_modules/axios/lib/helpers/isURLSameOrigin.js",
      "file": "static/chunks/c2a8.27def4aa4c3e39bcd218.js",
      "publicPath": "static/chunks/c2a8.27def4aa4c3e39bcd218.js"
    }
  ],

Expected behavior
From my understanding I expect that there is only one chunk.
System information

OS: Ubuntu
Version of Next.js: 8.1.0

This is expected because these are dynamic chunks and they are not used in enough locations to warrant the submodule to be extracted to the commons chunk.


Typo

Hi,
Great work!

Fredrik

Thanks!


[FEATURE] add simple Homepage

Thecyberhub-app is a react-native app
We have to build the same home page
Here is our home screen:

and we want to this exactly same on our home screen

website link https://www.thecyberhub.org/
website source code is also available on the GitHub
If you need any help or having doubts, let us know:
Thanks

I would like to work on this issue. could you please assign it to me?

ill take this up

ill take this up.
the Hamburger menu seems unnecessary on mobile.
since you already have a tab navigator on the bottom.

Hey @ATH42
Yes we don't have to implement Hamburger
Sure, I am assigning you this issue.
If you will have any questions or doubts, let us know below
Thanks.

If you want to make bottom-navbar with your own style, you can make
And can restructured the components and names.

could you refer me to the background image that you used on the web page?
should i also add the cyberworld logo as a splashscreen?

could you refer me to the background image that you used on the web page?
should i also add the cyberworld logo as a splashscreen?

Splashscreen, great, yes you can use thecyberworld logo as splashscreen
And about background image
We are using a video in a background.


Add IP SLA and redirect

I believe we need to add the redirect health group for destinations as well.


Make y_obs optional for marginal_plot()

There are use cases where we would like to inspect a model but we have no observed data at hand. For example if the model itself is a ensemble of models, the observed values are not easily available and we would still like to see the total effect of one feature in the form of the partial dependence plot as well as the average prediction per feature.
Proposition
Allow y_obs to be None. In that case only plot average of predicted as well as the histogram of the feature.

This feature sound reasonable to me. It would mean allowing for

plot_marginal(y_obs=None, ..)
compute_marginal(y_obs=None, ..)

PR welcome. I would recommend to start with the latter.

Ups mir fehlen noch einige Skills mit Open Source,,, VS Code war so nett und hat mit einen Fork erstellt, als ich in einen Branch bei dir pushen wollte.
Hier ein einfacher Vorschlag, der beim Beispiel Test gleich funktionierte: https://github.com/samathizer/model-diagnostics/tree/Isssue-178
Und ich habe vorher und nachher die selben 4 RuntimeWarnings erhalten:
￼
Gruss Samuel

On 1 Oct 2024, at 19:27, Christian Lorentzen @.***> wrote:
This feature sound reasonable to me. It would mean allowing for
plot_marginal(y_obs=None, ..)
compute_marginal(y_obs=None, ..)
PR welcome. I would recommend to start with the latter.
—
Reply to this email directly, view it on GitHub https://github.com/lorentzenchr/model-diagnostics/issues/178#issuecomment-2386569846, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ5CEY2VEZTTG2FUTEC7RLLZZLLPXAVCNFSM6AAAAABPE5H6ZGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBWGU3DSOBUGY.
You are receiving this because you authored the thread.


dw doesn't work with neovim 0.2.0 but works with 0.2.2

Tried running delete word with dw but didn't work with neovim 0.2.0. Updated to 0.2.2 and it worked. I'm on MacOS High Sierra with ST3156 and the latest version of ActualVim. Is this a known problem? May be beneficial to add in the README.

I added neovim version recommendation in readme, closing.


Remove explicit enrollmentsUrl override

The LMS is now setting this properly, so it can be removed from here.

bower linked this on my local instance, seems good. (I mean, it would either properly fetch enrollments or it wouldn't... and it did.)


Fritz Dashboard

Can you you add your fritz dashboard for grafana?

I've added all my grafana dashboards in 0ed4b937eba97a5032d1881c4ea612a2888961b0.
I know they are not perfect and maybe you have some improvements to share with me. :)


Update docs to mark packages commands as deprecated

Issue #, if available:
#2270
Description of changes:
Packages commands were deprecated in #8240 from EKS-A v0.20.0 onwards. This PR updates the EKS-A docs to mark packages commands as deprecated.
Testing (if applicable):
make -C docs build
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/cherry-pick release-0.20

/lgtm
/approve


Organization and Testing for Decaf Batch

In https://github.com/phetsims/decaf/issues/79 I published a new batch of simulations for decaf testing.
In slack, @KatieWoe said:

I would make sure each could open on Win Chrome, Win Firefox, Win Edge, Win IE, Mac 15 Chrome, Mac 15 Safari, iPad 13 Safari and Chrome OS. Probably do a bit of testing on each to determine if anything is unusably slow on important platforms.

We should decide how to track the testing for these simulations.  We would like to make it easy to see when a simulation is ready for publication, or, if not, what the problems are, and make it easy to deploy new versions for testing.  I sketched out a spreadsheet to start the discussion.
https://docs.google.com/spreadsheets/d/10bO1Eimm5Y7TH4o7QPebpNUaIItxYK-VnKcSe24TXmc/edit#gid=0
Let's discuss organization around this testing and how to proceed.  And the batch mentioned in https://github.com/phetsims/decaf/issues/79 and listed again in the prototype spreadsheet is ready for testing.

Talked with Sam a bit already, added a device/platform, and chose what we would say about each (good, ok [meaning that it lags noticeably but doesn't hurt learning], and unusable).
Testing is underway.

Concurrent top priority with #494

Now in https://github.com/phetsims/QA/issues/499. Closing


Import-CSV doesn't understand sep= header

Excel saves csv with a sep= header to indicate the delimiter/separator.  The cmdlet doesn't understand this header so it treats it as the header row incorrectly.
Steps to reproduce
@"
sep=,
a,b,c
1,2,3
"@ > test.csv
import-csv ./test.csv

Expected behavior
a b c
- - -
1 2 3

Actual behavior
sep=
----
a
1

Environment data

Name                           Value
----                           -----
PSVersion                      6.1.0-rc.1
PSEdition                      Core
GitCommitId                    6.1.0-rc.1
OS                             Darwin 17.7.0 Darwin Kernel Version 17.7.0: Thu Jun 21 22:53:14 PDT 2018; root:xnu-4570.71.2~1/RELEASE_X86_64
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

I cannot get Excel 2016 on Windows to repro this in any of the following formats as far as inserting a sep=.
CSV (Comma delimited) (*.CSV)
CSV (Macintosh) (*.CSV)
CSV (MS-DOS. (*.CSV)

I see the your repro is on macOS. It is Numbers producting the CSV? Or possibly Excel on macOS is just different. I've worked with a lot of CSVs in my career and have never seen sep= before.

We need more info. I did not find any specifications.
Is "sep=" always in first line?
Is separator char always single char?

Only spec I found was rfc4180 which does not include sep=.

The original issue was reported to me by a customer.  Looking online.  It seems that sep=, is not standard and with latest Excel on my macBook, I don't repro this either.  At this time, it seems like it's a legacy Excel feature to insert that metadata.  I'm ok with closing this until there's more community feedback.


Comparison is false...

Azure Data Studio Version: 1.46.1
OS Version: Windows 11 Pro

Steps to Reproduce:
Start a SHEMA COMPARE between a LOCAL database and a Azure database
The result is false.  When we open both files in ADS both are absolutely identical yet from the comparison we see the local not having [dbo].[...] but in fact it have it when we open the stored procedure so again both files are identical but the comparison sees them differently !?

Does this issue occur when all extensions are disabled?: Yes/No
YES

I clicked the REPORT HERE from the Extension page.  Hope it's ok !

Do you have an example dacpac that you can share to repro the problem here?


I don't know why, but it has bug with navigate when i use callback

Link component and another navigate tools has bug
href='/sign_in'  - it's works correct
href={routes.signIn()} - it's fail

Closing as outdated. Feel free to reopen if this is still relevant.


Add project maintainers as codeowners

We are making our open source project maintainers explicit with a CODEOWNERS file. Let me know /f these are not the correct maintainers for this project

I haven't been maintaining this project for a while, and I'm afraid I can't really allocate the time.
(Huge thanks to all the other contributors for picking up the ball (.rs) that I dropped)

Sure thing, I just got this from the existing list of maintainers which is outdated. Who should be the maintainer now?

@hrydgard Might make sense? (Sorry for throwing you under the bus Henrik)

I've updated this now, let me know if this is good to merge. Thanks

This this waiting on anything in particular?

I think it just drowned in a flood of issues.

Always the way! Thank you
On Mon, 7 Dec 2020, 15:47 Henrik Rydgård<EMAIL_ADDRESS>wrote:

I think it just drowned in a flood of issues.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EmbarkStudios/physx-rs/pull/108#issuecomment-740001680,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABOZVBXLKDPML7CJ42GNB5LSTT2KZANCNFSM4UJA3SHQ
.


fix import path in .travis.yml script

In e12752a we forgot to change build script import path.

@SchSeba @booxter  #68 introduced an issue on travis deploy script, I believe this could fix it.
https://travis-ci.org/kubevirt/ovs-cni/builds/502455100#L647


Firefox: Alt text for images on article pages not being declared before the caption beneath the image

What

The Alt text for the images within articles is being read out after the caption beneath the image which makes the page compositon confusing if you are using a screen reader.
It is  not in the correct order. The screen reader should be reading vertically down the page so the image should be declared with the alt text and then the quote under it.
This may be because there is what looks like an image/fig caption being used for a quote. If there is a fig caption then the alt text would usually be excluded as it would be read twice. So its possible it is due to composition rather than code.
Where

This does appear to be consistent behaviour in Firefox (minimal reproducible example in Codepen). I haven't been able to find an obvious way around it while continuing to use the figure and figcaption elements which feel highly semantically appropriate. An obvious alternative is to use divs, but I think this would be worse for other reasons.


Sketching-based Matrix Computations

TL;DR: I think having an implementation for some randomize matrix algorithms would be great. I would like to discuss if the community finds it useful too.
Randomized matrix algorithms have been a hot topic in research the last years. Recent developments have shown their utility in large-scale machine learning and statistical data analysis applications.
Sketching is a way to compress matrices that preserve essential matrix properties. For some problems, sketches can be used to get faster ways to find high-precision solutions to the original problem. This tool can be used for least-squares and robust regression, eigenvector analysis, non-negative matrix factorization, etc...
The main idea of sketching matrices is not new. One of the most famous concepts behind the efficiency of random projection is the Johnson-Lindenstrauss lemma. It is used for random projections, and it has a "crude" implementation in scikit-learn
An example of sketching is Uniform Sampling. It is quick and easy, but doesn't always yield good sketches; however, there are sophisticated sampling methods that do produce good sketches. I have been working on a xdata open source project for the last year called libSkylark. The library is suitable for general statistical data analysis and optimization applications, but it is heavily focused on distributed systems.
Although many ML applications would take advantage of the implementation of these algorithms the scope of them goes far beyond. I believe it would be a good idea integrate some of them in Scipy and I would like to know the opinion of others to check if it would be worth it.
Edit: I joined the Scipy dev mailing list yesterday. If the discussion is here successful I would discuss what/how to implement in-depth on the mailing list.

Hi @jomsdev, interesting. First a question on scope fit: you say you're focused on distributed systems, however distributed algorithms and data structures are out of scope for scipy. Are you proposing a non-distributed version here?
Also, how standalone is your code? I see libSkylark having dependencies on FFTW and MPI for example - both of those are showstoppers.

Hi @rgommers,
Thank you for the fast response.
As you say, LibSkylark has "hardcore" dependencies on MPI and FFTW. It is a production-ready implementation, and it extraordinarily scales. In fact, this is one of its strength/weakness. I made some python bindings and they are compatible with  numpy/scipy.sparse but the goal of the library is not this one. These methods are really powerful, and I would like having more people with access to them.
For this reason, what I am proposing here is a non-distributed approach.  Some time ago Matlab implemented Blendenpik, an algorithm based on these techniques that outperforms LAPACK by significant factors and scales significantly better than any QR-based solver.
Scipy is one of the most successful libraries in the Scientific Python community and makes sense for me having these methods in it.

I would like to propose a clear roadmap of what/how would be implemented. To my eyes, Randomized Numerical Linear Algebra is big/useful enough to has its submodule. However, this is something that the community has to decide.
Meanwhile, to demonstrate that these methods can be useful I would implement them under scipy. linalg,  where the scipy linear algebra functions live.
The idea is:

Implement CWT. CWT lets us create a matrix S such as SA is shorter than A.

Compute QR = SA using scipy.

From here, there are two potential features:

Implement Blendenpik for solving Linear Square problems faster without losing any accuracy.

Compute the squared row norms to obtain leverage scores.

I can implement both but I will start with Blendenpik, which is far more common.
I am thinking about doing it for sparse matrices first because CWT can take advantage of it. However, I wonder if the QR method on scipy takes advantage of sparsity too.
I have already set up the development environment so I could start as soon as we decide what to do. As I said I am open to discussion so anyone who could be interested is welcome.
Thank you,
Jordi.

@jomsdev I can't follow the link to CWT; do you have another?

@stefanv fixed.

However, I wonder if the QR method on scipy takes advantage of sparsity too.

No, linalg.qr doesn't know about sparse matrices.

To my eyes, Randomized Numerical Linear Algebra is big/useful enough to has its submodule.

I'm not an expert, but it's not obvious from the list above. If it's CWT, Blendenpik and leverage scores, there's no need for a separate submodule right?
These algorithms do look quite promising, and they've been cited enough and been around long enough that it looks to me like they're fine to add to SciPy.

If it's CWT, Blendenpik and leverage scores, there's no need for a separate submodule right?

I took the combination of CWT/Blendenpik because:

CWT takes advantage of sparsity (can be performed in O(nnz(A))
CWT can be used as a first step for many algorithms that look like "sketch and solve".
Blendenpik uses sketching as a preconditioner, but it yields the real vector x  which solves linear square regression.

However, there are many randomize algorithms for low-rank approximation, matrix decomposition, eigenvalues, etc.. that does not yield the "optimal" solution but an approximation. In many cases there is a parameter than can be tuned to achieve the accuracy desired taking into account the trade-off between speed/precision.
If after Blendenpik we see some traction from the community to these methods, I am glad to implement them.
I mentioned the module to be sure that people don't get it wrong (Why if I execute the same multiple times it gives me a different result? Or, Why it does not give the same result than using X algorithm?)
Anyway, the random linalg module (rlinalg??) is something that we should not worry about for now (and maybe never!)

I think this would be a fantastic feature addition. Us consumers of LA/ML algorithms would greatly benefit from having robust, tested, solutions to sketching problems.

I already started to implement the Clarckson-Woodruff Transform in scipy
I created a new file under /scipy/linalg/ called random_projections.py. I will implement in it some random projections.
I am now writing some tests (because TDD) but I do not know why I can't execute them. I created a new file scipy/linalg/tests/test_random_projections.py following other tests as example but it does not run when I execute the command python runtests.py -s linalg
Any idea about why is this happening?

I think your check_clarkson_woodruff_transform method needs to start with test_ to get picked up as a test method. (I.e., use test_clarkson_woodruff_transform.)
Also, the current recommendation is to add any new files as private modules (so _random_projections.py), to discourage them from being imported directly by users.

**In order to make the process as accessible to others as I can this message is a copy of the message sent to the official mailing list **
Hello,
I just made a pull request to the official repository with the implementation of the Clarkson Woodruff Transform (Count Min Sketch) for dense matrices.
You can find more details of the pull request here.
I would not like to made it badly (this is my pull request to the project) so any advice is welcome (Main implementation/tests/documentation).
Jordi Montes.

If anyone who's a potential user or is familiar with the algorithm (@quattro, @lesshaste, @mhsouto ?)  would like to review or test the PR (gh-7809), that would be very helpful.

I did some basic testing in REMLSketch and everything ran w/o issue.
I can't speak for future improvements (e.g., sparsity, FLJT, etc), but the current implementation seems to just work.

Thanks @quattro

I'll close this issue, now that we're agreed on it being a good idea to add some of these methods to scipy.linalg and the first PR being merged. Thanks all!

It would be good to have a list of PRs here for the interested reader.

@lesshaste sure, we can keep linking to or adding to this issue even when it's closed. Also note that each added function will show up at http://scipy.github.io/devdocs/linalg.html within a day of merging the PR for it.

Is there a general consensus/recommendation on what the appropriate sketch size for an n x k matrix? It might be helpful to add some recommendation/suggestion in the docs to guide users not familiar with this. I see mention of an error and suspect its related to those parameters, but again, the connection isn't explicit.


refactory metadata original message

Signed-off-by: zhaohuabing<EMAIL_ADDRESS>

LGTM

LGTM

LGTM


Publish to Maven Central Repository

I would find it convenient to just download from Maven. Thanks!

The problem is actually coming from the synchronization between bintray and maven central.
I need to look more into this.
For reference, here's the error in the bintray interface.
Last Sync Errors: No public key: Key with id: (93f9de56b3b1d69d) was not able to be located on http://keyserver.ubuntu.com:11371.
Upload your public key and try the operation again.
No public key: Key with id: (93f9de56b3b1d69d) was not able to be located on http://pool.sks-keyservers.net:11371/.
Upload your public key and try the operation again.
No public key: Key with id: (93f9de56b3b1d69d) was not able to be located on http://pgp.mit.edu:11371/.
Upload your public key and try the operation again.
Dropping existing partial staging repository.

I'm confused. I just put the public and private keys in the Bintray account, and even re-release (v0.0.3) so it had a chance to sign the files on upload, and it is still having the issue.
After 45 minutes of messing with this I have run out of time and am not finding what needs to be fixed :-/

Same here, my understanding of the error is that the pgp public key used to sign the jar is not published anywhere (keyserver.ubuntu.com, pgp.mit.edu are the main repos).
I have no idea how the signing is made and where the key is found.

It's easy to publish directly to Maven Central: http://central.sonatype.org/pages/ossrh-guide.html

ping
I've published these previously as snapshots under my own groupId to sonatype, and could publish as releases (I do for other projects) to maven central.  But the right fix must be getting this set up cleanly with reactivesocket-project, not the ossrh guide.  I don't think I can do that.

We have involved our eng-tools team to see why the gradle plugin is not working.
They're working on it.

Awesome to know. We will work around for now. Thanks!

https://mvnrepository.com/artifact/io.reactivesocket/reactivesocket-core
thanks @benjchristensen


Book on Testing JS applications

What does this PR do?
Add resource(s)
For resources
Description
Testing JavaScript Applications is a book to learn effective tests through examples. It covers tools that teach how to think about testing from a business perspective.
Why is this valuable (or not)?
This book will help you write high-quality software in less time, with more confidence.
How do we know it's really free?
Available in GitHub for free.
For book lists, is it a book? For course lists, is it a course? etc.
It is a book.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

In case you approve my PR, could you please tag my pr with the hacktoberfest-accepted label. Much appreciated!

github content is not the book, which is not free


ROS2 Body behavior

This requires https://github.com/bit-bots/dynamic_stack_decider/issues/73 and https://github.com/bit-bots/bitbots_behavior/issues/256

Part of the work done in https://github.com/bit-bots/bitbots_behavior/pull/260


Couldn't switch tabs correctly before

I hope this solves following problem:
original Bug report (via mail):
On the https://owncloud.org/install/ page, if you click on the "Mobile client", you can not go to the "install desktop client" tab. I checked on windows 10 google chrome and Edge.

I'll merge to test it on staging.

looks fine, I'll bring it live.


[release-0.41] Manual cherry pick of #5891

Don't delete temp pods that aren't associated with VMI;
We do not want to delete temporary pods that are not owned by the VMI.
Signed-off-by: Alex Kalenyuk<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
https://bugzilla.redhat.com/show_bug.cgi?id=1974289
Special notes for your reviewer:
Release note:

BugFix: Pending VMIs when creating concurrent bulk of VMs backed by WFFC DVs

/hold

/retest

/retest

/retest

/retest

/retest

Hey @jean-edouard, following the discussion in https://github.com/kubevirt/kubevirt/pull/6025#issuecomment-881435927,
do you think we should consider overriding here as well?

/retest

/retest

Hey @jean-edouard, following the discussion in #6025 (comment),
do you think we should consider overriding here as well?

Sure, but then we should look into fixing this, or just disable the test in this branch...

/retest

/lgtm
/approve
/override pull-kubevirt-e2e-k8s-1.18

/unhold


Android Java Support

Hi there, can we use this lib from java and is there any tutorial for that ? Because not all the apps are refactored to Kotlin yet

No, 2.0 is intentional made for Kotlin and there are things like extension functions that are now fundamental to it.

if so, delete this line from Readme :
The whole library was completely re-written in Kotlin. All the layouts and views were remade as well. This library is now designed specifically to work with Kotlin - it technically will work with Java, but not pleasantly.

That line still holds true. The library works from Java because Kotlin compiles to Java bytecode. But the features in the library which rely on extension functions won't be visible from Java.


MAINT: NumPy complex shim needed?

Following https://github.com/numpy/numpy/pull/24376, I'm having trouble building SciPy alongside NumPy main, with errors in Pythran that may relate to NumPy 2.0-geared API changes? Most likely complex-related changes?
[778/1588] Compiling C++ object scipy/stats/_stats_pythran.cpython-311-x86_64-linux-gnu.so.p/meson-generated_..__stats_pythran.cpp.o
FAILED: scipy/stats/_stats_pythran.cpython-311-x86_64-linux-gnu.so.p/meson-generated_..__stats_pythran.cpp.o
c++ -Iscipy/stats/_stats_pythran.cpython-311-x86_64-linux-gnu.so.p -Iscipy/stats -I../scipy/stats -I../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/pythran -I../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/numpy/core/include -I/home/treddy/github_projects/spack/opt/spack/linux-ubuntu22.04-skylake/gcc-11.3.0/python-3.11.2-4qncg2nqev5evxfmamde3e6rnb34b4ls/include/python3.11 -fvisibility=hidden -fvisibility-inlines-hidden -fdiagnostics-color=always -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -std=c++14 -O2 -g -fPIC -DENABLE_PYTHON_MODULE -D__PYTHRAN__=3 -DPYTHRAN_BLAS_NONE -Wno-cpp -Wno-deprecated-declarations -Wno-unused-but-set-variable -Wno-unused-function -Wno-unused-variable -Wno-int-in-bool-context -MD -MQ scipy/stats/_stats_pythran.cpython-311-x86_64-linux-gnu.so.p/meson-generated_..__stats_pythran.cpp.o -MF scipy/stats/_stats_pythran.cpython-311-x86_64-linux-gnu.so.p/meson-generated_..__stats_pythran.cpp.o.d -o scipy/stats/_stats_pythran.cpython-311-x86_64-linux-gnu.so.p/meson-generated_..__stats_pythran.cpp.o -c scipy/stats/_stats_pythran.cpp
In file included from ../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/pythran/pythonic/numpy/complex64.hpp:6,
                 from ../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/pythran/pythonic/types/ndarray.hpp:25,
                 from ../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/pythran/pythonic/types/tuple.hpp:8,
                 from ../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/pythran/pythonic/builtins/bool_.hpp:6,
                 from ../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/pythran/pythonic/types/NoneType.hpp:6,
                 from ../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/pythran/pythonic/types/slice.hpp:5,
                 from ../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/pythran/pythonic/core.hpp:46,
                 from scipy/stats/_stats_pythran.cpp:1:
../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/pythran/pythonic/types/complex.hpp: In static member function ‘static std::complex<_Tp> {anonymous}::pythonic::from_python<std::complex<_Tp> >::convert(PyObject*) [with T = long double; PyObject = _object]’:
../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/pythran/pythonic/types/complex.hpp:231:15: error: request for member ‘real’ in ‘val’, which is of non-class type ‘__complex__ long double’
  231 |   return {val.real, val.imag};
      |               ^~~~
../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/pythran/pythonic/types/complex.hpp:231:25: error: request for member ‘imag’ in ‘val’, which is of non-class type ‘__complex__ long double’
  231 |   return {val.real, val.imag};
      |                         ^~~~
../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/pythran/pythonic/types/complex.hpp:231:29: error: could not convert ‘{<expression error>, <expression error>}’ from ‘<brace-enclosed initializer list>’ to ‘std::complex<long double>’
  231 |   return {val.real, val.imag};
      |                             ^
      |                             |
      |                             <brace-enclosed initializer list>
../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/pythran/pythonic/types/complex.hpp: In static member function ‘static std::complex<_Tp> {anonymous}::pythonic::from_python<std::complex<_Tp> >::convert(PyObject*) [with T = float; PyObject = _object]’:
../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/pythran/pythonic/types/complex.hpp:246:15: error: request for member ‘real’ in ‘val’, which is of non-class type ‘__complex__ float’
  246 |   return {val.real, val.imag};
      |               ^~~~
../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/pythran/pythonic/types/complex.hpp:246:25: error: request for member ‘imag’ in ‘val’, which is of non-class type ‘__complex__ float’
  246 |   return {val.real, val.imag};
      |                         ^~~~
../../../python_venvs/py_311_scipy_dev/lib/python3.11/site-packages/pythran/pythonic/types/complex.hpp:246:29: error: could not convert ‘{<expression error>, <expression error>}’ from ‘<brace-enclosed initializer list>’ to ‘std::complex<float>’
  246 |   return {val.real, val.imag};
      |                             ^
      |                             |
      |                             <brace-enclosed initializer list>

cc @rgommers @ngoldbaum @mtsokol

pythran/pythonic/types/complex.hpp:231:15: error: request for member ‘real’ in ‘val’, which is of non-class type ‘__complex__ long double’

ping @lysnikolaou
Could this be related to changes to numpy's wrappers for the C99 complex types?
See https://github.com/numpy/numpy/pull/24085

This was already fixed in gh-2128. If you use a numpy nightly, you should temporarily install Pythran from latest master, just like the SciPy pre-release job does.
There's nothing else to do here I think, aside from waiting for a new Pythran release and bumping the minimum version needed in SciPy.

To be fair, the pythran master branch testsuite failed for me locally with NumPy main so I just gave up on it--I guess I should have tried using it anyway.
111 failed, 3402 passed, 56 skipped, 65 warnings in 2603.89s (0:43:23)
Many of the test failures proceed through distutils.errors.CompileError, but I think the problem is that pythran can't dig around for my OpenBLAS without using a ~/.pythranrc.


Conflict beetween ADE date component and AngularUI date

When I use the ADE date component in a page which contain an AngularUI date element, i have this error :

TypeError: Object [object Object] has no method 'destroy'

I think this should be fixed now.


[UpdateAzureSdkCodes.ps1] Install autorest from $SdkRepoRoot/package-lock.json

Fixes https://github.com/Azure/autorest.csharp/issues/4406
Reverts all changes from previous attempted fix #3862

@m-nash, @JoshLove-msft, @hallipr: This should fix the problem, and should be ready to merge as soon as someone signs off.


Set Locale

Hi, great work.
One thing I noticed is that this only works when we have a locale column in the users table.
But, what about when user decides to logout? Then at the login screen we don't have access to that.
A small tip is that the Nova::serving isn't enough, we have to use Nova::booted in order to have the translations available also at the login screen.
So, I thought that will be great if instead of updating and relying to a users' table field, to be able to play around with app()->setLocale() and therefore the login page issue will be solved because locale will be available across the project
without any users table dependancy.
I gave it a try and replaced the code where locale column being updated (at onSwitchLocale) and used app()->setLocale($request->post('locale'));
No luck although, when I use dd(app()->getLocale()); I am always getting en, the default locale.
Any thoughts about what might going wrong and does not set the locale?
Thanks in advance.

Hi, thank you very much for the thorough solution and explanation! I think will help others also in the future.
Cookie was one of the scenarios I was thinking as a workaround, I tried and it works properly, in fact I kept both of them, I am using locale field and cookie, the field in case that user wants to login from another pc so it will use the user's locale and cookie for login screen etc.


feat(biome_css_analyzer): implement noUnknownTypeSelector

Closes https://github.com/biomejs/biome/issues/2543
Summary
implements selector-type-no-unknown
Test Plan
Add snapshots.

Hey @Kazuhiro-Mimaki, are you still interested in this PR?

I'm sorry but I forgot this issue... 🙇♂
I'm still interested in and I'll do tomorrow.

Hi @Kazuhiro-Mimaki, I rebased your PR. Would you mind addressing the latest comments so we can merge them?


Contributing guide, set upstream not working

git branch --set-upstream-to=upstream/master doesn't work, tried on Windows.

Error snippet?


Can't extract public Facebook group video

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.09.20
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '--username', 'PRIVATE', '--password', 'PRIVATE', 'https://www.facebook.com/groups/203844543912465/permalink/235228274107425/']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.09.20
[debug] Python version 3.6.9 (CPython) - Linux-4.19.112+-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.8, ffprobe 3.4.8
[debug] Proxy map: {}
[facebook] Downloading login page
[facebook] Logging in
[facebook]<PHONE_NUMBER>07425: Downloading webpage
[facebook]<PHONE_NUMBER>07425: Downloading webpage
ERROR: This video is only available for registered users. Use --username and --password or --netrc to provide account credentials.
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/youtube_dl/YoutubeDL.py", line 797, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/lib/python3.6/dist-packages/youtube_dl/extractor/common.py", line 532, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/lib/python3.6/dist-packages/youtube_dl/extractor/facebook.py", line 484, in _real_extract
    video_id, fatal_if_no_video=True)
  File "/usr/local/lib/python3.6/dist-packages/youtube_dl/extractor/facebook.py", line 351, in _extract_from_url
    self.raise_login_required()
  File "/usr/local/lib/python3.6/dist-packages/youtube_dl/extractor/common.py", line 941, in raise_login_required
    expected=True)
youtube_dl.utils.ExtractorError: This video is only available for registered users. Use --username and --password or --netrc to provide account credentials.

Description

Hello! i was trying to download this video frm a facebook page. They posted it to a group, but the group is public, so anyone can view the video. When I first tried it, it said I had to login, so i put in my credentials. However, it still said I had to login even after I logged in. What's the issue?

try to use proxy, maybe you're limited by facebook

I've tried using a few proxies but they never work. Can you give me a proxy command that actually works?

Any update on this issue? Was it fixed?


Allow explicitly for null EPP resources in History objects

This change is

sorry for all the commits, just a lot of semantic merge conflicts + rebuilding based off a newly-not-broken master


RoutablePage routes without trailing slashes don't work

Issue Summary
Use case: I want to use my wagtail website without trailing slashes in the url.
(WAGTAIL_APPEND_SLASH = False and APPEND_SLASH = False)
When I create a RoutablePage with a @route without a trailing slash in the regex, the route will not resolve. This issue was also mentioned in https://github.com/wagtail/wagtail/issues/2871.
Steps to Reproduce

Start a new project with wagtail start myproject
Edit models.py as follows

from django.db import models
from django.shortcuts import HttpResponse

from wagtail.core.models import Page
from wagtail.contrib.routable_page.models import RoutablePageMixin, route

class HomePage(Page):
    pass

class TestPage(RoutablePageMixin, Page):
    @route(r"^year/(\d+)$")  # r"^year/(\d+)/$" does work however!
    def year(self, request, year):
        return HttpResponse("This is the page for year {}".format(year))

Add this to base.py

WAGTAIL_APPEND_SLASH = False
APPEND_SLASH = False

Migrate and create a new TestPage with slug test in the Wagtail admin.
Visiting http://localhost:8000/test/year/2020 won't work, 404 error.
Visiting http://localhost:8000/test/year/2020/ (with a trailing slash) won't work either.

I tried setting WAGTAIL_APPEND_SLASH = True APPEND_SLASH = True but that doesn't matter.
I think we should be able to define routable page routes without a trailing slash.
Technical details

Python version: 3.8.1.
Django version: 3.0.3.
Wagtail version: 2.8.
Browser version: Chrome 79 on macOS Catalina

I think this happens because a slash is automatically appended here.
Here is a possible fix.
I wanted to add a unit test, but whatever I do WAGTAIL_APPEND_SLASH is True so the serve_pattern is not properly set in the urlconf.
I tried to override settings, mock wagtail.core.utils.WAGTAIL_APPEND_SLASH but none of this work.
Thoughts?

I am currently running into this issue as well

Due to the way I have my serve() method set up, I didn't suffer from this bug when I added custom routes:
The method decorator has a trailing slash in its URLpattern to produce a resolver match for its method.
@route(r"^year/(\d+)/?$)
def year(self, request, year):

My page's serve() method returns a redirect response if the request URL ends with a slash.
def serve(self, request, *args, **kwargs):
    if request.path_info.endswith("/"):
        return HttpPermanentRedirect(request.path_info.rstrip("/")
    return super().serve(request, *args, **kwargs)

Hope this helps anyone encountering this problem.

Tried to fix that too, but the tests get stuck using WAGTAIL_APPEND_SLASH = True and it seems impossible to reset the urlpatterns at test runtime


Clean up: show usage when global --help present

Signed-off-by: Lei Jitang<EMAIL_ADDRESS>As the comment in https://github.com/docker/docker/blob/master/docker/docker.go#L58
says
// if global flag --help is present, regardless of what other options and commands there are,  		// just print the usage.
but not if provide a daemon flag.
This PR do two things:

handler global --help flag before handleGlobalDaemonFlag() and clientCli := client.NewDockerCli(stdin, stdout, stderr, clientFlags)
handleGlobalDaemonFlag() before  clientCli := client.NewDockerCli(stdin, stdout, stderr, clientFlags), before if provide daemon flag -d --daemon the clientcli is never used.

But, anyway, handleGlobalDaemonFlag() will deprecated in the future once -d --daemon is retired, so feel free to close if unnecessary.    :-)

LGTM

LGTM


New bug: Names that are 9 or 10 characters long result in an error when advertising is started

To reproduce: https://github.com/taks/esp32-nimble/commit/4e44350b814d76c660fe39a7ca1fbafc856bb0d9
Names I tried that don't work:

Bob456789
Bob4567890
123456789
1234567890
OpenSensor

Names I tried that do work:

Bob
12345678
OpenSensor_

I think this bug wasn't there before. I'll try to find out what commit broke this.

Actually I went back in past commits and I can't find a commit in 2024 that doesn't have this bug.

This bug was present since the beginning of the implementation.
There was a bug in the calculation of the data length of service_uuid.


Show Job ID on missing output error

Currently, a missing output file leads to an exception indicating the file, but not the job id. Tracing issues is much easier if at least the internal job ID is available to find matching log files (which may not contain the respective file name).

Thx - sorry - was going to check what formatting black wanted but didn't get around to it.


"g:OmniSharp.popup.position" does not seem to work. (throws error)

Hello, when using the new popup features I noticed that the option "g:OmniSharp.popup.position" does not seem to be working, When I set this option in my vimrc, I get the following error: "Undefined variable: g:OmniSharp".
Here is my vimrc entry:
let g:OmniSharp.popup.position = 'peek'

Here is the error:

You need to create it. The following are equivalent:
let g:OmniSharp = {}
let g:OmniSharp.popup = {}
let g:OmniSharp.popup.position = 'peek'

let g:OmniSharp = {
\ 'popup' = {
\   'position' = 'peek'
\ }
\}

My thinking was that you can end up declaring all options in a single variable, for consistency and to prevent polluting the global scope. But perhaps this is a bit confusing?

A possible alternative could be starting config with a call, e.g. call OmniSharp#Init() which initialises any config dictionaries required?

https://github.com/nickspoons/omnisharp-vim/blob/master/doc/omnisharp-vim.txt#L48
There's a better description of how I imagined this working. But I'm seriously rethinking it now, I think I'd better remove that single g:OmniSharp dict.

Oh, of course, somehow I did not occur to me. Thank you! <3

No problem, I hadn't been completely comfortable with that g:OmniSharp dictionary, I'm glad I got rid of it quickly, before it caused too much confusion for too many people


LaTeX build failing: missing figures

@ericagol The LaTeX build on travis is failing because there are two missing figures. I like to generate the figures on the fly with travis (rather than committing them with git) -- can you commit the scripts you use to generate them? I'll try to get julia to run on travis and get this working.

Is there a special syntax I need for the scripts in order for them to be generated automatically?

@ericagol I got things all set up on Travis to run the Julia scripts and build the paper. Currently your three figures are placeholders built by dummy julia scripts like this one. Just replace the code in there with your own, and make sure to keep the figure file names the same.

@rodluger I've replaced the dummy scripts with julia scripts.  I think I've added the needed dependencies in this julia/ directory, but let me know if it still has trouble generating the figures.

Seems to be working great. Thanks!


Adjustments responsive design

Now:

Unfortunately, the changes are not correct. As the current jsonConfig is structured, everything is correct and no changes are required for a responsive design on smaller screens

Changed

thx

fixed 3.1.0


Be explicit about which items are included in retention policy?

[Enter feedback here]
Re: "Excluding specific types of Exchange items from a retention policy", can you be explicit about the items which are included in the retention policy?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3a1c2e44-357c-8096-5bdf-40e205778ded
Version Independent ID: 19c7b5d9-1e0c-501b-7b70-df5aab48b36e
Content: Learn about retention for Exchange - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/retention-policies-exchange.md
Service: o365-seccomp
GitHub Login: @cabailey
Microsoft Alias: cabailey

Looking into this & waiting confirmation from engineering

This section needs to clarity that this PowerShell option is only if the retention action is retain-only (not retain and then delete, or delete-only) and I'm told that by default, retention for Exchange includes all items in a user mailbox with these exceptions: Contacts, calendar and tasks with no end date, documents, and distribution lists. I'll add this clarification to the documentation.
Is that sufficient or are you looking for the list of items that can be excluded with the PowerShell parameter ExcludedItemClasses?

The clarification is definitely useful, but for completeness I think it still needs to be explicit about the excludeditemclasses which can be used.
The reference page which is linked shows all of the classes, but I presume not all values are valid.

@qubika  For the Exchange locations, I've got confirmation that mail messages (includes drafts) with any attachments, tasks and calendar items when they have an end date, and notes are included.  I've added this to the page.  I'll follow up separately for a list of items that can be excluded by using PowerShell - the documentation for the ExcludedItemClasses parameter is not at all clear (https://docs.microsoft.com/en-us/powershell/module/exchange/set-retentioncompliancerule?view=exchange-ps#parameters)

@qubika  As a follow-up, the PowerShell description has now been updated to add a link to the message classes for Exchange but also discovered there is no validation on the values entered because it allows for customization.  That's why it can only talk about "examples".  Everything I've learned about this parameter (retain-only, no set list of values, no validation) leads me to conclude that this is not a standard configuration and for specialized use only, so I'm going to remove this section from the core docs.


Remove SimpleBlinds2 from config.schema.json and index.js

SimpleBlinds2 no longer exists as an accessory type since this commit https://github.com/iRayanKhan/homebridge-tuya/commit/f85c4e3d9a1c3c138162daed68f86b5c03c8d835

Merged into 3.1.0 as i will publish that as the next version. This will get merged into main upon testing completion.


fix nullary constructor libcpp set/map

fix for bug #4657

Thanks. Since you seem to have found this in previously working code, could you add a test, to make sure it keeps working in the future? There are test in tests/run/cpp_stl_*.pyx.

In the interests of getting this merged I added some tests in https://github.com/cython/cython/commit/cb035e34955649c93bc1808e3fec854f498925db (which can hopefully be picked over to here)

Hi @fried, do you still have time to work on this PR?
If it's not the case, I am interested in pursuing your work.

@jjerphan I think I'd already proposed a commit with the necessary tests, so i don't think there's anything to do.

This is fixed in #4528 as well along with a couple of other issues of the current iterator implementation. However at least the test of @da-woods should be added.

@maxbachmann feel free to just copy-paste my tests if you want - they aren't the sort of thing that needs credit preserving through git history or anything like that.

Closing because it was fixed elsewhere (https://github.com/cython/cython/commit/169876872f3cb6198971a1db07e5b8a9d12b3dac) - thanks for submitting the PR anyway