Document the meaning of icons

First off, thank you for making such a useful and easy to use extension for testing.
I don't understand the meaning of this icon:

Would it be possible to document in the README.md the meanings of all the different icons this extension provides?

Hi there @loligans,
Really sorry for the delay in responding to this.
This is a great idea! With that said, these are actually part of the UI for VSCode Test Explorer, which is the parent extension we've created an adapter for. We don't have much control over this as it's a separate project entirely.
With that said, from the looks of it, the icon you're pointing out is this one:
https://github.com/hbenl/vscode-test-explorer/blob/master/icons/pending.svg
Pending is a bit of an odd name, but it effectively means "this test has never been run in the time since it was loaded into the explorer. One you press play on a test or a test suite, other state icons showing success, failure or a currently running status will be shown.
I've added an issue in the test explorer repository (https://github.com/hbenl/vscode-test-explorer/issues/171) to see if there's a good document that already exists for us to point people to. I'll leave this issue open while we wait for a response, if there isn't one already, it might be something we can produce as a document for the test explorer directly.


Reorganize Volumes Page for Readability

This PR is a rewrite of the concepts/storage/volumes page. I've reorganized it for improved readability.

The changed page
https://deploy-preview-24574--kubernetes-io-master-staging.netlify.app/docs/concepts/storage/volumes/

Related to PR #24077

Hi @kbhawkey
I sent you an email (address from latest commit), about coordinating our efforts. I'd also be happy to communicate on here.
Thanks!

👀

Um, not sure why you are merging in my branch?

I simply wanted to get the changes onto my machine without setting up another upstream/remote, so I just did a cross repo PR with the github gui. Would it have been better to rebase? I assumed that since I would eventually sqash and rebase onto the master branch, the particular method wasn't important.

I simply wanted to get the changes onto my machine without setting up another upstream/remote, so I just did a cross repo PR with the github gui. Would it have been better to rebase? I assumed that since I would eventually sqash and rebase onto the master branch, the particular method wasn't important.

Hi @geoffcline .
There is no need to merge my changes into your branch as my branch has not been merged to master and may never merge to master. It all depends if the changes are worthwhile merging into the docs.
I suggest that you work on your pull request, get the changes to a state where you are happy, get the changes reviewed.
If your branch merges to master before my pull request and I want to continue to work on my changes, I will need to rebase.
It is unfortunate that the PRs came in at the same time and that the changes are not straightforward.
Do you understand?

/cc @kubernetes/sig-storage-pr-reviews

Yes, I understand
I have gone ahead and just closed the PR, to work in private

/remove-area blog
/remove-language es fr id it ja ko ru zh


Small url change that allow everyone to access the messenger app from we...

...b
Details:
Some users (myself included) can only access the app on the url messenger.com/new. Without the "/new" they're redirected to the page where you can download the app for mobile platforms.

This unfortunately interferes with https://github.com/rsms/fb-mac-messenger/commit/606b0ffeafcb884159fb6df6fb3d769aaf200334
One solution could be to detect redirect to wherever you are redirected to with the marketing stuff and at that point change URL to /new.

606b0ff actually had the same effect of my PR, haha.
Thanks for the great work. Nice job!


Error text shown above and in top bar; top and bottom bars transparent

I am assuming that #21 was referencing this problem as well though it was not expanded upon. I have posted a screenshot for clarity:

Interestingly, for some reason, the bottom bar does not show up in the screenshot. It looks similar to the top bar except that there is no error text and only a power off symbol shown in the center with nothing else. The picture was cropped when I was looking at it; the bottom bar does show up in the image above.
I installed Übersicht through its desktop installer (I can't find the Homebrew package...) disabled my desktop using the provided command, and installed supernerd through cloning the repository to the Übersicht widgets folder as outlined in the README. Other widgets are working properly, so this doesn't seem to be an installation method issue or one with Übersicht itself, I believe. I have disabled all other Übersicht packages for the moment for troubleshooting and that has not helped.
I use Little Snitch for network monitoring. I tried to disable this to ensure it wasn't breaking any connections which caused downstream errors in supernerd. After disabling network monitoring and both restarting and refreshing Übersicht, the error persisted.
The animations are functional (mousing over the volume icon reveals volume percentage, etc.) but are likely not properly styled.
No modifications were made to the provided scripts. I am not certain whether the error text visible in the interface contains the entire error or only part of it (probably the latter).
Assistance would be much appreciated!

Thank you for the quick response!
I had used the brew install ubersicht command instead of brew cask install ubersicht before, which is why I could not find the package. I uninstalled the old Übersicht package installed from the desktop installer through CleanMyMac to (hopefully) get rid of any configuration files as well and have reinstalled Übersicht through the brew cask install ubersicht command.
I then checked the Übersicht widgets folder (default location at $HOME/Application\ Support/Übersicht/widgets) and confirmed that it was empty. I issued the provided git clone command from the README to clone the repository to this directory. Here is a screenshot of the git clone command and tree on the Übersicht directory:

(note: username redacted, but the path is simply the expansion of $HOME)
Even after all of this, the errors persist, unfortunately. However, there now seems to be overlapping text at the very top-left corner of the screen instead:

(note: the bottom bar remains the same as in the initial screenshot post so I have only posted the relevant top portion of the screen)
OS & app versions:

macOS 10.13.4
Übersicht Version 1.2

I notice in the error at the top-left of the top bar that it mentions: chunkc: connection failed! Is chunkwm required to run this widget? I have it installed (also through homebrew) but disabled it until I could work out some conflicting keyboard shortcuts.
P.S. Your last README commit seems to have creates a huge code chunk object of all text after your modified brew cask install ubersicht command! Just wanted to point that out here instead of bothering you with another issue :+)

Thank you for the quick response!
No worries.

Ok so thanks for getting that tree, it definitely rules out a bunch of possible issues with the file positioning. So this brings us to something you mentioned...

I notice in the error at the top-left of the top bar that it mentions: chunkc: connection failed! Is chunkwm required to run this widget? I have it installed (also through homebrew) but disabled it until I could work out some conflicting keyboard shortcuts.

Yes! That's where it gets the focused desktop information. And I'm going to go ahead and assume that all of the other errors might just be referencing scripts you are not running. So, for instance, it seems that you are running the widget that controls mpd through mcp – if you are not running either, that could be the cause of the error. So to sum up:

chunkwm is required to get the focused desktop indication
mpd and mcp are required to get the music indicator. If you don't use mpc/mcp you should disable apps-and-music.coffee (for instance by renaming it to apps-and-music.coffee.optional) and enable apps-and-itunes.coffee.optional (by renaming it to apps-and-itunes.coffee).

With regards to the disappearing bars, the issue seems to be that you are not using wal, but your css-loader.coffee is looking for wal's output to gather colors from it. If you don't want to use wal, you should open up css-loader.coffee and change this line:
    @import url(supernerd.widget/styles/colors-wal.css);
to this:
   @import url(supernerd.widget/styles/colors.css);
Then you can edit the colors by editing colors.css.
Let me know if it works!

