Wrong java used while JaveSE-17 is configured

For work reason I have both JDK1.8 and JDK17 (mandrel-java17) installed. The JAVA_HOME points to JDK1.8 path.
However in the VSCode workspace setting (.vscode/settings.json), I have
{
    "java.configuration.updateBuildConfiguration": "automatic",
    "java.jdt.ls.java.home": "~/.local/mandrel-java17",
    "java.configuration.runtimes": [
        {
            "name": "JavaSE-17",
            "path": "~/.local/mandrel-java17",
            "default": true
        }
    ],
}

When trying to debug the Quarkus app within vscode I still get
Caused by: java.lang.UnsupportedClassVersionError: io/quarkus/maven/components/BootstrapSessionListener has been compiled by a more recent version of the Java Runtime (class file version 55.0), this version of the Java Runtime only recognizes class file versions up to 52.0

Please help.

We had a very similar bug from a few days earlier, https://github.com/redhat-developer/vscode-java/issues/2421 . Also filed against Quarkus which I moved to vscode-java.
I'll try again to reproduce this.

Ok, so looks like your project was compiled with Java 11 (55.0) but Java 1.8 (52.0) is being used to run the project (since you set it as JAVA_HOME).
If you want to try with Java 1.8, you could look at the Quarkus project build files (Maven or Gradle) and change the compiler source/target to 1.8 to get everything building. However, I think recent versions of Quarkus require Java 11 at a minimum.
Alternatively, you could try setting JAVA_HOME=~/.local/mandrel-java17. That should ensure that your project, which is compiled as Java 11 uses a runtime (Java 17) that should be compatible.

I close this issue since @rgrunber give a great explanation.
@tigerinus if you have again problem, please reopen it.


分岐の順序変更機能

https://github.com/sunfish-shogi/electron-shogi/issues/93

Codecov Report
Base: 69.50% // Head: 69.49% // Decreases project coverage by -0.01% :warning:

Coverage data is based on head (7bcd0b6) compared to base (de37f0e).
Patch coverage: 69.49% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #305      +/-   ##
==========================================
- Coverage   69.50%   69.49%   -0.02%
==========================================
  Files          65       65
  Lines       12006    12065      +59
  Branches     1601     1616      +15
==========================================
+ Hits         8345     8384      +39
- Misses       2082     2087       +5
- Partials     1579     1594      +15

Impacted Files
Coverage Δ

src/store/index.ts
52.36% <50.00%> (-0.02%)
:arrow_down:

src/store/record.ts
69.78% <50.00%> (-0.38%)
:arrow_down:

src/shogi/record.ts
65.65% <76.74%> (+0.09%)
:arrow_up:

