Create first draft of tutorial

[x] Go through cdalton's tutorial video && write up rough README.md to walk through students
[ ] Refine README.md with code solution blobs && emojis
[ ] Create solutions file
[ ] Fork cdalton's repo, or just use the foundry mix repo, and get the solutions file to work with make test

At this point, the strategy is working with a local mainnet fork.

[ ] Incorporate Scaffold-ETH v1 into the mix

Deploy localhost, edit deployment scripts to make it work with yearn strategies (may have to pull reference from the deploy scripts used with foundry and convert them over to whatever format is used for ScaffoldETH and hardhat)
Troubleshoot until the yearn strategy solutions show up in the troubleshooting UI for localhost
Test with its function calls && come up with a good workflow to showcase how it works

Refining Tutorial.md (renaming to README.md && replacing the old one):
Current thoughts:

[ ] Get solutions file working w/ backend (mainnet local fork testing - make test)
[ ] Replace README.md w/ Tutorial.md && write up copy so it makes sense
[ ] Add emojis && clean formatting
[ ] Get Austin's approval to create a new repo under scaffold-eth for DeFi Sub Branch stuff

New repo create  where yearn strategy tutorials will live. This is because:

Yearn tests written with their foundry_strategy_mix should be used when teaching strategy writing
Good to maintain consistency with yearn strategies that actually use foundry_strategy_mix by using this template repo

Creating a fork and PRing to @charlesndalton's repo was considered but I decided to make a new repo since:

foundry_strategy_mix may be updated since @charlesndalton's tutorial

easier to upkeep our own repo that is based off of the foundry_strategy_mix

[ ] WIll need to consider whether or not we need to see how to bring in Scaffold-ETH into this new repo, or make another new repo that integrates yearn's foundry_strategy_mix into its framework.

Moved issue to new repo.


Add usage of SRT subtitles to the VideoActivity [8-16h]

[ ] Add usage of SRT subtitle files to https://github.com/literacyapp-org/literacyapp-android/blob/master/app/src/main/java/org/literacyapp/content/task/VideoActivity.java
[ ] Display subtitles below video
[ ] Highlight word by word as they are spoken

@shashank7200 See http://stackoverflow.com/questions/13422673/looking-for-a-working-example-of-addtimedtextsource-for-adding-subtitle-to-a-vid


Update WFRP4Charsheet.html

Include the name of the sheet you made changes to in the title.
Changes
Changed a few minor errors and omissions that was necessary to fix. Primarily the setting of rolling WS or BS directly for weapons instead of skills.
Provide a few comments about what you have changed.
Additional comments
Anything else we should know?.
Roll20 Requests

If changes fix a bug or resolves a feature request, be sure to link to that issue.
For pull request that change multiple sheets please confirm these changes are intentional for all sheets. This will help us catch unintended submissions.
When updating existing sheets if you are changing attribute names please note what steps you have taken, if any, to assist players in keeping their data.

Thanks for the helpful comment.


Fix utils.combine_frames for Series with different level of indexes.

This fixes the bug at https://github.com/databricks/koalas/pull/923#discussion_r334500746.

Codecov Report

Merging #926 into master will decrease coverage by 2.24%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #926      +/-   ##
=========================================
- Coverage   94.45%   92.2%   -2.25%
=========================================
  Files          34      34
  Lines        6369    6367       -2
=========================================
- Hits         6016    5871     -145
- Misses        353     496     +143

Impacted Files
Coverage Δ

