Can't perform a React state update on a component that hasn't mounted yet

Describe the bug
Error received when navigating to a table using getGroupedRowModel() such as:
  const table = useReactTable({
    data,
    columns,
    getCoreRowModel: getCoreRowModel(),
    getGroupedRowModel: getGroupedRowModel(),
  });

As seen in this example, navigating to the basic table works fine where as navigating to the grouped table produces an error in the console. The only difference between the two is getGroupedRowModel().
I am using the code from the examples pretty much as is (stripped down for simplicity).
Your minimal, reproducible example
https://github.com/tacomanator/tanstack-table-unmounted
Steps to reproduce

Clone example, install dependencies, and run
While viewing the console, navigate to the grouped table
Observe error

Expected behavior
No error output
How often does this bug happen?
Every time
Screenshots or Videos
https://github.com/TanStack/table/assets/63794/30edbbcf-8f7e-459f-b624-b7d82aac3f0a
Platform
macOS, Chrome 115.0.5790.170 arm64
react-table version
8.9.3
TypeScript version
5.1.6
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

I am having the same problem in version 8.9.1 when I call table.getRowModel() to conditionally render rows in tbody and I can confirm that it probably has something to do with getGroupedRowModel.
If i remove getGroupedRowModel: getGroupedRowModel() from the hook the error disappears.

I'm seeing this exact same issue today.

I investigated this issue this morning and discovered that the error only occurs when in dev mode. Same with @tacomanator's example. When I run the example in dev mode I get the error, but if I do a npm run build and npm start then the error disappears.

I think this is due to the fact that these errors don't show up in production builds but still happen.
Happy to be proven wrong.

I think it has something to do with pagination. When you add getGroupedRowModel it ends up calling setPagination in the render and that ends up calling setState.
So if you set onPaginationChange in the useReactTable definition to onPaginationChange: () => undefined then the error also goes away.

I think this is due to the fact that these errors don't show up in production builds but still happen.

It's possible that it's a dev-only error because of React's strict mode which will double renders and effects. For me it's happens only on the second round of the very first render.
It's likely that there's a internal hook that's using useState in an useEffect for memoization. setState is called after the render and the component will already be unmounted in dev mode.

This occurs here and exclusively in the context when the getGroupedRowModel function is utilized. It can be resolved by simply verifying whether the component is mounted before call setState in onStateChange. It seems somewhat hacky but can be helpful.

This also happens on getFacetedUniqueValues

same issue in getFacetedUniqueValues

same issue in getFacetedUniqueValues

also facing the same

getFacetedUniqueValues

Same here, is there any other solution?

I faced a similar issue. in my case, the affected functionality is due to function getSortedRowModel() from checking "isEmpty" of my data which this caused the error , my temporary solution is :
getSortedRowModel: !data.isEmpty ? getSortedRowModel() : null

Using latest version of tanstack @ 8.11.6, getting this when calling table.getIsAllRowsSelected() in dev mode. I can confirm this still errors when strict mode is off.
{
        id: "select-col",
        enableResizing: false,
        enableSorting: false,
        header: ({ table }) => (
          <div className="flex h-full w-full items-center justify-center">
            <input
              type="checkbox"
              className="rounded-sm bg-neutral-500 text-orange-600 focus:ring-0 focus:ring-offset-0"
              id="checkbox"
              checked={table.getIsAllRowsSelected()}
              onChange={table.getToggleAllRowsSelectedHandler()}
            />
          </div>
        ),
      });
    }

I receive this when using getRowModel but not when using any of the other getRow functions. Figure it's something getRowModel does during render.

With me this error occurred in 8.20.1 as well, in table.getColumn(<COLUMN_ID>) in the Toolbar component, while passing column prop, and updating the DataTableFacetedFilter shadcn-ui component.
Which is absolutely expected behavior when passing mutable props.
Updating column prop via useState hook,  solved the problem.
const [statusCol, setStatusCol] = useState(null);

useEffect(() => {
    if (table.getAllColumns().length) {
      setStatusCol(table.getColumn('status'));
    }
  }, [table.getAllColumns().length]);

// then

{statusCol && (
  <DataTableFacetedFilter
    column={statusCol}
    // ...
  />
)}

I receive this error as well. For now I disable getGroupedRowModel and the error is gone.

I've been looking for a solution to this problem for a while. After reading further up about the pagination state causing the error I added manualPagination: true to the useReactTable hook props and it seems to fix it.
My use case has no need for client side pagination, so I guess it would be ok to set manualPagination to true?

just add:
const [isMounted, setIsMounted] = React.useState(false)

  // Fix error on react table, when the table is not mounted
  React.useLayoutEffect(() => {
    setIsMounted(true)
  }, [])

// react table hook, and other codes...
// Fix error on react table, when the table is not mounted
if (!isMounted) return null

@nabilfatih as a workaround sure, but important to let people know this workaround will hurt performance and change the loading profile of their app.

i get this same error on this line: table.getRowModel().rows?.length

Same problem here. Seems that getPaginationRowModel is causing the problem. tanstack table version 8.20.5. autoResetPageIndex: false seems to suppress the error but it messes up with filtering when changing the first page.

Any fixes?

adding this to hook useReactTable solved the issue for me:
autoResetPageIndex: false,

@ApacheEx
I'm using table.getSelectedRowModel().rows.length I guess that doesn't help in my case.


Avoid crashing when ContainerStatus.Started is nil

Fixes: #1656
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>(cherry picked from commit 9c1bff0bdda5ce70353036b8f775551f74c38e61)

(adding to the 0.12 board for visibility only - this is actually against the release-0.11 branch).


use CTags without .MacOSX/environment.plist on Mac os x Lion

I hope this package  can run with $PATH from from system's shell environment

Yep. I personally am using that identical setup on OSX. The docs should be updated, since the plist hack is really not the preferred way to map to an ideal CTags distribution.

@Greg,
Thanks for your reply.
May i have your latest repo address of the CTags for using ?
B.R
-- jojo
2012-2-2212:04 Greg Williams д

Yep. I personally am using that identical setup on OSX. The docs should be updated, since the plist hack is really not the preferred way to map to an ideal CTags distribution.

Reply to this email directly or view it on GitHub:
https://github.com/SublimeText/CTags/issues/25#issuecomment-4077638

Any version should work fine. Just make sure that your path is instrumented to include a recent-ish version of CTags. If you are on OSX, Homebrew is the most popular option for installing CTags. See the README (which also shows up on the overview page) for some pointers.
The details for accessing the repo are on the overview page for this repo:
git://github.com/SublimeText
Anyways... read-only access git URL is:
git://github.com/SublimeText/CTags.git


训练数据和代码？

楼主你好，最近在学习这块，想知道这部分代码和数据预计什么时候会开源？

Can you share the data first, thank you.

知乎过来的  https://www.zhihu.com/question/391094945/answer/1362816977

We have released our datasets in this link: https://github.com/3d-hand-shape/hand-graph-cnn/tree/master/data


Add yaml parser

This is a much improved prototype for the MetadataDefinitions and a new yaml file structure to build the schema.   It passed my tests.  Assuming Travis builds it with no issues, I think i should be assimilated into master.
You may run a test of the new features with this short script:
from mspasspy import MetadataDefinitions
mdef=MetadataDefinitions()
print("testing readonly on chan")
print(mdef.readonly("chan"))
print("testing is_normalized method on sta")
print(mdef.is_normalized("sta"))
print("testing normalization getters")
print("unique_id_key for sta=",mdef.unique_id_key("sta"))
print("return of table method for key of sta=",mdef.table("sta"))
print("return of method normalize_data for key=sta is ",mdef.normalize_data("sta"))

Change from table to collection is done and just pushed.
We have another branch pending that will definitely have conflicts with this one.   I think this one is now good to merge.   If you do that, I will take care of resolving the conflicts with the other branch pending.   I know exactly what they are and can resolve the differences quickly.


Upgrade exp to latest version 48.0.2

Hello @jevakallio !
Thanks for this awesome work :)
Is that possible to upgrade Expo client?
Bye

Upgraded! Thanks for the contribution 🎉


feat(import-repo-modal): note private repos cause errors

Why
I attempted importing a private Github repo and got an error message that was not related to the repo's visibility.

How
Updated the error copy

Test Plan
Repro:

Make an expo project
Place that expo project in a private github repo
Attempt to import that private github repo

We should probably aim to have more verbose errors in the future, but I think this is an improvement in the near-term.

Hi! Thank for the PR, this is a good suggestion. This has been fixes as part of PR #157.
Cheers


fix(Object): use pure _dirty to prevent unnecessary saves with ar referrences

https://leanticket.cn/tickets/14080

Codecov Report

Merging #515 into master will increase coverage by 0.1%.
The diff coverage is 93.33%.

@@            Coverage Diff            @@
##           master     #515     +/-   ##
=========================================
+ Coverage   67.01%   67.11%   +0.1%
=========================================
  Files          36       36
  Lines        2959     2959
=========================================
+ Hits         1983     1986      +3
+ Misses        976      973      -3

Impacted Files
Coverage Δ

src/object.js
82.9% <93.33%> (+0.51%)
:arrow_up:

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 3230b10...a352e39. Read the comment docs.


Unity WebGL doesn't work because of CORS policy

