Artifacts credprovider & Gitlab runner

I wanted to write up my troubleshooting from yesterday as to how the microsoft artifacts credprovider and the Gitlab runner system interacted for me as it took me several hours to resolve yesterday and I don't want anyone else to go through the pain if they don't need to!
I received error : Unable to load the service index for source and error :   Response status code does not indicate success: 401 (Unauthorized) when attempting to run dotnet restore when attempting to connect to an Azure artifact feed from a CI / CD script I was running that uses the simple scripting interface for Gitlab runners on a self-hosted Windows machine. The environment variable VSS_NUGET_EXTERNAL_FEED_ENDPOINTS had been set using Gitlab CI CD Variables, and was checked using SET.
When I logged in as myself and tried a dotnet restore it worked. When I used PsExec to login as system (whoami - nt authority system - the same as the runner) dotnet restore also worked.
I was running the installcredprovider.ps1 script as part of my script, and was being told on repeated runs that it was already installed. However, when I navigated to C:\WINDOWS\system32\config\systemprofile\.nuget there was no plugins folder. When I ran installcredprovider.ps1 manually (when logged in as system) the folder appeared. After that my CI / CD script worked.
I'm not sure if this is a credprovider bug, a Gitlab runner bug, an interaction between the two, or just due to my setup!

@HughGrovesArup, when you executed the installcredprovider.ps1, was the script being run as system or another user context?  The cred provider script ought to install the providers into %userprofile%.nuget -- do you see it installed in any user profile other than system after executing the script?

The script was being run as a system account via the Gitlab Runner. I didn't see it being installed anywhere else. When I logged in as system and installed it worked fine, and thereafter the Gitlab Runner worked correctly.
whoami
nt authority\system

powershell.exe -noexit -file "%CI_PROJECT_DIR%\installcredprovider.ps1" -AddNetfx
Windows PowerShell
Copyright (C) Microsoft Corporation. All rights reserved.

The netcore and netfx Credential Providers are already in C:\WINDOWS\system32\config\systemprofile\.nuget\plugins

However, when using windows explorer from a user account the folder C:\WINDOWS\system32\config\systemprofile\.nuget\plugins did not exist. After logging is as system and running the script the plugins folder appeared.

I had similar problem. I ended up copying plugins folder from %UserProfile%\.nuget to C:\WINDOWS\system32\config\systemprofile\.nuget

Hi ,
We are closing this issues because there have been no activity. If you are still experiencing this issues, please feel free to reopen it.
Thanks


feature: Add configuration for build Docker image in server module

Ⅰ. Describe what this PR did
Add configuration for build Docker image in server module.
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
This PR can't add test cases in code.
Ⅳ. Describe how to verify it

Add REGISTRY_USERNAME and REGISTRY_PASSWORD environment which docker account and password.
Execute mvn clean package -P image and then will push latest tag image in your docker hub.

Ⅴ. Special notes for reviews

Need to add two environment parameter in Travis: REGISTRY_USERNAME and REGISTRY_PASSWORD
The namespace seata.io need to check if is belong to us, or need to change others.
Add one profile for docker image named image
There has two scene to build image:

Use image profile, will build and upload the latest tag image.
Use release profile, will build and upload the latest and ${project.version} tag images.

And there has a very important thing is that we need change path of registry.conf and  file.conf, now if we want mount configuration volume to container, we need add not only registry.conf and file.conf, but also all files under resources. So move the two files to another folder maybe is a good choice

Codecov Report

Merging #1885 into develop will decrease coverage by 0.05%.
The diff coverage is 36%.

@@              Coverage Diff              @@
##             develop    #1885      +/-   ##
=============================================
- Coverage      55.17%   55.12%   -0.06%
- Complexity      2379     2381       +2
=============================================
  Files            422      422
  Lines          14392    14409      +17
  Branches        1729     1732       +3
=============================================
+ Hits            7941     7943       +2
- Misses          5708     5718      +10
- Partials         743      748       +5

Impacted Files
Coverage Δ
Complexity Δ

...src/main/java/io/seata/server/ParameterParser.java
46.8% <36%> (-9.86%)
11 <6> (+3)

...in/java/io/seata/server/session/GlobalSession.java
84.54% <0%> (-0.49%)
67% <0%> (-1%)

...o/seata/server/coordinator/DefaultCoordinator.java
48.01% <0%> (-0.4%)
28% <0%> (ø)

...server/store/file/FileTransactionStoreManager.java
56.64% <0%> (-0.32%)
29% <0%> (ø)

@helloworlde How to control the account permissions of docker.io?

@helloworlde How to control the account permissions of docker.io?

As we discussed in Dingtalk group, use profile and environment.

Has been discussed with @helloworlde , use profile for configuration isolation, use travis env environment variable to control account.


[Suggestion] Support for Forestry Bottler

I would love to be able to add/remove recipes to Forestry's Bottler machine. I use it a lot because I don't have Thermal Expansion or any other mod installed where I can fill up containers with liquids.

Why? The Bottler should automatically add recipes for all fluid/fluidcontainer combinations.

Well, it'd be the same as the Filling Machine from IE. Iirc @BluSunrize has recipes for all fluid / container recipes as well but has custom support for those out of that area.

Sorry it took long to delay. Weekends are not ideal for me.
Even though there is no option to empty liquid into the machine and keep containers such as cans and capsules like in Thermal Expansion, I want to add custom recipes for some high tier items to force players to use the Bottler to fill up with certain fluid.

a cautious maybe


Collapse Issue

