page does not refresh when _components/XXX.tsx is modified

Version
v2.0.1
Platform
Linux
What steps will reproduce the bug?

Enable TSX
have a .tsx page reference a component in _components/
load the .tsx page in browser
edit the component

How often does it reproduce? Is there a required condition?
100%
What is the expected behavior?
the page should refresh to new content
What do you see instead?
The page is stuck. Manual refresh doesn't work. I have to restart lume to see the new page.
Additional information
No response

Hi.
Can I see the code of how you're using the component?
In Lume, components are loaded automatically in the comp variable. Example:
export default function ({ comp}: Lume.Data ) {
  return
<comp.Button />

}

this can be workaround using watch or hmr.

run echo 'import "lume/cli.ts"' > _main.ts on terminal.
change tasks in deno.json to following:

 	"tasks": {
-		"lume": "echo \"import 'lume/cli.ts'\" | deno run -A -",
-		"build": "deno task lume",
-		"serve": "deno task lume -s"
+		"build": "deno run -A _main.ts",
+		"serve": "deno run -A --unstable-hmr _main.ts -s"
 	},

it should work like following:
$ deno task serve
Task serve deno run -A --unstable-hmr _main.ts -s
HMR Process started.
Loading config file file:///home/scarf/repo/etc/esbuild-issue/_config.ts
🔥 /main.js <- /main.tsx
🍾 Site built into ./_site
  1 files generated in 0.01 seconds
  Server started at:
  http://localhost:3000/ (local)
  http://<IP_ADDRESS>:3000/ (network)
Changes detected:
- /_config.ts
HMR Replaced changed module file:///home/scarf/repo/etc/esbuild-issue/_config.ts

In Lume, components are loaded automatically in the comp variable. Example:

I use import Component from '../Component.tsx'

The problem is Deno caches the imported modules and there's no way to invalidate the cache to refresh the modules.
In Lume, this limitation is solved loading the file with a hash (for example, import foo from "./foo.ts#hash123"). Components are loaded in this way so if a file changes, Lume reload the file with a different hash.
Deno added recently a way to do hot module replacement (https://deno.com/blog/v1.38#hot-module-replacement) but it's still unstable and doesn't work well with Lume.

Related: https://github.com/denoland/deno/issues/8327

this can be workaround using watch or hmr.
1. run `echo 'import "lume/cli.ts"' > _main.ts` on terminal.

2. change tasks in `deno.json` to following:

I totally missed this... now my site is mostly stable I don't have a easy way to test this. Hopefully someone else will find this useful.

This does not work for tsx files. It worked with Lume 1, and upgrading to Lume 2 seems to have broken this.
The terminal output this:
Changes detected:
- /scripts/feature-flags.page.tsx
Changes sent to the browser

But the brother never reloads that page.
Even though I can see:

https://github.com/lumeland/lume/blob/972d55cae716a8acd847afc6db949ae4a979ac0d/middlewares/reload_client.js#L112-L115
looks like it's because live reload client doesn't handle extension other than js.


Passing environment variable to React app

Hi There,
Is there way to pass environment variables to React app? In our application we need to pass an environment variable to React app. Usually in React app the environment variable must prefix REACT_APP_, but passing variable at run time doesn't work.
There is option to have environment specific files but then you need different Docker image for each application.
Any solution to this your mind?
Regards,
Syed

Probably the best solution is adding a new sbt setting that would allow passing these variables to underlying npm commands. Should be easily doable.


Polygon txs sometimes fail with "transaction underpriced"

Describe the bug
In using the eth-predict readme (on Polygon), I sometimes encounter an error "transaction underpriced". What's happening: network rejects the tx because it doesn't think you're paying a high enough gas fee.
Workaround in predict-eth: if I retry 1-2 times, it usually goes through.
But we need to fix this properly because:

the workaround is not suitable for fully automated flows
the workaround is poor UX for manual flows like READMEs
and what if it doesn't go through, and we need to pay a higher gas fee? It's not so easy within ocean.py right now

Overall goal
Get txs through more reliably, more often.
Proposed solution

If the error comes back, try again 1-2 times.
If there are still issues, then multiply gas price by e.g. 1.2 and try again.
Repeat (multiply by 1.2, try again) until it works, or until e.g. 50 tries done.

@trentmc I marked this as Critical because if users cannot publish/consume via ocean.py it inflicts directly the ETH-Predict challenge.
For now, we can do this:

There is an environment variable for the gas multiplier. Initially, @calina-c is going to make sure that this variable is correctly used in ocean.py.
In the ETH-Predict mention setting a 2.0 multiplier in ocean.py (@juanpablovs). We don't want to hardcode this as it is used for all the networks.
In the second version of ocean.py(with brownie integrated), there is a config through which we can specify the multiplier for each network, and then we can add a bigger polygon default gas multiplier.

Or, simpler yet: only do (3). This is possible if the next eth-predict challenge only starts after ocean.py with brownie is released. We can set challenge dates as such -- I'm iterating with Ray & Andrea on the dates.
Then the only Q is: when will ocean.py with brownie be released? To help on planning towards that, I've set a meeting among you (Ana) & Calina & I to discuss.

copy that, since it doesn't make sense to have this replicated and we are doing option 3 I am going to close the issue I opened this morning. If you would prefer to leave it open please let me know and I'll reopen it at once, thanks!

Oh, good to know this @trentmc :)
My assumption was that we need to thoroughly test ocean.py + brownie as it includes a big refactoring. If the ETH-PRedict challenge can wait till everything is done and tested we can go on that path. Thank you all!

@trentmc @AnaLoznianu can we close this?


Hand-packaged 'An ode to code duplication' Module Manager 2.6.1

Jenkins is being a douche, author says it resolves locally and we really need this added so I'm merging it.


Add file pining to favorites

What's the Problem?
File exploer now has them i feel files should too.
Solution/Idea

Alternatives

Priorities
No response
Files Version
No response
Windows Version
No response
Comments
No response

@Josh-65 can you give a little more context how this would improve your workflow?

Honestly i dont have a use case for it. But other may. i could also see it beeing useful when im helping others as they are not typically organised well.

@Josh-65 I think that bundles feature does the job for this.

Should this be reconsidered since bundles aren't supported anymore. Had a few people request this on discord too.

Bundles will be replaced with widget for tags.


Create minimal python client library for OpenAI Realtime API

Use official javascript library as example/guide
Not necessary to cover every feature just ones needed for us


failing tests on make test

--- FAIL: TestTCPPortCheckMarksNodesHealthyOrUnhealthy (0.10s)
        Error Trace:    work_test.go:193
	Error:      	Should be true
	Test:       	TestTCPPortCheckMarksNodesHealthyOrUnhealthy

@shubham7saxena These tests are working fine even on the CI build.

we tested it again and seemed to work fine. the tests are flaky 😞

Hmm lets look into this. Closing this issue for now


still pulls in underscore

there is still a underscore dependency in .versions, so it will pull in underscore, altough its not used anymore

@macrozone I'm not sure if .versions files in meteor packages are actually used for anything?
According to meteor list: https://github.com/raix/Meteor-EventEmitter is the culprit here
<EMAIL_ADDRESS>
├──<EMAIL_ADDRESS>(top level)
├─┬<EMAIL_ADDRESS>
│ ├─┬<EMAIL_ADDRESS>
│ │ ├──<EMAIL_ADDRESS>(top level)
│ │ └─┬<EMAIL_ADDRESS>
│ │   ├─┬<EMAIL_ADDRESS>
│ │   │ └──<EMAIL_ADDRESS>(top level)
│ │   └──<EMAIL_ADDRESS>(top level)
│ ├──<EMAIL_ADDRESS>(top level)
│ ├──<EMAIL_ADDRESS>(expanded above)
│ ├─┬<EMAIL_ADDRESS>
│ │ └─┬<EMAIL_ADDRESS>
│ │   └──<EMAIL_ADDRESS>
│ ├─┬<EMAIL_ADDRESS>
│ │ ├──<EMAIL_ADDRESS>(top level)
│ │ └──<EMAIL_ADDRESS>(expanded above)
│ ├─┬<EMAIL_ADDRESS>
│ │ ├──<EMAIL_ADDRESS>(top level)
│ │ ├──<EMAIL_ADDRESS>(expanded above)
│ │ ├──<EMAIL_ADDRESS>(expanded above)
│ │ └─┬<EMAIL_ADDRESS>
│ │   └──<EMAIL_ADDRESS>(top level)
│ ├─┬<EMAIL_ADDRESS>
│ │ └──<EMAIL_ADDRESS>(top level)
│ ├──<EMAIL_ADDRESS>(expanded above)
│ └──<EMAIL_ADDRESS>(top level)
├──<EMAIL_ADDRESS>(top level)
├─┬<EMAIL_ADDRESS>
│ └──<EMAIL_ADDRESS>
└─┬<EMAIL_ADDRESS>
  └──<EMAIL_ADDRESS>(top level)

this is fixed in raix:eventemitter, but i think @raix  did not yet release a new version

its now released as<EMAIL_ADDRESS>

Published 3.0.2 with<EMAIL_ADDRESS>dependency. Run meteor add<EMAIL_ADDRESS>or in some other way ensure that the 1.0.0 version ends up in your .versions file.

Even with 3.0.2 and raix:emitter1.0.0 in my .versions file, I still see this package importing an entire copy of underscore. Not sure this is resolved!


fix: use string for url instead of URL

The reason for that is that URL is putting a trailing / at the file which becomes an issue in downstream libraries such as esplora-client. This is a temp fix, a proper fix will follow.
Note: make sure to pass in ESPLORA_ENDPOINT always without a trailing /, i.e. ESPLORA_ENDPOINT=http://<IP_ADDRESS>:3000

Thanks for fixing! 👍

Thanks for helping debugging it.


Remove rebuild_container tasks and variables in favour of just using systemd

For example looking at https://github.com/Capgemini/Apollo/blob/coreos/roles/mesos/templates/mesos-master.service.j2#L7-L25 it will pull the correct image and start it correctly on a service restart (if the template file is updated with a new docker image)

Closing via https://github.com/Capgemini/Apollo/commit/127772fc5ffc3183dcbbef24e56aa9ed6c93da2d


Add Subscription RP commands.

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[X] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

Command Guidelines

[X] Each command and parameter has a meaningful description.
[X] Each new command has a test.

(see Authoring Command Modules)

View a preview at https://prompt.ws/r/Azure/azure-cli/5121
This is an experimental preview for @microsoft.com users.
(It may take a minute or two for your instance to be ready)
Email feedback to 'azfeedback' with subject 'Prompt Feedback'.


