added phraseBERT and coreference resolution

solved the issues below

Use phrase-BERT for entity encoding Use phrase-BERT for entity encoding #49
Add co-reference resolution Add co-reference resolution #1

FYI, it took about 11hours to implement coreference resolution to the trump dataset.

Does the coreference resolution method get slower and slower during the loop? This could be linked to the use of list.append at each iteration.


refactor/01_03_52-disable_lock_whitelist

i was not able to run it separately with npm run run-single-test-file tests/01_03_52-disable_lock_whitelist.js pls advice...

pipeline:run

pipeline:run

pipeline:run

pipeline:run

pipeline:run

pipeline:run

pipeline:run

pipeline:run


Devel

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

67 of 69 (97.1%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.5%) to 91.592%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/bitset/bitset.go
67
69
97.1%

Totals

Change from base Build<PHONE_NUMBER>1:
2.5%

Covered Lines:
1536

Relevant Lines:
1677

💛  - Coveralls


[BUG] Not able to swap over MAYA

When trying to quote a swap over MAYA the error Swap amount too small gets prompted.
Even though the chains are cheap and 30$ should be more than sufficient to make a swap.

@realpaaao  should be resolved

Fixed , Closing


Question

Hi,
I hope all is fine,
Thank you so much for sharing this repository,
my question is can I apply this optimization on Yolox or just this optimization for bert and yolov5 ??

Hey @LamnouarMohamed.
We currently do not support yolox. However, given that we have had major success with sparsifying yolov5, and, at least at a first glance, yolox shares a lot of architectural similarities, I think it would a good idea to use our Sparsify tool to automatically optimize your model.

Hi @dbogunowicz
thank you so much for reply,
if I don't have mistake,  you can correct me
I think there is a huge difference between yolov5 and yolox, because yolov5 based on CNN, and yolox based on transformers, I think doesn't make sense to apply same recipe.

To close this issue: we do not have a specific sparsification integration for yolox. We are about to roll out sparsified yolov8, which, according to the benchmarks, is superior to yolox.
We are also working on the possibility for the users to sparsify any model, follow us to stay up-to-day with the developments!


Modify Host added by the user

After a host has been added to the system an user should be able to modify the details subject to the following constraints:

If a host is not attached to any of the volumes then all the fields can be modified.
If a host is attached to a volume then only the availability zones and initiators can be modified. A user will be able to only add new initiators and and or remove availability zones.
The availability zone that belongs to both host and volume can not be deleted.

The code is ready but needs to be added after the #210 PR is merged. Needs to be tested.

Closing since PR is merged.


opentabletdriver: <IP_ADDRESS> -> <IP_ADDRESS>

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for opentabletdriver is: "Open source, cross-platform, user-mode tablet driver"
meta.homepage for opentabletdriver is: "https://github.com/OpenTabletDriver/OpenTabletDriver"
meta.changelog for opentabletdriver is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS> with grep in /nix/store/hc7dwdvi4d6ncqy2w2sz7rwip5v4w5j8-OpenTabletDriver-<IP_ADDRESS>
found <IP_ADDRESS> in filename of file in /nix/store/hc7dwdvi4d6ncqy2w2sz7rwip5v4w5j8-OpenTabletDriver-<IP_ADDRESS>

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
0 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
opentabletdriver

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/hc7dwdvi4d6ncqy2w2sz7rwip5v4w5j8-OpenTabletDriver-<IP_ADDRESS> \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A opentabletdriver https://github.com/r-ryantm/nixpkgs/archive/9a2377c8a6f047f4d71debd0226fff9556869d6e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/hc7dwdvi4d6ncqy2w2sz7rwip5v4w5j8-OpenTabletDriver-<IP_ADDRESS>
ls -la /nix/store/hc7dwdvi4d6ncqy2w2sz7rwip5v4w5j8-OpenTabletDriver-<IP_ADDRESS>/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

opentabletdriver

Maintainer pings
cc @thiagokokada for testing.

Converting to draft to avoid @r-ryantm to creating another PR.

Update script no longer recommends the earlier version, so I think this can be closed.


removing extra columns from mysql full db creation sql

Remove extra columns from mysql database creation script.  I assume these are for an in-progress feature(s) that hasn't been merged to master yet, but having them included in the master db creation script causes an error when saving a bin due to missing values for NOT NULL columns.

Hi guys,
I'm facing the same issue today with jsbin 3.27.3 and mysql 5.6.22.
@loudwinston  How did you solve the problem please ?

can anybody post the solution to this problem ? I cannot seem to wrap my head around it..  should we modify the code in models/bin.js  ??

Okay if I add default values to  "streaming_read_key/active_tab and  active_cursor -1 ?"
CREATE TABLE `sandbox` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `javascript` mediumtext COLLATE utf8mb4_unicode_ci,
  `html` mediumtext COLLATE utf8mb4_unicode_ci,
  `created` datetime DEFAULT NULL,
  `last_viewed` datetime DEFAULT NULL,
  `url` char(255) COLLATE utf8mb4_unicode_ci DEFAULT NULL,
  `active` char(1) COLLATE utf8mb4_unicode_ci NOT NULL DEFAULT 'y',
  `reported` datetime DEFAULT NULL,
  `streaming` char(1) COLLATE utf8mb4_unicode_ci DEFAULT 'n',
  `streaming_key` char(32) COLLATE utf8mb4_unicode_ci NOT NULL,
  `streaming_read_key` char(32) COLLATE utf8mb4_unicode_ci NOT NULL DEFAULT '',
  `active_tab` varchar(10) COLLATE utf8mb4_unicode_ci NOT NULL DEFAULT '',
  `active_cursor` int(11) NOT NULL DEFAULT '-1',
  `revision` int(11) DEFAULT '1',
  `css` mediumtext COLLATE utf8mb4_unicode_ci,
  `settings` mediumtext COLLATE utf8mb4_unicode_ci,
  PRIMARY KEY (`id`),
  KEY `viewed` (`last_viewed`),
  KEY `url` (`url`(191)),
  KEY `streaming_key` (`streaming_key`),
  KEY `spam` (`created`,`last_viewed`),
  KEY `revision` (`url`(191),`revision`)
) ENGINE=InnoDB AUTO_INCREMENT=9002830 DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci

now i still need to work out why the code in the ui is not working ..
I get a lot of ERR_SSL_PROTOCOL_ERROR in chrome console... very strange