(function(){
var searchTerm, panelContainerId;
// Create a new contains that is case insensitive
$.expr[':'].containsCaseInsensitive = function (n, i, m) {
return jQuery(n).text().toUpperCase().indexOf(m[3].toUpperCase()) >= 0;
};
$('#accordion_search_bar').on('change keyup paste click', function () {
searchTerm = $(this).val();
$('#accordion > .panel').each(function () {
panelContainerId = '#' + $(this).attr('id');
$(panelContainerId + ':not(:containsCaseInsensitive(' + searchTerm + '))').hide();
$(panelContainerId + ':containsCaseInsensitive(' + searchTerm + ')').collapse (searchTerm);
});
});
This is my jQuery file, I have filtered my search and able to display. But it is displayed in collapse form, i want to expand and show the particular result.

Bug reports must include a live demo of the problem. Per our contributing guidelines, please create a reduced test case via JS Bin and report back with your link, Bootstrap version, and specific browser and OS details.
This is a saved reply.

On top of that, this looks like it's JS for v3. We're not maintaining that right now.


JQuery Ajax loading spinner only on first request

Describe the bug
Spinner is loading once at the first time of reload of the page.
Steps to reproduce

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

Expected behavior
bla bla bla
Screenshots
No response
Desktop

OS: [e.g. iOS]
Browser: [e.g. chrome, safari]
Version: [e.g. 22]

Smartphones

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
No response

@wasimreja Can you please assign this issue to me?


setup.py error

❓ Questions and Help
Before asking:

search the issues.
search the docs.

What is your question?
Installing the setup for fairseq using pip install --editable . gives following error
C:\Program Files (x86)\Microsoft Visual Studio\2019\BuildTools\VC\Tools\MSVC
\14.27.29110\include\yvals.h(12): fatal error C1083: Cannot open include file: '
crtdbg.h': No such file or directory
Code

What have you tried?
What's your environment?

fairseq Version (e.g., 1.0 or master):
PyTorch Version (e.g., 1.0)
OS (e.g., Linux):
How you installed fairseq (pip, source):
Build command you used (if compiling from source):
Python version:
CUDA/cuDNN version:
GPU models and configuration:
Any other relevant information:

Sorry, I don't have access to a windows machine to test this. Hopefully someone from the community can help (or if you figure it out please share the solution back!)

Was there a solution here? All I see is the person who opened it closed it.


best version for Symfony2.8

How can I find out what's the suited version to a Symfony version?
Thx,
Kim

You can safely go for the latest release 5.0 from symfony 2.3+ et 3.0 and php 5.3+. If you use earlier version of symfony looking at the composer.json for a given tag will help :)

Awesome! Thanks.


story63

Query/Filter Action implemented under BDD; fix a small error

Codecov Report

Merging #68 (2e77ef9) into master (17dcc17) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #68   +/-   ##
=======================================
  Coverage   97.82%   97.82%
=======================================
  Files           7        7
  Lines         322      322
=======================================
  Hits          315      315
  Misses          7        7

Continue to review full report at Codecov.

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


Package name should be WinCopies.PushBinding

Thus, the user knows that this package was built from a fork.

This issue has been fixed and the old NuGet package will be marked as deprecated.


Working on genieacs with docker, can I set static IP for the ubuntu container ?

general@general-M93p:~$ sudo docker run -t -i ubuntu:14.04 ip link add vipr2 link eth0 type macvlan mode bridge
[sudo] password for general:
RTNETLINK answers: Operation not permitted
general@general-M93p:~$
I tried like this from the suggestion of someone. But I got operation not permitted error ..
So, to run genieacs I need a static IP. When I run the container, its IP gets changed each time..
Can I get a solution for it ?
Thank you.

@Siddeshi For now I'd suggest something like github.com/jpetazzo/pipework for something like that.
Also, by default containers do not have the capability to manipulate network settings unless you explicitly enable it with your docker run command by adding --cap-add NET_ADMIN
For support issues, please use one of the following:

#docker on freenode IRC
docker-user mailing list
forums.docker.com

Closing since this isn't a docker issue.
Please feel free to reach out in any of the mentioned support forums above.

Thank you.


[Web] Updating login page

Updating login page with new clarity components.

@gnomeontherun
Is this something we want to merge?


Redirect after login

Description of bug
When having more than one language activated the redirect after login doesn't work
Steps to reproduce
List the steps to reproduce the behavior:

Go to security;
Set a redirect after login page;
Go to language;
Enable a second language, for example dutch;
login with a registered user;
You wil see it doesn't redirect to the page but goes to the homepage;
Deactivate the second language;
You will see the redirect does work again

Current result
After login with a registered user it doesn't redirect to the redirect page
Expected result
It should redirect to the page set under security - redirect after login page
Error log
no errors in the log
Affected version

[x] 9.2.1

Affected browser
All browsers:

[ ] Chrome
[ ] Firefox
[ ] Safari
[ ] Internet Explorer
[ ] Edge

@wbonekamp This issue will be included in the 9.3.0 release

@mitchelsellers, @ohine , @sleupold
When we are on home page and using login popup, then this feature redirect us to page set in "Redirect After Login" setting.
When logging into the application via URL in browser either {siteurl}/login or {siteurl}/login.aspx then what should be the behavior of the application, currently it is redirecting to home page

@Mhtshum
IMO, if a page is set for "Redirect after Login", the user should always be redirected to this page unless another page is specified in URL as returnURL. Note: page set in site setting "Redirect After Login" is language specific, i.e. it depends on the language used after login, which page gets displayed in ML sites.

@mitchelsellers
I agree that this should not be different whether using popup or not.
However, the page after login should be overridable via an URL parameter, there are a number of use cases for this.


chore(noetic): release 1.8.0

:robot: I have created a release beep boop
1.8.0 (2023-08-28)
Features

add rosdeps update warning (#15) (52b77af)
improve ros shutdown and codebase (#20) (a263b6f)
migrate to gymnasium and cleanup codebase (#9) (e4fe84d)

Bug Fixes

fix 'PickAndPlace' reset velocities (#19) (86a1da4)
fix action/observation space dtype (#17) (c484443)
remove singleton anti-pattern (#7) (c2f163c)

Documentation

add pre-commit note (#16) (e3c1819)
update BLC reference to SLC (27a098b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/rickstaa/ros-gazebo-gym/releases/tag/v1.8.0 :sunflower:


need to remove office 2016 ProPlus and install O365

we have about 30 machines on our domain that someone used the wrong install file and installed Office 2016 instead of using the script I created for O365.  I knwo I can easily go to each machine and manually uninstall 2016 then run the script but that will take a very long time to do all 30 machines.   I was hoping to create a GPO to uninstall 2016 and install 2016 in a fairly easy all in one package.    there is no fixit for 2016 and cant use offscrub as far as I can tell.  I am thinking I may have to move all the copmputers to another OU to be able to do this.  it would be nice to have the script look for office 20016 and if it exist to remove it then install O365 so I can just apply that script to a GPO on the whole domain without worrying if it will uninstall the already existing 300+ other versions of O365

Have you heard anything back regarding this? We have a similar issue


Specify nonzero positive value for items created with seeds.rb

The default value for Item value is zero, and that make sense for real world use.
However, for seed (sample) data, it might make more sense to use a nonzero value, so that we can see price extensions (cost per item times quantity) and totals.
I propose putting something like this (or something more performant) at the end of seeds.rb:
Item.all.each { |i| i.value_in_cents = 100; i.save }

I am attaching 2 screenshots to show what the donation page looks like with and without the change. With the change, we can easily verify visually that the totals are working correctly, since each item's value is identical, $1.00.

This will also help because I wanted to simplify the calculation of the total value and this will easily enable visually verifying that the calculation is still correct.

+1 for this, except instead of i.value_in_cents = 100, how about i.value_in_cents ||= 100, just in case something was set previously to it? (or if ||= doesnt' work because it's initializing to 0, then do something like Item.where(value: 0).each { ... }?

This works:
Item.where(value_in_cents: 0).each { |item| item.value_in_cents = 100; item.save }
I tested it and all records had a value of 100 with both approaches, but your way is safer in case anyone adds something later to seeds.rb to initialize the values before it gets to this code.

Why not ? Item.where(value_in_cents: 0).limit(1).update_all(value_in_cents: 100)

@JuarezLustosa Yes, your way will use SQL instead of AR, and is the better way to do that. I just was not familiar with that syntax. (I'm a long time Rubyist but have little Rails experience.) However, I don't think we want the limit, do we? We want to operate on all records whose value is zero.

@armahillo This issue is straightforward, I guess. What do you think about the "Good First Issue" label? I can grab it, but I think that would be more productive to let it open to new contributors.

I think it would be a good first issue, but on the other hand, it would be nice if this were done ASAP (I am willing to do it also). I'm hoping to contribute many issues and pull requests, and many build upon each other, so delays can impair my productivity. If we choose to make it a first issue, could we do so ASAP?

@JuarezLustosa Yes, your way will use SQL instead of Ruby, and is the better way to do that. I just was not familiar with that syntax. (I'm a long time Rubyist but have little Rails experience.) However, I don't think we want the limit, do we? We want to operate on all records whose value is zero.

Yeah, don't need. Just in case.

👋 Hi! I'd be happy to work on this issue!

That works for me but we should probably wait for @armahillo to confirm.

@katestud It's all yours, if you're still available to work on this.

@keithrbennett I think doing this for some items is a great idea, but I don't think we should do it for all of them, as my impression is many diaperbanks will have Items that do not have this filled out.
In general, for seeds data, we should NOT fill a value for an attribute if that attribute is not required.

That's an interesting point.  I was not aware that the value_in_cents field was optional. My main motivation was to be able to see the data in the UI, to ensure that it was displayed correctly. For that, omitting all the values doesn't seem like a good idea either. So maybe some of the values?
On the other hand, if we assume that some diaper banks will want values for all items, then maybe it does make sense to have values for all items. Better to display values in the optional columns to see what it would look like, and tell people that the field is optional, rather than not be able to see what it would like if the option were exercised, no?

@mdworken Sorry about the way I worded that comment. You were suggesting assigning values to some might be best, and I made it sound like you were suggesting assigning to none of them. I'm editing my comment to correct that.

It looks like another manifestation of this issue is the issue reported by @armahillo at https://github.com/rubyforgood/diaper/issues/1237. Although assigning values to some of the items may work, I think the only practical way to guarantee that all pages displaying values will have values to display is to assign values to all the items.

It sounds like there are a couple things happening.

We want to feel confident that totals are displaying on the donation page
We want to feel confident that the totals we are displaying are accurate

I think that controller tests (and maybe some unit tests) as outlined in https://github.com/rubyforgood/diaper/issues/1237 can address both of those requirements, but it sounds like we also want a quick visual check when working locally, which can be addressed by adding totals to some of the items in the seeds. Is that an accurate summary? If so, I'd be happy to update https://github.com/rubyforgood/diaper/pull/1227 to only add values to the some of the seed items. I'd also be happy to help with writing some tests.

@katestud yes, that is exactly correct!

@jeduardo824 You're correct! Looks like it's moot now, but good call on "Good First Issue" :) I've added it anyways because why not, haha :)

@katestud @mdworken  Would we want to be sure that there would be values visible on any page displayed with the seed data? If so, how would we determine how many or which items get values? (I don't know which items are used where, and are they randomized, etc.)

@keithrbennett I think that would probably be helpful! I need to dig more closely into how the seeds are set up to see if that's doable, but I will try to find a way. 😎


Doesn't work without listeners

Most of the code crashes when scrolling without listeners. Let me show you where the problem is:
@Override
    protected void onItemSelected(int position, Object item) {
        if (onMinuteSelectedListener != null) {
            onMinuteSelectedListener.onMinuteSelected(this, position, convertItemToMinute(item));
        }
    }

    @Override
    protected void onItemCurrentScroll(int position, Object item) {
        if (lastScrollPosition != position) {
            onMinuteSelectedListener.onMinuteCurrentScrolled(this, position, convertItemToMinute(item));
            if (lastScrollPosition == 11 && position == 0)
                if (onMinuteSelectedListener != null) {
                    onMinuteSelectedListener.onMinuteScrolledNewHour(this);
                }
            lastScrollPosition = position;
        }
    }

In the onItemSelected, you check whether onMinuteSelectedListener is set or not. In onItemCurrentScroll you don't (actually you do, but not everywhere). This doesn't only apply to WheelMinutePicker but any WheelPicker.
I understand that normally. programmers would attach the listener but for a quick prototyping when you don't actually use selected values this doesn't work.
PS: Tested against latest version 1.2.0

can you make a pull request for that ?


[1.10.2] Oxygen vent won't turn on.

After setting up a space station, I placed down a power source, oxygen vent ontop of it, C02 scrubbers on both sides with cartridges, and gave it a redstone signal to power on, but nothing happens. Am I missing something?
here is a picture of the setup with level set to on.
http://i.imgur.com/XzPuPtd.png

Only thing I can think of is that the room may not be sealed for some reason.  Also can you verify that the machine actually has power in the buffer?

The machine definitely has power in the buffer.
Here's some screenshots of the room:
http://i.imgur.com/PjaDIjS.jpg
http://i.imgur.com/LADOkrp.jpg
http://i.imgur.com/NMaxik6.png
The walls are made of structural glass from Mekanism, so it can be a bit hard to see all the edges, but I assure you the walls are solid :)
The airlock area is partly made out of chisel slabs,a s you can see in the picture, would that cause it? Also, the airlock doors should create as seal, yes?

you can always lookup the chisel block registry name (f3 + h I think to enable debug tags, then hover over the item) name and add it to the config. But they would also seal when you leave half the block horizontaly open to air


Can't reach container by templated hostname 1.13

Description
Created service with templated hostname
https://github.com/docker/swarmkit/pull/1738
Steps to reproduce the issue:

docker service create --name tmpl --hostname "{{.Service.Name}}-{{.Task.Slot}}" --network overlords --replicas 2 nginx:latest
docker exec -ti tmpl.1.53l54w9j8p38kewybm0341bsc /bin/bash
apt-get update && apt-get install links netcat -y

ping -c1 tmpl-1 - ok
links http://tmpl-1 - ok
nc -zv tmpl-1 80 - ok
all above for tmpl-2 - unknown host
all above for tmpl-2 CONTAINER ID - ok
Describe the results you received:
Got unknown host reaching container by templated name
Describe the results you expected:
Connecting to container by its templated name
Additional information you deem important (e.g. issue happens only occasionally):
docker 1.13.0~rc2-0~debian-jessie amd64
Output of docker version:
Client:
 Version:      1.13.0-rc2
 API version:  1.25
 Go version:   go1.7.3
 Git commit:   1f9b3ef
 Built:        Wed Nov 23 06:17:45 2016
 OS/Arch:      linux/amd64

Server:
 Version:             1.13.0-rc2
 API version:         1.25
 Minimum API version: 1.12
 Go version:          go1.7.3
 Git commit:          1f9b3ef
 Built:               Wed Nov 23 06:17:45 2016
 OS/Arch:             linux/amd64
 Experimental:        false

Output of docker info:
Containers: 6
 Running: 4
 Paused: 0
 Stopped: 2
Images: 1
Server Version: 1.13.0-rc2
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 15
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
Swarm: active
 NodeID: ia9vbuazsu1rfmyrkck9oekiw
 Is Manager: true
 ClusterID: jtnyl5pllcxoicbnxuhm7qsao
 Managers: 3
 Nodes: 3
 Orchestration:
  Task History Retention Limit: 5
 Raft:
  Snapshot Interval: 10000
  Number of Old Snapshots to Retain: 0
  Heartbeat Tick: 1
  Election Tick: 3
 Dispatcher:
  Heartbeat Period: 5 seconds
 CA Configuration:
  Expiry Duration: 3 months
 Node Address: <IP_ADDRESS>
 Manager Addresses:
  <IP_ADDRESS>:2377
  <IP_ADDRESS>:2377
  <IP_ADDRESS>:2377
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 03e5862ec0d8d3b3f750e19fca3ee367e13c090e
runc version: 51371867a01c467f08af739783b8beafc154c4d7
init version: 949e6fa
Kernel Version: 3.16.0-4-amd64
Operating System: Debian GNU/Linux 8 (jessie)
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 3.686 GiB
Name: ip-172-31-60-101
ID: EYJD:4JFM:IOGW:4PBL:AQNK:AINV:NOXH:XBRR:NC52:VT4D:E4FR:M4V7
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
WARNING: No memory limit support
WARNING: No swap limit support
WARNING: No kernel memory limit support
WARNING: No oom kill disable support
WARNING: No cpu cfs quota support
WARNING: No cpu cfs period support
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

Additional environment details (AWS, VirtualBox, physical, etc.):
AWS/physical
Thanks a lot in advance!
Really hoping this will help to sort out (at least) some difficulties with clustering hostname-demanding software like couchbase and rabbitmq.

The --hostname of the container is not the name under which the container is reachable on the network, it only sets hostname inside the container. To access a service on the network, use the service name (tmpl)  or the container's name (tmpl.1.53l54w9j8p38kewybm0341bsc) to access individual containers

Thanks for the info, using consul anyway.


Getting "Runtime node closed unexpectedly - no connection" on example tpu notebook

Hi. I wanted to try it with coral but my livebook dies on the last cell in the tpu.livemd
I tried to run it as mix task but also getting
[1]    55100 segmentation fault (core dumped)  mix classify_image -m /tmp/mobilenet_v2_1.0_224_inat_bird_quant_edgetpu.tflit

I have the coredump but it's 175MB in size

Hi @dkuku, which version of tflite_elixir was it in the Notebook dependencies and setup cell? And could you please let me know if you're on an x86_64 or aarch64 Linux? Thanks!

I was using the latest master on intel. In the deps there is 0.1.5

Oh I kinda know what went wrong here. I haven't added reference counting mechanism to the EdgeTPUContext and related things. So Erlang dropped some resources while they are meant to live as least as long as the Interpreter.
I'll fix this in the next version.
With that said, if you would to test and see if it can communicate with your TPU, then the normal model also works on TPU (but not optimised). For example,
$ mix classify_image --model test/test_data/mobilenet_v2_1.0_224_inat_bird_quant.tflite \
  --input test/test_data/parrot.jpeg \
  --labels test/test_data/inat_bird_labels.txt \
  --use-tpu \
  --tpu usb
----INFERENCE TIME----
78.4ms
-------RESULTS--------
Ara macao (Scarlet Macaw): 0.70703

this works:
mix classify_image --model test/test_data/mobilenet_v2_1.0_224_inat_bird_quant.tflite \                                                                             1 ✘  1 ☑  3 
  --input test/test_data/parrot.jpeg \
  --labels test/test_data/inat_bird_labels.txt \
  --use-tpu \
  --tpu usb
----INFERENCE TIME----
713.8ms
-------RESULTS--------
Ara macao (Scarlet Macaw): 0.70703

Hi @dkuku, v0.1.6 is out and it should work as expected!

Thanks @cocoa-xu
It works 🥳, Thanks.


add large AP handle service and api of proxy

Signed-off-by: jinrui<EMAIL_ADDRESS>

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


Flakey tests in TestWorkflowTemplateSuite

TestWorkflowTemplateSuite/TestSubmitWorkflowTemplate
TestWorkflowTemplateSuite/TestNestedWorkflowTemplate
TestWorkflowTemplateSuite/TestSubmitWorkflowTemplateWithEnum
Example runs:
https://github.com/argoproj/argo/pull/4823/checks?check_run_id=1651690361
https://github.com/argoproj/argo/pull/4427/checks?check_run_id=1532851026

It might be a k3d issue. Workflow didn't start on time. Currently, it is not happening.  We will keep eye on it.

It might be a k3d issue. Workflow didn't start on time. Currently, it is not happening.  We will keep eye on it.


build: disable all shared libs

Disable shared libs for the other targets. Never used and simplifies builds.

Just curious: why did you close this branch?

Ummm good question. By accident!


UnboundLocalError: local variable 'status' referenced before assignment

The output from python -c "import tensorflow; print(tensorflow.__version__)":
python -c "import tensorflow; print(tensorflow.version)"
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcublas.so locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcudnn.so locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcufft.so locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcuda.so.1 locally
I tensorflow/stream_executor/dso_loader.cc:128] successfully opened CUDA library libcurand.so locally
0.12.1
Here's the error:
Epoch 2/2
19125/19125 [==============================] - 78s - loss: 0.4568 - acc: 0.8681 - val_loss: 2.1682 - val_acc: 0.4104
Exception ignored in: <bound method BaseSession.del of <tensorflow.python.client.session.Session object at 0x7f747161e160>>
Traceback (most recent call last):
File "/home/p3/anaconda3/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 581, in del
UnboundLocalError: local variable 'status' referenced before assignment

This should've been fixed in https://github.com/tensorflow/tensorflow/pull/7386 , can you see if error persists in latest nightly version?

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

I've got this in tensorflow (1.0.1).

Still persists.

Happens intermittently with tensorflow/tensorflow:1.0.1-gpu-py3 docker image and keras.
Exception ignored in: <bound method BaseSession.__del__ of <tensorflow.python.client.session.Session object at 0x7f3a69e8c470>>
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 582, in __del__
UnboundLocalError: local variable 'status' referenced before assignment

Also happened for me upon saving a model in keras:
>>> tf.__version__
'1.0.1'
>>> keras.__version__
'2.0.4'

Happend to me too today on our GPU-Cluster:
uname -a
Linux [...] 4.8.0-46-generic #49~16.04.1-Ubuntu SMP Fri Mar 31 14:51:03 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

On a GeForce GTX 980 Ti.
python --version
Python 3.5.3 :: Continuum Analytics, Inc.

>>> import tensorflow
I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcublas.so.8.0 locally
I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcudnn.so.5 locally
I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcufft.so.8.0 locally
I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcuda.so.1 locally
I tensorflow/stream_executor/dso_loader.cc:135] successfully opened CUDA library libcurand.so.8.0 locally
>>> tensorflow.__version__
'1.0.1'
>>> import keras
Using TensorFlow backend.
>>> keras.__version__
'2.0.3'

@zheng-xq , could you take a look?

The exception comes from session.py. Passing to Derek fro now.

I'm pretty sure this has been fixed since the 1.1 release. I'm closing this issue for now, but feel free to reopen it if you can reproduce with a newer version of TensorFlow.

I got this error today while training my network on keras.
Epoch 10/10
19308/19286 [==============================] - 62s - loss: 0.0220 - acc: 0.1795 - val_loss: 0.0231 - val_acc: 0.1846
dict_keys(['val_acc', 'val_loss', 'acc', 'loss'])
Acc
[0.1782905629139073, 0.1796146675076869, 0.18056705298013245, 0.17775015535748767, 0.18656870860927152, 0.17137973897756442, 0.17694536423841059, 0.18142738762061394, 0.17860099337748345, 0.17951108349797049]
Loss
[0.032948918208874613, 0.029692213859633675, 0.028213497242020654, 0.027055541726004047, 0.025659046015572666, 0.025640698771453779, 0.024098622677416005, 0.023113698205617934, 0.02143701246557202, 0.02202132222925083]
Validation Loss
[0.027665206672329651, 0.026977688277141799, 0.026762272280297782, 0.025648139429476319, 0.024419905385002494, 0.024124456143497338, 0.024165717970677895, 0.022718481179619013, 0.021766701916234244, 0.023056170290434046]
Exception ignored in: <bound method BaseSession.del of <tensorflow.python.client.session.Session object at 0x7fc9535b4a90>>
Traceback (most recent call last):
File "/home/carnd/anaconda3/envs/carnd-term1/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 581, in del
UnboundLocalError: local variable 'status' referenced before assignment

I also get this error, version 1.5
2018-03-19 02:51:05.913 EDT The replica master 0 exited with a non-zero status of 1. Termination reason: Error. Traceback (most recent call last): File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main "__main__", fname, loader, pkg_name) File "/usr/lib/python2.7/runpy.py", line 72, in _run_code exec code in run_globals File "/root/.local/lib/python2.7/site-packages/object_detection/train.py", line 167, in <module> tf.app.run() File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 124, in run _sys.exit(main(argv)) File "/root/.local/lib/python2.7/site-packages/object_detection/train.py", line 163, in main worker_job_name, is_chief, FLAGS.train_dir) File "/root/.local/lib/python2.7/site-packages/object_detection/trainer.py", line 366, in train saver=saver) File "/usr/local/lib/python2.7/dist-packages/tensorflow/contrib/slim/python/slim/learning.py", line 790, in train return total_loss UnboundLocalError: local variable 'total_loss' referenced before assignment To find out more about why your job exited please check the logs: https://console.cloud.google.com/logs/viewer?project=301256752837&resource=ml_job%2Fjob_id%2Fshreya_object_detection_1521441960&advancedFilter=resource.type%3D%22ml_job%22%0Aresource.labels.job_id%3D%22shreya_object_detection_1521441960%22

i also get this error, version 1.6
Traceback (most recent call last):   File "object_detection/train.py", line 167, in <module>     tf.app.run()   File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 126, in run     _sys.exit(main(argv))   File "object_detection/train.py", line 163, in main     worker_job_name, is_chief, FLAGS.train_dir)   File "/usr/local/lib/python2.7/dist-packages/tensorflow/models/research/object_detection/trainer.py", line 360, in train     saver=saver)   File "/usr/local/lib/python2.7/dist-packages/tensorflow/contrib/slim/python/slim/learning.py", line 791, in train     return total_loss UnboundLocalError: local variable 'total_loss' referenced before assignment

I also get a similar error when attempting to train using the object detection API (identical to the comment above)
Traceback (most recent call last):
File "object_detection/train.py", line 167, in
tf.app.run()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 126, in run
_sys.exit(main(argv))
File "object_detection/train.py", line 163, in main
worker_job_name, is_chief, FLAGS.train_dir)
File "/home/dave/tensorflow-master/models/research/object_detection/trainer.py", line 370, in train
saver=saver)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/contrib/slim/python/slim/learning.py", line 791, in train
return total_loss
UnboundLocalError: local variable 'total_loss' referenced before assignment


Access control defaults should not be permissive

It's preferable if the defaults are not permissive, i.e. they error or deny access. Users should not have a permissive system by forgetting to configure access control, so users will have to configure access control to have a functioning system.
This is difficult in the short term, because our configuration (including for unit tests and integration tests) is not confined to one place. Therefore, because access control is baked into the IM, tests can fail if they haven't configured access control. Having more permissive defaults helps us develop until we reach feature complete, and until we improve our startup/shutdown story (including for tests).
Production code will also include the access control cluster, which will configure the access control module. Therefore, all a build (such as an example app) needs to do is include the cluster, and the system will be correctly configured for operation.
Once we hit feature complete, we can reorganize this (e.g. find all the tests that need to also configure access control) and make the defaults less permissive again.

Spec-compliance:

The Access Control system is rule-based with no implicit access permitted by default. Access to a Node's Targets is denied unless the Access Control system grants the required privilege level to a given Subject to interact with given Targets on that Node. Initial Access Control privileges are <<ref_ACL_Bootstrapping, bootstrapped>> during the commissioning phase, and maintained thereafter by Administrators.
All access privileges, from the <<ref_AccessControlEntryPrivilegeEnum>> enumeration, SHALL be granted by Access Control. Thus, the Initiator ("Subject" hereafter) of any Interaction Model action SHALL NOT succeed in executing a given action on a Node's Target unless that Node's Access Control system explicitly grants the required privilege to that Subject for that particular action.


Add plotly to requirements in setup.py

  File "/Users/alana/ls/envs/magic/lib/python3.9/site-packages/alana/color.py", line 6, in <module>
    from plotly.graph_objs._figure import Figure
ModuleNotFoundError: No module named 'plotly'

We might also be able to remedy just by not importing alana.color in __init__.py, if we want to avoid importing plotly for some reason.


Port Supplier issue when filling in process list getting memory corruption.

I'm getting a failure when loading a Node within the
The failure appears in out CProcess class which is implemented in the following way:
public class CProcess : IDebugProcess2,
                        IDebugProcessEx2,
                        IDebugProgram2,
                        IDebugProgramEx2,
                        IDebugProgramNode2,
                        IDebugProgramEngines2

{
....
}
The crash comes in while attaching to a process dialog and changing the connection target.

The call to: public int GetInfo(uint Fields, PROCESS_INFO[] pProcessInfo) to fill the process information looks correct.
The sequence continues until after calling EnumPossibleEngines. Then it crashes with the followng:
System.AccessViolationException
HResult=0x80004003
Message=Attempted to read or write protected memory. This is often an indication that other memory is corrupt.
Source=
StackTrace:

This exception was originally thrown at this call stack:
Microsoft.VisualStudio.PlatformUI.VsCommand.Execute(object)
MS.Internal.Commands.CommandHelpers.CriticalExecuteCommandSource(System.Windows.Input.ICommandSource, bool)
System.Windows.Controls.MenuItem.InvokeClickAfterRender(object)
System.Windows.Threading.ExceptionWrapper.InternalRealCall(System.Delegate, object, int)
System.Windows.Threading.ExceptionWrapper.TryCatchWhen(object, System.Delegate, object, int, System.Delegate)
System.Windows.Threading.DispatcherOperation.InvokeImpl()
MS.Internal.CulturePreservingExecutionContext.CallbackWrapper(object)
System.Threading.ExecutionContext.RunInternal(System.Threading.ExecutionContext, System.Threading.ContextCallback, object, bool) in executioncontext.cs
System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, object, bool) in executioncontext.cs
System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, object) in executioncontext.cs
...
[Call Stack Truncated]
====================================================================================
Microsoft.VisualStudio.PlatformUI.VsCommand.Execute(object), I'm not sure what this is a hook to.
Is there any sequence diagrams for the Attach to process dialog, so I can see what is causing this crash?
I've saved the m_pProcessInfo within the class to see if this may be the issue but it doesn't appear to be.
Also it doesn't appear to be the EnumPossibleEngines call because it's returning HR = ERROR_INSUFFICIENT_BUFFER as recomended by the docs.
public int EnumPossibleEngines(uint celtBuffer, Guid[] rgguidEngines, ref uint pceltEngines)
{
m_IServer.LogDebug(10, "CProcess: EnumPossibleEngines(uint celtBuffer, Guid[] rgguidEngines, ref uint pceltEngines)");
// There is only one supported engine, the INtime debugger
//See:
//https://docs.microsoft.com/en-us/visualstudio/extensibility/debugger/reference/idebugprogramengines2-enumpossibleengines?view=vs-2022
        if ((0 == celtBuffer) && (null == rgguidEngines))
        {
            pceltEngines = 1;
            unchecked
            {
                int hr = (int)0x80070000 + ERROR_INSUFFICIENT_BUFFER;
                return hr;
            }
        }
        else if (0 == celtBuffer)
        {
            return VSConstants.E_INVALIDARG;
        }
        else if (celtBuffer == 0)
        {
            ThrowWin32Error(ERROR_INSUFFICIENT_BUFFER);
        }
        else if (null == rgguidEngines)
        {
            return VSConstants.E_INVALIDARG;
        }

        rgguidEngines[0] = Constants.DebugEngineGuid;
        return VSConstants.S_OK;
    }