P.S. Your last README commit seems to have creates a huge code chunk object of all text after your modified brew cask install ubersicht command!
I disabled apps-and-music.coffee, enabled apps-and-itunes.coffee, made the suggested changes to css-loader.coffee, and started skhd and chunkwm.
Colors work now, yay! I'll get around to tinkering with wal later but it's good to know it works without it.
There seems to be some build error with apps-and-itunes.coffee. No modifications were made to this file other than removing the trailing ".optional" portion of the original name to activate the module. While the module is active, a text error persists in the top-left corner as pictured in the previous screenshots but with some nondescript error (but here's a picture, just in case):

Disabling apps-and-itunes.coffee results in that error text disappearing. Even while the above error is showing, playing a song in iTunes results in unformatted plain text displayed after the error message as [artist name] - [track name].
Similarly, enabling time-date.coffee gives the following error, while disabling it results in that error disappearing from the top-left of the screen:

Ok, so, mixed response here.

time-date.coffee: I'll have to do a mea culpa here. I did not maintain this at all because I maybe maybe might have completely forgotten that this widget existed at all. I'm actually going to remove it because it makes no sense to have it in given how everything else is set up, until I can actually work out a more modular system that let's users mix and match every indicator exactly how they want to. I recommend you use date-weather.coffee instead.

apps-and-itunes.coffee: Weird, because my apps-and-itunes works fine. So now I'm wondering if it might depend on whether you have or not all of the apps installed. Do you not have Spotify installed? If that's the culprit I will have to split it in two separate widgets, one for iTunes, one for Spotify...

I think there might be an issue upstream here – I don't know how to handle errors because it seems like the ubersicht code shits itself before the issue can be handled by my code.
So I was messing around with different frameworks, and I got a bit carried away so I remade the entire bar as a npm module. This handles error extremely well. If you have the time I'd love for you to check it out, I'm now evaluating if I should consider switching development altogether.


Add Vlad Gheorghiu to the roster

I forgot to update the readme.

Thanks Ry!  Will you be able to add him to the tsc team as well?

@dstebila please see #16


Breakpoints for uncaught exceptions do not get triggered for extension development

VSCode Version: Code 1.15.1 (41abd21afdf7424c89319ee7cb0445cc6f376959, 2017-08-16T17:15:57.756Z)
OS Version: Darwin x64 16.7.0
Extensions:

Extension
Author (truncated)
Version

ActiveFileInStatusBar
Ros
1.0.3

find-word-at-cursor
mks
1.8.0

code-spell-checker
str
1.3.8

I'm assuming that when "Uncaught Exceptions" is checked under "Breakpoints" in the debug view, it will cause a breakpoint to get automatically set and execution will stop when an uncaught exception is encountered so that the developer can debug the issue.
That's not the behavior I'm observing. Instead, I see this in the Extension Development Host:

...and no breakpoint is set in the VS Code window where the source code is opened.
Steps to Reproduce:

Use yo code to generate an extension.
Open the extension in VS Code
Add bad code some where or throw an error
Run the extension with F5
Call the extension with extension.sayHello
No breakpoint gets triggered even though "Uncaught Exceptions" is checked (by default)

@msafi please try reproducing this with vscode insiders since we have changed our debug backend for extension debugging.
https://code.visualstudio.com/insiders
If you still see the issue then @roblourens can investigate more.

@isidorn @roblourens I tried it in Code Insiders. Same problem.
See this video

But the exception is caught, by vscode's code. It must be, for the error dialog to be shown.  It's too much noise. Not really usable.
I feel the current design harms the extension authoring experience. For example, if you don't show the dialog or some how allow the error to propagate up so that the debugger will consider it uncaught, that will make things better.
Can't you rethrow the error after showing the dialog so that the debugger will consider it uncaught or maybe change the behavior of VS Code so that when it's an Extension Development Host and the error is generated by the extension being hosted, it doesn't catch the error?

You bring up some good points, I think we can make this better. I think the best way to do that is to provide a way to not break on exceptions in code that isn't part of the user's extension, by our own definition, with the "skipfiles" feature.
@weinand "Toggle skipping file" isn't enabled for extensionHost debug, can we change that? https://github.com/Microsoft/vscode-node-debug/blob/master/package.json#L113
Debugging with "All exceptions" enable is annoying, I break on exceptions from other vscode EH code, our builtin extensions (especially TS), and other random extensions I've installed. We could introduce another special skipfiles namespace like <eval> and <node_internals>, that would cover all the EH code that isn't part of the currently debugged extension. Or a couple new ones, <extensionhost_internals>, <other_extensions>. And let users skip exceptions in those files.

Got here after searching with the exact same problem as @roblourens - Totally agree with what he's said above and it would be fantastic to be able to focus entirely on my own code even with "All exceptions" and/or "Promise rejects" checked while debugging. There's currently far too much noise with other extensions, node internals etc. being caught by the debugger.


e2e: remove white spaces in username

What does this PR do?
The username can contain a white space so I removed it to avoid any failure on the aws ec2 create-tags call.
aws ec2 create-tags --resources [...] --region us-east-1 --tags Key=repository,Value=github.com/DataDog/datadog-agent Key=branch,Value=HEAD Key=commit,Value=3aef42c7 Key=user,Value=Julien Balestra

Error parsing parameter '--tags': Expected: '=', received: 'EOF' for input:
Balestra

Note :disappointed: :
This is super strange because I never had this issue before (I didn't change my own username).

Succeeded on the branch.


Support dynamic app templates

Allows dynamic app templates that we can pass Javascript variables to.
Also fixes default template for multiple environments.
Had this idea from the discussion in https://github.com/mui/mui-toolpad/pull/1429, should also be a long-term fix for these types of issues.

This makes it impossible to update the template by importing it in toolpad, changing and exporting again.
Let's keep it simple? We could just remove the map with paths and replace it with functions that load the correct template
const TEMPLATES = {
  defaultTemplate: async () => {
    const tpl = await readJson(path.resolve(__dirname, './default.json'))
    tpl.nodes['1313wn3'].attributes.query.value.url.value = DEFAULT_TEMPLATE_DATA_URL
    return tpl
  },
  imagesTemplate = async () => readJson(path.resolve(__dirname, './images.json'))
}

This makes it impossible to update the template by importing it in toolpad, changing and exporting again. Let's keep it simple? We could just remove the map with paths and replace it with functions that load the correct template
const TEMPLATES = {
  defaultTemplate: async () => {
    const tpl = await readJson(path.resolve(__dirname, './default.json'))
    tpl.nodes['1313wn3'].attributes.query.value.url.value = DEFAULT_TEMPLATE_DATA_URL
    return tpl
  },
  imagesTemplate = async () => readJson(path.resolve(__dirname, './images.json'))
}

That works too and has that advantage, I'll change it now.

Updated for new approach suggested by Jan.


Video overlays

Is there any way to achieve partial 360 video overlay on top of base layer (regular cubeface maps)?

In principle yes, but nobody has attempted it that I'm aware of.
The only thing that's special about video is that it requires logic to control playback, which only exists in the demos and sadly never made it into Marzipano proper (if it were to make it, it could probably be simplified, since it dates back to a time when lots of workarounds were necessary to support most browsers).


feat(expect): assertions

@antfu the only step remaining is that we increment assertionsCalls on every expect() call!
Any suggestions of how we do that, maybe cloning the expect function!


Update the subject eligibility for our International Relocation Payment

As quick fix, we are going to streamline this to just physics for now, as this is better than the current setup, which is wrong.

 Update the subject eligibility for our International Relocation Payment (IRP) email


Fixed an "UnsupportedOperationException cannot be cast" error during sbt launcher.

I got this error  when trying to load the project on my Macbook. Bumping SBT resolved the error, but I also had to bump the sbt-converter, and subsequently had to fix a couple compilation errors.

LGTM. Do you have other changes you want to add ?

That's it for the moment. I forgot to mark this PR as ready.

Thanks !


Only Initializing SoLoader Once Broke Dynamic Feature Module.

https://github.com/facebook/SoLoader/commit/3b241005a2226aca68d73853c2eaf56d1304e5c0
This commit checks if SoLoaders have been loaded once via
if (sSoSources != null) {
      return true;
}

But this broke DFM functionality. let's imagine that we have SoLoader init  call on MainApplication. Then the user download a DFM module with multiple .so files packed inside the DFM module, when we call SoLoader.init again on DFM module so loader will not load and search .so files inside DFM module and instead do nothing.
cc @simpleton .

Do you have a reproduction? this only tries to prevent multiple init on the
SoLoader side. the soSource updated should be handled by sSoSourcesVersion .
On Fri, May 20, 2022 at 11:48 AM Agastya Darma @.***>
wrote:

3b24100
https://github.com/facebook/SoLoader/commit/3b241005a2226aca68d73853c2eaf56d1304e5c0
This commit checks if SoLoaders have been loaded once via
if (sSoSources != null) {
return true;
}
But this broke DFM functionality. let's imagine that we have SoLoader init
call on MainApplication. Then the user download a DFM module with multiple
.so files packed inside the DFM module, when we call SoLoader.init again on
DFM module so loader will not load and search .so files inside DFM module
and instead do nothing.
cc @simpleton https://github.com/simpleton .
—
Reply to this email directly, view it on GitHub
https://github.com/facebook/SoLoader/issues/101, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADO6YTA4HHYT2M3MGRMBETVK7NBHANCNFSM5WQI7BQQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi thank you for the quick reply @simpleton. When we are debugging it earlier, the .so files inside DFM module never getting picked up when we call SoLoader.init for the second time inside a DFM module.
Maybe this is the intended behaviours  of so loader?. If this is indeed the intended  behaviours how can we update the soSource so that it will also include all the so.files inside DFM? we make a quick and dirty fix to solve this problem by removing all the sSoSources != null check. But maybe this "hack" is not the best way to do this. do you have any suggestion?

Hi @simpleton sorry for bothering you again. What's exactly is the proper way to update the soSource? Because of DFM i need to init SoLoader again on the DFM module in order to update the soSource, is there any other way?


Adding debug log to log the user object for all user callable transport actions

Description of changes:
Adding user audit information for all transport actions
CheckList:

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@thalurur Did we want to fix up the failure (not sure if it's a flaky one or something in PR) and get this in?

@thalurur Did we want to fix up the failure (not sure if it's a flaky one or something in PR) and get this in?

@dbbaughe looks like a detekt issue, let me fix it.


set_ak_column: problem with a new column on the reduction step.

I'm using set_ak_columnf32 function and want to add a variable for the existing Muon object on the selection step. Here is an example:
events = set_ak_column_f32(events, "Muon.mT", mT_values)
The docstring for the parent function set_ak_column says that if there is no intermediate field, they should be created automatically. I have the Muon field, so I expect for mT column to be added to the Muon object.
Unfortunately on the reduction step, I get the error:
 File "/afs/cern.ch/user/s/stzakhar/work/higgs_cp/modules/columnflow/columnflow/columnar_util.py", line 709, in set_ak_column
    ak_array = ak.with_field(ak_array, value, sub_route.fields)
  File "/afs/cern.ch/user/s/stzakhar/work/higgs_cp/data/software/venvs/venv_columnar_dev_3cbb5aff/lib/python3.9/site-packages/awkward/_dispatch.py", line 68, in dispatch
    return gen_or_result
  File "/afs/cern.ch/user/s/stzakhar/work/higgs_cp/data/software/venvs/venv_columnar_dev_3cbb5aff/lib/python3.9/site-packages/awkward/_errors.py", line 67, in __exit__
    self.handle_exception(exception_type, exception_value)
  File "/afs/cern.ch/user/s/stzakhar/work/higgs_cp/data/software/venvs/venv_columnar_dev_3cbb5aff/lib/python3.9/site-packages/awkward/_errors.py", line 82, in handle_exception
    raise self.decorate_exception(cls, exception)
ValueError: cannot broadcast nested list (in compiled code: https://github.com/scikit-hep/awkward/blob/awkward-cpp-24/awkward-cpp/src/cpu-kernels/awkward_ListArray_broadcast_tooffsets.cpp#L27)

This error occurred while calling

    ak.with_field(
        <NanoEventsArray [<event 1:                32:                31197>, ...] type='...'>
        <Array [[], [], [], [], ..., [], [], []] type='53146 * var * float32'>
        ('Muon', 'mT')

This means that the column didn't belong to the Muon object and when the skimming had been done, the mT column was not affected. To calculate mT, I'm using kinematic variables of the muon so storing this variable attached to the Muon object is useful.
Did you see this behaviour before?

Hi @hephysicist ,
thanks for the question! It's a little hard to make a definitive answer, can you point us to a full example somewhere?
That being said, I have seen similar cases where the problem was actually that the dimensions of the arrays do not match. In these cases, there was a mismatch on the axis 0, i.e. the number of events had changed when creating the new column (in your case Muon.mT). Did you apply any masks when you calculated your mT column? Note that during the SelectEvents task, we do not actually discard any events yet. From the top of my head, there are two mechanism you can use to create meaningful values under consideration of any mask:

you can use the ak.mask function, which will not reduce dimensions but will fill None values where your mask is False. Afterwards, you can use the ak.fill_none function to fill the None values with any default value, e.g. the EMPTY_FLOAT value from cf (which is also what I would recommend in your case)
If you already have a column with the right dimensions on axis 0, you can also use the ak.where to fill values according to the mask.

But I think the problem will become clearer if you can provide us with an example to reproduce the error.
Cheers,
Philip

Hi @pkausw
Let me describe the problem in detail.
I want to produce the mT column using the producer function listed below:
@producer(
    uses =
    {
        f"Muon.{var}" for var in ["pt","phi"]
    } | {
        f"MET.{var}" for var in ["pt","phi"]
    } | {attach_coffea_behavior},
    produces={
        "Muon.mT"
    },
)
def mT(self: Producer, events: ak.Array, **kwargs) -> ak.Array:

    def calc_mt(muon, met):
        cos_dphi = np.cos(muon.phi - met.phi)
        mT_vals = np.sqrt(2*muon.pt*met.pt*(1. - cos_dphi))
        return ak.fill_none(mT_vals, EMPTY_FLOAT)

    initial_dim = ak.num(events.Muon)
    (_, met_br) = ak.broadcast_arrays(events.Muon, events.MET[:,np.newaxis])

    flat_muon = ak.flatten(events.Muon, axis=1)
    flat_met  = ak.flatten(met_br,      axis=1)

    mT_values = ak.unflatten(calc_mt(flat_muon, flat_met), initial_dim,axis=0)

    events = set_ak_column_f32(events, Route("Muon.mT"), mT_values)
    return events

Then in the main selector, I simply call this function:
events = self[mT](events, **kwargs)

It has to be done in the selection section cos I want to use this variable to perform the selection. It's required to have mT as a field of the Muon object since I'm using muon kinematics and MET, so for control, it is useful that the reduction step will also skim mT variable.
Several functions later I perform the selection of a single Muon for each event and produce a muon index array that will be used in the reduction step. So the output of the last function before the reduce() looks like:
...
 return events, SelectionResult(
        steps = {
            "extra_lep_veto" : extra_lepton_veto,
        },
        objects={
            "Muon": {
                "Muon": pair_mu_idx
            },
            "Tau": {
                "Tau": pair_tau_idx
            }
        }
    )

Where pair_mu_idx contains indices of the selected muons.
I'm aware of the solution with masking and then filling Nones. This requires the production of a separate mT column that isn't connected to the Muon object. However, this is not quite safe and could potentially cause future bugs because I'll have to skim only one variable by hand independent of the Muon object set. Also there gonna be more variables to be added in the selection step.
Best,
Stepan

Hi @hephysicist, this is not really solving the issue you have potentially found, but as an alternative solution, you could also try calculating you mT feature without flattening + unflattening, e.g. like this:
cos_dhi = np.cos(events.Muon.delta_phi(events.MET))
mT_values = np.sqrt(2 * events.Muon.pt * events.MET.pt * (1 - np.cos(dphi)))
events = set_ak_column_f32(events, Route("Muon.mT"), mT_values)

I personally prefer to work with awkward arrays as much as possible, but of course, this is up to preferences.

Hi @mafrahm thanks for the suggestion! I'll try to refactor the code.
Just for an update: I tried to remove mT column at the end of the selection and everything worked well. However, now have to create it twice: at the selection and production tasks. So it's probably connected to how the selector treats the Muon object.
Cheers,
Stepan.

Hey @hephysicist ,
Thank you for providing the code that caused the issue for you. I just tried it out with the latest version of columnflow. Here is what I tried

tried to run the producer by itself
tried to include it into a selector
tried it out with tt_sl and tt_dl events.

It all worked fine for me and I was not able to reproduce your error. Can you try to update columnflow to the latest version on the master branch and try again? Also, which version of cf do you currently use (i.e. which commit hash)?
Cheers,
Philip

Hi Philip!
I updated columnflow and checked everything on the new version.
Still, see the problem and here is the hash of the commit:
16d35bb2f25f62f9110a8f1089e8dc5c62b29825
I'm also trying to have some checks to point out the exact place where the error happens.
Best,
Stepan.

https://github.com/hephysicist/higgs_cp/tree/master
Here it is.

Hi @hephysicist ,
thanks for pointing me to your code. I set it up, but had to change the following things:

I don't think you have included your cmsdb entries in a branch, did you? In any case, I had no access to your default run3 ntuples you produced, so I had to switch to the run2 2017 nanoAODv9 samples
Since I don't have access to your nanoAODs, I had no access to some of columns (all related to DeepTauv2p5). I excluded these selectors in the default selector
I also commented out the remove_ak_column command that would have removed the new Muon.mT column

I ran this command
law run cf.SelectEvents --dataset tt_dl_powheg --branch 0 --version test --selector default --config run2_2017_nano_v9_limited

I didn't try a lot, and right now I don't select any events at all, but this command still runs through. Note that the definition of the producer in your repo is different from the definition you've sent us in this thread, is that on purpose? Overall, I still can't reproduce the error, but I also can't do the exact same setup of course. Maybe this already helps debugging, or maybe wen can also

iterate again to synchronize the setups,
talk offline to try to figure this out.

Hi @pkausw ,
Thanks a lot for your efforts! For now, I have a quick fix for this problem, so let's talk offline when you have free time.
Best,
Stepan.

Hi @hephysicist , is this still an issue? If not, we'd appreciate it if you could close it :)

Since there aren't any updates in this issue, I'll close it


Add Multiple Delete Feature

Brief Summary

Users are now able to delete multiple contact at one go.
Different success/failure message will be displayed based on whether single / multiple delete is being used.
User can multiple delete up to 10 contacts at once.
If more than 10 indexes are entered, FAST will display an error message. Nothing will be deleted.
User cannot add duplicated indexes in the same command.
If duplicated indexes are detected, FAST will display an error message. Nothing will be deleted.
If there is an invalid index, FAST will delete all contacts from the valid indexes before the invalid index is detected
If there are multiple invalid index, FAST will only detect the first invalid index.
Can choose to individually type the multiple index or to type a range of index

del 2 4 7 9
del 5-9

Change command word for add, edit delete and mark appointment

appt to aa
eppt to ea
dappt to da
done to ma

Minor update to UG to reflect the changes above

close #33  #124

Codecov Report

Merging #127 (707d7e5) into master (f32b8b7) will increase coverage by 0.50%.
The diff coverage is 92.53%.

@@             Coverage Diff              @@
##             master     #127      +/-   ##
============================================
+ Coverage     71.95%   72.45%   +0.50%
- Complexity      650      676      +26
============================================
  Files           102      102
  Lines          2104     2164      +60
  Branches        249      257       +8
============================================
+ Hits           1514     1568      +54
- Misses          528      530       +2
- Partials         62       66       +4

Impacted Files
Coverage Δ

.../seedu/fast/logic/commands/AppointmentCommand.java
100.00% <ø> (ø)

.../fast/logic/commands/DeleteAppointmentCommand.java
96.00% <ø> (ø)

...du/fast/logic/commands/EditAppointmentCommand.java
95.31% <ø> (ø)

...du/fast/logic/commands/MarkAppointmentCommand.java
93.10% <ø> (ø)

...c/main/java/seedu/fast/logic/parser/CliSyntax.java
91.66% <ø> (-0.65%)
:arrow_down:

.../java/seedu/fast/logic/commands/DeleteCommand.java
91.11% <89.47%> (-8.89%)
:arrow_down:

...a/seedu/fast/logic/parser/DeleteCommandParser.java
93.93% <96.42%> (-6.07%)
:arrow_down:

...main/java/seedu/fast/commons/core/index/Index.java
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Force X/Y Min/Max values.

Is there a way to force the range of a graph?
Use case:
If I know my chart has an X range of -1, 1 and a Y range of 0, 1, is there a way I can force the axes to show those values, even if a data point doesn't reach it?
The inserted image shows a chart with the aforementioned range, however the X axis stops at 0.83 as that is the highest data point that is available. Is there a way I can always have the axes show the min/max that I want?
Thanks!

The above mentioned fix with the x & y domain works fine in the live edit (for line+bar with focus chart), but when I specify the option in my javascript like so:
yDomain: [0,100]
It doesn't have any effect. I also tried it in the plunker for the graph and still no effect.
Any suggestions?


add site url in GitHub interface

Add site url https://thefaylab.github.io/lab-manual/ by clicking the gear icon in this image 'cause it's hard to find the live Faylab manual site without it

I don't see the gear icon anymore - Gavin prob has to make this change


Level 1 chooses between trival and dense layout selection, depending on the layout score

On top of #3035, #3018, and #3321 (therefore, on hold).
Fixes #2845

i'd like to get some timing on this for a very large circuit before we activate it for Level 1. Because it does a circuit deepcopy.

i'd like to get some timing on this for a very large circuit before we activate it for Level 1. Because it does a circuit deepcopy.

Using this 40-depth circuit in FakeMelbourne.
qc = QuantumCircuit(14)
for i in range(39):
    qubit = i % 13
    qc.h(qubit)
    qc.cx(qubit, 13)

I timed them using (because, at least in my laptop, the result was varying a lot)
timeit(lambda: passmanager.run(qc), number=100)

The exact code here: 3324_rollback.txt
Without rollback
Using the following "level 1" (like the one on this PR but without the control flow):
pm1.append([DenseLayout(coupling_map, backend_properties), Layout2qDistance(coupling_map, property_name='dense_score')])

The result is:<PHONE_NUMBER>11
With rollback
With the PR as it is 77d678a.
The result is:<PHONE_NUMBER>6399998.
Thanks a lot @kdk for helping me with this.
Comparing against master
Considering the benchmark qft.QftTranspileBench.time_ibmq_backend_transpile(2), this PR is 1.78x slower than master. However, it's important to notice that master is not running DenseLayout at all.
Possible "solutions"
@kdk suggested a great possible optimization to rollback_if: Do not rollback (e.i. deepcopy) the dag if all the passes in the pass set are analysis pass (which is the case when layout selection).  I would really like to implement that optimization, but one that this PR is merged, because the stack of PRs is getting complicated.

Other result, 600 depth circuit:
qc = QuantumCircuit(14)
In [ ]: qc = QuantumCircuit(14)
   ...: for i in range(599):
   ...:     qubit = i % 13
   ...:     qc.h(qubit)
   ...:     qc.cx(qubit, 13)
   ...: print(qc.depth())
   ...: start = time.time()
   ...: passmanager.run(qc)
   ...: print(time.time()-start)
600
11.103873014450073

@kdk suggested a great possible optimization to rollback_if: Do not rollback (e.i. deepcopy) the dag if all the passes in the pass set are analysis pass (which is the case when layout selection).

Good news! Adding this optimization was trivial (see https://github.com/Qiskit/qiskit-terra/pull/3324/commits/ec8e31ac6311b8a1a903f6423b0f1a9edd26c055). I will rerun the tests.

(because I'm getting full different set of values (probably related with changes in master or my machine is just less loaded) here are the new outputs:
Without rollback
The result is: 58.22561879
With rollback
The result is: 58.96733729300001
Comparing with level 2
Level 2 is running in:<PHONE_NUMBER>14
Comparing against master
1.74x slower for qft.QftTranspileBench.time_ibmq_backend_transpile(2)

Closing in favor of #3657


Publish new version of the website

Release Notes:

New video on homepage
Add link to confluence

Closing this PR due to issues on the pipeline.


read_csv last whitespace line not ignored when skip_blank_lines=True

data = "1,2,3\n4,5,6\n "
print(pandas.read_csv(pandas.compat.StringIO(data), skip_blank_lines=True))
    1   2   3
0   4   5   6
1 NaN NaN NaN

these are related / maybe dupe
#10548, #10164

closed by #10825


Auto-select type="text" for DateTimeOffset values

#6648
a different take on #4871
DateTime can round-trip DateTimeKind.UTC with [DataType("datetimeoffset")] or [UIHint("datetimeoffset")]
since they're now handled differently by default, add more DateTime tests
expand tests involving Html5DateRenderingMode.CurrentCulture

Make VS-suggested changes to files updated in this PR

/cc @Eilon, @DamianEdwards for 2.0.x patch consideration.
This fix breaks one of the workarounds I described in #6648 (here in particular) but only if a developer uses the <input asp-for="ADateTimeOffset"/><select ...>...</select> approach and relies on auto-selection. In that case, adding type="datetime-offset" to the tag helper, adding [DataType("datetime-local")] to the property, or other kill-the-auto-select changes restore the browser's wizard.

Working on failures. A few expected strings are obviously set for my locale ☹

7e4a8fe47
(thanks @pranavkm)


S3 Static Website Hosting

[ ] Configure our bucket for s3 static website hosting
[ ] Upload an index.html
[ ] Upload an error.html
[ ] Update our output for static website hosting url


Document constraints on Sid grammar

According to
aws accessanalyzer validate-policy --policy-document policy.json --policy-type SERVICE_CONTROL_POLICY

the Sid can only have characters [a-z, A-Z, 0-9].
At any rate a Sid containing underscores is rejected by the API.
It would be nice if the correct constraints were in https://docs.aws.amazon.com/IAM/latest/UserGuide/reference_policies_elements_id.html.
Thank you @ransford!

Hello @sourcefrog ,
The issue description mentions Sid but the provided link is for policy element Id. These are two different policy elements, Sid and Id.
Can you clarify if you're referring to the Sid or Id policy element?
Reference links:

https://docs.aws.amazon.com/IAM/latest/UserGuide/reference_policies_elements_id.html

https://docs.aws.amazon.com/IAM/latest/UserGuide/reference_policies_elements_sid.html

The Sid element supports ASCII uppercase letters (A-Z), lowercase letters (a-z), and numbers (0-9).


panther-13 process has died

I don't know if this happened as a result of that my gazebo version is 9.And whether there is something wrong else or not.Could you please give me some advise?thank you very much!

Now there is a new issue after changing the virtual environment from Conda to venv.
By the way,I try to compile the develop branch and it said "undefined reference to `casadi::Matrix::sparsity() const' "

Now I have solved the issue by using stable-baselines3 with version 1.1.0.

现在，我已经通过使用 1.1.0 版本的 stable-baselines3 解决了这个问题。
The same issue persists even after updating to version 1.1.0 of stable-baselines3 and recompiling.


community-profiles: potsdam - changed SSID

I know there was a recently PR #1634 on this profile, but there was an ongoing discussion about "fixed" SSID's. Finaly we came to a conclusion, so here is a new PR withe only a ssid change.
Commit message:
community-profiles: potsdam - changed SSID

After some discussion in our meetings and on the maling list we decided to
change the SSID to a fixed one.

Signed-off-by: Hannes Fuchs<EMAIL_ADDRESS>

This PR is only for 17.01 branch, but you likely want that also for master, right?

Yes. Should I create a nother PR for the master branch. The 17.01 branch is currently used by the freifunk-firmware.
BTW: Wich is the best work-flow to get changes in both branches?

Create PR into master, then cherry pick?
Create two PRs, for both branches?

It is ok to either:

create two PRs, one for each branch
create PR for master, then cherry-pick to 17.01.

As you already have 17.01 PR, just create one for master.


[Removal]: Fancy Hemp Swan

Hotspot Name
Fancy Hemp Swan
Hotspot b58 Address
117mDz4KxW3UvsSw8c3x6NqDzYEun4u6y6nttSeBJBS8vXjB1HB
Discord Handle
No response
Hotspot Manufacturer
Bobcat
Removal Reason
Because I have rich field installation resources, on January 10, I bought 20 Bobcat hotspots from the second-hand market. After I got the machine, I transferred 8 to my friends nearby to network together. To my surprise, on January 12, the rejection list was published on the Internet. Nine hot spots in my batch of machines are on the blacklist.This is the first one. At that time, the seller transferred to me was in Shenzhen and used several wallet addresses to transfer the hot spot. The original owner's address of this hot spot was 144dna5jbxlnayhn9fyavqeswn8r8jbk5ohrtk3kudyp8gxsmgz As you know, in China, the price of buying second-hand hot spots is very high. Due to my carelessness, I bought hot spots on the blacklist, and so far I have basically no income, which has caused great harm to me. After that, I argued with the seller, but there was no result. I suffered great property losses because of the rejection list. I hope the administrator can help me remove it. My hot spots are all deployed and installed in reality.
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information
At the same time, I curse the seller. He has many wallets, such as 13jdtrgculqbrvn3c4dsqe7eaucjgxnitveqk8xe4nlfjiez9mk7
13cG76FbzevoTyfFNTra8BnmYYsFq8WZ4k3gACpzQBziHqcrxQ4
14HAEQNkDjC4sRz5fBW1KyzkWhoq4PZyRnKCg3CqPGi7Ni1ef15
144Dna5JBxLNAyhN9FYavQeSwn8r8JBk5ohrtK3KUDyp8gXsMgz
I hope you will punish this garbage that destroys the ecology of the community.

Thank you for your advice. However, crypto--related activities are not protected by law in China, and police and courts do not handle such disputes. The seller didn't want a refund, so I had to bear the loss. As you can see, my hotspot was traded before the  denylist was published, so there was no subjective fraudulent removal. Thank you for your concern and I hope you can solve my difficulty.

Buy from legitimate shop next time.

We do not have sufficient evidence to remove these hotspots from the denylist. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.

We have taken into account this removal request in the next iteration of the denylist. It may take some time before it's processed. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


Move Services into ServiceProvider

Some time ago we added KernelServiceProvider and RoutingServiceProvider. I have taken the rest of the services defined directly in the Silex\Application class and moved them into those two service providers.
This makes Application more of a delegation class.

I moved the two services and updated a whitepsace issue

Rebased and the build is now green.

Nice work :)

+1

:+1:

Thank you @henrikbjorn.


The help isn't working in 1.3 from PSGallery

None of the Help is working in 1.3 from the PSGallery.
It is just giving the generic help.

I realized the problem after posting this. I was looking at a different version I had gotten off github, and not the active version that was loaded in a different module folder. The help xml was intact in the version that wasn't loaded.
I also realized that you had mentioned you were working on a fix in a separate issue.
Anyway, love the module and especially love the ApiKey code.


Remove default page loading & add UrlFormatter

This is a support change for the basic navigation implementation on the wolvic side. Get rid of the default page loading since it'll be handled by the wolvic browser. Also add UrlFormatter dependency to support url fomatting consistent with chromium.

depends on https://github.com/Igalia/wolvic-chromium/pull/5


keyMatcher.Path cannot resolve images nested within a list

What steps did you take:
I followed this documentation to reference a nested key. Specifically, the following instruction:

path (optional; array) specifies key path from the root of the YAML document (e.g. [data, sidecarImage], [spec, images, {allIndexes: true}])

To demonstrate the issue, here is a repro, containing a kbld config, an imgpkg lockfile and a manifest I wish to transform:
---
apiVersion: kbld.k14s.io/v1alpha1
kind: Config
minimumRequiredVersion: 0.29.0
searchRules:
- keyMatcher:
    path: [spec, default]
- keyMatcher:
    path: [spec, list, {name: first-item}, default]
- keyMatcher:
    path: [spec, list, {name: second-item}, nested, default]
---
apiVersion: imgpkg.carvel.dev/v1alpha1
images:
- annotations:
    kbld.carvel.dev/id: original-image:1.2.3
  image: updated-image@sha256:2005721c33a949d82bab23099aa3f577932c350bc24d5931932e7440856f6276
kind: ImagesLock
---
spec:
  default: original-image:1.2.3
  list:
    - name: first-item
      default: original-image:1.2.3
    - name: second-item
      nested:
        default: original-image:1.2.3

What happened:
I ran the kbld command, and only some of the image references were replaced with the resolved image path:
$ kbld -f manifest.yaml
resolve | final: original-image:1.2.3 -> updated-image@sha256:2005721c33a949d82bab23099aa3f577932c350bc24d5931932e7440856f6276
---
metadata:
  annotations:
    kbld.k14s.io/images: |
      - Metas:
        - Type: preresolved
          URL: updated-image@sha256:2005721c33a949d82bab23099aa3f577932c350bc24d5931932e7440856f6276
        URL: updated-image@sha256:2005721c33a949d82bab23099aa3f577932c350bc24d5931932e7440856f6276
spec:
  default: updated-image@sha256:2005721c33a949d82bab23099aa3f577932c350bc24d5931932e7440856f6276
  list:
  - default: original-image:1.2.3
    name: first-item
  - name: second-item
    nested:
      default: original-image:1.2.3

What did you expect:
I expected all instances of original-image:1.2.3 in the manifest to be replaced with updated-image@sha256:2005721c33a949d82bab23099aa3f577932c350bc24d5931932e7440856f6276. I also expected to be able to reference the list items with the syntax {name: first-item}, {name: second-item}, etc.
Anything else you would like to add:
I had expected this to work like the select function in jq, e.g.:
$ cat manifest.yaml | yq -j eval | jq -r '.spec.list[] | select(.name == "first-item").default'
original-image:1.2.3

Environment:

kbld version (use kbld --version): 0.39.0
Docker registry used (e.g. Docker HUB): N/A
OS (e.g. from /etc/os-release): Ubuntu 18.04.5 LTS

If building images using docker:

Docker CLI Version (e.g. docker version): N/A

If building images using buildkit:

Buildkit version (e.g. kubectl buildkit version): N/A
Kubernetes version (e.g. kubectl version): N/A

If building images using pack

Pack Version (e.g. pack version): N/A

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

At this point in time keyMatcher.path can only match on a specific key name for an object or all the array or a specific index on the array. You cannot add a matcher like {name: first-item} that would try to match a key value on an array item.
For your case in particular you could use these matcher rules:
- keyMatcher:
    path: [spec, list, {allIndexes: true}, default]
- keyMatcher:
    path: [spec, list, {allIndexes: true}, nested, default]

This would try to match spec.list[*].default and spec.list[*].nested.default, so basically any array item that contains the default key or any array item that contains the keys nested.default.
The only problem with this approach is if you had the following YAML
spec:
  list:
    - name: something-that-is-not-an-image
      default: ubuntu

In this case, ubuntu would be resolved and maybe that is not the intention.
Since kbld is working as intended I am going to remove the bug label and convert it into enhancement.

i could see us adding a matcher that matches on the content of the array item e.g. [foo, {contains: {name: "blah"}}, blah]. this does look a bit more involved from the code perspective, so would be nice to see more real life examples where allIndexes is not good enough.

We'll move this to our Unprioritized Backlog for now. For folks that stumble upon this issue, please surface any additional examples that would support this feature request.


need better handling of select_multiple data on export

formhub is nice in that in the data export it will also add a column for each option and then will have a boolean to show if that option was selected. makes it easier to analyze. omkserver just puts the selected values in the single column as space separated:

would be nice to have: "marktype": "1 2 4", also be represented in the json data as:

"marktype/1": true,
"marktype/2": true,
"marktype/4": true,

so then the OMK csv output could have a column for each so (or N/A or null or whatever for false)

marktype  |  marktype/1  |  marktype/2  |  marktype/3  |  marktype/4  |  marktype/5
------------------------------------------------------------------------------------
1 2 4     |  true        |  true        |  false       |  true        |  false

what's the best way to accomplish this?

xform-to-json can be passed a field name to use for the geojson coordinates when the output is meant to be geojson format. should we see about introducing a option where you can pass an array of field names to be interpreted as select_multiple and do the above? OMK Server can read the form xml and find get the field names for questions with type="select" (these are select_multiple, and select_one show up as type="select1") before converting submission data?
if we would need to get the question field names anyways, just process the json after xform-to-json, splitting the value on spaces and then creating the needed key/value pairs from it?
???

xform-to-json only knows how to handle submissions. We needed some way to understand the structure of the XForm itself. util/xform.js now includes a parser that will pull out the instance name (usually the filename) as well as field names. Select values can later be extracted using similar techniques (which is now done in aggregate-submissions.js to expand submission fields per ^^).

https://github.com/posm/posm/issues/278


"Parsing error: The file does not match your project config" for only some of the files in a directory.

I have a pretty big setup and, for some reason, some of the files have a parsing error. It looks like it stops detecting after some hundred files.
I have two files, in the same folder of the same project. One is correctly parsed, the other one throws an error at me. Nothing in the configuration excludes this particular file.
Here is the context:
I have a global tsconfig.json and eslintrc.json as follow:
tsconfig.eslint.json
{
    "compilerOptions": {
        "target": "es2017",
        "lib": ["es2017", "dom"]
    },
    "include": ["packages/**/src/**/*", "packages/**/test/**/*"],
    "exclude": ["packages/**/node_modules/*", "packages/**/aurelia_project/*", "packages/**/dist/*", "packages/**/src/resources/wwwroot/*", "packages/**/static/*", "packages/**/locales/*", "packages/**/shared-components/**/*"]
}

.eslintrc
{
    "root": true,
    "extends": ["prettier"],
    "plugins": ["prettier"],
    "ignorePatterns": ["**/aurelia_project/**", "**/custom_typings/**", "**/shared-components/**", "*.d.ts", "*.ejs", "**/main.ts", "**/src/resources/**"],
    "settings": {
        "jsdoc": {
            "mode": "typescript"
        }
    },
    "overrides": [
        {
            "files": ["*.ts"],
            "extends": ["plugin:@typescript-eslint/recommended-requiring-type-checking", "plugin:jsdoc/recommended"],
            "parser": "@typescript-eslint/parser",
            "parserOptions": { "project": "./tsconfig.eslint.json", "debugLevel": true },
            "rules": {
                "prettier/prettier": "warn",
                "@typescript-eslint/no-unsafe-call": "off",
                "@typescript-eslint/no-unsafe-member-access": "off",
                "@typescript-eslint/no-floating-promises": "off",
                "@typescript-eslint/no-unsafe-return": "off",
                "@typescript-eslint/member-delimiter-style": "error",
                "@typescript-eslint/no-unsafe-assignment": "off", //function.bind returns any
                "@typescript-eslint/member-ordering": "error",
                "@typescript-eslint/no-unused-expressions": "error",
                "@typescript-eslint/prefer-for-of": "warn",
                "@typescript-eslint/type-annotation-spacing": "warn",
                "@typescript-eslint/restrict-template-expressions": "off",
                "comma-dangle": [
                    "warn",
                    {
                        "objects": "always-multiline",
                        "arrays": "always-multiline",
                        "imports": "always-multiline",
                        "exports": "always-multiline",
                        "functions": "never"
                    }
                ],
                "complexity": ["warn", { "max": 15 }],
                "curly": "error",
                "default-case": "error",
                "eqeqeq": ["error", "smart"],
                "max-lines": ["warn", 1000],
                "new-parens": "error",
                "no-caller": "error",
                "no-console": "error",
                "no-eval": "error",
                "no-new-wrappers": "error",
                "no-redeclare": "error",
                "quotes": "off",
                "jsdoc/require-jsdoc": [
                    "error",
                    {
                        "checkConstructors": false,
                        "publicOnly": false,
                        "enableFixer": false,
                        "require": {
                            "FunctionExpression": true,
                            "MethodDefinition": true
                        }
                    }
                ],
                "jsdoc/newline-after-description": ["warn", "never"],
                "jsdoc/require-description": "warn",
                "jsdoc/no-multi-asterisks": "off",
                "jsdoc/require-returns-description": "off",
                "jsdoc/no-undefined-types": "off",
                "jsdoc/require-param-description": "off"
            }
        },
        {
            "files": ["*.spec.ts"],
            "rules": {
                "@typescript-eslint/no-unsafe-call": "off",
                "@typescript-eslint/no-unsafe-assignment": "off",
                "@typescript-eslint/no-unsafe-member-access": "off",
                "no-invalid-this": "error"
            }
        },
        {
            "files": ["*.fake.ts"],
            "rules": {
                "@typescript-eslint/no-unsafe-call": "off",
                "@typescript-eslint/no-unsafe-assignment": "off",
                "@typescript-eslint/no-unsafe-member-access": "off",
                "jsdoc/require-jsdoc": "off"
            }
        }
    ]
}

and then for my projects, I also have files that import the global file:
tsconfig.eslint.json
{
    "extends": "../../tsconfig.eslint.json",
}

.eslintrc.json
{
    "extends": ["../../.eslintrc.json"],
    "ignorePatterns": "**/selector/meta-data/entities/*.ts",
    "overrides": [
        {
            "files": ["**/components/**/*.ts"],
            "rules": {
                "@typescript-eslint/unbound-method": "off"
            }
        },
        {
            "files": ["**/samples/**/*.ts"],
            "rules": {
                "@typescript-eslint/no-unsafe-call": "off",
                "@typescript-eslint/member-ordering": "off",
                "@typescript-eslint/unbound-method": "off",
                "jsdoc/require-jsdoc": "off",
                "no-console": "off"
            }
        }
    ]
}

I have these two files in the same folder:
packages\genius-ui-components\src\components\grid\genius-grid-v2.ts
packages\genius-ui-components\src\components\grid\patate-pomme.ts
They are parsed correctly when a run eslint in the console:

I can see the error in the genius-grid-v2.ts file using the extension:

But the other one does not and give me a parsing error:

I think it might be because this project is significantly larger (800 something files to parse), but I'm not getting any "out of memory" or other such things in the console.
I have 14 other projects and I don't have this problem anywhere else, so I'm pretty stuck on what I'm doing wrong here.
All install versions:
Node 14.15.6
NPM 6.14.6
"@typescript-eslint/eslint-plugin": "4.26.1",
"@typescript-eslint/parser": "4.26.1",
"eslint": "7.28.0",
"eslint-config-prettier": "8.3.0",
"eslint-plugin-jsdoc": "35.1.3",
"eslint-plugin-prettier": "3.4.0",
Output for extension looks like this:

Please help! Thanks!

When you run ESLint in the console in which directory do you do that. If it is not the workspace folder directory then you need to configure a valid set of working directories using eslint.workingDirectories.

Oh, I see. I thought it would just take the one in the root since I have one. But adding the workingDirectories options makes for a much smoother experience indeed!
Thanks!


wow you ripped off someone else's code

well done u mf

i helped i added everything from the school cheats blooket and still adding so plz be nice for what im am trying to do and if u go to global and press one of the ones u want at the top i promise it works


Wrap barlists for transactions, downloads, etc in collapsable components

A barlist like this should be wrapped in a collapsable since the info won't be useful to "90% of users"

I'm going to re-assign this back to you @ccerv1
This should be achievable with the built-in antd Collapse component.
Just put it under the Card, but above the BarList to achieve what you want.

Just made these changes for contract transactions and NPM downloads on Plasmic


Added ability to create a resthook subscription without returning a p…

…ayload using an HTTP POST for notification
I removed a validation for a blank payload, removed setting a default payload, and added the ability to send a HTTP POST notification to the subscriber's server if there is no payload.

Coverage decreased (-0.06%) to 72.346% when pulling eb2d21ac4bff0c999fd3e55a4e26e753b793da85 on subscriptionsWithoutPayloadPOST into fe37c87e78f43a737fc103fa24d3344002618990 on master.

Looks good, merging now!


Allow 0 value for bank account

Regarding this trello card https://trello.com/c/LHyXzalI/1112-p1-bank-accounts-in-eh-are-not-able-to-have-nil-values and this PR https://github.com/Thinkei/employment-hero/pull/3204
Explain: This modification is based on the keypay meaning of json attributes:
Abbreviation: (1): fixed_amount of primary bank account
(2): fixed_amount of normal bank accounts
(3): percentage_amount of primary bank account
(4): percentage_amount of normal bank accounts
+ Case bank_split_type = 'percentage': (1) and (2) are null, (3) and (4) is NOT null
+ Case bank_split_type = 'amount': (1) is null and 2 is NOT null, (3) = 100 and (4) is null
Reference api: POST

https://staging.yourpayroll.com.au/api/v2//business/{businness_id}/employee/unstructured

The title was named based on the trello card. However, nil value means 0 in the context, not NULL. On the UI, when the user leaves blank/empty amount field, it would be converted to 0.


One-to-many relation: ResultSet contains ... multiple times

Whenever an element class of an aggregate has its own field for the ID that is also used as keyColumn for the one-to-many relationship to its aggregate root, access to these elements will cause duplicated columns in the generated SQL and consequently warnings like:
ResultSet contains id multiple times

The problem is described in detail in this StackOverflow question.
Here is a small project repo to demonstrate the problem.
Desired behaviour: No warnings, the column accesses should probably be "squashed".

Hi,
Any news here?
Kr,
Henry

How is it going now ?

@codering It looks like the patch provided by @schauder wasn't satisfactory for the spring-data project lead :(

The project lead was quite happy with it (that's me). But Mark wasn't satisfied and I failed to convince him or improve the PR. I'll try to revisit it next week.


Add Log capability to GameEngine

This is for the log displayed on the MapScreen.

[x] Add new function addLog (Style, message)
[x] Style represents the color

GOOD - green - Something positive happened
INFO - white - Something noteworthy happened
WARN - amber - Something unusual happened
CRIT - red - Something really bad happened

[x] Store these messages in vector
[x] Return const reference to log vector
[x] MapScreen iterates backwards through vector painting logs from bottom to top

Dev complete


aur: add --version output

If wanting to support different aurutils versions in the same script, having the version directly from aur can be useful. (e.g. if aurutils is installed manually through the Makefile)

Thanks for the quick review, I think I'll release 3.1.1 right away 🎉


Connecting to bootnode fails

⚠ Support requests in an issue-format will be closed immediately. For support, go to Swarm's Discord.
Context

Summary
Connecting to bootnode fails:
"time"="2024-11-20 15:49:53.360738" "level"="warning" "logger"="node/kademlia" "msg"="connect to bootnode failed" "bootnode_address"="/ip4/<IP_ADDRESS>/tcp/32490/p2p/QmZsYCbkUXWpfR34PmUwMJvHwJtGfbcMMoAp1G2EydkpRA"
"time"="2024-11-20 15:49:53.360749" "level"="debug" "logger"="node/kademlia" "msg"="discover to bootnode failed" "bootnode_address"="/dnsaddr/sepolia.testnet.ethswarm.org" "error"="discover /dnsaddr/bee-0.sepolia.testnet.ethswarm.org: discover /ip4/<IP_ADDRESS>/tcp/32490/p2p/QmZsYCbkUXWpfR34PmUwMJvHwJtGfbcMMoAp1G2EydkpRA: network status unknown: connect full close stream reset"

Expected behavior

Actual behavior

Steps to reproduce

Possible solution

with https://github.com/ethersphere/bee/pull/4871 bootnodes started blocklisting peers
which caused first time connecting peers to be blocklisted almost instantly because of protocol requests like pullsync.
generally these requests are not fired after the warmup period but if the warmup period is reduced then the fresh node can be blocked from joining the network through the bootnode.
the fix should be that peers should be sending any protocol requests to the bootnode, ever which is handled in  https://github.com/ethersphere/bee/pull/4909


Torch multiprocessing probelms

What do I want? The ability to use a pipeline that looks something like:
pipeline = gunpowder.torch.nodes.Predict(...) + gunpowder.torch.nodes.Scan(...)

What goes wrong:
First error: RuntimeError: Cannot re-initialize CUDA in forked subprocess. To use CUDA with multiprocessing, you must use the 'spawn' start method
Attempted to solve with using multiprocessing.set_start_method("spawn")
Second error: RuntimeError: context has already been set
I think this is the first problem that needs fixing: importing gunpowder should not set the spawn method before a user gets the chance to determine what the spawn method should be. I've tracked this down to the class variable __next_id created in the class Batch in file "gunpowder/batch.py". It seems this line sets the spawn method to "fork" if no method has been set yet.
Additionally the LocalServer in "gunpowder/tensorflow/local_server.py" also has a class variable that leads to the spawn method being set on import.
Further problems:
After removing the two mentioned variables that cause Multiprocessing to set a default spawn method, you can set the spawn method with multiprocessing.set_spawn_method("spawn"). You then get an error: AssertionError: can only join a started process. This seems to be getting thrown in "gunpowder/producer_pool.py" on line 76 at self.__watch_dog.join(). It seems to happen
Does this mean we can't use double underscore methods if we want to be multiprocessing friendly?
Continuing:
After removing all "__{name}" methods and replacing them with "_{name}" in ProducerPool, the precache.py tests still seem to hang forever. This seems to be an issue with logging errors in tests, so I created some simple scripts to do the same thing outside of a testing framework. Now I get the following error:
RuntimeError: Attempted to send CUDA tensor received from another process; this is not currently supported. Consider cloning before sending.
The stack trace here is particularly unhelpful so I am having a lot of difficulty figuring out where this is coming from.
See branch torch-multiprocessing for changes I have made to explore this problem.

The problem you are running into actually stems from torch.Predict. In its constructor, it is already initializing CUDA (by sending the model to the "cuda" device, if one exists). This only works if torch.Predict is used in a single process. Scan, however, does spawn processes (if num_workers is larger than 1), which now already have CUDA initialized, leading to the error you observe.
One solution¹ is to defer initialization of anything that relates to CUDA to the process in which it is used. gunpowders predict nodes (torch.Predict and tensorflow.Predict) inherit from GenericPredict, which contains a start() method for exactly this purpose. Its documentation says:

This method will be called before the first call to predict, from the same process that predict will be called from. Use this to initialize your model and hardware.

I fixed the implementation of torch.Predict to initialize CUDA in start(), see branch torch_predict_multiprocessing. With this change, the following example script runs fine:
import gunpowder as gp
import numpy as np
import torch

class ArraySource(gp.BatchProvider):

    def __init__(self, array_key, array):
        self.array_key = array_key
        self.array = array

    def setup(self):
        self.provides(self.array_key, self.array.spec)

    def provide(self, request):

        batch = gp.Batch()
        batch[self.array_key] = self.array.crop(request[self.array_key].roi)
        return batch

if __name__ == '__main__':

    raw = gp.ArrayKey('RAW')
    pred = gp.ArrayKey('PRED')

    raw_array = gp.Array(
        np.ones((100,), dtype=np.float32)*10,
        gp.ArraySpec(roi=gp.Roi((0,), (100,)), voxel_size=(1,)))

    model = torch.nn.Sigmoid()
    model.eval()

    scan_request = gp.BatchRequest()
    scan_request[raw] = gp.Roi((0,), (10,))
    scan_request[pred] = gp.Roi((0,), (10,))

    request = gp.BatchRequest()
    request[raw] = gp.Roi((0,), (100,))
    request[pred] = gp.Roi((0,), (100,))

    pipeline = (
        ArraySource(raw, raw_array) +
        gp.torch.Predict(model, {'input': raw}, {0: pred}, device='cuda') +
        gp.Scan(scan_request, num_workers=10))

    with gp.build(pipeline):
        batch = pipeline.request_batch(request)

    print(batch[raw].data)
    print(batch[pred].data)

This was tested with the following packages:
cudatoolkit               10.0.130                      0
cudnn                     7.6.5                cuda10.0_0
python                    3.7.7                hcff3b4d_5
pytorch                   1.3.1           cuda100py37h53c1284_0
pytorch-gpu               1.3.1                         0

Let me know if this fix works for your case.
¹: The other solution is to use the spawn start method, as the error message suggests. Unfortunately, the error message fails to mention that the better solution is to avoid having CUDA shared between processes in the first place, if possible.

I think this is the first problem that needs fixing: importing gunpowder should not set the spawn method before a user gets the chance to determine what the spawn method should be.

I think in this case, if a user really wants to change the start method, it should be set before importing gunpowder:
import multiprocessing
multiprocessing.set_start_method('spawn')
import gunpowder

Does this mean we can't use double underscore methods if we want to be multiprocessing friendly?

No, __ methods have nothing to do with multiprocessing. They are just Python's version of "private" functions that can not be overwritten by subclasses. __init__ and __repr__, for example, use this concept. See the documentation for more details.

As discussed off-site, this was indeed the problem. Fixed in 3e9e1dad09b7d95c8bad25e9bf4986b24a584811, together with a similar issue in torch.Train.


Patch fullfillment inbound

Added listPrepDetails and setPrepDetails operations.

Official Announcement
https://developer-docs.amazon.com/sp-api/changelog/update-fulfillment-inbound-api-v2024-03-20-now-supports-preparation-details


Report the state of agents back to clients (e.g., reached destination)

Clients need to be able to know when they've reached the destination! Or what other state agents are in.

I've seen, that there is already a corresponding function to check, if the Agent reached its target (pub(crate) fn has_reached_target in Agent.rs) and checked in update of lib.rs. For me, it looks like it could be enough to add an enum with the agents state and set accordingly in the update loop of landmass to expose it over the Agent struct.

Yeah I like this! We should have a few variants of this state. I'm picturing Idle, ReachedTarget, Moving, NoPath (maybe that's part of #8 though?). In the future, we'll probably want LookingForPath as well to handle async pathfinding or something, but we don't have to worry about that for now.

Actually if we stick PathStats in the Moving and NoPath variants, that might also be enough to also solve #8. Although maybe users would want to recieve an event every time a path failed? Or we can just have every pathfind generate a unique ID, and users have to keep track of when that changes 🤔


[ZGC] Improve parsing for Java 17 style output

ZGC was marked production ready in Java 17. There were a number of changes made to the ZGC log output in the past 2-3 years that have made it into the 17 release. I've updated the tests with samples from a Java 17 run on a sample app to test changes made to the ZGCCycle class.  Java sample app was invoked with the following flag -Xlog:gc=info,gc+start=info,gc+phases=info,gc+load=info,gc+mmu=info,gc+marking=info,gc+metaspace=info,gc+nmethod=info,gc+ref=info,gc+reloc=info,gc+heap=info:file=/logs/gclog:level,tags,uptime:filecount=10,filesize=250m

elastic metaspace was introduced in this commit https://github.com/openjdk/zgc/commit/7ba6a6bf003b810e9f48cb755abe39b1376ad3fe#diff-9aa1419e172b665d42f187e7f9d46753f396ab6e0d192906d659ff81b3b6cf12 which removes the capacity field. A specific metaspace class was added to reflect these changes.

Memory pool summary no longer emits reserved heap, removed to reflect changes in this file https://github.com/openjdk/zgc/blob/1f8813495e8184b6c38319df1c2cb70de7811a76/src/hotspot/share/gc/z/zStat.cpp#L1432-L1503

MarkFree phase was added and exported in the ZGCCycle summary. May be opportunity to add Mark Continue phase in the future. Phase are defined here https://github.com/openjdk/zgc/blob/26c03c1860c6da450b5cd6a46576c78bea682f96/src/hotspot/share/gc/z/zDriver.cpp#L45-L54

Reclaimed and MemorySummary were updated to export value in kilobytes based on the unit strings

Relocation summary is not yet implemented so it was removed form the ZGCParserRulesTest class as the old capture doesn't work with the new format. I'll update this in a separate PR.

ZGCParserTest was update with a new format, along with all test values, that were hand calculate to ensure we're accurately parsing the data.

One of the motivations here for upgrading for the latest output format is because ZGC was not really considered "production ready" in an LTS release until Java 17 (from Per Liden's ZGC talk) . I suspect that the majority of users who want leverage ZGC would be doing it on Java 17+ not Java 11 given its less than ideal performance on Java 11. Please let me know if you disagree with this assertion.

@kirk-microsoft - Thoughts? I'm pretty sympathetic to this line of thinking (ZGC for 17+ only).


ShouldUploadFileAsync -> PASS

TestableStream was created which inherits from MemoryStream and overrides the ReadTimeout and WriteTimeout properties because the base getter implementation in MemoryStream throw the exception  System.InvalidOperationException : Timeouts are not supported on this stream. which causes FluentAssertions to fail.
HttpContentHeadersTypeComparer was created to compare HttpContentHeaders with CompareLogic because CompareLogic doesn't support comparing collections with a non integer indexer and, in this case, throws the the exception System.Exception : Cannot compare objects with a non integer indexer for object File.Headers.NonValidated.


DSUB Extensions - 2023 - CP integration

List of integrated CP in DSUB Extensions:
1105-02

Closes #
📑 Description

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed
[ ] I have selected a committee co-chair to review the PR

ℹ Additional Information

@mattblackmon review please


Added linkedin to the testimonials section

Related Issue
Closes #1175

Description
Hey @rohansx I have added LinkedIn URLs to the testimonials section so that users can see the details of the admins an mentors.
Screenshots

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

Add twitter link & github link also

okay

@rohansx done

@rohansx This PR issue is similar to my PR issue #1187 so can you please look into this matter before it create problems.

@rohansx This PR issue is similar to my PR issue #1187 so can you please look into this matter before it create problems.

okay sure I have added all the three links

@sumitkr2000 please look into it as this issue has already been solved.

@sumitkr2000 please look into it as this issue has already been solved.

i have added the twitter link also

@sumitkr2000 please look into it as this issue has already been solved.

i have added the twitter link also

Actually #1136 is the similar issue and your's was assigned later. Still can approve for twitter links!
Requesting to change title.

@sumitkr2000 please look into it as this issue has already been solved.

i have added the twitter link also

Actually #1136 is the similar issue and your's was assigned later. Still can approve for twitter links! Requesting to change title.

done ... i have changed the title


sed error in device install script

Describe the bug
When installing on the device I see an 'sed' error (It does not seem to effect anything but still)
bash -c "$(curl -fsSL https://getsshnpd.noports.com)" -- --args "-s -u -v"
  [X] Missing required dependency: unzip
Client address (e.g. @alice_client): @lorawan123_client
Device address (e.g. @alice_device): @lorawan123_device
Device name: dragino

Downloading sshnpd from https://github.com/atsign-foundation/sshnoports/releases/download/v3.4.2/sshnp-linux-arm.tgz
Installed binaries: sshnpd at_activate sshrv
sed: can't read /home/lorawan123/.local/bin/sshnpd@lorawan123_client: No such file or directory
Installed cron job: '@reboot nohup /home/lorawan123/.local/bin/sshnpd@lorawan123_client ><EMAIL_ADDRESS>2><EMAIL_ADDRESS>Installed cron job: '40 6 * * * logrotate -f<EMAIL_ADDRESS>Service entry added: sshnpd@lorawan123_client cron
Starting sshnpd@lorawan123_client service...

Installation complete!

Steps to reproduce

use install script

Expected behavior
no errors
Screenshots
No response
Smartphones

armbian (debian)

Were you using an atApplication when the bug was found?
No response
Additional context
No response

Going to close this ticket, it's non breaking, and this installer is soon to be replaced.
The problem is that we try to sed a file when renaming/installing, but that file doesn't exist on install so it complains. (but doesn't break anything)


npm run dev:assets failing due to TypeError

I was trying to build the theme on my dev environment (OS: ArchLinux) but I've got this weird error:
$ npm run dev:assets

><EMAIL_ADDRESS>dev:assets /home/my_user/projects/jane
> webpack --watch --display='errors-only'

Webpack is watching the files…

(node:23444) UnhandledPromiseRejectionWarning: TypeError: Cannot create property 'message' on string 'Checksum error in glyf'
    at /home/my_user/projects/jane/node_modules/iconfont-webpack-plugin/lib/loader.js:34:19
    at process._tickCallback (internal/process/next_tick.js:68:7)
(node:23444) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:23444) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

I looked at the "iconfont-webpack-plugin" repo and no issue were reported about this problem, so I am wondering if it's not related to my local installation...
I'll continue to investigate and update this issue if I find anything.

Firstly, try to run
$ npm rebuild node-sass --force

If it doesn't help or if the issue appears again later, try to set up node virtual environment for older version of node.
I also faced this problem some time ago and I figured out that some of dependencies (which we can't change directly) is not ready for new version of node.
I have node version v9.11.1, npm version 5.10.0.

Thanks a lot for the quick response. That would explain why it worked on my mac but not on Arch... The command did not help.
I installed a new node env and it worked :smile:
In case anyone is having the issue:
sudo pip install nodeenv
nodeenv env --node 9.11.1 #This will create a new env folder in your current dir
source env/bin/activate
npm rebuild node-sass --force
npm run dev:assets #It works :D :D :D

Thanks again @Zebradil !


Add rustfmt.toml to unify format

The purpose of this change is to get stable results with stable and nightly rustfmt, right?
stable rustfmt is changed?


Add more levels of navigation to sidebar

This PR adds support for multi-level drop downs in infite amounts.
The docs needs to be updated for this. I've tested it down to level 5. Most common use-case would be 3 levels I guess.
We achieve this by running recurive building of sub-menus.
Not sure if it breaks anything which I have not on my radar but demo app works fine for me. Feel free to test the PR with all your needs.

@MarvinKlein1508 Thanks for the PR. Can you add one or two examples?

@gvreddy04 sure. I'm not familiar with your demo project structure though. The way to use this is bascially the same as for the two level navigation.

For example:
yield return new NavItem
{
    Id = "nav-test",
    IconName = IconName.Gear,
    Text = "Test Level 1",
    IconColor = IconColor.Success,
    Match = NavLinkMatch.All
};

yield return new NavItem
{
    Id = "nav-test2",
    IconName = IconName.Gear,
    Text = "Test Level 2",
    IconColor = IconColor.Success,
    Match = NavLinkMatch.All,
    ParentId = "nav-test"
};

yield return new NavItem
{
    Id = "nav-test3",
    IconName = IconName.Gear,
    Text = "Test Level 3",
    IconColor = IconColor.Success,
    Match = NavLinkMatch.All,
    ParentId = "nav-test2"
};

You can do this unlimited times.

I've added another commit to change the demos.

@MarvinKlein1508 I'll test all the scenarios.

@gvreddy04 just out of curiosity. Can you tell me what's the main purpose of the SidebarDataProviderRequest
For me it looks like it is just sorting the items and create nesting. Wouldn't it be much easier to simply define a List<NavItem> and pass this one to the component?
It would less code to write since you don't need to rely on all the ids because you can simply add another NavItem as a child to another NavItem and so on.


[Bug/Feature]: Field to paste ETA template missing in Better BibTex Prefs for QuickCopy

Debug log ID
Q23ZJ4PJ-refs-euc
What happened?
I am still missing aan issue similar to https://github.com/retorquere/zotero-better-bibtex/issues/2457  Where to paste ETA template?

The Better BibTeX prefernces open via Button in a seperate window.

The Export -> QuickCopy tab looks like the screeenshot
BUG: The former UI Input field is missing
Even after restarts!

I am on Zotero 6.0.26 on an MBP M1 under MacOS Monterey, with:

Better BibTex for Zotero 6.7.79 autoinstalled at 09. Mai 2023
Calling the eta-template directly and via Zutilo 3.10.0 Shortcut

The eta-template code is reachable via Zotero Config-Editor in
extensions.zotero.translators.better-bibtex.quickCopyEta

The QuickCopy export using the template is executed well on this machine.

checked the commit in tag 6.7.63 fixing #2457 and compared it with current 6.7.79.

https://github.com/retorquere/zotero-better-bibtex/commit/570c08df618abea5407bd7bfdb5e54501b04fa5e
and in content/Preferences/export/quickcopy.pug

https://github.com/retorquere/zotero-better-bibtex/commit/b6c9ee8cb546998a84ca0198253ec01383266564

I cannot find myself a location where the bug was reintroduced if still the same/similar to #2457 (no screenshot provided).

add proper title

Test build test build <IP_ADDRESS>59 is not available for me (anymore?) even after login. didnt recognize the notification to give feedback, sorry

Test build test build <IP_ADDRESS>59 is not available for me (anymore?)

Test builds expire after approximately one week.

It seems that the eta-template field is not available on next visit of the BetterBibTex Prefs as expected.
Steps to reproduce
Close the BetterBibTex Prefs and reopen it:

in the tab "Export" -> QuickCopy" the Input field for LaTeX citation shows up with label LaTeX command
only after triggering a change to the dropdown e.g. to "Zotero select link" and then back to "Eta-template" unveils the field for the Eta-template.

My Zotero Prefs setting in tab Export  QuickCopy "Format for Entries" points to: "Better BibTeX Citation Key Quick Copy"
Screenshot of the initial view.

I can confirm that with test build <IP_ADDRESS>00:
The Eta-Template field now does not vanish and stays visible on initial visit of the tab. THX!
The tight width of the textbox remains, as shown in my screenshot.
But one thing after the other.
Fixing two things at the same time is not a good idea.
I open a new ticket, and close this as resolved for me.


fetch cross origin rest api

CORS is not working, because origin of fetch.get is null.
any idea?

Could you provide us with a few more details on the issue? Which sample on which OS are you using? What are you trying to fetch?
Maybe you could post a code snippet illustrating the place where you encounter this problem.

Hi,
i'm based on Photino.Vue-Sample, within JS-File (referenced in HTML-File) i'm doing a fetch to rest-api.
const response = await fetch(url, {
method: "get",
mode: "cors",
headers: {
"authorization": bearer,
"Content-Type": "application/json",
"Accept": "application/json",
}
In Developer-Tools i can see, that Origin-Header is null and Preflight-(Options)-Call  is not working because of CORS.

One way to get around this issue is to start a localhost with ASP.NET (you can combine the ASP.NET template dotnet new webapp with Photino) and load your Angular app over the localhost connection.
Please refer to this SO thread on the topic: https://stackoverflow.com/questions/8456538/origin-null-is-not-allowed-by-access-control-allow-origin
This is a common issue when trying to load resources from the local filesystem and modern browser try to prevent malicious code to be executed from it.

Refer to https://github.com/tryphotino/photino.NET.Server


Feature request: Maya 2025 Integrated submitter support

Describe the problem
Currently the integrated submitter relies on PySide2 to create the UI, which is no longer supported by Maya 2025.
To add support we'll have to update the submitter code to handle either Pyside2 and PySide6 so Maya 2023-2025 can be covered.
Proposed Solution
Autodesk has a list of QT6 changes that can be used to direct what changes need to be done.
And QT has their own guide on porting from 2 to 6.
Example Use Cases
Just bumping the imports to PySide6 isn't enough, a proper migration would need to be done.
Example:
"/opt/DeadlineCloudSubmitter/Submitters/Maya/scripts/deadline/maya_submitter/ui/components/scene_settings_tab.py", line 43, in init
lyt.setMargin(0)
^^^^^^^^^^^^^
AttributeError: 'PySide6.QtWidgets.QHBoxLayout' object has no attribute 'setMargin'

Just like the deadline cloud client library, it should be a relatively simple change of swapping to QtPy and re-testing the submitter to ensure all the elements are still hooked up correctly.

I have gone through and done this I have a working version for Maya 2024 and 2025

Nice! Is that something you're willing to share back with a PR? Otherwise, this is on our radar as something to address in a future change.

No, I would love to. how would you like me to provide it back to you?
On Wed, Oct 16, 2024 at 12:07 PM Morgan Epp @.***>
wrote:

Nice! Is that something you're willing to share back with a PR? Otherwise,
this is on our radar as something to address in a future change.
—
Reply to this email directly, view it on GitHub
https://github.com/aws-deadline/deadline-cloud-for-maya/issues/180#issuecomment-2417703494,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQHCCODV6XUMZYSYICC4QDZ322PXAVCNFSM6AAAAABPVHX2OOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJXG4YDGNBZGQ
.
You are receiving this because you commented.Message ID:
@.***>

--
Derek Zavada
cell<PHONE_NUMBER>
@.***

That's awesome! If you're familiar git, we've got a revamped contributing guide you can use: https://github.com/aws-deadline/deadline-cloud-for-maya/blob/mainline/CONTRIBUTING.md#contributing-via-pull-requests.
It should hopefully give enough detail on how to submit a pull request to this github repository. Let us know!


Fix metadata does not get updated

Description
Please describe the changes you made.
Currently the metadata key was dropped before it will be processed in a changeset. This PR preserved the metadata key until it is processed in Ecto changeset.
I'm not sure of it the function name is, and should it be in the sanitize_metadata pipeline or having to call it in controller after the Customers.sanitize_metadata function
Issue
Link to the issue (if available)
This PR fixes #995
I haven't added a test for this since I assumed  there is a recently pushed PR for metadata test.

I'd like to ask for thoughts on my changes @reichert621.

Hey @reichert621, @jeepers3327! Do you think you would be able to get this PR merged soon, please? I'd be incredibly valuable to us. Thanks again for your time! 🙌

Hi @CharlesMangwa,
I'm not sure about that. It seems that the maintainer haven't reviewed my PR yet. I will try to reach out to them in slack if they can check on it.

Hey @jeepers3327,
Thanks for the quick reply! I did see the maintenance announcement though I was under the impression that @reichert621 would consider this PR to fall into the exception mentioned in the 2nd sentence.
Regarding the Slack workspace, I believe this message was coming from you?

@jeepers3327 this issue isn't fresh in my mind, do you mind including test cases that demonstrate that the broken behavior is now fixed?

Hi @reichert621
I updated the code and added a test.
Please check, thank you.

@CharlesMangwa ,
Based on my code changes in this PR, If you want to update the metadata field, please update the param.
from:
{
   "customer":{
      "metadata":{
         "random_custom_field":"034c0f85-0be3-4086-acf8-85ed8f4f1228"
      }
   }
}

to:
{
   "customer":{
      "metadata":{
         "custom_metadata" =>  {
            "random_custom_field":"034c0f85-0be3-4086-acf8-85ed8f4f1228"
         }
      }
   }
}

Hey @jeepers3327, thanks for the tests and steps! @reichert621 hopefully this PR is good for a merge now

Hey @reichert621, checking again real quick if you'd have the time to review and potentially merge this PR this week by any chance, please?


HBASE-27872 xerial's snappy-java requires GLIBC >= 2.32

We need to add a native library load check with a helpful error message if xerial snappy fails to initialize due to a too old glibc or similar reason, and disable the unit test if the native library fails to load.

After the proposed changes:
2023-05-18T16:49:25,730 ERROR [Time-limited test {}] xerial.SnappyCodec(58): The Snappy native libraries could not be loaded
java.lang.UnsatisfiedLinkError: /home/apurtell/src/hbase/hbase-compression/hbase-compression-snappy/target/snappy-1.1.9-1a39c2aa-088d-44df-a86b-7bc7d8682d47-libsnappyjava.so:
/lib/aarch64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by /home/apurtell/src/hbase/hbase-compression/hbase-compression-snappy/target/snappy-1.1.9-1a39c2aa-088d-44df-a86b-7bc7d8682d47-libsnappyjava.so)
    at java.lang.ClassLoader$NativeLibrary.load(Native Method) ~[?:1.8.0_332]
    at java.lang.ClassLoader.loadLibrary0(ClassLoader.java:1950) ~[?:1.8.0_332]
    at java.lang.ClassLoader.loadLibrary(ClassLoader.java:1832) ~[?:1.8.0_332]
    at java.lang.Runtime.load0(Runtime.java:811) ~[?:1.8.0_332]
    at java.lang.System.load(System.java:1088) ~[?:1.8.0_332]
    at org.xerial.snappy.SnappyLoader.loadNativeLibrary(SnappyLoader.java:180) ~[snappy-java-<IP_ADDRESS>.jar:<IP_ADDRESS>]
    at org.xerial.snappy.SnappyLoader.loadSnappyApi(SnappyLoader.java:155) ~[snappy-java-<IP_ADDRESS>.jar:<IP_ADDRESS>]
    at org.xerial.snappy.Snappy.init(Snappy.java:70) ~[snappy-java-<IP_ADDRESS>.jar:<IP_ADDRESS>]
    at org.xerial.snappy.Snappy.<clinit>(Snappy.java:47) ~[snappy-java-<IP_ADDRESS>.jar:<IP_ADDRESS>]
    at org.apache.hadoop.hbase.io.compress.xerial.SnappyCodec.<clinit>(SnappyCodec.java:55) ~[classes/:?]
    at org.apache.hadoop.hbase.io.compress.xerial.TestWALCompressionSnappy.setUpBeforeClass(TestWALCompressionSnappy.java:53) ~[test-classes/:?]
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_332]
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[?:1.8.0_332]
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:1.8.0_332]
    at java.lang.reflect.Method.invoke(Method.java:498) ~[?:1.8.0_332]
    at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:59) ~[junit-4.13.2.jar:4.13.2]
    at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) ~[junit-4.13.2.jar:4.13.2]
    at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:56) ~[junit-4.13.2.jar:4.13.2]
    at org.junit.internal.runners.statements.RunBefores.invokeMethod(RunBefores.java:33) ~[junit-4.13.2.jar:4.13.2]
    at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) ~[junit-4.13.2.jar:4.13.2]
    at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) ~[junit-4.13.2.jar:4.13.2]
    at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:299) ~[junit-4.13.2.jar:4.13.2]
    at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:293) ~[junit-4.13.2.jar:4.13.2]
    at java.util.concurrent.FutureTask.run(FutureTask.java:266) ~[?:1.8.0_332]
    at java.lang.Thread.run(Thread.java:750) ~[?:1.8.0_332]