databricks/koalas/utils.py
97.26% <100%> (-0.68%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
24.54% <0%> (-72.73%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
78.04% <0%> (-7.32%)
:arrow_down:

databricks/conftest.py
91.48% <0%> (-6.39%)
:arrow_down:

databricks/koalas/groupby.py
89.34% <0%> (-2.05%)
:arrow_down:

databricks/koalas/frame.py
94.52% <0%> (-1.53%)
:arrow_down:

databricks/koalas/namespace.py
85.57% <0%> (-1.26%)
:arrow_down:

databricks/koalas/plot.py
93.33% <0%> (-0.96%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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


Link broken for download of Sept-2019 PBID files

Hi, the links for the downloads of Sept-2019 PowerBI Destkop installation files are broken (404). August are there.
Thanks!

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

ID: 93f23670-d83a-dffc-c17a-8cef53d0de7b
Version Independent ID: 1b63f449-9fbd-3544-bee5-17c70fe55229
Content: Previous monthly updates to Power BI Desktop - Power BI
Content Source: powerbi-docs/desktop-latest-update-archive.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment, @zenzeinet I've assigned this issue to the doc author for review.


Enable OpenMP for armv8 builds

Description
Enabled OpenMP in the dockcross/arm image
Checklist
Essentials

[x] This is a minor PR
[x] Changes are complete

Changes

[x] Made a code change and opened a PR for dockcross
[x] Build the change and uploaded it to mxnetci/dockcross-linux-arm64 dockerhub with an updated tag
[x] Reused this tag in this PR
[x] Enabled OpenMP for MXNet armv8 build

Did we test this?


Wrong arrays comprasion

"get_diff" returns unnecessary :update to arrays comprasion
old = [1, 2, 3]
new = [1, 2, 3, 4, 5]
JsonCompare.get_diff old, new
=> {:append=>{3=>4, 4=>5}, :update=>{3=>4}}

I think it is what #9 says, so it is duplicated issue.


Need for contributing file

I have searched the repo but could not find it.This is a popular application it has over 13k stars, it's time that we have a contributing.md file. Off the top of my head it can talk about formatting the code and adding a changelog entry. It saves new people time and that's the first thing some people search for in a repository. Thoughts?

I think this is a great idea and don't see how it would harm anyone.
I can't say what exactly I'd expect to be inside it right now, but maybe my checklist for code reviews can help with some baselines what people should watch out for.
FUNCTIONALITY: Does it actually work? (includes `cargo test`)
PERFORMANCE: Does it work good enough for Alacritty?
STYLE: Is the code clean?
DOCUMENTATION: Is the code documented?
TESTS: Are there any test that could be added?
BENCHES: Are there any benches that could be added?
CHANGELOG: Is an entry present?

I'll usually make sure I've gone through all of these points before any PR is merged. So someone who is contributing should probably check these too to reduce the potential back and forth. Note that these are just quick notes for myself and shouldn't be taken literally the way they're presented here.


Input and output claims need to be distinct

Tested claims transformation with same claim as input and output, it doesn't get updated

When we give the output claim as unique, it works.

@Karthikmuthiahr : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@yoelhor - fyi
#sign-off


Addition of basic auth support

Following the pattern defined in https://github.com/splunk/splunk-sdk-ruby/pull/43, I have implemented Basic Auth in this PR.
from splunklib.client import Service

service = Service(host="splunk.myhost.net", port=8089, scheme="https", version=6.3,
                  basic=True, username="steve", password="puppydoge")

service.indexes['test'].submit("My message!", host="localhost", sourcetype="text")

@itay any idea what I'm doing wrong getting this test set up?
https://travis-ci.org/splunk/splunk-sdk-python/jobs/98574591#L602

@itay ^

Hey @matthewsullivan-wf - sorry it took so long to reply. I finally had some time to look today.
Anyway, I found the issue with your test. I put a comment on the specific line, but I wanted to ask you how you wanted to proceed. I can merge your PR once that's fixed (and if you want to rebase it to the tip of develop and squash your commits), or I can just fix it and get that merged. I wanted to give you the option of doing it yourself so you get "credit" for the contribution on GitHub, since you did the hard work :)
Let me know how you want to proceed.

@itay appreciate the consideration for credit, but I don't care - I just need it fixed so we can start using the SDK at my place of work :)  Go ahead if you've got everything staged that you need.

Superceded by #133


Shallow Water demo, and water normal methods

Is the shallow water demo functional?  HTML5, Mac, and Flash targets aren't rendering the water, and mouse interaction is freezing the water instead of rippling:

In general, water normals seem to have issue on c++ targets.
In my OpenFL Away3D Terrain Demo, water is spectacular on HTML5 and Flash targets, but fails to render on c++.

Water implemented as:
    waterMethod = new SimpleWaterNormalMethod(
        Cast.bitmapTexture("assets/water/water_normals.jpg"),
        Cast.bitmapTexture("assets/water/water_normals.jpg"));

    waterMaterial = new TextureMaterial(new BitmapTexture(new BitmapData(512, 512, true, 0xaa404070)));
    waterMaterial.alphaBlending = true;
    waterMaterial.lightPicker = lightPicker;
    waterMaterial.repeat = true;
    waterMaterial.normalMethod = waterMethod;
    waterMaterial.addMethod(new EnvMapMethod(cubeTexture));
    waterMaterial.specularMethod = fresnelMethod;
    waterMaterial.gloss = 100;
    waterMaterial.specular = 1;

Are these known issues?
Probably unrelated, but the Terrain Demo c++ target is showing a corrupt buffer:

Configuration:
away3d: 5.0.1
hxcpp: 3.4.64
lime: 4.0.2
openfl: 4.9.1

Using the latest dev builds of lime and openfl as of today, I managed to get the Terrain demo up and running fine targetting C++ & neko (on OSX at least).

Could you check against the latest dev builds?
As for the Shallow water demo, looking at the demo code, it appears there is a dependency PixelBender shaders to manipulate the water ripples. These currently aren't implemented in OpenFL and the embeds are commented out so wouldn't even work on the flash target.
https://github.com/openfl/away3d-samples/blob/master/advanced/ShallowWaterDemo/Source/shallowwater/ShallowFluid.hx#L65-L75
So until PixelBender shaders are implemented or the shaders are re-worked in a more OpenFL way it's not going to work as in the original demo.
I'll leave this open for now so it's still on the radar.

I think there's a regression in Lime 4.x that I'm trying to resolve, causing the corruption issue

@jgranick One thing I've seen - probably not related though - is that performance (having tried a few of the away3d samples) of C++ and Neko on the mac is very bad in comparison to webgl - even non-debug versions. ShallowWater (on old 2009 MacBookPro) on Neko (non-debug) takes ages to start, may 20-30secs of black screen then 1-2FPS, C++ (debug version) gives 15FPS whilst HTML5 gives a pleasant 52FPS (debug as well).
I've yet to look into potential reasons why but struggled to create an xcode project to debug with. Need to look into using hxScout again as well - it's been a while.
Have you seen similar poor performance on C++ or Neko targets?

The latest HXCPP and Lime work with HxScout, use -Dtelemetry when you build to enable it.
The Stage3D code has been undergoing accuracy focus, and likely has some low-hanging fruit for performance optimization. I've seen it vary on the system, it could also be an unoptimized code path hitting Cairo too often. Is the demo you're running using AwayFPS or some other object with shapes?

Yes - using AwayFPS. I'll try and get some time to dig into it and see where the bottleneck(s) are.

I found a regression in render-to-texture support in OpenFL, here's the TerrainDemo after the fix:

...and testing the C++ target on your sample as well:

All around awesome - thanks to you both.
Per performance of c++ targets on mac, in my experience they load quick and are responsive achieving 60 fps.  Though I haven't pressed any limits, I haven't observed any substandard performance in either debug or release builds.
Not quite up to speed, but I can take a stab at profiling on Mac.
OS X El Capitan v10.11.6, retina, 2.4 GHz Intel Core i7, 8 GB 1600 MHz DDR3, NVIDIA GeForce GT 650M 1024 MB

@Greg209 the performance might be slower HXCPP debug performance, I have optimizations in Lime and OpenFL that flag core classes for release even in debug builds, which should help (but only workshows in Haxe 3.4+)


SF UI API support

When will the SF UI API be supported? In particular, we are looking for "Get Values for All Picklist Fields of a Record Type" to identify dependencies.

I cannot speak to this API specifically, but you can use JSForce to call these api's (and most other api's as well) by calling the request functions conn.request() or conn.requestGet().
I have submitted a PR for the docs to add instructions and example for working with request().
https://github.com/jsforce/jsforce-website/pull/10
Here is what I have submitted to the docs - hopefully you find it useful as I have used the UI API in some of the examples:
Request
Make REST api calls to APIs that are not explicitly supported by JSForce.
Setting the URL
The Endpoint URL can be in one of the following formats:

Absolute URL: https://na1.salesforce.com/services/data/v32.0/sobjects/Account/describe.
Relative path from root: /services/data/v32.0/sobjects/Account/describe.
Relative path from version root: /sobjects/Account/describe.

This is only supported if you have explicitly set a default version.

Making Requests
You can use Connection#request() to make api requests.
For GET requests, you can pass in a string URL.
/* @interactive */
conn.request('/services/data/v47.0/ui-api/object-info').then(response => {
  console.log(response);
});

If you prefer to use callbacks instead of promises, pass a callback as the second parameter.
conn.request('/services/data/v47.0/ui-api/object-info', function(err, response) {
  console.log(response);
});

For other HTTP methods, you can pass an object to the request method. Ensure that you serialize the body of the request.
/* @interactive */
// Bulk API 2.0 - Query
const requestBody = {
  operation: 'query',
  query: 'SELECT Id, Name FROM Account LIMIT 1000',
};

conn
  .request({
    method: 'POST',
    url: '/services/data/v47.0/jobs/query',
    body: JSON.stringify(requestBody),
    headers: {
      'content-type': 'application/json',
    },
  })
  .then(response => {
    console.log(response);
  });

Request Helper Methods
In addition to Connection#request(), JSForce provides the following helper methods that can also be used:

Connection#requestGet()
Connection#requestPatch()
Connection#requestPost()
Connection#requestPut()
Connection#requestDelete()

For requestPatch, requestPost and requestPut, these will be serialized automatically and the content-type will be set to application/json.
/* @interactive */
const requestBody = {
  operation: 'query',
  query: 'SELECT Id, Name FROM Account LIMIT 1000',
};

conn.requestPost('/services/data/v47.0/jobs/query', requestBody).then(response => {
  console.log(response);
});

Request HTTP Options
All request methods allow setting HTTP options to be passed to the HTTP request.

Name
Type
Description

responseType
string
overrides the content-type from the response to change how the response is parsed. Valid values are text/xml, application/xml, application/json, text/csv. If you do not want JSForce to auto-parse the response, set this to any other value, e.x. text.

noContentResponse
any
Alternative response when no content returned in response (= HTTP 204)

transport
Transport
Transport for http api - you should not need to set this option.

If you would like to opt-out of parsing, you can set the responseType to text. This is useful if you want the raw response from Salesforce instead of having the results automatically parsed.
// Get raw CSV data instead of allowing JSForce to parse the CSV to JSON
requestGet('/services/data/v47.0/jobs/query/7502J00000LYZC4QAP/results', { responseType: 'text' }).then(response => {
  console.log(response);
});

Any oppositions to me submitting a PR that implemented SF UI-API support?


[Question] S3 as deep storage usage is high

Hi folks,
I'm new to Pinot and try setting deep storage.
The data usage in S3 is increasing high after days even though it's not much on server.
Can setting retention on tables will reduce S3 deep store usage by having old segment files removed ?

Yes. Retention manager should be able to remove segments from deep store after they expire

You may ask questions in our Slack troubleshooting channel, where you might be able to find answer faster :)


Make failed

Hi,
I followed the 'getting started' documentation, cloned the git repository, installed dependencies and ran make on Ubuntu. I got the following error:
$ make
if which rebar.cmd > /dev/null; \
then ERL_LIBS=.:$ERL_LIBS rebar.cmd compile; \
elif which rebar > /dev/null; \
then ERL_LIBS=.:$ERL_LIBS rebar compile; \
else \
make  erlc-lfec; \
fi
make[1]: Entering directory '/home/kerry/Projects/lfe'
bin/lfec -I include -o ebin -pa ../lfe src/cl.lfe
/usr/bin/env: lfe: No such file or directory
Makefile:68: recipe for target 'ebin/cl.beam' failed
make[1]: *** [ebin/cl.beam] Error 127
make[1]: Leaving directory '/home/kerry/Projects/lfe'
Makefile:78: recipe for target 'compile' failed
make: *** [compile] Error 2

Check out issue #182 where this problem is discussed and some simple solutions presented.


Update initial position sampling in maximize_likelihood

Use the same code in #152 for maximize_likelihood.

@kazewong Other parts of this function require changes as well. I am not sure how to do that, do you mind taking a look later?

I am not sure what you are trying to do here. The original PR has already been merged.

The function I changed is in likelihood.py. It has the same code structure as the code you merged in #152. Maybe I can change the base to jim_dev. It might be more obvious.

The loops before and after the code I modified still need some work.


Adds Craft CMS project type, docs, and tests closes #3594

The Problem/Issue/Bug:
There is no built-in Craft CMS project type, and no quickstart documentation for Craft CMS
How this PR Solves The Problem:
This PR adds a Craft CMS project type, tests, and quickstart documentation for people working with Craft CMS
Manual Testing Instructions:
mkdir path/to/my-project
cd path/to/my-project
ddev config --project-type=craftcms
ddev start
ddev composer create craftcms/craft
ddev launch

Automated Testing Overview:

Craft CMS webserver test
Craft CMS commands test

Related Issue Link(s):
https://github.com/drud/ddev/issues/3594
Release/Deployment notes:
n/a

I recommend using ddev composer for everything. Purists would never have composer or php on their local machine, so all the quickstarts use ddev composer create or ddev composer install. There are loads of problems with using local php/composer, including version and extension mismatches.

Yes, you're of course right @rfay, I've updated the quickstart docs to read as follows:
mkdir path/to/my-project
cd path/to/my-project
ddev config --project-type=craftcms
ddev start
ddev composer create craftcms/craft
ddev launch

Sorry the full tests weren't being run. Forgot you were first-time contrib so they have to be authorized just during this PR. If you want to do the docs part first, we can get that one through faster and then (I think) it won't balk about running tests on this one.

No worries @rfay -- the docs are all set!
Let us know what we need to do from here.

I was just saying you'll probably have a lot of tests work to do still, and you may need me to ping each push you do. Also remember to run the failing tests manually while debugging them so you don't have to wait for all the test runners.

Oh, if you can't see the buildkite failures PM me your email and I'll add you to buildkite.

So near as I can tell, it's failing in TestApptypeDetection which I believe is failing because we need to provide you with some kind of a test app?
If you could help us out with where to go from here, that'd be helpful. I have looked at:
https://ddev.readthedocs.io/en/stable/developers/project-types/
&
https://github.com/drud/ddev/blob/master/docs/content/developers/building-contributing.md
But I'm a little unclear on what we do from here in terms of getting a test project built, delivered, etc.

I've started a PR to add a test site at https://github.com/khalwat/ddev/pull/11, but I think we need direction from @rfay on next steps.

I don't think TestAppTypeDetection can work until you have a craftcms stanza in  https://github.com/khalwat/ddev/blob/777386662756585a8d2519e8e3a34a48d481d8db/pkg/ddevapp/ddevapp_test.go#L40
Because IsCraftCMS() can't succeed in https://github.com/khalwat/ddev/blob/1f62a43dd46ae3007b7eafc182046c51a3e17754/pkg/ddevapp/craftcms.go#L13-L15 without the craft command being there.

@rfay we have our test site at https://github.com/khalwat/ddev/blob/ed7cb298f279bf636c49ad12a0ab92810335c185/pkg/ddevapp/ddevapp_test.go#L238-L251 but it looks like IsCraftCMS() is still failing. Do you have any pointers for us?

Looks like you have
			SourceURL:                     "https://github.com/khalwat/ddev/files/9488101/craftcms-4.2.3.tar.gz",
			ArchiveInternalExtractionPath: "craftcms-4.2.3",

but there is no subdir in that tarball, no ArchiveInternalExtractionPath. the tarball starts at the root.
Congrats, though, it certainly works with ddev config on the extracted archive!
I do think the archive extraction should have failed on you because that subdir isn't in there.
This is a good time to get Goland going though (or vscode, but it's more work) so you can easily run a test and step through it.

I updated the source, ensuring that craftcms-4.2.3 is a subdirectory in the tarball, but tests are still failing. Any other ideas?

We both have it up and running in Goland... it's a little opaque to me what's going on tho.
It seems like what is happening is somehow, the craftcms project isn't in the right place, so it fails the check for project type craftcms when it looks for the craft command.
However, I've compared what Ben did with the archive files to some of the other project types that do seem to pass, and I'm missing what's wrong.

Looks like it was just missing an empty craft file in pkg/ddevapp/testdata/TestApptypeDetection/craftcms/
Tests are running, so there may still yet be more errors, but at least that one is down...

Are you $#%@ kidding me?! Well, tests seem to be passing, so great catch!!

You're awesome, will try to look tomorrow.

So @bencroker I did some sleuthing... you switched over to use .zip in the test app:
		// 13: craftcms
		{
			Name:                          "TestPkgCraftCms",
			SourceURL:                     "https://github.com/khalwat/ddev/files/9497111/cms-4.2.3.zip",
			ArchiveInternalExtractionPath: "cms-4.2.3/",
			FilesZipballURL:               "https://github.com/khalwat/ddev/files/9497148/craftcms-4.2.3-files.zip",
			DBZipURL:                      "https://github.com/khalwat/ddev/files/9496399/craftcms-4.2.3-db.sql.zip",
			FullSiteTarballURL:            "",
			Type:                          nodeps.AppTypeCraftCms,
			Docroot:                       "web",
			Safe200URIWithExpectation:     testcommon.URIWithExpect{URI: "/test.html", Expect: "Thanks for testing Craft CMS"},
			DynamicURI:                    testcommon.URIWithExpect{URI: "/", Expect: "Thanks for installing Craft CMS"},
			FilesImageURI:                 "/happy-brad.jpg",
		},

But alas, it may allow for .zip for the SourceURL but it does not for other parts of the tests:
Files:
		// Get files before start, as syncing can start immediately.
		if site.FilesTarballURL != "" {
			_, tarballPath, err := testcommon.GetCachedArchive(site.Name, "local-tarballs-files", "", site.FilesTarballURL)
			require.NoError(t, err)
			err = app.ImportFiles(tarballPath, "")
			assert.NoError(err)
		}

Database:
		if site.DBTarURL != "" {
			_, cachedArchive, err := testcommon.GetCachedArchive(site.Name, site.Name+"_siteTarArchive", "", site.DBTarURL)
			if err != nil {
				assert.NoError(err)
				continue
			}
			err = app.ImportDB(cachedArchive, "", false, false, "db")
			if err != nil {
				assert.NoError(err, "failed to import-db with dbtarball %s, app.Type=%s, database=%s", site.DBTarURL, app.Type, app.Database.Type+":"+app.Database.Version)
				continue
			}
		}

...so the files were never being imported, and also the database was never being imported, causing the tests to fail.
So I think you'll need to recreate the .tar.gz archives for the files and database, and use the FilesTarballURL and DBTarURL respectively.

New archive files in tar.gz format:
craftcms-4.2.3-files.tar.gz

But alas, it may allow for .zip for the SourceURL but it does not for other parts of the tests:

So you're saying that our testing is uncovering quirks with ddev's testing framework, rather than with our code ;)
Anyway, great to see you get to the bottom of this and get tests finally green!!

Ironically, Craft just got an update that requires us to change the .env.example to .env.example.dev 😆

We will have to check for both. Fortunately it is an easy change

The test tarballs are now at https://github.com/drud/ddev_test_tarballs/releases/tag/v1.1 but I changed the cms-4.2.3.zip to craft-cms-4.2.3.zip

This PR is waiting on the addition of a project type action that executes after ddev composer create, so that we can rewrite the .env or env.example / env.example.dev then, rather than inside of the configOverrideAction: craftCmsConfigOverrideAction as we do now.
@rfay has said he's going to look into adding such a project type action.
This will allow us to greatly simplify the quickstart documentation, and will also allow for the resolution of:
https://github.com/drud/ddev/issues/3636 & https://github.com/drud/ddev/issues/3590

Strike that comment above, after a discussion with @rfay I was able to refactor the code so the setup flow is much cleaner. We may not need the additional project action at all.

Thx for all your help @rfay -- I just push some very minor docs changes to clean up the formatting.
But we are ready. Let's do this!

Congrats, great work!


need to check if Get-VMSwitch cmdlet is available

Get-VMSwitch fails without Hyper-V feature installed. Need to check if Hyper-V feature is installed before calling the cmdlet.
Ran into the issue running the script with Get-VMSwitch cmdlet on Windows Server 2019 server core where Hyper-V feature wasn't installed.

Currently, it's not possible to use the script to troubleshoot network related issues if Hyper-V feature is not installed. The proposed change was created to help troubleshoot container networking issues if one does not have Hyper-V feature enabled on the host. Since the proposed solution is claimed to be an overkill, please consider providing a separate utility/script that can assist with container networking issues when Hyper-V feature is not enabled on the host and container runtime is facilitated solely by Windows Containers feature.
The request is described in issue #1264.

Hey @ivansharamok, thanks for the eye-opening elaboration. I have implemented the request bit differently (#1273) introducing slight optimisations. Hope it finds you well.


I want to keep video continues from its current playback position when changing the quality .

I want to keep video continues from its current playback position when changing the quality but media kit has no player.setDataSource(...).
Now I needed to declare duration for loading and sometimes if load takes less time then video start from first position. How I can acheive without fixed duration?

That's pretty vague. When source has multiple video qualities (like a dash or hls), just add a track selector. If url is different, save start position, load new url, set start position.


[FEATURE] Sort Icons alphabetical in README

Description
Sort Icons alphabetical in README
https://github.com/EddieHubCommunity/LinkFree#available-icons

Can I work on this?


migrate from Java-Markdown-Generator to own classes to drop this dep

create new module logchange-md
packageing dev.logchange.md and create there classes like Link, Heading etc

@witx98

@marwin1991 Migration - https://github.com/logchange/logchange/pull/242


Async Job - Two Parallel Jobs

I have a situation where I need to run two same jobs parallel. Right now it's executing sequentially.  Like I have scheduled both jobs with startnow function. it supposed to excute in Async but its executing in sequence.
JobManager.create(this).addJobCreator(new CookJobCreator());
PersistableBundleCompat extras = ci.toBundle();
        int jobId = new JobRequest.Builder(CookSyncJob.TAG)
                .startNow()
//                .setBackoffCriteria(200L, JobRequest.BackoffPolicy.EXPONENTIAL)
                .setExtras(extras)
//                .setExact(1)
//                .setPersisted(false)
//                .setRequirementsEnforced(true)
                .setUpdateCurrent(false)
                .build()
                .schedule();

The library doesn't make any promises about how jobs are executed. In your case I highly recommend to create two separate threads inside of your job and to run them in parallel. You need to block your job until the execution is done. A similar sample can be found here: https://github.com/evernote/android-job/wiki/FAQ#how-can-i-run-async-operations-in-a-job

what if I create the job with separate tags??
I need separate Jobs as both are completely independent of each other.
Is there any way we can execute jobs irrespective of other running jobs??

If they're independent, why do you need to run them in parallel? Yes, they should run irrespective of each other.

yes, they should. By parallel I mean they should run irrespective of each other. But they are running in sequence.

Sorry for the delay. I finally had time to test this myself and can reproduce the behavior. The behavior depends on the API. Exact jobs or jobs using the JobScheduler under the hood run in parallel. However, if you start jobs immediately, then a JobIntentService is used, which doesn't run jobs in parallel.
I don't have a good solution for this since the JobIntentService is required for Android O+.

so what do you suggest?? should I use Exact Job?

You're probably starting your job immediately. I recommend to not use the library in this case and use instead two background threads. If the background works could take longer and you want to use a job, then I suggest to use just one job like before.

Closing because of inactivity


npm create indiekit fails with a "Cannot find package 'debug'" from store-github

Description of the bug
I'm working through the getting started guide and can't get the first step to work:
❯ npm create indiekit my-indiekit

> npx
> create-indiekit my-indiekit

Welcome to Indiekit! (create-indiekit v1.0.0-beta.18)
If you encounter a problem, visit https://github.com/getindiekit/indiekit/issues to file a new issue.

> Gathering details…
✔ What is your website’s URL? … https://micro.jacobian.org/
✔ Do you want to use a publication preset? … yes
✔ Which publication preset do you want to use? › Hugo
✔ Where do you want to store your content?  › YAML
node:internal/modules/esm/resolve:854
  throw new ERR_MODULE_NOT_FOUND(packageName, fileURLToPath(base), null);
        ^

Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'debug' imported from<EMAIL_ADDRESS>    at packageResolve (node:internal/modules/esm/resolve:854:9)
    at moduleResolve (node:internal/modules/esm/resolve:927:18)
    at defaultResolve (node:internal/modules/esm/resolve:1169:11)
    at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:540:12)
    at ModuleLoader.resolve (node:internal/modules/esm/loader:509:25)
    at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:239:38)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:96:40)
    at link (node:internal/modules/esm/module_job:95:36) {
  code: 'ERR_MODULE_NOT_FOUND'
}

