fwup not yet in Chocolatey's repo

fwup not installed. The package was not found with the source(s) listed.
If you specified a particular version and are receiving this message, it is possible that the package name exists but the version does not.
Version: ""
Source(s): "https://chocolatey.org/api/v2/"

Soon, I hope:
https://chocolatey.org/packages/fwup/0.9.1

It's still not on chocolatey, but look here now:
https://chocolatey.org/packages/fwup/0.9.2

Just posted now!
https://chocolatey.org/packages/fwup

🎉 🚀 ✨


[BUG]: AttributeError: type object 'Trainer' has no attribute 'add_argparse_args'

🐛 Describe the bug
Description:
When I running the command "python main.py --logdir /tmp/ -t -b configs/Teyvat/train_colossalai_teyvat.yaml" to rekoad the diffusion model, I encountered the following error:
AttributeError: type object 'Trainer' has no attribute 'add_argparse_args'
Steps to Reproduce:
Clone the repository.
Run the command "python main.py --logdir /tmp/ -t -b configs/Teyvat/train_colossalai_teyvat.yaml".
Expected Behavior:
The command should run without any errors.
Actual Behavior:
╭─────────────────────────────── Traceback (most recent call last) ────────────────────────────────╮
│ /root/ColossalAI/examples/images/diffusion/main.py:535 in                                │
│                                                                                │
│   532 │   sys.path.append(os.getcwd())                                                           │
│   533 │                                                                                │
│   534 │   parser = get_parser()                                                                  │
│ ❱ 535 │   parser = Trainer.add_argparse_args(parser)                                             │
│   536 │                                                                                │
│   537 │   opt, unknown = parser.parse_known_args()                                               │
│   538 │   if opt.name and opt.resume:                                                            │
╰──────────────────────────────────────────────────────────────────────────────────────────────────╯
AttributeError: type object 'Trainer' has no attribute 'add_argparse_args'
Additional Context:
The error message suggests that there is a problem with the "Trainer" object and its "add_argparse_args" attribute. Can u help me to resolve it. Thank u.
Environment
No response

