[Feature Request] Disable/Change frequency of GraphQL Introspection Queries

Insomnia Version: 6.0.2
Operating System: Mac OS 10.14

Details
Please add a feature to lower the frequency of (or disable entirely) Insomnia sending introspection queries to the URL. I'm in the process of creating and debugging a GraphQL API using AWS Lambda+Serverless, and every time I make a call to my local instance, GraphQL follows it up with an introspection query, which produces a massive literal wall of text in the log output making debugging incredibly frustrating. I've been able to work around the issue by filtering out introspection queries on the API side, but that doesn't stop Insomnia from trying anyway and producing error popups every time it does. And when I'm not actively debugging the API itself, the autocomplete feature is nice to have, but it's annoying to have to keep going into the API to turn the filter on and off.
Proposed features

Add an options menu for GraphQL, either as part of the preferences window or in the GraphQL tab (perhaps next to the "Prettify GraphQL" button).
Add an "Enable Automatic Introspection" check box that, when unchecked, disables the automatic introspection.
Add an "Enable Introspection After Query" check box that, when unchecked, disables the introspection query made after a normal GraphQL query.
Add a "Change Introspection Interval" text field that accepts a positive integer for how many seconds pass between introspection queries. (This text box is grayed out if "Enable Automatic Introspection" is unchecked.)

I'm not sure what behavior the autocompletion should take if introspection is disabled entirely. Either it should be disabled as well, or it should use the last successful introspection that it received. Perhaps that can be a configurable setting as well.

I agree, it's a lot of fetching. It could probably be optimized a bit to not fetch so often but a manual toggle would be easy to add, and very useful.
There's already a way to manually trigger the introspection request, so I suggest just adding a "manual mode" toggle beside that for now.

+1000 for this, it's soooo annoying!  was okay when my schema was small to medium size. Now I need to increase my terminal line buffers in order to scroll back to the top see my debug message.
It's doing it after every request. I'd love to me able to manually trigger it when I need


Change log

where can we find the log of changes to the SPO cmdlets?

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

ID: 65bf2c0b-b0ec-2748-e47e-af66011efe30
Version Independent ID: be3968f4-d766-0b45-384b-c51c47328a86
Content: SharePoint Online PowerShell
Content Source: sharepoint/sharepoint-ps/sharepoint-online/sharepoint-online.md
Service: sharepoint-powershell
GitHub Login: @techwriter40
Microsoft Alias: kirks

@DeanGross - you can see the history on the GitHub file. It tracks every commit and merge. This applies for the repo as a whole and also each individual file. To see the history for a specific file, navigate to that file in the GitHub repo (tip, if you are looking at the file on docs.microsoft.com then you can click the Edit button in upper right of page to go directly to GitHub source) and then click the History button. You will see all of the commit history and can click and see changes. There are different views to see the changes but the one I prefer shows red for removed text and green for added text.
Here are some references:
https://help.github.com/articles/differences-between-commit-views/
https://git-scm.com/book/en/v2/Git-Basics-Viewing-the-Commit-History


Cursor Redesign

This commit provides a redesign of Cursor. The reasons for this redesign

ACursor is not sealed. This created some modeling and correctness issues around certain operations. For example, when attempting to create an efficient representation of a particular ACursor to the root of a JSON object (for errors and the like), we had to handle the case that the given ACursor we were working on was not a known type.
Certain useful fields such as lastCursor and lastOp could not be exposed safely as they might be null.
Certain cursor operations were not modeled in the CursorOp, e.g. withFocus, creating a difficult situation where null in the cursor history both indicated the beginning of the history, as well as an unknown operation.
HCursor exposed and relied upon operations like addOp and replace which permitted corruption of the cursor history.
Certain other methods had odd semantics (see root).

Cursor is primarily just moving all of the ACursor, HCursor, ObjectCursor, ArrayCursor, TopCursor, and FailedCursor, into a single file so we can seal it.
It provides better error information in FailedCursor (which will be addiative to decoding errors, not replace them). Safely exposes certain desirable methods, e.g. lastCursor and lastOp.
We also add a new CursorOp.Replace so we can fully record history on the cursor.
Cursor values may be converted to ACursor or HCursor values and the majority of the implementation of circe Cursors in this commit now use Cursor under the hood.

Things need to be deprecated, tests need to be added. There are also some TODO values added that need to be addressed in this or followup PRs.
@zarthross @zmccoy, assuming you guys are on board with this change (targeting 0.15.x), how much do you want in this PR vs follow up PRs. It's the tension of a complete implementation vs. smaller PRs.

Codecov Report

Merging #2004 (1bdef07) into series/0.15.x (3c74a83) will decrease coverage by 1.43%.
The diff coverage is 67.93%.

:exclamation: Current head 1bdef07 differs from pull request most recent head 76f6546. Consider uploading reports for the commit 76f6546 to get more accurate results

@@                Coverage Diff                @@
##           series/0.15.x    #2004      +/-   ##
=================================================
- Coverage          86.06%   84.62%   -1.44%
=================================================
  Files                 74       72       -2
  Lines               2554     2699     +145
  Branches             120      127       +7
=================================================
+ Hits                2198     2284      +86
- Misses               356      415      +59

Flag
Coverage Δ

2.12.15
84.48% <67.93%> (-1.50%)
:arrow_down:

2.13.8
84.98% <71.55%> (-1.19%)
:arrow_down:

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

Impacted Files
Coverage Δ

...s/core/shared/src/main/scala/io/circe/Cursor.scala
65.12% <65.12%> (ø)

...core/shared/src/main/scala/io/circe/CursorOp.scala
55.55% <66.66%> (ø)

.../core/shared/src/main/scala/io/circe/HCursor.scala
81.25% <81.25%> (-9.38%)
:arrow_down:

.../core/shared/src/main/scala/io/circe/ACursor.scala
100.00% <100.00%> (+28.57%)
:arrow_up:

.../shared/src/main/scala/io/circe/FailedCursor.scala
75.00% <100.00%> (+2.27%)
:arrow_up:

...re/shared/src/main/scala/io/circe/MapDecoder.scala
92.59% <100.00%> (ø)

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
88.30% <0.00%> (ø)

Continue to review full report at Codecov.

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

Closed in favor of #2015


add search bar to contacts screen

Adds the feature requested in #27

can you rebase @jrbuhl93 ?

Yes, I am going to look into using the react native elements searchbar instead of input as well.


removed lines introduced by commit f53b98d which break the cache purging

this is a fix for issue #104
the second continue effectively breaks cache cleanup
r,
Sam

