[Errno 21] Is a directory: '/sandcastle/sources'

https://sentry.io/organizations/red-hat-0p/issues/1422284472
https://github.com/osbuild/osbuild/pull/199

any chance this is gonna be solved by the fixes which came in recently?

negative, sir, another one just came in: https://sentry.io/organizations/red-hat-0p/issues/1426674268

I can reproduce with vanilla packit:
09:09:40.624 upstream.py       DEBUG  new %autosetup line:
%autosetup -n osbuild-7.41.g6a14ba4
ERROR    Preparing of the upstream to the SRPM build failed: [Errno 21] Is a directory: '/home/tt/g/osbuild/osbuild/sources'
Traceback (most recent call last):
  File "/home/tt/g/user-cont/packit/packit/api.py", line 472, in create_srpm
    self.up.prepare_upstream_for_srpm_creation(upstream_ref=upstream_ref)
  File "/home/tt/g/user-cont/packit/packit/upstream.py", line 656, in prepare_upstream_for_srpm_creation
    self.specfile.download_remote_sources()
  File "/usr/lib/python3.7/site-packages/rebasehelper/specfile.py", line 143, in download_remote_sources
    LookasideCacheHelper.download('fedpkg', os.path.dirname(self.path), self.header.name, self.sources_location)
  File "/usr/lib/python3.7/site-packages/rebasehelper/helpers/lookaside_cache_helper.py", line 125, in download
    for source in cls._read_sources(basepath):
  File "/usr/lib/python3.7/site-packages/rebasehelper/helpers/lookaside_cache_helper.py", line 65, in _read_sources
    with open(path, 'r') as f:
IsADirectoryError: [Errno 21] Is a directory: '/home/tt/g/osbuild/osbuild/sources'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/tt/g/user-cont/packit/packit/cli/utils.py", line 61, in covered_func
    func(config=config, *args, **kwargs)
  File "/home/tt/g/user-cont/packit/packit/cli/srpm.py", line 75, in srpm
    srpm_path = api.create_srpm(output_file=output, upstream_ref=upstream_ref)
  File "/home/tt/g/user-cont/packit/packit/api.py", line 476, in create_srpm
    ) from ex
packit.exceptions.PackitSRPMException: Preparing of the upstream to the SRPM build failed: [Errno 21] Is a directory: '/home/tt/g/osbuild/osbuild/sources'

The problem is that rebase-helper thinks that the osbuild upstream repo is a dist-git repo, because path "sources" exists - but in dist-git it's meant to be a file, while here it's a directory - hence the error message.


Add keyInput for NODES buttons

The issue is when we want to add keycode input right now we need to use ENUM but the enum list can be long. Since we have a specific zui component for keycode input we use this now with a dedicated type for buttons.

Should we close this in favor of the future custom Nodes ?https://github.com/armory3d/zui/issues/134

👍  Sounds good, continuing at https://github.com/armory3d/zui/issues/134.


MAL watched states

For me if I take and sync watched states from MAL using the option provided it doesn't seem to do anything. And also I wonder if it is possible to have generic files an example is.
If you have watched Akame Ga Kill 10/24 episodes and missing like episode 15 it will say that there is 13 unwatched even though there should be 14.

Fixed in 3.6
https://github.com/japanesemediamanager/jmmserver/pull/332#event-734683616


ensure that logging output is sent to Runner.log

Or SurveyManRunner.log -- right now it sounds like it's being routed to SurveyMan.log

@rhuamani : next time you run, can you (1) nuke the logs folder and (2) check to see if the runner is writing to logs/Runner.log?


Improve host errors

Some host errors are not directly actionable. For instance:
         	            	Value: Status(HostStorageError(AccessToUnknownEntry))

        	            	Backtrace (newest first):
        	            	   0: <unknown>
        	            	   1: soroban_env_host::storage::Storage::get
        	            	   2: soroban_env_host::host::data_helper::<impl soroban_env_host::host::Host>::load_account
        	            	   3: <unknown>
        	            	   4: <unknown>
        	            	   5: soroban_env_host::host::Host::invoke_function
        	            	   6: <unknown>
        	            	   7: preflight_host_function
        	            	   8: _cgo_3031edd0e0b1_Cfunc_preflight_host_function
        	            	   9: <unknown>

It indicates an entry is missing but it doesn't tell you which one, so that you can debug the problem further.
I suspect that's the case with many other errors.
It would help if we could add as much detail as possible to the error.

In this case I think we could get the additional details by outputting debug events in the host when looking up a storage item fails.

Here's another (not very helpful) error:
Value: Status(HostAuthError(NotAuthorized))

        	            	Backtrace (newest first):
        	            	   0: <unknown>
        	            	   1: <unknown>
        	            	   2: <unknown>
        	            	   3: <unknown>
        	            	   4: <unknown>
        	            	   5: <unknown>
        	            	   6: <unknown>
        	            	   7: <unknown>
        	            	   8: <unknown>
        	            	   9: <unknown>
        	            	  10: <unknown>
        	            	  11: <unknown>
        	            	  12: <unknown>
        	            	  13: <unknown>
        	            	  14: <unknown>
        	            	  15: <unknown>
        	            	  16: <unknown>
        	            	  17: <unknown>
        	            	  18: <unknown>
        	            	  19: <unknown>
        	            	  20: <unknown>
        	            	  21: <unknown>
        	            	  22: <unknown>
        	            	  23: <unknown>
        	            	  24: <unknown>
        	            	  25: <unknown>
        	            	  26: __libc_start_main
        	            	  27: <unknown>

It would be much more useful if we got a clear message indicating why it wasn't authorized and how to fix it

The plan here is actually to be making the errors a bit more coarse (i.e. less informative) but also to add more debug events, so that you get more information in the debug log while developing or running on an rpc server that's able to report debug events.

I think this was largely done in https://github.com/stellar/rs-soroban-env/pull/806 -- feel free to file followups if you see cases where there isn't quite enough context still.


chore: change rust instructions to nightly

Closes #69

This item belongs to payment request #19A990 on OnlyDust:

from shramee to enitrat
14 items included
$500 for 1 day of work


FBX: Disable importer when canceling FBX2glTF setup

Pretty hacky solution but it's better than an infinite loop.
All this import setup needs to be redone, it's very difficult to properly bail out from an invalid import without triggering reimport loops.
Also fix underline not visible at default editor scale in LinkButton.

Fixes #73319.
Fixes #74368.

Cherry-picked for 4.0.1.


Use relative paths for file = {...} field of .bib export.

Sorry if this is the wrong place, since I report ID is not applicable.
The difference is in the file =  field.
This is related, but not exactly equivalent, to Zotero's
user_pref("extensions.zotero.baseAttachmentPath", "$base_path")
setting.
Exporter used:
Better BibLaTeX auto exporter ("keep updated")
Desired behavior:
@inproceedings{bohlin2000path,
  title = {Path Planning Using Lazy {{PRM}}},
  booktitle = {Proceedings 2000 {{ICRA}}. {{Millennium}} Conference. {{IEEE}} International Conference on Robotics and Automation. {{Symposia}} Proceedings (Cat. {{No}}.{{00CH37065}})},
  author = {Bohlin, R. and Kavraki, L. E.},
  date = {2000-04},
  volume = {1},
  pages = {521-528 vol.1},
  issn = {1050-4729},
  doi = {10.1109/ROBOT.2000.844107},
  file = {thesis/literature/zotero/data/storage/WW8FF8QM/Bohlin and Kavraki - 2000 - Path planning using lazy PRM.pdf},
  organization = {{IEEE}}
}

Actual behavior:
@inproceedings{bohlin2000path,
  title = {Path Planning Using Lazy {{PRM}}},
  booktitle = {Proceedings 2000 {{ICRA}}. {{Millennium}} Conference. {{IEEE}} International Conference on Robotics and Automation. {{Symposia}} Proceedings (Cat. {{No}}.{{00CH37065}})},
  author = {Bohlin, R. and Kavraki, L. E.},
  date = {2000-04},
  volume = {1},
  pages = {521-528 vol.1},
  issn = {1050-4729},
  doi = {10.1109/ROBOT.2000.844107},
  file = {/Users/goretkin/projects/thesis/literature/zotero/data/storage/WW8FF8QM/Bohlin and Kavraki - 2000 - Path planning using lazy PRM.pdf},
  organization = {{IEEE}}
}

BTMK7BEX-euc

You can turn on this at the cost of losing caching.

(the debug ID also gives me a copy of your BBT settings, versions of zotero and plugins, etc, so even if I don't need the log, this gives me a better sense of what context BBT works in)

Oh, I didn't notice that option. Thanks so much!
On the topic of caches, I am experimenting with keeping my zotero data directory versioned under git (using this to get some meaningful-ish diffs for the sqlite databases, though it's still pretty inscrutable). I ignored the files in $datadir/better-bibtex/cache.* because they were changing often, appearing and reappearing, etc. Do you have any opinion on this, or generally, how to maintain version history?

You're not really expected to know about the hidden preferences, it's OK to ask here.
There's one sqlite file that may be worth backing up, that holds the citation keys. There's a bunch of json files, those will disappear on start and reappear on shutdown. Those are just caches, I don't see much value in versioning those. They will be automatically rebuilt when missing or corrupted.

Does this address your questions?

Yes, it does. Thank you!


Feedback - Add details on how to use custom handler and what's supported in context of GitHub actions

If we want to deploy the custom handler Azure Function by GitHub Actions, it's not straight forward. For example, the following document explains how to deploy the Azure Function by GitHub Actions.
https://docs.microsoft.com/en-us/azure/azure-functions/functions-how-to-github-actions?tabs=dotnet
However, the document doesn't have an option for the custom handler.
Also, one of the options to use AZURE_FUNCTIONAPP_PUBLISH_PROFILE didn't work. I couldn't find any documentation that says "The custom handler can't use this deploy option" but I figured it out from the following link.
https://issueexplorer.com/issue/Azure/functions-action/81#19894089
Current behaviour
We don't have an official document about the deployment of custom handler by GitHub Actions. And we can't use AZURE_FUNCTIONAPP_PUBLISH_PROFILE.. This should be documented.

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

ID: 9a0ad9be-7b06-1d81-0339-3776b7fedca2
Version Independent ID: 83637bff-1ba9-3421-1380-41cfaa5138f1
Content: Use GitHub Actions to make code updates in Azure Functions
Content Source: articles/azure-functions/functions-how-to-github-actions.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@amanjeetsingh Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.


Persist buffer in internal field when using zero-copy fromBufferSync

Fixes #580
/cc @springmeyer

Before merging this I'd like to manually test. While I'm sure it is correct the lcons->InstanceTemplate()->SetInternalFieldCount(2); is freaky.

It's not that freaky! :stuck_out_tongue_winking_eye:

Hrm, so failing tests here look like the buffer may not be persisted correctly...

Closing, fixed in #677


uinit: Don't lose debug output in default cmds

The quiet flag is used to decide between running the subcommands with
-d or not, but it doesn't make a difference since the output is
captured and discarded.
In the spirit of uinit being a "script", redirect the output of the
commands to stdout and stderr. This is done both when quiet and when
not, since the -d flag passed to the boot commands is already used to
increase or decrease verbosity

Codecov Report

Merging #75 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #75   +/-   ##
=======================================
  Coverage   47.97%   47.97%
=======================================
  Files          11       11
  Lines         419      419
=======================================
  Hits          201      201
  Misses        193      193
  Partials       25       25

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 e6718e1...9e9149e. Read the comment docs.


Tracing revisited

Split this back out into a fresh pull request, since I don't see how to reopen the old one.
Issues:

supporting needed RPMs / python libraries
providing hook for allowing  tags/attributes, the function decorator can't do that.

Now github says this is the same as master, but I have diffs on my end.. sigh.


button to open from inside

Is it possible to use one more pin for a button which will trigger the relay to open a door from inside?

You can connect button to relay circuit.

ok:)

I add the push button to trigger relay from the inside. Maybe possible to add in to the code?

I think esp-rfid should not endorse additional hardware other than simply basic RFID reader support and a door lock trigger mechanism.  LCDs, Buzzers, Speakers, LEDs, Buttons does not fit for all, some are comfortable with Buzzer, some are implementing LCD.
We should keep it this way.

Hi @teriall, I'm interested in adding a button.
Cloud you please share with the community your changes?
I would really appreciate that!
Thank you, have a nice day! :)