src/shogi/kakinoki.ts
79.91% <0.00%> (+0.27%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add fields to leboncoin bridge

To complete Leboncoin bridge and make search more accurate this PR add some fields
Here the modifications:

Rename variables to prepare new fields addition (more understanding by humans)
Add Department field (find item arround you)
Add Price field (Money Rules The World)
Add Estate, Rooms, Square fields (for property)
Add Mileage,  Year, Cubic capacity, Fuel fields (for vehicule)
Finishing with a quick code clean/code style pass (polishing)

~More fields incoming (does it need an other PR?)~
Hope it will helps!

does it need an other PR?

I don't think that is necessary. You are already changing existing parameters, so the bridge isn't compatible anymore. Adding more parameters shouldn't do any harm.
Let us know when this PR is ready to be merged.

It's Ok for me. This PR is ready.

I see it more as a condition like
if($this->getInput('category') == ID and (!empty($this->getInput('mileage')) or !empty($this->getInput('brand'))...) {
    die('Cannot input mileage nor brand for anything but cars !');
}

Ok... You really want it in this PR?
I'm not comfortable with that way. It will bring a lot of complexity:
mileage is only for category 1, 2, 3, 4, 5
fuel is only for category 2, 5
estate is only for category 9, 10
rooms is only for category 9, 10
square is only for category 9, 10
Your code is inclusive about category, it's better to be exclusive:
if (!array_key_exists(ID, $vehicule_compat_categories) and ...)
{ error about fields not use right }

So we need to create constant for category ids and array to store compatibility. That what I was meaning when I told about "mapping".
We should create an issue for this.

You're right, I didn't see the complexity behind it, as I don't use the bridge
Thanks for maintaining the bridge and for the PR !

Hi,
If I may a suggestion:
because it's complicated to add all the cities, zipcodes to cover an area, it would be nice to add a field to leboncoin bridge: a kilometres radius.
Leboncoin website allows this location criterium.
For example: https://www.leboncoin.fr/recherche?text=auto&locations=Paris__48.85790600716752_2.358844068809977_10000_20000
Would it be complicated to add to your bridge?

@JackNUMBER Sorry I didn't notice the Feature request flag before, it's done: https://github.com/RSS-Bridge/rss-bridge/issues/2080


Prompt Regex Escaping Confusion

First of all thanks for the new Prompt RegEx feature ❤
I did just try it out and after some fiddeling around with the escaping, it works like a charm.
Expected behaviour
First of all I think in some edge cases it  might make a difference to mention that the RegEx you type in your config.py is a JS RegEx and not a python one.
So that said IMHO it should work similar to this:
copybutton_prompt_text = r"<valid_js_reg_ex> "
copybutton_prompt_is_regexp = True

🌈 🦄
What I did
In my case the prompt I wanted to get rid of was of the form
r"In \[\d*\]: |\.\.\.: "

which is the jupyter console style and pretty much the same as the iPython style from the docs
"\\[\\d*\\]: |\\.\\.\\.: "

So it should be straight forward
"In \\[\\d*\\]: |\\.\\.\\.: "

as in the docs and I would be done?
But it didn't work, so I did some digging and ended up with:
"In \\[\\d*\\]: |\\.\\.\\.: |\\$ "

What internally happens
In my config.py, I set copybutton_prompt_text
copybutton_prompt_text = r"In \[\d*\]: |\.\.\.: "

which is the same as:
copybutton_prompt_text = "In \\[\\d*\\]: |\\.\\.\\.: "

>>> r"\\" == "\\\\"
True

Sphinx now replaces the context variable copybutton_prompt_text in copybutton.js_t and generates copybutton.js.
But when we look at that string which should have just been passed as is, it is missing backslashes.
'In \[\d*\]: |\.\.\.: |\$ '

Well this looks like a totaly valid JS RegEx, so this is fine?
And it would be if written like this (JS just wants keep people on their toes, so why not a different character for RegEx strings?):
/In \[\d*\]: |\.\.\.: |\$ /

Now the string gets passed to the RegExp constuctor, which expects strings to be escaped like python, if you don't use raw strings.
But it doesn't match.
> "In [1]: bar".match(new RegExp('^(In \[\d*\]: |\.\.\.: )(.*)'))
null

Why is this going wrong and why didn't the tests catch this?
TLDR
IMHO this all comes down ro JS RegExp 'helping' you in some cases when it can interprete what you might have ment, which gives you false security (false positiv match in the tests).
Details
Disclaimer I'm in no way a JS expert with insight into the internals, I just found this due to WTF moments + trail and error.
> let proper_regex = new RegExp('^(>>> |\\$ |In \\[\\d*\\]: |\\[\\d*\\]: |\\.\\.\\.: )(.*)')
undefined
> let false_positive_regex = new RegExp('^(>>> |\\$ |In \[\d*\]: |\[\d*\]: |\.\.\.: )(.*)')
undefined
>"$ bar".match(proper_regex)
(3) ["$ bar", "$ ", "bar", index: 0, input: "$ bar", groups: undefined]
> "[1]: bar".match(proper_regex)
(3) ["[1]: bar", "[1]: ", "bar", index: 0, input: "[1]: bar", groups: undefined]
> "[1]: bar".match(false_positive_regex)
(3) ["[1]: bar", "[1]: ", "bar", index: 0, input: "[1]: bar", groups: undefined]
> "...: bar".match(proper_regex)
(3) ["...: bar", "...: ", "bar", index: 0, input: "...: bar", groups: undefined]
> "...: bar".match(false_positive_regex)
(3) ["...: bar", "...: ", "bar", index: 0, input: "...: bar", groups: undefined]

So far so good, all testcases work fine, as we would expect since they pass.
> "In [1]: bar".match(proper_regex)
(3) ["In [1]: bar", "In [1]: ", "bar", index: 0, input: "In [1]: bar", groups: undefined]
> "In [1]: bar".match(false_positive_regex)
null

The only explaination I got for this behaviour, is that JS wants to "help" the user by guessing it's intention, like the famous adding thingy:
> "5" + 1
"51"

but at some points it is like "I don't get it", and the pattern that worked before fails.
Possible fix
Use repr in add_to_context
If this line was changed to:
      {"copybutton_prompt_text": r"{}".format(repr(config.copybutton_prompt_text))[1: -1]}

I know it looks ugly, but would add extra backslashes for escaping (maybe someone else has a more elegant solution).
I'm not sure how much escapeRegExp would need to be adjusted to those changes.
I didn't have luck with backslash replacing in JS:
> 'In \[\d*\]: |\.\.\.: '.replace(/\\/, '\\\\');
"In [d*]: |...: "
> 'In \[\d*\]: |\.\.\.: '
"In [d*]: |...: "

So in conclusion:
Python ❤
JS 😠
P.S.: Wish we had the sphinx-copybutton on github, for the JS blobs I posted 😢

hmmmm - a couple quick thoughts:

Maybe @chrisjsewell can advise on whether the regexp behavior you're describing looks like a JS thing, or maybe it's a Jinja thing? If it's a Jinja thing then I think we can try to design around this in other ways.
@s-weigand would you be interested in making a PR to the docs with some "sample" regexes that will match very common patterns? E.g., the In[*] and Out[*] pattern? That could be helpful in saving people time in the future!

I can make a PR at the weekend, maybe we even get the escaping thingy resolved by then. 😄
What definitely is a JS thing is that two different regexs match the same string.
Ha I just found tha jinja2 has the format filter builtin.
so changing:
https://github.com/executablebooks/sphinx-copybutton/blob/d56f2cf69e7ad8346b9dad950b3cd03bdaba2c16/sphinx_copybutton/_static/copybutton.js_t#L89
to:
  return formatCopyText(target.innerText, {{ "{!r}".format(copybutton_prompt_text) }}, {{ copybutton_prompt_is_regexp | lower }}, {{ copybutton_only_copy_prompt_lines | lower }},  {{ copybutton_remove_prompts | lower }})

Also does the trick, looks nicer and should be more reliable than my repr solution.
If this looks fine to you I can make the PR for docs and the template.

Ha I just found tha jinja2 has the format filter builtin.

Ah cool didn't notice that, yep that looks like it should improve the issue thanks @s-weigand, all them backslashes are a right pain

nice catch @s-weigand !


Inconsistent usage of HTML Web Browser

When invoking the web browser, sometimes focus shifts to the Browser tab and sometimes it doesn't. For example Wallet tab > ... > Buy crypto and Wallet tab > ... > Swap

That's intentional. But will check back to make sure which uses which correctly.


Embed button not visible on toolbar

I've added the embed_provider as indicated here and the embed button to the toolbar group settings, but it doesn't show, tried clearing the cache, refresh, new DNN install on local machine with same results. Using it on DNN 7.4.2.
Really appreciate any help, thanks!

I think there is a problem with the icons, can you post a screenshot of the editor?

Thanks! screenshot attached, icon should be at the left of code snippet icon.

What happens when you click on the icon left next to the code snippet icon. At least for me the embed plug in has math plug in icon

That opens the formula editor, screenshot attached.

Okay, so i guess its a different problem. Is the embed plugin in the list of extraPlugins in the Editor config? are you using the latest version of the Provider?

Yes, it is on the ExtraPlugins field: embed,tableresize,textselection,notification,stylesheetparser,qrcodes,newsarticleslinks,codemirror,dnnpages,xmltemplates,wordcount
I installed 2.1.11 but said it was 2.1.10, so did a repair, still shows as 2.1.10.

I run out of ideas what could be the problem, i assume you loaded the correct toolbar?
You cab check in the browser console if the plugin if you enter
CKEDITOR.instances.EDITORINSTANCE.plugins.embed
you need to replace editorinstance with your id the browser console should suggest you the correct id

any update?

Yes! problem was in the CKToolbarSets.xml, the icon/plugin was being saved as <ToolbarGroup><items><string>embed</string>... , it works fine as <ToolbarGroup><items><string>Embed</string>...
Thanks and Happy Holidays!


Optimize jade_style

Input                             | Time per 10^7 run (ms)
----------------------------------|------------------------
'padding: 8px; margin: 9px'       |  500 =>   55
'padding: 8px; margin: 9px;'      |  705 =>  206
{ padding: '8px', margin: '9px' } | 6930 => 2119
{}                                | 3614 =>  205

LGTM.
Thanks for putting this together, I was thinking the same after taking a look at the other functions, I just cannot find the time to work on it right now.


fix(spark-core): interceptors ignore plugin-logger

I kept the console as a fall-back, just in case; AFAIK, options.logger was the wrong reference.
It's not the case (without the work of yet another interceptor) that the logger is on options.
Effectively, this meant (for us, and likely others) that logging would always fall-back to console.
IIRC, the interceptor's this reference has a spark, on which the logger plugin is registered.
Also cross-referencing #441 for an oft-related (and still existing, but minor) issue involving replacing the logger. We hacked around that issue by explicitly registering the stock logger plugin first, then replacing it with our own, which works for us since we never use the default instance.

Second commit adjusts static create to be consistent w/ e.g. AuthInterceptor.create.
@ianwremmel: does this look in good shape for you to merge? If so:
I remember how to ferry this through the Jenkins (validated the merge).


About pretrained model

@zafirshi @elnino9ykl @MasterHow
Hi, I am very interested in your work and code. Thanks for your inspiring reserach.
I would like to download the pretrained model on the pitts250k and yq360 for the inference process. Is there the download link?
Please help me , thanks a lot!

Thank you very much for your interest in our work！ 😄
The pre-trained model weights for PanoVPR will not be made public for now. You can get them in the logs folder after running the train.py file.
Of course, as mentioned in issue#3, you will need to tune the parameters for different framework configurations.


[NEW] Get copyright message from package.json

Avoid injecting copyright notice into translation files, keeping a single source of truth.

Do we care about internationalization on this string? @engelgabriel


func _process(_delta) should be default preset instead of func _process(delta)

Godot version:
3.1 stable
Issue description:
func _process(delta) results in a warning, func _process() is not possible
Hence func _process(_delta) should be the default preset.
When a new user adds a default script to a node the first time, they should not get a warning.

I disagree, variables shouldn't be "ignored" by default and I think it's the user's job to choose put the underscore when desired.

I disagree too.
The proper solution IMO is what I proposed in https://github.com/godotengine/godot/issues/27576#issuecomment-478745395 i.e. split this warning into two warnings:

One for user-defined methods (ON by default), so my_method(arg_i_wont_use) will raise a warning
One for engine callbacks (OFF by default), so _process(delta) won't raise a warning unless you enabled this warning yourself

The proper solution IMO is what I proposed in #27576 (comment) i.e. split this warning into two warnings:
One for user-defined methods (ON by default), so my_method(arg_i_wont_use) will raise a warning
One for engine callbacks (OFF by default), so _process(delta) won't raise a warning unless you enabled this warning yourself

Yes, this sounds like a much better solution!

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


VM migration fails due to VR in "Migrating" state

ISSUE TYPE

Bug Report

COMPONENT NAME

CLOUDSTACK VERSION

<IP_ADDRESS>

CONFIGURATION

OS / ENVIRONMENT

VMware 6.7
SUMMARY

While ESXi host is being put into maintenance mode in ACS - if the VM is being migrated while its VR is also in the Migrating state - the UserData can’t be updated after the user VM migration is done (because VR is in “Migrating” state) - thus ACS cancels the user VM migration job (even though the VM was already moved to a new host) and issues Stop command, causing VM to stop.
STEPS TO REPRODUCE

Put host into maintenance mode - optionally, since this could be hard to reproduce in the non-busy/test env, consider manually updating the VR state to "Migrating" in databases (not reproduced at the moment of creating this issues)

EXPECTED RESULTS

Migration completes.

ACTUAL RESULTS

Migation "physically" finishes (user VM moved to another host), migration job cancelled, and VM stopped as a result of failed migration job.

Log excerpt:
2023-02-07 15:39:59,157 DEBUG [c.c.a.m.ClusteredAgentAttache] (Work-Job-Executor-100:ctx-117cc6cd job-2450367/job-2455893 ctx-69f4e98e) (logid:106487b6) Seq 39519-3613294276034694276: Forwarding Seq 39519-3613294276034694276:  { Cmd , MgmtId:<PHONE_NUMBER>15, via: 39519(b-esx-inter-003.mgmt.c.domain.local), Ver: v1, Flags: 100011, [{"com.cloud.agent.api.CheckVirtualMachineCommand":{"vmName":"i-390-172822-VM","wait":"20","bypassHostMaintenance":"false"}}] } to<PHONE_NUMBER>89 2023-02-07 15:39:59,198 DEBUG [c.c.a.t.Request] (AgentManager-Handler-3:null) (logid:) Seq 39519-3613294276034694276: Processing:  { Ans: , MgmtId:<PHONE_NUMBER>15, via: 39519, Ver: v1, Flags: 10, [{"com.cloud.agent.api.CheckVirtualMachineAnswer":{"state":"PowerOn","result":"true","wait":"0","bypassHostMaintenance":"false"}}] } 2023-02-07 15:39:59,198 DEBUG [c.c.a.t.Request] (Work-Job-Executor-100:ctx-117cc6cd job-2450367/job-2455893 ctx-69f4e98e) (logid:106487b6) Seq 39519-3613294276034694276: Received:  { Ans: , MgmtId:<PHONE_NUMBER>15, via: 39519(b-esx-inter-003.mgmt.c.domain.local), Ver: v1, Flags: 10, { CheckVirtualMachineAnswer } } 2023-02-07 15:39:59,227 DEBUG [o.a.c.e.o.VolumeOrchestrator] (Work-Job-Executor-100:ctx-117cc6cd job-2450367/job-2455893 ctx-69f4e98e) (logid:106487b6) Releasing 2 volumes for VM: 172822 from host: 39516 2023-02-07 15:39:59,268 DEBUG [c.c.n.NetworkModelImpl] (Work-Job-Executor-100:ctx-117cc6cd job-2450367/job-2455893 ctx-69f4e98e) (logid:106487b6) Service SecurityGroup is not supported in the network id=24593 2023-02-07 15:39:59,304 WARN  [c.c.n.e.VirtualRouterElement] (Work-Job-Executor-100:ctx-117cc6cd job-2450367/job-2455893 ctx-69f4e98e) (logid:106487b6) Failed to update userdata for vm VM instance {id: "172822", name: "i-390-172822-VM", uuid: "cc2b97f6-b3be-4f0b-857f-2b9fb22b846b", type="User"} and nic NicProfile {"broadcastUri":"vlan:\/\/1206","iPv4Address":"<IP_ADDRESS>","id":407159,"reservationId":"bb8d02b1-cd90-4367-b099-2ce888c31308","vmId":172822} 2023-02-07 15:39:59,354 DEBUG [o.a.c.n.t.BasicNetworkTopology] (Work-Job-Executor-100:ctx-117cc6cd job-2450367/job-2455893 ctx-69f4e98e) (logid:106487b6) SAVE USERDATA TO ROUTE RULES 2023-02-07 15:39:59,355 WARN  [o.a.c.n.t.BasicNetworkTopology] (Work-Job-Executor-100:ctx-117cc6cd job-2450367/job-2455893 ctx-69f4e98e) (logid:106487b6) Unable to apply save userdata entry, virtual router is not in the right state Migrating 2023-02-07 15:39:59,355 WARN  [c.c.v.VirtualMachineManagerImpl] (Work-Job-Executor-100:ctx-117cc6cd job-2450367/job-2455893 ctx-69f4e98e) (logid:106487b6) Unable to migrate VM instance {id: "172822", name: "i-390-172822-VM", uuid: "cc2b97f6-b3be-4f0b-857f-2b9fb22b846b", type="User"} to Host {"id": "39519", "name": "b-esx-inter-003.mgmt.c.domain.local", "uuid": "5ee7908b-b344-4e83-8034-d4c7a1868339", "type"="Routing"} due to [Resource [DataCenter:1] is unreachable: Unable to apply save userdata entry, virtual router is not in the right state] com.cloud.exception.ResourceUnavailableException: Resource [DataCenter:1] is unreachable: Unable to apply save userdata entry, virtual router is not in the right state 	at org.apache.cloudstack.network.topology.BasicNetworkTopology.applyRules(BasicNetworkTopology.java:412) 	at org.apache.cloudstack.network.topology.BasicNetworkTopology.saveUserDataToRouter(BasicNetworkTopology.java:352) (edited)

@andrijapanicsb this behaviour is also experienced on KVM

Fixed with #7285

Fixed with #7285

Fixed in #7360


Make it easier to run hathor-explore-service locally

Description
Today, running hathor-explorer-service is a trial-and-error process. The developer must know how serverless work, and how to install all dependencies.

Renaming the Dockerfile will break the deploy job in Github Actions.
We need to use the new name where need in the Makefile


[Issue] Can't find local baseline for 15.1.1531.10

Describe the issue
This error display while trying to use .\CompareExchangeHashes.ps1 script to check Exchange Health.
Script Output
If applicable, add the exception that you are seeing that wasn't handled.
PS C:\Users\FME\Downloads> .\CompareExchangeHashes.ps1
Do you want to run software from this untrusted publisher?
File C:\Users\FME\Downloads\CompareExchangeHashes.ps1 is published by CN=Microsoft Corporation, O=Microsoft
Corporation, L=Redmond, S=Washington, C=US and is not trusted on your system. Only run scripts from trusted publishers.
 Help (default is "D"): R
[03/11/2021 15:44:01] Started...
Found exchange version: 15.1.2176.2
Can't find local baseline for 15.1.1531.10
Downloading baseline file from GitHub to C:\Users\FME\Downloads\baseline_15.1.1531.10.zip
LoadFromGitHub : C:\Users\FME\Downloads\baseline_15.1.1531.10.zip not found... please open issue on https://github.com/microsoft/CSS-Exchange/issues, we will work on it
At C:\Users\FME\Downloads\CompareExchangeHashes.ps1:647 char:13

        LoadFromGitHub -url $zip_file_url -filename $zip_file -in ...

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,LoadFromGitHub

PS C:\Users\FME\Downloads>
Thanks for the help. ;-)

I'm experiencing this issue as well. When I go to the releases page, there's no baseline for "15.1.1531.10".

Duplicate of #258


Add connection assume_role configuration

Is your feature request related to a problem? Please describe.
Currently in order to configure a connection to an AWS account, you need to provide a aws config profile or the credentials in the configuration file. This can make connecting to a large number of accounts difficult to maintain the configuration.
Describe the solution you'd like
It would be ideal to be able to provide a assume_role to the connection config in place of credentials and this role would be assumed for the connection.
Describe alternatives you've considered
A way to dynamically generate the needed aws profiles and dynamically generate the connection configuration file. This is not ideal in any way.
Additional context
This type of configuration would make dynamically adjusting the connection information simple and scale well with large multi account configurations.

This looks like it would be a great addition!
Our main difficulty is generating this .aws/config file in ECS, but having the ability to assume role directly from the connection configuration would make things so much easier!
Happy to help out to push this through if needed, although not contributed here before either.  Let me know 😄

@brittandeyoung Thanks for opening the issue and submitting the PR!
I like the idea of being able to specify a role ARN for easier creds setup, but I do have some concerns around how to pass options/config when assuming the role. For instance, if you want to assume a role with an external ID or MFA serial number in the AWS CLI or the AWS plugin today, you'd setup the AWS profile like:
[profile myrole]
role_arn = arn:aws:iam::234567890123:role/SomeRole
mfa_serial = arn:aws:iam::234567890123:mfa/saanvi
external_id = 123456

If you were to use Steampipe today to authenticate to an AWS account and reference your profile in aws.spc, the AWS SDK would automatically take in this profile information and pass it along to the STS AssumeRole API call.
So if we wanted to offer the same functionality in the AWS plugin, we'd most likely need to offer additional config args, e.g., assme_role_mfa_serial = '...', or have the assume role config arg be an object and essentially re-create the AWS profile format, e.g.,
assume_role_config = {
  role_arn = '...'
  mfa_serial = '...'
}

Is your use case just for assuming a role with no other configuration/options specified, or do you have use cases for this as well?
@johnsmyth Any thoughts on how we could support a feature like this?

@cbruno10
My thoughts on this:
If we were deploying Steampipe to ECS or similar, we would need to have:

The .aws/config configuration stored in a Docker image
AWS CLI installed (since I don't believe Steampipe directly reads .aws/config, it resolves the values through AWS CLI)

Whereas with this implementation, the roles are assumed within Steampipe itself, and does not depend on AWS CLI at all.
I don't think you will need to support mfa_serial since the user would have already authenticated with their default profile - a profile that already has permissions to assume roles.

@p5 Thanks for the examples!
I don't think AWS CLI would need to be installed, as the AWS plugin uses the AWS SDK Go v2, which I don't believe relies on the AWS CLI (but I could be wrong on this), as it handles creds and API calls directly in its code.
The Terraform AWS provider provides an assume_role block, which allows users to pass in other options, e.g.,:
provider "aws" {
  assume_role {
    role_arn     = "arn:aws:iam::123456789012:role/ROLE_NAME"
    session_name = "SESSION_NAME"
    external_id  = "EXTERNAL_ID"
  }
}

So we could provide something similar in this plugin (along with assume_role_with_web_identity now or in the future).

looking at the schema for the steampipe plugin, it does not look like they support configuration blocks. I will be updating the pr to have top level items for the following:
assume_role_arn
assume_role_duration
assume_role_external_id
assume_role_session_name

@brittandeyoung We have an example in the Kubernetes plugin you can follow:

https://github.com/turbot/steampipe-plugin-kubernetes/blob/main/config/kubernetes.spc#L63-L71
https://github.com/turbot/steampipe-plugin-kubernetes/blob/main/kubernetes/connection_config.go#L14

In this example, we allow any top level key in the map, e.g., "my_chart_name", but for this case, we would know all of the key names.

@cbruno10 Thank you for the example I am currently looking to see if i can make the needed adjustments to move it to a block using your example.

@brittandeyoung Can you please also test what happens when you try to run a query past the role session expiration time when using your new config arg(s)? For instance, if you run the following steps:

Setup aws.spc to have an AWS connection that specifies an assume role timeout of 15 mins
Start Steampipe (steampipe service start)
Run a query (steampipe query "select name from aws_iam_role")
Wait 20 minutes
Run another query (steampipe query "select name from aws_s3_bucket")

Do you get an error, does the request succeed, or is there another outcome?
I believe this scenario is already handled in the AWS plugin by the AWS SDK when pointing to a profile that has configuration details in the credentials or config file (except when a static MFA token is passed in), so it should be supported for this new config arg as well in order to allow users to use long running Steampipe processes.

@cbruno10 I am struggling to get the configuration working in the config block. The configuration that I currently have pushed to the PR works great with the root level configurations.
One thing I am struggling with is that the schema for the plugin you are referencing is dynamic, while the schema of this plugin is statically defined. I do not see a way in the static definition to define a block, in other projects like terraform providers this is typically done by a List of one, but I  do not see that as an option in the plugin schema.
Any tips from here are appreciated.
Here is what I tried, I removed the static schema config, and set the schema to be dynamic. This alone causes the plugin to crash even without adding any additional configuration.
This is the error when attempting to run with dynamic schema:
ERROR: failed to start plugin 'hub.steampipe.io/plugins/turbot/aws@latest': timed out waiting for hub.steampipe.io/plugins/turbot/aws@latest to startup after 20 seconds (SQLSTATE HV000)
I am very new to steampipe development, so i could be making a simple rookie mistake =) .

Hi @brittandeyoung, thanks for submitting the PR. The changes look great!
For the above config example, I think you need to mention the cty tags as well to provide type hints to certain keys for parsing HCL configurations.
Could you please try the below config struct?
type awsConfig struct {
	Regions               []string          `hcl:"regions,optional"`
	DefaultRegion         *string           `hcl:"default_region,optional"`
	Profile               *string           `hcl:"profile,optional"`
	AssumeRole            *assumeRoleConfig `hcl:"assume_role,optional"`
	AccessKey             *string           `hcl:"access_key,optional"`
	SecretKey             *string           `hcl:"secret_key,optional"`
	SessionToken          *string           `hcl:"session_token,optional"`
	MaxErrorRetryAttempts *int              `hcl:"max_error_retry_attempts,optional"`
	MinErrorRetryDelay    *int              `hcl:"min_error_retry_delay,optional"`
	IgnoreErrorCodes      []string          `hcl:"ignore_error_codes,optional"`
	EndpointUrl           *string           `hcl:"endpoint_url,optional"`
	S3ForcePathStyle      *bool             `hcl:"s3_force_path_style,optional"`
}

type assumeRoleConfig struct {
	RoleARN     *string `hcl:"role_arn" cty:"role_arn"`
	Duration    *string `hcl:"duration,optional" cty:"duration"`
	SessionName *string `hcl:"session_name,optional" cty:"session_name"`
}

Let us know if you need any help, thanks!

I have updated the PR with a working assume role configuraiton. Below is an example config that will work for assuming roles:
connection "aws" {
  plugin  = "aws"
  assume_role = {
    role_arn = "arn:aws:iam::1111111111111:role/steampipe"
    duration = "300s"
    external_id = "testing"
    session_name = "steampipe"
  }
  // default_region = "us-east-1"
  regions = ["us-east-1"]
}

I am still having issues with the optional config items not actually being read into the config. In this example, eexternal_id , session_name and duration. I am assuming there is something missing with my structure tags. If I add a cty tag for the optional fields, steam pipe crashes and fails to load at all. If I remove the optional tag and add a cty tag, they load, but then they MUST be provided in the config.

Still would love to have this available.


gpio.h: remove extraneous extern C declaration

Signed-off-by: Jon Trulson<EMAIL_ADDRESS>

Is there anything else that needs to be done to get this fix committed?  It causes build problems with C based drivers.

Not really, I'll take a good look at a bunch of PRS and merge tmrw. Sorry and thx for the poke

Thanks!

Merged - would be interested in seeing what the issues with this are - can you link me to what failures have been happening?

Since the clause was not protected by an "#ifdef __cplusplus", any C code compiled with JAVACALLBACK or SWIGJAVA defined would fail, as 'extern "C" ...' isn't valid C.
Since the file is already properly protected via this guard at the top and bottom, this clause was not necessary anyway.


CIF-899 - Display category image on generic category page

Description
Currently, Venia uses a feature in their store front that display a category image on the category page. By implementing this feature in our AEM Venia store front, we can make our store front visually more appealing.

Because of API changes in the sling model, I bumped the current snapshot version to 2.0.
Added image to product list component. Will only be displayed when showImage property from the design dialog is true and the category has an actual image.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] I have signed the Adobe Open Source CLA.
[X] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[X] I have added tests to cover my changes and the overall coverage did not decrease.
[X] All unit tests pass on CircleCi.
[X] I ran all tests locally and they pass.