Okay as a temporary fix I chenged the following line
https://github.com/jsbin/jsbin/blob/108e15e15350a294cee957cd9f928372a1fe79ce/views/runner.html#L33
to  {{#scripts}}<script src="{{static}}{{.}}"></script>


Do not allow users to enter invalid instagram URL

https://userfeeds.github.io/cryptobeep/5865
Better option might be to fix the url if the user only enters username.

Done!


add table name in query profile, OlapScanner: -> OlapScanner(lineitem):

Proposed changes
Issue Number: close #xxx
Problem Summary:
in order to know which table is scanned, add table name in query profile, just OlapScanner: -> OlapScanner(lineitem):
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

already done.


Build for ESP32: Fatal error: etstimer.h: No such file

I followed the instructions up to the point 5. Build example.
I deleted the component  WS2812FX due to previous build error (see #84 ).
But then I got:
... esp-homekit-demo/components/led-status/led_status.c:3:22: fatal error: etstimer.h: No such file or directory
compilation terminated.

make[1]: *** [led_status.o] Error 1

make: *** [component-led-status-build] Error 2

Additional infos:
I am building on Mac. Added the sdkconfig below.
sdkconfig.txt

led-status is an example for esp8266. ESP32 examples are under examples/esp32 and right now there is only one example there.

I am also getting the same build error while compiling esp32/led example.

ok, this caused by the fact that some of components were esp8266-/esp32-only and build system was dumb enough to help separating them.. So I just pushed a change that splits components into 3 categories.. This should fix problem of component clashes.
Should compile now.

It compiled successfully.
Thanks


Tougher, less trusting splitter function

Maybe use yaccety_sax, but so far it's baffled me.

Moved splitter out into XmlMunger module


Update model and defaults to 20.01

This updates the default model to Galaxy 20.01 along with the default image values. The version of Postgres chart is also updated to the latest version (the model was built using the new version).

Just a heads up, you don't need to include a postgres dump (assuming that's for faster startups) since https://github.com/galaxyproject/galaxy/pull/8300 / 19.09

Just confirmed this working as advertised @mvdbeek! Thanks for letting us know.

@afgane The web pods seem to be ok, but wanted to check on the job pods. I believe they are applying the migrations one at a time.


Increase support for multiple-choice questions

Current the tool only supports simple yes/no multiple choice questions.
@joelanman has been experimenting with support for multiple-choice more generally, but has found that Claude starts hallucinating.

@joelanman's PR #7 implements multiple-choice. I'll raise another issue for improving it's performance.

@joelanman's PR #7 implements multiple-choice. I'll raise another issue for improving it's performance.


Avoid deprecation warning as long as we need to support older versions

Avoid deprecation warning:
Deprecated: Return type of TwbsHelper\View\HtmlAttributesSet::offsetGet($key) should either be compatible with ArrayObject::offsetGet(mixed $key): mixed, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /var/www/html/vendor/neilime/twbs-helper-module/src/TwbsHelper/View/HtmlAttributesSet.php on line 47

See
https://php.watch/versions/8.1/ReturnTypeWillChange

Hi @becoded,
You should rebase your branch with main as it may fix the failing tests

Hello @neilime, could you give it an other try please?

Thank you @becoded


Record leaf nodes support

Python version - 2.7.13
Google-cloud-bigquery - 0.22.1
I am trying to insert data using the big query python client library. I am getting an error that record leaf types are not supported. The schema contains nested records that are repeatable. Is there a way using the client to insert nested repeatable records ?
my schema is as follows:

Thanks for reporting @vigbk!
Do you have some example code/stacktrace?
@tseaver is this kind of related to https://github.com/GoogleCloudPlatform/google-cloud-python/issues/2354?

@daspecster Here is an excerpt of the code I used
from google.cloud import bigquery
import os
import json
from datetime import datetime

def transform(item):
    newItem = {"insertId": getUUID(), "json": item}
    return newItem

def getTemplateSuffix():

    utcDate =  datetime.utcnow().strftime("%Y%m%d")
    templateSuffix = "_" + utcDate
    return templateSuffix

def uploadToBigQuery(data):

     rowData = json.loads(data)

     os.environ["GOOGLE_APPLICATION_CREDENTIALS"] = "credentials.json"

     # Instantiates a client
     bigquery_client = bigquery.Client("logs")

     dataset = bigquery_client.dataset("eventlogs")
     table = dataset.table("events")

     # Reload the table to get the schema.
     table.reload()

     rows = [rowData]
     print(rows)
     errors = table.insert_data(rows=rows, template_suffix=getTemplateSuffix())

     if not errors:
         print('Loaded 1 row into {}:{}'.format(dataset, table))
     else:
         print('Errors:')
         print(errors)

data = '{"user_dim":{"user_id":"12345","user_properties":[{"key":"subdomain_id","value":{"string_value":"custom.zendesk.com"}}]},"event_dim":{"date":"20170116","category":"ALERT","name":"NUMBER_NOT_A_MOBILE","timestamp_micros":1484580530384000,"params":[{"key":"network","value":{"string_value":"Singapore"}},{"key":"country","value":{"string_value":"Singapore"}}]}}'

uploadToBigQuery(data)

Is this a duplicate of #2951? (It seems like it is, but I am hesitant to close without being sure.)

@tswast Can you confirm / deny the duplicateness?

These two issues look like duplicates to me, too.

Okay. I marked the other one as important, and am going to close this one for personal sanity / tracking. :-)


"Open Web UI at Launch" stays at  /blank instead of /

WebUI seems to have some leftovers related to settings, which produce an error:

I believe ipfs-desktop is not using a release version of webui, but omne from https://github.com/ipfs-shipyard/ipfs-webui/pull/1445

@lidel yes, it stays at /blank. Will fix in a minute.
And yes, we're using the master branch of Web UI. You're still using an older version and that's why you're getting that error. Try rebuilding the Web UI part.
We'll need a new release of Web UI before releasing this.


Is it possible to create the permalink structure from the filename structure?

My permalink config is
permalinks: {
	structure: ':path/index.html'
}

I set the path in the YFM of every file and it's working fine, the files are compiled like '/jogos/lista/index.html', '/jogos/buraco/index.html', etc
But I wanted to have something more automatic, I don't want to do this in every single html file.
I was looking for a way to have something like that
permalinks: {
	structure: ':basename/index.html'
}

So the name of the files would be something like "jogos__lista.html", "jogos__buraco.html" and I would have a way to intercept the basename and replace all "__" with "/".
Is this currently possible?
(I'm using assemble 0.4.41 and permalinks 0.3.6)

@doowb this is a nice workaround but perhaps this should be built in


PoC of using pdoc to generate API docs

This draft PR adds support for generating docs using pdoc. It looked like one of the simpler options, less complex than something like sphinx.
I only focused on the main doc page, and the METAR page. The other links on the main page won't work yet.
I renamed some of the functions in metar.py to start with ``__` so that they wouldn't show up in the generated documentation. (They aren't in the existing documentation, so I assumed you wanted them hidden.)
Like the previous draft PR, if this seems like something worth pursuing, I'm happy to finish up the PoC and issue a full PR with the updated docs.
You can preview the docs by running poetry run pdoc avwx, which will start up a doc server on http://localhost:8080
(I do actually have a project that I want to build on top of this library, but I got distracted 😄.)

pdoc looks like it will do exactly what I want. I love that it uses the existing md in the docstrings too.
I want to push a quick data update build before merging this and playing with it. So far, the only weird thing I need to config is showing inherited class methods like Metar.parse since that is core to every report class.
I'll also dive deeper into the pods and Python docs on discoverability. Most of those metar.py functions don't get used outside of the file they are in, but some of them could still be useful. I should go back through and audit what should and should not be considered a public function. For example, most of the avwx.parsing.core functions are just meant to be used by avwx report parsers, but I've used some of them in outside projects because they were already implemented.

I poked around at this a bit more this evening, and updated some more docs.
pdoc's docs talk a bit about discoverability. The data/__init__.py file uses the __all__ approach.
Let me know if you'd like more help with these documentation updates, I'm happy to keep poking at it. Otherwise, I'm going to go start using the library. Thanks again for the work you've put into this.

Thank you for your work starting the auto doc. I'll merge this in and keep playing with it. I'll finally get the documentation pipeline fixed too 😄

Thanks for the help on this. If you email me, I'll send you some stickers :)


Update DLAMBDA to DELTA_LAMBDA to match picca

Also changed output RA and DEC to radians to match picca

can we keep DLAMBDA as well? Or does it break the correlation function code? I'll merge and we can discuss later.


Test branch merge titles

Oh interesting. Seems that GitHub will always add the $username/ before the branch name in the merge commit title.


fix: make logging infrastructure consistent and add logging checks

Related Issues/PRs

#xxx
What changes are proposed in this pull request?
Briefly describe the changes included in this Pull Request.
How is this patch tested?

[ ] I have written tests (not required for typo or doc fix) and confirmed the proposed feature/bug-fix/change works.

Does this PR change any dependencies?

[ ] No. You can skip this section.
[ ] Yes. Make sure the dependencies are resolved correctly, and list changes here.

Does this PR add a new feature? If so, have you added samples on website?

[ ] No. You can skip this section.
[ ] Yes. Make sure you have added samples following below steps.

Find the corresponding markdown file for your new feature in website/docs/documentation folder.
Make sure you choose the correct class estimators/transformers and namespace.
Follow the pattern in markdown file and add another section for your new API, including pyspark, scala (and .NET potentially) samples.
Make sure the DocTable points to correct API link.
Navigate to website folder, and run yarn run start to make sure the website renders correctly.
Don't forget to add <!--pytest-codeblocks:cont--> before each python code blocks to enable auto-tests for python samples.
Make sure the WebsiteSamplesTests job pass in the pipeline.

/azp run

/azp run

Codecov Report

Merging #1755 (10c1819) into master (557470b) will decrease coverage by 85.93%.
The diff coverage is 0.00%.

:exclamation: Current head 10c1819 differs from pull request most recent head 06865a1. Consider uploading reports for the commit 06865a1 to get more accurate results

@@            Coverage Diff             @@
##           master   #1755       +/-   ##
==========================================
- Coverage   85.93%   0.00%   -85.94%
==========================================
  Files         271     271
  Lines       14130   14140       +10
  Branches      733     739        +6
==========================================
- Hits        12142       0    -12142
- Misses       1988   14140    +12152

Impacted Files
Coverage Δ

...re/synapse/ml/cognitive/CognitiveServiceBase.scala
0.00% <ø> (-80.54%)
:arrow_down:

...ynapse/ml/cognitive/anomaly/AnomalyDetection.scala
0.00% <ø> (-81.74%)
:arrow_down:

...gnitive/anomaly/MultivariateAnomalyDetection.scala
0.00% <ø> (-89.92%)
:arrow_down:

...re/synapse/ml/cognitive/bing/BingImageSearch.scala
0.00% <ø> (-89.29%)
:arrow_down:

...crosoft/azure/synapse/ml/cognitive/face/Face.scala
0.00% <ø> (-81.38%)
:arrow_down:

...ynapse/ml/cognitive/form/FormOntologyLearner.scala
0.00% <ø> (-89.80%)
:arrow_down:

...ure/synapse/ml/cognitive/form/FormRecognizer.scala
0.00% <ø> (-84.05%)
:arrow_down:

...e/synapse/ml/cognitive/form/FormRecognizerV3.scala
0.00% <ø> (-88.00%)
:arrow_down:

...oft/azure/synapse/ml/cognitive/openai/OpenAI.scala
0.00% <ø> (-81.25%)
:arrow_down:

...zure/synapse/ml/cognitive/search/AzureSearch.scala
0.00% <ø> (-87.77%)
:arrow_down:

... and 342 more

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

/azp run


Added Link to Device build for React native.

Client had a query for building react client for their own infrastructure.  They were failing on make build-android.
Please make sure you've read the pull request section of our code contribution guidelines.
When filling in a section please remove the help text and the above text.
Summary
Added additional documentation as requested to readme.
Ticket Link
Conversation on Private Channel

Thanks @onepict for the pull request!
Per the CONTRIBUTING.md file displayed when you created this pull request, we need to add you to the list of approved contributors for the Mattermost project.
Please help complete the Mattermost contribution license agreement?
This is a standard procedure for many open source projects. Your form should be processed within 24 hours and reviewers for your pull request will be able to proceed.
Please let us know if you have any questions.
We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

Thanks for the changes @onepict. One of these days, we need to take a look at making the README more useful, but this is a good start


Some files in /mnt appear only 2 bytes long => flurry of errors in /mnt/wslg/stderr.log

Version
Windows version: 10.0.19045.2311
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 22.04
Other Software
No response
Repro Steps
Install Ubuntu 22.04 via .appx package from https://aka.ms/wslubuntu2204 (or perhaps wsl --install Ubuntu).

Good symbolic links in C:\Users\colem\AppData\Local\Microsoft\WindowsApps\
Actual Behavior
See Repro Steps.
Diagnostic Logs
No response

https://stackoverflow.com/questions/71697488/follow-hard-links-reparsepoints-to-files-windows-terminal

is an example of yet another kind of reparse point: an AppX reparse point aka app execution alias aka (internally) AppExecLink.
These are used by UWP applications, which notably includes all applications downloaded from the Microsoft Store.
While they are conceptually similar to symlinks, there are important differences:
The link target isn't necessarily the true target executable; it may just be the generic UWP launcher, C:\WINDOWS\system32\SystemUWPLauncher.exe, such as in the case of Microsoft Edge.
In other words:
You should treat an app execution alias just like any other (regular, self-contained) executable and not as a link to another executable.

It seems that WSL cannot parse this kind of reparse point.

Apparently WSLg is trying to open a remote destop connection, but the file wslg.rdp looks incomplete:
audiocapturemode:i:2
authentication level:i:0
disableconnectionsharing:i:1
enablecredsspsupport:i:0
hvsocketenabled:i:1
remoteapplicationmode:i:1
remoteapplicationprogram:s:dummy-entry

This command from a CMD console opens a small window [RemoteApp] "Starting your app" that never connects.
"c:\Program Files\WindowsApps\MicrosoftCorporationII.WindowsSubsystemForLinux_<IP_ADDRESS>_x64__8wekyb3d8bbwe\msrdc.exe" /v:022336E4-DDD9-4DF6-BDF2-4063D81F69D9 /hvsocketserviceid:401D9BBF-FACB-11E6-BD58-64006A7986D3 /wslg /plugin:WSLDVC_PACKAGE /wslgsharedmemorypath:WSL\022336E4-DDD9-4DF6-BDF2-4063D81F69D9\wslg "C:\Program Files\WindowsApps\MicrosoftCorporationII.WindowsSubsystemForLinux_<IP_ADDRESS>_x64__8wekyb3d8bbwe\wslg.rdp"

@colemar the issue is not the reparsing. If you look at the output when you run fsutil reparespoint query it points to the actual RDP binary located at:
PS C:\> ls 'C:\Program Files\WindowsApps\MicrosoftCorporationII.WindowsSubsystemForLinux_<IP_ADDRESS>_x64__8wekyb3d8bbwe\msrdc.exe'

    Directory: C:\Program Files\WindowsApps\MicrosoftCorporationII.WindowsSubsystemForLinux_<IP_ADDRESS>_x64__8wekyb3d8bbwe

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
-a---           12/1/2022  6:59 PM        2568096 msrdc.exe

In my case is version 1.0.0. Looks like you're on version 1.0.3 already.

Wait, on my system /mnt/wslg/stderr.log refers to /mnt/c/Users/colem/AppData/Local/Microsoft/WindowsApps/MicrosoftCorporationII.WindowsSubsystemForLinux_8wekyb3d8bbwe/msrdc.exe
and
root@VERDE:~# ll '/mnt/c/Users/colem/AppData/Local/Microsoft/WindowsApps/MicrosoftCorporationII.WindowsSubsystemForLinux_8wekyb3d8bbwe/msrdc.exe'
-r-xr-xr-x 1 colem colem 2 Dec  2 20:06 /mnt/c/Users/colem/AppData/Local/Microsoft/WindowsApps/MicrosoftCorporationII.WindowsSubsystemForLinux_8wekyb3d8bbwe/msrdc.exe*

root@VERDE:~# cat '/mnt/c/Users/colem/AppData/Local/Microsoft/WindowsApps/MicrosoftCorporationII.WindowsSubsystemForLinux_8wekyb3d8bbwe/msrdc.exe' | wc -c
2
root@VERDE:~# xxd '/mnt/c/Users/colem/AppData/Local/Microsoft/WindowsApps/MicrosoftCorporationII.WindowsSubsystemForLinux_8wekyb3d8bbwe/msrdc.exe'
00000000: 4d5a                                     MZ

root@VERDE:~# wslpath -w '/mnt/c/Users/colem/AppData/Local/Microsoft/WindowsApps/MicrosoftCorporationII.WindowsSubsystemForLinux_8wekyb3d8bbwe/msrdc.exe'
C:\Users\colem\AppData\Local\Microsoft\WindowsApps\MicrosoftCorporationII.WindowsSubsystemForLinux_8wekyb3d8bbwe\msrdc.exe

Now in CMD.EXE
C:\>dir "C:\Users\colem\AppData\Local\Microsoft\WindowsApps\MicrosoftCorporationII.WindowsSubsystemForLinux_8wekyb3d8bbwe\msrdc.exe"
 Volume in drive C is System
 Volume Serial Number is 16F5-BA47

 Directory of C:\Users\colem\AppData\Local\Microsoft\WindowsApps\MicrosoftCorporationII.WindowsSubsystemForLinux_8wekyb3d8bbwe

02/12/2022  20:06                 0 msrdc.exe
               1 File(s)              0 bytes
               0 Dir(s)  167.923.810.304 bytes free

(zero bytes?)

C:\>"C:\Users\colem\AppData\Local\Microsoft\WindowsApps\MicrosoftCorporationII.WindowsSubsystemForLinux_8wekyb3d8bbwe\msrdc.exe"
(this opens a "Remote Desktop Usage" window nonetheless)

It seems to me that WSL sees the reparse point as a file of 2 bytes size containing only "MZ", while Windows understands the reparse point (at least when launching the process msrdc.exe).

And it seems strange for WSL to refer to C:\Users\colem\AppData\Local\Microsoft\WindowsApps instead of C:\Program Files\WindowsApps.
Even if colem is the user that installed WSL, why?

I see the same symptoms as @colemar describes on this box,
> wsl --version
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>
WSLg version: 1.0.47
MSRDC version: 1.2.3575
Direct3D version: 1.606.4
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.19045.2311

Using either Debian or Alpine (both installed via the MS Store), /mnt/wslg/stderr.log is constantly adding pairs of lines with pid incremented by 2 like (lines wrapped for readability):
[13:23:25.130] <5>WSLGd: Run:104: pid 4875 exited with status 127, /mnt/c/Users/gwhite/AppData/Local/Microsoft/WindowsApps\
/MicrosoftCorporationII.WindowsSubsystemForLinux_8wekyb3d8bbwe/msrdc.exe \
/v:21CDBDE0-A785-4340-B9C0-5E37C2A39CBF /hvsocketserviceid:7E061424-FACB-11E6-BD58-64006A7986D3\
 /silent /wslg /plugin:WSLDVC_PACKAGE \
/wslgsharedmemorypath:WSL\21CDBDE0-A785-4340-B9C0-5E37C2A39CBF\wslg C:\Program Files\WindowsApps\
\MicrosoftCorporationII.WindowsSubsystemForLinux_<IP_ADDRESS>_x64__8wekyb3d8bbwe\wslg.rdp
/mnt/c/Users/gwhite/AppData/Local/Microsoft/WindowsApps\/MicrosoftCorporationII.WindowsSubsystemForLinux_8wekyb3d8bbwe/msrdc.exe: \
line 1: MZ: command not found

When I run a X11 application such as xeyes, the app runs, but the window never appears.
In linux, ...\msrdc.exe appears to be the same MZ two bytes.

In Linux msrdc.exe show as MZ because of the interop. Here is the setting Linux side:
tux@eleven:~$ cat /proc/sys/fs/binfmt_misc/WSLInterop
enabled
interpreter /init
flags: PF
offset 0
magic 4d5a

There is no /proc/sys/fs/binfmt_misc/WSLInterop in this WSL2 debian.   I have PowerShell Core 7.3.0 in Windows and have done Install-Module -Name WslInterop in Windows:
PS C:\Users\gwhite> get-installedmodule wslinterop

Version              Name                                Repository           Description
-------              ----                                ----------           -----------
0.4.1                WslInterop                          PSGallery            Integrate Linux commands into Windows wi…

which is supposed to work with PS Core 7.3.0.

@elsaco
I downgraded WSL <IP_ADDRESS> to <IP_ADDRESS>.
I can confirm your findings about MZ magic signature.
Now WSL is able to start msrdc.exe via the reparse point:
colem@VERDE:/mnt/wslg$ /mnt/c/Users/colem/AppData/Local/Microsoft/WindowsApps/MicrosoftCorporationII.WindowsSubsystemForLinux_8wekyb3d8bbwe/msrdc.exe

If I remember well, version <IP_ADDRESS> was not able to start it. I am reinstalling this version to confirm.

I unistalled WSL <IP_ADDRESS> and installed WSL <IP_ADDRESS> (via Appx package).
WSL <IP_ADDRESS> is also able to start the reparse point, and I don't see anymore the original problem about /mnt/wslg/stderr.log.
I don't know why.
I see other issues in stderr.log, but I suppose these are not relevant:
Missing Wayland requirements for glamor GBM backend
Failed to initialize glamor, falling back to sw
The XKEYBOARD keymap compiler (xkbcomp) reports:
> Internal error:   Could not resolve keysym XF86FullScreen
Errors from xkbcomp are not fatal to the X server
The XKEYBOARD keymap compiler (xkbcomp) reports:
> Warning:          Unsupported maximum keycode 569, clipping.
>                   X11 cannot support keycodes above 255.
> Internal error:   Could not resolve keysym XF86FullScreen
Errors from xkbcomp are not fatal to the X server


Updating SfBO to Teams

Docs Build status updates of commit bcac05b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Set-CsCloudCallDataConnector.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 633962b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Set-CsCloudCallDataConnector.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@dariomws @tseward @get-itips

@tiburd Please merge this PR. Thanks


Desync after enabling altert

All players in multiplayer will be desyncronized after one of the players activates the alert.
Map version: 1.0.0
Mod: 0.1.2

I haven't had the chance to test the mod in MP environment yet. Will do as soon as i can.
In the mean time, can you maybe provide more information.

Does desyncronization happen instantly at mod activation ?
Are spawners consuming pollution ?

Thanks !

Does desyncronization happen instantly at mod activation: yes
It seems to happen only once in a while for each player. After rejoining again, the mod can be activeted without desync for the same player. If another one activates it, desync happens again.
It also seems to happen only when other players are logged in to the server.
Not sure about the spawners consuming pollution at the moment when desync happens. But once the mod is activated, it works really well. But it has to be activated each time joining the server.

Alright, thanks for the clarifications!
I think i know where the bug comes from.
It will be fixed in the next release.

Should be fixed in cc89588a7e3ba63aa26fa94405f2c4ee2a5b7f63.
@MalthusEx I couldn't test the fix in MP, can you tell me if you don't encounter the bug anymore ?


tor: hidden services can bring-their-own-key

Motivation for this change
This change adds a new script to run before Tor starts. For any hidden services, if they have a predefined keyPath, it will be copied and used as the hidden service's hs_ed25519_secret_key.
In this way, you can generate Tor hidden service keys, encrypt them with sops-nix, embed them into an image, and have them securely, automatically boot into serving a given hidden site.
This is particularly nice because you can now effectively predictably automate a NAT-hole punch with a "stateless" static image, in conjunction with any unlocker that sops supports (Azure KeyVault, KMS, GPG, etc).
For reviewers: please advise on other options for how I can prevent copying the private key to the torDirectory. I am using sops-nix, and I'm not sure what the problem was, but I could not get any sort of symlink to work, no matter how much I ensure the permissions were exactly correct.
But, I really don't like copying the private key to disk, especially when sops-nix goes so far out of its way to keep it off disk. I think I may need to try to consult with the Tor folks to see if is particular in this regard.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Maybe it would be better to write a script that uses the Tor socket api to create the service using the key data directly?

note to self, per mic92's suggestion:

https://github.com/NixOS/nixpkgs/blob/790714af65bf03234efce4c6ec9237166697bf86/nixos/tests/slurm.nix#L92
ok, from my perspective you are just copying a new file, which should be doable with systemd-tmpfiles

I have some recollection that someone did this and tailscale more or less fulfills any reason I was ever curious about this. Closing.


metrics-overview: simplify metrics collection

Pull Request Checklist
This is a small refactoring following #95.
General

[x] Update Changelog following the conventions laid out here

[ ] Update README with any necessary configuration snippets

[ ] Binstubs are created if needed

[x] RuboCop passes

[x] Existing tests pass

Purpose
This is a simple refactoring which remove a bunch of duplicated code detected while implementing #95.
The functionality of the plugin is the same otherwise.
Known Compatibility Issues
Output is the same as before.

released: https://rubygems.org/gems/sensu-plugins-rabbitmq/versions/5.4.0


Recruitment performance report bigquery

Context
Add a class that makes requests to the application_metrics_by_provider table in bigquery.
Public methods include:

#provider_data
#national_data

Changes proposed in this pull request
Refactor existing DfE::Bigquery implementation.

ApplicationMetrics is used to get data from BigQuery and save it so it can be used to populate the MonthlyStatistics Report.
The class exposes a number of class methods that each make a request to the Bigquery API
These methods all get called in order for the MonthlyStatistics report to be complete.
These methods all share state in the form of cycle_week value.
The current implementation requires that this value be passed as a parameter to each class level method

Instead we can create an instance of the ApplicationMetrics service, setting the cycle_week as an instance variable.
Instance methods on this class can make individual calls to the Bigquery API.
The response from Bigquery is used to instantiate a nested Result class which separates the request from the response.
It also allows us to reduce the api surface area of the ApplicationMetrics class.
This approach is continued to be used in the ApplicationMetricsByProvider Service.
Guidance to review
Link to Trello card
Trello Ticket
Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated added to the Azure KeyVault

 Create ApplicationMetricsByProvider < BigQuery classes


Code Coverage incorrect when NuGet packages contain pdb's

Environment

VSTS
Project with nUnit testcases

Issue Description
VS 2017 Enterprise 15.8.3 reports a total code coverage of 92.97% while VSTS build of the same project reports a coverage of 48.60%
The difference in the coverage files is the inclusion of foreign nuget packages into the results, where the criteria to have them included seem to be:

Touched by unittest
.pdb present

My assumption would be that both give (nearly) the same result, and certainly would not differ in the affected files to calculate over.
VS result (correct)

VSTS result (incorrect)
Coverage file loaded from build into VS for display

@SartriX you will have to add these to the exclusion filter in the runsettings to prevent code coverage from being calculated for these.
Ideally the platform will not try to understand which is a user dll and which is not. you will have to include/exclude these in the runsettings

@SartriX Is this issue resolved?

I consider the suggested solution a workaround.
It brings a lot of overhead though, since the build-definitions are maintained by a smaller crew. Consider a workplace where not all developers have the same priviliges and checkin policies apply. It'll create the followng process:

a developer decides to require an additional nuget package and is added to the solution
builds will (partially) fail, due to policy on drop in coverage
developer needs to contact another developer with correct priviliges to update the exclusions
meanwhile developer is stranded, because pullrequest won't make it through a succesful build
Worst part of it: since VS does it fine, the developer has no cue up front that he's going to create an issue

From a product quality point of view I would find it unacceptable to have differences between VS and VSTS default behaviour.
Ideally, the platform should only determine coverage on code which has source. Having a .dll or a .pdb is insufficient for the determination.

@SartriX
Can you share following info with us at<EMAIL_ADDRESS>Please mention issue link in the mail.
VSTS info:

Set system.debug variable with value true
Share diagnostics logs and code coverage file.

VS info:

Close VS instances.
Uncomment system.diagnostics element in vstest.console.exe.config. This will set debug mode to true. Read comments in vstest.console.exe.config to do this correctly. You can find this file at location similar to %\Program Files (x86)%\Microsoft Visual Studio\Enterprise\Common7\IDE\Extensions\TestPlatform\vstest.console.exe.config
Start VS and run the tests with code coverage
Share diagnostics logs and code coverage file.

@SartriX Can you please share info asked above?

@SartriX Closing this issue due to no response. Please re-open to follow-up (along with the info asked above.)


Issue#1850: Upgrade our slf4j dependencies from 1.7.24 to 1.7.30

Upgraded dependency version. Ran all tests successfully.
This will resolve #1850.

Thank you for your contribution! :)


#[contract_library_function] allows pub parameters

Aim
contract Foo {
    #[contract_library_method]
    fn foo(x : pub u8) -> u8 {
        x
    }
}

Expected Behavior
An error explaining pub is not allowed on non-entry point functions.
Bug
The function is accepted, except for a pub return type error related to https://github.com/noir-lang/noir/issues/3344
To Reproduce

Installation Method
None
Nargo Version
No response
Additional Context
No response
Would you like to submit a PR for this Issue?
No
Support Needs
No response

@guipublic what is the status of this issue?

The issue  is resolved so I close it.


Determine tensor sharing by tensors not underlying data

Tensor sharing is now determined by underlying data.
This is done to allow shared tensor have different metadata.
But this is causing some delicate issues.

we can never going to know which one is shared to which one until allocated. This losses meaning of sharing a tensor.
Same underlying data now has two meaning: memory clash or tensor is shared

:octocat: cibot: Thank you for posting issue #1619. The person in charge will reply soon.


How building process takes place in angular 6?

I'm new one to angular 6.
I know how to build the application but i don't know the inner process happened while taking build.
May i know which bundling system Angular CLI using and Is there necessary to use external bundling system like rollup, browsify, ...
Further i need to add the tree shaking feature in my application.
please assist me
Thanks,
Buhari Mohamed Ismail

This is not an issue, Best that you redirect support questions to the gitter channel
https://gitter.im/angular/angular-cli
Thanks.

Hello, we reviewed this issue and determined that it doesn't fall into the bug report or feature request category. This issue tracker is not suitable for support requests, please repost your issue on StackOverflow  using tag angular.
If you are wondering why we don't resolve support issues via the issue tracker, please check out this explanation.

I raised the same as in gitter and stackoverflow but no one is responding i'm still waiting for the solution.

Angular CLI is using webpack under the hook. And tree-shaking is provided out of the box.
https://github.com/angular/angular-cli/wiki/build

@BUHARIMOHAMEDISMAIL Thanks for the inquiry, the build system currently uses webpack under the hood.


Is there any way to maximize energy efficiency through software?

Your current example code uses void loop() and pings the phone every one second which I suppose is for the sake of keeping it simple.
Is there any way that I could get a special message on my esp whenever the phone receives a new notification (and then do a request from the esp using the /notification command) so that I don't have to ping it all the time? This looks incredibly wasteful of energy

At the moment no such feature exists in this app, it's in need of some reworking. Personally in my watch project for which this app was designed (https://github.com/Bellafaire/ESP32-Smart-Watch) I poll the phone only once when the watch wakes from sleep. After the first connection the app spawns a background service and will automatically reconnect to the ESP32 as long as its advertising. There's currently no way to check the number of notifications other than to read the full string, I'm thinking of ways to improve this app but I'm not totally sure on which direction to take it quite yet.

This feature is now supported, the ESP32 example shows how to receive a callback on notifications being posted or removed.


error CS1061: 'DataRecognitionClient' does not contain a definition for 'WaitForFinalResponse' and no extension method 'WaitForFinalResponse'

I am trying to call WaitForFinalResponse, but getting error, According to sdk documentation, this method is supported (https://staging.www.projectoxford.ai/Files/Doc/Speech/Windows/html/ff4169e5-6142-558a-92af-48618f592f39.htm)
this.dataClient.WaitForFinalResponse();
kindly help. Thanks

This issue was moved to Microsoft/Cognitive-Speech-STT-Windows#6


Responsive Design for ContributorCard Component using styled-components

This pull request introduces changes to the ContributorCard component to make it responsive using the styled-components library.
Changes include:

Removal of display: flex; on small screens to stack the image and text vertically.
Addition of display: flex; on larger screens to align the image and text horizontally.
Adjustment of the ImageContainer div and the image within it to take up 100% of the width of their parent element on small screens, and have a fixed size of 150px on larger screens.
Use of styled-components to create Card and ImageContainer components with media queries for responsive design.

These changes ensure that the ContributorCard component adapts to different screen sizes, improving the overall user experience on both small and large screens. The use of styled-components allows for more modular and reusable CSS in JS.

Hello Aly,
Feel free to take a look and let me know if everything aligns with the expected changes. Thanks again for your collaboration!

Done


Datasets page - UI formatting issue with long dataset names (Safari)

The UI Datasets page, https://cloud.soda.io/datasets/overview,  doesn't format well for long dataset names (full screen)

Safari

Version 15.3 (176<IP_ADDRESS>.8)

Thanks for filing this issue @jasonicarter. I've opened an issue on the internal backlog of the Soda Cloud team so we can take a look at this! :pray:

This has been resolved by the Cloud Team last week.


How can I pass through arguments from Python to Google App Script?

For this script: https://github.com/gsuitedevs/python-samples/blob/master/apps_script/execute/execute.py
I'd like to pass through SalesID to Google app script's function.
def main(SalesID):
    """Runs the sample.
    """
    SCRIPT_ID = '1WChnVrk5gycQEtumI7mPi5PexXafuhBAWN7-VnBK2aPkFpzMHtUp0cnx' #Actual Google Sheet Sample

Google app script's function:
function myFunction(e) {
var ss = SpreadsheetApp.getActiveSpreadsheet();
var sheet = ss.getSheetByName("Sheet1");
var timestamp = new Date();

sheet.getRange("A2").setValue(e.parameter.SalesID);
sheet.getRange("B2").setValue(timestamp);
}

Sample google spreadsheet: https://docs.google.com/spreadsheets/d/1Z4PAY3CCaRorn5LRdFQKn4-EcAHxwxHJsABzEgsSQk0/edit#gid=0
How can this be achieved? Basically I'd like to be able to pass the SalesID from shell to Google app script to process. Thanks!

Python version 3.7.2
OS CentOS 7

Looks like I need to specify it in the request body:
request = {"function": "myFunction", "parameters": [{"salesID" : 123}]}


ObjectID as _id shows localforage warning

From the localforage code:
// Cast the key to a string, as that's all we can set as a key.
if (typeof key !== 'string') {
    console.warn(key +
        ' used as a key, but it is not a string.');
    key = String(key);
}

Perhaps add an idGeneration flag similar to the Meteor api?

Let me know if this is still an issue in 2.0.0-rc.5

I transitioned my project off meteor.  Closing this issue.


Implementation of BkParallelTest (#227)

Resolved puzzle https://github.com/yegor256/takes/issues/227  in src/test/java/org/takes/http/BkParallelTest.java:32-34 This unit test not implemented but has to be. E.g. you could pass your own a ExecutorService implementation to the ctor and verify execute() calls. But feel free to try another way....

Implemented unit-test

@cyberone Many thanks, I will find someone to review it before we merge

@pinaf this pull request is for you, please review

@cyberone PMD violation:

[ERROR] /src/test/java/org/takes/http/BkParallelTest.java[51-51]: To be compliant to J2EE, a webapp should not use any thread. (DoNotUseThreads)

@cyberone 3 comments above.

@pinaf I'ive accepted 2 of 3 of your comments. Ok for now?

@cyberone sure, let's wait for the CI builds

@rultor merge

@rultor merge

@pinaf Thanks for your request. @yegor256 Please confirm this.

@cyberone it's a wrong design, see my comment above

@cyberone @yegor256 please close this issue

@ypshenychka please, review this task for compliance with our quality rules

@cyberone
According to our QA Rules:

Pull request title explains the problem it is fixing.
Pull request description explains the solution proposed and contains a link to the original ticket it is related to.

Title of this PR is not informative and description is copied from puzzle. Please provide appropriate title and description.

@yegor256
According to our QA Rules:

Messages in a ticket always start with a name of a user they are addressed to.

Please  indicate a person you're referring to in the beginning of your comment.

@davvd @ypshenychka I'm sorry but I have to inform you that this PR will not be merged. The issue https://github.com/yegor256/takes/issues/227 was reassigned from me and closed 26 days ago. There's no need reviewing this PR and making it conform our rules.

@ypshenychka now we have a problem because if you mark the quality as bad I won't get paid :(

@cyberone I was asked to review this ticket, so please make the appropriate corrections so that I could accept the ticket.

@pinaf No worries. I'm just waiting here for @cyberone and @yegor256 to fix the remarks, and ticket will be accepted.

@ypshenychka good for now?

@cyberone Title's good. Please provide appropriate description that explains the solution proposed and contains a link to the original ticket it is related to.

@ypshenychka there's implemented unit-test with a link to a ticket. Is it enough?

@cyberone

src/test/java/org/takes/http/BkParallelTest.java:32-34 This unit test not implemented but has to be. E.g. you could pass your own a ExecutorService implementation to the ctor and verify execute() calls. But feel free to try another way....

This description is copied from puzzle and is barely readable. Leave only "Resolved puzzle #227" and
briefly describe what was done in scope of this PR  - if it was only unit test, so please provide its name and purpose in several words.

@ypshenychka does it fit now?

@cyberone better now. Thanks

@ypshenychka I fixed mine

@yegor256 Thanks

@davvd Quality is acceptable.

@davvd Quality is acceptable.

@ypshenychka thank you, let's try to make it "good" next time

@pinaf just added 10 mins to @ypshenychka (for QA), payment ID is 59038958; Thanks so much! Your account was topped up for 21 mins (transaction ID is AP-33W64962L2411120R, the task took 861 hours and 11 mins); extra minutes for review comments (c=6); +21 added to your rating, current score is: +6596


No checkpoint saved for TM training

Hi,
I am trying to train a TM using the following command:
python nmt_prior.py --config ../configs/transformer/trans.deen_base.yaml --name final.trans.deen_base
The training was good until the end of the last epoch, an error appeared:

Traceback (most recent call last):
File "/home/ubuntu-user/mc35180/lm-prior-for-nmt/models/nmt_prior.py", line 143, in
eval_best(trained_model)
File "/home/ubuntu-user/mc35180/lm-prior-for-nmt/models/../models/nmt_prior_helpers.py", line 121, in eval_best
eval_nmt_checkpoint(trainer.best_checkpoint,
File "/home/ubuntu-user/mc35180/lm-prior-for-nmt/models/../models/translate.py", line 110, in eval_nmt_checkpoint
_base, _file = os.path.split(checkpoint)
File "/home/ubuntu-user/anaconda3/envs/lmprior/lib/python3.9/posixpath.py", line 103, in split
p = os.fspath(p)
TypeError: expected str, bytes or os.PathLike object, not NoneType

And there is no any checkpoints ever saved in the experiment directory. How can I fix this?

I added a CheckpointCallback in line 99 of nmt_prior.py and there is a saved checkpoint now:
  callbacks = [
      LossCallback(config["logging"]["log_interval"]),
      GradientCallback(config["logging"]["log_interval"]),
      ModuleGradientCallback(["encoder"], config["logging"]["log_interval"]),
      SamplesCallback(config["logging"]["log_interval"]),
      EvalCallback(config["logging"]["eval_interval"], keep_best=True,
                   early_stop=config["optim"]["early_stop"]),
      CheckpointCallback(config["logging"]["checkpoint_interval"], # line 99
          only_best=True,
          early_stop=config["optim"].get("early_stop", 100)),
  ]

Is this correct?

After the modification, a new error shows up: "index_select_out_cuda_impl" not implemented for 'Float' during translating.


Extra margin makes buttons look less orderly

The buttons have an extra margin on the left and right that makes everything look less ordered visually.  That is to say, the left button has a larger left margin, and the right button has a larger right margin.

(I'm just describing the visual layout, not the structure of the markup—I realize that it is almost certainly an element encompassing all buttons which has the margins—since it is the visual appearance that is relevant to the average user.)
Compare that to the GitHub editor, where the different elements following the same line creates a sense of visual order:

Here that line is added to make the connection explicit (although viewing it with an explicit line instead of an implicit line makes it look worse):

This is expected behavior, if you don't like it you can disable it by overriding styles. The reason behind it is that the toolbar is focusable. It looks very awkward when the focus outline of the toolbar juts up against the button. The other option is to add the same amount of padding to the editor. Tbh I'm not a designer. Thank you for this and I'll see if I can attach it to #46

@brandondrew Fixed in 0.5.0.


Do not stop if one unit test fails

Currently, our unit test process stops as soon as one test fails thereby making it hard to evaluate what else breaking without actually fixing the error and relaunching test again. We need to first run all tests and then print summary of failed functions.

hey is this  issue still exists? if so can i work on this?

Oh yes, the issue still exist. Feel free to work on it. I was actually referring to this one: https://github.com/google/gcp_scanner/blob/main/src/gcp_scanner/test_unit.py. It fails as soon as the first assert is not satisfied. I'd use unittest.mock module to keep consistency.

oo ok gotcha

@Pavan-Nambi Can I pick this up?

hey @jatinagwal  yeah feel free to do so , i got into some work so wasn't able to do , appologies for delay

I am also interested in this issue if it's open.

@mshudrak I think its best practice to write single assert per test function. If you agree, I can take on the issue and refactor the tests.
I am looking forward to your opinion 🙂


[question]how to create a desktop app

in the old days,I want to create a BS application for users ,but due to media files like video ,pdf can drive the traffic fee blast, I want     user to download this desktop app and install on their own computer, then he can open the deployed url either in browser or just open the desktop app, also if we can compile it into a mobile app would be much better.
Can i achieve this through niva

I'm not sure if I fully understand you correctly: you want a set of code that can be run on both the Web and the Desktop.
My answer: Niva is just a tool for building Desktop apps, you just need to deploy your front-end code (for urls) on your server and use niva to build your Desktop packages. As for building on the mobile side, I think that's what similar tools like React-native do, but not niva

i  See


feat(backend): Configuring GridFs in spring boot application for hand…

@stephane-segning and @mbunwe-victor i think this should have a DBObject which will implement store methods

@Christiantyemele, the implementation of GridFsTemplate handles everything that has to do with the store method.The GridFsTemplate encapsulates the functionality needed to interact with GridFS for file storage

Please write some unit tests and integration tests to ensure this is working

Done


Fresh changes made from scratch.

Hi @Court72 I hope this PR doesn't have any issues. I have redone all the changes from scratch.

@ju-shim
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Court72 - I am not the right writer to set on these docs. Let me find the right contact and then these changes should be reviewed by them and their aliases/usernames should be set to the articles above.

Hi @nithyaramakrishnan , I'll be taking over this PR. I'll do a review and submit any required edits by EOD. @ju-shim I'll update the work items as needed!

@mattmcinnes,
Can you review this old PR and determine whether it needs to be closed or merged?

@mattmcinnes,
Can you help us resolve this aging public-repo PR? Should the changes be made or discarded?
Thanks, PR review team

PR review team emailed author today

PR review team emailed author today

Hi @Court72 any update on this PR? Have the changes been accepted and merged?

I sent a Teams message to the author today.

I sent an email to the content owner today.

I sent email to the content owner today.

#sign-off

@ShannonLeavitt / @Court72 -- PR is approved. Please merge changes when you get a chance!

@nithyaramakrishnan @ju-shim - we can't review and merge this PR until the merge conflicts are resolved.
@nithyaramakrishnan - can you resolve these conflicts and tag me or sign off? thanks!

#hold-off

@Jak-MS I don't understand, when I submitted this PR on May 30th, I had fixed all conflicts. But there was a lot of delay from the team to merge it.

@nithyaramakrishnan - let me see what i can do.

@nithyaramakrishnan - the issue is the metadata in articles/confidential-computing/how-to-fortanix-confidential-computing-manager-node-agent.md which has been changed in the main branch.  Our team can usually just fix this, but for some reason we do not have edit perms on your fork/branch and so you will have to do this.
Can you change the metadata in your file to this:
---
title: How To - Run an application with Fortanix Confidential Computing Manager
description: Learn how to use Fortanix Confidential Computing Manager to convert your containerized images.
services: virtual-machines
author: ju-shim
ms.service: virtual-machines
ms.subservice: confidential-computing
ms.topic: how-to
ms.date: 03/24/2021
ms.author: jushiman
---

After you do that, i think the merge conflicts should resolve. Thanks.

@Jak-MS can you check now?

@nithyaramakrishnan - lots of changes here that we can't fully test because unlike the private repo, we aren't able to stage content here. That said, with the exception of the above suggestions, which you should address in a new PR, I've reviewed this and it should render okay. Please, check the live page to make sure there are no broken images or odd rendering.
@ju-shim - fyi
@nithyaramakrishnan - remember to follow up to make the two suggestions above.
Thanks, I'm going to merge these changes


PICNIC-548: Service Row

Fixes a few issues with the user results where the prettyName of a user could be the same as the keybase username.

Removed scrollview on desktop

Sets margin correctly when service icon is the only thing in the bottom row.

PICNIC-584 is a follow up ticket to share the display logic between people-result and user-result

@keybase/react-hackers bump


fix iddr and oddr pack port bug

in the iddr and oddr, S and R is in one port SR, we should remove one port in pack.

That's merged. Many thanks for the contribution!


"View profile" not working

Describe the bug
The "View Profile" button does not seem to perform any action.
To Reproduce
Steps to reproduce the behavior:

Enter someone's profile
Go to the "Posts" tab
Scroll until you find a repost
Click on the "+" sign
Click on the "View profile" button

Expected behavior
To go to the another person's profile
Screenshots
https://github.com/user-attachments/assets/1960d70e-efa7-4e50-82e8-a17c373fa8d7
Details

Platform: Android
Platform version:
App version: 1.90.1

Additional context

Just tested it on iOS version and it is working fine

Tested on Android. Could not replicate.

I tested again and could not replicate too


[WARNING] Exception encountered when loading plugin: AntiVPN

git:BungeeCord-Bootstrap:1.13-SNAPSHOT:9ea82e9:1351
08:44:02 [INFO] Enabled plugin BungeeTabListPlus version 2.8.8 by CodeCrafter47
08:42:40 [WARNING] Exception encountered when loading plugin: AntiVPN
java.lang.LinkageError: loader constraint violation: when resolving method "org.slf4j.impl.StaticLoggerBinder.getLoggerFactory()Lorg/slf4j/ILoggerFactory;" the class loader (instance of net/md_5/bungee/api/plugin/PluginClassloader) of the current class, org/slf4j/LoggerFactory, and the class loader (instance of net/md_5/bungee/api/plugin/PluginClassloader) for the method's defining class, org/slf4j/impl/StaticLoggerBinder, have different Class objects for the type org/slf4j/ILoggerFactory used in the signature
    at org.slf4j.LoggerFactory.getILoggerFactory(LoggerFactory.java:418)
    at org.slf4j.LoggerFactory.getLogger(LoggerFactory.java:357)
    at org.slf4j.LoggerFactory.getLogger(LoggerFactory.java:383)
    at me.egg82.avpn.lib.ninja.egg82.patterns.events.EventHandler.<clinit>(EventHandler.java:13)
    at me.egg82.avpn.lib.ninja.egg82.sql.SQLite.<init>(SQLite.java:50)
    at me.egg82.avpn.Loaders.loadStorage(Loaders.java:57)
    at me.egg82.avpn.AntiVPN.onEnable(AntiVPN.java:145)
    at net.md_5.bungee.api.plugin.PluginManager.enablePlugins(PluginManager.java:227)
    at net.md_5.bungee.BungeeCord.start(BungeeCord.java:281)
    at net.md_5.bungee.BungeeCordLauncher.main(BungeeCordLauncher.java:56)
    at net.md_5.bungee.Bootstrap.main(Bootstrap.java:15)

That is odd, especially considering I tested it on only a few versions prior and it worked fine. I wonder if they got rid of some stuff without mentioning anything.

Haven't been able to reproduce on my end, sadly. Not sure where that would be coming from :/


Update homepage to use photos from our chapters

Description
Currently, we're using a generic photo at the top of the homepage. @Cara-Jo made the suggestion of using photos from our chapters instead (YES!). See screenshot below of current homepage image FYR.

To Do

[ ] Create image cropped at 1400 x 499 pixels
[ ] Remove stale homepage images from repo (e.g. photo3.png, photo2.png, photo.png)
[ ] Add new image

Will this be driven by our site on its own (e.g. uploads in admin) or maybe an API from an image-storing site? Would be nice to automate/simplify the image upload process while dynamically pulling new photos.

@kstack7 all great suggestions! What do you think would work best?

I guess it depends on the comfort level of the contributors. Built-in is always a great choice, but not sure if putting it in the back-end of the site is intuitive enough? Or if it can work alongside AA? That'd rock. Is space an issue?
APIs are tempting, but always leave room for lag/disconnection.

Hand-picking them is probably ideal for the sake of the launch. We can always whip up some crazy integrated masterpiece when the deadlines are outta the way!

@kstack7 right now we only have one spot where images, other than the chapter leader profile images, are being used: the homepage. I vote that we keep this in mind for future integration, but hold off until we have enough spots,with custom images, to worry about it. Make sense? Sound ok?

Oh for sure...just spitballing beyond the designs to see what it could be. I'd be happy to sniff out some options for whenever we are ready to look at it. Def not a pre-launch project!

I like the idea of having 'hero' images uploadable. It would be great if users could upload an image and then be presented with a way to crop images to the right aspect ratio. I've worked with companies before with uploading images that need to be a specific aspect ratio and found it was helpful for them to have an option to crop on upload.
@juliaelman I don't have a stock pile of GDI images, but I will send a message out to the leaders asking them to send them our way!

@Cara-Jo that is a GREAT idea and love the idea of developing this for future releases. As I mentioned to @kstack7, I think it would be good to keep it simple for now :+1:
Is there a particular newer hero image you'd like to put on the homepage? I think we should just change that in the code and call it a day for now. We're close to release and want to make sure to get this accomplished prior to Galentines Day.

Oh, totally down for doing it simple and dirty.
I'll check my photo archives for some awesome student photos.

I'm going to create an issue with the future implementation of a hero image upload in the admin. That way we can bring the focus of this issue back to finding the most perfect image for the home page.

@Cara-Jo :+1:

@Cara-Jo okay to assign this one to you?

Sure can!!! Let's get it done!

:rainbow: @Cara-Jo

Could we use this one in the meantime? I cropped to 1400 x 499.

+1 having a pool of 5-10 images to have a different one appear on each refresh of the page.
Also @Cara-Jo I took some really solid images of leaders building the site at the summit w your fancy :camera:. The lighting at Meetup HQ was lovely, and I think we might have a good one or two from there. Are they shared somewhere? On Dropbox?

Heyo @corinnepw ...my immediate thought is that the animated text will have to be changed to a lighter color. I do love the idea of using our group photo so maybe a designer would like to play with what looks best?

I’ve tried this image in the past on the home page and it is way too busy for it. It’s better suited for an inside page where there isn’t much going on.
--
CARA JO MILLER | Co-founder | Girl Develop It Boulder/Denver |<PHONE_NUMBER> |<EMAIL_ADDRESS>| girldevelopit.com | @GDIboulder | @GDIDenver | @corrinajo
On March 8, 2015 at 11:08:55 PM, kstack7<EMAIL_ADDRESS>wrote:
Heyo @corinnepw ...my immediate thought is that the animated text will have to be changed to a lighter color. I do love the idea of using our group photo so maybe a designer would like to play with what looks best?
—
Reply to this email directly or view it on GitHub.

Yeah...I thought maybe we had discussed this one before. We'll find one!

@Cara-Jo @kstack7 noted! Can we find a way to get this on the Donate page?

Scratch that — this would be best on About!

I just want to drop my two cents and say that we should make a concerted effort to represent people who are not white on the homepage. Unfortunately, Vermont is the pastiest place on earth so I can't really contribute diverse imagery from my chapter, but I'm hoping we have chapters that can help us with this?

+1000000

@mmcelaney Absolutely agree— we should always strive to show images that are representative of our incredibly diverse communities. We should look for images that show women of color and age diversity.

I think we should reconsider the text layout/design as well since it tends to limit the images in use. I know we tried a few other options, but they only worked well with consistently lighter photo options. Let's explore that more, too :)

I'd just like to chime in and mention that WOCinTechChat has amazing stock photos of actual women of color who actually work in tech, and they're free with attribution! https://www.flickr.com/photos/wocintechchat/albums


Redis rate limiter

Im implementing a rate limiter in my loopback4 project and I have some issues when trying to customize the values for the rate-limit config. Im using the redis-loopback-connector.
For example:
this.bind(RateLimitSecurityBindings.CONFIG).to({
     name: 'redis',
      max: 20, // limit each IP to 20 requests per windowMs
      windowMs: 1 * 1000, // one second
      keyGenerator: rateLimitKeyGen,
    });

Im trying to set the max parameter to 20 and windowMs to 1000 ms, but it looks like the windowMs parameter is being ignored because when Im testing it is using the default value 6000 ms.
Does anyone know how I can fix this?

Seems like a bug. @akshatdubeysf can you put a fix to this asap?

@jhimy-michel can you confirm if this issue is resolved?

@jhimy-michel we have merged the PR that resolves the issue, and we are closing the issue, if you find the issue is not resolved for you, you can reopen this issue.


Correctly parent azure-test-watcher

The java - aggregate-reports pipeline has been failing. The odd thing I noticed in the failure was the version of the maven-site-plugin that was being used was 3.3 which doesn't match the version we're supposed to be using. It took some time to track down where this was getting overridden and it turns out it was from azure-test-watcher not having a parent set to azure-client-sdk-parent. Since azure-test-watcher is the first module listed in pom.xml it ended up causing the version of maven-site-plugin to be whatever the default version is (guessing 3.3) instead of using the 3.7.1.
The fix here is to correctly parent azure-test-watcher but this also requires maven-enforcer-plugin configuration to allow org.junit.jupiter:junit-jupiter and org.junit.jupiter:junit-jupiter-api dependencies.

/azp run java - aggregate-reports

/check-enforcer evaluate

/check-enforcer reset

/check-enforcer evaluate


Generation of procedures with positional parameters

Is there a way to generate by default (globally) interface with positional parameters for the dart functions and not keyword parameters?

Currently there hasn't, but feel free to PR (which may not be super hard: just change f({A b}) to f(A b), or I will try to squeeze some time to implement it!

Yes, I need this feature. It is too bad that rust is so hard to be mastered for me.
I am going to see what I can do. I will learn a lot if I find time to look to frb code.

Take your time! This feature does not need to master rust, just treat Rust as simple scripting language is enough (because we do not need to handle complex lifetime etc in the code generator). If you find any issue just ping me.

At the end, this feature would not be a solution for me : I need to generate some functions with positional parameters and some other with keyword parameters.
For this reason and for some other reasons that I don’t want to elaborate, I decided to implement a simple wrapper above the library.

Ah I see. Feel free to discuss if you think frb can do anything!


wiremock#2004 Make proxy timeout configurable

This PR fixes #2004 by adding a new configuration option to set proxy request timeout in milliseconds. The documentation for the new configuration option is in this PR: https://github.com/wiremock/wiremock.org-sources/pull/10

Thanks!


Add global matchers

close #121

Coverage remained the same at 100.0% when pulling dcea0913fe0efcbb038213536d9b23bafb1e1a8a on global into 426848129f078f2ff3acc3e173a54bd206f8a89f on master.

Coverage remained the same at 100.0% when pulling a8fdc809f5a212166725014db197cd5d134cf9cc on global into 426848129f078f2ff3acc3e173a54bd206f8a89f on master.

Coverage remained the same at 100.0% when pulling 98db04c23f2625c0981671939d57a16c3ae9ab97 on global into 426848129f078f2ff3acc3e173a54bd206f8a89f on master.


Add travis and coveralls badges

...so we are reminded that more coverage would be nice ;-)

Coverage remained the same at 33.47% when pulling 402cf3a3b3b082d45ee7323cea06aa633a120c40 on zimeon:badge into cf2856409949cc438c19434a37a25ef06dba3810 on IIIF:master.


feat: deploy ralph with uvicorn

Description
This PR updates the Ralph deployment to use Uvicorn, updates the logging configuration of include the timestamp and removes the unused ralph-job

Thanks for the pull request, @Ian2012! Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
Please let us know once your PR is ready for our review and all tests are green.

@Ian2012 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.


WSANOTINITIALISED 10093 Error Code

Dependencies Used

Visual Studio 2019
Windows 10
ggpo.lib
ggpo.dll
SDL2

Summary
When I attempt to build ggpo as a .dll and .lib file to use as a dependency for my own project, I end up receiving the following error below:

Looking at the code snippet below (udp.cpp):
void
Udp::SendTo(char *buffer, int len, int flags, struct sockaddr *dst, int destlen)
{
   struct sockaddr_in *to = (struct sockaddr_in *)dst;

   int res = sendto(_socket, buffer, len, flags, dst, destlen);
   if (res == SOCKET_ERROR) {
      DWORD err = WSAGetLastError();
      Log("unknown error in sendto (erro: %d  wsaerr: %d).\n", res, err);
      ASSERT(FALSE && "Unknown error in sendto");
   }
   char dst_ip[1024];
   Log("sent packet length %d to %s:%d (ret:%d).\n", len, inet_ntop(AF_INET, (void *)&to->sin_addr, dst_ip, ARRAY_SIZE(dst_ip)), ntohs(to->sin_port), res);
}

I end up getting a SOCKET_ERROR when the sendto(...) function is called. It turns out the error I get back is error code: 10093. Based on Microsoft's documentation, error code  https://docs.microsoft.com/en-us/windows/win32/winsock/windows-sockets-error-codes-2, error code 10093 means that Win Sockets dependency hasn't been initialized yet.
Steps to Reproduce:

The following simplified code will reproduce the issue:

// assume callbacks, game state, non game state,  and port numbers are already defined somewhere else

const int NUM_PLAYERS = 2;
GGPOSession *ggpo = NULL;

GGPOSessionCallbacks cb = { 0 };
cb.begin_game      = vw_begin_game_callback;
cb.advance_frame	 = vw_advance_frame_callback;
cb.load_game_state = vw_load_game_state_callback;
cb.save_game_state = vw_save_game_state_callback;
cb.free_buffer     = vw_free_buffer;
cb.on_event        = vw_on_event_callback;
cb.log_game_state  = vw_log_game_state;

ggpo_start_session(&ggpo, &cb, "rollback_tech_demo", NUM_PLAYERS, sizeof(int), localPort);

ggpo_set_disconnect_timeout(ggpo, 3000);
ggpo_set_disconnect_notify_start(ggpo, 1000);

// Calls to ggpo_add_player() made but omitted to keep simple

int maxTimeIdleMillis = 6;
ggpo_idle(ggpo, maxTimeIdleMillis);

// sends input request over the network using UDP protocol internally
// Looks like this fails because Winsockets hasn't been initialized yet.
ggpo_add_local_input(ggpo, ngs.local_player_handle, &localInputsThisFrame, sizeof(localInputsThisFrame));

ggpo_synchronize_input(ggpo, (void *)allInputs, sizeof(int) * NUM_PLAYERS, &disconnectFlags);

ggpo_advance_frame(ggpo);

I have proposed a fix in a following PR: https://github.com/pond3r/ggpo/pull/69


ARROW-7343: [Java][FlightRPC] prevent leak in DoGet

This changes ArrowMessage and FlightService/FlightClient so that if gRPC fails to actually write the message to the stream (as it occasionally does), we don't leak the memory associated with ArrowMessage. It also fixes a case where clients could block indefinitely in DoPut.

Well that's embarassing, the test that we don't leak memory leaks.

Let's hope this fixes it! There are some code paths where gRPC throws, causing us to skip cleanup.

Wow. I find it funny that Java needs some massaging for this where C++ would do fine :-)

cc @emkornfield for possible review.

Mostly nits and some questions on potentially making new unit tests more predictable.  I'm not an expert on flight/java grpc but I didn't see anything tha caught my eye.s

@pitrou the problem is try to jam manual resource management :(

Thanks for the comments, hopefully all addressed. I simplified the tests as they did not need so many loops, as Micah pointed out.

Wow. I find it funny that Java needs some massaging for this where C++ would do fine :-)

Yes, I find that refcounting without RAII is error-prone, especially with Flight where we do not control the entire stack (and have to do weird things like this as a result).

Alright, I've updated as per Jacques's suggestions and un-ignored the flaky tests...sometime I need to dig through what is making them still flake.

@jacques-n just to follow up, how does the new approach here look to you?


[release/v0.8.0] Implement choreo specific functionality in keda components

This PR adds the choreo specific functionality and bug fixes to the branch release/v0.8.0. These changes are already implemented in the main branch which is currently being used. Following fixes are added with this PR.

interceptor: request queue cooldown changes
interceptor: upstream retries fix for 503
interceptor: count middleware host validation
scaler: scaler queue tick duration (KEDA_HTTP_SCALER_STREAM_INTERVAL_MS will be used separately instead of hard coding the values)
interceptor: add cilium annotations to deployment manifest

Fixes #

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


Fixed

Hello,
I'm trying to buy floats with my birthday values; however, whether I'm trying to find one, it takes ages to go through all of the pages for each skin. Could you please tell me if there's a way to make the extension show more than 100 listings on one page or, even better, when the float filter is used, hide the listings that didn't meet the requirements and show only the ones that met. I will be very thankfully for your help^^

Please close the issue instead of renaming it to "Fixed".


add documentation for the vision scripts

documentation for the scripts in https://github.com/UHHRobotics22-23/marimbabot/tree/main/marimbabot_vision/scripts


Writing to Terminal with \n inconsistent on Windows vs Linux and vs other libraries

On Windows, Terminal.Out/OutLine will not do a carriage return when receiving a string with embedded \n in it. Given this code:
Vezel.Cathode.Terminal.OutLine("Line 1\nLine 2");
On Windows the output is:
Line 1
      Line 2

If I make that \n into a \r\n, then it does the right thing.
Line 1
Line 2

If I run the \n version on Linux, then it also is correct. Only on Windows is it wrong.
Other libraries also do the right thing. I tested with both Spectre and .NET BCL. On Windows and Linux, with just \n, either of these is also correct:
Spectre.Console.AnsiConsole.WriteLine("Line 1\nLine 2");
System.Console.WriteLine("Line 1\nLine 2");

I also noticed that if I change the System.Console test to include a bit of Cathode, it starts doing the wrong thing.
var _ = Vezel.Cathode.Terminal.System;
System.Console.WriteLine("Line 1\nLine 2");

I know the analyzer blocks this because they don't play well together, but simply causing the WindowsVirtualTerminal to get constructed alters the meaning of \n for the system console.

Ok it looks like the issue is that DISABLE_NEWLINE_AUTO_RETURN is being set in the native driver.
Since this is deliberate, I'm wondering why?
I can work around the issue for myself by flipping the bit back off after ensuring the native console has been initialized, but what might I break when I do this? (Guess I'll find out!)

Can you confirm whether clearing the DISABLE_NEWLINE_AUTO_RETURN flag actually helps? That would be surprising to me since it's a flag meant to improve compatibility with Unix terminals in regards to writing in the bottom-rightmost cell of the screen.

I was actually aware of this issue since the beginning of the project. I recall that I concluded that it was an unfixable platform difference. How or why I concluded that, though, I don't exactly remember. But let me try...
I think it comes down to the fact that CR and LF have distinct functions in a terminal: CR moves the cursor to the leftmost cell without changing its vertical position, and LF moves the cursor down a cell without changing its horizontal position. It just so happens that on Unix platforms, the ONLCR flag is set as part of the default cooked mode, so LF gets translated to CR + LF by the kernel terminal driver, letting you get away with just LF (but only in cooked mode!). The Windows console does the ~same translation by default, but I believe that changes once ENABLE_VIRTUAL_TERMINAL_PROCESSING is in play. Unfortunately, that flag (and its input equivalent) is a weird mix of cooked mode and raw mode -- which happened because the developers, perhaps rightly, expected most folks to want raw mode -- and I believe there wasn't a good way to enable both VT100 support and retain the LF -> CR + LF translation.
... But I could also just be completely misremembering.
An easy workaround is to just use Environment.NewLine, or if you're not in control of constructing the original string, just call str.ReplaceLineEndings(). This is arguably the right thing to do for portability anyway.

This code indeed works around the problem for me:
        using var handle = Win32_PInvoke.CreateFile(
            "CONOUT$",
            (uint)(GENERIC_ACCESS_RIGHTS.GENERIC_READ | GENERIC_ACCESS_RIGHTS.GENERIC_WRITE),
            FILE_SHARE_MODE.FILE_SHARE_READ | FILE_SHARE_MODE.FILE_SHARE_WRITE,
            null,
            FILE_CREATION_DISPOSITION.OPEN_EXISTING,
            0,
            null);

        Win32_PInvoke.GetConsoleMode(handle, out var consoleMode);
        consoleMode &= ~CONSOLE_MODE.DISABLE_NEWLINE_AUTO_RETURN;
        Win32_PInvoke.SetConsoleMode(handle, consoleMode);

I can go find everywhere I'm using the Terminal specific streams, or one of the Terminal 'out' functions, and run them through helper functions instead that do the newline replacement. But I'd rather just set the mode once, as I did above. It's much less error prone.
Indeed this is a strange situation. I've written loads of little cli tools over the years for Windows Mac and Linux, and have always (*) used plain old \n. All my tooling, the company VCS's, etc. have been set to unix EOL. I normally strip \r's and avoid direct/indirect use of Environment.NewLine, because those \r's cause portability annoyances in practice. But here, I need to add them back in to work right!
Philosophy aside, the main argument I have is that it's different from System.Console and Spectre, and therefore surprising. Also only shows up with embedded newlines, which may be a less common case than people hitting OutLine (this is how I came across the issue today). I'm imagining the Mac user who is now trying out their cool little cli tool on Windows, only to find the most surprising behavior..
(* Ok maybe only in the last 15 years, when Windows dev tooling became more unix friendly.)

One clarification:

it's different from System.Console and Spectre, and therefore surprising

I don't mean to say that it shouldn't have the ability to be different. Just that the default behavior is different. If it's cooked because that is what people expect (I think I saw that in a comment in your code, and agree), then I'd say that people also generally expect that \n will end up on with an implicit \r as well.

OK, the fact that DISABLE_NEWLINE_AUTO_RETURN causes this is actually super surprising to me and not at all what I would have expected.
I wonder if this is a bug in the console host. :thinking:
I'll need to do some more investigation and see what can be done. I do agree that it isn't ideal; consistency between platforms is preferable.

OK. Yeah. The docs for that flag are just completely super wildly wrong. And the funny part is that I filed a bug for it! I just completely forgot, apparently.
The TL;DR of that bug tail seems to be that we should set this flag for raw mode and clear it for cooked mode, similar to how we set ONLCR for cooked mode and clear it for raw mode in the Unix driver.

@scottbilas v0.13.25 has the fix and should appear on NuGet soon™.

Amazing, at multiple levels. :D


Fix Knative integration in AES

Prior to this commit, the mappings being generated from Knative Ingress resources did not have a namespace assigned to them. Also, the namespace was being fetched from the mapping dict without .get() method. This PR fixes both of these issues to fix Knative integration.

BTW nice catch! and mea culpa for breaking it. 🙁


fix: default the full param of AV._encode to true

Codecov Report

Merging #455 into next will increase coverage by 0.13%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##             next   #455      +/-   ##
========================================
+ Coverage   65.87%    66%   +0.13%
========================================
  Files          32     32
  Lines        2725   2724       -1
========================================
+ Hits         1795   1798       +3
+ Misses        930    926       -4

Impacted Files
Coverage Δ

src/av.js
81.34% <100%> (+2.37%)
:white_check_mark:

src/file.js
73.33% <0%> (-0.61%)
:x:

src/object.js
81.75% <0%> (+0.06%)
:white_check_mark:

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 3676b34...21b3bc5. Read the comment docs.


Updates dependencies

fixes #1456 (that second part mentioned here https://github.com/ManageIQ/manageiq-ui-service/issues/1456#issuecomment-407077066)

¯_(ツ)_/¯

Now the specs timed out... another go...


Unrecognized BigQuery alias for unquoted table reference

When querying an unquoted table reference, BigQuery automatically provided alias for the table name in case insensitive manner.
Example:
SELECT
  BLOCKS,
  blocks
FROM
  bigquery-public-data.crypto_bitcoin.blocks
WHERE
  timestamp_month = "2024-09-01"
LIMIT
  1

However, when parsing such query I got exception saying BLOCKS is not a recognized name:

This won't work on quoted table reference, for example BigQuery will refuse to execute this query:
SELECT
  BLOCKS
FROM
  `bigquery-public-data.crypto_bitcoin.blocks`
WHERE
  timestamp_month = "2024-09-01"
LIMIT
  1

So, I think this error is caused by the toolkit automatically adding quotes to all table reference. I'm not sure about this, haven't look into it in more detail.
Anyone available to check and fix is really appreciated. Thanks.

Confirmed automatic quoting name paths on this line:
https://github.com/GoogleCloudPlatform/zetasql-toolkit/blob/e196efea5d8c70491dffd95b343e5c23bf9a4b13/zetasql-toolkit-core/src/main/java/com/google/zetasql/toolkit/ZetaSQLToolkitAnalyzer.java#L210
Tried to comment it but it ends up making the parser not recognizing the table name. I'm thinking of other workaround.

Tried this workaround by creating a statement rewriter and adding an alias to all table path expression that doesn't currently have alias. Since BQ automatically created alias by using the table name, we can do the same thing here.
I won't open a PR for this, because I believe the core package can do something better than rewriting the query. Therefore I will keep this issue open for now.
Here's the code that I use to add alias. You can call AddAliasForTableName.rewrite( {{ your query here }} ) before parsing your query.
import com.google.zetasql.Parser;
import com.google.zetasql.parser.ASTNodes;
import com.google.zetasql.parser.ParseTreeVisitor;
import com.google.zetasql.toolkit.ParseTreeUtils;
import com.google.zetasql.toolkit.StatementRewriter;
import com.google.zetasql.toolkit.catalog.bigquery.BigQueryReference;
import com.google.zetasql.toolkit.options.BigQueryLanguageOptions;
import java.util.ArrayList;
import java.util.HashSet;
import java.util.List;
import java.util.Objects;
import java.util.stream.Collectors;

public class AddAliasForTableName {
  private static StatementRewriter.Rewrite createRewrite(int from, int to, String content) {
    return new StatementRewriter.Rewrite(from, to, content);
  }

  private static StatementRewriter.Rewrite createRewrite(ASTNode node, String content) {
    return createRewrite(
        node.getParseLocationRange().start(), node.getParseLocationRange().end(), content);
  }

  private static String tableNameFromTablePath(ASTNodes.ASTTablePathExpression expression) {
    return expression.getPathExpr().getNames().stream()
        .map(ASTNodes.ASTIdentifier::getIdString)
        .collect(Collectors.joining("."));
  }

  private static boolean shouldAddAlias(ASTNodes.ASTTablePathExpression expression) {
    String tableName = tableNameFromTablePath(expression);
    return (Objects.isNull(expression.getAlias())
        && BigQueryReference.isQualified(tableName)
        && !BigQueryReference.isWildcardReference(tableName));
  }

  private static List<StatementRewriter.Rewrite> addAlias(ASTNodes.ASTFromClause fromClause) {
    ArrayList<StatementRewriter.Rewrite> rewrites = new ArrayList<>();

    HashSet<ASTNodes.ASTTablePathExpression> expressionsToAlias = new HashSet<>();

    fromClause.accept(
        new ParseTreeVisitor() {
          @Override
          public void visit(ASTNodes.ASTTablePathExpression node) {
            if (shouldAddAlias(node)) {
              expressionsToAlias.add(node);
            }
          }
        });

    expressionsToAlias.forEach(
        expr -> {
          String tableNameStr = tableNameFromTablePath(expr);
          BigQueryReference ref = BigQueryReference.from("", tableNameStr);
          rewrites.add(createRewrite(expr, tableNameStr + " AS " + ref.getResourceName()));
        });

    return rewrites;
  }

  private static List<StatementRewriter.Rewrite> rewriteSelect(ASTNodes.ASTSelect select) {
    ASTNodes.ASTFromClause fromClause = select.getFromClause();
    return addAlias(fromClause);
  }

  public static String rewrite(String query) {
    ASTNodes.ASTScript parsedScript = Parser.parseScript(query, BigQueryLanguageOptions.get());

    List<ASTNodes.ASTSelect> selects =
        ParseTreeUtils.findDescendantSubtreesWithKind(parsedScript, ASTNodes.ASTSelect.class);

    List<StatementRewriter.Rewrite> rewrites =
        selects.stream()
            .flatMap(select -> rewriteSelect(select).stream())
            .collect(Collectors.toList());

    return StatementRewriter.applyRewrites(query, rewrites);
  }
}


Update ingress api versions

The Ingress object is being deprecated in the near future in favor of the gateway object. We should update our Ingress definitions to use the new standard.

This was a misundertanding. The ingress object is not being deprecated, but support for the beta API is.

Migrate to use the Stable (generally available) API versions before your clusters are upgraded to Kubernetes v1.22.
Hello Google Kubernetes Engine (GKE) Customer,
We are writing to let you know that as part of the Open Source Kubernetes lifecycle, previously deprecated Beta APIs in v1.22 have been removed in GKE. As Kubernetes evolves, alpha and beta APIs can be changed on a minor version boundary. When the v1 APIs graduate, the older alpha and beta APIs are deprecated and eventually removed. The beta APIs removed in 1.22 were deprecated in favor of their generally available replacements between 2018 and August 2020.
What do I need to know?
As of Kubernetes 1.22, API clients will no longer be able to use previously deprecated Beta API versions. Manifests using those versions will no longer be able to be applied. Note that all previously persisted objects remain functional.They can be read and updated via the new API versions, before and after the Kubernetes 1.22 upgrade.
This change is part of the project’s policy that once the v1 APIs graduate to the next level, the old alpha and beta APIs are deprecated and eventually removed.
Software known to be impacted, but not limited, by these removed APIs include:
Anthos Service Mesh 1.10.3 and below
Istio 1.9 and below
The GKE release schedule outlines the cluster upgrade schedule.
If we are able to detect usage of deprecated beta APIs in Kubernetes v1.22, we will pause automatic upgrades until March 2022 for GKE clusters v1.21 to ensure seamless transition from previously deprecated beta APIs in v1.22 to generally available (GA) APIs.
Please note, regardless of whether your cluster is enrolled in a release channel or not, normally cluster control planes are always upgraded on a regular basis. For an additional level of control over when an auto-upgrade can occur (or must not occur), you can configure maintenance windows and exclusions.
Even if your clusters are running older Kubernetes versions, we urge you to start migrating to v1 stable APIs that are available in earlier Kubernetes versions as outlined in this email.
What do I need to do?
Migrate to use the GA API versions before your clusters are upgraded to Kubernetes v1.22 to ensure your API clients and resource manifests can access and update API resources without interruption. More information is available from the GKE v1.22 deprecated API documentation and from the Kubernetes project. Clusters with Google's Cloud Operations suite enabled can use Admin Activity log query to show the use of 1.22 APIs.
For Anthos Service Mesh upgrades, see Anthos Service Mesh version 1.10 documentation for version-specific upgrade instructions.
For Istio, migrate to the Google-supported Anthos Service Mesh or see Istio upgrade instructions at istio.io.
Your projects below use the deprecated beta APIs and will require you to migrate to the GA API versions before your clusters are upgraded to Kubernetes v1.22:
Biosimulations: (biosimulations)
biosimulations
Need more information or help?
You can learn more about how cluster upgrades work here. More information is also available from the GKE v1.22 deprecated API documentation and from the Kubernetes project.
If you have any questions or require assistance, please reply to this email to contact Google Cloud Support.
Thanks for choosing Google Kubernetes Engine.

Done for the dev branch. Same thing can be copied over for prod once #32 is merged