Any help would be appreciated, so I can port this over correctly.

Here is a screen shot of the ProcessInfo before the crash.

I aplogize I noticed the first sentence is incomplete.
It should be..     I'm getting a failure when loading a Node within the Connection Target dropdown.

More Information,
Using Native Debugger,  The call that crashes in Visual Studio is:
00007FF8249D7DEC  call        CVsAttachToProcessesDialog::CProcessInfo::CProcessInfo (07FF8249CF960h)
Not sure if this is related to PROCESS_INFO that is set during GetInfo

More information:
00007FF8249D7DEC  call        CVsAttachToProcessesDialog::CProcessInfo::CProcessInfo (07FF8249CF960h)
00007FF8249CF9CA  lea         rax,[CDebugEventSink::`vftable' (07FF824BB18D0h)]
Our new implementation doesn't call GetInfo for the count of EnumProcesses.
It calls GetPhysicalProcessId and GetProcessId for the number of processes in the Enum.
However, for GetInfo it is only getting called once.
Where as in our visual studio 2017 implementation using the original Port Suppler,  GetInfo get's called for the number of processes in the enum.
That's where stuff get's broken and fails to attach to the process dialog.

Not sure yet, if enumProcesses got corrupted.  But by the fact that visual studio aquires GetPhysicalProcessId , and GetProcessId  for the correct number of items, it appears to break betwen those calls and the GetInfo Calls.

I put a callback to the Port for the enumProcesses.  When it enters GetInfo, I'm able to look at it and it appears to be ok.

The index that the GetInfo is using of the enum is 0, for the first item, and gets that info, but doesn't continue to get the rest of the processes information.
It just goes through the sequence for process 0 and crashes when it exits EnumPossibleEngines.

Any help would be appreciated with this.
I created a test to see if I could fill in the PROCESSINFO with some psuedo information.
IDebugProcess2.GetInfo(...)
This is a psuedo implementation to track down my issue.  I am setting the strings in the pProcessInfo with psuedo values.
And I end up getting memory corruption, with the strings.
If the strings, and the Fields are not filled in for the items listed next, then GetInfo is called for each item in the Enumerator.
Comment out the following in the Example Code:
bstrAttachedSessionName , bstrFileName , bstrBaseName, bstrTitle, including the Field for those items.
Of course the process window doesn't show any items, but there isn't a problem  with the strings.
Example Code:
    public int GetInfo(uint Fields, PROCESS_INFO[] pProcessInfo)
    {
        m_IServer.LogDebug(10, "CProcess: GetInfo(uint Fields, PROCESS_INFO[] pProcessInfo)");
        INtimeProcessEnum enumProcs = (INtimeProcessEnum)m_Port.EnumProcs;
        m_pProcessInfo = pProcessInfo;

        pProcessInfo[0].Fields = 0;

        pProcessInfo[0].bstrAttachedSessionName = String.Format("{0}", "bstrAttachedSessionName");
        pProcessInfo[0].Fields |= (uint)enum_PROCESS_INFO_FIELDS.PIF_ATTACHED_SESSION_NAME;

        pProcessInfo[0].bstrFileName = String.Format("{0}", "bstrFileName");
        pProcessInfo[0].Fields |= (uint)enum_PROCESS_INFO_FIELDS.PIF_FILE_NAME
            ;
        pProcessInfo[0].bstrBaseName = String.Format("{0}", "bstrBaseName");
        pProcessInfo[0].Fields |= (uint)enum_PROCESS_INFO_FIELDS.PIF_BASE_NAME;

        pProcessInfo[0].bstrTitle = String.Format("{0}", "bstrTitle");
        pProcessInfo[0].Fields |= (uint)enum_PROCESS_INFO_FIELDS.PIF_TITLE;

        //pProcessInfo[0].ProcessId.ProcessIdType = (uint)enum_AD_PROCESS_ID.AD_PROCESS_ID_SYSTEM;
        //pProcessInfo[0].ProcessId.dwProcessId = m_RtHandle;
        //pProcessInfo[0].Fields |= (uint)enum_PROCESS_INFO_FIELDS.PIF_PROCESS_ID;

        pProcessInfo[0].dwSessionId = 1;
        pProcessInfo[0].Flags = 0;

       ......
       ......
  }

Screen Shots:

I modified our native implementation close to the same way:  Which shows the process window as shown below.
HRESULT CProcess::	GetInfo(
PROCESS_INFO_FIELDS	Fields,
PROCESS_INFO*		pProcessInfo)
{
char*			p;
NTXHANDLE		hNode, hRoot, hProc, hLRS;
int				len;
LRS_STRUCT*		pLRS;
TRACEFUNT;
if (!pProcessInfo)
	return E_INVALIDARG;

// Get the fake process information
memset (pProcessInfo, 0 , sizeof (PROCESS_INFO));

pProcessInfo->Fields = 0;

pProcessInfo->bstrAttachedSessionName = SysAllocString(L"bstrAttachedSessionName");
pProcessInfo->Fields |= PIF_ATTACHED_SESSION_NAME;

pProcessInfo->bstrFileName = SysAllocString(L"bstrFileName");
pProcessInfo->Fields |= PIF_FILE_NAME;
    ;
pProcessInfo->bstrBaseName = SysAllocString(L"bstrBaseName");
pProcessInfo->Fields |= PIF_BASE_NAME;

pProcessInfo->bstrTitle = SysAllocString(L"bstrTitle");
pProcessInfo->Fields |= PIF_TITLE;

pProcessInfo->dwSessionId = 1;
pProcessInfo->Flags = 0;

@ssemon Could you share a crash dump at the point of the crash?

Yes, I'll start working on that.   I've got to change the pkgdef from c++ to c#. I'm working on another item at the moment, so it may be a day or two.

Here is the crash dump, Unfortunately, I wasn't able to see anything, that relates to the strings.  The only thing showing up is wait for mutliple objects.   However, some symbols failed checksums.
Won't let me upload a zip of the crash dump.

devenv.zip
rename the following files to the following from .txt
devenv.z01
devenv.z02
devenv.z03

It won't let me load the splits

Visual_Studio_Dump2.zip

Visual_Studio_Dump3.zip

Visual_Studio_Dump4.zip

Visual_Studio_Dump5.zip

Visual_Studio_Dump6.zip

Visual_Studio_Dump8.zip

Visual_Studio_Dump9.zip

Visual_Studio_Dump10.zip

Visual_Studio_Dump11.zip

The previous 13 zips contain the 63 pieces used with the first zip.  The first zip needs the pieces to extract the whole dump file.
Thanks,
ssemon

After testing the extraction xxx10.zip was missing a piece.   A new file is attached.
Visual_Studio_Dump10.zip

@ssemon Could you file a feedback item on Visual Studio using the Help > Send Feedback > Report a Problem? You can attach the full crash dump directly there.

Ok I submitted the details with the dump.

Could you share a link to the feedback item?

https://developercommunity.visualstudio.com/t/Visual-Studio-Extension---PortSupplier--/10005302?entry=problem&ref=native&refTime=1649260231548&refUserId=f19fec0e-f3bc-4f56-8875-02b11e9aa9c0

https://developercommunity.visualstudio.com/t/Visual-Studio-Extension---PortSupplier--/10005302?entry=problem&ref=native&refTime=1649260231548&refUserId=f19fec0e-f3bc-4f56-8875-02b11e9aa9c0

Could you upload the crash dump to the ticket? It seems like it hasn't gotten uploaded.

It's there, devenv.DMP

Closing as this issue has been moved over to a feedback item.


ENH Replace gpuci_conda_retry with gpuci_mamba_retry

mamba was recently added to gpuCI build environment, testing usage and solvability with this PR which should speed up build times.

Thanks Dillon! 😄
This seems reasonable, but it looks like Jenkins is running into some kind of issue:
Error response from daemon: manifest for gpuci/rapidsai-driver:21.10-cuda11.0-devel-centos7-py3.7 not found: manifest unknown: manifest unknown
Docker pull failed, retry 1 of 3...

Is this know/being tracked somewhere?

rerun tests

@dillon-cullinan I think we can merge this after removing the Draft status

I touched base with Dillon offline and he said this is on hold for the moment

Yes, there is a large batch of these PRs and we want to merge them at the same time. There are only two that are failing right now due to some missing 21.10 packages. Just waiting on those.

@gpucibot merge


workspaceclient v2023, WSP-only

Hello, following your comments here, I tested your packages for WSP on Fedora39 and encountered a conflict:
Last metadata expiration check: 0:33:22 ago on Fri Jan  5 11:07:03 2024.
Error:
 Problem: problem with installed package redhat-lsb-core-4.1-62.fc39.x86_64
  - package lsb_release-3.2-2.fc39.noarch from updates conflicts with redhat-lsb-core provided by redhat-lsb-core-4.1-62.fc39.x86_64 from @System
  - package lsb_release-3.2-2.fc39.noarch from updates conflicts with redhat-lsb-core provided by redhat-lsb-core-4.1-62.fc39.x86_64 from fedora
  - package workspacesclient-wsp-2023.2.4580-1.x86_64 from copr:copr.fedorainfracloud.org:vorona:aws-rpm-packages requires lsb_release, but none of the providers can be installed
  - conflicting requests
(try to add '--allowerasing' to command line to replace conflicting packages or '--skip-broken' to skip uninstallable packages)
[root@sebastian ~]#

This is from a Fedora upgraded release after release from F27! Researching on the net for the conflict did not lead to anything recent.

@claer is it possible to remove redhat-lsb-core before installing workspacesclient?

Good one! I didn't thought to remove it. Here are the deps removed as weel, nothing important on my system (sorry for the french):
==================================================================================================
 Paquet                          Architecture           Version                       Dépôt                   Taille
==================================================================================================
Suppression:
 redhat-lsb-core                 x86_64                 4.1-62.fc39                   @fedora                  39 k
Suppression des dépendances inutilisées:
 esmtp                           x86_64                 1.2-22.fc39                   @fedora                  98 k
 libesmtp                        x86_64                 1.0.6-28.fc39                 @fedora                 180 k
 liblockfile                     x86_64                 1.17-6.fc39                   @fedora                  64 k
 mailx                           x86_64                 12.5-44.fc39                  @fedora                 500 k
 spax                            x86_64                 1.6-10.fc39                   @fedora                 422 k

After that, the installation of workspaceclient-wsp package installed successfully.
Once the program starts, I only have a blank page. I can understand that the web page is here but all the text and images are not shown : grey text over grey background, no image but text is selectable and button clickable.

@claer please try to play with workspacesclient-wsp-2023.2.4580-5

@claer hey man, i'm waiting for your feedback
the latest release 2023.2.4580-7  has size around 7MB instead of previous 22MB, and it works!

Hello,
Thanks for your patience. Your package installs nicely but I still have the grey box when starting the client. Sounds like a rendering issue. I can select the invisible text and paste it in a text editor. If I try to login by filling the invisible fields I can start the DV client. I also saw that some window decorations are missing in the dv client. Wild guess, I'm missing a theme or sth for webkit.

Hello,
Thanks for your patience. Your package installs nicely but I still have the grey box when starting the client. Sounds like a rendering issue. I can select the invisible text and paste it in a text editor. If I try to login by filling the invisible fields I can start the DV client. I also saw that some window decorations are missing in the dv client. Wild guess, I'm missing a theme or sth for webkit.

looks like it is discrete graphics issue.... I do not have options to help with this.
btw, i saw that you have 32bit libs installed ... i do not have any one


Style fix in xEnvironment

getting rid of extra indentation and adding import-module to integration tests

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

Tests/Integration/MSFT_xEnvironmentResource.EndToEnd.Tests.ps1, line 24 at r1 (raw file):

        BeforeAll {

            # Import Environment resource module for Get-TargetResource, Test-TargetResource, Set-TargetResource

This new section here isn't needed. These functions aren't called in these tests

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable


Export for Easycraft

Hey @derfloh205
Added a button to export recipes for Easycraft (see screen).

It's mostly based on the export function you wrote for ForgeFinder but I added some extra data to export which I needed.
Also added two util functions, atob that encode a string to base64 and IsTWWRecipe that check if a recipe is related to TWW.
Let me know if you need me to change anything!

And please resolve the recent conflicts

All good!


Dashboard functionality is working as expected

Can create, update, delete todos. Can view and drop courses the user has signed up for.

fixed all merge conflicts just needs a look over


jaco v0.9.5?

Hey, I know this isn't really an "issue" but I'm looking to add mp3 support to my plugin and I noticed that you have v0.9.5 but I only see v0.9.4 online. Where did you get this? Looking at your commits it looks like you may have built it yourself? Just curious what the story is behind this version! It looks like it has extra methods like add(File file) etc that v0.9.4 doesn't have.

I actually did find a v.0.10.2 at https://sourceforge.net/p/jacomp3player/code/HEAD/tree/JACo MP3 Player v3/dist/ so I'm also curious what the differences are there.

I honestly cannot remember how I came to that specific build, sorry!

Ok no problem, thanks for the response :)


Invisible not triggering

The invisible (hidden) condition isn't triggering a DC11 check.

I've handled that (and added a few other things) in https://github.com/jessev14/pf2-flat-check/pull/2 (or, rather, I treated Invisible as Undetected).

#2 merged.


24638 has weird sky fit on r3; bad trace fit

No obvious impact on QA, but the sframes look pretty bad on r3.
https://data.desi.lbl.gov/desi/spectro/redux/daily/tiles/cumulative/24638/20220329/tile-qa-24638-thru20220329.png

https://data.desi.lbl.gov/desi/users/raichoor/main-status/sframesky/sframesky-20220329-00127995.png

@julienguy could you check this case and whether it is a "runaway PCA correction" or other problem?

This is caused by a (rare) trace shift fit on sky lines that went wrong.
 plot_fiber_traces --inpsf /global/cfs/cdirs/desi/spectro/redux/daily/exposures/20220329/00127995/psf-r3-00127995.fits --other-psf /global/cfs/cdirs/desi/spectro/redux/daily/calibnight/20220329/psfnight-r3-20220329.fits

The wavelength calibration adjustment on sky lines can be improved with a median filtering of the spectra. This takes a bit more time but more importantly it should be tested on a large data sample (with for instance radial velocities comparison).  This is studied in https://github.com/desihub/desispec/issues/1827.

@akremin, can we mark r3 on 20220329/127995, tileid 24638 as bad and reprocess?

Since this was a single exposure the standard star fitting fails without r3, so I've flagged the entire petal as bad for this exposure and reprocessed.
Specifically, I've flagged a3 for (TILEID 24638, NIGHT 20220329, EXP 127995). for both the svn exposure_tables and the daily versions (which are a snapshot of the daily processed data). I've removed the old reductions and rerun. I have also updated the tsnr table (tiles-daily.csv) for that night, as well as running desi_night_qa with --recompute.

Great.  This looks good now.  It's been updated in the surveyops repo.  I'm closing the ticket.


If a user is deleted, simple JWT allows token refreshing and verifying the user's token

Whenever a user is deleted and the user tries to refresh the token, it is refreshed or tries to verify the token, it is verified with 200 ok.
Due to the new token is not valid, frontend starts to refresh the token constantly. Then this causes the django server is down.
I love using simple JWT. Hope it is fixed soon.
Thanks in advance.

The same happens when the user model has .is_active set to False

One of the reasons of using access tokens is to reduce the number of database calls, but In my opinion that whenever a new access token is issued using a refresh token, the database must be touched in order to ensure that account exists and is active.
To bypass above specified use cases, I've created a custom refresh token serializer and used it for the refresh token view.
# serializers.py

from django.contrib.auth import get_user_model
from rest_framework import exceptions
from rest_framework_simplejwt.state import token_backend
from rest_framework_simplejwt.serializers import TokenRefreshSerializer

class CustomTokenRefreshSerializer(TokenRefreshSerializer):
    """
    Inherit from `TokenRefreshSerializer` and touch the database
    before re-issuing a new access token and ensure that the user
    exists and is active.
    """

    error_msg = 'No active account found with the given credentials'

    def validate(self, attrs):
        token_payload = token_backend.decode(attrs['refresh'])
        try:
            user = get_user_model().objects.get(pk=token_payload['user_id'])
        except get_user_model().DoesNotExist:
            raise exceptions.AuthenticationFailed(
                self.error_msg, 'no_active_account'
            )

        if not user.is_active or user.email != token_payload['user_email']:
            raise exceptions.AuthenticationFailed(
                self.error_msg, 'no_active_account'
            )

        return super().validate(attrs)

# views.py

from .serializers import CustomTokenRefreshSerialize

class CustomTokenRefreshView(TokenRefreshView):
    """
    Refresh token generator view.
    """
    serializer_class = CustomTokenRefreshSerializer

Gist: Use blacklist and revoke on user deactivate. Please read the rest, though, as a proper implementation of this package.
Hmm, if you use the blacklist app, this wouldn't be an issue, so long as you revoke the refresh token on user "deletion." That would be the proper method, but obviously, that would also be the only method to avoid this problem.
@selmanceker @dimmg @T-101
In any JWT application, including those who use OAuth protocol, the views should be subclassed so that they can be rate limited (using Django RateLimit, great package and maintained by some core Django developers) with status code 429 if user rate limited to prevent DoS or memory overload. I maintain https://github.com/noripyt/django-cachalot which is an all-table caching mechanism, and I always use its caching on the blacklist tables, in combination with rate limiting, to serve my web apps.
That means, for the most part, an average 0.1% cache miss rate when the a user refreshes a token. Hope that helps!

@Andrew-Chen-Wang how do you revoke refresh token on user deletion? I couldn't find anything

import the blacklist token model and add the token that should not be used again


Make master build again

Resolved issues

master does not build currently build in travis (missing travis_wait)
description in readme does not reflect status of project (directory web not yet in master)

please merge after review

Merged #17 for an updated elm-bridge version

I reviewed the changes manually and the branch builds


Scroll to top on route change

Note: for support questions, please use stackoverflow. This repository's issues are reserved for feature requests and bug reports.

I'm submitting a ...

[ ] bug report
[x] feature request
[ ] support request => Please do not submit support request here, see note at the top of this template.

Do you want to request a feature or report a bug?
feature

What is the current behavior?
on route change does not scroll to top

If the current behavior is a bug, please provide the steps to reproduce, ideally also a screenshot or gif if it's a style issue

What is the expected behavior?
on route change scroll to top

What is the motivation / use case for changing the behavior?
easy of navigation

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

That's implemented


lookup function StatefulSet gives error

Output of helm version:
version.BuildInfo{Version:"v3.5.2", GitCommit:"167aac70832d3a384f65f9745335e9fb40169dc2", GitTreeState:"dirty", GoVersion:"go1.15.7"}

Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.3", GitCommit:"1e11e4a2108024935ecfcb2912226cedeafd99df", GitTreeState:"clean", BuildDate:"2020-10-14T12:50:19Z", GoVersion:"go1.15.2", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.5+k3s2", GitCommit:"746cf4031370f443bf1230272bc79f2f72de2869", GitTreeState:"clean", BuildDate:"2020-12-18T01:41:55Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):
k3os
adding the following lookup call:
#image: {{ (lookup "apps/v1" "StatefulSet" "" "")}}

results in
helm upgrade -f smat-xims-values.yaml xims ./xims --debug
upgrade.go:123: [debug] preparing upgrade for xims
Error: UPGRADE FAILED: YAML parse error on xims/templates/cronjob_mongo.yaml: error converting YAML to JSON: yaml: line 25: could not find expected ':'
helm.go:81: [debug] error converting YAML to JSON: yaml: line 25: could not find expected ':'
YAML parse error on xims/templates/cronjob_mongo.yaml
helm.sh/helm/v3/pkg/releaseutil.(*manifestFile).sort
        /private/tmp/helm-20210204-99981-1f8vc0y/pkg/releaseutil/manifest_sorter.go:146
helm.sh/helm/v3/pkg/releaseutil.SortManifests
        /private/tmp/helm-20210204-99981-1f8vc0y/pkg/releaseutil/manifest_sorter.go:106
helm.sh/helm/v3/pkg/action.(*Configuration).renderResources
        /private/tmp/helm-20210204-99981-1f8vc0y/pkg/action/action.go:165
helm.sh/helm/v3/pkg/action.(*Upgrade).prepareUpgrade
        /private/tmp/helm-20210204-99981-1f8vc0y/pkg/action/upgrade.go:215
helm.sh/helm/v3/pkg/action.(*Upgrade).Run
        /private/tmp/helm-20210204-99981-1f8vc0y/pkg/action/upgrade.go:124
main.newUpgradeCmd.func2
        /private/tmp/helm-20210204-99981-1f8vc0y/cmd/helm/upgrade.go:155
github.com/spf13/cobra.(*Command).execute
        /Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:850
github.com/spf13/cobra.(*Command).ExecuteC
        /Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:958
github.com/spf13/cobra.(*Command).Execute
        /Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:895
main.main
        /private/tmp/helm-20210204-99981-1f8vc0y/cmd/helm/helm.go:80
runtime.main
        /usr/local/Cellar/go/1.15.7_1/libexec/src/runtime/proc.go:204
runtime.goexit
        /usr/local/Cellar/go/1.15.7_1/libexec/src/runtime/asm_amd64.s:1374
UPGRADE FAILED
main.newUpgradeCmd.func2
        /private/tmp/helm-20210204-99981-1f8vc0y/cmd/helm/upgrade.go:157
github.com/spf13/cobra.(*Command).execute
        /Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:850
github.com/spf13/cobra.(*Command).ExecuteC
        /Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:958
github.com/spf13/cobra.(*Command).Execute
        /Users/brew/Library/Caches/Homebrew/go_mod_cache/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:895
main.main
        /private/tmp/helm-20210204-99981-1f8vc0y/cmd/helm/helm.go:80
runtime.main
        /usr/local/Cellar/go/1.15.7_1/libexec/src/runtime/proc.go:204
runtime.goexit
        /usr/local/Cellar/go/1.15.7_1/libexec/src/runtime/asm_amd64.s:1374

Without the above line everything works as expected. Replacing StatefulSet with Deployment works as expected...
Any Idea?
Thanks
Ognian

A key thing to look at is error converting YAML to JSON: yaml: line 25: could not find expected ':'. could not find expected ':' is happening when the package that handles this is trying to understand the YAML in order to convert to JSON.
This makes me wonder, how may Deployments and how many StatefulSets do you have? If Deployments is 0 or 1 it would generate one line of YAML. If StatefulSets is multiple it could product multiple lines in the YAML. Once on to a second line in the generated YAML is would not be properly formed.
Might something like this be happening?

Yes, I have 1 deployment and 4 StatefulSets...
When I use the 4th parameter to provide the name of one of the StatefulSets I get an empty map...
So I have no way to get the info about the StatefulSet

OK I start to understand...
This is all about how the templates work with the yaml parser.
If someone needs this here is one possible solution:
              image: {{ range $index, $sts := (lookup "apps/v1" "StatefulSet" "" "").items -}}
                 {{- if eq $sts.metadata.name $.Values.xxx.statefulset_name  -}}
                 {{- ( first $sts.spec.template.spec.containers).image -}}
                 {{- end -}}
               {{- end }}

When I use the 4th parameter to provide the name of one of the StatefulSets I get an empty map...

To clarify, you were only providing the 4th parameter, but omitting the 3rd parameter. Is that correct?
I think you need to provide the namespace along with the name if you want lookup to find the correct statefulset. For example lookup "apps/v1" "StatefulSet" .Release.Namespace "mystatefulset".
Omitting the namespace makes the lookup function assume it is looking for that object at the global scope. A StatefulSet client lookup the correct resource at the global scope; it can only perform List operations. That would explain why lookup can't find your StatefulSet.

@bacongobbler you are totally right!
and this is the oneliner:
              image: {{( first (lookup "apps/v1" "StatefulSet" "default" $.Values.xxx.statefulset_name).spec.template.spec.containers).image }}


RuntimeError: input tensor has to be contiguous with DCNv2 on Res2Net

Hi, I am trying to add DCNv2 on Res2Net backbone. Interestingly, I got an RuntimeError: input tensor has to be contiguous with batch size = 2 but when I set batch size = 1, the error was gone. This bug also didn't appear on DCNv1.

I met this problem before. Adding this line may help.

I met this problem before. Adding this line may help.

Thanks! I will give it a try~

Hi, @zehuichen123, I have also run into the exactly same problem and tried the fix given by @PanJianning, but it seems that the fix doesn't work.  Have you solved the problem? Thank you.

Hi, @zehuichen123, I have also run into the exactly same problem and tried the fix given by @PanJianning, but it seems that the fix doesn't work. Have you solved the problem? Thank you.

One possible solution would be to change sp = self.convs[0](spx[0]) into sp = self.convs[0](spx[0].contiguous()) and change  sp = self.convs[i](sp) into sp = self.convs[i](sp.contiguous())

I met this problem also with DCNv1.
@hyz-xmaster's solution fixed both DCN version on my environment, so I sent PR.


All attribute hooks are not rendered

I'm having some problems with value attribute being rendered as [object Object] instead of its actual value. This is because a soft hook is created and the following code does not render it because of the second condition value.namespace.
if (isVHook(value) && value.namespace) {
   ret += ' ' + createAttribute(name, value.value, true);
   continue;
}

As an example this test fails if added:
it('should render hooks', function () {
  var node = h('input', { type: 'submit', value: 'add' });
  assert.equal(toHTML(node), '<input type="submit" value="add">');
});

@marlun let me know if the pull request help.

@marlun let us know if it works for you now

Just so that other people can see, I answered in #10 and everything is working fine.


I am a candidate of GSSOC'24 and I want to contribute a Web Development project

I am a candidate of GSSOC'24 and I want to contribute a Web Development project which is an Image Carousel which allows user to add and remove images and shows the images. It also have a chat box API integration. Please assign me this

Hey, I am interested in fixing this issue. could you assign it to me?

Hey, I am interested in fixing this issue. could you assign it to me?

Please create your own new issue with new project idea .


Hi！ Could you provide a version of generative for Text Processing？

Hey!
My background is mostly about computer vision.
I'm sorry I can not help you for now.


Miss 'seeked' or 'playing'  event

It looks like miss 'seeked' or 'playing' after 'seeking' event.
This problem can not be reproduced 100%!!!
Do you have any ideas or tips which could help us to identify the issue?
Thanks.
mux sdk: v1.0.0

Hey, sorry to hear you're still having trouble.
Something I noticed is that both of the cases seem to involve seeking during an ad. I'd like to ask you for some information that may be too sensitive for GitHub.
Would you mind filling out a support request on our website? I've let them know to expect you. Thank you!

A fix for this is now available in v1.0.1 of the SDK. Please reach out if you see any more issues!


Fix doctor message about missing lib32stdc++6

Steps to Reproduce
On a clean Ubuntu 16.04 install follow Flutter setup guide then run flutter doctor. Observe message:
Downloaded executables cannot execute on host (see https://github.com/flutter/flutter/issues/6207 for more information)

This message is not useful. #6207 is closed by #8798, which in turn simply points back to the closed issue. The issue thread does not give you a straight solution, but rather to a stream of comments one of which happens to be a solution. There does not seem to be an open issue to fix this permanently any time soon. We should either include the lib32stdc++6 setup step in the setup guide, or have the error message point to a special page that offers a solution more directly.
This is happening as of https://github.com/flutter/flutter/commit/f68c6fb814bd93e2061fb7ad595e275490fb999e

Hi @yjbanov  is this issue still relevant?

Nope. Thanks for catching! Closing.


xVMHyperV - Added support for AutomaticCheckpointsEnabled

Pull Request (PR) description
Adds support for AutomaticCheckpoints of VMs as described in #130
This Pull Request (PR) fixes the following issues:
Fixes #130
Task list:

[x] Change details added to Unreleased section of CHANGELOG.md?
[x] Added/updated documentation, comment-based help and descriptions in .schema.mof files where appropriate?
[x] Examples appropriately updated / added?
[x] New/changed code adheres to Style Guidelines?
[x] Unit and (optional) Integration tests created/updated where possible?

This change is

Submitted a PR for this even if it isn't 100% complete yet, hope that is OK.
I'll complete that task list as soon as I have a little time again, any feedback on the code changes are greatly appreciated.

Codecov Report

Merging #131 into dev will increase coverage by <1%.
The diff coverage is 93%.

@@         Coverage Diff          @@
##            dev   #131    +/-   ##
====================================
+ Coverage    71%    71%   +<1%
====================================
  Files         9      9
  Lines      1032   1048    +16
  Branches     12     12
====================================
+ Hits        733    748    +15
- Misses      287    288     +1
  Partials     12     12

If I've understod the contribution requirements correctly this should be ready for review.
Took the liberty to expand the Sample_xVMHyperV_Complete.ps1 to actually create a complete VM by also creating the required VHD.

@SimonWahlin  Can I get you to have a look at the failing unit tests and resolve the merge conflicts on this one? I'll review it after that.

@SimonWahlin  OK you should be able to do a pull from the dev branch now and get something that will fix the tests. Once that's all green I'll do a review of it all

Thank you @BrianFarnhill I've merged the upstream commits now.

Reviewed 2 of 5 files at r1, 3 of 3 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


20-transfer miscellaneous clean-up

[ ] Clarify comment about preventing transfers in transfer.proto (no cross-chain per-denom enable/disable)
[ ] Use SHA256 instead of imported tmhash
[ ] Add comment explaining why base denom is not validated (coming from another chain)
[ ] Rename validatePrefixedDenom
[ ] Replace "ibc" with constant
[ ] Remove hardcoded numbers (e.g. [4:] should be length of a constant) in DenomTraceFromHash and elsewhere
[ ] Add comment why the acknowledgement is 1
[ ] Abstract out denomination calculation functions
[ ] Add comment about zero-coins
[ ] Move events around (after state changes, not in-between)
[ ] Comment to clarify security model around denomination space namespacing in RecvPacket
[ ] Improve naming of trace functions generally
[ ] Note that ordered channels are considered dangerous

closing as stale


Fixed wrong use of to throw that will fail with unexpected v11

Description of the Change
I was testing Mocha's compatibility with the upcoming version of Unexpected. For that version we remove support for satisfying against a function as that has confused people.
I found one test that will fail with the next version as it was using to throw to satisfy against an error. This test will actually just call TypeError with the throw error, so it wont really check anything. This PR fixes the test to assert that the throw error is a TypeError.
Benefits
Bugfix.
Possible Drawbacks
None

nice catch.

@sunesimonsen 	BTW, how are you running compatibility tests?

@boneskull I was just doing it manually. We have a beta out and I tried to run it against the Mocha tests suite. We like to run the major version betas on as many test suites we can to make sure things works as expected. We also try to check all of the plugins and fix them for compatibility.


docs: expand adapter-node polyfill docs

Add some extra info to the adapter-node polyfill option's docs.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[ ] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

How ready for prime time is nodejs.dev? I haven't been following that. I do notice that their homepage currently says that the most recent LTS is 18.15.0 (not 18.16.0) and that the latest is 19.8.1 (not 20.0.0), which doesn't inspire tremendous confidence. I'd prefer linking to the docs on nodejs.org for now.

How ready for prime time is nodejs.dev? I haven't been following that. I do notice that their homepage currently says that the most recent LTS is 18.15.0 (not 18.16.0) and that the latest is 19.8.1 (not 20.0.0), which doesn't inspire tremendous confidence. I'd prefer linking to the docs on nodejs.org for now.

Honestly, I'm not sure. It was just the first place Google took me to when I was investigating the issue and it looked reputable. 🙃


How do I carry around dynamic dependencies on e.g. headers?

Thanks for the great work. I have a design question for a problem I've been trying to solve. I'm new to buck2 and I'm not a previous Bazel or buck user, so it's been a little rough getting started. If I make any conceptual mistakes, please point them out :)
For context, this is without the prelude -- I would like to keep much simpler rules on hand that don't need to be as flexible and get better at implementing rules.
I have a project made up a bunch of logical groupings of C files, created by a rule c_inputs that returns the provider CInputsInfo:
CInputsInfo = provider(
    doc = "Info on a set of C inputs.",
    fields = [
        "srcs",
        "hdrs",  # dict mapping { str (header path): source }, globbed from all qualified_inc_paths
        "qualified_inc_paths",
    ]
)

Downstream rules consuming CInputsInfo might compile the sources into a binary, run a linter on them, and so on.
I want to have a dynamic dependency on the headers consumed by these C files, generated by the compiler (-M family of options in gcc/clang). I would like to do the actual dependency generation only once and then get to use the sources many times. I see a couple of general approaches to doing this:

dynamic_outputs: I implemented this for one use case (c_objects), and it works well, but I'm having trouble seeing how I would do this for c_inputs because we wouldn't yet have an artifact to bind.

def _c_objects_impl(ctx: "context"):
  ...

  for src in ctx.attrs.inputs[CInputsInfo].srcs:
    ...

    def compile_with_header_deps(ctx, artifacts, outputs, dep_file_json=dep_file_json, object=object, src=src):
        cmd = cmd_args([compiler, src, flags, inc_flags, "-c", "-o", outputs[object].as_output()])

        # for every header in the list, add a hidden dependency on it to the cmd
        dep_header_list = artifacts[dep_file_json].read_json()
        for header in dep_header_list:
            cmd.hidden(ctx.attrs.inputs[CInputsInfo].hdrs[header])

        ctx.actions.run(cmd, category = "compile", identifier = str(outputs[object]))

    ctx.actions.dynamic_output(dynamic=[dep_file_json], inputs=[src], outputs=[object], f=compile_with_header_deps)

dep_file: Unless I'm mistaken, these can't be used across actions.

Do you have any ideas on how to approach this kind of thing in buck2?

Could you make your srcs be a list of pairs of (source, dep-of-source)? Then you should be able to generate them easily enough in the c_inputs rule. Does that work? Note that the flags that are used by the thing compiling an artifact may change which dependencies it relies upon (e.g. if you #ifdef over an #include), which may mean that the compiling thing (c_object) might be the first place that can do it. If so, you might need to look at anon_targets to share work between rules.

@ndmitchell That works, and I actually later implemented it exactly that way -- CInputsInfo looks like:
CInputsInfo = provider(
    doc = "Info on a set of C inputs.",
    fields = [
        "srcs",
        "inc_paths",
        "header_deps",
    ]
)

where srcs is a list of source and header_deps is a dict mapping source to list of headers it depends on, made with dynamic_output and gcc/clang's -M family of options.
This works okay, although:

it would be nice to have a way to not have to .hidden() on the headers in upstream rules, but debatable
The cli flags should be the same for c_inputs and c_objects, etc, because, as you said, they can change the dependencies.


Fix default type inference of CombinePerKey.

More typehints propagation of CombinePerKey, MapTuple, FlatMapTuple

get_type_hints returns an empty TypeHint object. Add with_default to
initialize the typehints from the fn when the hints are generated.
[default_type_hints() and in the map functions]

In MapTuple and FlatMapTuple, the input type hint is still ignored.
This is because
IOTypeHints cannot distinguish *args from an ordinary argument. I feel there
can be more complications even if this one is tracked.

For CombinePerKey, also override infer_output_types() to perform an inference
pass on the fn. This is more accurate than the default types. There
is some redundancy with default_type_hints()

Also fix strip_output_annotations (bug discovered while fixing the broken users
from the above changes)

It needs to inspect into the composites (Tuples etc), but wasn't doing so.
For now we return Any if any inner hint of a composite is annotated.
Expanded the comment.

This shall be filed / synced as https://github.com/apache/beam/pull/16351

BUG=36682198
CL=418042595

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @robertwb

Can you add some tests illustrating the improvement in behavior? It's fine to inspect pcollection.element_type manually.

Thanks for the ping. I will revisit this and add some tests this week!
On Wed, Jan 26, 2022 at 6:10 PM Ahmet Altay @.***>
wrote:

@rainwoodman https://github.com/rainwoodman - Could you please respond
to the open comments?
—
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/16351#issuecomment-1022778123, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABBWTB5TCQ5O4PLWY6WC7LUYCSQTANCNFSM5KVUR4HA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

I added two tests.

Checking for new type hints logic. It was hard to isolate, but after some fiddling I finally realized this PR mainly deals with the case where the type hints are from annotation. ;) We don't typically have unit tests covering them.
Added a test for visit_inner_types.

I didn't add a test for _strip_output_annotations. It was an internal method.
I didn't find a test for the stripping behavior and didn't add one for the new recursion logic neither.

Ping?

@robertwb - pinging explicitly.

R: @yeandy

Can you help me better understand the behavior before your changes to default_type_hints() and _strip_output_annotations(), and then after the changes?

It's been a while, and this is what I still remember. Before the change

output annotations (TaggedValue etc) are sometimes leaked to the default_type_hints, causing unintended type check errors when we were trying to rollout type propagation with GBK.
the output types are missing even if the python functions used to construct Map, CombinerFn have Python type annotations.

After this change 1 and 2 are fixed. The input types are still missing in the Tuple Map variants as noted in the comments.

Do we want to close this or keep it open?

It has been dragging for very long, but it looks like the conflicts are trivial.

@robertwb - could you PTAL when you get a chance?


When will you release the dataset？

Hi， I am very interested in your work and the dataset. When will you provide the access of the dataset publicly? Thank you.

Thank you very much for your interest in our dataset. We are doing the final refinement. The Youmakeup Dataset will be released by the end of this year.


⚠ ClamAV has degraded performance

In 231a82a, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 88 ms

Resolved: ClamAV performance has improved in ef28715.


Smart code editor gets out of sync on Mac

Environment

Elixir & Erlang/OTP versions (elixir --version):

Erlang/OTP 25 [erts-13.0.2] [source] [64-bit] [smp:12:12] [ds:12:12:10] [async-threads:1] [jit:ns]

Elixir 1.13.3 (compiled with Erlang/OTP 24)

Operating system:  Mac 12.5.1
How have you started Livebook (mix phx.server, livebook CLI, Docker, etc):  Livebook desktop
Livebook version (use git rev-parse HEAD if running with mix):
Browsers that reproduce this bug (the more the merrier): Brave

Current behavior
Include detailed steps to reproduce error, errors and stacktraces if appropriate.
Smart code editor gets out of sync when used with the tested cell library.  For some reason only existing smart cells, not the new ones, have the sync issue.  I have been collaborating with @BrooklinJazz, the creator of the library, and we are both stuck and unsure how to debug the problem.  We can only reproduce the issue on my computer.
The following smart cell input

Produces an incorrect output in the livemd file.

Is there any reason for the smart code cell to get out sync?
Expected behavior
Smart cell code editor and .lmd file should be in sync.

I believe this has been fixed in main, right @jonatanklosko? We will ship a new version really soon.
You can try main by downloading the Universal installer from our staging page: https://deploy-preview-47--livebook-dev.netlify.app/#install
Alternatively, check the steps in the README and let us know how it goes!

It's actually a different issue. If you look at the source of the notebook here, you can find \r\n in the persisted smart cell content. We generally turn all CRLF into LF on import, however in this case the CR is serialized in a JSON string, hence it is kept.
Starting with the next release, the editor will always produce LF line endings, so we shouldn't run into cases like this in the first place. The Livebook version used while editing that smart cell must've added CRLF in the editor.

What's surprising though is that you are using Livebook desktop, I assume v0.6.3? I would expect the issue to only appear on Livebook main. @wojtekmach any chance #1337 made it to the v0.6.3 desktop build?

@jonatanklosko yes, Wojtek did more recent builds for sure. :)

Yeah I wouldnt miss out on such elite pr

@jonatanklosko Yes, I am using 0.6.3 and I will keep a watch for the next version.  Thank you all.

I used the universal installer as you suggested @josevalim however the behavior is still there.  I captured it in the video below in case it helps.
Unfortunately, 720p was too large.
https://user-images.githubusercontent.com/2127099/189892345-4b81deaf-7008-40e3-8eac-4314d249ed52.mov

@Billboz the issue should've been fixed in #1400, so to test you need to run Livebook main or wait for the new nightly build of the desktop app :) If you can reproduce it on main, please let me know!


Cannot connect to On-premise network using Self hosted as a Proxy

am not able to connect on-premise enviroment from the Azure SSIS, and I am using self hosted runtime as a proxy, my package runs successfully from the azure vm where I have the VPN gateway, but not able to run using the Azure SSIS.

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

ID: cc15b40b-fcef-856c-3da0-01222122dccd
Version Independent ID: d3a3777f-043c-5e07-c3bc-36fe395730f9
Content: Configure a self-hosted integration runtime as a proxy for SSIS - Azure Data Factory
Content Source: articles/data-factory/self-hosted-integration-runtime-proxy-ssis.md
Service: data-factory
GitHub Login: @swinarko
Microsoft Alias: sawinark

SHIR as a proxy for SSIS IR currently supports only a few components, see https://docs.microsoft.com/azure/data-factory/self-hosted-integration-runtime-proxy-ssis#current-limitations.  Please raise a support ticket for us to investigate your issue in more details. #please-close

@swinarko we have a suppot ticket for this but we are not getting much help there can you look into this
SR number:<PHONE_NUMBER>04867

Please see this

SHIR as a proxy for SSIS IR doesn't currently support Execute SQL Task.

@swinarko so will need to setup Azure VPN Gateway for this?

To run Execute SQL Task on SSIS IR that connects to your on-prem SQL Server, you'll need to join your SSIS IR to a VNet connected to your on-prem network via ExpressRoute/VPN Gateway for now.

@swinarko thanks for the confirmation, can you also throw some insights as we are using CDC and CT implementation here, are those CDC splitter sequence containers and all those flows supported when we setup S2S connectivity between on-premise and Azure?

With VNet injection, everything's supported.


macOS minimize (iconify) bug

when i try to minimize my client, the client just bugs out the right hand side panels disappear.
here are the screenshots

^ this shows how it looks like before i minimize it

^this shows that i have clicked the minimize (iconify) icon its highlighted

^this shows that the right hand side pannel just disappears and i can't minimize it and i have to close it from the windows or what ever you call it on MAC

^ thats how i close it to re-open the client

This is a duplicate of #2967


Allow password reset rule to be overridden.

I also set the password min length to match the Registrar service rule.

Send to 5.1 please. This could be breaking for people using that property for something else.

5.1 is the master btw. :)


[Draft] Tables swagger spec

DO NOT MERGE - only for CI validation check

/azp run automation - sdk

Can one of the admins verify this patch?

/azp run automation - sdk


Route Groups strange behavior

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Here's some sample code :

Flight::group('/profile', function () {
    $uc = new UserController();

    Flight::route('GET /edit', function () use ($uc) {
        AuthMiddleware::handle();
        $uc->edit();
        Flight::render('profile/edit', [], 'content');
        Flight::render('layout');
    });

    Flight::route('GET /@id', function ($id) use ($uc) {
        AuthMiddleware::handle();
        $uc->view($id);
        Flight::render('profile/view', [], 'content');
        Flight::render('layout');
    });
 });

In this case :
GET /profile/4 : works
GET /profile/edit : works
When i invert the position of the 2 sub routes like this :
Flight::group('/profile', function () {
    $uc = new UserController();

    Flight::route('GET /@id', function ($id) use ($uc) {
        AuthMiddleware::handle();
        $uc->view($id);
        Flight::render('profile/view', [], 'content');
        Flight::render('layout');
    });

    Flight::route('GET /edit', function () use ($uc) {
        AuthMiddleware::handle();
        $uc->edit();
        Flight::render('profile/edit', [], 'content');
        Flight::render('layout');
    });
 });

GET /profile/4 : works
GET /profile/edit : returns 404 error
Expected behavior
same behavior in the 2 cases.
Environment (please complete the following information):

OS: Fedora linux 41
Browser Firefox latest
PHP Version 8.3.14
Flight Version 3.13

Probably because @id covers also edit as a valid value. Maybe you could try to limit @id like: @id:[0-9]

@kuopassa is right on the money! Because @id is at the end of the route and it's a variable (and it's not specifying that it's only an int) it assumes that anything that's passed into that route is valid. So if you went to your code on the /edit route you'll notice that it will get passed to the @id route and the $id variable will actually say 'edit'.
The way to structure your routes in Flight (and other frameworks too) is specific routes up top and more general routes on the bottom.


BREAKING CHANGE: Adl object refactor

This PR is mapped to this Swagger change: https://github.com/Azure/azure-rest-api-specs/pull/1429

Revised the inheritance structure and broke up objects for customer-experience clarity

For more details, please refer to the PR above

Found two bugs that I needed to fix in order to run tests in Playback mode:

including vcrpy in requirements.txt
resource-manager to data-plane in swagger_to_sdk_config.json for catalog.json and job.json

There were also a couple modifications that I made to azure-mgmt/tests/test_mgmt_datalake_analytics.py in order for the tests to pass.  However, all the tests in the automatic PR passed, which was odd.

@ro-joowan,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@lmazuel here is the aggregated PR for python that I promised you :). I reviewed this with @ro-joowan and helped him get it done. For some reason the automatic PR passed while this one required some test updates, so I am glad that we did it this way and caught the test failures :).

Codecov Report

Merging #1324 into master will increase coverage by 0.04%.
The diff coverage is 84.48%.

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

Please merge and publish!
Thank you,
Joo Wan


Support environment variables when run all tests

It couldn't be the best way to do this because It will set all functions environment variables before start the first test.

That's a great PR, @mpuittinen could you merge this, please?

@inakiabt I can't understand how the PR changes the functionality. SetEnvVars was moved from before getFilePaths (which should be renamed to getTestFilePaths to be more descriptive) to after it. The only thing that happens in getFilePaths is that it looks up paths to test files (in test folder) based on the lambda function names evt.options.paths. Since SetEnvVars needs the lambda function name, not the test file name (these happen to be identical), using the test file paths returned by getFilePaths is logically wrong.

Currently, when the tests are run without providing a function name (e.g. sls function mocha-run), the evt.options.paths variables is empty and therefore the SetEnvVars does not set any environment variables. However, as the getFilePaths checks for empty function arguments lists and if it's empty, gathers all files, SetEnvVars receives proper file list.
The other way to fix it is to collect filenames in SetEnvVars or move creation of promise to line 152 and reuse getFilePaths in SetEnvVars

@rafaljanicki Thanks for the clarification, I'll see how this could be solved so that we are logically doing things right. The test folder may contain also tests created manually by the developer, i.e. it is not guaranteed that there is a 1:1 match between all test cases in the test folder and functions.

That's right, I just have found out that error in the PR.

@rafaljanicki @inakiabt I'll probably fix it so, that in case that there is no functions specified in the mocha-run command, I'll check for available functions, and build the test filenames based on that.
In case developers have tests that are not generated by this plugin / named differently / etc... there is always the option to run mocha directly.

I think that makes sense, thanks!

BTW, if you are working on this, could you also add a check on var func = S.getProject().getFunction(funcName).toObjectPopulated(config); if the function actually exists and throw an error? It'd look much better than TypeError right now :)

This is now fixed in 0.5.7 (just out), included better handling of non-existent functions.