Codecov Report

Merging #68 into master will decrease coverage by 0.47%.
The diff coverage is 23.07%.

@@             Coverage Diff              @@
##             master      #68      +/-   ##
============================================
- Coverage     62.65%   62.18%   -0.48%
- Complexity      166      168       +2
============================================
  Files            26       26
  Lines           948      960      +12
  Branches         64       64
============================================
+ Hits            594      597       +3
- Misses          325      334       +9
  Partials         29       29

Flag
Coverage Δ
Complexity Δ

#karma
94.92% <ø> (ø)
0 <ø> (ø)
:arrow_down:

#unittests
53.73% <23.07%> (-0.46%)
168 <2> (+2)

Impacted Files
Coverage Δ
Complexity Δ

...ore/components/models/productlist/ProductList.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...nternal/models/v1/productlist/ProductListImpl.java
31.06% <27.27%> (-0.12%)
16 <2> (+2)

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 28cf4bc...0bb5c2e. Read the comment docs.


[Proposal] On the meaning of "terminated" "truncated" "done" etc. I miss "causality break" flag

Proposal
Arrive to a wide agreement on the meaning of "done", "terminated" and "truncated".
Motivation
With the new API, now we have two boolean variables (terminated, truncated) instead on one, which gives 4 possible situation. However in the Gymnasium docs it is said that reset should be called when terminated=True and also that reset should be called when truncated= true.
So it seems that with respect to resetting the environment, there is no difference, at least in specification and intention.
It also seems that both conditions point to an event which is very significative:
The rupture of the causal chain in the temporal series of the MDP.
This rupture, signaled previously by "done" and now by both "terminated and "truncated" has been used widely in RL libraries to do several things, like processing the data in replay buffer, decide in the programed "collectors" some conditions etc.
However, there are situations where you need to assume a sequence is "complete", i.e, for n-step returns, total accumulated discounted return calculation etc. however, the causal chain does not end as in continuing taks but still you want to communicate that outside so that collectors can save stats and process replay buffer memory but reset should not be done.
Of course, we can add this into the info field, but we can then have "truncated" inside info as was done previously.
So basically, it would be useful to take advantage of the 4 possible meaning on the true,false of these two variables and have a consensus on this so that RL library developers take that into account.
¿What event is now signaling causal chain rupture exclusively without requiring a reset?
Pitch
Arrive to a wide agreement/consensus  on the meaning of "done", "terminated" and "truncated". and take into account the causal chain rupture.
Alternatives
add a specific output variable to step indicating causal chain rupture.
Additional context
No response
Checklist

