Some question about test and pval file

Hello, this software works very well but I have some questions.

In the pval file, does betadir/taudir means the direction of allel effect?
Is there a way to calculate or display the allel effect in a score test?wald test is very time-consuming
Wald test seems not to work with allel types such as "TC" "AAC"
Also I found a small problem in line 278 of the "gwas.jl", it should be @warn instead of warn

Thank you for your interest in our package.

Yes, betadir and taudir mean the direction of allele effect with respect to the first allele (Plink, BGEN) / alternate allele (VCF).
One way would be running the score test first, and running the Wald test only with the significant alleles. We have filtering tools in SnpArrays.jl and VCFTools.jl.
Could you provide a little more detail or a minimal working example for this one, please?

Thank you for spotting the issue. Line 278 will be fixed soon.

In my bim file some snp will be this form
16	rs10680802	0	53726175	CAG	C
biallelic is not simply A/T/C/G
Wald testing will be stopped in these SNPs with"ERROR: ArgumentError: matrix contains Infs or NaNs"

It sounds like a numerical issue rather than the SNP being AG insertion/deletion. I don't think that kind of issue has something to do with allele type. Could you please paste the full error message including the call stack and which line this is happening in?
I might need to look into the numbers later to resolve this kind of issue.

like this?
run = 2, ‖Δβ‖ = 1.068269, ‖Δτ‖ = 1.890190, ‖ΔL‖ = 0.023151, status = Optimal, time(s) = 6.472817
┌ Warning: Ipopt finished with status Invalid_Number_Detected
└ @ Ipopt ~/.julia/packages/Ipopt/QF8Lc/src/MPB_wrapper.jl:195
┌ Warning: Optimization unsuccesful; got Error; run = 1
└ @ WiSER ~/.julia/packages/WiSER/etxnv/src/fit.jl:63
run = 1, ‖Δβ‖ = NaN, ‖Δτ‖ = 0.000000, ‖ΔL‖ = 0.000000, status = Error, time(s) = 0.530174
┌ Warning: Ipopt finished with status Invalid_Number_Detected
└ @ Ipopt ~/.julia/packages/Ipopt/QF8Lc/src/MPB_wrapper.jl:195
┌ Warning: Optimization unsuccesful; got Error; run = 2
└ @ WiSER ~/.julia/packages/WiSER/etxnv/src/fit.jl:63
run = 2, ‖Δβ‖ = NaN, ‖Δτ‖ = 0.000000, ‖ΔL‖ = 0.000000, status = Error, time(s) = 0.583764
ERROR: ArgumentError: matrix contains Infs or NaNs

Actually, the full output would be more helpful. Everything you get, including the lines below the "ERROR" line.

The full output will be more helpful but initially it looks like it could be an issue with numeric instability. Are the variants you mention very rare? That could potentially cause issues.
Can you also try another solver some examples are shown here trajgwas() takes solver as an input.
You may also want to try standardizing predictors to see if that helps.

This is my first time using julia, so I'm not very familiar with it.
 We will need to check allele frequencies, try to normalize predictors, and/or try to change solver settings. I will post some code to check allele frequencies later today.

@RoyeSie
Would you try running this code?
using SnpArrays
s = SnpData("<prefix of your plink data>")
println(maf(s.snparray)[s.snp_info[!, :snpid] .== "rs10680802"]) # and other SNPs that your code fails on

julia> println(maf(s.snparray)[s.snp_info[!, :snpid] .== "rs10680802"]) # and other SNPs that your code fails on
[0.2970530641070913]

Sorry for missing this out. This seems to be a kind of numerical issue we were not aware of. The SNP is not that rare...

Hi, I am having a related issue even with the example dataset: Wald test seems not to be working if you select >= 50 SNPs. I am using precisely the same model specifications as you do in your tutorial (the model in the "Basic Usage" section), with the only modification being that I changed "test = :score" to "test = :wald". And I get the same error as @RoyeSie (as posted on 22 July). My rough guess is that this is purely about the optimization.

Thank you for providing a working example. We were having trouble reproducing the issue. In this situation, one SNP fails in the optimization, stopping the entire pipeline. We can first print a message that testing for a specific SNP fails and move on, not stopping the whole run. Ideally, I hope to be able to fix the issue entirely. We recently added the init keyword to try another initial condition. e.g., adding
init = x -> WiSER.init_ls!(x; gniters=0),

Changing the initialization helped resolve some other situations (discussed off-GitHub), but it did not help in this case. I will look into it to find out what is going on.

In most situations with common alleles, just changing the initialization is good enough to resolve the issue.
However, with the hapmap3 example, it is happening with a SNP with a very low MAF (0.004). Our development is mainly focused on making sure that the score test is working with this low MAF, but we cannot really guarantee that Wald test works well with a rare variant. At this point, we cannot get the Wald test run with that specific SNP using the set of initialization methods we have in WiSER.jl. Hence, the action plan is just to print out that the test failed, display MAF, and move on to the next SNP. It will be finished next week.

Updated ver 0.2.2:
Wald test is now much more robust to failure. It will try with another initialization first, and then if it fails, it will give you a warning with effect size NaN and p-value -1.


Modify the path of "sifter-injector" in sifter.py

Since we decide to place binaries in /sbin, the path of "sifter-injector" should be modified.

I have updated release 1.02 with this change.


Extra input files

It should be possible to specify paths to extra input files (like TypeScript typings) from the build configuration instead of editing the gulpfile directly.

This is done in the current 2.0 dev!


feat!: Bootstrap test integration and change in buckets creation

Hello folks,
This PR contains:

The integration test for the bootstrap step, with the following resources:

Buckets
Source Repositories
Cloud Build Triggers
Service Accounts

Add "location", "trigger_location" and "tf_apply_branches" variables in "tf_cloudbuild_workspace" module.

Change in bucket creation:

Add the "bkt" prefix
Creation of a state bucket for each step and each environment (development, production and non-production)

/gcbrun

TestBootstrap 2024-03-11T15:13:59Z command.go:100: Running command gcloud with args [source repos get-iam-policy eab-applicationfactory --flatten bindings --filter bindings.role:roles/viewer --format json]
TestBootstrap 2024-03-11T15:14:00Z command.go:185: ERROR: (gcloud.source.repos.get-iam-policy) PERMISSION_DENIED: The caller does not have permission
TestBootstrap 2024-03-11T15:14:00Z command.go:185: - '@type': type.googleapis.com/google.rpc.RequestInfo
TestBootstrap 2024-03-11T15:14:00Z command.go:185:   requestId: 8ff955bda0df4baf8de7be978eb5cb68

/gcbrun

Hi, @gtsorbo
Could you take a look at the PR please? Whether it will be necessary to make any changes before merging. Thanks.


Adding "windows" label to Windows node.

Additional steps to add label to Windows Nodes.  This will allow admin to filter Windows node when creating a service like this.
docker service create --constraint engine.labels.os==windows --name winonly microsoft/nanoserver  The same steps are on this article: https://success.docker.com/article/how-can-i-assign-a-service-to-windows-nodes-only

Proposed changes

Unreleased project version (optional)

Related issues (optional)

@ekitagawa, @JimGalasyn I thought swarm would check the platform.os listed on the image manifest, and if it was Windows (you're trying to deploy a Windows image), then swarm would make sure to schedule the service on a Windows node.
Do you know if that really happens? If that's the case you don't really need to add constrains to your services.

This has been stalled for a while, and it doesn't hurt to add labels to your nodes. Let's merge it. I've filed #6690 to bring this to UCP 3.0 docs too.

so sorry for the delay @joaofnfernandes... Yes, in an ideal world, task assignment does check the platform.  but we had an issue like this https://github.com/docker/orca/issues/8591 so it's good to add the label.  thank you for merging this!


ABBREVIATION:

Description
kindly add abbrevation fka. It means formerly known as.

Can I fix this?

@Njong392 can I fix this?

@milekboy any updates on this maybe?

Good day! If the issue is open I would like to take it.

Good day! If the issue is open I would like to take it.

Sure thing! I got no replies, so I guess it is all yours

Ok, then I take it.

It seems I have missed the notification for this issue.😞


[Doc] Modify comments of scaled_dot_product, attention utils

Stack from ghstack (oldest at bottom):

#205
#213
-> #204
#176

Differential Revision: D38283965

Codecov Report

:exclamation: No coverage uploaded for pull request base (gh/langong347/17/base@ea39b07). Click here to learn what that means.
The diff coverage is n/a.

@@                   Coverage Diff                    @@
##             gh/langong347/17/base     #204   +/-   ##
========================================================
  Coverage                         ?   93.11%
========================================================
  Files                            ?       47
  Lines                            ?     2802
  Branches                         ?        0
========================================================
  Hits                             ?     2609
  Misses                           ?      193
  Partials                         ?        0

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 ea39b07...acb6602. Read the comment docs.

@langong347 has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.

@langong347 has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.


Cartographer level trade causes crash 0.2.8

when trading with a cartographer and they level up from a novice up, you freeze get disconnected and then get a server watchdog crash as your player is still connected but not able to do anything / able to reconnect.  it seems to be when its creating a map trade idk.
for version:
numismatic-overhaul-0.2.8+1.19.jar.
File attached is a log for the crash (may be missing a bit as i had to copy it from the terminal rather than the file)
crashlog.txt

This issue is related to vanilla, and happens due to the server lagging out while the cartographer is searching for a specific structure. It is not something we can exactly fix within Numismatic Overhaul.
I do recommend a mod like Structure Essentials, as it heavily improves this search algorithm, possibly preventing your server from lagging out.
https://www.curseforge.com/minecraft/mc-mods/structure-essentials-forge-fabric
Alternatively, in your servers server.properties file (server config), you can increase the max-tick-time to something higher. I believe 120000 ticks (2 minutes) is usually required for slow servers + heavy modpacks.


[AutoRelease] t2-machinelearningservices-2023-05-12-76717(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/4161
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1006
BuildTargetingString
azure-mgmt-machinelearningservices
Skip.CreateApiReview
true

issue link:https://github.com/Azure/sdk-release-request/issues/4161

/check-enforcer override


Only update run count when user is not already a stargazer

Stack:

#155
#154 ⮜
#153
#152
#151
#150
#149
#148
#147

Checking the file into Git, don't want it to change on its own.

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

Although, I personally enjoy sometime looking in there and seeing how many times I ran spr:
showPRLink: true
logGitCommands: true
logGitHubCalls: true
statusBitsHeader: false
stargazer: true
runcount: 2245

That's a lot of times! I'm only getting started :-) Already proving useful.

The main reason for the counter is for the stargaze prompt, so I'm ok with this change.
I'm not sure I understand why you are checking in the user config file?

I have all config files in my home directory checked into Git for synchronization :)

✓ Commit merged in pull request #156


Setting default values for hours and minutes

I need to set default value for minutes and hours and make the text readonly. Is it possible using any method or do i have to customise it?

You can use the date picker and show the time in the picker button/ion-item/whatever only. It is currently not possible to disable any part of the picker, only changeable inputs should be shown to user.


🐛 BUG: [esinstall] outdated/wrong documentation?

Quick checklist

[X] I am using the latest version of Snowpack and all plugins. N/A

What package manager are you using?
npm
What operating system are you using?
Ubuntu 20 under WSL2
Describe the bug
I followed the directions for esinstall and it doesn't do what it says in the README.
import { install, printStats } from "esinstall";

