LogFactory - Ensure to flush and close on shutdown

Fixes bug introduced with #1906 and reported in #1998 (Abused the medium-trust-AppDomain-unit-test to verify flush is done properly on AppDomain-unload)

Codecov Report

Merging #2010 into master will increase coverage by <1%.
The diff coverage is 92%.

@@           Coverage Diff           @@
##           master   #2010    +/-   ##
=======================================
+ Coverage      82%     82%   +<1%
=======================================
  Files         286     286
  Lines       19658   19659     +1
  Branches     2296    2296
=======================================
+ Hits        16042   16069    +27
+ Misses       3047    3034    -13
+ Partials      569     556    -13

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 586a737...41f42c6. Read the comment docs.

Excellent work!

Probably fixes #2030 and #2031


Can I use this with Python in Atom.io

If yes, then please show me how? I got ruby installed and also got seeing is believing gem installed. Now what should I do. I am not a pro in programming still wan to learn. Please help. :-)

Unfortunately, it won't work with Python. It works by parsing the code and rewriting it with additional code to capture the results of each line, but not affect behaviour. The rewritten code is then run in a separate process where the environment is setup to allow for capturing things like printing. The line-result code also writes the events it sees to a file descriptor, which is sent back to the main process.
So, to make it work with Python, I'd probably have to change certain assumptions within the structure of Seeing Is Believing, as well as figure out how to rewrite Python, and enough about how Python works to be able to fake out the environment sufficiently. Unfortunately, that's very nontrivial, and I've used Python so infrequently, that there's very little incentive to work on such a thing (if I were going to do something like that, I'd probably try to do it for JavaScript).

@JoshCheek Thanks


Workflow failure - processing Summary ServicePrincipalsAppRoleAssignments

Here is the error message I see in the github actions logs:
processing Summary ServicePrincipalsAppRoleAssignments
AzADServicePrincipalInsights.ps1: /home/runner/work/_temp/ab3aa0e7-ff1c-4b58-8b6c-194328e0298d.ps1:3
Line |
   3 |  . .\$($env:ScriptDir)\$($env:ScriptFile) -ScriptPath ${env:ScriptDir} …
     |  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Value cannot be null. (Parameter 'key')
Error: Error: The process '/usr/bin/pwsh' failed with exit code 1

This error started happening with v1_20240209_1, I believe...

I have the very same issue since updating to v1_20240209_1
processing Summary ServicePrincipalsAppRoleAssignments ##[error]Value cannot be null. (Parameter 'key') ##[error]PowerShell exited with code '1'. Added TLS 1.2 in session. Disconnect-AzAccount -Scope Process -ErrorAction Stop Clear-AzContext -Scope Process -ErrorAction Stop Finishing: Run AzADServicePrincipalInsights v1

@cjtous1 @DarxonHU thanks for reporting. Can you try the branch fix_issue27


Align layer/legend titles, content, ordering

Feedback:

It might be more clear if the headers on the legends matched the name of the layers in the clickable list (right now the info ro the "air quality forecast" is labeled "projected air quality index" and "air quality sensor network" is labeled "air quality sensors").
Also, when both air quality layers are clicked on, the information about the sensor network is below the info about the forecasts, but that's not the order they're listed in the clickable layers.  Can the sensor info go above the projected AQI when both are clicked on?
Would it be possible to make the PurpleAir legend match the Projected AQI legend so that it has the good/moderate/unhealthy text?
And maybe also add the sentence from the projected air quality information "this layer shows AQI" to the information that goes with the PurpleAir legend?  It would be good for each layer's information to stand alone in case people only have one layer clicked on.
Can we also add a sentence to send people here for more information on PM2.5?

[ ] Legends match titles
[ ] Fix order of legends to match layer order
[ ] Use same legend table for PurpleAir and Air Quality Forecast layers
[ ] Mirror language between PurpleAir / Air Quality Forecast layers
[ ] Add link to EPA PM site


Error running Recalcuate Rollup from Parent

@jamessimone It looks like the Recalculate Rollup is working as expected now. It only generated 1 batch. However, I think as a side effect to the DateTime comparison changes, it's erroring out:

The field Grant_Date_From_Grant__c is a Date formula field (not a DateTime). I'm not sure if I have the clause formatted incorrectly (no quotes, yyyy-mm-dd) or if it's reading the field type incorrectly:

@jkenzer it shouldn't matter if it's a formula field; I will fix this issue, but for now you should be good to go by changing the date to this 2021-04-01 00:00:00

Updating it to 2021-04-01 00:00:00 fixed the issue. Do you want me to leave this open to allow for a date without the time designation?

@jkenzer yes please! I have just fixed the underlying issue and will look to get this updated version out in the next day or so.


Add reference counters to meshes, shaders (and probably something else)

Avoid leaking memory when continuously creating/destroying the app.

Implemented


build(client): Override transitive deps on ip

Addresses CVE-2023-42282.

Can we use the new flub modify lockfile so we don't need to keep the overrides?

Ooooh, I'll try it!

Can we use the new flub modify lockfile so we don't need to keep the overrides?

Ooooh, I'll try it!

@alexvy86 Worked well! This PR is now updated.


Batch anonymize CSV files via HTTP endpoint

Hi @omri374,
I'm looking for ways to handle PII before it enters our dataplatform hosted in Azure. If this is not the proper way to ask such questions let me know!
Use case
We're trying de identify and anonymize PII that's coming from a local database. This flow should be mainly hosted in an Azure Data Factory self hosted integration runtime.

Flow:

Get data from source and write to local folder /RAW/file1.csv
Get file paths of files to process
Loop over all files that need to be processed

Send csv to local http docker endpoint (docker container)
(PRESIDIO) Anonymize columns X, Y, Z completely and look for PII in column A
Write anonymized file to /STAGING/file1.csv

Copy local folder /STAGING/file1.csv to Azure Data Lake storage

In most cases we should hash the complete column contents, in other cases it's part of a long text string.
SOURCE

Id
Name
Description
Additional field

1
Jane
This line is about Jane
Nothing secret here

2
Josh
This line is about Jane
Nothing to do here

TARGET RESULT

Id
Name
Description
Additional field

HASH41425
HASH65243
This line is about NAME
Nothing secret here

HASH95862
HASH19765
This line is about NAME
Nothing to do here

I went through the documentation already and I'm sure if the current state of Presidio can support this use case entirely. I've came across this discussio.
Things I'm uncertain about:

Can Presidio handle CSV files already?
Is it possible to tell Presidio to hash column X entirely?
Is the API able to batch anonymize? Based on the API reference it seems it's not.

Hi @dsfrederic,
Analyzing and anonymizing structured/semi-structured data is not yet supported in Presidio. In the meantime, we are collecting some feedback from users and looking for community contributions in this space.
There is no integrated support for anonymizing an entire column, but a naive approach would be to call the anonymizer on each cell.
This is definitely something on our roadmap, but until we get to it, we would be happy to help with specific questions around implementation.

Hi @dsfrederic we have support for batch analysis on structured data available, see sample here https://microsoft.github.io/presidio/samples/python/batch_processing/. Input data is Panda dataframe

batch analyzer is now available in presidio https://microsoft.github.io/presidio/samples/python/batch_processing/


Scrollable Tooltips is incompatible on 1.19.3

When attempting to start up 1.1.2 of scrollable tooltips, The game will just crash.

Closing issue because it went stale.


ConcurrentLinkedQueue relies on setting next to this, but that leaks memory

When a ConcurrentLinkedQueue dequeues a node, the Java implementation relies on setting its next field to itself as a way to mark the node as removed. It also uses that technique to prevent unbounded retention (in other words, only live nodes should be GC-reachable).
The technique leaks memory when it's translated to Objective-C. I wonder if we can add a marker object to the translated version, similar to what we did for #605.
The code sets next to this with a compare-and-swap call and compares this to next in a lot of places in the non-obvious form of if (p == q) (example), and so if we take this approach we'll need to carefully examine the changes.
There is a potential problem of reading stale values in a non-GC environment – the "ABA problem" discussed in the original paper as well as in the code comment. For example, when we finally get to compare-and-swap tail->next with the new node we're enqueuing, we want to make sure that tail's memory content is still what we saw before (because the node at tail may actually have been dequeued in another thread at that point, and the address at tail may mean something else in a non-GC environment). The paper pairs pointers with counters to mitigate the problem. I wonder if this has to be introduced back into the Objective-C translation.
I haven't looked into it too much, but LinkedTransferQueue appears to have the same issue.
(Edit: Example for the ABA problem.)

I'll be using the marker object solution as you suggest. The ABA problem should not affect us because our translation of volatile variables ensure that they live for at least the duration of the autorelease pool where they are read.

Fixed ConcurrentLinkedQueue and fixed LinkedTransferQueue.

Thanks for the fixes!

Sure. Let me know if you find more.


Consistency: haskell_prebuilt_library vs. prebuild_*_library

Just a nitpick, but I guess the Haskell rules are still young enough to be able to change the name to prebuilt_haskell_library to be consistent with the other ones, before it is too late?

Alas, all these names are compatible with Buck1, so we have a lot of legacy - the name is years old. My guess is once we eliminate Buck1 usage internally we'll probably make a lot more of these things consistent, but until then, it is probably more hassle than it is worth :(
We do have the idea of trying to gradually migrate internal things to sensible outcomes via macros (so we do if buck1 then use the legacy name, otherwise use the Buck2 name), but it's a fair amount of work internally, so won't happen quickly.

Oh, yes, of course, I haven't thought of Buck 1.


Fix build and test issues on PowerShell v4

This change is

Created this by mistake. Hence closing it.


Adds debug and warn messages for migrations

Adds a debug message for better identification of migration issues.
Adds a warn message when a table exceeds the migration max.

This seems related to #4469

@dlmarion Any other changes needed?


MemError with taxii-push command

I had been successfully pushing STIX files to my OpenTAXII until a few days ago since it started failing with the following MemError on the gunicorn service run:
{"timestamp": "2017-04-12T07:27:29.439255Z", "logger": "opentaxii.utils", "api": "opentaxii.persistence.sqldb.SQLDatabaseAPI", "event": "api.initialized", "level": "info"}
{"timestamp": "2017-04-12T07:27:29.535209Z", "logger": "opentaxii.utils", "api": "opentaxii.auth.sqldb.SQLDatabaseAPI", "event": "api.initialized", "level": "info"}
{"timestamp": "2017-04-12T07:27:29.536024Z", "logger": "opentaxii.server", "event": "opentaxii.server_configured", "level": "info"}
("'b3BlbnRheGlpOmZvcnRpbmV0'", 24)
[2017-04-12 09:27:51,420] ERROR in app: Exception on /services/inbox-a [POST]
Traceback (most recent call last):
  File "/home/fortinet/venv/local/lib/python2.7/site-packages/flask/app.py", line 1982, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/fortinet/venv/local/lib/python2.7/site-packages/flask/app.py", line 1614, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/fortinet/venv/local/lib/python2.7/site-packages/flask/app.py", line 1517, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/fortinet/venv/local/lib/python2.7/site-packages/flask/app.py", line 1612, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/fortinet/venv/local/lib/python2.7/site-packages/flask/app.py", line 1598, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/home/fortinet/venv/local/lib/python2.7/site-packages/opentaxii/middleware.py", line 76, in wrapper
    return _process_with_service(service)
  File "/home/fortinet/venv/local/lib/python2.7/site-packages/opentaxii/middleware.py", line 142, in _process_with_service
    get_content_type(request.headers), request.data)
  File "/home/fortinet/venv/local/lib/python2.7/site-packages/opentaxii/taxii/utils.py", line 46, in parse_message
    result = validator_parser.validator.validate_string(body)
  File "/home/fortinet/venv/local/lib/python2.7/site-packages/libtaxii/validation.py", line 170, in validate_string
    etree_xml = parse(xml_string)
  File "/home/fortinet/venv/local/lib/python2.7/site-packages/libtaxii/common.py", line 36, in parse
    e = etree.parse(s, get_xml_parser()).getroot()
  File "src/lxml/lxml.etree.pyx", line 3427, in lxml.etree.parse (src/lxml/lxml.etree.c:81117)
  File "src/lxml/parser.pxi", line 1811, in lxml.etree._parseDocument (src/lxml/lxml.etree.c:117848)
  File "src/lxml/parser.pxi", line 1837, in lxml.etree._parseDocumentFromURL (src/lxml/lxml.etree.c:118195)
  File "src/lxml/parser.pxi", line 1741, in lxml.etree._parseDocFromFile (src/lxml/lxml.etree.c:117107)
  File "src/lxml/parser.pxi", line 1138, in lxml.etree._BaseParser._parseDocFromFile (src/lxml/lxml.etree.c:111653)
  File "src/lxml/parser.pxi", line 595, in lxml.etree._ParserContext._handleParseResultDoc (src/lxml/lxml.etree.c:105109)
  File "src/lxml/parser.pxi", line 706, in lxml.etree._handleParseResult (src/lxml/lxml.etree.c:106817)
  File "src/lxml/parser.pxi", line 629, in lxml.etree._raiseParseError (src/lxml/lxml.etree.c:105589)