We're using Unity 2018.4.2 and latest Scatter Sharp lib version.
Local functions like Scatter.Connect() and Scatter.GetIdentity() work fine.
Under WebGL are not working EosBase.CreateTransaction() and EosBase.GetAccount() - functions related to the nodes.
I see this error message in the console:
Access to XMLHttpRequest at 'https://jungle2.cryptolions.io/v1/chain/get_block' from origin 'https://0x.games' has been blocked by CORS policy: Request header field content-type is not allowed by Access-Control-Allow-Headers in preflight response.
UnityLoader.js:4 {"raw_error":"{"ClassName":"EosSharp.Core.Exceptions.ApiException","Message":null,"Data":null,"InnerException":null,"HelpURL":null,"StackTraceString":"  at EosSharp.Unity3D.HttpHandler.BuildApiError (System.String content, System.Int32 statusCode) [0x00000] in <00000000000000000000000000000000>:0 \n  at EosSharp.Unity3D.HttpHandler.CheckUnityWebRequestErrors (UnityEngine.Networking.UnityWebRequest uwr) [0x00000] in <00000000000000000000000000000000>:0 \n  at EosSharp.Unity3D.HttpHandler+d__6`1[TResponseData].MoveNext () [0x00000] in <00000000000000000000000000000000>:0 \n  at

That is a normal error for when a EOS node does not allow cross origin requests. I would suggest you talk with crypto lions if is there a alternative of you.
Or you could spin a local testnet with CORS disabled and connect to that.

It can not be the normal issue because:

we notice the same problem on your node
this problem was not present when we used Scatter's js api

here is the log from your node:
Access to XMLHttpRequest at 'https://nodes.get-scatter.com/v1/chain/get_block' from origin 'https://0x.games/' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: It does not have HTTP ok status.

ok I will look into it in more detail

@mmcs85 Some context: We use Scatter in two games 0xWarriors and 0xRacers. Earlier we used js api. But it stopped working on Scatter 11 (on Scatter 10 still works fine). We decided to switch to Scatter sharp api, because you wrote that it works on WebGL. But it is not. As a result both our games don't work now. Please help us solve this problem

@BestStream I'm happy to help as mutch as I can.
But I want to clear right now that I'm not responsible for your decisions. This is open source software and with the use of any lib there is risk and one should give it a try before rewriting software.

So I have identified the issue. Some nodeos nodes dont return on the CORS preflight stage the header access-control-allow-headers=content-type.
Its required to allow json requests to nodeos nodes.
This worked before and might happen on a a nodeos version just dont know wich one.
Nodes should be configured with this:
access-control-allow-origin = *
access-control-allow-headers = Origin, X-Requested-With, Content-Type, Accept
try this one for jungle: https://api.jungle.alohaeos.com

@mmcs85 Ok, we’ll check https://api.jungle.alohaeos.com for Jungle environment. But I still don’t understand the following

Why was there no problem on Scatter JS API, and it appeared after switching to Scatter Sharp API?
What node to use in the release environment? Will you fix your node nodes.get-scatter.com ?

The issue is related with eosjs and not scatter.

eosjs sends requests with Content-Type: text/plain;charset=UTF-8
eos-sharp sends with Content-Type: application/json.

The correct way of doing ajax requests in json is the way eos-sharp does. And as I stated it worked before.
But somehow Chrome browser started sending on CORS situations where Content-type: application/json the above headers and then some nodeos apis did not responde on the access-control-allow-headers.
There is really two ways to go: change the content-type of eos-sharp.
Change the nodeos configs to support this situations.
On mainnet you can use this: https://api.main.alohaeos.com
I'm talking with @nsjames on the best solution and to look into the node

@mmcs85  I have checked and https://api.jungle.alohaeos.com works fine for Jungle! Thanks!

@mmcs85  https://api.main.alohaeos.com - it works too. Thanks for help! The problem is resolved


Update Blog “why-is-it-important-for-the-board-of-pharmacy-specialties-bps-to-recognize-pharmacy-informatics-as-a-specialty”

Automatically generated by Netlify CMS

👷 Deploy Preview for cool-toffee-ab3b21 processing.

Name
Link

🔨 Latest commit
81989cc2ba29c9d23c6f7f666d967da9761715c2

🔍 Latest deploy log
https://app.netlify.com/sites/cool-toffee-ab3b21/deploys/64a44aa18a4b190008ffa90e


Check for blank fields as any number of whitespace characters

Blank fields are now considered as any number of whitespace characters (space, tab, newline, non-breakable space, …).
I separated the test for spaces and the test for others characters. This allows also to test inside quotes (needed by the newline). Feel free to merge the tests if you prefer.
Fixes #16

seems great. Thanks!

Included in 3.0.2, released just now.

(sorry for the delay, I was waiting for CI to pass and forgot to come back to it!)

No problem! There is no hurry! Thanks.


Question: Node or browser?

Question: Node or browser?

Hi @rahulmehtas, if you're using the library in server-side JavaScript (such as Node) use this import:
import ollama from 'ollama'
If you're using JavaScript which runs in the browser (such as a basic script) use this import:
import ollama from 'ollama/browser'
Hope that helps.

Thanks for detailed instruction.


Request: Always resolve source maps on Errors

For this worker:
export default {
  fetch: async () => {
    console.error(new Error("Pretty error test"));
    return new Response("OK");
  },
};

I would expect the console.error call to print the error with a source mapped stack trace (like it does if I throw the error), but it doesn’t. I hunted through the Miniflare code and found that we do construct source maps for thrown errors, but don’t override prepareStackTrace by default.
I tried removing the line Error.prepareStackTrace = originalPrepareStackTrace; in getSourceMapper() and calling getSourceMapper() on launch, but it didn’t seem to override prepareStackTrace. Nor did passing NODE_OPTIONS=--source-map-support to wrangler dev.
I’m not expecting this to be fixed anytime soon (although I’ll happily take pointers on how to fix it myself!), but I wanted to raise the issue so it’s tracked somewhere. Cheers again for all the hard work on this repo, it’s not unappreciated ❤

I might be mistaken but I think that, in latest Miniflare 3, the error comes already stringified from Workerd and Miniflare just prints it without modifying the stack trace. Somewhere around here. Maybe that's the place to add sourcemaps 🤔

Can’t believe I didn’t see that—thank you! I’ll leave it to you to whether you wanna close this issue now or after that ships :)


Create ISSUE_TEMPLATE.md

attempting to quell the stream of invalid issues relating to OS compatibility

's Not a bad idea.  LGTM.


GHG Center: Add profile options for workshop

Refs https://github.com/NASA-IMPACT/veda-jupyterhub/issues/32
Adding some compute-optimized options for containers.
This:

Adds a c5.4xlarge instance type to the eksctl config to create a nodepool with that instance type (compute optimized)
Adds resource options to the profile list for the compute optimized containers

Questions:

Should we include the workshop in the display_name or so to distinguish these options?
@yuvipanda please let me know if the way the resources are specified seems okay - I did some rough conversions between bits and GiB and put in values that seemed appropriate
Also, I've set the cpu_limit to be what's requested, but the cpu_guarantee a bit lower in line with the other resource options, lemme know if that seems okay

@yuvipanda let me know if these seem okay - happy to make any changes :)
cc @sunu @slesaad

@batpad looks like the CI found a real issue too.

@yuvipanda updated the PR based on our conversation:

Remove the additional node type and just use an existing node-pool for now while Sourish is testing
Added a comment to explain this and listing the steps to take after Sourish decides what resource allocations are ideal, including then figuring out the ideal node type, etc.

Thanks :-)

hmm, not sure why pre-commit is failing :(

hmm, not sure why pre-commit is failing :(

oh, lol - maybe my YAML commenting style?

@yuvipanda fixed all the things, thanks you and pre-commit for the catches.

ty for the fix!


Add WikiCFP to CFP aggregators list

WikiCFP focuses on more academic science and technology conferences, which tend to be underrepresented in language- or location-focused lists.

Thank you, @edunham Seems like a good resource to me. Merged, if you're ok with that also, @matteofigus

Sure! Thanks @edunham and @peterkokot


Dnslog init failed!

Hello!
I've added the .jar file to my Burp Extensions extender tab and I get this following error:
initDomain failed: www.dnslog.cn: No address associated with hostname

This plugin needs to use the echo of dnslog.cn as the basis for judgment, please confirm whether you can access http://www.dnslog.cn normally.

can not access this can you share any alternative.


Cannot resolve method with()

Facing error Cannot resolve method with() while fetching bitmap
Bitmap result= Picasso.get().
load(imgUrl).
with(getActivity());

That is Picasso, not OkHttp, and Picasso has no with API.


Highlights only work with -nick, not -word

The IrssiNotifier script doesn't recognize inlined highlights (namely, those set up as -word instead of -nick) and thus never sends those notifications. The result is that I'm never notified for channel highlights because all my highlights use -word.
I use hilightwin.pl, so perhaps it could be configurable to trip notify whenever the hilight window has a new entry (other than "Day changed")?
On the other hand, hilightwin.pl can recognize -word highlights (via ($dest->{level} & (MSGLEVEL_HILIGHT))), so I should think irssinotifier.pl should be able to as well.

Not sure if this works as supposed based on #178.


SAA-157 API Endpoint: activity schedule instances

Refinement of api for activity schedule instances - prisoner number becomes optional request param
this allows us to select all the activity scheduled instances for a prison or a prisoner

Codecov Report
Base: 74.68% // Head: 74.32% // Decreases project coverage by -0.35% :warning:

Coverage data is based on head (95ee1fd) compared to base (acc6fd2).
Patch coverage: 37.50% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #63      +/-   ##
============================================
- Coverage     74.68%   74.32%   -0.36%
  Complexity      287      287
============================================
  Files            67       67
  Lines          1035     1040       +5
  Branches         25       26       +1
============================================
  Hits            773      773
- Misses          243      247       +4
- Partials         19       20       +1

Impacted Files
Coverage Δ

...smanagementapi/service/ScheduledInstanceService.kt
64.28% <28.57%> (-35.72%)
:arrow_down:

...api/resource/ActivityScheduleInstanceController.kt
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Update to latest CoffeeNet parent version 0.32.0

New CoffeeNet parent version 0.32.0 was released! Please check https://github.com/coffeenet/coffeenet-starter/blob/master/CHANGELOG.md for more information

Pull Request Test Coverage Report for Build 290

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.827%

Totals

Change from base Build 288:
0.0%

Covered Lines:
220

Relevant Lines:
237

💛  - Coveralls


Update Node.js minutely probes documentation

This commit updates the documentation about Node.js minutely probes to document the existence of the default minutely probe (fixes #532), including the hostname tag added in appsignal/appsignal-nodejs#543. It also documents how to disable the default minutely probe and how to disable the minutely probe system, taking into account the change in behaviour for the enableMinutelyProbes config option introduced in appsignal/appsignal-nodejs#547.
I also removed several references about the probes system being an EventEmitter. While this is technically true, using it as an EventEmitter directly makes no sense in the context of how the probes are meant to be used. You can't simply register a listener and expect it to be called every minute; instead, you have to use the register method we provide for that to happen, which means there's no point to all the wording about how its usage should be familiar because it's just an EventEmitter. In fact, we should probably just use an EventEmitter internally, instead of Probes extending EventEmitter.
@shairyar Feel free to change anything in this PR as you see fit for the Node.js documentation revamp 📖💪

I have changed references to the as-of-yet unpublished version 2.2.10 to refer to 2.3.0, since the next version will be a minor version, not a patch.

Waiting for #531 (Revamp nodejs docs) to be merged, to rebase this on top of its changes afterwards.

Merging this, as the changes it refers to are now live and the PRs don't seem to have any overlap.


fix(types): PropsWithChildren type argument issue

Problem
The current CalendarProps type definition is incorrect, leading to type errors when using the component. This issue arises because React.PropsWithChildren is a generic type that requires a type argument to specify the component's props.
Changes
This PR fixes the type definition by correctly combining React.PropsWithChildren with CalendarProps. This ensures proper type checking and prevents potential runtime errors.
Screenshots

Contribution checklist

[x] I have followed the contributing guidelines.
[x] I have added sufficient test coverage for my changes.
[x] I have formatted my code with Prettier and checked for linting issues with ESLint for code readability.

looks like the linter is failing, can you review and fix?


added json rule match in java

eg:
  @Test
  public void should_match_rule_json() throws Exception {
        final String ruleJsonText = Jsons.toJson("#value['foo'] == 'bar2'");
//        final String ruleJsonText = Jsons.toJson(of("foo", "#value == 'bar2'"));
        final String requestJsonText = Jsons.toJson(of("foo", "bar2"));
        server.request(rule(json(ruleJsonText))).response("foo");
        running(server, () -> assertThat(helper.postContent(root(), requestJsonText), is("foo")));
    }

result:
Results: SUCCESS (1 tests, 1 successes, 0 failures, 0 skipped)
BUILD SUCCESSFUL in 4s

eg:
  @Test
    public void should_match_rule_json_with_resource() throws Exception {

        final String ruleJsonText = "\"#value[name].equalsIgnoreCase('linus')||#value[name]=='Dreamhead'\"";
//        final String ruleJsonText = "{\"name\":\"#value.equalsIgnoreCase('linus')\"}";
        final String requestJsonText = "{\"name\":\"Linus\",\"location\":\"Portland\"}";
        final String responseText = "${req.json.name} is such a cool guy from ${req.json.location}.";
        server.request(rule(json(ruleJsonText))).response(template(responseText));
        running(server, () -> assertThat(helper.postContent(root(), requestJsonText), is("Linus is such a cool guy from Portland.")));
    }

result:
Results: SUCCESS (1 tests, 1 successes, 0 failures, 0 skipped)
BUILD SUCCESSFUL in 5s

rule match in java（commit- d37939a）：
 @Test
    public void should_rule() throws Exception {
        server.request(rule(uri("#value.startsWith('/foo')"))).response("bar");

        running(server, () -> {
            assertThat(helper.get(remoteUrl("/foo/a")), is("bar"));
            assertThat(helper.get(remoteUrl("/foo/b")), is("bar"));
        });
    }

result:
Results: SUCCESS (1 tests, 1 successes, 0 failures, 0 skipped)
BUILD SUCCESSFUL in 11s
4 actionable tasks: 3 executed, 1 up-to-date

 @Test
    public void should_rule_for_resource() throws Exception {
        server.request(rule(header("foo"), "#value.endsWith('bar')")).response("bar");

        running(server, () -> {
            assertThat(helper.getWithHeader(root(), of("foo", "Abar")), is("bar"));
            assertThat(helper.getWithHeader(root(), of("foo", "Bbar")), is("bar"));
        });
    }

result:
Results: SUCCESS (1 tests, 1 successes, 0 failures, 0 skipped)
BUILD SUCCESSFUL in 4s
4 actionable tasks: 1 executed, 3 up-to-date

rule match in json（commit- 569df57):
the content of file:
[
  {
    "request": {
      "uri": {
        "rule": "#value.startsWith('/foo')"
      }
    },
    "response": {
      "text": "uri_match"
    }
  },
  {
    "request": {
      "uri": "/text-match",
      "text": {
        "rule": "#value.startsWith('foo')"
      }
    },
    "response": {
      "text": "text_match"
    }
  },
  {
    "request": {
      "uri": "/header-match",
      "headers": {
        "content-type": {
          "rule": "#value.startsWith('application')"
        }
      }
    },
    "response": {
      "text": "header_match"
    }
  },
  {
    "request": {
      "uri": "/query-match",
      "queries": {
        "foo": {
          "rule": "#value.startsWith('b')"
        }
      }
    },
    "response": {
      "text": "query_match"
    }
  },
  {
    "request": {
      "uri": "/json-match",
      "json_rule": {
        "name": "#value.equalsIgnoreCase('Linus')"
      }
    },
    "response": {
      "text": "json_match"
    }
  }
]

The result of executing Mocoruletest.java  ：

GET /query-match HTTP/1.1
content-length: 0
Connection: Keep-Alive
Host: localhost:12306
Accept-Encoding: gzip,deflate

24 九月 2021 11:37:04 [pool-1-thread-2] INFO  Response return:

HTTP/1.1 200
Content-Length: 11
Content-Type: text/plain; charset=utf-8

query_match

24 九月 2021 11:37:04 [pool-1-thread-2] INFO  Request received:

GET /query-match HTTP/1.1
content-length: 0
Connection: Keep-Alive
Host: localhost:12306
Accept-Encoding: gzip,deflate

24 九月 2021 11:37:04 [pool-1-thread-2] INFO  Response return:

HTTP/1.1 200
Content-Length: 11
Content-Type: text/plain; charset=utf-8

query_match

24 九月 2021 11:37:04 [Test worker] INFO  Server stopped.
24 九月 2021 11:37:04 [Test worker] INFO  Server is started at 12306
24 九月 2021 11:37:04 [pool-1-thread-14] INFO  Request received:

GET /foo/bar HTTP/1.1
content-length: 0
Connection: Keep-Alive
Host: localhost:12306
Accept-Encoding: gzip,deflate

24 九月 2021 11:37:04 [pool-1-thread-14] INFO  Response return:

HTTP/1.1 200
Content-Length: 9
Content-Type: text/plain; charset=utf-8

uri_match

24 九月 2021 11:37:04 [pool-1-thread-14] INFO  Request received:

GET /foo/blah HTTP/1.1
content-length: 0
Connection: Keep-Alive
Host: localhost:12306
Accept-Encoding: gzip,deflate

24 九月 2021 11:37:04 [pool-1-thread-14] INFO  Response return:

HTTP/1.1 200
Content-Length: 9
Content-Type: text/plain; charset=utf-8

uri_match

24 九月 2021 11:37:04 [Test worker] INFO  Server stopped.
24 九月 2021 11:37:04 [Test worker] INFO  Server is started at 12306
24 九月 2021 11:37:04 [pool-1-thread-10] INFO  Request received:

GET /header-match HTTP/1.1
content-length: 0
Connection: Keep-Alive
Host: localhost:12306
Accept-Encoding: gzip,deflate
Content-Type: application/json

24 九月 2021 11:37:04 [pool-1-thread-10] INFO  Response return:

HTTP/1.1 200
Content-Length: 12
Content-Type: text/plain; charset=utf-8

header_match

24 九月 2021 11:37:04 [pool-1-thread-10] INFO  Request received:

GET /header-match HTTP/1.1
content-length: 0
Connection: Keep-Alive
Host: localhost:12306
Accept-Encoding: gzip,deflate
Content-Type: application/xml

24 九月 2021 11:37:04 [pool-1-thread-10] INFO  Response return:

HTTP/1.1 200
Content-Length: 12
Content-Type: text/plain; charset=utf-8

header_match

24 九月 2021 11:37:04 [Test worker] INFO  Server stopped.
24 九月 2021 11:37:04 [Test worker] INFO  Server is started at 12306
24 九月 2021 11:37:04 [pool-1-thread-13] INFO  Request received:

POST /json-match HTTP/1.1
content-length: 19
Connection: Keep-Alive
Host: localhost:12306
Accept-Encoding: gzip,deflate
Content-Type: text/plain; charset=UTF-8

{
	"name":"Linus"
}

24 九月 2021 11:37:04 [pool-1-thread-13] INFO  Response return:

HTTP/1.1 200
Content-Length: 10
Content-Type: text/plain; charset=utf-8

json_match

24 九月 2021 11:37:04 [pool-1-thread-13] INFO  Request received:

POST /json-match HTTP/1.1
content-length: 19
Connection: Keep-Alive
Host: localhost:12306
Accept-Encoding: gzip,deflate
Content-Type: text/plain; charset=UTF-8

{
	"name":"linus"
}

24 九月 2021 11:37:04 [pool-1-thread-13] INFO  Response return:

HTTP/1.1 200
Content-Length: 10
Content-Type: text/plain; charset=utf-8

json_match

24 九月 2021 11:37:04 [Test worker] INFO  Server stopped.
24 九月 2021 11:37:04 [Test worker] INFO  Server is started at 12306
24 九月 2021 11:37:04 [pool-1-thread-3] INFO  Request received:

POST /text-match HTTP/1.1
content-length: 6
Connection: Keep-Alive
Host: localhost:12306
Accept-Encoding: gzip,deflate
Content-Type: text/plain; charset=UTF-8

foobar

24 九月 2021 11:37:04 [pool-1-thread-3] INFO  Response return:

HTTP/1.1 200
Content-Length: 10
Content-Type: text/plain; charset=utf-8

text_match

24 九月 2021 11:37:04 [pool-1-thread-3] INFO  Request received:

POST /text-match HTTP/1.1
content-length: 7
Connection: Keep-Alive
Host: localhost:12306
Accept-Encoding: gzip,deflate
Content-Type: text/plain; charset=UTF-8

fooblah

24 九月 2021 11:37:04 [pool-1-thread-3] INFO  Response return:

HTTP/1.1 200
Content-Length: 10
Content-Type: text/plain; charset=utf-8

text_match

24 九月 2021 11:37:04 [Test worker] INFO  Server stopped.
Disconnected from the target VM, address: 'localhost:1726', transport: 'socket'
Results: SUCCESS (5 tests, 5 successes, 0 failures, 0 skipped)
BUILD SUCCESSFUL in 5s

@dreamhead  If you have any suggestions, please let me know in time.


add roles, roles_mapping ,indices on kibana can't synchronize to configuration file of sgconfig

I found that these new configurations could not be synchronized to the relevant configuration files on sgconfig,When I add a roles,role_mapping,internal user to the kibana page.and when
i upload the changes via sgadmin ,configuration files is gone on kibana page . I wonder why settings isn't synchronous.

Please do not ask questions here.
For that refer to https://groups.google.com/forum/#!forum/search-guard or, if you are a paying customer, contact our support.
Github is a place only for reporting bugs or feature requests.
If you report a bug please include always the following informations or the issue will likely be closed as invalid:

Search Guard and Elasticsearch version
Kibana plugin version
JVM version and operating system version
Number of nodes in your cluster
Description of the bug
sg_*.yml config files
kibana.yml file
elasticsearch.yml file

These informations are optional but are very valuable and
typically would lead to get bugs fixed faster:

Steps to reproduce
Logfiles on DEBUG level
A failing unittest


[SPARK-18950][SQL] Report conflicting fields when merging two StructTypes

What changes were proposed in this pull request?
Currently, StructType.merge() only reports data types of conflicting fields when merging two incompatible schemas. It would be nice to also report the field names for easier debugging.
How was this patch tested?
Unit test in DataTypeSuite.
Print exception message when conflict is triggered.

Can one of the admins verify this patch?

This is actually the message users face in some cases. Isn't it :)?
val df1 = spark.range(10).selectExpr("id as intcol", "cast(id as int) as longcol")
df1.write.parquet("/tmp/a")
val df2 = spark.range(10).selectExpr("id as intcol", "id as longcol")
df2.write.parquet("/tmp/b")
spark.read.option("mergeSchema", true).parquet("/tmp/a", "/tmp/b").show()

Before
...
Caused by: org.apache.spark.SparkException: Failed to merge field longcol: Failed to merge incompatible data types IntegerType and LongType
  at org.apache.spark.sql.types.StructType$$anonfun$merge$1$$anonfun$apply$3.apply(StructType.scala:480)
  at org.apache.spark.sql.types.StructType$$anonfun$merge$1$$anonfun$apply$3.apply(StructType.scala:471)
  at scala.Option.map(Option.scala:146)
...

After
...
Caused by: org.apache.spark.SparkException: Failed to merge incompatible data types IntegerType and LongType
  at org.apache.spark.sql.types.StructType$.merge(StructType.scala:515)
  at org.apache.spark.sql.types.StructType$$anonfun$merge$1$$anonfun$apply$3.apply(StructType.scala:473)
  at org.apache.spark.sql.types.StructType$$anonfun$merge$1$$anonfun$apply$3.apply(StructType.scala:471)
...

BTW, it looks the test in DataTypeSuite was not added by mistake.

Thanks for the review @HyukjinKwon !
Are your stacktrace of Before and After swapped?
Do you mean the message is confusing? How about correcting it to Failed to merge field longcol: incompatible data types IntegerType and LongType?
For the missing test case, do you mean a merge case that are not StructType as in your example?
Or is it something else that I'm missing?

Dho, yeap, I just edited. I just simply meant I support this PR :).

Can one of the admins verify this patch?

ok to test

@bravo-zhang Sorry for the late response. Could you please also add a test case for capturing the new error message?

Test build #77970 has started for PR 16365 at commit 9bc9f8a.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77970/
Test FAILed.

retest this please

Test build #77999 has started for PR 16365 at commit 9bc9f8a.

Test build #77999 has finished for PR 16365 at commit 9bc9f8a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77999/
Test PASSed.

ping @bravo-zhang for adding the test.

Test build #79921 has started for PR 16365 at commit 54a898f.

Test build #79921 has finished for PR 16365 at commit 54a898f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79921/
Test PASSed.

@HyukjinKwon @gatorsmile Test added.

retest this please

Test build #80064 has started for PR 16365 at commit 54a898f.

Test build #80064 has finished for PR 16365 at commit 54a898f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80064/
Test PASSed.

Test build #80088 has started for PR 16365 at commit f500a11.

Test build #80088 has finished for PR 16365 at commit f500a11.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80088/
Test FAILed.

Test build #80090 has started for PR 16365 at commit f4892c9.

Test build #80090 has finished for PR 16365 at commit f4892c9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80090/
Test PASSed.

Thanks! Merging to master.


How to Use Parsley Validation?????

What kind of issue is this? (put 'x' between the square brackets)

[ ] Question. This issue tracker is not the place for questions. If you want to ask how to do
something, or to understand why something isn't working the way you expect it to, use
http://stackoverflow.com/questions/ask .
Provide working code, starting from http://codepen.io/marcandre/pen/jqbzyN?editors=101.
We monitor the tag parsley.js.

[ ] Bug report. If you’ve found a bug, you must provide a minimal example in a CodePen,
starting from http://codepen.io/marcandre/pen/jqbzyN?editors=101 .

[ ] Feature Request. Make sure there's no good way to do what you want first;
consider asking on http://stackoverflow.com/questions/ask first.

How to Use Parsley Validation?????


Issue running scan.py (ValueError)

Hi there,
I am new to python and monocle and enjoying the challenge - but please bare with me on the below as I haven't got a clue on how I can get past the following error to get the map up and running.
Firstly I followed all the steps on the github site, I am running postgresql and have set up all the details in the config.py as per the site.
The error I get right now is when running scan.py is as follows:
c:\Monocle-develop>scan.py
Traceback (most recent call last):
File "C:\Monocle-develop\monocle\worker.py", line 87, in init
self.account = self.extra_queue.get_nowait()
File "", line 2, in get_nowait
File "C:\Users\Lenovo\AppData\Local\Programs\Python\Python36-32\lib\multiproce
ssing\managers.py", line 772, in _callmethod
raise convert_to_error(kind, result)
queue.Empty
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Monocle-develop\monocle\worker.py", line 90, in init
self.account = self.captcha_queue.get_nowait()
File "", line 2, in get_nowait
File "C:\Users\Lenovo\AppData\Local\Programs\Python\Python36-32\lib\multiproce
ssing\managers.py", line 772, in _callmethod
raise convert_to_error(kind, result)
queue.Empty
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\Monocle-develop\scan.py", line 218, in
main()
File "C:\Monocle-develop\scan.py", line 203, in main
overseer.start(args.status_bar)
File "C:\Monocle-develop\monocle\overseer.py", line 92, in start
self.workers = tuple(Worker(worker_no=x) for x in range(conf.GRID[0] * conf.
GRID[1]))
File "C:\Monocle-develop\monocle\overseer.py", line 92, in
self.workers = tuple(Worker(worker_no=x) for x in range(conf.GRID[0] * conf.
GRID[1]))
File "C:\Monocle-develop\monocle\worker.py", line 92, in init
raise ValueError("You don't have enough accounts for the number of workers s
pecified in GRID.") from e
ValueError: You don't have enough accounts for the number of workers specified i
n GRID.
Now, I initially had 5 accounts and quite a big area, so I reduced the area to a very small area 1-2 houses and added more accounts - I now have 10 accounts in the accounts.csv file.
Any clue as to what I can do to fix the error? perhaps an earlier error/exception is causing this?
Thanks in advance for any assistance, let me know if you need more info on my set up I will happily provide it.
Regards
Jay

found (and reduced) the GRID variable in the config file : )


axum: adding routes

Current arrangement cleans up lib.rs as it moves relevant fns to where they should be. E.g. if and when we want to add /v2/ we simply add .merge(v2::router(ctx.clone())) to our app in lib.rs and leave the rest of it alone.
Each route can be easily found, as you need to look in their respective file. In total we'll create 5 more files but it'd be worth it as helper functions will also move to where they belong, and so will imports.
This will make each file more specialized and more readable.


Revert "llvm: use proper syntax for --config option"

This reverts commit 577d47f3f1ea6c624b76c0afb1bc3089f79605f2.
The --config= syntax with the equal sign does not work with clang 15 which is less than one year old (https://releases.llvm.org/). It does not work with clang 14 either.
clang --verbose
clang version 15.0.7

clang --help | grep config

  -cl-std=<value>         OpenCL language standard to compile for.
  --config   <value>      Specifies configuration file
  --cuda-feature=<value>  Manually specify the CUDA feature to use

clang: error: unsupported option '--config=/home/runner/work/...

The reverted commit mentioned "issues in some situations" without providing any example or other information.
This revert fixes the SOF fuzzer build, see more details in #61778.

So --config=file.cfg was added in clang 16, which was released only a few months ago.
The older -config  file.cfg syntax was kept and I have found no plan to deprecate it.
clang   --config   --config=
----------------------------
14        YES         NO
15        YES         NO
16        YES        YES

Longer story in #61778

Requiring clang 16 is a big change and since ubuntu 22.04 LTS has clang 14, requiring clang >14 is a no from me

Ya this never was my intention to require clang 16 :) I just thought I was fixing a bug (see #61778 discussion) by using the alternate --config= syntax, and simply didn't realize it had only been introduced in 16.0.0.
FWIW CI is using clang 16, and it might actually make sense to use clang ~14 for now (also current version on OSX).


WIP: Support for Flang on Windows

This adds support for using flang on Windows. It appears @xoviat beat me to it, but I figured I'd post what got everything building for me for reference as well.
Things that still need to be addressed:

[ ] Verify that none of these changes break anything when building with gfortran in CMake
[ ] Check that the TIMER variable is chosen correctly for flang
[ ] Verify that all the desired routines are actually exported from the compiled dll.

Sorry. I guess I was a little too eager.

That or I was a little too slow. No worries.

Pending insertinterestingnamehere/OpenBLAS#1.

Quick word on how I arrived at the lists of routines present in the CMake files for lapack and lapacke: I copied over the file lists from the more recent vendored copy of LAPACK in the repo and then adapted them by removing multiply defined symbols that conflicted with existing C files found elsewhere in the repo and adding the source files for any missing symbols (like the matgen routines).

Appears to work fine with gfortran/gcc 7.2 from MinGW-w64. That build includes more tests by default than the current clang-cl/flang build though.

Looks like we have some problems. I would recommend b0afe10.

cherry-picked. Thanks. It's odd that that wasn't needed on my local machine.

@insertinterestingnamehere, can you remove the last commit and add https://github.com/isuruf/OpenBLAS/commit/98c317c579f355b7b5de2ff7a0f56e54b9b65a06 ?
You wouldn't have seen this locally because you probably were running cmake from a build directory and  the gensymbol script mistakenly assumes that the current working directory is the script directory.

Sorry for the delay there. I've cherry picked that one instead.

Yep, I've been running it from a separate build directory. That explains it.

Isn't creating a separate build directory generally recommended by cmake or projects that use it ? (A change that allows either kind of builds to work is still good in my opinion)

Isn't creating a separate build directory generally recommended by cmake or projects that use it ?

yes, it is recommended. Some projects explicitly disable building in tree. I'm okay with either.

Alright, I just checked that the correct TIMER setting is NONE. It is. At least for the time being, etime is not provided by flang. Since @isuruf already handled the code to make sure all the correct symbols are exported, this should be done now.

@insertinterestingnamehere, can you also check that exports/gensymbol is not outdated?

Sure, you mean that it exports symbols consistent with LAPACK 3.7?

Okay, 3.7.0 support appears to have been added in https://github.com/insertinterestingnamehere/OpenBLAS/commit/a836fe8ec1250019d474b8dea12503b8b5ed5eae. The list added there is a superset of the routines listed as new in the 3.7.0 release notes (probably because auxiliary routines aren't mentioned in the release notes). Not all the routines mentioned in the release notes have their corresponding LAPACKE wrappers included for some reason (e.g. lapacke_cgelq).

Omission of LAPACKE_cgelq &friends was actually an oversight in netlib 3.7.0 as well, see https://github.com/Reference-LAPACK/lapack/pull/118 . I think I copied that PR as part of  #1095, but in any case it seems I never thought to check gensymbol.

Got it, so for LAPACKE, we need to add *geqr, *gelq, *gemlq. Thanks. That limits the searching I need to do pretty considerably. I'll go ahead and add that here.

Okay, I believe that should fix it. Builds all still pending.
Note: in this PR, I got the updated lapack.cmake by copying lists of source files from the netlib version and then removing files corresponding to multiply defined symbols. Do you remember if there were any missing fortran routines there that I'd need to add back in?

I do not remember, but any multiply defined symbols can only come from the small set of functions reimplemented in "lapack" (as opposed to "lapack-netlib")

Yah, those are the same routines I had to remove from the copied lists of sources to originally get this to work. Thanks for the input.

It looks like we need additional changes from https://github.com/Reference-LAPACK/lapack/pull/118. It may be worth just upgrading to 3.7.1 to get the relevant bugfixes. Thoughts?

If you're up for it, then I don't see why not. It only depends on how much work you are willing to do.

That seems like the easier way out of the current rabbit hole. It's better to just update than to keep picking out additional patches. I'm pretty much out of time to work on this today though, so it'll probably take another day or two.

That's fine with me.

My suggestion is to crawl out of the rabbit hole back to the working code you had about four hours ago, and leave fixing the unrelated, pre-existing issues with missing functions to another PR. (I had optimistically claimed to provide the PR bringing 3.7.1 to OpenBLAS "in a week or so" in mid august, #1273 but things went downhill from there for me)

Yeah there are some changed makefile targets in 3.7.1 and the introduction of a LAPACKE_WITH_TMG define as noted in #1273. (Possibly makes sense to update to netlib git rather than 3.7.1 even...)

FYI netlib appears to be preparing a 3.8.0 release right now.

I put the 3.8.0 update into PR #1366 (Not that I wanted to make your work obsolete, it just seemed easier to me to do it that way rather than build on your branch. Still ended up with a push-pull game worthy of Alice in Wonderland)

Awesome. I'll try that branch out. Thanks for taking care of that. I wasn't meaningfully further than the work already shown here, and even that was basically just backporting a few patches to 3.7.0.


fix: empty breaking or additive items in changelog

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

/azp run

/azp run go - aztemplate - ci


Unabled to find OTP to update its sentTo field

Added MFA. Use instruction - https://pocketbase.io/docs/authentication/#multi-factor-authentication
Received: WARN Unabled to find OTP to update its sentTo field (either it was already deleted or the id is nonexisting)

The email was sent correctly. And has OTP code.
MFA: email + OTP
Tables in database:
_otps and _mfas is empty
Code like this:

PocketBase v0.23.5

I'm not able to reproduce it. Here is a minimal pb_public/index.html example:
<!DOCTYPE html />

    <meta charset="utf-8" />
    <title>Test</title>

Make sure that:

you are testing against the correct environment
the OTP you are targeting hasn't expired
you don't have custom hooks that could interfere with the _mfas/_otps records creation

If you are still not able to identify the culprit, please create a minimal and I'll try to investigate it further but as of now it works correctly for me.

Also note that in the dual screenshot, the received otpId from left differs from the submitted one on the right side.

Also note that in the dual screenshot, the received otpId from left differs from the submitted one on the right side.

This is not important in this moment. Question about empty tables.
I does not see any INSERT to _otps or _mfas in DB on --dev mode.

Does not use any hooks for _mfas
Duration OTP - 180s

Use PB as framework

There is a cron that automatically delete expired tokens every 30mins so it is possible what the missing records in your case to be from that.
Again it works correctly for me. Please test the above minimal example and if you are still having the same issue, feel free to create your own minimal repo that triggers the error and I'll try to investigate it.

I found issue. This is the refactoring bug after update code  to v0.23. On some my global hook -> OnRecordCreate
return nil ---> return e.Next().
You were right about hook. I didn't finish watching


Windows10 Error building extension

Windows10
VS 2017
PyTorch 1.3.1
CUDA 10.1
Any idea? Thanks
[3/3] "C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.16.27023\bin\Hostx64\x64/link.exe" fused_bias_act.o fused_bias_act_kernel.cuda.o /nologo /DLL c10.lib torch.lib torch_python.lib _C.lib "/LIBPATH:C:\Program Files\Python37\libs" "/LIBPATH:C:\Program Files\Python37\lib\site-packages\torch\lib" "/LIBPATH:C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v10.1\lib/x64" cudart.lib /out:fused.pyd

FAILED: fused.pyd

"C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.16.27023\bin\Hostx64\x64/link.exe" fused_bias_act.o fused_bias_act_kernel.cuda.o /nologo /DLL c10.lib torch.lib torch_python.lib _C.lib "/LIBPATH:C:\Program Files\Python37\libs" "/LIBPATH:C:\Program Files\Python37\lib\site-packages\torch\lib" "/LIBPATH:C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v10.1\lib/x64" cudart.lib /out:fused.pyd

   Creating library fused.lib and object fused.exp

fused_bias_act_kernel.cuda.o : error LNK2019: unresolved external symbol "__declspec(dllimport) public: __cdecl c10::cuda::CUDAStream::operator struct CUstream_st *(void)const " (__imp_??BCUDAStream@cuda@c10@@QEBAPEAUCUstream_st@@XZ) referenced in function "class at::Tensor __cdecl fused_bias_act_op(class at::Tensor const &,class at::Tensor const &,class at::Tensor const &,int,int,float,float)" (?fused_bias_act_op@@YA?AVTensor@at@@AEBV12@00HHMM@Z)

fused_bias_act_kernel.cuda.o : error LNK2019: unresolved external symbol "__declspec(dllimport) class c10::cuda::CUDAStream __cdecl c10::cuda::getCurrentCUDAStream(short)" (__imp_?getCurrentCUDAStream@cuda@c10@@YA?AVCUDAStream@12@F@Z) referenced in function "class at::Tensor __cdecl fused_bias_act_op(class at::Tensor const &,class at::Tensor const &,class at::Tensor const &,int,int,float,float)" (?fused_bias_act_op@@YA?AVTensor@at@@AEBV12@00HHMM@Z)

fused.pyd : fatal error LNK1120: 2 unresolved externals

ninja: build stopped: subcommand failed.

Seems like that you need to do some configurations for compiling in windows: https://github.com/pytorch/pytorch/issues/15757#issuecomment-452113121 https://pytorch.org/docs/stable/notes/windows.html#cpp-extension

Seems like that you need to do some configurations for compiling in windows: pytorch/pytorch#15757 (comment) https://pytorch.org/docs/stable/notes/windows.html#cpp-extension

@rosinality  I have used 'x86_x64 Cross Tools Command Prompt for VS 2017' but still have the same troubles about compiling these CUDA extensions under win10. I have read above links carefully, but unfortunately I am new for torch c++ extensions and could not to find the point. Would you plz give some more detailed hints?

@neoragex2002 I don't know much about compiling cuda extensions in windows, too. I think you need to check you can compile cuda kernels in windows first. Then please give me compiler error logs.

@neoragex2002 Finally I found a hack :D
in .\op\fused_act.py:

in .\op\upfirdn2d.py:

The ldflags of 'c10_cuda.lib' is the key to solve the problem. sorry for my kinda messy screenshots!

@neoragex2002 Interesting! Glad to hear that you have resolved problems.

@neoragex2002 Interesting! Glad to hear that you have resolved the problem.

I think this way of hacking may not be portable (I didn't test in linux). so it is not a patch, but a suggestion for your consideration. Thanks for your help!


rotaryWithScroll not working with ScalingLazyColumn

Hi, I created a very basic sample with a ScalingLazyColumn using rotaryWithScroll to be able to scroll via the crown/RSB. But if I rotate the RSB, the list isn't scrolling. I also tried it with onRotaryScrollEvent which is suggested here which works.
So is it intended to use the old onRotaryScrollEvent or is rotaryWithScroll with ScalingLazyColumn just not working correctly? Or am I doing sth. wrong here?
Used on Pixel Watch with Wear OS 4 and Horologist 0.5.14
@OptIn(ExperimentalHorologistApi::class)
@Composable
fun WearApp() {
    MyApplicationTheme {

        val scrollState = rememberScalingLazyListState()
        val focusRequester = rememberActiveFocusRequester()
        LaunchedEffect(Unit) {
            focusRequester.requestFocus()
        }

        ScalingLazyColumn(
            modifier = Modifier
                .fillMaxWidth()
                .rotaryWithScroll(
                    focusRequester = focusRequester,
                    scrollableState = scrollState,
                ),
        ) {
            items(List(100) {i -> "$i"}) {
                Box(modifier = Modifier
                    .fillMaxWidth()
                    .height(50.dp),) {
                    Text(text = it)
                }
            }
        }
    }
}

You need to pass the scrollState into the ScalingLazyColumn.
Also you don't need the LaunchedEffect.  rememberActiveFocusRequester is enough.


Propogate missings with arithmatic

This issue: https://github.com/JuliaLang/julia/issues/28570 points out that arithmetic with dates and missing values throws an error. Instead, dates should behave like numbers in this instance, propagating missing values.
This PR adds 4 methods for operations with missing and TimeType. +, -, and both orders for each.
However it is likely this adds too much behavior, as it seems not all TimeType subtypes support + (you can't add years).  I would appreciate some guidance for more specific methods to add.

This github repository is deprecated; please make PRs against the stdlib/Dates/ directory of JuliaLang/julia repository

Yeah that's what I thought.
This is what the remote was when I dif ]dev Dates, which is weird, right?


Minor doc fixes

Motivation

I was reading through the docs and noticed some typos and formatting inconsistencies.
Approach

Fixed typos
Added missing import in an example
Comments in shell start with #
Set OSS link to effection in the header to / and now displays v3 instead of v2
Capitalized H in GitHub for the header link
There were two places where an operation was written as function *foo instead of function* foo. although it works either way, it was written as function* foo everywhere else so I fixed the two for consistency
An example for streams was missing channel. for sending messages

@minkimcello Can you set up commit signing?

@minkimcello I went ahead and rebased it for you.


dev-server reloadShell "server is not started"

I have a minimal application for trying out Native Federation, using angular 18 with esbuild and Native federation 18.2.2, Node 20.17.0
Using executors @angular-architects/native-federation:build for serving, works fine and I can get the shell to load the remote app. However, when making a change in either the shell or the microfrontend and trigger a reload I get the following error, stopping the serve:
WORKROOT\node_modules\@angular-architects\native-federation\src\utils\dev-server.js:60         throw new Error('server is not started');         ^ Error: server is not started     at reloadShell (WORKROOT\node_modules\@angular-architects\native-federation\src\utils\dev-server.js:60:15     at Timeout._onTimeout (WORKROOT\node_modules\@angular-architects\native-federation\src\builders\build\builder.js:208:75     at listOnTimeout (node:internal/timers:569:17     at process.processTimers (node:internal/timers:512:7)
running both shell and microfrontend with command:
nx run PROJECTNAME:serve
the configuration for "serve" taken from examples online (in this case for the remote-app, shell is on different port):
    "serve": {       "executor": "@angular-architects/native-federation:build",       "options": {         "target": "remote-app:serve-original:development",         "rebuildDelay": 0,         "dev": true,         "port": 4200       }     },
Is there any specific configuration that I am missing for this to work?

the issue seems to be resolved once removing "port" field from the "serve" config. Closing.


Py37

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Why we are dropping the tests? We spotted multiple problematic corner cases  and errors thanks to these tests. And this is a testing framework - it should better be well-tested...

Why we are dropping the tests?

Heavy testing like that should be done upstream. Getting the source via git and running all the battery of tests is too costly for the CIs.

We spotted multiple problematic corner cases and errors thanks to these tests. And this is a testing framework - it should better be well-tested...

I wonder if that was due to packaging issues or real failures b/c we do expect the package to be tested upstream before a release.
With that said all the tests were passing when I sent this PR with the exception of one failure due to a false positive on a pandas warning. That is one reason why heavy testing here can be problematic, it can delay the publication of a fully functional package.
With that said. I'm not against good testing practices. I just don't want an unnecessary package maintenance and CI burden.

You are probably right. This is costly. In this case upstream has a full set of tests and CI - but still some bugs slipped through. I am just afraid we will pack up a faulty package and people will count on it for their own testing (e.g. my personal packages do).

but still some bugs slipped through

I really think those were packaging issue :-/

I am just afraid we will pack up a faulty package and people will count on it for their own testing (e.g. my personal packages do).

I also use it on production every day, that is why I want a faster release cycle.
Let's merge this one, if it turns out I break it we re-instate the tests ASAP. OK?


showHeaderOnEveryPage

looping back on the open issue #222 and some code changes i am proposing to address it (in line with  #262 by David Moore). code is meant to fix some missing offset to line up headers and table spacing on every page. code can benefit from some refactoring but just tested it in 2.3.3 and looks good .

submitted by mistake against master not develop base

submitted by mistake against master not develop base


'@material-table/exporters/csv' does not contain a default export (imported as 'ExportCsv')`

I just updated to v1.2.0 and I'm getting the following error:
Attempted import error: '@material-table/exporters/csv' does not contain a default export (imported as 'ExportCsv').
Is there a new way I'm supposed to import the module? The current README states:

I'm currently importing it in the way that the README suggests, and it appears to not be working. Either the README should change, or the underlying code.

We are currently working on this

Please try this:
import { ExportCsv } from "@material-table/exporters/csv";
import {  ExportPdf } from "@material-table/exporters/pdf";

Hi, thanks for your response. Unfortunately, I'm still seeing the same error:

I just updated to v1.2.11:

Importing as instructed:

Running a create-react-app project on node v18 and npm v8.19.3. Any thoughts? Thanks again.

Hi,
any workaround?

@brunohpg I had to pin mine to version 1.1.0 to get it work for now.

@brunohpg I had to pin mine to version 1.1.0 to get it work for now.

Thank you. It work for now.

@Domino987 Could you pls let us know is this issue fixed?

The latest version should just work with import {NAME} from "..."

Thanks @Domino987, it appears to be working for me now on v1.2.17.

Thanks @Domino987, it appears to be working for me now on v1.2.17.

Thank @Domino987
It is working fine in latest ver 1.2.17


Suggestion / request: use Dependabot to keep dependency verisons secure

First of all, thanks for Slate!
I've got a suggestion / request: would you be up for using Dependabot to automatically create dependency update PRs for this repo when dependencies have security vulnerabilities? I ran it against my fork and it generated these PRs. I've ported the nokogiri one across to this repo in #1026.
I built Dependabot, but I'm honestly only suggesting it because I hope it can save you some time. I'd love any feedback, and obviously having open source repos using Dependabot helps boost its profile, but if it's not helpful to you then it's not really worth anything. I'd suggest you run it in "security updates only" mode in Slate, so you just get a PR when there are new security vulnerabilities, which is pretty rare (maybe once a month)
You can install it from here or here if you decide to give it a try. It's been through GitHub's security testing (to be allowed in the GitHub Marketplace) and is used by a few thousand organisations, and the source code is here.
:octocat:

Dependabot added given how useful it is on meaning I don't have to manually check rubygems for versions released of dependencies.
And wouldn't you know it, it has already found a couple of out-of-date dependencies we could bump for the next version release.


[Doc][Iceberg] Improved iceberg documentation

subtask of #4572

Purpose of this pull request

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

good , Thanks for your contribution. Looks good  PTAL : @hailin0

When can use the function of sink to iceberg


Allow users to customize default variant placement

This PR makes it possible for users to customize where the default (unaltered) variant for a utility is placed in the final CSS relative to other variants generated for that utility.
This is done by explicitly specifying default in the variant list if you want the default variant to be placed anywhere other than first in the list.
For example, this CSS:
@variants focus, default, hover {
  .order-2 {
    order: 2
  }
}

...will generate this output:
.focus\:order-2:focus {
  order: 2
}

.order-2 {
  order: 2
}

.hover\:order-2:hover {
  order: 2
}

If the default variant is not specified in the list, the default variant will be invisibly prepended to the variants list, ensuring that the default variant is always rendered first.
This feature is useful in situations where you want to create a custom variant that should still be overridable by a default variant when necessary, like outlined by @benface here:
https://github.com/tailwindcss/tailwindcss/issues/496#issuecomment-399441184

@xpromx
Or even
function ensureIncludesDefault(variants) {
  return [...new Set(['default', ...variants].reverse())].reverse()
}


Metadata Cache

Unit Tests need to be added once the design has been reviewed :)