(async () => {
  const result = await install(["preact", "htm"]);

  if (result.success) {
    printStats(result.stats);
  } else {
    console.error(JSON.stringify(result));
  }
})();

Output:
{"importMap":{"imports":{"htm":"./htm.js","preact":"./preact.js"}},"stats":{"direct":{"htm.js":{"size":1234,"gzip":662,"brotli":596},"preact.js":{"size":10043,"gzip":3992,"brotli":3674}},"common":{}}}

There is no success property in there?
(I don't know why we need to configure this with a build-script in the first place? Documentation is never a good substitute for something that actually emits an error message if you're using it wrong... Please consider adding a CLI front-end like we had in the old days with Pika? Installing an npm package as an ES module should be easy - no other project makes that easy since we lost Pika, and then again since the role of Snowpack completely changed from being essentially this... You had the simplest, most elegant solution to this... Please, consider making this simple again. 🙏)
Steps to reproduce
(see above)
Link to minimal reproducible example (optional)
No response

Even this doesn't work:
import { install, printStats } from "esinstall";

(async () => {
  const result = await install(["preact", "htm"]);

  printStats(result.stats);
})();

printStats just doesn't do anything - although from the console output above, you can see the stats property is there.
19k weekly downloads tells me somebody is successfully using this tool, or at least trying to?
Is this package maintained and supported or is it defunct?

Oh, okay, printStats doesn't actually print, heh... console.log(result.stats) and it works.
I'm going to guess the missing success flag has been replaced by a rejected Promise or something?
Looks like this is just a documentation issue.
(It would be really nice with a CLI front-end though - this really shouldn't be this hard.)


handle resize like most window managers

As the title suggests, this issue encourages the use of mod4 and right click instead of mod1 and left click.
This way this window manager would be more convenient for newbies who want to experiment.
I've come back to this window manager a few times until I noticed how to resize windows.
I've already tried to implement this, but my C knowledge is very limiting.
Keep up the good work!
Hope this gets implemented.

imo there are two main problems here,
The first is discoverability - it should be intuitive for folks to find out how to resize windows via the documentation. We could try adding something to the man pages or docs to help with this. Ideally I think we could add some sort of functionality to allow regional-window resizing (i.e. if your cursor is at the bottom right of a window you can drag to resize it there, etc).
The second is configurability - currently we hard-code the fact that window resizing and movement via the pointer is initiated by a left-pointer click. However, we could make this configurable to the right mouse button quite easily. If you're interested in implementing this, I suggest looking at this commit for reference. I can try taking a swing at this, but I can't guarantee I'll have bandwidth anytime soon.

Documenting and changing the resize behaviour seem good suggestions.
However, I am unable to figure out how to change resize from left click to right click. Could you give me a hint, or even better, look at it yourself?
As I mentioned above, my C knowledge is not really good.

I've made an attempt to implement the feature in question in a separate branch
You can modify the buttons used for moving and resizing during compile time by modifying config.h.
#define MOVE_MASK Mod4Mask
#define MOVE_BUTTON 1
#define MOVE_BUTTON_MASK Button1Mask

#define RESIZE_MASK Mod4Mask
#define RESIZE_BUTTON 3
#define RESIZE_BUTTON_MASK Button3Mask

Feel free to check out the commit history to see what I changed
P.S: Button3 is the right mouse button

to add to this, I've merged @Sidd-Dino's changes with #184 on my own fork here.
i'm not entirely sure if i've done everything correctly (i wasn't sure if i could merge branches across forks), and i'm pretty sure both of these changes already have a pr associated with them, so i've just grouped them together for convenience if anyone stumbles in here while we wait for our based @JLErvin to return to us. :pray:

Hey @letterlock,
The changes I made in my fork are a bit janky..
They were intended as a proof of concept.
I might make changes to them so that they are... less jank.

by all means! i just wanted easier access to a stable berry build with your fix and #184 together so i didn't have to manually rebuild it if i needed to reinstall etc.
i don't know enough c to be able to recognise janky code, let alone run with it and build other stuff on top of it, if that was your concern.

I encourage you to open a PR for this @Sidd-Dino, that's the most effective way for me to give feedback so that we can merge this into the main repo (btw, you can create PRs for forks).
In general I think it's pretty close so I think it could be worth doing :)

#191
A new PR is born.
The new PR has modifications that allows configuration during compile time (via config.h) and during runtime using berryc.
The config.h options being
#define MOVE_BUTTON   1
#define RESIZE_BUTTON 1

The berryc options being
berryc move_button    1
berryc resize_button  3

that looks great. is there any way i can help test these changes (if necessary)? how much does the code differ from your fork?

Just try running it and see if moving/resizing windows with the configured mouse buttons works or not.
configure the mouse buttons using berryc
Mouse buttons
1 - Left Mouse Button
2 - Middle Mouse Button (Pressing down on the scroll wheel)
3 - Right Mouse Button

The branch mentioned in the PR differs greatly from the branch i mentioned previously in this thread.
The IPC stuff is just stuff needed for berryc to communicate with berry

sweet. tested with
berryc resize_mask "mod4"
berryc resize_button 3

in my autostart file. works as expected. fiddling around with different buttons seems also to work fine, no responding to buttons outside of the specified commands or not responding to the correct ones.

so - unsure if i should be double posting here or editing my earlier reply, so sorry if this is bad form.
it seems that this build breaks the ability to drag windows by the title bar of their decorations. it's a very minor issue imo and i only notice it because i tend to swap between my gaming rig and my linux box quite often. i just figured i would mention it, as it seems related to @Sidd-Dino's changes.

Whooops my bad.
Working on fixing it ASAP

that does indeed fix it.

P.S : Please mention any new issues caused by my changes in this PR https://github.com/JLErvin/berry/pull/191. It's easier to keep track of things :D

right, of course! will do that from now on. i knew there was a better place but i was blanking. x^)


Initial translation of handling-events

@nutboltu vai, can you please review again?
The tests actually passed. You can see the details in the netlify website.


Shortcut for Min BIN no longer works on updated Web app

When M is pressed it does not list the selected player for minimum price as stated.

Thanks for the heads up. I don't play FUT anymore, so I wouldn't have caught this issue. A few things broke because of their update, but I believe I've patched them. I just published a new version to the store, and it should be available within the hour.


About Azure AD Authentication for Azure Database for MySQL Flexible Server listed in the table

In the table provided in the "Comparing the MySQL deployment options in Azure" section, the item "Azure AD Authentication" for Azure Database for MySQL Flexible Server is listed as "Not Supported" when it should be "In preview" I think.
(Because the feature of Azure AD authentication for Azure Database for MySQL – Flexible Server has been announced as public preview on 09/28/2022)

Public preview: Azure AD authentication with Azure Database for MySQL – Flexible Server | Azure updates | Microsoft Azure
https://azure.microsoft.com/en-us/updates/public-preview-azure-ad-authentication-with-azure-database-for-mysql-flexible-server/

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

ID: cd33da12-a2f9-fe16-fef2-2143551b9a7f
Version Independent ID: b9c9faff-b0f1-23cd-be91-cd0a4222e7b4
Content: Selecting the right deployment type - Azure Database for MySQL
Content Source: articles/mysql/single-server/select-right-deployment-type.md
Service: mysql
Sub-service: single-server
GitHub Login: @savjani
Microsoft Alias: pariks

@YuKogasaka
Thanks for your feedback! We will investigate and update as appropriate.

Hi @savjani ,
Could you please look into this document enhancement request which needs update in table mentioned by the user.

@YuKogasaka
Thank you for your contribution.
Azure AD is now GA as of last week.
The article will be updated by the end of the week.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion, and we will reopen the issue. To ask a question directly to our Q&A environment, please visit https://docs.microsoft.com/en-us/answers/support/ for more information on asking questions about the product or service.
Thank you.
#please-close


Title Order and Creator order

Since multiple titles will either be in random order or alpha order in the current Sufia 7 implementation, I have checked production to see what kind of impact that will have.
We currently have 39 depositors using more than one title:
{"sxs551"=>1, "mjf25"=>5, "gmr159"=>1, "mjl9"=>1, "jsm334"=>3, "sem268"=>31, "gnf1"=>5, "bqw5095"=>1, "dgn2"=>2, "sxh36"=>6, "tlr15"=>1, "dvo"=>3, "sjs49"=>1, "cwc5"=>1, "mus54"=>1, "jdw5170"=>221, "mjp5130"=>1, "adm135"=>1, "tdp11"=>1, "lcb11"=>3, "cpl2"=>4, "jjd264"=>1, "jjm38"=>1, "oom5021"=>1, "jgm106"=>1, "pmh22"=>2, "ajs23"=>2, "dmc186"=>1, "gac2"=>1, "esc10"=>1, "aus50"=>1, "ard19"=>1, "wuw12"=>1, "cej14"=>1, "bal19"=>1, "bah1"=>1, "bzl1"=>4, "ahh139"=>1, "lmo11"=>1}
Those depositors have deposited 317 files into the system.
Will add Creator information soon.
@olendorf Any other fields where order has an implied meaning?

Here is a link to Joshua Wilkins files in ScholarSphere: https://scholarsphere.psu.edu/catalog?depositor="jdw5170"&search_field=advanced
It looks like most of Sarah Chilenski's files are private: https://scholarsphere.psu.edu/catalog?depositor="sem268"&search_field=advanced

FYI: We can continue to use the latest Sufia, which is the forthcoming 7.2 release, and maintain the current behavior of a pseudo-preserved ordering. However, we will we need to pin our Gemfile to RDF ~> 1.99 and ActiveFedora ~> 10.3, so we'd be unable to take advantage of newer features in ActiveFedora 11, for example.
This should at least buy us some time to decide on alpha-sorted fields, or wait for a complete solution on the issue.

@cam156  @awead Should I be assignee on this?

@olendorf I believe so.  We need to decide what we want to do with the multiple titles and creators when we move to hyrax.  I think the original thought was that we could reach out to existing users to see what they meant by having multiple creators and titles so we could modify our metadata appropriately. That said I do not think this is needed for 3.0

Oh right. I actually did that. Creator order is important as entered. For manuscripts, one person said it wasn't too important because the manuscript counts. However, for things that are only published in Scholarsphere, expecially things like data, it becomes more important. Multiple titles, no one thought that they would do that.

ref #764

For creator and title we should preserve the order they input the items into the system.
Possibly using a secondary field that contains the display order of all items.

possible solutions:

Smash all the values into the current creator field
Use Order proxies
Keep current creator field as is and add a new predicate for storing display order

The last option is the current fan favorite.


Migrate etcd, master, node and nodepool stanzas

etcd, master, node and nodepool stanzas
This ticket is to remove the old stanzas in the heading from ansible/roles/kraken.config/files/config.yaml  and update the related Ansible tasks to match. When doing this the k2 schema, ansible/roles/kraken.config/files/schema.json , should be updated as well.

Notes
The nodepool stanza contains a mishmash of concepts: number of nodes, shared kube/container/os settings, provider specific volume configuration and node specific mount points. Under the k2recon, this information has been moved to the “nodes” (number of nodes and shared configuration)) and “nodeConfig” (provider and node) stanzas. Most of the configuration work should be completed by #269, #270, #271, #272, and #273 .
provider volume and node mountpoint configuration is stored in a new nodeConfig stanza. Is there any benefit to breaking these up a la k8s volume and volumeMounts stanzas?

