Problem with pip install

Orion version: 0.1.0
Python version: 3.11, and 3.10
Operating System: Windows & Linux(Google Colab)

Description
Error locally and on Google Colab when running the command:
pip install orion-ml
Colab Error:
Collecting orion-ml
  Downloading orion_ml-0.1.0-py2.py3-none-any.whl (38 kB)
Collecting baytune<0.3,>=0.2.3 (from orion-ml)
  Downloading baytune-0.2.5-py2.py3-none-any.whl (28 kB)
Collecting mlblocks<0.4,>=0.3.0 (from orion-ml)
  Downloading mlblocks-0.3.4-py2.py3-none-any.whl (26 kB)
Collecting mlprimitives<0.3,>=0.2.2 (from orion-ml)
  Downloading mlprimitives-0.2.4-py2.py3-none-any.whl (179 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 179.7/179.7 kB 6.7 MB/s eta 0:00:00
Collecting mongoengine<0.17,>=0.16.3 (from orion-ml)
  Downloading mongoengine-0.16.3.tar.gz (149 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 149.0/149.0 kB 5.3 MB/s eta 0:00:00
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> See above for output.

  note: This error originates from a subprocess, and is likely not a problem with pip.
  Preparing metadata (setup.py) ... error
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

I also tried running this locally,
Python 3.10
Windows 11
This is the error I get when running locally:
Collecting baytune<0.3,>=0.2.3 (from orion-ml)
  Using cached baytune-0.2.5-py2.py3-none-any.whl (28 kB)
Collecting mlblocks<0.4,>=0.3.0 (from orion-ml)
  Using cached mlblocks-0.3.4-py2.py3-none-any.whl (26 kB)
Collecting mlprimitives<0.3,>=0.2.2 (from orion-ml)
  Using cached mlprimitives-0.2.4-py2.py3-none-any.whl (179 kB)
Collecting mongoengine<0.17,>=0.16.3 (from orion-ml)
  Using cached mongoengine-0.16.3.tar.gz (149 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [1 lines of output]
      error in mongoengine setup command: use_2to3 is invalid.
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

I tried installing a version of mongoengine > 0.2 and then installing orion-ml with the --no-dependencies flag, but after I do that if I try to import orion, I get the following error:
ModuleNotFoundError: No module named 'orion'

Hi @gh-wf, thank you for using Orion!
Can you try downgrading to python 3.8? Currently, I am still patching up the library to work with python 3.9+ and figuring out issues (it should be released soon!).

Hi @gh-wf, I just made a new release and you should be able to install Orion 0.6.0 on python 3.10 and 3.11!


Example not Working

In the example below, using a fresh install (no DataFramesMeta package), I had to install DataFramesMeta and then add:
import DataFramesMeta.Chain
using Tidier
using RDatasets
movies = dataset("ggplot2", "movies");
@chain movies begin
@mutate(Budget = Budget / 1_000_000)
@filter(Budget >= mean(skipmissing(Budget)))
@select(Title, Budget)
@slice(1:5)
end

Tidier doesn’t require DataFramesMeta to be installed so I’m not sure why that would be the case. Additionally, Tidier re-exports the @chain macro directly from Chain.jl so you shouldn’t need to grab the macro using DataFramesMeta.
I notice that the M in @Mutate is capitalized. Is that the cause of the error?
Can you share what error you are getting?

julia> movies = dataset("ggplot2", "movies");
julia>
julia> @chain movies begin
@mutate(Budget = Budget / 1_000_000)
@filter(Budget >= mean(skipmissing(Budget)))
@select(Title, Budget)
@slice(1:5)
end
ERROR: LoadError: UndefVarError: @chain not defined
in expression starting at REPL[5]:1

After restarting Julia it works

Glad to hear! Will close this issue.


Add Windows theme

A windows theme can be added using https://pub.dev/packages/fluent_ui and a if statement on builder using GetPlatform.isWindows.

I think one single theme for everyone is more coerent, if someone still wants, feel free todo.


Improve usability of constants

Hey,
This is a update on @abrown 's PR https://github.com/bytecodealliance/wasmtime/pull/2468.
The main changes are:

The introduction of i128's into DataValue
The Trait based ConstantGenerator and PooledConstantGenerator

This was also discussed as potential fix to https://github.com/bytecodealliance/wasmtime/issues/2906. But I haven't checked if it fixes it yet.

Maybe you should add "closes #2468" to the PR description?


Benchmark on different cpu architectures

https://github.com/Boshen/oxc/pull/23#discussion_r1116336138

I have found that to_bitmask on ARM NEON can be cause parsing to be slower compared to a scalar version. My block comment parser only uses SIMD on x86, not ARM.

On x86_64, to_bitmask is one instruction. On AArch64, there is no instruction and to_bitmask needs to be emulated. See: https://godbolt.org/z/531oEo5d1 It looks like the Rust compiler currently does a terrible job on AArch64. You can definitely do better by hand, but it still might not be faster than the scalar version.

I think I'm satisfied with the linux / mac / windows benchmarks for now. Figuring out all the cpu architectures is beyond my knowledge at the moment. I may come back to this if I ever find the time.


fix: Ensure bindings are set once presenter has owner set

closes #900

@Mergifyio backport release/stable/5.0

@Mergifyio backport release/stable/4.2


Improve the architecture of SpiFuture

After trying to use SpiFuture in the real world, I encountered some
problems that these changes address. Specifically, I added the ability
for the Spi struct to be part of some user type. The AsSpi trait
provides an interface for the SpiFuture. This has the added benefit of
simplifying the implementation.
Summary
[describe your changes here]
Checklist

[x] CHANGELOG.md for the BSP or HAL updated
[x] All new or modified code is well documented, especially public items
[x] No new warnings or clippy suggestions have been introduced (see CI or check locally)

@bradleyharden, I quite like this. I think it definitely makes SpiFuture more flexible.

I still need to backport some other bug fixes. That's why I it's still a WIP


End user documentation for openwiki usage

Create very simple end user documentation for how to use:

[x] includes
[x] links
[x] macros
[x] wikis

with plenty of examples aimed at non-developers
child of threefoldtech/home#162

Send docs to @VonSub when done for clarity verification

Estimation: 1 working day

Will skip incomplete macros like gslides (https://github.com/threefoldtech/digitalmeX/issues/53 is not done yet).

Fixed dot macro (https://github.com/threefoldtech/jumpscaleX/commit/330db95cedcc29002a4a8102da95d0585247eca2)
Will try to clean developer docs more and add more examples, also, fixing some dead links (see https://github.com/threefoldtech/digitalmeX/issues/42#issuecomment-481182813 and https://github.com/threefoldtech/digitalmeX/issues/42#issuecomment-481207503)

Done https://github.com/threefoldtech/digitalmeX/tree/development/docs/publishing_platform/wiki
Sent to @VonSub to review.

@rkhamis A question regarding how to use wikis, what is meant by this exactly
What i understand, if it's for end-users, he should be editing the wikis, not using technical tools or deploy his own wiki? right?

@abom perhaps it's best to have the usage of wiki usage documentation link to a more in-depth, yet step by step, documentation on

how to deploy own wiki
how to use more advanced tooling

Done, added a link to developer docs, it contains tool usage and reference to lapis-wiki.

Looks good to me.


fix: scan private registries

fixes #67
Full context:
Currently, the private registry docker images cannot be scanned, even when providing credentials, even when using GitHub PAT.
With the changes in this PR, I was once again able to scan images normally without issue.
P.S. Please install dependabot or renovatebot. I'll be happy to contribute on that side too if I get a confirmation from the maintainers.

ping @craigbox @avinashupadhya99

@meysam81 thanks for the suggestion, I'd like to have a renovabot action to maintain the image tag up-to-date please!

Well @matthyx , I love renovatebot and I fully support your decision.
But, please, whatever you end up executing, we want this yesterday :)
The official kubescape-action is not working for private images :(

does https://github.com/kubescape/github-action/pull/70 solve your issue?

@matthyx
no, it does not
this line is failing the CI:
https://github.com/kubescape/github-action/blob/c9749b84d138c0cbbb702b258774954f5463b82e/entrypoint.sh#L149


feat: catalog (DO NOT REVIEW)

Description

addresses continuous test failure: https://github.ibm.com/GoldenEye/issues/issues/8129
addresses appid permission issue: https://github.ibm.com/GoldenEye/issues/issues/8118
creates FSCloud profile, example and tests.
refactors code so that root module supports creation of CD and users.
remove advanced example
onboard catalog

Release required?

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

@rajatagarwal-ibm I think this can be closed right?


Missing file section 3.1

Cannot find the example file of section 3.1, the .csv file that should contain the name of ROIs.

Yes, this file is missing. Is there a preferred format for this file? If regions can have commas in their name, perhaps tab-separated file would be preferable? Or would we like to read in a file defining (new) regions by polygonal coordinates?

I think the second option would be preferable, as I normally attribute a class to my ROIs and then export them from there.

Perhaps the default behavior could be:
No input: export ALL existing ROIs as in the SCiLS Lab file
Input a list of names of ROIs: export ONLY those ROIs that are in the list
Input a list of names and polygons: generate new/overwrite existing ROIs and export those.
Do you have an example of a CSV file with exported regions?

Agree.
Here is an example:
EpithelialROIs_test.csv

This ROI seems to be a list of pixel coordinates, whereas SCiLS Lab also defines ROIs as polygons, where the coordinates are the vertices. We need to think about if and how to support both of these. For now, averageRegions averages and outputs all regions already defined in the SCiLS Lab file, but ignores the first input.


Hera API

Heras GraphQL API serving data to Heras website.

removed from board because it isn't part of the current milestone. rather the current milestone are things to be done before api, website and huge additions to hera should be done


Collapse all should cancel creating New file

VSCode Version: 1.31.1
OS Version: MacOS Mojave 10.14.3 (18D109)

Steps to Reproduce:

Start creating new file in some directory
Press Collapse All button in the Explorer
The project tree is now grayish until opening the directory with the New file name box.
It is even possible to have several New files in separate directories.

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

Fixed in latest vscode insiders


Allowing user to set CommandKey and ThreadPool

Currently in tenacity command , the property key given by user is used as same for command key and threadpool key. This does not allow the user the freedom of executing various command on same threadpool

When the project first started we tried to avoid mixing and matching different threadpools with different keys and vice versa because we found it became pretty hard to reason about past behavior if these threadpools and which keys ran in them change (observing metrics, etc.). A little context :-)
The library forces you at the moment to adhere to that, but I don't see any reason why we shouldn't add the ability to make it so you can choose the propertykey you want to use for the command key and threadpool key. I'll get this in the next release.

Should be available in 0.8.1!


XACRO information about TB not used by Gazebo

I've been trying to analyze deviation in ranges that I read from /scan topic while Turtlebot3 (burger) is stationary, but then I realised I should just look in models and this is what I found in turtlebot3_burger.gazebo.xacro (omitting some detail)
<gazebo>     <plugin name="imu_plugin" filename="libgazebo_ros_imu.so">       ...       <gaussianNoise>0.0</gaussianNoise>       <imu>         <noise>           <type>gaussian</type>           <rate>             <mean>0.0</mean>             <stddev>2e-4</stddev>             <bias_mean>0.0000075</bias_mean>             <bias_stddev>0.0000008</bias_stddev>           </rate>         </noise>       </imu>     </plugin>   </gazebo>
and then in another place
<gazebo reference="base_scan">     <sensor type="ray" name="lds_lfcd_sensor">       <ray>         <scan>...</scan>         <range>... </range>         <noise>           <type>gaussian</type>           <mean>0.0</mean>           <stddev>0.01</stddev>         </noise> </ray>
So I'm a bit confused about which Gaussian noise you are modeling (which mean/std) and why these particular values? I want to later on move on from simulation to using the actual TB3 and I would like to use realistic values.

Hi :)
Your right.  is not corresponds to real TB3. I will modify it or if you want, you can change it and PR to develop branch.
But i still wonder how to check 'wheelSeparation' value in Gazebo. Please tell me where you get a information of it.
Best regards
Darby

This issue will be closed since there were no actions for a while.
You can reopen this issue to show this issue to the users whenever.
Thanks. :)


not_landed: avoid to nag when there are open dependencies (depends_on)

Coverage decreased (-0.09%) to 30.651% when pulling 822fa3b88009ab6ce95b31ad65cd359334aa2341 on calixteman:fix_not_landed into 9c22356473de4b3e7b982805182a2bce28272860 on mozilla:master.


Add Death indicator fix

Fix Issue #3676 where the death indicator plugin was clearing the Bounty Hunter hint arrow. This fix is implemented by changing the Death hint arrow to instead be a tile overlay that marks the players death location as recommended by Adam.

I think that a config option for this is a little overkill. The arrow can conflict with content so I would just do the tile overlay.

I agree @Adam- , I went ahead and fixed this PR so that the plugin will now just show a tile overlay where the player died, instead of complicating things with a hint arrow that can conflict with other content. Let me know if you'd like to see any other changes! 👍

I think I am done making changes to this PR now. I went ahead and did the following: Fixed the hint arrow conflicting with Bounty Hunter arrow by changing the hint arrow to instead be a tile highlight (as Adam requested), updated the plugin to now use the OverlayManager, fixed issue #3743 by checking if the current time is after the saved death time plus 3600 seconds (1 hour), fixed an unreported issue I found where the markers were showing for any account that I played on locally (not just the account that died) by saving the player who died's name and verifying on login that the current player is the player that is recorded as dead, and added yellow text to the tile highlight overlay and minimap that says "Grave" (like konduit does, iirc). I extensively retested all previous scenario's plus the new ones after the additions and could not find any other issues.
Current Death Markers

Just let me know if anything needs to be changed. Thanks.

I would still keep hint arrow, but make it disabled by default, but keep it for people that want it even if it interferes with existing stuff.

This PR is really old, outdated, and all around pretty bad looking back at what I'd done here. I'm just going to close it. I'll consider revisiting the hint arrow as Deathbeam mentioned above, I like that idea.


DR workflow fixes

Context
A couple of fixes for the DR workflows that were just added
Changes proposed in this pull request
Due to some non-standard settings in apply

change AZURE_RESOURCE_PREFIX to RESOURCE_NAME_PREFIX
change DB server name suffix from -pg to -psql

Guidance to review
PTR restore: https://github.com/DFE-Digital/apply-for-teacher-training/actions/runs/11686160618
PTR backup: https://github.com/DFE-Digital/apply-for-teacher-training/actions/runs/11686644424
Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated added to the Azure KeyVault
[ ] Inform data insights team due to database changes
[ ] Make sure all information from the Trello card is in here
[ ] Rebased main
[ ] Cleaned commit history
[ ] Tested by running locally
[ ] Add PR link to Trello card

 add DR workflows to aftt


[Documentation] experimental etcd snapshot/restore procedure

K3s v1.19.1+k3s1 release will include etcd snapshot and restore. Similar to embedded etcd, this will be experimental in this release. We will plan in the next patch release or two to fully support both of these new features.
The etcd snapshot and restore PR does a good job outlining the new flags this feature adds. It talks about how it can be leveraged. @rancher-max has been testing this and may be able to provide additional context as well if he is needed.

The correct list of potentially relevant flags are:

   --datastore-endpoint value                 (db) Specify etcd, Mysql, Postgres, or Sqlite (default) data source name [$K3S_DATASTORE_ENDPOINT]
   --datastore-cafile value                   (db) TLS Certificate Authority file used to secure datastore backend communication [$K3S_DATASTORE_CAFILE]
   --datastore-certfile value                 (db) TLS certification file used to secure datastore backend communication [$K3S_DATASTORE_CERTFILE]
   --datastore-keyfile value                  (db) TLS key file used to secure datastore backend communication [$K3S_DATASTORE_KEYFILE]
   --etcd-disable-snapshots                   (db) Disable automatic etcd snapshots
   --etcd-snapshot-schedule-cron value        (db) Snapshot interval time in cron spec. eg. every 5 hours '* */5 * * *' (default: "0 */12 * * *")
   --etcd-snapshot-retention value            (db) Number of snapshots to retain (default: 5)
   --etcd-snapshot-dir value                  (db) Directory to save db snapshots. (Default location: ${data-dir}/db/snapshots)

   --cluster-init                             (experimental/cluster) Initialize a new cluster [$K3S_CLUSTER_INIT]
   --cluster-reset                            (experimental/cluster) Forget all peers and become sole member of a new cluster [$K3S_CLUSTER_RESET]
   --cluster-reset-restore-path value         (db) Path to snapshot file to be restored

Bringing up a cluster using --cluster-init defaults to using embedded etcd, so it is not required to pass --datastore-endpoint etcd.
I recommend using the /etc/rancher/k3s/config.yaml when setting these configurations, especially if wanting to pass a different cron value. I have been unable to successfully pass that flag in the arguments to INSTALL_K3S_EXEC due to improper escape sequences with asterisks. Even when I thought I escaped it properly, I didn't see it using the cron value I chose. There are probably plenty of users better with bash or using a different OS (I used Ubuntu 20.04) where this will work, but it is much easier using config.yaml
Using --cluster-reset is essentially a two step process:

Restart the cluster using the cluster-reset flag (and optionally the restore-path to restore from a snapshot)
This should fail to start and give a message in the logs saying it can be restarted without that flag now, so remove both of the flags and start k3s again. It should run successfully (and be restored from the snapshot specified, if any).

Closing this issue because https://github.com/rancher/docs/pull/2684 was merged


Autoupdate?

Perhaps it would be nice to have automatic updates. are some examples in userscripts.org

The autoupdate feature is already supported by Greasemonkey.
Anyway, it will be nice to have this feature for Google Chrome.
Can you send me a link to a good autoupdate example?

No tengo idea si sera bueno o no:
http://userscripts.org/scripts/show/2296


How to do authenticated schema introspection requests?

Hi there! Love this module, exactly what I needed to build a headless WordPress GraphQL website.
One thing though: My /graphql endpoint is only allowing authenticated schema introspection requests (which is recommended anyway). So my config is:
export default defineNuxtConfig({
  // ...
  graphqlMiddleware: {
    graphqlEndpoint: process.env.NUXT_GQL_HOST,
    serverFetchOptions() {
      return {
        headers: {
          Authorization: process.env.NUXT_GQL_TOKEN,
        },
      };
    },
    codegenConfig: {
      schema: {
        [process.env.NUXT_GQL_HOST as string]: {
          headers: {
            Authorization: process.env.NUXT_GQL_TOKEN,
          },
        },
      },
    },
  },
});
´

When executing this though, the CLI errors:
Failed to load schema from https://mywpurl.com/graphql:
    GraphQL introspection is not allowed, but the query contained __schema or __type
        GraphQLError: GraphQL introspection is not allowed, but the query contained __schema or __type
    at createGraphQLError (file:///Users/fabian/Desktop/code/madebyfabian/node_modules/.pnpm/@graphql-tools+utils@9.2.1_graphql@16.6.0/node_modules/@graphql-tools/utils/esm/errors.js:6:12)
    at file:///Users/fabian/Desktop/code/madebyfabian/node_modules/.pnpm/@graphql-tools+wrap@9.3.5_graphql@16.6.0/node_modules/@graphql-tools/wrap/esm/introspect.js:10:71
    at Array.map (<anonymous>)
    at getSchemaFromIntrospection (file:///Users/fabian/Desktop/code/madebyfabian/node_modules/.pnpm/@graphql-tools+wrap@9.3.5_graphql@16.6.0/node_modules/@graphql-tools/wrap/esm/introspect.js:10:58)
    at file:///Users/fabian/Desktop/code/madebyfabian/node_modules/.pnpm/@graphql-tools+wrap@9.3.5_graphql@16.6.0/node_modules/@graphql-tools/wrap/esm/introspect.js:41:32
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async UrlLoader.load (file:///Users/fabian/Desktop/code/madebyfabian/node_modules/.pnpm/@graphql-tools+url-loader@7.17.11_graphql@16.6.0/node_modules/@graphql-tools/url-loader/esm/index.js:216:29)
    at async file:///Users/fabian/Desktop/code/madebyfabian/node_modules/.pnpm/@graphql-tools+load@7.8.12_graphql@16.6.0/node_modules/@graphql-tools/load/esm/load-typedefs/load-file.js:14:39
    at async Promise.all (index 5)
    at async loadFile (file:///Users/fabian/Desktop/code/madebyfabian/node_modules/.pnpm/@graphql-tools+load@7.8.12_graphql@16.6.0/node_modules/@graphql-tools/load/esm/load-typedefs/load-file.js:12:9)
    at async file:///Users/fabian/Desktop/code/madebyfabian/node_modules/.pnpm/@graphql-tools+load@7.8.12_graphql@16.6.0/node_modules/@graphql-tools/load/esm/load-typedefs/collect-sources.js:201:25

Tried to use codegen without the module to make sure it works, so the working codegen.ts for demo purposes would be:
// codegen.ts
import type { CodegenConfig } from "@graphql-codegen/cli";
import * as dotenv from "dotenv";
dotenv.config();

const config: CodegenConfig = {
  schema: {
    [process.env.NUXT_GQL_HOST as string]: {
      headers: {
        Authorization: process.env.NUXT_GQL_TOKEN,
      },
    },
  } as any,

  documents: [
    "./server/trpc/**/*.ts",
    "./server/api/v2/**/*.ts",
    "./api/**/*.ts",
    "!./graphql/**/*",
  ],

  generates: {
    "./graphql/": {
      preset: "client",
      plugins: [],
    },
  },
};

export default config;

Hey! I‘m glad you find this module useful!
The codegenConfig property contains the actual config, so without the [process.env.NUXT_GQL_HOST as string]:
export default defineNuxtConfig({
  graphqlMiddleware: {
    codegenConfig: {
      headers: {
        Authorization: process.env.NUXT_GQL_TOKEN
      },
    },
  }
})

I quickly looked through the code and it seems like that config is not passed to the underlying function call that downloads the schema, only for TS codegen. I will fix that sometime today. Because the config is different to the one used for TS codegen I‘ll introduce a new property graphqlMiddleware.schemaConfig that will be used for this codegen part.

Thanks so much!

Just a note: I think the TypeScript types on codegenConfig are wrong:

Type '{ headers: { Authorization: string; }; }' is not assignable to type 'TypeScriptDocumentsPluginConfig'.
  Object literal may only specify known properties, and 'headers' does not exist in type 'TypeScriptDocumentsPluginConfig'.

I also discovered that graphqlMiddleware.codegenConfig.documents doesn't work. My usecase is that I want to exclude some .gql files entirely, since the schema generation fails due to an upstream bug https://github.com/dotansimha/graphql-code-generator/issues/9010
So I want to use something like
{
  graphqlMiddleware: {
    graphqlEndpoint: process.env.NUXT_GQL_HOST,
    serverFetchOptions: {
      headers: {
        Authorization: process.env.NUXT_GQL_TOKEN || "",
      },
    },
    codegenConfig: {
      documents: [
        // Exclude due to infinite loop
        "!pages/_graphql/SinglePage.query.gql",
        "!pages/_graphql/SinglePost.query.gql",
      ],
      headers: {
        Authorization: process.env.NUXT_GQL_TOKEN || "",
      },
    } as any,
  },
};

also tried it with the documents and autoImportPatterns config, but regardless of which I set, the module is always trying to generate schema for those .gql files I don't want it to.

I released a new beta that adds support for schema-ast configuration<EMAIL_ADDRESS>For the authentication headers see this doc page here:
https://github.com/dulnan/nuxt-graphql-middleware/blob/module-server-options/docs/configuration/using-auth-headers.md
This beta also contains a breaking change I prepared two weeks ago where some configuration options that are needed during server runtime have been moved to a separate file that is inlined into the .output server build. Previously the module tried to load nuxt.config.ts at runtime, which failed if only the .output folder is deployed (which actually is the correct way).
If you use a method for graphqlEndpoint or the serverFetchOptions, onServerError or onServerResponse options, you'll need to move those to ~/app/graphqlMiddleware.serverOptions.ts. That was the only way to make this work. See this page for an explanation: https://github.com/dulnan/nuxt-graphql-middleware/blob/module-server-options/docs/configuration/server-options.md

@dulnan Sounds great, currently implementing it. I now get an infinite "Reloading Server" with no additional info. Did that happen to you too?

Does that mean that everything I load into ~/app/graphqlMiddleware.serverOptions.ts is available on the client bundle, making people able to inspect for my auth token for example?

I didn't get that inside the module's playground, but Nuxt's playground setup tends to sometimes work, but then installing the module in an actual Nuxt app produces errors. Have you tried removing .nuxt and trying again?
The serverOptions file is not part of the client bundle, only the server bundle. This workaround is needed because Nuxt doesn't offer a way for module authors to provide module users a way to pass in methods as configuration. Runtime config doesn't work for methods because it goes through JSON.stringify. The workaround is to take the serverOptions file during build and telling Nitro to inline it in the server bundle.
Of course if you accidentally import the file in a plugin or component (which is pretty hard to do accidentally :smile: ) you could probably get its contents in a client bundle. But then again, the token is likely in an ENV variable and is not resolved to a string during build, so you should be safe even in this case.

Hi, so I managed to remove that error, was some wierd other thing in my repo I did while upgrading.
Alright sounds great! Using the beta now and the header config for both generation and runtime query execution works. That plugin saves me so much code 😄 (mostly generated stuff to be fair):

By the way, the generated schema.graphql should be commited to git or not? Since it's generated on every build/dev start.


No longer works in Obsidian Desktop Version 1.5.12

It appears that this add on stopped working... do you mind fixing it?
This is a crucial part of my daily note. Thanks.

I'm currently using page-gallery 0.8.4 on Obsidian 1.5.12 on both Mac and Windows without any issues. Can you confirm what error messages you're seeing, if any, what version of obsidian-dataview you're using, and post a sample page-gallery code block that isn't working for you?

Thanks for looking into this. I think the issue was caused by my dataview not running correctly. I reinstalled dataview and everything works fine now. Will close this issue.


Initialize Setup

Initialize poetry and setup venv for our project

Closing issue, added Poetry and initialized the main file


Set JIRA Issue Type ID

Currently the JIRA integration only uses the issue type of 1. However that issue type does not exist on my JIRA Server. Is there a way to update the JIRA Service to support at the very least a field to set the Issue ID Type.

Hey @bassrock,
Thanks for reaching out on this. As you mentioned, we do need an issue type of 1 to create new issues in JIRA. Adding in support to customize this is something that we're looking into, but I don't have a timeline for this currently. I'll re-open this if the work to add this is taken on, but thanks again for raising this up!
-Mike

We are experiencing the same problem. With JIRA Cloud you cannot set your own ID, so it is impossible to connect. It would be very useful (and, probably, very simple) to add a field for the issue id type.

Thanks @AlessandroLacroce

@mikebonnell Tested out that pull request and looks like it'll work. Let me know if you're able to merge in the near future.

We're hoping to provide a resolution for this issue soon, will update this issue with status as we make progress.

Thanks.

On Jan 20, 2016, at 9:11 AM, Josh Ladieu<EMAIL_ADDRESS>wrote:
We're hoping to provide a resolution for this issue soon, will update this issue with status as we make progress.
—
Reply to this email directly or view it on GitHub https://github.com/crashlytics/crashlytics-services/issues/110#issuecomment-173277213.

A fix for this issue has been deployed.  Now Jira integrations will use issueType:name 'Bug' by default, but if you have problems with this, you can provide an override in the Jira setup UI on the Fabric dashboard.  Thanks for bringing this to our attention!

Just tried it out and works great. Thanks.


Bug: does not work on Win11

Login page is stuck loading forever, reload goes to about:blank
How do I do token?

What is your problem? Can I see the screen?

Wh is invald

it works fine on windows 11, these are not windows 11 specific issues.
for bugs, please open individual issues with actual detail (see the templates)
for support requests, please see the discord


[Bug] 资源使用问题

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
分支：data_quality_design
资源使用模块，设计到shell，python等控件
选择多个控件，前端只显示1个，但是数据库里面是有多个，如果下次编辑忘记选择，就会变成只有一个

What you expected to happen
前端Bug
How to reproduce
前端Bug
Anything else
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

@caiyt2001
For the convenience of communication, please use English


Add a concept of trusting a project for access to GPS data

To test you'll need to have a user that is the admin of a project and you'll need another user that is a member of that project and trusts the project.  The member will need an observation whose GPS info is hidden, but is associated with the project.  There is now a button on the show project page that allows members to trust a project.  Once this is setup, the admin user should be able to see the GPS info of the observation with the hidden GPS info.  Other users who are not either the owner of the observation or an admin for this project should not be able to see the GPS info.
Specific places where the GPS info is show include the show observation page, the label output, and reports.  The code for the reports is particularly tricky since it is trying to minimize the number of queries that get run.

coverage: 94.457% (+0.02%) from 94.441%
when pulling 32299884cbc31c3cc46704a9ca04b89a94c38f49 on njw-project-members
into 3020c43c4a6b2501e9da84f363f6b81b19100c45 on main.

Suggestion:
Either:

Make Revoke Trust retroactive, or
Add a warning or other text indicating that it applies only to future Observations.
(I expected it to be retroactive.)

Lat/Lon is missing from a spreadsheet downloaded by the Project Admin after Trust Revoked, but is present in printed Labels.
Here's what I did:

As me:

Created a project Administered by me (in Chrome).
Added user OMS to that project.

Logged in as OMS in a different browser (Firefox).

Trusted the Project.
Ceated an Observation with hidden coordinates.
Revoked Trust.

As me,

went to the Project,
clicked Observations,
Download Observations
Selected CSV spreadsheet, UTF-8, clicked Download
Result: spreadsheet without Lat/Lon
observations_1oQg6.csv
Clicked UTF-16, then UTF-8 (in order to un-grey the Print Labels button)
Clicked Print Labels
Result: RTF label that includes Lat/Lon

I also note that the label rounds lat and lon to 1 decimal place. That seems wrong. (I don't think it should be rounded.) But that's another story.

@JoeCohen I don't know what you mean by making "Revoke Trust" retroactive.  It's boolean.  Either it is on or it's not.  It defaults to false, but gets explicitly set to true for the owner of each project.  However, now that I think about it, I don't think the owner should trust the project for existing projects since I believe the current user automatically gets access to their own lat/long info.

The rounding behavior pre-dated this PR.  We could change it, but that makes me nervous since we're then changing behavior that was created a long time ago.

What I meant by "retroactive" was: does it apply to Observations that were
earlier added to the project.
In particular, I note that if a user:

Trusts a Project
adds an Obs with hidden coordinates to the Project
Revokes Trust

then the Project admin still sees the coords when viewing the Obs.
So it appeared to me that Revoke Trust is not "retroactive"
On Sat, Nov 11, 2023 at 7:11 PM Nathan Wilson @.***>
wrote:

@JoeCohen https://github.com/JoeCohen I don't know what you mean by
making "Revoke Trust" retroactive. It's boolean. Either it is on or it's
not. It defaults to false, but gets explicitly set to true for the owner of
each project. However, now that I think about it, I don't think the owner
should trust the project for existing projects since I believe the current
user automatically gets access to their own lat/long info.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1686#issuecomment-1806986329,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFGHCC7AOTLIY2X3MQTYEA44XAVCNFSM6AAAAAA67PI7ISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBWHE4DMMZSHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for the explanation. Makes sense to leave it as-is.
On Sat, Nov 11, 2023 at 7:13 PM Nathan Wilson @.***>
wrote:

The rounding behavior pre-dated this PR. We could change it, but that
makes me nervous since we're then changing behavior that was created a long
time ago.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1686#issuecomment-1806986639,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFBQGAO7RTUJ7A4DSVTYEA5EFAVCNFSM6AAAAAA67PI7ISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBWHE4DMNRTHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks Nimmo and Nathan.
I didn't understand that about .l and .t.
On Sat, Nov 11, 2023 at 7:14 PM Nathan Wilson @.***>
wrote:

@.**** commented on this pull request.
In app/controllers/projects/members_controller.rb
https://github.com/MushroomObserver/mushroom-observer/pull/1686#discussion_r1390322407
:

 if params[:commit] == :change_member_status_revoke_trust.l

   flash_notice(:change_member_status_revoke_trust_flash.l)

I used .l since the original code had .l. As Nimmo points out .t is only
needed in the case that there are embedded translation strings and in these
case they are kind of being used as constants so .l seemed reasonable to
me. Don't have a strong opinion.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1686#discussion_r1390322407,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFER6FGUSPVMWASGGF3YEA5KRAVCNFSM6AAAAAA67PI7ISVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTOMRWGE3DINJZGA
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

@JoeCohen I was not about to reproduce the case where after revoking trust the admin can see the coordinates.  What you're describing doesn't make sense to me since the trust info is not stored on a per observation basis.  Could it be that the admin user you are using is the owner of the observation?  If that's true, then yes, it will show the coordinates.

The Obs owner and Project user are different.
I'll dig into it in a while to try to figure out what's going on.

Project owner & membership
[image: Project members.png]
Member OMS's view of Project. Note that it not Trusted (at the moment).
[image: Member OMS view of Project.png]
Project owner's view of Observation. Note that coords are visible.
[image: Project Owner's view of Obs.png]
On Sun, Nov 12, 2023 at 12:11 PM Nathan Wilson @.***>
wrote:

@JoeCohen https://github.com/JoeCohen I was not about to reproduce the
case where after revoking trust the admin can see the coordinates. What
you're describing doesn't make sense to me since the trust info is not
stored on a per observation basis. Could it be that the admin user you are
using is the owner of the observation? If that's true, then yes, it will
show the coordinates.
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1686#issuecomment-1807226653,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFE2D66NQGUREAXIUELYEEUPRAVCNFSM6AAAAAA67PI7ISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMBXGIZDMNRVGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Re Project admin seeing hidden coordinates after user revokes trust.
It looks like Observation#can_edit? is returning the wrong thing, or at least the opposite of what I expect.
(Ditto for Observation#project_admin?)
See below. I will keep digging.

With the project admin (me, User 4468) logged in,
and a breakpoint in Observation#reveal_location?
then when viewing the Obs in question, I see
   701|   def reveal_location?
=> 702|     debugger
   703|     !gps_hidden || can_edit? || project_admin?
   704|   end
=>#0    Observation#reveal_location? at ~/mushroom-observer/app/models/observation.rb:702
  #1    block in observation_details_where_gps at ~/mushroom-observer/app/helpers/observations_helper.rb:207
  # and 169 frames (use `bt' command for all frames)
(ruby) User.current
#<User 4468: "Joseph D. Cohen (Joe Cohen)">
(rdbg) self
#<Observation:0x00000001159fa8b0
 id: 516708,
 ...
 user_id: 5485,
...
 lat: 0.458963e2,
 long: -0.918735e2,
 ...
 log_updated_at: Sat, 11 Nov 2023 15:38:27.000000000 PST -08:00>
(ruby) !gps_hidden
false
(ruby) can_edit?
true
(ruby) project_admin?
...
false

Some Progress.
I think:

You can replicate the bug by making the Project non-open-membership.
The underlying problem:

ObservationsHelper#observation_details_where_gps causes gps to be displayed if obs.reveal_location?
Observation#reveal_location? returns true if (among other things) <observation>.can_edit? is true:

  def reveal_location?
    !gps_hidden || can_edit? || project_admin?
  end

Observation#can_edit? calls Project#can_edit?, rather than seeing if the Obs itself is editable by the current user.

  def can_edit?(user = User.current)
    Project.can_edit?(self, user)
  end

That seems strange to me. And I don't think it's what's needed in this case; ObservationHelper just wants to know if the current user can edit the Obs.

I don't know what the fix is.  Would something like this work:

Delete the current definition of Observation#can_edit?.  (So <observation>.can_edit? will call the superclass.)
Change the calls to Observation#can_edit?-- except reveal_location? -- to directly call Project#can_edit?

That is correct behavior as far as I know.  Specifically, before there was the concept of open membership projects, all members and admins were considered to have edit permissions for anything that was part of that project.  The concept being that they had to be invited to be part of the project.  When open_membership is true, the ability for members and admins to edit is turned off.

Non-open membership projects essentially implicitly trust the project.  We could change that I suppose.  At minimum it seems like maybe we shouldn't allow such users to even mess with the trust level since it's kind of meaningless.

This might do a more intuitive thing:
% git diff
diff --git a/app/models/project.rb b/app/models/project.rb
index 28e145c44..86ce27944 100644
--- a/app/models/project.rb
+++ b/app/models/project.rb
@@ -159,6 +159,7 @@ class Project < AbstractModel
     group_ids = user.user_group_ids
     obj.projects.each do |project|
       next if project.open_membership
+      next unless project.trusted_by?(obj.user)
       return true if group_ids.member?(project.user_group_id) ||
                      group_ids.member?(project.admin_group_id)
     end
diff --git a/db/migrate/20231019202655_update_project_users.rb b/db/migrate/20231019202655_update_project_users.rb
index e8e97677e..752a3bc68 100644
--- a/db/migrate/20231019202655_update_project_users.rb
+++ b/db/migrate/20231019202655_update_project_users.rb
@@ -4,8 +4,8 @@ class UpdateProjectUsers < ActiveRecord::Migration[6.1]
   def up
     Project.find_each do |project|
       project.user_group.users.each do |user|
-        ProjectMember.create(project:, user:,
-                             trusted: project.user_id == user.id)
+        trusted = (project.user_id == user.id) || not project.open_membership
+        ProjectMember.create(project:, user:, trusted:)
       end
     end
   end

Non-open membership projects essentially implicitly trust the project. We could change that I suppose. At minimum it seems like maybe we shouldn't allow such users to even mess with the trust level since it's kind of meaningless.

Thanks for the explanation. I agree about not letting users mess with the trust level.

My proposed change would make both GPS visibility and edit privileges for admins and members non-open projects depend on the owner trusting the project.  The migration change would default all members of non-open projects to trusting that project.  Should probably make it so new created ProjectMembers trust the Project if it's not an open project.

Oof, that broke 18 tests.  My brain is fried.  I'll look at this more tomorrow.

So I dug into some of the test failures.  All the ones I've looked at so far are due to objects that are owned by the test user "mary" for tests being run as the user "dick".    The fact that mary is not a member and hence cannot trust the bolete_project means that these objects have switched from editable to non-editable.  I think simplest thing to do is add mary to bolete_users and have her trust the project.  That reduces the errors from 18 to 13.  I'll have to dig into those.

Lat/Lon is missing from a spreadsheet downloaded by the Project Admin after Trust Revoked, but is present in printed Labels. Here's what I did:

As me:

Created a project Administered by me (in Chrome).
Added user OMS to that project.

Logged in as OMS in a different browser (Firefox).

Trusted the Project.
Ceated an Observation with hidden coordinates.
Revoked Trust.

As me,

went to the Project,
clicked Observations,
Download Observations
Selected CSV spreadsheet, UTF-8, clicked Download
Result: spreadsheet without Lat/Lon
observations_1oQg6.csv
Clicked UTF-16, then UTF-8 (in order to un-grey the Print Labels button)
Clicked Print Labels
Result: RTF label that includes Lat/Lon

I also note that the label rounds lat and lon to 1 decimal place. That seems wrong. (I don't think it should be rounded.) But that's another story.

I don't understand what you want here.  I am simply following what the code currently does.  For reports gps_hidden removes the lat/long.  For labels it rounds to .1.  Change either of these should be handled in a different PR.

As I said in the latest commit comment:
What an adventure!  This is entirely to enable the one line change to
app/models/project.rb.  This broken many tests (18?) and led me to
discover that the way the bolete_project was configured was weird
(user mary had observations but was not a member and hence didn't
trust the project).  I ended up fixing the bolete_project except that
it was also being used to test some special cases for a project that
has a single user.  This led me to create the lone_wolf user and
lone_wolf_project.  This in turn led me to find what I consider to be
a bad test in: test/integration/capybara/lurker_integration_test.rb
since it is dependent on the first observation in the rss_log to not
have images.  I then found a bug in
app/classes/query/modules/active_record.rb via
test/models/query_test.rb where an error gets thrown if the new
observation, :wolf_fart (English translation of Lycoperdon), has two
images.  I changed the fixtures to work around the lurker test issue
and the active_record bug.  The active_record should get fixed,
but I wasn't able to figure out a satisfactory solution.
I left comments in the code to reproduce the two issues that I worked around by finessing the fixtures.  I really don't like how incredibly complex some of our tests are and how brittle the system is to changes in the fixtures.  It seems to me that you should always be able to add new stuff to the fixtures and not break existing tests.

@JoeCohen Is there anything you're still waiting for on this PR?  More manual testing would be great.  I've been buried in the test/fixture issue and haven't done much manual testing.  I think the latest changes will work better for closed projects.

@mc-nathan There's nothing more I'm waiting on. (A lot of my problem was that i didn't understand the desired functionality.)
I will try to exercise the branch more.
But I think we should get the pending Project-related PRs merged. (#1658, #1686, #1691)
(Because of the potential for conflict, I'm uncomfortable with having three of them pending. But maybe that's just me. I wanted to add )
I agree about the brittleness of the fixtures and tests. What's the solution? Fewer fixtures + more factories?

@JoeCohen Is there anything you're still waiting for on this PR? More manual testing would be great. I've been buried in the test/fixture issue and haven't done much manual testing. I think the latest changes will work better for closed projects.

Ok, I'd like to do a bit of manual sanity checking before merging, but I'm going to head that way.  Personally, I'm not worried about having multiple project related PRs open, but it would of course be good to get stuff merged.
Regarding fixing the tests, I believe that all of the tests we want to do could be written in a way that allows for additions to fixtures.  In my view our tests are often too complicated.  We are definitely not following best practices for how to write tests.  My recommendation is that we figure out which tests are fragile to fixture changes and work to fix those tests.  I think we could find them by just adding a disconnected new fixture to each fixture file.
We could consider switching to factories.  In my view it is a better way to do tests at scale, but it's not without it's problems.  Given how committed we currently are to fixtures, I don't think it would be worth the effort to switch.


Type narrowing doesn't work inside Record

Bug Report

🔎 Search Terms
explicit type narrowing record
🕗 Version & Regression Information
Tested on lastest version 4.8.4 and nighty version 4.9.0-dev.20221017
⏯ Playground Link
Playground link with relevant code
💻 Code

interface Child {
    num?: number;
}

interface Parent {
    children: Record<string, Child>;
}

function allGreaterThanFive(parent: Parent): boolean {
    for (const child in parent.children) {
        if (
            parent.children[child].num === undefined ||
            5 - parent.children[child].num <= 0
        ) {
            return false;
        }
    }
    return true;
}

🙁 Actual behavior
parent.children[child].num is being reported that it could be undefined
🙂 Expected behavior
Typescript narrowing down that parent.children[child].num can't be undefined

Duplicate of #10530. Type narrowing does not occur for indexed access forms e[k] where k is not a literal.
Just store it in a variable (which IMO is much cleaner anyway).


Fix documentation: -- or -

Readme suggests the flags use double dashes. For example --dry-run but the v1.0.0 binary still uses single dashes eg. -dry-run

More confusingly, the help menu in the tool itself suggests:
$ ./monaco
Please provide environments yaml with -e/--environments

But the output shows:
Usage of arguments:
  -environments string
        Mandatory yaml file containing environments to deploy to.

Finally, the tool actually excepts both -environments and --environments

Go flags support both cases, and I wouldn't change that behaviour. As it's common in UNIX we should support both UNIX -e and GNU style flags --environments. But I agree with you, we need more consistency and we should make sure:

shorthand options are in UNIX style: -e, -p
long options are in GNU style: --environments, --project

The issue here is that Go supports - for shorthand and long options, as well as the standard --shortand & ---long options.
The documentation was purposely written to follow the norm for options.
The help text is generated by Go flags - which shows the single dash exclusively - I'm not aware of a way to change that.
Funnily enough, even the documentation of flags reads:

Use -h or --help flags to get automatically generated help text for the command-line program.

Gotcha. I've learned something new today :) I'll close this as not an issue.


Improvements

Refactored the code
Fixed a minor bug when using multiple characters in the west mark.
Use the ul package to merge data.

:tada:


Can't install libxmljs 0.18.8

Hi!
I get a HTTP 301 error when trying to install libxmljs 0.18.8 with yarn or npm. I'm able to download the file from my browser but it seem nod-pre-gyp does not handle the redirection very well...
Yarn error:
node-pre-gyp http GET https://github.com/libxmljs/libxmljs/releases/download/v0.18.8/node-v64-darwin-x64.tar.gz
node-pre-gyp http 301 https://github.com/libxmljs/libxmljs/releases/download/v0.18.8/node-v64-darwin-x64.tar.gz
node-pre-gyp ERR! Completion callback never invoked!
node-pre-gyp ERR! System Darwin 18.7.0
node-pre-gyp ERR! command "/Users/ce/.nvm/versions/node/v10.15.3/bin/node" "/Users/ce/git/node_modules/libxmljs/node_modules/.bin/node-pre-gyp" "install" "--fallback-to-build" "--loglevel" "http"
node-pre-gyp ERR! cwd /Users/ce/git/node_modules/libxmljs
node-pre-gyp ERR! node -v v10.15.3
node-pre-gyp ERR! node-pre-gyp -v v0.9.1

Npm error:
node-pre-gyp http GET https://github.com/libxmljs/libxmljs/releases/download/v0.18.8/node-v72-darwin-x64.tar.gz
node-pre-gyp http 301 https://github.com/libxmljs/libxmljs/releases/download/v0.18.8/node-v72-darwin-x64.tar.gz
node-pre-gyp ERR! Completion callback never invoked!
node-pre-gyp ERR! System Darwin 18.7.0
node-pre-gyp ERR! command "/Users/ce/.nvm/versions/node/v12.13.0/bin/node" "/Users/ce/test/node-typescript-boilerplate/node_modules/libxmljs/node_modules/.bin/node-pre-gyp" "install" "--fallback-to-build" "--loglevel" "http"
node-pre-gyp ERR! cwd /Users/ce/test/node-typescript-boilerplate/node_modules/libxmljs
node-pre-gyp ERR! node -v v12.13.0
node-pre-gyp ERR! node-pre-gyp -v v0.9.1
node-pre-gyp ERR! This is a bug in `node-pre-gyp`.
node-pre-gyp ERR! Try to update node-pre-gyp and file an issue if it does not help:
node-pre-gyp ERR!     <https://github.com/mapbox/node-pre-gyp/issues>
npm WARN<EMAIL_ADDRESS>No repository field.

npm ERR! code ELIFECYCLE
npm ERR! errno 6
npm ERR<EMAIL_ADDRESS>install: `node-pre-gyp install --fallback-to-build --loglevel http`
npm ERR! Exit status 6
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:

Node: v12.13.0
Npm: 6.12.0
yarn: 1.22.4
Thanks for your help!!

Hi Guys, is there any fix for this? Thanks!

I think I fixed it by using node.js 10


Org/space/app name annotations

Description
The Log Egress (Loggregator) team would like to send logs from apps to many downstream consumers, most of which have historically depended on app metadata to help filter and annotate those logs. Some of those are native to the platform, like Log Cache, but others are offsite, such as Splunk, Papertrail, or AppDynamics. To make this happen, annotations on the StatefulSet/Pods with the name of the application, org and space would be extremely useful.
While it is possible for this information to become out of date using the cf rename-* commands, the current implementation in CF continues to send the original name, so it would be fine for Eirini not to update them.
Suggested Fix
Given that the cloudfoundry.org/application_name and space_name annotations already seem to exist (https://github.com/cloudfoundry-incubator/eirini/blob/ca5617920508538c055485a30362b4240b2d35ee/k8s/statefulset.go#L28), we suggest adding a cloudfoundry.org/org_name or organization_name annotation.

This should work in the latest version. Feel free to reopen the issue if still see problems


Simple form not valid inside of html template.

I noticed the simple_form component doesn't seem to work when using the liveview-client-html library with a render component.
Here's my template:

Chat App

    <%= for message <- @messages do %>

<%= message.name %>

<%= message.content %>

    <% end %>

<.simple_form for={@form} id="form" phx-submit="send-message">
    <.input field={@form[:name]} placeholder="From..." />
    <.input field={@form[:content]} placeholder="Say..." />
    <:actions>
        <.button type="submit">
            Send Message
        </.button>
    </:actions>
</.simple_form>

Here's my render component:
defmodule ChatWeb.ChatLive.HTML do
  use ChatNative, [:render_component, format: :html]
end

Very weirdly it's interpreted as text. (see the image below)

@BrooklinJazz are you using ~H or ~LVN ?

For the render function, I'm using ~H

But in the example, I'm using a separate template file

@BrooklinJazz Did you ever figure this out? I'm having the same issue when using a template file.  Seems to work in a render/2 function but just prints out the HTML as text in a template file.

The issue is related to the Native module. I've got a local working branch to fix this but I'm currently in the middle of implementing LiveComponents in LVN and will circle back to resolve this problem after. Sorry!


KeyValue atomic delete and purge methods

Proposed change / Use case
See https://github.com/nats-io/nats-architecture-and-design/issues/271

Completed https://github.com/nats-io/nats.java/pull/1092


Split into patches & global info

Hi,
It helps to save memory by spliting image into patches, but each patches are treated individually, is there another way to ultilize the global info of the original image for harmonization with limited GPUs?
Another question is, I'm trying to do video harmonization, sometimes the results of adjacent frames are different while the background is quite similar.  Also, the same thing happens in many other works. Is it  an unsolved problem for image harmonization?

For question 2, I extract the 'intermediates' from first frame and fix it for the following frames. It improves the results but does it make sense?

Hi @wzking09 ,
To my knowledge, currently, the quite efficient way is to predict the params of color filters, which can lead to a lightweight and efficient network easily (yet lead to inferior performance). I'd like to recommend Harmonizer as a reference for color-filter-based methods.
For the second question, yes, the inconsistency of adjacent frames is a problem valuable to research. Currently, there exist some solutions to this problem, which you may refer to in Section V-D in the newest version of our paper.
--------------------
Your possible solution for question 2 may work for short clips, but how to generalize to long videos with varying scenes?

Thanks!
I've tried Harmonizer on my own data. The predicted filter params vary a lot among frames. Maybe there is a  huge gap between train/test data sources. And yes, all test data are very short videos.


Implemented cdh_utils for Python

Translation of cdh_utils.R in Python (cdh_utils.py)

Hi Daniel, thanks for this contribution! Please consider adding CI and code coverage like I did for the R library. I'm using Travis CI and codecov. Cheers


Cannot call system fonts on macOS

\setmainfont[Scale=1.3]{Angsana New} gives error Package fontspec Error: The font "Angsana New" cannot be found.

Is this issue fixed? @DobaKung

Actually, this issue should be closed because macOS doesn't have Angsana anyway. We have another problem instead though.


Surrealdb-migrations compiled binary file

Surrealdb-migrations is an awesome project. It would be very convenient for  developers not using  Rust  to use this tool If a compiled binary file is provided.

Hi Zafar,
This is great idea.
The compiled binaries are now integrated on every release. You can download the binaries from the latest release here: https://github.com/Odonno/surrealdb-migrations/releases/tag/v0.9.4


Add property contentcreated

Working with ninjs I noticed that a property for the date when the content as such was created is missing.
IPTC NewsML-G2 has contentCreated and the IPTC Photo Metadata has a Date Created.
ninjs has versioncreated but this aligns with the NewsML-G2 versionCreated: its the timestamp of when this version of this news item was created. Photos may be taken and audio may be recorded significantly before a news item (version) is created and this needs to be reflected.

Good idea.


Expired session after adding new device

Found some oddity in how keybase sets up on a 2nd device. I think. :)
The situation:

Keybase set up and working fine on local machine and used often.
I was trying to get a secondary machine, a VPS, access to my keybase filesystem.
2nd machine is headless (no UI). Not sure if this is a factor.
Installed keybase on second machine.
keybase device add -- added second machine.
ls /keybase/public// --- sits there forever.
run_keybase
ls /keybase/public -- shows my username
ls /keybase/public// --- sits there forever.
keybase login -- I login thinking maybe there is a dialogue box that I am not seeing.
ls /keybase/public// --- sits there forever.
keybase status -- I see nothing obvious, but I save the output
keybase unlock
ls /keybase/public// --- WORKS!
keybase status -- save new output and diff...

-Session:       toddwarner [loaded: yes, cleared: no, expired: no]
+Session:       toddwarner [loaded: yes, cleared: no, expired: yes]

Now why would the session be expired right off the bat? I even rebooted to see if that would remedy things auto-magically. It did not. I had to manually unlock things.
Anyway. It smells like there is a bug here somewhere. Or at least, a sub-optimal workflow. I hope this helps.
my log id: 3b091934cdac5a9bb3ec921c

Cc: @patrickxb. This is similar to the situation that @malgorithms hit a few days ago.  @taw00 I agree there is some bug here.

@taw00 how did you provision the second device?

@taw00 how did you provision the second device?

Sorry that this issue sat in my dust bin forever. I don't recall how I provisioned that device.


Get-ChildItem: Cannot find path '/home/runner/work/maester-tests/maester-tests/tests/' because it does not exist.

After the last update I am getting the following error
WARNING: The version '5.5.0' of module 'Pester' is currently in use. Retry the operation after closing the applications.

Get-ChildItem: Cannot find path '/home/runner/work/maester-tests/maester-tests/tests/' because
it does not exist.
Error: System.Management.Automation.RuntimeException: No test files were found and no scriptblocks were provided. Please ensure that you provided at least one path to a *.Tests.ps1 file, or a directory that contains such file. Or that you provided a ScriptBlock test container.
Message
  at Invoke-Pester<End>, /usr/local/share/powershell/Modules/Pester/5.5.0/Pester.psm1: line 5042
  at Invoke-Maester, /home/runner/.local/share/powershell/Modules/Maester/0.0.128/public/Invoke-Maester.ps1: line 254
  at <ScriptBlock>, /home/runner/work/_temp/52d05bc6-33f9-4f[83](https://github.com/Cyber-Monks/maester-tests/actions/runs/9196716704/job/25295446060#step:6:84)-ab45-30f22f7e11ea.ps1: line 17
  at <ScriptBlock>, <No file>: line 1
Get-Content: /home/runner/work/_temp/52d05bc6-33f9-4f83-ab45-30f22f7e11ea.ps1:20
Line |
  20 |  $summary = Get-Content test-results/test-results.md
     |             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Cannot find path
     | '/home/runner/work/maester-tests/maester-tests/test-results/test-results.md' because it does not exist.
Error: Error: The process '/usr/bin/pwsh' failed with exit code 1

it seems like is checking the wrong folder but this is my main.yml:
name: Maester Daily Tests

on:
  push:
    branches: ["main"]
  # Run once a day at midnight
  schedule:
    - cron: "0 0 * * *"
  # Allows to run this workflow manually from the Actions tab
  workflow_dispatch:

permissions:
      id-token: write
      contents: read
      checks: write

jobs:
  run-maester-tests:
    name: Run Maester Tests
    runs-on: ubuntu-latest
    steps:
    - uses: actions/checkout@v4
    - name: Set current date as env variable
      run: echo "NOW=$(date +'%Y-%m-%d-T%H%M%S')" >> $GITHUB_ENV
    - name: 'Az CLI login'
      uses: azure/login@v2
      with:
          tenant-id: ${{ secrets.AZURE_TENANT_ID }}
          client-id: ${{ secrets.AZURE_CLIENT_ID }}
          allow-no-subscriptions: true
    - name: Run Maester
      uses: azure/powershell@v2
      with:
        inlineScript: |
          # Get Token
          $token = az account get-access-token --resource-type ms-graph
          # Connect to Microsoft Graph
          $accessToken = ($token | ConvertFrom-Json).accessToken | ConvertTo-SecureString -AsPlainText -Force
          Connect-MgGraph -AccessToken $accessToken
          # Install Maester
          Install-Module Maester -Force
          # Configure test results
          $PesterConfiguration = New-PesterConfiguration
          $PesterConfiguration.Output.Verbosity = 'None'
          # Run Maester tests
          $results = Invoke-Maester -Path tests/ -PesterConfiguration $PesterConfiguration -OutputFolder test-results -OutputFolderFileName "test-results" -PassThru
          # Add step summary
          $summary = Get-Content test-results/test-results.md
          Add-Content -Path $env:GITHUB_STEP_SUMMARY -Value $summary
          # Flag status to GitHub
          if ($results.Result -ne 'Passed'){
            Write-Error "Status = $($results.Result): See Maester Test Report below for details."
          }
        azPSVersion: "latest"

    - name: Archive Maester Html Report
      uses: actions/upload-artifact@v4
      if: always()
      with:
        name: maester-test-results-${{ env.NOW }}
        path: test-results

Which is the same I was using before and same as published on website with the only exception I am running all tests and not just Maester ones.
So, after the execution these are the errors I am getting:
Run Maester TestsSystem.Management.Automation.RuntimeException: No test files were found and no scriptblocks were provided. Please ensure that you provided at least one path to a *.Tests.ps1 file, or a directory that contains such file. Or that you provided a ScriptBlock test container.

Run Maester TestsError: The process '/usr/bin/pwsh' failed with exit code 1

Run Maester TestsNo files were found with the provided path: test-results. No artifacts will be uploaded.

What does the directory structure of your repo look like? sounds like your tests/ folder isn't in the right place.

What does the directory structure of your repo look like? sounds like your tests/ folder isn't in the right place.

Hi Adam, effectively I just solved the issue. I am not sure for which reasons (probably the Layer 8 as usual) all folders were in the root folder and not tests. Stupidly enough I didn't check this but I was focusing on the code.


Old FileSearchProvider session not properly cancelled

Implement a traditional fileSearchProvider.
In provideFileSearchResults, listen for options.session?.onCancellationRequested
While debugging the extension, open a file picker, then close it.
The session token's onCancellationRequested never gets triggered when the session is done (aka when the file picker is closed) :bug:

This contradicts https://github.com/microsoft/vscode/blob/2e4a043ef387856c67ac934c36ca67073f56624d/src/vscode-dts/vscode.proposed.fileSearchProvider.d.ts#L37

regression from https://github.com/microsoft/vscode/pull/226687


Value conversions from strings to other types

Resolves #11492
Finally I have found some time to work on this, sadly too late for 2.1.
This PR adds value conversions from string to the following types:

bool
char
DateTime
DateTimeOffset
Enum
Guid
Number
TimeSpan

I will also add unit tests, but I think it would be better, if someone briefly review this first.
Cc: @ajcvickers

@SSkovboiSS, @smitpatel has a bot in his pocket, let me test it 😄.
@dotnet-bot test windows release x64 build

Thanks for try @ralmsdeveloper, but actually I supposed that build will fail.
I have fixed tests now and also revisited numeric convertions.
I will later add more unit tests, but I would like to get answers for my questions regarding promoting logic to base classes instead of duplicating it, because this way it will be possible to reduce duplicating in unit tests aswell.

I did a rebase on latest dev and fixed a case when provider type is unknown.
It seems like build does not work at all 😕

@dotnet-bot test

@dotnet-bot test windows release x64 build

@dotnet-bot test ubuntu 16.04 release build
@dotnet-bot test OSX 10.12 release build

@dotnet-bot test windows release x64 build

@ajcvickers I resolved all of your points. Waiting for further feedback.

@SSkovboiSS Thanks!


Feature request: Refactor to initialize instance through ctor

class C
{
public int I1 { get; set; }
public int I2 { get; set; }
}
should be refactored to
class C
{
public C(int i1, int i2)
{
this.I1 = i1;
this.I2 = i2;
}
public int I1 { get; set; }
public int I2 { get; set; }
}
optionally making setters private :), althrough this would mean to fix all ctor call sites. So maybe not.
This refactoring could help to make types immutable.

This is my most used feature in Resharper.
public class Foo {
    public string Bar { get; }
}

To
public class Foo {
    public Foo(string bar) {
        Bar = bar;
    }

    public string Bar { get; }
}


gNMI-1.14.2 Test gNMI Subscribe with "ON_CHANGE" mode for backplane capacity counters

Request for testing gNMI Subscribe with "ON_CHANGE" mode for backplane capacity counters
WBB is required to support gNMI Subscribe with ON_CHANGE mode for various counters. This test if to verify that DUT supports gNMI Subscribe with on_change mode for backplane capacity counters
Implement code for gNMI-1.14.2

I think we need to also OC descriptions for /components/component/integrated-circuit/backplane-facing-capacity.  I see https://github.com/openconfig/public/pull/990 in progress could be refined with the below:

consumed-capacity is the sum of the admin-up front panel interface speeds connected to the integrated-circuit
available-percent is total-operational-capacity / consumed-capacity

Orion will subscribe ON_CHANGE to available-percent and take action if available-percent < 100
gNMI-1.4 functional test should be updated to implement the above.


UI crashes on "search page" if we add multiple filters with the same category

UI crashes on Search Page if we add multiple filters to the same category.
Expected Behaviour
There can be scenarios where having multiple search filters for the same category can be useful, e.g. maybe we could have a checkbox to include or exclude items that have a certain badge (e.g. PII). Having such a checkbox can be more user friendly if we're trying to build a complex search query as many analysts might not be able to follow proper search syntax, as is expected in the search filter field.
Current Behaviour
If we add multiple search filters having the same "categoryId", the search page breaks upon switching back and forth between resource types.
Possible Solution
Perhaps, this is happening because "category Ids" get passed further as "html ids" which are supposed to be unique. Maybe a better support to specify html id in the config or handling of this edge case in code to accommodate duplicate category ids is needed.
Steps to Reproduce

Add two filters having the same categoryId in the config-default.ts or config-custom.ts file.

After building and running, navigate to the "Search" page and upon switching back and forth between different resource types (e.g. Tables and Dashboards etc.), you will notice that the search filter with the duplicate "categoryId" keeps on getting duplicated in FilterSection (sidebar) e.g. if we switch 3 times, there will be 3 extra filters displayed (one added per transition).

Screenshots (if appropriate)

Context
This affects a very useful and highly requested feature of ours.  Would be great if it can be fixed.
Your Environment

Amundsen version used: common-0.30.0 (e0373e17a03b833f4a5c6097b12b8d7f6446d2f1)
Data warehouse stores: BigQuery
Deployment (k8s or native): k8s
Link to your fork or repository:https://github.com/deliveryhero/amundsen

@Golodhros @kristenarmes @allisonsuarez can you please take a look and help with this issue if possible?

Closing this, as it has been fixed by this PR.


[bazel,dvsim] update dvsim.py to use Bazel to build SW

After previously rolling this change back due to CI resource constraint
issues, this again fixes #11563 by updating dvsim.py, and the chip-level
testbench, to build SW artifacts with Bazel (instead of meson).
Signed-off-by: Timothy Trippel<EMAIL_ADDRESS>

@GregAC @rswarbrick looks like the private CI job still took 1hr20min.

Thanks @timothytrippel, I did do a run of the previous commit that added the bazel sw build (https://github.com/lowRISC/opentitan/commit/b42dbc2d004358a7036ab4eee2c02b79e091bead) and that took 40 minutes (https://dev.azure.com/lowrisc/opentitan-private/_build/results?buildId=81688&view=results), so hopefully things a bit variable but 40 minutes will be more the norm.
I've set up an hourly run of this PR on private CI: https://dev.azure.com/lowrisc/opentitan-private/_build?definitionId=16&_a=summary let's see how it gets on. If it's looking to be working well I'll move it to every 30 minutes tomorrow. Which should give it a decent stress test.

SG thanks @GregAC !

The hourly run worked well, 30-40 minutes per run. Sadly it all blew up with the stress test, I set it up to run every 20 minutes to give it a decent workout and like last time lots of jobs hitting 2 hours run-time and timing out.
One of the jobs didn't hit the timeout, but failed with this message:

Building "//sw/device/tests/sim_dv:uart_tx_rx_test_sim_dv" on network connected machine.
./bazelisk.sh: line 64: /azp/agent/_work/1/s/.bin/bazelisk: Text file busy

An indication something isn't happy.


Kansas NG911 Partial State Addresses!

Hi Justin,
You can submit a request for the NG911 statewide address points through the NG911 Program Portal (https://portal.kansas911.org/).  You will need to register for an account.  Once your registration has been approved, you will receive an email containing your password.  Log into the Portal and click on the GIS Data Request menu item to submit your request.  The purpose of your request is included in our communication to the PSAPs, so I would encourage you to provide a short, but robust description of the purpose of your request.
Like Sherry mentioned, the data is owned and maintained by the individual PSAPs.  Your request will be sent out to every Kansas PSAP for their approval.  Once we allow 1-2 weeks for their response, we'll process the PSAP approved data and you'll receive an email containing a download link.
Since the counties approved the distribution of the data, there are no restrictions on how the data is used.
Please let me know if you have any further questions.
Thanks,
Eileen

UPLOADING TO DROPBOX NOW

This is great! Thanks for doing this work, @justinelliotmeyers.

@justinelliotmeyers can you zip the .dbf/.shp/.shx for each county together rather than individually? Machine won't know what to do with them as individual zip files.

ummmm, might take me a while. But I can do it, or does anyine have an arcmap license? Data came in FGDB? Im fine with either

or should i just export a flat CSV?

here is a CSV: https://www.dropbox.com/s/ostce96gor4639s/ks_ng911_GCS_North_American_1983.zip?dl=0


Update Binary Compatibility page

Update Binary Compatibility between Visual Studio 2015 and Visual Studio 2019, to add a section about upgrading from from older to newer VC++ Redist.

@msalehmsft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@mikeblome : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@GitHubber17 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Possible issue with DiffMarkupFormatter.Instance

We are using Verify with BUnit on a fairly large project, with many tests using .Verify  We increasingly see the following exception in our tests when we run the entire suite.  When running smaller sub-groups the error will not occur and we can continue.
As shown below the error occurs in the AngleSharp PrettyMarkupFormatter when it tries to indent by a negative number.
The problem I can (easily) see is that Verify.Bunit uses the static instance of DiffMarkupFormatter meaning that it is keeping count of open tags across all tests, so any test that is invalid, will break all tests after that.
Locally I made the change to create a new DiffMarkupFormatter each time instead of using .Instance and it appears to have solved the problem.  I can submit a PR, but being very new to this code base I wanted to get some feedback first.
System.ArgumentOutOfRangeException
Specified argument was out of the range of valid values. (Parameter 'count')
   at System.Linq.ThrowHelper.ThrowArgumentOutOfRangeException(ExceptionArgument argument)
   at System.Linq.Enumerable.Repeat[TResult](TResult element, Int32 count)
   at AngleSharp.Html.PrettyMarkupFormatter.IndentBefore(Int32 i)
   at AngleSharp.Html.PrettyMarkupFormatter.OpenTag(IElement element, Boolean selfClosing)
   at Bunit.Diffing.DiffMarkupFormatter.OpenTag(IElement element, Boolean selfClosing) in /_/src/bunit.web/Diffing/DiffMarkupFormatter.cs:line 51
   at AngleSharp.Dom.Node.ToHtml(TextWriter writer, IMarkupFormatter formatter)
   at AngleSharp.FormatExtensions.ToHtml(IMarkupFormattable markup, IMarkupFormatter formatter)
   at MarkupFormattableToString.Convert(IMarkupFormattable markup, IReadOnlyDictionary`2 context) in C:\projects\verify-bunit\src\Verify.Bunit\MarkupFormattableToString.cs:line 4
   at VerifyTests.VerifierSettings.<>c__DisplayClass166_0`1.<RegisterFileConverter>b__0(T o, IReadOnlyDictionary`2 context) in /_/src/Verify/Splitters/Settings_Typed.cs:line 30
   at VerifyTests.VerifierSettings.<>c__DisplayClass167_0`1.<RegisterFileConverter>b__0(Object o, IReadOnlyDictionary`2 context) in /_/src/Verify/Splitters/Settings_Typed.cs:line 39
   at VerifyTests.InnerVerifier.Verify(Object target) in /_/src/Verify/Verifier/InnerVerifier_Object.cs:line 67
   at VerifyXunit.Verifier.<>c__DisplayClass9_0.<<Verify>b__0>d.MoveNext() in /_/src/Verify.Xunit/Verifier.cs:line 74
--- End of stack trace from previous location ---
   at Web.Pages.Students.SemesterDetailsTests.RendersCorrectly__WhenStudent__AndViewIs__Certificate() in C:\code\work\gnl\gnl.sams\app\Gnl.Sams.Ops.Web.UnitTests\Pages\Students\SemesterDetailsTests.cs:line 440
   at Xunit.Sdk.TestInvoker`1.<>c__DisplayClass47_0.<<InvokeTestMethodAsync>b__1>d.MoveNext() in /_/src/xunit.execution/Sdk/Frameworks/Runners/TestInvoker.cs:line 259
--- End of stack trace from previous location ---
   at Xunit.Sdk.ExecutionTimer.AggregateAsync(Func`1 asyncAction) in /_/src/xunit.execution/Sdk/Frameworks/ExecutionTimer.cs:line 48
   at Xunit.Sdk.ExceptionAggregator.RunAsync(Func`1 code) in /_/src/xunit.core/Sdk/ExceptionAggregator.cs:line 90

Just a quick follow up, I switched our test runs to be serial and it resolved the issue, so seems like it is the thread safety of the DiffMarkupFormatter causing the issue
	/// <summary>
	/// Gets an instance of the <see cref="DiffMarkupFormatter"/>.
	/// </summary>
	/// <remarks>
	/// The <see cref="DiffMarkupFormatter"/> is not thread safe, so using this singleton
	/// instance to format elements may not result in the desired effect.
	/// </remarks>
	public static new readonly DiffMarkupFormatter Instance = new();


LIT-528106 Allow handling of duplicated ids

@mbartenev-atypon is this the only PR related to the duplicate IDs problem?

@mbartenev-atypon is this the only PR related to the duplicate IDs problem?

Here is the other one
https://github.com/Atypon-OpenSource/manuscripts-article-editor/pull/239/files

@mnatsheh looks like we don't need that PR anymore. Closing.

@mbartenev-atypon was the fix included somewhere else? did we handle the issue in some other way?

hm, I was under impression that it was used on some other branch for the purposes of testing zax and was merged then. But now I'm checking the code and I see that's not on master so it was never merged, although the ticket is closed. Let me know if we need to merged it.

I can still see tickets being filed with the "duplicate ids in model map" error (e.g. LEAN-3047).


Scrapper, Alisa Karpova - 22FPL2

You need to parse author more carefully - see the error from tests attached


Make it work on DragonFly/FreeBSD

Tested on DragonFly BSD. Should work similarily on FreeBSD.

Thanks!


service: provide formula accessor.

Use f because it's generally recognised as a formula and this shouldn't be widely needed/used.
Needed for https://github.com/Homebrew/homebrew-core/pull/107726

Review period skipped due to critical label.

Should @formula usage be switched out too?

@SMillerDev I think that's fine/more readable. Will push example to homebrew-core PR shortly.


ToDo: Update Readme

Needs an updated overview, and really an entire re-design.

all the internal links should be good, now, for the whole project.
still need to work on this readme, but first adding content, because the more subjects having their own pages changes the structure, and so... need to keep working out the main categories I want to focus on...
The standards area, of course.
Related Regulation: Personal Data, Privacy, Digital Identification.
I've gone through about half of the project filling out a few areas that needed attention,
but am hesitant to spend too much time on this readme, while the site structure is in flow.


Excellent work --- :) clap

Hi, DCMMC!
I have the pleasure of perusing your paper. But I was confused about how to transform raw pcap files to .traffic format. Can you update the code of this part or provide some reference?

Hi, DCMMC! I have the pleasure of perusing your paper. But I was confused about how to transform raw pcap files to .traffic format. Can you update the code of this part or provide some reference?

I have the same question and hope it gets resolved

Hi, DCMMC! I have the pleasure of perusing your paper. But I was confused about how to transform raw pcap files to .traffic format. Can you update the code of this part or provide some reference?

I have the same question and hope it gets resolved

Is this problem solved?

Hi, DCMMC! I have the pleasure of perusing your paper. But I was confused about how to transform raw pcap files to .traffic format. Can you update the code of this part or provide some reference?

I have the same question and hope it gets resolved

Is this problem solved?

I have the same question and hope it gets resolved ，thanks

Hi, DCMMC! I have the pleasure of perusing your paper. But I was confused about how to transform raw pcap files to .traffic format. Can you update the code of this part or provide some reference?

I have the same question and hope it gets resolved

Is this problem solved?

I also have the same issue and hope to get it fixed

I also have the same confusion

Hi, DCMMC! I have the pleasure of perusing your paper. But I was confused about how to transform raw pcap files to .traffic format. Can you update the code of this part or provide some reference?

I found the author's reply in another closed issue:

Because the preprocessed .hdf5 or .traffic files are too large, we only publish the original .pcap files. You need to preprocess these pcap files into a .traffic file. In the .traffic file, each line represents a preprocessed packet. For example, reddit//tcp0//0 45 10 00 48 00 00 40 00 40 .... To preprocess, you need remove irrelative packets (e.g., ARP, ICMP, DNS) and mask some fields according to the paper. .hdf5 files are just another form of .traffic files, you can read the codes.

we can just try it

Hi, DCMMC! I have the pleasure of perusing your paper. But I was confused about how to transform raw pcap files to .traffic format. Can you update the code of this part or provide some reference?

I found the author's reply in another closed issue:

Because the preprocessed .hdf5 or .traffic files are too large, we only publish the original .pcap files. You need to preprocess these pcap files into a .traffic file. In the .traffic file, each line represents a preprocessed packet. For example, reddit//tcp0//0 45 10 00 48 00 00 40 00 40 .... To preprocess, you need remove irrelative packets (e.g., ARP, ICMP, DNS) and mask some fields according to the paper. .hdf5 files are just another form of .traffic files, you can read the codes.

we can just try it

hello,have you ever solved this problem?


Build error

I Try build on Linux gcc 11.2 :
In file included from /usr/include/signal.h:328,
from /mnt/N1/MZ/Prog/PC_CPP/CppLibs/libcoro/test/catch.hpp:7955,
from /mnt/N1/MZ/Prog/PC_CPP/CppLibs/libcoro/test/main.cpp:2:
/mnt/N1/MZ/Prog/PC_CPP/CppLibs/libcoro/test/catch.hpp:10735:58: error: call to non-‘constexpr’ function ‘long int sysconf(int)’
10735 |     static constexpr std::size_t sigStackSize = 32768 >= MINSIGSTKSZ ? 32768 : MINSIGSTKSZ;
|                                                          ^~~~~~~~~~~
In file included from /usr/include/c++/11.2.0/bits/atomic_wait.h:44,
from /usr/include/c++/11.2.0/bits/atomic_base.h:41,
from /usr/include/c++/11.2.0/bits/shared_ptr_atomic.h:33,
from /usr/include/c++/11.2.0/memory:78,
from /mnt/N1/MZ/Prog/PC_CPP/CppLibs/libcoro/test/catch.hpp:2909,
from /mnt/N1/MZ/Prog/PC_CPP/CppLibs/libcoro/test/main.cpp:2:
/usr/include/unistd.h:640:17: note: ‘long int sysconf(int)’ declared here
640 | extern long int sysconf (int __name) __THROW;
|                 ^~~~~~~
In file included from /mnt/N1/MZ/Prog/PC_CPP/CppLibs/libcoro/test/main.cpp:2:
/mnt/N1/MZ/Prog/PC_CPP/CppLibs/libcoro/test/catch.hpp:10794:45: error: size of array ‘altStackMem’ is not an integral constant-expression
10794 |     char FatalConditionHandler::altStackMem[sigStackSize] = {};
|                                             ^~~~~~~~~~~~
ninja: build stopped: subcommand failed.

Looks like its time to upgrade catch2, I currently only have gcc 11.1 on my local box so might take some time to reproduce.

@Maziar123 I've updated #113 but I didn't have time to try on the latest compiler.  Mind giving it another whirl and see if this fixes your issue?

I'm going to go ahead and close this out, if you are still having issues please re-open


@opentelemetry/instrumentation-graphql's unit test pipeline not working correctly

What version of OpenTelemetry are you using?

<EMAIL_ADDRESS>
What version of Node are you using?

>=14 <=18

What did you do?

Run unit test in current repo on github action

What did you expect to see?

@opentelemetry/instrumentation-graphql's unit test run correctly and return list of test + code coverage report

What did you see instead?

@opentelemetry/instrumentation-graphql's unit test on github action run success but unable to list out list of test + only report code coverage for symbol.ts file

pipeline: https://github.com/open-telemetry/opentelemetry-js-contrib/actions/runs/5567559025/jobs/10169539286

test output:

lerna info run Ran npm script 'test' in '@opentelemetry/instrumentation-graphql' in 49.4s:

><EMAIL_ADDRESS>test /home/runner/work/opentelemetry-js-contrib/opentelemetry-js-contrib/plugins/node/opentelemetry-instrumentation-graphql
> nyc ts-mocha -p tsconfig.json 'test/**/*.test.ts'

------------|---------|----------|---------|---------|-------------------
File        | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
------------|---------|----------|---------|---------|-------------------
All files   |       0 |      100 |     100 |       0 |
 symbols.ts |       0 |      100 |     100 |       0 | 17-19
------------|---------|----------|---------|---------|-------------------

code coverage report:
https://app.codecov.io/gh/open-telemetry/opentelemetry-js-contrib/pull/1580/tree/plugins/node/opentelemetry-instrumentation-graphql/src

Additional context

I double check those test on local machine and unable to repproduce this behavior. There are no crash or anything.
I able to reproduce with codespace, seem like nyc issue when they trying to transform code for codecoverage report. When running ts-mocha without nyc everything seem to work.

@haddasbronfman I think you also face similar problem with mongodb test with #1487 . I can try to update nyc, ts-mocha etc... to see if that can resolve the problem.

@chigia001 in my case I think also the problem is because of nyc, although in my case I saw an empty table.

@chigia001 is it ok if I assign this to you since you have a PR open?


Wrap header rows in

Prework

[X] Read and agree to the code of conduct and contributing guidelines.
[X] If there is already a relevant issue, whether open or closed, comment on the existing thread instead of posting a new issue.

Description
Header rows should always be wrapped in
. In particular, this will mean tables output to Typst that break over pages get the header rows repeated on each page.
Reproducible example
Consider the following example:
from great_tables import GT, md, html
from great_tables.data import islands

gt_tbl = GT(islands)
# Show the output table
gt_tbl

This results in the following (partial) HTML:

  name
  size

...

If this table is used in Quarto with format: typst and is allowed to break over pages, the header rows are not repeated on the second page:

QMD
---
title: "Untitled"
format: typst
keep-md: true
---

```{=typst}
#show figure: set block(breakable: true)
```
```{python}
from great_tables import GT, md, html
from great_tables.data import islands

gt_tbl = GT(islands)
# Show the output table
gt_tbl
```

Expected result
E.g. in gt the header rows are nested inside
,

      name
      size

Then the header rows are repeated on the second page:

Development environment

Operating System: macOS Sonoma 14.4
great_tables Version: 0.10.0
Quarto: dev version

@cwickham we are adding thead in #421 -- does this look okay for the typst work?
Here's a snapshot showing the new thead placement (note that thead now closes on line 66 in the new code in the diff)

Yeah, that looks great!


Unicode character causes "Fatal error: memory allocation failed"

When opening a file containing the following unicode character  lite-xl instantly closes. I could not replicate this issue on lite itself. May be related to #75?
To reproduce:

Create a file with  as it's contents
Open the file using lite-xl

System: Debian 10

Thank you for reporting the problem. This issue should be fixed in the new release, 1.16.2.

Can confirm it's fixed in 1.16.2. Thanks for all the great work you do @franko.


[google_maps_flutter] Ground Overlays support

Google Maps SDK for iOS and Android support ground overlays, however the feature is not present in google_maps_flutter package. This PR adds the support for Ground Overlays on both iOS and Android (thanks to @bytestreme) platforms
Fixes https://github.com/flutter/flutter/issues/26479
Pre-launch Checklist

[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[ ] I updated CHANGELOG.md to add a description of the change.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I signed the CLA.
[ ] All existing and new tests are passing.

PS. This is my first pull request, so any feedback will be helpful.
PPS. I modified the pubspec.yaml file to use relative google_maps_platform_interface package instead of the global one as the global doesn't have GroundOverlay class

Would be awesome to have this in the next versions!

Hopefully this PR will not end as #2085

rebased the branch onto master branch and migrated the modified code to null-safety. not sure though how good I migrated it))

Really looking for this feature. I hope your PR could be validated and included in a future release

Any news?

/cc @UDAYBIDKAR (since this overlaps with https://github.com/flutter/plugins/pull/4410; perhaps there's a way to pool effort here)

@strigePC Are you planning on adding the tests as mentioned above, and addressing the unit test failures showing up in CI?
For Java as mentioned above the effort can likely be combine with #4410, but iOS tests will need to be written.

Thank you for your contribution. I'm going to close this PR for now since there are outstanding comments, just to get this off our PR review queue. Please don't hesitate to submit a new PR if you have the time to address the review comments. Thanks!


GEODE-5579: Add -PforceTest property to :test tasks

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[n/a] Have you written or updated unit tests to verify your changes?

[n/a] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

I noticed significant duplication in test.gradle with respect to the other test targets and have streamlined and unified that configuration.  If that seems beyond the scope of this ticket, I'm happy revert that last commit and make it its own ticket / let it be subsumed in the future work to remove FlakyTest as a whole.

@PurelyApplied please remove the last commit or add a block to add the flaky exclusion back. This change will result in the remaining flaky tests being executed in the CI.

I hadn't noticed that we got rid of the FlakyTest box in our pipeline.  That's awesome!  But yes, that does mean those tests aren't getting run elsewhere.

@PurelyApplied the changes to the submodules seems to be negatively impacting the integration tests. Let's decouple those changes from this issue please.

@pivotal-jbarrett The integration failure appears to be due to a bad commit on / test-merge against develop.  See https://concourse.apachegeode-ci.info/teams/main/pipelines/develop/jobs/IntegrationTest/builds/257.


Change imports for mapbox-style-spec

Currently, this module imports ES6 code directly from the mapbox-style-spec module. This makes it difficult to use in projects that do not transpile the node_modules directory.
This PR imports from the built ES5 code in the mapbox module.

There is a huge trade-off between build size and convenience here. When importing from '@mapbox/mapbox-gl-style-spec', we import a lot of stuff we do not need, which tree-shaking also cannot get rid of. In terms of build size, we are talking about 115 kB larger bundles, just by making such a change.

Actually after removing the babel-loader, bundle size increases only by 10 kB instead of 115 kB. I have created #94 for further discussion.


fixes #621

@DTAIEB those aren't single quotes. They are "back ticks". I tried single quotes and that didn't work but stackoverflow gave me the back tick solution. Therefore a quote in the field name should probable work (but should be tested). I did the space in name check because some on stackoverflow worried that this was an unstable hack, so I only wanted to do it when necessary.


BLE Mesh node name (IDFGH-6530)

Hi All
according to
issue  https://github.com/espressif/esp-idf/issues/3196 and
issue  https://github.com/espressif/esp-idf/issues/6619
in ble mesh device we can set name for unprovisioned device by function esp_ble_mesh_set_unprovisioned_device_name();
BUT
when a device is provisioned and you want to connect to that (in connect menu of nRF-Mesh app) you can see default node name (ESP-BLE-MESH). show it for device name.
and this mean (I think!) the device uprov and proved have different name
and there is another function that you don't explain about that in documents
bt_mesh_gatts_set_local_device_name();
this function set default name for nodes -> (ESP-BLE-MESH)
so I can not use bt_mesh_gatts_set_local_device_name() this function I think I should do something else
anyone know ???!!!

Could you share the code to set the name?
This comment should be noted.

Try esp_ble_mesh_set_unprovisioned_device_name()  after esp_ble_mesh_init() .
this worked for me


Extract links from decoded_body instead of raw body

If a URL in the body of the email was split to multiple lines, URI.extract was not extracting the full URL.

Hi, thank you for the PR. Could you please explain why did you switched from gsub to decoded_body? They are seems to be pretty different things.

Sure thing! The original solution of using gsub to remove "\r\n" handled most instances of links not being parsed out properly, but there were still some Quoted-printable artifacts in the body causing issues (mostly instances of "=3D") so instead of trying to handle everything with gsubbing, I switched to Mail's Body#decoded

I'm going to close this pull request - it was from master of my fork. I want to make it come from a branch on my fork instead, so I can work on other things too.


Enabling OSM Notes and then adding them via the 'Add Note' button fails

Console logs the following message when the map is clicked to add the note:

error: do not call context.selectedNoteID anymore
context2.selectedNoteID @ iD.js:109610
noteSaveSection @ PixiLayerMapillaryPhotos.js:93
call_default @ iterator.js:4
noteEditor @ PixiLayerMapillaryPhotos.js:69
call_default @ iterator.js:4
sidebar.show @ iD.js:105949
enter @ full_screen.js:127
context2.enter @ iD.js:109568
_click @ keepRight_editor.js:86
emit @ index.js:181
_processClick @ duplicate_way_segments.js:383
_pointerup @ duplicate_way_segments.js:254
emit @ index.js:181
InteractionManager2.onPointerComplete @ rapid_poweruser_features_dialog.js:1242
InteractionManager2.onPointerUp @ rapid_poweruser_features_dialog.js:1302

Fixed in 204e8501567703d68a0813669f94968fa78fc1a2.
We had two problems:

BehaviorDraw 'click' events now send the entire event, not just a mouse coord to listeners, so the handler in the ModeAddNote.js needed to update to expect an event, grab its mouse coords, and invert them into a lat/lng coord to make the note.
The layer handling code needed to check the context.selectedIDs method, because context.getSelectedNoteID() is deprecated and generates the above stack trace.


feat: change @elizaos/eliza to @elizaos/core

This PR mass-renames the @elizaos/eliza package to @elizaos/core

An essential change!