I'll try to do it somehow


Edit Class

User Story
As a teacher, I want to be able to edit my class.
AC
WHEN I hit the edit button on the single day view page.
THEN I should be taken to a form to edit the class
AND When I submit my changes
THEN Those changes should change in the database and be reflected on the page
Dev Notes

Use an "edit" state on the class form component for adding a student that will toggle whether or not the form is in edit mode or create mode.
Check that no other class is in the day/time slot requested
Make API call to update class in the database and pull down the new data to refresh the page.


Fixed typo in docs/topics/db/fixtures.txt.

I think this adheres to the contribution guidelines -- is a single commit for a trivial typo fix. Let me know if I need to change anything.

@ashwindharne Thanks :+1: Welcome aboard :boat:

Hmm interesting... I've never read this part of the docs but most of these headings are not phrased correctly to be questions, rather as statements.
Take "Where Django looks for fixtures?"; It should either have the question mark removed or be "Where does Django look for fixtures?" 😛


Angus johnson/clipper2#171

some more CMake clean up and some helper scripts for windows developers to quickly get up and running.
I have removed the use of the cmake install() command as it was not being used for the intended purpose.
I have started working on getting the install stuff to work properly but I think I will make a new issue/branch for that work as I am not sure when I will be able to give it enough attention to be completed.

Great stuff Nicholas. Thanks!


AppArmor confinement support

Add a hidden feature to allow advanced users to confine apps with AppArmor.
Without this, there is not a simple way for custom stage1 implementors to tweak the generated app systemd units and add the AppArmorProfile= option (systemd.exec(5)). The only alternative I can see, would be to reimplement the /init entrypoint from scratch, which is overkill for a single one-line addition to one of the units.
This is implemented as a rkt specific annotation, that can be set with carefully crafted Pod manifests. Ideally this should be an isolator (and I started it that way), but since the AppContainer spec is frozen, we can't define new isolators. It seemed reasonable to start with a rkt specific annotation in experimental mode.
The annotation only has effect on stage1 images that support it. It will be a noop on all stage1 images currently bundled with rkt (coreos, fly, and kvm flavors). The simplest way to get a stage1 image that supports apparmor is using the host flavor, and installing a systemd build on the host that has +AppArmor support. I also added a ./configure --with-stage1-flavors=src --enable-stage1-systemd-apparmor option to have a systemd built with apparmor support for the src flavor. That helps advanced users building their own custom stage1 images based on the src flavor to have apparmor support.
There is also a global flag (--insecure-options=apparmor) to completely ignore the apparmor annotations when needed. Similarly to what is done for seccomp filters.
Since this is only to enable/unblock advanced users with custom stage1 images to have AppArmor support, I decided to not add any CLI flags to configure it. Functional tests for this is also tricky, since it requires a custom stage1, and I don't have much control on the CI environment to add a new build type.
Later, if other bundled stage1 images have interest in adding AppArmor support, and if we can add a new AppArmor isolator to the appc spec (OCI has it), we could consider making it a more general feature of rkt, but having it as a hidden advanced toggle is OK to unblock us.

the CI failure seems unrelated.

appc/spec#695 added an AppArmor isolator to the spec. If you'd like, I could carry this PR and modify it to use the new isolator.

@cyphar thanks! It is very unlikely I'll be able to carry more changes into this PR though. I'm happy to hand it off to others if there are any volunteers.


Godot project crashes when language is not English

Tested versions
Every version 4.0 >
System information
Windows 11 - Godot 4.22
Issue description
When opening a project from the launcher with a language other than English, the project is likely to crash.
Steps to reproduce
Literally the description of the problem.
After changing the language from Italian to English it stopped crashing.
Minimal reproduction project (MRP)
N/A

Can't reproduce. Does it happen with any project? If it's project-specific please attach a minimal reproduction project that exhibits this issue.

Every version 4.0 >

Is that true though? Many users use non-English editor and this is probably the first report of this crash.

Ok, I've explored this a bit. It's not Godot's problem. It's my problem, it seems that after installing some launcher like GOG, Vulkan gets corrupted and that's what was causing my problem.
I was sure it was a language problem, but after restarting the computer a few times it started happening again.
Sorry for wasting your time, with this comment I close the issue.


Pass command line arguments to node-gyp

I have an electron application which depends on a native module that I am writing.
When I run electron-forge package, it fails at Preparing native dependencies: 0/1:
gyp: Undefined variable BASE_DIR in binding.gyp while trying to load binding.gyp

I usually pass the BASE_DIR as a command line argument to node-gyp (this also works for npm install):
node-gyp rebuild --BASE_DIR=`pwd`
npm install ../addon --BASE_DIR=`pwd`

Is there a way to pass this command line argument with electron-forge? I have tried
electron-forge package --BASE_DIR=`pwd`