Blocked on #276.

Looking only at etcd, this looks like it will be a fair amount of work. There are > ~80 references. Therefore I am assigning to myself until I either complete the work or figure out how to break it up into more manageable pieces.

So 2 things remain here:

The provider role requires more thought. Even though #269 was intended to handle that migration, the way things are currently written, I can't finally kill off the master node stanza from the old style config without making changes to the provider role.

We need to decide on what idiom we want to use to avoid {{ item }} name conflicts. {{ item }} is the default name given to loop iterations.

Since all of the include tasks list set the variable, any include which also uses iteration needs avoid name conflicts. The possible ways are here: http://docs.ansible.com/ansible/playbooks_loops.html#loop-control
a) loop_control results in less code - name: clean the terraform state generated prefix and other misc files.   file:     path: "{{ config_base | expanduser }}/{{ item.0.name }}/{{ i }}"     state: absent   when: kraken_action == 'down'   with_items:     - aws_prefix     - terraform.tfstate     - terraform.tfstate.backup   loop_control:     loop_var: i , but it would require less thought, and therefore be more robust if we just set facts, by convention, at the beginning of any task list. E.g., - set_fact:   cluster: "{{ item.0.name }}"   node: "{{ item.1.name }}". That way {{ item }} can retain it's generic use.

So, I've here's what the current code does. In each main.yaml, we use a loop_control to avoid polluting the namespace of included tasks files.
--
- include: service-account-key.yaml
  with_subelements:
    - "{{ kraken_config.clusters }}"
    - nodes
  loop_control:
    loop_var: outer_item
  when: item.1.apiServerConfig is defined

Then, at the beginning of each task file, I set to facts so that we can use descriptive, and shorter, names in any code that follows.
- set_fact:
  cluster: "{{ outer_item.0.name }}"
  node: "{{ outer_item.1.name }}"

We arguable do with just setting the facts, but then the item variable would be aliased and ansible would complain.

Another note, I don't think I will migrate the provider just yet. Not doing so just means we have to live with old style and new style configs for a bit  longer. Since we might want to do that anyway, to give people time to update their configs, I think it's more important to press on with this isolated change. Thought are definitely welcome.

PR merged:  https://github.com/samsung-cnct/k2/pull/306


Any idea, why zabbix agent service is not detected?

Describe the bug
When trying to query zabbix-agent on centos7, systeminformation returns running:false, however, the service is running on the machine.
 [root@s1-h4-tbocdn dist]# service zabbix-agent status
 Redirecting to /bin/systemctl status zabbix-agent.service
 ● zabbix-agent.service - Zabbix Agent
    Loaded: loaded (/usr/lib/systemd/system/zabbix-agent.service; enabled; vendor preset:  disabled)
    Active: active (running) since Wed 2019-08-07 14:04:10 CEST; 5min ago
   Process: 350 ExecStop=/bin/kill -SIGTERM $MAINPID (code=exited, status=0/SUCCESS)
   Process: 368 ExecStart=/usr/sbin/zabbix_agentd -c $CONFFILE (code=exited, status=0/SUCCESS)
  Main PID: 370 (zabbix_agentd)
    CGroup: /system.slice/zabbix-agent.service
            ├─370 /usr/sbin/zabbix_agentd -c /etc/zabbix/zabbix_agentd.conf
            ├─371 /usr/sbin/zabbix_agentd: collector [idle 1 sec]
            ├─372 /usr/sbin/zabbix_agentd: listener #1 [waiting for connection]
            ├─373 /usr/sbin/zabbix_agentd: listener #2 [waiting for connection]
            ├─374 /usr/sbin/zabbix_agentd: listener #3 [waiting for connection]
            └─375 /usr/sbin/zabbix_agentd: active checks #1 [idle 1 sec]

systeminformation returns:
{"name":"zabbix-agent","running":false,"startmode":"","pids":[],"pcpu":0,"pmem":0}]
To Reproduce
Steps to reproduce the behavior. Called as:
const data = await si.services("zabbix-agent");
Environment (please complete the following information):

"systeminformation": "^4.14.4",
OS: CentOS Linux release 7.6.1810 (Core)

I think I see the issue.
  [root@s1-h4-tbocdn ~]# service --status-all
 iptables: Chain already exists.
 netconsole module not loaded
 Configured devices:
 lo eth0
 Currently active devices:
 lo eth0

service --status-all returns wierd things on centos 7.6, Will try to figure out, whether it's a bug on my side or some changes to centos.
It might be needed to be changed to systemctl list-units --type service --all

@fr6nco : just one question:
To get the satus of the Zabbix agent, vou are calling is like this:
const data = await si.services("zabbix-agent");

So then  - as you defined a service - service --status-all should not be called.
What is the output of ps -axo pcpu,pmem,pid,command? Do you see the zabbix-agent here?
Anyway ... when calling it with "*" for all services I should really consider changing to systemctl list-units --type service --all

@fr6nco can you also provide the output of systemctl list-units --type service --all? Would help changing it ... Thank you in advance!

Hi @sebhildebrandt,
yes Im calling it like you mentioned. Here's a quick test
const si = require('systeminformation');

si.services("zabbix-agent").then((se) => {
	console.log(se);
});

which returns false for running, however the app is running.
[root@s1-h3-cdn tests]# node index.js
[ { name: 'zabbix-agent',
    running: false,
    startmode: '',
    pcpu: 0,
    pmem: 0 } ]

the command systemctl list-units --type service --all returns zabbix-agent in correct state:
zabbix-agent.service            loaded    active   running Zabbix Agent

Also the command you mentioned:
[root@s1-h3-cdn tests]# ps -axo pcpu,pmem,pid,command | grep zabbix
 0.0  0.0 13038 grep --color=auto zabbix
 0.0  0.0 14243 /usr/sbin/zabbix_agentd -c /etc/zabbix/zabbix_agentd.conf
 0.0  0.0 14255 /usr/sbin/zabbix_agentd: collector [idle 1 sec]
 0.0  0.0 14256 /usr/sbin/zabbix_agentd: listener #1 [waiting for connection]
 0.0  0.0 14257 /usr/sbin/zabbix_agentd: listener #2 [waiting for connection]
 0.0  0.0 14258 /usr/sbin/zabbix_agentd: listener #3 [waiting for connection]

So can you modify your code, that you ask for „zabbix_agent“ (underscore) instead of „zabbix-agent“ (hyphen). That should work ...

oh god..,,
yeah that works. thanks.
[ { name: 'zabbix_agent',
    running: true,
    startmode: '',
    pids: [ '14243', '14255', '14256', '14257', '14258' ],
    pcpu: 0.010129732753413528,
    pmem: 0 } ]


[Popover] Fix PopOver animation origin when targetOrigin.horizontal=middle

When the targetOrigin.horizontal of a popover (in my case a DropDownMenu) is set to middle,  it replaces it with vertical which is an invalid value, causing the transform-origin style to be rejected and default to the (horizontal and vertical) center of the menu.
This changes it to transform middle to center, which is the correct css value for centering horizontally.

[ ] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[x] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

@tsholmes Thanks


Add revision_id column to policies

:construction: Erchef Updates Incoming :construction:
Adds a revision_id column to policies. Per the draft spec, this column is required, so it should be NOT NULL, but we can't add this constraint if there's any rows that violate it. Which means we either need to do a data migration or nuke the data. Given the status of the feature, I think it's acceptable to nuke.

Going to abandon this branch and start over. Based on how Authz is specified in the RFC, I need to create quite a few more tables.


Browser Sync doesn't work, issue Pug rendering to HTML

Whenever I run the app with npm start, the latest changes to the Pug file(s) are being rendered and build into /dist. That's good, as intended. As soon as I start changing code in the Pug file(s), the Browser Sync seems not to be working. It doesn't auto sync the browser page to the latest changes. Only when I Terminate batch job and re-run the npm start command.
Is this how it is intended or am I missing something?

Upath has been added to the npm scripts to work for windows machines, so this should be working now!

Hi David, thank you. Will be testing that today!


Ability to mint a "farming gauge"

The idea here is to increase the target market of gauges from just DAOs to include speculatooooors and passive incoooomers.
If you mint a gauge with a 0x0 staking token, all the emissions accumulate to the gauge owner. They could then burn their MERC into the gauge to increase its value on the secondary market or exit their MERC as income.

This is cool. Basically 0x0 staking token means no one can stake to it.


Extend creation of changelog for multiple repositories

Hi,
would be nice if it would be possible to have the changelog generator create one central changelog for multiple repositories. So if your projects consists of multiple repos you can just get one single changelog file generated which you can maintain in your main repo.

:+1:

How do you think it should be arranged? Like to change logs one-by-one or amended in one list? Please, prvide some template, how are you see change log with multiple repos.

Hi @skywinder,
I will try to get a hol on a template for this. Actually should be something like an amendmend for each repo.
so basicially:

#Temp Changelog
6.1-SP0_33 (2015-08-04 13:18)
Repo 1
Bugfixes

