Request for Access

Please forgive me if this isn't the correct place.  This is my first contribution to... well, any open source git project, actually, so I'm not sure what protocol is here.
I would like to contribute to RoslynPad.  I use it a lot, and there are a few things I think could be done to improve things.  (That sounded ominous; really I have a few ideas and the ability to implement the myself, so I thought I would try to contribute.)
What triggered this message is the fact that I actually have implemented a solution to Issue 185 (option to disable bracket completion).  I tried to push the branch from Visual Studio and got a 403 error.  So here I am. :)
What do I need to do in order to get permissions to create a branch?
Thanks!
-Merovech

Hey @Merovech - you might want to read up a little on Pull Requests, which is the standard for contributing on GitHub: https://help.github.com/articles/about-pull-requests/
Might I also suggest you open separate issues for the improvements you have and discuss them with @aelij just in case he has other ideas for their implementation, or even implemented them already in a private branch (these things happen all the time)...

Hi @ohadschn !
Thanks for the reply.  I'm more than happy to adjust with whatever I need to; I'm just trying to understand the process so I can get started. :)
I actually use git at work, so I'm familiar with branching and pull requests.  That's what got me to making this post, in fact - I cloned the repo, made a topic branch for my changes (bracket-disable-option), applied my changes to it, then went to publish the branch so I could make a PR out of it.  When I went to publish the branch (really, sync to GitHub for the first time), I got this:

Error encountered while pushing branch to the remote repository: Git failed with a fatal error.
unable to access 'https://github.com/aelij/RoslynPad.git/': The requested URL returned error: 403

I assumed that it was a permissions error, so here I am.
Looking at the article you linked, though, I'm wondering: should I be requesting permissions to contribute here or is it preferred that I fork the whole thing and create a Pull Request across forks?
One more question: I did post about the improvements I have on the original issue.  I'm perfectly happy to open a new issue for it, but is that the preferred method here?  I'm used to looking at two issues addressing the same thing and one being declared a duplicate and closed. :)

@Merovech I think the problem is that you cloned the repo from here and not your fork. You can use git remote to change it, then push to your fork.
As @ohadschn pointed out, GitHub has detailed docs, so please consult them :) There's no special access required to create a PR to any public repo.

@aelij Makes sense.  Didn't realize we should PR from forks and not branches.  Thanks!

@Merovech regarding duplicate issues, that is a valid concern, but that's what the search button is for :)


[Bug] Documentation navbar incorrect

What Happened?
Navbar says "contributing" when it really should say "Setup"
Steps to Reproduce

Go to https://jury.mikz.dev
Click "Contributing" on navbar

What device(s) are you seeing this issue on?
No response
What browser are you seeing this issue on?
No response
Possible Solutions
No response
Additional Info
No response

Hey @MichaelZhao21  Can you please assign this issue to me?

not needed


Replace selected icon with svg

When you activate it should replace selected element with SVG element

On hold until someone requests

yes it can be cool or adding more selected icon library (like iconify fontanswone .. etc) ?

Hi @micker,
yes that sounds like a cool idea, I will consider about it and work on it if I will have some free time 🙂
I've added a task for it #22

Great rhanks


Allow customization of Jest: transformIgnorePatterns

There are a ton of issues about this, and after two hours of reading through them all, I still wasn't able to get a sense of the state of things.

It seems that some Jest config options are now available: https://github.com/facebookincubator/create-react-app/pull/1830
Please make transformIgnorePatterns also available.

We have a case where we have local libraries that are brought into node_modules, but then Jest stumbles whenever we import them, because it is not pre-processing the React components in node_modules. Transpiling our components before export is not an option because the constant transpiling every time one makes a change is a serious drag on development.
Some discussion threw out symlinks as a solution or workaround, but another developer on our team sunk many hours into that before finally getting stuck again.
There has to be a simple way to make LibraryA/MyReactComponent go into MyApp/node_modules, such that MyApp/src/MyComponent can import MyReactComponent during tests without any problems.
It would seem to be a common use case.

Could you explain more about how transformIgnorePatterns would help? Have you verified (e.g. by manually changing node_modules/react-scripts/utils/createJestConfig) that this particular option would help? What did you set it to?
Is this related to https://github.com/facebookincubator/create-react-app/issues/607?

Yes. I manually changed the transformIgnorePatterns to just an empty array, as a test case:
    transformIgnorePatterns: [
      // '[/\\\\]node_modules[/\\\\].+\\.(js|jsx)$'
    ],

I also had to change:
node_moldules/jest-config/build/defaults:
  transformIgnorePatterns: [], // removed NODE_MODULES_REGEXP

It ran a lot slower, obviously, since that caused it to preprocess EVERYTHING in the node_modules folder. But it finally ran without any “UnexpectedToken <“ errors.
With the ability to customize this, I could add to package.json a regular expression that would cause it to pre-process everything in node_modules except the subfolder containing my local libraries. As it stands, the createJestConfig requires Jest to skip pre-processing everything in node_modules, which puts us in the position of constantly having to transpile…
To get an idea of the size of the problem, we have 10 different local libraries. We were having to transpile to export every time we changed one line of code. I basically listened to my computer whirr all day long, and saw it eat through 25% of a brand new solid state drive in three months (because of the near constant writing and re-writing to disk) before we finally said enough to the transpiling.
But now we’re in a position where we can’t run a Jest test on anything that imports anything that ultimately imports a React component from another library.

On Jun 14, 2017, at 7:07 PM, Dan Abramov<EMAIL_ADDRESS>wrote:
Could you explain more about how transformIgnorePatterns would help? Have you verified (e.g. by manually changing node_modules/react-scripts/utils/createJestConfig) that this particular option would help? What did you set it to?
Is this related to #607 https://github.com/facebookincubator/create-react-app/issues/607?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/facebookincubator/create-react-app/issues/2537#issuecomment-308582971, or mute the thread https://github.com/notifications/unsubscribe-auth/AMQpjMHaavuyI9I2e2jQ8p6STygdw5O9ks5sEGemgaJpZM4N6iYN.

Is there any chance you could share what was the issue with symlinks?

Unfortunately, no. The person who went down the long path there is no longer on the project… Could you point me toward good documentation for how to use symlinks to solve this problem?

On Jun 14, 2017, at 7:48 PM, Dan Abramov<EMAIL_ADDRESS>wrote:
Is there any chance you could share what was the issue with symlinks?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/facebookincubator/create-react-app/issues/2537#issuecomment-308589382, or mute the thread https://github.com/notifications/unsubscribe-auth/AMQpjBLCw8A-H_VC4wN60OzCUmfL8gJoks5sEHFRgaJpZM4N6iYN.

I’m not entirely sure myself. Unfortunately I’m overloaded with React work right now and can’t look at this in very close future, but this feedback is helpful. We know sharing components between apps is pretty important. If you could describe your setup in more detail (how many apps, how many components they share, your folder layout) this would help for future work.

Foo
- node_modules
- - Bar
- - - MyUntranspiledReactComponent

- src
- - MyJestTestedComponentWhichImportsMyUntranspiledReactComponent
** Jest configuration with create-react-app prevents you from transforming files in node_modules via customizing transformIgnorePatterns, so “UnexpectedToken <“ is thrown.**

Bar
- src
- - MyUntranspiledReactComponent
** Does not transpile when copied to Foo/node_modules because constantly transpiling in development is not possible given the volume of folders. **

We have approximately 20+ libraries encompassing several hundred modules all following this structure. Many of the libraries depend on each other as well, compounding the file-transpile-and-copy problem.

Symlinks (to avoid copying) or being able to use Jest transformIgnorePatterns (to avoid transpiling before copying to Foo/node_modules) would both be helpful solutions.

> On Jun 14, 2017, at 7:52 PM, Dan Abramov<EMAIL_ADDRESS>wrote:
>
> I’m not entirely sure myself. Unfortunately I’m overloaded with React work right now and can’t look at this in very close future, but this feedback is helpful. We know sharing components between apps is pretty important. If you could describe your setup in more detail (how many apps, how many components they share, your folder layout) this would help for future work.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub <https://github.com/facebookincubator/create-react-app/issues/2537#issuecomment-308589954>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AMQpjK7jpo0fNTew4IshlmNR-78L_RMMks5sEHJZgaJpZM4N6iYN>.
>

I see, thanks. Again, we don't have any immediate solution to this, but I feel free to participate in https://github.com/facebookincubator/create-react-app/issues/1492.

I have the same issue with an ejected app. Components in node_modules that need to be transpiled by Jest/Babel.
I ended up changing transformIgnorePatterns to ["/node_modules/(?!(our-react-components-.*?\\.js$))"].
Now JS files in our-react-components-* folders in node_modules are being transpiled.
I understand the discussion is around a non-ejected app, I just wanted to give a solution to those that don't mind ejecting.

Thanks a lot

@gaearon This seems like developers screwing over other developers. This isn't production stuff so why wouldn't we let them supply their own test config. I'm happy to submit a patch if there are eyes to review.

We make use of OpenLayers, so sadly we need to eject every project, just to be able to have transformIgnorePatterns as an option. There should be a way to pass this down.

@Primajin @michaeldewolf85 @mandysimon88 It's true that the transformIgnorePatterns option is currently not available via the jest property in package.json, however, you can still tap into Jest's configuration options without ejecting using the CLI:
npm test -- --transformIgnorePatterns "node_modules/(?!(your-untranspiled-module|some-other-module)/)"

Via the docs:

Every one of Jest's Configuration options can also be specified through the CLI.

Note: CLI options take precedence over values from the Configuration.

That last bit is what tipped me off (maybe a documentation improvement @gaearon).  Obviously configuring Jest via package.json doesn't work, and my jest.config.js seem to get trumped by CRA's Jest config, so CLI args was the saving grace.  Hope that helps!

@john-goldsmith, you are my hero.

@Primajin In my case, all works without any errors.
How I use this parameter in package.json:
"test": "react-scripts test --transformIgnorePatterns \"node_modules/(?!ui-core)/\" --env=jsdom"

@bogdosarov
Did you resolve?
``
FAIL  src/containers/App.test.tsx
● Test suite failed to run
/Users/me/path/node_modules/decimal.js/decimal.mjs:4789
export var Decimal = clone(DEFAULTS);
^^^^^^