Addressed, please recheck.


Fix: Change color of Spinner dropdown item in TimeTravel dialog

Fixes #236
Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Run the unit tests with gradlew app:connectedGmsDebugAndroidTest to make sure you didn't break anything (with an emulator running or phone connected)

[x] If you have multiple commits please combine them into one commit by squashing them.

Looks good to me - thanks for the fix.


Themed Chips

Added themed Chips.
#497

What do you think?

Done.

I saw this and I love this feature. Is there a way to integrate this with md-chips? Like all of the inputted chips there will have a theme.


Crash After Permission Disable

Describe the bug
The app is crashed if the user disabled permission for photos. And after that tried to capture it, application crashed.
To Reproduce
Steps to reproduce the behavior:

Open Application
After giving all permission, go back to settings
Disable photos permission
Tried to capture the picture, it crashes.

Expected behavior
It should ask permission and save image instead of crashing
Environment (please complete the following information):

Device: Any Device
OS: 15.1
Xcode Version 13
Swift Version  5.3

Installation Type

SPM

If you try it for the first time and the first screen is the camera, it also crashed and then ask for permission rather than first ask for permission and then allow you to capture an image.

Try https://github.com/Yummypets/YPImagePicker/releases/tag/5.1.1 it should fix the crash. If found - open the new issue.