2023-05-18T16:49:25,736 WARN  [Time-limited test {}] xerial.TestWALCompressionSnappy(54): Snappy codec cannot be loaded.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m 39s
master passed

+1 :green_heart:
compile
0m 17s
master passed

+1 :green_heart:
checkstyle
0m  9s
master passed

+1 :green_heart:
spotless
0m 42s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
0m 21s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 22s
the patch passed

+1 :green_heart:
compile
0m 16s
the patch passed

+1 :green_heart:
javac
0m 16s
the patch passed

+1 :green_heart:
checkstyle
0m  9s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
12m 25s
Patch does not cause any errors with Hadoop 3.2.4 3.3.5.

+1 :green_heart:
spotless
0m 41s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
0m 25s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 11s
The patch does not generate ASF License warnings.

30m 33s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5245/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5245

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux f82964fb906c 5.4.0-148-generic #165-Ubuntu SMP Tue Apr 18 08:53:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 71d7996047

Default Java
Eclipse Adoptium-11.0.17+8

Max. process+thread count
83 (vs. ulimit of 30000)

modules
C: hbase-compression/hbase-compression-snappy U: hbase-compression/hbase-compression-snappy

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5245/2/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Issues implementing built-in intent

Expected Behaviour
Ability to implement a built-in intent into a handler.
Actual Behaviour
When implementing a built-in intent into a handler the intent has a . (period) in the name. PHP does not allow such a character in a function name, therefore, it is not possible to implement a built-in intent.
How To Reproduce The Issues
`<?php
class handler
{
public $alexa;
public function AMAZON.yesIntent()
{
$this->alexa->response->speak('Howdy there!');
$this->alexa->response->emit();
}
}`