Fix cleaning up the test directory on windows CI

The windows CI runs multiple tests in different build directories as the same user. Since #3290 this means that the tests use the same temporary directory, and the cleanup_test may remove tests from other runs. This adds an environment variable KUZU_TEST_TMP_DIR which can be used to configure the temp directory, and uses it to create job-specific test directories on the windows runner.

I've updated this to do the same for the macos runner.


Add interface for bindInput

What's this PR do?
Adds an interface for bindInput.
Where should the reviewer start?
FormContainer.tsx
What are the relevant tickets / issues?
#18
How should this be manually tested?
There is no functional change, so manually test that all works as it did previously.
Any background context you want to provide?
The idea for this is when bindInput needs to be passed down as a prop, the type definition can be defined.
Questions
Will this be exported (and thus importable) in the npm package?

Happy to merge @vitkon ?

sorry for a delay, I'm on it now ;)

1 naming suggestion. See what you think @mattdell
Happy to merge otherwise.

Do we need to update the package semver? Also if you add me to npmjs.org I can deploy as well.

@mattdell try to publish

@vitkon published! 🎉


Ability to send any TF Build command

Since some TF Build commands are undocumented, such as results.publish, would you be willing to  expose a public method so that Cake users can opt to type their own command strings? That way users can take responsibility to use TF Build commands which Cake does not have because they are new or undocumented.
This is Cake's current implementation, used internally:
https://github.com/cake-build/cake/blob/f11b299d7d4b9782874cb0a9b6c9dee03219ac8b/src/Cake.Common/Build/TFBuild/TFBuildCommands.cs#L321-L328
It's doing the equivalent of vsts-task-lib's command (see also _writeline, TaskCommand.toString), so I'd suggest that we keep the same parameter names and types as the ones which are exported from that public library.
Maybe something like this?
namespace Cake.Common.Build.TFBuild
{
    public interface ITFBuildCommands
    {
        void WriteWarning(string message);
        void WriteWarning(string message, TFBuildMessageData data);
        void WriteError(string message);
        void WriteError(string message, TFBuildMessageData data);
+       void WriteCommand(string command, IReadOnlyDictionary<string, string> properties, string message);
        void SetProgress(int progress, string currentOperation);
        void CompleteCurrentTask();
        void CompleteCurrentTask(TFBuildTaskResult result);
        Guid CreateNewRecord(string name, string type, int order);
        Guid CreateNewRecord(string name, string type, int order, TFBuildRecordData data);
        void UpdateRecord(Guid id, TFBuildRecordData data);
        void SetVariable(string name, string value);
        void SetSecretVariable(string name, string value);
        void UploadTaskSummary(FilePath markdownPath);
        void UploadTaskLogFile(FilePath logFile);
        void LinkArtifact(string name, TFBuildArtifactType type, string location);
        void UploadArtifact(string folderName, FilePath file);
        void UploadArtifact(string folderName, FilePath file, string artifactName);
        void UploadBuildLogFile(FilePath logFile);
        void UpdateBuildNumber(string buildNumber);
        void AddBuildTag(string tag);
    }
}

And it would be used like this? (compare TestPublisher.publish and Microsoft/vsts-tasks’s publishtestresults.ts)
TFBuild.Commands.WriteCommand("results.publish", new Dictionary<string, string>
{
    ["type"] = "NUnit",
    ["runTitle"] = "Integration tests",
    ["resultFiles"] = testResultFile.FullPath,
    ["publishRunAttachments"] = "true"
}, message: null);

@cake-build/cake-team what are your collective thoughts on this one?

Sure, it gives more flexibility. I’ve sometimes wished for same functionality in TeamCity provider also.

@gep13 and @jnm2 i have similar pull request #2109 for Issue #1609  for this as well to add upload test results and code coverage. The thing is that some of these commands are only support in TFS2017+ or VSTS.

@louisfischer I think the concern Gary expressed in the chat was that committing to an undocumented protocol could result things breaking and it being Cake's responsibility, whereas this PR would allow it to be the user's responsibility to respond to a breaking change in an undocumented protocol. I do like your PR too though. Wouldn't mind having both! 😃

This would be great. I just had to resort to manual logging commands with Information to upload artifacts by a directory path since there aren't Cake commands to support it yet. Something like this would be a lot less ugly.

@gtbuchanan What were you looking to do that wasn't supported? We've already got the UploadArtifact aliases for uploading files (and creating artifacts) that you can see in the API here.
Just wanting to understand where our gaps are :)

@agc93 UploadArtifact in Cake currently only accepts FilePath but Azure Pipelines also supports directories. See:
https://github.com/Microsoft/azure-pipelines-tasks/blob/master/Tasks/PublishBuildArtifactsV1/publishbuildartifacts.ts#L50
https://github.com/gtbuchanan/repo-template-cs/blob/master/build.cake#L200

@gtbuchanan I think that mandates a separate issue.

@patriksvensson I agree. I was simply responding to the question. Haven't had the time to create another issue/PR yet. Nonetheless, my main point was this feature would be helpful in the interim while waiting for any updates in the future.

@gtbuchanan Absolutely. I just don't want a required change to get lost in the comments 😄


⚠ Cioos Pacific CKAN has degraded performance

In eaff629, Cioos Pacific CKAN (https://catalogue.cioospacific.ca) experienced degraded performance:

HTTP code: 200
Response time: 9004 ms

Resolved: Cioos Pacific CKAN performance has improved in 1959785 after 11 minutes.


AudioPlayerDelegate doesn't get called with time updates playing a live stream

Hi.
I've observed that audioPlayer(_ audioPlayer: AudioPlayer, didUpdateProgressionTo time: TimeInterval, percentageRead: Float) doesn't get called while playing a live stream. I've checked the code and it's because the player cannot know the total duration, therefore it skips calling the delegate.
In my opinion, this callback is still useful even if you don't know the total duration of the stream. For instance, for a given live stream we know which song is being played at a certain moment, if we can know when the stream started and the progression so far. But maybe there is another, easier way to accomplish this?
What do you think? Thanks.

Hi Daniel,
Correct me if I'm wrong but I think the time is only relative to when the user started playing the livestream isn't that right? I mean at the time he presses "Play" the time is 0, after 10 secs it's 10 etc etc. How would you know what is being played based on this relative time?

Yes, I think that's correct.
Although you could know what's being played if you save the starting date (when the user presses play) and track the progress. This is useful as well if you want to seek back to a certain time, because if it's later than when you started, you already have the new current date just adding the progress.
Another approach I wanted to follow is to get the timed metadata from the stream and call the backend with that info to know the data of the whole item, that's why I added the other PR that you already merged.
But in any case I think it's a nice feature to be able to track the progress of a stream, and right now that's not possible if it doesn't have a duration.
BTW, love the new structure of the code, it's so easy to go through it and make changes, congrats 💯

Makes sense. Working on that right now.


Types not following properly for properties of a function

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

VS Code Version:
OS Version:

Steps to Reproduce:

Authoring my library.

Code is a bit big, but i'll try to narrow down. I've built a library where functions have properties attached which are functions as well. such as:
const getMessage = async (messageId: string) => { /* call to backend */ }
getMessage.locally = (messageId: string) => { return fromCache(...) }

Once compiled, the .d.ts for this method is generated and everything looks fine.

Using my library

In another project (imported with npm link), I can hover the function in vscode and see the .d.ts file properly:

but when using, the auto-completion does not resolve the typing according to the .d.ts that it's able to read:

How to make VS code to read my types properly ?

Please share a minimal, self contained example of the problem


Release?

Hey @TomAugspurger,
would you be happy if we'd release the current status as 0.3.0 or so?

Sure.
We'll need to bump https://github.com/stac-utils/stac-geoparquet/blob/main/stac_geoparquet/init.py#L2.
And we'll need to figure out the upload to PyPI permissions. In TomAugspurger/stac-geoparquet I had a secret to upload to PyPI. IIRC, the stac-utils organization might have a secret we can use, similar to https://github.com/stac-utils/pystac/blob/ebe5c6b32c38c9f4a2acfb7936fc84c026f67482/.github/workflows/release.yml#L28. I'm not sure if that secret is available in this repo or not (I can't seem to view secrets. Can you?)

I bumped the version number.
I see a TWINE_PASSWORD, PYPI_STACUTILS_PASSWORD and PYPI_STACUTILS_USERNAME in the repo/org secrets.
So should it be
          TWINE_USERNAME: "__token__"
          TWINE_PASSWORD: ${{ secrets.TWINE_PASSWORD }}

or
          TWINE_USERNAME: ${{ secrets.PYPI_STACUTILS_USERNAME }}
          TWINE_PASSWORD: ${{ secrets.PYPI_STACUTILS_PASSWORD }}

in the publish.yml? Or does both work? Then it should work already. @TomAugspurger

Also updated the repo URL in various places: https://github.com/stac-utils/stac-geoparquet/commit/09f545c7aac938ba7d879f67cebc5052ca382ff3

PyPI acutally has a new feature for "trusted publishers" that I'm trying out https://docs.pypi.org/trusted-publishers/adding-a-publisher/.

With https://github.com/stac-utils/stac-geoparquet/pull/22 merged, I think things are set up to upload a release on GitHub releases. I probably messed something up, but we'll see. I'll tag a release now.

https://github.com/stac-utils/stac-geoparquet/actions/runs/5600077559/jobs/10241944419 is the GitHub action building and uploading the release to PyPI.

Close :-)
It works locally for me though, weird...

Maybe https://github.com/stac-utils/stac-geoparquet/commit/e073fe08c026696f2a7328a743294e1f0358e73a got it. I'll try a 3.0.1.

Looks like we need to change this?
https://github.com/stac-utils/stac-geoparquet/issues/21#issuecomment-1642111936

I'm going to try one more time. I think I had publish.yaml in one place and publish.yml in another.

Success! https://pypi.org/project/stac-geoparquet/0.3.2/

Thank you, Tom!


nix-env -qaP stops working

Hi,
when I add the musinx-channel, I get the following error:
# nix-env -qaP
error: cannot auto-call a function that has an argument without a default value (‘config’)

Interesting.
I must to confess that I don't have musnix as a channel locally, so I haven't had a chance to test this approach myself.  It was a change we added in https://github.com/musnix/musnix/commit/7c01dc4142e98502c905abf90cc322c9d19051e5.
I will go ahead and see if I can track down the problem.
cc @yegortimoshenko - any ideas?

Thanks for picking that up Henry,
the usage as outlined in the README.md actually works pretty well. Please excuse my brevity in the last post. Thank you for providing us with musnix.
Regards
Tobias

You're welcome!  Your post was quite helpful, however brief 😎 .

Okay, I can reproduce this over here.
@milnet2: what do you get from nixos-version?

$ nixos-version
17.09.3008.c831224528c (Hummingbird)

Hey @henrytill, I am getting this as well.
$ nix-env -qaP
error: cannot auto-call a function that has an argument without a default value (‘config’)
$ nixos-version
17.09.3205.12512f25e9e (Hummingbird)

I've added a pre-18.03 branch to address problems with compatibility on master.
You can update your channel and try using that branch.  If you have any issues, don't hesitate to open a new ticket.


Detachable joint: support for nested models of the same name

🦟 Bug fix
Fixes #
Summary
Not sure if this is a bugfix or a feature tbh. Currently, if we are loading multiple models with child models that have the  same name, the detachable joint will pick the wrong one. This is because given the current logic we only check for a child model by name. My understanding is that we wrote this plugin so we can detach sibling models.
The proposed fix in this PR is that if there are multiple models with a similar name, the system will look for the model which is a child of the parent model. This is by no means the best solution, as I do not handle grandchildren or the case where there are multiple children of the same name but it is a solution. Would be happy to have other suggestions as to how I should handle this.
Checklist

[x] Signed all commits for DCO
[x] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[x] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

Removing beta label. Let me know if you're planning to get this in before Harmonic is released.

I'll address this after the code freeze.

@arjo129 Do you want to keep this PR?

Yep. Lets try to get this in. I'll address the feedback.

Looks like there's a windows test failure (https://build.osrfoundation.org/job/gz_sim-pr-win/741/testReport/(root)/DetachableJointTest/NestedModelsWithSameName/). @arjo129 would you be able to take a look?

Will do in the morning.
On Tue, Aug 13, 2024, 11:13 PM Addisu Z. Taddese @.***>
wrote:

Looks like there's a windows test failure (
https://build.osrfoundation.org/job/gz_sim-pr-win/741/testReport/(root)/DetachableJointTest/NestedModelsWithSameName/).
@arjo129 https://github.com/arjo129 would you be able to take a look?
—
Reply to this email directly, view it on GitHub
https://github.com/gazebosim/gz-sim/pull/1097#issuecomment-2286500837,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEEMQFRJFHB2XWTWV3XWZDZRIPB7AVCNFSM6AAAAABMC6IWMOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBWGUYDAOBTG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Looks like the other test cases in the detachable_joint integration test are actually disabled on Windows:
TEST_F(DetachableJointTest, IGN_UTILS_TEST_DISABLED_ON_WIN32(LinksInSameModel))
Looking at the test log, the physics engine is not correctly loading in windows, which could cause the test failure:
[Err] [C:\J\workspace\gz_sim-pr-win\ws\gz-sim\src\systems\physics\Physics.cc:761] Failed to find plugin [libignition-physics-dartsim-plugin.so]. Have you checked the IGN_GAZEBO_PHYSICS_ENGINE_PATHenvironment variable?
@azeey I think we can disable the new test case on Windows for now and figure out the underlying issue with the physics engine not loading post-release?

https://github.com/gazebosim/gz-sim/pull/2523 disables the added test case on Windows


Add missing make config-reset to "useful make commands" in developer-workflow

…

Summary
I was chatting with Jesse Hallam who recommended this command.  When I went to go read documentation on it, I couldn't find it anywhere.  This seemed like a good place for it so others know about it!
Ticket Link
This is a one line fix, so there's no ticket.  I believe this is OK per the guidelines, but lemme know if not!

@justinegeffen, not much to "review" here, but would love your second set of eyes on a small but useful change like this :)