but this does not work:
$ electron-forge package --BASE_DIR=`pwd`
√ Checking your system
  error: unknown option `--BASE_DIR'

Hm, can you show me your binding.gyp file for that native module.  This is actually an issue with electron-rebuild but we can discuss it here for now 👍

The binding.gyp is very long, and I think I am not allowed to post it here in its entirety, but the relevant part is in the very beginning:
{
  'variables': {
    "BUILD_TESTS": "false",
    "PYTHON_DIR": "<@(BASE_DIR)/../../opt/Python-3.6.0",
    "BOOST_DIR": "<@(BASE_DIR)/../../opt/boost_1_63_0"
  }
  ...
}

I set the BASE_DIR via command line because locating all our dependencies is not that easy in our current setup.
I am also not sure if it is purely an electron-rebuild issue because electron-forge fails very early when I pass in extra command line arguments...

OK, I think I have a solution (or workaround) for now... I include a variables.gypi file in my bindings.gyp:
{
  'includes': [
    'variables.gypi',
  ],
  'variables': {
    "BUILD_TESTS": "false",
    "PYTHON_DIR": "<@(BASE_DIR)/../../opt/Python-3.6.0",
    "BOOST_DIR": "<@(BASE_DIR)/../../opt/boost_1_63_0"
  },
  ...
}

which is empty by default. Then, before running electron-forge, I can populate it with the correct values, maybe even from environment variables.

I am also not sure if it is purely an electron-rebuild issue because electron-forge fails very early when I pass in extra command line arguments...

That's because -forge doesn't expect random arguments
Even if forge accepted the argument it would need to know to pass it through to -rebuild which would then need to know to pass it through to node-gyp.  This first needs to be solved at the -rebuild level

OK, thanks for helping me investigate. Since this sounds like a hacky solution, I will use my workaround for now.
OTOH, npm install seems to do it right - maybe they just pass all unknown args to all subsequent processes?

I believe they do in the case of node-gyp but I consider that unsafe as it means random args can flow down the spawn tree and cause unexpected behavior.

Closing this out as it should be tracked upstream in electron-rebuild if it isn't already 👍


ERROR HINDERING MY ABILITY TO USE TENSOR FLOW

TF Error.docx
Thank you for submitting a TensorFlow documentation issue. Per our GitHub
policy, we only address code/doc bugs, performance issues, feature requests, and
build/installation issues on GitHub.
The TensorFlow docs are open source! To get involved, read the documentation
contributor guide: https://www.tensorflow.org/community/contribute/docs
URL(s) with the issue:
Please provide a link to the documentation entry, for example:
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/MyMethod
Description of issue (what needs changing):
Clear description
For example, why should someone use this method? How is it useful?
Correct links
Is the link to the source code correct?
Parameters defined
Are all parameters defined and formatted correctly?
Returns defined
Are return values defined?
Raises listed and defined
Are the errors defined? For example,
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/feature_column/categorical_column_with_vocabulary_file#raises
Usage example
Is there a usage example?
See the API guide: https://www.tensorflow.org/community/contribute/docs_ref
on how to write testable usage examples.
Request visuals, if applicable
Are there currently visuals? If not, will it clarify the content?
Submit a pull request?
Are you planning to also submit a pull request to fix the issue? See the docs
contributor guide: https://www.tensorflow.org/community/contribute/docs,
docs API guide: https://www.tensorflow.org/community/contribute/docs_ref and the
docs style guide: https://www.tensorflow.org/community/contribute/docs_style

Having successfully downloaded tensor flow on Anaconda, I continue to encounter an error when I try to import tensorflow on Jupyter. ImportError: DLL load failed: The specified module could not be found. Please advise about how I can go about this challenage.

@fanabis
Please refer to this link and let us know.


Dry run not working

I am using version 4.
I would like to run conventional-changelog-action (as a dry run)  first to get the next version in order to build my installation files. Once I know nothing fails I would like to run conventional-changelog-action in commit mode to really reflect the changes into version.json, and create the tag.
With "dry run" I mean to run conventional-changelog-action to get the next version but not writing anything. For this I use following modifiers:
          git-push: false
          skip-commit: true
          skip-tag: true

But this does not seem to work as the version retrieved is always the fallback one:
Using GIT to determine the new version
The version could not be detected, using fallback version '0.1.0'.
Changelog generated
# 0.1.0 (2024-02-16)

The normal run reads from version.json getting the real version (0.6.0).
How can I achieve a dry run?

Could you share your full workflow + log?
Only setting git-push to false should be sufficient, skip-commit and skip-tag can't be used together as it can't determine any version at that point so it will fallback to the fallback version.
After everything is than ok you won't need to run the action again but could just git push if i'm not mistaken.

Thank you for the quick reply. I was running conventional-changelog-action  two times. But running git push at the end makes more sense. I'll try that. Thanks.


Increase Developer Experience by replacing Create React App with Vite

Motivation

Decrease the start and build time significantly. Start under 500ms, build under 5 seconds.

Changes

To be updated.
Stay tuned. The implementation is completed. I will update the changes to PR description very soon. For now, you can experience the speed by npm run start and npm run build

How to test

Just run:

npm i
npm run start

Screenshots

@nvh95 Thank you again for the time you spend on Best of JS, I appreciate your help.
I'm aware that the code is not very clean, types have been added but not everywhere, so I'm sorry if you had trouble with types, I should have cleaned before.
Some context about Create React App: the reason I chose it is their concept of 0 configuration, no upgrade nightmare with dependencies like Babel, Webpack, ESLint, SVG loading, fast refresh and everything, it just works without having to think about dependencies.
Also I like their TypeScript integration: they always upgrade when it's stable and I like the fact the compiler checks types in development mode after every change. (I know some people prefer to have the type checking only when they build for production).
That being said, I know Vite is a great solution, so I'm open to any solution to improve the "DX", so I'm very curious about your result and I will check your branch.
One more thing: I was working on a new architecture using Next.js framework to enjoy some features like SSR but this task has been delayed, it's still a prototype.

@michaelrambeau
I'm the one who needs to say thank you. I learn a lot from https://bestofjs.org/ for the last year.

I'm aware that the code is not very clean, types have been added but not everywhere, so I'm sorry if you had trouble with types, I should have cleaned before.

Your code is super clean already. I also learned a lot. I can see in some places we can improve the types and I will try to add it to this PR as well.

Some context about Create React App

Yeah. We all used CRA. But it's not in active development anymore (maintenance mode with very slow progress). I adopted Vite as an alternative for CRA about 8 months ago and I'm very happy with that. It's not like zero-configuration but it's not very complicated. And the most important thing is the speed gain when develop and build an app (Vite leverages native ESM support in modern browsers, eliminates the concept of bundling during develop, it also use esbuild underhood which is super fast).

I'm very curious about your result and I will check your branch.

I will update the PR description with the changes I made and some things I need your opinions on. I will also add a benchmark table as well.

I was working on a new architecture using Next.js framework to enjoy some features like SSR

That would fix #114. I also want to do that but it's not an easy task. So I chose to migrate CRA to Vite first to improve the DX first. And leave the SSR task for future development. We can consider Next.js (the most mature option now), or Remix (potential candidate), Vite SSR (which I'm looking forward, version 3 is coming).
I would love to help with the work on Next.js. I haven't had a chance to work with Next.js on production, so it might be a good one for me. Let me know if you need any help.
I used to bootstrap a new app using Vite. But I haven't had a chance to migrate a CRA app to Vite so bestofjs-webui is a great opportunity for me to practice my skills. I learned a lot during the migration. I will mark this PR as Ready to Review and mention you very soon.
Thank you very much.

Thank you for your comments @nvh95 !
About the migration to Vite, you convinced me 🤝 😄 , I will check your branch as soon as I fixed the "Timeline" page, adding what is missing.
About the future...
The current app is limited by the current architecture (SPA approach, we load a huge JSON file and all search queries happen in the browser) I'd like to handle more projects, having a server to query.
Next.js has great SSG features (it would be great to generate a new home page on the server everyday when data is updated, for example).
Remix approach looks very interesting too and maybe more appropriate to Best of JS use case (we have a lot of search requests that can combine query and sort filters, it's impossible to generate statically all the combinations, so the "dynamic" approach of Remix could be great).
We can talk if you are interested, we are on Discord or I can invite you to our Slack team, as you want.
Both spaces are very quiet!

@michaelrambeau I would love to join. Can you send me slack and discord invitation links to my Direct Message on my twitter? Thanks.

we load a huge JSON file and all search queries happen in the browser
I see. I think I need to get involved more to fully understand the current limitation. Some keywords pop up in my head

Algolia: very famous, free for open-source, technical sites
Flexsearch: full text search solution used by Nextra (an example: https://swr.vercel.app/)

@michaelrambeau I updated the PR description. Please take a look

About the testing setup
The testing setup approach with SWC is described by a post from @nvh95 himself:
https://hung.dev/jest-vite

@michaelrambeau It might also be worth checking out in the future is Astro. I haven't used vite much before directly but it's built on top of vite and has a lot of nifty features (like querying data at build time to be used for SSG)


RDS_NOT_PUBLIC object is not subscriptable error

The lambda errors with the following

'NoneType' object is not subscriptable: TypeError
Traceback (most recent call last):
File "/var/task/RDS_NOT_PUBLIC.py", line 315, in lambda_handler
compliance_result = evaluate_compliance(event, configuration_item, valid_rule_parameters)
File "/var/task/RDS_NOT_PUBLIC.py", line 80, in evaluate_compliance
is_publicly_accessible = configuration_item["configuration"]["publiclyAccessible"]
TypeError: 'NoneType' object is not subscriptable

any ideas?

Hey, have you deploy the rule with configuration change on "AWS::RDS::DBInstance"? This error is usually due at a periodic trigger (the rule is not designed for that).

sorry my bad. That was it. Thanks!

Cool!
An advice @mbotmcc : Consider using the RDK to deploy rule: https://github.com/awslabs/aws-config-rdk
The rules in a folder in this repo are compatible.
0 - Have AWS CLI and a default profile set up
1 - Copy the RDS_NOT_PUBLIC folder in your working folder
2 -
pip install rdk
rdk deploy RDS_NOT_PUBLIC

3 - Enjoy


Let owners create custom tags for moderators to apply when they review messages

for rejecting (and accepting)?

Right now we provide some pre-selected tags that moderators can choose to apply to rejected messages ("spam", "insult", etc.) Rather than just providing the defaults, we should allow squad owners to pick tags in their settings they want their moderators to be able to apply, and show these tags in the moderation interface.

and let them give descriptions of these tags for mods: i.e. put things about my work under tag x, put things that are harassing but not threatening under tag y.


Bluetooth sleep time

Is it possible to configure the time for it to go sleep when cable is off? Can't find on examples.
Right now is just a couple minutes, pretty anoying. I was thinking about some hours.

I didn't see such option in any software for keyboard.


It's still possible to write into search field with displayed SmartFAQs

When SmartFAQ input loses focus, it's possible to fill-in search form in the background.
Video: http://recordit.co/NNk1C0Qe0e

Bug or feature? :joy:

Closed by https://gitlab.skypicker.com/frontend/frontend/merge_requests/5171/diffs

I have serious doubts that this can be solved to perfection without a major revamp of how forms work in FE. I'd say this potentially has a higher cost to implement than the benefit we get from implementing this, and it's definitely not our lowest hanging fruit in keyboard usability.

I believe it can be solved quite fairly, it just didn't have the highest priority now :)

Hmm.. I think, that I could not test the issue..

@MagdalenaAlmanova yes, you’re right, it’s not possible to test in the standalone app

:tada: This issue has been resolved in version 0.1.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[CT-2964] Backport #7630 to 1.3 and 1.4

The security fix made for #7630 in 1.5 and 1.6 to exclude some profile fields from Jinja rendering now needs to be backported to 1.3 and 1.4.

We have decided not to include unit tests for the 1.3 and 1.4 backports of this fix. The reason is that the automated tests written for 1.5+ do not work with the older logging infrastructure in 1.3 and 1.4, combined with the fact that regressions of this functionality are unlikely in 1.3 and 1.4. The regressions are unlikely both because these versions are rarely modified and also because the code in question is relatively self contained and itself in a rarely modified module.
I did verify the fixes both in the 1.3 and 1.4 patch branches by manually testing that the unwanted error messages appeared before the fix, but not after.


Not able to make it work on Apache Win32 and Win64

When I visit "localhost/sailor" it just prints out what is inside the index.lua file. I added everything mentioned here in the apache conf file.
My conf file
My error log
I can't seem to figure out where it's going wrong. I have Lua 5.1 installed.
I was getting 500 Internal Server error in the beginning but after some refactoring of the conf file it changed to this

Hello! I noticed from the log file it mentions "Apache/2.4.18 (Win64)". In this case you will need the Lua C libraries compiled for Win64 - the libraries included with the Lua For Windows installer, for example, are 32-bit.

You should consider a parallel 32-bit installation of Apache/2.4.18 - just make sure the LuaPackageCPath points to the right path where Lua for Windows is installed and it should work as expected

Okay. I downloaded the 32 bit version of Apache. I still get an error.
Here is my error log

Could Lua have been installed to C:/Program Files (x86)/Lua instead of C:/Program Files/Lua/ as indicated in LuaPackage* directives?

I was about to edit my previous comment with the updated error log. I noticed my mistake and changed it. This is the new log

By any chance has it got something to do with backward() slashes used in windows file paths instead of forward(/)  ?

It looks like you need some missing dependencies in the 'C:/Program Files (x86)/Lua/5.1/lua/' directory, this one: https://github.com/sailorproject/remy/tree/master/src

Yes. Noticed that in the logs. I was missing latclient also. Now, after installing both, when I point my browser at the Sailor sample app it just says "webpage not available" and "connection reset". Any pointers on that ?

Can you post the new log? These dependencies were included with this repo in the past. Now they are installed through luarocks, this is why they are missing - an updated version of the doc will mention this.

Logs

Do you have https://github.com/sailorproject/lua_at_client/tree/master/lua?

Yes. I installed it using luarocks. This one right ?

Yes, 0.4-1 is the latest. Please feel free to update the docs. Your contribution is very welcome. I'm not sure what is wrong now - the log is not indicating the issue... I think.

I think I will just downgrade my Lua installation in my Linux system and poke around Sailor using that

Can you post a more clean version of the log without the previous errors that were fixed?

This is the final log. I am not able to find anything relevant from them

I will try to reproduce this later today. Are you using Windows 10 64-bit?

No. Windows 7. 64 bit

I am going to try a clean install of lua/luarocks now. Hopefully that solves the problem

I am going to try a clean install of lua/luarocks now. Hopefully that solves the problem

@rnikhil275 I didn't check your error logs (too lazy, sorry :P) but I did checked your conf file.
Under <Directory "c:/Apache24/htdocs">, you should change AllowOverride None to AllowOverride All. That is definitely an issue there (probably not the only one). You should restart apache after that.

I am sorry I don't have access to a windows machine now. It's at my home and I am staying at college hostel since my exams are going on. I cannot provide any updates on the issue for another 5 days. :worried:


fix: remove node selector

Description
fix: remove node selector
Related issues

Close #1635
Close #1610 1610

Checklist

[x] I've read the guidelines for contributing to this repository.
[x] I've added tests that prove my fix is effective or that my feature works.
[x] I've updated the documentation with the relevant information (if needed).

Thank, @chen-keinan . Let me check

@truongnht Note: its not yet released , it will be out with v0.17.0 unless you want to build it manually from source code

hey @chen-keinan , when do you guys plan to cut v0.17.0 release?

hey @chen-keinan , when do you guys plan to cut v0.17.0 release?

by end of next week

xref: https://github.com/aquasecurity/trivy-kubernetes/commit/c7e2f804481a7abfc6a5bcedcd7be0e94942a65c

Issue is not fixed in v0.18.0. Node collector still has node selectors that tries to schedule on the control plane that taints forbid this.
NAME                                  COMPLETIONS   DURATION   AGE   APP
job.batch/node-collector-65f99f5bd6   0/1           99s        99s

NAME                                  READY   STATUS    RESTARTS   AGE     APP
pod/node-collector-65f99f5bd6-x7ksn   0/1     Pending   0          2m15s   node-collector

Node-Selectors:              kubernetes.io/hostname=k8ststmaster-1
Tolerations:                 node.kubernetes.io/not-ready:NoExecute op=Exists for 300s
                             node.kubernetes.io/unreachable:NoExecute op=Exists for 300s
Events:
  Type     Reason            Age    From               Message
  ----     ------            ----   ----               -------
  Warning  FailedScheduling  2m48s  default-scheduler  0/6 nodes are available: 3 node(s) didn't match Pod's node affinity/selector, 3 node(s) had untolerated taint {node-role.kubernetes.io/control-plane: }. preemption: 0/6 nodes are available: 6 Preemption is not helpful for scheduling..

Workaround : disable infraAssessment scanner for trivy-operator.
OPERATOR_INFRA_ASSESSMENT_SCANNER_ENABLED="false"

I face same issue with pod node-collector is pending and version is v0.18.1

@chen-keinan yes, we have some nodes using taints/tolerations, some nodes not using taints/tolerations

@chen-keinan nodes are working (not in shutting status) and I deploy latest version of trivy. Yes, we have some nodes are using taints/tolerations, some nodes are not using taints/tolerations

are you using toleration setting in operator

@chen-keinan thanks, I have just added toleration as suggestion however it does not work. Even without toleration it should locate to nodes without taints.

@chen-keinan thanks, I have just added toleration as suggestion however it does not work. Even without toleration it should locate to nodes without taints.

I see , then my suggestion to you is to remove toleration and use the excludeNodes param to ignore nodes by labels , so you could ignore these specific node which keep pod in pending state.

@chen-keinan I see this property exclude nodes from scanning by label however I expect these nodes are still scanning.
# -- excludeNodes comma-separated node labels that the node-collector job should exclude from scanning (example kubernetes.io/arch=arm64,team=dev)
  excludeNodes:

@chen-keinan I see this property exclude nodes from scanning by label however I expect these nodes are still scanning.
# -- excludeNodes comma-separated node labels that the node-collector job should exclude from scanning (example kubernetes.io/arch=arm64,team=dev)
  excludeNodes:

the reason the pod hang on Pending state it because it is not schedule to a Node from some reason

the reason the pod hang on Pending state it because it is not schedule to a Node from some reason

The job tries to schedule pods on control_plane servers, which have NoSchedule taint. This is the issue to fix.

if we want Node collector pods to be scheduled on control_plane, so we need a toleration in the podSpec to tolerate the NoSchedule taint
if we don't want, so we must not generate pods with "Node-Selectors" that lead to schedule on control_plane.

@chen-keinan I see this property exclude nodes from scanning by label however I expect these nodes are still scanning.
# -- excludeNodes comma-separated node labels that the node-collector job should exclude from scanning (example kubernetes.io/arch=arm64,team=dev)
  excludeNodes:

the reason the pod hang on Pending state it because it is not schedule to a Node from some reason

@chen-keinan we have some nodes that could any pod (no need config toleration) locate. Could you reproduce this case?

It is on prem or cloud ? I can give it a try but I do not recall such issue on my env.

@chen-keinan yes, it is on cloud. We are using eks (aws).

It is on prem or cloud ? I can give it a try but I do not recall such issue on my env.

On cloud too, with "vanila" kubeadm installations - almost similar to on-premise clusters.
Control_plane  servers are tainted NoSchedule, and they host also the etcd cluster.
It's no fancy, by the way.

It is on prem or cloud ? I can give it a try but I do not recall such issue on my env.

On cloud too, with "vanila" kubeadm installations - almost similar to on-premise clusters. Control_plane servers are tainted NoSchedule, and they host also the etcd cluster. It's no fancy, by the way.

@ledroide thanks for the input @chary1112004 do you want to try it ?

It is on prem or cloud ? I can give it a try but I do not recall such issue on my env.

On cloud too, with "vanila" kubeadm installations - almost similar to on-premise clusters. Control_plane servers are tainted NoSchedule, and they host also the etcd cluster. It's no fancy, by the way.

@ledroide thanks for the input @chary1112004 do you want to try it ?

@chen-keinan from our side, we no need manage kubeadm as it is managed by aws. So it is not in our case. I understand that @ledroide describe about the environment that is facing the issue, not solution.

It is on prem or cloud ? I can give it a try but I do not recall such issue on my env.

On cloud too, with "vanila" kubeadm installations - almost similar to on-premise clusters. Control_plane servers are tainted NoSchedule, and they host also the etcd cluster. It's no fancy, by the way.

@ledroide thanks for the input @chary1112004 do you want to try it ?

@chen-keinan from our side, we no manage kubeadm as it is managed by aws. So it is not in our case. I understand that @ledroide describe about the environment that is facing the issue, not solution.

@chary1112004 I think he suggested to add toleration in nodeSpec for NoSchedule taint but if it doesn't work for you then need to understand why node-collector pods are pending for this specific Nodes

@chen-keinan After checking then it relates to node selector even I added all tolerations needed
...
nodeSelector:
kubernetes.io/hostname: ip-192-168-120-211.eu-central-1.compute.internal
...
message: '0/x nodes are available: x node(s) didn''t match Pod''s node affinity/selector.
Could we disable node selector for pod node-collector?

@chary1112004 if you disable this flag , node-collector will not be triggered

@chary1112004 if you disable this flag , node-collector will not be triggered

Yes, I saw it in some threads to disable this one, actually could you please give summarize what purpose of node-collector then we could disable it for now. Thanks!

@chen-keinan After checking then it relates to node selector even I added all tolerations needed Here is yaml pod node-collector:
...
nodeSelector:
    kubernetes.io/hostname: <ip>.<region>.compute.internal
...

message: '0/x nodes are available: x node(s) didn''t match Pod''s node affinity/selector.
Could we disable node selector for pod node-collector?

So it is useful information for enable infra assessment.
Base on message of pod then I have checked again node affinity & node selector that is configured in pod node-collector.

affinity: no config
node selector:

nodeSelector:
    kubernetes.io/hostname: <ip>.<region>.compute.internal

I have taken a look in default value file & all helm chart however there is no config for nodeSelector in value file then it may be generated from somewhere else. Then could we have option to disable nodeSelector in pod node-collector?
Thanks!

@chen-keinan After checking then it relates to node selector even I added all tolerations needed Here is yaml pod node-collector:
...
nodeSelector:
    kubernetes.io/hostname: <ip>.<region>.compute.internal
...

message: '0/x nodes are available: x node(s) didn''t match Pod''s node affinity/selector.
Could we disable node selector for pod node-collector?

So it is useful information for enable infra assessment. Base on message of pod then I have checked again node affinity & node selector that is configured in pod node-collector.

affinity: no config
node selector:

nodeSelector:
    kubernetes.io/hostname: <ip>.<region>.compute.internal

I have taken a look in default value file & all helm chart however there is config empty setting for nodeSelector (nodeSelector: {}) in value file then it may be generated from somewhere else. Then could we have option to disable nodeSelector in pod node-collector?
Thanks!

it possible could be very easy change unless you want it to scan every node

@chen-keinan I understand this merge request is merged and node selector should be removed already then we do not need disable infra assessment or does it need any additional config that I need to update? Thanks!

@chen-keinan I understand this merge request is merged and node selector should be removed already then we do not need disable infra assessment or does it need any additional config that I need to update? Thanks!

OK , do you mind opening issue for it ?

@chen-keinan I understand this merge request is merged and node selector should be removed already then we do not need disable infra assessment or does it need any additional config that I need to update? Thanks!

OK , do you mind opening issue for it ?

Yes, I will open issue for this.

Does it mean hopefully that v0.18.2 will fix the issue so I can set back OPERATOR_INFRA_ASSESSMENT_SCANNER_ENABLED="true" ?

Does it mean hopefully that v0.18.2 will fix the issue so I can set back OPERATOR_INFRA_ASSESSMENT_SCANNER_ENABLED="true" ?

I'll be adding a config flag to choose if to use nodeSelector or not

@chen-keinan I understand this merge request is merged and node selector should be removed already then we do not need disable infra assessment or does it need any additional config that I need to update? Thanks!

OK , do you mind opening issue for it ?

I create the ticket https://github.com/aquasecurity/trivy-operator/issues/1779
Thanks!


Omit empty and optional values in form data in Try It

Draft PR for : https://github.com/stoplightio/elements/issues/1829

@GeliXav Hello! Just an FYI that we know your PR exists and we've not forgotten about it. We've been working on new features for Stoplight Studio and have been focused on that. We will make sure someone reviews your PR though!

Hey @Nezteb, any news on when this PR will be reviewed ? Thanks a lot !

@GeliXav Thank you for your patience on this and the ping.
I tested this out to get a demo video for my team:
https://user-images.githubusercontent.com/3588798/151081400-cfb36e2e-174e-4645-8752-6caa0f8be1e6.mov
Overall, I like the approach! I'm glad to see that you account for required fields. 😄
My only concern is the UI/UX of the checkboxes. I'm going to make sure our product manager and designer take a look and provide you with feedback. (@mnaumanali94 @noor-ahmad)
Thank you for the contribution! I will make sure we don't lose this one.

Okay @GeliXav, I got feedback!
In order to make it clear which fields the checkboxes are referring to, our designer suggested indenting them (i.e. adding some padding so that each checkbox is moved to the right a bit). Also changing the language from "Omit the field value" to just "Omit <actual-field-name>". These two things will make it extra clear for users!
Our designer also came up with a future design idea to clarify the UX even more:

But you've already done a lot of good work and we'll happily merge the PR without the eye icons. 😄

Hey @Nezteb, I applied your suggestions outside of the eye icons :) !

Awesome @GeliXav, we love what you've done! I fixed that typo @jvyang mentioned too, so we're all good! I will merge!

Hey @Nezteb, thanks for spending time and effort on this. Any idea on when this will get merged ? I see a test not passing, but I don't think it's related to my changes.

@GeliXav Thank you for your patience! I reran the failed tests and they passed, but now there are merge conflicts. Another Stoplighter moved some CSS around in https://github.com/stoplightio/elements/commit/9f9ae13782edde922f4299a3db817a545095aa55, so I'll try to fix that for you!

@Nezteb Thanks a lot, I did the merge by myself, I think this time we should be good for a merge ! :D


heatplot.ado does not verify whether colrspace is installed

According to the README:

The palettes package and, in Stata 14.2 or newer, the colrspace package are required. To install these packages, type
. ssc install palettes, replace
. ssc install colrspace, replace

The ado-file verifies that palettes is installed and provides a helpful error message if it is not. But it does not check whether colrspace is installed, which left me briefly confused as a new user:
capt which colorpalette
if _rc {
    di as err "the palettes package needs to be installed; type {stata ssc install palettes, replace}"
    exit 499
}

When colrspace was not installed, I received the following uninformative error message:
colrspace_library_palettes.sthlp not found
file not found
r(601);

Hi, thanks for reporting this. I now updated heatplot on github so that it will throw a better error message in this case. The change will also eventually make it into the version at the SSC archive.
Your comment actually pointed my to another problem. Installing palettes from SSC also installs file lcolrspace.mlib because in earlier days this was all that was needed from colrspace to make palettes (to be precise, the colorpalette command) work. However, at some point I revised colrspace and now additional files need to be installed (e.g. colrspace_library_palettes.sthlp). The issue now is that colorpalette looks for lcolrspace.mlib and throws an informative error message when not found; if found, however, colorpalette assumes that colrspace is installed and continues without error message, but then, later on, will bump into the uninformative error you reported.
In sent a note to the maintainer of the SSC archive to remove lcolrspace.mlib  from the installation files of the palettes package. This will help preventing confusions similar to the one you reported.
ben

@benjann Thanks for making such a careful fix and such useful packages!


No app config

Hi there. As documented, I do next:
ini:
[celery]
BROKER_URL = redis://<IP_ADDRESS>:6379/0
CELERY_IMPORTS = my.module.tasks

[app:main]
egg = another.module
pyramid.includes =
    ...
    pyramid_celery
...

another.module:
<...>
def main(global_config, **settings):
    <...>
    config.configure_celery(global_config['__file__'])

my.module.tasks
from pyramid_celery import celery_app as app

print app.conf['PYRAMID_REGISTRY']

@app.task()
def my_task():
    <...>

Then I run worker or I run application with pserve by .ini file, and application fails with
KeyError: 'PYRAMID_REGISTRY'.
Question is: what am I doing wrong? How do I access pyramid config in my tasks?

@dbajal The app.conf wont be registered at import time, it will be registered after your application is started in the main of your pyramid app or within the task itself.   If you move your print to application init time or within the task it'll work.


Fix Issue #39 - Add Client Provisioning

Issue #39 asks to include a name in the updateclientpolicy() function, but is looking for a function to implement clients. Created provisionclient() to do just that.

Can't seem to find provisionclient() in the API, was this added?

Hi @z3roCoo1 please search for provision in the official API docs site to find what you need: https://developer.cisco.com/meraki/api-latest/


validator - add clear sky power model

@wholmgren this function will take in system and model metadata and a DatetimeIndex, and give back clearsky power.  It's a wrapper around pvmodel.calculate_solar_position, pvmodel.calculate_clearsky and pvmodel.irradiance_to_power.  I think a separate function is a good idea, because the calculation is likely to be needed in several places.
Would you like the function in pvmodel?  That would be my choice.

If you're going to wrap all of those functions I think you'd want to take in a datamodel.SolarPowerPlant and DatetimeIndex.
I am ok with this as a convenience function in principal. I worry about unnecessary recalculations of solar position in practice. I don't have a good sense of the whole workflow yet, though, so maybe I am off base here. Two potential wrappers that might make it more clear what I'm thinking about...
def validator_wrapper_efficient(power_plant):
    solar_position = pvmodel.calculate_solar_position(*power_plant_args)
    validator.test1(solar_position, etc)
    validator.test2(solar_position, etc)
    validator.test3(solar_position, etc)

def validator_wrapper_inefficient():
    validator.test1(power_plant, etc)  # calls calc solar position
    validator.test2(power_plant, etc)  # calls calc solar position
    validator.test3(power_plant, etc)  # calls calc solar position

I'll hold off on this function for now. I'm aiming at #22 next, which can be written to expect Series of clear sky power instead of calculating that internally.

Sounds good to me.

I think we can close this issue.  Functions are in pvmodel.py to calculate clearsky irradiance and to translate irradiance to power.

agree.


Split 'fileprivate' and 'private', but give them the same behavior for now

fileprivate is considered a broader level of access than private, but for now both of them are still available to the entire file. This is intended as a migration aid.
One interesting fallout of the "access scope" model described in 758cf64 is that something declared private at file scope is actually treated as fileprivate for diagnostic purposes. This is something
we can fix later, once the full model is in place. (It's not really wrong in that they have identical behavior, but diagnostics still shouldn't refer to a type explicitly declared private as fileprivate.)
As a note, ValueDecl::getEffectiveAccess will always return FilePrivate rather than Private; for purposes of optimization and code generation, we should never try to distinguish these two cases.
This should have essentially no effect on code that's not using fileprivate other than altered diagnostics, but if we need to back out I've put the interesting changes behind a new LangOptions flag, EnableSwift3Private.
Progress on SE-0025

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

A smoke test on macOS does the following:

Builds the compiler incrementally.
Builds the standard library only for macOS. Simulator standard libraries and
device standard libraries are not built.
lldb is not built.
The test and validation-test targets are run only for macOS. The optimized
version of these tests are not run.

A smoke test on Linux does the following:

Builds the compiler incrementally.
Builds the standard library incrementally.
lldb is built incrementally.
The swift test and validation-test targets are run. The optimized version of these
tests are not run.
lldb is tested.

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

@CodaFi More access control stuff to review.
@swift-ci Please test

@swift-ci Please test

LGTM.  Thanks.


fix(monitor): modify monitor serviceType in StartService()

这个 PR 实现什么功能/修复什么问题:
1.修改monitor服务service 代码中注册的globalServiceType:
alerter ==> monitor

是否需要 backport 到之前的 release 分支:

release/3.3
release/3.4
release/3.5
release/3.6

/cc @zexi  @swordqiu
/area monitor

Codecov Report

Merging #8977 (31e6e8e) into master (4a8eafb) will increase coverage by 4.75%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #8977       +/-   ##
===========================================
+ Coverage    5.88%   10.63%    +4.75%
===========================================
  Files         825      172      -653
  Lines      128808    18095   -110713
===========================================
- Hits         7575     1925     -5650
+ Misses     120554    16004   -104550
+ Partials      679      166      -513

Flag
Coverage Δ

aFlag
10.63% <ø> (+4.75%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/compute/models/dns_zonevpc.go

pkg/compute/models/guests.go

pkg/monitor/models/suggestsysalert.go

.../compute/models/loadbalanceropenstackcachedlbbg.go

pkg/compute/regiondrivers/zstack.go

pkg/compute/models/buckets.go

pkg/compute/regiondrivers/apsara.go

pkg/compute/models/dnsrecords.go

pkg/util/ansible/playbook.go

pkg/compute/models/billingresource.go

... and 643 more

Continue to review full report at Codecov.

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


Mime type for uploaded .m4a AAC audio files

I have uploaded several AAC encoded audio files with .m4a extension to Google Drive using google-drive-ocamlfuse on Ubuntu 19.04 LTS. I encountered a problem when playing these files in Safari on a Mac or Safari and Chrome on iOS - they refuse to play in the browser. I discovered the following issue:
Problem:
Files uploaded with a .m4a extension get audio/mpeg as mime type, which lets Safari assume it is a MP3 file.
Files uploaded via the Google Drive web interface get audio/x-aac as mime type, which does not show the described issue.
Suggested Solution:
Add a file extension match rule to src/mime.ml stating
| "m4a" -> "audio/x-aac"

Update:
Maybe it is also audio/x-m4a, I don't really know what is the best solution in this case.
I did not yet consider the case, if the .m4a file is ALAC encoded - maybe someone has a better solution for this problem.

It looks like the standard mime type is audio/mp4 (https://www.iana.org/assignments/media-types/media-types.xhtml#audio). I'm publishing v0.7.7 (on the beta ppa) than includes this patch. Let me know if it works for you. Thanks!

Small remark, I am asking myself (I was not going into the depths of your code...), why do you have to manually map the mime type? I think the Drive API should somehow provide auto detection...
Short search lead me to another project on Github:
https://github.com/gsuitedevs/drive-zipextractor/commit/01df4a5812e8f5e1532ee66b48f280a810bb5b82

You are right, in version 0.7.8 I've added a config option (autodetect_mime) that enables Google Drive autodetecting feature. The default is true so you don't have to turn it on. But if it doesn't work, you can turn it off.


fix: sso add cmd

What kind of change does this PR introduce?

The value of the flag --type saml is not being passed down to the underlying API call
Fix a bug with the metadata url scheme validation
Fix staticcheck errors

Can you please leave the capitalization in the errors as they are printed to the user?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 18 (16.67%)  changed or added relevant lines in 5 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.004%) to 61.83%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/sso/internal/render/render.go
0
1
0.0%

internal/sso/internal/saml/files.go
2
3
66.67%

internal/sso/create/create.go
0
3
0.0%

internal/sso/info/info.go
0
3
0.0%

cmd/sso.go
1
8
12.5%

Files with Coverage Reduction
New Missed Lines
%

internal/sso/info/info.go
1
0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.004%

Covered Lines:
4129

Relevant Lines:
6678

💛  - Coveralls


Spanish Language not working.

When using the locale to have a multilanguage RangePicker spanish is not working.
The placeholder does work, but the month and the week names doesnt change

Working,
English

Port

wrong place


visualizations for GC

What StrongLoop had: Heap still in use after a minor or a major gc https://docs.strongloop.com/display/SLC/Available+metrics
What Trace has: GC run counts for scavenge and mark-sweep AND time spent for GC https://trace-docs.risingstack.com/docs/metrics
What Appdynamics has: Number of full gc and incremenatal GC per min
https://docs.appdynamics.com/display/PRO42/Node.js+Metrics
Its not clear in the above whether different vendors are using different names for the same thing, or whether appmetrics could access and display more information. Research needed.

We are tracking the work for this on RTC @sam

Closing as discussion belongs in another project.

Not tracked on RTC, can you reopen, @sjanuary ? Or add me as a member to the repo so I can :-)


Unify toLower implementations

Description
I found 3 instances where different files needed to transform a string into all lower case letters. Two implementations were identical and a 3rd used a standard algorithm. 3 reimplementations of the same thing seemed like justification to create a new private abstraction to handle this. Following patterns I saw elsewhere, I added a priv namespace to String.hpp within which sf::priv::toLower is implemented.
Tasks

[ ] Tested on Linux
[ ] Tested on Windows
[x] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

This latest revision moves this utility out of String.hpp and into System/Priv/Utils.hpp. This is good because it means we don't need to bring in the entire String.hpp header when these 3 files just need this one utility. I think we can do better though. Because we ideally want these private implementation details to be unobservable to users, I think the ideal solution is to move this file into src/. I'm still considering how exactly to implement that in a way that is more granular than simply adding
include_directories(.)

to src/CMakeLists.txt. This works, but it means that the System module could, for example, depend on files from Network without having to express a dependency on that module which we probably don't want to allow. We can't add any new public include paths inside src/ since those will be visible to users so we might be forced to do something like that include_directories(.) line to allow sharing files between .cpp files without them being visible to users.

https://github.com/ChrisThrasher/SFML/commit/c8e963243c2e3feb9f12dd63c300c671428530d6
Here's one way we could potentially add private headers to src/ to prevent having to add more implementation details to public headers.

@ChrisThrasher It is already possible to put implementation headers in src/ without new CMake commands.
See https://github.com/SFML/SFML/blob/master/cmake/Macros.cmake#L212-L214

@ChrisThrasher It is already possible to put implementation headers in src/ without new CMake commands.
See https://github.com/SFML/SFML/blob/master/cmake/Macros.cmake#L212-L214

Oh, lovely. In that case I feel ever better about moving this function to src/SFML/System/Utils.hpp (or maybe ToLower.hpp). I merged that commit into this PR. I'll squash commits as requested once we all agree on a solution. I'll keep both commits for now in case we choose to remove the latest commit I added.

SFMLBuildMaster: Build this please.

SFMLBuildMaster: Build this please.

@vittorioromeo I found two more files with custom toLower implementations and was able to remove <algorithm> in two more places!

  D:\a\SFML\SFML\src\SFML\Audio\SoundFileWriterFlac.cpp(101): error C2039: 'min': is not a member of 'std' [D:\a\SFML\SFML\build\src\SFML\Audio\sfml-audio.vcxproj]
793
  D:\a\SFML\SFML\src\SFML\Audio\SoundFileWriterFlac.cpp(101): error C3861: 'min': identifier not found [D:\a\SFML\SFML\build\src\SFML\Audio\sfml-audio.vcxproj]
794
  D:\a\SFML\SFML\src\SFML\Audio\SoundFileWriterOgg.cpp(145): error C2039: 'min': is not a member of 'std' [D:\a\SFML\SFML\build\src\SFML\Audio\sfml-audio.vcxproj]
795
  D:\a\SFML\SFML\src\SFML\Audio\SoundFileWriterOgg.cpp(145): error C3861: 'min': identifier not found [D:\a\SFML\SFML\build\src\SFML\Audio\sfml-audio.vcxproj]
796
  D:\a\SFML\SFML\src\SFML\Audio\SoundFileWriterOgg.cpp(150): error C2039: 'min': is not a member of 'std' [D:\a\SFML\SFML\build\src\SFML\Audio\sfml-audio.vcxproj]
797
  D:\a\SFML\SFML\src\SFML\Audio\SoundFileWriterOgg.cpp(150): error C3861: 'min': identifier not found [D:\a\SFML\SFML\build\src\SFML\Audio\sfml-audio.vcxproj]

I may have been too quick to remove <algorithm>.
@vittorioromeo Don't we have a non-standard min/max implementation that we're using to avoid more inclusions of <algorithm>? I think we need to add that to src\SFML\Audio\SoundFileWriterOgg.cpp and src\SFML\Audio\SoundFileWriterFlac.cpp.

SFMLBuildMaster: Build this please.

I followed through on @vittorioromeo's recommendation but the PR still shows as being blocked by an unaddressed request. Not sure what to make of that but it appears that everything else is good to go.

the PR still shows as being blocked by an unaddressed request

It's shown as such, until the reviewer approves it 😉


[BUG] ManagedIdentityCredential misleading error

Describe the bug
When the IMDS endpoint returns a 400 response, ManagedIdentityClient wraps it with a misleading error:

ManagedIdentityCredential authentication unavailable, the requested identity has not been assigned to this resource.

And even worse, it sets the MsiType to Unavailable so subsequent requests give out this error (due to retries this is what I actually see, without the error content):

ManagedIdentityCredential authentication unavailable, no managed identity endpoint found.

While in reality, the error in my case was:

{"error":"invalid_resource","error_description":"AADSTS500011: The resource principal named ... was not found in the tenant named ...

Expected behavior
Always show the actual error. I would also change the wording in the wrapped error (400 does not necessarily mean "has not been assigned").
To Reproduce
Use DefaultAzureCredential in an environment with a MI and pass it an invalid scope.
Environment:

Name and version of the Library package used: Azure.Identity/1.1.1
Hosting platform or OS and .NET runtime version: .NET Core 3.1.5 on Linux
IDE and version: Visual Studio 16.6.3

Moving to August milestone

Could this be prioritized? It's a very serious bug. We've stopped using this class because of it.

Pinging again... Please prioritize this.

Another related issue - if ImdsManagedIdentitySource.TryCreateAsync fails, it is never retried, since the asyncLock field contains null and we get the MsiUnavailableError. May I suggest using IMDS as the fallback without even trying to connect to it?
https://github.com/Azure/azure-sdk-for-net/blob/4cc1a6811808cfe180294545b74aecd6b1e036b8/sdk/identity/Azure.Identity/src/ManagedIdentityClient.cs#L63
We're also seeing this failure in Az PowerShell:
> Connect-AzAccount -Identity
WARNING: Unable to acquire token for tenant 'organizations' with error 'ManagedIdentityCredential authentication unavailable. No Managed Identity endpoint found.'

This is a transient issue that a retry would've also solved. That's why I'm suggesting to get rid of the code that tests the TCP connection, and just try the HTTP call.

any update on it?

I believe this is resolved in the latest version.

Thanks


Release to Maven central

The project is currently not consumable as a dependency. Alpha release is better than no release. :)

Hi Antti,
I looked a bit into it and to get it into maven I need to create a signed
build artifact and have that work with Travis that I'm currently using for
build the project. I have not had any free time neither at work or a home
lately. But the coming weekend I should have some time.
Regards Troels
On Sep 29, 2015 13:29, "Antti Laisi"<EMAIL_ADDRESS>wrote:

The project is currently not consumable as a dependency. Alpha release is
better than no release. :)
—
Reply to this email directly or view it on GitHub
https://github.com/tlbdk/copybook4java/issues/1.

Hi,
thanks for the reply.
You most likely already found this but just in case: http://central.sonatype.org/pages/ossrh-guide.html . Regarding the maven configuration, the simplest way is to use org.sonatype.oss:oss-parent (even though it is deprecated) as parent pom and then you only need to set up the pgp keys and run mvn release:prepare/perform.

It has been released now:
http://search.maven.org/#search|ga|1|copybook4java


How to calculate total score？

Hi, after i get all dimension score. How calculate the total score？

We have updated the offline calculation script of the t2v model at https://github.com/Vchitect/VBench/blob/master/README.md#get-final-score-and-submit-to-leaderboard. You may try it locally and before your submit.


How can I use ntp to get time (CON-566)

I'm using esp32-h2, it's a matter-over-thread device, how can I get UTC-time in this non-wifi device?T T

To make a Thread device support ntp/sntp, the Thread BR should support NAT64 feature. We support this nat64 feature on our Thread BR. We can now visit the IPv4 HTTP servers with this feature but the APIs of sntp are still not supported for thread device.

Here are a example to use sntp on thread device.
Firstly, you should register a event handler for OPENTHREAD_EVENT_SET_DNS_SERVER. This event will be post after the thread device gets its NAT64 prefix.
And in the event handler you should set the sntp server by sntp_setserver(). The sntp server address should be an IPv6 address with a NAT64 prefix which can be obtained by esp_openthread_get_nat64_prefix(). Then call the sntp_init().
static void openthread_set_dns_server_handler(void* arg, esp_event_base_t event_base, int32_t event_id, void* event_data)
{
    ip_addr_t sntp_server;
    time_t now = 0;
    struct tm timeinfo = { 0 };
    char strftime_buf[64];

    sntp_server.type = IPADDR_TYPE_V6;
    esp_openthread_get_nat64_prefix(&sntp_server.u_addr.ip6);
    sntp_server.u_addr.ip6.zone = IP6_NO_ZONE;
    ip4_addr_t ip4_addr;
    IP4_ADDR(&ip4_addr, 209, 58, 185, 100); // IPv4 address for pool.ntp.org. It can be other IPv4 address for other sntp server.
    sntp_server.u_addr.ip6.addr[3] = ip4_addr.addr;
    sntp_setserver(0, &sntp_server);
    sntp_init();

    setenv("TZ", "CST-8", 1);
    tzset();
    while (1) {
        vTaskDelay(5000/portTICK_PERIOD_MS);
        time(&now);
        localtime_r(&now, &timeinfo);
        strftime(strftime_buf, sizeof(strftime_buf), "%c", &timeinfo);
        ESP_LOGI(TAG, "The current date/time in Shanghai is: %s", strftime_buf);
    }
}
......
esp_event_handler_register(OPENTHREAD_EVENT, OPENTHREAD_EVENT_SET_DNS_SERVER, openthread_set_dns_server_handler, NULL);
......

But there are some issues on APIs like sntp_setservername(). So the sntp APIs in esp_netif will also fail as esp_netif APIs will use sntp_setservername() . Maybe we need a hook to map the IPv4 address to IPv6 address for this function after the DNS resolving. But now we don't have any plans to fix it.

Here is an example to use sntp on thread device. Firstly, you should register an event handler for OPENTHREAD_EVENT_SET_DNS_SERVER. This event will be post after the thread device gets its NAT64 prefix. And in the event handler you should set the sntp server by sntp_setserver(). The sntp server address should be an IPv6 address with a NAT64 prefix which can be obtained by esp_openthread_get_nat64_prefix(). The last 32 bits for this address should be the IPv4 address of sntp server. Then call the sntp_init().
#include "esp_openthread_dns64.h"
#include "lwip/apps/sntp.h"

static void openthread_set_dns_server_handler(void* arg, esp_event_base_t event_base, int32_t event_id, void* event_data)
{
    ip_addr_t sntp_server;
    time_t now = 0;
    struct tm timeinfo = { 0 };
    char strftime_buf[64];

    sntp_server.type = IPADDR_TYPE_V6;
    esp_openthread_get_nat64_prefix(&sntp_server.u_addr.ip6);
    sntp_server.u_addr.ip6.zone = IP6_NO_ZONE;
    ip4_addr_t ip4_addr;
    IP4_ADDR(&ip4_addr, 209, 58, 185, 100); // IPv4 address for pool.ntp.org. It can be other IPv4 address for other sntp server.
    sntp_server.u_addr.ip6.addr[3] = ip4_addr.addr;
    sntp_setserver(0, &sntp_server);
    sntp_init();

    setenv("TZ", "CST-8", 1);
    tzset();
    while (1) {
        vTaskDelay(5000/portTICK_PERIOD_MS);
        time(&now);
        localtime_r(&now, &timeinfo);
        strftime(strftime_buf, sizeof(strftime_buf), "%c", &timeinfo);
        ESP_LOGI(TAG, "The current date/time in Shanghai is: %s", strftime_buf);
    }
}
......
esp_event_handler_register(OPENTHREAD_EVENT, OPENTHREAD_EVENT_SET_DNS_SERVER, openthread_set_dns_server_handler, NULL);
......

But there are some issues on APIs like sntp_setservername(). So the sntp APIs in esp_netif component will also fail as esp_netif APIs use sntp_setservername() . Maybe we need a hook to map the IPv4 address to IPv6 address for this function after the DNS resolving. But now we don't have any plans to fix it.

Thank you so much！！！T T LOVE ya

@moculll If query is answered, please close the issue.

Closing this now


unify navigation methods

Avoid having separate parameters for caches, searches, waypoints and so on. In most cases this can be unified to a coordinate only.

Yes, but then we need a method to support that use case (and other methods to support other use cases). Currently we take all possible parameters into one large method and guess inside the method which use case we currently want to execute. So in the end there may be the same amount of code, but it will be easier to understand. :)

I agree with @Bananeweizen. A recurring problem of changes in the internal map is that the 'use case' is handled at way to many places by interpreting the availability and value of a set of parameters in an opaque way. Identifying and separating the real 'modes of operation' would be a huge step forward.

@Bananeweizen Why did you make the change in cgeopopup regarding fromDetails to moreDetails
I now have the following problem:

Select nearby from the main screen
Click the map in action bar to display the 20 caches on the map
Select a cache
All you see is the basic information. I can't choose to view the cache without having to remember its name and go back to the list view.
I should be able to click More Details and have the cache loaded.

The condition when to show more details may be wrong. I am not sure what is correct there, but it should depend on the state of the cache object only.
Previously that flag was propagated through multiple activities and even through the map overlays, just to differentiate whether the user is coming from the detail activity or not. And that was clearly absolutely useless (and horrible from the viewpoint of encapsulation).

I think the more details button should always be shown. If we have full cache info then it will be displayed right away. If we don't, then it will be downloaded (just like is done for Live Map, which is why @blafoo had to change the code to
 if (moreDetails || cache.getType() == CacheType.GC_LIVE_MAP)
If you prefer, I can open a new issue for this.

I had a second look at this and you are completely right. All the boxes in the popup are now visible without conditions.


Double routerlink in Intro.js?

Hey, you seem to have the <RouterLink href="/#project-1"> twice in the Home/Intro.js file at the bottom, are you doing this for a reason, or is this just by accident?

It's intentional, one is for touchscreens (the down arrow) and the other for desktop (the mouse). There's a media query in the css to show the appropriate scroll indicator link


Not working behind proxy

Add an option to change proxy settings for the app to use.

Duplicate of #60.


Web3 build failed

I am working on substrate-node-template (https://github.com/substrate-developer-hub/substrate-node-template).
Integrate web3 as dependency in pallet/cargo.toml file .
Getting error named as sys not defined in this scope. similar more build errors are there, souch as socket.. error.

Can you suggest me the reason, How I can use this web3 in my code

Docs.rs is not a general support site. You can ask for help on users.rust-lang.org or on the community discord.


WIP: Add RAND_DRBG_bytes

I also like that you add additional data automatically.

I've pushed a new version

I've pushed a new version

What has changed w.r.t the version from two days ago?

Only the rand_drbg_get_additional_data() function, it should do something useful on at least unix and windows.

The travis failure with --strict-warnings is something I'm trying to deal with in #4757

I've pushed a new commit

I'll try to have a decent look at this tomorrow.


Cannot abstract out window contents, Ui is borrowed by build()

So here's a conundrum. I am currently working on an editor for my game engine. At this point I am trying to create document windows for different types of assets. I have a Document trait with a function taking an &mut Ui, so that it can specify what the document window contents should be. Unfortunately, this does not play nice with Rust's borrowing rules:
imgui::Window::new(&im_str!("{}", document.window_name())).focused(raise).opened(&mut window_open).position([320.0, 48.0], imgui::Condition::Appearing)
    .size([640.0, 640.0], imgui::Condition::Appearing).build(ui, ||
{
    document.do_ui(editor, ui);
});

error[E0500]: closure requires unique access to `ui` but it is already borrowed
    |
420 |     .size([640.0, 640.0], imgui::Condition::Appearing).build(ui, ||
    |                                                        ----- --  ^^ closure construction occurs here
    |                                                        |     |
    |                                                        |     borrow occurs here
    |                                                        first borrow later used by call
421 | {
422 |     document.do_ui(editor, ui);
    |                            -- second borrow occurs due to use of `ui` in closure

Is there any way to solve this?

Found a workaround: pass a Window builder object with some pre-set properties into the function, along with the Ui, so that the window is built inside the function where the Ui borrow isn't subject to the function barriers that the borrow checker seems to dislike so much.

Does it help if you use begin+end instead of build?

A bit late on this, but I did some recent refactoring to move to a tabbed interface and I started using the begin/end API. So far it's worked very well for me. I have noticed however that CollapsingHeader does not have begin/end. Created #481 for this.


UI: Add staggered animation delays to splash logo

TL;DR
Added delay animation parameters to the Krail logo letters in the splash screen.
What changed?
Added individual delay parameters to each letter in the animated Krail logo, creating a sequential animation effect. Each letter now has a specific delay: R (50ms), A (100ms), I (150ms), and L (200ms).
How to test?

Launch the app
Observe the splash screen animation
Verify that the letters "RAIL" appear in sequence with a slight delay between each letter
Confirm the animation looks smooth and professional

Why make this change?
To enhance the visual appeal of the splash screen by creating a more dynamic and polished animation sequence. The staggered letter appearance adds a professional touch to the app's initial loading experience.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#336  👈
#335
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ksharma-xyz and the rest of your teammates on  Graphite

Merge activity

Nov 11, 9:30 AM EST: A user merged this pull request with Graphite.


[BUG] I am not able to get png images from AQUOS sense 4 SHARP Android 10 device.

I am not able to get png images from AQUOS sense 4 SHARP Android 10 device.
I've tried everything and still can't get the png image on this device.
I need help .
Screenshots

Flutter version
Flutter Version : 2.10.5
Photo_manager: ^2.1.1
**Device **
Device :  AQUOS sense 4 SHARP Android 10

Try
https://github.com/fluttercandies/flutter_photo_manager/blob/d51761629915072c0957342eb4bc44d6b43c8081/lib/src/filter/filter_options.dart#L86

@AlexV525 Thank you, i tried the problem solved.

Hi!
I also having trouble getting the PNG images, can someone

please help me


Use https-proxy-agent instead of tunnel. Fixes #53

Used the module https-proxy-agent instead of tunnel seeing that with node 0.12.7 is not working.
Tested on our environment and it is working. Keeping on mind that I am not a node expert :D

:+1:

Merged, thanks!


Should keep index consistency across partitions

for example:
Create index_1 on column A in partition A
Create index_1 on column B in partition B
This is allowed by OAP, but this inconsistency will cause problem later, say, refresh index.

@songzhan01
you can copy you test case here for this issue.

@linhong-intel sumbit a task to check in parallel whether all existing index_1 point to column A?

This is result from that we allow operation on a single partition. Some other issues may be caused here, I'll create another issue tracking this, thanks!


WIP: added links

For issue #180
I've added links under our Devfile Samples page so users can access the docs repo as well. I thought a name change to Devfile resources would make it more intuitive to the users. If we like the name change idea, I'll have to change the name of the actual doc and in the TOC.
I'm trying to give more context as to why users would even want to visit our repos. If anyone has suggestions for more context, that'd be appreciated.
Also, to my knowledge, there's no communication channel to link users to, like a YouTube channel, blog page, Twitter account, etc. But those can be considerations, moving forward!

Thanks, @elsony, new, squashed commit is up.

@rkratky, this PR is ready to merge if it also looks good after your review. Thanks you, and thanks, Elson, for the SME review!

@rkratky, this PR is ready to merge if it also looks good after your review. Thanks you, and thanks, Elson, for the SME review!

Note: working with Eva to complete doc peer review.

Note: working with Eva to complete doc peer review.

LGTM!

LGTM!


[card-92] upload images while running specs

@goromlagche can you please rebase this PR. thanks.

closing this one. would suggest to give it a try using concurrent-ruby gem, and make that gem a soft dependency.


Nullpointer exception when parsing endpoint for a local instance of DynamoDB

When connecting to a local instance of DynamoDB, the AwsHostNameUtil.parseRegion returns null, since the endpoint is not a valid AWS endpoint. This null is then passed down to Regions.fromName and then an NPE is thrown and thus preventing the DynamoDBDelegate class from being instantiated.

Looks good - please squash all commits to one. And add your name to the contributors list in pom.xml. I'll review once more and then merge in!

I have squashed the commits and updated the contributor list.
I had a look at the build failing. Now that a connection is actually opened to Dynamo locally, it seems like the local instance of Dynamo is not there anymore when the tests run. Even though from the logs, it was started before the tests. I ran the tests locally a few times and it could connect fine. Will need to investigate what is happening with the build on Travis.
There seems to be permission issues as well further down in the logs:
Caused by: org.elasticsearch.ElasticsearchIllegalStateException: Failed to obtain node lock, is the following location writable?: [/tmp/searchindex/data/elasticsearch]
If there is anything I can help with to get the build going again, let me know.

@F2006 please create an issue for using a different directory for the ES root as it makes sense that I don't have permission to write to /tmp in the Travis container.

@F2006 You could fork the project (you already did) and set up your own Travis CI for the time being. But I can work on the ES permission issue to start. Good catch.

@F2006 created #80 to track the /tmp/elasticsearch fix


chore: update cmd/fitconv

run fitgen 21.113

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%.  Consider uploading reports for the commit 36ea950 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #174   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           42        42
  Lines         3471      3471
=========================================
  Hits          3471      3471

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix #7064: Ignore key events without scan code

Up until #4999 we deferred all key events to the character event handler
for which ToUnicodeEx returned a valid character and alternatively
those who aren't a special key combination as listed in
TerminalInput's implementation.
Since #4999 we started acknowledging/handling all key events no matter
whether they're actually a known key combination. Given non-ASCII inputs
the Win32 SendInput() method generates certain sequences that aren't
recognizable combinations though and if they're handled by the key event
handler no follow up character event is sent containing the unicode
character.
This PR adds another condition and defers all key events without scan
code (i.e. those not representable by the current keyboard layout) to
the character event handler.
I'm absolutely not certain that this PR doesn't have a negative effect
on other kinds of inputs.
Is it common for key events to not contain a scan code? I personally
haven't seen it happen before AutoHotKey/SendInput.
Before this PR is merged it'd be nice to have a good testing plan in
place in order to ensure nothing breaks.
Validation Steps Performed
Remapped AltGr+8 to » using AutoHotKey using <^>!8::SendInput {Raw}».
Ensured » is printed if AltGr+8 is pressed.
Closes #7064
Closes #7120

This seems reasonable to me, but I also don't really know if there are other kinds of inputs without scan codes. I guess I haven't seen it, but I'm also not usually looking for that.
I'm not sure how we could be more pre-emptive about other input methods that we don't know, but we could make sure to add tests to cover this case, so that we don't regress it again in the future. Maybe something like just testing that HandleKey with a 0 scancode returns false? I know that doesn't check that we still get a character event afterwards, but it's something...

I‘ll add a unit test as soon as I can. 🙂

I don't really see a usage of 0 scan code that would turn into something we could know how to handle without further user32 intervention.
So I'm okay with this being "it's better than it was and if something proves us wrong, we'll refine further then."

PR fixed. 👍

Okay, let's scream test this!

As always, thank you @lhecker!


Logs to the console also when asked to just write on file.

I have this command:
const log = require('simple-node-logger').createSimpleLogger('project.log');
I just want to log to the file, but I also see it logging to the console. How can I stop it to log to the console ?

if you just want a file logger use 'createSimpleFileLogger()' (see examples/simple-file-logger.js)


az monitor log-analytics workspace table show returns retention as null

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

using az cli to update a table seems to work and to return the correct payload. but when using get table the retention returns as null. in addition when using REST request directly to ARM the correct payload is received.
Rest request sent:
GET https://management.azure.com/subscriptions/a8732c41-c4a7-4fa1-a180-f3efb7843f6f/resourcegroups/testworkspaces/providers/Microsoft.OperationalInsights/workspaces/TestWorkspaceWestUS/tables/Heartbeat?api-version=2015-11-01-preview HTTP/1.1
Rest response:
{"properties":{"RetentionInDays":40,"IsTroubleshootingAllowed":false},"id":"/subscriptions/a8732c41-c4a7-4fa1-a180-f3efb7843f6f/resourcegroups/testworkspaces/providers/microsoft.operationalinsights/workspaces/testworkspacewestus/tables/heartbeat","name":"Heartbeat"}
Example using cli:
PS /home/tom> az monitor log-analytics workspace table show --resource-group testworkspaces --workspace-name TestWorkspaceWestUS --name Heartbeat
{
"id": "/subscriptions/a8732c41-c4a7-4fa1-a180-f3efb7843f6f/resourcegroups/testworkspaces/providers/microsoft.operationalinsights/workspaces/testworkspacewestus/tables/heartbeat",
"name": "Heartbeat",
"resourceGroup": "testworkspaces",
"retentionInDays": null,
"type": null
}
PS /home/tom> az monitor log-analytics workspace table update --resource-group testworkspaces --workspace-name TestWorkspaceWestUS --name Heartbeat --retention-time 40
{
"id": "/subscriptions/a8732c41-c4a7-4fa1-a180-f3efb7843f6f/resourcegroups/testworkspaces/providers/microsoft.operationalinsights/workspaces/testworkspacewestus/tables/heartbeat",
"name": "Heartbeat",
"resourceGroup": "testworkspaces",
"retentionInDays": 40,
"type": null
}
PS /home/tom> az monitor log-analytics workspace table show --resource-group testworkspaces --workspace-name TestWorkspaceWestUS --name Heartbeat
{
"id": "/subscriptions/a8732c41-c4a7-4fa1-a180-f3efb7843f6f/resourcegroups/testworkspaces/providers/microsoft.operationalinsights/workspaces/testworkspacewestus/tables/heartbeat",
"name": "Heartbeat",
"resourceGroup": "testworkspaces",
"retentionInDays": null,
"type": null
}
To Reproduce
in order to reproduce just run
az monitor log-analytics workspace table update
and then
az monitor log-analytics workspace table show
and you will see the retention returns as null
Expected behavior
the expected behavior is that the retention will return when running the "show" command
Environment summary
i tried to run from CLI in azure portal, but i guess it is the same for all environments
Additional context
the type is also null - i am not sure what it supposed to contains, but maybe worth fixing it as well

monitor

The property 'retentionInDays' is not capitalize in swagger. But in response body it names 'RetentionInDays'.

Please let me know the current status of this article.

@sameergMS @dadunl Please help with this issue. Thanks!

Can you please tell me the current status of this event?

Can you please tell me the current status of this event?

Monitor service team should look into this issue.

Monitor service team should look into this issue.

CX is required for an urgent resolving this issue.
When are you planning to resolve this issue?

CX is required for an urgent resolving this issue.
When are you planning to resolve this issue?


All() is missing?

Hello
I don't know what's going on but the All() is missing everywhere!!!
My program has more then 200 calls to All() and it does not compile anymore.
Would you be so kind to put All() back everywhere so I can continue to make my program? That code worked perfectly.
I'm a beginner and have no knowledge of Async and threads etc etc... I'm making a very simple program that uses All() to get the result. I have no experience with threads and also don't need this in my program. I just wait until mouse cursor is set to normal again in my code after All() has finished.
I'm very sad now, development of my program is stopped because of 2 reasons:

.NET 4 support is dropped
All() is removed

I hope you put it back. I worked for months on this program.
Best regards
Jessica

I really don't want to add the non-asynchronous method calls back into the library, they were an example of bad coding and should be removed. You should not be using synchronous code for getting data from a Web API. if you really don't want to make your program asynchronous then you can always add a .Result to the end of the AllAsync method, although this is a major code smell.
I will be adding multi targets though

So AllAsync().Result you mean? If I run it this way, does it block the current thread?
Is a Backgroundworker a correct way to run your AllAsync?
see:
https://docs.microsoft.com/en-us/dotnet/api/system.componentmodel.backgroundworker?view=net-5.0
Thx for adding multi targets!!!

No that's the old way of doing thing things. You need to use await and async keywords

I understand AllAsync() runs in a different thread and when called does not block the thread it was called in , in my case the UI thread.
If this is'correct, in which way do I need to call AllAsync:

in a new thread
in a new Task

?
My program's user interface must know when AllAsync() is completed.
When I used All() the UI thread was blocked and when All() had finished its work, I updated my UI.
So if'you can explain what way to use (thread or task) in order to know when AllAsync() has completed.
thx

I never used a async call, so that's why I ask.
Maybe you can write an example which you include in the library source code in 'examples' of how a user can get all sets using AllAsync() with a way to know when the AllAsync() has completed its work?
If I run AllAsync().Result , does it block the thread it is called in (so my UI thread)?
I didn't understand?

https://docs.microsoft.com/en-us/dotnet/csharp/async

Will you never remove .Result in an update?
Because I want to use it now.

yes


Question: is there a setting to control which key combos are handled by VS Code and which by NeoVim?

Iin VS Code I often use ctrl+j to focus/show then hide the terminal. In the vscodevim extension, this was accomplished by adding:
"vim.handleKeys": {
  "<C-j>":false
}

to the settings json. Is there any comparable feature with this extension?

Go to keybindings, remove


fix: allow high contrast mode for checkboxes

Description
Fix for checkbox part of #786.
This changes checkboxes to work in the following ways in WHCM

Have a visible focus indicator using the highlight colour
Use Disabled colour when disabled
Not use error colour when in error
be able to determine the checked/unchecked/mixed state

Note - I am unsure how to mark error state and am in discussions with Spectrum. IMO this can be tackled in a later PR.
How and where has this been tested?

Run using Windows High Contrast mode both White on Black and Black on White
Note that the colors now match the windows system colors
Edge 85.0.564.30 on Windows 10

Screenshots

To-do list

[ ] If my change impacts other components, I have tested to make sure they don't break.
[ ] If my change impacts documentation, I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.

[ ] This pull request is ready to merge.

Thanks for this!  We will put some time into reviewing it and get back to you soon.


Implement --generate option in build script

This will rebuild the admin UI static filesystem for builds when the --generate option is used. This is a failsafe in case the admin UI static filesystem is not rebuilt when it is changed, see https://github.com/influxdata/influxdb/issues/5780.
A machine running the build script with the --generate option will also need to have github.com/rakyll/statik installed and in the PATH. Not sure about the proper place to put that disclaimer.
@rossmcdonald

+1, with one comment

@rossmcdonald thanks for pointing out run(). I had just uncommented the existing TODO code.
I decided to remove the whole run_generate function and call run("go generate services/admin") in main. Let me know if that's too minimal.

@gunnaraasen Actually I'd prefer if it was in run_generate(). I'm trying to keep everything below line 130 independent of the application that it's building, so that it's easier to port functionality between the Kapacitor and Telegraf build scripts.

No problem. I added run_generate() back in.
There are still comments that reference go generate services/admin instead of go generate ./... below the fold. I figure those will need to be edited anyway since they were already InfluxDB specific.

+1, looks great


domProps innerHTML does not work with svgs in IE 11

Version
2.5.16
Reproduction link
https://codepen.io/anon/pen/vrqYEG
Steps to reproduce
Set domProps: { innerHTML } of svg tag and it will not work in IE
What is expected?
SVG should contain provided HTML
What is actually happening?
SVG contains no HTML

React had the same issue a while ago
https://github.com/facebook/react/issues/10013

innerhtml is not supposed to work with svg(per spec). See https://github.com/vuejs/vue/issues/5310#issuecomment-290006053
But apparently there is a shim that would allow you to set it when I did a quick search