MemoryError

Any idea hwo to fix this?
Thanks in advance,

@maxymax68 recently I've noticed these kind of errors in my setup as well. So far, I was not been able to reliably reproduce this.
Does this happen when you push a particular XML? Would you mind sharing it, if it is not a sensitive information?

Unfortunately, this is sensitive file that I can't share.
What I can say is this file is 174MB of size if that can help.
Where you running python3 or python2.7 when you had the issue in your environment?
Thanks,

@maxymax68 try with xml_parser_supports_huge_tree: no in your OpenTAXII config? maybe it will work with lxml's huge_tree property disabled
I run OpenTAXII in py3.4 env mostly

"xml_parser_supports_huge_tree: no" did help

just keep in mind, that having this property set to no will not allow OpenTAXII to work with non-XML content blocks larger than 16MB
I'll continue investigation of that MemoryError

I split my XML file in chunks of 10MB and all was nicely pushed into OpenTAXII. Might hqve to do with how you manage memory for large XML files.

OpenTAXII does not manage any memory, it is lxml's job

I'm going to close this until we see this again / have more info


Reset Password via CLI tools

Allow users to reset their password using the CLI tools
Part of: https://github.com/3Blades/3blades/issues/115

N/A


Why did you say 'bottleneck'?

The authors say 'bottleneck' but why call that so? It seems the authors did not explain what 'bottleneck' refers to in the paper? Can you clarify?

Hey,
Scenic is hosting multiple projects related to multiple paper. Can you elaborate which project/paper you're referring to?


Automating release using GoReleaser

Using GoReleaser to automate releases
@sethvargo could you please add the required GitHub actions secret if you are ok with this PR?
Should fix https://github.com/GoogleCloudPlatform/gcr-cleaner/issues/52

I can't - this is what I was hinting at in https://github.com/GoogleCloudPlatform/gcr-cleaner/issues/52. This project is in a Google-managed organization which has strict org policies. I cannot export JSON service account keys and I cannot (currently) use Workload Identity Federation. This is why I haven't gone down this path yet - still waiting for internal approvals.


Refactor openai completion api w.r.t Prefix Cache

Refactored openai completions api with respect to the newly added Prefix Cache feature.

Thanks for the contribution! Could you merge with the main branch? We will start reviewing after fixing the conflict, thanks!

@LiuXiaoxuanPKU @WoosukKwon we'd love for this to land so we can use prefix cacheing with the OpenAI server. Is there someone who can review this PR?

Are there any update on this? Thanks!!!


Upgrade to Fedora 36 RPMs and cri-o 1.24

Jun 08 13:10:50 ip-10-0-46-223 bootkube.sh[2000]: Error: error creating container storage: the container name "etcdctl" is already in use by "92bead90214bcd21e31e1a26f06f9fd5e357b51093f1a7bfe5d8a666e33a5982". You have to remove that container to be able to reuse that name.: that name is already in use
Jun 08 13:10:50 ip-10-0-46-223 bootkube.sh[1259]: etcdctl failed. Retrying in 5 seconds...

/retest

This is a very common error now on the bootstrap node.  At some point some logic is going to have to be built into the bootkube.sh script to handle/cleanup these errors without having to do a reinstall.

/override ci/prow/e2e-aws


upgrade PicketLink to 2.5.5.SP1

Linux Build 8493 outcome was FAILURE using a merge of 697e4db32e26d600f04986104bffd5be932b3545
Summary: Tests passed: 3188, ignored: 360; compilation error: Compiler (new) Build time: 00:53:38

Windows Build 3527 outcome was FAILURE using a merge of 697e4db32e26d600f04986104bffd5be932b3545
Summary: Tests passed: 3188, ignored: 360; compilation error: Compiler (new) Build time: 00:53:43

This doesn't compile because the SAML2LoginModule is not available. This class was directly referenced in the standalone-picketlink.xml config in EAP 6 (and it's still there in WF 10) so this needs to be resolved somehow. Breaking compatibility like that is not good; if it can't be avoided there needs to be some sort of migration strategy.
I recommend you reopen #8237 so we can get 2.5.5.Final into master ASAP without blocking on getting this sorted out.

Looking at it. In case you need to release soon, use 2.5.5.Final from https://github.com/wildfly/wildfly/pull/8237


Consider adding audio cues for inline chat

Now we have audio cues - which are off by default atm - in the chat view. We might want these also in the inline chat.

cc @jrieken


Pickable_Branch is not editable

I'm trying to make the branch that you pick up from the ground drop something different, but it appears that the pickable_branch's drop list is not editable. I'm unsure if this is a bug or because that particular item is handled different than most other pickables. I just thought I'd let you know.

I asked the developer in the modding Discord. They were very helpful to advise that Pickable isn't supported yet. Not sure if there are plans to do so. I have limited time right now to consider forking or to do it myself so might just wait to see if there are plans.


Update OpenConext-deploy to correctly point to the SF2 version of EngineBlock

This issue is imported from pivotal - Originaly created at Sep 24, 2015 by Daan van Renterghem
Modify the vhost templates to point to the correct location. May only be done once the conversion to SF2 is complete.

This is currently still pending merging, and therefor cannot be delivered. The last blocking element is waiting for a conclusive answer from SURF. (Daan van Renterghem - Nov 5, 2015)


Refactor out "clarkparsia"

Perhaps make it match iBioSim paths.  Likely becomes a module that is an iBioSim dependency at some point.

SBOLDesigner package structure now mimics iBioSim package structure.


Rajah

Source name
Rajahentai
Source link
https://rajahentai.xyz/
Language
Indonesian
Other details
"+18/NSFW =Yes"
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a title with source name.
[X] I have checked that the extension does not already exist on the website extensions list or the app.
[X] I have checked that the extension does not already exist by searching the GitHub repository and verified it does not appear in the code base.
[X] I will fill out all of the requested information in this form.

🥱

Duplicate of #530


[pinmux] Tracker for various modifications and updates for Silver

This is a feature tracker for all the pinmux improvements and updates that need to go in for Silver.
For a more detailed overview, see also:

Pinmux Update Slides
Pinmux Programming Model

Tasks:

[ ] Consolidate pinmux and padctrl, https://github.com/lowRISC/opentitan/pull/5128
[ ] Update sleep behavior, add sleep status bits, https://github.com/lowRISC/opentitan/pull/5128
[ ] Update wakeup detector connections and configuration, https://github.com/lowRISC/opentitan/pull/5128
[ ] Add fine grained CSR lock bits for all multiregs, https://github.com/lowRISC/opentitan/pull/5128
[ ] Add HW strap sampling support
[ ] Add DFT TAP hooks, implement JTAG mux
[ ] Add hierarchy for physical instances to padring, connect power sequencing signals
[ ] Make USB wakeup detector instantiation parametric
[ ] Rework WARL behavior of pad attribute CSR (this needs topgen tooling support since the pad type will be defined in then top-level hjson)
[ ] Rework Hjson datastructure and the associated parts in topgen and the templates
[ ] Automatically generate pinmux documentation tables
[ ] Make padring more parametric such that topgen can propagate the pad configuration from the Hjson down into RTL (either via parameters or a template)
[ ] General refactoring of padring and ASIC top-levels, try to share code and make these top-levels more uniform

CC: @GregAC @silvestrst @tjaychen @vogelpi

I've split out the remaining low-prio tasks as separate issues, such that we can close this one down.


Add VIP through DHCP

Investigate using DHCP to add a VIP address
cc / @mylesagray

Time to turn this into GO
sudo ip link add mymacvlan1 link  ens192 type macvlan mode bridge
sudo ifconfig mymacvlan1 up
sudo dhclient mymacvlan1
ip addr show mymacvlan1

8: mymacvlan1@ens192: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue state UNKNOWN group default qlen 1
    link/ether ca:aa:6f:4a:bc:2c brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global mymacvlan1
       valid_lft forever preferred_lft forever
    inet6 fe80::c8aa:6fff:fe4a:bc2c/64 scope link
       valid_lft forever preferred_lft forever

This is being tacked in #55

Closed with #55


Some POSTFIX_RAW_CONFIG parameters should not be lowercased

For instance, there is a TLS-related field named smtpd_tls_CAfile.
With all the names for POSTFIX_RAW_CONFIG_* being lowercased, it becomes impossible to set the field with something like -e POSTFIX_RAW_CONFIG_SMTPD_TLS_CAFILE.
I guess making anything after POSTFIX_RAW_CONFIG_ case-sensitive would be a solution. Maybe add an option to actually make POSTFIX_RAW_CONFIG parameters case-sensitive for backwards compatibility?

done

image: marvambass/versatile-postfix:2024


Error: Failed to query available provider packages

Error:
$ tf init

Initializing the backend...

Initializing provider plugins...
- Finding latest version of appgate.com/appgate/appgate...

Error: Failed to query available provider packages

Could not retrieve the list of available versions for provider
appgate.com/appgate/appgate: cannot search $HOME/.terraform.d/plugins: lstat
$HOME/.terraform.d/plugins: no such file or directory

Config & main.tf
$ cat $TF_CLI_CONFIG_FILE
provider_installation {
  filesystem_mirror {
    path    = "$HOME/.terraform.d/plugins"
    include = ["appgate.com/*/*"]
  }
  direct {
    exclude = ["appgate.com/*/*"]
  }
}