[WAF] Block response content type based on Accept header

Addresses PCX-5269.

Addresses PCX-5269.

What si they raison ?

Heyo @pedrosousa, going to close out as this is a few years old.


HDFS-13775. Add a conf property to retry dns reverse lookups during datanode registration.

406

 Quality gate: Failed
4 Issues

 4 Code issues

Details

 Quality gate: Failed
3 Issues

 3 Code issues

Details


In iOS I cannot initialize the library

RN version: 0.60.5
Mixpanel RN library: 1.0.1
Hi there! In Android I could do the implementation well everything worked for me following the documentation but in iOS I cannot initialize the library with the function
configMixpanel = async () => {        this.mixpanel = await Mixpanel.init("Your mixpanel token");    }
in the debug I see that this function returns it undefined

what can it be?
I'll appreciate your help
Regards

hi @pixsolution , would you mind providing more sample regarding how you put configMixpanel? Also you may want to try our demo app to see if that works for you. https://github.com/mixpanel/mixpanel-react-native/tree/master/MixpanelDemo

Hello @zihejia  Thanks for your reply, suddenly start working on iOS perfect and now stop working on Android (the strange thing is that it does not show any error) the only think I do was pass from version 1.0.1 to 1.0.2. I think it's a problem with library thats no sending the events to Mixpanel. If It was my code never work or show an error...

hi @pixsolution , If iOS is working fine, in that case, I will close this one and let's use this issue to track instead. https://github.com/mixpanel/mixpanel-react-native/issues/27


'callHackageDirect' not defined in Adding Packages example

error: attribute 'callHackageDirect' missing, at /home/patrick/ob/default.nix:21:18
(use '--show-trace' to show detailed location information)

Can you show what command you ran to get this error?

Are you using master branch of obelisk? Try ob thunk .obelisk/impl --branch develop to use develop instead and see if this happens.

Thanks for the quick response!
ob run after following the instructions in the readme, i.e. master: nix-env -f https://github.com/obsidiansystems/obelisk/archive/master.tar.gz -iA command
# ob run -v
Starting Obelisk </nix/store/ryihixldr6wb22ypp8hxvscivagvb9b1-obelisk-command-0.1/bin/.ob-wrapped> args=["run","-v"] logging-level=Debug
Handing off to ./.obelisk/impl/.attr-cache/command.out/bin/ob
Starting Obelisk </nix/store/ryihixldr6wb22ypp8hxvscivagvb9b1-obelisk-command-0.1/bin/.ob-wrapped> args=["--no-handoff","run","-v"] logging-level=Debug
Creating process runNixShellAttr: 'nix-shell' '-A' 'shells.ghc' '--run' '/nix/store/ryihixldr6wb22ypp8hxvscivagvb9b1-obelisk-command-0.1/bin/.ob-wrapped --no-handoff --verbose internal run-static-io 01489bf44eb91e7f793dea3f236c5989'
error: attribute 'callHackageDirect' missing, at /home/patrick/ob/default.nix:43:18
(use '--show-trace' to show detailed location information)

After doing the thunk update everything seems to be working as expected.

This isn't the first time that the mismatch between obelisk's default of using master and the GitHub configuration's default showing develop has caused confusion. Perhaps we should consider changing that.


Try out weechat

