spark-xml for hadoop 2.9+

For future reference, if you try to use the spark-xml library with newer hadoop libraries i.e. 2.9+ it will fail due to the XMLInputFactory and XMLOutputFactory defaulting to com.ctc.wstx.stax.*.
To resolve you can force the use of the old libraries:
// force use of com.sun.xml.* libraries
System.setProperty("javax.xml.stream.XMLInputFactory","com.sun.xml.internal.stream.XMLInputFactoryImpl")
System.setProperty("javax.xml.stream.XMLOutputFactory","com.sun.xml.internal.stream.XMLOutputFactoryImpl")

I don't see quite that issue after https://github.com/databricks/spark-xml/pull/343 which may help by not changing the version of Hadoop used from whatever Spark depends on.
I do see test failures like...
[info] - Convert current structure to string *** FAILED ***
[info]   "[[Stax Event #1][Stax Event #4][Stax Event #1][Stax Event #4][Stax Event #2][Stax Event #1][Stax Event #1][Stax Event #4][Stax Event #2][Stax Event #1][Stax Event #4][Stax Event #2][Stax Event #2][Stax Event #2]]" did not equal "[<info>
[info]         <name>Sam Mad Dog Smith</name><amount><small>1</small><large>9</large></amount></info>]" (StaxXmlParserUtilsSuite.scala:77)

So does seem to be some issue with whatever Hadoop does to the XML parser. If we can accommodate it that may be more flexible, especially as I don't know the impact of trying to override the environment's XML parser in this case.

I can fix the above with the change in https://github.com/databricks/spark-xml/pull/344 , but we still get test failures like:
[info] - DSL save with nullValue and treatEmptyValuesAsNulls *** FAILED ***
[info]   Set("[null,bk101,Gambardella, Matthew,[1,Computer],44.95,[WrappedArray([null,1,10,2000])],XML Developer's Guide]", "[<book id="bk102">
[info]           <author>Ralls, Kim</author>
[info]           <genre>
[info]               <genreid>2</genreid>
[info]               <name>Fantasy</name>
[info]           </genre>
[info]           <price>5.95</price>
[info]           <publish_dates>
[info]               <publish_date tag="">
[info]                   <day>16</day>
[info]                   <month>12</month>
[info]                   <year>2000</year>
[info]               </publish_date>
[info]               <publish_date tag="lastest">
[info]                   <day>17</day>
[info]                   <month>12</month>
[info]                   <year>2000</year>
[info]               </
[info]
[info]   publish_date>
[info]           </publish_dates>
[info]           <title>Midnight Rain</title>
[info]       </book>,null,null,null,null,null,null]", "[null,bk103,Corets, Eva,[2,Fantasy],null,[WrappedArray([null,7,11,2000], [null,8,11,2000])],Maeve Ascendant]") did not equal Set("[bk101,Gambardella, Matthew,[1,Computer],44.95,[WrappedArray([null,1,10,2000])],XML Developer's Guide]", "[bk102,Ralls, Kim,[2,Fantasy],5.95,[WrappedArray([null,16,12,2000], [lastest,17,12,2000])],Midnight Rain]", "[bk103,Corets, Eva,[2,Fantasy],null,[WrappedArray([null,7,11,2000], [null,8,11,2000])],Maeve Ascendant]") (XmlSuite.scala:432)
[info] - Write values properly as given to valueTag even if it starts with attributePrefix *** FAILED ***
[info]   11 did not equal 12 (XmlSuite.scala:462)

Weird -- there are extra lines in closing the tag. It happens there just because that's the point where the woodstox writer buffer flushes, but I couldn't directly see any issue in how woodstox handles this. Something is inserting newlines there but can't figure out why.

@srowen Any work around for this issue ?

I haven't looked at this in a while, but no I could not figure out any workaround last time.

thanks @srowen

@a137705 I think I fixed it - https://github.com/databricks/spark-xml/pull/417


Prop className did not match.

When migrating a create-react-app project to Next I stumbled upon the following error during hydration. Maybe I'm doing something wrong, but I think it might be a bug in tss-react. After a lot of tinkering I managed to make the following minimal repro. It's a little bit contrived, to make it as small as possible.
You can reproduce it the following way:

git clone https://github.com/nphmuller/tss-react-classmatch-issue.git
cd tss-react-classmatch-issue
npm i
npm run dev

Now when opening the page in the browser the following error gets logged:

Warning: Prop className did not match. Server: "mui-ek3luc-icon mui-1-icon-ref" Client: "mui-ek3luc-icon mui-0-icon-ref"

Some details on how I got the error

In page.tsx we render the following:

    <>
      <OtherComponentThatCallsMakeStyles />
      <ComponentWithIssue />
    </>

The 1st component just calls makeStyles() and renders nothing. It doesn't matter what it renders, just that makeStyles() is called before the component with the issue.
The main issue lies in the 2nd components, which contains the following code:

const useStyles = makeStyles<void, "link" | "icon">()(
  (_theme, _props, classes) => ({
    link: {
      // Removing this fixes the issue
      [`& .${classes.icon}`]: {
        color: "blue",
      },
    },
    icon: {},
  })
);

const ComponentWithIssue = () => {
  const { classes } = useStyles();

  return (

      Icon

  );
};

The [`& .${classes.icon}`]: clause seems to cause the different classname to be generated server-side vs client-side.
Additional details
> node -v
v20.9.0

    "@emotion/styled": "11.11.5",
    "@mui/material": "5.15.15",
    "@mui/material-nextjs": "5.15.11",
    "next": "14.2.3",
    "react": "^18",
    "react-dom": "^18",
    "tss-react": "4.9.6"

Mui was setup the following way (via layout.tsx):
import { ThemeProvider, createTheme } from "@mui/material/styles";
import { AppRouterCacheProvider } from "@mui/material-nextjs/v14-appRouter";

...

  <AppRouterCacheProvider>
    <ThemeProvider theme={createTheme()}>{children}</ThemeProvider>
  </AppRouterCacheProvider>

Ok it's a confirmed bug, thanks for reporting and taking the time to setup a repro repo. That's refreshing.
I'm on it. I can already see that my past self left a message about this for present me:

Thank you so much for the lightning fast response. I should have suspected it was a usage error. It usually is 😁.
I'll make the necessary changes and I'll close this issue now.
Thanks again!

@nphmuller,
No, no! You did good opening this issue, as I said revisiting the code made me found another bug!
Beside, the documentation about this wasn't clear enough.

Hello @ArtemAstakhov,
Sorry I can't reproduce.
I created a reproduction repo with your code. No issue
https://github.com/garronej/tss-react/assets/6702424/aa5c081a-76d1-45f9-b9d9-6af9bfe710c7
Can you please try do fork and modify the repo to make the issue your experiencing appear:
https://github.com/garronej/tss-issue-212
If I can reproduce I will adress it switfly.

@ArtemAstakhov Great! Thank you for the update!


Ignore Debug, Release, bin, and obj folders created by PlaneFinding when building and running tests

Addresses #9

Hi @ForrestTrepte, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Forrest Trepte). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


Test timeouts in ci

I've had backend tests timeout multiple times today in the Rafiki GitHub Actions workflows:
https://github.com/interledger/rafiki/runs/5486418533?check_suite_focus=true
Output includes:
notice(message_bus): message queue for peer tigerbeetle.src.message_bus.union:392:19{ .replica = 0 } full, dropping message

The workflows appear to succeed on a retry.

Thanks @wilsonianb is this log output from the TB client?
Scrolling through the CI log it looks like replica 0's connection was not stable and was being reconnected a few times before the message queue full errors started popping up.
Would it be possible to get the log output from all replicas in the cluster?

I've pushed an updated log message to main (Zig 0.9.1) that will include the type of message being dropped for more visibility: https://github.com/coilhq/tigerbeetle/commit/1f07119a3df3ab590217273d868ad620aba99791

Scrolling through the CI log it looks like replica 0's connection was not stable and was being reconnected a few times before the message queue full errors started popping up.

If you're referring to the info(message_bus): connected to replica 0 log lines:
The tests repeatedly start/stop (init/deinit) the client; I think that's what you're seeing. This is part of how each test file sets up & tears down its environment (appContainer = await createTestApp(deps) and await appContainer.shutdown() respectively).

I've been unable to reproduce the issue in a few dozen attempts today, some of which with the tigerbeetle logging change:
https://github.com/wilsonianb/rafiki/pull/84
:confused:

I'm encountering the issue again today
https://github.com/wilsonianb/rafiki/actions
@jorangreef It looks like Github Actions are disabled in the tigerbeetle repo, so there isn't a recent Docker image for Rafiki to use to update to the latest tigerbeetle-node (and use the updated log message). Could you try enabling Github Actions and/or triggering the docker workflow on the latest commit to main?

The updated log message is merged :point_up_2:
It looks like there's a combination of ping and request messages being dropped:
https://github.com/interledger/rafiki/runs/5872664693?check_suite_focus=true
info(message_bus): message queue for peer tigerbeetle.src.message_bus.union:384:19{ .replica = 0 } full, dropping request message
info(message_bus): message queue for peer tigerbeetle.src.message_bus.union:384:19{ .replica = 0 } full, dropping ping message

with more logging:
https://github.com/wilsonianb/rafiki/runs/5874848460?check_suite_focus=true

The queued: log lines are from https://github.com/coilhq/tigerbeetle/blob/1a2863d2cfc3568efae18ef1749a28c21a245985/src/message_bus.zig#L604-L616.
The message bus connection's send queue is full of retries for the register request.
info(message_bus): message queue for peer tigerbeetle.src.message_bus.union:384:19{ .replica = 0 } full, dropping request message operation=register client=243897649607243345099344348854093531920 op=0 checksum=144046189412347373692193060048604389285 parent=0 context=0 connection.state=connecting
info(message_bus): queued: command=request operation=register client=243897649607243345099344348854093531920 op=0 checksum=144046189412347373692193060048604389285 parent=0 context=0
info(message_bus): queued: command=request operation=register client=243897649607243345099344348854093531920 op=0 checksum=144046189412347373692193060048604389285 parent=0 context=0


cmake: Fixed drivers/sensor/CMakeLists.txt

Signed-off-by: Sebastian Boe<EMAIL_ADDRESS>

Fixes https://github.com/zephyrproject-rtos/zephyr/issues/4864

tested ok on disco_l475_iot1


time complxity of this fsm

I aprreciate the code you shared. But  it seems that every time this fsm handles an event,  it needs to search the transition table. So the time complexity for each call of process_event is O(N) (N is the count of rows) , not O(1), which makes this fsm unpractical .

For time complexity etc. see the references given in the README.

sorry but i can't find the references in README. I only found a documentation of api.

I look through the linkage of MSM but still not found anything explicitly about time complexity. Is there anything wrong with my analysis of O(N) complexity above?