$ file ~/.terraform.d/plugins/appgate.com/appgate/0.3.3/darwin_amd64/terraform-provider-appgate
/Users/drmullen/.terraform.d/plugins/appgate.com/appgate/0.3.3/darwin_amd64/terraform-provider-appgate: Mach-O 64-bit executable x86_64

$ cat main.tf
terraform {
  required_providers {
    appgate = {
      source = "appgate.com/appgate/appgate"
      # version = "0.3.3"
    }
  }
}

original post https://github.com/appgate/sdp-terraform-provider/issues/86

What terraform version are you using?
I know they have changed the behaviour of loading custom providers between versions, so it might differ.
In your example, I believe its that dev.tfrc cant resolve $HOME, that you need to use absoulute path
provider_installation {
  filesystem_mirror {
    path    = /Users/drmullen/.terraform.d/plugins"
    include = ["appgate.com/*/*"]
  }
  direct {
    exclude = ["appgate.com/*/*"]
  }
}

$ file ~/.terraform.d/plugins/appgate.com/appgate/0.3.3/darwin_amd64/terraform-provider-appgate

I believe this should be 1 more subdir,
< ~/.terraform.d/plugins/appgate.com/appgate/0.3.3/darwin_amd64/terraform-provider-appgate
---
> ~/.terraform.d/plugins/appgate.com/appgate/appgate/0.3.3/darwin_amd64/terraform-provider-appgate

fml. it was the 2nd appgate. my apologies. the $HOME does work though, so thats a +
seems like youre working pretty hard on this recently; ill try to give you feedback where possible
thanks


Getting Started restructure and Pub/sub quickstart

Description

New pub/sub quickstart
Reorganize Getting Started area for better accessibility
Freshness grammar/style pass on docs w/i Getting Started area

Need

repo used for git clone in pub/sub quickstart

Issue reference
#2059

Closing since PR is out of date (against v1.5). Will open new PR against v1.6.


Semantic diff reports configs have same policies - but policies are (syntactically) different

When running semantic diff on two policies which are (rule-wise) semantically equivalent but rules are syntactically different we can get a message saying that the two configs "have the same network topology and the same set of policies.".
For example, running nca.py --semantic_diff tests/istio_testcases/example_policies/bookinfo-demo/bookinfo-policy-equiv-principals-by-condition-1.yaml --base_np tests/istio_testcases/example_policies/bookinfo-demo/bookinfo-policy-test-principals-1.yaml -r tests/istio_testcases/example_podlist/bookinfo_pods_list.json
The two policies have one rule that is written differently but is semantically equivalent.

Not planned


New Field: Property Type

We miss a field in "Property Objects":
field name: propertyType
description: The type of property
not required
string
enum: [single_family_house, condominium, vacation_house, vacation_condominium, agricultural_farm, 2or3_family_house, multi_family_house, residential_building_plot, building_plot_other, mixed_property, commercial_condominium, office_building, industrial_building, special_object]

I am unaware of agricultural_farm or 2or3_family_house being part of WP or IAZI in the normal evaluation.
For me more category "income_property".

Agreement MP / TKB on 29.09.22: Ok


fix(line): distance bug in vertical or horizontal line #17162

Updated at Sep. 10.

Brief Information
This pull request is in the type of:

[x] bug fixing
[ ] new feature
[ ] others

What does this PR do?

Fixed issues

#17162:

(a)Label distance X option does not work when a vertical line is set.
(b)Label distance Y option does not work when a horizontal line is set.

Details
Before: What was the problem?

Distance value is default ( [x, y] = [20, 8] ).

Distance value is [x, y] = [20, 5].

Distance value is [x, y] = [0, 0]. In this case, there is no problem before I fixed this issue.

After: How does it behave after the fixing?

Distance value is default ( [x, y] = [20, 8] ). I confirmed distance y value was working in horizontal line and other label positions didn't changed.

Distance value is [x, y] = [20, 5]. I confirmed distance x value was working in vertical line and other label positions didn't changed.

Distance value is [x, y] = [0, 0]. I confirmed any label positions didn't changed.

Document Info
One of the following should be checked.

[x] This PR doesn't relate to document changes
[ ] The document should be updated later
[ ] The document changes have been made in apache/echarts-doc#xxx

Misc
ZRender Changes

[ ] This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs
N.A.
Others
Merging options

[ ] Please squash the commits into a single one when merging.

Other information

@Ovilia Thank you for your review. I checked and updated the test case in test/markLine.html. Also, I updated the screenshots in the pull request comment. Could you check again?

🔼 The result is generated by visual test. You can try it by yourself. npm run test:visual

Thank you for your comment. I'm sorry, I misunderstood the expected behavior. I'll fix the code and commit again later.

@horitaka Never mind. Thank you for your contribution!

After my investigation, I think the issue #17162 may not be a bug but correct behavior.
This issue is about the case position = end. I checked the current behavior for position = insideEndTop, insideEndBottom.
(1)For horizontal line, with position = insideEnd.
Vertical position isn't changed even if distance.y is set to [0, 30]
(2)For horizontal lines, position = insideEndTop, insideEndBottom.
Vertical position is changed if distance.y is set to [0, 30]

Issue #17162 has the same behavior as (1). Therefore, I do not think this is a problem. If we want to move label position verticallly for a horizontal line, I think new options like position = endTop, endBottom are needed.
Of course, if this issue is determined to be a bug, I can fix this issue.

Thanks for your contribution anyway!


force:package:beta:version:create command fails if there is unpackaged metadata

Hi,
During managed package version creation with sfdx-cli/7.174.0 win32-x64 node-v16.17.1 and the beta command sfdx force:package:beta:version:create we've observed, that if the package contains unpackaged metadata, then the version creation is failing during test execution.
Steps To Reproduce:

Make sure you have the newest version of the CLI (minimum 7.174.0)
Create a sample manage package with unpackaged metadata - for example it can be an approval process (for details go here: https://developer.salesforce.com/docs/atlas.en-us.sfdx_dev.meta/sfdx_dev
/sfdx_dev_dev2gp_unpackaged_md.htm). Make sure, that you've got some apex tests, that are using that approval process or any other unpackaged metadata that you are using.
Try to create a new version of the package with the following command: sfdx force:package:beta:version:create -v yourDevHubUser -p "YourPackageAlias" -k yourInstallationKey -w 30 --codecoverage

Expected result
New version is being created without any problems
Actual result
Apex unit tests are throwing an error:
System.DmlException: Process failed. First exception on row 0; first error: NO_APPLICABLE_PROCESS, No applicable approval process was found.: []

thank you so much for creating this @paweliwanowicz 🎉

Hi @paweliwanowicz could you please try using the beta commands on version 7.177.1, I believe this has been fixed by this comment https://github.com/forcedotcom/cli/issues/1743#issuecomment-1317288953
would love to verify this - thank you 👍

Hi @WillieRuemmele,
I've checked the newest version 7.179.0 and everything works just perfectly. Thank you for the help! :)


Submodule Updates

Description
Updates submodule

📚 $\text{Stack Overview}$
Pulls submitted in this stack:

#98 👈
main

This comment was automatically generated by st.


feat: add codeql ci/cd workflow for javascript

Issue that this pull request solves
Closes: #413
Proposed changes
Files added

.github/workflows/codeql.yml

To run this workflow you need to enable the following permission:

Go to your project repository settings
On left pane > Actions > General

Scroll down and Go to Workflow permissions > select "Read and write permissions"

Click save and its done.

Types of changes

[ ] Bugfix (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)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist

[x] I am a SSoC'24 contributor
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings

@singodiyashubham87 Please review this PR. After merging the PR, don't forget to add gssoc24 ,level labels and assign this PR to me, otherwise my points won't be added.
Thanks

@harmeetsingh11 Is giving read/write permission mandatory? If yes, why?


Open Camera directly, instead of image chooser

I want to open camera instead of opening image chooser or gallery. Please guide me in this regard.
Thank you

https://github.com/ArthurHub/Android-Image-Cropper/blob/5fb46a6c2efa6ce53d0a56ded5ea9adf992d7ad9/cropper/src/main/java/com/theartofdev/edmodo/cropper/CropImage.java#L237
I have the same question, I am analyzing these lines of code for the moment.

private fun openCamera() {
        val outputFileUri = getCaptureImageOutputUri(applicationContext)

        val captureIntent = Intent(MediaStore.ACTION_IMAGE_CAPTURE)

        if (outputFileUri != null) {
            captureIntent.putExtra(MediaStore.EXTRA_OUTPUT, outputFileUri);
        }

        startActivityForResult(captureIntent, CropImage.PICK_IMAGE_CHOOSER_REQUEST_CODE)
    }

Hi I have the same question! I need open the camera directly in Java Android.. Can you help me

@jjimenez0611 ,,i think this is ur solution.    sorry if i am wrong.
public class MainActivity extends AppCompatActivity {
ImageView iv;
Bitmap bitmap;
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_main);
   iv  = (ImageView)findViewById(R.id.image);