SyntaxError: Unexpected token export


Poly functions cause flush: short draw message errors.

I fixed the dopoly function so it writes the proper length messages.

@rsc @cmarcelo Are you guys interested in this fix? I'd appreciate if you could take a look.


mobileconfig: Error: undefined method `[]' for nil:NilClass

Mobileconfig type/provider is not managing individual Payloads properly.
The :content property in the Mobileconfig type accepts an array of Payload hashes each with its own PayloadType and PayloadIdentifier key.
This makes creating compound profiles (configs which span multiple PayloadTypes) simple and straightforward. However, after adding one of these unique Payloads to a Mobileconfig resource successfully, attempts to remove a Payload will fail.
Example:
Each of these parameters establishes settings in a different PayloadType...
# This corresponds to PayloadType "com.apple.screensaver" key "askForPassword"
managedmac::security::ask_for_password: true
# This corresponds to PayloadType "com.apple.systempolicy.control" key "EnableAssessment"
managedmac::security::gatekeeper_enable_assessment: true

When these params are passed to the managedmac::security class, TWO distinct Payloads are created.
The bug manifests itself when you attempt to remove one of these params/PayloadTypes by dropping one of the parameters...
# This corresponds to PayloadType "com.apple.screensaver" key "askForPassword"
managedmac::security::ask_for_password: true

You can successfully remove the entire resource, but excising 1 or more individual Payloads produces an error:
Error: undefined method `[]' for nil:NilClass`

This is a bug in the Mobileconfig type, here:
https://github.com/dayglojesus/managedmac/blob/master/lib/puppet/type/mobileconfig.rb#L136
It's an out-of-bounds error on the array. This was introduced when we began using PayloadUUIDs as MD5 sums.
Easy fix. We perform an early return if the "is" and "should" arrays are not of equal length.


Use Dockerfile paths to determine which test data is in scope

This updates the test project to reflect the changes that were made in https://github.com/dotnet/dotnet-docker/pull/4207 and to conform to the new test script contract defined by https://github.com/dotnet/docker-tools/pull/1082. This means the test project no longer filters tests by product version but rather by Dockerfile path.
This change made it very cumbersome to attempt to adapt the existing SkippableTheoryAttribute class to use the Dockerfile path data in order to determine which tests to skip. So instead of doing that, I got rid of it entirely and went with a different approach. Each of the test projects defined its own set of test data. This data was largely duplicated amongst the classes except for WCF and SDK tests which needed to limit which versions they tested (hence the need for SkippableTheoryAttribute).
I consolidated all of that test data into one set which all the test classes use. Then I made use of the Xunit.SkippableFact NuGet package. This provides its own SkippableTheoryAttribute that, when used, allows the code to throw a SkipException to indicate that the test should be skipped. It also provides some helper methods that will throw a SkipException based on a condition. This then allowed me to reimplement the skip logic based on version in a much more straightforward manner.

I made some changes to handle a case where it was still running into the XUnit issue of having an empty set for the member data. That can happen when the paths filter causes the image for a given image type to be filtered out. For example, a 3.5 test leg would have the following paths specified:

src/runtime/3.5/windowsservercore-ltsc2019
src/sdk/3.5/windowsservercore-ltsc2019
src/aspnet/3.5/windowsservercore-ltsc2019

There is no wcf path here, by design, so this causes the WCF tests to end up with an empty set of image data and XUnit throws. To work around this, I've added code to compensate for this scenario by injecting a null ImageDescriptor as a placeholder to allow the test to detect and skip execution in that case. I've also added some validation logic to ensure we only end up in that state for this expected scenario. This also applies to the SDK-only tests.


Add Website to Google Search Console

Description:
This website is not added to Google Search Console, which is the reason a new user can't find it without the URL. Adding this website to Google Search Console will improve its visibility and allow users to find it more easily.
Steps to Resolve:

Sign in to Google Search Console.
Add the website as a property.
Verify ownership using one of the provided methods (HTML file upload, HTML tag, Google Analytics, Google Tag Manager, or domain name provider).
Submit the website’s sitemap.

Assignee:
Assign this task to me. I have experience with adding websites to Google Search Console and can solve this issue.

@PATILYASHH It is a new concept for me, and I will look into this topic to expand my knowledge further. Could you possibly share the documentation for learning Google Search Console?

I don't have specific learning sources for it because I learned from chatgpt . But I can explain to you

Will start a Discord community within this weekend or in the upcoming weeks. We can learn further together on this issue. For now, can you work on issue #3?

Will start a Discord community within this weekend or in the upcoming weeks. We can learn further together on this issue. For now, can you work on issue #3?

oky

Will start a Discord community within this weekend or in the upcoming weeks. We can learn further together on this issue. For now, can you work on issue #3?

I have some more plans for this project we will discuss on discord

Will start a Discord community within this weekend or in the upcoming weeks. We can learn further together on this issue. For now, can you work on issue #3?

I have some more plans for this project we will discuss on discord

sure, possibly we can make the app the best

Will start a Discord community within this weekend or in the upcoming weeks. We can learn further together on this issue. For now, can you work on issue #3?

I have some more plans for this project we will discuss on discord

sure, possibly we can make the app the best

We will

Will start a Discord community within this weekend or in the upcoming weeks. We can learn further together on this issue. For now, can you work on issue #3?

I have some more plans for this project we will discuss on discord

sure, possibly we can make the app the best

We will make

Hey Can you also help me with my other projects too. As I started a new repo for new learners of Git, HTML, CSS, JS
contribution-cards

Will start a Discord community within this weekend or in the upcoming weeks. We can learn further together on this issue. For now, can you work on issue #3?

I have some more plans for this project we will discuss on discord

sure, possibly we can make the app the best

We will make

Hey Can you also help me with my other projects too. As I started a new repo for new learners of Git, HTML, CSS, JS
contribution-cards

Yes


Chime Shown, Homebase not shown

Describe The Bug:
Doorbell chime is recognised and maps the security states successfully (home, away, etc). But the Homebase itself does not appear.
To Reproduce:
Chime and Homebase hardware installed, run plugin.
Expected behavior:
Make the Homebase the alarm system and ignore the chime
Logs:
[6/11/2021, 9:01:31 AM] [EufySecurity] Adding new accessory: Doorbell
[6/11/2021, 9:01:31 AM] [EufySecurity] Adding new accessory: Chime
[6/11/2021, 9:01:50 AM] [EufySecurity] Eufy Guard Mode:  { value: 1, timestamp:<PHONE_NUMBER>000 }
[6/11/2021, 9:01:50 AM] [EufySecurity] Eufy Guard Mode:  { value: 1, timestamp:<PHONE_NUMBER>000 }

Plugin Config:
            "username"<EMAIL_ADDRESS>            "password": "myPassword",
            "hkHome": 1,
            "hkAway": 0,
            "hkNight": 3,
            "hkOff": 63,
            "platform": "EufySecurity"

Screenshots:

Environment:

Plugin Version: 0.1.1
Operating System:  macOS >

Update: removing the chime and pairing the doorbell with the HomeBase is a workaround.

Interesting. Didn't know that Chimes were treated as stations in eufy-security-client. The code should support having multiple Stations - but if we only receive either a Chime or Hub then we can only add what was returned. I don't have a Chime so I can't test this.

I have a Homebase, keypad, motions sensors and contacts.  The doorbell I had first and was associated with the chime. So I removed the chime, associated the doorbell to the Homebase and all is working perfectly.

If we introduced "the this.log.debug('Found hub' + hub.getName());" in the hub iteration we would be able to see how many hubs the client return.

Thanks, but I suspect my case is pretty niche and I’ve moved the doorbell over to the Homebase, so I’d say focus on bigger thing. Really appreciate the reply and work you’re doing. Absolutely awesome, thank you.

Added the logging. When it is merged you can retest.

@PeterAlexander75 The logging is merged into master now.

Please re-test if possible.

Closing this issue due to inactivity


Ignore dependabot upgrades for react 18 types

Description
The types for react 18 contain some breaking changes. While updating our code is possible, the libraries we depend on haven't been upgraded yet either, so the type checking still fails. I think it's best if we wait until the major libraries we depend on (e.g. patternfly, easy-peasy) are upgraded to handle this correctly
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Code is clear and sufficiently documented
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[x] ~End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )~

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Merged into branches master, iso5 in 39563ec91b4216c9031152c562874265752c1699


Enable and document how to render MyST as part of other applications or as a standalone component

As part of the frx-challenges project, we wanted a way for data challenge administrators to write MyST Markdown for basic formatting of pages served via a Django app. We wanted a way of quickly rendering MyST and output nice HTML, but without the full-blown MyST document engine provided via the CLI.
However, we couldn't figure out how to do this easily. It would be helpful if this were more clearly documented, or enabled if it isn't possible.
Examples for inspiration
For example, we wanted something similar to how JupyterLab-MyST renders "snippets" of MyST markdown with basic MyST functionality, but not everything).
Another workflow we wanted was something similar to markdown-it. In that case, you can render HTML from markdown-it markdown like so:
import markdownit from 'markdown-it'
const md = markdownit()
const result = md.render('# markdown-it rulezz!');

The sample snippet covers what i'd have liked pretty well! Take string of markdown (+ possible options), return HTML


Release version for GHC 9.2

The published versions accept GHC 9.2 but fails to build, e.g. 2.0.1:
$ cabal install fast-tags

Build profile: -w ghc-9.2.3 -O1
In order, the following will be built (use -v for more details):
 - fast-tags-2.0.1 (lib) (requires download & build)
 - fast-tags-2.0.1 (exe:fast-tags) (requires download & build)

[ 5 of 10] Compiling FastTags.LexerM  ( src/FastTags/LexerM.hs, dist/build/FastTags/LexerM.o, dist/build/FastTags/LexerM.dyn_o )

src/FastTags/LexerM.hs:428:9: error:
    • Couldn't match expected type ‘Word8#’ with actual type ‘Word#’
    • In the pattern: 0##
      In a case alternative: 0## -> ptr#
      In the expression:
        case indexWord8OffAddr# ptr# 0# of
          0## -> ptr#
          9## -> ptr#
          10## -> ptr#
          13## -> ptr#
          32## -> ptr#
          92## -> ptr#
          _ -> goBack# (ptr# `plusAddr#` -1#)
    |
