fix check of available dates of next month

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/28622
PROPOSAL: https://github.com/Expensify/App/issues/28622#issuecomment-1743236069
Tests

Ensure today's date is the last date of month. If it isn't change the date manually on the device.
Click on FAB and then Request Money and switch to Manual tab
Input Money
Click Next
Select any user and click Next
Click on Show More and then click on Date
Change the month to the previous month with < arrow
Verify that the next arrow > is not disabled

[x] Verify that no errors appear in the JS console

Offline tests

Ensure today's date is the last date of month. If it isn't change the date manually on the device.
Click on FAB and then Request Money and switch to Manual tab
Input Money
Click Next
Select any user and click Next
Click on Show More and then click on Date
Change the month to the previous month with < arrow
Verify that the next arrow > is not disabled

QA Steps

Ensure today's date is the last date of month. If it isn't change the date manually on the device.
Click on FAB and then Request Money and switch to Manual tab
Input Money
Click Next
Select any user and click Next
Click on Show More and then click on Date
Change the month to the previous month with < arrow
Verify that the next arrow > is not disabled

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/102477862/fc948b13-5d49-4fd9-9570-9fdb0be9a3db

Android: mWeb Chrome
https://github.com/Expensify/App/assets/102477862/3f012993-ccd3-4e4d-9a17-3e95f3651d76

iOS: Native
https://github.com/Expensify/App/assets/102477862/1cb33577-6f74-47ef-baf9-888f1bced48e

iOS: mWeb Safari
https://github.com/Expensify/App/assets/102477862/5d6cf166-00a1-47fe-8664-5c2f288c501d

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/102477862/0562e401-fa81-491b-b3f2-9e57f443581e

MacOS: Desktop
https://github.com/Expensify/App/assets/102477862/2c8923ab-eb31-40b6-a180-5d4f714d13ce

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/108292595/a29c515e-9c26-4559-80d5-846ff01774f0

Mobile Web - Chrome
https://github.com/Expensify/App/assets/108292595/bd3cbb3f-ec59-40ea-9a47-92167ae4a9c5

Mobile Web - Safari
https://github.com/Expensify/App/assets/108292595/d0359ad5-91e2-4c44-ae12-35a9019ecc65

Desktop
https://github.com/Expensify/App/assets/108292595/63271817-5c61-4ac4-a7e6-d421b40a8723

iOS
https://github.com/Expensify/App/assets/108292595/23a02f34-4f51-43ab-b32e-400baae917e1

Android
https://github.com/Expensify/App/assets/108292595/bd3cbb3f-ec59-40ea-9a47-92167ae4a9c5

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/cristipaval in version: 1.3.84-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/francoisl in version: 1.3.84-10 🚀

platform
result

🤖 android 🤖
skipped 🚫

🖥 desktop 🖥
skipped 🚫

🍎 iOS 🍎
skipped 🚫

🕸 web 🕸
skipped 🚫

🚀 Deployed to production by https://github.com/francoisl in version: 1.3.84-10 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/cristipaval in version: 1.3.85-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/francoisl in version: 1.3.85-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Numpy setup.py fixes

I've provided some examples of how I was able to fix the couple of issues (#151 and #152) I recently raised in this repo.
Feel free to pick them apart or use as-is, or suggest changes!

Coverage increased (+3.6%) to 92.287% when pulling f47dd6c51093321e7c51533eee8b363225d66f2f on aodn:numpy-setup-fix into 812009c9d13b25b11cad6118e05f4c0f27d4ac7d on SciTools:master.

Coverage increased (+3.6%) to 92.287% when pulling f47dd6c51093321e7c51533eee8b363225d66f2f on aodn:numpy-setup-fix into 812009c9d13b25b11cad6118e05f4c0f27d4ac7d on SciTools:master.

Coverage increased (+3.6%) to 92.287% when pulling f47dd6c51093321e7c51533eee8b363225d66f2f on aodn:numpy-setup-fix into 812009c9d13b25b11cad6118e05f4c0f27d4ac7d on SciTools:master.

@lwgordonimos Awesome, thanks :+1:

No worries, makes perfect sense to keep it contained. Sorry, I glanced at your example and jumped the gun with copy/pasting without noting the details!

@lwgordonimos Cool, thanks for the PR, much appreciated! :beers:
I've already tagged the patch release 2.1.2, and I'm in the process of building and uploading to PyPI, and I'll also update the recipe on conda-forge, so hopefully this will unblock you.
Nice one, all the best!


Raf sample code directories

Code files for Indexer
TODO:

[ ] Update README.md with overview code samples and directory structure

Tested Java examples: Pass.
Ready for merge @rfustino ?


throw camera drift.

Is your feature request related to a problem? Please describe.
add camera drift on release like this:
https://danielefornari.github.io/three.js/examples/?q=arcball#misc_controls_arcball
Describe the solution you'd like
I'd like camera spin drift damping with these values

drag release to throw.
Describe alternatives you've considered
not using F3d even though I love it.
Additional context
save the tendons in our forarms please.  long drag actions are RSI accelerants.
Contribution
able but no time.
Please tell us if you are able to contribute (Can you develop this new feature?)
Don't worry, you can still post a feature request if you cannot contribute!*

Looks like a fun feature to implement :)

I've seen a lot of renderer using this kind of interaction.
It's particularly useful with touch screens.

I'm not sure about the implementation for this one. Use timers to move the camera ?

This should now be implementable thankgs to the event loop


Request to provision "Class home page" template failed

Your request to provision   "Class home page"    template failed!   Please report   the issue here providing the following provisioning action correlation   ID: 885b2a29-2e24-43d7-a187-8fe95a161666 !