///start image import from default CAMERA
Intent cameraIntent = new Intent(android.provider.MediaStore.ACTION_IMAGE_CAPTURE);
startActivityForResult(cameraIntent, 0);
}
///now onActivityResult
@Override
protected void onActivityResult(int requestCode, int resultCode, Intent imageReturnedIntent) {
    super.onActivityResult(requestCode, resultCode, imageReturnedIntent);
        switch(requestCode) {
        case 0:
            if(resultCode == RESULT_OK){
                Uri selectedImage = imageReturnedIntent.getData();
                if(imageReturnedIntent.getData()==null){
                    bitmap = (Bitmap)imageReturnedIntent.getExtras().get("data");
                    iv.setImageBitmap(bitmap);
                }else{
                    try {
                        bitmap = MediaStore.Images.Media.getBitmap(this.getContentResolver(), imageReturnedIntent.getData());
                        iv.setImageBitmap(bitmap);
                    } catch (IOException e) {
                        e.printStackTrace();
                    }
                }
                Log.d("selectedImage: ", imageReturnedIntent.getData()+"");

            }

            break;

Thanks, I will implement this solution. I really appreciate your help.

The above solution by @Biplovkumar is not remotely close to the answer. The above 'answer' doesnt mention anything related to the CropImage library.
@jjimenez0611 your best bet so far is to open the default camera intent by parsing the uri, then on result of that start CropImage.activity().start(uri) with that uri instance.
I'm trying this and for the most part its working. But its not ideal, I would much rather like to open the Crop Intent with camera straight away, but doesnt seem like this is an option.
I've logged a ticket for this as well. (https://github.com/ArthurHub/Android-Image-Cropper/issues/510)
Also I've not seen many answers by the developer of this awesome library, which is weird, because it really is an awesome library.

Hi
I found a solution to my problem.
I open the camera normally and them onActivityResult open the crop activity on this way:
CropImage.activity(CropImage.getCaptureImageOutputUri(SiffApplication.getInstance().getComponent().getContext()))
.setGuidelines(CropImageView.Guidelines.ON)
.setFlipHorizontally(false)
.setFlipVertically(false)
.setActivityTitle(getString(R.string.location_image_capture))
.setAspectRatio(1, 1)
.start(getContext(), this);
And then get the crop image onActivityResult....

Hi
I found a solution to my problem.
I open the camera normally and them onActivityResult open the crop activity on this way:
CropImage.activity(CropImage.getCaptureImageOutputUri(SiffApplication.getInstance().getComponent().getContext()))
.setGuidelines(CropImageView.Guidelines.ON)
.setFlipHorizontally(false)
.setFlipVertically(false)
.setActivityTitle(getString(R.string.location_image_capture))
.setAspectRatio(1, 1)
.start(getContext(), this);
And then get the crop image onActivityResult....

I have Used your solution still it is not working.
Fallowing exception is coming.
Failed to load sampled bitmap: file:///storage/emulated/0/Android/data/
cache/pickImageResult.jpeg (No such file or directory)

Hi @BhargaviDeviYamanuri.. I really don't remember exactly in what OS do you need to do a FILE PROVIDERS in the manifest
<meta-data
                android:name="android.support.FILE_PROVIDER_PATHS"
                android:resource="@xml/file_paths">
        </meta-data>

And in the res folder do you need to have the file_paths.xml with the following information:

And then this es my code
override fun onRequestPermissionsResult(requestCode: Int, permissions: Array, grantResults: IntArray) {
super.onRequestPermissionsResult(requestCode, permissions, grantResults)
when (requestCode) {
cameraPermissionsRequest -> {
            if (grantResults.isEmpty() || grantResults[0] != PackageManager.PERMISSION_GRANTED) {

                Log.i("log", "Permission has been denied by user")
            } else {
                takePicture()
            }
        }
    }
}

override fun onActivityResult(requestCode: Int, resultCode: Int, data: Intent?) {
    super.onActivityResult(requestCode, resultCode, data)

    if (requestCode == CAMERA_CAPTURE_IMAGE_REQUEST_CODE && resultCode == RESULT_OK) {
        setPictureToView()
    }
}

private fun createImageFile(): File {
    // Create an image file name
    val storageDir: File? = getExternalFilesDir(Environment.DIRECTORY_PICTURES)
    return File.createTempFile(
        "JPEG_TEMPORAL_", /* prefix */
        ".jpg", /* suffix */
        storageDir /* directory */
    ).apply {
        // Save a file: path for use with ACTION_VIEW intents
        currentPhotoPath = absolutePath
    }
}

private fun takePicture() {

    Intent(MediaStore.ACTION_IMAGE_CAPTURE).also { takePictureIntent ->
        // Ensure that there's a camera activity to handle the intent
        takePictureIntent.resolveActivity(packageManager)?.also {
            // Create the File where the photo should go
            val photoFile: File? = try {
                createImageFile()
            } catch (ex: IOException) {
                // Error occurred while creating the File
                null
            }
            // Continue only if the File was successfully created
            photoFile?.also {
                val photoURI: Uri = FileProvider.getUriForFile(
                    this,
                    "com.example.TEST.fileprovider",
                    it
                )
                takePictureIntent.putExtra(MediaStore.EXTRA_OUTPUT, photoURI)
                startActivityForResult(takePictureIntent, CAMERA_CAPTURE_IMAGE_REQUEST_CODE)
            }
        }
    }

}

private fun setPictureToView() {
    // Get the dimensions of the View
    val targetW = 400
    val targetH = 400

    val bmOptions = BitmapFactory.Options().apply {
        // Get the dimensions of the bitmap
        inJustDecodeBounds = true
        BitmapFactory.decodeFile(currentPhotoPath, this)
        val photoW: Int = outWidth
        val photoH: Int = outHeight

        // Determine how much to scale down the image
        val scaleFactor: Int = Math.min(photoW / targetW, photoH / targetH)

        // Decode the image file into a Bitmap sized to fill the View
        inJustDecodeBounds = false
        inSampleSize = scaleFactor
    }
    BitmapFactory.decodeFile(currentPhotoPath, bmOptions)?.also { bitmap ->
        //imageView.setImageBitmap(bitmap)

        val stream = ByteArrayOutputStream()
        bitmap.compress(Bitmap.CompressFormat.JPEG, 60, stream)
        val imageInByte = stream.toByteArray()
        val houseImage = Base64.encodeToString(imageInByte, Base64.DEFAULT)

        inspectionAdapter.setPhoto(houseImage)

    }
}

Hi @jjimenez0611 thanks for the reply,sorry for late response .I have solved the issue my logic is almost similar to your logic. But there is one problem that is   android.os.FileUriExposedException  coming to solve that i have used following code.
//This line is used for to avoid android.os.FileUriExposedException need to do further checking         StrictMode.VmPolicy.Builder builder = new StrictMode.VmPolicy.Builder();         StrictMode.setVmPolicy(builder.build());

Here is a solution for anyone who faces this issue
  ///Open CAMERA Only
                            List<Intent> allIntents = new ArrayList<>();
                            PackageManager packageManager = mContext.getPackageManager();
                            StrictMode.VmPolicy.Builder builder = new StrictMode.VmPolicy.Builder();
                            StrictMode.setVmPolicy(builder.build());
                            // collect all camera intents if Camera permission is available
                            if (!CropImage.isExplicitCameraPermissionRequired(mContext) && true) {
                                allIntents.addAll(CropImage.getCameraIntents(mContext, packageManager));

                                //Choose Camera Only
                                Intent target;
                                if (allIntents.isEmpty()) {
                                    target = new Intent();
                                } else {
                                    target = allIntents.get(allIntents.size() - 1);
                                    allIntents.remove(allIntents.size() - 1);
                                }

                                // Create a chooser from the main  intent
                                 Intent chooserIntent = Intent.createChooser(target, "Take Photo");
                                  chooserIntent.putExtra(Intent.EXTRA_INITIAL_INTENTS, allIntents.toArray(new
                                  Parcelable[allIntents.size()]));
                                  YOURACTIVITY.startActivityForResult(chooserIntent,
                                 CropImage.PICK_IMAGE_CHOOSER_REQUEST_CODE);


keywords to increase discoverability

I just found this library today and wish I had 2 years ago.
Try using these plugin keywords to better categorize this on the chai.js website:

contains
deep
like
match
similar

Nice suggestion! Just added some new keywords in package.json. They will appear with next release of this plugin and build of chaijs.org

Thanks!


Allow user deletion

Trello ticket
Description
Introduce the possibility of deleting users via rake task.
Users won't be effectively deleted from the system but flagged as deleted.
When an user is marked as deleted:

Their cases will be assigned to their team.
Won't be able to sign in.
Won't be able to receive a password reset.
Won't be able to be invited to any team.
Won't receive any email notifications.
Won't be displayed in the team member list.
Won't appear in the user list for filtering cases.
Won't appear in case list/assignment autocomplete.
Their name will be displayed as User Name [user deleted] when possible.

To delete a user:
<EMAIL_ADDRESS>rake user:delete

or
 ID=id-sequence rake user:delete

Checklist:

[x] Have you documented your changes in the pull request description?
[ ] Has acceptance criteria been tested by a peer?

 User deletion/deactivation

Had a discussion with @scruti.
I reckon two different functions to decorators for names across two different files isn't ideal. We've suggested leaving as-is for now, but following up with a story to clean up the display_name function, and probably move it out of the user model.

@scruti did have a look into https://github.com/rubysherpas/paranoia ?

@StupidCodeFactory I am trying to avoid to rely on default scopes as much as possible (and Paranoia seems to be relying on them) + seems their own maintainers are discouraging to adopt it nowadays. But was nice to peek around the project :+1:

@slorek

I am not sure but I think we might need to implement display_name in the TeamDecorator too for instances where the assignee or source was a Team. I think this will bubble the call up to the Team class but that is inconsistent.

Yes I relied in the method call being handled by the model delegation. Got tired of replacing things around and left it like it was :sweat_smile:  Will change it to keep the consistency.

@slorek has a good point we need to add #display_name to the team decorator as these are polymorphic model we need to maintain the same interfaces! I wish ruby had interfaces like Java that would be amazing.

One general thing is calling decorate inside view. I would prefer to pass already decorated users to the view.

@frankieroberto about deleted_at & deleted_by I just took the simplest approach that works with the current requirements.
deleted_at could replace deleted field as flag field. But... do we need to know this info? Also the user.deleted? would require to have logic to wrap the timestamp check instead of being automatically provided by AR. But I recognise the cost of this is marginal and I know there are some recommendations for using null/timestamp as alternative for false/true flag attributes.
For the deleted_by I expect to be introduced whenever we develop the possibility of marking users as deleted from the user interface. At this stage of development, triggered by developers on the terminal, this concept didn't make sense to me. So left it out.


Embedded malicious code

Hello,
Just wanted to highlight that on July 13, someone committing as @jakob-everson added malicious code to this repo.
Initial commit is here: https://github.com/basilio/responsiveCarousel/commit/aec0aa71fcdd189e4983c9dedc52e32f2cf4a29f
You can see the malicious code here
: https://github.com/basilio/responsiveCarousel/blob/master/responsiveCarousel.js#L1201
I'm reporting this to cdnjs and GitHub. @basilio - can you clean this up?

Thanks @basilio. Can you explain why @jakob-everson was given access to the repo? And why they are adding malicious code?


Add capability to configure isort

It seems one can only use isort in its default configuration with this plugin?
It would be great to add support for configuring it for everyone that does not...
For instance, to make isort play nice with black, it's been said that using these options work best:
isort --trailing-comma -m 3

I may have opened this too fast - it looks like I can simply configure isort directly by providing the paramaters in one of the config files it recognizes. nevermind!


cuda error: an illegal memory access was encountered

Hi @pureexe, thanks for your great work, but when I trained on m own datasets for a while, I got a Cuda error, when I change to train on only one GPU, it can train for a longer time, but can also trigger this error
the error messages like below:
train.py in forward cof=pt.repeat_interleave(cof,args.sublayers,0)
runtimeerror: an illegal memory access was encountered
But when I use the demo-room datasets to train, it seems the training phase is normal. I use the colmap to preprocess the datasets and get the hwf_cxxy.txt and poses_bounds.npy using the scripts you provided.
btw, when train on my own datasets, how to set the plane.txt?  hope you can give some advice, thanks~

Could you give the command that you use to run the train.py code? What config that you used?
Do you set up the environment with our environment.yml? If you use your own environment, what is your PyTorch version?
Could you give us your dataset? We will test it on our side.
Also, what is your machine spec? (GPU, CPU, RAM)

my gpu is  TITAN X,cuda version is 10.1,drive version is 418.56,torch1.4.0. it seems that this error is related to my environment, but since try to reinstall all the above is complex,  I will try to debug this error and see if there are other way to solve it, thanks~

I also encountered this problem，have you solved it？


Add more mocked tests

Now the coverage is not so good. Especially fetch_parameterized() and its related functions.

Close this in favor of adding "my.cnf" option file.


Cheatsheet: Fixed my pull request to support other screen sizes. Added optional dark theme and spiced up the CSS.

I'm afraid I won't be adding that option for the moment.


There is a vulnerability in quartz 2.3.0,upgrade recommended

https://github.com/apache/incubator-dolphinscheduler/blob/17c06ce966fc5c6a6136ee142e4698312fe6532f/pom.xml#L67
CVE-2019-13990
Recommended upgrade version：2.3.2

+1

+1

Updated


Add batmat to allowed releasers

(This PR template only applies to permission changes -- ignore for changes to the tool)
Description
Would like to add myself to list of people allowed to release the Jenkins plugins parent pom: (https://github.com/jenkinsci/plugin-pom). Stephen Connolly asked me to handle the release
Permissions pull request checklist
Always

[X] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

N/A

For a new permissions file only

N/A

When adding new uploaders (this includes newly created permissions files)

[ ] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable

actually, authorization given upstream, see https://github.com/jenkinsci/plugin-pom/pull/42#issuecomment-273713875

[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] All newly added users have logged in to Artifactory at least once

Thanks @orrc


fix: Fix queue time calculation

Description
Fixes how the queue time calculation works.
Because the started_at time changes between queued and in_progress events the current calculation does not work as expected - the queue time is nearly always 0. This PR introduces a cache to compare the queued event and in_progress event started_at times for a more accurate queue time calculation.
It also removes the state from the existing workflow job runtime metric as measuring the job runtime and the queue time have different accuracy requirements, so one metric for both is not suitable. In return, a new metric has been introduced to measure the queue time specifically and the old metric will only measure the time the job spent in progress.

Really neat + readable PR 👌


Compiling under FreeBSD

To compile under FreeBSD there's no need to create macros for be32toh() and so on, they are already declared that way. So you have to change include/endian.h:119 to:
#elif defined(__NetBSD__) || defined(__DragonFly__)
i.e.: remove checking for __FreeBSD__ definition.
P.S.: I don't know if the same applies to NetBSD and DragonFly also, but it's like to be: NetBSD and DragonFly BSD don't follow the OpenBSD naming standard for betohxx(), they uses the bexxtoh() naming standard instead.

Thank you for reporting this issue. I have removed the macro definitions for FreeBSD. Unfortunately I don't have access to a NetBSD or a DragonFly system to test if they need to be removed there too, so I have left them in there.


How to set configurations to client?

Is there a way to send client config settings to a language client, e.g pyls?
I read the example from #32, where a simple example was provided. However the vscode api changed, in fact the provided example does not work.
I tried to adapt the example to the changed api, but my config is not send to pyls.
Is there any minimal working example for this?

@TypeFox Any update here? There is no documentation provided on how to do this, and without such documentation it is impossible to use this project with Python language servers among others.

Oh thank god I finally figured it out (by spending some hours inspecting the innards of vscode-languageserver-node). You have to add a middleware, like this:
new MonacoLanguageClient({
  name: "Your Name",
  clientOptions: {
    documentSelector: [{ pattern: "**" }],
    middleware: {
      workspace: {
        configuration: (params, token, configuration) => {
          return [{ foo: "bar" }];
        },
      },
    },
  },
  connectionProvider: {
    get: (errorHandler, closeHandler) =>
      Promise.resolve(
        createConnection(connection, errorHandler, closeHandler)
      ),
  },
})

(I won't guarantee the other stuff in that code will work for you, I had to hack a bunch of other things as well. But the middleware key should be correct. Of course you must adjust the actual content of the LSP configuration to taste.)
The relevant part of vscode-languageserver-node is where the middleware is pulled out and turned into config.

Hello @raxod502. What is the middleware key are you referring to? Is it the name of the language server? Or is it something else?

By "middleware key" I simply meant the data that appears immediately after middleware: in the sample code.

In Riju, I see that you are using Microsoft's python language server. Is the binary available somewhere? Also, what steps did you take to configure it. It would be helpful for my use case to use a standalone server written in a single language rather than a patchwork of different libraries and extensions.

I suggest referring to the code.

Installation
Configuration

I am running Microsoft's python language server on a separate machine and want to connect with the monaco editor. And as per language server documentation I need to pass "python.linting.enabled": true config setting to the language server for enabling linting support.
I am using below config for creating MonacoLanguageClient instance.
new MonacoLanguageClient({
      name: 'Simple Client',
      clientOptions: {
        // use a language id as a document selector
        documentSelector: ['python'],
        initializationOptions: {
          interpreter: {
            properties: {
              UseDefaultDatabase: true,
              Version: '2.7',
              InterpreterPath: '/usr/bin/python',
              LibraryPath: '/usr/lib/python2.7'
            }
          },
          asyncStartup: true,
        },

        middleware: {
          workspace: {
            configuration: (params, token, configuration) => {
              return[{python: { linting: {enabled: true}}}];
            },
          }
        },
        synchronize: {
          configurationSection: ''
        },

        // disable the default error handler
        errorHandler: {
          error: () => ErrorAction.Continue,
          closed: () => CloseAction.DoNotRestart
        }
      },
      // create a language client connection from the JSON RPC connection on demand
      connectionProvider: {
        get: (errorHandler, closeHandler) => {
          return Promise.resolve(createConnection( connection as any, errorHandler, closeHandler));
        }
      }
    });

Its always sending blank setting to language server.
{"jsonrpc":"2.0","method":"workspace/didChangeConfiguration","params":{"settings":{}}}
Is there anyway to send custom setting to language server?

I can't speak for the didChangeConfiguration message, Riju (linked above) has support for setting the initializationOptions in the initialize method (key init in Riju language configuration) as well as sending configuration in response to the workspace/configuration message (key config in Riju language configuration, currently unused but I just tested that it does work).

@itthought did you ever have any success populating the settings payload?
@raxod502 it looks like Riju was taken down, did you perhaps rename it?

it looks like Riju was taken down, did you perhaps rename it?

Unfortunately, that's not what happened. Please see https://intuitiveexplanations.com/tech/replit/ for the details.


Generalize the texture class to 2D and 3D

This commit simply make Tex3D a generic N-D texture with implementation for bilinear (2D) interpolation as well as the existing trilinear (it would be simple to extend it to 4D if necessary but it's not yet provided).
A template wizard could probably make a generic  template-recursive version instead of using explicit specializations.

Tested on linux plateform only ?

Windows too iirc. why ?

Just checking.


CSP Notebook: Outputs

I show previously hidden outputs. Also, removed matplotlib warnings. Javascript widget error warnings remain.
EDIT:
Note that in one of the outputs, all that is printed is pointers to some objects. I'm not sure what the purpose of this is. I left them in, but personally I feel that that output should be hidden, or maybe have the cell removed completely.

I agree that "In[3] australia, usa, france: adds no value. We should either delete that cell, or add a useful str/repr method to CSP.
It is depressing that we can't get the javascript to load in github. I'm not sure what a better solution is.


Submit the training job fails

there is an error thrown when trying to run the first command of "Submit the training job"
az batchai job create --cluster nc6cluster --name cifar_distributed --resource-group batchai.horovod --workspace batchaidev --experiment cifar --config-file job.json --storage-account-name mystorageaccount
Error occurred in request., RetryError: HTTPSConnectionPool(host='management.azure.com', port=443): Max retries exceeded with url: /subscriptions/<subscription>/providers/Microsoft.Storage/storageAccounts?api-version=2018-02-01&%24skiptoken=<token> (Caused by ResponseError('too many 502 error responses',))
Traceback (most recent call last):

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

ID: 1b33df02-8f80-f8a2-ab56-7e0054305f9e
Version Independent ID: 1dd76f73-2011-5a14-9dc8-ffb865d24805
Content: Tutorial - Distributed training with Azure Batch AI and Horovod
Content Source: articles/batch-ai/tutorial-horovod-tensorflow.md
Service: batch-ai
GitHub Login: @johnwu10
Microsoft Alias: danlep

@burstingninja Hi, according to the document https://docs.microsoft.com/en-us/azure/bot-service/bot-service-resources-links-help?view=azure-bot-service-4.0  of Bot product group, I don't think they help in Git
Please go to https://stackoverflow.com/questions/tagged/botframework for help. Thanks a lot!


Cant load the .env. file via react-native cli

Hi,
I am having a wired issue while using this lib. I have used this article when configuring the lib. The instructions are similar to the one given here. https://dev.to/calintamas/how-to-manage-staging-and-production-environments-in-a-react-native-app-4naa
For android it works fine. But for ios, if I run it via xcode then it works fine. (i.e. loads config). But it doesnt work when I do ENVFILE=.env.uat react-native run-ios .
react-native: 0.61.5
react-native-config: 0.12.0
Any idea?

You need to direct the CLI to use the scheme you created.  Add --scheme "scheme_name" to the end of your react-native run-ios command

Thanks @district-michael that works. But it cant be proper solution. It should work wiht ENVFILE parameter.

Perhaps.  I think it may be difficult to read environment variables from within Xcode's build pipeline, which is why they have to write the name of the env file to /tmp/envfile.  If you don't want to type it every time, we just added new commands to our package.json to run dev and prod, so now we run npm run ios-prod.

This should have been mentioned in readme. Spent so much time investigating this.

What is the point of mentioning the ENVFILE option if it does not work and you will have to literally replace the file anyway? 🤔❓


Implement provision actions using Terraform

Description
Currently for each supported provider there is a driver implementation and also action scripts for each operation (deploy, cancel, reboot, etc...) When a new provider is added, these driver and scripts need to be implemented, which requires too much effort.
The idea is to remove these (driver and action scripts) and add Terraform to deploy, undeploy and execute all the operations. This will reduce the complexity and will make it easier to add a new provider in the future.
Use case
Uses cases are multiple, but the main ones are reduce the complexity of adding a new provider and be able to grow up the supported providers catalag.
Also, it would have the ability to create more resources in the remote provider, not only hosts, but networks or storage.
Interface Changes
Changes in provision and maybe some CLI adjustments.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

PRs to merge in master:

code
infra
docs
tests


ES6 string interpolation

Good pm why's that ES6 string interpolation  doesn't work for me the ${sample} it result as a string as it is.

This doesn't sound related to VS Code


Upgrade debug and other deps

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Does cypress test still run with the upgrade?

@pjfitzgibbons Is this for CVE fix? If so can you please update the title and description for it. It is easier to track CVE fixes this way later.


Correctly fetch typing.Annotated metadata in typing.NamedTuple

Signed-off-by: Samhita Alla<EMAIL_ADDRESS>TL;DR
Annotated metadata isn't included in namedtuple typing. include_extras should fix the issue.
Type

[x] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
Modified the import of get_type_hints from  typing to typing_extensions since include_extras is available starting Python 3.9.
Tracking Issue
https://github.com/flyteorg/flyte/issues/
Follow-up issue
NA
OR
https://github.com/flyteorg/flyte/issues/

Can you add a unit test?

Done!

@pingsutw, @eapolinario, can I get a +1 again, please? Since the codecov check was still failing, I had to push a minor update.


training many domains: failure of CA step (too many open files)

Training a system with 6090 domains (= 6090 bilingual corpora) failed during the step
(4 of 7) Context Analyzer training
as follows

Command 'java -cp /home/ubuntu/ModernMT/build/mmt-0.13.jar -Dmmt.home=/home/ubuntu/ModernMT -Djava.library.path=/home/ubuntu/ModernMT/build/lib eu.modernmt.cli.ContextAnalyzerMain -l en -i /home/ubuntu/ModernMT/engines/st2_bil/models/context -c /home/ubuntu/ModernMT/runtime/st2_bil/tmp/training/clean_tms' failed with exit code 1

The log reports a problems related to the numerb of files (too many):

Exception in thread "Thread-1" java.lang.IllegalStateException: Bucket not open
at eu.modernmt.context.lucene.storage.CorpusBucket.close(CorpusBucket.java:125)
at org.apache.commons.io.IOUtils.closeQuietly(IOUtils.java:303)
at java.util.HashMap$Values.forEach(HashMap.java:972)
at eu.modernmt.context.lucene.storage.CorporaIndex.close(CorporaIndex.java:136)
at org.apache.commons.io.IOUtils.closeQuietly(IOUtils.java:303)
at eu.modernmt.context.lucene.storage.CorporaStorage$BackgroundTask.run(CorporaStorage.java:288)
Exception in thread "main" eu.modernmt.context.ContextAnalyzerException: Unable to add domain 4077
at eu.modernmt.context.lucene.LuceneAnalyzer.add(LuceneAnalyzer.java:56)
at eu.modernmt.cli.ContextAnalyzerMain.main(ContextAnalyzerMain.java:72)
Caused by: java.io.FileNotFoundException: /home/ubuntu/ModernMT/engines/st2_bil/models/context/storage/_4077 (Too many open files)

The system limit is 1024, that is the command
$> ulimit -n
1024

Does Lucene open all the files simoultaneously? This would explain the the error.
In this case is there any way to constrain Lucene to use less files simoultaneously?
Or do you envisage alternative solutions?
We need to manage thousand of domains ...

Hi @cattoni
for sure we have to add the option to limit the amount of open files in MMT. BTW in production environment, for performance reason, is a good idea to have that limit increased and let the program to handle more open files at the same time.
For your testing purpose I'd suggest to change that limit to a high value so not to have annoying error messages anymore.
Davide

Thanks @davidecaroselli
I've set in  /etc/security/limits.conf
* soft nofile 16384
* hard nofile 16384

and the training completed successfully

👍

@davidecaroselli please add a note to the INSTALL documentation

@marcotrombetti done!
https://github.com/ModernMT/MMT/blob/master/INSTALL.md#max-open-files-limit

@davidecaroselli @cattoni I also updated the documentation for Docker.
Closing. Please re-open if I am missing something.


How to use with Gradle

as title

I've searched for this also, and couldn't find any Yested (or plain Kotlin2JS) project with Gradle, so I've created an example.
Can be seen it here: https://github.com/vilmosnagy/yested-gradle-kotlin2js-example


Update getRoundUpdate method to return a deep copy

fixes #1443

Codecov Report

Merging #1450 (1ff26cb) into master (828364b) will decrease coverage by 10.19%.
The diff coverage is n/a.

:exclamation: Current head 1ff26cb differs from pull request most recent head e434e03. Consider uploading reports for the commit e434e03 to get more accurate results

@@             Coverage Diff             @@
##           master    #1450       +/-   ##
===========================================
- Coverage   50.51%   40.31%   -10.20%
===========================================
  Files         145      146        +1
  Lines        7544     9651     +2107
===========================================
+ Hits         3811     3891       +80
- Misses       3066     5132     +2066
+ Partials      667      628       -39

Impacted Files
Coverage Δ

pkg/core/consensus/testing/node.go
0.00% <0.00%> (-93.55%)
:arrow_down:

...ore/consensus/blockgenerator/candidate/testutil.go
0.00% <0.00%> (-90.91%)
:arrow_down:

pkg/core/consensus/testing/utils.go
0.00% <0.00%> (-75.61%)
:arrow_down:

...core/consensus/blockgenerator/candidate/genesis.go
0.00% <0.00%> (-69.24%)
:arrow_down:

...nsensus/blockgenerator/candidate/blockgenerator.go
0.00% <0.00%> (-64.29%)
:arrow_down:

pkg/gql/query/transactions.go
13.43% <0.00%> (-52.55%)
:arrow_down:

pkg/rpc/client/session.go
21.42% <0.00%> (-47.15%)
:arrow_down:

pkg/core/consensus/reduction/aggregator.go
47.36% <0.00%> (-43.94%)
:arrow_down:

pkg/util/nativeutils/eventbus/eventbus.go
57.14% <0.00%> (-42.86%)
:arrow_down:

pkg/rpc/client/client.go
3.73% <0.00%> (-36.61%)
:arrow_down:

... and 168 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Mocking traits with &impl and &mut impl parameters

Both functions give me compiler errrors for cargo build --tests. I use entrait with the unimock feature as dependency. Is there a way to make them work with entrait/unimock?
use entrait::entrait;

pub trait MyTrait {}

#[entrait(Foo, mock_api=FooMock)]
fn foo<D>(_: &D, _foo: &impl MyTrait) -> i32 {
    unimplemented!()
}

#[entrait(Bar, mock_api=BarMock)]
fn bar<D>(_: &D, _bar: &mut impl MyTrait) -> i32 {
    unimplemented!()
}

fn main() {}

Errors:
error[E0658]: `impl Trait` in type aliases is unstable
 --> src/main.rs:6:25
  |
6 | fn foo<D>(_: &D, _foo: &impl MyTrait) -> i32 {
  |                         ^^^^^^^^^^^^
  |
  = note: see issue #63063 <https://github.com/rust-lang/rust/issues/63063> for more information

error[E0658]: `impl Trait` in type aliases is unstable
  --> src/main.rs:11:29
   |
11 | fn bar<D>(_: &D, _bar: &mut impl MyTrait) -> i32 {
   |                             ^^^^^^^^^^^^
   |
   = note: see issue #63063 <https://github.com/rust-lang/rust/issues/63063> for more information

error: unconstrained opaque type
 --> src/main.rs:6:25
  |
6 | fn foo<D>(_: &D, _foo: &impl MyTrait) -> i32 {
  |                         ^^^^^^^^^^^^
  |
  = note: `Inputs` must be used in combination with a concrete type within the same impl

error[E0599]: the method `unimock_try_debug` exists for mutable reference `&mut _::<impl MockFn for BarMock>::Inputs<'_>::{opaque#0}`, but its trait bounds were not satisfied
  --> src/main.rs:10:1
   |
10 | #[entrait(Bar, mock_api=BarMock)]
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ method cannot be called due to unsatisfied trait bounds
   |
   = note: the following trait bounds were not satisfied:
           `&mut _::<impl MockFn for BarMock>::Inputs<'_>::{opaque#0}: Debug`
           which is required by `&mut _::<impl MockFn for BarMock>::Inputs<'_>::{opaque#0}: ProperDebug`
           `_::<impl MockFn for BarMock>::Inputs<'_>::{opaque#0}: Debug`
           which is required by `_::<impl MockFn for BarMock>::Inputs<'_>::{opaque#0}: ProperDebug`
   = note: this error originates in the attribute macro `::entrait::__unimock::unimock` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0308]: mismatched types
   --> src/main.rs:6:18
    |
5   | #[entrait(Foo, mock_api=FooMock)]
    | --------------------------------- arguments to this function are incorrect
6   | fn foo<D>(_: &D, _foo: &impl MyTrait) -> i32 {
    |                  ^^^^   ------------
    |                  |      |
    |                  |      the expected opaque type
    |                  |      this type parameter
    |                  expected `&_::<impl MockFn for ...>::Inputs<'_>::{opaque#0}`, found `&impl MyTrait`
    |
    = note: expected reference `&_::<impl MockFn for FooMock>::Inputs<'_>::{opaque#0}`
               found reference `&impl MyTrait`
    = help: type parameters must be constrained to match other types
    = note: for more information, visit https://doc.rust-lang.org/book/ch10-02-traits.html#traits-as-parameters
note: function defined here
   --> /Users/ander/.cargo/registry/src/github.com-1ecc6299db9ec823/unimock-0.4.12/src/macro_api.rs:209:8
    |
209 | pub fn eval<'u, 'i, F>(unimock: &'u Unimock, inputs: F::Inputs<'i>) -> Evaluation<'u, 'i, F>
    |        ^^^^

error[E0308]: mismatched types
   --> src/main.rs:11:18
    |
10  | #[entrait(Bar, mock_api=BarMock)]
    | --------------------------------- arguments to this function are incorrect
11  | fn bar<D>(_: &D, _bar: &mut impl MyTrait) -> i32 {
    |                  ^^^^       ------------
    |                  |          |
    |                  |          the expected opaque type
    |                  |          this type parameter
    |                  expected `&mut _::<impl MockFn for ...>::Inputs<'_>::{opaque#0}`, found `&mut impl MyTrait`
    |
    = note: expected mutable reference `&mut _::<impl MockFn for BarMock>::Inputs<'_>::{opaque#0}`
               found mutable reference `&mut impl MyTrait`
    = help: type parameters must be constrained to match other types
    = note: for more information, visit https://doc.rust-lang.org/book/ch10-02-traits.html#traits-as-parameters
note: function defined here
   --> /Users/ander/.cargo/registry/src/github.com-1ecc6299db9ec823/unimock-0.4.12/src/macro_api.rs:209:8
    |
209 | pub fn eval<'u, 'i, F>(unimock: &'u Unimock, inputs: F::Inputs<'i>) -> Evaluation<'u, 'i, F>
    |        ^^^^

Some errors have detailed explanations: E0308, E0599, E0658.
For more information about an error, try `rustc --explain E0308`.

I changed the syntax to generic functions which compiles just fine:
use std::fmt::Debug;

use entrait::entrait;

pub trait MyTrait: Debug {}

#[entrait(Foo, mock_api=FooMock)]
fn foo<D, T: MyTrait>(_: &D, _foo: &T) -> i32 {
    unimplemented!()
}

#[entrait(Bar, mock_api=BarMock)]
fn bar<D, T: MyTrait>(_: &D, _bar: &mut T) -> i32 {
    unimplemented!()
}

fn main() {}

Sidenote: when using the module version of the entrait macro, I need to use distinct generic type parameters for all functions. This example will not compile and complain about the name `T` is already used for a generic parameter in this item's generic parameters
use std::fmt::Debug;

use entrait::entrait;

pub trait MyTrait: Debug {
    fn foor();
}

#[entrait(pub Foo, mock_api=mock, no_deps)]
mod my_module {
    pub fn foo<T: super::MyTrait>(_foo: &T) -> i32 {
        unimplemented!()
    }
    pub fn bar<T: super::MyTrait>(_bar: &mut T) -> i32 {
        unimplemented!()
    }
}

fn main() {}


Keep reporting errors while writing code

UncaughtException: TypeError: Cannot read property 'getProp' of undefined. Restarting Server...

+1 failing constantly after upgrade to latest 0.19.1

Downgraded to 0.18.5 and haven't seen given error popup for last 24h .

Maybe you're using an older version of bootstrap. Check for that in your header

Have to say that upgrade to latest 0.20.0 fixed an issue for me.

Same here


Docker setup for development and testing environments

Drawing from:

Avalon's multistage Dockerfile: https://github.com/avalonmediasystem/avalon/blob/master/Dockerfile
Hyku's docker-compose.yml: https://github.com/samvera/hyku/blob/master/docker-compose.yml
https://dev.to/bgord/multiple-postgres-databases-in-a-single-docker-container-417l

Also switched internal test hyku to use temporary hyku fork that has specific commits from hyku master and PRs but without the user elevation feature: https://github.com/ubiquitypress/hyku/tree/hyku_2/3_without_user_elevation

Closing in favor of #9.


fix: set no_include flag properly in tonic plugin

Fixes some copypasta that breaks the flag.

Many thanks! I'll get this corrected and out this afternoon.


java.net.BindException occasionally

Android device report log:
D/OkHttp: <-- HTTP FAILED: java.net.BindException: bind failed: EADDRINUSE (Address already in use)

occasionally. Don't know why.

Could you provide an executable test case? There’s nothing here we can act upon.


build broken with vis-network 5.2.4

After updating vis-network to 5.2.4 in my angular project i keep getting build errors on util.d.ts in the vis-util library inside the node_modules of vis-network.
ERROR in node_modules/vis-network/node_modules/vis-util/dist/types/util.d.ts(214,61): error TS1005: ',' expected. node_modules/vis-network/node_modules/vis-util/dist/types/util.d.ts(214,62): error TS1005: ',' expected. node_modules/vis-network/node_modules/vis-util/dist/types/util.d.ts(215,64): error TS1005: ',' expected. node_modules/vis-network/node_modules/vis-util/dist/types/util.d.ts(215,65): error TS1005: ',' expected. node_modules/vis-network/node_modules/vis-util/dist/types/util.d.ts(223,52): error TS1005: ',' expected. node_modules/vis-network/node_modules/vis-util/dist/types/util.d.ts(223,53): error TS1005: ',' expected.
I've tried to also install vis-util 1.1.6 in my own project but that did not help.
So how can i fix this problem? or is this something you guys need to fix in the build itself?

Hi @ydg2018,
I just tested it and it doesn't happen to me. Could you provide an MWE please?
One thing to try is to update TypeScript, we use 3.5.3. Because this looks like the kind of error 3.3 tsc would show with 3.4 project but that's just wild guess.
Thanks.

ah yes that is a good possibility as our project is still on 3.2.4 because an other library is not updated for the newer versions of typescript. I guess we are coming closer to a time where we need to update typescript and find other solutions for the old library :)
Thanks for the answer.


Deepslate/stone variants for both ores

For parity with vanilla, both sulfur and potassium should have both stone and deepslate ores even if they do not naturally generate.

I don't see why that would be necessary due to I've purposely made sulfur only spawn at deepslate levels while potassium in regular stone.  It would just create more files which are never used.

I don't see why that would be necessary due to I've purposely made sulfur only spawn at deepslate levels while potassium in regular stone. It would just create more files which are never used.

Just to keep in line with the way Vanilla handles ores.  All ores have deepslate and stone variants just for consistency.

With that logic there would be stone and deepslate variants for all the nether ores. I will keep them separated.


fix: vue-ts template checkbox type mismatch

Fix for issue: vue/ts template broken

Actually there is still the same issue. Will come back with another fix.


refactor: persist job and resource dependency

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 13 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 77.136%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
7132

Relevant Lines:
9246

💛  - Coveralls


fix #948

Coverage remained the same at 79.31% when pulling c60b097f4aa119fda7d4d56c14262eda79b21b8b on antgonza:fix-948 into 01ac199d172409edc20c99da4b67b05d1f92e614 on biocore:master.

:+1:


[CI flake] increase mockserver timeouts

Follow up from https://github.com/apollographql/apollo-kotlin/pull/5698

It failed on another test, just reran them


Allow creating env variables shared across multiple projects

Prerequisites

[x] I am running the latest version. (up upgrade)
[x] I searched to see if the issue already exists.
[x] I inspected the verbose debug output with the -v, --verbose flag.
[x] Are you an Up Pro subscriber?

Description
I would like to request a new feature.
I've noticed that env variables are being pushed to EC2 SSM Parameter store (which by the way is an amazing idea), and follow the naming convention /up/<project>/<stage>/<ENV_VARIABLE>. I wonder if it would be easy to add /up/<stage>/<ENV_VARIABLE> that would be shared across all projects.
I would image the order of precedence would be the following (first one has the lowest priority):
/up/all/<ENV_VARIABLE>
   ^
/up/<stage>/<ENV_VARIABLE>
   ^
/up/<project>/all/<ENV_VARIABLE>
   ^
/up/<project>/<stage>/<ENV_VARIABLE>

Steps to Reproduce
None. Feature request
Love Up?
Loving it.
Would appreciate if this would be considered for a future release. Thank you

I've given this some more thought, and there is a potential downside. If multiple functions would be sharing some env variables, for example api keys to connect to a 3rd party app, then it would make sense to put those several lambda functions into one. It would "force" good programming practice, as env variables would "dictate" in a sense the scope of an application/function(s). I don't have that much experience with lambda functions / server-less so I am open to be corrected.
I've just started using apex-up a week ago and we already built one functionality at my workplace, so I have yet to encounter a situation where env variables need to be shared across multiple functions, but will keep an eye on that.
This would definitely be an opt-in.

On #535 I mentioned a latency issue as well that I didn't think about until going to implement it haha. Probably best to avoid it for now at least. I think personally I'd still prefer to update each app, even if it's a bit of work, at least then you can slowly migrate to a new database etc, bit more control.


fix(sbb-form-field): fix accessibility when slotting form error

Previously, aria-describedby of the input was not correctly updated when removing or first adding a sbb-form-error. Additionally, to support removing and adding sbb-form-error multiple times for screen readers, we had to move the aria-live region to a single element, although according to documentation the aria-live region should be on a container.

Codecov Report

Merging #1922 (8cf5dbd) into master (ab917b0) will decrease coverage by 17.22%.
Report is 1529 commits behind head on master.
The diff coverage is 31.20%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##           master    #1922       +/-   ##
===========================================
- Coverage   54.85%   37.63%   -17.22%
===========================================
  Files          49      268      +219
  Lines        1659     9699     +8040
  Branches      406     2104     +1698
===========================================
+ Hits          910     3650     +2740
- Misses        671     5814     +5143
- Partials       78      235      +157

Files Changed
Coverage Δ

...ts/sbb-accordion-item/sbb-accordion-item.events.ts
100.00% <ø> (ø)

.../sbb-accordion-item/sbb-accordion-item.stories.tsx
0.00% <0.00%> (ø)

...mponents/sbb-accordion-item/sbb-accordion-item.tsx
0.00% <0.00%> (-37.21%)
:arrow_down:

...components/sbb-accordion/sbb-accordion.stories.tsx
0.00% <0.00%> (ø)

...ents/sbb-action-group/sbb-action-group.stories.tsx
0.00% <0.00%> (ø)

...onents/sbb-alert-group/sbb-alert-group.stories.tsx
0.00% <0.00%> (ø)

src/components/sbb-alert/sbb-alert.stories.tsx
0.00% <0.00%> (ø)

...onents/sbb-autocomplete/sbb-autocomplete.events.ts
100.00% <ø> (ø)

...ents/sbb-autocomplete/sbb-autocomplete.stories.tsx
0.00% <0.00%> (ø)

...-breadcrumb-group/sbb-breadcrumb-group.stories.tsx
0.00% <0.00%> (ø)

... and 57 more

... and 216 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add enum options to target lowering tablegen

@Abhishek-Varma Could you help review?

cc @frafranz

@Abhishek-Varma Can this PR be merged?

@Abhishek-Varma Can this PR be merged?

Absolutely!
Can you rebase with current master and force-push? I'll get this in once the CI passes again.

@Abhishek-Varma Can this PR be merged?

Absolutely!
Can you rebase with current master and force-push? I'll get this in once the CI passes again.

I updated with main.


feat(coverage): added test cases for coverage

PR Checklist

[ ] Unit Tests have been added for new changes
[ ] API tests have been updated if applicable
[ ] All commented code has been removed
[ ] If you've added a dependency, you've ensured license is compatible with Apache 2.0 and clearly outlined the added dependency.

What are you changing?

Anything the reviewer should know when reviewing this PR?
If the there are associated PRs in other repositories, please link them here (i.e. open-amt-cloud-toolkit/repo#365 )

Codecov Report

Merging #478 (ed075ac) into main (a17d616) will increase coverage by 11.77%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #478       +/-   ##
===========================================
+ Coverage   63.88%   75.66%   +11.77%
===========================================
  Files          43       43
  Lines        1282     1282
  Branches      144      144
===========================================
+ Hits          819      970      +151
+ Misses        396      240      -156
- Partials       67       72        +5

Impacted Files
Coverage Δ

src/app/dashboard/dashboard.component.ts
80.00% <0.00%> (+10.00%)
:arrow_up:

...devices/device-toolbar/device-toolbar.component.ts
65.62% <0.00%> (+12.50%)
:arrow_up:

src/app/devices/kvm/kvm.component.ts
57.89% <0.00%> (+17.54%)
:arrow_up:

src/app/devices/devices.component.ts
76.71% <0.00%> (+17.80%)
:arrow_up:

src/app/core/toolbar/toolbar.component.ts
85.71% <0.00%> (+28.57%)
:arrow_up:

src/app/wireless/wireless.service.ts
100.00% <0.00%> (+45.45%)
:arrow_up:

src/app/profiles/profiles.service.ts
100.00% <0.00%> (+52.17%)
:arrow_up:

src/app/domains/domains.service.ts
100.00% <0.00%> (+54.54%)
:arrow_up:

src/app/auth.service.ts
80.64% <0.00%> (+61.29%)
:arrow_up:

src/app/configs/configs.service.ts
100.00% <0.00%> (+62.96%)
:arrow_up:

... and 1 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 a17d616...ed075ac. Read the comment docs.


refactor(admob): delete unused snippets

The build is failing in #285 due to AdMob Breaking Changes.
I was going to upgrade this code to the new admob v20.0.0, but then I noticed that most of it is not even shown on our docs. Maybe it's time to clean it up?

@rosariopfernandes I think you're right, these are unused. Thanks for catching this!

Oops this is my bad but two of these snippets are actually used:
https://firebase.google.com/docs/admob/android/quick-start#add-and-initialize-sdk
I'll have to restore those.

@samtstern The link you posted takes me to this section:

If you're talking about the MobileAds.initialize() snippet, it was not removed by this PR. It's still in this repo.
Other that this one, I don't see any other snippet in that docs page you linked.

Yep you're right, it was a false alarm! Something else broke the page and I assumed it was this.


SQLite (SQL generally?) visibility problems with custom text fields

Expected Behavior
Searching by a custom text field should work, as it does using the ES backend.
Actual Behavior
After upserting the custom attribute from a workflow with a value like configdriven-stresstest, subsequent queries fail with errors like the following:
"level":"error","ts":"2023-08-03T16:49:59.640-0700","msg":"service failures","operation":"CountWorkflowExecutions","wf-namespace":"default","error":"CountWorkflowExecutions operation failed. Query failed: SQL logic error: no such column: stresstest (1)

or
error":"ListWorkflowExecutions operation failed. Select failed: SQL logic error: no such column: stresstest (1)",

My guess is there is some kind of escaping or parsing issue with dashes going on here.
Steps to Reproduce the Problem

temporal operator search-attribute create --name CustomStringField --type Text
Upsert the value from a workflow with something with dashes in it like foo-bar
Attempt to query/list for WFs matching that value

Specifications

Version: 1.21.4
Platform: Temporal CLI using SQLite backend

Thanks for reporting this issue.
It appears that the issue you're experiencing is related to how Temporal handles special characters like dashes (-) in custom text fields.  You're correct that dashes can cause problems, and Temporal's SQLite backend does not yet support them in custom text fields.
It's best to avoid dashes in your custom search attribute values.  Consider using underscores (_) instead for now, as it is currently supported.
We understand this limitation and are working on improving the support for special characters in future releases.
In the meantime, you can find more information about using search attributes in the Observability section of our documentation.

Thanks for reporting this issue. It looks like you're running into an issue where custom text fields with dashes are not being parsed correctly when querying workflow executions using the SQLite backend.
This appears to be a known issue and we are currently working on a fix. You can find more information about this issue and its potential solutions in our documentation.
In the meantime, you can try the following workaround:

Use underscores instead of dashes: Replace the dashes in your custom field values with underscores. For example, instead of configdriven-stresstest, use configdriven_stresstest.
Use a different backend: If possible, consider using a different backend such as Elasticsearch or MySQL for your Visibility store.

We apologize for any inconvenience this issue may cause. We appreciate your patience as we work on a resolution.


scrcpy-killed it self

i face a problem that scrcpy doesn't mirror the phone and suddenly killed it self when connected wirelessly
scrcpy
INFO: scrcpy 1.14 <https://github.com/Genymobile/scrcpy>
/usr/share/scrcpy/scrcpy-server: 1 file pushed. 2.3 MB/s (31693 bytes in 0.013s)
killed

i tried with usb but scrcpy keep won't mirroring
i use adb version 1:8.1.0+r23-7
and scrcpy version 1.14
android version 8.1

adb version 1:8.1.0+r23-7

Hmmm, what? What is the result of:
adb --version

On which platform? How did you install it?

suddenly killed it self when connected wirelessly

i tried with usb but scrcpy keep won't mirroring

So it also prints killed over USB?

Thank you for respond

adb --version
Android Debug Bridge version 1.0.39
Version 1:8.1.0+r23-7
Installed as /usr/lib/android-sdk/platform-tools/adb

i use kali linux and i install it with apt install scrcpy

So it also prints killed over USB?

unfortunately yess

Any errors in adb logcat?

i don't know but i will see the logcat now and send it to you

here's the logcat
https://drive.google.com/file/d/1RYVnZaQJBCkBf54uAELtV1_5cJXQOo3X/view?usp=sharing

here's the logcat
https://drive.google.com/file/d/1RYVnZaQJBCkBf54uAELtV1_5cJXQOo3X/view?usp=sharing

08-29 23:32:11.292 25990 25990 I /system/bin/adbd: type=1400 audit(0.0:4175): avc: denied { write } for comm=73796E6320737663203232 name="tmp" dev="mmcblk0p50" ino=1483223 scontext=u:r:adbd:s0 tcontext=u:object_r:system_data_file:s0 tclass=dir permissive=1
08-29 23:32:11.292 25990 25990 I /system/bin/adbd: type=1400 audit(0.0:4176): avc: denied { add_name } for comm=73796E6320737663203232 name="scrcpy-server.jar" scontext=u:r:adbd:s0 tcontext=u:object_r:system_data_file:s0 tclass=dir permissive=1
08-29 23:32:11.292 25990 25990 I /system/bin/adbd: type=1400 audit(0.0:4177): avc: denied { create } for comm=73796E6320737663203232 name="scrcpy-server.jar" scontext=u:r:adbd:s0 tcontext=u:object_r:system_data_file:s0 tclass=file permissive=1
08-29 23:32:11.292 25990 25990 I /system/bin/adbd: type=1400 audit(0.0:4178): avc: denied { write open } for comm=73796E6320737663203232 path="/data/local/tmp/scrcpy-server.jar" dev="mmcblk0p50" ino=1483224 scontext=u:r:adbd:s0 tcontext=u:object_r:system_data_file:s0 tclass=file permissive=1
08-29 23:32:11.312 25990 25990 I /system/bin/adbd: type=1400 audit(0.0:4179): avc: denied { getattr } for comm=73796E6320737663203232 path="/data/local/tmp/scrcpy-server.jar" dev="mmcblk0p50" ino=1483224 scontext=u:r:adbd:s0 tcontext=u:object_r:system_data_file:s0 tclass=file permissive=1
08-29 23:32:11.312 25990 25990 I /system/bin/adbd: type=1400 audit(0.0:4180): avc: denied { relabelfrom } for comm=73796E6320737663203232 name="scrcpy-server.jar" dev="mmcblk0p50" ino=1483224 scontext=u:r:adbd:s0 tcontext=u:object_r:system_data_file:s0 tclass=file permissive=1
08-29 23:32:11.312 25990 25990 I /system/bin/adbd: type=1400 audit(0.0:4181): avc: denied { relabelto } for comm=73796E6320737663203232 name="scrcpy-server.jar" dev="mmcblk0p50" ino=1483224 scontext=u:r:adbd:s0 tcontext=u:object_r:shell_data_file:s0 tclass=file permissive=1

It seems shell does not have the permissions to write to /data/local/tmp/ on your device.
$ adb shell touch /data/local/tmp/a
$ adb shell ls -ld /data/local/tmp

$ adb shell ls -ld /data/local/tmp
drwxrwxrwx 3 root root 4096 2020-08-30 00:59 /data/local/tmp

the output has write permission now but still get same error

If you still get permission denied in the logs, then there is a permission issue. Maybe restrictions defined by selinux.

Can you create a file in /data/local/tmp from adb shell?

Can you create a file in /data/local/tmp from adb shell?

yes i can there is a file after i ran this command

$ adb shell touch /data/local/tmp/a

If you still get permission denied in the logs, then there is a permission issue. Maybe restrictions defined by selinux.

hmm i don't think so because it went perfectly normal before i reinstall scrcpy, i was removing scrcpy with apt and i tried install it again manually with prebuilt server in your README but it's fail after i ran ninja -Cx. because it fail so i remove manually all scrcpy in my /usr file and install it again with scrcpy and suddenly that is happen to me.

What is the result of adb shell ls -l /data/local/tmp?
In particular, is there some scrcpy-server.jar? What are its permissions? Is it a directory or a file?

$ ls -l /data/local/tmp
-rw-rw-rw- 1 shell   shell       0 2020-08-30 00:59 a
-rw-rw-rw- 1 shell   shell   31693 2020-07-01 14:59 scrcpy-server.jar

it's a file sir, tried to remove scrcpy-server.jar and rerun scrcpy still got the same error..
scrcpy can make directory tmp by it self right ? if so, i tried remove tmp directory and rerun scrcpy again show me this error

$ scrcpy
INFO: scrcpy 1.14 https://github.com/Genymobile/scrcpy
adb: error: failed to copy '/usr/share/scrcpy/scrcpy-server' to '/data/local/tmp/scrcpy-server.jar': remote secure_mkdirs failed: Permission denied
/usr/share/scrcpy/scrcpy-server: 0 files pushed. 1.8 MB/s (31693 bytes in 0.017s)
ERROR: "adb push" returned with value 1

i reboot my device and try scrcpy and it works, i don't know what happen to my phone.. so weird
thank u for your advice sir.


why is there no resolution input in the distance_map computing(scipy.ndimage.distance_transform_edt)?

Hello,
Thanks for sharing the code. it helps me a lot. The paper (loss odyssey ) is also great for me to know more about loss in medical segmentation.
I have found that in your code, the resolution is not considered in distance_map computing. However, it is used herehttps://github.com/LIVIAETS/boundary-loss.
Why is the resolution  not be used here? Looking forward to your reply. Thanks a lot.

Thanks for your interest.
We didn't use it because nnunet already resamples them into the same spacing during preprocessing.


Wrong amount in templates by edit of an estimates

[ ] Steps

Create estimates with an item
Update estimates by adding another item.

issue closed


Comments in .tmLanguage.json disable syntax highlighting (with JSON with comments)

VSCode Version: 1.23.1
OS Version: macOS 10.12

Steps to Reproduce:

Add e.g. MASM language extension
Add a // comment in masm.tmLanguage.json
Reload Window
Syntax highlighting no longer works in .asm files

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

That's as designed: Comments are not valid in tmLanguage.json.
You can use put comments in custom properties, such as here:
https://github.com/Microsoft/vscode/blob/f92bacec327be5308c0f045873a5eb84b1a80006/extensions/pug/syntaxes/pug.tmLanguage.json#L12

Even with this in User Settings?:
"files.associations": {     "*.json": "jsonc" },


Should the flink-sql-runner continue collecting and shading connecters?

In the long term, should the flink-sql-runner continue collecting and shading more and more connectors? Or should they be environmental, supplied by whatever flink classpath they are running on?
An alternative is the Dockerfile could download connectors such as flink-sql-avro-confluent-registry and flink-sql-connector-kafka to the flink classpath somewhere.

Resolved by https://github.com/streamshub/flink-sql/pull/20


Could not find play-services-measurement-base.aar (com.google.android.gms:play-services-measurement-base:15.0.2)

I'm suddenly getting this error after removing and readding the android platform:
FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'android'.
> Could not find play-services-measurement-base.aar (com.google.android.gms:play-services-measurement-base:15.0.2).
  Searched in the following locations:
      https://jcenter.bintray.com/com/google/android/gms/play-services-measurement-base/15.0.2/play-services-measurement
-base-15.0.2.aar

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
Picked up _JAVA_OPTIONS: -Xmx512M

I've already edited build.gradle as suggested in certain topics:
repositories {
	maven { url 'https://maven.google.com' }
        jcenter()
}

@andreszs was this issue resolved on your side?


WIP: Add capabilities to apko

Add capabilities to apko

Needs go generate ./...

closing as old/inactive. please re-open if necessary


export model to onnx

Hi, Nvidia AI team. thanks for your opensource sample code for deploying the pointpillar on Xavier.
By the export to onnx part, I have a question. How can you ensure you only export the middel part of the network(after voxelization and encode to 10 feature per pillar), not the whole part which include voxelization, pillar feature extraction, scatter to bev, backbone, postprocess?
 torch.onnx.export(model,                   # model being run
          (dummy_voxel_features, dummy_voxel_num_points, dummy_coords), # model input (or a tuple for multiple inputs)
          "./pointpillar.onnx",    # where to save the model (can be a file or file-like object)
          export_params=True,        # store the trained parameter weights inside the model file
          opset_version=11,          # the ONNX version to export the model to
          do_constant_folding=True,  # whether to execute constant folding for optimization
          keep_initializers_as_inputs=True,
          input_names = ['input', 'voxel_num_points', 'coords'],   # the model's input names
          output_names = ['cls_preds', 'box_preds', 'dir_cls_preds'], # the model's output names
          )

thanks!

They're using onnx-graphsurgeon to drop those operation after exporting model as whole.
Relevant script;
https://github.com/NVIDIA-AI-IOT/CUDA-PointPillars/blob/main/tool/simplifier_onnx.py