BUGFIX: Lorem ipsum...  [#1396] (http://link.go.es/here)
BUGFIX: Lorem ipsum...  [#1396] (http://link.go.es/here)
BUGFIX: Lorem ipsum...  [#1396] (http://link.go.es/here)

Features

FEATURE: Lorem ipsum...  [#1396] (http://link.go.es/here)
FEATURE: Lorem ipsum...  [#1396] (http://link.go.es/here)
FEATURE: Lorem ipsum...  [#1396] (http://link.go.es/here)

6.2-SP0_15 (2015-08-04 08:47)
6.1-SP0_32 (2015-08-04 08:17)
6.1-SP0_31 (2015-08-04 07:33)
Bugfixes

ETBUGS-9458: Kalenderaufrufe #1425
BUGFIX: fixed date time format for payment date in SEPA mandate #1421
BUGFIX: fixed mapping for data exchange object company #1420
ETBUGS-9352: Rechnungssicht in erweit. Druckeinstellungen nicht aenderbar #1419
ETDFB-337: Wiedereintritt FCN Kunden #1418
BUGFIX: rename import response for using in SAPETCustomerImport webservice #1415
ETBUGS-9435: Export of payment methods are empty #1414
BUGFIX: Force soap interface to use soap 1.1 for generic interfaces #1411
BUGFIX: Reload iban reductions #1410
ETBUGS-8246: Anzeige Gutschein-Info im alten Design und ohne Gutschein-Daten #1408
ETBUGS-8689: Buchung einer Aufwertungsveranstaltung über Tickettroublepoint #1407

OR also possible:

6.1-SP0_33 (2015-08-04 13:18)
Bugfixes

''Repo1'' - BUGFIX: Lorem ipsum...  [#1394] (http://link.go.es/here)
''Repo1'' - BUGFIX: Lorem ipsum...  [#1395] (http://link.go.es/here)
''Repo1'' - BUGFIX: Lorem ipsum...  [#1396] (http://link.go.es/here)
''Repo2'' - BUGFIX: Lorem ipsum...  [#1294] (http://link.go.es/here)
''Repo2'' - BUGFIX: Lorem ipsum...  [#1295] (http://link.go.es/here)
''Repo2'' - BUGFIX: Lorem ipsum...  [#1296] (http://link.go.es/here)

Features

''Repo1'' - FEATURE: Lorem ipsum...  [#1391] (http://link.go.es/here)
''Repo1'' - FEATURE: Lorem ipsum...  [#1392] (http://link.go.es/here)
''Repo1'' - FEATURE: Lorem ipsum...  [#1393] (http://link.go.es/here)
''Repo2'' - FEATURE: Lorem ipsum...  [#1291] (http://link.go.es/here)
''Repo2'' - FEATURE: Lorem ipsum...  [#1292] (http://link.go.es/here)
''Repo2'' - FEATURE: Lorem ipsum...  [#1293] (http://link.go.es/here)

6.2-SP0_12 (2015-08-03 12:11)
Bugfixes

''Repo1'' - BUGFIX: Lorem ipsum...  [#1394] (http://link.go.es/here)
''Repo1'' - BUGFIX: Lorem ipsum...  [#1395] (http://link.go.es/here)
''Repo1'' - BUGFIX: Lorem ipsum...  [#1396] (http://link.go.es/here)
''Repo2'' - BUGFIX: Lorem ipsum...  [#1294] (http://link.go.es/here)
''Repo2'' - BUGFIX: Lorem ipsum...  [#1295] (http://link.go.es/here)
''Repo2'' - BUGFIX: Lorem ipsum...  [#1296] (http://link.go.es/here)

Features

''Repo1'' - FEATURE: Lorem ipsum...  [#1391] (http://link.go.es/here)
''Repo1'' - FEATURE: Lorem ipsum...  [#1392] (http://link.go.es/here)
''Repo1'' - FEATURE: Lorem ipsum...  [#1393] (http://link.go.es/here)
''Repo2'' - FEATURE: Lorem ipsum...  [#1291] (http://link.go.es/here)
''Repo2'' - FEATURE: Lorem ipsum...  [#1292] (http://link.go.es/here)
''Repo2'' - FEATURE: Lorem ipsum...  [#1293] (http://link.go.es/here)

@eryx12o45 Thanks for clarification! :+1:
IMHO first option is more structured.

Sure :-) Either way, would be great to have such a thing. I guess more people a using separated repositories for one project. Thanks for your great tool, btw. ;-)

I would also appreciate such a feature. I prefer the second suggest of @eryx12o45, but you don't need the repo as prefix in each line, instead the link to the issue contains the repo. E. g. organisation/repo#issue

Hi,
any news on that?

@eryx12o45 it's not in my priority list for now. But I'm ready to review PR, is someone would like to help to implement this :+1:

any news on that?

any news on that?

I up for this feature too.
Second option on the log layout is preferable. But having the possibility to add labels (to the configuration file categories) from two (or more) repos and combining those on a single changelog output would be awesome!

👍 - second option would be great!

Having this feature would be great!
Is there any news? 🙏🏻


Restyle "welcome back" message

Should fit in the space between the top and the hero content.

This looks good on staging so I'm closing the issue.


(firefox) Automatic update broken - file URL has changed

Expected Behavior
cup firefox updates Firefox to v65.0.1
Current Behavior
Package update is still in moderation
Possible Solution
The URL has changed: according to https://download-installer.cdn.mozilla.net/pub/firefox/releases/65.0.1/win64/en-US/, the filename includes the version number now and is no longer held generic

I'm not sure how exactly that happened, but https://gist.github.com/choco-bot/837ec070ea1277c58e491d4e4637a163 lists that the download should happen from https://download-installer.cdn.mozilla.net/pub/firefox/releases/65.0.1/win64/en-US/Firefox Installer.exe

I'm guessing that when we initially checked for updates, they hadn't yet uploaded the correctly named installer.
I'm working on an update to the checker to not use the redirected urls, which hopefully will prevent situations like this to occur in the future.

This is now fixed, and a I'm forcing a resubmit of the latest package version

The updated package works, thank you!


The nav having extra heading and it's not sticky

Hello, I'm a potential GSoC student, I'm working on a demo and I'm using those components
the nav has extra space, I can work on it
@halkeye @krisstern

The component shouldn't decide how and where it is on the page.

I just removed all the CSS, that will affect the nav, I will check again, I have explored the component, and yeah you are right but Its getting me confused :laughing:

The most basic fixed top (as seen on accounts.jenkins.io) is
.fixed-top {
  position: fixed;
  top: 0;
  right: 0;
  left: 0;
  z-index: 1030;
}

None of the other sites have it sticky.

Thanks, I will use them instead to force the ui, and see, I will be back to you


Retweets TEXT is truncated, retweeted_status not available

Per https://developer.twitter.com/en/docs/tweets/data-dictionary/overview/tweet-object the text field is truncated for retweets, with the full value available in retweeted_status. Can retweeted_status be added to the schema for this connector?
At the moment any RT is only available in its first 140 characters (give or take for user name length etc):
ksql> select retweet, max(len(text)) as max_tweet_length  from all_tweets group by retweet emit changes;
+--------------------------------------------------------------------+--------------------------------------------------------------------+
|RETWEET                                                             |MAX_TWEET_LENGTH                                                    |
+--------------------------------------------------------------------+--------------------------------------------------------------------+
|false                                                               |411                                                                 |
|true                                                                |152                                                                 |

@rmoff in the API I am using getRetweetedStatus and getQuotedStatus are full Status objects. If I pulled these in would it work for you?

yes that would be 💯 what I need 👍


added using wrapping the response content stream

as discussed on twitter, just making disposing of the response content stream with the using

Thanks Ricardo!  Going to publish a new release in the beginning of next week :)


[FEAT]: CI - Pass full backstopjs package name/version to Docker

Summary
Let's allow the backstopjs package installation to be passable in args during Docker build.
Why?
This will allow GitHub actions to pass the current backstopjs branch to Docker, so build, publish, and tests are run with the latest changes this branch has introduced.
How?

[ ] rename the Dockerfile ARG "BACKSTOPJS_VERSION" → "BACKSTOP_INSTALL_SOURCE"
[ ] adjust the ENV BACKSTOPJS_VERSION=$BACKSTOPJS_VERSION → BACKSTOP_INSTALL_SOURCE=$BACKSTOP_INSTALL_SOURCE
[ ] adjust npm scripts as needed
[ ] create GitHub Actions var BACKSTOP_INSTALL_SOURCE to concatenate installation source path
[ ] update GitHub Actions to pass the BACKSTOP_INSTALL_SOURCE to Docker builds
[ ] check if any $TAGs need to be adjusted

@garris FYA — another improvement for your consideration. This would include any changes to Dockerfile as part of the overall regression suite. Currently one needs to commit a change to Dockerfile to include the GitHub path to their branch to test changes within npm backstop itself.
Super hard to explain. This feature is a good candidate for first-hand review when it's ready.


Spot intensity tune

Refactored get spot intensity to make search bounding box smaller so it doesn't "wander off" and misses the spots.

Please feel free to merge after resolving the conflicts with master.

Codecov Report

Merging #69 into master will increase coverage by 5.09%.
The diff coverage is 52.82%.

@@            Coverage Diff             @@
##           master      #69      +/-   ##
==========================================
+ Coverage   35.80%   40.89%   +5.09%
==========================================
  Files          23       25       +2
  Lines        1606     1692      +86
==========================================
+ Hits          575      692     +117
+ Misses       1031     1000      -31

Flag
Coverage Δ

#unittests
40.89% <52.82%> (+5.09%)
:arrow_up:

Impacted Files
Coverage Δ

array_analyzer/load/debug_plots.py
14.28% <0.00%> (-0.15%)
:arrow_down:

array_analyzer/transform/point_registration.py
7.86% <0.00%> (ø)

array_analyzer/workflows/interpolation_wf.py
25.00% <0.00%> (+0.75%)
:arrow_up:

array_analyzer/workflows/registration_workflow.py
20.00% <0.00%> (-0.23%)
:arrow_down:

array_analyzer/extract/img_processing.py
13.23% <11.11%> (+0.48%)
:arrow_up:

array_analyzer/transform/array_generation.py
10.93% <14.28%> (+2.48%)
:arrow_up:

array_analyzer/load/report.py
23.33% <18.75%> (+9.04%)
:arrow_up:

array_analyzer/extract/image_parser.py
9.18% <33.33%> (+0.02%)
:arrow_up:

array_analyzer/extract/constants.py
100.00% <100.00%> (ø)

array_analyzer/utils/spot_regionprop.py
100.00% <100.00%> (ø)

... and 6 more

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 601efec...5f29c93. Read the comment docs.


UNIXServer does not delete socket file on destruction

require "socket"

def check_file(prefix, file)
  if File.exists? file
    puts "#{prefix} File exists"
  else
    puts "#{prefix} No file"
  end
end

path = "/tmp/unixtest_cr.sock"

check_file "1/", path
UNIXServer.new(path)
begin
  check_file "2/", path
  UNIXServer.new(path)
rescue err
  puts "Error raised !"
  puts
  check_file "Final/", path
  puts "Error was: #{err.message}"
end

Exemple run with carc.in using Crystal 0.20.1 : https://carc.in/#/r/1hp3
1/ No file
2/ File exists
Error raised !

Final/ No file
Error was: Error binding UNIX server at /tmp/unixtest_cr.sock: Address already in use

Exemple run on master, using Crystal 0.20.3 :
Using compiled compiler at .build/crystal
1/ No file
2/ File exists
Error raised !

Final/ No file
Error was: bind: Address already in use

I'm expecting the output to be :
1/ No file
2/ No file

I'm on Archlinux, kernel 4.8.13
@ysbaddaden you worked on this, any idea ? (Btw, awesome work on the sockets 👍)

It seems that there is no destructor, which could allow to close the socket..
It is not documented, but I think we can do this using GC#add_finalizer, but I didn't find any usage of it..

This is by design. You must either explicitly close the socket using #close or using the .open constructor, that will eventually close it.
The IO::FileDescriptor ancestor should already have a finalize that will call #close, but that will only happen when GC collects the memory.

Does the GC collects memory on program exit ?
So does the finalize functions get called on program normal quit (not crashed) ?

Does the GC collects memory on program exit ?

No, the OS will already to the cleaning (allocated memory, opened sockets, ...). Yet, it won't delete the socket created by UNIXServer. You may tell it to, thought:
server = UNIXServer.new("/tmp/service.sock")
at_exit { server.close }

IO::FileDescriptor's finalizer does call close, so the socket file is eventually deleted when the object is collected by the GC. One should use the following examples (there are other solutions) to have better control over when the file is actually deleted.
UNIXServer.open("/tmp/my.sock") do |server|
  # ...
end

server = UNIXServer.new("/tmp/my.sock")
at_exit { server.close }

Thanks it's clear now!


remove Jenkins.get().getRootUrl() to generate relative url

@Jochen-A-Fuerbacher get we get this merged please?

This commit introduced issue when jenkins is not in root directory of given URL.
In example, if you have jenkins in localhost/jenkins/..... then badge functionality is broken.
That is why this uses Global Configuration Entry before. Please revert this.

I just released a new version reverting this PR.


ci: add a github workflow to label pull requests based on their title

Description of the change

Related issue number

How to verify the PR

Was the PR tested

[x] I have added >=1 unit test(s) for every new method I have added.
[x] I have ensured all unit tests pass

@Ssukriti @anhuong If we are comfortable with this we can merge.

It puts a message like https://github.com/konveyor/move2kube/pull/1191#issuecomment-2264954757 as soon as a PR is created.
It creates an action which will fail if it does not have ['feat', 'fix', 'docs', 'style', 'refactor', 'perf', 'test', 'build', 'ci', 'chore', 'revert'] as the prefix of the PR.


Camera - FPS Imgui indicator

Allow to reduce, through an option, the fps (default: 60).

Camera is now considered as a configurable object and FPS is a camera option

Removing CLI option in favor of a dedicated Imgui widget


Offer async API

Hi,
have you thought about providing an async API? Since you are using reqwest, it should be easily possible by using reqwest::async::Client and returning futures instead of results.

I started https://github.com/magnusmanske/mediawiki_rust/tree/async but I'm not really happy with the approach, maybe because async is still a bit unstable, maybe because I haven't really worked with it yet. Comments and PRs welcome!

Default Api struct is now async (use ApiSync for the old one)


Optional chaining

We should allow JavaScript-style optional chaining: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Optional_chaining
struct Bar {
    baz: i64
}

struct Foo {
    bar: Bar?
}

function main() {
    let mut foo = Foo(bar: None)
    let baz = foo?.bar?.baz
}

Implementation wise, this should become an expression that returns an Optional<i64>, since that's the type of the last ?.
If we short-circuit on any of the ?. operators, we should return an empty Optional<i64>.

i'm interested in picking this one up.

[not sure where to ask this]
I'm wondering if there is an even more elegant approach syntax for these different types of accesses (normal a.b, optional a?.b, and infallible a!.b).
Typically, a chain of property accesses "degrades in certainty", so writing a?.b!.c would be ambiguous. Does it mean that c must exist if b does?
Also, the most common case would be something like a!.b?.c?.d?.e?.f?.(). Would it make more sense to "carry over" the degree of certainty unless overridden, like a!.b?.c.d.e.f().

Implemented in dc4e06920f30dc9b27f54bca04f5aa4b9d6cc867.


Allow reading only n tail bytes from a file

Carrying on from #415, a user can specify a start and end range to read from a remote file. The JSON API also allows reading n tail bytes from a file, e.g. range=-100 would be the last 100 bytes. Our API currently looks like:
file.createReadStream({ start: 0, end: 10 })

We could allow specifying a range in another way:
file.createReadStream({ range: "0-10" })

Which would open the doors for:
file.createReadStream({ range: "-100" })

Definitely open to other ideas of how we can support ranges of both types: "start and end" and "end offset".
Note: start and end was added to match the native fs module's createReadStream's arguments, which behaves the same was currently as ours.

Another idea I'm not too attached to but figured I'd throw it out. What about:
file.createReadStream({ tail: 10 })

This would override or be always overridden by start/end. Problems include introducing another parameter, parameters that void other parameters, but benefits are it's clear its purpose.

How about just allowing:
file.createReadStream({ end: -10 })

and adding documentation to explain that's reading -10 tail bytes?

I'm :+1: to end: -10 (the same way in CSS you can do right: -10px for example). Definitely would rather this than adding a new parameter... But you guys are the experts.

Cool cool. Let's try it out and if we find the community hates it, we can try again!


Are the questions only available in English?

Are the questions on the 'OpenChain Self Certification Questionnaire' only available in English?  I plan to translate the questions into Korean. However, "To add a language:" in the Readme file doesn't mention how to add the translate questions.

Oh, I see. Thank you!


nginx 1.11.1

this version of nginx has HTTP/2 support.
note: both the previous packaged nginx and this new one errors out on startup with this error, so additional post-install would be better. unsure what to do to improve situation
nginx: [alert] could not open error log file: CreateFile() "logs/error.log" failed (3: The system cannot find the path specified)
2016/07/05 13:05:10 [emerg] 12452#10072: CreateFile() "C:\Users\m\dev\scoop/conf/nginx.conf" failed (3: The system cannot find the path specified)

I wonder if it's just a matter of creating the log and conf directories... not sure. This is good for now though.


Generation of TypeScript SDK using OpenAPI JSON file generated by Swashbuckle generates invalid models

Generated models do not correctly implement Parsable. TypeScript compiler gives the following error.
Class 'WeatherForecast' incorrectly implements interface 'Parsable'.
  Property 'additionalData' is missing in type 'WeatherForecast' but required in type 'Parsable'.ts(2420)

Additional information

Created new ASP.NET/React app using dotnet new react.
Added Swashbuckle with dotnet add package Swashbuckle.AspNetCore
Updated Startup.cs to generate OpenAPI by:

Add following code at end of ConfigureServices:services.AddSwaggerGen(c =>
{
    c.AddServer(new Microsoft.OpenApi.Models.OpenApiServer
    {
        Url = "https://localhost:5001"
    });
    c.SwaggerDoc("v1", new Microsoft.OpenApi.Models.OpenApiInfo {
        Title = "Weather API",
        Version = "v1"
    });
});

Add folowing code to Configure just before the app.UseHttpsRedirection(); line:app.UseSwagger();
app.UseSwaggerUI(c =>
{
    c.SwaggerEndpoint("/swagger/v1/swagger.json", "Issues API V1");
    c.RoutePrefix = "swagger";
});

Run the project. Once project is running, run kiota.exe as follows (from root of project):kiota.exe --language TypeScript --openapi https://localhost:5001/swagger/v1/swagger.json --output .\ClientApp\src\api

From your command-line, go the ClientApp directory in the project and run npm start. You'll see something like:TypeScript error in C:/Source/temp/KiotaTest/ClientApp/src/api/weatherForecast/weatherForecast.ts(3,14):
Class 'WeatherForecast' incorrectly implements interface 'Parsable'.
  Property 'additionalData' is missing in type 'WeatherForecast' but required in type 'Parsable'.  TS2420

    1 | import {Parsable, ParseNode, SerializationWriter} from '@microsoft/kiota-abstractions';
    2 |
  > 3 | export class WeatherForecast implements Parsable {
      |              ^
    4 |     private _date?: Date | undefined;
    5 |     private _summary?: string | undefined;
    6 |     private _temperatureC?: number | undefined;

The problem seems to be caused by the following line in swagger.json: "additionalProperties": false. Removing this line causes the generation to succeed.
Generated OpenAPI
{
  "openapi": "3.0.1",
  "info": {
    "title": "Weather API",
    "version": "v1"
  },
  "servers": [
    {
      "url": "https://localhost:5001"
    }
  ],
  "paths": {
    "/WeatherForecast": {
      "get": {
        "tags": [
          "WeatherForecast"
        ],
        "responses": {
          "200": {
            "description": "Success",
            "content": {
              "text/plain": {
                "schema": {
                  "type": "array",
                  "items": {
                    "$ref": "#/components/schemas/WeatherForecast"
                  }
                }
              },
              "application/json": {
                "schema": {
                  "type": "array",
                  "items": {
                    "$ref": "#/components/schemas/WeatherForecast"
                  }
                }
              },
              "text/json": {
                "schema": {
                  "type": "array",
                  "items": {
                    "$ref": "#/components/schemas/WeatherForecast"
                  }
                }
              }
            }
          }
        }
      }
    }
  },
  "components": {
    "schemas": {
      "WeatherForecast": {
        "type": "object",
        "properties": {
          "date": {
            "type": "string",
            "format": "date-time"
          },
          "temperatureC": {
            "type": "integer",
            "format": "int32"
          },
          "temperatureF": {
            "type": "integer",
            "format": "int32",
            "readOnly": true
          },
          "summary": {
            "type": "string",
            "nullable": true
          }
        },
        "additionalProperties": false
      }
    }
  }
}

Any service driven by OData is always going to require AdditionalProperties to be true, which is the default.  To support non-OData services that are explicit about not supporting AdditionalProperties, I guess we will need to refactor the Parseable interface.

yeah that was my first thought too, we'll need to divide Parseable in two interfaces, probably still Parseable and AdditionalPropertyStore something.


tabUrlPath not used

Ionic version:  (check one with "x")
[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[ x ] 3.x
I'm submitting a ...  (check one with "x")
[x ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
tabUrlPath is not used and results in tabs URL like /#/nav/n4/tabs.page/tabs/t0/my-page/my.page.
Expected behavior:
tabUrlPath should be used.
Steps to reproduce:

Create a new app with tabs starter
Add tabUrlPath to the tabs
Reload the page and look at the URL

Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
global packages:

    @ionic/cli-utils : 1.4.0
    Cordova CLI      : 7.0.1
    Ionic CLI        : 3.4.0

local packages:

    @ionic/app-scripts              : 1.3.12
    @ionic/cli-plugin-cordova       : 1.4.0
    @ionic/cli-plugin-ionic-angular : 1.3.1
    Cordova Platforms               : android 6.2.3 browser 4.1.0
    Ionic Framework                 : ionic-angular 3.5.0

System:

    Node       : v8.1.3
    OS         : Linux 4.4
    Xcode      : not installed
    ios-deploy : not installed
    ios-sim    : not installed
    npm        : 5.0.3

Thanks for opening an issue. I think @danbucholtz is currently working on the navigation/URLs.

@HugoHeneault,
Shoot, I'll check it out and make sure we account for that. Thanks. I didn't know that was a thing.
Thanks,
Dan

Thanks @danbucholtz
I'm using lazyloading if it can help ;)

@HugoHeneault,
I'm currently using tabUrlPath on four <ion-tabs> and it's working properly on 3.5.0... How are you using it? Are you sure you're passing a string?

I too get "http://localhost:8100/#/nav/n4/page-name". Also using lazy loading.

Yeah I'm passing a string.

@HugoHeneault Yes, I'm using lazy loading and the latest @ionic/app-scripts (2.0.0).

I'm on 3.5 with app-scripts 1.3.7 and tabUrlPath is also not working for me

tabUrlPath does not appear to be working for me either
global packages:
@ionic/cli-utils : 1.4.0
Cordova CLI      : 7.0.1
Ionic CLI        : 3.4.0

local packages:
@ionic/app-scripts              : 1.3.12
@ionic/cli-plugin-cordova       : 1.4.0
@ionic/cli-plugin-ionic-angular : 1.3.1
Cordova Platforms               : none
Ionic Framework                 : ionic-angular 3.5.0

System:
Node       : v6.11.0
OS         : Windows 10
Xcode      : not installed
ios-deploy : not installed
ios-sim    : not installed
npm        : 3.10.10

@FdezRomero And what URL are you having? I'll check with the new scripts.

@HugoHeneault I used to have /nav/n4/tabs/t0/tab-url-path-here but I noticed that I could get rid of the first navigation segment (/nav/n4) by moving the <ion-tabs> directly to app.component.ts. This left the URL as /tabs/t0/tab-url-path/page-name.
Then I added a name to the tabs with <ion-tabs name="app"> and finally got /tabs/app/tab-url-path/page-name.
In any case, the tabUrlPath input was working well for me since the beginning. I have no idea why isn't working for you.
Do you have a Plunker showing how is your navigation set up? How are your URLs?

I'll try adding the <ion-tabs name="app"> and moving to the app.component.ts (is that a good practice?).
My problem is for making deeplinks (from push notifs for example). I thought I could use /my-tab instead of the /tabs/t0/tab-url-path/my-tab thing

Hi,
This issue no longer exists.
Thanks,
Dan


Blog RSS workaround via redirect using 302 status code

Followup to #5520
Contributes to #5651
Thanks to @jord1e for point that out. /cc @dmathieu. Can you confirm that it still works for you (it should).
Once I get a confirmation, I'll double check other 301s to see if they should be 302s.

Redirect test:

https://deploy-preview-5652--opentelemetry.netlify.app/blog
https://deploy-preview-5652--opentelemetry.netlify.app/blog/index.xml

$ curl -I https://deploy-preview-5652--opentelemetry.netlify.app/blog/index.xml
HTTP/2 302
...
location: /blog/2024/index.xml
...

/fix:text


feat(firestore-bigquery-export): Add validation regex for collection path parameter.

fixes https://github.com/firebase/extensions/issues/410

firestore-bigquery-export was the only extension without a regex for its Firestore collection path param.

Attempts at odd number of path segments:

Codecov Report

Merging #418 into next will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             next     #418   +/-   ##
=======================================
  Coverage   39.77%   39.77%
=======================================
  Files          19       19
  Lines         636      636
  Branches       85       85
=======================================
  Hits          253      253
  Misses        381      381
  Partials        2        2

@laurenzlong is this the same as the bug you identified in #437?

No this is unrelated.


Bug: generate_adversarial_data does not work with colored images

generate_adversarial_data doesnt work with colored images
It says value error that element at index 0 has 3 dimensions while element at index 1 has 4

I fixed the issue by changing the code in adversarial_utils.py in line 30 from
adv_images = np.empty((0, original_images.shape[1], original_images.shape[2]),
                          dtype=int)

to
adv_images = np.empty((0, original_images.shape[1], original_images.shape[2], \
original_images.shape[3]), dtype=int)

Thank you @JafarBadour for opening this issue and suggesting the fix.
Please re-open if there is still a problem.


mv doesn't work

A brief description
The mv command doesn't work as it does on Ubuntu's bash.

Expected results
I expect it to behave the same was as on Ubuntu's bash.

Actual results (with terminal output if applicable)
It produces an error (see below).

Your Windows build number
Microsoft Windows [Version 10.0.14393]

Steps / All commands required to reproduce the error from a brand new installation

maxw@DESKTOP-PCK3HLA:~$ mkdir a_tmp
maxw@DESKTOP-PCK3HLA:~$ mkdir b_tmp
maxw@DESKTOP-PCK3HLA:~$ mv a_tmp/ b_tmp/
mv: cannot move ‘a_tmp/’ to ‘b_tmp/a_tmp’: No such file or directory
maxw@DESKTOP-PCK3HLA:~$

Note that the directory names are completed automatically by pressing the tab key. IE, I type m, v, , a, , , b, , .

Strace of the failing command

maxw@DESKTOP-PCK3HLA:~$ strace mv a_tmp/ b_tmp/
execve("/bin/mv", ["mv", "a_tmp/", "b_tmp/"], [/* 24 vars */]) = 0
brk(0)                                  = 0x1001000
access("/etc/ld.so.nohwcap", F_OK)      = -1 ENOENT (No such file or directory)
mmap(NULL, 8192, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f7f993d0000
access("/etc/ld.so.preload", R_OK)      = -1 ENOENT (No such file or directory)
open("/etc/ld.so.cache", O_RDONLY|O_CLOEXEC) = 3
fstat(3, {st_mode=S_IFREG|0644, st_size=21966, ...}) = 0
mmap(NULL, 21966, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f7f993ca000
close(3)                                = 0
access("/etc/ld.so.nohwcap", F_OK)      = -1 ENOENT (No such file or directory)
open("/lib/x86_64-linux-gnu/libselinux.so.1", O_RDONLY|O_CLOEXEC) = 3
read(3, "\177ELF\2\1\1\0\0\0\0\0\0\0\0\0\3\0>\0\1\0\0\0\0[\0\0\0\0\0\0"..., 832) = 832
fstat(3, {st_mode=S_IFREG|0644, st_size=134296, ...}) = 0
mmap(NULL, 2238192, PROT_READ|PROT_EXEC, MAP_PRIVATE|MAP_DENYWRITE, 3, 0) = 0x7f7f98dd0000
mprotect(0x7f7f98df0000, 2093056, PROT_NONE) = 0
mmap(0x7f7f98fef000, 8192, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_DENYWRITE, 3, 0x1f000) = 0x7f7f98fef000
mmap(0x7f7f98ff1000, 5872, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_ANONYMOUS, -1, 0) = 0x7f7f98ff1000
close(3)                                = 0
access("/etc/ld.so.nohwcap", F_OK)      = -1 ENOENT (No such file or directory)
open("/lib/x86_64-linux-gnu/libacl.so.1", O_RDONLY|O_CLOEXEC) = 3
read(3, "\177ELF\2\1\1\0\0\0\0\0\0\0\0\0\3\0>\0\1\0\0\0`\34\0\0\0\0\0\0"..., 832) = 832
fstat(3, {st_mode=S_IFREG|0644, st_size=31168, ...}) = 0
mmap(NULL, 2126336, PROT_READ|PROT_EXEC, MAP_PRIVATE|MAP_DENYWRITE, 3, 0) = 0x7f7f98bc0000
mprotect(0x7f7f98bc7000, 2093056, PROT_NONE) = 0
mmap(0x7f7f98dc6000, 8192, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_DENYWRITE, 3, 0x6000) = 0x7f7f98dc6000
close(3)                                = 0
access("/etc/ld.so.nohwcap", F_OK)      = -1 ENOENT (No such file or directory)
open("/lib/x86_64-linux-gnu/libattr.so.1", O_RDONLY|O_CLOEXEC) = 3
read(3, "\177ELF\2\1\1\0\0\0\0\0\0\0\0\0\3\0>\0\1\0\0\0\300\20\0\0\0\0\0\0"..., 832) = 832
fstat(3, {st_mode=S_IFREG|0644, st_size=18624, ...}) = 0
mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f7f993c0000
mmap(NULL, 2113760, PROT_READ|PROT_EXEC, MAP_PRIVATE|MAP_DENYWRITE, 3, 0) = 0x7f7f989b0000
mprotect(0x7f7f989b4000, 2093056, PROT_NONE) = 0
mmap(0x7f7f98bb3000, 8192, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_DENYWRITE, 3, 0x3000) = 0x7f7f98bb3000
close(3)                                = 0
access("/etc/ld.so.nohwcap", F_OK)      = -1 ENOENT (No such file or directory)
open("/lib/x86_64-linux-gnu/libc.so.6", O_RDONLY|O_CLOEXEC) = 3
read(3, "\177ELF\2\1\1\0\0\0\0\0\0\0\0\0\3\0>\0\1\0\0\0P \2\0\0\0\0\0"..., 832) = 832
fstat(3, {st_mode=S_IFREG|0755, st_size=1840928, ...}) = 0
mmap(NULL, 3949248, PROT_READ|PROT_EXEC, MAP_PRIVATE|MAP_DENYWRITE, 3, 0) = 0x7f7f985e0000
mprotect(0x7f7f9879a000, 2097152, PROT_NONE) = 0
mmap(0x7f7f9899a000, 24576, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_DENYWRITE, 3, 0x1ba000) = 0x7f7f9899a000
mmap(0x7f7f989a0000, 17088, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_ANONYMOUS, -1, 0) = 0x7f7f989a0000
close(3)                                = 0
access("/etc/ld.so.nohwcap", F_OK)      = -1 ENOENT (No such file or directory)
open("/lib/x86_64-linux-gnu/libpcre.so.3", O_RDONLY|O_CLOEXEC) = 3
read(3, "\177ELF\2\1\1\0\0\0\0\0\0\0\0\0\3\0>\0\1\0\0\0\260\27\0\0\0\0\0\0"..., 832) = 832
fstat(3, {st_mode=S_IFREG|0644, st_size=252032, ...}) = 0
mmap(NULL, 2347200, PROT_READ|PROT_EXEC, MAP_PRIVATE|MAP_DENYWRITE, 3, 0) = 0x7f7f983a0000
mprotect(0x7f7f983dd000, 2093056, PROT_NONE) = 0
mmap(0x7f7f985dc000, 8192, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_DENYWRITE, 3, 0x3c000) = 0x7f7f985dc000
close(3)                                = 0
access("/etc/ld.so.nohwcap", F_OK)      = -1 ENOENT (No such file or directory)
open("/lib/x86_64-linux-gnu/libdl.so.2", O_RDONLY|O_CLOEXEC) = 3
read(3, "\177ELF\2\1\1\0\0\0\0\0\0\0\0\0\3\0>\0\1\0\0\0\320\16\0\0\0\0\0\0"..., 832) = 832
fstat(3, {st_mode=S_IFREG|0644, st_size=14664, ...}) = 0
mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f7f993b0000
mmap(NULL, 2109744, PROT_READ|PROT_EXEC, MAP_PRIVATE|MAP_DENYWRITE, 3, 0) = 0x7f7f98190000
mprotect(0x7f7f98193000, 2093056, PROT_NONE) = 0
mmap(0x7f7f98392000, 8192, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_FIXED|MAP_DENYWRITE, 3, 0x2000) = 0x7f7f98392000
close(3)                                = 0
mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f7f993a0000
mmap(NULL, 8192, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f7f99390000
arch_prctl(ARCH_SET_FS, 0x7f7f99390840) = 0
mprotect(0x7f7f9899a000, 16384, PROT_READ) = 0
mprotect(0x7f7f98392000, 4096, PROT_READ) = 0
mprotect(0x7f7f985dc000, 4096, PROT_READ) = 0
mprotect(0x7f7f98bb3000, 4096, PROT_READ) = 0
mprotect(0x7f7f98dc6000, 4096, PROT_READ) = 0
mprotect(0x7f7f98fef000, 4096, PROT_READ) = 0
mprotect(0x61c000, 4096, PROT_READ)     = 0
mprotect(0x7f7f99222000, 4096, PROT_READ) = 0
munmap(0x7f7f993ca000, 21966)           = 0
statfs("/sys/fs/selinux", 0x7fffe69cba90) = -1 ENOENT (No such file or directory)
statfs("/selinux", 0x7fffe69cba90)      = -1 ENOENT (No such file or directory)
brk(0)                                  = 0x1001000
brk(0x1022000)                          = 0x1022000
open("/proc/filesystems", O_RDONLY)     = 3
fstat(3, {st_mode=S_IFREG|0444, st_size=0, ...}) = 0
mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f7f99380000
read(3, "nodev      sysfs\nnodev      root"..., 1024) = 367
read(3, "", 1024)                       = 0
close(3)                                = 0
munmap(0x7f7f99380000, 4096)            = 0
open("/usr/lib/locale/locale-archive", O_RDONLY|O_CLOEXEC) = 3
fstat(3, {st_mode=S_IFREG|0644, st_size=2919792, ...}) = 0
mmap(NULL, 2919792, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f7f97ec7000
close(3)                                = 0
geteuid()                               = 1000
ioctl(0, SNDCTL_TMR_TIMEBASE or SNDRV_TIMER_IOCTL_NEXT_DEVICE or TCGETS, {B38400 opost isig icanon echo ...}) = 0
stat("b_tmp/", {st_mode=S_IFDIR|0777, st_size=0, ...}) = 0
lstat("a_tmp/", {st_mode=S_IFDIR|0777, st_size=0, ...}) = 0
lstat("b_tmp/a_tmp", 0x7fffe69cb730)    = -1 ENOENT (No such file or directory)
rename("a_tmp/", "b_tmp/a_tmp")         = -1 ENOENT (No such file or directory)
open("/usr/share/locale/locale.alias", O_RDONLY|O_CLOEXEC) = 3
fstat(3, {st_mode=S_IFREG|0644, st_size=2570, ...}) = 0
mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f7f99380000
read(3, "# Locale name alias data base.\n#"..., 512) = 512
read(3, "PURPOSE.  See the\n# GNU General "..., 512) = 512
read(3, " entries are case independent.\n\n"..., 512) = 512
read(3, "859-1\nfinnish         fi_FI.ISO-"..., 512) = 512
read(3, "R\nko_KR\t\tko_KR.eucKR\nlithuanian "..., 512) = 512
read(3, "SO-8859-9\n", 512)             = 10
read(3, "", 512)                        = 0
close(3)                                = 0
munmap(0x7f7f99380000, 4096)            = 0
open("/usr/share/locale/en_GB.UTF-8/LC_MESSAGES/coreutils.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale/en_GB.utf8/LC_MESSAGES/coreutils.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale/en_GB/LC_MESSAGES/coreutils.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale/en.UTF-8/LC_MESSAGES/coreutils.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale/en.utf8/LC_MESSAGES/coreutils.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale/en/LC_MESSAGES/coreutils.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale-langpack/en_GB.UTF-8/LC_MESSAGES/coreutils.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale-langpack/en_GB.utf8/LC_MESSAGES/coreutils.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale-langpack/en_GB/LC_MESSAGES/coreutils.mo", O_RDONLY) = 3
fstat(3, {st_mode=S_IFREG|0644, st_size=15001, ...}) = 0
mmap(NULL, 15001, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f7f993d2000
close(3)                                = 0
open("/usr/share/locale-langpack/en.UTF-8/LC_MESSAGES/coreutils.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale-langpack/en.utf8/LC_MESSAGES/coreutils.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale-langpack/en/LC_MESSAGES/coreutils.mo", O_RDONLY) = 3
fstat(3, {st_mode=S_IFREG|0644, st_size=619, ...}) = 0
mmap(NULL, 619, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f7f993cf000
close(3)                                = 0
open("/usr/lib/charset.alias", O_RDONLY|O_NOFOLLOW) = -1 ENOENT (No such file or directory)
write(2, "mv: ", 4mv: )                     = 4
write(2, "cannot move \342\200\230a_tmp/\342\200\231 to \342\200\230b"..., 45cannot move ‘a_tmp/’ to ‘b_tmp/a_tmp’) = 45
open("/usr/share/locale/en_GB.UTF-8/LC_MESSAGES/libc.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale/en_GB.utf8/LC_MESSAGES/libc.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale/en_GB/LC_MESSAGES/libc.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale/en.UTF-8/LC_MESSAGES/libc.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale/en.utf8/LC_MESSAGES/libc.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale/en/LC_MESSAGES/libc.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale-langpack/en_GB.UTF-8/LC_MESSAGES/libc.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale-langpack/en_GB.utf8/LC_MESSAGES/libc.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale-langpack/en_GB/LC_MESSAGES/libc.mo", O_RDONLY) = 3
fstat(3, {st_mode=S_IFREG|0644, st_size=3595, ...}) = 0
mmap(NULL, 3595, PROT_READ, MAP_PRIVATE, 3, 0) = 0x7f7f993ce000
close(3)                                = 0
open("/usr/share/locale-langpack/en.UTF-8/LC_MESSAGES/libc.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale-langpack/en.utf8/LC_MESSAGES/libc.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
open("/usr/share/locale-langpack/en/LC_MESSAGES/libc.mo", O_RDONLY) = -1 ENOENT (No such file or directory)
write(2, ": No such file or directory", 27: No such file or directory) = 27
write(2, "\n", 1
)                       = 1
lseek(0, 0, SEEK_CUR)                   = -1 ESPIPE (Illegal seek)
close(0)                                = 0
close(1)                                = 0
close(2)                                = 0
exit_group(1)                           = ?
+++ exited with 1 +++
maxw@DESKTOP-PCK3HLA:~$

Required packages and commands to install

See our contributing instructions for assistance.

Thanks for reporting the issue!  Based on the strace this looks like a duplicate of #1008 where rename fails with a trailing slash.  This is fixed for insider builds if you want to give those a try.
rename("a_tmp/", "b_tmp/a_tmp")         = -1 ENOENT (No such file or directory)

sounds likely. I'll just wait for it. Thanks.

Thanks for checking, I'll close this out for now.


Supports version 2 of docker-compose

Supports version 2 of docker-compose.
version: '2'
services:
account:
image: ${project.rootProject.stackwork.imageId}
ports:
- "8100"
networks:
default:
external:
name: xxxyyy
this template cause error to start docker-compose
docker-compose -f .../stackwork-plugin/docker-compose.yml -p 3BH90V164 up -d version services networks

The most basic version 2 support is now implemented.
We still need:

multiple networks support
not to depend on indentation in case of extending docker-compose.yml.template files
prove that named volumes work (not tested currently)
verification that the current version works with a remote docker host (Mac is easy to test this on)

This list can probably be turned into separate issues.
To do this all, the docker compose template files and base files must be parsed and joined together logically. This is not difficult, but it is some work. The base docker info must be loaded into the Gradle object to be available runtime for all 'extending' modules.


Need More Details of How to use?

@sionzee  I try to integrate this in sveltekit for inline svg.
But for option part didn't get much info?
All svg brokes after using it.
Can you provide into the details for the community?

@code-masala sorry for late response, could you provide more into on all broken svgs?
Actually you just have to install the package and put it into plugin field in rollup.config.js
i.e.:
import inlineSvg from 'rollup-plugin-inline-svg';
export default {
  input: "src/input.js",
  output: "dist/output.js",
  plugins: [inlineSvg()]
}

And then you can include your svg anywhere in the code processed by rollup.
i.e.: whatever.mjs
import UserSvg from "../icons/user.svg"

console.log(UserSvg); // will print content of the svg
document.getElementById("user-icon").innerHTML = UserSvg;


AddDataProtection().PersisKey

Hi,
I use MongoDbCache, it replace the StackExchange.Redis library.
The problem is that we have a "Services.AddDataProtection().PersistKeysToStackExchangeRedis" but there is no equivalent for MongoDB.
Can you provides it ?
PS: here is the source code https://github.com/aspnet/DataProtection/blob/master/src/Microsoft.AspNetCore.DataProtection.StackExchangeRedis/RedisDataProtectionBuilderExtensions.cs

I'm closing this issue as it is not related to this project


FEATURE: Ability to get parsed list of virtual desktops via dispatcher

Good day to you!
is there  any ability to get parsed list of virtual desktops  via dispatcher?
usecase: jumping between desktops via fuzzy selector
hyprctl vdesks -j | rofi -dmenu

thank you!

Hello again!
So, I went looking at Hyprland's code again and, as far as I can tell, there's no way of printing via hyprctl for a plugin. This issue should be raised in Hyprland's repo, and see what vaxry thinks about this :)

how to describe issue on Hyprland's repo?
"Ability print  plugin's runtime info"?

No, not really. This is more related to the ability of printing anything at all from an hyprctl dispatcher: this is impossible for plugins and hyprland code itself, as far as I can tell.
So, best thing would probably be something like "ability for plugins to print from hyprctl dispatchers" or "allow plugins to print to stdout from hyprctl dispatchers"

@levnikmyskin
vaxry had implemented some changes.
is it enough for this feature?

Hi,
Yes that is everything I need. I'll update you here on this ticket when I implement the feature. Maybe this weekend :)

Hey,
just to give you an update: I didn't forget about this issue. It's just that Hyprland's implementation in v 0.35 was incomplete for this to work.
I already pushed (and vaxry merged) the changes upstream, so we will have this feature in 0.36 :)


use kiali 1.4 so it can pick up patches

Based on recommendation from kiali team and what we have done in 1.2.
[ ] Configuration Infrastructure
[ ] Docs
[x ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Yes, using just vX.Y (as opposed to vX.Y.Z) is how we've been doing it since Istio 1.1 and 1.2:
https://github.com/istio/istio/blob/release-1.1/install/kubernetes/helm/istio/charts/kiali/values.yaml#L7
https://github.com/istio/istio/blob/release-1.2/install/kubernetes/helm/istio/charts/kiali/values.yaml#L7

@linsun: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pilot-multicluster-e2e_istio
751aa41a5d2fb0488dddf7a8dbd74097888f8b08
link
/test pilot-multicluster-e2e_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

i think using a pinned version is a better idea but given that 1) we clearly aren't good at keeping it update to date and 2) we had it this way before, this seems fine. Thanks!

I think using a pinned version may leads the version we are testing in the release is different from the version user is using. That may have bug that we cannot cover in our release. Anyway, I think we should switch to kiali operator to leverage the kiali's experts for that.


Added action to assign to issue

Added an assign action. Because there was no assignment action in the JIRA pack.

Thanks @sinnershiki !


No resource identifier found for attribute 'windowLayoutInDisplayCutoutMode' in package 'android'

Information

Apktool Version (apktool -version) - 2.4.0
Operating System (Mac, Linux, Windows) - Windows
APK From? (Playstore, ROM, Other) - Other

Steps to Reproduce

Decompile apk
Recompile APK

APK
APK Link: https://www78.zippyshare.com/v/y11Q3m2c/file.html
Questions to ask before submission

Have you tried apktool d, apktool b without changing anything? Yes
If you are trying to install a modified apk, did you resign it? No
Are you using the latest apktool version? Yes

升级你的android.jar包试试

File has expired and does not exist anymore on this server

That attribute was added in API 27 - https://developer.android.com/reference/android/R.attr#windowLayoutInDisplayCutoutMode so you'll need to update framework files.


Add FlashAttention3 to CP implementations

Description
Enable FA3 in all CP implementation. Work in BF16 and FP16 now.
Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Infra/Build change
[ ] Code refractor

Checklist:

[x] I have read and followed the contributing guidelines
[x] The functionality is complete
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/te-ci pytorch


python3 support

test and update codechecker if needed

the default interpreter for Arch linux is already python3
next Ubuntu release (16.04) will also change the default interpreter to python3 https://wiki.ubuntu.com/Python/FoundationsXPythonVersions

thrift 0.10 will be released soon, and it is the first version to support python 3.

Thrift 0.10 is out and available in the pypi for python 3. Source: https://pypi.python.org/pypi/thrift/0.10.0
The last blocker to support python 3 should be gone.

It's not gone.
Ubuntu 17.04 Zesty and 17.10 Artful still packages thrift-0.9.x and python2 as python in their system.
Thrift could be compiled from source, and even the generated files of 0.9.1 work with 0.10.0 installed from PIP work in Python2 mode.
However, updating Python and Thrift would help a lot in a pressing API issue about encoding large UTF-8 codepoints, such as #726.

Thrift 0.11 is already released but it looks like the community is not very quick with packaging it.
There is a 0.10 in debian experimental repositories: https://packages.debian.org/experimental/thrift-compiler
And it is not available in any of the ubuntu versions.

Debian and Ubuntu isn't about getting the newest versions of everything, philosophically.

Reminder that python2 will be fully and completely EOL in 1 month 12 days.

yes, we know about it, thanks for the reminder

Followup reminder:
DEPRECATION: Python 2.7 reached the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 is no longer maintained. A future version of pip will drop support for Python 2.7. More details about Python 2 support in pip, can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support

Is there a tracking issue on what's missing for python3?

Hi,
There is no tracking issue, just a project with some notes I've collected.
I'm working on the port to py3 it is not finished yet (The tests run on my machine successfully).
I still need to cleanup the commits and we are discussing in the team how to handle the generated thrift stubs (py3 stubs need at least thrift 0.11.0, but on Ubuntu LTS  that version is not available only 0.9.1).
Probably they will be published to pypi and npm but we still try to figure out what would be the easiest user/developer workflow in case of an api change.


Update fluid version to 0.8.6

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

fixes #XXXX
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

/lgtm
/approved

/lgtm
/approve


Confocal microscopy evidence [GO synapse term request]

is_a microscopy evidence [ECO_0001098]

Added the term confocal microscopy evidence (ECO:0005587).
is_a microscopy evidence [ECO_0001098]
Closing the ticket now.


Documentation on just.systems not updated?

Recent updates to README.md are not being reflected on https://just.systems/man/en/ .  Looks like the last README.md change to make it there was https://github.com/casey/just/commit/eb8227744eaf574a36a90d89e429ee6a488978b1 , so if this is caused by a change in this repo, the regression range maybe https://github.com/casey/just/compare/eb8227744eaf574a36a90d89e429ee6a488978b1...daf4520a2b8259ea26db15c1c55972170dfb9dc2 ?

Good catch! I fixed it, but I still have no idea what went wrong. It does look like it happened around the time of those changes to the workflow files, although I fixed it, I think, by just turning on and off GitHub pages, and didn't have to change any workflow files. Hopefully it stays fixed!


Patch rollout-batch-id even without rollout-id in workload

Ⅰ. Describe what this PR does

patch rollout-batch-id even without rollout-id in workload

Ⅱ. Does this pull request fix one issue?
https://github.com/openkruise/rollouts/issues/244
Ⅲ. Special notes for reviews

simply use revision id may not works for rollback cases, rollback also support mutli-batch rollback.

simply use revision id may not works for rollback cases, rollback also support mutli-batch rollback.

can you give me a specific scenario so I can reproduce it? maybe can use the SafeEncodeString func to encode the revision id.


Fixes deadlocks on AppVeyor

Fixes deadlocks in xunit tests based on Chill when those run on an AppVeyor build agent.

:+1:
It's a fix, so if you want to release this, it should be 2.2.1.


Enums to MagicConstants

Because #perfmatters - replace ConnectivityState, Strength and Type with integer constansts.

Done in v2.0.0-beta1.


Move logic to before determining user agent

This moves the logic of analytics.exclude before trying to create the user agent.

@grantholle back with another contribution, thanks mate!
Maybe I'm missing it, but what's the purpose of the change in order? 😄

My thinking is to try and cut the request off as soon as possible. So if I'm excluding some routes, check those before parsing and configuring the user agent to use later.
Probably a very negligible performance gain... but seemed logical in my head.

Makes sense, works for me, thanks dude.


install package pods error.

all owdev-install-packages pods error in the init openwhisk cluster, The specific performance is as follows:

install the packages locally first into docker containers .


Run EKS-D on microk8s on MacOS

What would you like to be added:
I'd like to run EKS-D on microk8s on MacOS.
Why is this needed:
This page: https://aws.amazon.com/blogs/opensource/introducing-amazon-eks-distro/, mentions being able to use microk8s to deploy an EKS-D compatible cluster.
Clicking through that link brings me here: https://snapcraft.io/eks
However, there is no 'snap' support on MacOS. On MacOS microk8s is installed via 'HomeBrew'.
I couldn't find equivalent instructions for microk8s on MacOs. If all I have to do is pass a channel name on microk8s launch, I didn't find it documented.

Thanks, I'll do some research on this.

As you probably know, you can install microk8s https://ubuntu.com/tutorials/install-microk8s-on-mac-os#1-overview on mac using brew. I've only run it on Ubuntu. I don't think Snap is going to be happening, but that is something to take up with Canonical.

As you probably know, you can install microk8s https://ubuntu.com/tutorials/install-microk8s-on-mac-os#1-overview on mac using brew. I've only run it on Ubuntu. I don't think Snap is going to be happening, but that is something to take up with Canonical.

This issue isn't about getting Snap support on MacOS. That may never happen.
MicroK8S has documentation here with instructions for switching to a different k8s channel.
While the snap instructions to select 1.20 are:
snap install microk8s --classic --channel=1.20/stable
the instructions for macOS would be:
microk8s install --channel=1.20/stable, which to be honest, is also poorly documented.
However, neither Ubuntu nor AWS has documented which channel (if there is one) to choose to deploy EKS-D via microK8S.
AWS has clearly worked with Ubuntu to get EKS-D to work with microK8S, so the community would naturally expect AWS to provide an answer. I'm sure many were/are excited at the prospect of being able to run an EKS-like cluster on their own machines.

It may be useful for reference to look at how Canonical  did their integration which is in the Canonical EKS snap repo
I've only used microk8s on Ubuntu, but it would be nice to have the option documented.

@ktsakalozos do you have any advice here as to which channel should be used to get the eks-d based distro?

@ktsakalozos do you have any advice here as to which channel should be used to get the eks-d based distro?

Sorry for the late response.
To get any snap on MacOS you need a Linux VM. After you have a VM you can snap install eks --classic --channel=1.18/edge.

Sorry for the late response.
To get any snap on MacOS you need a Linux VM. After you have a VM you can snap install eks --classic --channel=1.18/edge.


[YouTube] Downloading premieres with --live-from-start raises an error and leaves behind 'partial' files

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I'm running yt-dlp version 2023.10.13 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
United States
Provide a description that is worded well enough to be understood
Downloading a premiering YouTube video (live with a predetermined duration) with --live-from-start has yt-dlp retry at the end of the premiere, which subsequently raises a KeyError on manifest_url and causes the application (or script, if invoked without wrapping a try-except) to terminate, leaving behind .part and .ytdl files.
Downloading the premiering video without --live-from-start has yt-dlp exit properly at the end of the premiere.
I haven't had the opportunity to test if any of the 'retries' options would work in this case, nor am I familiar with how a zero retry behaves in this situation. Results of experimenting with those options were unsuccessful, per follow-up comment
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', '--live-from-start', 'https://www.youtube.com/watch?v=redacted_id']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b634ba742] (pip)
[debug] Python 3.9.2 (CPython AMD64 64bit) - Windows-10-10.0.19041-SP0 (OpenSSL 1.1.1i  8 Dec 2020)
[debug] exe versions: ffmpeg 5.0.1-full_build-www.gyan.dev (setts), ffprobe 5.0.1-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.18.0, brotli-1.0.9, certifi-2023.05.07, mutagen-1.45.1, sqlite3-3.34.0, websockets-10.0
[debug] Proxy map: {}
[debug] Loaded 1890 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>yt-dlp is up to date<EMAIL_ADDRESS>[youtube] Extracting URL: https://www.youtube.com/watch?v=redacted_id
[youtube] redacted_id: Downloading webpage
[youtube] redacted_id: Downloading ios player API JSON
[youtube] redacted_id: Downloading android player API JSON
[youtube] redacted_id: Downloading MPD manifest
[youtube] redacted_id: Downloading MPD manifest
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, size, br, asr, vext, aext, hasaud, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info] redacted_id: Downloading 1 format(s): 247+140
[debug] Invoking dashsegments downloader on "https://manifest.googlevideo.com/api/manifest/dash/expire/<redacted>"
[dashsegments] Total fragments: unknown (live)
[download] Destination: redacted_id [redacted_id].f247.mkv
[debug] File locking is not supported. Proceeding without locking
[dashsegments] Total fragments: unknown (live)
[download] Destination: redacted_id [redacted_id].f140.mkv
[debug] [youtube] [redacted_id] Generating fragments for format 247
[debug] [youtube] [redacted_id] Generating fragments for format 140
[download]    8.39MiB at      0.00B/s (00:01:06) (frag 38/38)
[download]    2.89MiB at      0.00B/s (00:01:06) (frag 38/38)[youtube] redacted_id: Downloading webpage
[youtube] redacted_id: Downloading ios player API JSON
[youtube] redacted_id: Downloading android player API JSON
[debug] Loading youtube-nsig.9d15588c from cache
[debug] [youtube] Decrypted nsig ClKGCwrlWJFQk6 => Z3NhWuOsspbg3A
[youtube] redacted_id: Downloading m3u8 information
WARNING: [youtube] redacted_id: Video is no longer live. Retrying (1/3)...
WARNING: [youtube] redacted_id: Video is no longer live. Retrying (2/3)...
WARNING: [youtube] redacted_id: Video is no longer live. Retrying (3/3)...
[download] 100% of    8.39MiB in 00:01:15 at 114.43KiB/sERROR: 'manifest_url'
Traceback (most recent call last):
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\YoutubeDL.py", line 1567, in wrapper
    return func(self, *args, **kwargs)
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\YoutubeDL.py", line 1723, in __extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\YoutubeDL.py", line 1782, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\YoutubeDL.py", line 2921, in process_video_result
    self.process_info(new_info)
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\YoutubeDL.py", line 3334, in process_info
    success, real_download = self.dl(temp_filename, info_dict)
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\YoutubeDL.py", line 3108, in dl
    return fd.download(name, new_info, subtitle)
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\downloader\common.py", line 455, in download
    ret = self.real_download(filename, info_dict)
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\downloader\dash.py", line 60, in real_download
    return self.download_and_append_fragments_multiple(*args, is_fatal=lambda idx: idx == 0)
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\downloader\fragment.py", line 421, in download_and_append_fragments_multiple
    result = result and future_result(job)
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\downloader\fragment.py", line 397, in future_result
    return future.result(0.1)
  File "<redacted>\python\3.9.2\lib\concurrent\futures\_base.py", line 433, in result
    return self.__get_result()
  File "<redacted>\python\3.9.2\lib\concurrent\futures\_base.py", line 389, in __get_result
    raise self._exception
  File "<redacted>\python\3.9.2\lib\concurrent\futures\thread.py", line 52, in run
    result = self.fn(*self.args, **self.kwargs)
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\downloader\fragment.py", line 385, in thread_func
    return self.download_and_append_fragments(
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\downloader\fragment.py", line 510, in download_and_append_fragments
    for fragment in fragments:
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\downloader\fragment.py", line 407, in interrupt_trigger_iter
    for f in fg:
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\downloader\dash.py", line 71, in _get_fragments
    for i, fragment in enumerate(fragments):
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\extractor\youtube.py", line 2893, in _live_dash_fragments
    should_continue, last_seq = _extract_sequence_from_mpd(True, no_fragment_score > 15)
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\extractor\youtube.py", line 2849, in _extract_sequence_from_mpd
    mpd_url, stream_number, is_live = (mpd_feed(format_id, 5 if expire_fast else 18000)
  File "<redacted>\python\3.9.2\lib\site-packages\yt_dlp\extractor\youtube.py", line 2811, in mpd_feed
    return f['manifest_url'], f['manifest_stream_number'], is_live
KeyError: 'manifest_url'

Both --retries 0 and --fragment-retries 0 also redownload the webpage at the end of the premiere, so that doesn't do any good.

I am facing this same issue when downloading a livestream, and would really appreciate any here here. Thank you!

@mekicha If you're open to using other tools, https://github.com/Kethsar/ytarchive currently works great for both premieres and live streams.

@nosoop oh great, thanks a lot. Let me check it out.