[X] I have checked that there is no similar issue in the repo

Interesting question, Im not sure I totally understand what you mean
Could you look at these page and explain what definition we are missing

https://gymnasium.farama.org/content/migration-guide/#environment-step
https://github.com/pseudo-rnd-thoughts/gym/issues/1
https://github.com/Farama-Foundation/Gymnasium/pull/105

This issue has been inactive for over a year and the OP uses the term "causal chain rupture" which I can not understand what it means closing


Add list of available commands to README.md

The list of commands should be placed between the sections "Error Codes" and "Log-Entry Error Levels"
The list of commands should contain all available commands, including those that are only accessible to committee members

It is preferable to have a list of features rather than just commands. Closed as a duplicate.


doc: fix broken website

/azp run

/azp run

Codecov Report

Merging #1251 (d2f095e) into master (6459f3c) will decrease coverage by 30.85%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #1251       +/-   ##
===========================================
- Coverage   83.11%   52.26%   -30.86%
===========================================
  Files         300      248       -52
  Lines       13699    11984     -1715
  Branches      660      660
===========================================
- Hits        11386     6263     -5123
- Misses       2313     5721     +3408

Impacted Files
Coverage Δ

...m/src/main/scala/com/microsoft/lightgbm/SWIG.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...la/org/apache/spark/ml/param/TypedArrayParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

.../microsoft/azure/synapse/ml/io/binary/Binary.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...oft/azure/synapse/ml/codegen/GenerationUtils.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...m/microsoft/azure/synapse/ml/codegen/CodeGen.scala
0.00% <0.00%> (-98.44%)
:arrow_down:

...m/microsoft/azure/synapse/ml/stages/Batchers.scala
3.70% <0.00%> (-92.60%)
:arrow_down:

.../execution/streaming/continuous/HTTPSourceV2.scala
0.00% <0.00%> (-92.09%)
:arrow_down:

...che/spark/sql/execution/streaming/HTTPSource.scala
0.00% <0.00%> (-90.00%)
:arrow_down:

...ql/execution/streaming/continuous/HTTPSinkV2.scala
0.00% <0.00%> (-89.75%)
:arrow_down:

...microsoft/azure/synapse/ml/codegen/Wrappable.scala
10.25% <0.00%> (-89.11%)
:arrow_down:

... and 162 more

Continue to review full report at Codecov.

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

/azp run


WebGL w/Build Setting "Development Build" on WalletConnectSession.DisconnectSession may throw 'Memory Access Out of Bounds'

Summary
In Unity3d for WebGL if Development Build in Build Settings is checked, when WalletConnect.Disconnect is called a "memory access out of bounds" error is thrown.
Steps to reproduce
Unity 2020.3.34 or 2021.2.5

Set WebGL as Platform
Check 'Development Build'

Click Run and Build
When app is running in browser click on button or cause other event that calls WalletConnect.Session.Disconnect().
Error in console log