create a database model

Key Changes:
1. Promotion Model Creation:

Added a Promotion model in SQLAlchemy to represent a promotion in the database.
Fields include:

id (UUID): Primary key, auto-generated.
product_ids (JSONB): A JSON array for storing product IDs associated with the promotion.
name (VARCHAR): The name of the promotion.
description (TEXT): Detailed description of the promotion.
start_date and end_date (TIMESTAMP): Promotion validity period.
active_status (BOOLEAN): A flag to indicate if the promotion is currently active.
created_by and updated_by (UUID): Track the creator and updater of the promotion.
created_at and updated_at (TIMESTAMP): Track the record's creation and last update.
extra (JSONB): A field for storing additional metadata (e.g., promotion type, discount value).

2. Update Serialization and Deserialization:

Implemented methods to serialize a Promotion instance to a dictionary.
Implemented methods to deserialization a Promotion.

Looks good to me. Once @wencfy approve then we can merge.


initial version of DictionaryProtocol

This is a first attempt at implementing a DictionaryProtocol protocol as discussed in #293
This is not complete yet and needs discussion in order to improve!
For now it conforms stdlib Dictionary and OrderedDictionary to the newly introduced DictionaryProtocol.
A basic first definition of a MutableDictionaryProtocol has been created but no conformances added yet. (Want to get the base protocol right first and perhaps identify methods that we would want a mutable version to have)
Hoping to get some feedback on the current implementation and will add conformances to the other Dictionary objects next.
Would we also like to introduce a SetProtocol? Should we consider this first since Dictionaries often depend on some type of Set of keys? Or do we keep the keys collection more generic than constraining it to a Set kind of type?
Not all checks are done since this is still in development :) Would love to hear people's thoughts!
Checklist

[x] I've read the Contribution Guidelines
[x] My contributions are licensed under the Swift license.
[x] I've followed the coding style of the rest of the project.
[ ] I've added tests covering all new code paths my change adds to the project (if appropriate).
[ ] I've added benchmarks covering new functionality (if appropriate).
[x] I've verified that my change does not break any existing tests or introduce unexplained benchmark regressions.
[ ] I've updated the documentation if necessary.

Ah forget about the SetProtocol remark. Forgot that the regular Dictionary's Keys is not strictly a Set.

@lorentey Would you have a bit of direction for me? I'd love to get this to the finish line but could really use some guidance from you. See also my comments in #293


Which event fire when release the mouse from the cells selection ?

Hi,
I am using your plugin and its great to use it. Now i have a requirement to get the range of cells when mouse released from the selection of cells. Actually I have to store that range in database and that range should be  pre-selected when user comes next time.  Please give me a solution.
Thanks

Hi Gulshan, thank you for sharing your experience. The following example will show you how to programmatically select columns on jExcel. http://jsfiddle.net/b9kj4ouq/1/

No This not what i wanted . I want that which event occur when i released the mouse after cells selection ?

I want to select the columns manuaally and store those in database after releasing the mouse selection. Please give any solution ?

I have added the onselection and oncellselection. For a callback after the onmouseup event you should follow the example: http://jsfiddle.net/s1ef8do1/1/

Its not working here , Please check it again

It is not working because you get the old file in cache.

yes its working now thanks Paulhodel , Aswm


Cannot find name 'require' when set to language mode: javascript

VSCode Version: 1.3.1
OS Version: OS X Yosemite

Steps to Reproduce:

Create index.es6 file
Set language mode to javascript
Note linter errors for any require statements, export statements, etc.

I've also tried setting the "file.associations" property in the JSON settings file, same result. Red underlines under all "exports" and "require" statements.

@Dygerati pls provide sample workspace/file. Which linter extensions do you have installed? How are they configured? Do you see the same issue when starting vscode with the --disable-extensions command line option? Thx!

The file's really simple; attached:
example.zip
This is what I'm talking about:

The issue persists with no extensions, and I never really installed anything other than themes so far. The red underlines, even though I've got it set to javascript, are the problem.
Let me know if you need anything further!