Node.js v20.17.0
npm error code 1
npm error path /Users/jacob/c/jacobian
npm error command failed
npm error command sh -c create-indiekit my-indiekit
npm error A complete log of this run can be found in: /Users/jacob/.npm/_logs/2024-10-02T22_24_51_978Z-debug-0.log

Here's that log file mentioned in the error message: 2024-10-02T22_24_51_978Z-debug-0.log
The log file mentioned
Steps to reproduce

Run npm create indiekit

Expected behaviour
I expect not to get an NPM error 😄
Environment

Indiekit version: 1.0.0-beta.18
OS: macOS 15 ("sequoia")
Node: 20.17.0 (installed via mise)

Additional context
No response

Thanks for reporting this issue; I missed adding the debug dependency to the @indiekit/store-github package in a recent commit, this will fixed in the next beta release.


change(ci): add acceptance test for getblocktemplate RPC in CI, and fix RPC bugs

Motivation
We want to test the getblocktemplate RPC with a synced Zebra instance.
We tried this in PR #5653, but the RPC had some bugs, so we had to revert it.
Closes #5652.
Solution

Re-apply PR #5653 (do a revert of the revert)
Fix bugs in the RPC method

An assert that was doing the wrong check, causing a panic
A fee that wasn't actually negated, causing a panic