428 |         0##  -> ptr#
    |         ^^^
...

There seems to be an unreleased PR fixing this:

https://github.com/elaforge/fast-tags/pull/58

But the versions on Hackage are nevertheless broken.
I revised the versions on Hackage so the cabal constraint solver does not pick them with GHC 9.2: https://hackage.haskell.org/package/fast-tags-2.0.1/revisions/

I just updated to 2.0.2 on hackage, hopefully this solves the problem!

Thanks!


JSON Parser Should allow array as root element.

The current parser will not accept an array as a root element.  This is valid JSON, so it should be allowed.

Fixed parser to allow root element to be array, also fixed UBJSON parser too.


⚠ Bofrost.de has degraded performance

In 51602d9, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1948 ms

Resolved: Bofrost.de performance has improved in 0fd6816 after 6 minutes.


pythonPackages.flask-httpauth: 4.3.0 -> 4.4.0

Motivation for this change
Bump flask-httpauth and drop support for Python 2, https://github.com/miguelgrinberg/Flask-HTTPAuth/commit/e690ce56827de9d669718fa5d0fcda63112f8008
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 126533 run on x86_64-linux 1

4 packages built:

onionshare
onionshare-gui
python38Packages.flask-httpauth
python39Packages.flask-httpauth


Does not work on Ubuntu 18.04

I really loved this little helper, but it seems not to work anymore.
Anyone got an idea how to fix it?
Manually typing the source-command didn't work...
I added the crucial lines to my bashrc-file.

https://github.com/jimeh/git-aware-prompt/pull/64
Like literally at the bottom... Fixed it! Yay!


How to use Mamba2

Mamba2 is provided in the classification folder, how to apply to VMamba

You can follow the examples in vmamba.py to use it. One of the examples is as follows:
def vmamba_tiny_m2():
    return VSSM(
        depths=[2, 2, 4, 2], dims=96, drop_path_rate=0.2,
        patch_size=4, in_chans=3, num_classes=1000,
        ssm_d_state=64, ssm_ratio=1.0, ssm_dt_rank="auto", ssm_act_layer="gelu",
        ssm_conv=3, ssm_conv_bias=False, ssm_drop_rate=0.0,
        ssm_init="v2", forward_type="m0_noz",
        mlp_ratio=4.0, mlp_act_layer="gelu", mlp_drop_rate=0.0, gmlp=False,
        patch_norm=True, norm_layer="ln",
        downsample_version="v3", patchembed_version="v2",
        use_checkpoint=False, posembed=False, imgsize=224,
    )


Is locobot mover broken?

Type of Issue
Select the type of issue:

[x] Bug report (to report a bug)
[ ] Feature request (to request an additional feature)
[ ] Tracker (I am just using this as a tracker)
[ ] Refactor request
[ ] Documentation Ask

Description
File "/home/locobotm/djan/locobot/agent/../../base_agent/core.py", line 23, in step
    self.perceive()
  File "/home/locobotm/djan/locobot/agent/../../base_agent/loco_mc_agent.py", line 206, in perceive
    v.perceive(force=force)
  File "/home/locobotm/djan/locobot/agent/perception/perception.py", line 176, in perceive
    rgb_depth = self.vision.input()
  File "/home/locobotm/djan/locobot/agent/perception/handlers/core.py", line 31, in __call__
    return self.handle(*input)
  File "/home/locobotm/djan/locobot/agent/perception/handlers/input.py", line 25, in handle
    rgb_depth = self.agent.mover.get_rgb_depth()
  File "/home/locobotm/djan/locobot/agent/locobot_mover.py", line 221, in get_rgb_depth
    pts_in_cam = np.multiply(self.uv_one_in_cam, depth)
ValueError: operands could not be broadcast together with shapes (3,230400) (307200,)

Steps to reproduce

