soulforger archetype essence powers needs rework and missing res.

https://2e.aonprd.com/Archetypes.aspx?ID=102
so with this archetype under the soulforgers essence powers. what would be useful is if these choices were selectable feats in a feat tree similar to how other feat trees work in the feat page.
like so:

raw you can only choose one type of armament from this list, https://2e.aonprd.com/Rules.aspx?ID=1561
once choose one of those when you go up in level you must select a different catergory so the previous one chosen gets locked out.
currently the list looks like this and there are alot of choices in the essence powers that lack res. all of these abilities only last 1 minute and cant be used until a long rest has occured.

adaptable persona could have the re for choice of move speed with the option of an re for fly speed at lvl 8.  with an re for -10 corruption flaw.
bounding spirit could have the re for gaining a 30ft ranged ability and returning
deep seeded fear missing +2 intimidation checks RE
Harmful Malice adding 1d4 negative to strikes re, missing harm spell link,  corruption flaw missing re for reducing damage by half your level.
heroic heart, +1 status bonus to checks missing re to attack rolls, perception checks, skill checks and saves. missing re for corruption flaw - 1 penatly to attack rolls.
Healing Grace, fast healing re missing to half your level,  missing re to reduce healing by half pcs level.
Magical Resilience (abjuration) Armor or shield only. You gain a +2 status bonus to saving throws and AC against spells. missing re.
Corruption Flaw You take a -1 penalty to saving throws and AC against spells. missing re.
Planar Bond (abjuration) Armor or shield only. When you manifest the essence form, choose one damage type: acid, chaotic, cold, electricity, evil, fire, good, lawful, negative, positive, or sonic. You gain resistance equal to your level + 2 to damage of the selected type. missing re,  Corruption Flaw You gain weakness 1 to all damage. missing re.
Planar Pain (evocation) Weapon only. When you manifest the essence form, choose one damage type: acid, chaotic, cold, electricity, evil, fire, good, lawful, negative, positive, or sonic. Attacks with the weapon deal this type of damage instead of their physical damage with a +2 status bonus to the damage. missing re to select the damage choice.
Corruption Flaw When you deal damage with a weapon or unarmed attack, you take 2 damage of the last type you chose for planar pain, even if the damage type normally wouldn't harm you, such as good damage if you aren't evil. (missing RE)
Pull of Stasis (transmutation) Weapon only. Any time you hit with the soulforged weapon, the target takes a -10-foot penalty to its Speeds for 1 round. (missing re) for the speed.
Corruption Flaw You take a -10-foot penalty to all your Speeds. missing re.
Reflecting Spirit Armor or shield only. You gain a +2 status bonus to AC against physical ranged attacks. (missing RE) If an enemy's physical ranged attack misses you, you can use your reaction to immediately attempt a ranged Strike against the attacker using the projectile that missed. Corruption Flaw You have weakness 5 to physical ranged attacks. (missing RE)
Resolute Defiance Shield only. The shield gains a +2 status bonus to its Hardness and gains temporary Hit Points equal to your level + 2 that last until the essence effect ends (missing re)

There’s this initial work, but it’s on hold until affliction support is in.
https://github.com/foundryvtt/pf2e/pull/5220


draft: adds possibility to add link to card

This pull request adds the possibility in the CMS to add a link to a card in the cardblock.

@antw would you be willing to review this issue? I've been doubting about wrapping a card into a link or making a stretchable link in the card. I couldn't find a conclusive answer online, so for now I've made a conditional wrapper that links the card if a link has been added in the CMS. But I would be very interested in hearing you opinion on this. And of course on any other issue you come across.

I've been doubting about wrapping a card into a link or making a stretchable link in the card.

Personally, I'd use something like a stretched link. Wrap whatever part of the card you want to be clickable in className="relative" then apply the stretched link class to the link.
/* styles/globals.css */

@layer base {
  .stretched-link::after {
    content: "";
    @apply z-10 absolute inset-0 bg-transparent pointer-events-auto;
  }
}

// components/StretchedLink.tsx

function StretchedLink({ content, className, ...linkProps }: React.ComponentProps<'a'>) {
  return (
    // Use "a" or Link as appropriate.
    <a {...linkProps} className={`${className || '' } stretched-link`}>
      {content}

  );
}

// Inside the component

function CardTitle({ condition, children, ...rest }) {
  // Pseudo code
  if (condition) {
    return <StretchedLink href="..." className="mb-3 block">...</StretchedLink>
  }

  return ...;
}

function Card(...)
  return (
    {/* stuff */}

      <CardTitle ...>{cardItem.title}</CardTitle>

        <RawHtml html={cardItem.text} />

    {/* more stuff */
  );
}

@antw Thanks for the feedback.  Good idea to make it a separate component. I've implemented your suggestion. Could you please review it?
There is still a problem in this branch with the migration files, causing a failed check, we are solving this currently.

@antw Sorry, would you mind approving this issue again? We had to change some things in the migration files before it would pass all tests. I've pushed this and now it needs to be approved again. Nothing changed in the code you approved earlier.


Update tedious connection config

Tedious connection config options changed. The docs need to be updated to reflect the new config format.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: David-Engel sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

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

@David-Engel, this pull request is for the live branch. We can't merge to Live. Would you resubmit your changes to the master branch? Thanks.

@jborsecnik I thought that was the process and tried to select master, but there was no master branch in the list:

@jborsecnik Here's my entire list. Which branch should I use?

@David-Engel, for help, see https://docs.microsoft.com/en-us/contribute/.

@David-Engel, for help, see https://docs.microsoft.com/en-us/contribute/.

@jborsecnik I've read that. It's not helpful here. How can I submit to the master branch when it does not exist?
@MightyPen This is the fix for issue #2389. Feel free to take my change and apply it to the private repo.

This fix was transferred to private repo PR 11187.


Build pip package locally using docker

Hello,
I'm trying to build bazel tests locally on my linux machine running Ubuntu 20.04. I tried using docker with the provided Dockerfile and was able to build the container correctly. However, when I tried to run:
bazel test waymo_open_dataset/metrics:all
inside the container, I get the following error:
ERROR: error loading package '': cannot load '//tf:workspace_tf2_7_0.bzl': no such file
INFO: Elapsed time: 0.096s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded)
FAILED: Build did NOT complete successfully (0 packages loaded)

we have not added support for tf2.7 which has upgraded c++ version. You are welcome to add its support if you want.  We may add support for it in the future.

Understood, thanks!


multiple repos for a single website.

I have a site that is built from two github databases.
one of the is the actual website documents. the other is a project code, but we auto-generate documentation on it, and want to keep it up to date.
(currently, we do manual "git clone" of the 2nd site to generate the code for it)
I need to trigger a netlify build when the 2nd repo is modified.

Hi @drortirosh, can you please follow the feature request template?
I'm having a hard time if this is a request for Netlify CMS, or Netlify the platform.
If it's the latter please submit your question in https://answers.netlify.com/

Hi, we believe that you have a problem, but in order for anybody to help you, you need to provide as much detail as possible. The issue template contains the bare minimum of what is required (how to reproduce, expected behaviour, screenshots, etc..).
I'm closing this issue for now. If the problem persists, please open a new issue with all of the details filled out. Also, before posting a new issue, please verify that everything is all right with your environment (computer, internet, browser) and verify whether this issue was already reported by someone else. For general help, please turn to our Slack.


update for 1.1.0

Checklist