Fix coverage bugs in the test

The original RPC query is sent right after Zebra activates the mempool, when it is usually empty
Wait for transactions to download and verify in the test, then call the RPC again

Review
This is a high priority, because it stops us adding more bugs to the getblocktemplate RPC.
Reviewer Checklist

[ ] Will the PR name make sense to users?

[ ] Does it need extra CHANGELOG info? (new features, breaking changes, large changes)

[ ] Are the PR labels correct?
[ ] Does the code do what the ticket and PR says?
[ ] How do you know it works? Does it have tests?

Follow Up Work

#5686

I have run this RPC manually thousands of times, and it seems to work (and it's very fast)


Undetected forge mods

I'm currently hosting a "Pixelmon Reforged" server and using your plugin to stop hack mod/clients.
There's one : Pixelradar. It is unseeable in the console and is considered a hack.
I don't know how possible it'd be to try and detect it., but I'd love if it could be.

It's a bit of a ghost to find online, but I have the jar if you want to see it.

Hello! Thanks for you report.
I have already looked at it partially and I am not really sure yet how to handle it from the plugin. But, I will keep trying. Maybe I will find something useful for it.
Otherwise... I am thinking that a client side mod would be able to handle it. Such mod would have to be added to the modpack and be present on all clients which want to connect to the server. Furthermore, that mod would be communicating with the plugin on the server side and while connecting to the server, it would send the real modlist to the server.

Thank you for the insight and work!
If I can help further just ask.

Just a little idea: Would it be possible to read client-side config names?
It makes a client CFG file named "pixelradar"
Just an idea, I don't know the ins and outs of modding so excuse if this is stupid!

I doubt it is possible to do it from the server. But I will check if it is possible.
Also, I am planning to create ClientInspector mod that will be used on the client side and it will do the thing that I described above, that means, it will send the real mod list to the server.
Let’s leave this issue open so that I can post updates here.

It’s been a long time since last update here.
So, let’s get it started. 😄
I have just finished development of Client Inspector Forge mod.
This mod will completely replace existing Sponge plugin.
It works almost the same like the plugin with one difference, that is, you have to install it on the client side.
The reason is that the server side needs to somehow ask the client side for installed mods. Because we need to send a special request packet to the client, we need to have a mod on the client side. Such mod will properly handle the request packet and will return us a mod list with all mods, even with those that are “hiding”. And that is what Client Inspector is doing.
Furthermore, the Client Inspector mod will be published both on Sponge and CurseForge and its dev release is already available at my discord server. I have also tested it already with the mod you mentioned in your message and it did properly send its name to the server.
Some extra thoughts:
I was thinking about having a possibility for having the mod installed only on the server side and letting the server owner to decide if they want to require Client Inspector on the client side but it does not seem worth. Firstly because you are not fully protected and normally as a server owner you would rather want full protection than half. And secondly, you are already making a mod pack and adding an additional mod on the client side will not make a big difference in the terms of performance but it will benefit in additional protection. Also, you will not need to worry about this mod when you are building separate packs for server (with additional plugins) and what for client (with client side only mods) as it will go to both packs.


Add tests for common.go

Summary
Provide a brief overview of the changes and the issue being addressed.
Explain the rationale and any background necessary for understanding the changes.
List dependencies required by this change, if any.
Fixes #(related issue)
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[ ] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 55.637% (+0.3%) from 55.345%
when pulling 5b3c45a0032b70e1745888b600f35f1e51e068c5 on add-tests-common.go
into b69d87de33106bb520e680d416f0ce97909bd97c on main.


Implement Frontend User Authentication and Authorization

This issue focuses on the implementation of user authentication and authorization in the frontend of the web application using React and Redux.
Tasks:

Create Registration and Login Forms:

Develop user interface components for registration and login.
Forms should collect username, email (for registration), and password.

Handle Form Submission:

Implement logic to handle form submissions.
Capture form data and send it to backend endpoints.

Interact with Backend Authentication Endpoints:

Use HTTP requests to interact with Django backend.
Handle responses, especially the authentication tokens.

Manage Authentication State Using Redux:

Define actions and reducers for authentication state.
Update Redux store based on authentication operations.

Implement Protected Routes/Components:

Manage routing using React Router.
Create protected routes accessible only to authenticated users.
Redirect unauthenticated users to login.

Store Authentication Tokens Securely:

Store tokens in browser's local storage.
Attach token to HTTP request headers for protected endpoints.

Implement Logout Functionality:

Clear authentication token from local storage and update Redux state.

Error Handling and Validation:

Handle errors during authentication.
Validate user input on frontend before sending to backend.

Responsive and User-Friendly UI:

Ensure UI is responsive and user-friendly.
Provide feedback during form submissions (like loading indicators).

Testing:

Test functionalities thoroughly for different scenarios and edge cases.

Completion of these tasks will establish a robust user authentication system on the frontend of the application.

Progress Update on Task 1: Frontend User Authentication and Authorization
Here's a summary of what has been done so far for implementing user authentication and authorization in the frontend:

Created Registration and Login Forms:

Developed RegistrationForm.js and LoginForm.js components.
These forms include input fields for username, email (for registration), and password, along with submit buttons.

Implemented Form Submission Handling:

Added logic in both forms to handle user input and form submissions.
Utilized React's useState hook for managing form states.

Interacted with Backend Authentication Endpoints:

Used axios to make HTTP POST requests to Django backend for user registration and login.
Handled responses including extracting and storing authentication tokens upon successful login (stored in local storage).
Prepared for error handling during login and registration processes.

These steps establish the foundational work for the frontend authentication system. The next steps will include securing the token handling, managing global user state, and integrating protected routes in the application.
Feel free to review the code and provide any feedback or additional requirements.

Progress Update on Tasks 1-4
We've made significant progress on implementing frontend user authentication and authorization. Here's a summary:

Registration and Login Forms: Created RegistrationForm.js and LoginForm.js components to handle user inputs and form submissions.

Form Submission Handling: Implemented logic to send user data to the backend endpoints for registration and login, using axios for HTTP requests.

Backend Interaction: Set up interaction with the Django backend. The frontend successfully sends data to the backend and handles responses, particularly dealing with authentication tokens.

Redux Integration: Initiated the setup of Redux for state management. Created an authSlice to manage authentication state, including actions and reducers for login and logout functionalities.

API URL Management: Discussed the use of environment variables for managing the API URL, allowing for flexibility between development and production environments.

Logout Implementation: Started the implementation of logout functionality, including creating a logout endpoint in the backend and a corresponding logout function in the frontend.

The application is shaping up well, with key authentication functionalities being integrated into the frontend. The next steps will include finalizing the logout functionality, securing token handling, and further integrating Redux for global state management.
Feel free to review the updates and provide any feedback or additional requirements.

Progress Update on Task 5: Implementing Protected Routes/Components
We've made significant progress on Task 5, focusing on managing routing and securing components in the React application:

React Router Integration: Integrated React Router to manage application routing. This setup allows for the creation of distinct navigable pages within the app.

Protected Route Implementation: Developed a ProtectedRoute component. This component checks the authentication state and ensures that only authenticated users can access certain routes. Unauthenticated users are redirected to the login page.

Redux State Persistence: Addressed an issue where the Redux state was resetting on page reloads. Implemented redux-persist to ensure the persistence of the Redux state across sessions and page reloads. This change maintains the user's logged-in status even after refreshing the page.

Non-Serializable Warning Resolution: Identified and addressed a warning regarding non-serializable values in Redux actions, which is a common occurrence with redux-persist. Updated the Redux store configuration to safely ignore serialization checks for specific redux-persist actions.

With these implementations, the application now has a robust system for route protection and state persistence, enhancing both security and user experience.
The next steps will involve finalizing the overall authentication flow, testing the complete functionality, and making any necessary refinements.
Please review the progress and provide any feedback or additional requirements.


Spatial Mapping Renderer =  Hot Pink

After adding the HoloToolkit to a new Unity project and assigning it to collection of holograms, the inside of my office is rendered in spectacular hot pink. By default, my holograms should be occluded by my office walls. I suspect that the renderer is not locating the occlusion material.
My fix:

Set the Renderer Mode to Material, and
Drag and drop HoloToolkit/SpatialMapping/Materials/Occlusion on to Render Material.

Hi @richiecarmichael,
When did you pull the HoloToolkit? Change #57 should have addressed this. It was just checked-in yesterday.

Hi @angelaHillier, I downloaded the toolkit on the Tuesday evening (two days ago) so it may not include your recent fix.

Hi @angelaHillier, I just created a new Unity app from scratch with the toolkit's Spatial Mapping Renderer. Still hot pink. Had to explicit set mode to "material" and material to "occlusion".

I'm up to date with master and am also seeing hot pink SR meshes.

Create an empty scene.
Create Empty object in scene.
Add SpatialMappingRenderer component to the object.
Build, Deploy, Run
Observe pink walls.

I'm focusing on #49, but thought another repro would be good on this. I can also confirm that the work around works for me too.

Just cloned the repo today and enabled spatial occlusion in my project. I also saw hot pink. @richiecarmichael thanks for the heads-up.

Found the problem. Resources.Load() is being used to load the occlusion material, but that material does not exist under a Resources folder. Since loading materials from Resource is actual not recommended, I'm going to expose two public properties on the Spatial Mapping Renderer component: Occlusion and Render Material, which will have the occlusion and wireframe materials pre-mapped.

I currently have a pull request out to help address this issue. Note: my change will still require that you manually set the 'Occlusion Material' and 'Render Material' for the Spatial Mapping Renderer component, or else you will see hot pink for the mesh. I considered making a prefab that has these materials pre-wired, but it did not offer as much flexibility as keeping the components separate....

I've updated the rendering component to expose an 'Occlusion Material' property. You still have to set the material manually to avoid seeing hot pink though! We've moved away from loading the materials for you, since there is no guarantee that they'll be there and/or that the folder structure won't change again.


Custom prompts and easier customization

Instead of using a raw talon list, we now use a capture user.modelPrompt for all the model commands. The capture is formed via the set union of static prompts and custom prompts. Custom prompts are empty by default. There is an example that can be copied into the user directory from the GPT folder.
The user can create a custom prompt talon list file outside this repo in their user directory and it will automatically be the most specific  (i.e. you don't need to do weird context matching on the hostname or stuff that is semantically meaningless)

@jaresty when you get a chance could you try this out locally and see if you like the UX? Also curious on the naming scheme and if you think customPrompt and modelPrompt both make sense. (as well as static prompt still).
modelPrompt = staticPrompt U customPrompt <- does that seem fair if you had no prior background with this repo?
If it seems reasonable I will document this in the readme for GPT/ and then merge

This is great! 😊 I really like the new interface and I think I can use it easily to experiment with new prompts. Thank you for putting this together! 🌟

Sweet will doc this then merge. Just to confirm, did you have a chance to test locally? If not it is ok, just want to know for context

I did give this a try locally and it worked for me.


n_jobs for logisticRegression is not usable

**Description of the issue :  **
We are proposing a n_jobs parameter for the asreview.models.classifiers.logistic  classifier . This parameter is passed to the underlying sklearn.linear_model LogisticRegression. However this classifier makes use of n_jobs>1 only in case of multi class classification (and with another solver than liblinear that is used as default in asreview). Because we are only facing a binary classification task in ASReview, this n_jobs parameter seems not needed.
Changes needed:
I would suggest to remove the n_jobs parameter from the asreview logistic classifier because changing it does not change anything to the learning pipeline. I will be happy to implement a PR if this is pertinent.
NB: I am not aware that we are doing some HP selection on top of the ASreview logistic classifier. However, if this is the case, we should keep n_jobs=-1 for the asreview.logistic classfiier to pass it to HP selection but we can still remove it from the sklearn.linear_model Logistic regression.

I like the idea. PR is welcome!

PR provided here :)  https://github.com/asreview/asreview/pull/1742


paket.template should allow comments

For example: # this is because of foo.js

Fixed by pull request 989


Add support for Python 3.12

According to https://github.com/pytorch/pytorch/issues/110436 PyTorch 2.2 has at least some Python 3.12 support now.

We should ditch 3.9 then.

Why? 3.9 still has 1 year, 8 months left until EOL.
Agree that if something pops up where we have to change code to build around support both we can ditch 3.9, but why would we ditch it just because?

Why? 3.9 still has 1 year, 8 months left until EOL.
Agree that if something pops up where we have to change code to build around support both we can ditch 3.9, but why would we ditch it just because?

Not just because. My experience is that it is kinda painful to support more than 3 Python versions back. But it is biased because the package contained compiled code. Here, for pure Python, it might be less of a hassle.

@awennersteen Torch doesn't play nicely with py3.12, right ? This is why you closed the issue on PyQ ?

Actually it got sorted with Torch, and in PyQ it's been enabled for 3.12. With the next release of PyQ we can bump it here too

closing since its supported now


Add feature to generate Spring controllers based on contract DSL

From @jkubrynski on May 12, 2015 11:13
Copied from original issue: Codearte/accurest#64

Wow.

The work required to do this (with all the endpoint stubbing, pojo creation) is not worth the results. We're dropping this.


Maintenance week updates

What does this PR do?
Updates app according to our maintenance week documentation
Helpful background context
This app is used by DDC staff for local processing of top container data and does not have any of the AWS/Docker-related code typically found in our apps. I do not foresee this app ever moving into AWS based on the current usage.
How can a reviewer manually see the effects of these changes?
Run make test and make lint to confirm they still pass
Includes new or updated dependencies?
YES
Changes expectations for external applications?
NO
Developer

[x] All new ENV is documented in README
[x] All new ENV has been added to staging and production environments
[x] All related Jira tickets are linked in commit message(s)
[x] Stakeholder approval has been confirmed (or is not needed)

Code Reviewer(s)

[ ] The commit message is clear and follows our guidelines (not just this PR message)
[ ] There are appropriate tests covering any new functionality
[ ] The provided documentation is sufficient for understanding any new functionality introduced
[ ] Any manual tests have been performed or provided examples verified
[ ] New dependencies are appropriate or there were no changes

Good call @jonavellecuerdo, just added a general description to README.md, let me know more detail would be useful!


Capacitor 2.0 support

Hi, looks like my project was unable to import the plugin:
Cannot read property 'enable' of undefined
I'm currently using capacitor 2.0beta (because I want to use androidx support). Maybe that's the problem? Thanks in advance.

My bad, it works with capacitor2.0, but im unable to run the enable() method of the plugin. It looks like it won't exist in the plugin. Any advice? Thanks in advance.

This is now addressed with @capacitor-community/firebase-analytics package. Closing the issue.


Aggregate Control does not work on dynamically added elements

The aggregate control correctly uses the jQuery .on() function to bind to body to watch elements added to body (or removed from), but it does not work. For example on the new tags page in branch fix_596 after filtering the aggregate control no longer recognizes selected elements.

Above committed fix has a known issue. Some selectors cause double selection for a reason I do not know. Example: In the tags page handler, replace the aggregate selector with input[name=tags]. Every selected checkbox will appear twice.


changing native dictionary to a included words resource

I've been using randexp in several environments, so I think that would be a good approach include a word resource inside of project.

Configuration of dictionary path looks interesting, but considering Windows users, all of them would have to look for a wordlist resource or maybe create one, with the popularization of the Randexp, we could have more and more Windows users.


[IE TESTS] add definition set_device_name for cuda unit tests

@lohika-denis-kotov Please review & merge the changes.
FYI: We have not to link shared functional tests lib to unit tests binary. It's different testing level: unit is for class entity level (internal & etc), functional is for public API level testing. https://github.com/openvinotoolkit/openvino/blob/master/src/tests/README.md

@lohika-denis-kotov Please review & merge the changes.

@iefode Why do you add this function also for unit tests ?

FYI: We have not to link shared functional tests lib to unit tests binary. It's different testing level: unit is for class entity level (internal & etc), functional is for public API level testing. https://github.com/openvinotoolkit/openvino/blob/master/src/tests/README.md

It probably was done by mistake, simple copy-paste ... Dependency should be tried to remove and see if it compiles

@iefode

@lohika-denis-kotov Please review & merge the changes.

Why do you add this function also for unit tests ?

FYI: We have not to link shared functional tests lib to unit tests binary. It's different testing level: unit is for class entity level (internal & etc), functional is for public API level testing. https://github.com/openvinotoolkit/openvino/blob/master/src/tests/README.md

It probably was done by mistake, simple copy-paste ... Dependency should be tried to remove and see if it compiles

It was my point. I don't understand why the shared lib was linked to the unit tests. But anyway I have a linking problem in the OV PR ( https://github.com/openvinotoolkit/openvino/pull/13340)


Release 0.6 prints 1.0 with help -v

squeakr help -v
version 1.0

but release is 0.6 ?
This would be good to have:
% squeakr version
squeakr 0.6

Relates to #27

Fixed in release 0.7.


Ordered chaining for .default() values

Hi, is it possible to have an ordered default value chain on a field with zod?
For now, I have only tried doing it this way, but it doesn't seem to work:
import env from "env-var"

const schema = z.object({
    port: z.number()
        .default(env.get("PORT").asPortNumber()!)
        .default(env.get("CONFIG.PORT").asPortNumber()!)
        .default(4444)
})

Basically what I want to do here, is to assign 3 default values in an ordered way. So for example, if the PORT variable is undefined | null zod will try to default to CONFIG.PORT, if that's undefined | null too, the value default to 4444.
[original field = undefined] --Try defaults--> undefined -> undefined -> 4444 (default)

If there's no default value at the end of the chain and the value from the previous matcher is undefined | null zod would throw an error.
[original field = undefined] --Try defaults--> undefined -> undefined -> ERROR (no default values)

Can this be achieved with zod?
Thanks!

What about this:
import env from "env-var"
const defaultPort = env.get("PORT").asPortNumber() ||
  env.get("CONFIG.PORT").asPortNumber() ||
  4444;
const schema = z.object({
    port: z.number()
        .default(defaultPort)
})

What about this:
import env from "env-var"
const defaultPort = env.get("PORT").asPortNumber() ||
  env.get("CONFIG.PORT").asPortNumber() ||
  4444;
const schema = z.object({
    port: z.number().default(defaultPort)
});

That's a nice solution @scotttrinh but I don't want to leave the zod object(z.object) scope and declare a constant elsewhere for both readability purposes and to not cause undefined behavior at runtime. With ordered chaining, I don't think this will be an issue.

@michaelgrigoryan25

I don't want to leave the zod object(z.object) scope and declare a constant elsewhere for both readability purposes and to not cause undefined behavior at runtime

I'm not sure I understand what you're trying to say here: z.object, etc. is "constructor" of sorts, so when you run this code, it copies the value of defaultPort into the ZodType class instance, so there is no undefined behavior at runtime. If you just mean that you want to colocate the definition visually, you can certainly inline it:
const schema = z.object({
    port: z.number().default(
      env.get("PORT").asPortNumber() ||
        env.get("CONFIG.PORT").asPortNumber() ||
        4444
    )
});

With ordered chaining, I don't think this will be an issue.

The actual technical issue with some kind of proposal to have an array of defaults built by chaining calls to default has to do with the type checker. default expects that the value is of the type of the base schema so that we can ensure the type hold true at runtime, in this case number. As your example notes, you have to lie to the type checker and assert that your value is not undefined or null with a !. That means, there is nothing stopping you from having unsound runtime behavior like this:
const schema = z.object({
    port: z.number()
        .default(env.get("PORT").asPortNumber()!)
        .default(env.get("CONFIG.PORT").asPortNumber()!)
})

Oops! Since those asPortNumber return values are actually potentially undefined, but you've told the type checker that they're actually number. And until we get to runtime, we can't possible know if those values are defined or not, now the schema is unsound.
From a design perspective, it doesn't make sense to hold onto an array of default values unless we allow the default to be undefined and/or null and treat default the same as optional and/or nullable (depending on how permissive the default method is), which would make that field number | undefined | null (or whatever combination we allow to be returned from default).
Since || (or better yet ??) has the desired fallback behavior you're looking for, I think using that built in language feature makes the most sense. If you want something that uses function invocation rather than an operator, there are a few functional programming related libraries (e.g. fp-ts) that can make this a bit more functional/chain-y looking:
import * as O from 'fp-ts/Option'
import { pipe } from 'fp-ts/function'

const schema = z.object({
    port: z.number().default(
      pipe(
        O.fromNullable(env.get("PORT").asPortNumber()),
        O.alt(() => O.fromNullable(env.get("CONFIG.PORT").asPortNumber()),
        O.getOrElse(4444)
      )
    )
});

—but it's not clear why you'd want to do that.

The funny thing is that I totally forgot that I could use || or ?? operators, oof 😆. I don't know what I was thinking about. Weird stuff happens...
Anyways, thanks for your help!


MobileApp News 11.3

for I do the minor corrections for the app in a new pr and merge this one as is. thanks for contributing.
https://github.com/axonivy/dev.axonivy.com/pull/326


Error: Don't know how to format @ngDoc: module

No formatting (and therefore documentation pages) available for modules. Any reason this is the case?

+1 This seems like a weird issue, how are we supposed to group services.

You can use overview tag for documenting modules. Please refer to https://github.com/angular/dgeni-packages/blob/master/NOTES.md#ngdoc-tags

That doesn't seem to group them together?

angular.module('common.analytics', ['common.moment', 'common.d3Service'])
/**

@ngdoc overview
@name common.analytics

@requires common.moment
@requires common.d3Service

@description
enter description using overview

@example
enter example
/
.service('analyticsService', function analyticsService(API, $http, dateService, d3Service) {
/*
@ngdoc service
@name analyticsService
@requires ng.$http
@requires API
@requires dateService
@description uses service
*/

Note at the top the analytics are in separate groups, the only way I can get them to group together is names common.services.module, but it can't auto group the child services form modules.


App: Freezes when equipping items

After dragging an item from the inventory (i) to the slot, the app freezes.

This happens regardless of the inventory UI, I spawned a sword from prefabs list and equipped and it froze.

 There was a bug in the wearables components lookup.

It doesn't freeze anymore but wearables and pets don't attach and preview avatar doesn't show up.

I guess we need to wait until the new menu page is done on a few weeks.

Folding this into https://github.com/webaverse/app/issues/895


Using huey

Short Description
Instead of rq we want to use huey. The reason is that we encountered some problems using RQ because we have to use different processes for different workers which does not work with ERiC and a security stick.
Changes

Remove RQ
Add huey component and use it for all the perform_job functions
Adapt the tests
Add adr

Feedback

I thought that we already had an adr for RQ. Somehow, that was missing. Do you think we should add it now or just go with it and at least have one for huey.
Does the structure/naming of the huey component make sense to you?
Do you see any problems that I might have missed.
Do you see any specific part where we could/need to add documentation for huey?

How to review this Pull Request
Link to Confluence (internal)

@VictorDelCampo Maybe you can have a look, too. If you have any questions, let me know.

I thought that we already had an adr for RQ. Somehow, that was missing. Do you think we should add it now or just go with it and at least have one for huey.

I'd say what you wrote is detailed enough.

Does the structure/naming of the huey component make sense to you?

Structure yes, for naming I have commented suggestions.

Do you see any problems that I might have missed.

No.

Do you see any specific part where we could/need to add documentation for huey?

No.


Update list of actively maintained plasma implementations

The current list of implementations on /build is a little stale. I figure we can remove the ones that haven't been touched in 3+ months and add any new implementations that have popped up since!

@therevoltingx Is https://github.com/luciditytech/lucidity-plasma still actively maintained?

@kfichter
Hmm, kinda not really at this point. We're focusing on Plasma Cash because it uses some concepts that we use in our own system (which we call Plasma Bank.)
We do plan on using Plasma MVP style exits and using it as a library but we're not quite there yet.
Ideally we'd want to collab on a repo that's shared and more community driven so we don't create our own version. Then we want to be able to use it as libraries within our own products and smart contracts.

Cool, makes sense. Will remove it from this as I want to point people to places they can go to help out and get help. We should chat a community implementation later this month if you're around.

Sounds good to me! Yeah let's chat later about co-ordinating those efforts.


FR: Oven Set Temperature

Is it possible to have a sensor created with the temperature the oven is set to? This information is visible within the app, so I hope it's exposed to the api.
IE: When the oven is turned on, it defaults to being set to 350F. Can a sensor be generated to read that data?
I've tried making a template sensor that calculates the raw_temperature and user_temp_offset, but the user_temp_offset doesn't seem to update very quickly so the template sensor usually just ends up displaying the raw_temperature value.

Sorry, got Covid so I've been offline for a while.  I think that this can be accomplished by looking at the attributes of the oven entity, it publishes the target_temperature as an attribute that you should be able to read in a template sensor.  Does that give you what you need?

First and foremost, I hope you're doing well!
This is what I see for attributes while the oven is on:
min_temp: 76.7
max_temp: 287.8
operation_list: Off, Bake
current_temperature: 19.4
temperature: 190.6
target_temp_high: null
target_temp_low: null
operation_mode: Bake
display_state: Preheat
probe_present: false
display_temperature: 100
raw_temperature: 67
icon: mdi:stove
friendly_name: Oven
supported_features: 3

I do see a target_temp_high and target_temp_low but they both report null. But the temperature attribute does seem to be reporting the target temp so I can use this and make a template sensor to get what I need.
Basically I'm playing with different ideas on how to best control the oven through HA. The water_heater platform adjusts temps 1 degree at a time which means 50 button presses to bring it from 350 to 400. I'd give up and just go set it at the oven itself lol. I may do a virtual climate device or something like a simple input_select with the most common set temps.
Thanks for pointing me in the right direction!

Ah, you may want to look at https://github.com/nervetattoo/simple-thermostat.  It can be used with water_heater entities and you can set the step size to something more reasonable for an oven.  It's what I'm using in my HA dashboards.


Documentation

Hey i was looking at your mod, seems very useful for my usercase, however under regexpFilterWords = [] is not very well explained, i want to filter exactly with this regex expression : \[1 (apple), 1 (golden_apple), 1 (enchanted_golden_apple), 1 (banana)\]
simply to filter [1 apple, 1 golden_apple, 1 enchanted_golden_apple, 1 banana]
using that regex expression throw us crash, how i suppose to use properly?
Thanks.

Please provide your crash report, so we ccould diagnose the problem you've ran into.

Sorry for being rude, but seriously?


fix(healthchecker) port 2.x lock fixes to 1.5.x

Porting the changes from #112 to the 1.5.x tree.
This is still a WIP (I think I have a failing test).

ed494c0 is kinda interesting... it fixes what seems like an unrelated bug, but I'm yet not sure why the test for it wasn't failing before.

LGTM :+1:
@locao I want to make a couple minor changes to schedule() and run_in_timer() to make sure we log errors from the locked function when running in the timer context (instead of silently discarding them), but I think I'll do that in a follow-up PR. If that sounds good to you I say go ahead and merge.


Name resolution can fail in the simulator depending on operator order

I found a strange edge case in which name resolution of an imported operator incorrectly fails, but only if the operators are imported in a specific order and the main module is loaded in a specific way :woman_shrugging:
This issue is light on details because I've opened a draft PR with a failing integration test in #1036.

Very strange! Thanks for the detailed example

Thanks for taking a look! This is very strange indeed. I’m sorry I wasn’t able to derive a more minimal reproduction, but it seems so sensitive to the exact order of defs that every little thing I changed would avoid the problem!


No Bins are generated - not even with the write_fasta_bins.py script

Describe the bug
Hi,
I've installed GraphMB from source copying the data folder to /src/graphmb and the installation seemed to be working fine. I proceeded to bin my assembly (Flye v2.9.2) with the following command:
graphmb --assembly /path/to/folder/with/all/files --outdir /path/to/outdir --marker "" --cuda --post writebins
However, no bins were generated. I reran the software by removing the --post flag and tried to use the write_fasta_bins.py script  to generate the bins. I downloaded the script from https://github.com/AndreLamurias/binning_workflows/blob/main/write_fasta_bins.py since the file is not in the /src folder.
I used the graphmb_0_best_contig2bin.tsv  and my original .fasta file as inputs for the script.
However, I am getting a parsing error:
contig_name, bin_name = line.strip().split("\t")
ValueError: not enough values to unpack (expected 2, got 1)
The generated CSV file has 2 columns separated by a \t and initially I thought it could be an empty line at the end of the tsv file, but I removed it and it's still not working. Can you please let me know how to solve this?
Thanks

Hi, can you share the generated graphmb_0_best_contig2bin.tsv file?
It's strange that it is not writing bin files, I will check that later. But if the tsv file was generated, that script should be able to read it.

Hi Andre, thanks for getting back to me.
I can't share TSV files on github, so I compressed it. Please, find it attached.
Cheers,
graphmb_0_best_contig2bin.tsv.zip

The issue was the headers. The write_fasta_bins.py file did not account for the fact that the output files have header lines (starting with @) to make it compatible with AMBER. I have fixed that file, but you can also delete those lines, and it should work. let me know if you still have issues with that. I'm also going to add a "--writebins" option to GraphMB to make it easier.

Hi Andre,
I tried removing all the header lines before and it still didn't work. I had a different error instead. Regardless, the new script is working like a charm. Thanks for that.


improve performance by removing some code

I'm excited to shware some pawsitive changes with you today! UwU So, I wecentwy submited a puww wequest that wemoves some code fwom our pwoject. 🎉 I know, I know, it sounds a wittle scawy at fiwst, but bewieve me, it’s fow the best!
As pawt of my changes, I carefuwwy scwutinized our codebase and found a few aweas where we had some wedundant or unnecessawy code. This code was not onwy taking up space, but it was awso swowing down our appwication a wittle bit. 😿
By wemoving this code, we can now impwove the ovewaww pewformance of ouw appwication. That means faster load times, mowe efficient wesouwce usage, and a smooder expewience fow evewyone! ✨
I made sure to test evewything thwoughoughly to ensure that nothing bweaks and that the appwication continues to wun smoothwy. 🌟 So, wet's cewebwate this win and wook fowward to an even bettew and fastew appwication!
Thank you fow youw suppowt, and pwease wet me know if you have any questions ow concwerns! UwU 💖

please fix typo

it doies not compile


Widget loads, returns Error displaying widget: model not found

It's possible that this is a duplicate of https://github.com/matplotlib/jupyter-matplotlib/issues/112, but I have tried the following steps, and am receiving the errors that the model is not found.
this is from a clean 18.04 VM:

:$ sudo apt update
:$ sudo apt upgrade
:$ wget https://repo.anaconda.com/archive/Anaconda3-2019.10-Linux-x86_64.sh
:$ chmod +x Anaconda3-2019.10-Linux-x86_64.sh
:$ ./Anaconda3-2019.10-Linux-x86_64.sh
[logout]
[login]
(base) :$ conda install -c conda-forge ipympl
(base) :$ conda install nodejs
(base) :$ jupyter labextension install @jupyter-widgets/jupyterlab-manager
(base) :$ jupyter labextension install jupyter-matplotlib
(base) :$ git clone https://github.com/matplotlib/jupyter-matplotlib.git
(base) :$ cd jupyter-matplotlib
(base) :$ jupyter-lab

navigate to examples/ipympl.ipynb
Second cell returns:
Error displaying widget: model not found
I have tried to follow the instructions precisely -- am I missing something here?
I'm happy to provide any debug information that would help. I have also tried other iterations including updating conda, but they have all given this error.

Me too!

Same here. Set up as docker image, leads to the error "model not found". If I run
jupyter labextension install @jupyter-widgets/jupyterlab-manager jupyter-matplotlib
again it sometimes changes the error to "Loading widget..." but nothing happens.
Docker based on nvidia/cuda:10.2-base which is ubuntu 18.04 with the latest conda packages as of today.

I don't see anything wrong in your Docker image... Which version of JupyterLab are you using? It must be >=1.0

I am using jupyterlab 1.1.4.

Same problem here. The configuration below does not work.
ipympl                    0.2.1                 py36_1001    conda-forge
ipython                   7.10.2           py36h39e3cac_0
ipywidgets                7.5.1                      py_0    conda-forge
jupyterlab                1.0.2            py36hf63ae98_0
matplotlib                3.1.1            py36h5429711_0
nodejs                    10.13.0              he6710b0_0

I tried different versions of jupyterlab, nodejs......nothing worked...

A full restart of jupyterHub solved the issue for me. Not a server restart from the web-interface but a restart of the jupyterHub process.

I fixed this with https://github.com/matplotlib/jupyter-matplotlib/issues/130#issuecomment-567931502

I fixed this with #130 (comment)

your solution truly works, thank you!


optimize performace

@zhangyafeikimi,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Add possibility to deploy flowforge container to stage environment

Description
Add the possibility to deploy flowforge container to the stage environment.
Related Issue(s)

Checklist

[x] I have read the contribution guidelines
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

[ ] Upgrade instructions
[ ] Configuration details
[ ] Concepts

[ ] Changes flowforge.yml?

[ ] Issue/PR raised on flowforge/helm to update ConfigMap Template
[ ] Issue/PR raised on flowforge/CloudProject to update values for Staging/Production

Labels

[ ] Backport needed? -> add the backport label
[ ] Includes a DB migration? -> add the area:migration label

Do the failed steps need to pass before we merge this?

All green now.


neon/{padal,paddl} + clang 9, 10, & 11 on riscv64 test errors

/arm/neon/padal/vpadal_u8/native/c   [ ERROR ]
/<<PKGBUILDDIR>>/test/arm/neon/padal.c:219: assertion failed: r[0] == simde_vld1_u16(test_vec[i].r)[0] (45917 == 34167)

/arm/neon/padal/vpadal_u16/native/c  [ ERROR ]
/<<PKGBUILDDIR>>/test/arm/neon/padal.c:276: assertion failed: r[0] == simde_vld1_u32(test_vec[i].r)[0] (535960123 ==<PHONE_NUMBER>)

/arm/neon/padal/vpadal_u8/native/cpp [ ERROR ]
/<<PKGBUILDDIR>>/clang9_test/arm/neon/padal.cpp:219: assertion failed: r[0] == simde_vld1_u16(test_vec[i].r)[0] (45917 == 34167)

/arm/neon/padal/vpadal_u16/native/cpp[ ERROR ]
/<<PKGBUILDDIR>>/clang9_test/arm/neon/padal.cpp:276: assertion failed: r[0] == simde_vld1_u32(test_vec[i].r)[0] (535960123 ==<PHONE_NUMBER>)

/arm/neon/paddl/vpaddl_u8/native/c   [ ERROR ]
/<<PKGBUILDDIR>>/test/arm/neon/paddl.c:136: assertion failed: r[0] == simde_vld1_u16(test_vec[i].r)[0] (28866 == 291)

/arm/neon/paddl/vpaddl_u16/native/c  [ ERROR ]
/<<PKGBUILDDIR>>/test/arm/neon/paddl.c:169: assertion failed: r[0] == simde_vld1_u32(test_vec[i].r)[0]<PHONE_NUMBER> == 64615)

(and likewise for the cpp versions and the emulated versions of both)
https://buildd.debian.org/status/fetch.php?pkg=simde&arch=riscv64&ver=0.7.0~rc-1-1&stamp=1606043276&raw=0

This persists with the latest SIMDe development code https://buildd.debian.org/status/fetch.php?pkg=simde&arch=riscv64&ver=0.7.3~0git20210814191010.3f186a0-1&stamp=1629143973&raw=0

I ran the tests against https://github.com/simd-everywhere/simde/commit/7e70d02dda84bdfb1e97777bdb9622f3f29682b4 and I was unable to reproduce this in a Ubuntu focal container running qemu-riscv64-static version 4.2.1 (Debian 1:4.2-3ubuntu6.24)

[x] clang 10.0.0-4ubuntu1 "clang version 10.0.0-4ubuntu1"
[x] clang 11.0.0-2 "Ubuntu clang version 11.0.0-2~ubuntu20.04.1"
[x] clang 12.0.0-3ubuntu1 "Ubuntu clang version 12.0.0-3ubuntu1~20.04.5"

With clang 9.0.1-12 "clang version 9.0.1-12 ", I do see errors about the test aparatus itself; however clang-9 is rather old, so I'm going to close this issue. If clang-9 is important to someone else and they run into problems, please comment!


Fix for: [FATAL]:1:324866: Invalid byte 2 of 2-byte UTF-8 sequence (https://github.com/olap4j/olap4j/issues/25)

It corrupts data when system locale is different from Utf-8 and Xmla response encoded in Utf-8 and contains non english characters
//-- ? new String(entry.getResponse()).getBytes()
//++? Arrays.copyOf(entry.getResponse(), entry.getResponse().length)

Even better: entry.getResponse().clone()

Agree


Question. I'm still confused about the new style templates.

[Enter feedback here]Question. I'm still confused about the new style templates using the top level statements.
If I was trying to use Serilog to write log information to a file, what would I do to replace the startup.cs entries and where would I put it?
The old code
using Serilog;
using Serilog.Events;

public class Program
{
    public static int Main(string[] args)
    {
        Log.Logger = new LoggerConfiguration()
            .MinimumLevel.Override("Microsoft", LogEventLevel.Information)
            .Enrich.FromLogContext()
            .WriteTo.Console()
            .CreateLogger();

        try
        {
            Log.Information("Starting web host");
            CreateHostBuilder(args).Build().Run();
            return 0;
        }
        catch (Exception ex)
        {
            Log.Fatal(ex, "Host terminated unexpectedly");
            return 1;
        }
        finally
        {
            Log.CloseAndFlush();
        }
    }

    public static IHostBuilder CreateHostBuilder(string[] args) =>
            Host.CreateDefaultBuilder(args)
                .UseSerilog() // <-- Add this line
                .ConfigureWebHostDefaults(webBuilder =>
                {
                    webBuilder.UseStartup<Startup>();
                });
}

Thank you

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

ID: 447e23ae-0256-d8c1-4dca-d31fe21602a5
Version Independent ID: 926db74c-be79-f7cd-a4e9-e156dafa6287
Content: C# template changes in .NET 6 - .NET
Content Source: docs/core/tutorials/top-level-templates.md
Product: dotnet-fundamentals
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

@IEvangelist This is a logging question - can you help?

Hi 👋 @scottgurvey
Thank you so much for posting this issue. We really appreciate it.
I'm happy to help. The new style templates use top-level statements. They're functionally equivalent. So wherever you were plugging in Serialog before, you continue to do so. But these are templates we're talking about. For example, file new scenarios... there is no reason to update existing / functional apps to use top-level statements. Nonetheless, here is how you could do so:
using Serilog;
using Serilog.Events;

Log.Logger = new LoggerConfiguration()
    .MinimumLevel.Override("Microsoft", LogEventLevel.Information)
    .Enrich.FromLogContext()
    .WriteTo.Console()
    .CreateLogger();

try
{
    Log.Information("Starting web host");
    CreateHostBuilder(args).Build().Run();
    return 0;
}
catch (Exception ex)
{
    Log.Fatal(ex, "Host terminated unexpectedly");
    return 1;
}
finally
{
    Log.CloseAndFlush();
}

static IHostBuilder CreateHostBuilder(string[] args) =>
    Host.CreateDefaultBuilder(args)
        .UseSerilog() // <-- Add this line
        .ConfigureWebHostDefaults(webBuilder =>
        {
            webBuilder.UseStartup<Startup>();
        });

Please let me know if this is helpful, and we'll close this issue.

Yes, this is a great help.
Thank you. I really appreciate it.
HSG
From: David Pine @.>
Sent: Wednesday, August 18, 2021 1:31 PM
To: dotnet/docs @.>
Cc: scottgurvey @.>; Mention @.>
Subject: Re: [dotnet/docs] Question. I'm still confused about the new style templates. (#25673)
Hi 👋 @scottgurvey https://github.com/scottgurvey
Thank you so much for posting this issue. We really appreciate it.
I'm happy to help. The new style templates use top-level statements. They're functionally equivalent. So wherever you were plugging in Serialog before, you continue to do so. But these are templates we're talking about. For example, file new scenarios... there is no reason to update existing / functional apps to use top-level statements. Nonetheless, here is how you could do so:
using Serilog;
using Serilog.Events;
Log.Logger = new LoggerConfiguration()
.MinimumLevel.Override("Microsoft", LogEventLevel.Information)

.Enrich.FromLogContext()

.WriteTo.Console()

.CreateLogger();

try
{
Log.Information("Starting web host");

CreateHostBuilder(args).Build().Run();

return 0;

}
catch (Exception ex)
{
Log.Fatal(ex, "Host terminated unexpectedly");

return 1;

}
finally
{
Log.CloseAndFlush();

}
static IHostBuilder CreateHostBuilder(string[] args) =>
Host.CreateDefaultBuilder(args)

    .UseSerilog() // <-- Add this line

    .ConfigureWebHostDefaults(webBuilder =>

    {

        webBuilder.UseStartup<Startup>();

    });

Please let me know if this is helpful, and we'll close this issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/dotnet/docs/issues/25673#issuecomment-901297704 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AAZAD3DGFJ6TGG66DA7VRWDT5PU4RANCNFSM5CKQHRIQ .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .

I'm going to close this issue as I believe my response has helped the user. Feel free to open another issue if you need more help.

That's fine and thank you. I've got it working. Somehow it seems I've gone back to the old style and I wish I could see how the new style works but this is fine and does the job. I suppose someday someone will generate some examples based on the Core 6 templates. Regards.On Aug 23, 2021 8:57 AM, David Pine @.***> wrote:
I'm going to close this issue as I believe my response has helped the user. Feel free to open another issue if you need more help.
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android.


Instance-profile, update user data

HI , can you add support to "Instance-profile" and manual enter the user data or just give the option to not "runner_no_update"
github is auto update the runner ver at start , there is not way to add "--disableupdate" in the user data section
Thanks in advance
Noam

Would an option to pass raw CLI args to the runner startup command via a space delimited string be okay ? e.g "--sometime value --somethingElse value2"

HI , look like this part is working i add the "github_action_runner_extra_cli_args" i can conform its working
Many thanks

I'll merge it to main and draft a release later this evening.
I'd like to keep this issue open for the Instance-profile work

for the instance profile i think this will help
https://docs.aws.amazon.com/AWSJavaScriptSDK/v3/latest/client/ec2/command/AssociateIamInstanceProfileCommand/

@mahdi-torabi Hi, did you find a way to add instance-profile

@noamgreen I'm working on the InstanceProfile option as I need it for a project,
https://github.com/NextChapterSoftware/ec2-action-builder/pull/29

Merged. I'll verify and create a release this evening. There's a couple of other changes that I'd like to get out along with this


[Linux] CSGO Crashes after the last update [02/03/2023]

For some reason as soon as I issue the command to inject Osiris into the game it crashes and stops working.
I tried both a previous version and downloaded and compiled the latest version of the project for nothing.
Here is the output of
sudo gdb -p $(pidof csgo_linux64) -ex "call (void*)__libc_dlopen_mode(\"$PWD/libOsiris.so\", 2)"
(I used the debug version)
Attaching to process 158540
[New LWP 158541]
[New LWP 158542]
[New LWP 158543]
[New LWP 158544]
[New LWP 158546]
[New LWP 158547]
[New LWP 158548]
[New LWP 158549]
[New LWP 158554]
[New LWP 158555]
[New LWP 158556]
[New LWP 158557]
[New LWP 158621]
[New LWP 158622]
[New LWP 158623]
[New LWP 158625]
[New LWP 158626]
[New LWP 158628]
[New LWP 158629]
[New LWP 158630]
[New LWP 158631]
[New LWP 158632]
[New LWP 158634]
[New LWP 158635]
[New LWP 158636]
[New LWP 158637]
[New LWP 158638]
[New LWP 158639]
[New LWP 158640]
[New LWP 158643]
[New LWP 158646]
[New LWP 158647]
[New LWP 158648]
[New LWP 158649]
[New LWP 158660]
[New LWP 158661]

warning: Could not load shared library symbols for bin/linux64/launcher_client.so.
Do you need "set solib-search-path" or "set sysroot"?
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
0x00007fa05e6e5868 in __GI___clock_nanosleep (clock_id=clock_id@entry=0, flags=flags@entry=0, req=0x7ffe1413e780, rem=0x7ffe1413e770) at ../sysdeps/unix/sysv/linux/clock_nanosleep.c:78
78	../sysdeps/unix/sysv/linux/clock_nanosleep.c: No such file or directory.
--Type <RET> for more, q to quit, c to continue without paging--c
$1 = (void *) 0x6a009700
(gdb)
(gdb) c
Continuing.
[Thread 0x7fa00afff640 (LWP 158661) exited]
[Thread 0x7fa00a7fe640 (LWP 158660) exited]

Thread 1 "csgo_linux64" received signal SIGABRT, Aborted.
__pthread_kill_implementation (no_tid=0, signo=6, threadid=140326757489600) at ./nptl/pthread_kill.c:44
44	./nptl/pthread_kill.c: No such file or directory.

yeah,steam update 2 days ago,then linux cheat engine like Osiris、kali、gamesneeze cant load in game now(csgo will crash after sucessfully injected).I try many system,but the result is same .

I don't know what's the issue here; hopefully some folks comment about how injection works and why it fails after updates.

This issue was fixed by the author after this commit


:tada: Initial commit.

Summary of Changes
After successful merge of https://reviews.apache.org/r/72440/ to master it's now possible to use much more flexible query parameter to conduct search. Previously, sorting only worked with entityFilters clause that used different handler, was less performant, required specific indication of fields on which searching should be done and did no full text processing (like tokenization, analyzers). Query parameter enforces full text search on all immediate attributes.
Tests
What tests did you add or modify and why? If no tests were added or modified, explain why. Remove this line
Documentation
What documentation did you add or modify and why? Add any relevant links then remove this line
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[x] PR passes make test

Codecov Report

Merging #105 into master will decrease coverage by 0.86%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #105      +/-   ##
==========================================
- Coverage   75.86%   75.00%   -0.87%
==========================================
  Files          17       17
  Lines         609      608       -1
  Branches       75       75
==========================================
- Hits          462      456       -6
- Misses        121      127       +6
+ Partials       26       25       -1

Impacted Files
Coverage Δ

search_service/proxy/atlas.py
49.03% <100.00%> (-5.25%)
:arrow_down:

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 e9a8cc9...7510640. Read the comment docs.

cc @verdan @dwarszawski

@feng-tao just noticed that should've been merged

thanks @mgorsk1 for the ping, lgtm

Codecov Report

Merging #105 into master will decrease coverage by 0.85%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #105      +/-   ##
==========================================
- Coverage   76.01%   75.16%   -0.86%
==========================================
  Files          17       17
  Lines         613      612       -1
  Branches       76       76
==========================================
- Hits          466      460       -6
- Misses        121      127       +6
+ Partials       26       25       -1

Impacted Files
Coverage Δ

search_service/proxy/atlas.py
49.03% <100.00%> (-5.25%)
:arrow_down:

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 5c864e7...3ae9525. Read the comment docs.


R2R Mapping Designer - Step 1 - "sign in" error and some other enhancements

Please find attached a file with comments about R2R Mapping Designer:

One bug to be resolved, concerning the "sign in" label, and
Three other issues, as "nice to have or consider" - for UI improvement, if you all agree.
Thank you.

thank you, @bkapourani. We will be fixing these bugs and integrating your ideas in the new design.

not relevant in the new transformation tool version


Revert to FOP 1.1

During testing it was discovered that FOP 2.x comes with a major performance degradation. This PR reverts back to FOP 1.1 that was originally bundled, but still uses Gradle dependency management for FOP JAR files.
Signed-off-by: Jarno Elovirta<EMAIL_ADDRESS>

This PR is redundant. There seems to be no performance hit from using FOP 2.1. The original reports were invalid.


Issue not being closed

Hi Peter,
I am having a little trouble using your action. I have a yml file that does some python work, adds a comment, removes a label and my last step is
    - name: Close issue
      uses: peter-evans/close-issue@v1
      with:
          issue-number: ${{ github.event.issue.number }},
          comment: "Auto-closing this issue."
          token: ${{ secrets.GITHUB_TOKEN }}

the response in the action is failure, the output to screen is
Run peter-evans/close-issue@v1
  with:
    issue-number: 27,
    comment: Auto-closing this issue.
    token: ***
    repository: ...
  env:
    issue_number: 27
    pythonLocation: /opt/hostedtoolcache/Python/3.8.6/x64
Adding a comment before closing the issue
Error: Not Found

Not sure what is wrong here as the issue certainly exists.
Best wishes

Hi @MatBenfield
Fairly sure the problem is the comma after the issue number.
    - name: Close issue
      uses: peter-evans/close-issue@v1
      with:
-       issue-number: ${{ github.event.issue.number }},
+       issue-number: ${{ github.event.issue.number }}
        comment: "Auto-closing this issue."
        token: ${{ secrets.GITHUB_TOKEN }}

Ah! Ugh!
Thanks so much.


Is there a way of getting previous messages?

Skype Web and Skype on the phone seems to get a history of messages. I'm assuming there must be a way to get the message history.

I'm going to close this issue. I can get the messages back by editing the get-messages.ts file. I will do a pull request when I have cleaned up my code.

Fantastic, thanks for your future contribution and problem solving:)

Hi @James91
Is your version available somewhere? If you don't have much time right now, I might do it now that I finally got back to this library.
I'll keep the issue open until this feature is merged.

Hi @demurgos
Sorry, I've been away for two days. My code is a bit of a hack together and I need to tidy it up before I would be happy releasing it. I kind of just copied the conversation class and changed it to the message URL. I have added the ability to add a start date too.
I'm in the middle of a project deadline and don't really have time to fix it up for the next few weeks.

Ok, I can totally understand deadline so no problem about it. (I was myself not very active for some time).
I'll assign myself to the issue when I'll start working on it: if your deadlines end before, feel free to send a PR, even it's just a hack we'll be able to iterate.
I am mostly interested about your other issue: #55. Could you check it out quickly?


Add missing PrismaVectorStore filter operators

Summary
Add support for missing SQL operators in PrismaVectorStore class filter:

isNull
isNotNull
like

Fixes #3317

Thank you!