Hello, could I know your version of Pytorch Lightning in your environment?
I had the same "AttributeError: type object 'Trainer' has no attribute 'add_argparse_args'" error with the lastest version of lightning==2.0.0rc1 (Release on Mar 2, 2023), but I tried with lightning==1.9.0 which is the same as [https://github.com/hpcaitech/ColossalAI/blob/main/examples/images/diffusion/requirements.txt], the above bug disappeared.

Hi, same issue here, using stable version of pytorch lighnining=2.0, just installed from conda

I have same issue. my version is 2.0.0

Same issue, version is 2.02

https://github.com/Lightning-AI/lightning/pull/16708

Same issue! version is 2.0.3

Hi,
I'm experiencing the same issue pytorch_lightning version is 2.0.4.

I installed pip install pytorch-lightning==1.06 and it WORKED!!

same issue

Same error on pl_version:
'2.0.2'

Same with 2.2.1

Same here. Pl 2.2.1

I found this problem solved with pytorch_lightning==1.6.3

same error with the pytorch_lightining == 2.2.4

same with 2.1.0

same with 2.2.0 either
Looking at the previous logs, the problem continues to occur repeatedly. Is there any post expressing your position on this?
Sorry to say this but it's a little frustrating. The version is updated, but the error is still not resolved and the code is unusable. I wish I could know what the situation is.

Same issue 2.3.3

same issue 2.4.0

Same issue with pytorch-lightning 2.4.0


chore: improved docs and ci config

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


no-op PR to rebuild img

https://github.com/open-cluster-management/backlog/issues/14437
Signed-off-by: Song Song Li<EMAIL_ADDRESS>

/hold

/approved

/unhold


Fresh install fails to fetch version (Win11)

please complete the following information:

OS version: Windows 11
Editor version [e.g. 1.45.0-insider]: 1.63.2
Programming language: N/A
TabNine extension version: 3.5.11

Issue Details:
It looks like there is a deadlock when the https.request from src/utils/download.utils.ts:48 is awaited from src/binary/binaryFetcher/bundleDownloader.ts:56.  It just hangs while trying to get the latest version number from your server. There's no indicator of failure or success, just a deadlock.
gz#10267

I'm also seeing other random deadlocks around initializing tab nine 😞


Android notification icon can't be changed

it seems that in the Android notification screen the smallIcon is hardcoded to be the play icon, while it should be configurable by passing data.icon to Player.updateOptions
https://github.com/react-native-kit/react-native-track-player/blob/b6d1272118712b2d0928055d7a20dccdb20bef23/android/src/main/java/guichaguri/trackplayer/metadata/components/MediaNotification.java#L69

I have the same issues using this piece of code:
TrackPlayer.setupPlayer();
		TrackPlayer.updateOptions({
			stopWithApp: true,
			capabilities: [
			  TrackPlayer.CAPABILITY_PLAY,
			  TrackPlayer.CAPABILITY_PAUSE,
			  TrackPlayer.CAPABILITY_SKIP_TO_NEXT,
			  TrackPlayer.CAPABILITY_SKIP_TO_PREVIOUS,
			  TrackPlayer.CAPABILITY_STOP
			],
			compactCapabilities: [
			  TrackPlayer.CAPABILITY_PLAY,
			  TrackPlayer.CAPABILITY_PAUSE
            ],

            icon: require('./../imgs/ic_logo_notification.png'),
		});

That's not the same problem. Please, open a new issue instead, this one is over a year old.


Default duration should be only an example

Issue
Within API documentation the default session duration of 24 hours is only given as an example:

"... or the default session expiration time has been reached (within an example provider implementation it is set to 24hrs)"
"When not specified, a default session duration (e.g. 24 hours) is applied"

but in the API YAML file it is fixed:
duration:
          type: integer
          example: 86400
          description: |
            Session duration in seconds. Maximal value of 24 hours is used if not set.
            After session has expired the client will receive QOS_STATUS_CHANGED event with
             - qosStatus is 'UNAVAILABLE', and,
             - statusInfo is 'DURATION_EXPIRED'.
            See notification callback.
          format: int32
          minimum: 1
          maximum: 86400
          default: 86400

Proposal:
Omit "Maximal value of 24 hours is used if not set" and the default: 86400 line from the general YAML, 24 hours is then still the maximum and implicit default value, if there is not other default value defined by the network operator. A network operator can set a specific default for their network enviroment. A PR should also make the documentation more precise and encourage developers to set the duration if they expect a certain duration.
Background:
We would like to set a short default duration in certain networks (using environment variable), e.g. in case of test environments (as a mitigation to issue described in #101).
Also a default of 24 hrs might be too long from business perspective in production networks.

Sounds reasonable to me. As you say the maximum implies an upper limit, so no implementation should allow a higher value, but we may allow different policies regarding the default. In any case, the client can always request for a value explicitly, and it will be informed of the applied value in the response.

We propose to not fix the issue. For test/lab deployments we have internal (not schema conform) solution to configure the default value. Leaving out the default value in the API spec would make the description incomplete.
A an alternative (correct) solution would be to make the duration paramater mandatory, but that would be a different issue.

Won't fix.

Won't fix.


re-land "Retry cURL downloads"

Description
This is a re-land of #50731, which was reverted because dart infra's version of curl didn't support --retry-connrefused. This re-lands the retry flags and leaves off the --retry-connrefused flag.
Related Issues
#50730
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read [Handling breaking changes].

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: Replace with a link to your migration guide

Did we ever get an answer to https://github.com/flutter/flutter/pull/50731#discussion_r383372136 from @zanderso?

Any reason not to keep the default exponential backoff behavior?

Did we ever get an answer to #50731 (comment) from @zanderso?

Any reason not to keep the default exponential backoff behavior?

Nope. @fkorotkov?

@christopherfujino 🤷♂ yeah, let's keep it simple then with just retry count.

@jmagman removed explicit 5 second delay, PTAL


Model issue: Weapon models not reading NIF properties

As you are probably aware, they changed the model extension from .dff to .nif for this port of the game, and this caused a LOT of issues.
Models do not read certain properties from .nif files, in the PS2 version this was not an issue and models displayed properly. The information that they are refusing to read are rotation, origin, and scale from within .nif NiTriStrips blocks.
Since it is completely ignored, it causes issues such as:
Weapons having incorrect sizes:
PC version:

.nif information (notice how it's supposed to be resized to be smaller):

PS2 version:

Weapons "floating away" from character's hands:

.nif information:

I manually edited the models to see if it would fix the issue, and it displayed them properly.
I'm not sure if this is something that could be fixed with SilentPatch, but I wanted to make you aware of this issue.

Upon further investigation, while this would fix plenty of issues, it should NOT be used for the map models and other props, because apparently, they have this information too, but they are placed properly (fixed) with the .ipl position instead of relying on the model's data.

Bumping this
i think props in certain classes suffer from this too. as seen here https://youtu.be/0fZt4MVflG0?t=203
It works fine on mobile port https://youtu.be/r_Mm1B_Nz1I?t=57 and also x360 version https://youtu.be/GC5yNmqNFfk?t=420


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3356

Questions ?
Please refer to the Backport tool documentation


How do i remove any animations when i filp the card ?

How do i remove any animations when i swipe the cards ?
I want to remove the animation that the cards are shrink when they are swiped

Hey @freemjstudio. If you just want to disable swipe animations (cards continuing out of the screen) then you can specify the animation of the cardStackConfiguration.
This would set the animation time to zero for example:
CardStack(/* card stack init */)
    .environment(\.cardStackConfiguration, .init(animation: .linear(duration: 0)))

There is no way to change the transition itself at the moment I'm afraid.


Renamed Validation errors to Issues identified

#1616
Changelog:

Changed "Validation errors" to "Issues identified" in the upper part of the report
Changed the text in the tooltip in the buttons as well

 Reports | Change Validation Errors label to Issues for compliance and exclusion reporting |1|


question

Hello, how to mark multiple objects.
When I press numbers to change the color on the GUI interface, there is no response at all. How can I do it?

See #11

It has been settled. Thank you very much


Results on PACS and VLCS

Thank you for the interesting work.
Reading your paper, I noticed that your results for PACS are 88.1 in Table 2, yet are 87.1 in the "ablation study" Table 5.
Similarly, you reported 79.1 for VLCS in Table 2 and then 78.9 in Table 5.
What causes these differences?
PS: it would be very helpful if you could include your HP of the SWAD-specific hyperparameters in the hparams_registry.py, to facilitate future comparison.

They has different dataset splits. For Table 5 (and Table 3), we splat the in-domain datasets into 6:2:2 for the training:validation:test, to report in-domain performance. On the other hand, there is no in-domain test set in the other experiments (this is the default DomainBed setting), so we splat the datasets into 8:2 for the training:validation.


allows msa programs to build with std::move, as added by xlat-i

Xlat-i adds std::move to function arguments while unmarshalling for zero-copy support. This did not preclude compilation of the MSA library, but anything trying to use its headers will fail to build. This patch allows MSA programs to build again.

I would merge this but there is some kind of glitch with the CI. @matthiasdiener do you know what is happening? @Wingpad for your next PR try pushing a topic branch to your forked repository and requesting a pull from it instead of your master branch.

I would merge this but there is some kind of glitch with the CI. @matthiasdiener do you know what is happening? @Wingpad for your next PR try pushing a topic branch to your forked repository and requesting a pull from it instead of your master branch.

I'm not sure what the issue is, but having a pull request from the 'master' branch might be the explanation, like you said. This seems to be a bug in the CI pipeline. I set the check to optional so that you can merge this. @Wingpad If you prefer, you can create new branches directly on UIUC-PPL/charm, no need for a fork.


Remove upgrading reinforced windows to shuttle windows

About the PR
The ability to upgrade reinforced windows to shuttle windows has been removed
Why / Balance
This specific upgrade is largely used to upgrade the windows to brig and command offices which is very powergamey and not very cool. Itd be one thing if these were meant to be destroyed but its 500 hp and its insane reflect makes it unreasonable for a 2 plasteel upgrade.
Technical details
removed the upgrade edge on the reinforced window graph
Media
Requirements

[x] I have read and am following the Pull Request and Changelog Guidelines.
[x] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

:cl: clinux

remove: Removed the ability to upgrade reinforced windows to into shuttle windows

Would it not be better to change cost instead rather than remove the feature?

Would it not be better to change cost instead rather than remove the feature?

I was leaning towards adjusting their health as an alternate over the cost (b/c frankly them having anything over 100hp is silly). I'm more than willing to massively nerf the shuttle walls and windows (as well as increase the cost for construction & upgrading) if that's what the general consensus yields. But since these structural parts are really only used for evac and from what I remember the main reason they have such large heath is to prevent EORG/shuttlebombing (not that it really helps, evac gets spaced and has holes blown in it consistently)
it both doesn't make sense for them to be so easily utilized and nerfs would get more maintainer pushback than removal, I opted to remove them instead.

In terms of nerfs to the shuttle wall and windows I was thinking this:
Windows-
Hp decreased from 500->100
Construction cost increased from 2 plasteel -> 4
Walls-
Hp decreased from 600->400 (for total destruction, 200 to destroy into girder) (this gives it 100 and 50 more hp than a plasma wall respectively)
Construction cost increased from 4 total plasteel -> 5
What do you guys think?

Shuttle structures are intentionally robust. I wouldn't make them weaker. Increase the cost and add to the complexity(extra steps) if you want to decrease the scenarios you mention.

Shuttle structures are intentionally robust. I wouldn't make them weaker. Increase the cost and add to the complexity(extra steps) if you want to decrease the scenarios you mention.

I fully get why they are so beefy, but an upgrade to a 500hp window with anything that can be ordered from cargo for a reasonable price & quantity is insane at best. You should not be able to upgrade the brig windows to be almost the same hp as a reinforced wall with roundstart materials.
What kind of steps do you even have in mind? The entire point of the window upgrade system is so that you can quickly upgrade windows. If you make the upgrade process complex then what's the difference between it and just deconstructing the existing window and constructing a shuttle one? The "extra steps" would be to deconstruct the existing window in this case, aka removing the upgrade path (which this PR does).

I would say it's fairly reasonable to remove from the upgrade graph as long as they can still be built from scratch. But if it's preferable to leave it in, then look at the time it takes to build from scratch and add a welding step to make up the difference

But if it's preferable to leave it in, then look at the time it takes to build from scratch and add a welding step to make up the difference

I can agree to that, up to the maintainers tho imo.

Regarding window upgrades -
I'm in agreement that it should actually be really difficult for command/sec to upgrade windows. This is the 1# reason they weren't included in the RCD and that bypasses that step explicitly. Otherwise you do get command/sec upgrading windows with near indestructible windows every round and that's something we do not want to have gameplay wise.

Otherwise you do get command/sec upgrading windows with near indestructible windows every round and that's something we do not want to have gameplay wise.

This is what prompted the PR lol
Over the past month on lizard windows have increasingly been upgraded to shuttle windows in the bridge/brig/hop office and I had like 4 rounds in a row where the brig was reinforced with this. I'm sure this isn't as much as an issue on mrp servers but it's gotten very annoying on lrp.

If you swap this PR to a full revert it'd get my approval. This definitely needed a maintainer review before it was merged.

Ehhh... Im rather fond of the other window upgrades and really only think the issue lies with the shuttle windows.

If you swap this PR to a full revert it'd get my approval. This definitely needed a maintainer review before it was merged.

Ehhh... Im rather fond of the other window upgrades and really only think the issue lies with the shuttle windows.

The main issue is that the original PR accidentally bypassed any official maintainer discussion.
As it is right now, it's also skipping the fact that we excluded this from RCDs for a reason - due to it being really powerful. Which is why we required the actual construction steps for these kind of windows. It's also why things like R-Walls and other specialty type walls aren't in the RCD/upgradable.

I just think they look ugly when next to normal or reinforced walls so I'm on board :godo:

The main issue is that the original PR accidentally bypassed any official maintainer discussion.
As it is right now, it's also skipping the fact that we excluded this from RCDs for a reason - due to it being really powerful. Which is why we required the actual construction steps for these kind of windows. It's also why things like R-Walls and other specialty type walls aren't in the RCD/upgradable.

I can change this to a revert tomorrow if you really want but since we are on that biweekly release schedule would it not make more sense to have that maintainer discussion now and request changes to this PR? (Instead of reverting, opening a new PR re-adding the old stuff, maintainer discussion there) I'm down to make any requested changes since it's just yaml.

The main issue is that the original PR accidentally bypassed any official maintainer discussion.
As it is right now, it's also skipping the fact that we excluded this from RCDs for a reason - due to it being really powerful. Which is why we required the actual construction steps for these kind of windows. It's also why things like R-Walls and other specialty type walls aren't in the RCD/upgradable.

I can change this to a revert tomorrow if you really want but since we are on that biweekly release schedule would it not make more sense to have that maintainer discussion now and request changes to this PR? (Instead of reverting, opening a new PR re-adding the old stuff, maintainer discussion there) I'm down to make any requested changes since it's just yaml.

We already had a lengthy talk about this, was going to make a post later tonight, but if you could swap this to a revert that'd make things easier.
The biggest two reasons for a revert:

It doesn't fit the game.

Only thing that really prevents people from spamming windows is materials. This is one of the reasons these window types weren't added to the RCD. To build these stronger windows, one of the biggest balance for them is the time & steps it would take to replace.
It removes the need for tools (welding, screwdriver, etc) to repair/remove/replace.
It makes space-facing windows a non-issue to deal with. To replace a space facing window you should bring space protection and holofans with you, and air refill as needed.

It skipped the previous maintainer process unintentionally, so there was no discussion behind merging it back then.

Note: these to fixes-
fix: Windows have been made to correctly display their damage visuals.
fix: Reinforced plasma windows have been given the correct amount of hp and no longer have 12x the amount they should.
Have been reverted as well.

Could you keep the fix for damage visuals? Reinforced Plasma glass can stay reverted, would need to talk to the other devs about that.
Feel like the fix for the damaged visuals is fine, just not the upgrades.

@cohanna tagging and requesting changes for visibility

Seen, will implement changes as soon as I get the time to

Tests went over fine. This is good to go.


Build, flash, and run the FreeRTOS demo project in Aws esp32 in VS code

Describe the bug
Issue at step Hey, Build, flash, and run the FreeRTOS demo project" . I followed [https://docs.aws.amazon.com/freertos/latest/userguide/getting_started_espressif.html#setup-espressif-idf33] this documentation.
System information

Hardware board: [esp32]

Operating System: [ Windows  ]

Expected behavior
I am stuck at CMake step could not figure out how to proceed .Do I require VS Code for it.

Hello @aditi230 ,
Could you provide more information about what probelm you have? If possiable could you paste the command you use and the error result you got? The VS Code is not required. Once you set up correctly, you can directly run the Cmake command from the amazon-freertos root derectory from a terminal.
Regards,
Ming

cmake -DVENDOR=espressif -DBOARD=esp32_wrover_kit -DCOMPILER=xtensa-esp32 -S . -B your-build-directory
In this command what should be written under your-build-directory and also I am using esp32_devkitC

Hi @aditi230,
the -B option is to set the folder location (can be a new folder which will be created during the build process) where to store the building artifacts and the build image. For example, if you build from the amazon-root directory and want to save the image under "./build" folder. Then you coulde set the -B as ./build. Then the command would be:
cmake -DVENDOR=espressif -DBOARD=esp32_wrover_kit -DCOMPILER=xtensa-esp32 -S . -B ./build
Regards,
Ming

Got It ! Thank you

After this this is the error I ma facing.

other then that every thing is working fine
How will i resolve this.

Build and flash FreeRTOS on Windows, Linux, and MacOS (ESP-IDF v4.2)
Use the idf.py script to build the project and flash the binaries onto your device.
Note
Some setups might require that you use the port option "-p port-name" with idf.py to specify the correct port, as in the following example.
idf.py -p /dev/cu.usbserial-00101301B flash
To build and flash the project
Navigate to the root of your FreeRTOS download directory.
In a command line window, enter the following command to add the ESP-IDF tools to your terminal's PATH.
Windows
vendors\espressif\esp-idf\export.bat
This is giving me error:
`Setting IDF_PATH: I:\FreeRTOS\vendors\espressif\esp-idf
Adding ESP-IDF tools to PATH...
fatal: not a git repository (or any of the parent directories): .git
Traceback (most recent call last):
File "I:\FreeRTOS\vendors\espressif\esp-idf\tools\idf_tools.py", line 1358, in
main(sys.argv[1:])
File "I:\FreeRTOS\vendors\espressif\esp-idf\tools\idf_tools.py", line 1354, in main
action_func(args)
File "I:\FreeRTOS\vendors\espressif\esp-idf\tools\idf_tools.py", line 980, in action_export
idf_python_env_path, idf_python_export_path, virtualenv_python = get_python_env_path()
File "I:\FreeRTOS\vendors\espressif\esp-idf\tools\idf_tools.py", line 840, in get_python_env_path
idf_version_str = subprocess.check_output(['git', '--work-tree=' + global_idf_path, 'describe', '--tags'], cwd=global_idf_path, env=os.environ).decode()
File "C:\Python39\lib\subprocess.py", line 424, in check_output
return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
File "C:\Python39\lib\subprocess.py", line 528, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['git', '--work-tree=I:\FreeRTOS\vendors\espressif\esp-idf', 'describe', '--tags']' returned non-zero exit status 128.`


Add FiniteElementCollection::GetVDim in order to fix a bug with AMS when using ND_R1D_FECollection or ND_R2D_FECollection

This PR addresses the discussion in #3524 here: https://github.com/mfem/mfem/pull/3524#issuecomment-1624383241. In particular, the spaces introduced by PR #1989 have vdim != space_dim for interpolation, so the AMS solver setup needs special treatment. The results of mpirun -np 4 ./ex31p -o 2 reveal the issue: Running with master converges in 90 iterations, while with the fix from this branch only 27 iterations are required.

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#3774
@sebastiangrimberg
@mlstowell
@psocratis + @tzanio
7/11/23
⌛due 7/25/23
⌛due 8/1/23

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

Hi @tzanio, sorry to bother you here but would you mind taking a look at this when you get the chance? It would be nice to fix this bug in master. Thank you!

This would be a great bug fix to include as part of https://github.com/mfem/mfem/pull/3841 in the upcoming release if possible, as it continues to affect master. @tzanio would you mind taking a look when you get a chance or is there anything I can do to help move this forward?

Hi @sebastiangrimberg,
I am sorry for this overly delayed response.
I was debating on what to do about GetVDim(), which is now present in FiniteElementSpace, FiniteElement,  FiniteElementCollection and several other classes.
Do you think that the different meaning in FiniteElementSpace and FiniteElementCollection will be confusing, and if so, which one we should rename and to what?
Resolving this is probably not for this PR, but I thought we can at least indicate the possible confusion with some additional comments...

Hi @tzanio, thanks for taking the time to get back to us here. I fully agree with your concern and can already recall at least a few situations for confusion between the different meanings of FiniteElementSpace::GetVDim and FiniteElement::GetVDim. The new FiniteElementCollection::GetVDim does not introduce a new meaning, fortunately, as it's the same as that of FiniteElement but I'll still propose two different solutions to improve things here for users and developers:

As you suggest, clarify the different in comments. This can easily be done as part of this PR.

Rename FiniteElement and FiniteElementCollection::GetVDim to something more appropriate which doesn't conflict. These methods are used less than FiniteElementSpace::GetVDim (and certainly exposed less outside of the library) and so the impact for end-users and throughout the library will be less, and not so bad I believe. I would suggest calling it GetInterpDim for the new name, to reflect the dimension of the basis interpolation operation from dof -> quadrature points. If this sounds good or at least like an interesting idea I can make a PR into this one with the rename to investigate just how widespread the change in names would be.

If this sounds good or at least like an interesting idea I can make a PR into this one with the rename to investigate just how widespread the change in names would be.

Okay lets do that, thanks @sebastiangrimberg! Another possible name could be GetValueDim.

Great, I'll have something ready tomorrow.

See a2557a3.
Another appropriate name option could be GetRangeDim (to match GetRangeType/GetDerivRangeType. Perhaps you or @v-dobrev have thoughts here? Happy to go either way but in my opinion this is actually a good change to make as part of this PR.

I like GetRangeDim. Unless someone else objects, I am happy to approve if you make that change.

Ping @mlstowell

Merged in next for testing...

From nightly testing: the iteration counts in sample runs in ex31p and ex32p are improved by this PR. However, the following sample run from ex8p gets worse (from 13 iteration to no convergence in 200 iterations):
mpirun -np 4 ./ex8p -m ../data/star-surf.mesh -o 2 -no-vis

From nightly testing: the iteration counts in sample runs in ex31p and ex32p are improved by this PR. However, the following sample run from ex8p gets worse (from 13 iterations to no convergence in 200 iterations):

The issue was that the order p for the nodal space was being set to p-1 for dim = 1 (for this example, dim = 2 and we are constructing the AMS solver on the trace space). This happened because ND_FECollection for dim = 1 returns p-1 for GetOrder. Previously for trace spaces the GetOrder call was on the RT_FECollection for ex8p which correctly returns p.

Re-merged in next for testing...

There are a number of diffs in tonight's autotest on the following runs:
mpirun -np 4 ex31p -m ../data/hexagon.mesh -o 2 -no-vis
mpirun -np 4 ex31p -m ../data/star.mesh -no-vis
mpirun -np 4 ex31p -m ../data/square-disc.mesh -o 2 -no-vis
mpirun -np 4 ex31p -m ../data/square-disc-nurbs.mesh -o 3 -no-vis
mpirun -np 4 ex31p -m ../data/amr-quad.mesh -o 2 -rs 1 -no-vis

mpirun -np 4 ex32p -m ../data/hexagon.mesh -o 2 -no-vis
mpirun -np 4 ex32p -m ../data/star.mesh -no-vis
mpirun -np 4 ex32p -m ../data/square-disc.mesh -o 2 -n 4 -rs 1 -no-vis
mpirun -np 4 ex32p -m ../data/square-disc-nurbs.mesh -rs 3 -o 3 -no-vis
mpirun -np 4 ex32p -m ../data/amr-quad.mesh -o 2 -rs 1 -no-vis

They all seem to be improvements, for example
--- sample-runs-baseline-tux426.saved.filt	2023-10-20 03:49:50.657058995 -0700
+++ sample-runs-baseline-tux426.out.filt	2023-10-20 03:49:50.226055789 -0700
@@ -163927,59 +163927,33 @@ <examples> mpirun -np 4 ex31p -m ../data/hexagon.mesh -o 2 -no-vis
   cpu clock time  = x.xx seconds
   cpu MFLOPS      = 0.000000

-<C*b,b>: 2.611336e+02
+<C*b,b>: 3.163512e+02

 Iters       ||r||_C     conv.rate  ||r||_C/||b||_C
 -----    ------------    ---------  ------------
-    1    2.947654e+00    0.217820    1.824085e-01
-    2    2.022022e+00    0.685977    1.251280e-01
-    3    1.483375e+00    0.733610    9.179510e-02
-    4    1.056575e+00    0.712278    6.538363e-02
-    5    7.000339e-01    0.662550    4.331992e-02
-    6    4.250542e-01    0.607191    2.630346e-02
-    7    2.584280e-01    0.607988    1.599220e-02
-    8    1.290529e-01    0.499377    7.986132e-03
-    9    9.767519e-02    0.756861    6.044396e-03
-   10    8.573402e-02    0.877746    5.305444e-03
-   11    6.515222e-02    0.759934    4.031789e-03
-   12    6.672521e-02    1.024143    4.129130e-03
-   13    4.698466e-02    0.704151    2.907533e-03
-   14    2.499886e-02    0.532064    1.546995e-03
-   15    1.326472e-02    0.530613    8.208554e-04
-   16    7.271846e-03    0.548210    4.500008e-04
-   17    4.000522e-03    0.550138    2.475627e-04
-   18    2.221832e-03    0.555386    1.374928e-04
-   19    1.515255e-03    0.681985    9.376796e-05
-   20    1.104933e-03    0.729206    6.837616e-05
-   21    6.843608e-04    0.619368    4.235003e-05
-   22    3.749040e-04    0.547816    2.320004e-05
-   23    2.007864e-04    0.535568    1.242519e-05
-   24    1.370456e-04    0.682544    8.480742e-06
-   25    6.029602e-05    0.439970    3.731275e-06
-   26    3.041494e-05    0.504427    1.882156e-06
-   27    1.623125e-05    0.533660    1.004432e-06
-   28    8.347502e-06    0.514286    5.165652e-07
-   29    5.112137e-06    0.612415    3.163523e-07
-   30    3.641462e-06    0.712317    2.253432e-07
-   31    2.689342e-06    0.738534    1.664235e-07
-   32    1.397238e-06    0.519547    8.646476e-08
-   33    7.133001e-07    0.510507    4.414087e-08
-   34    3.537619e-07    0.495951    2.189171e-08
-   35    1.819985e-07    0.514466    1.126254e-08
-   36    1.045075e-07    0.574221    6.467194e-09
-   37    6.698242e-08    0.640934    4.145047e-09
-   38    3.575714e-08    0.533829    2.212745e-09
-   39    1.526792e-08    0.426989    9.448186e-10
-   40    7.652251e-09    0.501198    4.735412e-10
-   41    3.593526e-09    0.469604    2.223767e-10
-   42    1.494784e-09    0.415966    9.250114e-11
-   43    5.713986e-10    0.382262    3.535963e-11
-   44    2.420154e-10    0.423549    1.497655e-11
-   45    1.109640e-10    0.458500    6.866742e-12
-   46    5.449979e-11    0.491148    3.372589e-12
-   47    2.272773e-11    0.417024    1.406451e-12
-   48    8.730966e-12    0.384155    5.402950e-13
+    1    1.424255e+00    0.092254    8.007614e-02
+    2    2.473365e-01    0.173660    1.390604e-02
+    3    6.760617e-02    0.273337    3.801033e-03
+    4    1.541750e-02    0.228049    8.668206e-04
+    5    5.745541e-03    0.372664    3.230325e-04
+    6    1.803782e-03    0.313945    1.014143e-04
+    7    8.121549e-04    0.450251    4.566192e-05
+    8    3.183839e-04    0.392024    1.790055e-05
+    9    1.210442e-04    0.380183    6.805488e-06
+   10    3.258535e-05    0.269202    1.832051e-06
+   11    8.965635e-06    0.275143    5.040764e-07
+   12    2.157339e-06    0.240623    1.212924e-07
+   13    5.985692e-07    0.277457    3.365345e-08
+   14    1.658318e-07    0.277047    9.323586e-09
+   15    4.463431e-08    0.269154    2.509482e-09
+   16    1.090579e-08    0.244336    6.131580e-10
+   17    2.729355e-09    0.250267    1.534530e-10
+   18    9.045941e-10    0.331431    5.085914e-11
+   19    2.722810e-10    0.300998    1.530850e-11
+   20    7.201621e-11    0.264492    4.048979e-12
+   21    1.966339e-11    0.273041    1.105538e-12
+   22    4.831783e-12    0.245725    2.716581e-13

 =============================================
@@ -163991,8 +163965,8 @@ <examples> mpirun -np 4 ex31p -m ../data/hexagon.mesh -o 2 -no-vis
   cpu clock time  = x.xx seconds
   cpu MFLOPS      = 0.000000

-PCG Iterations = 48
-Final PCG Relative Residual Norm = 5.40295e-13
+PCG Iterations = 22
+Final PCG Relative Residual Norm = 2.71658e-13

 || E_h - E ||_{H(Curl)} = 0.0306458

@sebastiangrimberg, I will email you the full log to make sure these are expected and look good.

There are a number of diffs in tonight's autotest on the following runs:

They all seem to be improvements, for example

Yep, this is expected for examples 31p and 32p. This PR will fix a bug on master for any use of AMS with Nedelec spaces of the type ND_R1D_FECollection and ND_R2D_FECollection.


feat(model-client): extend JS API

Expose ReplicatedModelJS to access information about the replicated version.
Add method in ClientJS to initialize a repository.
Add method in ClientJS to set a user ID.
Add documentation to public code in the JS API.
Mark some code as internal in the JS API.
To be verified by reviewers

[ ] Relevant public API members have been documented
[ ] Documentation related to this PR is complete

[ ] Boundary conditions are documented
[ ] Exceptions are documented
[ ] Nullability is documented if used

[ ] Touched existing code has been extended with documentation if missing
[ ] Code is readable
[ ] New features and fixed bugs are covered by tests

:tada: This PR is included in version 8.10.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Slack messaging for scheduled tasks

Description of change
Enable 2 of the scheduled tasks - "automatic company archive" and "get company updates" - to send summary messages to a Slack channel upon completion
Checklist

[Y] If this is a releasable change, has a news fragment been added?

[Y] Has this branch been rebased on top of the current develop branch?

[Y] Is the CircleCI build passing?

Thanks, ok, will fix all these 👍
On Mon, 6 Apr 2020 at 21:05, Pawel Szymanski<EMAIL_ADDRESS>wrote:

@elcct commented on this pull request.
In datahub/core/test/test_realtime_messaging.py
https://github.com/uktrade/data-hub-api/pull/2756#discussion_r404355000:

   """

   Test that sends are aborted if the channel is not set.

   """

   caplog.set_level('INFO')

   realtime_messaging = RealtimeMessaging()

   realtime_messaging.send_realtime_message('Hello')

   expected_message = 'Setup was incomplete. Message not attempted.'

   assert expected_message in caplog.text

+@pytest.fixture()
+def realtime_messaging():

"""
Set up a realtime_messaging client
"""
with override_settings(SLACK_API_TOKEN='API_TOKEN',

Our coding style requires that each argument is in its own line if the
line is too long. For example:
with override_settings(
SLACK_API_TOKEN='API_TOKEN',
SLACK_MESSAGE_CHANNEL='MESSAGE_CHANNEL',
):
Would be good to fix this in all other places.
You can have a look:
https://github.com/uktrade/data-hub-api/blob/develop/docs/Coding style.md
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/uktrade/data-hub-api/pull/2756#pullrequestreview-388576601,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADTNLFMI4E2JBDZGJBG753RLIYYTANCNFSM4MCPCNEA
.

--
*Mark Henwood *| Tech Lead | Data Hub | Digital, Data & Technology
*Department for International Trade *| Windsor House, 50 Victoria Street,
SW1H 0TL
M: +44 (0) 20 3286 4394 [redirects to mobile] | E:
<EMAIL_ADDRESS>gov.uk/dit
https://www.gov.uk/government/organisations/department-for-international-trade?utm_source=staffsignature&utm_medium=email&utm_campaign=DIT
| GREAT.gov.uk
http://www.great.gov.uk/?utm_source=staffsignature&utm_medium=email&utm_campaign=GREAT
| DIT Twitter https://twitter.com/tradegovuk | DIT LinkedIn
https://www.linkedin.com/company/department-for-international-trade
| Business
is GREAT Facebook http://on.fb.me/1atEmJr
--
Communications with the Department for International Trade may be
automatically logged, monitored and/or recorded for legal purposes.

I think that last push addresses all your concerns @reupen. Thanks for the suggestions.


Bundle blacklist.txt to Docker image

Bundle the regex in blacklist.txt to Docker image.

We should instead of blacklisting every single file, move relevant files from the project root to their own directory. So we can write
!<dir>

once and not for each file.

We should also consider using a single folder in the project root for all files required for the project to function. Like in Node.js projects, all relevant files are stored in the src directory. We should seek a similar solution for this Rust project.

These file are already mounted to the container as images, so that they can be modified on runtime if necessary.
See docker-compose.yml for this

These file are already mounted to the container as images, so that they can be modified on runtime if necessary. See docker-compose.yml for this

This should be documented and all relevant configuration files should use the same method of inclusion in the container (as in, they will be included via docker-compose.yml or via Dockerfile, but all from the same file).

I agree, the files that are included (not excluded) in .dockerignore should all be requirements for building the project and nothing more that that. Everything else can and should be mounted as an image for the to be able to be modified quickly if necessary.

After some investigation it appears that the mounts are actually successful in docker-compose.yml (removed !award_id_blacklist.txt line from .dockerignore). This means that the problem is not the mount but something after it. Maybe they files are never read?

After some investigation it appears that the mounts are actually successful in docker-compose.yml (removed !award_id_blacklist.txt line from .dockerignore). This means that the problem is not the mount but something after it. Maybe the files are never read?

This should be easy enough to figure out with a few breakpoints in the code. I suggest @lajp should build the project in the production server with a few logs about what points are reached and if the file is being read or not.

Ok, Ill be closing this pr, and also reverting #23.
I'll start working on some documentation later today.

After some investigation it appears that the mounts are actually successful in docker-compose.yml (removed !award_id_blacklist.txt line from .dockerignore). This means that the problem is not the mount but something after it. Maybe the files are never read?

This should be easy enough to figure out with a few breakpoints in the code. I suggest @lajp should build the project in the production server with a few logs about what points are reached and if the file is being read or not.

I think the problem is just that the build is so outdated that it doesn't yet mount the files.

After some investigation it appears that the mounts are actually successful in docker-compose.yml (removed !award_id_blacklist.txt line from .dockerignore). This means that the problem is not the mount but something after it. Maybe the files are never read?

This should be easy enough to figure out with a few breakpoints in the code. I suggest @lajp should build the project in the production server with a few logs about what points are reached and if the file is being read or not.

I think the problem is just that the build is so outdated that it doesn't yet mount the files.

I suggest you (@lajp) create a separate issue about the blacklist not working to track fixes.

Will do, closing this now


Bot not working with latest Chrome version

Currently running Version 115.0.5790.99 of chrome and getting this message when attempting to start
WebDriver generation failure!
Is Google Chrome installed?
Is there a network problem? Check VPN availability if one connected
If it still doesn't work, try reinstalling Google Chrome.
Press Enter to exit.
Was working fine before the update

webdriver not match 115 now.downgrade to 114 will fix this issue

webdriver not match 115 now.downgrade to 114 will fix this issue

看这篇文章，大概意思是115以上的版本不会发布在chrome driver上了？
After reading this article, it probably means that the version above 115 will not be released on the chrome driver?
https://chromedriver.chromium.org/downloads/version-selection

webdriver not match 115 now.downgrade to 114 will fix this issue

看这篇文章，大概意思是115以上的版本不会发布在chrome driver上了？ After reading this article, it probably means that the version above 115 will not be released on the chrome driver?
https://chromedriver.chromium.org/downloads/version-selection

等undetected-chromedriver库更新吧 等他修复这个问题应该就没事了

Uninstall your chorme retain data
install old version
114.0.5735.199_x64
https://mega.nz/file/4G5FFBhI#bPZNi3SGV0E_ZSf5s1Tr7zc8CQbuBepw1oabDpPY6t8

2.1.2 fix it.


How to convert the data from SQL Server to Azure API For FHIR database.

I have all of my data present in the SQL Server. I have thousands of patients. I cannot insert my data manually.
Is there any ETL process that can handle the data ingestion process? Our C-CDA does not match the current version.
Has anybody here written any mapper which converts data directly from the SQL server to Azure Cosmos DB for the Azure API For FHIR?
I will really appreciate your help on this. Please let me know if you have any questions for me.

Hi @Kedar782 , currently we don't have an ETL process that can copy data from SQL server to Azure API for FHIR. You have to write the ETL pipeline yourself.


Use git branch name by default when creating a db branch

Feature request
Use the current git branch name as the default name – when no branch name is given – for the db branch when running supabase db branch create.
Describe the solution you'd like
The current implementation requires that I pass the <branch name> as an argument, but it would be nicer if the argument were optional and the CLI used my currently checked out branch in git for the branch name when no branch name is given.
Result in a db branch being created named feb23-livestream, after the git branch name
╭─iamwill@zues ~/code/OneSignalDevelopers/ngosi ‹feb23-livestream●›
╰─$ supabase db branch create

If a name is given, use that instead
╭─iamwill@zues ~/code/OneSignalDevelopers/ngosi ‹feb23-livestream●›
╰─$ supabase db branch create some-other-branch-name

This is now the default behaviour for preview branching https://github.com/supabase/cli/pull/1313


Module fix

Fully removing RobustToolbox submodule in order to re-add it fresh. This will cause ALL the builds and tests to fail. I'm not going to merge until I re-add it.

I'm done. I'm not allowed to panic fix Harmony at night anymore.


Optimize makefile and use buildkit where possible

Change-type: patch
Signed-off-by: Kyle Harding<EMAIL_ADDRESS>

@balena-ci rebase

Superseded by https://github.com/balena-os/leviathan/pull/568


Question on Inference performance

Hi,
I see that it is taking around 10s on A100 to generate a 512 stylyzed image (Without knowing the technical details of the architecture, I would imagine the style transfer to be faster than generating images from scratch using the models like GANs or text-2-img models which can generate new images in a couple of seconds or even less, I apologize if this is less informed comment from me).
I am just curious if there is anything that can be warmed up or tweaked to make the inference time in the order of a couple of seconds? I see that VGG model gets loaded every time.. but that's inconsequential compared to the overall time. Is there anything that can be warmed up or tweaked to improve the performance?
Thanks.

Hi Vanga,
This is an optimization based method, so there isn't a specialized neural network that does the stylization; instead the output image is iteratively updated to minimize the objective function (which is based on features extracted by VGG). This is inherently much slower than the forward pass of a neural network. You can play with reducing the number of iterations, but it will make results worse.
Best,
Nick


⚠ Cioos Pacific CKAN has degraded performance

In fd6b170, Cioos Pacific CKAN (https://catalogue.cioospacific.ca) experienced degraded performance:

HTTP code: 200
Response time: 5120 ms

Resolved: Cioos Pacific CKAN performance has improved in c2da5ad after 5 minutes.


PSA: Xargo is now in maintenance mode

Also see the status updates below. Original announcement by original maintainer follows.

It actually has been in maintenance for a while but now I'm officially announcing it.
What does this mean?
It means that I am not going to add new features to Xargo or fix known papercuts. If you would like to work on that then I can add you as collaborator. I do, however, will fix future issues (hopefully zero of them), caused by nightly, that break my ARM Cortex-M development workflow.
What should I do now?
I have proposed adding Xargo's functionality to Cargo here. Doing that will fix the known papercuts (#189, #123, etc.) and make it easier to add desired features (e.g. #139). I suggest that you comment on that issue to get it green lighted and implemented (by someone other than me).
Why are you doing this?
Xargo was conceived as a temporary solution to the missing "build the sysroot" functionality in Cargo (cf. rust-lang/rfcs#1133). This is just me being firm about the "temporary" part.

@ofek This integration got punted, because we were able to land direct support for the platforms Xargo is typically used for. There's interest in doing this work in Cargo still, but not until 2019.

I would like this to be a key focus for Cargo in 2019. We're working on a more precise roadmap, but I discuss it in my recent blog post at https://www.ncameron.org/blog/cargos-next-few-years/

It would be really really great if this could happen.

Linking to related RFC: std Aware Cargo

I would like this to be a key focus for Cargo in 2019. We're working on a more precise roadmap, but I discuss it in my recent blog post at https://www.ncameron.org/blog/cargos-next-few-years/

As well as broad impact, I think we should focus on cross-compilation this year because:

So what's the current state?  Does cargo+rustup already do everything we need for cross development?

Current status:

std Aware Cargo RFC: closed July 29, 2020
wg-cargo-std-aware repo: created July 2020

The issue tracker seems to be the place for current discussion

build-std exists in nightly Rust

https://github.com/johnthagen/min-sized-rust#optimize-libstd-with-build-std

Xargo maintenance status update
The current status of xargo maintenance is as follows: we haven't seen @japaric in a while, so not sure how much they are still keeping their ARM Cortex-M development workflow working.
I am maintaining xargo for the purpose of Miri, which needs to check-build the official Rust standard library (unchanged). As long as Miri needs xargo, as part of maintaining Miri I will make sure this part of xargo keeps working. But I will likely not have time to work on issues that do not affect Miri. Your PRs are welcome as long as they keep CI green and don't break Miri. :)

As for me, my need for Xargo has basically gone away so any time I spend reviewing issues & PRs is solely community service.

Another status update: Miri has stopped using xargo, so I will also not maintain xargo any more.

@japaric, even now at the end of 2024 some people still use Xargo(and it's mentioned in the OSDev wiki for making an OS for raspberry pi in rust) and I'd like to be added as a maintainer so that I can try to fix known bugs and the such. I also will add features as they are requested.


Separate business handlers

There are some thing missing like the swagger docs and the updates to the queries. I'll let you know when I think is ready so we can do a review, change what's left and merge @teh0xqb

@teh0xqb This is ready for integration I think. I tested and is working fine. Please check out the code and let me know if we need further modifications.
The abstractions can go in another release

There are some thing missing like the swagger docs
The swagger docs go in schemas.clj, you can take a look how the query params are described for /inspections, and use the same format for /bussinesses. I think we should take care of that in this PR.

I did the refactor for some common functionalities. I see other patterns susceptibles to abstraction but I can't quite get exactly how right now. I added the swagger thing. @teh0xqb

I see other patterns susceptibles to abstraction but I can't quite get exactly how right now

That's fine. There's only one outstanding thing. I guess I could fix, but I already asked not sure if you're taking care of it already.

Yes I changed county to countyNumber @teh0xqb

Remember to close https://github.com/Code-for-Miami/restaurant-inspections-api/issues/62 with the merge, when you merge, if you merge...

There's a bug around the zipCodes query param. I'm trying out your branch and I can query by one zip code, but when querying by multiple zipCodes I get no data (data: []).
Tried this query:
http://localhost:8083/businesses?perPage=5&zipCodes=32609,32615
Calling zipcodes individually works. Multiple zip codes do work for inspections:
http://localhost:8083/inspections?perPage=5&zipCodes=32609,32615
I'm trying to debug a bit, but have been unable to find the problem. Let me know if you find it @marcoslhc

Thanks for noticing this @teh0xqb. The problem is I didn't add zipCodes to the services/format-business-params before passing them to the query. It's fixed. I tested also Notifications and it's working fine.
try http://localhost:8080/businesses?zipCodes=33317,33139&perPage=200

+1, merged.


CircleCI - Overview Dashboard

PR Details
Adding: The new CircleCI SumoLogic App based on Webhooks.
Description

Checklist

[ ] Changes follows the best practices from the given docs.
[ ] Updated CHANGELOG.md.
[ x] Ran unit tests locally.
[ x] Updated screenshots/sample logs if any.

ISSUES
Following the contributor's guide: https://github.com/SumoLogic/sumologic-public-partner-apps/blob/master/CONTRIBUTING.md

In step 5 To update the manifest run the below command.
a. Is this a required step? Or for those updating an existing app?
b. sumoapputils is not an installed command
b1. This also doesn’t appear to be a typo of either sumoapp utils or sumoapp sumoapputils
In step 11, it is instructed to add your changes to the commit, but the README file is not added in these instructions.
a. I was unsure if this was intentional and added the README.md.

@sumo-drew You had made mention of ensuring this was a "version 2.0" of the CircleCI app. We do not appear to already exist in this repository and I didn't see any mentioning of versioning in the contributor's guide. Though I see "manifestVersion": "0.1",. Should we update this?

@sumo-apps-team - Should we have this released as a second version? The prior version depended on the Orb and this new version requires their new webhooks functionality.

@sumo-apps-team - Should we have this released as a second version? The prior version depended on the Orb and this new version requires their new webhooks functionality.

@KyleTryon Should we release this app as second version? In the documentation link mentioned in Readme I do not see any details of configuring webhook functionality

Hello,
Yes, that link/documentation is planned to be replaced with new documentation based on the same info in the "Partner App Doc Template" which has a step that essentially links to: https://circleci.com/docs/2.0/webhooks/?section=projects#setting-up-a-hook.
Does the info in that doc appear near the integration on your platform?

@KyleTryon  So the old doc will no longer exist? I mean the orb based approach would be replaced with webhook based approach? If yes then we can replace the old app with the new app else if both the approaches will coexist then we will have to create new app version.

@himanshu219 - No need to do a formal version as this will replace the old app. I'm working with @KyleTryon on the updated CircleCI docs. Once those are ready let's go ahead and publish!


PV update using PNet

Hi Hyeonbeen,
Thank you for providing the great material on automating RD using Python.
I'm currently trying to move from C# ProcessNet to Python, and have a question regarding the necessity of the "model_document.UpdateDatabaseWindow()" method when the PV is changed.
In my experience, simply using the command "PV.Value = PVvalue" did not update the PV in the model.
In the RD GUI, when we implement a model, we press the "OK" or "Apply" button in the Parametric Value Tab after modifying some values.
This triggers an internal model update (redraw) process.
However, it appears that the update process is not explicitly performed when using ProcessNet.
When I was using PNet in C#, I preferred to use the model document updating or model redraw functions because of the reason.
I often encountered a similar behavior when following the code you provided (RunDOE_GUI).
It would be greatly appreciated if you could share your experience regarding this matter.

Hi, thank you for pointing out the potential issue.
Update
In my environment (RecurDyn V9R5 and 2023), just passing PV.Value=value worked fine.
However, as you have pointed out, this method doesn't seem to seamlessly communicate with the RecurDyn GUI.
I have updated the function modeling.ChangePVvalue to complement the pointed deficiency.
Existing issue
However, this also does not explicitly change PV value in the RecurDyn GUI. As I know, calling model_document.UpdateDatabaseWindow() demands high computational load if the model is large. Therefore if the function ChangePVvalue does not work as expected, it is considerable to additionally call model_document.UpdateDatabaseWindow(). Since the expected behaviors can differ by RecurDyn versions and OS, it will be ultimately the user's choice.


bump ajs-integration and tester version to 3.x

@f2prateek noticed this... should probably upgrade

are there no breaking changes in the upgrade?

@f2prateek I don't think so... I think the main changes were using the new ecommerce events. But this is low priority and we can revisit


Dropped events

This integration is dropping events. I can see events in the Posthog UI which do not show up in snowflake. I reported the bug to Posthog but wanted to post here as well for other folks who are debugging this and trying to figure out what's up.

+1 same issue

I have reached the same conclusion after investigating some missing events in Snowflake. Data shows up in Posthog just fine but some events are missing in Snowflake.
I have done a bit of investigation but I haven't been able to identify a root cause yet:

It looks like the export plugin sometimes either fails to run or has no files to run on. When looking at the Snowflake copy history in the information schema, we can see a drop in copy activity that matches the discrepancy in the data (in one case, I have one day with nearly 0 synced events and just a couple of copy queries on that day). You can look into the copy history with:

SELECT
 FILE_NAME,
 STAGE_LOCATION,
 LAST_LOAD_TIME,
 ROW_COUNT,
 ROW_PARSED,
 ERROR_COUNT,
 initcap(replace(STATUS, '_', ' '), ''),
 PIPE_NAME,
 PIPE_CATALOG_NAME,
 PIPE_SCHEMA_NAME,
 FILE_SIZE,
 FIRST_ERROR_MESSAGE,
 FIRST_ERROR_LINE_NUMBER,
 FIRST_ERROR_CHARACTER_POS,
 FIRST_ERROR_COLUMN_NAME,
 ERROR_LIMIT,
 PIPE_RECEIVED_TIME
FROM
 table(
   "YOUR_POSTHOG_DB".INFORMATION_SCHEMA.COPY_HISTORY(
     table_name => '"YOUR_POSTHOG_DB"."PUBLIC"."EVENTS"',
     start_time => TO_TIMESTAMP_LTZ('2023-05-19T07:00:00.000Z', 'AUTO'),
     end_time => TO_TIMESTAMP_LTZ('2023-05-20T07:00:00.000Z', 'AUTO')
   )
 )
ORDER BY
 LAST_LOAD_TIME DESC

The exportEvents metrics page in Posthog shows no errors. But the app logs show a lot of errors:

Failed to copy snowflake-export-2023-05-31-20230531-222757.117Z-a69fa7c9b0a1589b.csv,
snowflake-export-2023-05-31-20230531-222758.743Z-81912340ca352a99.csv,
snowflake-export-2023-05-31-20230531-222758.992Z-efab940327d1e991.csv,
snowflake-export-2023-05-31-20230531-222800.063Z-75fe2e2bc5d7c61e.csv,
....
from object storage into Snowflake. Retrying in 24576s.

It is unclear to me:

What's causing these copy errors. I haven't found any corresponding logs on Snowflake's end
If these errors are properly retried (they are retried up to 15 times according to the plugin code) and if they actually all made it into Snowflake at some point
What happens if we reach the limit and these don't get inserted into Snowflake. Will it show up as an error on the metrics page?

I noticed that the COPY INTO operation on Snowflake that performs the actual load from the CSV files into the tables, is configured with ON_ERROR = 'skip_file', (doc), which may or may not be a good thing? I'm not a Snowflake expert so I can't tell if this is a desirable behavior. The sure thing is that error mechanisms are not properly documented in this plugin's readme.
@eboswort Did you hear from Posthog? Did you make any progress in figuring out the problem?

Thanks both. It's running behind schedule but at least the bulk of the work was merged in with https://github.com/PostHog/posthog/pull/15649
I still need to hook up Snowflake as well, and get the frontend merged in.

Closing as this plugin has been deprecated in favor of our Snowflake destination for batch exports.


CORE: Remove all files and references to a TripRecipe object - repo, service, domain, controller

Done. Global search was super helpful here


Sort and Sort By Key fail beyond certain size for custom types on NVIDIA and AMD

I was working with std::pair sorting. The kernel fails with Out of Resources error.
This only happens on NVIDIA and AMD. Intel works fine.
A standalone code to test the bug is attached (rename to .cpp).
Compilation command:
g++ -std=c++11 sort_vector.cpp -o sort_vector -I /path/to/compute/include/ -lOpenCL -lm

Gist: https://gist.github.com/shehzan10/938d22d2ed68baf4b48ad85f66a612f1
Attachment: sort_vector.txt

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/33203659-sort-and-sort-by-key-fail-beyond-certain-size-for-custom-types-on-nvidia-and-amd?utm_campaign=plugin&utm_content=tracker%2F402515&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F402515&utm_medium=issues&utm_source=github).

Problem location:
https://github.com/boostorg/compute/blob/master/include/boost/compute/algorithm/detail/insertion_sort.hpp#L137

This is related to https://github.com/boostorg/compute/issues/343. We do not have comparison sort implementation yet. I'll add before July.

Welp, forgot that we reported the other issue as well.

I get the same error when trying to sort vector of any custom type (and obviously custom compare func) with length >  4096 elements. Is there currently any workaround for this?

@dot3bumpmap Currently, there is not a universal workaround for this. I'm working on a comparison sort for GPUs. We also need radix_sort to be rewritten using meta_kernel class.

Fixed, see https://github.com/boostorg/compute/pull/619.

Awesome work! Solves an issue for me.


Redirect URI blocked error

Hi,
Following this documentation, we got the Redirect URI blocked error when trying to login.
On opening the developer console and observing the actual login request, we found the redirect uri parameter to be of the form: https://login.microsoftonline.com/te/your-tenant-name.onmicrosoft.com/oauth2/authresp
When we added this to facebook app's oauth settings, it worked.
Can you please clarify, whether this behaviour is because this documentation is not updated, or is it because we are using the legacy b2c app registration?

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

ID: aeb73901-3320-4d53-767c-bfd03b3b0354
Version Independent ID: 404e18f1-d6da-5654-3e68-8e888e11433e
Content: Set up sign-up and sign-in with a Facebook account - Azure Active Directory B2C
Content Source: articles/active-directory-b2c/active-directory-b2c-setup-fb-app.md
Service: active-directory
Sub-service: b2c
GitHub Login: @mmacy
Microsoft Alias: marsma

@manu4rhyme,
The Redirect URL https://login.microsoftonline.com/te/your-tenant-name.onmicrosoft.com/oauth2/authresp is legacy URL. We are in process of migrating to b2clogin.com domain, which results in the Redirect URL to be https://your-tenant-name.b2clogin.com/your-tenant-name.onmicrosoft.com/oauth2/authresp. Currently both URLs are supported by B2C but going forward, https://login.microsoftonline.com/te/your-tenant-name.onmicrosoft.com/oauth2/authresp is going to be deprecated. The documentation is updated with the new Redirect URL.
In your case, your application is configured to use login.microsoftonline.com domain instead of your-tenant-name.b2clogin.com, which is why the constructed redirect URL is https://login.microsoftonline.com/te/your-tenant-name.onmicrosoft.com/oauth2/authresp. Please update your application to use your-tenant-name.b2clogin.com domain. Refer to https://docs.microsoft.com/en-us/azure/active-directory-b2c/b2clogin for the migration to the b2clogin.com domain.


Publish the Python bindings to PyPI?

I am experimenting with the compilation interface and wanted to create a script in Python to do that. I referred to http://onnx.ai/onnx-mlir/UsingPyRuntime.html and found that I needed to link PyCompile manually for Python to discover it. It would be great if they can be published to PyPI as a package and be used as e.g. onnxmlir.PyCompile.

It would be great if they can be published to PyPI as a package and be used as e.g. onnxmlir.PyCompile.

Anyone knowns how to do this?


"Quantum" random?

How do you implement a quantum random generator? Being quantum is a physical characteristic of the process znd it is hard to achieve in practice but anyway it makes sense only when speaking about physics and typically at either experimental or fundamental level.
You can not achieve a true quantum outcome only by means of a classical software code.
Is it really quantum  what you call quantum ? Or what's it indeed?

moved to talos


Other Models result in error

When I try other models that I have access to I receive an error consistently. Sometimes it starts printing the response, sometimes it only prints the error...
{
  "data": {
    "id": "chatcmpl-91yH74w78DzwnwgfeB8pH1dvl7raP",
    "object": "chat.completion.chunk",
    "created":<PHONE_NUMBER>,
    "model": "gpt-4-0613",
    "system_fingerprint": null,
    "choices": [
      {
        "index": 0,
        "delta": { "content": "Weather" },
        "logprobs": null,
        "finish_reason": null
      }
    ]
  }
}

my toml is set as
api_key = "REDACTED"
model = "gpt-4"
max_tokens = 2048
temperature = 0.8

Same issue here. Gpt-3.5-turbo works with no issues, but as soon as I switch model to gpt-4 I get the same issue as OP. I'm usage tier 1, so I have access to gpt-4.
Am I missing something?

@TheJedinator, @patrykchojecki, @u-foka version 2.0.3 is now available in Brew (https://github.com/Homebrew/homebrew-core/pull/165965). It should be installed automatically when doing brew upgrade.
Please let me know whether the issue is fixed for you all.


Rename Cauchy_diff to CMRF cuqipy, training and papers

I believe this should be closed,
@jakobsj, @nabriis, please feel free to reopen if this is not the case.


AddDate returns wrong result when used outside main()

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go1.8.1
What operating system and processor architecture are you using (go env)?
CentOS Linux release 7.3.1611 (Core)
GOHOSTARCH="amd64"
GOHOSTOS="linux"
What did you do?
https://play.golang.org/p/Ldn92AdcbD
What did you expect to see?
2017-05-15 10:43:18.66663326 +0300 +03
What did you see instead?
2017-05-16 10:43:18.66663326 +0300 +03

AddDate() returns a time.Time variable, you are not using it in your example
See: https://play.golang.org/p/5oTGW3kRPF

Yeah as drmarduk said AddDate does not change the receiver. You have to assign the returned new date to a variable. Closing, since there's no bug.


Selection menu stays after I do a single click on a selection

Hello,
thank you for putting such a neat demo. I found an issue with de-selection.
Once text is selected and you single click on the selection - selection clears, but popup menu remains.

Hello, Thanks for advise ! :)

@VincentLoy Shall I work on this bug/enhancement ?

Hello @madhankumar028 ! Any contribution is welcome ! Just create a new PR and I will make a code review. If everything good, I will merge.
Thanks for your help !

@VincentLoy I am working on it 😊


Incorrect/missing entries

<IP_ADDRESS>/25,US,US-IN,Indianapolis,
Traffic from the US/US-MN/Minneapolis PoP is egressing from at least some IPs in this prefix.
<IP_ADDRESS>/24
Traffic egressing from some variation of this prefix, but it's not defined.
Thank you!

Me this afternoon: "Is this even worth filing an issue over? Does anything actually use this?"
Me this evening: 🤬


Add support for acknowledgments

I want to make a scheme where I can be reasonably certain that n-peers have received a block before I communicate any confidence that a block is safely "committed". This PR goes with the new flag in the protocol handshake and asks peers to acknowledge any blocks they receive. Of course this does not make any guarantees about byzantine peers, so you should only trust the ack if you trust the peer.
Still needs more tests.

landed in latest


Footnotes don't work when embedded in a list

This code (parsed with footnotes: true, of course):
* here is a footnote in a list[^1]

[^1]: my footnote from the list

Results in the following AST:
{"ul", [], [{"li", [], ["here is a footnote in a list[^1]"], %{}}], %{}}
It's swallowing the footnote content and not creating a link in the list item.
I thought I'd try to be helpful by starting a PR on this but not sure it's useful without a solution!

Nice one! 👍👍

appreciated @michaelforrest


Update trace-until-event to use shared code from dotnet/diagnostics

Summary
Now that the shared version of EventMonitoringPassthroughStream is available in public dotnet/diagnostics builds, switch over to using it. NOTE: The shared version splits EventMonitoringPassthroughStream into two separate classes (EventMonitor and PassthroughStream).

Release Notes Entry

Could you target release/7.x and then we'll merge back to main?


Not working with Cordova 9 and XCode 10

I'm using ionic AppFlow and when trying to build a production package with XCode 10.  I'm getting the following error:
` Installing "cordova-plugin-qrscanner" for ios

Installing "cordova-plugin-add-swift-support" for ios
Adding cordova-plugin-qrscanner to package.json
Failed to restore plugin "cordova-plugin-qrscanner" from config.xml. You might need to try adding it again. Error: Using "requireCordovaModule" to load non-cordova module "glob" is not supported. Instead, add this module to your dependencies and use regular "require" to load it`

After reinstalling the plugin the error remains. Ionic team tells me this is due to plugin incompatibility with Cordova 9.
Would much appreciate your comments on this.

See: https://github.com/apache/cordova-plugin-camera/issues/444#issuecomment-484002970
"Uses requireCordovaModule on line 5 and 6 for non-Cordova modules. (seen in Windows platform)"
"You will need to ask the plugin developers to replace requireCordovaModule with require for the non-Cordova modules."

Currently using Cordova 9 and targeting Windows 10 as well. This fix would surely be a great help.


Profile picture and updated menu

Züri Trash Bot is missing a profile picture. Also, the current menu is kind of boring. Let's add some emojis to lighten things up.

Profile picture added today. Menu update scheduled with version 2.0 (PR #13)


Copr build process stuck at "Pending — Starting RPM build... "

Under https://github.com/oamg/convert2rhel/pull/122, the copr builds are not being created. The last message from packit is "Pending — Starting RPM build... ".
When looking at the linked copr build job (https://copr.fedorainfracloud.org/coprs/g/oamg/convert2rhel/build/1775484/), it says "importing - Package sources are being imported into Copr DistGit."

Hmm, looks like some problem on Copr side -- please, let us know if this happens again. The /packit build should help (like you've tried).
I am moving the issue to the packit-service repo...


bring back the github.com in url, its neeed at the scm stage

Confirmed, fixes issue from https://ci.centos.org/view/Devtools/job/devtools-almighty-core-build-master/82/console in https://ci.centos.org/view/Devtools/job/devtools-almighty-core-build-master/83/console


Checking for user-defined cookies

Hi there,
I'm trying to set a cookie under "Advanced Settings" -> "Don't cache pages that use the following cookies" . But it looks like this configuration does not take that into account, and the pages are cached nonetheless.
Is there a way to fix this?

Unfortunately, the script cannot be dynamic and adjust the configuration by entering new values like you are doing.
You will have to alter the cookie management section and add your cookies there (cookie1|cookie2|...) at line 98 of the Rocket-Nginx configuration.
If a contributor would help with issue #38, what you are asking could be automated in a generation script.

Hmm, I figured there would be no way. But changing this config file could cause conflicts with future updates from this repository.
Maybe there's a more elegant solution? Maybe including an external file in the config which is added to this repository's .gitignore which has a list of all cookies?
Not sure if that would be the best way to do it, just a suggestion!

Of course, editing the configuration manually is not the best option in the long run.  But it's an option for now.
When the configuration generator will be completed, it will be easier to make these kinds of modifications.

Fixed in 2.0


Examples bind

https://github.com/qt/qtbase/tree/dev/examples

https://github.com/qt/qtbase/tree/dev/examples%2Fwidgets%2Fitemviews


Update authorization-code-flow.md

As per internal testing and confirmation of the Oauth2.0 specification, scope is not a required parameter for /token endpoint calls regardless of whether the grant_type is for an authorization_code or for a refresh_token

Hello Kenga, please feel free to reach me at my alias aubeauch on Teams.

@kengaderdus

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Aubeauch:
Thanks for your contribution! #sign-off.


Code completion is not suggesting argument label of existing argument

The following test case only suggests lineCap and not dash, which is causing a stress tester failure. One could argue that the current behavior is expected, which would mean that the stress tester needs to be updated.
func test(lineCap: Int = 1, dash: Int = 1) {}
test(#^COMPLETE^# dash: 1)

This change in behavior is caused by https://github.com/apple/swift/pull/71054

Hm, I didn't think about it deeply when I reviewed the PR, but I believe suggesting the entire call pattern was the expected behavior, because for example:
func foo(arg: Int, option: Opiton) -> Int
func bar(arg: Int, option: Option) -> Int

bar(arg: foo(#^COMPLETE^#, option: .default)

we can't decide the option: .default was meant for  foo call, or the outer bar call.


Implement VTRANSFORM opcode

Fixes #831.
 It needs testing to make sure it's actually implemented properly.

@thegrb93 this is broken with the revert commit you added - the generated code calls VM:ReadVector(address_1, VM.VMODE), which was a new function added in the first commit and that you removed in the second one.

Ah, I'll fix that.


Permission Issue assetstore in DSA Girder and Mongo DB on Docker

In Docker Compose, I tried to use Mongo DB but I still can't access the /data/db/. I think it's because of the permission of the Mongo DB's DSA in Docker Compose. When I comment it, I can access the Mongo DB. The service is running fine. But then the DSA Girder 1 is not working. This happens after I clone this repository to the DSA repository. I still can't run the frontend apps until now. I tried to uninstall the Docker and change the configuration of the Docker Compose, but nothing works.
MongoDB Service:

Mongo wants to have access to /data/db/journal.
I manually create the journal directory, the error changed to WiredTiger permission denied.
I changed the Docker Compose, removing the user from the Mongo DB service works. But now the file is owned by Mongo not the host.

Girder Service:

Unable to write into directory /assetstore in Girder

Setting DSA_USER variable from the shell or providing .env file doesn't work.

Hi, Is it possible to go back to the default configuration? if so, then to resolve the first error that cannot access the /db folder. I have faced this problem before. So as a simple solution I have changed the permission of the folder using this command sudo chmod -R 777 db/ from inside the dsa folder. then try to use docker compose again, but before using docker compose up you can use docker-compose down -v to remove preconfigured images.


Bump WindowsAppSdk and Win2D versions

As the title says. The Win2D package removes the C++ dependency and that makes our Windows publishing story for .NET MAUI better.

@jfversluis did you update the project references?
You might need to search as this is in a few places.

Of course: https://github.com/dotnet/Microsoft.Maui.Graphics/pull/415
🙄


RFC: Link for Software Testing course is broken - 404 (under Advanced programming section)

The link for the software testing course is broken. I searched for it online, but I think the course might not be available anymore for free.
If the course is still available could somebody fix the link, if not we would need to find a replacement.
Thanks in advance,
Sven

Hey

Closing as a duplicate of this open issue: https://github.com/ossu/computer-science/issues/1179
Note that the curriculum was just updated with the video playlist of the previous course.


Add Performance.now and add time stamp information to requestAnimationFrame bindings

More info in commit log entries.

Was this meant to be closed? I don't see my commits in any branches.

the improved-base branch was removed (since it's been merged into master), which is why all pull requests to this branch were automatically closed. Can you resubmit a pr on master?

Oops, sorry, thought I got to all the PRs that got closed inadvertently.


LibreSpeed: Upload speed reported to high

Allgemeines

[X] Ich habe auf die neuste Version von MySpeed aktualisiert.
[X] Mein Bug wurde noch nicht gemeldet

Der Fehler
I have a cable connection providing 50 mbit upstream. When choosing LibreSpeed as provider it reports about 100 mbit so about double the maximum speed possible.

Im Browser
Auf welchem Betriebssystem läuft deine MySpeed-Instanz?
Linux

Hi, sorry for the late reply. Just to verify: Could you try running a speedtest directly using the LibreSpeed CLI?
You can either do this with the docker run --rm ghcr.io/danieletorelli/librespeed-cli command or by downloading the CLI from their releases and running it manually.
Also, does the same behavior happen when using different servers?

Tried the librespeed container directly. On my NAS the upload speed is
bogus. On my PC though the upload speed is shown correctly.
Am Fr., 21. Juni 2024 um 10:15 Uhr schrieb Mathias Wagner <
@.***>:

Hi, sorry for the late reply. Just to verify: Could you try running a
speedtest directly using the LibreSpeed CLI?
You can either do this with the docker run --rm
ghcr.io/danieletorelli/librespeed-cli command or by downloading the CLI from
their releases
https://github.com/librespeed/speedtest-cli/releases/latest and running
it manually
https://github.com/librespeed/speedtest-cli?tab=readme-ov-file#usage.
Also, does the same behavior happen when using different servers?
—
Reply to this email directly, view it on GitHub
https://github.com/gnmyt/myspeed/issues/694#issuecomment-2182242893, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSDFUCVWU5RD34ZVSEUKJDZIPOKBAVCNFSM6AAAAABIK3LMVSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBSGI2DEOBZGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Create timeline panel

Please read the help wanted guide before start.

Create react (preact) component of the timeline panel. It should take 100% of parent width.
The precise sizes in pixels:

Colors:

small dash: #AE9BAE
large dash and number label: white
purple background: #512750

Shadow of the panel: 0 2px 4px black
All time labels are exactly in the middle of the large dash.
Requirements for the component
As you can notice from size illustration, gap between dashes is 4px, it should be defined in em units. So if you set font-size on the root of the component to 10px, the gap should be set to 0.4em so the result will be equal to 4px.
Having that component will receive scale property and the timeline will be scaled accordingly, simply by setting root's font-size to appropriate value. For instance, if scale property is 1.5 the font-size should be set to 10 * 1.5 = 15px. This will allow to scale the timeline by means of em units. All other sizes must not use ems, only the size of dash gap should scale.
The timeline should cover at least 20 seconds or 20 000 milliseconds.
Entire sketch mockup could be found here: link and here link.
Cheers! ~

Will do


Add segment membership in Profile schema

https://github.com/adobe/xdm/issues/379

@kstreeter can you please put this in the June 15th milestone. We have been working on this schema for several months across the teams on segmentation.

@harleensahni @cdegroot-adobe, It seems like one field that may be missing is the producer/namespace of the segment.  How would I know a segment came from AAM vs AA vs Target?

@cluby-adobe I feel the producer/namespace is part of segment definition rather than the actual realization of segment.

@cluby-adobe I feel the producer/namespace is part of segment definition rather than the actual realization of segment.

+1

I'm generally ok with this PR, but think that we should use the less technical term "audience" instead of the more technical term "segment" here.
In some of the comments, we are speaking of "entities" being part of a "segment" which obfuscates the fact that we have people or profiles that are part of an audience.

@trieloff - segment vs audience is a context thing. The terminology int his area is a inconsistent in normal day to day usage, but we are being specifically consistent here.
Correct Definitions:

A Profile qualifies for Segments.
A list of Profiles that all qualify for the same Segment is an Audience.
The rule that defines if a Profile qualifies for a Segment or not is a Segment Definition.

There are a couple of small fixes that @cdegroot-adobe called out we need to fix, but this otherwise looks good.  @cluby-adobe, @harleensahni are you guys good with the latest here?

Hi @rkpandey, please make the small changes @cdegroot-adobe suggested regarding the file names, and fix the conflicts on package.json.  I will then pull this in.  This is the last item in the milestone so if you can do that early today we will tag the milestone later today.  Thanks!


"allow" does not allow in the same phase

SecAction "phase:1,allow"
SecAction "phase:1,deny"
=> denied
msr->allow_scope is only checked at the beginning of the phases, but it doesn't stop the current processing.

Cannot reproduce it with trunk

Hi @marcstern,
Can you give us a little more details on this issue report? I am not sure if I understood. Keep in mind that those reports will be here for the community, as a knowledge base. Therefore, there will be good to have a very detailed description of the issue that you are facing.

Cannot reproduce in httpd 2.4 with MS 2.9.2 => ignore this


The deps.rs site is down and just returns 503 Service Temporarily Unavailable

I want to report the same on my side.

How can we help getting it back up?

We're already working in restoring the service


184 fix unit tests

Fixing the unit tests for issue 184. They were missing a manual fail when they didn't throw an exception.

Fixes #184


Launch Rstudio add-in gives "Error: RStudio not running"

Hi, I did what is suggested in the document to activate Rstudio addin support.
After I call the command R: launch Rstudio addin, I can see a list of addins in the command palette.

But after I select it, it gives me an error in the terminal
[ins] r$> xaringan:::inf_mr()
Error: RStudio not running

I also tried other addins listed, but they give me the same error...
Did I miss anything here?
Thank you!

This addin works for me.
The error suggests the rstudioapi functions have not been loaded by VSCode, however the fact that you are able to get to the addin menu suggests they have.
I can reproduce your behaviour by doing the following:

Start an R terminal with the VSCode extension using either command R: Attach active terminal or R: Create new terminal.
Create a new default terminal (running bash or powershell) and run the R executable with the command R or R.exe
With the terminal from step 2. active, run the addin from the menu and get your error.

The addin picker just passes a command to be run in the active terminal. If that active terminal is not one that has been initialised (attached) by the VSCode-R extension, RStudio addins, and many other VSCode-R extension features, will not work as expected.
Could running an R session not initialised by the vscode-R extension be a contributing factor in your case?

Thank you @MilesMcBain
I tried again, and it was not the scenario you described, as esquisser works
[ins] r$> .vsc.attach()

[ins] r$> xaringan:::inf_mr()
Error: RStudio not running

[ins] r$> esquisse:::esquisser()
Loading required package: shiny

Listening on http://<IP_ADDRESS>:5829
Browsing http://<IP_ADDRESS>:5829

Could it be something related to my .Rprofile or VS-code setting? Also, you mentioned this addin works for you, did you use the newest version? As I just installed it yesterday, it might be something about it?
remotes::install_github('yihui/xaringan')

Thank you so much!

If xaringan were doing something weird in the latest version I would expect a different error about rstudioapi functionality not supported by vscode.
I'm thinking it could be your .Rprofile. I've seen situations where things in the profile cause the attach process to terminate abnormally.
Do you mind sharing what you have in there?

Ah-ha, I figured it out, but have no idea why.
So I played with my .Rprofile for a while, and found out that it was due to library(tidyverse).
After I comment out this single line, the error disappeared.

Oh yes okay this makes sense to me. On certain platforms tidyverse makes a call to see if the rstudioapi is available and what version it is for the purposes of working out how many terminal colours it can use and if icons are supported.
The rstudioapi emulation works by injecting functions into the package namespace as part of a custom 'on load' hook. So it seems that because the tidyverse already loaded the rstudioapi namespace, our custom hook doesn't get a chance to fire.
We might be able to cater for this by running the hook manually if we detect the namespace is already loaded.

fixed in #610


[diversity] Use inclusive language

As discussed in the TSC meeting on 2020-07-27 and the ASWF #diversity Slack channel, the term "sanity check" should be avoided as it is considered ableist language. I checked the OCIO source code for other terms, but there are just a few ones that should potentially be substituted.

sanity check: validate (perform check), smoke test, validity / plausibility / soundness check
sane, insane, sanity: valid, invalid, validity (syntactic?)
dumb: basic? (draft implementation)
lame-ass: boring, unpleasant, crude? (support code)
whitelist: approved list, allow-list (CLA)
master: main? (Git branch)

1 and 2 are urgent to be renamed in time for v2, as they are used in a public interface. 3, 4 and 5 are trivial to replace. 6 would be a huge task... related: https://github.com/github/roadmap/issues/63
I also found the terms stupid, crazy and crippled in third-party code (comments/test data), but changing that seems out of scope.

Hi @Simran-B, Can we close the issue?

Yes and no... it is kind of a reminder for me to also check other ASWF projects and add something about inclusive language to the documentation.

If you want to keep track of other ASWF projects, the best would be to create a defect in their repo.
On the other hand having a disclaimer in the documentation is a great idea.

Can I work in this issue?

@Simran-B , we have a volunteer for this issue.  I know a lot of this has already been implemented.  Please let us know if there is anything left to do or else close the issue.  Thx!


patch mathigon studio

Fixes #799
Changes
patch the progress model in @mathigon/studio by changing the index of the progresses collection
Implementation details
the progress model includes userId and courseId fields. however its index was referencing user and course.  as a result, when storing progress data it would fail. the field names in the index need to be updating to accurately coincide with the model field names.

if this PR gets merged the progresses collection index will need to be deleted so new one is created with this update

good catch here 👀


Add HubConnectionBuilder to Java Client

It's not all that configurable yet but the experience should be there as we bring this closer to the other clients.

Done here: https://github.com/aspnet/SignalR/commit/94f99c3a29786fc912ca5a51460229ce4b23a600


Custom Data notebook: sacremoses tokenization fix, sacrebleu tokenization in yaml, other fixes.

Apparently it's supposed to be sacremoses -l eng tokenize file.txt, not sacremoses tokenize -l eng file.txt
added a mkdir command for saving to drive after training.
Adding quotations around bash variables. This seems to have improved the notebook's ability to handle spaces in file paths.
Set default Sacrebleu tokenizer for test data to "none", and added comment.
small typo fixes, etc.

:100: Thanks so much @ColinLeongUDRI!!


difference between Aws Iot Shadow _Get() and  SHADOW _Get() how they works

This is not any issue its just informational question i know i can put this on stack overflow but i just thought it is place where i can find perfect answer,
i just want know there working and definition  if both are same then why two ?

Hi @sameerdasarwad
SHADOW_Get belongs to a higher level aws_shadow.h API that uses the aws_iot_shadow.h API containing AwsIotShadow_Get.
They both can be used to get data related to AwsIotShadowDocumentInfo_t:
typedef struct AwsIotShadowDocumentInfo
{
    const char * pThingName; /**< @brief The Thing Name associated with this Shadow document. */
    size_t thingNameLength;  /**< @brief Length of #AwsIotShadowDocumentInfo_t.pThingName. */

    IotMqttQos_t qos;        /**< @brief QoS when sending a Shadow get or update message. See #IotMqttPublishInfo_t.qos. */
    uint32_t retryLimit;     /**< @brief Maximum number of retries for a Shadow get or update message. See #IotMqttPublishInfo_t.retryLimit. */
    uint32_t retryMs;        /**< @brief First retry time for a Shadow get or update message. See IotMqttPublishInfo_t.retryMs. */

    union
    {
        /* Valid for Shadow get. */
        struct
        {
            /**
             * @brief Function to allocate memory for an incoming Shadow document.
             *
             * This only needs to be set if #AWS_IOT_SHADOW_FLAG_WAITABLE is passed to
             * @ref shadow_function_get.
             */
            void * ( *mallocDocument )( size_t );
        } get; /**< @brief Valid members for @ref shadow_function_get. */

        /* Valid for Shadow update. */
        struct
        {
            const char * pUpdateDocument; /**< @brief The Shadow document to send in the update. */
            size_t updateDocumentLength;  /**< @brief Length of Shadow update document. */
        } update;                         /**< @brief Valid members for @ref shadow_function_update. */
    } u;                                  /**< @brief Valid member depends on operation type. */
} AwsIotShadowDocumentInfo_t;

however SHADOW_Get only utilizes a subset of AwsIotShadowDocumentInfo_t as it uses ShadowOperationsParams_t to store its results.
typedef struct ShadowOperationParams
{
    /** @brief Thing name of Shadow. */
    const char * pcThingName;

    /**
     * @brief Input parameter for #SHADOW_Update; output parameter of #SHADOW_Get.
     * - For #SHADOW_Update, set this to point to the Shadow update document.
     * - For #SHADOW_Get, this is the output parameter pointing to a buffer which
     * holds the Shadow document. If #SHADOW_Get fails, this will be set to NULL.
     * - This member is ignored by #SHADOW_Delete. */
    const char * pcData;

    /**
     * @brief Input parameter for #SHADOW_Update; output parameter of #SHADOW_Get.
     * - For #SHADOW_Update, set this to length of the Shadow update document.
     * - For #SHADOW_Get, this is the output parameter specifying the length of
     * #ShadowOperationParams_t.pcData. If #SHADOW_Get fails, this will be set
     * to @c 0.
     * @note If the received Shadow document is longer than this value, the
     * document will be truncated. A warning message will be printed if
     * #shadowENABLE_DEBUG_LOGS is @c 1.
     * - This member is ignored by #SHADOW_Delete. */
    uint32_t ulDataLength;

    /**
     * @brief Output parameter of #SHADOW_Get.
     * - This member is ignored by #SHADOW_Update.
     * - For #SHADOW_Get, this is the output parameter for the MQTT Buffer Handle.
     * This buffer must be returned to the MQTT Client after the user is done
     * with the received document by calling #SHADOW_ReturnMQTTBuffer.
     * - This member is ignored by #SHADOW_Delete. */
    MQTTBufferHandle_t xBuffer;

    /**
     * @brief Controls whether topic subscriptions remain active after a Shadow
     * operation completes.
     *
     * Allows the MQTT subscriptions of the operation to remain active if set
     * to @c 1, saving time if the same operation is performed again. Set this
     * value to @c 0 to deactivate the operation's MQTT subscriptions after the
     * operation completes.
     * @warning Users may be billed for extraneous messages received on an
     * operation's MQTT topics. If other clients are publishing to the same topics,
     * it is best to deactivate the subscriptions. */
    uint8_t ucKeepSubscriptions;

    /**
     * @brief MQTT qos when publishing from the Shadow Client @b to the Shadow Service.
     * @note The Shadow services always publishes to the Shadow Client with qos 1. */
    MQTTQoS_t xQoS;
} ShadowOperationParams_t;

You can see SHADOW_Get mapping these data members here and here, for example.
Besides their different interfaces, a key difference is that SHADOW_Get is a blocking function that internally calls AwsIotShadow_TimedGet to implement a timeout whereas AwsIotShadow_Get is non-blocking and can invoke a callback once the Thing's data is retrieved.
As far as when to use one or the other, that depends on if/how you want to handle timeouts for retrieving the Thing's data.

Hi thanks for valuable information i was little confused while looking into conveyor, drink dispenser and aws shadow demo thanks for clarification

Hello @sameerdasarwad,
Thanks for  letting us know that the information helped. I'm closing this issue, if you still have questions regarding this issue,  please feel free to re-open the issue or create a new one.
Thanks !


How to parse element name having specialCharacters

Your JSON dictionary:
How do i parse json element name Start with special character like "@"
{
  "@name": "ObjectMapper",
  "@url": "https://github.com/Hearst-DD/ObjectMapper"
}

Your model:
struct Repo: Mappable {
  var name: String!
  var url: URL!

  init(_ map: Map) {
    name <- map["@name"]
    url <- map["@url"]
  }
}

What you did:
let repo = Mapper<Repo>().map(myJSONDictionary)

What you expected:
I expected something like:
Repo(name: "ObjectMapper", url: "https://github.com/Hearst-DD/ObjectMapper")

What you got:
Repo(name : nil , url : nil)

I have just tested a similar scenario locally and it is working just fine for me. Is this still an issue for you?


Failed to read SASL handshake header: EOF

Issue

When trying to start a bridge with kafka

Info

Running consortium through Kaleido
Authentication details fetched from one of the nodes in the consortium

Command

./ethconnect kafka -T [REPLIES] -t [REQUESTS] -b [BROKER] -u [USERNAME] -p [PASSWORD] -g consensys-solutions -r [RPC_URL] -d 2

Log

DEBU[2019-02-12T14:15:04Z] Log level set to 2
INFO[2019-02-12T14:15:04Z] Starting Kafka bridge
DEBU[2019-02-12T14:15:04Z] JSON/RPC connected.
DEBU[2019-02-12T14:15:04Z] Kafka TLS Enabled=false Insecure=false MutualAuth=false ClientCertsFile= PrivateKeyFile= CACertsFile=
DEBU[2019-02-12T14:15:04Z] Kafka ClientID: 42aee518-8fa4-4c28-4f40-8f20efa13d88
DEBU[2019-02-12T14:15:04Z] Kafka Bootstrap brokers: [kafka.eu-central-1.kaleido.io:9093]
DEBU[2019-02-12T14:15:04Z] [sarama] Initializing new client
DEBU[2019-02-12T14:15:04Z] [sarama] client/metadata fetching metadata for all topics from broker kafka.eu-central-1.kaleido.io:9093
DEBU[2019-02-12T14:15:04Z] [sarama] Failed to read SASL handshake header : EOF
kafka: client has run out of available brokers to talk to (Is your cluster reachable?)

Figured it out. Missing -e option (TLS)

Glad you got this working @ehanoc 👍
If you continue to experience any issues specific to use of ethconnect with the Kaleido SaaS platform, it's probably best to raise a ticket via<EMAIL_ADDRESS>


Added possibility to use gcs push endpoint on pubsub subscription

Checklist

I applicable, I acknowledge that I have:

[X] Read the contributing guide
[X] Ran terraform fmt on all modified files
[X] Regenerated the relevant README.md files using tools/tfdoc.py
[X] Made sure all relevant tests pass

thanks @luigi-bitonti! this lgtm.

It's a pleasure. Do I have to fmt also the documentation? I see that it didn't pass a check about formattation.

Yes, just run your code through terraform fmt


reopen 182

The bot closed issue #182 while it is a valid issue (about EKSA on M1 macs). Please reopen it.

See the response on #182


Better status checking by ArgoCD on cluster creation via RHACM

Improve how ArgoCD check for the successful completion of a cluster deployment via RHACM.
Currently ArgoCD does not check on the status of the Cluster once it begins is cluster deployment.
A method should be created to do a post sync to check success or failure of deployment.

Argo Application Health Check has been added to check status of cluster deployment and update health check upon success/failure.
Closing issues