[x ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

the auto-rick bot beat me to it.


插件工作一段时间卡死，不更新数据

HA core更新2024.7.1后插件出现问题，ha才启动一段短时间能够正常工作，之后不更新数据。python版本3.12.3，通过esphome的webserver查看培正模块工作正常。ha的报错日志如下：
日志记录器: custom_components.peacefair_energy.modbus
来源: custom_components/peacefair_energy/modbus.py:111
集成: Peacefair Energy Monitor (文档, 问题)
首次出现: 10:53:26 (1101 次总计出现)
上次记录: 13:35:32
Error in gathering, Caught blocking call to sleep with args (0.3,) inside the event loop by custom integration 'peacefair_energy' at custom_components/peacefair_energy/modbus.py, line 88: return self._client.read_input_registers(address, count, **kwargs). (offender: /opt/homeassistant/lib/python3.12/site-packages/pymodbus/transaction.py, line 271: time.sleep(delay)), please create a bug report at https://github.com/georgezhao2010/peacefair_energy/issue For developers, please see https://developers.home-assistant.io/docs/asyncio_blocking_operations/#sleep
Error in gathering, Caught blocking call to sleep with args (2,) inside the event loop by custom integration 'peacefair_energy' at custom_components/peacefair_energy/modbus.py, line 88: return self._client.read_input_registers(address, count, **kwargs). (offender: /opt/homeassistant/lib/python3.12/site-packages/pymodbus/framer/rtu_framer.py, line 215: time.sleep(self.client.comm_params.timeout_connect)), please create a bug report at https://github.com/georgezhao2010/peacefair_energy/issue For developers, please see https://developers.home-assistant.io/docs/asyncio_blocking_operations/#sleep
补充错误日志：
日志记录器: homeassistant.const
来源: helpers/deprecation.py:197
首次出现: 10:38:30 (26 次总计出现)
上次记录: 10:38:43
ELECTRIC_CURRENT_AMPERE was used from peacefair_energy, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfElectricCurrent.AMPERE instead, please report it to the author of the 'peacefair_energy' custom integration
POWER_WATT was used from peacefair_energy, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfPower.WATT instead, please report it to the author of the 'peacefair_energy' custom integration
ENERGY_KILO_WATT_HOUR was used from peacefair_energy, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfEnergy.KILO_WATT_HOUR instead, please report it to the author of the 'peacefair_energy' custom integration
FREQUENCY_HERTZ was used from peacefair_energy, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfFrequency.HERTZ instead, please report it to the author of the 'peacefair_energy' custom integration
TEMP_CELSIUS was used from midea_ac_lan, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfTemperature.CELSIUS instead, please report it to the author of the 'midea_ac_lan' custom integration

我的也是这样，如何解决？


stackage R&D

Research the feasibility of replacing the native schemax.Collection types with go-stackage Stack instances.
Package stackage affords all of the capabilities schemax needs, and its integration would (seemingly) reduce the mass of the code base quite a bit. Additionally, stackage is designed to work seamlessly with type-aliasing, which is absolutely what schemax should do, given the number of types needed.

Implemented by #16


The invisible Connection Error

Bug Report
When I start my bot he cant find any textchannels or voicechats.
Expected Behavior
I want to send a message in a chat, wich I, get it by the getTextChannelById Methode
Code Example or Reproduction Steps
N/A
Code for JDABuilder or DefaultShardManagerBuilder Used

Exception or Error

that's probably because jda is not ready at the time you try to get the channel, either use awaitReady() after .build() or use the ReadyEvent. also, you should've filled out the template thoroughly

that's probably because jda is not ready at the time you try to get the channel, either use awaitReady() after .build() or use the ReadyEvent. also, you should've filled out the template thoroughly

I made 3 while loops too wait for the programm

I made 3 while loops too wait for the programm

lets not spam the issue with discussion here, join the support guild

lets not spam the issue with discussion here, join the support guild

The consoles says it all loaded and then my other code starts

The consoles says it all loaded and then my other code starts

My name on Discord is LPD#6591

My name on Discord is LPD#6591

join the support guild

join the support guild

I am on the discord server, where can i join the support guild?

I am on the discord server, where can i join the support guild?

guild = "server", go to #general

guild = "server", go to #general


Disable character intro in case character tab is closed

The character intro keeps playing in case we parse over a character in character tab and close it before the intro is finished.
https://user-images.githubusercontent.com/35926530/180801016-0f3b270f-68bb-4593-8790-7246b8120ecb.mp4

Done.


Having some trouble

I'm running into a few issues. Have you tested the package since upgrading to the new material-ui?
https://github.com/callemall/material-ui/issues/5573

@acomito not yet, was planning to launch it with the last remaining feature added.

Okay. If I come up with anything helpful I'll report back. Great looking package even if I can't get it working yet!

@acomito haha thank you. Hope you get it working soon. Feel free to ask again if you have any questions or problems.

Tried to use this package again today but no luck. So are the steps to integrating it with a material-ui project anything more than installing from npm and calling:
ReactMaterialUiNotifications.showNotification({
      title: 'Title',
      additionalText: `Some message to be displayed`,
      overflowText<EMAIL_ADDRESS>      icon: <Message />,
    });

Or do I need to put the <ReactMaterialUiNotifications /> component in my root component?

Yes you need to call it and also put that in the react tree preferably as the root component or somewhere else  in the tree.

See this for an implemention example https://github.com/puranjayjain/react-materialui-notifications/blob/master/src/app/Main.js

Feel free to reopen this issue if you have any problems.

Where do have you been putting it in projects using react-router? Do you need something like redux at that point or is the object available globally?

It's not available globally you might need to pass that reference using an event emitter pattern or something equivalent, I haven't had the chance to explore redux at the moment.

Same problem here: i can't get notifications. I've took code(<ReactMaterialUiNotifications> and method calling) from Main.js of the project and nothing happens.

Hi, same problem here.. using React Router

Same problem. No router, have  on the page. Notifications are not shown.

@tiagowippel @akashihi can you give me the link to the place in the project where it is being implemented if it is open source.

@puranjayjain Component is defined here: https://github.com/akashihi/mdg-web-ui/blob/master/src/js/components/Notification.js and used here: https://github.com/akashihi/mdg-web-ui/blob/master/src/js/components/Main.js
Notifications are issued here: https://github.com/akashihi/mdg-web-ui/blob/master/src/js/util/ApiUtils.js#L12

So @akashihi the problem seems to be that the show notifications is not being called in the same component the element is hoisted in. You can either move the showNotification call to Notification.js or create a ref to the notification.js's ReactMaterialUiNotifications component and trigger an action on that ref.

@puranjayjain Well, i'm pretty new to the Js/React world. Could you please provide some intuition, how to implement that ref? Thanks in advance.

It does not work because you have to call a setState or forceUpdate on your own, if not the notifications will not be updated

@DevAlien @akashihi this issue has been fixed in v0.5.0 and released on npm also!


GITHUB hooks support

Is there an easy way to trigger a build when a repo has been pushed to in github

Yep! Just add your concrete URL to the Post-Receive URLs in your repo's service hooks. Whenever github receives your push they'll send a POST to your concrete instance and trigger a new build.

Sweet, yea should probably mention it on the docs.
That's would have saved me 5hrs messing around with Jenkins
Dharmesh
On 7 Feb 2012, at 19:02, Ryan Kee
<EMAIL_ADDRESS>wrote:

Yep! Just add your concrete URL to the Post-Receive URLs in your repo's service hooks. Whenever github receives your push they'll send a POST to your concrete instance and trigger a new build.

Reply to this email directly or view it on GitHub:
https://github.com/ryankee/concrete/issues/8#issuecomment-3853938


Fix problems with BOM encoding in file

Fix #326
Add new rule and closes #327
Closes #316

And of course my git auto fixed the problems I wanted to keep for tests in CLI :D I will try to add files with bom encoding and restart tests.

Codecov Report

Merging #328 (3e7a0c0) into master (78b8303) will decrease coverage by 1.26%.
The diff coverage is 92.30%.

@@            Coverage Diff             @@
##           master     #328      +/-   ##
==========================================
- Coverage   97.68%   96.41%   -1.27%
==========================================
  Files          23       22       -1
  Lines        1854     1871      +17
==========================================
- Hits         1811     1804       -7
- Misses         43       67      +24

Impacted Files
Coverage Δ

robocop/run.py
96.44% <71.42%> (-0.97%)
:arrow_down:

robocop/checkers/comments.py
97.05% <100.00%> (-2.95%)
:arrow_down:

robocop/checkers/misc.py
92.30% <0.00%> (-5.77%)
:arrow_down:

robocop/checkers/documentation.py
96.00% <0.00%> (-4.00%)
:arrow_down:

robocop/checkers/errors.py
94.73% <0.00%> (-2.64%)
:arrow_down:

robocop/utils/utils.py
96.29% <0.00%> (-2.47%)
:arrow_down:

robocop/checkers/__init__.py
90.80% <0.00%> (-2.30%)
:arrow_down:

robocop/checkers/naming.py
97.14% <0.00%> (-1.72%)
:arrow_down:

robocop/checkers/spacing.py
96.92% <0.00%> (-1.54%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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


Implement TRUNCATE TABLE

This PR implements TruncateStmt, and adds the truncate.sql fixture file to the tests.

sorry, I think I did this in the wrong order. Let me try and import your commits and fix the conflicts.

ok, not sure which one you wanted, here is the conflict:
<<<<<<< HEAD
    output.push(this.list(node.relations, ', ', '', 'truncate'));
=======
    output.push(node.relations.map((e) => this.deparse(e, 'truncate')).join(', '));
>>>>>>> truncatestmt

@acarl005 btw — thank you for your awesome work!  you're really improving the library!!! thank you :)


Update sveltekit-environment-variables.md

process.env.key isn't supported anymore. We have to use process.env[key] instead.

Can you share the source of this change?


Improve login time

Description
The API listApis currently has too many redundant calls to the DB, which causes slowness on the ACS login. This PR aims to improve the time of login on ACS by refactoring the classes that implement the APIChecker interface. In my local lab, the login usually took 13 seconds, without login cache. With this new version, the login took around 2 seconds, with no login cache as well.
Fixes: #5266
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[x] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[x] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?
I created multiple accounts with the roles based on the following table. I compared this PR with a version with the unmodified APIChecker. And the number of APIs were returned as expected, as the table shows.

Role Type
<IP_ADDRESS> version
PR version

Root Admin
624
624

Domain Admin
180
180

Read-Only Admin
312
312

Support Admin
202
202

User
268
268

Furthermore, I tried multiple functions through the UI with these accounts, and the APIs allowed were working correctly.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6412 (SL-JID-1639)

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6412 (SL-JID-1672)

@blueorangutan package

Packaging result: :heavy_multiplication_x: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 3625

Packaging result: :heavy_multiplication_x: el7 :heavy_multiplication_x: el8 :heavy_multiplication_x: debian :heavy_multiplication_x: suse15. SL-JID 7

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_multiplication_x: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_multiplication_x: suse15. SL-JID 3635

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan package

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@blueorangutan package

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian :heavy_multiplication_x: suse15. SL-JID 3662

@blueorangutan test

Trillian test result (tid-4405)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 38333 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6412-t4405-kvm-centos7.zip
Smoke tests completed. 98 look OK, 0 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File

@harikrishna-patnala @Pearl1594 can you review the PR and test an env if possible. The changes are in dynamic and project roles checker where we can use your expertise.

This is ready for merging, asking a final review from security PoV cc @harikrishna-patnala

(rekick travis against latest main)


Add support for passing environment variables to Cassandra process

I was looking for a way to pass ENV variable to Cassandra process, similar to EXECUTOR_ARG, however executor only passes JMX_PORT as a env variable to the process.
https://github.com/mesosphere/dcos-cassandra-service/blob/b407b9bc71870cf2c7d8d4b3c37dc5302154d2df/cassandra-executor/src/main/java/com/mesosphere/dcos/cassandra/executor/CassandraDaemonProcess.java#L254
Is there a plan to add such support? if it's not in your roadmap, would you be interested in PR for such feature?

This repo is deprecated and will be archived in one week.  Please see the latest version of Cassandra or DSE for DC/OS:
https://docs.mesosphere.com/service-docs/cassandra/
https://docs.mesosphere.com/service-docs/dse/  (enterprise-only)


[BEAM-2939, BEAM-10057] Ensure that we can process an EmptyUnboundedSource and also prevent splitting on it.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

R: @ihji @chamikaramj
CC: @TheNeuralBit (cherry pick into 2.22)

Is there any test coverage of this?

I suspect test coverage will be internal to Dataflow (hence during import).

I'm doing a manual validation for now.

Got it, thanks

Run Java PreCommit

Test coverage is by existing IOs that enable these features which we don't have enough of in Beam (since it requires portable runners to implement SDF to a greater extent then they do) so supplement it with internal testing in Google.

LGTM. Thanks.
Ran the Kafka test few times and it seems to be working.

ParameterizedUnboundedSourceWrapperTest.testWatermarkEmission[numTasks = 2; numSplits=2] failed with a timeout after 30s. Seems like it could be a related failure and not a flake...
But there's also a lot of precedent for this test flaking with a timeout: https://issues.apache.org/jira/issues/?jql=project %3D BEAM AND text ~ testWatermarkEmission

Run Java PreCommit

Run Java PreCommit


Add the ability to pass a custom SSH pair through to AWS

This was a collaborative effort with @timaa2k .
See https://jira.mesosphere.com/browse/DCOS_OSS-3575.

Codecov Report

Merging #981 into master will decrease coverage by 24.03%.
The diff coverage is 34.28%.

@@             Coverage Diff             @@
##           master     #981       +/-   ##
===========================================
- Coverage     100%   75.96%   -24.04%
===========================================
  Files          28       28
  Lines        1647     1681       +34
  Branches       60       61        +1
===========================================
- Hits         1647     1277      -370
- Misses          0      396      +396
- Partials        0        8        +8

Impacted Files
Coverage Δ

tests/test_dcos_e2e/backends/aws/test_aws.py
74.32% <32.14%> (-25.68%)
:arrow_down:

src/dcos_e2e/backends/_aws/__init__.py
95.78% <42.85%> (-4.22%)
:arrow_down:

tests/test_dcos_e2e/test_legacy.py
0% <0%> (-100%)
:arrow_down:

tests/test_dcos_e2e/test_enterprise.py
0% <0%> (-100%)
:arrow_down:

tests/test_dcos_e2e/test_node.py
0% <0%> (-100%)
:arrow_down:

...cos_e2e/_node_transports/_docker_exec_transport.py
36.17% <0%> (-63.83%)
:arrow_down:

src/dcos_e2e/node.py
63.46% <0%> (-36.54%)
:arrow_down:

src/dcos_e2e/_node_transports/_ssh_transport.py
66.66% <0%> (-33.34%)
:arrow_down:

tests/test_dcos_e2e/test_cluster.py
76.28% <0%> (-23.72%)
:arrow_down:

src/dcos_e2e/cluster.py
94.31% <0%> (-5.69%)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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


Add GH action for stale issues/PRs (#1)

Overview/Summary
Implements actions/stale to help manage stale PR/issues
This PR fixes/adds/changes/removes

Adds actions/stale

Breaking Changes
None identified
Testing Evidence
https://github.com/matt-FFFFFF/terraform-azurerm-caf-enterprise-scale/runs/4300835319?check_suite_focus=true
As part of this Pull Request I have

[x] Checked for duplicate Pull Requests
[x] Associated it with relevant issues, for tracking and closure.
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x] Performed testing and provided evidence.
[ ] Updated relevant and associated documentation.
[ ] Updated the "What's New?" wiki page (located in the Enterprise-Scale repo in the directory: /docs/wiki/whats-new.md)

For discussion on triage call on Friday

Closing in favour of fabric boit


Mounting UNC path via /etc/fstab doesn't work

Windows Version
10.0.22621.3447
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-1
Distro Version
Ubuntu 20.04
Other Software
No response
Repro Steps
/etc/fstab contains
'\\\\pnjnas\Qdownload'  /mnt/Qdownload  drvfs   defaults        0       0

This works from a bash prompt:
sudo mount -t drvfs '\\\\pnjnas\Qdownload' /mnt/Qdownload/

But processing via /etc/fstab fails:
$ sudo mount -a
mount: /mnt/Qdownload: special device '\\\\pnjnas\Qdownload' does not exist.
<3>WSL (162) ERROR: UtilCreateProcessAndWait:687: /bin/mount failed with status 0x2000

dmesg reports:
/mnt/Qdownload: special device '\\\\pnjnas\Qdownload' does not exist.

Expected Behavior
I would expect mount -a to succeed in mounting the UNC path.
Actual Behavior
$ sudo mount -a
mount: /mnt/Qdownload: special device '\\\\pnjnas\Qdownload' does not exist.
<3>WSL (162) ERROR: UtilCreateProcessAndWait:687: /bin/mount failed with status 0x2000

Diagnostic Logs
No response

Here are the logs:
WslLogs-2024-04-21_19-54-22.zip

Diagnostic information
Appx package is not installed

???

Don't use quotes inside fstab. Remove the single quotes around server/share.

This indeed works.
I feel... speechless...
I'd have sworn I'd tried all combinations of quotes / no quotes, escaping / not escaping bacsklashes...
For some reason I didn't.


Why perform one action vs another? (posting to resource journal vs job journal)

Technical documentation isn't really useful - for a complex product - unless it answers the question WHY. For example, on this page, there are two sections at the end, "To post time sheet lines in a resource journal" and "To post time sheet lines in a job journal".  WHY WOULD I PICK ONE VS THE OTHER? In what circumstances should i post time to a resource journal? when should i post to a job journal? Why would I make that decision? How does this affect billing for a project? Does it? Where do I go to find more information?

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

ID: dd49f3e2-3072-8a24-dcba-bc7aaba2625d
Version Independent ID: 00d7ace5-1e2a-7eab-ee6c-de453653635a
Content: Work with Time Sheets for Jobs - Business Central
Content Source: business-central/projects-how-use-time-sheets.md
Service: dynamics365-business-central
GitHub Login: @SorenGP
Microsoft Alias: edupont

Hello,
Thank you for your feedback! I'm sorry to hear that our documentation caused confusion and frustration, since that is definitely not our goal. Rather, the documentation tries to reflect the non-prescriptive nature of the software. Many organizations have a clear preference for how to carry out many tasks, and Dynamics 365 Business Central often supports two or more ways of doing things.
In this particular example, you can post the time sheet lines in two different journals. If you prefer to track cost by resource, then you might prefer to use the resource journal. If you prefer to track cost by project, then using the job journal might make more sense.
You can learn about project management and the Jobs functionality here:
https://docs.microsoft.com/en-us/learn/paths/create-jobs/
https://docs.microsoft.com/en-/learn/paths/calculate-post-job-wip/


sketch 52.1 broken

@morgancheung Have you restarted Sketch after installing?

hmm...no crashes, just does not launch? Could you launch the settings window through Plugins menu? Plugins -> Handy Menu -> Configure...

sketch 52.2 is ok

uhh...it seems that it was the Sketch's bug.


Billing plug-in - View external subscription price models

This change is

Reviewed 175 of 175 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


feat: App Positioning Feature Backend Implementation

Description
Implements backend functionality for App Positioning (Auto Layout) Feature
Fixes #19348
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

JUnit
Manual

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=88d7a976163879d8e2083271bf42086ac2990a4c

Hey Nilansh, I'm going to call this out again, we seem to be adding yet another set of unpublished and published fields in the base type in this PR. Do you think it is worth investing in consolidating this into one type, we're going to end up increasing the burden of migrations in the future at this rate. :)

Hey @nidhi-nair, I completely agree to what you are pointing towards and this has been a conscious decision to ignore the cleanup as part of this PR.
The aggregation of published and unpublished fields is our next top-most priority and will be picked up very soon. Issue reference here: https://github.com/appsmithorg/appsmith/issues/19280
Rest assured, this would probably be the last set of published and unpublished fields going in the base type.
Can you please review this PR, ignoring the cleanup and optimisation for published/unpublished fields at this point?
I will connect with you separately for #19280

/ok-to-test sha=b6193c55675c4c5915f18e8f056149a2ea45a5ee

/ok-to-test sha=200747fbbedf2a6041c47f728a79c4f32b7f4a07

/ok-to-test sha=5052758d4a5845a1d6fd195bdc08a060de913d98

/ok-to-test sha=acf999cd4c57703098bcc4a3548d355afe085015


Treatment Pathways Seemingly Does Not Generate Pathways

Describe the bug
I have generated 4 cohorts that I have passed into Treatment Pathways, and I get this message after running computePathways:
Construct treatment pathways, this may take a while for larger datasets.
Original number of rows: 0
After eraCollapseSize: 0
After combinationWindow: 0
Time needed to execute combination window 0.00844308535257975
Order the combinations.
After filterTreatments: 0
constructPathways done.

I followed the instructions exactly as per the documentation for TreatmentPatterns 2.6.2 and I do not understand why this is not generating any pathways. My cohorts are rather small (only about ~500 with the smallest having only ~100 patients) but I'd imagine this shouldn't matter.
Could it be that I did something wrong with my cohort definitions?
To Reproduce
Here were the cohorts I used:

Target: Depression
Event: Fluoxetine Exposure
Event: Sertraline Exposure
Event: Duloxetine Exposure

And here is the code I used:
library(dplyr)
library(CirceR)
library(DatabaseConnector)
library(tibble)
library(TreatmentPatterns)

Sys.setenv("DATABASECONNECTOR_JAR_FOLDER" = "driver")
connectionDetails <- createConnectionDetails(dbms="postgresql",
                                             server="localhost/synthea",
                                             user="thecedarprince",
                                             password="")
conn <- connect(connectionDetails)

cohorts <- tibble(cohortId = c(1,2,3,4), cohortName = c("Depression","Duloxetine","Fluoxetine", "Sertraline"), type = c("target", "event", "event", "event"))
cohorts <- cohorts  %>% mutate(across(cohortId, as.integer))

test <- computePathways(
  cohorts = cohorts,
  cohortTableName = "cohort",
  connectionDetails = connectionDetails,
  cdmSchema = "omop",
  resultSchema = "omop"
)

Also, if you want to fully reproduce this at all, here is the code I used to generate my cohorts:
# Fill the cohort set using  cohorts included in this
# package as an example
definition <- "depression.json"
cohortName <- tools::file_path_sans_ext(basename(definition))
cohortJson <- readChar(definition, file.info(definition)$size)
cohortExpression <- CirceR::cohortExpressionFromJson(cohortJson)
cohortSql <- CirceR::buildCohortQuery(cohortExpression, options = CirceR::createGenerateOptions(generateStats = FALSE))

# Create the cohort tables to hold the cohort generation results
cohortTableNames <- CohortGenerator::getCohortTableNames(cohortTable = "cohort")
CohortGenerator::createCohortTables(connectionDetails = connectionDetails,
                                                        cohortDatabaseSchema = "omop",
                                                        cohortTableNames = cohortTableNames)
# Generate the cohorts
cohortsGenerated <- CohortGenerator::generateCohortSet(connectionDetails = connectionDetails,
                                                       cdmDatabaseSchema = "omop",
                                                       cohortDatabaseSchema = "omop",
                                                       cohortTableNames = cohortTableNames,
                                                       cohortDefinitionSet = data.frame(cohortId = 1,
                                                           cohortName = cohortName,
                                                           sql = cohortSql,
                                                           stringsAsFactors = FALSE))

# Fill the cohort set using  cohorts included in this
# package as an example
definition <- "duloxetine.json"
cohortName <- tools::file_path_sans_ext(basename(definition))
cohortJson <- readChar(definition, file.info(definition)$size)
cohortExpression <- CirceR::cohortExpressionFromJson(cohortJson)
cohortSql <- CirceR::buildCohortQuery(cohortExpression, options = CirceR::createGenerateOptions(generateStats = FALSE))

# Generate the cohorts
cohortsGenerated <- CohortGenerator::generateCohortSet(connectionDetails = connectionDetails,
                                                       cdmDatabaseSchema = "omop",
                                                       cohortDatabaseSchema = "omop",
                                                       cohortTableNames = cohortTableNames,
                                                       cohortDefinitionSet = data.frame(cohortId = 2,
                                                           cohortName = cohortName,
                                                           sql = cohortSql,
                                                           stringsAsFactors = FALSE))

# Fill the cohort set using  cohorts included in this
# package as an example
definition <- "fluoxetine.json"
cohortName <- tools::file_path_sans_ext(basename(definition))
cohortJson <- readChar(definition, file.info(definition)$size)
cohortExpression <- CirceR::cohortExpressionFromJson(cohortJson)
cohortSql <- CirceR::buildCohortQuery(cohortExpression, options = CirceR::createGenerateOptions(generateStats = FALSE))

# Generate the cohorts
cohortsGenerated <- CohortGenerator::generateCohortSet(connectionDetails = connectionDetails,
                                                       cdmDatabaseSchema = "omop",
                                                       cohortDatabaseSchema = "omop",
                                                       cohortTableNames = cohortTableNames,
                                                       cohortDefinitionSet = data.frame(cohortId = 3,
                                                           cohortName = cohortName,
                                                           sql = cohortSql,
                                                           stringsAsFactors = FALSE))

# Fill the cohort set using  cohorts included in this
# package as an example
definition <- "sertraline.json"
cohortName <- tools::file_path_sans_ext(basename(definition))
cohortJson <- readChar(definition, file.info(definition)$size)
cohortExpression <- CirceR::cohortExpressionFromJson(cohortJson)
cohortSql <- CirceR::buildCohortQuery(cohortExpression, options = CirceR::createGenerateOptions(generateStats = FALSE))

# Generate the cohorts
cohortsGenerated <- CohortGenerator::generateCohortSet(connectionDetails = connectionDetails,
                                                       cdmDatabaseSchema = "omop",
                                                       cohortDatabaseSchema = "omop",
                                                       cohortTableNames = cohortTableNames,
                                                       cohortDefinitionSet = data.frame(cohortId = 4,
                                                           cohortName = cohortName,
                                                           sql = cohortSql,
                                                           stringsAsFactors = FALSE))

Desktop (please complete the following information):

OS: Fedora 38
Browser [e.g. chrome, safari]
Version [e.g. 22]

Thank you for reaching out!
It could be a couple things:

Subjects with depression (target) do not occur in your treatment cohorts (events).
Treatments happen before the occurrence of depression.

I'd like to verify a couple things:

Do records exist within the cohort table? Like you said we'd expect this to be between ~100 and 500.

DatabaseConnector::renderTranslateQuerySql(
  connection = conn,
  sql = "SELECT cohort_definition_id, COUNT(*) FROM omop.cohort GROUP BY cohort_definition_id;"
)

Do people in your event cohorts also exist in your target cohorts? You can change the cohort_definition_id per event cohort.

# Duloxetine in Depression
DatabaseConnector::renderTranslateQuerySql(
  connection = conn,
  sql = "
  SELECT subject_id
  FROM omop.cohort
  WHERE cohort_definition_id = 2 AND subject_id IN (
    SELECT subject_id
    FROM omop.cohort
    WHERE cohort_definition_id = 1
  );"
)

Hey @mvankessel-EMC,
I went through your debugging suggestions and here are my findings:

Do records exist within the cohort table? Like you said we'd expect this to be between ~100 and 500.

This was a great thought. I checked my cohorts and they were all 100+.

Do people in your event cohorts also exist in your target cohorts? You can change the cohort_definition_id per event cohort.

Sure enough, there were two events that had close to 0 patient overlaps. I removed those and these are now my patient counts with 200+ overlaps:
 cohort_definition_id | count
----------------------+-------
                    1 |   291
                    2 |   424

Treatments happen before the occurrence of depression.

I did a quick analysis and can confirm that the treatments do happen for my patients within the time period that they are part of the cohort (i.e. they are prescribed the drug I am considering as an event).

After all this, I tried only having one target and event cohort (with 100's of overlaps) but I still do not see any pathways generating. What else should I be trying or checking? Do you think I did something wrong with my definitions? The ones I am using now are the Depression one and the Fluoxetine Exposure
Thanks!

I don't think it is the cohort definitions, as subjects for both cohorts show up in your cohort table. TreatmentPatterns is completly agnostic as to how you define your cohorts, as long as they pick up subjects for your cohort table.
I took another look at the logs TreatmentPatterns spat out during your initial run:
construct treatment pathways, this may take a while for larger datasets.
Original number of rows: 0

The fact that this is being printed means that there are records in your cohort table, but lost along the way when asserting that the start date of the target cohort is earlier than the start date of the event cohorts. And that the start date of the event cohort is earlier than the end date of the target cohort. To put it a little but more programmatic:
# targetStartDate: 2021-03-06
# eventStartDate: 2021-06-01
# targetEndDate: 2022-12-31

targetStartDate < eventStartDate AND eventStartDate < targetEndDate

This is because includeTreatments = "startDate". You could try to set it to "endDate", and alter your periodPriorToIndex.

Hi @mvankessel-EMC
Hooray! After fiddling a bunch with your suggestions on periodPriorToIndex, I was able to get pathways generated! This was the code that ended up working:
test <- computePathways(
  cohorts = cohorts,
  cohortTableName = "cohort",
  maxPathLength = 5,
  connectionDetails = connectionDetails,
  cdmSchema = "omop",
  resultSchema = "omop",
  includeTreatments = "startDate",
  periodPriorToIndex = 18250
)

Out of curiosity, why do you think I had to set the periodPriorToIndex at all? I was under the impression that pathways would be generated by:

Look at my cohorts
Find when each patient starts in a cohort
Begin construction of pathways with respect to each patient entry
Look at occurrences of drug of interest per overlapping patient cohort
Generate pathways

But it seems like that is not truly how the algorithm underlying TreatmentPatterns works?
P.S. I'll close this issue for now since my problem is solved, but thanks for all the help! :smile:

Looking back at your previous comment:

I did a quick analysis and can confirm that the treatments do happen for my patients within the time period that they are part of the cohort (i.e. they are prescribed the drug I am considering as an event).

Do the event cohorts start during the target cohort? Or do they start before the target start date and look back and occur during the target cohort?
TreatmentPatterns assumes treatments start within the bounds of the target cohort for a patient.

In this image, event 3 would not get picked up at all, unless the periodPriorToIndex is extended to i.e. 10.
I can point you to the original article that includes this image:

Maybe this will clear up some things as to how the inclusion of events work.


intersect -> intersection with deprecation

Forgot to use the OGC standard and GeoInterface.jl function name for this...

Codecov Report

Merging #10 (3e4fa1b) into main (c7453f0) will not change coverage.
The diff coverage is 33.33%.

@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage   82.55%   82.55%
=======================================
  Files           1        1
  Lines          86       86
=======================================
  Hits           71       71
  Misses         15       15

Impacted Files
Coverage Δ

src/Extents.jl
82.55% <33.33%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Better system chaining / piping

What problem does this solve or what need does it fill?
There are many different methods to control the execution flow of a system set. I think many of them could be unified into an easy and intuitive API. For example, "piping" and "chaining" systems is essentially the same thing: you could look at "chaining" as piping () . Ideally, these two concepts should be merged into a nice and consistent API.
What solution would you like?
I think we can achieve a really nice looking API by utilizing the Shr (or Shl) traits.
For example:
Simple chaining
// Before:
(sys1, sys2, sys3).chain()

// After:
sys1 >> sys2 >> sys3

Simple piping
// Before:
sys1.pipe(sys2).pipe(sys3)

// After:
sys1 >> sys2 >> sys3

Complex chaining + piping
// Before:
(((sys1, sys2, sys3).chain(), sys4, sys5.pipe(sys6)), sys7).chain()

// After:
(sys1 >> sys2 >> sys3, sys4, sys5 >> sys6) >> sys7

What alternative(s) have you considered?
Keep the API as is
Additional context

We can also use declarative macros. That would offer more flexibility, but it will be harder to implement and maintain (and it's not guaranteed to make the API feel better)
This could open the gate for "1-N piping" (piping the output of a system into many systems)

If this is acceptable, I would like to work on the implementation.

Not a fan of using operator overloading like this. This doesn't behave how I would think a right shift operator should work.

Chaining and piping fundamentally don't work the same way. Piping is a "merging" of systems: they become one single unit, and are evaluated as a single system. Chaining is a set of .before / .after ordering; the systems retain their identity, their accesses stay disjoint and other systems can run in between them.

Not a fan of using operator overloading like this. This doesn't behave how I would think a right shift operator should work.

I'm not sure I completely understand. But would it help if the suggestion was optional, and everyone could choose whether to use 'chain' and 'pipe', or '>>'? After playing around with it I found it quite intuitive and easier on the eyes.

Chaining and piping fundamentally don't work the same way. Piping is a "merging" of systems: they become one single unit, and are evaluated as a single system. Chaining is a set of .before / .after ordering; the systems retain their identity, their accesses stay disjoint and other systems can run in between them

I'm aware that chaining and piping is different internally. But in my opinion, the difference in implementation should not necessarily reflect a different API for each. Under the hood '>>' will 'chain' systems with no output/ input, and it will 'pipe' systems with.
What do you think about adding this as an option, and not as a replacement?

Syntax wise I'm not sure the >> syntax is possible. Both sides of the operator generally have a function item type, which are not local to Bevy, so the Shr/Shl traits cannot be implemented.

Syntax wise I'm not sure the >> syntax is possible. Both sides of the operator generally have a function item type, which are not local to Bevy, so the Shr/Shl traits cannot be implemented.

That's true. I didn't realize that.


Can't open connection dialog

When pressing F1 and choosing "IBMi: Connect" I get the following error:
Command 'IBM i: Connect' resulted in an error
(command 'code-for-ibmi.connect' not found)
code-for-ibmi 0.0.6, VS Code 1.53.2 on linux

@dferrand That is certainly an odd issue. I am afraid you might have to debug that one yourself since I cannot recreate it.
https://github.com/halcyon-tech/code-for-ibmi#run-from-local

I have the same issue.  I saw this on the Midrange email list today and did the following steps.

Downloaded and installed VS Code (Version 1.53.2)
Installed Code for IBMi Extension
Installed IBMi Languages Extension
Tried following instructions on main page of this project where is says to press F1 and find 'IBM i: Connect'

When doing that, nothing shows up in the find result.

I think I solved it.  The 2 extensions I mentioned above appear to be for code highlighting, syntax, etc.
The extension that you need for connections and such is RPG for IBM i here:
https://marketplace.visualstudio.com/items?itemName=HalcyonTechLtd.code-for-ibmi
After installing that, I do see the connection option.

@dferrand @getfoundational I submitted a PR (https://github.com/halcyon-tech/code-for-ibmi/pull/15) that should resolve this problem on Linux (tested with Ubuntu 20.04.2), it seems to be a small file naming issue that didn't wasn't effecting Windows

Yeah found it too.
File is src/Instance.js and extension.js references src/instance.
It works on windows because the filesystem is not case sensitive


Basic Websocket Test

Adds a test for the websocket server. I needed to write some tests myself and this was the best I could come up with. Had to use the httparse::Headers so this is a bit more verbose than I would prefer, nothing else seemed to work for me 🤷 . Uses the lunatic::TcpStream for the tungstenite client.

This looks great! Though it seems like theres some linting issues causing the GH checks to fail

@tqwewe Sure, will format.

Thanks for the contribution @mpope9!


broken window screensharing in armcord flatpak and nix version of webcord-vencord in arch linux

Hyprland Version
hyprland-git 0.33.1.r44.79d8d14f-1
Bug or Regression?
Bug
Description
when I screenshare specifically a window hyprland will crash in both flatpak armcord and nix webcord-vencord
tested with stable hyprland it didn't crash there but it was a black screen
these issues don't exist on the aur version of both webcord and vencord
How to reproduce
enter any of the formentioned app and screenshare a window
screensharing a region or the screen works
Crash reports, logs, images, videos
hyprlandCrashReport6819.txt
hyprlandCrashReport8468.txt
one of them is webcord the other is armcord hopefully both of them point to the same bug
yeah peace and love

I want to point out that if this is a problem with flatpak or nix since no problem in the aur
at least I expect hyprland to not crash until session restore comes around at least

check with above

@vaxerski wasn't fixed sadly
hyprlandCrashReport23171.txt
hyprlandCrashReport28667.txt

check with above

@vaxerski its now fixed thanks a lot ヽ(´▽`)/


Regression: @primer/react is depends on esm only module @github/combobox-nav

Describe the bug
This is a similar problem to #2238. When I try to use the latest @primer/react in the Next.js app, I get the following error:
> next build

info  - SWC minify release candidate enabled. https://nextjs.link/swcmin
info  - Linting and checking validity of types
info  - Creating an optimized production build
info  - Compiled successfully
info  - Collecting page data ...Error [ERR_REQUIRE_ESM]: require() of ES Module<EMAIL_ADDRESS>from<EMAIL_ADDRESS>not supported.
Instead change the require of index.js in<EMAIL_ADDRESS>to a dynamic import() which is available in all CommonJS modules.
    at Object.<anonymous> (/Users/rokumura/work/tests/test-20220819/my-app/node_modules/@primer/react/lib/drafts/hooks/useCombobox.js:8:43)
    at Object.<anonymous> (/Users/rokumura/work/tests/test-20220819/my-app/node_modules/@primer/react/lib/drafts/InlineAutocomplete/_AutocompleteSuggestions.js:16:20)
    at Object.<anonymous> (/Users/rokumura/work/tests/test-20220819/my-app/node_modules/@primer/react/lib/drafts/InlineAutocomplete/InlineAutocomplete.js:22:55)
    at Object.<anonymous> (/Users/rokumura/work/tests/test-20220819/my-app/node_modules/@primer/react/lib/drafts/InlineAutocomplete/index.js:8:50)
    at Object.<anonymous> (/Users/rokumura/work/tests/test-20220819/my-app/node_modules/@primer/react/lib/FormControl/FormControl.js:28:50)
    at Object.<anonymous> (/Users/rokumura/work/tests/test-20220819/my-app/node_modules/@primer/react/lib/FormControl/index.js:13:43)
    at Object.<anonymous> (/Users/rokumura/work/tests/test-20220819/my-app/node_modules/@primer/react/lib/index.js:669:43)
    at Object.551 (/Users/rokumura/work/tests/test-20220819/my-app/.next/server/pages/_app.js:22:31)
    at __webpack_require__ (/Users/rokumura/work/tests/test-20220819/my-app/.next/server/webpack-runtime.js:25:42)
    at __webpack_exec__ (/Users/rokumura/work/tests/test-20220819/my-app/.next/server/pages/_app.js:51:39)
    at /Users/rokumura/work/tests/test-20220819/my-app/.next/server/pages/_app.js:52:28
    at Object.<anonymous> (/Users/rokumura/work/tests/test-20220819/my-app/.next/server/pages/_app.js:55:3)
    at Object.requirePage (/Users/rokumura/work/tests/test-20220819/my-app/node_modules/next/dist/server/require.js:58:12)
    at /Users/rokumura/work/tests/test-20220819/my-app/node_modules/next/dist/server/load-components.js:55:50
    at async Promise.all (index 1)
    at async Object.loadComponents (/Users/rokumura/work/tests/test-20220819/my-app/node_modules/next/dist/server/load-components.js:53:49)
    at async /Users/rokumura/work/tests/test-20220819/my-app/node_modules/next/dist/build/utils.js:673:21
    at async Span.traceAsyncFn (/Users/rokumura/work/tests/test-20220819/my-app/node_modules/next/dist/trace/trace.js:79:20) {
  code: 'ERR_REQUIRE_ESM'
}

> Build error occurred
Error: Failed to collect page data for /
    at /Users/rokumura/work/tests/test-20220819/my-app/node_modules/next/dist/build/utils.js:743:15
    at processTicksAndRejections (node:internal/process/task_queues:96:5) {
  type: 'Error'
}

I am guessing that @github/combobox-nav is the ESM only module.
To Reproduce
Steps to reproduce the behavior:

Create Next.js + TypeScript app via create-next-app

Edit package.json as follows:
{
  "name": "my-app",
  "version": "0.1.0",
  "private": true,
  "scripts": {
    "dev": "next dev",
    "build": "next build",
    "start": "next start",
    "lint": "next lint"
  },
  "dependencies": {
    "@primer/react": "35.7.0",
    "next": "12.2.5",
    "react": "17.0.2",
    "react-dom": "17.0.2"
  },
  "devDependencies": {
    "@types/node": "18.7.6",
    "@types/react": "17.0.48",
    "@types/react-dom": "17.0.17",
    "eslint": "8.22.0",
    "eslint-config-next": "12.2.5",
    "typescript": "4.7.4"
  }
}

Edit pages/_app.tsx as follows:
import type { AppProps } from 'next/app'
import { SSRProvider } from '@primer/react'

function MyApp({ Component, pageProps }: AppProps) {
  return (
    <SSRProvider>
      <title>Test</title>
    </SSRProvider>
  )
}

export default MyApp

Run npm run build command

Expected behavior
This error does not occur when using<EMAIL_ADDRESS>so I expect the latest version work as well.
Screenshots
N/A
Desktop (please complete the following information):
N/A
Smartphone (please complete the following information):
N/A
Additional context
N/A

I can verify that I am experiencing the same issue

Hi, thanks for the report. It looks like we'll have to make some upstream changes in several dependencies:

@github/combobox-nav
@github/markdown-toolbar-element
@github/paste-markdown

Unfortunately that's a slower process than the fix for #2238 and may take some time. In the meantime, I would recommend staying locked to version 35.5.0 until we can get this resolved.

Looks like a few other primer related things might be an issue in next
Portals create a div on every render, during render
Primer behaviors isMacOs reads window.navigator without guarding against typeof window === 'undefined'
useLayoutEffect instead of useIsomorphicLayoutEffect to avoid hydration warnings

I have created a Next.js app using yarn create next-app my-app (without typescript) and am having this same issue. Any fix for this?

I haven't used this before, or tried to setup with primer, but it looks like there's a bit of nextjs specific tooling for handling these transforms. https://www.npmjs.com/package/next-transpile-modules
that might be the best option temporarily if you need a CJS build of primer >= 35.6.0

Was the portal issue already a problem before the MarkdownEditor work? I don't think that work introduced anything unique or new regarding portals, though I could be wrong.
The second two issues are definitely new since the MarkdownEditor work though. Maybe the best strategy for now is to hasten the extraction of experimental components out of this package. That would at least prevent us from introducing breaks like this while experimenting.

There is some resistance (Slack thread) from the @github/web-systems team to adding the CommonJS build to combobox-nav (https://github.com/github/combobox-nav/pull/63) because they would rather not keep supporting a legacy system and instead focus on ESM.

There is some resistance (Slack thread) from the @github/web-systems team to adding the CommonJS build to combobox-nav (github/combobox-nav#63) because they would rather not keep supporting a legacy system and instead focus on ESM. Not relevant to the other SSR issues Matt mentioned above, but crucial for this particular issue.

Curious to know whether primer/react would work with just the esm build in next.

Curious to know whether primer/react would work with just the esm build in next.

I believe it would. I don't have any context to know if the non-esm versions are required elsewhere.
If my understanding is correct, this should be broken for any users of @primer/react who are either in a node environment, or are using require("@primer/react").
Note that this works for web environments using import, because they don't match the "node" condition:
https://github.com/primer/react/blob/1e860b0d32b959d11e17fc8aa01604a21d890dd0/package.json#L7-L33

Hi there! 👋
This should be addressed in latest (v35.12.0) through updates to the "exports" field and the CommonJS entrypoint. The CommonJS entrypoint will be an interim fix until the project moves to ESM-only.
Going to close this out as a result, feel free to comment and I can re-open if this is still an issue!


merge with original project

Hey, maybe you could contact original maintainer and merge those projects?

Hi, I'm not sure why I would want to do that. They gave me access to the NPM package which is the important thing. It makes sense for the GitHub to be under my account rather than theirs because they are no longer involved. (I was the primary maintainer for some time while it was under their account, but they chose to archive it, which is their choice)


Update README.md

fixup function declarations. endpoints.Context doesn't exist, but context.Context does.

signed!
On Sun, Aug 21, 2016 at 12:21 AM googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project. Before we can look at your
pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data https://cla.developers.google.com/clas and verify
that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please let us know the
company's name.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/go-endpoints/pull/147#issuecomment-241242991,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAV5vBnqCRAsBieGSO8E6SW4lXzckFxWks5qh_xYgaJpZM4JpPj-
.

Hey thanks for the PR!
Just merged this change via a different PR: https://github.com/GoogleCloudPlatform/go-endpoints/pull/124
Cheers :D


Try installing extra deps such that bcrypt and cryptography can be installed in arm/v7

Docker build is currently failing

@basnijholt I have the home assistant docker image cloned already, how can I use it to execute the tests locally?

This is not the same as the Home Assistant image.
Until the build passes (currently it was failing in ARM v7 (so e.g., Raspberry Pi)) you would need to run the build yourself. And then run the image.


[Suggestion] It would be better to change log to "Delete slot 'xxx' Succeeded" in the ACTIVITYLOG window after deleting a slot

OS: Windows 10
Build Version: 20220513.10
Repro Steps:

Expand a web app.
Right click a slot -> Click "Delete Slot...".
Check the log message in the ACTIVITYLOG window.

Actual:
There is a "Delete web app 'xxx' Succeeded" log in the ACTIVITYLOG window.

Suggestion:
It would be better to change log to "Delete slot 'xxx' Succeeded" in the ACTIVITYLOG window after deleting a slot.

Issue also exists in Functions https://github.com/microsoft/vscode-azurefunctions/issues/3148


Break up the GetUserPools query

The getUserPools query takes >5s on my laptop.
This seems attributable to this function doing way too much and having a sub-optimal caching + server side vs client side strategy. https://github.com/osmosis-labs/osmosis-frontend/blob/4b0eab7475a7596efae19545de629a4d930c5ce9/packages/server/src/queries/complex/pools/user.ts#L31-L146
We can get each of the pool types independently. Furthermore some of this data already is known locally, e.g. queryAccountPositions

I think migration to SQS pools query, which includes a list of pool IDs as input, greatly improves response time since we no longer have to filter over all pools


Can't list all version to use like nvm

In nvm, we've a have a nvm list to list all version to use. Will be a great command to have.

This command already exists.

Only for installed version. Will be good a command for avaliable versions which you don't have.

npm list available.


fix: Leave combine/separate elements in index order - not alphabetica…

…l order.
Fixes: #624

one lint error
ERROR: /workspace/ui/src/app/lib/atlasmap-data-mapper/components/mapping/mapping-list.component.ts[73, 9]: Identifier 'fields' is never reassigned; use 'const' instead of 'let'.

ugh - sorry - will fix
--P
On Fri, Oct 19, 2018 at 8:58 AM Tomohisa Igarashi<EMAIL_ADDRESS>wrote:

one lint error
ERROR:
/workspace/ui/src/app/lib/atlasmap-data-mapper/components/mapping/mapping-list.component.ts[73,
9]: Identifier 'fields' is never reassigned; use 'const' instead of 'let'.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/atlasmap/atlasmap/pull/637#issuecomment-431354395,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC1-Lb86Aws7UF2sYU5JAgfKQibK-vctks5umcxqgaJpZM4XvQBY
.


Loose Configured URL and ApiToken Using Spring Cloud Starter Kubernetes Config

If you drop the following dependency in the samples application, then even if you configure the following properties, it always generates a new key and uses the developer URI.  I found this behavior in my app and was able to reproduce in the sample app in this repo.  I expect many people will want to use both spring cloud kubernetes and wavefront.  If I remove the depenency and keep the properties in my sample app application.properties, then a new token is not generated.  That is the expected result I want with the dependency.
management.metrics.export.wavefront.api-token=<YOUR_KEY>
management.metrics.export.wavefront.uri=https://surf.wavefront.com
	<dependency>
		<groupId>org.springframework.cloud</groupId>
		<artifactId>spring-cloud-starter-kubernetes-config</artifactId>
		<version>1.1.2.RELEASE</version>
	</dependency>

Thanks for the report, this is a duplicate of #24


WMCore-09-2016-issues

Bug fixes high priority:
#7126 WQE created with wrong priority (S)
#7114 ExecutableNotFound not really informative (S)
#6791 Aborted workflows acquiring WQE and creating/submitting jobs (S)
#6710 force complete not completing (S)
#6187 ACDC for TaskChain wfs creating more jobs than expected (A)
#6078 MCFakeFile IntegrityError in LQ (A) (already in production - unittest to be fixed)
#5809 WQ trying to abort workflows in aborted-archived (S)
#7012 CMSSW Release used for merges within TaskChain (A)
#7115 Files are not injected to PhEDEx but DBS is updated. (S)
#7031 fix the bottleneck on creating jobs due to previous subscription creation (S)
#7130 ReqMgr2 parameters validation fix (A)
#7132 Memory is inactive in recovery procedure
High Priority:
#6868 Expose the site of failure for SubmitFail (S)
#4851 show when a merge job skips a file (S)
#7134 Create a new api which give statistics for by each exit code
#5631 redo abort and force complete (S)
#6789 Change abort workflow to complete ALL subscriptions
#6787 Stepchain: sort out duplicate outputModules (A)
#6941 AcquisitionEra/ProcessingString for TaskChain Merge ACDC (A) (testing)
#6932 ReqMgr2: changes to view/edit fields in web UI (A)
#6931 Request assignment to > 1 team should fail (A)
#7058 ReqMgr: make only editable field in reqmgr2 GUI. JSON format will have all the field.(S) done
Bug fixes medium priority:
#7118 Combine threshold for Processing and Production job type (S)
#7127 Update condor version and build new wmagent release (D)
#6907 memoryRequirement should not be set for utilitarian job types
#6830 Differentiate error code for time out
#6906 Upload new values to Multicore, Memory and Performance params to workload doc
#6881 ReqMgr2: filter out some keys before persisting them in the reqmgr2 couch doc
#6952 Create outputmodule validate function based on its length (A)
#6845 getChunkInfo doesn't assumes all the files in that chunk has the same location (A)
#7133 Make a list of accepted parameters for creation and another for assignment
#7123 calculate the submit threshold by priority (S)
#7121 Review Lexicon rules (procdataset, acqname, etc) (A)
#7119 Changing pendingSlotsSitePercent takes no effect in ASW (A)
#4966 Adding tasks info into RC fails silently (A)

@ticoann @amaltaro here is the list of Ops prefered items
Vital:
#7138 TrusPUSiteList=True not functionning in ACDC assignment
=> because we cannot run ACDC for premix DR80.
Agent Stability Bundle:
I actually did not find GH issue about any of the crashes that happen daily/weekly and prevent us from smooth operation. Or have I missed it ?
#7XYZ Report crashes to wmstats stack-trace and logs from the agents for dev inspection
Then in decreasing priority:
Error Report Bundle:
#7115 Files are not injected to PhEDEx but DBS is updated. (S)
#7114 ExecutableNotFound not really informative (S)
#6868 Expose the site of failure for SubmitFail (S)
#4851 show when a merge job skips a file (S)
#7012 CMSSW Release used for merges within TaskChain (A)
=> difficult to understand what happened to workflows and increase latency in making ACDCs
#7132 Memory is inactive in recovery procedure
=> difficulty to efficiently create recovery/ACDC with increase memory requirement without risking to be killed by local queues. Mitigated by HTcondor mem rounding
Priority Bundle:
#7126 WQE created with wrong priority (S)
#7123 calculate the submit threshold by priority (S)
=> otherwise work does not get done in the correct order.
Cherry on top:
#6787 Stepchain: sort out duplicate outputModules (A)

List of issues for October created in #7275


some simple string parsing for ds9 regions files

This is still extremely rough code. Working through the grammer of ds9 regions file to create region objects.

@keflavich @joleroi @cdeil @sosey - what is the status of this, is it superseded by the code we've already merged in?

you can ignore my pull request, that was just early playing around...


Potential documentation issue on aws-otel.github.io site

Hello
I believe there might be a documentation issue in this section of the aws-otel site: Collecting infrastructure metrics
The section says: ADOT Collector uses the AWS Container Insights Receiver and the link leads to awscontainerinsightreceiver GitHub folder however, below in a list of supported platforms, the following bullet - Amazon ECS with cluster- and service-level metrics, leads to a site section describing the usage of a completely different receiver - awsecscontainermetrics.
Is that intentional or a documentation problem? Can awscontainerinsightreceiver actually be used with AWS ECS to collect cluster and service level metrics (e.g. for Fargate)?
Cheers,
Marcin

Hi @marcinbelczewski thanks for flagging! We'll take a look and fix.

Hi @marcinbelczewski can you please file an issue on the ADOT docs site at https://github.com/aws-otel/aws-otel.github.io/issues Thanks!

Hi @alolita, thank you for replying. I have just created the issue as suggested: https://github.com/aws-otel/aws-otel.github.io/issues/252
Cheers,
Marcin

We are tracking this in the website repo.


fix: typos

Related Issue

Information about the related issue

Closes: #73
Describe the changes you've made

Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist

[ ] My code follows the guidelines of this project.
[ ] I have performed a self-review of my own code.
[ ] I have commented on my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

Screenshots (if applicable)

Original
Updated

original screenshot
updated screenshot

Code of Conduct

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

This is a duplicate of #73, sorry.

@Panquesito7 I think the label should be "duplicate" not "invalid"

@Panquesito7 I think the label should be just "duplicate" not "invalid"

Yep wait, I'll correct that :)


internal: ceph-deploy may have ceph-create-keys INFO in syslog

This is not a concern for error and should be ignored.
Signed-off-by: Loic Dachary<EMAIL_ADDRESS>

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1812/
Test PASSed.


Unable to insatll on Windows

I have installed all pre requisites and trying to install project however not able to install as ruby 2.4.2 is using JSON 2.04
I am getting following error and wonder how i can fix JSON issue?  ruby gem list is also after the error
//***********************************************************************
ERROR
bundle install
Fetching gem metadata from https://rubygems.org/............
Fetching gem metadata from https://rubygems.org/..
Resolving dependencies...........
Using rake 11.2.2
Using i18n 0.7.0
Fetching json 1.8.3
Installing json 1.8.3 with native extensions
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.
current directory:
C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/json-1.8.3/ext/json/ext/generator
C:/Ruby24-x64/bin/ruby.exe -r ./siteconf20171101-13364-1152m0u.rb extconf.rb
creating Makefile
current directory:
C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/json-1.8.3/ext/json/ext/generator
make "DESTDIR=" clean
current directory:
C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/json-1.8.3/ext/json/ext/generator
make "DESTDIR="
generating generator-x64-mingw32.def
compiling generator.c
generator.c: In function 'generate_json':
generator.c:861:25: error: 'rb_cFixnum' undeclared (first use in this function);
did you mean 'mFixnum'?
} else if (klass == rb_cFixnum) {
^~~~~~~~~~
mFixnum
generator.c:861:25: note: each undeclared identifier is reported only once for
each function it appears in
generator.c:863:25: error: 'rb_cBignum' undeclared (first use in this function);
did you mean 'rb_cFixnum'?
} else if (klass == rb_cBignum) {
^~~~~~~~~~
rb_cFixnum
make: *** [Makefile:242: generator.o] Error 1
make failed, exit code 2
Gem files will remain installed in
C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/json-1.8.3 for inspection.
Results logged to
C:/Ruby24-x64/lib/ruby/gems/2.4.0/extensions/x64-mingw32/2.4.0/json-1.8.3/gem_make.out
An error occurred while installing json (1.8.3), and Bundler cannot
continue.
Make sure that gem install json -v '1.8.3' succeeds before bundling.
In Gemfile:
rails was resolved to 4.2.7, which depends on
actionmailer was resolved to 4.2.7, which depends on
actionpack was resolved to 4.2.7, which depends on
actionview was resolved to 4.2.7, which depends on
rails-dom-testing was resolved to 1.0.7, which depends on
rails-deprecated_sanitizer was resolved to 1.0.3, which depends on
activesupport was resolved to 4.2.7, which depends on
json
//***********************************************************************
Here is my Ruby gem list
actioncable (5.1.4)
actionmailer (5.1.4)
actionpack (5.1.4)
actionview (5.1.4)
activejob (5.1.4)
activemodel (5.1.4)
activerecord (5.1.4)
activesupport (5.1.4)
addressable (2.4.0)
arel (8.0.0, 6.0.3)
awesome_print (1.7.0)
bcrypt (3.1.11 x64-mingw32)
bigdecimal (default: 1.3.0)
builder (3.2.3, 3.2.2)
bundler (1.16.0.pre.3, 1.15.4)
chronic (0.10.2)
coderay (1.1.1)
coffee-script-source (1.10.0)
concurrent-ruby (1.0.5, 1.0.2)
cookiejar (0.3.3)
crass (1.0.2)
daemons (1.2.4)
database_cleaner (1.5.3)
did_you_mean (1.1.0)
diff-lcs (1.2.5)
eco-source (1.1.0.rc.1)
ejs (1.1.1)
erubi (1.7.0)
erubis (2.7.0)
eventmachine (<IP_ADDRESS> x64-mingw32)
excon (0.51.0)
execjs (2.7.0)
ffi (1.9.14 x64-mingw32)
formatador (0.2.5)
globalid (0.4.1)
guard-compat (1.2.1)
hashdiff (0.3.0)
hashie (2.1.2)
http_parser.rb (0.6.0)
httpauth (0.2.1)
i18n (0.9.0, 0.7.0)
io-console (default: 0.4.6)
jasmine-core (2.4.1)
jshint_on_rails (1.0.3)
json (default: 2.0.4)
kramdown (1.11.1)
loofah (2.1.1)
lumberjack (1.0.10)
mail (2.6.6)
method_source (0.9.0, 0.8.2)
mime-types (3.1)
mime-types-data (3.2016.0521)
mini_portile2 (2.3.0, 2.1.0)
minitest (5.10.1, 5.9.0)
multi_json (1.12.1)
multipart-post (2.0.0)
mysql2 (0.4.4 x64-mingw32)
nenv (0.3.0)
net-telnet (0.1.1)
netrc (0.11.0)
newrelic_rpm (<IP_ADDRESS>8)
nio4r (2.1.0)
nokogiri (1.8.1 x64-mingw32)
openssl (default: 2.0.5)
orm_adapter (0.5.0)
phantomjs (<IP_ADDRESS>)
pkg-config (1.1.7)
power_assert (0.4.1)
psych (default: 2.2.2)
rack (2.0.3, 1.6.4)
rack-test (0.7.0)
rails (5.1.4)
rails-dom-testing (2.0.3)
rails-html-sanitizer (1.0.3)
rails_serve_static_assets (0.0.5)
rails_stdout_logging (0.0.5)
railties (5.1.4)
rake (12.0.0, 11.2.2)
rb-fsevent (0.9.7)
rdoc (default: 5.0.0)
request_store (1.3.1)
rspec-support (3.5.0)
ruby-openid (2.7.0)
ruby_dep (1.3.1)
rubygems-update (2.6.14, 2.6.13, 2.0.3)
safe_yaml (1.0.4)
sass (3.4.22)
sexp_processor (4.7.0)
shellany (0.0.1)
slop (3.6.0)
spring (1.7.2)
sprockets (3.7.1)
sprockets-rails (3.2.1)
test-unit (3.2.3)
thor (0.20.0, 0.19.1)
thread_safe (0.3.6, 0.3.5)
tilt (2.0.5)
timecop (0.8.1)
tzinfo (1.2.4)
unf_ext (<IP_ADDRESS> x64-mingw32)
vagrant (1.5.0)
vcr (3.0.3)
websocket-driver (0.6.5)
websocket-extensions (0.1.2)
xmlrpc (0.2.1)

Could you please try setting up using Docker? We updated the README with the details.


Read config file from global location

Now tool reads .howfairis.yml file in current working directory. It would be nice if it could find a config file in a global location so I don't have to cd to some place first. Global location could be /etc/howfairis.yml or ~/.config/howfairis/config.yml.

In the help it says The configuration file needs to be on the remote so this issue is probably not inline with the design decisions of the tool.
Why was it chosen to have a config file remotely?

In the help it says The configuration file needs to be on the remote so this issue is probably not inline with the design decisions of the tool.
Why was it chosen to have a config file remotely?

The configuration file model was recently changed such that the python module comes packaged with its own default config file. This can be (partially) overwritten by including an optional file in the repository (default: .howfairis.yml in the root of the repo). The result of the overwrite can subsequently be (partially) overwritten by including a file on the local file system, e.g. either a global one or a local one.
For reference, the relevant code is here: https://github.com/fair-software/howfairis/blob/0.12.0/howfairis/Config.py
As to

Why was it chosen to have a config file remotely?

, I think there can be good reasons to override one or more checks. For example if somebody includes a custom checklist as a MarkDown table in their README.md or as checklist.md, that would not be found by a howfairis analysis, even though it complies with the recommendations from https://fair-software.nl/recommendations/checklist.

The configuration file model was recently changed such that the python module comes packaged with its own default config file. This can be (partially) overwritten by including an optional file in the repository (default: .howfairis.yml in the root of the repo). The result of the overwrite can subsequently be (partially) overwritten by including a file on the local file system, e.g. either a global one or a local one.
For reference, the relevant code is here: https://github.com/fair-software/howfairis/blob/0.12.0/howfairis/Config.py
As to

Why was it chosen to have a config file remotely?

, I think there can be good reasons to override one or more checks. For example if somebody includes a custom checklist as a MarkDown table in their README.md or as checklist.md, that would not be found by a howfairis analysis, even though it complies with the recommendations from https://fair-software.nl/recommendations/checklist.

Agree its more about the repo owner showing he/she is fair then someone checking if repos are fair.

Agree its more about the repo owner showing he/she is fair then someone checking if repos are fair.


bump helm chart version to 0.3.0

Description
bump helm chart version to 0.3.0
Issue reference
#1358 1358
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation
[ ] Specification has been updated
[ ] Provided sample for the feature

/ok-to-test


Tweak manage-fluent-repositories.sh

Remove needless S3 profile argument for deb and rpm commands
Drop xenial support
Add dry-download and dry-upload commands
Automate copying generated files of apt repositories

LGTM.
It may be better if fluentd-apt-source_*.deb also copied into FLUENT_RELEASE_DIR too.

Thanks, I'll consider it later.


Latest tag seems borked with authentication

With latest:
time="2021-08-03T15:05:59Z" level=info msg="Fetching https://api.github.com/orgs/repos?per_page=100 \n"
time="2021-08-03T15:06:00Z" level=error msg="Error scraping API, Error: Error: Received 404 status from Github API, ensure the repsository URL is correct. If it's a privare repository, also check the oauth token is correct"
time="2021-08-03T15:06:18Z" level=info msg="Fetching https://api.github.com/orgs/repos?per_page=100 \n"
time="2021-08-03T15:06:18Z" level=error msg="Error scraping API, Error: Error: Received 404 status from Github API, ensure the repsository URL is correct. If it's a privare repository, also check the oauth token is correct"

instead with 2.0.5-alpha is working fine.
I've set both ORGS and GITHUB_TOKEN

Same problem here, alpha works fine but this latest gives this same error.

It rather seems to be a bug when assembling the URLs. https://api.github.com/orgs/repos?per_page=100 is invalid, it should be https://api.github.com/orgs/ORGNAME/repos?per_page=100


Support ovirt CPU Policy

When importing a VM from oVirt it has the property of CPU Pinning Policy.
Only having none or dedicated is supported on CNV. Therefore, only both
of them will be respected.
As for dedicated the user should enable the CPU manager on the nodes,
or else the VM won't be able to schedule and the migration will hang.
NUMA is out of the scope for this PR.
More about the feature at the dedicated cpu resources page.

Validation PR: https://github.com/konveyor/forklift-validation/pull/49

lgtm under the assumption that the case in which the cpu pinning policy property is not reported is verified

based on checking other part, when a field wasn't available from the API, I had: Required value error(empty string). Therefore I think we are good to go

lgtm under the assumption that the case in which the cpu pinning policy property is not reported is verified

based on checking other part, when a field wasn't available from the API, I had: Required value error(empty string). Therefore I think we are good to go

not sure I understand - so if we work against RHV version that doesn't report the CPU pinning field we'll get an error that says "Required value"? I wouldn't expect to get an error in this case but treat that as 'none'..

ok, that's good. but while speaking on that, I'd check if we have a good place to replace empty string to 'none', just to prevent someone adding code that enumerates the values that are listed and otherwise raise an error


Added Amazon Web Service (AWS) Arabic Playlist

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Greeting Sir,
Update the Link

Thanks, and welcome to free-programming-books!

Thanks, and welcome to free-programming-books!

The name of the channel and the playlist still not correct here

@AhmedElTabarani could you submit a PR?

done
في الخميس، 6 أبريل 2023 في 11:10 م تمت كتابة ما يلي بواسطة Eric Hellman
@.***>:

@AhmedElTabarani https://github.com/AhmedElTabarani could you submit a
PR?
—
Reply to this email directly, view it on GitHub
https://github.com/EbookFoundation/free-programming-books/pull/7437#issuecomment-1499635425,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQQEJIAE4SZW5SUTJY3BKBLW74WMTANCNFSM6AAAAAAQ3YE2GE
.
You are receiving this because you were mentioned.Message ID:
@.***>


help

HOW to train my own dataset?I need to use my image to detect the defects,so I test the VOC2007, it works.But when I change the images the Keyerror happens.In addition, CAN the model in the program be changed?or use a simpler model?please connect with<EMAIL_ADDRESS>

Did you solve this problem??
I am also confused.

many people communicate with me that the training program seems doesn't work and I don't know why


Multi-class 3D segmentation visualization in Patch-Based training

Hello @fepegar ,
I'm working on a multi-class 3D segmentation project which contains 3 classes, including background.
I've utilized Medical image segmentation with TorchIO, MONAI & PyTorch Lightning the example provided in your GitHub repository (https://github.com/fepegar/torchio-notebooks/blob/main/notebooks/TorchIO_MONAI_PyTorch_Lightning.ipynb). The key difference in my approach is that I'm training the model in a patch-based manner as provided in your GitHub repository (https://github.com/fepegar/torchio-notebooks/blob/main/notebooks/TorchIO_tutorial.ipynb).
Could you help me understand how to modify the testing part of the patch-based method to fit multi-class segmentation? Specifically, I need to visualize and check the model output for each class alongside the input image and the corresponding labels for each class.
Your assistance would be greatly appreciated.

Closing in favor of https://github.com/fepegar/torchio/discussions/1177


As an admin, I want to create staff projects that aren't R&D and have them show up on the Staff Projects page so I can highlight new work like Startwords.

We already have a "staff projects" grant type, but it isn't configured to make projects display here. Just need to edit:
https://github.com/Princeton-CDH/cdh-web/blob/96404f5296d0cd9973d21530c3e540813243bd62/cdhweb/projects/models.py#L42
and include it.
After this is done, change the Startwords project grant type back to "staff projects".
testing notes
when this is working, you should be able to find "Startwords" under "staff projects". If you click it and scroll down to the bottom to see the list of CDH grants, it should read "Staff Projects" (not "Staff R&d").

Looks good--only thing is that it says "2019– staff project," and that's a bit confusing.
If the dash just means "2019 to present," I guess that's fine. But the dash looks a little bit like a hyphen, like it means something equivalent to "2019: staff project."


json error

HOOBS 3.2.6
My json
{     "accessory": "SwitcherBoiler",     "name": "Boiler",     "debug": true }
this is the error
[Boiler] SyntaxError: Unexpected end of JSON input     at JSON.parse (<anonymous>:null:null)     at Socket.<anonymous> (/home/hoobs/.hoobs/node_modules/homebridge-switcher-boiler/lib/api.js:20:27)     at Socket.emit (events.js:322:22)     at Pipe.<anonymous> (net.js:672:12)
Thanks for the plugin

11.6.2020, 22:42:00 [Boiler] Discovering Switcher Device...
11.6.2020, 22:42:00 [Boiler] Could NOT discover devices!! -> Checking in storage for cached config...
11.6.2020, 22:42:00 [Boiler] Traceback (most recent call last):
File "/home/hoobs/.hoobs/node_modules/homebridge-switcher-boiler/lib/switcher.py", line 116, in
print("{ "status": "success", "deviceID": "" + device_id + "", "deviceIP": "" + switcherIP + "" }")
TypeError: can only concatenate str (not "bytes") to str
11.6.2020, 22:42:00 (node:731) UnhandledPromiseRejectionWarning: Error: NOT DISCOVERED & NOT CACHED
11.6.2020, 22:42:00 at /home/hoobs/.hoobs/node_modules/homebridge-switcher-boiler/index.js:103:13
11.6.2020, 22:42:00 at processTicksAndRejections (internal/process/task_queues.js:97:5)
11.6.2020, 22:42:00 (node:731) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
11.6.2020, 22:42:00 (node:731) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
11.6.2020, 22:42:00 [Boiler]
11.6.2020, 22:42:00 [Boiler] SyntaxError: Unexpected end of JSON input
at JSON.parse ()
at Socket. (/home/hoobs/.hoobs/node_modules/homebridge-switcher-boiler/lib/api.js:20:27)
at Socket.emit (events.js:322:22)
at Pipe. (net.js:672:12)
11.6.2020, 22:49:59 [Boiler] Getting Switcher State (interval)
11.6.2020, 22:50:03 [Boiler] { "status": "failed", "message": "[Errno -2] Name or service not known" }
11.6.2020, 22:50:03 [Boiler] ERROR Getting State from the Switcher:
11.6.2020, 22:50:03 [Boiler] Error: [Errno -2] Name or service not known

Ok those are 2 different issues...
Regarding the first, I'll try to correct this JSON.parse error to give me better view on the error. Is there a chance there is already more logs you've not shared that might be relevant?
Also, if both of you can please share your device information? What device? What is the firmware version?
Thanks

I have a V2 with firmware 3.21
More from the log:
[Boiler] { "status": "failed", "message": "[Errno -2] Name or service not known" } 12.6.2020, 14:25:11 [Boiler] ERROR Getting State from the Switcher: 12.6.2020, 14:25:11 [Boiler] Error: [Errno -2] Name or service not known 12.6.2020, 14:25:13 [People] Changed occupancy state for h-iPhone-sl-wr to true. Last successful ping 2020-06-12T05:25:13-06:00 , last webhook none . 12.6.2020, 14:25:14 [Boiler] { "status": "failed", "message": "[Errno -3] Temporary failure in name resolution" } 12.6.2020, 14:25:14 [Boiler] ERROR Getting State from the Switcher: 12.6.2020, 14:25:14 [Boiler] Error: [Errno -3] Temporary failure in name resolution

all errors should be fixed now with version 1.0.5
please update the plugin (delete the plugin and reinstall if not working) and close the issue if it's resolved

Thank you!


Investigate why release fails

This happened again: https://github.com/strawberry-graphql/strawberry/runs/5430322854?check_suite_focus=true

Fixed using a fork of autopub: https://github.com/strawberry-graphql/strawberry/blob/main/.github%2Fworkflows%2Frelease.yml#L47
I've opened an issue on the main repo: https://github.com/autopub/autopub/issues/26


BrowserRouter getUserConfirmation does not work correctly for the Browser Back button

Version
"react-router": "^4.3.1"
+
"react": "^16.5.2",
"react-dom": "^16.5.2",
"react-router-dom": "^4.3.1",
"react-scripts": "2.0.5"
Google Chrome
Version 69.0.3497.100 (Official Build) (64-bit)
Version 70.0.3538.67 (Official Build) (64-bit)
Test Case
https://github.com/nicewaytodoit/browser-back-bug
Steps to reproduce
Simple logic for getUserConfirmation has been written and attached to the BrowserRouter:
BrowserRouter getUserConfirmation={userConfirmation}
const userConfirmation = (msg, callback) => {
    console.log(' @--- User confirmation', msg);
    var data= JSON.parse(msg);
    // Just simple circular logic when links are pressed
    if (data.allowBack) {
        callback(true);
    } else {
        callback(false);
    }
}

In App.js a simple logic regulates when to display a Prompt question:
        <Prompt
          when={(!allowBack)}
          message={() => this.messageGetter(allowBack.toString())}
        />

And the function that sends the data info to getUserConfirmation when page is switched:
  messageGetter = (allowBack) => {
    return {
      "allowBack": ${allowBack}
    }
  }

Expected Behavior
It is expected that each time, when user clicks on the Link or clicks on Browser Back button, if getUserConfirmation Callback return FALSE it will just output a single log line "@--- User confirmation".
In other words, React controls will not refresh or go into control's life cycle.
Actual Behavior
Instead, there are 3 type of behaviors:

When LINK is clicked if getUserConfirmation's callback is FALSE control will display a correct behavior.

When [Browser Back] button is clicked and getUserConfirmation's callback is FALSE, it will stop going to other page - but it will go to trough life cycle of update for both App and the Page user is currently on as shown below:

@ App: Will Update App.js:64
@ App: RENDER Page5.js:18
@ 5page: Will Recieve Props Page5.js:22
@ 5page: Will Update Page5.js:34
@ 5page: RENDER Page5.js:26
@ 5page: DID Update App.js:41
@ App: DID Update

At some extent this is not desirable but can be acceptable/tolerable.
This behavior is kept even if we do the right click on [browser back button] and choose some of other pages that will respond to the described logic.

Now, a strange thing happen if we already visited a few pages. Let say in following order Page1, Page2, Page3, Page4, Page5, then if we check right click on the [browser back button] it should be 5 pages. At this point if we click on the browser tab REFRESH button and then click [NO BACK] inside of the page to set the state that has been lost in the process, functionality should prevent user going back through mechanism of getUserConfirmation. But, after clicking on the [Browser Back button] current page (let say Page5) will stay and control will not go trough life cycle -- but with each click, address bar will change the link and Browser History will reverse for one page, until blank page is reached at that point browser will display blank tab or switch to previous web site.

Bonus: Furthermore, in the version v3 react-router with React 15.4 (code is slightly different) behavior is similar for the point 1 and 3 and in regards to point 2 there is double PopUp behavior with first '/pagex' and action PUSH and then the same address '/pagex' with POP action. Both App and PageX will  render a full React component life cycle.
I am not sure how does is work currently so I will trow few ideas/question at this point:
Is full react history kept in react-router or history component?
Does the the browser refresh causing lose of that state too?
Would it be possible to hook or allow user to hook that state and persist the history state within local storage or any other way (e.g. database, sessions etc.)?
Third behavior is not acceptable, although it could be circumvented by let say additional popup that will prevent user closing the page and refreshing  -- but that is not a solution it is just a hack.
So, question is what is good measure between what is desired behavior and what is possible?

Does it work at all I am trying this and it doesn't work...
https://stackoverflow.com/questions/53876066/using-react-browserrouter-v4-how-do-i-recognize-when-the-location-has-changed


Remove ingress Timeout

Ingress's timeout was set to super long timeout by
https://github.com/knative/serving/pull/8965.
But the current conformance test for omitted timeout  proved that we can drop
the timeout so knative/serving and knative/networking are dropping it.
Please see https://github.com/knative/networking/pull/228 and https://github.com/knative/serving/pull/9852.
Hence this patch removes timeout in net-kourier to follow up them.
/cc @davidor @jmprusi @markusthoemmes

/lgtm

/approve