Hi,
Thanks for using the Look Book site. Based on the event log, we can see that you applied the template providing the URL of a site that was a "Communication Site" (SITEPAGEPUBLISHING#0), while the template was expecting to have a "Modern Team Site" (GROUP#0). Please, try again providing a correct site target. In case of any issue, please create another item and provide the corresponding Correlation ID for further investigation. Thanks for your cooperation.


Install instructions

It seems like the installation instructions should be updated.

[ ] Add go get github.com/bjwbell/gensimd
[ ] Remove https://
[ ] go install https://github.com/bjwbell/gensimd/simd/sse is wrong path

Sorry about that, I never tested the install directions. I'll have it fixed shortly

No problem. Just wanted to put it in, so I didn't forget it.

Hopefully the new instructions are correct.


Update OpenSearch versions in main and 1.x branches

Please update the version consumed in main and 1.x branch according to OpenSearch main and 1.x respectively.

There are CI failures in https://github.com/opensearch-project/index-management/pull/119/ when trying to use OpenSearch 1.1.


Compile errors in Visual Studio 2013

Building the master branch with the following commands
mkdir msbuild
cd msbuild
cmake .. -DCMAKE_INSTALL_PREFIX=""
msbuild.exe opencc.sln /p:Configuration=Release /t:libopencc;opencc;opencc_dict /fl

gives errors:
  z:\work\opencc\src\PhraseExtract.hpp(79): error C2039: 'function' : is not a member of 'std' [Z:\work\OpenCC\msbuild\src\libopencc.vcxproj]

  z:\work\opencc\src\UTF8StringSlice.hpp(177): error C3861: 'memmem': identifie
r not found [Z:\work\OpenCC\msbuild\src\libopencc.vcxproj]

std::function cannot be resolved; we should import <functional> explicitly
The function memmem() is a GNU extension; we should consider rewriting that part using strstr() or std::string::find()
There is one more error that happened to src/PhraseExtract.cpp: msvc cannot process UTF-8 encoded string in source code, unless the file containing Unicode string literals starts with a UTF-8 encoded Byte-order Mark (U+FEFF)

Fixed by https://github.com/BYVoid/OpenCC/commit/0184be7f3b58d4c35af1ef24fca8d8f133a4bd7e


Configuration option to tell go.nvim to never load guihua

I was able to track this down behavior found in utils.load_plugin, where if packer or lazy.nvim are detected, go.nvim will attempt to load guihua. However, unlike the pcall attempts for checking for lazy or packer, the call to load guihua is done so unconditionally:
      require('lazy').load({ plugins = name })

This results in a vim.notify error every time any function might load guihua. Unfortunately, while it might be easy to say "just use it then", this makes it impossible for me to use the current vim.ui.input plugin that I actually use, and prefer to use, over guihua. This is because projects where I write go are usually multi-lingual and so once guihua is loaded my input and select settings are changed until I reset my neovim instance.
Given that the function in question can return nil, I would very much prefer there to be a "bring your own vim.ui.input/vim.ui.select plugin" setting if possible to just unconditionally return nil if found.

Not sure if there a specific module/function breaks? As input will be override only during the time you using it and will be reset back later on. Here is code from go rename which uses input:
  local guihua = utils.load_plugin("guihua.lua", "guihua.floating")
  local input = vim.ui.input

  if guihua then
    vim.ui.input = require('guihua.input').input
  end
  vim.lsp.buf.rename()
  return vim.defer_fn(function()
    vim.ui.input = input  -- after rename it will reset
  end, 1000)

It seems to set it across all buffers once it's been enabled, and if I swap to a different buffer mid-rename it'll continue to use it and that's happened on occasion, but only when using gui clients where buffers don't steal the entire context input.
That said, I should point out that in the README, it does say guihua is an optional dependency for floating arguments, but the way utils.load_plugin executes above, it will always results in an error under lazy.nvim, and if I don't have a custom notify that pops up under :messages. tbqh I wasn't sure if I should just request a way to disable it, or ask that how all plugins are loaded with lazy.nvim change. I figured asking for a configuration option would be the least intrusive change. 😅

Use GoRename for example, if guihua is installed, it use guihua. After rename finished, the vim.ui.input will reset back. It does not affect other components.
guihua is an optional package and if it fails to load it will fallback to default non- UI output instead of preventing you from using the plugin.
Also, you can adjust the vim.notify to be less noisy and only popup when warn/error. Alternatively, you can override go.utils.info to a print function or empty function if you think it is necessary

Unfortunately, lazy.nvim always errors when guihua is not installed, hence why I'm asking, if this configuration could be added to just skip all of this together. I would need to turn off all of notify itself, or modify lazy.nvim directly.

This error occurs everytime I use anything that might load guihua in go.nvim, the primary one being GoRename

I have updated a change so it will be less noisy.

Hi, I was looking at the commit and it looks like it's currently in the fixtures branch. 😅
I appreciate the work though. Looking forward to the commit making it into the main branch 🙂

Thanks for the heads up. The branch was merged

Thank you! I'll close this now 🙂


Pull Request

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.

Description

a website that shows my social medias
Link to Website

https://ricethe.pages.dev/

CLA Assistant Lite bot: Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

You can retrigger this bot by commenting recheck in this Pull Request

I have read the CLA Document and I hereby sign the CLA


An error occured, please submit a bug report. Error: Error: General Photoshop error occurred. This functionality may not be available in this version of Photoshop.

I tried to export my file as a sprite sheet, but was met with this error.
Photoshop 21.0.2 Release
macOS Mojave 10.14.6

Same error here in middle 2022!

I was getting this and then I realized I needed to put all of my layers into animation frames in the timeline for it to work...


The dark/light mode keeps switching when changing sections in Firefox

I hope this message finds you well!
First of all, I want to commend you on the fantastic work you’ve done with the Sphinx Awesome theme. The aesthetics and functionality are truly impressive and have greatly facilitated my work with documentation.
I’m reaching out because I’m experiencing an issue with the dark/light mode compatibility in Firefox. While the theme's documentation seems to work well, I’m having trouble getting it to function properly in my own project.
If there’s any configuration or workaround that I might try, I would greatly appreciate your guidance.
Thank you again for the amazing theme and your dedication!

@kai687 Thank you for your quick response!
I managed to work around the issue I mentioned earlier regarding the light/dark mode. It turns out that in production, the behavior is correct and no longer shows the problem I initially observed.
However, I’m now experiencing an issue when navigating between sections of the documentation. The screen briefly "flickers" as if the entire page is reloading. While it’s subtle, it does cause a small disruption in the navigation flow. Monitoring the network activity, I noticed that a new HTTP request is made each time I switch sections, which suggests the page is being fully reloaded.
Is this behavior expected with the theme, or is there a configuration I can adjust to avoid full reloads and make the navigation smoother?
Thank you again for your help and for the amazing work on the theme!


Option to opt-out from Cascade Layer

Summary
I would like an field in PigmentOptions that allows to opt-out from wrapping generated CSS into a Cascade Layer (which was introduced in #226)
Examples
No response
Motivation
I'm building a website using Pigment that requires to support iOS versions as old as 13, which do not support @layer.
Search keywords: layer

The official browser support policy for Pigment CSS is https://mui.com/material-ui/getting-started/supported-platforms/#browser. https://caniuse.com/css-cascade-layers fits into it.
Looking at the relative use https://caniuse.com/css-cascade-layers

I'm not sure we should entertain this further. What's the need to support iOS 13?

Hello,
I had a talk with management, it turns out our decision to support iOS 13 and above was based on outdated visits statistics, I was able to convince them to support iOS 15.4 and above instead.

@axeleroy Alright, let's close this issue for now then. By the time Pigment CSS becomes stable, browser support should have improved enough.
Now, It does make sense to me for https://mui.com/material-ui/getting-started/supported-platforms/#browser to have 3 different tiers of support:

Full support, like today
Best effort support (like this case could be)
No support (like IE 11 is under)


Indexing a dask array with a decreasing slice results in negative dimension size

On dask 0.16:
In [39]: import dask.array as da

In [40]: x = da.ones(10, chunks=-1)

# notice the negative shape!
In [41]: x[6:3]
Out[41]: dask.array<getitem, shape=(-3,), dtype=float64, chunksize=(-3,)>

# at least it gets computed correctly.
In [42]: x[6:3].compute()
Out[42: array([], dtype=float64)

This manifested itself in an issue for xarray: https://github.com/pydata/xarray/issues/1778

That's clearly wrong. Do you know if this has always been the case or if it was this some kind of regression?

Not sure. It could have been here for a while -- decreasing slices are pretty rare, they aren't actually useful for anything.


Vespa 8: Remove filedistribution element in services.xml

Files are distributed using HTTP and no need to support:
<filedistribution>
  <downloadbitrate>1024k</downloadbitrate>
  <uploadbitrate>4.5M</uploadbitrate>
</filedistribution>

remove https://docs.vespa.ai/documentation/reference/vespa-cmdline-tools.html#vespa-status-filedistribution too

Fixed


getaddrinfo doesn't accept port service names

socket.getaddrinfo and loop.getaddrinfo in Python 3.5.1 can accept service names for its port argument, i.e.
>>> socket.getaddrinfo('<IP_ADDRESS>', 'http')
[(<AddressFamily.AF_INET: 2>, <SocketKind.SOCK_DGRAM: 2>, 17, '', ('<IP_ADDRESS>', 80)),
 (<AddressFamily.AF_INET: 2>, <SocketKind.SOCK_STREAM: 1>, 6, '', ('<IP_ADDRESS>', 80))]

In 3.5.2 with our custom _ipaddr_info such calls will fail with an exception:
>>> import asyncio, socket
>>> loop = asyncio.get_event_loop()
>>> loop.run_until_complete(loop.getaddrinfo('<IP_ADDRESS>', 'http', type=socket.SOCK_STREAM))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/yury/dev/py/asyncio/asyncio/base_events.py", line 589, in getaddrinfo
    info = _ipaddr_info(host, port, family, type, proto)
  File "/Users/yury/dev/py/asyncio/asyncio/base_events.py", line 108, in _ipaddr_info
    port = int(port)
ValueError: invalid literal for int() with base 10: 'http'

/cc @ajdavis

Honestly I thought that such service names were a thing of the past. But ah well...

Looks like we need to use https://docs.python.org/3/library/socket.html#socket.getservbyname if we can't cast port to an int.

You mean a numeric string like '80' would also be accepted?

Yes, and event b'80', since socket.getaddrinfo accepts it. Apparently people use that.

This little project still has more corners of socket programming to teach me! I've dealt with '80' and b'80' already, this last patch also handles 'http' and b'http'.

Closing this issue -- the relevant PR has been merged.


fix(compiler): return undefined for getScriptVersion

Summary
Close #1639
Test plan
Not sure how to test this. But from code perspective, using ! operator is wrong.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
N.A.

Pull Request Test Coverage Report for Build 4835

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 92.853%

Totals

Change from base Build 4834:
-0.04%

Covered Lines:
1061

Relevant Lines:
1096

💛  - Coveralls


cannot remove breakpoint if breakpoint is on empty line

When editing a file that has breakpoints, sometime I end up in a situation where the breakpoint sits on an empty line:

If this happens I'm no longer able to remove the breakpoint by clicking on it in the gutter.
I suggest that you allow to remove a breakpoint on any line.

Maybe I should just allow breakpoints to be set on empty lines? This was added before we nicely adjusted breakpoints once a session starts.
e.g chrome allows to set on empty lines and then they adjust

@isidorn yes, please allow for setting breakpoints in empty lines. That will make it possible to support debugging Whitespace programs!

Whitespace users around the world can rejoice now!


Render tiles upper in the stack in front of the ones below

Fixes #8026.

As a temporary stopgap measure until we explore better alternatives, we have decided to tackle the above issue by putting tiles upper in the stack in front of the ones below.
We need to put tiles into a separate div with its own stacking context, so that tiles do not overlap other elements such as the cursors or the wrap-guide.
/cc: @nathansobo @abe33 @rbu

@rbu: it'd be great if you could try this out and check if it actually solves the problem you reported. In the screenshot above it seems like this did the trick, but I'd like to make sure it works on the other platforms as well. Thank you! :pray: :sparkles: (if you don't know how to build an Atom branch, please follow the build instructions or hit me up on Slack)

Code looks good to me if it solves the problem.

Timeline

master

as-tiles-z-index

:+1:

You might want to inset that unicode oddity in the first line of a new tile.
Out of interest: since this is meant as a stopgap, shouldn't you let the bug #8026 open to track resolution of the bigger issue?

@as-cii, thanks for the ping. I could not build Atom yet, as it would require an npm update on my system (which I have no package for) and now you were faster than myself. Anyway, thanks for fixing it, I'll check it out in the next release!

You might want to inset that unicode oddity in the first line of a new tile. :)

@dwt: yeah, we are aware it's not a super comprehensive solution but we wanted to make sure that a common case such as the underscore was covered in the meantime.

Out of interest: since this is meant as a stopgap, shouldn't you let the bug #8026 open to track resolution of the bigger issue?

That specific issue should be fixed as of now, but I agree we should probably track the bigger problem in a new issue (linking to #8026 and similar bug reports). Thanks for pointing that out! :bow:

I could not build Atom yet, as it would require an npm update on my system (which I have no package for) and now you were faster than myself. Anyway, thanks for fixing it, I'll check it out in the next release!

@rbu: sounds great! Thanks for taking the time to report this problem and to follow its progress! :sparkles:


v0.11.0-rc1: draft logs error: seek ...: invalid argument

When using v0.11.0-rc1, I am seeing errors calling draft logs on a failing build:
><> draft up
Draft Up Started: 'example-go'
example-go: Building Docker Image: FAIL ❌  (1.0026s)
example-go: Build ID: 01C8DQY3HXWCWRVR9TX4TWTWZT
><> draft logs example-go 01C8DQY3HXWCWRVR9TX4TWTWZT
Error: error getting logs from server: rpc error: code = Unknown desc = seek /tmp/draft-logs869081101/01C8DQY3HXWCWRVR9TX4TWTWZT: invalid argument

><> draft version
Client: &version.Version{SemVer:"v0.11.0-rc1", GitCommit:"5c9c07809e5f67e8a2711fff5fa5c89ba4bd2d9f", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v0.11.0-rc1", GitCommit:"5c9c07809e5f67e8a2711fff5fa5c89ba4bd2d9f", GitTreeState:"clean"}

I found that this doesn't happen on a failed build, but most calls to draft logs.

after extensive testing by myself and @fibonacci1729, neither of us were able to reproduce the issue. It's possibly a misconfigured cluster or my setup was in a weird state. Regardless we can close this, but let's re-open if this comes up again. :)


Android custom card com.braintreepayments.api.exceptions.AuthorizationException for sandbox environment

After successful setup unable to get nonce for custom.
XPLATlient.setup(braintreeDemoToken)
.then(function (success) {
console.log('XPLATlient setup success', success);
})
.catch(function (err) {
console.log('XPLATlient setup err', err);
//error handling
});
const card = {
number: "4111111111111111",
expirationDate: "10/20", // or "10/2020" or any valid date
cvv: "400",
}
XPLATlient.getCardNonce(card)
.then(function (nonce) {
console.log('XPLATlient nonce', nonce);
})
.catch(function (err) {
console.log('XPLATlient err', err);
//error handling
});
In the react console nor success nor errors.
After debagging android code figured out that Braintree lib give exception:
com.braintreepayments.api.exceptions.AuthorizationException: You are unauthorized to perform input validation with these authentication credentials.
Debuger point on the:
@Override
public void failure(Exception exception) {
fragment.postCallback(exception);
fragment.sendAnalyticsEvent("card.nonce-failed");
}
Exception in  postCallback from the BraintreeFragment.

mArgument in that fragment: Bundle[{com.braintreepayments.api.EXTRA_INTEGRATION_TYPE=custom, com.braintreepayments.api.EXTRA_SESSION_ID=555b135dbbab4a12958a9037d3a492e8, com.braintreepayments.api.EXTRA_AUTHORIZATION_TOKEN=}]

Found whats wrong.
Problem with validation.
In the Braintree.java (this library):

CardBuilder cardBuilder = new CardBuilder() .validate(true);

in the getCardNonce method
I set it to false and now working fine.
I hope it is help if somebody get same error.
Also the question It is related with sandbox environment?
Somebody had validation works with production environment?

I set to false
CardBuilder cardBuilder = new CardBuilder() .validate(false);
Still not working.


Identifiers validation failed for dataset Uso y monitoreo de los recursos naturales en el Corredor Biológico Mesoamericano (áreas focales Xpujil-Zoh Laguna y Carrillo Puerto) (Bosques)

Identifier validation failed for the dataset Uso y monitoreo de los recursos naturales en el Corredor Biológico Mesoamericano (áreas focales Xpujil-Zoh Laguna y Carrillo Puerto) (Bosques):

Crawler attempt: 307
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 412
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 412 total records; 412 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3740160/step

New IDs sample:

Old IDs sample:

02a353f037f583140b304c5ff831ee9b
02c815c025a0825603a1f04a87857a01
05e3235eaa92dac7c0e438eb319cd867
07797956eeec703b106de355d13b92cf
092d1fd3484f91ec4e8f958cb0adfb30
0b6f2524d27cc48efb58db484290835f
14f0cac485d2fb4e18384783af4614af
19ab97e23a5e9a633bdb7e7076088f6a
1d9f08fe3200250b83c5614ccf019eda
25c81291c70b2bb7d8cdd394cf8028e5

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Uso y monitoreo de los recursos naturales en el Corredor Biológico Mesoamericano (áreas focales Xpujil-Zoh Laguna y Carrillo Puerto) (Bosques) : https://doi.org/10.15468/hdlrbx.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Identifier validation still  failing for the dataset Uso y monitoreo de los recursos naturales en el Corredor Biológico Mesoamericano (áreas focales Xpujil-Zoh Laguna y Carrillo Puerto) (Bosques):

Crawler attempt: 308
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 412
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 412 total records; 412 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3741095/step

New IDs sample:

Old IDs sample:

02a353f037f583140b304c5ff831ee9b
02c815c025a0825603a1f04a87857a01
05e3235eaa92dac7c0e438eb319cd867
07797956eeec703b106de355d13b92cf
092d1fd3484f91ec4e8f958cb0adfb30
0b6f2524d27cc48efb58db484290835f
14f0cac485d2fb4e18384783af4614af
19ab97e23a5e9a633bdb7e7076088f6a
1d9f08fe3200250b83c5614ccf019eda
25c81291c70b2bb7d8cdd394cf8028e5

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Uso y monitoreo de los recursos naturales en el Corredor Biológico Mesoamericano (áreas focales Xpujil-Zoh Laguna y Carrillo Puerto) (Bosques) : https://doi.org/10.15468/hdlrbx.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.

Identifier validation still  failing for the dataset Uso y monitoreo de los recursos naturales en el Corredor Biológico Mesoamericano (áreas focales Xpujil-Zoh Laguna y Carrillo Puerto) (Bosques):

Crawler attempt: 310
Publishing organization: Comisión nacional para el conocimiento y uso de la biodiversidad
Publishing country: MEXICO
Installation: IPTCONABIO
Number of occurrences indexed for this dataset at the time of the issue: 412
Cause: GBIF ID problems exceed 50% threshold: 100% duplicates; 412 total records; 412 absent records
Pipelines execution steps: http://api.gbif.org/v1/pipelines/history/execution/3741713/step

New IDs sample:

Old IDs sample:

02a353f037f583140b304c5ff831ee9b
02c815c025a0825603a1f04a87857a01
05e3235eaa92dac7c0e438eb319cd867
07797956eeec703b106de355d13b92cf
092d1fd3484f91ec4e8f958cb0adfb30
0b6f2524d27cc48efb58db484290835f
14f0cac485d2fb4e18384783af4614af
19ab97e23a5e9a633bdb7e7076088f6a
1d9f08fe3200250b83c5614ccf019eda
25c81291c70b2bb7d8cdd394cf8028e5

Publisher email
Hello,
I am contacting you from the GBIF Secretariat about a dataset published by the Uso y monitoreo de los recursos naturales en el Corredor Biológico Mesoamericano (áreas focales Xpujil-Zoh Laguna y Carrillo Puerto) (Bosques) : https://doi.org/10.15468/hdlrbx.
We noticed that the occurrenceIDs were changed. We have temporarily paused the ingestions of this dataset.
As you might already know, when an occurrence record has a new occurrenceID for a given dataset, our system considers it to be a new occurrence.
This means that it will be given a new gbifid and a new occurrence URL (like this one: https://www.gbif.org/occurrence/1252968762) and the old gbifid and URL will be deprecated.
In this case, this means that the occurrence URLs would be deprecated when ingesting the newest versions of these datasets.
We would like to check with you if those changes were intentional. Do you know if this is the case?
Please let us know, thanks! We are happy to resume the dataset ingestion.
Note that some users rely on those occurrence URLs and gbifids (like https://bionomia.net for example). In an attempt to improve
the stability of the occurrence URLs and gbifids, we have implemented a warning system to detect these type of changes in datasets (see this news item).
If the data publisher can provide us with a list of old and new occurrenceIDs per record, we can avoid the identifier and URL changes. Could that be an option?
Please let us know if you have any question. Thanks!
All the best,

You can skip/fix identifier validation using the registry UI.


Implement break and continue in M_state_while

Correct creation of a continuations for break and return. Either call/cc or tail recursion is used. The continuations are created in the M_state function for while loop. The continuations are included everywhere they are needed. The continuations work correctly.

; test cases for M_state_while
(check-equal? (M_state_while '(< x 10) '(begin (= x (- x 1)) (break) (= x (+ x 100))) '(((x) (0)))) '(((x) (-1))))
(check-equal? (M_state_while '(> x -10) '(begin (= x (- x 1)) (continue) (= x (+ x 100))) '(((x) (0)))) '(((x) (-10))))
(check-equal? (M_state_while '(< x 10) '(begin (while (< x 100) (begin (= x (+ x 1)) (if (> x 3) (break)))) (= x (+ x 5))) '(((x) (0)))) '(((x) (15))))
(check-equal? (M_state_while 'true '(begin (break) (= x (+ x 13))) '(((x) (0)))) '(((x) (0))))


issues loading ftc

I have attempted to load ftc 10th edition and the same errors keep happening.
it wont load the terrain feature, the selector to chose deployment type, and the dice app does not roll dice
It says in red, error in script for start menu, extract terrain, spawn game tools and mat obj surface
it worked perfectly yesterday but seem to fail every time today
i have tried deleting and reloading but the same issue happens every time.

??

further to my issues the following assets wont load and "show error in script, attempt to index a nil value"
extract terrain
spawn game tools
matObjsurface


make avatar-recorder/avatar-replayer Inspector-ready

Fix tests.
Add autoPlay to avatar-replayer so we can control if it starts playing immediately when set on the scene.

Working on summarizing all of the changes made...

Need to fix tests and explain every change in this PR, but this can now be roughly tested at https://ngokevin.github.io/aframe-inspector/example/ while I polish up.


fix: plugins typescript import

Hi, tried to upgrade to v4.0 rc, seems like typescript dont see the svgo.d.ts import.

Work if changed to:
import type { Config } from './svgo.d.ts';

export type * from './svgo.d.ts';

My tsconfig:
{
    "compilerOptions": {
      "esModuleInterop": true,
      "skipLibCheck": true,
      "target": "es2022",
      "allowJs": true,
      "resolveJsonModule": true,
      "moduleDetection": "force",
      "isolatedModules": true,
      "verbatimModuleSyntax": true,
      "strict": false,
      "noUncheckedIndexedAccess": true,
      "noImplicitOverride": true,
      "module": "NodeNext",
      "sourceMap": true,
      "lib": ["es2022"],
      "outDir": "dist"
    }
  }

Is something wrong on my end or it should be explicit imported?

Thats for sure not the fix :)
I'll take a look on it again later and will reopen it as issue or PR

tsconfig:
"moduleResolution": "bundler",
"module": "ESNext",

fixed it for me.
More info: https://www.totaltypescript.com/concepts/option-module-must-be-set-to-nodenext-when-option-moduleresolution-is-set-to-nodenext


chore: fixup bump

What: bump version doesn't work unless it has a previous version and go versioning is different per convention

:tada: This PR is included in version 0.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Related Conversation Widget needs UI Update

The old Related Conversation (class: ConversationMLTQueryBuilder, creator: query_related_mlt used the assumption that the first message in a conversation is the question and follow up messages are answers. This assumption breaks down when Smarti is used with public channels. In such a setting one would like to search for relevant sections in conversations and the Widget should show the most relevant message plus nearby messages on user interaction.
While the server side implementation was already updates as part of #303 this will also require updates to the response format and the UI of the widget.
With the new version the message initially shown (as most relevant within the section) may no longer be the first so the old format of a question and multiple answers does no longer fit.
The suggestion would be to just return the messages of the section and mark the most relevant message within this section so that the Widget can present it to the user

As discussed with @westei the UI for conversation MLT results inherits the message context UI from conversation search, with the difference that the context messages won't be interactive and the main message will be posted as a link, if enough data (room ID and message ID) is available. In this case there is also a link in the results UI to jump to the message.


Trees-Easy-2

Complete Level order traversal,Insertion and LCA in JAVA, C++ and Python with proper comments.
Add the file in the respective folder.

Assign to me for C++

I will do this in Java. Please assign this to me.

Assign this to me @adityabisoi

Hey! I'll do this in Python. Assign it to me

I can do this in Python or C++. Please assign it to me

@Aarushi11H @JayeshGChandan Already assigned. Please add new algos as issues.

@subhangi2731 @AmanSingh0-0 Make and correct a PR ASAP. Otherwise, you will be unassigned

Incase they aren't done, please assign them to me in either C++ or Python


How to use?

Can you provide an example of how to properly call this module?
I am having difficulty with the syntax.
...sorry for being dense

Yeah I need to add a synopsis to this and a better README. But anyway run it like this:
`
Set-SystemSetting -Path 'HKLM:\SOFTWARE\Microsoft\Windows\CurrentVersion\ImmersiveShell' -Name 'UseActionCenterExperience' -Type DWord -Value 0 -Force -TryLGPO:$True -LGPOBinaryPath c:\windows\temp\LGPO.exe
Set-UserSetting -Path 'SOFTWARE\Policies\Microsoft\Windows\Explorer' -Name 'DisableNotificationCenter' -Type DWord -Value 1 -Force -TryLGPO:$True -LGPOBinaryPath c:\windows\temp\LGPO.exe
`
You will need to specify the path to the LGPO.exe file.

I've updated the module and it is now in Powershellgallery. Please review the README for how to.

I've updated the module to add the cmdlets to ExportCommands. Try it again but install it from Powershell gallery. Run this to update the module:

Uninstall-Module LGPO -AllVersions
Install-Module LGPO -Force
Import-Module LGPO
Get-Module -Name LGPO

Thank you,
Ill dig into this tonight, unfortunately however, I am unable to install from the internet in this case, so I am pre-downloading the files. Will that break the process?
Thanks,
-Tom
From: @.>
Sent: Tuesday, March 8, 2022 5:26 PM
To: @.>
Cc: @.>; @.>
Subject: Re: [PowerShellCrack/LGPOPSmodule] How to use? (Issue #1)
I've updated the module to add the cmdlets to ExportCommands. Try it again but install it from Powershell gallery. Run this to update the module:

Uninstall-Module LGPO -AllVersions
Install-Module LGPO -Force
Import-Module LGPO
Get-Module -Name LGPO

—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FPowerShellCrack%2FLGPOPSmodule%2Fissues%2F1%23issuecomment-1062459916&data=04|01||20561caff064425a6e6208da016bce88|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637823859789388464|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=Yidf6LGGJL6aQmwrpZGUbFy3qU%2BRCbwF2f56gaPPgGA%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAJOERC2KTAKEZGNCCGMXPTU6743PANCNFSM5PIGM2KA&data=04|01||20561caff064425a6e6208da016bce88|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637823859789388464|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=NT%2Fqd6s%2FJAwbfcUD%2Bf%2FhkmvAFj33HqtodAp4x6kejxM%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01||20561caff064425a6e6208da016bce88|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637823859789388464|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=6phJPnNyzlO9dfc6VM0mjsuQJbTUfoi1erXdOS%2BCxXY%3D&reserved=0 or Androidhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01||20561caff064425a6e6208da016bce88|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637823859789388464|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=6AEvSzWKjweaVWW%2FYNCfHXeE5MFFOog0JdDjS7cD%2FGA%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

I've updated this repo as well. It Should work

I am still not seeing the policy change, however I do see the module being called and reporting that it is running successfully.
despite uninstalling all versions, I do see 2 versions reported when I call the module, <IP_ADDRESS> and 0.0
I still feel that I am missing something here
see attached:

Your not setting registry items; your trying to call an item in the administrative template path. This module is designed to take a registry key and set it that administrative template item; not the other way around.
For instance:
You trying to set this policy:
Computer Configuration > Administrative Templates > Windows Components > Push To Install`
to disabled right?
So you want to run the registry key for it:
Set-LocalPolicySetting -Path 'HKLM:\SOFTWARE\Policies\Microsoft\PushToInstall' -Name 'DisablePushToInstall' -Type DWord -Value 0

I bet it did set some invalid registry keys though from that screenshot. you might want to check those.

Your not setting registry items; your trying to call an item in the administrative template path. This module is designed to take a registry key and set it that administrative template item; not the other way around.
For instance: You trying to set this policy:
Computer Configuration > Administrative Templates > Windows Components > Push To Install
to disabled right?
 I was trying to use this to change the local sec policy, which currently has the PushToInstall service, Disabled. It seems that I have just taken the long road around to come back to square 1. Lucky I'm working on a test VM, so no long term damage by this...
I did try your suggestion, but the service is still being stopped from starting. :/
any ideas?

Are you trying to stop it or start it?  I saw your repo. I used to own two KITT cars.

I need to disable the LGPO that is explicitly preventing the service from starting. Scripting it to stat or set to auto/delayed start is pretty easy. Unfortunately, were not using a DC and we really should be at this point. I was hoping to find a way to script LGPO changes easily without the need to import, export, or transport new policies between machines.
On the KR question, yes, yes I am 😃
I have a few projects that orbit that topic, including an openauto build running on a pi4, I was sampling ki3t’s voice to generate a speech profile to tie into something like an open version of alexa to run in the background, and I have just received the proto boards for the scanner bar control box.
Thanks,
From: @.>
Sent: Thursday, March 10, 2022 5:28 PM
To: @.>
Cc: @.>; @.>
Subject: Re: [PowerShellCrack/LGPOPSmodule] How to use? (Issue #1)
Are you trying to stop it or start it?  I saw your repo. I used to own two KITT cars.
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FPowerShellCrack%2FLGPOPSmodule%2Fissues%2F1%23issuecomment-1064680772&data=04|01||1df0577bf12a44ca718408da02fe6564|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637825588895746044|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=OPe6M%2FWNDpXALUamTGEAZ9M3kfQ80eoIJiL84lpDTLY%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAJOERAN4WOWI35ODQGDZFDU7KOSNANCNFSM5PIGM2KA&data=04|01||1df0577bf12a44ca718408da02fe6564|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637825588895746044|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=vr2wzm%2F4%2F45gUpHhn8MF6ib7p06xeqi3YfLo22Ekaes%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01||1df0577bf12a44ca718408da02fe6564|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637825588895746044|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=0f%2FTZPc2MjSBYbBmjjbrbtUDzTwBCV%2B5AqCXPcOb8sc%3D&reserved=0 or Androidhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01||1df0577bf12a44ca718408da02fe6564|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637825588895746044|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=5hm3exkoqFkLJEIY2WVBb0A1UOqPw2e1IvT0I2vU7xk%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

So that also can be managed by LGPO using a security template (eg. GptTmpl.inf). I wrote a similar script to utilities the LGPO in this repo https://github.com/PowerShellCrack/STIGSCAPWinTool nut I don't think it will help.
LGPO can be ran like this:
LGPO.exe **/s** <pathtofile>\gptTmpl.inf
The inf file is formatted like this
[Unicode]
Unicode=yes
[Version]
signature="$CHICAGO$"
'Revision=1 [Service General Setting] "PushToInstall",4,""`
Change the 4 to a 2 and save file call it by LGPO /s
I could write this into the module to support these types of configs, but it would have to be sometime this weekend or next.

So.... that can also be managed by LGPO using a security template though (eg. GptTmpl.inf). I wrote a similar script to utilize the LGPO as well in this repo https://github.com/PowerShellCrack/STIGSCAPWinTool but I don't think it will help.
in summary LGPO can be ran like this:
LGPO.exe /s <pathtofile>\gptTmpl.inf
The inf file is formatted like this
[Unicode] Unicode=yes [Version] signature="$CHICAGO$" Revision=1 [Service General Setting] "PushToInstall",4,""
Change the 4 to a 2 and save file call it by LGPO /s
I could write this into the module to support these types of configs, but it would have to be sometime this weekend or next.
here are some screenshots

ok, i just created the inf file in npp as displayed and attempted to apply it with lgpo.exe /s, it keeps erroring out with exit code 1. also, get-service is not recognized. i am wondering if i dont have something installed that is required.
also, what is the signature value $CHICHAGO$?

Looks like problem is solved.


Path improvement

This pull request suggests making life easier for the end user so that they do not have difficulty entering the path of the downloaded file, but that this detection happens automatically.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 55.55556% with 12 lines in your changes missing coverage. Please review.

Project coverage is 94.96%. Comparing base (f8d3216) to head (95015fd).

Files with missing lines
Patch %
Lines

src/main.py
36.84%
12 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
- Coverage   99.23%   94.96%   -4.27%
==========================================
  Files           2        2
  Lines         261      278      +17
==========================================
+ Hits          259      264       +5
- Misses          2       14      +12

Flag
Coverage Δ

unittests
94.96% <55.55%> (-4.27%)
:arrow_down:

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

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


MEI Garage Customization Service issues when using local source file

The RelaxNG file generated by MEI Garage's Customization Service has some issues when using a local source file.
The source file was canonicalized using Oxygen (attached next), and this "canonicalized" version was used as the MEI Source file in the Customization Service. Regardless of which type of Customization file I used—either one of the pre-established options in the Customization Service or a local customization file—the generated RelaxNG is behaving wrong.
mei-source-canonicalized.xml.zip

If I select one of the pre-established customization files in the website, the schema generated is empty (see mei-mensural.rng file attached next)

mei-mensural.rng.zip

If I select a customization file from my own machine, the RelaxNG file isn't empty (which is good!), but it doesn't behave as expected. I proceeded to use this RelaxNG schema file (see mei-local (2).rng attached next). I created a new XML document in Oxygen and clicked on "Customize" to add the schema (the path to the RelaxNG file that I just downloaded from MEI Garage). This file is also attached next ( Oxygen_prefileld_this_file_based_on_schema_provided.xml ). You can clearly see that the root element is assigned the wrong namespace (see screenshot and attached file).

LocalCustomization_and_MEIfile.zip
The Customization Service works perfectly when using one of the pre-established MEI release versions as the source file and using a local customization file. The issue is just when using a local source file (with either a pre-established or a local customization file, generating two different issues in each of them).

It seems more appropriate to handle this in the https://github.com/Edirom/oxgarage/ repo, so we'll take it from there. Thanks for pointing this out :-)


Default attributes cause Jenkins master to be installed with version incompatible with this cookbook

Cookbook version
2.6.0
Chef-client version
12.12.15
Platform Details
Ubuntu 15.04
Scenario:
Installing Jenkins master
Steps to Reproduce:
Create a basic wrapper recipe that includes the minimum required to install the Jenkins master via this cookbook:
include_recipe 'jenkins::java'
include_recipe 'jenkins::master'

Expected Result:
Jenkins is installed with a version that is compatible with the cookbook, i.e. < 2.X
Actual Result:
The latest stable version of Jenkins is installed (currently 2.19) which is not compatible with the cookbook
I am probably missing something, but is it possible to mention proper steps to get started with this cookbook in the README file, or, if this is a real issue, mention it somewhere in the same file?

Same here...

On top of that there doesnt even seem to be a way to specify a version since only the latest is included in the apt-repo which is at 2.7.4 as of this moment.

I've specified war installation method and downgraded to a version that the cookbook works for now...

which version is that?

I'm using 1.642

The following attributes work for me:
default['jenkins']['master']['install_method'] = 'package'
default['jenkins']['master']['use_system_accounts'] = false
default['jenkins']['master']['home'] = '/opt/app/jenkins'
default['jenkins']['master']['host'] = 'localhost'
default['jenkins']['master']['port'] = '8080'
default['jenkins']['master']['endpoint'] = "http://#{node['jenkins']['master']['host']}:#{node['jenkins']['master']['port']}/"
default['jenkins']['master']['jvm_options'] = '-Dhudson.diyChunking=false -Djenkins.install.runSetupWizard=false '
'-Djenkins.slaves.JnlpSlaveAgentProtocol3.enabled=true'
default['jenkins']['executor']['timeout'] = 300
default['packagecloud']['proxy_host'] = node['mnet_jenkins']['proxy_host']
default['packagecloud']['proxy_port'] = node['mnet_jenkins']['proxy_port']
Contrary to the documentation and the comments in the text, node['jenkins]['master']['host'], node['jenkins']['master']['port'] AND node['jenkins']['master']['endpoint'] MUST be set if you want to install a slave. The cookbook does NOT reliably determine the endpoint from the first two attributes alone.

Thanks Jörg. What version gets installed using those attributes?

The most current one, 2.23.

With 5.0.0 release we're support 2.x up so this can be closed.


Screen where creator can see incoming offers on Basilisk

We are missing one single screen where users can check their incoming offers on their NFTs
Show in table columns like

Item
Amount
Offer Maker
Time

nft name
bsx amount
maker address
relative date

Route could be /myoffers

Could user get some kind of notification of incoming offer?
I haven’t got any offers yet on my pieces but would love to know

ht @cryptodamsky

👋


Automated cherry pick of #10109

Cherry pick of #10109 on release-6.6.
/cc  @amyblais
NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Pull Request successfully merged
SHA: b1ce8a9e7a73bccc4346a2e322b72f9daa72de09


Question: Why have you added SDWebImage as a dependency for your SDK ?

We are trying to reduce dependencies on ObjC libraries in our codebase to improve launch times. We switched over to Kingfisher which does a great job and is completely in swift.  But since SDWebImage is a dependency for your SDK. We in effect have 2 image caching libraries in our app.
Questions:

Is there a big enough use case to include an image caching library?
Why not use a swift 3rd part dependencies?

We might have to consider dropping back to SDWebimage

Hi @sudeepdunzo
Apologies for the delay response.
Answers to your queries:

We are using the SDWebImage cache library in our engagement channels. For example, the App Inbox uses SDWebImage for Image Cache and async loading for the image. Another Example would be In-Apps that support gifs.

Currently, our SDK is written in Objective-c language. The reason why we can't use a swift dependency.

Having said that, we will definitely consider using the Swift dependencies when we write our SDK in the Swift language.
Thank you!


[Bug]: It still shows 'Connection' rather than 'DataSource' in operation records

ODC version
4.2.4
OB version
not related.
What happened?
Since ODC 4.2.4 introduced 'DataSource' to an alternative with 'Connection', the operation records still show 'Connection'.

What did you expect to happen?
It shows 'DataSource' instead of 'Connection'.
How can we reproduce it (as minimally and precisely as possible)?
Open operation records.
Anything else we need to know?
No response
Cloud
No response

Fixed In Commit 77ad728956062508dda106afe1e3b4f6caa870f5
------
陈朝俊
Fri Mar 29 09:31:54 2024 +0800

There are still issues


Feature Request: Additional option for late penalty calculation

Use case: I applied the settings so that 20% are deducted for every 48 hours for upto 48 hours (effectively, if students are one second late or 48 hours, they lose 20%).
This results in a score for the students that seems to be equivalent to: max(student_score - deduction_if_applicable, 0)
What I had hoped it would do is: (student_score * (1 - deduction_if_applicable) ) .
So if a student would have scored 5%, they currently end up with a 0, while I had hoped they'd end up with 4%.
Maybe I there's already a setting that does this, but I couldn't find it.
So I am requesting this as a new feature.

@mishaschwartz I'd like to +1 this request. It would be an ideal setting for me as well.


drone.telemetry.scaled_pressure() hangs

Hi there,
I am trying to get the atmospheric pressure from the drone but the code hangs forever at the async call without any error.
# get atmospheric pressure at takeoff location
async for pre in drone.telemetry.scaled_pressure():
    pressure=pre.absolute_pressure_hpa
    print("Pressure at takeoff location: %.2fmbar" %(pressure))
    break

Using mavsdk Python on Windows
PX4 mini and mavsdk have been updated to latest version
thanks for help

If you never receive any scaled_pressure in the async for, it suggests that PX4 does not send them. Could it be?

That makes sense! Do you know about any plans to make those available to telemetry? Havent found any parameter to turn them on...

There's no parameter to enable it. They are supported but not sent by default. You will have to request the autopilot to stream the messages using MAVLink MAV_CMD_SET_MESSAGE_INTERVAL or eanbling them by default using the mavlink module commands: https://dev.px4.io/master/en/middleware/modules_communication.html#mavlink.

found it, thank you. But the drone's barometric pressure is about 4m different than my ground station, so this is not very useful...

@olafwessler you might be able to correct/calibrate the baro altitude using a param. It might be this one: https://docs.px4.io/master/en/advanced_config/parameter_reference.html#SENS_BARO_QNH

@julianoes I am using the absolute pressure value from both drone and ground station. So applying the QNH calibration to both devices will not eliminate the difference..
By the way: As it is possible to receive pressure data from mavsdk and to set a calibration value by parameter, would it make sense to also allow to activate the data stream itself from PX4 by parameter?

I think you would only apply the calibration to one but maybe I don't understand the problem correctly. I just assume that these pressure sensors are not perfectly calibrated which is probably what you're finding.

activate the data stream itself from PX4 by parameter?

You mean so that you don't have to use set_rate_scaled_pressure()?

@julianoes The QNH calibration is to adjust the pressure-to-altitude conversion according to local weather conditions. It does not affect the absolute pressure value. My goal was not to get the altitude based on pressure but to get a relative height difference between ground station and drone. So I am using absolute pressure values on both ends.
But indeed acording to specs pressure sensors can have a tolerance of +/-10m.
Yes it would be good to be able to activate set_rate_scaled_pressure() by a parameter. The pressure value from mavsdk.telemetry cannot be used without the PX4 streaming this value. And to those only using the mavsdk without detailled knowledge of mavlink, a parameter would be more conveniant..

I was hoping the set_rate commands are convenient enough. That's why there here. The problem is that otherwise we need to add a parameter for each and every MAVLink message because everyone is going to want a specific one. :smile:

I thought something like the SENS_EN_MB12XX parameter to activate a distance sensor would be very convenient. In particular, as the barometric pressure sensor is a standard build-in sensor in every pixhawk and the altitude value is already supported by mavsdk.
But I totally understand its getting too much at some point...


libusb: Add support for windows, version 1.0.24 and fix includes

I need help with this:

I need to select the right libusb_*.sln depending on the most recent VS installed.
What was the purpose of the add_includedirs directive? From what I saw in debian packages you should include libusb-1.0/libusb.h to use it.
I couldn't find a way to build only the shared or static library so for now I build both and then copy the right files.

see https://github.com/xmake-io/xmake-repo/pull/278


Multiple JOIN aggregation

Hi,
I'm looking for support when it comes to doing multiple JOIN aggregation. Currently I gave up and wrote raw sql:
		rows, err := db.Client.Debug().QueryContext(context.Background(),
			`
			select
				players.name,
				players_with_profits.profit as "profit",
				players_with_experience.experience as "experience",
				players.level,
				players.vocation,
				players.world,
				g.name as "guild"
			from
							players
			left join (
				select
								players.id as id,
								sum(pdp.profit) as profit
				from
								players
				join player_daily_profits pdp on
								pdp.player_id = players.id
				where
								pdp.created_at >= $1
				group by
								players.id) players_with_profits on
							players_with_profits.id = players.id
			left join (
				select
								players.id as id,
								sum(er.relative) as experience
				from
								players
				join experience_records er on
								er.player_id = players.id
				where
								er.created_at >= $1
				group by
								players.id) players_with_experience on
							players_with_experience.id = players.id
			left join guilds g on
				g.id = players.guild_players
			where
				experience is not null and profit is not null
			order by
				experience desc,
				profit desc
			LIMIT $2`, beginningDay, limit,
		)

But I would prefer to stick to ent's built in queries interface. Is it currently possible to break down this query so it contains as little raw sql as possible?

Hey @Marahin  , have you had a look at these examples https://entgo.io/docs/predicates/#get-all-users-with-a-tesla-car ?


Can't update image override by changing override.value

This doesn't work although the docs suggest it should (https://developer.sketchapp.com/reference/api/#symbol-override):
Given a selected symbol layer...
let imageOverrides = selectedLayer.overrides.filter(override => override.property === 'image');
let override = imageOverrides[0];
let image = new Image({image: '/Users/giles/Desktop/icon.png'});
override.value = image.image;

Both override.value and image.image are of type ImageData
The image in the override disappears instead of being replaced and the top area of inspector panel disappears too.

just so you know, let image = new Image({image: '/Users/giles/Desktop/icon.png'}) creates an image layer. You should be able to set the override directly to '/Users/giles/Desktop/icon.png' (once it is fixed)


Testcases MiniLab

How will we receive the fhir files during the MiniLab? What is the procedure? Is it by using the send button in the test application?

@DiemerVD
Indeed!


Add textDocument/foldingRange Provider

✅ Status: Ready for Review
This PR is basically where I envisioned it, aka it's ready for review.
Description
This PR creates a folding range provider. See here for more about folding ranges:
https://microsoft.github.io/language-server-protocol/specifications/specification-3-15/#textDocument_foldingRange
Background
I recently raised an issue here:
https://github.com/elixir-lsp/vscode-elixir-ls/issues/170
The tl;dr is that folding ranges stopped working, but as ElixirLS was relying on the default folding ranges implementation to create them, there was no easy way to fix the situation.
@wingyplus made some progress on a home-grown implementation a few months back:
https://github.com/elixir-lsp/elixir-ls/issues/358
But they did not have the time to devote to finishing the work. I wanna thank them for getting me started -- I took their code as my starting point so I want to make sure they get some credit.
Methodology
High level
I make multiple passes (currently 4) through the source text and create folding ranges from each pass. Then I merge the ranges from each pass to provide the final ranges. Each pass gets a priority to help break ties (the priority is an integer, higher integers win). See:
apps/language_server/lib/language_server/providers/folding_range.ex
Indentation pass (priority: 1)
I use the indentation level -- determined by the column of the first non-whitespace character on each line -- to provide baseline ranges. All ranges from this pass are kind?: :region ranges.
Comment block pass (priority: 2)
I make a "comment block" -- consecutive lines that start with # -- to form regions. All ranges from this pass are kind?: :comment ranges.
Token-pairs pass (priority: 3)
I use pairs of tokens, e.g. do and end, to provide another pass of ranges. All ranges from this pass are kind?: :region ranges.
Heredocs (priority: 3)
I find heredocs in a pass. Ranges from this pass are either kind?: :comment if the heredoc is an @doc or @moduledoc heredoc or kind?: :region otherwise.
To Do

[x] Heredoc pass: kind?: :comment | :region
[x] Comment block pass: kind?: :comment

Any others?

|> chains: kind?: :region?
Import blocks: kind?: :imports?

Testing
Run
$ mix cmd --app language_server mix test test/providers/folding_range_test.exs --color

What to look out for when reviewing

Overall approach: Does the multiple passes with priority makes sense?
Speed: I tried to make each pass O(n) in the length of the source text. But I don't know how to benchmark my results. ~There's also some extra Enum.sort()s that we'll want to drop. I just got lazy with the tests.~ This is fixed now.
Typespecs: Overkill?
Idiomatic: Does this PR clash with the style used in the rest of the project?

Wow @billylanchantin, this is relly impressive. I left a few small remarks and I'm going to give it some testing.

Thanks, @lukaszsamson! I didn't realize how much I leaned on code folding until I didn't have it, so that was good motivation :)
Let me know how the tests go! I checked this implementation on some complicated codebases locally in addition to the tests in folding_range_test.exs, and everything seemed to work. But that's hardly exhaustive.

 Lots of good suggestions in your review :)

That's a really nice addition to the language server, code was fun to read!

Thanks!

One thing I didn't comment in-code is about the multiple instances where you do single-function pipes (eg result = input |> some_function()). I think in those cases it's more clear to just write the function call the "conventional" way (ie without a pipe).

Yeah this is a quirk of mine (@benwilson512 calls me out on it a lot 😛). It's one of those things that I think most developers grow out of after they get over the initial excitement at having the |> operator at their disposal, but I never did. I actually believe I can make a case for it not being such a bad choice. But it's certainly not the norm (and I'm inconsistent myself). I'll try to make a pass to address it soon.

That said I don't really know if that style is used elsewhere in the project.

Nor I. It seems exhausting to check haha.

I did some testing and I have to say it's pretty solid. I didn't notice any crashes and the performance is acceptable (at least on my machine). Great work @billylanchantin. So far I was able to find only a few small issues

heredocs with sigil ~S""" are strangely folded

defmodule SomeModule do
  @doc ~S"""
  Does something.

  ## Examples
      iex> SomeModule.some_fun(1.0)
      1
  """
end

multiline ordinary strings don't seem to work

"
a
multiline
string
"

charlist heredocs don't work

'''
a
charlist
heredoc
'''

multiline charlists don't work

'
a
multiline
charlist
'

IMO we should also support binaries

<<2,3,4,
  4,5,
  6,4>>

sigils don't work very well, e.g

~s(
    asd fdas fd
  dfs sdf
  sdf)

I'm not sure if we can/should support all 8 of sigil delimiterss though
~r/hello/
~r|hello|
~r"hello"
~r'hello'
~r(hello)
~r[hello]
~r{hello}
~r<hello>

@lukaszsamson Thanks for pointing these out!
Issues you've raised:

[ ] Sigil heredocs
[ ] Multiline regular strings
[ ] Multiline charlist strings
[ ] Binaries
[ ] All sigil delimiters

First bit of good news: Looks like binaries can be covered by a token pair. That's a one-line change (plus a test).
Now some bad news: I forgot to consider sigils or multiline regular/charlist strings at all, and they break a few of my assumptions. It won't be a tiny patch to fix these like I'd hoped.
Second bit of good news: Token.tokenize/1 does a lot of the work for me. Now that I know to look for these cases, I think I'll be able to transform the heredoc pass into a "special tokens" pass to cover everything else. The bones of how I approached heredocs should apply to these other cases.

Hm, older versions of Elixir all seem to be failing on test
test/providers/folding_range_test.exs:462

Any ideas why?

Any ideas why?

It looks like there is a slight difference in tokenizer output in older versions:
1.9
{:sigil, {2, 14, nil}, 83,
    ["I'm a @moduledoc heredoc.                            # 2\n"], [],
    "\"\"\""},

1.11
{:sigil, {2, 14, nil}, 83,
    ["I'm a @moduledoc heredoc.                            # 2\n"], [], 2,
    "\"\"\""},

@lukaszsamson Thank you for the insight! That seems to have fixed everything.
So here's where we stand:
I think the functionality is basically there. I have a few quibbles that we could address:

[ ] The {:ok, ranges} pattern isn't really used correctly since the sub-range providers default to an [] if there was an issue. Perhaps we want the sub-range-providers to return an {:error, reason} instead so it's clearer what went wrong, then override the result with an [] in FoldingRange.do_provide/1.
[ ] Documentation. Some of the module docs aren't filled out.

But we could ignore both those changes. Or we could make them in a follow on PR. It's up to how quickly y'all want to get this out the door.

@lukaszsamson Thank you for the insight! That seems to have fixed everything.

I'm glad I could help. I went through commit history of :elixir_tokenizer' and this seems to be the only significant change between 1.8 and 1.11. There are some heredoc and /r/n` related changes on master though. Let's hope they not break anything.

The {:ok, ranges} pattern isn't really used correctly since the sub-range providers default to an []

Is there any value in returning errors here?

Documentation. Some of the module docs aren't filled out.

IMO it's good enough. We're not likely to need it pushed to hex docs.

There are some heredoc and /r/n related changes on master though. Let's hope they not break anything.

Agreed. Although code folding already I don't mind patching if more issues come up.

The {:ok, ranges} pattern isn't really used correctly since the sub-range providers default to an []

Is there any value in returning errors here?

No, not really other than debugging and testing. I'd be fine dropping the whole {:ok, ranges} thing and just having all provide_ranges implementations return a list no matter what.
I suppose I pointed it out because when I see {:ok, something}, that signifies to me that it's possible {:error, reason} can happen as well. Since that can't happen here, I was wondering if we should either add errors or drop the tuples.

Thanks for this! I just wanted to drop a note that I am working my way through reviewing this PR, and it's looking great so far so I don't expect to request any major changes.

Here's the PR: https://github.com/billylanchantin/elixir-ls/pull/1

Thanks again for this :heart:!

@axelson You all are welcome! Sorry for the delay. I'd wanted to merge your changes in for a while, but I messed up my local build and had trouble getting things working again on my end. That and some life stuff sapped up all my free time.
So glad we got this across the finish line!

No worries! Life happens to all of us (definitely including myself, haha). I'm quite excited for this functionality and I'm hoping to get it out in the next release soon. Maybe next weekend.


v1.111 Snowball Survival

Thanks for the 112 update, but I've noticed the pre-app-update rounds of the new round are added as a new round. I assume they can be converted as was the case in previous updates when new rounds were added?

Should be fixed on v1.113

Thanks


Appending Tensorflow Layers to Pre-Trained Keras Models

I am trying to append a model to the pre-trained ResNet-50 using Tensorflow.  This is just an example to replicate the error.  I am trying to initialize all trainable parameters without erasing the pre-trained ResNet-50 weights.  I am using Python 3, and the latest version of Tensorflow and Keras.
import tensorflow as tf
from keras.applications.resnet50 import ResNet50

ckpt_path = 'example'

input_t = tf.placeholder(tf.float32, shape=(None, 224, 224, 3))
training = tf.placeholder(tf.bool)

with tf.variable_scope('new_model'):
    conv1 = tf.layers.conv2d(input_t, 3,
                             kernel_size=(3,3),
                             padding='same',name='conv1',
                             kernel_initializer=tf.glorot_normal_initializer())

    bn1 = tf.layers.batch_normalization(conv1, fused=True, training=training,
                               name='bn1')

    trainable_vars = tf.get_collection(tf.GraphKeys.TRAINABLE_VARIABLES,
                                           scope="new_model")
    resnet = ResNet50()(bn1)

    with tf.Session() as sess:
        saver = tf.train.Saver()
        sess.run(tf.variables_initializer(trainable_vars))
        saver.save(sess, ckpt_path+'/model.ckpt', 0)

I use BatchNormalization in my model, so I get the following error:
Traceback (most recent call last):

  File "<ipython-input-1-449753a374eb>", line 1, in <module>
    runfile('/home/rmk6217/Documents/ccbb_tf/example_error.py', wdir='/home/rmk6217/Documents/ccbb_tf')

  File "/home/rmk6217/anaconda3/lib/python3.5/site-packages/spyder/utils/site/sitecustomize.py", line 710, in runfile
    execfile(filename, namespace)

  File "/home/rmk6217/anaconda3/lib/python3.5/site-packages/spyder/utils/site/sitecustomize.py", line 101, in execfile
    exec(compile(f.read(), filename, 'exec'), namespace)

  File "/home/rmk6217/Documents/ccbb_tf/example_error.py", line 38, in <module>
    saver.save(sess, ckpt_path+'/model.ckpt', 0)

  File "/home/rmk6217/anaconda3/lib/python3.5/site-packages/tensorflow/python/training/saver.py", line 1594, in save
    raise exc

  File "/home/rmk6217/anaconda3/lib/python3.5/site-packages/tensorflow/python/training/saver.py", line 1573, in save
    {self.saver_def.filename_tensor_name: checkpoint_file})

  File "/home/rmk6217/anaconda3/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 889, in run
    run_metadata_ptr)

  File "/home/rmk6217/anaconda3/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1120, in _run
    feed_dict_tensor, options, run_metadata)

  File "/home/rmk6217/anaconda3/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1317, in _do_run
    options, run_metadata)

  File "/home/rmk6217/anaconda3/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1336, in _do_call
    raise type(e)(node_def, op, message)

FailedPreconditionError: Attempting to use uninitialized value new_model/bn1/moving_mean
	 [[Node: new_model/bn1/moving_mean/_4 = _Send[T=DT_FLOAT, client_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device_incarnation=1, tensor_name="edge_7_new_model/bn1/moving_mean", _device="/job:localhost/replica:0/task:0/device:GPU:0"](new_model/bn1/moving_mean)]]
	 [[Node: new_model/res3a_branch2c/kernel/_499 = _Recv[_start_time=0, client_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device_incarnation=1, tensor_name="edge_254_new_model/res3a_branch2c/kernel", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:CPU:0"](^_arg_new_model/save/Const_0_0, ^new_model/save/SaveV2/tensor_names, ^new_model/save/SaveV2/shape_and_slices)]]

Is there a way of initializing the moving mean's without using tf.global_variables_initializer()?  When I use that, it runs without error - but it deletes the pre-trained ResNet-50 weights.  Any thoughts would be appreciated.

This is a complicated TF workflow to do something that Keras makes really simple. I recommend you use Keras layers instead. Also please avoid using variable_scope in your code, whether it is raw TensorFlow or a mixture of TF and Keras. This will not do what you think it does, and it is guaranteed to lead to bugs.
Additionally, avoid relying on global collections in TF. Global shared states are also a common source of bugs.
This is probably what you want to do (much simpler, not relying on esoteric APIs):
import tensorflow as tf

input_t = tf.keras.Input(shape=(224, 224, 3))
conv1 = tf.keras.layers.Conv2D(3,
                        kernel_size=(3, 3),
                        padding='same',
                        name='conv1',
                        kernel_initializer=tf.glorot_normal_initializer())(input_t)

bn1 = tf.keras.layers.BatchNormalization(fused=True, name='bn1')(conv1)
resnet_output = tf.keras.applications.ResNet50()(bn1)

# Instantiate model so you have access to saving, listing weights, etc.
model = tf.keras.models.Model(input_t, resnet_output)
# these are your trainable weights: `model.trainable_weights`
# this is your input placeholder(s): `model.input`
# etc

# Save TF checkpoint
# (note that you are actually better off using `model.save(fname)`, the Keras API)
sess = tf.keras.backend.get_session()
saver = tf.train.Saver()
saver.save(sess, ckpt_path + '/model.ckpt', 0)

Hope this helps! Remember: Keras features are meant to help you, so sparse your sanity and use them.

The sess = tf.keras.backend.get_session() was EXACTLY what I was looking for.  This works perfectly...thanks @fchollet!


Installing error, subprocess-exited-with-error

I've already reinstalled the Python 3.8.9, and tried to install chatterbot by pip and this error keeps showing up.

I am finding exactly the same! Did you manage to fix this issue? If so please can you advise me what you did? Thanks.

i've just gave up lol


Update share.js

Fix content not secure (http -> https)

good catch, this should probably just use whichever protocol the page is using with just //fonts.googleapis.com.... instead of https://fonts.googleapis.com.....

hey, thanks for this PR, i updated the coffeescript source in #142. i changed my mind and decided on using https://, not just //, because i can't think of a reason why it shouldn't always be https


Add ViTamin models

Add the ViTamin model, which is trained on public DataComp-1B using OpenCLIP framework and obtains 82.9% zero-shot ImageNet-1K accuracy with 436M parameters. It achieves the state-of-the-art performance on zero-shot image classification, multi-modal retrieval, open-vocabulary detection and segmentation, and large multi-model models.
The code of ViTamin models are modified from vision_transformer_hybrid.py in the timm codebase.
This ViTamin work has been accepted to CVPR 2024 (https://arxiv.org/pdf/2404.02132).

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@Beckschen thanks, probably a few more changes before the tests pass, if you get stuck I can help in a few days, for starter current failure, the dataclass init needs to use the default factory pattern as here: https://github.com/huggingface/pytorch-image-models/blob/main/timm/models/maxxvit.py#L137`

Thanks very much, Ross! I have fixed the dataclass init problem.

@Beckschen this required more changes, including an addition to the base vit model for xlarge (disable pos embed). I think it's working now but haven't done extensive checks... can add support to OpenCLIP now fairly easily, easier to verify it's correct there.

I'm truly grateful for your help, @rwightman ! I saw there are changes regarding the compatibility with vision_transformer.py and vision_transformer_hybrid.py . Thanks again!
The version is designed to support both timm and OpenCLIP. While initiating this pull request in timm, I also started one in OpenCLIP (not being reviewed yet). Since OpenCLIP already supports timm, simply adding configs of ViTamin to model_configs should ensure the compatibility with OpenCLIP. I will initiate another pull request to OpenCLIP later.
Thanks again, @rwightman !


Fix typo

"set" or "customize" is fine, but not "set customize"

Find the CL here: https://dart-review.googlesource.com/c/sdk/+/176640

Thanks for the documentation fix! It looks good to me, reviewed it over at https://dart-review.googlesource.com/c/sdk/+/176640 and submitted it to the commit queue.

Thanks for the documentation fix! It looks good to me, reviewed it over at https://dart-review.googlesource.com/c/sdk/+/176640 and submitted it to the commit queue.


Compute highest level of SAR, SFR claimed, display with text information on web

Use extracted SAR and SFR claims, heuristically establish the likely achieved value (highest) and visually display (e.g., within the minimum and maximum range levels).
Example:
https://seccerts.org/cc/861434b03ddcac64/
Security target
Security Assurance Requirements (SAR)
ADV_FSP.5

@petrs From where should these be collected? Technically, they can appear at multiple places:

Security target keywords
Certification report keywords
In the linked protection profiles. For instance, some certificate links to a protection profile for which we have a processed sample in pp_dataset.json. That PP can also list SARs that need to be parsed on their own...
Directly in the security_level field parsed from csv, e.g.,

"security_level": {
                "_type": "Set",
                "elements": [
                    "ALC_FLR.3",
                    "EAL3"
                ]
},
```.

If there's a conflict in the SARs, how should the conflict be resolved?


Workbox and Notifications Spam

I'm always watching videos done by the highly skilled Google Chrome dev team. I watched a video a few weeks ago on Service Workers and Notifications Spam.
Basically the spot that came across to me was the Googler saying that if you add notifications to all your web pages on your website you are mostly likely just going to spam the User. For example, if a User comes to your home page and before they start to do anything they get bombarded with a pop-up asking them to accept Notifications. This to me is bad User experience! Plus the Notifications  that website will send at a later time will most likely be really poor quality to the User.
The Googler gave an example saying if the User has gone to a flights website and filled out the contact form and then gets a Notification to tell the User that the form has been processed that is a good User experience.
To me I like making websites with this kind of User experience! Which got me thinking and why I am writing this issue.
To be honest the way Service Workers are set up they are made to Spam the internet and I would like to see a feature in Workbox where a User could go to a home web page and not see the Notification pop-up. But only see the Notification pop-up after filling out a form and clicking on an option to receive the Notification.
So being able to whitelist certain web pages or folders etc. and turning off all the rest of the website thus not annoying Users.
I be interested to see what people suggest or how to properly set this up.

Hello @ayumihamsaki!
I understand and share the frustration you feel when visiting websites that show Notification prompts too aggressively.
Unfortunately, sharing that feedback in the Workbox issue tracker is not likely to lead to any changes in that behavior. I do think it's outside the scope of what we can accomplish with Workbox.
I'd encourage you to share specific ideas for browser-level changes via the corresponding issue tracker for each browser, e.g. https://crbug.com for Chrome-related changes.


Fix the flaky issue in ParallelInterleaveDatasetOpTest

This PR fixes the flaky issue in ParallelInterleaveDatasetOpTest. We reset the iterator before restoring it so that all the resources/tasks the old iterator holds can be released.
cc:@jsimsa

Should all roundtrip tests include this fix?

Yes, the roundtrip test for ParallelMapDataset has a similar flaky issue. Will add this fix to all other roundtrip tests soon.

This fix is added to all roundtrip tests via https://github.com/tensorflow/tensorflow/pull/27805/commits/edb84f20f105df979a372784e152de728b905a9d.

Good suggestion! I change it via this commit (https://github.com/tensorflow/tensorflow/pull/27805/commits/9a86868aa720c0e4e19f3fddf06772193d2505bd). Could you please have a look at the change?

Sorry that there is a typo issue. I just fix it (https://github.com/tensorflow/tensorflow/pull/27805/commits/bbed106374e5db95057f19fd17113811bd85c4b3). @rthadur Could you please re-trigger the test?

The test failure in Linux GPU is unrelated.