A potential fix would be to simply replace the . (period) with a _ (underscore).
This would need to be properly documented as it may not be obvious.

Issue fixed with commit cc8a46e. Built-in intents should remove the '.'. If the intent has arguments such as :
'AMAZON.SearchActionobject@WeatherForecast' the list of arguments will be passed as an array to the intent.


0.1.0 does not match version regex

The version regex only works with the alpha / beta builds and breaks on the 0.1.0 GA as it does not have the additional version components required in the selector logic for breaking down the version numbers. Changes necessary to make 0.1.0 installable with this module.

Thanks for your notification. We'll look into this ASAP.

This should be fixed in tag 0.3.1. I tested it on our infrastructure.
Please let me know if you experience any other problems.

I'm going to close this. Please feel free to re-open should you experience more problems with the version regex.


UI issues on the phone screen

Hi, The following small problem was found on the phone screen：
1: The "X" clear button in the input box remains after the answer is searched and clicked to jump.

2: When you click search, the page will automatically zoom in and focus, this user experience is very good! But when you finish the search and click on the answer to jump, this magnifying and focusing effect will still be maintained on the page after the jump.

Thanks！

Thanks for reporting, the first issue is a bug, we'll working on it later.
For the second one, we probably can't do anything, the zooming effect is applied by Safari. But you can try overriding the viewport meta yourself by configuring your docusaurus.