[[https://github.com/thefinn93/signal-weechat/blob/master/README.md][signal weechat]]
[[https://github.com/wee-slack/wee-slack][slack weechat]]

Integrations are very fragile and IRC is on its way out - I'll use a matrix bridge to freenode if I really want this

Integrations are very fragile and IRC is on its way out - I'll use a matrix bridge to freenode if I really want this


Allow inputs and ouputs to be rounded to 4 decimal places instead of 2

Hmm, this is doable, though I'm curious what the use case is. Where do you have a use for such small fractional item rates? The underlying values are all full precision, it's just the display that is rounded.

First off I just want to thank you for this awesome project! Its my first time with Rust but It has been a great intro.
I cloned the repo made the change and have really been enjoying it. I like to get as precise as possible, not sure if it matters or not at that scale? Puts my mind at ease I guess?
I thought maybe a restriction could be set on the clocking to 6 decimal places as well.

If you're interested in trying a fix yourself, the rounding for the balance display is done here (which I guess you found already): https://github.com/satisfactory-accounting/satisfactory-accounting/blob/6d57786fcfd103e72672be3d3aab54ae75a2aa31/satisfactory-accounting-app/src/node_display/balance.rs#L87
I for general release I would make it a setting, sort of like how BalanceSortMode is handled:

Part of user settings here:
https://github.com/satisfactory-accounting/satisfactory-accounting/blob/6d57786fcfd103e72672be3d3aab54ae75a2aa31/satisfactory-accounting-app/src/app.rs#L51

Configuravle here:
https://github.com/satisfactory-accounting/satisfactory-accounting/blob/6d57786fcfd103e72672be3d3aab54ae75a2aa31/satisfactory-accounting-app/src/app.rs#L1037

Updated here:
https://github.com/satisfactory-accounting/satisfactory-accounting/blob/6d57786fcfd103e72672be3d3aab54ae75a2aa31/satisfactory-accounting-app/src/app.rs#L575

Regarding the precision of the clock speeds, I don't believe those are rounded for display at all. However all balances are currently calculated using 32-bit floating point, so all numbers in the app currently have a maximum precision of about 6-7 decimal digits (total including the whole part of the number not just the fractional part). I can enter and view clock speeds up to six digits. Keep in mind that clocks are also clamped to between 1% (0.01) and 250% (2.5) to match the game, so you can only be precise between those, but things like 63/64 (0.984375) work fine, aside from that there isn't room for them and they get hidden under the next box somewhat

32 bit floating point does mean there's not really value to having precision of higher than ~6 digits since anything more will start getting rounded away regardless. Using 64 bit float increases that to ~15 digits, though I'm not sure its worth it for this application.

I’ll play around and see what I can do with the overlapping then. I have encountered this on small viewports so I expected it to be more than just changing the rounding. I’ll get the setting in. I’ll let you know what I come up with.

Don't worry about sizing the output field, I'm doing some more significant changes in the branch prefactor, which includes solving sizing for input elements (aebba1a503788eba2c6511b98f0c6820d9ea466c and e3189983123d3016e93e922924689a8d191bdda6)

I will revisit this after my current project, #12, and v1.2.11 is released.
I want to add two options:

Rounding precision (0-7 digits)
Bool of whether "hide empty balances" should be based on the precise value or the rounded value.

If 1.3.x does end up using integer fractions (which I am exploring as a possible option), then it will also get more balance display options, such as displaying exact fractions like 2⅓ with no rounding needed at all.


Fix a typo

currentContent should be currentContext

#sign-off

@nitinme: I'm sorry - only the author of this article, @edjez, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Updating rly tag to v2.2.0

Description
The existing tag we were using no longer exists.
Commit Message / Changelog Entry
see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

I was incorrect, the image is still there. Closing this for now. 2.2.0 doesn't work with both event formats


Feature Request

Please make a a feature in which I will be able to share my history results .
Steps to reproduce the behavior:

Go to 'History' from the three dot menu
Press and hold on the any result
There should be an option to share that particular history

Android (please complete the following information):

Device: Realme 6
OS: Realme UI 2.0
App Version 2.1.0

Additional context
Add any other context about the problem here.

Added to latest beta 2.1.2


Add metadata entry for the type of surface

Surfaces can contain measurements from:

The same sample
Multiple samples of the material

We should collect this information. My suggestion is to add it as metadata to the surface, and add a dropdown menu that allows to select "This surface contains measurements from a single specimen", "This surface contains measurements from multiple specimen of the same material".

The idea of a digital surface twin is that all data is from the same specimen.


WIP: Update .travis.yml, test on 0.3 & 0.4 for now

I haven't had a chance to debug the segfault on 0.4 any more.
I've put this on a branch in the main repo in case anyone tries to get travis testing working on 0.4 again.

FYI @garborg, I just rebased this branch onto master to take another look at it.

FYI @garborg, I just rebased this branch onto master to take another look at it.

The tests on this pass for 0.3 and get the same results for 0.4 as the older Travis build system, so I'm going to go ahead and merge this into master. This seems much more elegant than the previous travis file. I still hope to figure out how to fix that segfault on 0.4 someday, but it's tough when I've never managed to reproduce that problem locally.

The tests on this pass for 0.3 and get the same results for 0.4 as the older Travis build system, so I'm going to go ahead and merge this into master. This seems much more elegant than the previous travis file. I still hope to figure out how to fix that segfault on 0.4 someday, but it's tough when I've never managed to reproduce that problem locally.

Oh, I know, I bet the 'build failing' statuses triggered by the segfault are bugging you. Thanks for merging this.

Yep, they drive me crazy. But at least it works on 0.3 for now.


code.length

Hello
There is  code.length in foundry exercises .  What does it mean exactly?
For example
any address.code.length returns 118
sincerely
VeriDelisi

That is the number of bytes of code at that address. address(x).code returns a bytes memory variable with the code at the address, and address(x).code.length returns the length of those bytes

What does it mean 118?

118 bytes, please see the solidity docs or https://www.evm.codes/#3b?fork=shanghai for more info

There is no information in the evm playground.
118 is gas cost?

If address(x).code.length returns a value of 118 it means the code at address(x) is 118 bytes in length. I recommend reading the solidity docs and some introductory EVM material :)


Logging Workflow Emits

This Non-functional requirements can be used functional requirements such as:

Data Analytics
Notifications
Activity Streaming

Working on mobile notification ES-36 for team happiness requires this PR to be merged first.


SCons: Only use alias if SCons v4.3 or later

Fixes #91049
Alias array support was added in 4.3, so earlier versions will require the old syntax. Implemented this via a conditional against scons_ver, which is now evaluated at the very beginning of the file.

Thanks!

Hi, I'm on Scons v4.0.1 and getting this error on running scons platform=linuxbsd:
TypeError: SConsEnvironment._get_major_minor_revision() missing 1 required positional argument: 'version_string':

Is this unintentional or should I just update my scons to the latest version?

Hi, I'm on Scons v4.0.1 and getting this error on running scons platform=linuxbsd:

I just got the same thing on Scons v4.4.0 on Windows scons platform=windows.
I updated to v4.5.0 using python -m pip install --upgrade scons==4.5.0 and it worked fine; so it looks like 4.5.0+ is the new requirement.

This will be fixed by https://github.com/godotengine/godot/pull/91080.
But in general it's advised to use latest scons if you can, it tends to perform better.


Convert Windows style path to a unix style path when pasting in WSL

Description of the new feature/enhancement
On pasting windows style path (C:\Program Files\Git) in WSL profile, it should automatically be converted to unix style WSL path (/mnt/c/Program Files/Git)
This is a feature already available in cmder terminal emulator which I miss in Windows Terminal

Proposed technical implementation details (optional)
WSL provides the "wslpath" utility which does just this. I'm not sure how feasible it is, however I'm thinking if we can listen to paste events when in WSL profile, we could identify the argument as a windows style and pass it to the wslpath utility.

Thanks for the suggestion! This is actually already being tracked by another issue on our repo - please refer to #1772 for more discussion.
/dup #1772


Add documentation for GraphQL API

Fixes https://github.com/wso2-enterprise/choreo/issues/14761

Fixes https://github.com/wso2-enterprise/choreo/issues/14761


move all plate 1 waves into one folder

In this PR, I moved all of the waves for plate 1 into one folder as to separate it from the new plate 2 data coming up. I added a README for the Plate_1_data module to explain the different waves and changed the main README to include that there will be multiple plates.

@gwaybio Thank you for the review! I am merging now!


GeoDCAT-AP update for DCAT-AP 3.0

Dear GeoDCAT-AP team,
I know DCAT-AP 3.0 has only been out a couple of days but are there any plans to align GeoDCAT-AP with the new version of DCAT-AP (and by extension, DCAT-3)? And if so, what would roughly be the timeframe?
I'm working on a cataloguing project focussing on environmental datasets. Our principal reference model is DCAT-AP in order to be compatible with various national and European data catalogues, but we also require the specific scope of GeoDCAT-AP to align with INSPIRE-related requirements for some datasets. We have several use cases for the new DatasetSeries in DCAT-AP and would welcome being able to remain fully compatible with both profiles.

@daniwelter , work on aligning GeoDCAT-AP with DCAT-AP 3 is now underway:
https://github.com/SEMICeu/GeoDCAT-AP/labels/release%3A3.0.0

@daniwelter the GeoDCAT-AP revision timeline will be announced on the introductory webinar, feel free to join.

As this is not an actual GeoDCAT-AP issue, and the revision is underway, I am closing this. The progress on DatasetSeries can be monitored in #71 and #72


fixed bugs for responsive view

Chat - Button in last line need margin to down
Solved: added margin-bottom 30px

Manage Users - Additional users below first user have missing title headers in left side
Solved: added the headers within each user's row when the screen size is detected as mobile

Manage roles - Never ending loop occurs and title headers below first role are missing
Solved: No loop detected now, it seems that backend works slowly, added headers in mobile for all items

Domains page - missing title headers below first one
Solved:  added headers in mobile view for all items

Model Viewer is too high in height
Solved: added css

Photo/Video - close buttons were hidden, added styles for showing in mobile view

Hi,
3D Model Viewer not looking good in my Full HD display.

And not with small display resolution.

Also close modal button is missing.


Pr2 engine scope

Support for configuring the session scope of SparkSQLEngine.

@xjzrc could you fix the conflicts?

@xjzrc could you fix the conflicts?

@yaooqinn I have fixed the conflict


Simplify database config by having a single instance of each

We currently have separate Postgres instances for each service that needs a database, and eventually a dedicated backup container for each of those databases... This is a complex configuration to maintain, making backup and updates more difficult while simultaneously using more resources than needed. I think we should scale that down to a single instance, shared by all containers that need it (on separate databases and users of course)

https://www.amarjanica.com/docker-multiple-postgres-databases


RedHat Network fixes and documentation

Add a rhn_config parameter for non-CentOS systems that need the RHN configurations in rhn.repo
Remove mention of rhnrelease (unused) in favor of typerelease
Set default of typerelease to $release in repo.pp (as described in comments)
Document using the type parameter for rhn repos
Set default http_proxy and https_proxy to ' ' to workaround bug http://projects.puppetlabs.com/issues/19848
Update README.redhat.markdown and README.markdown (including examples)
Clean up lint errors

So sorry about the delay!  I have squashed and rebased, but I haven't done any testing yet.
I'm not too familiar with the process - can you see the updates in my pull request automatically, or do I need to do anything so you can see the changes in my branch?

Please ignore this pull request for the time being while I do a couple of tweaks.  Let me know if you'd prefer me to close this and re-open.

I attempted to rebase and squash commits, and it appeared to work for all my commits, but I'm not sure why it's showing previous commits as well.  Do I need to re-branch and try again?
I also noticed that its stating that it contains merge conflicts that must be resolved, but I've tried rebasing / squashing multiple times, so I'm not sure if there's something else I need to do, or if that's fixed on the upstream side.

I am so sorry - I have literally rebased and squashed 4+ times, and I still can't seem to get it to the right place.  For some reason, now the extra commits are back but the pull request can be automatically merged.
I have been attempting to follow the instructions here:
https://github.com/edx/edx-platform/wiki/How-to-Rebase-a-Pull-Request
Is there a better choice?  I've read that since the commits have been pushed up to github, I may need to rebase / squash / create a new branch?
I think I'm spending more time on trying to get the commit squashing to work than I did on updating the code. :/

@esalberg hrm, those docs look right. It may be easier to just close and reopen this. If you could close this when you create a new PR and mention the PR number we'll take a look. Thanks!

I got it! I think.  Please take a look.
(The missing piece was that I needed to run "git pull upstream master" before rebasing on my local master.  I thought I'd done the equivalent, but obviously not - as soon as my local master reflected that it was not behind the upstream master, the rebase appeared to work.)

Looks good, thanks @esalberg !


Update DynamoDB Table Billing Mode query for AWS CloudFormation

Platform
AWS CloudFormation
Provider
AWS
Description
Updating query to handle when the "BillingMode" is an array.

Mistook an if condition in a yaml file, for an array with this attribute. Upon looking at the documentation, the attribute "BillingMode" can only be a string.


Use an optimized crc32 library which is faster

Needs testing to ensure that this is actually faster (and is still correct), but may solve #255.

We may also want to check what architectures this will run on, because it uses CPU instructions that may not be available everywhere, and I don't know if it does fallbacks.

On a quick glance it uses +build flags and implements the stdlib version as a fallback, so we should be OK.

OK, let's run some benchmarks!

My benchmark shows no difference; possibly vagrant does not support the necessary instructions. Since golang profiling is broken on native MacOS, we may have to run something on one of our servers.

Even on a bare-metal server whose /proc/cpuinfo says it supports SSE4.2 I see the same amount of time spent in the fallback method... not sure what else I need to do to actually make the optimizations run.

Welp, found a bug in our code now when benchmarking. Another PR incoming.

Short version is: this basically removes CRC32 from the profile entirely, it drops from 15-20% of the CPU time in my test to a statistically insignificant number of samples. Sold.
Also, the upstream author has submitted this to Go, so there's a very good chance Go 1.6 will have this optimization in the stdlib.

:clap:


Website mwl.io is not allowed to be shown within an iframe

Not allowed to be shown in an iframe

we keep those sites in as we still crawl these for kagi results


[Bug]: Failed to convert to YAML

Describe the problem
I am trying to use the "Get-WGInstalled" command and keep getting this output for mRemoteNG and for Putty

After that message i receive alot of information about those packages (maintainer, version, description, etc) but its all yellow and not correctly formated
Expectation
No response
Additional Information
No response
PowerShell version
5.1
Platform
Windows 10 Pro or Enterprise
Additional Checks

[X] You are using the latest version of this module.
[ ] You have read this repository's README file.
[ ] You have read full help and examples for the command you are having problems with.
[X] You are running PowerShell in an elevated session.
[X] You are running in a traditional PowerShell console or Windows Terminal

Here is an example of the winget -list command showing correct output for mremoteng and jdk, though putty gives the yaml error and jdk doesnt. Its the same for putty

Parsing non-English results is an ongoing issue. I'm working on another set of fixes. It is difficult for me to test Winget running under different cultures, so I may need your help at some point to run some test code for me.

Parsing non-English results is an ongoing issue. I'm working on another set of fixes. It is difficult for me to test Winget running under different cultures, so I may need your help at some point to run some test code for me.

Sure, i am willing to help and make this a better tool

Parsing YAML, especially under different cultures, is practically an impossible task. I'm still hoping the Winget PowerShell module will eventually make this module moot.


removes material you option after flashing

i flashed this and i love it one issue tho this breaks material you (pixel expereince) material you works but i  cant configure it after installing this module

Huh? Strange, no Material You breaking here on fluid also people had tried on different rom like pixel stock, PE Rom also no one report this.
Try without installing Google Styles and Wallpaper

Material you works
But the styles and wallpapers app has no option to change it
On Sat, Jan 8, 2022, 6:06 PM Shivan Singh @.***> wrote:

Huh? Strange, no Material You breaking here on fluid also people had tried
on different rom like pixel stock, PE Rom also no one report this.
Try without installing Google Styles and Wallpaper
—
Reply to this email directly, view it on GitHub
https://github.com/Kingsman44/Pixelify/issues/42#issuecomment-1008002080,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARKDURFWPLFA5ILVUFG5GU3UVBAHPANCNFSM5LQUQ7PQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Huh? You mean change colours?

That does not exist for me
Can you send me the apk of wallpaper app with apk extractor?
Also if I uninstall the module pixel launcher keeps crashing and won't work
On Sat, Jan 8, 2022, 6:59 PM Shivan Singh @.***> wrote:

Huh? You mean change colours?
[image: Screenshot_20220108-202805_Wallpaper style]
https://user-images.githubusercontent.com/58323485/148648965-572f354a-4fe2-4f36-9509-cf6e8b4ce8f7.png
—
Reply to this email directly, view it on GitHub
https://github.com/Kingsman44/Pixelify/issues/42#issuecomment-1008015652,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARKDURBRBYE6A64NMDUNTI3UVBGMTANCNFSM5LQUQ7PQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Also if I uninstall the module pixel launcher keeps saying not responding
and it won't work and wallpapers app uninstalls if I reflash the module
pixel search is broken and can only search apps and wallpapers app is gone
and when I install it it has no material you option also g photos does not
work it says it's backing up for free but it's actually consuming storage
this is spoofed as pixel 5
On Sat, Jan 8, 2022, 9:25 PM Zainullah Khan @.***> wrote:

That does not exist for me
Can you send me the apk of wallpaper app with apk extractor?
Also if I uninstall the module pixel launcher keeps crashing and won't
work
On Sat, Jan 8, 2022, 6:59 PM Shivan Singh @.***>
wrote:

Huh? You mean change colours?
[image: Screenshot_20220108-202805_Wallpaper style]
https://user-images.githubusercontent.com/58323485/148648965-572f354a-4fe2-4f36-9509-cf6e8b4ce8f7.png
—
Reply to this email directly, view it on GitHub
https://github.com/Kingsman44/Pixelify/issues/42#issuecomment-1008015652,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARKDURBRBYE6A64NMDUNTI3UVBGMTANCNFSM5LQUQ7PQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

not responding or crashing??
Update Google Private compute services from playstore to fix search....
This one https://play.google.com/store/apps/details?id=com.google.android.as.oss

It works now
But pixel search is still broken :(
On Mon, Jan 10, 2022, 12:56 PM Shivan Singh @.***>
wrote:

not responding or crashing??
Update Google Private compute services from playstore to fix search....
This one
https://play.google.com/store/apps/details?id=com.google.android.as.oss
—
Reply to this email directly, view it on GitHub
https://github.com/Kingsman44/Pixelify/issues/42#issuecomment-1008659264,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARKDURG3ERDX3LTWEM2VUGDUVKNKDANCNFSM5LQUQ7PQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

but i cant uninstall the module it says pixel launcher not responding please help :(

This should be fixed on pixelily v2
feel free to reopen or create new issue if doesn't


Save document while composing

Create a new document

Related: https://github.com/mintterteam/mintter/issues/1251


OSX 10.13.3, Textual 7.0.8, ZNC bouncer -- doesn't seem to be working?

Hello, very excited about this plugin.  System is above, I followed the install instructions and restarted Textual but I see no preference panel and chat hasn't changed.
Not sure which part of my system might be causing the problem.

Any chance you meant 10.13.3? 10.3.3 is Panther which is pretty old. Either way, the plugin will likely need updates for High Sierra.

Yes, I meant High Sierra :)

Yeah I also have the same issue. I'm running macOS 10.13.3 and Textual 7.0.8.

I have a computer running High Sierra now, so should be able to take a look at this pretty soon.

wait @c17r how did you fix it?

Didn't, just closing it out

Apologies for never getting to this, I haven't used Twitch in a few years 😢
If anyone is interested in taking over plugin maintenance, feel free to shoot me an email (address on my profile)


making either store or config

fixes https://github.com/sopherapps/pydantic-redis/issues/29

I guess this is no longer necessary since we will have only one way of initializing the redis store as shown by #35


[WIP] docker-compose up

Fixes #39

Thanks a lot @icyflame for the follow-up. 😃
Feel free to send in a fix anytime you get time.
I hope you have push access to this branch, if not, let me know.

@kshitij10496 I opened a new PR. :slightly_smiling_face: I hope that's okay? Your commits are preserved in that as well.


X-Forwarded-For & X-Real-IP = Addr of GKE Ingress Controller

When using the example for NGINX on GCE, everything seems to work, but I am not getting the headers from the original client. It appears that the values for
X-Real-IP & X-Forwarded-For are the private ip addr of the gke ingress controller.

@DocBradfordSoftware if you are using NGINX example then you are actually using Nginx Ingress Controller, not GCE Ingress Controller. But traffic is forwarded to your Nginx Ingress Controller via GCE load balancer, I suppose.
If so, take a look at Proxy protocol usage for Nginx Ingress Controller.

At aws I would add this to my config map
use-proxy-protocol: "true"

If I try that with google container engine I get this error.
while reading PROXY protocol, client: <IP_ADDRESS>, server: <IP_ADDRESS>:443
2016-11-15T21:24:41.460205161Z 2016/11/15 21:24:41 [error] 198#198: *69 broken header

This is probably a gke issue, so I will head over there.

have the same error once the use-proxy-protocol is enabled. Does it means that we have to setup haproxy to use it properly if the nginx service is set as NodePort?

GCE Load Balancers don't support proxy protocol. To get the real IP addresses with an nginx ingress, you have to rely on this alpha feature introduced in kubernetes 1.4:
https://kubernetes.io/docs/user-guide/load-balancer/#loss-of-client-source-ip-for-external-traffic
Closing this as it's not a kube-lego bug/limitation. Let me know how it goes...

Work like a charm!
need to set the type as LoadBalancer.

@kenng have you found a solution? My nginx ingress still shows <IP_ADDRESS> only as client ip

@alex88, per the reference above, you need to add an annotation to the nginx controller service
apiVersion: v1
kind: Service
metadata:
  name: nginx
  namespace: nginx-ingress
  annotations:
    **service.beta.kubernetes.io/external-traffic: OnlyLocal**
spec:
  type: LoadBalancer
  ports:
  - port: 80
    name: http
  - port: 443
    name: https
  selector:
    app: nginx

@DocBradfordSoftware unfortunately that didn't help. :(
I've a domain pointing to the GCE load balancer IP with an A record, if I use curl with the domain name, nginx logs shows: <IP_ADDRESS> - [<IP_ADDRESS>], if I use curl with the load balancer IP address it correctly shows my client IP :/

I used the echo service in the example. and got these values
X-Forwarded-For=70.197.xxx.xxx
X-Real-IP=70.197.xxx.xxx
double checking with whatsmyip
http://www.whatsmyip.org/
i got the same ip: 70.197.xxx.xxx

Lucky you :) I still can't get my real ip if I use a domain to connect to the load balancer

I am not sure if what i am doing is correct, but the way I find my ip to hook up to the dns record, is that I go to gcloud console networking -> loadbalancing. there should be one instance, click on that and it will show the external ip.
Also if you look at the External IP Addresses tab, there should be one IP that points to a Forwarding Rule.  Clicking on the forwarding rule, should bring you to the loadbalancing page above.

@DocBradfordSoftware I've found the discriminant, if I use port 443, I get <IP_ADDRESS> in the nginx ingress logs, if I use port 80 I get the real client IP

@alex88 Did you find a solution to your issue? I'm seeing the same problem. What version of the Nginx Ingress are you on?

@artushin nope, still the same issue, maybe it's because of the ports that the tcp load balancer can use the proxy protocol with, I don't really know.

Just upgraded to 0.9.0-beta.8 (https://github.com/kubernetes/ingress/releases). Looks good now.
Looks like it was probably this: https://github.com/kubernetes/ingress/issues/233

Sorry forgot to mention I was using beta-5, I'll try with the beta 8. Anway, do you have any special config in the service? Like the service.beta.kubernetes.io/external-traffic: OnlyLocal annotation?

I do, but it's a one node dev cluster, and I'm running the nginx pods in a daemonset, so it would probably work even with out it. But yeah, based on the k8s docs, if you have nodes that are not running an nginx pod, you should use that annotation if you need source IPs.

Oh yeah it works! That's awesome! :D

Now it's just a matter of having nginx trust the forwarded ip and that's it :)

Did somebody also get it working when using a GCLB through an ingress in front of Nginx? The service.beta.kubernetes.io/external-traffic: OnlyLocal doesn't seem to work, because the service is just a NodePort.
Sorry for using this thread, but best way to get a solution :-)

I would check out this documentation:
https://kubernetes.io/docs/tutorials/services/source-ip/#source-ip-for-services-with-typeloadbalancer.
kubectl patch svc loadbalancer -p '{"spec":{"externalTrafficPolicy":"Local"}}'

replace loadbalancer with your nginx service and add --namespace=xxx if you need to.

@DocBradfordSoftware Problem is that our services is a type:NodePort, because we have an ingress in front of it. But I am now assuming that nobody in this thread has it setup that way.
PS. We use the ingress controller with GCP Http Loadbalancer, so we can use:

TLS termination at the HTTP loadbalancer at Google
ingress.gcp.kubernetes.io/pre-shared-cert
kubernetes.io/ingress.global-static-ip-name

You are correct, mine is on GKE, but I followed the kube-lego/examples/nginx and so the service is Type:LoadBalancer

@bviolier we have the same setup and issue and been talking to Google support for about a week for find a solution. So far the best they have been able to offer is to setup an L2 load balancer with type:LoadBalancer and host nginx ourselves including all the ingress routing, SSL termination, etc in the nginx configuration.

See my solution posted in https://github.com/kubernetes/ingress-nginx/issues/808#issuecomment-409466723


Error while running demo.sh "view size is not compatible with input tensor's size and stride"

I'm running into the following error while running demo.sh
==
Running forward
+ CUDA_VISIBLE_DEVICES=0 python demo.py --batch_size_v 80 --num_workers 4 --forward_save_path demo/forward
Namespace(G_GAN=1, G_VGG=1, G_nn=1, batch_size_t=128, batch_size_v=80, beta1=0.5, dataroot=False, dataset='MPV', dataset_mode='regular', decay_iters=10, epoch=200, face_L1=10, face_gan=3, face_img_L1=1, face_residual=False, face_vgg=1, fine_height=256, fine_width=192, forward='normal', forward_save_path='demo/forward', gan_mode='lsgan', gpu_ids=[0, 1, 2, 3], grid_size=5, init_gain=0.02, init_type='normal', input_nc_D_app=6, input_nc_D_face=6, input_nc_D_parsing=56, input_nc_G_app=26, input_nc_G_face=6, input_nc_G_parsing=36, isdemo=False, isval=False, joint=False, joint_G_parsing=1, joint_all=False, joint_parse_loss=False, lambda_L1=1, lr=0.0002, mask=1, mask_tvloss=False, momentum=0.9, n_layers_D=3, ndf=64, netD_app='resnet_blocks', netD_face='resnet_blocks', netD_parsing='basic', netG_app='treeresnet', netG_face='treeresnet', netG_parsing='unet_256', ngf=64, no_dropout=False, norm='instance', num_workers=4, output_nc_app=4, output_nc_face=3, output_nc_parsing=20, pool_size=100, print_freq=10, resume_D_app='', resume_D_face='', resume_D_parse='', resume_G_app='pretrained_checkpoint/app.tar', resume_G_face='pretrained_checkpoint/face.tar', resume_G_parse='pretrained_checkpoint/parsing.tar', resume_gmm='pretrained_checkpoint/step_009000.pth', save_epoch_freq=1, save_time=False, size=(256, 192), start_epoch=0, suffix='', train_mode='parsing', use_gmm=False, val_freq=200, warp_cloth=False, weight_decay=0.0001)
initialization method [normal]
initialization method [normal]
initialize network with normal
initialize network with normal
initialize network with normal
====================
====================
====================
====================
==>loaded model
/home/yash/projects/Detailed-virtual-try-on/utils/pose_utils.py:112: FutureWarning: `draw.circle` is deprecated in favor of `draw.disk`.`draw.circle` will be removed in version 0.19
  yy, xx = circle(joint[0], joint[1], radius=radius, shape=img_size)
Traceback (most recent call last):
  File "demo.py", line 184, in <module>
    forward(opt, paths, 4, opt.forward_save_path)
  File "demo.py", line 108, in forward
    warped_cloth = warped_image(gmm, result)
  File "/home/yash/projects/Detailed-virtual-try-on/utils/warp_image.py", line 29, in warped_image
    grid, theta = model(agnostic, cloth_image)
  File "/home/yash/venvs/dl/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/home/yash/venvs/dl/lib/python3.7/site-packages/torch/nn/parallel/data_parallel.py", line 159, in forward
    return self.module(*inputs[0], **kwargs[0])
  File "/home/yash/venvs/dl/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/home/yash/projects/Detailed-virtual-try-on/lib/geometric_matching_multi_gpu.py", line 430, in forward
    theta = self.regression(correlation)
  File "/home/yash/venvs/dl/lib/python3.7/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/home/yash/projects/Detailed-virtual-try-on/lib/geometric_matching_multi_gpu.py", line 126, in forward
    x = x.view(x.size(0), -1)
RuntimeError: view size is not compatible with input tensor's size and stride (at least one dimension spans across two contiguous subspaces). Use .reshape(...) instead.

hey
can you try changing that line to
x = x.contiguous().view(x.size(0), -1)
it worked for me
Thanks

hey
can you try changing that line to
x = x.contiguous().view(x.size(0), -1)
it worked for me
Thanks

hey can you try changing that line to x = x.contiguous().view(x.size(0), -1)
it worked for me Thanks

It also works for me.


Added support for Revapi

Added Revapi support for API analysis and change tracking to identify incompatibilities.
Resolves #3929.

@lihaoyi Please review.

Some nits but otherwise looks good!


Windows Shared Printer Unable use

question
hi,I have a question
I want to use the raspberry PI to print a string of data, but I can't connect to the printer.
My printer is shared on a Windows computer

Device info
Printer: EPSON L4150
Share os： Windows10
python-escpos version: latest
python version: 3.7

Hi @EngineerWill .
First, you must configure your remote printer in your Raspberry Pi's CUPS server, if you haven't done it yet, and assert it prints the test page, or any document, properly.
If you have other printers configured in CUPS, set the remote printer as the rpi's default printer.
After the remote printer is configured and working, you need to install pycups.
To install pycups via pip you'll need to install the CUPS development package for your system first:

sudo apt install libcups2-dev
pip install -U pycups

Now, you need to update the python-escpos version to the one in #348:

pip install -U git+git://github.com/python-escpos/python-escpos/pull/348/commits/f07d5e06109e37aa45a5a139f49f56d6399b98d3

Now you can use the experimental python-escpos CUPS connector, and send a job to the printer's queue:
Here's a snippet:

import escpos.printer as printer

cups_conn = printer.CupsPrinter(printer_name="")  # Default CUPS printer
cups_conn.textln("Hello CUPS!")  # or any python-escpos command
cups_conn.close()```

If Hello CUPS! is printed by your remote printer, you are success.
Don't doubt to ask for help if needed.
Looking forward to your feedback.

Hello! I am using CUPS to print from a Raspberry Pi. I also have pycups installed, and I'm able to print documents properly (both the print page from CUPS as well as pdf documents using pycups).
I wanted to use some ESC/POS commands but I followed the steps above and I couldn't get the string printed (but no error arose). In the terminal, it only said that the CUPS connection was closed.
I also tried the is_online() command, but then I had the "python escpos 'nonetype' object has no attribute query_status" attribute error.
Any suggestions? Thanks in advance.

Hi @DudeWhereIsMyTardis !
The instructions in this issue are a bit out of date,
Please follow these which add new parameters that allow the printer to connect to a remote server:

Update to the last commit:
pip install -U git+https://github.com/python-escpos/python-escpos.git@69e9dca761cbc2b03d5ae54865926482d8953b1d

Run replacing <the_server_ip> with the Raspberry Pi ip:

import escpos.printer as printer

cups_conn = printer.CupsPrinter(printer_name="", host="<the_server_ip>", port=631)
cups_conn.textln("Hello CUPS!")  # or any python-escpos command
# more commands...
cups_conn.close()

If you confirm it worked, I will update the outdated instructions above.
Thanks in advance.

Hello @belono! Thank you for your fast reply.

When replacing <the_server_ip> with the Raspberry Pi ip, there was the following error AtributeError: 'CupsPrinter' object has no attribute 'pending_job'.
After that, I tried using the CUPS server host ip, namely <IP_ADDRESS>. The result was the same as before, no errors, and no printing, only Closing CUPS connection to printer <printer_name>.

Hello again @belono!
I contacted the printer manufacturer (I have a Vretti 420B Thermal Label Printer), and the reply was that this printer doesn't support ESC/POS commands. Nevertheless, thank you for your help.

No problem @DudeWhereIsMyTardis !
Here we are if you ever get an ESC/POS compatible printer.


CASMTRIAGE-6368/6292 - fix sftp use in IMS customize jobs and increase job memory limits.

Summary and Scope
Integrate the changes from ims-sshd to fix sftp use on customize jobs, and increase the default memory requirements / limits to handle bigger images work without changing IMS defaults.
Code PR's:
https://github.com/Cray-HPE/ims/pull/100
https://github.com/Cray-HPE/ims-sshd/pull/39
Issues and Related PRs

Resolves CASMTRIAGE-6368
Resolves CASMTRIAGE-6292

Testing
Tested on:

Mug

Test description:
I did a clean helm install of the new IMS service and tested the ssh/sftp access on jobs, and verified the new updated memory settings are present.

Were the install/upgrade-based validation checks/tests run (goss tests/install-validation doc)? N
Were continuous integration tests run? If not, why? N
Was upgrade tested? If not, why? Y
Was downgrade tested? If not, why? N
Were new tests (or test issues/Jiras) created for this change? N

Risks and Mitigations
The changes are small, so the only risks are the normal ones associated with last-minute changes to a release.
Pull Request Checklist

[X] Version number(s) incremented, if applicable
[X] Copyrights updated
[X] License file intact
[X] Target branch correct
[X] CHANGELOG.md updated
[X] Testing is appropriate and complete, if applicable

/backport release/1.5


UML-916 Show viewer code content changes

Purpose
Changed content on show viewer code page to include the new view URL and remove the trial information
Fixes UML-916
NOT TO BE RELEASED UNTIL PUBLIC LAUNCH
Checklist

[x] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] The product team have tested these changes

Coverage remained the same at 81.604% when pulling d5e19a76697f1e27fc253c6337019a7cee3db429 on UML-916-Code-gen-page-content-changes into 93d644c659b83376e4d10d57661764137db99d1f on master.


Tasks deduplication

Description
When user trigger pipeline with identical tasks list multiple times, devlake would run those duplicated tasks in parallel, which is a big waste and took longer time for each pipeline to complete
Describe the solution you'd like
When creating pipeline, run dedupe check on existing tasks, and reuse those tasks.
Has the Feature been Requested Before?
No
Additional context
EE need this to prevent situation like:

User click on "sync jira" and EE send out "collect jira -> compute jira/gitlab metric"
User then click on "sync gitlab" and EE send out "collect gitlab -> compute jira/gitlab metric" right away
Causing jira/gitlab got computed twice

This is handled in this taks: https://github.com/merico-dev/lake/issues/775
If a new task is triggered while another one of the same type is running, cancel all running tasks of the same type and only execute the new one.

@joncodo I suggest we reopen this one, and close #775 instead.
for automatical cancelling, we need to consider more factors.

@klesh feel free to change anything you need. I trust your ideas :)

Now we had 2d array pipeline, this is no longer an issue. close for now.


The villagers have disappeared

Chunker Version
I am using chunker.app website
Input World Version
Java 1.20.4 (PC)
Output World Version
Bedrock <IP_ADDRESS> (android)
Platform / Operating System
Windows 10 / android
Issue Description
After the transfer of the world from java to bedrock, all the villagers of the village disappeared. At the same time, I tried to transfer the world to Java 1.21.1 (on my PC) and there was no such error.
P.S I'm sorry that I don't know the version of chunker. Nowhere does it say whose project the chunker.app website is, so I decided to write here. if anything, I moved the worlds two days ago.

This issue tracker covers the downloadable Chunker at https://github.com/HiveGamesOSS/Chunker/releases
Additionally Chunker doesn't support entity conversion which is why villagers will disappear.


Add dockerfile

This PR adds a dockerfile and some explanation in the docs.
I'm not sure about the CLI commands, so somebody better review.

Why not? Doesn't hurt, and it makes it easier to use in github actions or other CI/CD. Up to you


update documentation about mutate

Includes and extends #321 (credits to @akx)

Added a proper code example as suggested. @pacocoursey

Closed in favor of #321.


Cross targeting: Projects build all TFMs from VS

Ensure that for cross targeting projects we clear out the "TargetFramework" global property during explict builds in VS so that we build for all configurations. We do so by implementing a build-only properties provider that checks if the project configuration specifies "TargetFramework" as a dimension or not, and clear out the TargetFramework property if the dimension exists.
Fixes #504

Tagging @dotnet/project-system for review


Make the /healthz endpoint actually check some things

After https://github.com/kubernetes-incubator/service-catalog/pull/84, there will be an API server running that implements a /healthz endpoint which simply returns a 200 OK to the client. That endpoint should do something "real" and return the actual status of the process.

@arschles and others... any ideal what that "something 'real'" might be?

I think we need a healthcheck endpoint on every container process

@krancour: following up on @pmorie's comment just now. I think "something real" will depend on which healthcheck we're talking about, since we're going to implement checks for all running processes. We'll also probably have to iterate as functionality grows.

In the controller, for example, "something real" might mean that each control loop is responsive.

that each control loop is responsive

Determined how? The channels they iterate on aren't close? Might need to refactor a bit to make that happen-- not that I am opposed.

I'm not certain what logic this endpoint should check, @arschles @krancour do you guys have any ideas, now that we're much further down the road on this?

I'm moving this to MVP 2 to reduce the scope of the now-overdue MVP 1.

The health check should check that the TPRs are all created and ready

Dupe of #464; closing


Removed mutexes from AllMuxesState singletons used by muxes.

Muxes always execute on the main thread, no thread synchronization is required.
Signed-off-by: Dmitri Dolguikh<EMAIL_ADDRESS>ping @ggreenway, @htuch.

/retest

This seems right based on the rationale that muxes are only used for xDS config fetch and that should always be on the main thread. We do have gRPC clients that aren't on the main thread, but they don't use muxes. @ggreenway WDYT?

@KBaichoo: any chance you could take a look at the tsan failure? It looks like a FakeUpstream is still handling the reset when the test (ResettingStreamUnregistersAccount, http2) calls its dtor. Perhaps the test should wait for upstream to see the reset?

will take a look at it

LGTM will wait for tsan to green on main

The tsan failure should be fixed in https://github.com/envoyproxy/envoy/pull/17609.


[Feature Request][Kernel] Run unidoc as part of the Kernel CI tests

Feature request
Which Delta project/connector is this regarding?

[ ] Spark
[ ] Standalone
[ ] Flink
[x] Kernel
[ ] Other (fill in here)

Overview
Currently the Kernel CI job just runs the tests and doesn't generate any docs to make sure there are no errors introduced in the PRs. Doc generation is done as part of the delta-spark tests which are not run if the PR just contains Kernel only changes.
May be it is better to separate out the unidoc generation into a separate CI workflow that always runs?

Hello @vkorukanti, i am new to open source project project.
Can I get assigned to this as my first contribution ?

May be it is better to separate out the unidoc generation into a separate CI workflow that always runs?

This SGTM

Here we are only checking wether test_group  is not specified or it is spark. Hence unidoc is just generated for spark group here.
So I think allowing also test_group to be kernel group might fix the issue and correct me please if I am wrong.

@sokolat Here is one plan: Don't run unidoc as part of the run-tests.py. Instead create a new workflow here that just runs build/sbt unidoc. This new workflow will be run for every PR (@allisonport-db Any idea to make the PR updates run the new workflow?)

If you add a workflow to the .github/worflows/ directory it should run on the PR

hello @allisonport-db, in the job steps can I have a hint on what actions I need to to use ?


chore: be explicit about esbuild version and lock it to specific version

Still not great
Open to suggestions for making esbuild not bundled with cdk-nextjs in #36

LGTM, any reason for 0.15.18 and not the latest version?

Only reason was to not change it again


Completions on class: put statics first

TypeScript Version:  nightly<EMAIL_ADDRESS>Code
class C {
  static foo() {}
  static bar() {}
}

C.

Expected behavior:
Completions foo and bar are first.
Actual behavior:
They are hidden among apply, arguments, bind, call, caller, length, name, prototype, and toString.

I've taken a crack at this for my contribution to TypeScript.
When creating the completion entries in competions.ts createCompletionEntry all entries are given a sortText of "0" which means they end up in the default alphabetical order. My logic is to give the symbols merged from Function sortText that will cause them to appear after. The concept that seems to indicate these symbols are from an inherited class are the mergeId so I have gone for something similar to the following:
let sortText = ";

if(symbol.parent.mergeId) { // undefined on symbols directly from 'C' but numerical for Function symbols
    sortText = `0-${symbol.parent.mergeId}`;
}

If this doesn't sound too far off then I would happily follow the standard procedure and create a pull request etc.

@jameshulse sounds about right to me

@jameshulse I think we're not correctly testing sortText right now. I'll try to look into that on monday.

Cheers @andy-ms. A potential risk is whether all clients are happy to handle non numeric sortText.

I think we should be fine. The name is sortText after all, and even the built-in JS sort function sorts numbers as if they were strings 😆

Ha good point @RyanCavanaugh. I've created a pull request to show my suggested fix. I struggled with testing part so I'm sorry if this causes more work for you guys. If you can point me in the right direction I can give it another go (was also running in to mocha errors locally annoyingly)


Can't start test wait: http://<IP_ADDRESS>:4444/wd/hub does not respond in 20m0s

Hello,
I have the same question
https://stackoverflow.com/questions/68915182/how-to-close-nested-virtualization-popup-for-emulator-while-running-automation-t
My tests can't start cause sees  Nested Virtualization popup. (image default android 6.0)
I downloaded https://github.com/aerokube/images
then went so /selenium/ and started ./automate_android.sh with defaults
I got images android:8.1
then I added this to browser and started my tests
when I start this image Can't start test wait: http://<IP_ADDRESS>:4444/wd/hub
Max time for waiting 20m
environment: win10 i7 +VirtBox Ubuntu 18 + testng + gradle
May be you can help me!

@BorisPotapov should not be shown. https://github.com/aerokube/images/blob/master/selenium/android/Emulator.conf#L2

@vania-pooh thanks for fast answer.
I can't build image correctly with this emulator config.  After build i see wait: http://<IP_ADDRESS>:4444/wd/hub does not respond in 20m0s
or you mean I should switch this parameter manually? Please, send me example

@BorisPotapov should build your image from scratch.

@vania-pooh could you send me a steps for it, please. I am not sure that I can build image from scratch.
I see only this documentation "https://aerokube-com.translate.goog/images/latest/?_x_tr_sl=auto&_x_tr_tl=ru&_x_tr_hl=be#_building_images" and for creating android image i found two lines
$ cd selenium
$ ./automate_android.sh

I used they the previous time. this lines don't suite for my problem.
Interesting, I downloaded image from this repo budim/selenoid-android:11.0-no-google and got the same problem
If you send me more information i will appreciate

@vania-pooh I have idea.
Perhaps, this example https://github.com/aerokube/images was created without Enable Nested Virtualization because it is the same problem like I solved when installed Virtual Box to Win10.
I mean maybe default android images hasn't this config https://github.com/aerokube/images/blob/master/selenium/android/Emulator.conf#L2 and start correctly, but this example https://github.com/aerokube/images has this conf and haven't  Enable Nested Virtualization true.
What about this idea?

@BorisPotapov older images in Docker hub don't have this config.

@vania-pooh I build image with showNestedWarning=true and this image start works correctly like older images. I see this Nested Virtualization popup.
Now I am sure that showNestedWarning=false works with some problems for my configuration VirtBox Ubuntu 18 + android image for win10.

I did it.
I just rewrote showNestedWarning=false without space and it works for me

@BorisPotapov don't get how you fixed it.

I deleted extra spaces at the end

@BorisPotapov closing then?

@vania-pooh yes, please :)

May be need to delete extra spaces at the end of https://github.com/aerokube/images/blob/master/selenium/android/Emulator.conf#L2 too.

@BorisPotapov I don't see such spaces in raw file version. Probably that's a Windows BOM symbol.


How to use 2 types of charts together?

I was thinking about use this lib in my React Project, but I need to use 2 differents types of graphs together is there a way? Its Line and Area

The easiest way is via css, for example if you want the second series as a line you can use:
.areaChart > .area:nth-child(1) { fill: none !important; }

Thank you, I didnt start to use this library on my project, but it looks good.


The changelog check should skip if there is no milestone

If the changelog check returns a fail because the milestone isn't set, you have two statues that say "set a milestone" :laughing:

Maybe it is a feature and not a bug! Two failures should show you how important it is to fix? 😅


No Getters + Private Final

Hello! In lecture, I believe Prof said we cannot use getter methods because they would violate the OOP principle of Tell, don't Ask - but my TA has been telling me that their use is permissible. So I have been stuck on my lab assignment for a few days because I'm not sure who I should follow - and whether I will be penalised for doing so. Hence I would like to seek clarification on this because there's no point in me using getter methods if I'm going to be penalised, and if I could use getter methods, I probably would have solved the lab assignment sooner haha
P.S. is there any penalty if we cannot score A by our lab submission by 2359 today?

Hello! From what I have heard you can use getter methods but just discouraged from doing so unless there is no other way to get the information you need. Hope this helps!

Yup! i think my TA said that but i think im more worried about whether we will be penalised / ramifications for doing so in later labs... i think im just very lost because I want to submit a good piece of code but if i do something e.g., make a copy of ImList for local referencing that's also kind of defeating the purpose of the private final initialisation :(

The tell don't ask principle is quite a subtle one. During recitation Prof henry did mention that in some situation using getters is necessary. Take the Pair class as an example that was covered during recitation. If the Pair class does not provide anyway to retrieve the first or second value, what will be the use of Pair? To understand when getters is necessary, take for example Circle and area - if someone were to ask for radius and calculate the area by themselves, it violates tell don't ask since it's perfect possible that circle calculate the area afterall, circle can calculate area itself.

Issue expired.


Fix the namespace detection pattern.

As I tried with protobufjs 6.7.3, message types (Type) also has
'nested' field, but the value is undefined. It should check
this pattern too, otherwise non-namespace ones become an empty
object. That causes the failure of googleapis/gax-nodejs#135

@murgatroid99 ?

I don't think those tests are going to work. The dependency is still on Protobuf.js 5, so I wouldn't expect protobuf_js_6_common.loadObject(messages_proto) to work at all.
Also, though I don't object to this change, I would strongly recommend against using the message types that are output there.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Current package.json file specifies as '^6.7.0', am I missing something?
Anyways, added the logic to skip the test if protobufjs doesn't look like 6.

I would strongly recommend against using the message types that are output there.
I'm not sure I get this. Here are my understandings, correct me where I'm wrong.

With protobufjs 5.x, loadObject() works like:

if namespace, returns an object with the child nodes
if service, returns a wrapped class of the service
if message, returns the message class (by calling build())

My change is doing the same thing in protobufjs 6.x.
The current protobuf_js_6_common.js misunderstands message as namespace because message types also have nested property, so it returns an empty object. I think it should return the message class. And it looks like unnecessary to call build(), the value is already the message class.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

@murgatroid99  this PR seems abandoned. Closing, feel free to revive if still useful.

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.
After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

User @jmuk isn't covered by a CLA. They will need to complete the form at https://identity.linuxfoundation.org/projects/cncf

Regards,
CLA GitHub bot


Hedged-request: Handling in HTTP Client Configuration

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Initiation change for
https://github.com/thanos-io/thanos/issues/6712
Verification

@GiedriusS PTAL!

Hey ! @GiedriusS closing this one ! PTAL for #118


Accept zero without units as remaining time

Accept a time value of "0" without having to specify units when entering a remaining estimate (set to, or reduce by). JIRA has a default unit, which is another way of not having to specify units (future feature, maybe?).

Thank you for this.


Blurring Dimmable Conflicts with Ribbon

It doesn't seem possible to use ribbons on images in conjunction with blurring dimmables.
Dimmable:

Blurring dimmable:

Small thing, but slightly annoying and weird. At first I thought this was for all dimmables, but apparently it's only on blurring.

(Latest Firefox OSX)

That being said, even with non-blurring dimmable, the ribbon doesn't persist when being hovered over:


Bluetooth: controller: refactor to use min/max macro

"Good first issue" to refactor the BLE controller implementation, clean up the use of conditional statements with min/max macros.
Refer to review comment in #6149

@cvinayak my first issue. Would be grateful if you could look over the pull request when you get a chance.


Addition of college to list

Addition of Faminas to list.

@JoaoASouzaN I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host alunoFaminas.br not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Readme is missing demo.gif file in package

If you go to https://www.nuget.org/packages/Microsoft.Windows.CsWin32/0.2.158-beta#readme-body-tab then you'll see broken link.

I wonder... can README files in nuget packages now reference images from that same package? TBH, the README file was written only with presentation from github in mind.


Improve some map initializations

How to categorize this PR?

/area quality
/kind enhancement
What this PR does / why we need it:
This PR improves some map variable initializations, specifies the length of the map where possible.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

NONE

/retest

/approve


Add support for manototv.com

Checklist

[x] I'm reporting a new site support request
[x] I've verified that I'm running youtube-dl version 2021.06.06
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that none of provided URLs violate any copyrights
[x] I've searched the bugtracker for similar site support requests including closed ones

Example URLs

Single video: https://www.manototv.com/episode/12576
Playlist: https://www.manototv.com/show/2526

Description
Manoto is a free Iranian public TV network based in London (no copyright infringement), they broadcast dubbed and non-dubbed content. On their website they provide some of the content they show DRM free.

https://github.com/yt-dlp/yt-dlp/pull/767


bigint in cassandra for timestamp

Hi,
as already stated in a comment to a TS to C* comparison article,
you really don't need timestamp in C* row to be bigint
https://github.com/timescale/tsbs/blob/73ae4be837a5ae5644c1c615dc77ae0496b33392/cmd/tsbs_load_cassandra/creator.go#L57
Each cassandra row will only contain one day of values, so it's 24*60*60/10 = 8640 entries.
I suggest to use 2-byte smallint, which is 4 times smaller than 8-byte bigint.

Hi @curlup . you make a good point that the bigint type and nanosecond precision may be a bit overkill. Ideally we could keep the schema flexible enough to handle at least millisecond precision, which a smallint couldn't handle in a clean way (we'd need at least 86M values per day to represent the full range of possibilities, whereas a signed smallint tops out at 32K).
We could still shave the memory footprint in half and be able to use ms-precision timestamps with an integer type, which I'm definitely not opposed to exploring.


[request] Other color models

The game is amazing! I never had feeling for colors but after playing for half an hour, they start making sense to me. I would imagine it has its pedagogical role in courses.
I am wondering if you plan to add other color models.

Hello @pesho-ivanov, Thank you for your comment. I am glad you enjoyed this game. I do not have any plan to add other colour models at this time. However, anyone who wishes to extend or fork this game to add other colour models are very welcome to do so. I hope the HTML + JavaScript code is simple enough that it can be followed easily by someone who is eager enough to modify it. The source code is released under the MIT license, so anyone who wishes to create new variants of this game is very welcome to do so!


Getting dimension mismatch on running YoloV5s model

Box array shape : (1, 3, 40, 85)

Traceback (most recent call last):
File "main.py", line 172, in
predictions.append(extract_boxes_triton(response, result_filenames[idx], FLAGS.batch_size))
File "/triton_client_yolo/utils/postprocess.py", line 94, in extract_boxes_triton
l_box_array = box_array[i, argwhere, :]
IndexError: too many indices for array: array is 4-dimensional, but 5 were indexed

Hi @manish0079,
At the moment, the client is not very generic to be able to process all kinds of different dimensions of the box array. For a proof of concept, i gave a refence config.pbtxt file here I used for the model exported via pytorch2onnx.py file. Here the box array returned is of size dims: [1, 16128, 1, 4] as shown in the config file. I used the input image size 512 x 512 which also changes the number of boxes produced. If you are using a different model, which is YOLOv5s in this case, you will have to adjust the dimensions to your case.
Also try using a model of batch size 1, since batched input is not supported at the moment.


treat docx with merged table cell?

if I try to fill the mailmerge field in a table, and if the table has merged cell, doc-mailmerge will un-merge the merged cell.
how can we treat such case? Thanks

What do you mean by "merged" cell ? A cell that has a new value but is still a MERGEFIELD ?
This is not supported yet, I have plans to support keeping fields in the document in my fork of the docx-mailmerge if requested. I have a pretty good idea how to do it and I noticed some other issues for the same feature.


ResultKind and ResultLevel appear to serialize incorrectly

Hi there! First of all, thanks for this library -- it's been fantastic to use.
I'm trying to emit some well-formed SARIF in zizmor and I'm running into some weirdness with how ResultLevel and ResultKind are serialized. More precisely, they seem to always be serialized as null (i.e. serde_json::value::Value::Null), regardless of their actual value.
Here's a minimal failing reproducer:
#[cfg(test)]
mod tests {
    use serde_sarif::sarif::ResultKind;

    #[test]
    fn test_resultkind_serializes() {
        assert_eq!(
            serde_json::to_string(&ResultKind::Fail).unwrap(),
            "\"fail\""
        );
    }
}

The expected behavior is that this serializes to the JSON string "fail", but instead it serializes as null:
assertion `left == right` failed
  left: "null"
 right: "\"fail\""
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I took a quick look at the code, and it doesn't look wrong, but I'm not super familiar with strum (which seems to be controlling the serialization equivalents for each enum variant).
Let me know if there's any other information I can provide! I'll also keep debugging this on my end.

I think I've root-caused this: these enums are defined with #[serde(untagged)], which is known to cause "bare" variants to serialize to null per https://github.com/serde-rs/serde/issues/1560.
Instead of using untagged, these enums should be using #[serde(rename_all = "camelCase")]. I'll send a PR to make this change.

A follow-up problem here is that Result::level and Result::kind are both Option<Value> instead of Option<ResultLevel> and Option<ResultKind>, respectively. This results in API consumers needing to do a clunky round-trip through serde_json::Value.

Opened #770 for the last piece above!


New package: GPUArraysCore v0.1.0

Registering package: GPUArraysCore
Repository: https://github.com/JuliaGPU/GPUArrays.jl
Created by: @maleadt
Version: v0.1.0
Commit: c88227865b9972bb893117783f1949616221ad65
Reviewed by: @maleadt
Reference: https://github.com/JuliaGPU/GPUArrays.jl/commit/c88227865b9972bb893117783f1949616221ad65#commitcomment-76603557
Description: Reusable array functionality for Julia's various GPU backends.

Since this is just a *Core subpackage split off of the existing GPUArrays.jl package, it should be OK to merge this before the 3-day waiting period. Let me know if anybody doesn't think so, else I'll merge this later today.