Refactoring in test API

Mainly for reducing redundant window.getComputedStyle calls and improve readibility.

lgtm. thanks!


[Enhancement] Applescript API

See atom/electron#4418.

Hi, I have no plans for this. If this feature is merged into Electron it might be possible to expose it, but it is not planned.


chore: create testing matrix for tests action

nvim

0.9.0
0.9.1
0.9.2
0.9.3
0.9.4 (latest)
0.10.0 (nightly)

kitty

0.29
0.30
0.31 (latest)
0.32 (nightly)

:tada: This issue has been resolved in version 3.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update README.md

Current coverage is 93.95%

Merging #20 into master will not affect coverage as of ac585c3

@@            master     #20   diff @@
======================================
  Files           10      10
  Stmts          298     298
  Branches         0       0
  Methods
======================================
  Hit            280     280
  Partial          0       0
  Missed          18      18

Review entire Coverage Diff as of ac585c3

Powered by Codecov. Updated on successful CI builds.


Add links to previous blame

add link to previous blame (fixes #15109)
fix blame row alignment on firefox (commit info column was 1px higher than other columns due to avatar placement)

WIP, help wanted: this probably needs an SVG icon + i18n-ed tooltip, how would I do this with this non-template code..? D:

#15171 is a much better solution to this


Slim4 RendererInterface

Slim4 should provides a renderer interface which is not aware of PSR Response

Can you explain why you  are proposing this please?

Coverage remained the same at 96.075% when pulling 4289d0d4fbe3db1bb7eda3d37ec985d4fe873c49 on jonathankowalski:feature.renderer-interface into 77daca61ee271bbd8628bd4a199963bb48fbef21 on slimphp:4.x.

Due to this issue : https://github.com/slimphp/Slim/issues/2065

It's a little bit of a bad design to add an interface to a system that has no implementations, and no place of use; it just sits there with no real explanation of what it's for.
As this is purely related to views, and that views are not part of the core package, I feel this would be better placed in view packages.
While I appreciate the reasoning behind it, I'm against this going into the core.


better view for txt code

when writing a jianpu text file, it will become a chaos if it does not have separation to assist.
for example, when I write the following music:

the txt file is like this:

3' 6 1' 3' 3' - 2' - 2' - - 2'

It may become much better to write like this:

3' 6 1' 3' | 3' - 2' - | 2' - - 2' |

however, the symbol '|' is used in Repeat (with alternate endings): A{ 2 | 3 }
May you add an if-else judgement that only in A{} paired '|' can be converted, otherwise just ignore it?
Thanks a lot!

Sorry I have only just seen this: I didn't realise Github doesn't automatically give us email notifications on new issues.  When writing a complex jianpu file I normally just start a new line every bar, but I guess | is a more natural choice because it is the bar-check indicator in Lilypond (and Lilypond does not just ignore them, it also checks that the bar is correct length and warns you if not).  But I'm not sure it's a good idea to "overload" | so it means a different thing depending on whether or not you are in a repeat: doing that could cause less error-checking, e.g. if someone forgets the A{ at the start of their alternatives, they need an error to tell them something is wrong.  I could add an "ignore errors like this" option but it's probably not a good idea: people would switch it on and fail to realise if their input really does have an error.
I am now tempted to change the existing use of | into || and make | the standard bar-check character.  But changing an existing part of the syntax would break some people's existing input, so I don't want to do it without some thought first.  But perhaps I should not worry quite so much because (1) Lilypond itself has changed its syntax many times and (2) I can easily add code to check that there is a || inside each A{ block and error if not, so existing input files that use | within A{ will get an error and an explanation after they upgrade jianpu-ly (and I expect this is not a common use-case anyway).

... or perhaps }{ would be better than || as the alternate-separator syntax, as || could be confused with a section separator ...

Okay, I have read the source code and modify it as what I want.
...
def getLY(score):
...
            elif word=="|":
                pass
            elif word=="A{":
                repeatStack.append((2,0,0))
                out.append(r'\alternative { {')
            elif word=="||":
                #if not repeatStack[-1][0]==2:
                #    sys.stderr.write("| should be in an A{ .. } block (scoreNo=%d barNo=%d)" % (scoreNo,notehead_markup.barNo))
                #out.append("} {")
                try:
                    if not repeatStack[-1][0]==2:
                        sys.stderr.write("| should be in an A{ .. } block (scoreNo=%d barNo=%d)" % (scoreNo,notehead_markup.barNo))
                    out.append("} {")
                except IndexError:
                    pass

Also I strongly support your opinion

But changing an existing part of the syntax would break some people's existing input

So I will just use it as a personal version ^_^, thank you very much

Instead of "pass", try "out.append(' | ')" which will copy the | characters to Lilypond. That way Lilypond will warn you if you make a bar the wrong length.

Great, thanks a lot!


Two minor updates

Hi @troydai, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Remove the vs code settings. We can think about it next time.

/cc @Tratcher

Can you remove Win32.cs?

sure.

Updated

@Tratcher looking good?

:shipit:


Add dispersion models for metals

Lots of metals can be well-modeled with analytic permittivity models (Drude model, etc.). My understanding is these are very accurate. It would be very easy to add these. I'm not sure how well the underlying algorithm handles metals. In principle, it should work, but in practice, there might be something I am missing.

In the meantime, metals are available as n/k tables from the refractiveindex.info database: #8

This can now be implemented manually by using an arbitrary function of wavelength as of  b157a55a0b5618d715ab8753b8803a980dea862b. I don't think this on its own is enough of a value-add to justify it.


fix: leaderboard functions

Type

[x] Bug Fix
[x] Functions: Fixes leaderboard functions

Referenced Issue: #565
Description
Fixed leaderboard functions, and removes a check if users dont exist.

I’m guessing if we don’t provide hideZero value user, it’ll just say the replace message placeholder.
Otherwise everything looks good.

Also, make sure we update all leaderboard functions with the same placeholder to make it consistent with all functions.


How to Read / Write with PluginBLE Xamarin

Good night!
I am developing an Xamarin.Forms application for bluetooth connection with an external device ... The application can now display the available devices and make the connection, when looking for services I have the following
[0] Generic Access
[1] Generic Attribute
[2] Device Information
[3] TI SensorTag Smart Keys
I use [1] that would be the GATT (Generic Attribute Profile) where I send and receive information with the device, correct?
Each service has characteristics, this one specifically has only 1, where it is CanRead = false and CanWrite = false ... if I try to do a .Read (), it generates a null error

Hi @nowlights,
I'm the friendly issue checker.
It seems like (100.00 %) you haven't used our issue template :cry: I think it is very frustrating for the repository owners, if you ignore them.
If you think it's fine to make an exception, just ignore this message.
But if you think it was a mistake to delete the template, please close the issue and create a new one.
Thanks!

c


Merge main into release/6.0

Merge the following PRs into release/6.0:

https://github.com/apple/sourcekit-lsp/pull/1194
https://github.com/apple/sourcekit-lsp/pull/1208
https://github.com/apple/sourcekit-lsp/pull/1215
https://github.com/apple/sourcekit-lsp/pull/1195

@swift-ci Please test


Make crash my extension host on VSCode

Before submitting your bug report

[X] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[x] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS: Ubuntu 22.04
- Continue: 0.8.40
- IDE: VS Code

Description
At installation, it makes my Extension host
To reproduce
When I install it the bottom bar become empty.
Log output
No response

@MrMic Are you able to share anything from the logs? My guess is that the answer to the problem will be fairly obvious once we can see them, but fairly obscure before then

The bug description is lacking, but sounds like dup of https://github.com/continuedev/continue/issues/911

Ah yes this is a good call, I'm going to close this in favor of condensing conversation around a single issue


WIP: Allow initialization of the cluster from backups

Currently, replica's will already use the S3 backups if they are
available to create a $PGDATA directory.
However, if no master (endpoint) is available, the behaviour up to now
was to always do an initdb.
This may not be what the user wants, say - a user wants to suspend a
service - removing all the Kubernetes objects and only keeping the S3
backups.
In such a scenario, it would be great if the backup would be used to
initialize the whole cluster.

We also need to support the following scenario:

Bootstrap from s3 bucket a
Backup to s3 bucket b

This for example to mimick a disaster recovery scenario, without interfering with the current running cluster.
Currently, if there are multiple HA clusters pointing to the same s3 bucket, bad things will happen (with the backup and/or archiving) for at least one of the clusters, if not both.


Python Pipeline Test classes

Python Pipelines test cases are created for various input and output formats in 4 ML frameworks,
each framework are separated with new folder like keras,pytorch,scikitlearn,tensorflow.
Covered all 3 types like Raw,Classificatiion,Regression.
Input Formats like JSON,IMAGE,NDARRAY,NUMPY,ARROW
Output Formats like Formats like JSON,IMAGE,NDARRAY,NUMPY,ARROW
Need to install in CI machine required Python libraries.

@ShamsUlAzeem  please confirm the pull request.

Thanks. Let's wait till the CI is fixed and other issues to rise up before final reviewing

Closed due  to becoming old api


fetches records for map display as ajax

Overview
The PR suggests to fetch records as ajax for loading on map, instead of rendering geojson inline
Related Issue / Discussion
#948
Contributions and Licensing
(as per https://github.com/geopython/pycsw/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[x] I'd like to contribute [feature X|bugfix Y|docs|something else] to pycsw. I confirm that my contributions to pycsw will be compatible with the pycsw license guidelines at the time of contribution.
[x] I have already previously agreed to the pycsw Contributions and Licensing Guidelines

Thanks @pvgenuchten.  I guess the question here is weighing 2 HTTP requests vs 1 (with a bigger response length because of the data inline in JavaScript).  Adding @kalxas and @ricardogsilva for any comments.

closing this for limited interest


Drop pseudo-class hook type

The pseudo-class hook type doesn't actually seem that useful to me. Consider a hover hook:
createHooks({
  hover: ":hover",
})

If you didn't have the pseudo-class hook type, you'd just use the selector type instead, the difference just being a single character:
createHooks({
  hover: "&:hover",
})

And, of course, if you're using the recommended config API, it doesn't matter anyway:
createHooks(recommended({
  pseudoClasses: [":hover"] // produces a `&:hover` hook
}))

Any thoughts on dropping this hook type in v2? Do you have a use case I should consider?

the main use case is being able to write hover and not need to escape the object key in a string.
as long as that's still supported, i think it's fine to drop the hook type.

I personally favor it being explicit. If I see hover: ... I will first perceive it as CSS key value versus a selector, like &:hover or a media query, which I believe would always need to be quoted anyway. &:hover also is easier to search against without worrying about missing occurrences and is more consistent without tools like emotion and Styled Components work, I think. In any case, too much/too many shorthands I think can be considered harmful in general.

Dropped in v2


Release Cloud Workflows V1 libraries version 1.1.0

Changes in Google.Cloud.Workflows.Executions.V1 version 1.1.0:

Commit ac367e2: feat: Regenerate all APIs to support self-signed JWTs

Changes in Google.Cloud.Workflows.V1 version 1.1.0:

Commit ac367e2: feat: Regenerate all APIs to support self-signed JWTs

Packages in this release:

Release Google.Cloud.Workflows.Common.V1 version 1.1.0
Release Google.Cloud.Workflows.Executions.V1 version 1.1.0
Release Google.Cloud.Workflows.V1 version 1.1.0

Created release for Google.Cloud.Workflows.Common.V1-1.1.0
Created release for Google.Cloud.Workflows.Executions.V1-1.1.0
Created release for Google.Cloud.Workflows.V1-1.1.0


JsonSchema4.FromData() doesnt exist

Hello,
I think that method FromData() doesnt exist, but if you read README.md you see:
Generate JSON Schema from sample JSON data (JsonSchema4.FromData())
Maybe the actual method is  JsonSchema4.FromSampleJson();
Please when possible fix it.

Its FromSampleJson()

Yes, I realized.
Thanks


i_CTRL-N does not work

:help i_CTRL-N is not working when some completions appear, I've to either exit the insert mode and enter again or press ctrl-x before it.
To simulate:

Force the completions with compe#complete().
Close the menu with compe#close('<C-e>').
Try to use i_CTRL-N.

The same problem as here #269.

If you press twice compe#close('<C-e>'), i_CTRL-N works correctly.

I don't know if it is the right solution, but with extra <C-e> at the end it's working:
vim.api.nvim_set_keymap('i', '<C-e>', '<Cmd>lua require"compe"._close()<CR><C-e>', {noremap = true})

@stasjok weren't you able to use the public api with this workaround? https://github.com/hrsh7th/nvim-compe#mappings

You can try something like this:
inoremap <silent><expr> <C-e>     compe#close()..'<C-e>'

But I like my earlier solution more, because it doesn't have fallback to <C-e>. Personally I think, that <C-e> and <C-y> are pretty unusable with auto-completion (because of interfering popup), so I completely remapped them (<C-y> to complete or comfirm depending if popup is active, <C-e> to just closing popup).
Remember, that this mapping valid only until it's fixed. What I guess is happening here is neovim after require"compe"._close() still thinks there is a popup active. So hotkeys for popup like <C-n>, <C-p> doesn't work until you press <C-e> (it's a hotkey for canceling default completion popup).


SVG Spriting

Changes

Added gulp script for generating a sprite from a folder of SVG files.
Rendered sprite into pattern lib to test
Re-named npm task from build to compile to be more consistent with author app.

How to test

npm run compile.
go to /patterns/icons and verify.

I've run this fine and we're accepting that the under source control icon files:
    modified:   app/styles/base/_sprite.scss
    modified:   app/templates/patterns/components/icons.html

Will be changed by running the compile grunt task.


HIPPO texture analysis examples do not work for multi-atom CIFs

The lines
    if i > 0:
        editor.ref(key1='Biso', key2='Biso', value='0 1 100000', sobj1=phases[0], sobj2=phase)
    else:
        editor.set_val(key='Biso', sobj=phase, value='0.6')

do not work once a CIF file has multiple atoms. Can they be made more general, i.e. loop over all atoms in a structure?

As a second issue, the initial values for the peak broadening parameters are IMHO too broad. Instead of
editor.set_val(key='MicroStrain', sobj=phase, value='0.002')
editor.set_val(key='CrystSize', sobj=phase, value='500')
they should be something like
editor.set_val(key='MicroStrain', sobj=phase, value='8e-4')
editor.set_val(key='CrystSize', sobj=phase, value='1000')
The current parameters make peaks too broad, setting them up to get even broader. Should be on the skinnier side...

Sven, will revisit the tutorial..
For the multi atom bit, have you tried using multiple sobs inputs. You can create a chain of strings seperated by spaces that must be in the subordinate object string e.g. sobj="str1 str2 str3"
If I have 3 subordinate object names they are all concatenated like "subordinate_object1subordinate_object2subordinate_object3" and I assert that str1 and str2 and str3 are in the object and assert that any nsobj="str4 str5..." are not in the subordinate objects names.

I think I now understand the problem you are referring to.. I've made changes to the example to address it. The following code snippet makes the first biso in the parameter file the referenced bison and all others the tied to it
    editor.set_val(key='Biso', sobj='First', value='0.6')
    editor.ref(key1='Biso', key2='Biso', value='0 1 100000', nsobj1='First',sobj2='First')

I tried starting with a narrower peak width and the fit is good, but I think there is actually anisotropic peak broaden in the run001 case.


Button: No easy way to get Button's width

If I want to position a button against the right side of a View I have to do this:
			// Demonstrate AnchorEnd - Button anchored to bottom/right
			var anchorButton = new Button ("Anchor End") {
				Y = Pos.AnchorEnd () - 1,
			};
			anchorButton.X = Pos.AnchorEnd () - (anchorButton.Text.Length + 4);

			Win.Add (anchorButton);

This is awkward and error prone. What happens if Button changes in the future such that it's not +4 characters wide?

I wondered if I could do this:
			// Demonstrate AnchorEnd - Button anchored to bottom/right
			var anchorButton = new Button ("Anchor End") {
				Y = Pos.AnchorEnd () - 1,
				X = Pos.AnchorEnd () - Dim.Width(anchorButton);
			};

			Win.Add (anchorButton);

But that obviously wont work because a) anchorButton hasn't been defined yet and b) Dim.Width returns a Dim which can't be assigned to int.
I think if Dim had a Size property this might be OK:
			// Demonstrate AnchorEnd - Button anchored to bottom/right
			var anchorButton = new Button ("Anchor End") {
				Y = Pos.AnchorEnd () - 1,
			};
			anchorButton.X = Pos.AnchorEnd () - Dim.Size(anchorButton).Width;

			Win.Add (anchorButton);

That won't work (for now,obvious to me, reasons).
But then I found I could do this:
anchorButton.X = Pos.AnchorEnd () - (Pos.Right (anchorButton) - Pos.Left (anchorButton));

Thus, I think the feature request here is to extend Pos with a Width property implemented by extending PosView like this:
		internal class PosView : Pos {
			public View Target;
			int side;
			public PosView (View view, int side)
			{
				Target = view;
				this.side = side;
			}
			internal override int Anchor (int width)
			{
				switch (side) {
				case 0: return Target.Frame.X;
				case 1: return Target.Frame.Y;
				case 2: return Target.Frame.Right;
				case 3: return Target.Frame.Bottom;
				default:
					return 0;
				}
			}

			public override string ToString ()
			{
				string tside;
				switch (side) {
				case 0: tside = "x"; break;
				case 1: tside = "y"; break;
				case 2: tside = "right"; break;
				case 3: tside = "bottom"; break;
				default: tside = "unknown"; break;
				}
				return $"Pos.View(side={tside}, target={Target.ToString()}";
			}
		}

...

		/// <summary>
		/// Returns a Pos object tracks the Width (Right-Left) of the specified view.
		/// </summary>
		/// <returns>The Position that depends on the other view.</returns>
		/// <param name="view">The view that will be tracked.</param>
		public static Pos Width (View view) => new PosView (view, 4);

A quick PoC where I implemented the above shows this works. Before I submit a PR, can anyone think of anything I'm missing in my thinking?
			// Demonstrate AnchorEnd - Button anchored to bottom/right
			var anchorButton = new Button ("Anchor End") {
				Y = Pos.AnchorEnd () - 1,
			};
			anchorButton.X = Pos.AnchorEnd () - Pos.Width(anchorButton);
			Win.Add (anchorButton);

Is there any way to do this where it would work in an object initializer?

@migueldeicaza - I'm waiting for you to comment on this before I submit a PR. I don't really feel I understand how DimPos actually works well enough yet to have confidence doing what I suggest above is ok. Thanks.

This now works. Closing.


Python formatter very slow

VS Code version
1.62.3 (and previous one as well)
Extension version
v2021.11.1422169775
OS type
Windows
OS version
Windows 10 21H1
Python distribution
Other
Python version
3.9.7
Language server
Pylance
Expected behaviour
Quick file save
Actual behaviour
Saving file may take up to 2 minutes
Message shown:

Steps to reproduce

Edit file
Save file

Logs
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Closing in favor of https://github.com/microsoft/vscode-python/issues/17918


Add the popular npmlog logging module

Thank you for contributing to the Node.js Toolbox catalog! 🙏
Note: Node.js Toolbox is specifically for backend libraries. If a library works both in the browser AND in Node.js feel free to add it. Libraries that work ONLY in the browser are out of scope for now.
Before submitting your pull request, please make sure:

[x] To use the exact package name as it appears on NPM (and npm install <package-name>)
[x] The packages are listed in alphabetical order
[x] The package doesn't already exist in a different category
[x] Description doesn't have a dot (.) at the end
[x] To format with Prettier before committing (npm run prettier)

@koresar Nice one! Haven't heard about npmlog before  🙌

Live: https://nodejstoolbox.com/categories/logging


Disable possibility of uploading the two created layers inside JOSM

Event if they are not locked to work on it, we never want to upload them to OSM.

Thanks @alkino for the PR. This is a mirror repository only, please use https://gitlab.com/bagage/batimap/ instead!


Parse f5vpn urls

After an endpoint-inspection, the webpage forwards
to an f5vpn url, which can now be passed on to
gof5 to extract the session-id itself.
Only missing to be a proper f5vpn handler is
logging

@fwiesel can you provide an f5vpn URL example? I haven't seen it before.

Sure, it seems to have the following pattern:
f5-vpn://<server-fqdn>?server=<server-fqdn>&resourcename=<resource-path-with-slashes>&resourcetype=network_access&cmd=launch&protocol=https&port=443&sid=<32*n>&token=<32-hex>&otc=<32-hex>&otc=<32-hex>
I assume, OTC seems to stand for one-time-code, and we get two of them. The sid is presumably session-id, and it is bogus, as we provide it.

Just as a voice of support for this PR. For nearly two months I've been using this PR to enable me to use gof5 with a F5 VPN endpoint that uses a web page based authentication flow.
It has consistently worked 100% of the time.


feat(#483): remove .0pdd.yaml config

Try to remove .0pdd.yaml config to repair pdd bot.
Closes: #483.

PR-Codex overview
This PR removes the .0pdd.yaml file from the repository.
Detailed summary

Deleted the .0pdd.yaml file

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here


unpin mpl

This PR unpins matplotlib to use the latest version 3.1.1, at this point in time.
Note that, in general the image differences were due to:

legend label positioning
axis ticks and label positioning
default levels for contours
axes get_position returning a different bottom

Requires the associated https://github.com/SciTools/test-iris-imagehash/pull/26 PR to be merged first.

@lbdreyer @pp-mo @stephenworsley Any takers for this PR and the associated https://github.com/SciTools/test-iris-imagehash/pull/26 ?

@lbdreyer I'm happy to add the diff images to this PR if it helps?

@lbdreyer I'm happy to add the diff images to this PR if it helps?

Don't worry! I'm happy looking at them locally

Requires https://github.com/SciTools/test-iris-imagehash/pull/27 to be merged first.

This one kinda concerns me. I find it strange that there are colour differences in the plot but not the colour bar :confused:

This one kinda concerns me. I find it strange that there are colour differences in the plot but not the colour bar :confused:

Okay I think the colour differences are due to the scatter points (and the rest of the plot) shifting slightly, rather than the values of the scatter points actually being different!

Yeah, I agree... it might just be artifacts of the overplotting. Difficult to really say...

Just leaving a breadcrumb trail for my future self (or anyone else) to repeat the testing...
#!/usr/bin/env bash

# conda create -n iris-mpl -c conda-forge python=3.7 cartopy cf-units cftime dask matplotlib netcdf4 numpy pyke scipy six nose imagehash pip filelock iris-sample-data nc-time-axis
# . activate iris-mpl
# conda install -c conda-forge python-eccodes
# conda install -c conda-forge --no-deps iris-grib
# pip install -e .  # iris dev branch

BASE=lib/iris/tests

python -m iris.tests.runner --example-tests
python ${BASE}/experimental/test_animate.py
python ${BASE}/integration/plot/test_plot_2d_coords.py
python ${BASE}/integration/plot/test_vector_plots.py
python ${BASE}/test_mapping.py
python ${BASE}/test_plot.py
python ${BASE}/test_quickplot.py

@lbdreyer Tests are looking good.... all the graphical test failure have been hiding a doc building failure, which is the last thing to investigate.

@lbdreyer Since we were chatting about this, see:

https://blog.travis-ci.com/2019-10-07-multi-cpu-architecture-support, and
https://docs.travis-ci.com/user/reference/overview/

@lbdreyer Finally..... 😌 I think this is now good to go, thanks 👍

Thanks @bjlittle !


Allow mixed ints and Dimensions for reshape shape param

CF issue #11974
I think this should fix issue with PR #12127 by mapping the dimensions to a new list, avoiding any attempt to assign to the shape param if it is a tuple.

Can one of the admins verify this patch?

Hi, pmccarter.  #12127 has been revised in the case of tuple argument, see commit. If possible, could you take a review? Thanks.

Closing this out in favor of #12127


Memory leak in AsteriskQueueImpl

Scenario:
QueueManager.handleJoinEvent and QueueManager.handleQueueEntryEvent check for duplicates using AsteriskQueueImpl.getEntry. Then a new entry is created via AsteriskQueueImpl.createNewEntry.
Problem:
When createNewEntry is invoked more than once at the same time it may add duplicates. There is no synchronization in between of checking for duplicates and creating new entry. If duplicate is added it will never be removed.
In my case it happens when AsteriskServerImpl.forceQueuesMonitor is enabled and createNewEntry is invoked as a result of caller entering queue and Asterisk response to QueueStatusAction.

Can you provide pr with a fix,?
On Thu, 16 Jul 2020, 9:40 pm Bartosz Bańkowski<EMAIL_ADDRESS>wrote:

Scenario:
QueueManager.handleJoinEvent and QueueManager.handleQueueEntryEvent check
for duplicates using AsteriskQueueImpl.getEntry. Then a new entry is
created via AsteriskQueueImpl.createNewEntry.
Problem:
When createNewEntry is invoked more than once at the same time it may add
duplicates. There is no synchronization in between of checking for
duplicates and creating new entry. If duplicate is added it will never be
removed.
In my case it happens when AsteriskServerImpl.forceQueuesMonitor is
enabled and createNewEntry is invoked as a result of caller entering
queue and Asterisk response to QueueStatusAction.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/asterisk-java/asterisk-java/issues/336, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAG32OHSOOM3SWO7AWBN3N3R33RLJANCNFSM4O4C3EPQ
.

It is here #337

pr merged


Add more precise information to README

Hey @ClintonCao,
could you check this PR and add the XML to generate the reports, replacing some snippet?
Thx!
Moritz

Thx for the comment, addressed it.

XML snippet is added.

I think it's quite OK now.

I'll merge this then


Run docker container as current host user

Some explicitly user flag specified in docker container exec commands inside scripts found under the sbin directory is redundant. Can we remove them?


LUCENE-10510: Check module access prior to running gjf/spotless tasks

Looks fine to me. We can maybe do the same regarding errorprone. Errorprone does not run by default without -Dtests.nightly at moment, but we should do the same.

See here:

Looks fine to me. We can maybe do the same regarding errorprone. Errorprone does not run by default without -Dtests.nightly at moment, but we should do the same.

See here: https://github.com/apache/lucene/blob/7a950a3ae6bdb297b97058636ae9f88c2a42e29e/gradle/validation/error-prone.gradle#L20-L26

I've reorganized the build logic a bit to include all expected exported packages and an early errorprone check.


Adding Codeql task

Validating if this injects the CodeQL task correctly.

Nevermind - it looks like it was indeed run. I was just looking at the wrong build.