Thanks！


It would recommend including a section on data residency

It would recommend including a section on data residency and the option for multi-geo be able to store Teams, SharePoint, OneDrive and  Exchange online content in a local geo to comply with local regulations.  I find this is a key question that I am asked as a Gold Partner.
Microsoft Docs reference pages
Multi geo in teams https://docs.microsoft.com/en-us/microsoftteams/teams-experience-o365odb-spo-multi-geo
Multi geo https://docs.microsoft.com/en-us/microsoft-365/enterprise/microsoft-365-multi-geo?view=o365-worldwide
Multi geo SharePoint Online and OneDrive for Business https://docs.microsoft.com/en-us/microsoft-365/enterprise/multi-geo-capabilities-in-onedrive-and-sharepoint-online-in-microsoft-365?view=o365-worldwide
Location of data in Teams lhttps://docs.microsoft.com/en-us/microsoftteams/location-of-data-in-teams

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

ID: be4f3e5c-7dd7-ca98-8cd1-21177e7fe26d
Version Independent ID: 083f0f2c-8c72-582d-3e13-1679684ea88c
Content: Compliance options for Microsoft 365 groups, Teams, and SharePoint collaboration
Content Source: microsoft-365/solutions/groups-teams-compliance-governance.md
Product: microsoft-365-enterprise
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@MikePlumleyMSFT  Please add your comment and help us in resolving this issue. Thanks.

Thanks, @nikki-c, @yogkumgit. I added a data residency section to the doc.

@nikki-c Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Bucket fills the whole canvas after clear -> redo

Using the bucket tool after doing clear than redo will fill the whole canvas instead of the correct part using the fill function.
How to reproduce it:

Draw a circle
Clear
Redo
Try to fill the circle

Fixed in #3 .