@Dygerati Thanks! A picture always helps. It looks like you write a nodejs program.  For now, vscode does not know about nodejs out of the box. You have to teach it.  Since Vscode uses typescript behind the scenes (it's nicely explained in our doc) you do this by adding the appropriate typing files to your project. See the the TypeScript definition file section in our doc.

I appreciate the explanation, although I'm a little confused at your statement that VSC doesn't know about node.js, since it does seem to know (or at least not complain) as long as the extension is *.js. The issue only makes itself known with a non-standard extension, which I'd hoped could be quickly addressed by simply adding it to the fileAssociations.


Fix codecov.io's "No coverage uploaded for pull request base" with Jenkins

Codecov.io is mostly working with Jenkins on istio.io/manager. We still need to generate coverage reports upon successful merge into the master branch so codecov.io can properly generate coverage % diff against master (see warning message below). I think we just need to re-run coverage report on post-submit/commit trigger.

No coverage uploaded for pull request base (master@f3fdddb). Click here to learn what that means.

I am wondering if it ll make more sense to have a git repo with all the scripts that we use instead of having them in istio testing. then we can use a git submodule for those on each repo. That way we can share best practices more easily.

Sounds reasonable assuming the individual repos share common structure, development, practices, etc. But not sure how that would affect this particular issue? I thought the "no coverage" error was because we aren't generating coverage report on merged commits to master via post-submit hook?

For this issue I ll just create a shared method  to publish Code Coverage and will leave the /bin/codecov script.

this was resolved with #85.


Create a hierarchical table for syllables and motifs

WIP

Figuring out this error:
hdmf.build.errors.ReferenceTargetNotBuiltError: next_tier (root/processing/behavior/Motifs/next_tier): Could not find already-built Builder for TimeIntervals 'Syllables' in BuildManager

@CodyCBakerPhD Figured out that error, problem is I'm only adding the motifs table to the processing module, I have to add the syllables too to make the reference work!
# Add the hierarchical table to the processing module
behavior_module.add(syllables)
behavior_module.add(motifs_table)

Interesting, you can't have a hierarchical reference to the global TrialsTable? I suppose that's fine but I'd be curious as to why it's made that way


support for global functions?

Would it be possible to add the same functionality for global functions?

Let me think about it. Should be possible

Feature added. Could you give it a go

It work fine, thanks~~


Internal inclusion change in PJON

Ciao @xlfe thank you very much for PJON-cython 👍
I have changed in PJON the inclusion of internal sources chaging from <> to "": https://github.com/gioblu/PJON/commit/6fcfd759c6f376bd1328a681bd644790fea021b2
This was required to ease the integration with MySensors, do this change break something here?

Hi @gioblu
no problems building or tests with the current sources
log-73dd183e-5cd2-44aa-8974-e3b7e1470910.txt


API encoding configuration

Story
When fetching resources using HTTP, the json representation of the data currently includes href's to the linked resources:
Below is an example from resgate-test-app's  notesService.notes resource:
[
  {
    "href": "/api/notesService/note/1",
    "model": {
      "id": 1,
      "message": "One"
    }
  },
  {
    "href": "/api/notesService/note/2",
    "model": {
      "id": 2,
      "message": "Two"
    }
  }
]

In some cases, a more flat representation is wanted as opposed to the the HATEOAS style currently used.
[
  {
    "id": 1,
    "message": "One"
  },
  {
    "id": 2,
    "message": "Two"
  }
]

Solution
A new config option should be added:
apiEncoding (string)
Encoding used to represent web resources.

"json" - JSON encoding with resource references included in the data (default)
"jsonFlat" - JSON encoding with no resource references except for cyclic references.

Example: "json"
Notes
Cyclic references
By using the jsonFlat setting, there is no unambiguous way to represent cyclic references:
Example of the jsonFlat data representation of cyclic.a:
{
  "a": { "href": "cyclic.a" }
}

It is most likely a cyclic reference, but could also be that "a" links to a simple model "b" that just happens to have a single property, href, with the value "cyclic.a".
This is impossible to avoid with jsonFlat. Users just needs to be aware of the case.
Multiple encodings
The solution opens up for other encodings as well, such as xml.
Later on (not in this story), it might be possible to use file-endings in the URL to specify which encoding to use.

/api/notesService/notes.json - Default json encoding with href
/api/notesService/notes.flat.json - jsonFlat encoding
/api/notesService/notes.xml - XML encoding

The setting set for apiEncoding will be used when no file ending is given.

Solved in #88


Fix indexing of datasets with embedded locations

Explicitly always give ODC a location for indexed datasets.
(When the location is inside the document, rather than external via filesystem, the normal indexing api was not recognising them.)

Thanks @dunkgray  -- fix has been deployed in 0.28.1


Is Docker-in-Docker ( DinD ) possible with Windows containers ?

Linux containers support DinD, which streamlines a lot of build pipelines. Is this feasible or planned with Windows containers ?

Hi, thanks for reaching out and for your patience! Currently, docker-in-docker is not supported for Windows containers. However, we have noted this feature request for our long-term roadmap. Since this is currently not supported, I will be closing out this issue.


Cobalt menu

Platform
Windows
SDK commit
No response
Geode Version
v1.0.0
Mods Installed
Cobalt Menu v0.1.2
Expected Behavior
I expected it to update
Actual Behavior
It didnt fully download and gave me an error message
Steps to Reproduce
I honestly don't know, I just tried to update the Cobalt menu and it have me a bug error
Additional Information
No response

Same thing happens to me, o think this is an error with the mod repo/indexer

was fixed manually
~ courtesy of mat


chore(deps): Remove unused dependency on jasmine-core.

Jasmine comes bundled with karma-jasmine 0.1.0, and our dependency on jasmine-core is both unused and misleading (we're not using Jasmine 2). Removing it.

Looks good, tests pass in Travis. Recommend merge.


Add tenant min listing price config

will this config be available in the tenant info API?

It will be included in [GET] /config/tnt/<tenant> if it is set, and when creating listings will be enforced


DUPLO-13203 support for duplocloud_aws_host.allocated_public_ip

Change description
Adds support for the allocated_public_ip attribute on the duplocloud_aws_host. Previously, this attribute was only a passthrough value for the resource to the Duplo API. However, the Duplo API does not rely on this flag when determining the configuration of the network interface controller.
New Features

If the allocated_public_ip property is set to true (default is false) and there is no NetworkInterface config provided in the NetworkInterfaces attribute, then a NetworkInterface config is created for it with the subnet_id attribute set to be the tenant's external subnet assigned to the provided zone (default zone is 0).
For all existing NetworkInterface configs, if the network_interface_id was left blank, the subnet_id attribute is set to be the tenant's subnet assigned to the provided zone (default zone is 0). If the allocated_public_ip attribute was true, the external subnet ID for the zone is used and if it is false, the internal subnet ID for the zone is used.

New Validation

If a subnet ID is provided for a NetworkInterface config by the user and it conflicts with either the zone attribute or the allocated_public_ip attribute, an error is returned.  In detail, this means that either the subnet is not in that zone, it is an external subnet when the allocated_public_ip attribute is true, or it is an internal subnet when the allocated_public_ip attribute is false.
If the tenant ID seems to be invalid when retrieving subnets or no subnets were found, an error is returned.
If the zone is out-of-range, an error is returned.

Edge Cases

If there are one more more NetworkInterface configs provided by the user and all of them have the network_interface_id attribute set, then the allocated_public_ip attribute will have no effect.
If the network_interface_id attribute is not set, the subnet ID will be set according to the discovery logic outlined above. If there is another NetworkInterface config that has the network_interface_id attribute set alongside a config that does not (e.g. multiple NetworkInterface configs, mixed), the subnet ID of the interface without a network_interface_id will still be set according to the discovery logic outlined above without regard to how the interface referenced by ID is configured.

Type of change

[ ] Bug fix (fixes an issue)
[x] New feature (adds functionality)

Related issues

Fix #1

Task linked: DUPLO-13203 TF: duplocloud_aws_host: allocated_public_ip attribute is not working. Host is not getting created with public ip if value of attribute is true


chore: bdd-spring-1573084063 to 0.0.1

chore: Promote bdd-spring-1573084063 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


asyncStoragePersistor for React Native

Implementation
I looked at the createLocalStoragePersistor and just changed a few lines and necessary imports. Please feel free to suggest if there is anything I should change or improve.
How to use
import {asyncStoragePersistor} from './utility/createAsyncStoragePersistor';

const queryClient = new QueryClient({
  defaultOptions: {
    queries: {
      cacheTime: 1000 * 60 * 60 * 48, // 48 hours
      staleTime: 1000 * 60 * 60 * 24, // 24 hours
    },
  },
});

const storagePersistor = asyncStoragePersistor({
  asyncStorageKey: 'AsyncStorageReactQuery', // asyncStorageKey vs localStorageKey
  throttleTime: 1000,
});

persistQueryClient({
  queryClient,
  persistor: storagePersistor,
  maxAge: 1000 * 60 * 60 * 24, // 24 hours in milliseconds
});

How to test
Use some queries to get the data. Restart the React Native app. We should see the UIs with previously called queries immediately without needing to call the APIs.

looks good, but could you please streamline it with the other plugins, meaning:

typescript
add to tsconfig.types.json
docs for the plugins
package.json
add it to the rollup config

basically, if you search for createLocalStoragePersistor-experimental everywhere, you'll see what you'll need to add :)

Added docs for plugins. Sorry, I am not a type script person and.. not very familiar with all the rollup config... :(

no worries. I think I can check out the branch and add some commits I guess?

Please feel free to add more commits. I can verify that the code is working coz I am running the codes now and tested a few times by restarting the app and can confirm querycache data is there.

@TkDodo @Aung-Myint-Thein We are going to test this PR as well in the coming days. We'll keep you updated if we have any issues or comments.

can you please update your fork and branch with the latest release, as we've now made some changes to the webstorageplugin and I would like to avoid conflicts

build fails because:

Cannot find module '@react-native-community/async-storage' or its corresponding type declarations.

How are we going to go about this please? I'd rather not add this as a peerDependency. Maybe it's better to change the interface to pass the actual AsyncStorage as a parameter to asyncStoragePersistor ? We've done something similar to the WebStoragePersistor...

@TkDodo What about async setItem ?

@TkDodo What about async setItem ?

Yes, we'd need something as a parameter that has an async getItem and an async setItem method :)

we'd need something as a parameter that has an async getItem and an async setItem method :)

tried that here: 06126123

k, but we cannot just not throttle, can we? we can make it an async function to have it always return a Promise?

I think so :D

@TkDodo we could use smth like p-throttle https://github.com/sindresorhus/p-throttle wdyt?

@TkDodo we could use smth like p-throttle https://github.com/sindresorhus/p-throttle wdyt?

what would be the drawback of just using the normal throttle that doesn't return anything?

Could be there race conditions? I mean one operation could be faster than the other one but order is not maintained on JS side

Probably this package would even more fill our needs

I came up with this https://gist.github.com/TMaszko/a6b9cb876340569c7daf8d520509d3c0

This is all looking really good. I think the changes to make things more flexible is the right move. Inverting control out of the API to the user is always a good decision, same way we did it with the other web storage api's.
Keep cruisin'!

@TMaszko thanks, I used your asyncThrottle implementation instead of throttle here: 73a20c4b

I created a third-party module for this exact purpose a while ago: https://github.com/dan-lee/react-query-native-persist
It can handle multiple storage backends (with the help of adapters). Maybe this can serve as inspiration regarding @tannerlinsley's point in his last comment. I'd also be willing to contribute if wanted.

@TMaszko @Aung-Myint-Thein  the PR would be good to go from my side, but I haven't really tested it. If someone can make a quick test with AsyncStorage, and maybe as well with localForage to see if it's compatible with the api, that would be much appreciated

@TkDodo I've tested already AsyncStorage persistor in the project but in the form of custom persistor (gist code) and It was working correctly.

When will it be merged?

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Compiler does not recognise localStorage interface

Use of localStorage interface results in warnings from the compiler not recognising the function calls, necessitating the use of @ignore(localStorage) to compile without warnings.
Example compiler output:
controlpanel.Application: [65,11] Unresolved use of symbol localStorage.getItem
controlpanel.ActiveCalls: [49,11] Unresolved use of symbol localStorage.getItem
controlpanel.ActiveCalls: [85,8] Unresolved use of symbol localStorage.setItem
controlpanel.Filter: [43,10] Unresolved use of symbol localStorage.setItem
From source lines that look like:
if ( localStorage.getItem("application.phonesFilter") )

Done with #578


pre-allocate buffer in xmlhttprequest.js

This PR is associate to #1786

ci check still has 1 failed at XMLHttpRequest/responsetype.html, node.js v7.2.1, say set responseType timeout. Does this fail really matters?  I failed to figure out why this timeout happens...

Yes, that's a pretty crucial test; it needs to pass or we cannot accept this.

All 1300+ tests pass except XMLHttpRequest/responsetype.html. And I use the unchanged version of jsdom on my own machine (directly git clone from https://github.com/tmpvar/jsdom.git) and run test-wpt again, this case still failed with the same error result.  @domenic Have you ever seen such kind of error case?

I have not. But if you're saying it's failing before the change too, then yeah, it's probably not the fault of this PR. I'll do my best to investigate this weekend. With any luck you won't need to change anything and I'll just be able to merge after fixing whatever's wrong there.


322 nacl explain no conn query

contained in https://github.com/np-guard/vpc-network-config-analyzer/pull/341


feat: add @types/webpack-env

It looks like we can't delete declare const require: because of compatibility issues with older versions.

Thank you for the contribution.  However, not adding additional project level dependencies was an intentional choice in this case (PR).
This added package is also Webpack specific which is something that is generally avoided at the public interface level as Webpack is primarily considered an internal implementation detail.
Ideally, the usage of Webpack specific functionality in these files should eventually be removed but this may require larger changes in design and implementation to accomplish.


Provide an option to set default schema for ConfigurationDbContext and PersistedGrantDbContext

Hi,
Thanks for your work, If you could provide an option to set default schema for ConfigurationDbContext and PersistedGrantDbContext while adding those stores to IdentityServer host that would be great.

I cannot figure out how to use this. I've set the schema in the Migrations and EF created the tables in the proper schema.
How do I configure the ConfigurationDbContext to use that schema?
I keep getting an error in Startup when I try to populate Clients from the In Memory ones used during QuickStart. The error shows that the schema is not being used when querying for Clients:
An Exception occurred in the database while iterating the results of a query. Invalid object name 'Clients'.
The generated SQL is:
SELECT CASE
WHEN EXISTS (
SELECT 1
FROM [Clients] as [c])
THEN CAST(1 AS BIT) ELSE CAST(0 AS BIT)
END
My schema is not being used to query the table.

Have you set ConfigurationStoreOptions? These were released with RC2.
EF cannot use a custom schema without first being made aware of it.

Thanks for the quick response.
I'm away from the code but have something like this in ConfigureServices
services.Configure(options => DefaultSchema("idsrv");
I guess that is what I can't figure out. Should I just create a subclass?

Oops, I see the editor changed what I wrote. Now that I have the code in front of me, this is what I have in ConfigureServices:
`services.Configure(options =>
{
options.DefaultSchema = "idsrv";
});`

Why are you registering the options manually and not using something like:
services.AddIdentityServer().AddConfigurationStore(builder =>
    builder.UseSqlServer(connectionString,
         options => options.MigrationsAssembly(migrationsAssembly)),
    storeOption => storeOption.DefaultSchema = "idsrv");

Hi Scott,
Thank you!
I figured there was something I was doing wrong.
For my own enlightenment, where in the source code would I have found that assignment? I started to look for something like that but was not sure where it would be.
Again, thank you!

https://github.com/IdentityServer/IdentityServer4.EntityFramework/blob/release/src/IdentityServer4.EntityFramework/Extensions/IdentityServerEntityFrameworkBuilderExtensions.cs#L19


Error handing when selecting logfile path

Fixed in 1.2.7

Having problems selecting logfile path after upgrading to Windows 11.
Window automatically disappears 5 second after opening it.

Hi, could you try the following:
1. Stop Traxile
2. Delete the config file: \AppData\Roaming\TraXile\config.xml
3. Try to start TraXile again

In the same directory, there should be a logfile - maybe there is some helpful information?

Thanks. Deleting file worked.

Fixed in current version


Improve Profile Edit: Make date input for birthday

What is the feature?

Make User can input birth day by use calendar
*### How to test?
Manual try to use it.

Codecov Report

Merging #98 (91b4184) into beta (153acaa) will increase coverage by 0.11%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             beta      #98      +/-   ##
==========================================
+ Coverage   69.71%   69.83%   +0.11%
==========================================
  Files          25       25
  Lines        1027     1031       +4
  Branches      135      135
==========================================
+ Hits          716      720       +4
  Misses        286      286
  Partials       25       25

Impacted Files
Coverage Δ

users/forms.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Include the /Lang-property, when it exists, in the StructTree-data (issue 14261)

Please note: This is a tentative patch, since I don't have the necessary a11y-software to actually test it.

/botio-linux preview

@brendandahl Should we land this as-is for now, since I'm guessing that unless bug 1741122 is fixed directly in Firefox we'd probably need to make larger (and more invasive) changes to the entire StructTree code?

/botio unittest


k8s scanner detection

k8s scanner detection

This PR was created by Automated Detection Testing Service :robot:
Tested detections:
Kubernetes Scanner Image Pulling :x:

diskUsage: 163840
runDuration: 0.9460000000000001s
scanCount: 0

This PR was created by Automated Detection Testing Service :robot:
Tested detections:
Kubernetes Scanner Image Pulling :x:

diskUsage: 163840
runDuration: 1.164s
scanCount: 0

This PR was created by Automated Detection Testing Service :robot:
Tested detections:
Kubernetes Scanner Image Pulling :x:

diskUsage: 159744
runDuration: 1.025s
scanCount: 0

This PR was created by Automated Detection Testing Service :robot:
Tested detections:
Kubernetes Scanner Image Pulling :x:

diskUsage: 159744
runDuration: 1.718s
scanCount: 0

This PR was created by Automated Detection Testing Service :robot:
Tested detections:
Kubernetes Scanner Image Pulling :white_check_mark:

diskUsage: 163840
runDuration: 1.177s
scanCount: 1


[Feature Request] Use id_rsa when password is empty

Description
I'm able to ssh or scp to an host using my default id_rsa key.
Changes
When host is not in termscp ssh key storage, and password is not provided, try to use is_rsa (or previously ssh-added)

That could work with the IdentityFile parameter in the ssh configuration actually. If it's ok I put it in my backlog.

Implemented IdentityFile check in 64e3848c (0.10.0)


stripe-ruby-mock doesn't allow Plan.amount to be nil

The stripe api says Plan.amount should be positive integer or zero (https://stripe.com/docs/api/plans/object#plan_object-amount) - yet for Plans with pricing tiers that field is null (or nil).
stripe-ruby-mock currently has a number of checks to ensure that Plan.amount is a positive integer field, which breaks tests using Plans with pricing tiers.

looks like the plan validation was added here: https://github.com/stripe-ruby-mock/stripe-ruby-mock/commit/081f6103cf8469f0fbf1ca963e04226492590dee


Show operationID in generated collection

It would be useful to have operation ID shown in a generated collection. It might be request name extension in case if it's impossible to show as a separate field.

@armen-ch The operation ID is an optional field in the OpenAPI specification.
Postman has its own format and structure, which does not support the definition of mapping the OAS operation ID.
For now we don't plan to overwrite the default structure and keep the 1:1 mapping of the OAS summary => Postman Name & OAS Description => Postman description.

Hi @thim81, adding "operationId" to the postman collection structure doesn't affect the postman functionality and could be helpful for debugging postman collection. Though I understand your point of view of  1:1 matching to the postman structure.
Thank you

We are hoping that Postman would support the option to add "tags" to a request 🤞 that would be the ideal place to store the OperationId


Integration with Tidy5e Sheet Auto Exaustion

Is your feature request related to a problem? Please describe.
Tidy5e Sheet offers the possibility to leverage CUB instead of its own internally coded AE for exustion. I'd like an option to exist also to use CE if present.
Describe the solution you'd like
a PR mostly focused on this and the relative module settings
Describe alternatives you've considered
Cross-referencing my feature request against Tidy5e directly #408

Good suggestion. I never used that feature on Tidy and didn't know it integrated with anything. I'll leave this open here until sdenec gets a chance to provide their input.

Stale, no input from Tidy so 🤷


Only show top 3 authors in author distribution

Allow it to be expandable like the legend

And instead of showing 0%, show <1%


Allow a semantics to be passed an input string OR a matchResult.

Since a semantics instance is always associated with a particular, concrete grammar, we could support a shorthand that avoids calling g.match:
var g = ohm.grammar('Arithmetic { ... }');
s = g.semantics().addOperation('eval()', { ... });
s('1 + 2').eval();

If the input fails to match, I guess it would have to throw an exception.

I guess that makes sense without the object matching now.

Don't think we need this — closing.


Bring 17.5 branch up to date

I think https://devdiv.visualstudio.com/DevDiv/_git/VS/pullrequest/444185 is the last insertion into the 17.5, but the release/dev17.5 is not up to date.
It is on https://github.com/dotnet/razor/commit/1ad1a3de9c4ab27ec11955b613edcf671effefa8
So I think we need to bring this up to https://github.com/dotnet/razor/pull/8158/commits/e0f81639fc489b6b7427f53e5d71dfff47ca9276
Wait @davidwengier  to check if there is any other Razor insertion to 17.5.

I don't think I even knew we had a release/dev17.5 branch 😁
Happy to confirm that you've picked the right commit for what we're shipping though...
C:\Code> roslyn-tools vsbranchinfo -p razor -b rel/d17.5
Razor info from VS branch rel/d17.5:
  Build Number: 20230105.2
  Commit SHA: e0f81639fc489b6b7427f53e5d71dfff47ca9276
    Link: https://github.com/dotnet/razor/commit/e0f81639fc489b6b7427f53e5d71dfff47ca9276
  Source Branch: main
  Build: https://dnceng.visualstudio.com/7ea9116e-9fac-403d-b258-b31fcf1bb293/_build/results?buildId=2080563


[Component name] Web Component: Add Tags to the bottom of the card component - DDS consulting

User Story

As a [user role below]:
Carbon for ibm.com developer

I need to:
create/change the Card

so that I can:
provide the ibm.com adopter developers components they can use to build ibm.com web pages

Additional information

Story within Storybook with corresponding knobs
Utilize Carbon
Create with Shadow DOM and Custom Elements standards
See the Epic (#7281 ) for the Design and Functional specs information
Web Components Visual QA testing issue (#?designqa issue?)
Web Components Prod QA testing issue (#?prodqa issue?)

Acceptance criteria

[ ] Include README for the web component and corresponding styles
[ ] Create Web Components styles in styles package
[ ] Do not create knobs in Storybook that include JSON objects
[ ] Break out Storybook stories into multiple variation stories, if applicable
[ ] Create codesandbox example under /packages/web-components/examples/codesandbox and include in README
[ ] Minimum 80% unit test coverage
[ ] Update the Carbon for ibm.com website component data file to be sure the web site Component Status and Storybook links are correct
[ ] Use the Visual QA checklist to verify design quality
[ ] If a design is provided, the Designer is included as a Reviewer in the Pull Request
[ ] Provide a direct link to the deploy preview for the designer in the Pull Request description
[ ] A comment is posted in the Prod QA issue, tagging Praveen when development is finished

Hey team! Please add your planning poker estimate with ZenHub @andysherman2121 @annawen1 @emyarod

Please add your planning poker estimate with ZenHub @ariellalgilmore

Please add your planning poker estimate with ZenHub @BrunnoM7

Please add your planning poker estimate with ZenHub @jeffchew

Please add your planning poker estimate with ZenHub @guilhermelMoraes

Please add your planning poker estimate with ZenHub @IgnacioBecerra

Please add your planning poker estimate with ZenHub @kennylam

@shixiedesign @justinlam2 Looks like the updated design specs for the Card component haven't been added to the Card UI directory yet (https://ibm.ent.box.com/folder/95514001950?s=hgvu48r1fv2ga2h0yn3qshrtxjc71vch). Please let me know when they're ready so that we can move forward with this dev issue. Thanks!


OCPBUGS-12630: Address CVE-2022-41723

Bump go to 1.19. We're already using this version in our images.

/assign @pierreprinetti

/jira refresh

You’ll probably need to clone the bug to 4.14 and close it in Jira after setting the “depends on” link in the right direction.
/approve
/lgtm

/jira refresh

/jira refresh

/label backport-risk-assessed

@pierreprinetti Done. I've also extended this to ensure we install golang.org/x/http v0.7 now.

/hold
Do you want to add a comment to the go.mod replace, linking to the Jira bug?
Either way:
/lgtm

/hold Do you want to add a comment to the go.mod replace, linking to the Jira bug?

I think the commit message should do the trick?

/hold Do you want to add a comment to the go.mod replace, linking to the Jira bug?

I think the commit message should do the trick?

Ok, then feel free to remove the hold

/unhold

/label cherry-pick-approved


Fix SSL certificate check when hostname is an IP address

The previous logic in SSLFilter::Connect was assuming the host is always a DNS name so it was unconditionally calling X509_VERIFY_PARAM_set1_host even for hosts that were actually IP addresses. This in turn was causing a CERTIFICATE_VERIFY_FAILED: Hostname mismatch(handshake.cc:393)) exception when a client attempted to connect to an IP address (rather than a DNS name) host.
This CL fixes that by checking whether the host is an IP address or a DNS name and calling either X509_VERIFY_PARAM_set1_ip_asc or X509_VERIFY_PARAM_set1_host depending on the result.
PS: The certificates had to be regenerated because the previous configuration in sample_certificate_v3_extensions had the <IP_ADDRESS> and ::1 IP addresses stored as DNS subject alternative names. This meant that the newly introduced tests would still pass even with the previous logic, but for the wrong reasons.
Bug: #35462, #49183

/cc @brianquinlan

I've left a comment on the gerrit review: https://dart-review.googlesource.com/c/sdk/+/296643

The comment is actually on https://dart-review.googlesource.com/c/sdk/+/296722. This is rather puzzling - we seem to import the same PR twice under different CL numbers. (/cc @athomas)

Regarding Mac OS X failures. Among other things please make sure that you adhere to https://support.apple.com/en-us/HT210176 when generating test certificates - that's the most common source of issues.

@brianquinlan @mraleph Thanks for the prompt review.
I've dropped the certificate regeneration from this initial set of changes. Unfortunately, as I've mentioned above, this means the newly introduced tests in secure_server_client_certificate_test.dart would still pass even without the logic fixes from this pull-request.
I'm still planning to submit a separate pull-request which removes the IP addresses from the DNS alt-names and regenerate all the test certificates once I figure out how to that without breaking the tests.
However, I don't think it's worth blocking this pull-request until the certificates are regenerated, because they are arguably two different bugs:

SSL check is broken when hostname is an IP address (fixed by this pull-request)
testing SSL certificates are misconfigured to contain IP addresses in the DNS field (hopefully fixed in the next future pull-request)

I have submitted a separate change for regenerating the test certificates at: https://dart-review.googlesource.com/c/sdk/+/300000.


[MacOS] Build failed

build log here
custom DownloadFfmpeg.ps1

I think mono doesn't support WPF, so this is why it can't build it

get a stable window and Linux version first then try porting it to mac


Detailed documentation of the oplog entries.

Documentation for oplog.values with examples and some description of the values I found documentation for.

@haadcode can this be merged?

lgtm 👍 thank you @CSDUMMI 🙏


test: it's to test cross

/test-cs9-dev-replace

@shi2wei3 Our cross build works. Both x86_64 and aarch64 tests passed. And both of them was running on x86_64 testing farm runner.


lxd/cluster: Use hook for initial heartbeat

Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

@tomponline figured I'd give that a quick go to see if I can get rid of the remaining non-panic cluster failures.

@brauner @freeekanayaka should be good to go. The failures are either CEPH (which #5980 fixes) or dqlite segfaulting in unit tests.

Good to go again. Added a commit which should reduce the number of odd cluster heartbeat failures we've been seeing.


new: add k8s-metacollector binary to .gitignore

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
What this PR does / why we need it:
This PR adds the k8s-metacollector binary to .gitignore
Which issue(s) this PR fixes:
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

This could be also useful when the k8s-metacollector is imported as a git submodule

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Custom coverage format in json output

Do you want to request a feature or report a bug?
Feature / Question
What is the current behavior?
Not sure if this is the best place, but currently we can get the result programmatically via --json flag. The problem is when I try to integrate this to phabricator to collect code coverage, phabricator expects the coverage to have a specific format (for example NNNNNCCCNNNNNNNNCCCCCCNNNUUUNNNNN) as described here https://secure.phabricator.com/book/phabricator/article/arcanist_coverage/, but the json output cannot produce that out of the box. It outputs an object with statementMap, branchMap, etc inside the coverage property
Here's the current JSON output
{
  "testResults": [
    {
      "name": "/path/to/test/file",
      "coverage": {
        "/path/to/file/": {
          "path": "/path/to/file",
          "statementMap": {},
          "branchMap": {}
        }
      }
    }
  ]
}

What is the expected behavior?
Considering I heard FB also uses phabricator, do you think it's possible to support those custom format out of the box?
Here's the proposed JSON output
{
  "testResults": [
    {
      "name": "/path/to/test/file",
      "coverage": {
        "/path/to/file/": "NNCCCNNNNNNNNCCCCC"
      }
    }
  ]
}

Thanks :)

@kentaromiura is actually about to open source the code we use for coverage in phabricator at FB soon.
@ide who might care. This is super useful during code review.

Example:

@cpojer good UI. Let’s change it by default. And move HTML page to coverage/ (right now it is on subdir) for better UX.

@ai this is not for the coverage html output. This is for phabricator integration: https://www.phacility.com/

@cpojer oops :(. Should I create issue with little HTML coverage tweaks? Or it is not so important?

the HTML coverage report is coming from istanbul, not from Jest: https://github.com/istanbuljs

We open sourced jest-phabricator-coverage with Jest 19. Feel free to use it as testResultsProcessor.


Certificate arn config tests and README update

I've updated your code with the following changes:

Updated from latest master
Added tests and Readme updates for certifcateArn config

I'm hoping these changes will help you get your PR merged

Hey, thanks! This small project has been slipping to the bottom of my TODO list for weeks now


Add sort methods to Slice

Issue: https://github.com/crystal-lang/crystal/issues/6498

This PR depends on this fix: https://github.com/crystal-lang/crystal/pull/7591
I extracted the sort logic from Array to a module. If this approach is approved, I'll try to add some docs for the module (or if someone has more skills with writing docs :) )

You can now rebase against master :-)

What's the purpose of Sortable module?  It doesn't define an actual interface, just some class methods  can easily be placed in an existing namespace like Slice or Array. Only intro_sort is publicly visible at all (that might even be questionable).

Is it possible to move sort, sort!, sort_by etc into Sortable?

@yxhuvud Yes that should work. The exclamation methods need a tiny adjustment, but #to_unsafe returns the pointer in both Array and Slice.
But I'm not sure if this module would be much of a benefit. Duplicating the code for sort methods is one thing, but that's not really an issue. The question is, is having a Sortable interface useful to users?

@straight-shoota
I extracted the sort logic from Array. Should I just rename the module to Sort for example and just require it in Array and Slice? or do you have another suggestion?

@Maroo-b Your changes (the Sort module) don't show up for some reason...
My thoughts:

there's no need for a Sort or Sortable module that can be included. In fact there's no need for it to be generic, the T isn't used anywhere
I agree that the sort helper methods (currently in Sortable) should exist somewhere where this logic can be shared, but let's not use global names like Sort or Sortable which can collide with user-defined types. I think just putting the helper methods in Slice or Pointer::SortUtils or Pointer::SortHelpers is good enough.

@asterite thank you for the feedback!
Sure I agree, I'll move those methods to Pointer::SortUtils then :)

IMHO the name Pointer::SortUtils doesn't make much sense. The sort algorithm is implemented on pointers, but conceptually, it operates on collections.
I'm still not convinced that we need a separate namespace for these methods anyway, but if we do, I'd call it Slice::Sort or Array::Sort. Or maybe Crystal::IntroSort?

The way I imagine it, the sort methods belong to Slice. Then Array simply does Slice.new(to_unsafe, size).sort.

@straight-shoota @asterite just to confirm, should I move the sort logic to Slice and update Array to use Slice for sorting?
I'm concerned about creating a Slice instance just for sorting. is it fine?

It's fine, Slice is a struct. And yes, please move the methods to Slice. Thank you!

Creating a struct is basically free of cost.
I'd suggest to add Array#to_slice as a shortcut for Slice.new(to_unsafe, size). Might be useful as a public method, too.

@straight-shoota but we'd get back to https://github.com/crystal-lang/crystal/issues/5173#issuecomment-340286755 ?

@RX14 wrote: We removed Array#to_slice because it would cause a segfault if you kept a reference to the slice returned from that method and the array was resized.

Looks great!
I'd prefer to put the sort methods in a separate file (e.g. src/slice/sort.cr) though for some kind of compartmentalization.

What about moving the specs to spec/std/slice/sort_spec.cr ?

@bew it's an option, but I prefer to keep Slice specs in one file for the moment.

PR updated to resolve conflicts and apply changes from this PR: https://github.com/crystal-lang/crystal/pull/7639

Seems like GTG 🚀


RU Translate Issue

Unfortunately, you forgot to specify a link to the Russian translation in the module settings.

Yup, just saw that 2 minutes ago, sorry about that, 4.5.1 will be out within a few minutes.


Integration with orchestra/canvas

This would allow any developer to use Spatie stubs instead of the default bundled in orchestra/canvas for Package development.
Installation
On any package project.
composer require --dev "spatie/laravel-stubs" "orchestra/canvas:^5.2"

And then run
./vendor/bin/canvas preset package

And then modify the generated canvas.yaml file:
preset: Spatie\Stubs\Canvas\Package
namespace: YourPackageNamespace

https://github.com/orchestral/canvas

Thanks!


Help button for command list + Notepad update

List UIs can now have window title buttons.
Added a 'help' title button to cmd list that shows usage info.
Increased the default window size of lists.
Text size customization for notepad UI.

I always thought there should be a usage menu available from :cmds. Good work


Some problems about the dataset Human36M

I have tried to train the model on dataset human36M. There are some questions.
1.Why use the model "./data/pretrained/human36m/human36m_alg_10-04-2019/checkpoints/0060/results/train.pkl"?
If I don't use all the dataset Human36M to train, such as the "./data/human36m/extra/human36m-multiview-labels-GTbboxes.npy" only include S6,S7,S8,S11, The train.pkl doesn't work.
2.Could you provide the completed file "./data/human36m/extra/human36m-multiview-labels-GTbboxes.npy" for me ? Or you could provide the completed dataset Human36M rather than the link to download the it.
I'm looking forward to your reply.

Hi, @sunny0212452!

What exactly doesn't work?
You can find it on Google Drive. We cannot provide a complete dataset due to lisence issues.

Hi, @karfly
Sorry to bother you again.
I've trained a multi-view volumetric model successfully and evaluated the absolute MPJPE, the result is like below (with damaged scenes in S9):

    "S9": {
        "Average": 31.51959048284118,
        "Directions": 20.229816436767578,
        "Discussion": 21.484794921875,
        "Eating": 20.372833251953125,
        "Greeting": 76.18420780066288,
        "Phoning": 20.91855948311942,
        "Posing": 20.023085071194558,
        "Purchases": 20.52593300559304,
        "Sitting": 23.380282432154605,
        "SittingDown": 89.24025844520247,
        "Smoking": 20.969817446966243,
        "TakingPhoto": 21.79986317952474,
        "Waiting": 78.53070850861378,
        "Walking": 20.59368426983173,
        "WalkingDog": 22.292724609375,
        "WalkingTogether": 20.858835856119793
    }

So I want to evaluate the absolute MPJPE without damaged scenes. But when I set with_damaged_actions: false, I've met a problem that the number of the processed samples doesn't equal to the sample number you provide in "./data/pretrained/human36m/human36m_alg_10-04-2019/checkpoints/0060/results/val.pkl". It reports the error below

f"has {len(self.keypoints_3d_pred)}. Did you follow all preprocessing instructions carefully?"
AssertionError: [train=False, test=True] ./data/human36m/extra/human36m-multiview-labels-GTbboxes.npy has 2107 samples, but './data/pretrained/human36m/human36m_alg_10-04-2019/checkpoints/0060/results/val.pkl' has 2181. Did you follow all preprocessing instructions carefully?

I know the volumetric triangulation approach uses predictions obtained from the algebraic triangulation. But is there a way for me to drop out the damaged scenes in the 'human36m_alg_10-04-2019/checkpoints/0060/results/val.pkl'?
Thank you :)
Shirley

@ShirleyMaxx, please inspect the evaluation output more carefully and you'll find, that it contains both "with damaged" and "without damaged" metrics.

@ShirleyMaxx, please inspect the evaluation output more carefully and you'll find, that it contains both "with damaged" and "without damaged" metrics.

Sorry, I still couldn't find these metrics. Do you mean the "metric.json" file in the 'logs' directory? However, this json file only recorded the per_pose_error and per_pose_error_relative, and they all included the damaged scenes.

And I’ve checked the evaluation output folder many times, it seemed that only the metric.json file recorded the metrics. I've also checked the TensorBoard but couldn't find "with damaged" and "without damaged" metrics, either.

Did I miss something?

@ShirleyMaxx, oh yes, it contains absolute/relative errors, I forgot about it.
Sorry, now I can't help you with code to calculate metrics without damaged scenes, but it's not going to be too difficult to implement having human36m.py.

Okay, I've got it. Thank you! 👍


goole place autocomplete

goole place autocomplete doenot work on another componenet rather than app.componenet.ts

https://github.com/rajan-g/angular2-google-place-autocomplete/issues/22
duplicate


infinite loop when using show-curl-before-try

When using show-curl-before-try with big openapi files an infinite loop seems to hang out the rendering

is it possible to get a sample spec for us to recreate the issue at our end ?

Hello @mrin9
One of the users of eventcatalog reached out to us about the missing functionality for large specs. By removing that parameter  show-curl-before-try, it seems to be working.
Let me give some more context
i render the rapidoc in astro
---
.......
---
<rapi-doc
        spec-url={buildUrl(pathToSpec, true)}
        render-style="read"
        show-header="false"
        allow-authentication="true"
        allow-try="true"
        default-schema-tab="schema"
        show-curl-before-try="true"
        use-path-in-nav-bar="true"
        show-method-in-nav-bar="as-colored-block"
        theme="light"
        schema-style="table"
        class="relative top-0"
        style={{ height: '100vh', width: '100%', zIndex: 100 }}
        regular-font="ui-sans-serif, system-ui, sans-serif, Apple Color Emoji, Segoe UI Emoji, Segoe UI Symbol, Noto Color Emoji"
        bg-color="#ffffff"
        primary-color="#6b21a8"
        nav-bg-color="#fff"
        nav-text-color=""
        nav-hover-bg-color="#fff"
        nav-hover-text-color="#6b21a8"
        nav-accent-color=""
      />

I will check with the show-curl-before-try related code if there is any  infinite looping possibility.
It could be that the endpoint payloads may be too complex/large for it to generate all the curl syntaxes in a large spec.
can you try render-style="focused" and update me if that worked
For large specs we recommend focused render-style as it displays only one path at a time.

Thanks @mrin9 i see the focuse and show-curl-before-try true works great with a dummy spec
i ll reach the user to see if can try that and see if it works
but a problem with focused is for mobile users its not as desired for mobile user specs


FLOW 2: offering : uitwerken safety

In the offerings consumer we defined the need for "safety", a way to describe how secure the test should be. (locked in a room, locked down computers, none , etc)
This needs to be described in more detail

opties: (array, choose any combination)

Secured Computer
Fixed Location
Surveillance


Fixed #30733 -- Doc'd that datetime lookups require time zone definitions in the database.

In the QuerySet documentation, many of the APIs that work on
datetime fields have a variation of the same notice indicating
that when the USE_TZ setting is set to True, the database is
used to do timezone conversions.  This commit replaces the
variations with, IMO, the most helpful of these variations.

@recvfrom Welcome aboard :boat:


100% CPU on onnxruntime threads caused by spin-waits

Describe the bug
I observed onnx_runner.py threads consuming nearly 100% of 4 CPU cores on my 8 core PC when running Openpilot simulations with Carla. Basically, one thread seems to consume less than a full core, while the others are at a solid 100%. I then tried different values for "options.intra_op_num_threads" in onnxrunner.py. The following results are from the System Monitor utility, where 100% means all 8 cores are busy.
11% with CPUExecutionProvider  (1 thread)
21% with CPUExecutionProvider  (2 threads)
35% with CPUExecutionProvider  (3 threads)
44% with CPUExecutionProvider  (4 threads)
63% with CPUExecutionProvider  (6 threads)
3% with CUDAExecutionProvider (4 threads)
OS Version
Ubuntu 20.04LTS
openpilot version or commit
e178bed3016957ae09228a91f793c8d5c00d5573
Additional info
I was not able to get a "pip install onnxruntime-gpu" configuration working with my RTX2070 CUDA/CUDNN setup. I then built onnxruntime from source to see if it would help- it gave the same results for CPU operation, but now I got the CUDA/GPU working, with dramatic reduction of CPU time (after minor mods to onnx_runner.py to select CUDA execution). After a good deal of investigation it seems that onnxruntime CPU operation defaults to spin-waits on the input queues, which fully accounts for the measured results. Great for benchmarking, but not all that practical in this case. Given the low execution rate of the supercombo model, spin waits are major overkill; the operating system's scheduling should be entirely adequate.
Unfortunately, the latest onnxruntime APIs only support changing the allow_spinning option from C++; it isn't supported in the Python API. While the CUDAExecutionProvider solves my immediate simulation problems, it should be possible to rewrite the Python code for C++ disabling of the spinning.
There are probably broader issues than just my Carla simulation on a PC- the Comma2 and Comma3 (and other) devices may suffer from the spinning, resulting in higher power dissipation than necessary, along with loss of available processing capability. At the very least, it would be a good idea to allow tuning the "options.intra_op_num_threads" value from an environment variable, to better match hardware capability with processing load.

There are probably broader issues than just my Carla simulation on a PC- the Comma2 and Comma3 (and other) devices may suffer from the spinning, resulting in higher power dissipation than necessary, along with loss of available processing capability. At the very least, it would be a good idea to allow tuning the "options.intra_op_num_threads" value from an environment variable, to better match hardware capability with processing load.

Comma two & three use Qualcomm's SNPE, so onnx is only relevant for PC users.
@mitchellgoffpc You have been spending some time on the ONNX stuff. Can you look into this?

Good to hear that it's only on the PC. I'm continuing to explore possible fixes.

An additional observation.  When I tried rebuilding onnxruntime with allow_spinning set to false, it didn't reduce CPU load. Either I did it wrong (entirely possible) or there is an issue other than the designed-in spin waits.

i tried "pip install onnxruntime-openmp", and put "export OMP_WAIT_POLICY=PASSIVE" into the environment. This results in approximately 12-20% CPU usage. This may be a reasonable solution for many users, but I favor the CUDA-based setup.  Even while acknowledging that onnxruntime GPU  support is a fragile, implementation-dependent thing- I had to build from source. "All of the above" might be the right approach in the openpilot code- CPU,-only, CPU with OMP, CUDA, OpenVino, or others, depending on the system setup. Opinions are welcome!


Trivy: update node version in Docker

Describe what problem your feature request solves
The node version is reported as 16.4.2 in our docker image/container:
/ # node --version
v16.4.2

Describe the solution you'd like
threat-dragon$ node --version
v18.2.0

Additional context
Trivy is complaining about the node version
Note that when the node version is updated we may be able to remove an ignore from .trivyignore:
# Client-side only vulnerability according to redhat:
#    https://access.redhat.com/security/cve/cve-2021-3807
# This is also baked into the node:16-alpine image at this time (inside of nodejs itself)
# We will need to wait for node to update
# As a temporary mitigating factor, I've added a custom resolution for all projects
CVE-2021-3807

This has been done in 9e78fcf62f196b6007bc897e33f423128c58377e, so this issue can now be closed


Avoid passing "entities" around in persistence operations.

For example, rather than RemoveQueueMessage having a QueueMessage, it really only needs the message ID and the revision.
Using the "entity" values inside these operations necessitates keeping these values in memory, when there might be better in-memory representations to use in between persistence operations.
This is also in-line with the idea of keeping the persistence package as bare as possible.

I tried this out quickly, and while there are some benefits, it tended to increase the complexity of most things, closing for now.


issue-3455 - adding resource info to transaction bundle audit message

I would have liked to "squash and merge" this, but prasanna has other changes on top of these ones, so I'm keeping all the commits


updateRemoteUserUID has no effect

When using this CLI, the container or remote user's UID and GID are not updated. This issue is documented in https://github.com/microsoft/vscode-remote-release/issues/10030, but I think the core issue might be located here. I have an example repository up on GitHub. Steps to reproduce are below. Am I missing something in my configuration, or is this a bug?
git clone https://github.com/cadojo/devcontainer-uid-example
id
cd devcontainer-uid-example
devcontainer up --workspace-folder . --remove-existing-container --build-no-cache
devcontainer exec --workspace-folder . id

I see the following text on containers.dev... does this mean that on MacOS, or Windows, there is no way to sync the container user's UID and GID with the local user's?

On Linux, if containerUser or remoteUser is specified, the user’s UID/GID will be updated to match the local user’s UID/GID to avoid permission problems with bind mounts. Defaults to true.

This only applies on Linux. Windows doesn't have Unix-like UIDs/GIDs. On macOS you would get the UID/GID mapped by Docker Desktop, maybe you are using something else?


TASK: Allow administrators to remove last second factor of users

Very nice! But for readability i think it would be better to rewrite the conditions to sth. like:
// Check, if user is allowed to remove second factor first
if ($isAdministrator || ($isOwner && $this->secondFactorService->canOneSecondFactorBeDeletedForAccount($account))) {
    // User is admin ore more than one second factor exists.
    // Remove second factor and show flash message.
    return;
}

// Next, check if the user is the owner of the second factor
if ($isOwner) {
    // As it is the owner, the second factor to be remove must be the last one, so we can't remove it.
    // Show error flash message.
    return;
}

// All others (neither administrator nor owner) are redirected to index
$this->redirect('index');


Group replication seeds & Group replication credentials

Hi,
Love this POC, and it helped me a lot in getting group replication working on Kubernetes.
I have one tip which works for me. By adding the replication port to the service you can populate loose-group_replication_group_seeds with the service name, instead of adding instances to the setting. This prevents DNS look ups for non existing instances and all the instances can than be seeds for the group replication without manually adding them.
The other thing is, I ran into this with Docker and now also with Kubernetes. The Group replication credentials don't have to be added to the extra instances, however all the tutorials and documentation on MySQL say that you have to. Looking at your scripts you also don't create the replication user, where did you get that info? Should we request changing the documentation on MySQL ? Put this question also on StackOverflow https://stackoverflow.com/questions/48620247/group-replication-recovery-credentials-already-exists

Hi,
I'm glad that I could help :).
Setting loose-group_replication_group_seeds to service name, didn't work for me. First pod is starting ok, but second one failed with:
2018-02-21T18:03:21.176251Z 0 [Note] Plugin group_replication reported: 'connecting to mysql-rw.mysql-test 24901'
2018-02-21T18:03:21.177824Z 0 [Note] Plugin group_replication reported: 'client connected to mysql-rw.mysql-test 24901 fd 61'
2018-02-21T18:03:21.178272Z 0 [Note] Plugin group_replication reported: 'cli_err 2'
2018-02-21T18:03:22.178642Z 0 [Note] Plugin group_replication reported: 'cli_err 2'

Anyway, isn't service load-balanced between pods? So instead of hitting mysql-1:24901 request can by sent to mysql-0:24901? Can you share some examples?
About credentials. If you are coping /var/lib/mysql/ from one server to another, you don't need to create replication user. It's already there within mysql database. AFAIK MySQL documentation refer to situation where you have 3 or more uncofigured, standalone servers and you don't copy mysql database between them.

Isn't that error because of the report_host setting missing ? I set the report_host to the mysql server IP ("report_host=$ip") address and the (loose-)group_replication_local_address also to the IP address ($ip:24901") instead of DNS name.
The service does load balance between pods but only replicates (data) with the replication seeds you put in loose-group_replication_group_seeds. So by using the service also for replication seeding the scaling is more balanced.
On the credentials, I connect freshly installed empty servers, no data at all. After connecting it replicates the rpm_user ?!?!?! I see it in my local docker environment and in my Kubernetes cluster. This is getting weird ....


COMException Class not registered when initializing CollectionViewSource

Describe the bug
The following exception occurs when initializing a CollectionViewSource instance from a MsTest execution:
System.Reflection.TargetInvocationException
HResult=0x80131604
Message=Exception has been thrown by the target of an invocation.
Source=System.Private.CoreLib
StackTrace:
at System.RuntimeTypeHandle.CreateInstance(RuntimeType type, Boolean publicOnly, Boolean wrapExceptions, Boolean& canBeCached, RuntimeMethodHandleInternal& ctor, Boolean& hasNoDefaultCtor)
at System.RuntimeType.CreateInstanceDefaultCtorSlow(Boolean publicOnly, Boolean wrapExceptions, Boolean fillCache)
at System.RuntimeType.CreateInstanceDefaultCtor(Boolean publicOnly, Boolean skipCheckThis, Boolean fillCache, Boolean wrapExceptions)
at System.Activator.CreateInstanceT
at WinRT.WeakLazy1.get_Value()    at WinRT.ActivationFactory1.ActivateInstanceI
at Microsoft.UI.Xaml.Data.CollectionViewSource..ctor()
at Example.ExampleVm..ctor() in C:\Users\Terry Cox\Documents\Projects\Tests and Spikes\Example\Example\Example\ExampleVm.cs:line 36
at ExampleTest.UnitTest1.TestMethod1() in C:\Users\Terry Cox\Documents\Projects\Tests and Spikes\Example\ExampleTest\UnitTest1.cs:line 12
This exception was originally thrown at this call stack:
[External Code]
Inner Exception 1:
COMException: Class not registered (0x80040154 (REGDB_E_CLASSNOTREG))
The CollectionViewSource constructor executes properly from the Win32 Windows Application project rather than from the MsTest. The evident difference is that the CollectionViewSource constructor is invoked via reflection from the test session.
Steps to reproduce the bug:
Extract the attached Example.zip file. It contains an application created from the C# Blank App, Packaged (WinUI 3 in Desktop) Visual Studio template for Project Reunion 0.8.0-preview. An MsTest project was added to the solution using the Unit Test Project (.NET Core) template. Both the desktop app and the MsTest project were modified as shown in the code. To demonstrate the problem, build the solution and:

Set ExampleTest as the Startup Project and run or debug all tests. The test will fail with the above exception.
Set Example (Package) as the Startup Project and run or debug the app. The application will execute without an exception.
Comment out line 36 in file ExampleVm.cs ]colorSource = new CollectionViewSource();]
Set ExampleTest as the Startup Project and run or debug all tests. The test will run without issue.

Version Info

NuGet package version:
Microsoft.ProjectReunion 0.8.0-preview
Microsoft.ProjectReunion.WinUI 0.8.0-preview
coverlet.collector 1.3.0
Microsoft.NET.Test.Sdk 16.7.1
MSTest.TestAdapter 2.1.1
MSTest.TestFramework 2.1.1

Edition	Windows 10 Pro
Version	21H1
Installed on	4/17/2021
OS build	19043.1023
Experience	Windows Feature Experience Pack 120.2212.2020.0

I think this might be expected.  I don't think the MSTest harness is packaged with Winrt types, so I think similiar to how you can't launch an unpackaged app (with winrt types) from visual studio, you can't use winrt types in your MSTest app.  @MikeHillberg is that right? What is the correct pattern here? something similar to Winui2's test app that the MSTest process interacts with using Mita?

I disagree with this assessment. This is not a UWP issue; it's a WinUI 3 issue, regardless of whether it's a UWP or Win32 app.  Also, CollectionViewSource is intended as a binding source; that's it's purpose. If it's not possible to run MsTest against classes using CollectionViewSource (i.e, ViewModels), then it's not possible to test such apps.

I only had a quick glance at your example, but are you following this blog post?
I did not see the EnablePreviewMsixTooling attribute in your ExampleTest.csproj file.

Marv51,
Thanks. I hadn't seen the blog post and will check it out.


Manage read-only file flag from brackets

Hi it would be nice to have a possibility to manage file "Read-only" flag from Brackets, or to automatically disable this flag when you save file.
Now it is really annoying to go to file explorer to remove that flag from the file.
Notepad++ have nice feature that you can right click on the file tab and click "Clear Read-Only Flag".

:+1:

Force writing a read-only as a superuser by entering the password would also be a nice feature, just like what Atom does.


Unable to add SoftwareRequirement

Hi,
I'm trying to add SoftwareRequirement to a CommandLineTool instance like so
tool.hints.extend([
    cwlgen.SoftwareRequirement().SoftwarePackage(
       "bcl2fastq", version="2.20.0", specs=["https://identifiers.org/rrid/RRID:SCR_015058"]),
])

which seems to work, but produces invalid CWL where the SoftwareRequirement key is missing.
How do you recommend adding SoftwarePackages to the CommandLineTool def?
Thanks
M

Or do you recommend going with cwl-utils instead of this project?
M

Hey @matthdsm, you can add it to the requirements.py file.
There’s one issue with CWL Utils  #27 before I see it as a viable replacement, but the intention is to deprecate this project as it supports different specs (where this is limited to CWL v1.0).

Hi @illusional
Thanks for the quick reply!
The fact is, it's already there (great work on your part), but I'm unable to call it, since SoftwareRequirement takes no arguments.
How should I call cwlgen.SoftwareRequirement, so I can add a SoftwarePackage to it?
Thanks again
M

Ah @matthdsm,  you're absolutely right! I've added a quick fix to #37 for solve this. You can install it with pip3 install git+https://github.com/common-workflow-language/python-cwlgen.git, and then I'll release it if that's it.
In the currently released version, you could do eg:
sreq = cwlgen.SoftwareRequirement()
sreq.packages = [cwlgen.SoftwareRequirement.SoftwarePackage(**kwargs)]

In the new version (in Git master) you can do:
sreq = cwlgen.SoftwareRequirement([cwlgen.SoftwareRequirement.SoftwarePackage(**kwargs)])

I can kick off a point release if this is all good.

Awesome! Thanks for the quick fix!
M

Actually fixed now.

Thanks for the fixes, but it seems we're not quite there yet.
using
# hints
bcl2fastq_tool.hints.extend([
    cwlgen.SoftwareRequirement([cwlgen.SoftwareRequirement.SoftwarePackage(
        "bcl2fastq",
        version=["2.20.0"],
        specs=["https://identifiers.org/rrid/RRID:SCR_015058"]
    )])
])

yields the following yaml
hints:
- packages:
  - package: bcl2fastq
    specs:
    - https://identifiers.org/rrid/RRID:SCR_015058
    version:
    - 2.20.0

which is missing the SoftwareRequirement key and is invalid according to cwltool
The following formatting registers as correct
hints:
  SoftwareRequirement:
    packages:
      bcl2fastq:
        specs: ["https://identifiers.org/rrid/RRID:SCR_015058"]
        version: ["2.20.0"]

It seems to me the SoftwareRequirement key needs to be added and the package key from SoftwarePackage needs to be dropped.
Cheers
M

Hi @matthdsm, there was some special handling for requirements that didn't happen for hints. Could you give the current master another look, I've added #39 to address this separate issue?

Magnificent! Everything work great!
Thanks a lot, and thanks for the quick responses!
Cheers
M


fix broken link

link under "Submit feedback to help shape Windows for your business - button: "Submit and track feedback", should be: https://docs.microsoft.com/en-us/windows-insider/at-work-pro/wip-4-biz-feedback, not https://docs.microsoft.com/en-us/windows-insider/at-work/wip-4-biz-feedback

Hi,
what is the typical turn around time for this?

@lizap - can you please assist to check, approve and merge the content for this PR?
Thank you.

@lizap - I would like to follow up on this. Can you please assist to check, approve and merge the content for this PR?
Thank you.

@kenwith - would you be able to merge the content of this PR for a broken link issue?
Thank you.

@mypil this one

@jessyrod68 - the status which shows "Merged" means that this Pull Request that you created has been resolved but it may take a few days for the merged content to appear in the article. Hope this helps to clarify things. Thank you.


September 2023 (version 1.83) fails to load extension

The new September release causes the extension to fail:

Reverting back to August the extension loads successfully.
@DumpySquare @mikeoleary

Just released 3.15. that should fix this.  Please confirm

@jmcalalang @mikeoleary ; have you had a chance to confirm the latest fix for this issue?

Can confirm, I'm upgraded and working.
tested fast/as3


style(codemirror): add highlight class

🧰 What's being changed?
Won't dim code if .CodeEditor also has .CodeEditor-Input_highlight (you can test this by manually adding the class)
🧪 Testing
Provide as much information as you can on how to test what you've done.

@runnabro @kanadgupta Published to 10.4.4


Difficult to make a responsive App Bar

I have an App Bar with several items: site title, search box, two buttons, and a Menu. On mobile, there's no space for the buttons, so I want to show those items in the Menu instead when the window width is small. I tried two ways of doing this, neither of which worked well:
Two Menus: one for narrow screens and one for wide.
I used CSS media queries to show and hide the Menus. But if a Menu is hidden when it is created, the click handlers don't register, so you can only click the one that was visible when the page first loaded.
One Menu: use CSS media query to show and hide menu items.
I used media queries on the Menu items themselves to show and hide them. Unfortunately, the menu doesn't recalculate its height when items are shown and hidden this way, so it keeps whatever dimensions it calculated when the page first loaded. Sometimes that leaves whitespace at the bottom of the menu, and sometimes it means the bottom two items are cut off.

Hi @alltom in future releases we will focus in the redesign of all components with a perfect fit in any kind of screen.
stay tunned ;)

:D

Closed in favour of #224


Adding Picture of WWWD Rig

Social Media
Discord: black_magic
Twitter: Nope
Rig Pics:
SGS5 w/ Wigle & Highway Radar
SGS20 w/ Wigle
Toshiba NB205 w/ Kali SD live boot
4x AWUS 036-ACM
1x AWUS 036-NHA w/ 7dbi
BU-353-S4 USB GPS
Cameo: My headless raspi wardriving rig which is still ever-changing.

Wigle: kmfdmk

Can you please try making a PR where you add things instead of deleting all the files?


update revive lint to latest commit

as title

Codecov Report

Merging #12921 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12921      +/-   ##
==========================================
- Coverage   43.09%   43.05%   -0.04%
==========================================
  Files         658      658
  Lines       72454    72454
==========================================
- Hits        31221    31195      -26
- Misses      36178    36211      +33
+ Partials     5055     5048       -7

Impacted Files
Coverage Δ

services/pull/check.go
31.53% <0.00%> (-16.16%)
:arrow_down:

modules/process/manager.go
72.50% <0.00%> (-2.50%)
:arrow_down:

modules/log/event.go
57.54% <0.00%> (-1.89%)
:arrow_down:

models/pull.go
54.54% <0.00%> (-0.63%)
:arrow_down:

models/gpg_key.go
53.90% <0.00%> (+0.57%)
: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 63e8bda...f2d8c7b. Read the comment docs.


Enforce privacy for leading underscore naming convention

Although the NameFormat linter has an option to allow for leading underscores, it does not enforce the convention. It would be really great to either have a linter that does this, or teach NameFormat to do this.
What I mean by enforcing privacy for the leading underscore convention is, for variables, functions, and mixins that start with a leading underscore, we should verify that:

any definitions are used within the same file
any uses are defined within the same file

And potentially:

any uses are preceded by their definitions

Bad
$_foo: red;

p {
  color: blue;
}

Bad
p {
  color: $_foo;
}

Bad
p {
  color: $_foo;
}

$_foo: red;

Good
$_foo: red;

p {
  color: $_foo;
}

And so on for functions and mixins.

In addition it would be nice to make the prefix configurable.


Remove all usage of from_url_and_kw. Make applets compatible with s3.

Note that with this PR QuartiCal main requires dask-ms master. This is temporary.


Add Dashboards var for UI Metric Collectors

Description
Add OpensearchDashboard variables corresponding to UI Metric Collectors introduced in the given PR.
Issues Resolved
https://github.com/opensearch-project/OpenSearch-Dashboards/pull/6203/
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thanks @peterzhuamazon . I have updated the PR with the suggested changes.


File change triggered every keystroke

I like this plugin, but I've started using it along with jest --watch, and as the plugin triggers file changes every keystroke, my tests just keep on reloading when I type.
This issue has been reported on SO. Unfortunately, I couldn't find any other way than disabling the plugin, which means that I'm not seeing live linting in the editor :(
It'd be cool if the plugin could use some other type of event which doesn't trigger these events. If someone has a solution, I'll take it :)
Anyways, thank you for this plugin, apart from this issue it does the job well

I too am being pained by this. I don't see any options to turn off auto-watch/run for the ESLint plugin. I'd like it only to run on explicit file save.

@harmon what I ended up doing is uninstalling the plugin and using the native eslint support. I didn't have any problems after that.

Hi @GabeAtWork,
Oh, cool. I'm not sure why I installed this plugin then. Maybe I didn't think it was natively supported in Webstorm and just went through the plugin install motions... this explains why I was very confused why there were two menus for ESLint in the settings panel! I removed the plugin as well and the native ESLint stuff seems to be working just fine. Thanks for pointing this out!

I ran into this problem as well. It was driving me nuts until I went to Webstorm support and they told me "third party ESLint plugins" can cause the problem. It's a shame this hasn't been resolved in over a year.


chore: bump kcl version to 0.10.8

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

chore: bump kcl version to 0.10.8

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

0 of 0   changed or added relevant lines in 0 files are covered.
13 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.04%) to 70.564%

Files with Coverage Reduction
New Missed Lines
%

kclvm/sema/src/resolver/mod.rs
1
99.1%

kclvm/tools/src/LSP/src/state.rs
12
81.14%

Totals

Change from base Build<PHONE_NUMBER>8:
-0.04%

Covered Lines:
51988

Relevant Lines:
73675

💛  - Coveralls


docs(snippets): const for gsi name, comment in put

Coverage remained the same at 94.843% when pulling 65b8f174dea5c1212dbeb50bb11bfd841d7ea0f8 on #184-finalize-snippets into 7e97c85a57b5c39ed7b0eebf8d9a89a42cb289ff on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add Interface::exists()

Adds an exists() method to interface types to allow users to check if a given interface name corresponds to an existing interface on a system.

MacOS utun implementation requires some special effort to destroy an interface--it doesn't accept SIOCIFDESTROY, but it doesn't delete itself by default when the file descriptor pointing to it is closed...

NOTE: MacOS has strange behavior for utun interfaces.
They don't conform to the usual SIOCIFDESTROY ioctl used generally to remove interfaces. Instead, a given utun interface will only disappear once all routes that make use of it have been deleted. This has caused issues elsewhere:
https://forums.developer.apple.com/forums/thread/682767
https://serverfault.com/questions/1129536/how-to-delete-utun0-in-my-macos
So it seems that manually deleting routes causes a utun device to go away. But, there is an alternative: SIOCDIFPHYADDR. I'm not 100% sure what it does under the hood, but the source code for MacOS's ifconfig util uses it for ifconfig deletetunnel. This may or may not cover the edge cases shown in the above links.
If further bugs arise, keep chasing this thread; for now, it works for the general case.


Bump kubectl version to 1.10.5.

Based on email thread of someone having issues w/ interacting w/ OKE and suggestion to update to latest kubectl.

I used 3.9.0 for step version to leave a few minor revisions in the step version in case there is ever a need to publish kubectl 1.8.x or 1.9.x.  If there is could use 3.7.x step version for the 1.8.x kubectl series and 3.8.x step version for the 1.9.x kubectl series.


JsonSerializerSettings can't be set

Hi,
i saved a time in String UTC Format in the Cosmos GraphDB.
As "yyyy-MM-ddTHH:mm:ssZ"
As i retrieved it, i though i would come back as string but it comes back as DateTime.
Is think this comes from the NewtonJsoft SerializerSettings in the Gremlin.net
Is there any way to change this default Settings to my behavior?
I want to do something like this:
	    JsonConvert.DefaultSettings = () => new JsonSerializerSettings
	    {
	        DateParseHandling = DateParseHandling.None,
	        DateFormatString = "yyyy-MM-ddTHH:mm:ssZ"
	    };

Thanks!

I just verified your problem with a simple unit test:
[Theory, MemberData(nameof(Versions))]
public void DeserializeDateTimeString(int version)
{
    var dateTime = DateTime.MinValue;
    var dateTimeString = dateTime.ToString("yyyy-MM-ddTHH:mm:ssZ");
    var graphSon = $"\"{dateTimeString}\"";
    var reader = CreateStandardGraphSONReader(version);

    var readValue = reader.ToObject(JToken.Parse(graphSon));

    Assert.Equal(dateTime, readValue);          // passes, but shouldn't
    Assert.Equal(dateTimeString, readValue);    // should pass, but fails due to a type mismatch
}

(CreateStandardGraphSONReader simply creates a GraphSON reader for version 2 or 3, so both can be tested.)
So, this is definitely something currently not possible with Gremlin.Net. I expected your approach to work to be honest as JsonConvert.DefaultSettings is a static property and should therefore also work for the deserialization of Gremlin.Net. I don't see why it doesn't work and I also don't know of another way to solve your problem directly.
Could you please create a ticket in the TinkerPop issue tracker? Gremlin.Net is now part of Apache TinkerPop and therefore I don't really maintain this repository any more.


test: add npm caching to pnpm examples

If cypress-io/github-action finds a pnpm-lock.yaml file, it uses the pnpm command pnpm install --frozen-lockfile by default to install dependencies. It does not however cache these dependencies for a future run of the same workflow.
This PR is a workaround to the pnpm example to make up for this missing functionality.
It adds dependency caching of the pnpm store contents using the command pnpm store path to find the location of the store and actions/cache to cache the store contents. This is applied to

the workflow example-basic-pnpm.yml and to
the README > pnpm section.

There is no change to the action itself.

A separate enhancement request is logged under https://github.com/cypress-io/github-action/issues/1044.

Verification
Run the workflow example-basic-pnpm.yml.
Check that each of the caches under github-action/actions/caches

macOS-pnpm-store-*
Linux-pnpm-store-*
Windows-pnpm-store-*

is approximately 6MB large.
Run the workflow example-basic-pnpm.yml again.
Check the logs files and examine each job.

basic-pnpm-ubuntu-20
basic-pnpm-ubuntu-22
basic-pnpm-on-windows
basic-pnpm-on-mac
basic-pnpm-binary

Under the step "Cypress tests" look for the line

Progress: resolved 179, reused 179, downloaded 0, added 179, done

The number next to downloaded must be 0.

@davidr-cy
Are you intending to review this PR?

@MikeMcC399 Sorry about that. I think we tagged the wrong person to review this!
Thanks for the ping, I'll get that sorted out.


GCP SM does not support external_account type

Describe the solution you'd like
GCP SM does not support external_account type which is required to support workload identity on baremetal Anthos cluster.
What is the added value?
required to support workload identity on baremetal Anthos cluster.
Give us examples of the outcome
In the pkg/provider/gcp/secretmanager/auth.go file changing the snippet in the function serviceAccountTokenSource
config, err := google.JWTConfigFromJSON(credentials, CloudPlatformRole)
if err != nil {
return nil, fmt.Errorf(errUnableProcessJSONCredentials, err)
}
to
Use google. tokenSource will add support to multiple such scenarios

Hi I've also been trying to use workload identity with Anthos, but on VMWare.  I created a SecretStore as per the guide, using 'global' for the cluster location.  I also tried setting serviceAccountRef.audiences (based if some info from here.  The error I am seeing is failed to create GCP secretmanager client: unable to fetch identitybindingtoken: could not get idbindtoken token, status: 404.

FWIW I managed to get something to work with our Anthos on VMWare clusters by using the pod identity method on the ESO deployment.  Unfortunately this restricts usage to ClusterSecretStore which is not ideal for finely scoping access to specific secrets from specific namespaces.
Installing with Helm I configured the following values.yaml
extraEnv:
  - name: GOOGLE_APPLICATION_CREDENTIALS
    value: /var/run/secrets/tokens/gcp-ksa/google-application-credentials.json

extraVolumes:
  - name: gcp-ksa
    projected:
      defaultMode: 420
      sources:
      - serviceAccountToken:
          path: token
          audience: <WORKLOAD_IDENTITY_POOL>
          expirationSeconds: 172800
      - configMap:
          name: google-application-credentials
          optional: false
          items:
            - key: "google-application-credentials.json"
              path: "google-application-credentials.json"

extraVolumeMounts:
  - name: gcp-ksa
    mountPath: /var/run/secrets/tokens/gcp-ksa
    readOnly: true

serviceAccount:
  annotations:
    iam.gke.io/gcp-service-account: <GCP_SERVICE_ACCOUNT_EMAIL_ADDRESS>
  automount: true

And then had to apply the google-application-credentials configmap as a separate step as couldn't find a way to deploy it through the Helm chart.  Looks like:
kind: ConfigMap
apiVersion: v1
metadata:
  namespace: external-secrets
  name: google-application-credentials
data:
  google-application-credentials.json: |
    {
      "type": "external_account",
      "audience": "identitynamespace:WORKLOAD_IDENTITY_POOL:IDENTITY_PROVIDER",
      "service_account_impersonation_url": "https://iamcredentials.googleapis.com/v1/projects/-/serviceAccounts/GSA_NAME@GSA_PROJECT_ID.iam.gserviceaccount.com:generateAccessToken",
      "subject_token_type": "urn:ietf:params:oauth:token-type:jwt",
      "token_url": "https://sts.googleapis.com/v1/token",
      "credential_source": {
        "file": "/var/run/secrets/tokens/gcp-ksa/token"
      }
    }

See https://cloud.google.com/anthos/fleet-management/docs/use-workload-identity for info on how to obtain audience value.

Yes it is working