RuntimeError: memory access out of bounds,RuntimeError: memory access out of bounds
    at __ZN21InterfaceFuncInvoker3IiP52ByteU5BU5D_tDBBEB0E8362242FA7223000D978B0DD19D4B0726iiE6InvokeEtP11Il2CppClassP12Il2CppObjectS1_ii (http://localhost:58545/Build/wgl.wasm:wasm-function[21526]:0x913cea)
    at dynCall_iiiiiii (http://localhost:58545/Build/wgl.wasm:wasm-function[82595]:0x15cca65) ...

What is the current bug behavior?
Appears to be related to the Web Request in the Session.Disconnect method, throws a 'memory out of bounds' error.
What is the expected correct behavior?
Disconnect does not throw an error.
Possible fixes
Appears to be related to a known Unity3D WebGL issue (i.e. https://answers.unity.com/questions/1798594/webgl-runtimeerror-memory-access-out-of-bounds-2.html)
Try to trap and handle error?

Thank you for reporting this, I do believe the correct thing to do here is simply catch the error and (maybe) log it
@gigajuwels can you take a look at this?

closed due to v1 depreciation


chore: disable rediscovery test

ill get that fella one day, but today is not the day

#9844  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Maddiaa0 and the rest of your teammates on  Graphite

Its time will come.


Improve error thrown when deserializing invalid JSON.

Currently, if an RPC value contains invalid JSON, we just get an SyntaxError with a message like "Unexpected token T in JSON at position 0".
This is not usually an issue, since we never expect backends to respond with malformed JSON (even in error responses).  However, we may get non-JSON responses from gateways or proxys that sit between a frontend and backend, and when that does happen, including the entire malformed JSON in the error message is much more helpful.

This reminds me of a question i asked @swbsi this morning. There used to be a lint rule that enforced that arguments and return types of RpcMethods be only of primitive or ArrayBuffer types. Steve says it's not running any more. @calebmshafer is there a way to reinstate that rule? It should now also be run against IpcInterface classes.

@kabentley we didn't re-implement that rule in ESLint but we'll add it to the backlog.
@paulius-valiunas can you add that to the backlog? I remember there was a reason we skipped it but don't remember the reason... Do you?

This reminds me of a question i asked @swbsi this morning. There used to be a lint rule that enforced that arguments and return types of RpcMethods be only of primitive or ArrayBuffer types. Steve says it's not running any more. @calebmshafer is there a way to reinstate that rule? It should now also be run against IpcInterface classes.

@kabentley we didn't re-implement that rule in ESLint but we'll add it to the backlog.
@paulius-valiunas can you add that to the backlog? I remember there was a reason we skipped it but don't remember the reason... Do you?


Permission denied (publickey). fatal: Could not read from remote repository.

➜  BryanBo-Cao.github.io git:(master) git remote rm origin
➜  BryanBo-Cao.github.io git:(master) git remote add origin git@github.com:BryanBo-Cao/BryanBo-Cao.io.git
➜  BryanBo-Cao.github.io git:(master) git push origin master
Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

Reference:


[bug] fixing signup issue and login issue

error in signing up and logging in ....error in fetching profiles

@RohitGupta1237  can you please assign me.


Unescaped command line args

Some youtube video IDs begin with a hyphen. These are not properly quoted, causing youtube-dl to interpret it as a command line option.
const ytdl = require('youtube-dl')
ytdl.getInfo('-3VOMOuH05U', (err, info) => console.error(err))

Error: Command failed: /Users/andy/Documents/stem/alpha/node_modules/youtube-dl/bin/youtube-dl --dump-json -f best -3VOMOuH05U
Usage: youtube-dl [OPTIONS] URL [URL...]

youtube-dl: error: no such option: -3

    at ChildProcess.exithandler (child_process.js:206:12)
    at emitTwo (events.js:106:13)
    at ChildProcess.emit (events.js:191:7)
    at maybeClose (internal/child_process.js:877:16)
    at Socket.<anonymous> (internal/child_process.js:334:11)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:188:7)
    at Pipe._handle.close [as _onclose] (net.js:498:12)

Please provide relevant url.

URL to what exactly? The video I'm trying to get info for? It would be https://www.youtube.com/watch?v=-3VOMOuH05U.
The youtube-dl CLI can take a URL or a youtube video ID, so in the code above I've simply provided the video ID rather than the full URL. It does work when the full URL is used, as the URL does not begin with a hyphen. But it breaks when the video ID is used instead because of the hyphen. It's broken for some IDs. It could be fixed by explicitly ending the command line options with --. Like so:
# whats currently generated
$ /path/to/bin/youtube-dl --dump-json -f best -3VOMOuH05U

# what would work for all cases
$ /path/to/bin/youtube-dl --dump-json -f best -- -3VOMOuH05U

@acarl005 you obviously know that you want to download video from YT so to avoid confusion and possible mismatching of services it would be a lot easier if just to pass the youtube url https://www.youtube.com/watch?v= before the id as per https://github.com/rg3/youtube-dl#how-do-i-download-a-video-starting-with-a-- instead of building that into the module and have possible false positive outcome.


Project Status

Hello @codenoid @yeka
What's the status of this project? I'm interested in adding features but there hasn't been any updates in over a year. The repository also doesn't contain a license making it legally impossible to fork and continue developing.
Thanks!

Hi @gaby, thankyou for your interest in this project
About the status of the project, We still use it internally (company-level), however there is some living bug that hasn't been reported or fixed yet...
And I'm still trying to steal a time to fix my old OSS project

Thanks for adding a license! I just submitted a PR with some minimal changes.

@codenoid Any updates on this project?

@gaby  I Would like to rewrite entire code and uses svelte as a frontend
Actually, I Already did it month ago, but it's still on my private git

@codenoid Nice! I was planning on forking the project if it was abandoned.

Here's the new version I just made
https://github.com/codenoid/temporar


clientv3: some testcases fail

vimal (master) clientv3 $ go test -v > err.txt 2>&1
vimal (master) clientv3 $ grep FAIL: err.txt
--- FAIL: TestDialTimeout (2.36s)
--- FAIL: TestTxnPanics (0.36s)

but when i run these test cases individually, test cases pass
vimal (master) clientv3 $ go test -v -run TestDialTimeout
=== RUN   TestDialTimeout
2016-10-18 11:20:40.626987 I | etcdserver/api/v3rpc: grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp <IP_ADDRESS>:12345: getsockopt: connection refused"; Reconnecting to {localhost:12345 <nil>}
2016-10-18 11:20:41.626497 I | etcdserver/api/v3rpc: grpc: addrConn.resetTransport failed to create client transport: connection error: desc = "transport: dial tcp <IP_ADDRESS>:12345: getsockopt: connection refused"; Reconnecting to {localhost:12345 <nil>}
2016-10-18 11:20:42.629205 I | etcdserver/api/v3rpc: Failed to dial localhost:12345: context canceled; please retry.
--- PASS: TestDialTimeout (2.00s)
PASS
ok  	github.com/coreos/etcd/clientv3	2.032s
vimal (master) clientv3 $
vimal (master) clientv3 $ go test -v -run TestTxnPanics
=== RUN   TestTxnPanics
--- PASS: TestTxnPanics (0.00s)
PASS
ok  	github.com/coreos/etcd/clientv3	0.026s

err.txt

It's picking up the cluster that's launched in TestMain for the Example tests.  ./test doesn't complain because it passes in -run=Test so the cluster never starts.
--- FAIL: TestDialTimeout (2.36s)
        leak.go:96: Test appears to have leaked a Transport:
                encoding/binary.(*bigEndian).Uint64(0xdc4aa0, 0xc422ff0de8, 0x8, 0x8, 0x8)
                        <autogenerated>:20
                encoding/binary.Read(0xd6ef80, 0xc4226bd4c0, 0xd7c5a0, 0xdc4aa0, 0x80c7e0, 0xc422ff0de0, 0x1, 0x0)
                        /usr/local/go/src/encoding/binary/binary.go:187 +0x4ce
                github.com/coreos/etcd/rafthttp.(*messageDecoder).decode(0xc4201d5c70, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
                        /Users/vimal/work/gowork/src/github.com/coreos/etcd/rafthttp/msg_codec.go:53 +0x112
                github.com/coreos/etcd/rafthttp.(*streamReader).decodeLoop(0xc420142680, 0xd733c0, 0xc4226bd4c0, 0x98e0d4, 0x7, 0x2, 0x0)
                        /Users/vimal/work/gowork/src/github.com/coreos/etcd/rafthttp/stream.go:358 +0x20e
                github.com/coreos/etcd/rafthttp.(*streamReader).run(0xc420142680)
                        /Users/vimal/work/gowork/src/github.com/coreos/etcd/rafthttp/stream.go:311 +0x6f4
                created by github.com/coreos/etcd/rafthttp.(*streamReader).start
                        /Users/vimal/work/gowork/src/github.com/coreos/etcd/rafthttp/stream.go:296 +0xb

Ideally this would work without the extra -run=Test stuff, but I don't think it's possible with the current go testing framework. Closing for now.


Set velocity direction separately from rotation

I'm trying to create particles that float straight up, with each particle having a random static rotation. It seems like a fairly straightforward situation. I've tried a lot of different behavior combinations but as far as I can tell this isn't possible - because the speed behavior relies on rotation for setting its direction. Is that correct?

That is correct. The Path, Speed, and Acceleration movement behaviors rely on the initial rotation to pick their direction. The simplest solution would probably be to make some sort of "ExtraLateStaticRotationBehavior" (that you may or may not use in combination with a regular rotation behavior) to set the rotation after it has been read by your movement initialization. Make a copy (or subclass) of StaticRotationBehavior and set its order to BehaviorOrder.Late + 1 (or 6), and register it on Emitter. The NoRotationBehavior does this exactly, but uses a defined rotation instead of a random rotation.


Fix bump.sh and publish.sh for new crate structure

I have added a new crate extendr-engine (#50) to remove the engine part from the default api. However, the scripts bump.sh and publish.sh don't reflect this change yet.

Claus has suggested bumping the version right after the publish. This will help us to distinguish versions that are pulled from github.

I like the versioning recommendations in the R packages book, which is to add a dev version (e.g., .9000) right after a package has been published, and then bump to the next non-dev version upon release. Is this possible with the Rust versioning system?

I think in Rust we have to go 0.1.10 -> 0.1.11-alpha -> 0.1.11 instead of 0.1.10 -> <IP_ADDRESS>00 -> 0.1.11.
https://docs.rs/semver/0.11.0/semver/

Or maybe we should just delete these two scripts and transition to cargo workspaces:
https://crates.io/crates/cargo-workspaces
cargo workspaces version can bump versions.
cargo workspaces publish can publish to crates.io.

cargo workspaces version can bump versions.

Sorry, I forgot to comment that this doesn't handle the versions specified with path. So we need some tweaks about these lines:
https://github.com/extendr/extendr/blob/41772722a01062b064320c98d687e3eae889790a/extendr-api/Cargo.toml#L13-L14
Btw, be careful when you try this command as this immediately commits and pushes the new version to remotes... When I tried this, I wasn't the member of extendr so the attempt was just rejected... --no-git-commit seems to save us.

If you have some experience with this tool, could you write up a workflow and add to the README, where it currently talks about bump.sh and publish.sh?

After experimenting a bit more, I think cargo workspaces version is currently broken. I'll try to fix it there, but I'm not sure if it gets done before the next release.

Sorry, it turned out it's just I didn't understand the usage... I think I can write up a workflow soon.


Error creating: Internal error occurred: failed calling webhook "sidecar-injector.istio.io": Post "https://istiod.istio-system.svc:443/inject?timeout=30s": Address is not allowed

Bug Description
HI!
We have EKS v1.19 cluster in AWS in the VPC CIDR <IP_ADDRESS>/16 with 2 subnets: <IP_ADDRESS>/24, <IP_ADDRESS>/24
Since we reached limit of free IPs for nodes, podes in those subnets (<IP_ADDRESS>/24, <IP_ADDRESS>/24) we created a new VPC CIDR (<IP_ADDRESS>/16) and configured EKS to use the pods' IPs from the new CIDR - following the article https://aws.amazon.com/premiumsupport/knowledge-center/eks-multiple-cidr-ranges/
The issue is that once we do all that above the istio-operator, istiod, isito-ingressgateway pods up and running with the IPs from the subnets from the new CIDR (<IP_ADDRESS>/16) all the containers are failing with the error:
Error creating: Internal error occurred: failed calling webhook "sidecar-injector.istio.io": Post "https://istiod.istio-system.svc:443/inject?timeout=30s": Address is not allowed
Version
kubectl version --short
Client Version: v1.19.12
Server Version: v1.19.13-eks-8df270

"istio-operator" version 1.9.2

Additional Information
Error creating: Internal error occurred: failed calling webhook "sidecar-injector.istio.io": Post "https://istiod.istio-system.svc:443/inject?timeout=30s": Address is not allowed

@pluksha
I have face similar issue.
Do you find how to solve?

Hi!
Yes, try to restart EKS control nodes - see
https://www.reddit.com/r/kubernetes/comments/dsj5on/address_is_not_allowed_when_deploying_metrics/
—
Regards,
Pavel Luksha
Devops Engineer, Klika Tech, Inc.
Mobile: +48 572 443 350
Email: @.***
www.klika-tech.com
NOTICE: This email may contain confidential and privileged material for the
sole use of the intended recipient(s). Any review, use, distribution or
disclosure by others is strictly prohibited. If you are not the intended
recipient (or authorized to receive for the recipient), please contact the
sender by reply email and delete all copies of this message.
On Thu, Nov 3, 2022 at 1:42 PM Jay Sithiporn @.***>
wrote:

@pluksha https://github.com/pluksha
I have face similar issue.
Do you find how to solve?
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/36702#issuecomment-1302041786, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AUVGRIOI6DTL6EOSW56JBRLWGOXMLANCNFSM5LH3BNGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Provide reason for note about using .bashrc instead .bash_profile on Ubuntu/Fedora

At https://github.com/yyuu/pyenv in Installation/Basic GitHub Checkout section there's the following note:

Ubuntu and Fedora note: Modify your ~/.bashrc file instead of ~/.bash_profile.

I would like to know what's the reason for this? I think this note should be expanded with this information.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/41037895-provide-reason-for-note-about-using-bashrc-instead-bash_profile-on-ubuntu-fedora?utm_campaign=plugin&utm_content=tracker%2F282009&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F282009&utm_medium=issues&utm_source=github).

issue tracker is not question forum

This is issue raised against documentation...


k8s maintenance: k8s node pool upgrades on EKS clusters

#4007 upgraded all control planes to 1.29, but we need to bring the node pools to 1.29 as well. Documentation on how to do this was updated in #4099 and is made available at https://infrastructure.2i2c.org/howto/upgrade-cluster/aws/ - where step 4 about upgrading the control plane is already done.
Clusters with node groups to upgrade

[x] 2i2c-aws-us
[ ] catalystproject-africa
[ ] earthscope
[ ] gridsst
[ ] jupyter-meets-the-earth
[ ] nasa-cryo
[ ] nasa-esdis
[ ] nasa-ghg
[ ] nasa-veda
[ ] openscapes
[ ] opensci
[ ] smithsonian
[ ] ubc-eoas
[ ] victor

Should be paired with someone else (TBD)

@sgibson91 agreed to do this work this during this sprint. I'm around to discuss anything if requested.

Most clusters are done, there are only the remaining clusters which have some user servers still up

Cluster
Number of running singleuser servers

nasa-cryo
2

nasa-ghg
3

2 of the nasa-cryo user servers have been up for >2days and 1 of the nasa-ghg user serves has been up for 36 hours. I suspect these might be abandoned so maybe they're fine to kick off?

2 of the nasa-cryo user servers have been up for >2days and 1 of the nasa-ghg user serves has been up for 36 hours. I suspect these might be abandoned so maybe they're fine to kick off?

You could do a rolling upgrade without using drain on the node pool they are using, and in the "taint and wait step" you push the changes up to that point and get a PR merged - leaving only a comment saying we need to also delete a node pool, something that hopefully can be done next week at least.

I've done that for nasa-cryo. I was about to do it for nasa-ghg, but the public ssh key isn't in the repo and so eksctl commands failed 😕

Is there a way to reverse this command?
kubectl taint node manual-phaseout:NoSchedule -l alpha.eksctl.io/nodegroup-name=nb-r5-4xlarge

@sgibson91 ah hmmm okay!
Hmmmm, deleting a taint can be done by kubectl edit node <node name>, maybe there is a one-liner kubectl also but I've not found out yet though.

Thanks Erik. I used that command to remove the taint, and have opened https://github.com/2i2c-org/infrastructure/issues/4122

Upgrading the core node group for nasa-ghg was also going to remove node group "nb-c5-4xlarge", which I believe comes from https://github.com/2i2c-org/infrastructure/pull/4100, so I used --exclude="nb-c5-4xlarge" to leave this alone

I've put in a reminder for myself on Tuesday to check if the old node groups have drained (Monday is a holiday in the UK and I'll be coming back from a weekend away)

Hmmmm, deleting a taint can be done by kubectl edit node <node name>, maybe there is a one-liner kubectl also but I've not found out yet though.

@consideRatio I learned a one-liner! Example, to remove the node-role.kubernetes.io/control-plane taint from all nodes (I don't know why you'd want to do this, but it's the example that came up in my course).
kubectl taint nodes --all node-role.kubernetes.io/control-plane-

The key is the extra - at the end which is the syntax for removing a taint

Thank you for sharing this @sgibson91!!


docs content doesn't match what happens on Panel

Article: https://docs.microsoft.com/en-us/azure/azure-functions/functions-proxies
Problem:
The wording In the left pane, select New proxy. doesn't reflect the current version of the panel. The plus button for adding a proxy only shows up if you select "Proxies" after you clicked on your function app.
Also in my case Proxies were in read only mode. See answer here: https://stackoverflow.com/questions/53630773/how-to-disable-read-only-mode-in-azure-function-app

@videlalvaro Please submit this feedback to the article itself. You can do this by using the feedback section at the bottom


how to use random credential

i have 3 token and save them as credentials,names "c1","c2" and "c3", i wanna use them random.
like this:
http://somesite/withtoken/{% credential c? %}.....
thanks for help

There's room for improvement of Huginn/Liquid here; Liquid does not have a random number generator and Huginn's credential tag only takes a bare key name.
For the first matter, I think you could use a millisecond value to get a pseudo random number.  For the second matter, I'm afraid there's no way to obtain a credential value with a dynamically composed key name; you'll have to write a lengthy conditional:
{% assign n = "now" | date: "%L" | modulo: 3 | plus: 1 -%}
http://somesite/withtoken/
{%- case n -%}
{%- when 1 -%}
{%- credential c1 -%}
{%- when 2 -%}
{%- credential c2 -%}
{%- when 3 -%}
{%- credential c3 -%}
{%- endcase -%}
......

@knu thanks a lot for your help.
what's the meaning about symbol '-' near '%' ?
i'm sorry for that i can't write english so well.


Create a model for tap-zuora

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/440
Originally created by @valexieva on 2019-03-01 14:46:44

Problem to solve
(Summarize the problem we are trying to solve in the form of we need to do [A], so that [B] can [C])
To have a full user story, we need to have model files created in order for the user to be able to see and interact with data in the UI
Target audience
(For whom are we doing this? Include a persona)
Us (e2e testing), demo users
Further details
(Include use cases, benefits, and/or goals)
Without model files, a user cannot view the data in the UI and test the charting and reporting functionality
This will also help us understand what is the benefit of these files, how difficult it is for a non-technical user to write them, is this a scalable approach, etc.
Proposal
(How are we going to solve the problem? Try to include the user journey)
@iroussos to create a model file with dimensions, metrics and aggregates as per user story. I have added more data, let me know if you need more.
What does success look like, and how can we measure that?
(Define both the success metrics and acceptance criteria. Note that success metrics indicate the desired business outcomes, while acceptance criteria indicate when the solution is working correctly. If there is no way to measure success, link to an issue that will implement a way to measure this)
A user can run e2e demo with a zuora account
Links / references
Please note that this was taken from GitLab, to be changed accordingly
(cc: @dmor ) this should be a priority for next week, so that the user story for zuora is complete before we continue with the rest of the taps.

View 17 previous comments from the original issue on GitLab


Login Page

Modified MUI base components for Login and Sign Up to create those pages for the app.
Created a Copyright component.
Created a User context.
Updated root layout to include the copyright on all pages.
Removed extra from Profile.
Added a placeholder to link to cart (feature to add)

Reviewing the commit, the Sign Page is missing a few imports.


logger encoder output format (json)

To have a way to change the log output format, making it possible to set the logger to json.
timbre-json-appender

@avelino What if, instead of enabling a single format each time, we just decoupled the appender from moclojer, and just let the user create it from their end.
Specifically:
(defn setup-with-appenders
  [level appenders]
  (global-setup (.getParent (Logger/getGlobal))) ;; disable `org.eclipse.jetty` logs
  (let [config {:min-level level
                :ns-filter {:allow #{"com.moclojer.*"}}
                :appenders appenders}
        sentry-dsn (or (System/getenv "SENTRY_DSN") nil)]
    (timbre/merge-config! config)
    (when sentry-dsn
      (timbre/merge-config! {:appenders {:sentry (sentry/sentry-appender sentry-dsn)}}))))

(defn setup
  "default setup for println appender. Here for no breaking changes."
  [level stream]
  (setup-with-appenders
   level
   {:println (core-appenders/println-appender {:stream stream})}))

where the enduser would be able to call like this:
(setup :auto [["*" :info]]) ;; default (no breaking changes)
(setup :auto [["*" :info]] ;; new one
       {:test (my-obscure-appender)})


Replace hardcoded data primitives in EngineApiTreeHandler impl

Describe the feature
Replace hardcoded data primitives in method signatures in EngineApiTreeHandler impl. Use associated types on generic N: NodePrimitives on EngineApiTreeHandler to replace hard coded types, e.g. replace use of alloy_consensus::Header with N::BlockHeader.
https://github.com/paradigmxyz/reth/blob/c7c84f2d3fc59cbec2288f6a097dc029c9fb0375/crates/engine/tree/src/tree/mod.rs#L544-L2611
Additional context
No response

Can I take this one? @emhane

this is a bit to complex atm for a good first issue so


early support for new expeditions of 2019.07

Includes A4, B2, B3, B4 and 41-44. Subject to change.

Thanks for the kontribution, will take a look during weekends.

LGTM overall. the change to .estlintrc is not necessary (and it should only take one of unix or windows according to https://eslint.org/docs/rules/linebreak-style). But that's just tiny issue I'll fix myself.
I would still investigate a bit on the requirements as I know some sources that has more detail on the requirements (e.g. https://eslint.org/docs/rules/linebreak-style (in Simplified Chinese)). I'll open an issue to followup on that.
I do understand the need of keeping up with game updates, so expect next release no later than next week. (if not feel free to ping me on that xD)

Actually, I didn't read docs of eslint.
Looking forward to new release.


Make withServer provide the actual port

This PR proposes to pass the gRPC listening port to the continuation of withServer. This is useful if you pass port zero to the server, leaving the OS bind any available port, and want to get back the port number.
I have used this several times in tests where I had several servers.

I addressed @ixmatus' comments. PTAL

This LGTM. Please sync with latest awakesecurity:master and then we'll get this in. Thanks!


Router icon

Icon Request

Icon name: router
Use case: iot, internet, device connected to the internet
Screenshots of similar icons:

Nice request, we could sure do with one:

@karsa-mistmere Nice!

Closed by #957


feat(Search): Add text and categories search

This PR adds a text and category search into the sidebar and URL.
It should not be possible to:

[x] Filter items by text
[x] Filter items by category
[x] Combine filters together to find specific items
[x] Get the same results when reloading the page and using the browser history
[x] Clear the text search easily
[x] See the first two categories by default when no parameters exist in the URL
[x] Cache the result once loaded so it isn't requested from the API a second time
[x] Avoid changing the filters while a request to the API is happening
[x] Avoid text searches longer than 100 chars long (This limit can be changed)
[x] Show the Sidebar button only when opened (Now only says "Suche" and doesn't show the number of active filters, as the categories are always active by default)

New texts key added in Grist:

textSearchLabel
textSearchPlaceholder
textSearchCategorySelfHelp
textSearchCategoryAdvising
textSearchCategoryClinics
textSearchCategoryOnlineOffers
textSearchCategoryDisctrictOfficeHelp
optionalFurtherSearchIntroText
filtersButtonLoading

Changes texts key in Grist:

filtersSearchTargetLabel
filterLabel

Todos:

[x] URL Params Parsing
[x] API search
[x] Button count update
[x] Refactor complex functions
[x] Show current search config in list
[x] Show "No results UI" in list
[x] Advanced Text Field UI with confirmed state
[x] Limit and sanitize text seach query

To see the specific tasks where the Asana app for GitHub is being used, see below:

https://app.asana.com/0/0/1205209947864758

@dnsos & @Esshahn The branch is ready for review. There were many changes so please help me to thoroughly test the new feature. Thx! Let me know if you would change or improve anything. Merci!

@dnsos & @Esshahn The branch is ready for review. There were many changes so please help me to thoroughly test the new feature. Thx! Let me know if you would change or improve anything. Merci!

There is currently a bug on mobile devices that lead to a full redender of the components when changing filters. I'm looking into it 👀

@Esshahn & @dnsos The bug has been fixed and everything should work fine now. Ready to review!


Mapping surroundings icon no longer displays

What does it mean when you get the icon in the attached image instead of the usual "trying to map your surroundings icon. The main menu still pops up.

Rebboting the device corrected the issue ,but I'd still like to know what it means

I'm going to close this since it's a Shell issue and not an MRDesignLabs issue. For shell issues you'll have better luck posting on the official Hololens forums.


[Feature Request] Support Shortcuts

alk7mist — 09/15/2024 7:27 PM
@Syd711 | [MAT] Matt, as I've mentioned I've been doing a lot of updates/upgrades/and tabling testing with VPIn Studio which has saved me a lot of time.  How about implementing a feature hotkey for commonly used actions?  When right clicking a table change the Launch option to "Launch (Ctrl-L)"  or whatever letter you prefer?  And an instant launch of the table with one keystroke vs a right click and click Launch?  I think that would be sweet and an additional time saver.  I know of many applications that have this.

Fixed with 3.7.0


How can i change LSTM to BERT ?

I need to use BERT instead of LSTM in this example ..is it possible to just replace the word LSTM with BERT ?
    inputs1=Input(shape=(2048,))

    fe1 = Dense(128, activation='relu')(inputs1)

    inputs2 = Input(shape=(max_length,))

    se1 = Embedding(vocab_size, 256, mask_zero=True)(inputs2)

    se2 = LSTM(256)(se1)

    se3 = Dropout(0.5)(se2)

    decoder1 = Concatenate()([fe1, se3])

    decoder2 = Dense(128, activation='relu')(decoder1)

    outputs = Dense(vocab_size, activation='softmax')(decoder2)

    model=Model(inputs=[inputs1,inputs2],outputs=outputs)

or how can i start to use it please

Just recover
On Dec 11, 2021 4:50 PM, mathshangw @.***> wrote:
I need to use BERT instead of LSTM in this example ..is it possible to just replace the word LSTM with BERT ?
inputs1=Input(shape=(2048,))

fe1 = Dense(128, activation='relu')(inputs1)

inputs2 = Input(shape=(max_length,))

se1 = Embedding(vocab_size, 256, mask_zero=True)(inputs2)

se2 = LSTM(256)(se1)

se3 = Dropout(0.5)(se2)

decoder1 = Concatenate()([fe1, se3])

decoder2 = Dense(128, activation='relu')(decoder1)

outputs = Dense(vocab_size, activation='softmax')(decoder2)

model=Model(inputs=[inputs1,inputs2],outputs=outputs)

or how can i start to use it please
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/google-research/bert/issues/1281, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFQHH7F2J6W7WC2LFRU4U5LUQMNFJANCNFSM5J22LGPQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Just recover On Dec 11, 2021 4:50 PM, mathshangw @.***> wrote: I need to use BERT instead of LSTM in this example ..is it possible to just replace the word LSTM with BERT ? inputs1=Input(shape=(2048,)) fe1 = Dense(128, activation='relu')(inputs1) inputs2 = Input(shape=(max_length,)) se1 = Embedding(vocab_size, 256, mask_zero=True)(inputs2) se2 = LSTM(256)(se1) se3 = Dropout(0.5)(se2) decoder1 = Concatenate()([fe1, se3]) decoder2 = Dense(128, activation='relu')(decoder1) outputs = Dense(vocab_size, activation='softmax')(decoder2) model=Model(inputs=[inputs1,inputs2],outputs=outputs) or how can i start to use it please — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub<#1281>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFQHH7F2J6W7WC2LFRU4U5LUQMNFJANCNFSM5J22LGPQ. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

excuse me what do you mean by "Just recover" ?


Suport for GCS

Question
Is is possible to use Google Cloud Storage for storage?

Hi @aaron276h , at this moment we do not support Google Cloud Storage

thanks for the prompt response @dmthuc


after playing with two recruits, bot will try to collect feedback for both at the same time

added new messageRecruiter function that is promise-based, so a subsequent questionaire will only start once a recruit is completed


[CXF-6986] If no application classes found, install a default applica…

…tion.

Don't merge - discussion only right now.


skytemple: update dependencies to fix build

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

cc @marius851000

Add a :+1: reaction to pull requests you find important.

igraph build log: log.gz

Result of nixpkgs-review pr 286131 run on aarch64-darwin 1

8 packages marked as broken and skipped:

python311Packages.skytemple-dtef
python311Packages.skytemple-dtef.dist
python311Packages.skytemple-files
python311Packages.skytemple-files.dist
python311Packages.skytemple-ssb-debugger
python311Packages.skytemple-ssb-debugger.dist
skytemple
skytemple.dist

4 packages failed to build:

python311Packages.skytemple-rust
python311Packages.skytemple-rust.dist
python312Packages.skytemple-rust
python312Packages.skytemple-rust.dist

4 packages built:

python311Packages.explorerscript
python311Packages.explorerscript.dist
python312Packages.explorerscript
python312Packages.explorerscript.dist


Porting to Plasma 6?

I tested it and it seems it's not working. Are you planning to port the script to Plasma 6?

I use this script every day, so it's going to be fixed. But I don't have plasma6 on opensuse yet.. I didn't expect it to break honestly. I'll try to have a look at what they broke.
If you want to help, you can try to find some relevant logs is journalctl --user -u plasma-kwin_wayland.service -f

Hi. First of all, it seems that systemsettings is not detecting the installed script. I'm nor an expert or a programmer myself but comparing this script with a working Plasma 6 one, it seems that script proprieties are read from a metadata.json file rather than from the metadata.desktop file included here.
I've tried creating the metadata.json myself so systemsettings detects the script and it does so and I'm able to activate it from there.
Once it's enabled it does nothing. Running the journalctl command throws this error so it seems that at least it's trying to initializing it but it's not working yet.
mar 03 18:49:22 fedora kwin_x11[3424]: kwin_scripting: /home/Carlos/.local/share/kwin/scripts/dynamic-workspaces/contents/code/main.js:143: error: Cannot call method 'connect' of undefined
I'll attach too the metadata.json file I hope it helps even if it doesn't make the script to work by itself. I don't think making a PR for this is worthy.
metadata.json

Thanks for the json file, this will be a huge help since I haven't even heard of them changing the format.
I explored the forums this weekend and it seems the KDE guys didn't make new docs yet, neither the migration file. I found the changed apis in kwin headers, and there aren't many, and your error log points to one of those.
In the coming days I plan to set up a VM with kde neon or something and try rewriting.

Wow, plasma6 black-screens in virtual box, both on nixos and kde neon.

Wow, plasma6 black-screens in virtual box, both on nixos and kde neon.

It works on a Xorg session. Plasma 6 has defaulted to Wayland and this combination doesn't work at all in VirtualBox.

Wow, plasma6 black-screens in virtual box, both on nixos and kde neon.

I can recommend libvirt virtualization over virtualbox nowadays, the user experience has matured far enough and supports things such as hardware accelerated gpu virtualization. Using wayland in a qemu vm worked fine last time I used it.
As far as the compatibility with plasma 6 goes, running plasmapkg2 -l in my terminal yields a warning from plasmapkg:
kf.coreaddons.desktopparser: Unable to find service type for service "KWin/Script" listed in "/home/ags/.local/share/kwin/scripts/dynamic_workspaces//metadata.desktop" - service type search directories were ("/home/ags/.local/share", "/home/ags/.local/share/flatpak/exports/share", "/var/lib/flatpak/exports/share", "/usr/local/share", "/usr/share")
kf.coreaddons.desktopparser: Unable to find service type for service "KWin/Script" listed in "/home/ags/.local/share/kwin/scripts/dynamic_workspaces/metadata.desktop" - service type search directories were ("/home/ags/.local/share", "/home/ags/.local/share/flatpak/exports/share", "/var/lib/flatpak/exports/share", "/usr/local/share", "/usr/share")
kf.package: Invalid metadata for package structure "KWin/Script"
Package type "KWin/Script" not found
Listing service types: KWin/Script in /home/ags/.local/share/
kf.coreaddons.desktopparser: Unable to find service type for service "KWin/Script" listed in "/home/ags/.local/share/kwin/scripts/dynamic_workspaces/metadata.desktop" - service type search directories were ("/home/ags/.local/share", "/home/ags/.local/share/flatpak/exports/share", "/var/lib/flatpak/exports/share", "/usr/local/share", "/usr/share")

Thanks, I should try libvirt sometime. I only use vbox because I've used it for a long time and know it's interface like my five fingers.
I managed to run plasma6 on x11 on nixos yesterday, and started working on it. There have been some pretty big changes to how desktops work now, and I found a funny bug in pager.
@0x5a17ed plasmapkg2 was deleted though? It's not in nixos's plasma. The forums tell me to use kpackagetool6 now, and it works with metadata.json file that monsieur CarlosEkisde gave

@maurges Let me know if i can help somehow.
I updated to plasma 6 on arch yesterday and i am missing dynamic workspace management.

I made a straightforward rewrite and put it in this branch: https://github.com/maurges/dynamic_workspaces/tree/plasma6 - it seems to work on x11, and if any of you can verify it works on wayland, I would really appreciate it! Updated installation instuctions with kpackagetool6 are given in the readme.
Fun fact: you can now crash plasma from a kwin script.
I'm now going to make a version that works with both plasma 5 and 6.

I made a straightforward rewrite and put it in this branch: https://github.com/maurges/dynamic_workspaces/tree/plasma6 - it seems to work on x11, and if any of you can verify it works on wayland, I would really appreciate it! Updated installation instuctions with kpackagetool6 are given in the readme.
Fun fact: you can now crash plasma from a kwin script.
I'm now going to make a version that works with both plasma 5 and 6.

On install i got this:
qt.dbus.integration: QDBusConnection: error: could not send signal to service "" path "/KPackage/" interface "org.kde.plasma.kpackage" member "packageInstalled": Invalid object path: /KPackage/

I made a straightforward rewrite and put it in this branch: https://github.com/maurges/dynamic_workspaces/tree/plasma6 - it seems to work on x11, and if any of you can verify it works on wayland, I would really appreciate it! Updated installation instuctions with kpackagetool6 are given in the readme.
Fun fact: you can now crash plasma from a kwin script.
I'm now going to make a version that works with both plasma 5 and 6.

My wayland session is now dead, black screen and locked mouse pointer, only power button performs a shutdown. Even uninstalling (and deleting ~/.local/share/kwin) didn't fix it.

In my wayland session nothing happens. All works fine besides of the script which seems not to be active...

I confirm it works in x11 too. Thank you =D

Works great in Wayland here too. I'll keep driving it and report any bugs!

I've found and killed the culprit (it was not the script, ofc). It works flawlessly AFAICT. Thanks!

Works good here! One small issue I noticed: When switching away from an empty workspace, it does so happen immediately without the usual animation. I guess the workspace is removed instantly and that prevents Kwin from starting/finishing the transition(?)
Operating System: Arch Linux
KDE Plasma Version: 6.0.1
KDE Frameworks Version: 6.0.0
Qt Version: 6.6.2
Kernel Version: 6.7.9-arch1-1 (64-bit)
Graphics Platform: Wayland

@r0skar I see what you mean. I ported the changes I made to plasma 5, and I get the missing animation with wayland, but not x11. Yeah it sucks, I'll try to fix it.

The plasma6 branch seems to work for me on Arch with Wayland as well!
One minor thing though, when it creates a new workspace, that workspace is named "dyndesk". Ideally, it should follow KDE's default of Desktop %d and if I'm remembering right, that's what it did in Plasma 5.
Anyway, thanks for the quick solution!

Yet another confirmation, works as expected on Wayland, Plasma 6.0.1. dyndesk workspace name does appear frustratingly. Thank you so much for making this work and making it possible!

Changing dyndesk  to default isn't trivial, as it inherits from i18n, and some users could have them edited to Main, or First/second/...

Anyway I tested an intermediate solution and created a PR

dyndesk

This used to be easier in plasma 5, where it just picked a name for me. Maybe a similar api call exists in 6 and I just haven't found it yet.
I think a solution like "dyndesk $i" is good enough without internationalization.

I have a question, is there a repository where this port is being handled?
I'd like to follow along to learn a little more about kwin-script development.

@rodrigopmatias Just for you, I pushed a local branch where I did all of the work: https://github.com/maurges/dynamic_workspaces/tree/rewrite - be careful as there will be force-pushes

This used to be easier in plasma 5, where it just picked a name for me. Maybe a similar api call exists in 6 and I just haven't found it yet.
I think a solution like "dyndesk $i" is good enough without internationalization.

Good news, workspace.createDesktop(workspace.desktops.length, undefined); will accomplish this behavior. I've tested it on Plamsa 6 Wayland; it seems to behave as documented.
Documentation states name The name for the new desktop, if empty the default name will be used.
See https://develop.kde.org/docs/plasma/kwin/api/#functions-1.

Documentation states

The docs are still for plasma 5 so not completely trustworthy, I've been rather looking at kwin source. But it's good that you verified it, I'll use it then.
I've been kind of busy with life for the last month, so no activity in this repo. But I hope to release a version with plasma 6 support in the store in the coming weeks.

It's been very nice to use in the mean time 🙂
Thanks so much for writing this!


Controlling automatic session time out

Hello Volkan,
My users are complaining that the application is automatically logging them out (laziness, I know). Is setting the application pool idle time out the correct way to prevent automatic logout?

Many thanks in advance.

Nope, increase timeout value in web.config

Like this? Thanks.

Is this what I need to change? If I set timeout to 300, does it mean it will stay up for 5 hrs?
 <forms loginUrl="~/Account/Login" timeout="300" path="/" slidingExpiration="true" />

Yes, that is it


Fix S3 region when not value set in instance

What changes are proposed in this pull request?
Please outline the changes and how this PR fixes the issue.
Why are the changes needed?
Please clarify why the changes are needed. For instance,

If you propose a new API, clarify the use case for a new API.
If you fix a bug, describe the bug.

Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including

change in user-facing APIs
addition or removal of property keys
webui

i think if we initialize the s3 client with AmazonEC2ClientBuilder.standard() then we won't have this issue. https://docs.aws.amazon.com/sdk-for-java/v1/developer-guide/java-dg-region-selection.html

Codecov Report

Merging #14481 (975d115) into master (83d7b63) will decrease coverage by 24.89%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #14481       +/-   ##
=============================================
- Coverage     41.86%   16.97%   -24.90%
+ Complexity     9329     2708     -6621
=============================================
  Files          1495     1495
  Lines         87512    87522       +10
  Branches      10428    10431        +3
=============================================
- Hits          36640    14854    -21786
- Misses        47890    71430    +23540
+ Partials       2982     1238     -1744

Impacted Files
Coverage Δ

...ommon/src/main/java/alluxio/metrics/MetricKey.java
97.57% <ø> (-0.38%)
:arrow_down:

...n/java/alluxio/stress/cli/RegisterWorkerBench.java
0.00% <0.00%> (ø)

.../alluxio/stress/cli/StreamRegisterWorkerBench.java
0.00% <0.00%> (ø)

...java/alluxio/stress/cli/StressJobServiceBench.java
0.00% <0.00%> (ø)

...ain/java/alluxio/stress/cli/StressMasterBench.java
0.00% <0.00%> (ø)

...n/java/alluxio/underfs/s3a/S3AUnderFileSystem.java
0.00% <0.00%> (-17.50%)
:arrow_down:

...mon/src/main/java/alluxio/shell/CommandReturn.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...mon/src/main/java/alluxio/util/ExceptionUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/src/main/java/alluxio/wire/AlluxioProxyInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/main/java/alluxio/wire/AlluxioMasterInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 689 more

Continue to review full report at Codecov.

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

@yuzhu I tested on my side that this fix is sufficient, the standard may also works, can you help test on your side as well?

I didn't see standard() modify the s3 region information, looks like it's similar to what we do now
  public static AmazonS3ClientBuilder standard() {
    return (AmazonS3ClientBuilder)(new AmazonS3ClientBuilder()).withCredentials(new S3CredentialsProviderChain());
  }

    AWSCredentialsProvider credentials = createAwsCredentialsProvider(conf);
  public static AWSCredentialsProvider createAwsCredentialsProvider(
      UnderFileSystemConfiguration conf) {
    // Set the aws credential system properties based on Alluxio properties, if they are set;
    // otherwise, use the default credential provider.
    if (conf.isSet(PropertyKey.S3A_ACCESS_KEY)
        && conf.isSet(PropertyKey.S3A_SECRET_KEY)) {
      return new AWSStaticCredentialsProvider(new BasicAWSCredentials(
          conf.get(PropertyKey.S3A_ACCESS_KEY), conf.get(PropertyKey.S3A_SECRET_KEY)));
    }
    // Checks, in order, env variables, system properties, profile file, and instance profile.
    return new DefaultAWSCredentialsProviderChain();
  }
    AmazonS3ClientBuilder clientBuilder = AmazonS3Client.builder()
        .withCredentials(credentials)

Tested with s3 access key and secret key in ~/.aws/credentials
s3 bucket in us-east-1, us-west-1, Singapore
bin/alluxio runTests
all passed

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.7 please


295 - Clean up docker-compose.env

Issue Number
Closing: #295
Purpose/Implementation Notes
Adjusted the compose script and the dockier-componse.yml file, and deleted the dockier-componse.env file.
🗒  Since the sentry env var was removed, I've deleted the dockier-componse.env file which was solely used to set the env vars for containers' ports (also to avoid the need for additional env vars setup for the production and staging environments for those ports).
Types of changes
What types of changes does your code introduce?

Breaking change (fix or feature that would cause existing functionality to not work as expected)

Functional tests
Ran all the scripts locally and successfully started/removed the containers for client and storybook.
Checklist

[x] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Hey, so we actually want to keep the docker-compose.yml file. We just want it to contain default values where appropriate and remove it from being tracked from source control.
The docker-compose.yml file should look like this:
# Environment variables for docker-compose.yml
CLIENT_PORT=3002
STORYBOOK_PORT=6006
STAGE_SENTRY_DSN=''
STAGE_SENTRY_ENV=local-refinebio-client

Then we add it to .gitignore and we can populate it with sensitive values and they wont get tracked in version control.
Let me know if you have any questions.

I see. The "docker-componse.env" file is required to compose Docker containers in all environments ("dev" in localhost, "staging" and "production" in Vercel preview etc). So if you add it to .gitignore, we'll probably need some additional setup (the scpca-portal repo is committing it). Let me know how you want to go about it.

I see. The "docker-componse.env" file is required to compose Docker containers in all environments (dev in localhost, staging and production in Vercel preview etc). So if we are going to add it to .gitignore, we'll probably need some additional setup (the scpca-portal repo is committing it). Let me know how you want to go about it.

I don't think this will be a problem as we aren't using docker outside of local development.
Any env var in a deployed environment would be populated by vercel. It passes the env vars that are set in the admin panel to our build step which then get passed into our application context via next.config.js.
We should be good to make this change.

What if other developers want to run dev (locahost) for contributions? For example, in thescpca-portal repo, the docker-componse.env (includes the ports information and sentry) is currently committed, so that could be easily achieved and any errors can be reported to us promptly.

Yeah I will be removing that from there as well.
Some points here:

We dont really want to collect any contributors errors in sentry as that information isn't actionable.
We really only care about errors in prod / staging but we need local support to know everything is wired up correctly prior to deployments.
Sentry not being configured correctly doesn't break the site. External contributors ought to be able to clone the repo and run the project just fine.
If they want to collect their own errors, they can populate the empty SENTRY_DSN env var with their own DSN.
Any active SENTRY_DSN should be kept in a secret / env vars to help mitigate spam. Though this hasn't been an issue to date.

Just to clarify, the file will be committed but untracked which means when you populate values and do git diff docker-compose.env wont show up. But the file will exist and ready for use with docker compose.

In terms of security, since we are using the refinebio API, I recommended to monitor the error activity on the local dev.

Just to clarify, the file will be committed but untracked which means when you populate values and do git diff in the project root and docker-compose.env wont show up. But the file will exist and ready for use with docker compose.

As for the untracked files, I was talking about a cloned/forked repository codebase for starting docker containers (with the required ports). So, in this case, we'll need to update the README and add instructions for port setup for contributors.

In terms of security, since we are using the refinebio API, I recommended to monitor the error activity on the local dev so that we'll be able to properly identity or prevent any risks or faults.

Please create an issue on refinebio-admin if you would like to provide more detail about how this would work or benefit us.

As for the untracked files, I was talking about a cloned/forked repository codebase for starting docker containers (with the required ports). So, in this case, we'll need to update the README and add instructions for port setup for contributors.

Please see point number 3 in my earlier comment about this.
Yeah I agree we can add a blurb to the README about populating sentry credentials.
Just to reiterate and clarify, before docker-compose.env is untracked (not deleted or removed from the project). It should contain all of the env var names with values, sensitive information should be omitted and replaced with just an empty string. Specifically, in this PR it should just be SENTRY_DSN that has it's value replace with an empty string.

Based on the chat w/ @davidsmejia, the following changes will be made and this workflow should be documented somewhere (e.g., egg):

No longer reporting dev (localhost) errors to Sentry but staging and prod

Except, if prefer to track errors in dev for testing etc, internal developers can add the sentry DSN value (stored in our password manager) to the docker-compose.env file to their local repository (* it should never committed).

Adding the docker-componse.env file to .gitignore so that it won't be tracked (treat it as a template)

Except, if new values need to be added to this file, do the following:

remove it from .gitignore to track the change
commit the new change to the remote repository
add it back to .gitignore to untrack again

The docker-compose.env file:
# Environment variables for docker-compose.yml
CLIENT_PORT=3002 // constant unless changed
STORYBOOK_PORT=6006 // constant unless changed
STAGE_SENTRY_DSN='' // always empty value
STAGE_SENTRY_ENV=local-refinebio-client // constant unless changed

🗒  This issue is still in progress.

@davidsmejia I applied the feedback and updated  docker-componse.env with new required values (d910e5619c5e3aea23c8db025247f840b4b74db3).
This file needs to be commuted to dev before adding it to .gitignore in order to use it as a template file (https://github.com/AlexsLemonade/refinebio-web/pull/296#issuecomment-1821804612), since we need this file to start the project for development.
Please take a look.

This looks good, the only thing missing would be the STAGE_API_HOST and STAGE_API_VERSION which should just point to staging and be v1.

the STAGE_API_HOST and STAGE_API_VERSION which should just point to staging and be v1.

@davidsmejia I've added those env vars and adjusted the fetchAsync helper. PR is ready for another look!

If you have had this repo cloned before this file was ignored you can run:
git update-index --skip-worktree docker-compose.env to have git skip tracking changes to this file.
if you need to undo that you can use the same command but with --no-skip-worktree to make edits.


bibtex and arXiv

Originally on 2010-09-01
In export of arXiv records in BibTeX format we should follow  the recommendation on the arxiv.org

site:
"For the new style arXiv identifiers (April 2007 and onwards) we
recommend
these bib-style extensions:
 archivePrefix = "arXiv",
 eprint        = "0707.3168",
 primaryClass  = "hep-th",

"

Currently we export arXiv in front of eprint, as well as in prefix.   It should be stripped from eprint.

This was actually implemented in the INSPIRE overlay https://github.com/inspirehep/inspire/blob/master/bibformat/format_elements/bfe_INSPIRE_bibtex.py#L272-L287


Identicial byte array representation of replicated events

Replicated events must have the same byte array representation at all locations. This is currently not ensured as target locations use their own serializers to write replicated events to their event logs. Given this additional guarantee, application upgrade (#107) can be made without having temporary unidirectional replication to the location being upgraded. This ticket also covers #89.

Having identical byte array representation of events at all locations was supposed to avoid permanent state divergence during incremental distributed application upgrade (#107). This, however, won't work for applications that use persistent views, for example. They'd need to recreate these views to correct for divergent persistent state that could have occurred during the incremental update window. This is considered to be a too high administrative burden, hence this ticket is closed and replaced by #218.


Format HasData in snapshot to multiple lines

b.HasData(
    new
    {
        Id = new Guid("c6a32cba-c6ea-4597-955b-2ab76025dc7d"),
        ConcurrencyStamp = "e2115b27-76d3-432e-bac4-541c2d96e5bd",
        LockoutEnd = new DateTimeOffset(new DateTime(9999, 12, 31, 23, 59, 59, 999, DateTimeKind.Unspecified), new TimeSpan(0, 0, 0, 0, 0))
    });

Instead of
b.HasData(
    new { Id = new Guid("c6a32cba-c6ea-4597-955b-2ab76025dc7d"), ConcurrencyStamp = "e2115b27-76d3-432e-bac4-541c2d96e5bd", LockoutEnd = new DateTimeOffset(new DateTime(9999, 12, 31, 23, 59, 59, 999, DateTimeKind.Unspecified), new TimeSpan(0, 0, 0, 0, 0)) }
    );

@bricelam I can give this a shot if you want, let me know!

Awesome, go for it!

@bricelam @AndriySvyryd Which of the following outputs are correct?
Output 1:
b.HasData(
    new { Id = 42 },
    new
    {
        Id = 43,
        OptionalProperty = 4.3m
    });

Output 2:
b.HasData(
    new
    {
        Id = 42
    },
    new
    {
        Id = 43,
        OptionalProperty = 4.3m
    });

Our main goal is to make that file diff well for when there's a merge conflict. Given that, I'd say option 2.