Please accept this and push a new release to PyPI (https://github.com/danilop/yas3fs/issues/124).


Provide an MSI installer again

📝 Provide an MSI Installer
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
An MSI installer was provided for previous releases please provide the MSI installer along with the EXE installer. MSI installations are useful in enterprise deployment scenarios.
If you'd like to see this feature implemented, add a 👍 reaction to this post.

the current installer did not work. Nothing was installed.

@psc0425, can you provide logs or more detail?  the exe wraps the msi

Yes please !
At very least add a silent switch to the EXE installer !!

Bring back the MSI installer - EXE wrappers are notoriously unreliable when used in deployment systems

Another vote for an MSI installer please. We were deploying this through InTune

For me, running .exe file for 0.20 doesn't do anything. The .msi files of older versions seem to be working just fine.

Without a Silent installation option, this currently breaks the Chocolatey package.
For Chocolatey to work correctly, one of these options is required:

MSI installer (preferred)
Silent installation via parameters (e.g. by using /S , /SILENT, /VERYSILENT, --silent and so on)

yet another vote for an MSI installer. We need this to deploy via GPO

Can't get 0.20 through chocolatey anymore. Please provide an MSI installer.

done


➕ [RESOURCE] - <Add Web 3.0 Amazon Clone>

Idea Contribution

[X] I have read all the resource request issues.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Where do you want to add the resource? Specify the category and section.
i want to Add Web 3.0 Amazon Clone in web 3 project category
This project facilitates decentralized  Amazon Clone Step-By-Step with Solidity (Writing Smart Contracts & Tests), Javascript (React & Testing), Hardhat (Development Framework),Ethers.js (Blockchain Interaction), React.js (Frontend Framework).
Provide the URLs of the resources you are adding
https://youtu.be/X1ahXNYkpL8?si=6fIFKRkegDOoNO6l
Any alternative approaches/features
No response
Additional Context
No response
Code of Conduct

[X] I follow the CONTRIBUTING GUIDELINE of this project.

@jfmartinz  plz review https://github.com/jfmartinz/ResourceHub/pull/1000


Images Preloading

Hydrogen Image Preloading
Preloading above-the-fold content has a powerful effect on Largest Contentful Paint (LCP). Specially when it comes to fonts and images, as both images and text nodes can be LCP candidates. Hero images and large runs of text that are rendered using web fonts can benefit significantly from a well-placed preload hint, and should be used when there are opportunities to deliver these important bits of content to the user faster.
Current situation
Hydrogen currently does not offer a native abstraction, example or guide on how to effectively implement image preloading. Although Remix offers all the right primitives, developers have to go out of their way to recognize the importance and implementation details of preloading critical resources to improve performance.
https://github.com/Shopify/hydrogen/assets/12080141/3d370d46-8675-4195-a1f6-ffb7ae432b83
👎🏼 Main homepage hero image not preloaded

👍🏼  Main homepage hero image preloaded in the homepage

Example generated tag for skeleton's home route.

Proposal
The follow driven-development doc, demonstrates a proposal to facilitate images preloading.

[!NOTE]
A separate proposal will be made for handling web fonts and other critical resources

Code Changes
If we prefer to keep this as userland feature we should export hydrogen-react Image component inner utilities within @shopify/hydrogen

Utility
Function

generateImageWidths
Generates an array of sizes for Shopify images, for both fixed and responsive images.

generateSizes
Generates an array of widths, heights and crops needed for Imagery loader

generateSrcSet
Generates a srcSet for Shopify images based on an array of sizes

shopifyLoader
Appends a width, height and crop url query params to a given Shopify CDN url

Otherwise, we can export the genPreloadImageLinkMeta from hydrogen which internally uses the Image utilities
// src/genPreloadImageLinkMeta.ts in hydrogen-react
import {
  generateImageWidths,
  generateSizes,
  generateSrcSet,
  shopifyLoader,
} from './src/Image.ts';

type GenerateImageWidthsParams = Parameters<typeof generateImageWidths>;

type GenerateSizesParams = Parameters<typeof generateSizes>;

type GenPreloadImageLinkMetaProps = {
  url: string;
  width?: GenerateImageWidthsParams[0];
  srcSet?: {
    interval: GenerateImageWidthsParams[1];
    startingWidth: GenerateImageWidthsParams[2];
    incrementSize: GenerateImageWidthsParams[3];
  };
  sizes?: {
    aspectRatio: GenerateSizesParams[1];
    crop: GenerateSizesParams[2];
  };
};

// Default props match those of the Image component
const defaultProps = {
  url: '',
  width: '100%',
  srcSet: {interval: 15, startingWidth: 200, incrementSize: 200},
  sizes: {aspectRatio: '1/1', crop: 'center'},
} as const;

export function genPreloadImageLinkMeta({
  url,
  width = defaultProps.width,
  srcSet = defaultProps.srcSet,
  sizes = defaultProps.sizes,
}: GenPreloadImageLinkMetaProps) {
  // Assign default values if not provided
  const interval = srcSet?.interval ?? defaultProps?.srcSet?.interval ?? 15;
  const startingWidth =
    srcSet?.startingWidth ?? defaultProps.srcSet.startingWidth ?? 200;
  const incrementSize =
    srcSet?.incrementSize ?? defaultProps.srcSet.incrementSize ?? 200;
  const aspectRatio =
    sizes?.aspectRatio ?? defaultProps.sizes.aspectRatio ?? '1/1';
  const crop = sizes?.crop ?? defaultProps.sizes.crop ?? 'center';

  const widths = generateImageWidths(
    width,
    interval,
    startingWidth,
    incrementSize,
  );
  const imagesizes = generateSizes(widths, aspectRatio, crop);
  const imagesrcset = generateSrcSet(url, imagesizes, shopifyLoader);

  return {tagName: 'link', rel: 'preload', as: 'image', imagesrcset, href: url};
}

Implementation
On each applicable route, import the genPreloadImageLinkMeta utility and call it if a preload prop is present
in the loader return
// app/routes/product.$handle.tsx

+ import {genPreloadImageLinkMeta} from '@shopify/hydrogen'

export const meta: MetaFunction<typeof loader> = ({data}) => {
  const metas = [
    {title: `Hydrogen | ${data?.product.title ?? ''}`},
  ] as MetaDescriptor[];

+ if (data?.preload.image) {
+   const preloadImageLink = genPreloadImageLinkMeta({
+     url: data.preload.image.url,
+   });
+   metas.push(preloadImageLink);
+ }
  return metas;
};

export async function loader({params, request, context}: LoaderFunctionArgs) {
  // ...other code
  return defer({
    product,
    variants,
+   preload: {image: product.selectedVariant.image},
  });
}

[!NOTE] Repeat for any other routes requiring it such as index (main Hero), collection (first 4 products)

Additional Reading

Preloading responsive images
Preload critical assets to improve loading speed

Interesting take @frandiox
I initially tried using the Link header but AFAIK it does not support imagesrcset (to handle responsive images) which is critical in our use case, because our Image component uses it. e.g we don't know that far in advance what image source the browser will need for the given viewport.

Yeah, I imagine the src set headers could get huge.  Why not just:

Great point! Initially I was going to write two proposals one including a  at the root, but then I reverted back considering we just deprecated the  component in favor of using straight up meta.
I will update the implementation to eliminate the extra return from the loader.

I guess then there's no way to preload images for sub-sequent navigations? 😞

I was trying to think through this at the burst, and AFAIK there's no way to do it. Unless we simplify the image loading to not use imagesrcset, which is also undesirable.


fix: fox/eth polling

Description

Notice

[ ] Have you followed the guidelines in our Contributing guide?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #2825   👈

PR #2826

This comment was auto-generated by Graphite.


NotifyPublish seems hardly usable

Since amqp091.NotifyPublish tag is an incremental id that is bound to the amqp091.Channel, we cannot identify which message we receive a publish notification for after an automatic reconnect (since the tag is silently reset to 0).
This is an issue linked to how amqp091 secretly handles this id.
If possible, it would be better for this lib to associate the amqp091.Channel tag to a unique rabbitmq.Publisher tag (that is incremented each time we publish without error). That would remove the impact of re-connection, so the original behavior is kept.
Possible solution:

have an increment int in channelManager that is incremented on every successful publish when NotifyPublish is on
have a startIncrement int in channelManager that is equal to to the increment+startIncrement of the previous channel in case of re-connection.
when transmitting a NotifyPublish confirmation, add the startIncrement to the tag provided by amqp091.Channel's Confirmation.

This should ensure the original behaviour is kept even with reconnection.

Resolved in v0.8.1 with new "ReconnectionCount" field.


Script base templates for new scripts

An option to allow new scripts to be started from a base template when adding a new script using your $EDITOR instead of specifying the command in the commandline.
The template would really just be a dumb file that is inserted into the new script as a base, an example base template could look like this:
 tmpfile=$(mktemp)
 trap 'rm "$tmpfile"' 0 1 15

 # --- continues ---

Flag short / long name
I would suggest -T / --template since tag is already taken.
How would this look in practice?
A pier command using the templating flag could perhaps look something like this:
pier add -a my_alias --template sh-tmpfile

Abandoned.


[Bug]safari browser Unable to open，not support static {...}

Describe the Bug
safari browser Unable to open https://viewer.ohif.org/
Steps to Reproduce
safari browsers Error unable to open https://viewer.ohif.org/
The current behavior
error info: SyntaxError: Unexpected token '{'
The expected behavior
Normal open https://viewer.ohif.org/
and Normal open Tag<v3.8.2>
OS
macOS 12.0.1
Node version
16.20.2
Browser
Safari 15.1

it opens for me

update safari?

@sedghi First of all, thank you for your reply。Because this issue has indeed occurred on both my Mac and phone, the issue should be whether the browser supports the writing style of "static {...}", and whether it is possible for Babel to further optimize it

And I have tried many people's phones (including iPhone and Android), all of which have black screens, so I think it is a serious problem

I think it's because it didn't take effect: @babel/plugin-transform-class-static-block

I have also seen other users mention this issue (https://github.com/OHIF/Viewers/issues/3941)

@sedghi  I asked https://github.com/cornerstonejs/cornerstone3D The tsconfig.esm.json in the core and tools have both been added with module: 'es2015' and target: 'es2015', and Safari will eventually be able to access them. I hope to synchronize or use other methods, which is very important to me. Thank you again

Hmmm, how old is this safari? I was thinking to even target es2022 ...

Can you check the latest OHIF viewer, which now includes Cornerstone 3D 2.0?
Here are the migration guides:
Try OHIF locally: https://viewer-dev.ohif.org/localbasicTry Cornerstone3D 2.0 locally: https://www.cornerstonejs.org/live-examples/local.html
https://docs.ohif.org/migration-guide/3p8-to-3p9/


feat: Lint pr titles

The overall goal of this PR is to help improve the process of our release notes. Given that we use the titles of the squash commits on main to generate our changelog, improving the readability of our PR titles will hopefully allows us to spend less time manually fixing the names of each commit. Requiring the use of prefixes in our PR titles will eventually also allow us to automatically categorize the commits into the respective categories (like "Enhancement", "Fixes", "Docs", etc) but for now, will make it, in general, easier to organize our commits manually.
That said, I am aware that for cases where there is only one commit (not including merge commits), GitHub will use the commit message as the title for the squash commit. So this doesn't fix that issue. However, a majority of our PR's are multiple commits where the PR title will be used as the title for the squash commit so hopefully this will still help make release notes a bit easier.
For all outside contributors, this workflow will require our approval to run so that we only run this workflow on trusted code. If we approve the workflow and then the contributor pushes up new code, the workflow will require our reapproval to run again. Regardless if a contributor has contributed before, it will require our approval. This is one of the easiest ways to mitigate any bad actors from attempting to inject any malicious code.
Outside of the PR, I've written up a "PR Naming Guide" which I hope to publish on our Github wiki so that outsider contributors can reference it and understand these new requirements. I will also link that wiki in this PR so that if there is an error, you can click on the failed workflow and there will be a link to take you to the guide. (I have yet to finish writing this guide but that's the vision of it).
✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

How is this combining with the explorations that @devongovett was doing? I'd like to know the path we're taking forward

How is this combining with the explorations that @devongovett was doing? I'd like to know the path we're taking forward

I guess both work on making our changelog more readable but they don't really interfere with each other if that makes any sense. The way I see it for now is that this serves as an interim solution as Devon works on adding changesets.
This only runs on the PR after it is made on Github while what Devon is working on is ran (ideally) before pushing a PR. Each PR would contain it's own changeset which would get compiled later as the changelog once we do a release. With changesets, we wouldn't use the commit message on main to generate the changelog like we do now. The reason for linting PR titles right now is because it becomes the commit message on main. So, once the changesets gets released, this PR becomes a bit obsolete because what it solves now will  be solved by changesets but in a different way. That said, our commit messages on main will still come from the title of the PR's so I don't think it hurts to lint them even if they won't be used to generate our changelog anymore. It also doesn't take much time to run now so that's not as much of a concern anymore


Minor sample update

Minor update of the run-time performance of the RayTracer sample (now 20% faster).

On a side note, interestingly and anecdotally, looks like MS Edge is pulling ahead and is now at least 10-15% faster in executing JavaScript than Chrome and Node.js, for whatever reason. And Firefox is still way, way slower, unfortunately.

Thanks @ncave! Interesting, I'd assume Edge is using now the same V8 engine for JS as Chromium and Node.js but maybe it adds some optimizations on top.


during blog generation, github milestones are iterated over twice

Seems like once should be enough doing both checks at the same time. The second is for matching on tag, not sure what the first one is, but it routinely doesn't match.

=== UPDATE RELEASE TRAIN WIKI === then re-executes the milestone search. Could likely be cached?

Done via https://github.com/spring-cloud/spring-cloud-release-tools/commit/6efedbc324e4b94f21a6aa8bbf60dd9952009d48


Align Start/Ende to android app, fix date pattern to TT.MM.JJJJ

See https://github.com/corona-warn-app/cwa-website/issues/1153

Let's continue in #1159 👍


Upstream: D3D8 fixes that should be applied to D3D9 or DXSO too

It's possible some of the fixes in d8vk are accounting for weird behavior in d9vk / dxso but not D3D9. There may be a couple of these as D3D9 also supports vs1.1 and ps1.4.
See #43 for one possible example. D3D9 docs state that replicate swizzle should be used but DXSO seems to allow arbitrary swizzles, which may either be fixing games or breaking them.

#68 - AL16, R16, L16, AR16

There's a possibility that equivalent functionality to f974535b7058ada89cb3de886ef9687de73610e0 (EndScene on Reset) might want to be added to d3d9 if that's how Microsoft d3d9 behaves.

There's a possibility that equivalent functionality to f974535 (EndScene on Reset) might want to be added to d3d9 if that's how Microsoft d3d9 behaves.

The trace included with https://github.com/doitsujin/dxvk/issues/2184 seems to suggest that is the case, albeit I expect most d3d9 games to behave properly and not rely on it.

b2b89ee0b8f99ed35c063ad3c69649f91a66f70f - [d3d9] Initialize m_mapFlags to 0
4a7c627e3dc7d706916b870cd1e15f510c3951fd - [dxso] Implement bem instruction

Since d8vk has been upstreamed and this issue is only tracking #43 and #68 now I can close it and just deal with those on their own.


Modify validate_paths to work outside docker

If you haven't already, please read through our contributing guidelines before opening your PR
What is the purpose of this PR?
The current validate_paths function assumes that the user is within the docker and all paths are prefixed with /data. However, as we start to use these functions elsewhere and pip install the library, we don't want this assumption to necessarily be true.
How did you implement your changes
I've modified this function to give the option of dropping the /data prefix requirement, and updated all calls to validate_paths which are outside of a jupyter notebook to use this new logic.

All the ones that I found


CPT training is giving pretty unstalbe results with the learning rate 2e-5.

I am trying to conduct CPT with a mistral-instruct-v2. But every time, I notice an overshooting in the grad norm.  I tried different datasets and managed to re-produce the same issue.
I am using 8  80GB GPUs, and my effective batch size is 1024.
My config:
# Model arguments
model_name_or_path: mistralai/Mistral-7B-Instruct-v0.2
model_revision: main
torch_dtype: bfloat16

# Data training arguments
dataset_mixer:
  arcee-ai/sec-data-full: 1.0
dataset_splits:
  - train
preprocessing_num_workers: 12
text_column: "text"

# SFT trainer config
bf16: true
do_eval: False
evaluation_strategy: "no"
gradient_accumulation_steps: 64
gradient_checkpointing: true
gradient_checkpointing_kwargs:
  use_reentrant: False
hub_model_id: arcee-ai/mistral-instruct-v2-sec
hub_strategy: every_save
learning_rate: 2.0e-04
log_level: info
logging_steps: 1
logging_strategy: steps
lr_scheduler_type: cosine
max_seq_length: 4096
max_steps: -1
num_train_epochs: 1
output_dir: data/mistral-instruct-v2-sec-expanded
overwrite_output_dir: true
per_device_eval_batch_size: 1
per_device_train_batch_size: 4
push_to_hub: true
remove_unused_columns: true
report_to:
- wandb
save_strategy: 'steps'
save_steps: 50
save_total_limit: 2
seed: 42
warmup_ratio: 0.1


ArrayIndexOutOfBoundsException

Hi,
I'm using this library in my app and one of the users wasn't able to connect to their RTSP stream. I sent him a build of the latest sample app from this library to try to make sure it wasn't something I was doing wrong in my app.
I don't have more information other than these screenshots but I figured I'd post since it has an ArrayIndexOutOfBoundsException in it

also, here's the comment he made when reporting this issue.. it might help with the video type and phone OS
I just started setting up my fire hd 10 with rtsp feeds from my Unfi cameras. For some reason I keep getting a "SSL handshake timed out". I can play the feeds with VLC on the same tablet. Any idea what could be causing this? Thanks!


The resource  was blocked due to MIME type (“text/html”) mismatch (X-Content-Type-Options: nosniff).

Describe the bug
With the version 1.10.0, the web ui can't be loaded.
We have in the console the following error:
The resource from “https://********/temporal.4947095cc16fc87c0170.css” was blocked due to MIME type (“text/html”) mismatch (X-Content-Type-Options: nosniff).

To Reproduce
Steps to reproduce the behavior:

For security reason, our load balancer is configured to add the header X-Content-Type-Options: nosniff.
The main page failed to load (reproduce with chrome and firefox)

see : https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Content-Type-Options
**Expected
use mime type text/css for a css.
Screenshots
If applicable, add screenshots to help explain your problem. You can also use Loom to do short video bug reports.

Desktop (please complete the following information where relevant):

OS: [e.g. iOS] mac ox 10.15
Browser [e.g. Chrome, Safari] last version of firefox and chrome
Temporal Web Version [e.g. 1.7?] 1.10.0 (ok with 1.9.0)
Tctl Version [e.g. 1.7?] 1.10.5
are you using Docker or building from source? docker

Additional context
Add any other context about the problem here.

hate to +1 but... +1, i have this same issue today running the version from makeing the temporalio/ui-server

I get this issue when trying to enable the openapi for temporalio/ui


Document and generate docs for protected methods

This was spawned by the lack of docs on StateMachine.sessions. Let's doc these and include them in generated docs.

This should only be done for StateMachine


How to initialize weights of classifier at finetuning stage?

Dear @seasonSH ,
Would you like to teach us how to initialize weights of classifier at finetuning stage, which is not mentioned in your paper?

BTW. Can you point out DWI related code lines or blocks. I am not very familar with TF. Sorry for that

I have just uploaded the contents of DocFace+, including DIAM-Softmax. You can find the function in "src/tflib.py" (named diam_softmax). The weights are initialized in such a way that they should be distributed uniformly on the sphere. In practice, because I normalize all the weights onto the sphere every step (including first one), it is sufficient to just initialize them with a Xavier or Normal distribution.

Thanks.


ronn clone in js: marked-man

Hello,
just a note about that piece of software i maintain.
It gets some more features over the time (like recently, table support),
and uses a well-maintained markdown parser.
Cheers,
Jérémy

Looks like you forgot to link to it: https://github.com/kapouer/marked-man


Add SKU to products.json (LIFX Mini variant?)

I have a LIFX Mini (productID 59) that's not properly loaded since products.json doesn't include its metadata.  However, LIFX doesn't include this product ID in their API documentation.  I'm not sure if this is an updated SKU or variant of LIFX Mini (productID 49). The bulb was purchased as part of their multi-pack so maybe it's defined as a separate product ID?  Let me know if I should include any additional info about the light.
Model # L3A19MC08E26
Here's the payload from LifxLanParser._parsePayload (without modifying products.json):
{ vendorId: 1,
  vendorName: 'LIFX',
  productId: 59,
  productName: '',
  hwVersion: 0,
  features: null }

I'm currently working around this by appending the following in products.json:
{
  "pid": 59,
  "name": "LIFX Mini",
  "features": {
    "color": true,
    "infrared": false,
    "multizone": false
  }
}

Thank you so much for your feedback. I updated the products.json. Try the latest version.


Push Parse Server don't work

I have Parse 2.2.4 on Azure Server
I have :
serverURL: 'http://bran-parse.azurewebsites.net/parse' ,
  push: {
    ios: [
      {
        pfx: '/certificats/CertificatsDevBran.p12', // Dev PFX or P12
        bundleId: '-CRC57A3AR.BranStream',
        production: false // Dev
      },
      {
        pfx: '/certificats/CertificatsProdBran.p12', // Prod PFX or P12
        bundleId: '-CRC57A3AR.BranStream',
        production: true // Prod
      }
    ]
  }

I get this error
parse-server-push-adapter APNS
cannot find vaild connection for 9ee818d3b627b419f348f69c911982bd8cb2ae9a34cbe4123d5a0aef3a653201

I moved my key files to same directory as index.js. Seems to work but path should work as well.

Thank you for your help. But it doesn't work.
I activated VERBOSE and I have :
parse-server-push-adapter APNS
 APNS Connection 0 Connected
verb

parse-server-push-adapter APNS
 APNS Connection 0 Notification transmitted to 9ee818d3b627b419f348f69c911982bd8cb2ae9a34cbe4123d5a0aef3a653201
�[36mverbose�[39m: sent push! 1 success, 0 failures

verb

parse-server-push-adapter APNS
 APNS Connection 0 Disconnected
ERR!

parse-server-push-adapter APNS
 cannot find vaild connection for 9ee81

Do you think I have a trouble with a port ? Maybe 2195 is not open on Azure ?
I update from 2.2.4 to 2.2.6, same error...

What that means is:

parse-server-push-adapter APNS
APNS Connection 0 Notification transmitted to 9ee818d3b627b419f348f69c911982bd8cb2ae9a34cbe4123d5a0aef3a653201
�[36mverbose�[39m: sent push! 1 success, 0 failures

Your APNS configuration is correct, the connection can be established

parse-server-push-adapter APNS
cannot find vaild connection for 9ee81

This is likely that the device token don't match with the bundle ID for your certificate or some mis configuration.

I tested the certificat on Parse and it works well.
The bundle ID is ok. I don't understand..
Thank you for your help.

So that leaves the token that may have been invalidated. The push is not delivered right?

the push is not delivered and the best, the webapp crash and restart after two minutes.

What'S the crash?

Parse server seem restart if I try to send two notification push

and no logs? nothing, not a crash, an exception or anything?

no, only this error :
parse-server-push-adapter APNS
cannot find vaild connection for 9ee818d3b627b419f348f69c911982bd8cb2a

Can you try with only one certificate instead of 2, and then the other? Trying to identify which is failing

OK, thank you. I'm going to try only DEV

Try both and let me know. The APNS server is kinda picky, we have an implementation ready for the new shiny HTTP/2 APNS endpoints but HTTP/2 is not supported on node < 5. So we haven't merged it yet

log for DEV :
36mverbose�[39m: POST /parse/push { 'cache-control': 'max-age=43200',
  connection: 'keep-alive',
  'content-length': '284',
  'content-type': 'application/json',
  accept: '*/*',
  host: 'bran-parse.azurewebsites.net',
  'max-forwards': '10',
  'user-agent': 'curl/7.43.0',
  'x-parse-application-id': 'xxx',
  'x-parse-master-key': 'xxxx',
  'x-liveupgrade': '1',
  'x-arr-log-id': 'e0b12ab4-09ce-4242-b718-27717ef4de30',
  'disguised-host': 'bran-parse.azurewebsites.net',
  'x-site-deployment-id': 'bran-parse',
  'x-original-url': '/parse/push',
  'x-forwarded-for': '<IP_ADDRESS>:7443' } {
  "where": {
    "deviceType": {
      "$in": [
        "iOS"
      ]
    }
  },
  "data": {
    "title": "The Shining",
    "alert": "All work and no play makes Jack a dull boy."
  }
}

�[36mverbose�[39m: {
  "response": {
    "result": true
  }
}

�[36mverbose�[39m: sending push to 1 installations

ERR!

parse-server-push-adapter APNS
 cannot find vaild connection for 9fdac422418b4513590d05257742b4d42aa5d06b80fccf3f75761bfd5ec46ddc

2016-04-07T12:27:53  No new trace in the past 1 min(s).
2016-04-07T12:28:53  No new trace in the past 2 min(s).

Try with the prod one then.

Same error :
�[36mverbose�[39m: POST /parse/push { 'cache-control': 'max-age=43200',
  connection: 'keep-alive',
  'content-length': '284',
  'content-type': 'application/json',
  accept: '*/*',
  host: 'bran-parse.azurewebsites.net',
  'max-forwards': '10',
  'user-agent': 'curl/7.43.0',
  'x-parse-application-id': 'bran',
  'x-parse-master-key': 'xxx',
  'x-liveupgrade': '1',
  'x-arr-log-id': '4e086413-1012-46c0-abce-2296f86bf0c3',
  'disguised-host': 'bran-parse.azurewebsites.net',
  'x-site-deployment-id': 'bran-parse',
  'x-original-url': '/parse/push',
  'x-forwarded-for': '<IP_ADDRESS>:7390' } {
  "where": {
    "deviceType": {
      "$in": [
        "iOS"
      ]
    }
  },
  "data": {
    "title": "The Shining",
    "alert": "All work and no play makes Jack a dull boy."
  }
}

�[36mverbose�[39m: {
  "response": {
    "result": true
  }
}

�[36mverbose�[39m: sending push to 1 installations

verb

parse-server-push-adapter APNS
 APNS Connection 0 Connected
verb

parse-server-push-adapter APNS
 APNS Connection 0 Notification transmitted to 9fdac422418b4513590d05257742b4d42aa5d06b80fccf3f75761bfd5ec46ddc
�[36mverbose�[39m: sent push! 1 success, 0 failures

ERR!

parse-server-push-adapter APNS
 cannot find vaild connection for 9fdac422418b4513590d05257742b4d42aa5d06b80fccf3f75761bfd5ec46ddc
verb

parse-server-push-adapter APNS
 APNS Connection 0 Disconnected
verb

parse-server-push-adapter APNS
 APNS Connection 0 Connected

with only PROD, the webapp doesn't crash !

ok.. that's good to know.... Also, there seems to be a mismatch between the certificates and the device token...

Ok, So why does it work with parse normal ?
With prod I need to installed the app with test flight ?
Why the app crashes in dev?
Sofiane

Le 7 avr. 2016 à 14:45, Florent Vilmart<EMAIL_ADDRESS>a écrit :
ok.. that's good to know.... Also, there seems to be a mismatch between the certificates and the device token...
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

I'm short of answers here, I'll do my best to investigate. Many users have the push successfully configured on their system, did you try running on your local machine, to see if that would make a difference?

Not yet.
I have several solutions installed on AZure. I hope I will be not a problem.
Thank you for your help
Sofiane

Le 7 avr. 2016 à 14:59, Florent Vilmart<EMAIL_ADDRESS>a écrit :
I'm short of answers here, I'll do my best to investigate. Many users have the push successfully configured on their system, did you try running on your local machine, to see if that would make a difference?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

Try on your local machine, maybe add some logs in the push adapter itself to maybe have a better understanding of the errors

So I confirm. It works on PROD with a beta Test Flight.
I will not use DEV for push notifications. We can closed this topic.  Thank you

zalright!

I have exactly the same that App crashes and Push cannot be delivered. The log is pretty similar.

Sofianebenabdallah: I will not use DEV for push notifications. We can closed this topic. Thank you
@flovilmart
flovimart:zalright!
seb : can't help thinking this is wrong...
so you don't test in dev and just use prod environment and prod mongo db?
uhmmm... you should point out the root cause...
my humble opinion...
regards


Workaround for WFCORE-3977

@rachmatowicz Looks good?

Superseeded by https://github.com/wildfly/wildfly/pull/11467.


[4.1] SR-6449: FileManager.removeItem(atPath) leaks

FileManager.fileSystemRepresentation returns a heap allocated
pointer that needs to be freed by the caller.

@swift-ci please test

@swift-ci please test and merge


fix cpu reservation code

Fixes segfault with kletGetter.GetNode() returning nil.  GetNode() doesn't work until the status manager is up and the node has reported status to the API server.

Updated


Reverse proxy with apache

Problem Statement
Dear all,
I can get things to work with the built-in HTTPS support in openvpn-auth-oauth2 on port 9000. Next I tried to configure the reverse proxy in apache. On a virtual host that has ssl enabled, I have set
ProxyPass /openvpn "http://localhost:9000"
ProxyPassReverse /openvpn "http://localhost:9000"

In /etc/sysconfig/openvpn-auth-oauth2, I have set
CONFIG_HTTP_BASEURL=https://<my_fqdn>/openvpn/
I have obviously also disabled the built-in HTTPS for that. In the application properties in Entra ID, I have configured
https://<my_fqdn>/openvpn/oauth2/callback
When I try to connect, it takes me to the URL
https://<my_fqdn>/openvpn/oauth2/start?state=.....
and I get a 404 page not found. Is this type of reverse proxy not working? Is the idea to use openvpn as the prefix wrong, and it only works with
ProxyPass / "http://localhost:9000"
ProxyPassReverse / "http://localhost:9000"
Thanks and best wishes,
Christian
Environment

openvpn-auth-oauth2 Version: 1.19.1
OpenVPN Server Version: 2.6.9
Server OS: debian bookworm
OpenVPN Client (flavor, OS): 2.6.9 on win10-amd64

I have never used apache2 for year.
But can you try this:
ProxyPass /openvpn/ "http://localhost:9000/openvpn/"
ProxyPassReverse /openvpn/ "http://localhost:9000/openvpn/"

Thanks!!! That did it. Best wishes,
Christian

Could you please provide an apache 2 vhost configuration? Then I can add it here: https://github.com/jkroepke/openvpn-auth-oauth2/wiki/Configuration#reverse-proxy-nginx-traefik


ArrayList \ Append

append :: a -> [a] -> [a]
Append a to [a]

Taken care of by the cons function.


supporting partial wildcards on server names for filter chain matches

Currently, when doing a filter chain match on server names you cannot use a partial wildcard:
https://github.com/envoyproxy/envoy/blob/master/api/envoy/api/v2/listener/listener_components.proto#L126
From a very quick look at how the server names are looked up:
https://github.com/envoyproxy/envoy/blob/master/source/server/filter_chain_manager_impl.cc#L418
it seems like supporting this could be done without a huge perf impact.
We have a use case for this, which would allow us to split a bunch of vhost/route configs into their own chains. This would save us from having to support per route overrides for the filters that need to be globally disabled and only enabled
for some vhosts/routes.
Any reasons not to support this? Thoughts?
cc: @htuch @lambdai @lizan @fishcakez

Do you just want to do a prefix partial wildcard like in that doc example (*w.example.com), or do you need something like something.*.example.com?
I believe @PiotrSikora originally wrote the matching code; he may have opinions on this also.

Do you just want to do a prefix partial wildcard like in that doc example (*w.example.com), or do you need something like something.*.example.com?
I believe @PiotrSikora originally wrote the matching code; he may have opinions on this also.

The former -- I need something like *-foo.example.com.

In general, we can do these wildcards with tries. Even in vhosts, we do it by iterating over a linear list today. We had a trie implementation once upon a time in https://github.com/envoyproxy/envoy/pull/555, we could dig that up.

+1
As long as it's suffix match, regardless it's pure suffix or tokenized suffix, we can introduce a trie in filter chain manager.
The trie need a backtrack candidate so that when failing to find any match we go back to match empty server name.
Also it's O(1) overhead if user doesn't want to use this feature.

I still want/need this; will circle back in a bit.

+1 for this feature

+1 for this feature


allow building on Ubuntu21.10

I will no longer need to build in a docker

A little more background, from the changes you can see there were some minor issues with the dependencies.
I have build and run under Ubuntu 20.04 (docker) and Ubuntu 21.10 with no problems.


docker: build docker image for building quickly

Don’t copy the entire brooklyn tree (source and target/) to the docker daemon when none of it is needed. On a partially built tree this can be 1Gb+, when all that is needed is the Dockerfile itself.

@tbouron I've updated the Jenkinsfile as requested.

@kemitix Looks like Jenkins doesn't take into account your folder change

retest this please


onChange callback called twice

If using datetime or time, the onChange callback called twice, but when using date, month or year only once.
I think the problem is on the time, when i choose 10:55 and the callback called twice, the time from first call sometime 10:00 or 10:55 but the real value 10:55 is on the second call.
can you fix this?

In fact, it's triggered at the wrong place (right before time detail) in datetime mode. Investigating ...

On further investigation, it triggers at each major change point. I think that's ok, make your app reactive to changes as it should be - say the user didn't want time details. Of course, ideally onChange is triggered at the end ... if I have time, I'll go through the code and maybe issue PR. But so far, it's very well done!
Kudos @mdehoog !

@ramezrafla but, when onChange callback contain ajax request, it's not nice requesting same consecutive request to the server

To get a single call to onChange you need to replace in https://github.com/mdehoog/Semantic-UI-Calendar/blob/master/src/definitions/modules/calendar.js, lines 682 - 687
              if (mode === 'hour' || (mode === 'day' && module.get.date())) {
                //the user has chosen enough to consider a valid date/time has been chosen
                module.set.date(date);
              } else {
                module.set.focusDate(date);
              }

with
                module.set.focusDate(date);

@mdehoog, should we have a settings partialOnChange to control this behavior? Triggering half way through could be dangerous as it can easily send wrong data (say we change from Jan 1 at 10AM to Jan 2 11PM, right now we will get an intermediate onChange with Jan 1 at 10AM)

@rafee12 I agree. It should be correct data or nothing.

@mdehoog , we started using the startCalendar and endCalendar features .. and I have to say this: Well done! This is amazing!

I added mode to the onChange callback (3a2ab5bfc4cd0609d39afe869bdc2e4171137a72), so you could check the mode for minute for a complete change. Alternatively, I would suggest using the onHidden to call your AJAX method, and just track changes with the onChange event.

Hello,
I'm trying to figure out how to set the 'minute' mode properly:
$('#calendarid').calendar({             monthFirst: false,             ampm: false,             today: true             }, onChange: function (date, text, mode) {                 mode = 'minute'}});
Is that right?

Got it...


Stream returned using model_stream method has always a null action object

I'm sending an action like this
action.send(request.user, verb='refilled', action_object=self.product)  # self.product is an instance of ShoeProduct

The action is correctly sent and these are my action data at db level:
id	actor_object_id	verb	description	target_object_id	action_object_object_id	timestamp	public	action_object_content_type_id	actor_content_type_id	target_content_type_id
9	1	refilled			8479	2018-08-31 22:04:03.045199+02	t	91	12

Using model_stream in this way:
model_stream(ShoeProduct)[0]

I'm getting:
<Action<EMAIL_ADDRESS>refilled 1 minute ago>

and I can see the the action_object property is always None (action_object_object_id and action_object_content_type_id are None too)
Django: 1.11.9
Python: 3.5.1
django-activity-stream: 0.6.5

The problems is in class GFKQuerySet -> fetch_generic_relations.
The queryset returned by
qs.prefetch_related(*[g.name for g in gfk_fields])

set all the three generic relation to None.
Continuing the investigation...


Update prerequisites in Readme

Summary of the PR
Android-aot workload no longer exists, so it should be removed.
Added maccatalyst, since it is also included in .github/workflows/deploy.yml file. Not 100% sure if it is actually required.
Updated recommended Visual Studio 2019 to 2022, because 2019 does not officially support .NET6.

Would it actually be better if it mentioned VS and Rider? Neither are actually required, but are there really any other options?


Stricter POSIX sh compliance

This would allow more users to benefit from chruby!

POSIX sh users
ksh users
dash users
ash users
posh users
yash users
oil shell users

It seems there are two potential blockers for your proposal.
First, neither POSIX nor these shells have preexec functions. POSIX doesn't specify a pre-function trap, which is needed for auto.sh switching to work.
Second, features like local would have to be dropped to find a common subset of behavior that's undefined by POSIX but still portable.
As the Oil author writes, POSIX doesn't define enough common ground so shells have gone beyond it and diverged.

For bigger programs, limiting yourself to POSIX is not just inconvenient, it's also an ill-defined and virtually untestable concept...
Local variables are essential for writing maintainable shell scripts, but POSIX doesn't mention them.
This issue isn't theoretical — bash and dash differ in practice, but POSIX doesn't specify which is correct. In other words, POSIX is incomplete and out of date. (However, I've discovered that shells are highly conformant with respect to things the standard does specify.)
In other words, POSIX is incomplete and out of date.  (However, I've discovered that shells are highly conformant with respect to things the standard does specify.)

There's an mksh branch of chruby that's portable and works other than mksh not supporting preexec functions. I don't think any of the shells you list do, which would mean no auto switching until they implement a prexec function feature. A shell like Oil  or mksh might do that—but dash and friends likely never will. I saw preexec functions mentioned as a possible mksh function, but they haven't implemented it as far as I know.
Would you think it's worth targeting portability for any of these shells without support for auto switching?
P.S. It was fun looking into Oil shell. Thanks for mentioning it!

POSIX doesn't specify a pre-function trap, which is needed for auto.sh switching to work.

I think you might be right about that. I was under the impression that chruby and friends were simply aliasing cd. Is that not sufficient for implementing good version switchers?

From ReadMe:

Anti-Features

Does not hook cd.

I guess it’s because aliasing cd has already been in many user’s .bashrc.


Reimplement e2e-test for API & GUI with fabric operator

fabric-test repository is updated to use golang based test framework(ginkgo), instead of python based one (behave). To keep tracking the update of fabric, we had better to align with the same direction in the testing framework.
We could delete a lot of files related to e2e-test by going this change.

@xspeedcruiser @JeevaSang @nfrunza @adamk1230 Please review this change.
If this PR got merged into master, I'll add more test scenarios for verifying #79 . Thanks

After

/azp run

@nfrunza Fixed an issue that version of some materials are not deterministic.


If the Smart Card service is unavailable, keep trying until it's available instead of quitting

On Windows, the SCardSvr service is only on if there is a reader connected. If none is present, the service just shuts down. This fix makes the process keep refreshing the card context until it gets a valid handle, eg. until some reader is connected.

Implmented in pokusew/node-pcsclite@11f939ff3214d5458890fe8c88c594f6f30784e8


chore: add CODECOV_TOKEN secret

No change to logic. This adds the CODECOV_TOKEN secret to the GitHub workflow. This is meant to resolve the error messages where GitHub runs can't send results to Codecov.
Following this guide: https://docs.codecov.com/docs/adding-the-codecov-token

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

Project coverage is 60.13%. Comparing base (b893df9) to head (4819a09).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #45   +/-   ##
=======================================
  Coverage   60.13%   60.13%
=======================================
  Files           2        2
  Lines         148      148
=======================================
  Hits           89       89
  Misses         59       59

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


Allow php 8

We need to add php 8 to travis config. Althought it will not work at this moment.

We need to add php 8 to travis config. Althought it will not work at this moment.

I've just added it to travis, I think it's this PR https://github.com/php-build/php-build/pull/651 so it should be ready soon


PowerShell function provider template

https://github.com/anypackage/function-provider-template

@startautomating created a PowerShell function based provider template. Please take a look and see if it meets your needs.


dev: Make groth16 generated contracts (a bit) more user friendly.

Current behavior :
Generated groth16 contracts looks like this :
https://github.com/keep-starknet-strange/garaga/blob/6135bd6dec063851a8249e1dafc8a7e4c98abf9d/src/contracts/groth16_example_bls12_381/src/groth16_verifier.cairo#L81-L99
They call an internal function if the proof is verified with the public inputs parsed from the proof as parameter.
Minimal Changes requested :

Delete the internal function.
Change the signature of the verification function to -> Option<Span> :

If check==True, return Option::Some(groth16_proof.public_inputs), else return Option::None.

Update and provide clear comments on the template.

Rationale :
Users usually expect/are used to have a single verifier contract and interact with it, from another smart contract to forward proof to it.
Having the Option::Some(pub_inputs) or Option::None() pattern gives at the same time a True(Some)/False(None) and deserialize correctly the public inputs for the caller.
Users have more freedom on how to process the inputs.
How :

clone repo, make setup, source venv/bin/activate
Update https://github.com/keep-starknet-strange/garaga/blob/main/hydra/garaga/starknet/groth16_contract_generator/generator_risc0.py and https://github.com/keep-starknet-strange/garaga/blob/main/hydra/garaga/starknet/groth16_contract_generator/generator.py
Regenerate examples using make rewrite

hi , i take it


drivers: counter: Update drivers to use devicetree Kconfig symbol

Update counter drivers to use DT_HAS__ENABLED Kconfig symbol
to expose the driver and enable it by default based on devicetree.
We remove 'depend on' Kconfig for symbols that would be implied by
the devicetree node existing.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

@galak. This broke the RTC_STM32 support. After your change, CONFIG_USE_STM32_LL_EXTI, CONFIG_USE_STM32_LL_PWR, CONFIG_USE_STM32_LL_RCC, CONFIG_USE_STM32_LL_RTCare not enabled anymore, despiteCOUNTER_RTC_STM32` being correctly enabled.

@galak. This broke the RTC_STM32 support. After your change, CONFIG_USE_STM32_LL_EXTI, CONFIG_USE_STM32_LL_PWR, CONFIG_USE_STM32_LL_RCC, CONFIG_USE_STM32_LL_RTC are not enabled anymore, despite COUNTER_RTC_STM32 being correctly enabled.

Can you open a bug.  I did a quick test w/disco_l475_iot1 and having CONFIG_COUNTER=y and I see:
CONFIG_HAS_STM32CUBE=y
CONFIG_USE_STM32_LL_EXTI=y
CONFIG_USE_STM32_LL_PWR=y
CONFIG_USE_STM32_LL_RCC=y
CONFIG_USE_STM32_LL_RTC=y
CONFIG_USE_STM32_LL_UTILS=y

@galak Indeed I was also not able to reproduce it with the disco_l475_iot1 board. After trying to create a reproducer, I found out that removing the build directory and creating a new one fixed the issue. Sorry for the noise.


Update Issue - 20353611

default description
URL tested: http://ci.bsstag.com/welcome
Open URL on Browserstack

Property
Value

Browser
Chrome 76.0

Operating System
Windows 7

Resolution
1024x612

Screenshot Attached
Screenshot URL

Click here to reproduce the issue on Browserstack

Updated Screenshot Attached


Resolve TODO for link in consistent apphash test

https://github.com/celestiaorg/celestia-app/blob/9475c4bcb9f9cb3f27c9be054201303526c11f2f/app/test/consistent_apphash_test.go#L42
cc: @ninabarbakadze

thanks @rootulp


Mention the ability to disable timeouts

Apologies, last PR was against master.

Thanks @tjcela!

Whoops, @tjcelaya

Seems like a truly minor change but thanks for taking it seriously anyways.
Sent from my iPad

On Mar 19, 2015, at 4:39 PM, Tim Ferrell<EMAIL_ADDRESS>wrote:
Whoops, @tjcelaya
—
Reply to this email directly or view it on GitHub.


Style Guide: is the beta fase finished?

Based on this comment from @chrisvfritz, the Style Guide would be considered in beta for about one month. We're past two months now and I'm think if it's time to start translating the Style Guide, but no one wants to translate a beta content. Any news about the official release of the style guide?

The beta stage won't be finished until @yyx990803 has had a chance to do a thorough review, but I think you can consider it stable enough to translate at this point. I don't expect many more changes than other pages in the future.


Spelling fix for 'Partnering with ecosystem heros' section: per-determined -> pre-determined

Fixed typo under the "Partnering with ecosystem heros" section. Made per-determined say pre-determined

Thanks @sfaber34 !!


fix: use default max nominations

fix: use default max nominations if api.consts.staking.maxNominations doesn't exist

Closed in favour of https://github.com/novasamatech/nova-spektr/pull/1243


Unabel to debug test

Recording 2022-03-15 at 17.55.08.gif.zip
I don't even know what to do here, works fine with dev tools.

Nevermind, turns out this was with uncaught and caught exceptions enabled. Though what worries me is that it is really hard to understand it.


Should add a parameter for caps if a user have one

The latest dev version has a beta sketch built in. currently the angle is fixed, i do plan to make it settable tho.
To update run pip install git+https://github.com/benno1237/MinePI.git@develop#egg=MinePI --force-reinstall --no-cache-dir
Let me know what you think about it please


How to remove all data from the databases

Hello,
In my app I have 2 databases and everything is working fine, but when the user wants to log out from the app, I need to remove all the saved data. I guess I need to drop the tables or something like that.. Can anyone tell me how can I do that? I was searching for something like this but I didn't found anything.
Thanks!

context.deleteDatabase(MyDatabase.NAME);

@agrosner When I added this: deleteDatabase(TodoDatabase.NAME); - loged back again , and the data is not removed. Any idea?

Do you need to append ".db"? We add in the .db to the name of the database when its created. I should add that to the README

I dont understand..
I've just appended ".db" to the database name and I get errors while compiling. Here is what I've done:

@Database(name = FavoritesDatabase.NAME, version = FavoritesDatabase.VERSION)
public class FavoritesDatabase {
public static final String NAME = "Favorites.db";

public static final int VERSION = 1;

}

and here is what I get in logcat:
/app/build/generated/source/apt/debug/com/raizlabs/android/dbflow/config/ShoppingList/db$Database.java
Error:(5, 32) error: '{' expected
Error:(8, 34) error:  expected

Remove the .db from your database name except for when you call context.deleteDatabase(name + ".db"); I am putting in a fix for 2.0.1 that if the name starts with .db then it won't try to include that in the generated class name.


chore: CLI cleanup

Remove redundant code
Use the NUGET_PACKAGES env variable, if one exists (as per .NET docs)
dotnet format & warning cleanups
remove var vs explicit type

Since we removed the fallback to the current directory, I added the option to override the NUGET_PACKAGES folder (see docs).


[hardknott][honister] {melodic}{noetic} grid-map-filters: Convert to new override syntax

Merged to other branches as well:
05bd57644..0e1490e03  hardknott -> hardknott-next
096229e55..0410145ff  master -> master
096229e55..0410145ff  master -> master-next


Refactor DxEngine::_GetProposedFont into a static function

Ported from MSFT-21254947

With Dart/OpenConsole!3129098, I instantiate an entire DxEngine just to get the initial size of the control.
Turns out there's only two things that _GetProposedFont really needs from the instance - the dwriteFactory, and the _chainMode. If we make those params, then we could make GetProposedFont a public static function, and not need to instantiate an entire engine (including all the dx resources it needs) just to get the font size.

We should not be spinning up an entire DX Renderer to ask it how big a cell should be.

(We still shouldn't be spinning up an entire renderer just to measure text. Is that fixed?)

Stephen f Balogh
On Tue, Feb 6, 2024 at 7:16 PM Dustin L. Howett @.***>
wrote:

(We still shouldn't be spinning up an entire renderer just to measure
text. Is that fixed?)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Oh I came across this issue while searching for another rendering bug and thought this one would be irrelevant because it's DxEngine (I only read the title). I'll reopen it.


Adds experiment tracker example to docs

Details
I added a sample notebook (in .md format) to the docs, based on: https://ploomber.io/blog/experiment-tracking/
Solves: https://github.com/ploomber/ploomber-engine/issues/12

Update: I tried to compile the docs directly from my dev branch and all changes seem to be ok, but for some reason an error is spotted (it already appeared in the last commit, prior to this one) -
https://readthedocs.org/projects/ploomber-engine/builds/18922498/

by default readthedocs will fail if the docs fail with any warnings. the warning is this:

Updating searchtools for Read the Docs search... [etoc] missing index.html written as redirect to 'intro.html'

I think if we rename intro.md to intro.html this will be fixed. please check


Add support for latest version of swift

I am hitting this error
The “Swift Language Version” (SWIFT_VERSION) build setting must be set to a supported value for targets which use Swift. Supported values are: 3.0, 4.0, 4.2. This setting can be set in the build settings editor.
Kindly add support for latest version of swift. Thank you

Hi, just add empty Swift class to your iOS project, it will automatically add SWIFT_VERSION to your configuration. Plugin already have provided SWIFT_VERSION.

Hi, just add empty Swift class to your iOS project, it will automatically add SWIFT_VERSION to your configuration. Plugin already have provided SWIFT_VERSION.

How can we add SWIFT class. Please help. Thanks


Use app image as devcontainer base

Resolves #13
Uses the app image as the base for the devcontainer. Means requirements aren't duplicated. I am however keen to still support development outside a devcontainer and running inside the VSCode ecosystem

Image is missing util-linux and docker-in-docker doesn't work 😞


Overhaul ajax

Currently "ajax" calls and client side AND server side go through jquery ajax transport. Because of that we have the following things we want to get rid of:

dependency on jquery-mockjax for bootstrapping data
inability to specify api host with a proxy
you have to build your own api middleware to handle requests from the client side
if you point your api host back to your express server so that a single middleware handles client side and server side "ajax" calls, on the server, you pay an extra penalty for each ajax call
you cannot specify multiple api hosts

currently working on an overhaul here https://github.com/wawjr3d/brisket/tree/feature/no_jquery_transport with the help of @zomaish


MaxRSS at assignment web ui

With the latest developments related to memory <--> maxrss (where maxRSS gets set according to the Memory requirements), the tasks configuration have the proper settings during the request creation.
Problem is, the form given via assignment web page contains default values and that value comes from the request.schema definitions (which contains the workload default values, not the ones provided within the tasks/steps).
There are at least two ways to fix it:

properly build the Memory/MaxRSS value (from the tasks/steps if it's a chained request) and set it at workload (and schema level), during creation
remove MaxRSS field from the assignment web ui. Memory isn't there anyways...

I'm in favor of 2 because it's much easier and easier to maintain at a long run.
@ticoann any thoughts?

In addition to that, I'd remove a few other settings from the web ui assignment page and leave only what's is normally touched by Ops. Leaving the fields rarely changed to the rest api only.

@amaltaro, Alan, so even if MaxRSS is defined in the request, it gets overwritten when Memory value is set, is that correct?

MaxRSS can't be (manually) set during request creation, it is an assignment argument. Well, scratch that, actually it can:
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/WMSpec/StdSpecs/StdBase.py#L1054
If user explicitly sets it, then the expected behaviour is to use the user input (though I'm not sure it corresponds to reality :) Still, I don't think P&R team sets it during creation, only assignment.

Seangchan, I think I just figured what the problem is. I'll have a look at it soon and - if needed- you can follow it up on your timezone :)

Honestly, I think we should just "get rid of" this MaxRss and hard couple it to the Memory requirements, in a completely transparent way to the end users/P&R.

I completely agree if users are not going to set it in different value than memory. I can update the patch

However, looking at WmAgentScript code. It seems it is explicitly set the MaxRSS
https://github.com/CMSCompOps/WmAgentScripts/blob/master/reqMgrClient.py#L790
is MaxRSS has different unit than Memory?

They used to have, Memory is/was in MB and MaxRSS was in KB. We changed MaxRSS half an year ago to MB too, but we had to support KB too
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/WMRuntime/Monitors/PerformanceMonitor.py#L201
we should be able to replace that by now, I hope. I'm asking them in this issue:
https://github.com/CMSCompOps/WmAgentScripts/issues/248


Osra319 partners show action items

https://osraav.atlassian.net/browse/OSRA-319

This PR seems to go well beyond its intended scope, with most of its code dedicated to pending orphan lists, not partner. Can we just use a stub route ('#') for the Upload button for now and come back to pending orphan lists in due time?

The way I understood the story was to create the "action items" and the "action" part of that is mapping the link to a controller method and using a path helper for linking.


No feedback on connection limit reached for supabase connection pooler

This hasn't reproed for me after #3585


FPM initialization failed (Address already in use (48))

After installing PHP with the installation script, the execution of Magento is still very slow - without observing any bottleneck in the usage of resources.
The logs are full of entries like this:
==> /usr/local/var/log/php-fpm.log <==
[26-Aug-2020 12:56:21] ERROR: unable to bind listening socket for address '<IP_ADDRESS>:9072': Address already in use (48)
[26-Aug-2020 12:56:21] ERROR: FPM initialization failed

And killing the PHP process does not help, the errors appear almost instantly:
==> /usr/local/var/log/php-fpm.log <==
[26-Aug-2020 13:13:43] NOTICE: fpm is running, pid 14034
[26-Aug-2020 13:13:43] NOTICE: ready to handle connections
[26-Aug-2020 13:13:53] NOTICE: [pool www] 'user' directive is ignored when FPM is not running as root
[26-Aug-2020 13:13:53] NOTICE: [pool www] 'user' directive is ignored when FPM is not running as root
[26-Aug-2020 13:13:53] NOTICE: [pool www] 'group' directive is ignored when FPM is not running as root
[26-Aug-2020 13:13:53] NOTICE: [pool www] 'group' directive is ignored when FPM is not running as root
[26-Aug-2020 13:13:53] ERROR: unable to bind listening socket for address '<IP_ADDRESS>:9072': Address already in use (48)
[26-Aug-2020 13:13:53] ERROR: unable to bind listening socket for address '<IP_ADDRESS>:9072': Address already in use (48)
[26-Aug-2020 13:13:53] ERROR: FPM initialization failed
[26-Aug-2020 13:13:53] ERROR: FPM initialization failed

Solution:
The issue was caused by an old setup of PHP having a daemon process.
In $HOME/Library/LaunchAgents/ I had the right files:
ls $HOME/Library/LaunchAgents/nl.reachdigital.io.php*
$HOME/Library/LaunchAgents/nl.reachdigital.io.php-xdebug.plist<EMAIL_ADDRESS>$HOME/Library/LaunchAgents/nl.reachdigital.io.php.plist<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
(note: replaced by real file path with $HOME)
But in that folder there is also one extra item:
homebrew.mxcl.php

Executing launchctl unload "$HOME/Library/LaunchAgents/homebrew.mxcl.$PHP.plist" as advised by @paales, and restarting PHP via pkill php-fpm helped to start the PHP process correctly, with only one instance of it 🎉

Thanks for the reference!


Support open contrail network fabric

I don't think we want to do this and I have no desire to push for this so soon after moving to canal networking.  If someone disagrees, feel free to reopen.


EKS 1.15 Support

Opening this issue to handle all changes needed for EKS 1.15, which was released today: https://aws.amazon.com/about-aws/whats-new/2020/03/amazon-eks-now-supports-kubernetes-version-1-15/
I'm submitting a...

[ ] bug report
[X] feature request
[ ] support request - read the FAQ first!
[ ] kudos, thank you, warm fuzzy

Looks like EKS 1.15 no longer needs the VPC tags: https://docs.aws.amazon.com/eks/latest/userguide/network_reqs.html#vpc-tagging

This tag is not required or created by Amazon EKS for 1.15 clusters

Interesting!


(doc) Add link to C4B Azure offer

Description Of Changes
Add a link to the Azure Marketplace offer for C4B.
Motivation and Context
The summary doesn't contain a link to the offer. I had to keep reading to find it.
Testing

[ ] I have previewed these changes using the Docker Container or another method before submitting this pull request.

Change Types Made

[x] Minor documentation fix (typos etc.).
[ ] Major documentation change (refactoring, reformatting or adding documentation to existing page).
[ ] New documentation page added.

Change Checklist

[ ] Requires a change to menu structure (top or left hand side)/
[ ] Menu structure has been updated

Related Issue
N/A

@pauby should we be linking directly or via the short URL (https://ch0.co/C4B-Azure)?

@windows The Short URL makes sense. I took that link from the doc page. Let me update this.

@pauby wanted to ping you about this. Is this something you want to move forward with?

@gep13, thinking aboutt his one there are a few other instance where the longer URL is used that we could probably change to the short URL.
I'd say this can be merged, as it's consistent with the existing docs.
An issue can be created for changing any links to short links, where this makes sense to do so, in one go.


Retrieve point cloud point positions on click

This is something that's come up a few times recently on the forum. There's some good discussion on it in these threads:

how to get exact point location of a point cloud data
Snapping to closest point in a point cloud

Yes we have the same issue. Our solution was to use a spatialite database to do a spatial query on the point cloud as this was able to bypass the screen-space inaccuracies and the fact that there's no GPU-CPU transfer of point cloud data. Obviously not ideal for many reasons, but it enabled us to show details for the exact location of a point.
Omar, it's the definition of "exact" that's the issue. The buffer will get very close, but when dealing with point clouds with an accuracy of 1/10mm, the buffer isn't good enough. Plus it sometimes gives wildly inaccurate values if you're on the very edge of a point (in the order of hundreds of metres inaccurate).

Some notes from @lilleyse about how we could go about implementing this:

Does the data need to reside on the CPU or is it fast enough to do GPU copy with gl.bufferSubData? All point data is stored in vertex buffers.
If going with CPU approach, add an option that controls whether the data stays on the CPU. This is the line that normally deletes it: https://github.com/AnalyticalGraphicsInc/cesium/blob/master/Source/Scene/PointCloud.js#L1323.
Add an API for retrieving point data . Something imilar to Cesium3DTileFeature. The problem is we don't want to create a Cesium3DTileFeature per point, so we'll need something leaner.

Just want to revise something I said. The buffer call is gl.getBufferSubData and is only available for webgl2. Instead we could store per point data in textures and use readPixels.

This came up again on the forum here, specifically that you can't rely on scene.pickPosition to get point positions. This is reproduced in this Sandcastle. If you click on any point on the building, towards the horizon (such that the mountain is behind where you're clicking, but you're still clicking on the point cloud), you'll get a correct output like this:
Picked object {primitive: Cesium3DTileset, content: PointCloud3DTileContent}
Distance from known point 7.555343978706986

Where the "known point" is a point on this building. But sometimes, you'll get:
Picked object {primitive: Cesium3DTileset, content: PointCloud3DTileContent}
Distance from known point 19238.09656548114

So even though scene.pick knows you clicked on the point cloud, the pickPosition seems to click through and get a point far int he distance.
@lilleyse is there any known workaround for this, especially since scene.pick seems to correctly figure out you're clicking the point cloud?

Changing scene.pick(mousePosition) to scene.pick(mousePosition, 1, 1); seems to at least make the results consistent (so if you click on the edge of attenuated point, pickPosition will pick a point in the distance, picking through the point cloud, but so will pick).

Changing scene.pick(mousePosition) to scene.pick(mousePosition, 1, 1); seems to at least make the results consistent (so if you click on the edge of attenuated point, pickPosition will pick a point in the distance, picking through the point cloud, but so will pick).

That solved by problem. Thanks!

The need to click on a point and get its position for drawing lines on a point cloud came up again here:
https://groups.google.com/d/msg/cesium-dev/tla7PUio2Ys/qSz1csvHAwAJ

Hi! I am having the same issue on this. I am trying to only get the position inside the point cloud during left click. If I clicked on a less denser point cloud, I always get a position behind it. Is there any way to get the closest point in the point cloud based on the initial clicked position?

Is there any way to get the closest point in the point cloud based on the initial clicked position?

@sarahmeh Have you tried using drillPick? Also, though it's private API, there is drillPickFromRayMostDetailed . From them, you get an array of picked points, and you can iterate over them to get the one closest to the camera. Both functions take a width parameter, which allows to pick not only the clicked pixel, but also around it.


Unable to build an image

https://travis-ci.org/lavab/api/builds/66658861#L1047-L1062
What am I doing incorrectly?

I'm not sure, but I think it's related to https://github.com/moul/travis-docker/issues/17
For now I have no workaround, but I hope to fix your issue while fixing #17
You can try something, can you try different travis-docker configurations ?
You can use a matrix like this: https://github.com/moul/travis-docker-example/blob/master/.travis.yml#L1-L18, if one of the builds works, it may help to fix #17 and #20 :)


Doesn't change UDID. Help!

I installed the tool and followed all the instructions. But the UDID on Xcode is still the same. I tried on El Capitan with Xcode 7, and on Sierra with Xcode 8

Hi, didn't test it on Sierra yet, and didn't test it with latest Xcode version

I tried also on El Capitan with Xcode 7. I think the problem is installing Opee. What I did is opened opeeinstaller.sh and copied the commands one by one and ran them on Terminal. That's the way it gets installed right?

you should open terminal and type:
1- cd PATH_TO/Opee installer/
2- . opeeinstaller.sh
3- Entre and wait

I did the cd command. Anyway I am trying it and will let you know.
Thanks in advance

I got these errors, and still doesn't work
http://i.imgur.com/4vWbUqU.png

Check opee page to install it :( that's the only solution

I did. It gives the same errors on Sierra with Xcode 8 and on El Capitan with Xcode 8. Why did this happen?

Will Parasite work?

Did you disable SIP ?

Yes.

Hey, I followed the Opee page instructions and installed it successfully on Sierra. (My El Capitan virtual machine broke out). Then I tried and it doesn't open Xcode while clicking "Apply". (Maybe it is because of Sierra and Xcode 8). Anyway I will try again on El Capitan and will let you know. (In the meantime, could you test it on Sierra and Xcode 8? So if it won't work I would be sure it is Sierras and Xcode's fault)

I tried on El Capitan and worked!
Conclusion:

opeeinstaller.sh needs to be updated.
Doesn't work on Sierra and/or Xcode 8

OK, now I have to say that the apps don't install via another PC or via OTA. So this Apple limitation makes your tool useless. When you tested it, did you try installing via another PC?


Unable to edit failed sent e-mail

Expected behavior
When sending a e-mail has failed because of mistyping the recipient's address, edit the e-mail to correct the address, or move it to draft to edit it in the draft. The second option should abort to send e-mail.
Actual behavior
Once in the outbox, an e-mail can only be sent again or deleted.
Steps to reproduce

Write a new e-mail with a non-existent e-mail address as recipient
Send it!
Go to the outbox to try to move it to draft or edit it on place

Environment
K-9 Mail version: 5.206
Android version: 6
Account type (IMAP, POP3, WebDAV/Exchange): IMAP/SMTP
Thanks.

I tried to reproduce the error and when the step 1 and step 2 are done, the e-mail is moved to Sent folder though the mail is not sent. The message can be forwarded and when the Forward is clicked, it moves to an interface where changes can be done to that mail.
K-9 mail version: 5.207
Android version: 4.2.2
Haven't I reproduced the issue correctly or the issue is addressed by the newer version of K9 mail?
Thanks.

OK, your e-mail server did not refuse you misspelled address.

I just tried with 6.202 version and the problem has been solved: the e-mail that can't be sent is movable to the draft directory.


New version: joncampbell123.DOSBox-X version 0.84.0

Result: Installation Failed

Manifest
Add/Remove Programs

Name
DOSBox-X

Version
0.84.0

Publisher
joncampbell123 [DOSBox-X Team]

ProductCode

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 1868
Microsoft Reviewers: Open in CodeFlow

@wingetbot rerun

@wingetbot run


Settings GUI container extending too much to right

Issue Type: Bug

VSCode Version: 1.28.0-insiders
OS Version: Windows 10 1803 17134.320

Steps to Reproduce:

Open Settings GUI
Find Setting which utilises Add Pattern, in this case it is Files: Exclude Or find really long description
Hover over an item and see edit and delete is aligned too much to right OR text bleeds over just a bit.

Red Lines shows what I mean
Screenshot:

Text containers:

@roblourens

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

Unrelated.


Refactor assess_permission

Remove access_permission.get_full_data()
Remove access_permission.gcet_serializer_class()
get serializer class from model

@ostcar ping


fix<EMAIL_ADDRESS>CI failed

Description

Task Item
#minor

Screenshots

Coverage remained the same at 54.499% when pulling 6bab5657bead104a09ab1312c1a33010efb349d4 on zhixzhan/ci-error into dc0d900051b6423e32d15f14c165cc78acbdc8d3 on main.


Don't allow multiple workspaces to be named the same

(Note that test failures are related, looks like the regex has an issue.)

https://github.com/vercel/turbo/pull/2659#discussion_r1019837451
this comment really took me down a rabbit hole. Turns out:

pnpm i when you have no dependencies doesn't generate a lockfile. (not a big deal for this test, but I didn't know that before
npm i doesn't work with two workspaces with the same name, so it's hard to end up in this state. Previously I had run npm i to generate the lockfile before making the workspace names the same? Which is how the test was working in the first place?
turbo doesn't work even if the monorepo is configured to use pnpm, if workspaces aren't in package.json (weird and I didn't dig into it)
pnpm (at least v7) doesn't like the workspaces file have .yml extension, requires .yaml

For this test case, I ended up with making the fixture use pnpm and add workspaces to package.json in addition to the pnpm-workspaces.yaml file.


[Enhancement] Add create controlled term api to subject area omas

Is there an existing issue for this?

[X] I have searched the existing issues

Please describe the new behavior that that will improve Egeria
allow glossary terms to be used in a workflow. this means enhancing the subject area omas to

expose a create controlled terms API
expose a change status for the controlled term to allow it to move through different status's

Alternatives
No response
Any Further Information?
No response
Would you be prepared to be assigned this issue to work on?

[X] I can work on this

Also need to return the instance status when elements are retrieved and the type name of the element so caller can distinguish the controlled glossary terms from the ordinary ones.

@mandy-chessell thanks for you feedback. The SubjectArea OMAS already has a parent object called node that has the typename and a NodeType. I was thinking that the nodeType would be everything we know about that I could paint on the canvas. currently I have Activity -> Term -> GovernedNode -> Node. I am thinking of having a ControlledActivity -> Activity and a ControlledTerm-> Term as new NodeTypes which could have their own icon. The alternative is conditional code around the nodetype to look at whether the object is controlled or not.


Geocoding sample

Sample
Geocoding: Enter an address to get the geographic location of it. The pawn will be moved to that location and show a visual cue.
Reverse geocoding: Shift+ left click on a location to see the address. The queries are made through the Geocoding service API.
Summary
Enter an address on the text field in the top and the camera will move to that location and show a pin on that point as well as the address returned by the API. Ray casts will determine the elevation of the point.
Clicking on a location while holding the shift will make a query to get the address/description of that point. A visual cue shows the address. The scale of the visual cue is set based on the distance from the camera and the text will face the camera.
Tests
Windows
ArcGIS Maps SDK Version
Daily build 3598

Just noticed one thing. It says you tested with daily build 3598 but we want the samples to be configured to work with the last release if they are being merged into main.
I tested it on the last release so I know it works but for future reference the samples work should be locked onto the last release unless being merged into next


canonical modulus

"However, the behavior is the same for positive dividends."
What behavior?
Can someone clarify?

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

ID: 227f6a03-7076-a47a-24de-a2c3c31a90b5
Version Independent ID: 8538cb70-a092-1166-1cd9-b77f5e6a80d4
Content: % Operator (C# Reference)
Content Source: docs/csharp/language-reference/operators/remainder-operator.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

What it intends to say, @johnnyagerard, is that the remainder operation produces the same result as the modulus operation for positive dividends, but a different result for negative dividends.

@rpetrusha @pkulikov Ok, thanks.

I think at a minimum that we eliminate the ambiguity of "the behavior is the same for positive dividends". I'll assign this to myself for a future sprint. If either of you would like to do it in the meantime, @pkulikov and @johnnyagerard, please go ahead.

@rpetrusha I would like to update the article next week, if @johnnyagerard is fine with that.

Thanks, @pkulikov. I've assigned the issue to you.


Add an optional attribute to handle updates during atlas downloads

Description
What is this PR
Atlas download progress is currently handled within a Rich progress indicator, but external libraries do not have access to these updates. This PR adds an optional attribute to BrainGlobeAtlas to reference a handler that is updated during atlas downloads. This handler can be given as a function that takes the currently competed and total required bytes to download the given atlas.

[ ] Bug fix
[x] Addition of a new feature
[ ] Other

Why is this PR needed?
Provides access to track real-time atlas download updates by external apps. There may be better ways to provide this functionality...I'm all ears to alternatives.
What does this PR do?
Adds a hook called by bg-atlasapi during atlas downloads.
References
I am using this handler to update a progress bar in MagellanMapper when downloading atlases through BrainGlobe (see https://github.com/sanderslab/magellanmapper/pull/75).
How has this PR been tested?

Tested downloading the "kim_mouse_50um" atlas in bg-atlasapi in an interactive Python 3.6 session, where the handler is ignored and download proceeds as normal
Tested in MagellanMapper PR 75, which passes a function that updates the progress bar during atlas downloads

Is this a breaking change?
No
Does this PR require an update to the documentation?
Updated the docstrings with the changes. Also added a type hint to the handler to specify its expected parameters.
Checklist:

[x] The code has been tested locally
[ ] Tests have been added to cover all new functionality (unit & integration)
[x] The documentation has been updated to reflect any changes
[x] The code has been formatted with pre-commit

Attempted to fix formatting errors from the lint action by running the black package.

Looks good to me @yoda-vid. Just because everything else depends on this package, I'll wait for @vigji or @FedeClaudi to approve before merging.

Sounds good, thanks @adamltyson! Open to any feedback to make this best fit with BrainGlobe's goals.

Merged, thanks @yoda-vid!


"Unknown error" reading Parquet data from Google Cloud

https://stackoverflow.com/questions/76012391/pyarrow-fails-to-create-parquetfile-from-blob-in-google-cloud-storage

https://github.com/apache/arrow/issues/35318


Updates to perf tests

Updates to fix perf builds.

I don't know if we can merge with the flaky test, but I'd like to at least get it reviewed.

@prescod I'm fine with approving this, but what's the flakiness you reference? Is that one of the three Robot tests that is not currently passing?

@davidmreed Yes the three tests that are failing.


Add RSS/Atom

The blog doesn't have a way to subscribe, please add a RSS/Atom feed.

thanks for raising! doing a complete redesign right now so closing this.


Docs Page Does Not Render Correctly

Issue
The docs page is not rendering correctly
Development Environment:

[x] Machine (Local, Container, Virtual): Local
[x] Operating System (Name/Version): MacOS
[x] Web Browser (Name/Version): Safari
[x] Vapor Version: 3
[x] Swift Version: 4.1

Expected Behavior:
https://www.gitignore.io/docs renders the docs page
Actual Behavior:
https://www.gitignore.io/docs renders a dead site of https://www.gitignore.io
Reproduction Steps:
Navigate to https://www.gitignore.io/docs

There seems to be a bug in Vapor 3's routing engine which allows the site to flap between docs and /

This issue is related to vapor leaf rendering issue: https://github.com/vapor/leaf/issues/117


Feat:Scraping Rotten Tomatoes

Describe the feature
Scraping rotten tomatoes website using bs4.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC 23 contributor
[X] I want to work on this issue

Go ahead @alwenpy
Note

Create a separate module for this, as per the folder and project structure
Do not use the `selenium web driver as it is not compatible with all devices as well as cloud platforms.
Before making any changes, check if the module you want to add already exists or not. If yes, then add your functionality as a method only else make a separate module and class for it.


Update w3c_audio externs to last published version

See last published version from  08 December 2015 https://www.w3.org/TR/webaudio/, while these externs weren't updated since 2012.

This change is

Hi, any news on this ?

@zarov, there appear to be some problems with this externs file.
javascript/externs/w3c_audio.js:238: ERROR - property suspend already defined on superclass BaseAudioContext; use @override to override it
OfflineAudioContext.prototype.suspend = function() {};
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

javascript/externs/w3c_audio.js:243: ERROR - property resume already defined on superclass BaseAudioContext; use @override to override it
OfflineAudioContext.prototype.resume = function() {};
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

In this case, I think these 2 definitions can just be dropped. They're redundant anyway.
The compiler bails out at this point, so there may be more.

Yes indeed, it seems that I duplicated several values like this. Do you have a test to run for checking the others duplicates ?

@zarov since this is one of the standard externs files, any advanced mode compilation should have caught this problem.
@ChadKillingsworth suggests this:
echo "alert('hi')" | java -jar compiler.jar -O=ADVANCED -W=VERBOSE

@zarov let me know when you have an update & have squashed it to a single commit.
I'll re-do the import and restart testing at that point.

It appears we also have at least one internal project that uses the constants defined on
AudioBufferSourceNode.prototype such as PLAYING_STATE, so it's not safe to remove them yet.
Could you instead mark these as @deprecated?

Just updated, I got no warning/error on my side. I saw that I was a little too much picky when removing deprecated properties (but not marked as deprecated in the specification, only not present at all). Do you think it would be better if I move those properties as @deprecated for compatibility on your side, instead of deleting them ?

@zarov
Please do mark things @deprecated instead of removing them wherever possible.
Thanks.

@brad4d ok, just redid a complete pass on the file. I re-added deleted properties/methods as deprecated, change all @readonly to @const, so now it should be good. I got no error when testing on my side.

@zarov
Our PR import tool is choking on this PR.
We believe this is because your branch is based on a very old commit from google/closure-compiler master.
Could you please rebase onto the current head of our master branch?
Thanks.
(It choked the first time I tried to import it, too. I manually copied then as a workaround, but that isn't as practical as we go back and forth on changes.)

Oh sorry about that, thought I add rebase it ! Just did it, should be good now.

@zarov
Thanks a lot for rebasing. That fixed the import.
I've requested a few more changes, but I'll go ahead and see if what you have now passes testing.

@brad4d no problem at all, thanks for taking all this time for review. I reorder it and squash the commit of this PR. I checked all the nullable/non-nullable properties according to the specification, it should be good now.

@zarov
Thanks a lot. That made it much easier to review.
Sent you some more comments.

@brad4d just updated, should be even easier, lines changed have been reduced.

Testing a fresh import...

Looks like there's at least one duplicate definition.
I added a comment on the file.

@brad4d fixed, hoping I didn't missed anything again !

testing a fresh import...

Alas! IIRFilterNode is also duplicated.

Yes, that's because of another commit yesterday... rebasing the file from the current state !
See https://github.com/google/closure-compiler/commit/fc1caf7bfa49ec4c888411daf3c17e9b524d443a

Pushed the change back !

Importing and testing...

\o/ Tests passed.
Doing internal code review.

Had to rollback internal submit
REDACTED_FILE_NAME.js:24: ERROR - variable AudioBufferSourceNode.prototype.detune redefined with type (Object|null), original definition at javascript/externs/w3c_audio.js:546 with type AudioParam
AudioBufferSourceNode.prototype.detune;

More errors:
REDACTED/Sound.js:289: ERROR - actual parameter 1 of AudioNode.prototype.connect does not match formal parameter
found   : (DynamicsCompressorNode|null)
required: (AudioNode|AudioParam)
  gainNode.connect(this.compressor_);
                   ^^^^^^^^^^^^^^^^

REDACTED/Sound.js:316: ERROR - actual parameter 1 of AudioNode.prototype.connect does not match formal parameter
found   : (BiquadFilterNode|null)
required: (AudioNode|AudioParam)
    gainNode.connect(this.highFilter_);
                     ^^^^^^^^^^^^^^^^

The "redefined" error I posted first is because an internal project just defined something in their own externs that should be in ours.
I'll fix that internally.
The other errors look like a backward compatibility problem to me.

@zarov
You can hold off on making further changes for now.
I'm going to see what it takes to update the project's code to match your update.

@brad4d no problem ! As you said, the two other errors might need to be fixed internally, as non-primitive type should be non-nullable, as it is the problem here.
Let me know if I need to apply any other change. Thanks for the whole review ;)

Submitted an internal fix for the 'detune redefined' error.
I'm going to take a look at the errors in Sound.js next.

FYI, I'm doing internal changes to just make the first argument to AudioNode.prototype.connect non-nullable & fix whatever that breaks.

I've submitted fixes needed for 2 pieces of internal code, and I expect a change that makes the first argument to AudioNode.prototype.connect non-nullable to be submitted internally today.

Thanks for doing all that work 😃

@zarov please rebase to resolve the conflict with my commit.
Thanks

@brad4d done, basically just reverted ebaf022a94a81544fe4533a0541d8142acf0e0e6

Importing rebase and looking for the next fix to make...

I've had to update a few references from AudioChannelSplitter or AudioChannelMerger to ChannelSplitterNode or ChannelMergerNode.
I'm rerunning tests with those changes and hope to submit this PR internally tomorrow.

Discovered a couple more files referring to AudioChannel* instead of Channel*Node.
I've also discovered some cases that still use RealtimeAnalyserNode.
So this is going to take a bit longer.

Submitted changes to remove uses of RealtimeAnalyserNode.
Last round of testing found a few more errors.

@zarov
I believe I've discovered a bug in your commit.
Several onsomething properties are defined as {!EventTarget}.
I believe they should all be {!EventListener}.

@brad4d
Well in the spec, they are all specified as EventHandler. I thought the closest type was EventTarget, but I could have been misguided. If you confirm the referring type is EventListener indeed, I'm changing it !

@zarov
An EventTarget is an object on which events happen, not the code that does the handling.
I was mistaken about using EventListener, though.
Looking at other .onsomething properties in html5.js I see that we use '{?function(!Event)}',
so you should do the same here.

@brad4d that sounds good to me, just pushed changes back.

@brad4d I see new conflicts coming from nowhere, did I miss something or do I need to rebase and fix the conflicts on my side ?

@zarov
An internal user made a partial version of your change independently of me.
I tried rebasing and pushing back to your branch, but I don't think you've granted me permission to do that.
Please rebase.

@brad4d ok no problem, I added you in case it happens again ! Just rebase it back

@zarov
I've imported your latest changes and am testing them internally.
Unfortunately, I already know there are going to be a few more bits of internal code I need to fix.

test-all-the-things failed, too many things affected.
hmm, this worked before.
trying again.

I now have 2 classes of failures:
nullability problems
Solution here is to fix the client code to avoid passing things the compiler thinks might be null.
3 files.
Splitter/Merger issues
previously: AudioNode -> AudioChannelSplitter
now: AudioNode -> ChannelSplitterNode -> AudioChannelSplitter
AudioChannelSplitter is now deprecated and all APIs are defined to take and
return ChannelSplitterNode.
Same situation exists for ChannelMergerNode and AudioChannelMerger
4 files
fix:

Partial extern change: Add the new types to externs. Make methods and fields
accept the newer, base type, but have them return the older type for now.
Fix the client code to define everything in terms of the new type.
Partial extern change: update return values to use the new type.
maybe remove the deprecated type? Is it even really usable after step 3?

Submitted an internal change with almost everything from this PR.
There are TODOs for the bits that require internal fixes first.

I have 3 internal changes up for review to fix the nullability issues and splitter/merger type changes.
When they get submitted, I'll do a change that fixes the externs, and we'll be done.

Yes wonderful, very happy to be finally here :)

Last internal fixes were submitted on Friday.
I'm submitting the fix for the TODOs today to close this PR.

A change fixing the TODOs was submitted this morning.
It should get pushed to github tomorrow and automatically close this PR.

Thank you @brad4d for all this work !