copied the locobot/robot folder onto the locobot
executed the commands in ./launch_pyro.sh separately in 4 terminals (there is a bug in the launch file which I'll fix outside this issue) to launch the pyro4 env
ran locobot_agent.py

There was issue with image resolution reported for locobot agent from remote_locobot.py


Bug & Feature request:  add Forcibly reload scopes command for scope match error

I found that if I delete a whole line by using ctrl+shift+k, hscopes will give wrong match results. You can reproduce the bug by:

install draivin.hsnips & dravin.hscopes
add this code to markdown.hsnips

global
const vscode = require('vscode');
vscode.window.onDidChangeTextEditorSelection((e) => {
    console.log(vscode.extensions.getExtension('draivin.hscopes').exports.getScopeAt(e.textEditor.document, e.selections[0].active).scopes.toString())
});
endglobal

new a .md file and open execute Developer : toggle developer tools, then open console.
Execute reload snippets. Then the result is shown below:

https://user-images.githubusercontent.com/80528249/156931067-071d0583-2de2-425e-b519-6aa5f48a625d.mp4
I think that it is a good idea to add a command "reload scopes". I used to write markdown with draivin.hsnips but it randomly matches wrong scopes. Without the command "reload scopes", maybe we have to reload VSCode to get rid of the error.
Thanks! 😃

Thanks for the feedback, I'm a little busy right now, but I'll look into it early next week!

Is there any update on this? I think this is closely related to this HyperSnips issue and fixing it would improve the workflow for markdown users.

I have updated the cache logic so it should break less often, nevertheless, there is now a command to forcibly reload the scopes for the current editor.


Cleanup & Polish

Remove obsolete futures mod
Add CI
Adjust licenses to MIT and Apache 2.0`, as recommended.
Add MSRV
Add tags
Bump to 0.3.0

This is on top of #9.

@jamesmunns I'm not entirely sure why the CI doesn't run, might be setting in the repository?

@jamesmunns Also you mentioned that this crate is not really maintained; if you want I can become a co-maintainer. Although I don't think there will be much activity after this.

@Finomnis added you as a collaborator and invited you as a crate owner. Please feel free to make a 0.3/1.0 release.

@Finomnis added you as a collaborator and invited you as a crate owner. Please feel free to make a 0.3/1.0 release.

Cool! Thanks. I'll try my best to serve you and this crate well.


Take a maximum number of runners per provider into account

Just an idea that came to our mind and I wanted to share. Consider the following:

Have a host that is capable of running 20 runners in parallel
Have 2 orgs (A and B) registered

The problem is, there's now no way to specify that we are able to run 20 runners maximum. So when 10 runners are assigned to each org, all is good. On the other hand, when org A is rarely used, org B is limited to 10 runners, although most of the time more runners could be used, because resources are available.
If I am using autoscaling and set org A to max 10 runners and org B to max 20 runners, this will work fine, as long as org A has no runners active. But when org A ramps up, I end up with 30 runners, which will be a problem for the system.
What I was thinking as a rough idea:

Add a "max number of runners" setting to the provider to determine how many runners can be run at maximum for each provider
Add a "pool priority" setting to the pool to determine which pool should be preferred when autoscaling and we are reaching the maximum of the provider

So given my example above, this would:

Have org A with prio 10 (higher) and org B with prio 0 (lower)
Have org B with 19 runners and org A with 1 runner (when 1 is defined as the minimum for org A)
Scale back org B to free resources for org A when org A scales up (so when org A has 5 runners, org B should only have 15)
So when both orgs scaled to their maximum, we would end up with 10 runners for org A and 10 runners for org B

Hi @SystemKeeper !
Sorry for the late reply.
For the most part, this sounds like a good idea. Especially for providers that deal with limited resources, like LXD, Incus, K8s and potentially future providers for various other systems.
The current architecture of GARM is a really simple one. It's a single process, single server app. It doesn't currently scale horizontally, at all. It could with some refactoring, but up until this point it wasn't really needed. At least not for performance reasons. However, there are plans to split it up in the future into multiple components:

API (user facing - talks to controller)
Controller (deals with database writes - talks to API and Scheduler)
Scheduler (decides what goes where - picks a proper worker)
Workers (does the actual work, this would be where the provider executables reside)

At that point, I think we could start thinking about something along the lines of what you described. It wouldn't be impossible to have something like you described in the current code, but it would be difficult to add it in a way that would not make the code more difficult to decouple in the future. When we have a proper scheduler component, we can develop that further and potentially implement "filters" similar as a concept to what OpenStack has. A request could be passed through the scheduler which in turn would weigh that request using various filters that could be enabled in the scheduler and decide if a worker is returned to take care of the task, or if the request would be throttled for later re-queuing.
All of that, however is dramatically more complex than what currently exists in GARM, and is a large effort that will see us (probably) moving away from sqlite (even though there are some interesting projects out there that could help us stay with sqlite, but that would probably be like jamming a triangle in a square)
I will keep this open (potentially for a long time), but this is something that I acknowledge is useful in some cases.


Improve hover tool behavior on Curve types

Fixes https://github.com/ioam/holoviews/issues/986. Unfortunately in batched mode the hover tool will show all the values in the curve array rather than the specific coordinate you are hovering over so I've disabled displaying curve dimensions when in batched mode.
hv.NdOverlay({i: hv.Curve((range(4), (4-i,i*2, i+3, i))) for i in range(10)}, kdims=['z'])

Batched:

Non-batched:

Why is x repeated for the non-batched case?
I'm not sure how to parse 'Element: 4'; is that the 4th curve from the bottom?  Could you maybe assign names or values to the curves in the example, so that we can see that the name/value of that curve will be shown (hopefully true?)?
Also, apart from the name/value of the curve, presumably what we really want to see is the values of all other dimensions not mapped to the screen, at that point of that curve.  I guess that's not possible?  Presumably that's shown in the non-batched case?  Would be good to clarify that, and to put that in the docs.

Ok, that looks good.  Presumably the second x should be suppressed when it's the same as x and y, for the non-batched case?

Why is x repeated for the non-batched case?

I've fixed that and renamed the dimension the Curves are indexed by to 'z' in the example.

Also, apart from the name/value of the curve, presumably what we really want to see is the values of all other dimensions not mapped to the screen, at that point of that curve.

In the non-batched case that will work all dimensions associated which each curve sample will be shown. In the batched case we will unfortunately only see the NdOverlay dimensions and values.

Sounds good, thanks!  Worth explaining anywhere we explain batched=True.  Not sure why one can't do a lookup of whatever data we want to show, knowing the coordinates, but I guess that would turn this into something needing streams, whereas you are talking about Bokeh's native JS-based hover support?

I guess that would turn this into something needing streams, whereas you are talking about Bokeh's native JS-based hover support?

Right, indeed I don't really know how to modify the hover labels even using dynamic callbacks yet. This is purely about default hover behavior.

Ready to merge when tests pass.

Unfortunately, the new test_curve_overlay_hover test seems to be failing on Travis...

Ah, both a python3 issue and outdated test, will push shortly.

This was an actual bug in python3, hover was broken in some cases.

Good to have unit tests.

Ready to merge again.

Looks good. Merging.


Don't set GdsApi::Base.logger

We're currently setting GdsApi::Base.logger when sending something to panopticon. There's no real reason to use this. This code with extra logging was presumably copied from another repo - we don't need it here.
We still need the puts statements to log the output of the panopticon registration.
Removing the logger prevents noisy output during the tests and cleans up some special casing in the tests.

This looks good to me. Thanks. :+1:


[BUG] Unlocked cells do not maintain their unlocked status after reading and writing a workbook.

🐛 Bug Report
If I read in an xlsx file and then write it back out any unlocked cell will become locked.
Lib version: 3.8.2
Steps To Reproduce
Read and write an xlsx file.
Here is a before and after example, See E4 in the before and after.
ExcelJS-FormattingLockedCellAndDatavalidation.xlsx
ExcelJS-AFTER-FormattingLockedCellAndDatavalidation.xlsx
async load() {
  try {
    const wb = new ExcelJS.Workbook()
    await wb.xlsx.load(await this.file.arrayBuffer())
    const out = await wb.xlsx.writeBuffer()
    this.download(out)
  } catch (e) {
    console.error(e)
  }
}

The expected behaviour:
Unlocked cells stay unlocked after reading and writing a file.

See PR #1210 .


Bump isort in pre-commit

Codecov Report

Merging #183 (1a80960) into main (2209f3b) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #183   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines          184       184
=========================================
  Hits           184       184

Continue to review full report at Codecov.

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


Website autopush release with import group

Could you expand access to datcom-control?

It should already has access as the current version files are read from there.

Could you expand access to datcom-control?

It should already has access as the current version files are read from there.

Ah sorry for the confusion! I thought it was a pantheon project (not just a gcs bucket)


How do I use this library in an Angular project?

Check out this link: https://stackoverflow.com/questions/34329517/how-to-use-npm-module-in-angular
It describes how to use  "braintree-web" but the steps are the same for any other npm module (including this one)

Closing this as an answer has been provided.


Window Freezes after I click on source control

VSCode Version: 1.11.1
OS Version: OSX El Capitan

Steps to Reproduce:

Open VSCode
Click on Source Control section
VSCode freezes

If you wait long enough, does the view come back? If you run Disable Preview SCM, does the Git view work?

The icon changes to a clock. But still, can't click on it. If I Disable Preview SCM the Git view work.
Thanks!

If you Disable Preview SCM, how many changes are in your repository?

Duplicate of huge repo.


Use exact dependency versions and add npmrc with save-exact setting

What:

[x] fix dependency to be exact version
[x] add .npmrc config with save-exact so next time anyone runs --save or --save-dev - version will be saved without ^

[Current coverage][cc-pull] is 100%

Merging [#32][cc-pull] into [master][cc-base-branch] will not change coverage

@@           master   #32   diff @@
===================================
  Files           7     7
  Lines          47    47
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
  Hits           47    47
  Misses          0     0
  Partials        0     0

[][cc-pull]

Powered by Codecov. Last updated by [2506244...80268d8][cc-compare]
[cc-base-branch]: https://codecov.io/gh/kentcdodds/p-s/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/kentcdodds/p-s/compare/25062445a5cca2c2b80705ab937fe84c7a3b7cdb...80268d8021e0361ec36c63cbba1a2a6025c6d864?src=pr
[cc-pull]: https://codecov.io/gh/kentcdodds/p-s/pull/32?src=pr

LGTM. Could you make sure that your pull requests follow the commit message mentioned in the contributing guidelines? Thanks!

Sure, sorry


In-app communication to uploader users about transition

Problem Statement
As a current user, I would like multiple communication channels about the Uploader's move to SimpleReport. This is 1 of 2 communication methods, to inform current, logged in uploader users about the upcoming transition.
Note

Slack thread for context
Figma designs from SimpleReport for ReportStream's consideration

Acceptance criteria

In app message: After logging into reportstream, I'm taken to Upload page, and I see an alert message at the top of the screen about uploader's upcoming transition from reportstream to simplereport
public webpage: show same in-app alert message

To do

develop content for in-app alert message and public website documentation page
Create prototype view and share with eng on #5190

work is pulled into #5145 - closing this ticket.


Browse by organisation should be sorted

Feature Request
The link https://dockstore.org/organizations isn't sorted
Desired behaviour
Sorted list so easier to scan

@JWKaiqi  I think this can be taken into your design for organizational search

This should be sorted in ui2, but we do need to implement links to searches

Demoed previously


Cannot submit a tx on a chain that has a Bech32 prefix other than "cosmos"

Crate
modules, ibc-relayer
Summary of Bug
Testing the ibc-relayer-cli against two chains, one is an Akash chain and the other a Simapp chain. When I tried to perform a tx raw create-client got an error that invalid Bech32 prefix
Version
v0.1.0
Steps to Reproduce

Spin two chains, with at least one chain where the address prefix is not cosmos e.g. Akash is akash
Try to submit a tx raw

Basically the string "cosmos" is hard-coded in a couple of places. The logic needs to change to use the account_prefix value from the chain configuration (config file)
One location that needs to change is here https://github.com/informalsystems/ibc-rs/blob/b654af39c0c658d831cff42ccb11ac5562a43793/modules/src/address.rs#L11
And I believe there's another location here https://github.com/informalsystems/ibc-rs/blob/b654af39c0c658d831cff42ccb11ac5562a43793/relayer/src/keyring/store.rs#L149
Using the account_prefix might be the best option but might require some refactoring because the config (or this value) would need to be passed as a parameter to this functions

For Admin Use

[X] Not duplicate issue
[X] Appropriate labels applied
[X] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Doing some investigation on this issue, realised that the account_to_string method from address.rs in modules is used throughout the modules crate. But I have the feeling this function and the string_to_account method are a lot more closer to a chain implementation, maybe we should move this to the relayer crate and invoke the function from there because we need the config to know what is the account_prefix
For example, here's the create_client message, we should modify the signer in all these from Msg functions
https://github.com/informalsystems/ibc-rs/blob/b654af39c0c658d831cff42ccb11ac5562a43793/modules/src/ics02_client/msgs/create_client.rs#L104
Any thoughts @ancazamfir or @romac ?

We have a long due issue on this https://github.com/informalsystems/ibc-rs/issues/219. We don't really need the account info in the domain type messages. I would rather fix this issue now instead of adding a short-diff patch to fix this.

FYI also #416


Add number_aware_string_key

Add a method to calculate keys for list.sort and sorted from strings
that treats embedded strings of digits as integers.  This is useful to
produce an "expected" ordering of component names, pin names, etc.  For
example, we expect "U2" to be sorted before "U10" in such a list but the
default sort behavior would have the opposite ordering.

Style check failure can be ignored, will be fixed in #1012.


Tweak Retroarch audio latency setting

I experienced some pops and clicks in Super Mario 64 game that went away with this setting (previously set at 64ms). This shouldn't create side effects at all.

This change cannot be done here. There can be no files under /home/gamer in the system image. The actual home directory is mounted on top.
You can make this config change in the Steam Buddy repo.


Fix emtpy package on win32

The package was empty on win32
No need for compilers; just install the header
@conda-forge-admin please rerender

Woops, I just merged the automatic PR because I thought this was maintained by myself only. Just remember this used to be a mamba dep. is this still true @wolfv ?
Also, is the PR I merged defective @jschueller?
Should we rebase yours and merge? Cheers and thanks.

the arm64 PR just merged is fine
the problem is just on windows

@conda-forge-admin please rerender

Cool, thanks!


adjust book style

[ ] allow fold/unfold chapter lists.I tried this but it didn't work.
[x] add edit links to the originating docs.
[x] add link to martin repository. Should it link to marin or matin doc?
[ ] other stuffs need to talk?

comprehensive-rust  has something like this(see mdbook doc),does marin want it?
[output.html]
curly-quotes = true

Some stuffs comprehensive-rust used but I think martin dosen't need.

mdbook-svgbob
mdbook-exerciser

looks good so far!  Do you think we can somehow cleanup the sidebar too?  At the moment it is repetetive - it duplicates the title and the topic on every line:

Martin
Mdbook guide

I figured we might as well iteratively merge these things, no point in a "one big change"

I need a little bit reseach on it

looks good so far! Do you think we can somehow cleanup the sidebar too? At the moment it is repetetive - it duplicates the title and the topic on every line:
Martin	Mdbook guide

I need a little bit research on it.

Thx @sharkAndshark, I guess we should cleanup that summary, e.g. put all Using with docker/... under one group, like "How to run" or "Platform"', or some other heading like that. My concern that currently it looks kinda ugly with each topic repeated twice on the sidebar - https://maplibre.org/martin/


ci: skip ci if "[skip ci]" in last commit message

https://github.com/veggiemonk/skip-commit/issues/5

:tada: This PR is included in version 3.0.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


chore: recursive comparison in hermetic build IT

Fixes https://github.com/googleapis/sdk-platform-java/issues/2540
Sample error message when running library_generation/test/integration_tests.py
Generation finished successfully. Will now compare differences between generated and existing libraries
******************************
Checking for differences in 'java-apigee-connect'.
  The expected library is in /usr/local/google/home/diegomarquezp/Desktop/sdk-platform-java/library_generation/test/resources/integration/golden/java-apigee-connect.
  The actual library is in /usr/local/google/home/diegomarquezp/Desktop/sdk-platform-java/library_generation/output/google-cloud-java/java-apigee-connect.
  Some files (found in both folders) are differing:
   -  README.md
   -  proto-google-cloud-apigee-connect-v1/src/main/java/com/google/cloud/apigeeconnect/v1/ConnectionProto.java

Hi @diegomarquezp, thanks for fixing integration test.
Could you also try to fix the difference in the generated files by changing committish in configuration file?

Hi @diegomarquezp, thanks for fixing integration test.
Could you also try to fix the difference in the generated files by changing committish in configuration file?

Thanks @JoeWang1127, this definitely went right for google-cloud-java, but I'm seeing some strange behavior in java-bigtable that doesn't seem to only depend on the commitish. I'm tracking the issue in https://github.com/googleapis/sdk-platform-java/issues/2549 , I suspect is a special treatment case that should be done in generate_library.sh

@JoeWang1127 @blakeli0 what if we go for fixing java-bigtable as a follow up since the purpose of this PR is to fix the testing logic? I have disabled the bigtable test for now.

LGTM.


Adding types to ConfigInterface for callbacks

You may need to run yarn lint:fix to fix the prettier error 🙏

Fixed! Thanks


no information here when following help link

https://maproulette.org/admin/project/4816/challenge/9054/edit
links to
https://github.com/maproulette/maproulette2/wiki/Introducing-New-MapRoulette---Part-1.-Creating-and-Maintaining-Challenges#via-overpass
which says:
No information here

Good point. This is a MapRoulette front end issue, so I'll move it to MR3


KeyError

After:
from whotracksme.data.loader import DataSource
data = DataSource()

I get:
data available for months:
├── 2017-05
├── 2017-06
├── 2017-07
├── 2017-08
├── 2017-09
├── 2017-10
├── 2017-11
├── 2017-12
├── 2018-01
├── 2018-02
├── 2018-03
├── 2018-04
├── 2018-05
├── 2018-06
├── 2018-07
├── 2018-08
├── 2018-09
├── 2018-10
├── 2018-11
├── 2018-12
├── 2019-01
├── 2019-02
├── 2019-03
├── 2019-04
├── 2019-05
├── 2019-06
├── 2019-07
├── 2019-08
├── 2019-09
├── 2019-10
├── 2019-11
├── 2019-12
load trackers
update/create data for 2017-05/global/trackers.csv
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/kato/Applications/whotracks.me/whotracksme/data/loader.py", line 60, in __init__
    populate=populate,
  File "/home/kato/Applications/whotracks.me/whotracksme/data/loader.py", line 189, in __init__
    self.db.load_data('trackers', self.region, month)
  File "/home/kato/Applications/whotracks.me/whotracksme/data/db.py", line 312, in load_data
    [row[col] for col in name_columns] + \
KeyError: 'month'

In which context are you loading the module? From pip or just running code from within the project?

On linux I used source code, did a couple of steps:
$ pip install -r requirements.txt
$ pip install -e .

Thren open Jupyter notebook (using python3) or simple python3 prompt and run:
>>> from whotracksme.data.loader import DataSource
>>> data = DataSource()

Then I get the error (updated the output above - it prints months available before error).

I just tested this locally in a fresh virtualenv and didn't get an error. Do you have git-lfs installed and are the csv files in whotracksme/data/assets/ all downloaded?

Do you have git-lfs installed

Thanks a lot for this hint and your work in general, it was really helpful! It might be useful to add this to the read-me, unless it's already there, and I just missed it.


[ci] build and cache bitstream pre-merge

This is a test PR for enabling merge queue.

This took a lot of tries. Doesn't seem like we'll be able to use it for master unless we bring the queue down right?

We can increase the timeout


Solution to task ef135b50.json

Could you please explain what is the rule used to generate output in the following task https://github.com/fchollet/ARC/blob/master/data/training/ef135b50.json ?
Demonstrations 1 and 2 suggest "Paint all black fields lying between two red fields dark red", but then demonstration 3 would look differently and solution to the actual task would look differently as well.

It has been fixed with the above pr


HDDS-9369. Avoid deserializing OMRequest multiple times in OzoneManagerStateMachine.

What changes were proposed in this pull request?
In OzoneManagerStateMachine, the same log data is deserialized to the same OMRequest multiple times. It can be avoided by setting stateMachineContext.
What is the link to the Apache JIRA
HDDS-9369
How was this patch tested?
By existing tests.

cc @tanvipenumudy

In looking at this, I checked:
  public static OMRequest convertByteStringToOMRequest(ByteString byteString)
      throws InvalidProtocolBufferException {
    byte[] bytes = byteString.toByteArray();
    return OMRequest.parseFrom(bytes);
  }

And I noticed it takes the passed byteString and converts it to Bytes and then passes the bytes to OMRequest.parseFrom(). OMRequest.parseFrom also accepts a ByteString as a parameter, avoiding this copy - however, the parameter to this method is org.apache.ratis.thirdparty.com.google.protobuf.ByteString; and the parameter in the method is com.google.protobuf.ByteString.
I don't know much about how this Ratis stuff is put together, but it seems that we are paying the price for an extra copy on every message received due to "different" but likely the same classes being used behind the scenes.

We had the similar problem in HBase where the protobuf it used collides with the one shaded by Hadoop 3.3.
IIRC we solved it using reflection. Not the best solution IMO.

Proto 3 support wrapping a ByteBuffer to a ByteString but proto 2 does not.  We may be able to copy the proto 3 wrapper code to proto 2.  Then, the conversion can be done by
ByteString proto2 = wrap(proto3byteString.asReadOnlyByteBuffer());

Let me try it in a separated JIRA.

@kerneltime , @duongkame , @sodonnel , @jojochuang , thanks a lot for the review and ideas!

... We may be able to copy the proto 3 wrapper code to proto 2. ...

Tried it a little bit but not succeeded.  Many ByteString subclasses in proto 3 were missing in proto 2 and the wrap method needed them.


Table text-align inheritance

@jonathantneal thoughts on this?
table th, table td {
   text-align: inherit;
}

Yea, I actually really like that. Do we have examples of this on the web?

Considering this something to add.

I feel like I needed this at one point, but then went back to test it again, and it was doing it out of the box. I need to re-test

@dbox, so was this not an issue? If so, may we close it?

Not an issue! closing


chore(main): release mqtli-pub 0.1.1

:robot: I have created a release beep boop
0.1.1 (2024-12-30)
Miscellaneous Chores

Release 0.3.0 (99e33ef)
Release 0.4.0 (70d0310)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

mqtli-pub-v0.1.1
:sunflower:


development cycle | rest api, cleanup and more

The following holds a series of development changes to help improve the sphinxcontrib-confluencebuilder extension. Most notable changes include:

REST API Support (8a39c9d5448b7cca6392a6a167877c0d74b28229)
Page-prefix Support (1e525b7ac9dc1927915b9604e31e9c4a900a1735)
Proxy Support (ced564581ab6640f5a29cc23b8b0e9a4acd606c6)

Along with a series of several minor features and cleanup.

@tonybaloney Agreed; I'll try to dedicate some time to improve test coverage if a5abe1a6245a3a60c71e5ee8ace631cae84c2616 (and f5758443ebd7468c593dbdd3cf72528af1c55cfc) is merged into the master (in #21).


cypress: apply valid license admin

Overview
Add cypress tests for the apply valid admin license flow.

[ ] add testing environment that has no license

as a non-admin user

[ ] Request a trial license (get 403)
[ ] Apply a valid license (get 403)

as an admin user:

[ ] Attempt to login
[ ] Request a trial license
[ ] Fail to apply an invalid license
[ ] Fail to apply an expired license
[ ] Apply a valid license

Resources

Acceptance testing script

this work has been deprioritized.


Lesson 13: useTokenBalance from @usedapp/core returns undefined

I'm trying to get the token balance of the user and output it onto the front end but when I console.log tokenBalance, it just returns undefined. I'm not sure why this isn't working. I thought this could be a package issue but I reinstalled @usedapp/core. I also cloned the github repo and  I still got this issue.
import {Token} from "../Main"
import {useEthers, useTokenBalance} from "@usedapp/core"
import {formatUnits} from "@ethersproject/units"

export interface WalletBalanceProps {
    token: Token
}

export const WalletBalance = ({token}: WalletBalanceProps) => {
    const {image, address, name} = token
    const {account} = useEthers()
    console.log(account)
    console.log(address)
    const tokenBalance = useTokenBalance(address, account)
    const formattedTokenBalance: number = tokenBalance ? parseFloat(formatUnits(tokenBalance, 18)) : 0
    console.log(tokenBalance?.toString())
    return (<div>{formattedTokenBalance}</div>)
}

Any help would be appreciated :)

I got the same problem but no luck so far :(

Hi, I just found a way is to downgrade your usedapp package to 0.7.0 and it will return value as normal:
yarn remove @usedapp/core yarn add<EMAIL_ADDRESS>

I had this problem, but I created a new front-end separate from brownie, and re-install<EMAIL_ADDRESS>It's working fine now. I had a problem with older version when it comes to useContractCall, in newer version the useCall is working better for me.


MetalsInstall doesn't work on Apple Silicon (M1)

Describe the bug
Below is what I get when opening a Scala for the first time. This is expected

When I run :MetalsInstall I get the following error.

Expected behavior
Metal's Install to run and install correctly.
Operating system
macOS
Version of Metals
v0.11.1
Commit of nvim-metals
Cannot Find as Metals will not install.

Okay I was able to get :MetalsInstall to work after Installed Rosetta. It looks like Coursier needs to release a version that fully works with ARM.

Hey @ironfish, thanks for reporting. I'll leave this open to help discovery in case others hit on this. As you already found it it seems, there are issues with Coursier on M1. I see you already commented on them 😆 but I'll link them here as well:

https://github.com/coursier/coursier/issues/2069
https://github.com/coursier/coursier/issues/2335

Hey @ironfish, it looks like coursier merged in support for Graal 22.1.0 from my understanding supports M1. Do you know if this fixes your install woes?

Great, thanks for reporting on this @reidrac. I'll go ahead and close. If others hit on this, please do report back.


Update vendors.yml

Added Splashtop based on recent quote request.

I inquired with Splashtop what I needed to do to enable SSO on my account. To my surprise and dismay, they need 12x the monthly cost for their enterprise product.

Seconding this. Splashtop only offers SSO in their "Enterprise" product. Not sure if using the per endpoint pricing is better than using the concurrent technician sessions pricing. Billed annually, it's $199/concurrent session for their base "SOS" product versus $750/session for their "Enterprise" (received through quote during a demo as they don't list this pricing on the website).


[Thirsty Sword Lesbians] 10 new playbooks

Changes / Comments
Added in the support for ten additional playbooks from the upcoming Advanced Lovers & Lesbians supplement.
This should not impact existing character sheets and data at all (no player data at risk, so no steps necessary to preserve it, as it should remain untouched).
Older character records won't be able to access the new playbooks, but any newly created records will see them.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[x] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


changing the subject id will greatly reduce the performance

HI, thx for release your code, it helps me a lot.
I just find out that change the subject id in your tutorial (randomly pick 5 as test in 1-56) will greatly reduce the accurracy. I guess maybe you can give a cross-validation result or something....Since the result is higher than any other condition.

In general different results for different subjects are expected in EEG decoding.


Make screen readers and other assistive technologies read the response status as soon as the request completes

This is related to the discussion #3720. Currently, when we press CONTROL+ENTER to send a request, the screen reader speaks nothing when the request completes, requiring the user to navigate to the status element to view if the request succeeded or not. It would be nice when the request finishes, Insomnia notifies the new HTTP status, so the user won't need to find the element to know it.

@thgcode thank you for reporting and contributing a pull request. Your contribution was merged.
Please feel free to submit more issues and pull requests that will make screen readers and assistive tech experience better.


fix(@angular-devkit/build-angular): force event category to be something

If it is empty (or undefined), the Universal Analytics package will not send events.

@hansl  - Tests are failing

@vikerman are those flakes? They seem unrelated to this change.


Update testing-debugging.md

Bit of nitpick: The qsharp language header might be a bit confusing to someone new to the Q#. %%qsharp can only be used with jupyter notebook(python kernel with Q#) installed, so it might be helpful to say python, all along the way
Doc link: https://learn.microsoft.com/en-us/azure/quantum/user-guide/testing-debugging

@Manvi-Agrawal - take this out of draft mode when ready and @Bradben  will review.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @Manvi-Agrawal - thanks for the feedback. You raise a good point, but the code header needs to reflect the actual code used in the cell, which is Q#, even though it technically is a Python cell.  I've opened a work item to clarify the content around the examples to clarify that distinction. Thanks!

Already fixed in core docs.


Support investment

Hi,
I noticed that my "Zelf beleggen" orders weren't being imported correctly, so I fixed that :)
I can send you a test file via PM if you need it.
Let me know if you need any changes.

A test file would be great. You could also mail it to me (see my profile for the email).
I will look at the commit after the weekend.
Thanks for helping!

Thanks for taking a while to get back to you. Attached is an anonymised test file
testfile2.TAB.txt


Asset name (T) and location (lat/long – N, and location address, if used – T, if applicable)

Part of Asset Inventory: Asset Data Requirements


Install script for Windows

We currently only have an install script for MacOS

I run windows and can try and help if there are more specs here. Also, are all the python libraries windows compatible? I recently worked on some projects that were not and needed a VM/WSL. A good thing to clarify before going down a rabbit hole (like I did :) ).

Hey @Avr-Arnold i think all the packages are compatible with Windows but if you find one that isn't feel free to raise an issue

I want to take this up. There are 2 ways in which I'm thinking of addressing this issue:

We can use containerization software like docker to install the repo, we can fix the base image to linux. The advantage with this route is we need to maintain only one setup script and we don't need to maintain setup scripts for different oses.
There is a package manager for windows called winget, I try writing a powershell script such that required packages are installed from winget and the setup happens through powershell.

I have added the installation instructions for Windows OS in the above mentioned PR

Please check it @VictorOdede @abhigya-sodani


Feat: Add Neon Mainnet Network

Description
This PR adds Neon EVM to the Trust Wallet supported RPCs
How to test
./tools/build-and-test
[==========] 2374 tests from 579 test suites ran. (54854 ms total)
[  PASSED  ] 2374 tests.

Types of changes
Adds Neon EVM mainnet chain
https://docs.neonfoundation.io/docs/developing/connect_rpc
https://neonevm.org
Checklist

[x] Create pull request as draft initially, unless its complete.
[x] Add tests to cover changes as needed.
[x] Update documentation as needed.
[x] If there is a related Issue, mention it in the description.

If you're adding a new blockchain

[x] I have read the guidelines for adding a new blockchain.

@joehquak please also fix Android and iOS tests. You can run them using Android studio and XCode.
Or by typing
./tools/android-test
./tools/ios-test

@joehquak please also fix Android and iOS tests. You can run them using Android studio and XCode. Or by typing
./tools/android-test
./tools/ios-test

Will do so now, thank you 🙏

@satoshiotomakan Should be ready now I hope 😁


"Modify jQuery" section is incorrect

The "Modify jQuery" function is doing it wrong. See "Notes" section for the wp_register_script() function documentation:

The function should be called using the wp_enqueue_scripts action hook if you want to call it on the front-end of the site. To call it on the administration screens, use the admin_enqueue_scripts action hook. For the login screen, use the login_enqueue_scripts action hook. Calling it outside of an action hook can often lead to unexpected results and should be avoided.

You can get some additional insight from the old blog post by Pippin here as well.
In short, registering, and even more - enqueueing scripts on init hook is wrong. I'll submit a PR covering all cases when I get a minute. Also, it may conflict with #30

What a coincidence. I submitted a Pull Request for this today. https://github.com/taniarascia/wp-functions/pull/30

@viktorbijlenga Yes, I've got notification about your PR, that's why I actually had a look into the code and chimed in. Your PR replaces the init hook (which is correct) but does not mention non-frontend cases - admin and login and doesn't explained why the change is being made.
Having seen your PR is why I said if I'm submitting a PR covering all scenarios, it might conflict with yours. Actually, you can amend your commit with admin_enqueue_scripts and login_enqueue_scripts plus some inline docs explaining the cases and close this issue.

I read your comment a bit to fast, so saw your mention a bit to late.
Feel free to improve or refactor my code with a new pull request if you find the time. Your additions sounds like great improvements to me.

No worries, I'll do it in the next few days.


add autoscalingTemplate to the hpa

Added autoscalingTemplate to the hpa template and changed if clauses to with which is more effective:
https://helm.sh/docs/chart_template_guide/control_structures/ #59

Many thanks @mfuxi , I merged your changes in a separate branch and opened a new PR with some minor fixes and actual version bump - https://github.com/centrifugal/helm-charts/pull/62.


Resources are not saving if scene is not opened

Godot version:
Godot 3.1 Beta 6
OS/device including version:
Mac OSX Mojave and Windows
Issue description:
As requested by @reduz I created this new issue referencing an old issue regarding resource saving in the editor (#25832). Resources are saved along side scenes. When a scene is saved the resource is also saved, so if you are editing a resource even if it is not being used by the scene it gets saved, and that works nice.
The problem is that when there is no scene open and you are just editing a bunch of resources from your game there is no obvious way of saving them, other than running your game.
Cmd-S (saves only if the scene is opened) and Cmd-Alt-S saves only the script in the script editor.
there is no quick way to save the resource other than clicking in the save icon and selecting "Save" instead of "Save as" and this is tedious.
Another problem is that there is no obvious notification that your resource as been edited and not saved... if you try to close the project you don't get notified about unsaved resources (this led me to lose a lot of time spent in resource editing).
The current way of navigating through resources in the inspector also doesn't... (as you just have a "back" and "forward" button) you don't know how many resources you edited you don't have a visual cue of how many resources are actually open.
PS: sorry the shorts cuts are for OSX (I don't use windows only in VM).
Steps to reproduce:
N/A
Minimal reproduction project:
N/A

you don't have a visual cue of how many resources are actually open.

My guess on this is, they are never actually closed until you close Godot itself. They just sit in the history forever.


Write books via AsciidocFX (and a possible Book Template project)

I want to write actual books, not software documentation, via AsciidocFX. This is similar to what has already been done and linked to in the ReadMe.
To do this, I'm thinking of using these three tools:

Gradle
check and warn about broken links/references via HTMLSanityCheck
automatically build all the backends and store them in one folder
update the book's revision date / number programmatically
and other things that are better automated...
IntelliJ Idea
for GUI Git integration as an immediate workaround to #206
for editing the build.gradle file as needed
AsciidocFX
for actually writing and immediately seeing the result

First, do you know of any reasons why this might not work? If the IDE and AsciidocFX are both watching/modifying the same file, will that lead to problems?
Second, and assuming this does work, I thought it would be wise to create a book template that others can fork and then build upon. The current examples in asciidoc-gradle-examples all apply the java plugin because they assume it's being used for software documentation. However, in my case, this isn't necessary.
So, my current implementation of this template idea is asciidoc-gradle-book-template. I'd appreciate any help in getting this template properly configured and set up. Once it's done, perhaps we could link to this template project in the ReadMe.

Hello @JordanMartinez , what does not work ? AsciidocFX brings a comfortable writing experiences for developers and authors. It has an book sample menu, did you try it ? If you have any idea for this, I'm open for it.

It has an book sample menu, did you try it ?

Yes, I've tried it and it works quite well. However, I'm looking for more features than just what AsciidocFX currently offers: Git integration & automated testing of links. The VCS integration isn't supported but will be in the future I'm sure. I'm not so sure about the automated testing.
Additionally, I've noticed that there is a small but still noticeable lag when I'm using AsciidocFX to write asciidoc as opposed to my IDE. I'm not sure if this could be fixed if I changed a JVM argument. So, that's why I'm currently using my IDE and a few plugins instead of this project.

I also experience some small but annoying lag when I'm writing, even when the preview is disabled.

@damko Can you please share more details?

@rahmanusta with pleasure, what shall I provide exactly?
I have openJDK8 installed. I tried to tweak a  little the option file but there is no change. Everything works perfectly and the application is very good but it feels like rusted
If you had the change to work with remote terminals on poor connections, you can totally understand which kind of lags I'm experiencing: everything feels delayed by some 0.1 to 0.8 seconds (approximately).
No other editor I use has this issue but it's true that this is the only java based editor I have.
Maybe I should try Eclipse and see if it behaves differently?

Can you check https://bugs.openjdk.java.net/browse/JDK-8096192, and try using  -Dprism.forceGPU=true JVM parameter in .vmoptions file in AsciidocFX installation directory?


Wiring Formik Form to call a Reducer

I am new to React, Formik and everything else related to front end programming. I am trying to create a simple from with Formik that allows a user to register. The submit button should call my registerUser action which would then pass the information from the form to the action and then the saga and reducer. I think I have wired up the store/reducer/actions/sagas correctly but I can't figure out how to use Formik correctly. Below is what I have so far.  Can someone give me a guide on what to do? I have tried patching together numerous different version of the register-form.tsx component following the few examples I could find but can't get it to work.
My full app is here: https://github.com/infornite/n4nite-web-ui
register-form.tsx
import * as React from 'react';
import { Formik, Field, Form, FormikActions, InjectedFormikProps, withFormik  } from 'formik';
import { Dispatch } from 'redux';
import registerUser from '../../store/register/actions'
import * as Yup from 'yup';
import { connect } from 'react-redux';
import { bindActionCreators } from 'redux';
import { ApplicationState, ConnectedReduxProps } from '../../store'

interface FormProps {
  name: string;
  email: string;
  password: string;
  registerUser: typeof registerUser
}

interface FormValues {
    name: string;
    email: string;
    password: string;

}

// Our inner form component. Will be wrapped with Formik({..})
const InnerForm: React.SFC<InjectedFormikProps<FormProps, FormValues>> = (
    props,
  ) => (
    <form onSubmit={props.handleSubmit}>
      <input
        id="name"
        placeholder="Name..."
        type="text"
        onChange={props.handleChange}
        value={props.values.name}
      />
      {props.touched.name && props.errors.name &&

{props.errors.name}
}
      <input
        id="email"
        placeholder="Email..."
        type="text"
        onChange={props.handleChange}
        value={props.values.email}
      />
      {props.touched.email && props.errors.email &&

{props.errors.email}
}
      <input
        id="password"
        placeholder="Password..."
        type="text"
        onChange={props.handleChange}
        value={props.values.password}
      />
      {props.touched.password && props.errors.password &&

{props.errors.password}
}
      <button
        type="submit"
        disabled={props.isSubmitting}
      >
        Register
      </button>
    </form>
  );

  //const handleSubmit = (name: string, email: string, password:string { props: props }) => props.registerUser(values.name, values.email, values.password);

  const RegisterForm = withFormik<FormProps, FormValues>({
    mapPropsToValues: () => ({
      name: '',
      email: '',
      password: ''
    }),
    validationSchema: Yup.object().shape({
      name: Yup.string()
      },
    ),
    handleSubmit: (values, {props, setSubmitting }) => {
      setTimeout(
        () => {
          alert(JSON.stringify(values, null, 2));
          setSubmitting(false);
          console.log(values);
          console.log(props);
          props.registerUser(values.name, values.email, values.password);
         },
        1000,
      );
    },
  })(InnerForm);

  const mapStateToProps = ({ register }: ApplicationState) => ({
    name: register.name,
    email: register.email,
    password: register.password,
    loading: false,
    errors: ""
  })

  const RegisterUser = connect(
    mapStateToProps ,null
  )(RegisterForm);

  export default RegisterUser;

actions.ts
import { action } from 'typesafe-actions'
import { RegisterActionTypes } from './types'

export const registerRequest = (name: string, email:string, password:string) => action(RegisterActionTypes.REGISTER_REQUESTING, {name, email, password})
export const registerSuccess = () => action(RegisterActionTypes.REGISTER_SUCCESS)
export const registerError = () => action(RegisterActionTypes.REGISTER_ERROR)

export default registerRequest

reducers.ts
import { Reducer } from 'redux'
import { RegisterState, RegisterActionTypes } from './types'

// Type-safe initialState!
const initialState: RegisterState = {
  name: '',
  email: '',
  password: '',
  errors: undefined,
  loading: false
}

const reducer: Reducer<RegisterState> = (state = initialState, action) => {
  switch (action.type) {
    case RegisterActionTypes.REGISTER_REQUESTING: {
      return { ...state, loading: true }
    }
    case RegisterActionTypes.REGISTER_SUCCESS: {
      return { ...state, loading: false }
    }
    case RegisterActionTypes.REGISTER_ERROR: {
      return { ...state, loading: false }
    }
    default: {
      return state
    }
  }
}

export { reducer as registerReducer }

sagas.ts
import { call, put, takeLatest } from 'redux-saga/effects'
import { RegisterActionTypes } from './types'
import { handleApiErrors } from '../../utils/handleAPIErrors'

const registerUrl = process.env.INFORNITE_API_ENDPOINT + '/users' || ''

function signupApi (name: string, email: string, password:string) {
  return fetch(registerUrl, {
    method: 'POST',
    headers: {
      'Content-Type': 'application/json',
    },
    body: JSON.stringify({ name, email, password }),
  })
    .then(handleApiErrors)
    .then(response => response.json())
    .then(json => json)
    .catch((error) => { throw error })
}

function* registerFlow (action:any) {
  try {
    const { name, email, password } = action
    const response = yield call(signupApi, name,  email, password)
    yield put({ type: RegisterActionTypes.REGISTER_SUCCESS, response })
  } catch (error) {
    yield put({ type: RegisterActionTypes.REGISTER_ERROR, error })
  }
}

function* registerWatcher () {
  yield takeLatest(RegisterActionTypes.REGISTER_REQUESTING, registerFlow)
}

export default registerWatcher

types.ts
//The base API response type
export type ApiResponse = Record<string, any>

//Fields for the user we want to register
export interface RegisterUser extends ApiResponse {
  name: string,
  email: string,
  password: string,
}

//The types of actions that can be performed
export const enum RegisterActionTypes {
    REGISTER_REQUESTING = '@@register/REGISTER_REQUESTING',
    REGISTER_SUCCESS = '@@register/REGISTER_SUCCESS',
    REGISTER_ERROR = '@@register/REGISTER_ERROR',
  }

//A state which redux will manage for a User
export interface RegisterState extends RegisterUser{
    readonly loading: boolean
    readonly errors?: string
  }

Can you provide more context on what's wrong? Don't have time to read or run all that code you posted
I know it's not what you want to hear, but I'd suggest you try making a simplified version of the form without Formik. Typescript, React, Redux, and Redux-Saga is a WHOLE EFFING LOT of stuff to deal with all at once

From what I understand, Formik is designed to encapsulate the state of the form for you, managing the lower-level stuff like events and state while providing an API to do the actions we more care about like validation and submission. One of the guiding principles behind Formik is to not have to put the state of the form, which is inherently transient, or handle the individual DOM events in redux.

You might want to consider not using this library if you need to manage the state of the form explicitly.

@angelbeltran I would agree with you if there was no setFieldValue function, but there is:
https://jaredpalmer.com/formik/docs/api/formik#setfieldvalue-field-string-value-any-shouldvalidate-boolean-void
Why not go the extra distance and allow us to manage state as needed? Instead, I have to create custom handlers for specific fields that have complicated data (managing arrays of objects for instance).

@antgonzales how did it fare on your end? I am also in this scenario but having multiple arrays of objects as the form is quite complex. Resulting to lag on every keypress as when values of the context change - all of the dependents of the provider will trigger re-render.

@sshanzel sorry for the delay, we actually moved away from Formik because the forms we're delivering are extremely complicated. I think Formik is a great library just ended up being not best for my project's use case.


Make CORS configurable

Is your feature request related to a problem? Please describe:
Alternate web UIs cannot use the API because CORS is only configured to trust a couple local origins by default.
Describe the solution you'd like:
in src/main.ts the // setup cors section (line 93 currently) has two hard-coded localhost origins. It would be great if this setting could be configured through the config.json to allow alternate clients -- such as this one for retro devices.
Describe alternatives you've considered:
I have this working by setting up an apache2 reverse proxy where I can configure the CORS settings.
I have also made it work by modifying the running code at /usr/local/lib/node_modules/homebridge-config-ui-x/dist/main.js line 82505 to Set  origin: '*',
However, both of these solutions are too complex for most users. A configuration option would be greatly appreciated!

I do the same with Apache, but it would be nice to have this built-in/configurable.

I do the same with Apache, but it would be nice to have this built-in/configurable.

Any thoughts on giving the user control over CORS without having to hack at the source or setup a proxy?


Remove explicit snappy-java dependency

I would remove the snappy-java dependency from the Parquet subproject. This dependency is only required if you're using snappy compression, which is just one of the compressions for Parquet.
Besides that, nothing will change because the Parquet plugin requires the Avro plugin, transitively pulls in the library:
[INFO] --------< org.apache.druid.extensions:druid-parquet-extensions >--------
[INFO] Building druid-parquet-extensions 0.16.0-incubating-SNAPSHOT     [34/64]
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- maven-dependency-plugin:2.8:tree (default-cli) @ druid-parquet-extensions ---
[INFO] org.apache.druid.extensions:druid-parquet-extensions:jar:0.16.0-incubating-SNAPSHOT
[INFO] +- org.apache.druid.extensions:druid-avro-extensions:jar:0.16.0-incubating-SNAPSHOT:provided
[INFO] |  +- org.apache.avro:avro:jar:1.8.2:provided
[INFO] |  |  +- com.thoughtworks.paranamer:paranamer:jar:2.7:provided
[INFO] |  |  \- org.xerial.snappy:snappy-java:jar:<IP_ADDRESS>:compile

However, explicitly adding this to the pom is not correct in my opinion.

Any thoughts on this?

https://github.com/apache/incubator-druid/pull/8230 also removes the explicit snappy-java dependency

#8230 was merged and removed the explicit snappy-java dependency: https://github.com/apache/incubator-druid/pull/8230/files#diff-9876210e3556d07807fe3dbd7a8c3460L99

Awesome @ccaominh, thanks for the heads up!


When the params query contains the # symbol, the system ignores this symbol and everything after it, resulting in issues with the query results

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug
1. Issue Description:
When the params query contains the # symbol, the system ignores this symbol and everything after it, resulting in issues with the query results.
2. Expected Behavior:
The # symbol and everything after it in the query should be processed correctly, instead of being treated as a comment and ignored.
3. Steps to Reproduce:
Include the # symbol and its subsequent content in the query.
Run the query and observe the results.
Notice if the query results are missing the content after the # symbol.
4. Actual Behavior:
The content after the # symbol is ignored in the query results, leading to incomplete or incorrect results.
5. Environment Information:
Operating System: Windows
.bru file to reproduce the bug
ListNgsStepsV2.zip
Screenshots/Live demo link
This one's from bruno.

This one's from other app.

I think that the # and everything after is not sent, is expected behavior, see: https://blog.httpwatch.com/2011/03/01/6-things-you-should-know-about-fragment-urls/
You could change the # to the URL-Encodes equivalent → %23. But Bruno does not do encode the URL by default.
We should definitely update the query parameter parser to handle #, so it's more obvious where the URL fragment begins.

Bruno does not do encode the URL by default

This is the actual issue here. If there is a table where you enter parameters and the platform converts them into a query string (concatenates them with ? and &) the platform should also encode the parameters before concatenating. Everything else would be wrong and unexpected in my book.

On the same note: #2124.
The problem to keepn in mind here is that the conversions from query params in the URL field should automatically update the table and at the same time the updates in the table should automatically update the url.
Does that mean the table should ALWAYS contain non-url encoded values (or if they are provided url encoded, they get double encoded on send)?
What about the equals sign typed as value into the URL field?
E.g.: https://example.com/search?filter=color=red
is a url with query param filter having value of color=red . If I type it into url field, it's not wrong, and the table will be updated accordingly. Then, assuming the updates on in the table should automatically url-encode query param values in the URL field, any change to params table will change value in url field from:
https://example.com/search?filter=color=red to https://example.com/search?filter=color%3Dred, which seems undesired as well, right?

I think that the # and everything after is not sent, is expected behavior, see: https://blog.httpwatch.com/2011/03/01/6-things-you-should-know-about-fragment-urls/
You could change the # to the URL-Encodes equivalent → %23. But Bruno does not do encode the URL by default.
We should definitely update the query parameter parser to handle #, so it's more obvious where the URL fragment begins.

thx,Your advice worked

Hi, I am still facing this issue... can you check if the fix has been released in app binary (I am using windows 11) or the bug has occurred again?...

Hi, I am still facing this issue... can you check if the fix has been released in app binary (I am using windows 11) or the bug has occurred again?...

There was no fix in the app. You currently sitll have to manually encode the # into %23


Added CameraInfoService

Nixing this PR as the service needs a hell of a lot of work to be done to it.


💡 [REQUEST] - Enhance UI of the Contact us page

[x] I am from GSSoC'23.
[x ] I would like to be assigned this task.
I would like to change the ui of the contact us page by adding some cool effects.
So,please assign this issue to me .

Hi @Pragati-3003
Thank you for creating the issue. Please go ahead and start working on it. If you have any questions or need assistance during the process, feel free to ask.
Thank you for your contribution, and I'm excited to see your progress in resolving the issue.


Tensorflow 1.10.0 MKL build with bazel throws linker error

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): 1.10.0
Python version: 3.6 (doesn't really matter)
Bazel version (if compiling from source): 0.15.2
GCC/Compiler version (if compiling from source): VS2015 Update 3
CUDA/cuDNN version: N/A
GPU model and memory: N/A
Exact command to reproduce:

Describe the problem
Tried compiling tensorflow from source, cpu version, with mkl.
Source code / logs
Build fails with the following output:
ERROR: C:/ci/tensorflow-base_1535347248498/work/tensorflow/python/BUILD:3627:1: Linking of rule '//tensorflow/python:_pywrap_tensorflow_internal.so' failed (Exit 1000): link.exe failed: error executing command
  cd C:/users/nwani/_bazel_nwani/ozaxfmyw/execroot/org_tensorflow
  SET INCLUDE=C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\INCLUDE;C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\ATLMFC\INCLUDE;C:\Program Files (x86)\Windows Kits\10\include\10.0.10240.0\ucrt;C:\Program Files (x86)\Windows Kits\NETFXSDK\4.6.1\include\um;C:\Program Files (x86)\Windows Kits\8.1\include\\shared;C:\Program Files (x86)\Windows Kits\8.1\include\\um;C:\Program Files (x86)\Windows Kits\8.1\include\\winrt;
    SET LIB=C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\LIB\amd64;C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\ATLMFC\LIB\amd64;C:\Program Files (x86)\Windows Kits\10\lib\10.0.10240.0\ucrt\x64;C:\Program Files (x86)\Windows Kits\NETFXSDK\4.6.1\lib\um\x64;C:\Program Files (x86)\Windows Kits\8.1\lib\winv6.3\um\x64;
    SET PATH=C:\Program Files (x86)\Microsoft Visual Studio 14.0\Common7\IDE\CommonExtensions\Microsoft\TestWindow;C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\BIN\amd64;C:\WINDOWS\Microsoft.NET\Framework64\v4.0.30319;C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\VCPackages;C:\Program Files (x86)\Microsoft Visual Studio 14.0\Common7\IDE;C:\Program Files (x86)\Microsoft Visual Studio 14.0\Common7\Tools;C:\Program Files (x86)\Microsoft Visual Studio 14.0\Team Tools\Performance Tools\x64;C:\Program Files (x86)\Microsoft Visual Studio 14.0\Team Tools\Performance Tools;C:\Program Files (x86)\Windows Kits\8.1\bin\x64;C:\Program Files (x86)\Windows Kits\8.1\bin\x86;C:\Program Files (x86)\Microsoft SDKs\Windows\v10.0A\bin\NETFX 4.6.1 Tools\x64\;;C:\WINDOWS\system32
    SET PWD=/proc/self/cwd
    SET TEMP=C:\Users\nwani\AppData\Local\Temp
    SET TMP=C:\Users\nwani\AppData\Local\Temp
    SET USE_LINKER=1
  C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64/link.exe /nologo /DLL /SUBSYSTEM:CONSOLE -DEFAULTLIB:advapi32.lib /MACHINE:X64 @bazel-out/host/bin/tensorflow/python/_pywrap_tensorflow_internal.so-2.params /DEF:bazel-out/host/genfiles/tensorflow/python/pywrap_tensorflow_filtered_def_file.def /ignore:4070
LINK : warning LNK4044: unrecognized option '/lpthread'; ignored
LINK : warning LNK4044: unrecognized option '/lm'; ignored
LINK : warning LNK4044: unrecognized option '/ldl'; ignored
LINK : warning LNK4044: unrecognized option '/lpthread'; ignored
LINK : warning LNK4044: unrecognized option '/lpthread'; ignored
LINK : warning LNK4044: unrecognized option '/lm'; ignored
LINK : warning LNK4044: unrecognized option '/ldl'; ignored
LINK : warning LNK4044: unrecognized option '/lm'; ignored
LINK : warning LNK4044: unrecognized option '/lm'; ignored

bazel-out/host/bin/external/nsync/nsync_cpp.lib : fatal error LNK1000: Internal error during CImplib::EmitThunk

  Version 14.00.24215.1

  ExceptionCode            = C0000005
  ExceptionFlags           = 00000000
  ExceptionAddress         = 00007FF7EF196896 (00007FF7EF180000) "C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\bin\amd64\link.exe"
  NumberParameters         = 00000002
  ExceptionInformation[ 0] =<PHONE_NUMBER>000000
  ExceptionInformation[ 1] =<PHONE_NUMBER>000008

CONTEXT:
  Rax    =<PHONE_NUMBER>000000  R8     = 00007FF7EF27FBE0
  Rbx    =<PHONE_NUMBER>000000  R9     = 00007FF7EF27E9F0
  Rcx    =<PHONE_NUMBER>000000  R10    =<PHONE_NUMBER>000000
  Rdx    = 00007FF7EF27FBD8  R11    =<PHONE_NUMBER>000000
  Rsp    = 000000A6F5B2DE28  R12    = 00007FF7EF24D950
  Rbp    = 0000023D587FA330  E13    =<PHONE_NUMBER>000000
  Rsi    =<PHONE_NUMBER>008000  R14    =<PHONE_NUMBER>000000
  Rdi    = 0000023D59249700  R15    =<PHONE_NUMBER>000000
  Rip    = 00007FF7EF196896  EFlags =<PHONE_NUMBER>010246
  SegCs  =<PHONE_NUMBER>000033  SegDs  =<PHONE_NUMBER>00002B
  SegSs  =<PHONE_NUMBER>00002B  SegEs  =<PHONE_NUMBER>00002B
  SegFs  =<PHONE_NUMBER>000053  SegGs  =<PHONE_NUMBER>00002B
  Dr0    =<PHONE_NUMBER>000000  Dr3    =<PHONE_NUMBER>000000
  Dr1    =<PHONE_NUMBER>000000  Dr6    =<PHONE_NUMBER>000000
  Dr2    =<PHONE_NUMBER>000000  Dr7    =<PHONE_NUMBER>000000
Target //tensorflow/tools/pip_package:build_pip_package failed to build
INFO: Elapsed time: 2226.356s, Critical Path: 254.23s
INFO: 4537 processes: 4537 local.
FAILED: Build did NOT complete successfully
+ exit 1

i thought that bazel si currently not stable in windows

Related to https://github.com/tensorflow/tensorflow/issues/6396

While the build was running, I found the path to the parameter files and edited some lines to fix the problem:
sed -i 's,^/WHOLEARCHIVE:\(.*external.*\),\1,'  /c/users/nwani/_bazel_nwani/2womniv5/execroot/org_tensorflow/bazel-out/x64_windows-opt/bin/tensorflow/python/_pywrap_tensorflow_internal.so-2.params

sed -i 's,^/WHOLEARCHIVE:\(.*external.*\),\1,'  /c/users/nwani/_bazel_nwani/2womniv5/execroot/org_tensorflow/bazel-out/host/bin/tensorflow/python/_pywrap_tensorflow_internal.so-2.params

@tatianashp Could you reroute this issue to Intel?

Adding @agramesh1.
Ramesh - can you help here?

As this issue has invited community support, please remove the assignee. Otherwise, remove the community support label. Thank you.

As this issue has invited community support, please remove the assignee. Otherwise, remove the community support label. Thank you.


Container exits after signing in

After you sign in, the container will quit unexpectedly. Running the container again will work just fine.

Fixed in 1.0.1