EPIC: Reward function for grasping environment

From literature:
Sparse
Simple sparse reward

https://arxiv.org/pdf/1802.10264.pdf

+1 for successful grasp

https://ieeexplore-ieee-org.zorac.aub.aau.dk/stamp/stamp.jsp?tp=&arnumber=8460553

+1 for successful grasp

Sparse reward with extra guidance

https://arxiv.org/pdf/2007.04499.pdf

+10 for successful grasp (gripper holding object above certain height)
+1 (partial reward) if gripper comes in contact with the object
-1 if no action is taken
-0.025 for all time steps prior to termination

https://arxiv.org/pdf/1806.10293.pdf

+1 if the gripper contains an object and is above a certain height
−0.05 for all steps prior to termination (max 20 steps, but agent is allowed to terminate earlier)
At the beginning, this paper provides demonstrations of grasps from other algorithms providing 15-30% success rate. Then they use only the learned policy + exploration once 50% success it reached.

Dense
These don't seem to be that common in literate and those that are present are usually quite complex.

https://arxiv.org/pdf/1910.09470.pdf (very complex reward)

Reach reward (upon being close to the object, e.g. 0.01 m) +[0.0, 0.01]
Touch reward +1 upon contact with the object
Lift reward +1 (when above certain theshold
Task-specific reward (stacking objects)
All rewards are summed (specific function is used)

Other dense reward formulations

Increase in object's height (Final goal)

Reward could increase linearly as actor lifts object relative to their original position. This could be further restricted that there must be a contact between robot fingers and the object (such that actor holds the object rather than throwing it).

Touching an object (Helping step)

A small reward could be added for each time step robot touches an object with both fingers.

Guiding the agent
Demonstration of success
Add samples of success to the replay buffer at the beginning of training. These could be either from manual demonstration or by using another algorithm (could also be pre-planned).
Curriculum learning
Slowly increase the overall difficulty of the task, e.g. start from one object with restricted operating area.

https://arxiv.org/abs/2003.04960

The initial reward function was added in https://github.com/AndrejOrsula/drl_grasping/pull/54/commits/620207a010ed62e76a6467aaa4ae89fe58b2018e, with the possibility to select sparse or shaped rewards.
Shaped rewards consist of many aspects (distance to closest object, grasping of object, lifting the object, lifting above certain height)
Sparse reward gets triggered only when lifted above certain height.

I will leave this EPIC open for now as it is still relevant and many aspects must be tuned continuously.

Related epic: Curriculum learning was added in https://github.com/AndrejOrsula/drl_grasping/issues/62


Autoplay other direction (rtl instead of ltr)

Feature request is related to

[ ] embla-carousel (core package)
[ ] embla-carousel-react
[ ] embla-carousel-vue
[ ] embla-carousel-svelte
[x] embla-carousel-autoplay
[ ] embla-carousel-auto-height
[ ] embla-carousel-class-names
[ ] embla-carousel-docs (documentation website)

Is your feature request related to an issue?
In the documentation it is possible to set the direction of a slider to a different setting (ltr or rtl), but it seems not working if you use the autoslide plugin.
Describe the solution you'd like
An extra option for the autoplay plugin to set the direction the other way (rtl instead of ltr)

Hi @mkruip05,
I just tested the autoplay plugin with rtl and it's working just fine. Just follow these steps:

Fork the Autoplay CodeSandbox.
Follow these instructions in the docs.

That's enough to get it working.
Best,
David

Please read the contribution guidelines. It seems like you've missed quite a lot about how to write your report in order to get proper help. From the guidelines:

Do not: Provide a link to a production site. That's not a test case.

@davidjerleke sadly doesn't work. I will try to find another workaround. Thanks for your help.

Ok so that’s where the confusion comes from. You wrote ltr and rtl. These abbreviations are almost always used to specify the content direction in web development.
What you want is to be able to set the autoplay to move in the opposite direction. It should advance to the previous snap instead of next.
This is not possible with the current state of things.
I guess you missed this:

#321

Please read that. Thank you for understanding.
Best,
David


Generate aocx for my board

Hey guys.
I have the de1soc board and i like compile this CNN in my board.
For this, i need generate the aocx with my makefile.
How can I do this?

@Er1cZ it would be possible to provide 1 contact from you, I would love to speak with you.


make sure we deploy laravel4 with the deploy button

hi, we kept that commit in a different branch on purpose. do you have any issue deploying laravel 4?


CIV-11540 Updated WA in Claimant Response Journey

Add feature toggle before merging
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CIV-11540
Change description
Work Allocation in Claimant Response Journey
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ x] No

Created new PR for this https://github.com/hmcts/civil-wa-task-configuration/pull/488


Add completions using new Completion architecture

New completion architecture that uses posener/complete was added in
https://github.com/redhat-developer/odo/pull/893
It is nicely documented  https://github.com/redhat-developer/odo/blob/master/docs/completion-architecture.md
Currently it works only for odo service create.
TODO:

Suggesting for application name odo app delete, odo app describe, odo app, odo app set + every command that has --app flag
Suggesting for URL name odo url delete
Suggesting component names odo delete, odo component, odo link, odo push, odo watch, + every command that has `--compnent`` flag
Suggesting project names odo project delete, + every command that has --project flag
Suggesting storage names odo storage delete, odo storage mount, odo storage unmount
(I might have forget something)

It also works for odo service delete actually. And there's a bug I need to fix because right now the commands register their associated completion handlers using their names resulting in collisions.

@metacosm Is it possible to start working on adding completion for other commands or it will create too many collisions with your work for #871?

@kadel I think that it should be OK with this new architecture because the files more or less now follow what should happen after #871.

@kadel I would like to pick this up

Also started working on #903.

@mik-dass OK, lets do it one entity at a time. Pick one of the entities:  component, app, project, storage, url and make sure that it suggesting works for it.

@mik-dass let me know if you need help and/or you think the doc could be improved.

@kadel I would pick component for the start since now component names are very big and pain to delete :) @metacosm Thanks :+1:

@mik-dass note that #904 has changed some things, most notably the fact that you now need to register completion handlers after the command has been inserted in the command hierarchy, something to remember as if you don't do that, the app will exit on registration once #904 is merged.

@metacosm Looks like #904 is in. Will have a look at the new changes. Thanks :+1:

@kadel as previously we had odo utils completion bash for completion and the process to install bash completion has completelly changed (see https://github.com/redhat-developer/odo/blob/master/docs/completion-architecture.md#enabling--disabling-completion), we need to properly document this in the main help (currently in help it says --complete while in the linked doc says `-complete, but both works) for odo as well as in the main README.md. Completion is one of the most important features for productivity.
Also, is the install process odo -complete the final process to install completion? It doesn't look intuitive.

Also, is the install process odo -complete the final process to install completion? It doesn't look intuitive.

Yep, I don't like it either. I would prefer to move it back to odo utils, we already have the  PS1 (odo utils terminal), those two should be together

https://github.com/openshift/odo/issues/1171 has been closed (all tasks completed).
We can close this issue now.


Streetlevel imagery Mapillary, Bing Streetview, and Kartaview failing to load

URL
https://www.openstreetmap.org/edit#map=20/39.54776/-81.64010
How to reproduce the issue?
In my specific case, the imagery fails to load for Facebook since I block Facebook subdomains (thus Mapillary's CDN since its owned and served by Facebook) and am not operating iD in the Facebook container where it would have been permitted, KartaView was returning 404 for all imagery selected, and Bing Streetside throws some errors.
To reproduce:

Use Firefox
Set permission to strict security
https by default enabled for all pages

I tested with both enhanced tracking protection and privacy badger enabled and disabled.
See different issues depending on street level imagery selected. Console output attached. Screenshots attached as well as console output.
Console output attached:

Seeing mime type warning maybe iD?,
blocked (Facebook as expected) but could be handled better by disabling the option to select if the domain is blocked ie test and disable option to use if test fails,
404 for Kartaview (possibly their end issue).

This is an example of a technical issue that could present and a street level imagery tag would be applied in a changeset as noted in issue #9549.
Screenshot(s) or anything else?

console-export-2023-3-17_14-4-10.txt
Which deployed environments do you see the issue in?
Released version at openstreetmap.org/edit
What version numbers does this issue effect?
2.25.1
Which browsers are you seeing this problem on?
Firefox

See also:

#9230

There's multiple issues mentioned so it's unclear what you're reporting, but based on the title, having Mapillary fail to load since you're blocking it is expected.

having Mapillary fail to load since you're blocking it is expected.

How is "Set permission to strict security" choosing to block mapillary?
After some testing, the root cause seem to be strict security mode blocking facebook's CDN in the case of mapillary integration. So it's a legitimate upstream(firefox) issue. Which hopefully is valid to track here.
I reported it using the built in function when disabling "enhanced tracking protection" but that might not be enough and in any case doesn't give visibility for us here and possibility to communicate with upstream.
So if someone know the precise right place to report and has the time, that that would be the next step for this issue.


Style leftbar with bootstrap

At present, the leftbar is styled by custom classes, remove these custom classes and add Bootstrap classes there.

I would like to do this. Can u assign it to me?

@Aditya-ds-1806 go ahead.

Hey I made a PR please review.


Localized Agenda Header Text

Firstly, this is a really simple to use component for calendars. I need to make the calendar localized in Danish. However, a problem I have is that I cannot seem to replace the headers in the agenda view.
The localizations will localize everything except the headings, which seem to always be 'Date', 'Time' and 'Event'.
Please advise if this is doable. In the short run, this is fine but over the long term, we need to be able to change the text in the headers for agenda view.
Thanks.

Hi there. You can change the text via the messages prop it accepts a hash of string for replacing the various bits of text throughout the calendar with your localized messages.


question: how to install new grammar?

how to add new grammar that isn't listed in the menu?
where should i put grammar.js and the parser files?

Some of the info you're looking for is here in the README: https://github.com/sublime-treesitter/TreeSitter?tab=readme-ov-file#manage-your-own-language-repos-and-binaries
To manage your own languages, you need to do a few things in the STS settings. Search for Preferences: TreeSitter Settings in the command palette

Set python_path to an external Python 3.8 executable with a working C compiler. This way STS can compile its own TS language binaries. The Python that ships with Sublime Text can't compile TS languages
If the language you want to install isn't in this list here, add language_name_to_repo to your settings, and also add language_name_to_scopes

These settings dicts are merged with the dicts in src/utils.py

STS automatically downloads the language from its GitHub repo, builds the language binary, and uses this language for scopes defined in language_name_to_scopes

You don't have to manually add grammar.js files
To see where languages are installed on your machine, restart ST and open the ST console (View > Show Console); STS logs this location on startup

Here's an example of how to do this for e.g. dart
Your settings would look something like this. Then you could run TreeSitter: Install Language and install dart. STS would download the language repo, build the language binary, instantiate the language, and use it to parse code in any buffer with a scope of source.dart
{
  "python_path": "~/.pyenv/versions/3.8.13/bin/python",
  "installed_languages": [
    "python",
    "json",
  ],
  // Markdown parser slow for some reason
  "language_name_to_repo": {
    "dart": {"repo": "UserNobody14/tree-sitter-dart"},
  },
  "language_name_to_scopes": {
    "dart": [
      "source.dart",
    ],
  },
}


How to delete an image from the registry?

Is it possible to delete images from the registry?
I tried using skopeo delete to delete the tagged image from the registry but it only deleted the manifests, the blobs are still there even after the registry manifest has been deleted.
Is there any script to delete the blobs or are the blobs supposed to just stay there? ML apps' blobs can be in several GBs so I don't understand how keeping the blobs after deleting the manifests is a good idea! I saw this #PR but it hasn't been merged. If you do end up merging this pr, can you please give an example on how to use this endpoint to delete the layers?

Right now it's not possible to remove layers from the registry unless by building custom tooling, it's something we still need to look for a bit deeper.
The biggest problem here is that the safest way to perform GC with stop the world lock is not possible without heavy changes. There is obviously other ways to go about GC but not without drawbacks. I will be looking into the PR as it's a very key feature for a registry, but we need to make sure it prevents as much as possible data-races and unexpected data-loss.

I took a look today and I think this will help you! There is some docs on how to use the garbage collector.
https://github.com/cloudflare/serverless-registry/pull/48
Please let me know if you face any issues.

I have merged the garbage collection PR. I will close. Please re-open if you encounter any issues!


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v4.4.276",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=b5a2799cd62ed30c81b22c23028d9ee374e2138c"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "b5a2799cd62ed30c81b22c23028d9ee374e2138c",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "tty: serial: 8250: serial_cs: Fix a memory leak in error handling path\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.4.276 by commit b5a2799cd62ed30c81b22c23028d9ee374e2138c. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1001481


Ask the user whether a custom editor should be made the default

Was having a discussion with @kieferrm and we were thinking the extension should not say it should be the default, but rather indicate that it makes sense and would like to be the default. That way VS Code can ask the user when one of these default extensions is installed when opening a supported file whether it would like to be the default (eg. Luna Paint), but not for "secondary" custom editors where that doesn't make as much sense (eg. Hex Editor).
This will help if we recommend installing Luna Paint from the image viewer (https://github.com/microsoft/vscode/issues/116120) as the default wouldn't be changed out from under the user.

I'm not sure how we would know this is your first time opening a specific file type after a specific custom editor is installed. I think the easier flow would be something like themes which when installed shows you the current theme and gives you the option to switch your default.

@lramos15 the extensions list a set of extensions where it would make sense to be a default for. We could just keep a map in the storage service of the response the user gave to the question? eg.
answers['lunaPaint'] = {
  ".ico": true,
  ".bmp": false
}

Whether we ask on file open or extension install should probably be brought to the UX sync.

Closing for now as I've added a notification to prompt the user when there are two conflicting defaults


/rootCollections return non-collection objects

This endpoint appears to be returning non-collection objects

Should not send this to SBOLExplorer
Also should not send manage, shared, subCollections,


README.md: Drop link to OCI scope table

The OCI scope table doesn't exist any more so drop the link. Similar to https://github.com/opencontainers/image-spec/pull/824

+1 non binding
On Mon, Nov 20, 2023 at 12:05 PM Mike Brown @.***>
wrote:

@.**** requested changes on this pull request.
While there is no scope table and the charter has been "simplified", this
specification is still directed by the chart to adhere to the charter
values..
https://github.com/opencontainers/tob/blob/main/CHARTER.md#7-oci-values
Suggest adding "- OCI Values
https://github.com/opencontainers/tob/blob/main/CHARTER.md#7-oci-values
to the how this group operates list, located just above the removed text.
—
Reply to this email directly, view it on GitHub
https://github.com/opencontainers/distribution-spec/pull/493#pullrequestreview-1740306377,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPSINDUAXZHCCA7G7NEJDYFOEUHAVCNFSM6AAAAAA7TEABPGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONBQGMYDMMZXG4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
com>

--
Cheers,
Chris Aniszczyk
https://aniszczyk.org


Use new library for CLI arguments

Use docopts library instead of argparse -- it seems easier to use

Ok, will use only for new programs. Will not transform existing programs into this format.


fix(connectors): fix https://issues.redhat.com/browse/MGDCTRS-834

Description
The problem was happening specifically in case of expired namespaces that had connectors previously assigned.
The namespace expires and its status is set to deleting, the control plane delete the namespace and the status is updated to deleted. At this point  the namespace reconcile loop should run and correctly delete the namespace, unfortunately what happens inside the reconcile loop is that the namespace is re-detected as expired and placed back in deleting state before actually trying to delete it. In a never ending loop.
This PR fix this behavior.
See https://issues.redhat.com/browse/MGDCTRS-834
Verification Steps

OCM_ENV=integration make test/integration/connector

Checklist (Definition of Done)

[x] All acceptance criteria specified in JIRA have been completed
[x] Unit and integration tests added that prove the fix is effective or the feature works (tested against emulated and non-emulated OCM environment)
[x] Documentation added for the feature
[x] CI and all relevant tests are passing
[ ] Code Review completed
[ ] Verified independently by reviewer
[x] Required metrics/dashboards/alerts have been added (or PR created).
[x] Required Standard Operating Procedure (SOP) is added.
[x] JIRA has created for changes required on the client side

Can you resolve conflicts?


PDF file not rendering properly

Link to PDF file (or attach file here):
New.pdf
Configuration:

Web browser and its version:Chrome Version 50.0.2661.94 (64-bit),
also Safari Version 9.0.1 (116<IP_ADDRESS>)
Operating system and its version: Mac OS X v10.11.1
PDF.js version: pdfjs-1.4.20-dist, also tried pdfjs-1.5.188-dist
Is an extension: No

Steps to reproduce the problem:
Execute in dev tools console in Chrome:
PDFJS.disableWorker = true;
PDFJS.getDocument('<url to document>').then(function(pdf){
  pdf.getPage(1).then(function(page){
    viewport = page.getViewport(1);
    canvas = document.getElementById('render-doc-here');
    context = canvas.getContext('2d');
    canvas.width = viewport.width;
    renderContext = {canvasContext: context, viewport: viewport};
    page.render(renderContext);
  })
});

What is the expected behavior?

What went wrong?

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension): not hosted

Did you try to setup proper PDFJS.cMapUrl and PDFJS.cMapPacked ? Some documents require external CMaps to be loaded. It will be easier to tell if the complete example was published online (code snippet is not helping in your case). Can you publish that for us?

I made a standalone example and the page renders properly in that.  So it must be some interaction between PDFJS and the rest of my app which is a rails served Ember.js app.
Is there some documentation about CMaps? I don't see anything about that in the examples.

Is there some documentation about CMaps? I don't see anything about that in the examples.

See e.g. https://github.com/mozilla/pdf.js/blob/master/examples/components/simpleviewer.js

So it must be some interaction between PDFJS and the rest of my app which is a rails served Ember.js app

That is the important part of this issue. If you could provide minimal example online, we could see what you are doing wrong (plus probably created an example for future PDF.js Ember users with similar problem).

Incidently, it renders MANY other PDFs just fine, so there is something different about this pdf.
I'll think about how to reproduce in a minimal example.

here is a minimal example:
add pdf.js and pdf.worker.js, and the above attached New.pdf to the web directory of the pdf.js repo, and then add this file:
test.html:
<!DOCTYPE html>
<html>
  <head>
    <script src='pdf.js'></script>
    <script src='pdf.worker.js'></script>
    <script src="//cdnjs.cloudflare.com/ajax/libs/ember.js/2.4.3/ember.min.js" ></script>
    <script>
      PDFJS.disableWorker = true;
      PDFJS.getDocument('http://localhost:8888/web/New.pdf').then(function(pdf){
        pdf.getPage(1).then(function(page){
          viewport = page.getViewport(1);
          canvas = document.getElementById('render-doc-here');
          context = canvas.getContext('2d');
          canvas.width = viewport.width;
          canvas.height = viewport.height;
          renderContext = {canvasContext: context, viewport: viewport};
          page.render(renderContext);
        })
      });
    </script>
  </head>
  <body style="font-size:12px;">
    <canvas id='render-doc-here'></canvas>
  </body>
</html>

Then run gulp server, and load the page _http://localhost:8888/web/test.html_
when I load that page I get a display with only the two lines of text displayed above instead of a whole page of text. If I comment out the disableWorker = true line, it loads properly.  However in my real project that does not fix the problem. Eliminating the load of ember.min.js also makes it load properly.

I seems that the problem is with Ember, which is adding an enumerable _super property on Arrays. Hence this seem to be the same underlying problem as in #7064, please refer to the discussion there.
This is an issue with Ember, and not PDF.js, hence I'm closing this issue.

For future reference to others interested in using PDF.js with Ember, turning off extensions for Array appears to fix the issue of incorrect pdf rendering of some pdf files.
https://guides.emberjs.com/v2.5.0/configuring-ember/disabling-prototype-extensions/

@richkniles I am new to Ember.js and Node.js, and I'm having trouble using PDF.js with Ember. Can you please give me some guidance on proper installation and placement of code in the Ember app structure?

thx so much!!!!!
I used to struggle with this issue for a long time, because this issue happened cross different versions of pdfjs-dist. Some rendered correctly, while some not.


Severe Firefox crash...

Hi i-rinat,
With the latest version of the freshplayer plugin Firefox crashes even before its window opens.
Affects at least Firefox 40.0.3 and 41.0b5.
crashreport on crash-stats.mozilla.com
This happens with the version from ppa:nilarimogard/webupd8, but with my self-compiled version as well...

same here.

Should be fixed by 86f31237c65d695881ac1f0ce0e624160ae3613d.
I never set flash_command_line parameter in config file, that's why I didn't notice the bug.

Thank you very much i-rinat!
I have never seen software fixed so fast!
No crash (yet).

BTW, crash could be worked around by removing flash_command_line parameter from configuration file. That parameter have no visible effect anyway.

Thanks, that's fixed for me too.

Thanks for fixing this. I noticed it yesterday, so I went to email the Webupd8 site owner that the PPA he maintains with the development branch of this application suffered from the issue. But I just arrived here, and see not only that it's fixed, but there's a workaround. Yay!

Yup since Webupd8 added this to their ppa it sure hit a lot of users, including me. Chrome noticed count jump. I'm just glad it's fixed.

thanks


Upgrade to Atmosphere 2.4.2

Ah, I didn't notice that you have updated the PR.
I'll test it locally and update to 2.4.3 if everything is OK.
Thank you!

There are some problems with the PR.
Initially its single unit test was failing.
I've fixed it by using again TesterBroadcaster.
Now deployment at Tomcat 8.x fails with:
java.lang.NullPointerException
	org.atmosphere.handler.ReflectorServletProcessor.loadServlet(ReflectorServletProcessor.java:104)
	org.atmosphere.handler.ReflectorServletProcessor.loadWebApplication(ReflectorServletProcessor.java:84)
	org.atmosphere.handler.ReflectorServletProcessor.init(ReflectorServletProcessor.java:172)
	org.atmosphere.cpr.AtmosphereFramework.initAtmosphereHandler(AtmosphereFramework.java:1732)
	org.atmosphere.cpr.AtmosphereFramework.initAtmosphereHandler(AtmosphereFramework.java:1721)
	org.atmosphere.cpr.AtmosphereFramework.init(AtmosphereFramework.java:917)
	org.atmosphere.cpr.AtmosphereFramework.init(AtmosphereFramework.java:838)
	org.atmosphere.cpr.AtmosphereFrameworkInitializer.configureFramework(AtmosphereFrameworkInitializer.java:75)
	org.atmosphere.cpr.AtmosphereServlet.configureFramework(AtmosphereServlet.java:84)
	org.atmosphere.cpr.AtmosphereServlet.configureFramework(AtmosphereServlet.java:80)
	org.atmosphere.cpr.AtmosphereServlet.init(AtmosphereServlet.java:75)
	org.apache.catalina.authenticator.AuthenticatorBase.invoke(AuthenticatorBase.java:501)
	org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:79)
	org.apache.catalina.valves.AbstractAccessLogValve.invoke(AbstractAccessLogValve.java:610)
	org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:516)
	org.apache.coyote.http11.AbstractHttp11Processor.process(AbstractHttp11Processor.java:1086)
	org.apache.coyote.AbstractProtocol$AbstractConnectionHandler.process(AbstractProtocol.java:659)
	org.apache.coyote.http11.Http11NioProtocol$Http11ConnectionHandler.process(Http11NioProtocol.java:223)
	org.apache.tomcat.util.net.NioEndpoint$SocketProcessor.doRun(NioEndpoint.java:1558)
	org.apache.tomcat.util.net.NioEndpoint$SocketProcessor.run(NioEndpoint.java:1515)
	java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:61)
	java.lang.Thread.run(Thread.java:745)

Please take a look at branch https://github.com/apache/wicket/tree/pr-160-master and commit https://github.com/apache/wicket/commit/ebfa1bb3381b187b704b0170b742f06ad2b84c24

X-Ref: https://github.com/Atmosphere/atmosphere/issues/2140

I've currently have problems with the lookup of the atmopshere.xml in an embedded tomcat.
Maybe the deprecated methods should be updated with the update of the latest atmosphere version.
EventBus.java:
private static Broadcaster lookupDefaultBroadcaster()
	{
		BroadcasterFactory factory = BroadcasterFactory.getDefault();
		if (factory == null)
		{
			throw new WicketRuntimeException(
				"There is no Atmosphere BroadcasterFactory configured. Did you include the "
					+ "atmosphere.xml configuration file and configured AtmosphereServlet?");
		}

The BroadcasterFactory.getDefault(); is deprecated and instead you should use:
org.atmosphere.cpr.AtmosphereConfig.resourcesFactory()
It's only a hint...I'm not sure if its related to my problem.
https://github.com/Atmosphere/atmosphere/wiki/Getting-BroadcasterFactory-and-AtmosphereResourceFactory-with-2.2-and-newer

With the release of Atmosphere 2.2.0, the static getters BroadcasterFactory.getDefault() and AtmosphereResourceFactory.getDefault() were deprecated, and from 2.3+ they are completely removed.

Okay...the static getters are still there ... :)

Good evening,
Any news with this PR, please? I'd really like it to join the wicket-7.x branch.
Thanks in advance.

Please try to make https://github.com/apache/wicket/tree/pr-160-master running  againts latest Atmosphere version.
I've hit several issues with 2.4.x.

@payou Please close this PR! Wicket team decided to discontinue the support for Wicket-Atmosphere (see https://issues.apache.org/jira/browse/WICKET-6305). Thanks!

OK
Atmosphere is über complicated
So good luck with WebSocket


Support Commons RDF

(Migrated from https://openrdf.atlassian.net/browse/SES-2091)
The Commons RDF project is aiming to provide a common set of interfaces for RDF-1.1 components, based on Java-8.
More details can be found at GitHub:
https://github.com/wikier/commons-rdf

Is this still an issue ? It seems that Apache Commons-RDF has implemented an RDF4J  binding (https://commons.apache.org/proper/commons-rdf/implementations.html), so perhaps this ticket can be closed ?

Agreed.


added prefix transmute at transmute!

closes #1852

Codecov Report
Attention: Patch coverage is 73.17073% with 22 lines in your changes missing coverage. Please review.

Project coverage is 89.23%. Comparing base (ace5bb0) to head (7ace93d).

Files with missing lines
Patch %
Lines

src/util/macro_util.rs
68.11%
22 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2024      +/-   ##
==========================================
- Coverage   89.46%   89.23%   -0.23%
==========================================
  Files          16       16
  Lines        5838     5918      +80
==========================================
+ Hits         5223     5281      +58
- Misses        615      637      +22

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@joshlf , please review it

@jswrenn , I request you to review this pr.

This PR has huge swaths of commented-out code. As a rule, we don't leave commented-out code in the zerocopy codebase.


Git Rollback lines Error

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

Version: 1.78.0 (Universal)
Commit: 252e5463d60e63238250799aef7375787f68b4ee
Date: 2023-05-03T20:11:00.813Z (4 days ago)
Electron: 22.4.8
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 22.4.0
Sandboxed: No
Steps to Reproduce:

write some code in my project.
hit to rollback lines changed.

similar to https://github.com/microsoft/vscode/issues/176051

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Module request: factoriolab

Description
A web server for hosting factoriolab, supporting both local and public deployments.
Notify maintainers
@PatrickDaG

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

Not quite sure we really need a module for this. It's just a static website, which you can use any webserver to host. Since there is no configuration as far as I can see, a module would just be a very basic {nginx,caddy,traefik,...} configuration which I don't think would be to usefull since we'd have to make a lot of assumptions about systems.


Issue#60 pds3 image object from numpy array

Good progress @bvnayak!  My initial reactions are

Perhaps the interface for array data should be PDS3Image(data) instead of  PDS3Image.open(data).  Is there any benefit to using open?
I think your test would seem more direct and be more informative to users if you manually created the numpy array rather than reading it from a file.
More tests could be implemented to test other parts of the code by creating a variety of different arrays.

@godber That's similar to what I was thinking: https://github.com/wtolson/planetaryimage/blob/plugin-refactor/planetaryimage/image.py#L26

@wtolson ... Bhavin and I have both played with that branch at this point and may fiddle with it further.  We're addressing some internal needs right now and then will regroup and look at your plugin based approach.

More tests on different dtypes are coming soon !


chore: fixed docs link in readme

Codecov Report

Merging #90 (ada7cdb) into main (66189b0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #90   +/-   ##
=======================================
  Coverage   82.94%   82.94%
=======================================
  Files          18       18
  Lines         739      739
  Branches      145      145
=======================================
  Hits          613      613
  Misses        126      126


awscli2: enable building without pip in the environment

Description of changes
This package has a special check that runs prior to building that checks for pip in the environment. We do not need pip in the environment (it's only there right now by coincidence and will be removed soon) as we provide all the dependencies needed through PYTHONPATH. So we need to relax this check from upstream.
I did this in postPatch because it's not clear to me that this is something that should be upstreamed. (It looks like it has been added specifically to address some concern in their environment.)
Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
23.11 Release Notes (or backporting 23.05 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.

@ofborg build python310Packages.awscli2


Should Factory.CreateInstance ignore NLog.Web errors?

In my target I have a layout looking like this:
public Layout HostnameLayout { get; set; } = "${event-properties:hostname:whenEmpty=${aspnet-request-host:whenEmpty=${machinename}}}";

Basically, I want to try and render a hostname by looking at various properties. If the user specify a structured variable named hostname, I will use that. If not, I look for an aspnet-request-host. If not there I try to pull the machinename.
The main challenge here is that the CreateInstance method of the Factory class, throws an exception if someone adds my target to their code but not add the NLog.Web package. I want to allow people running without the NLog.Web package, but I don't want the config causing an exception.
Is there some way to avoid this or should it maybe be allowed to reference layout renderers that may or may not be there?

If you want a soft dependency on NLog.Web and NLog.Web.AspNetCore. Then I guess you need to assign the layout on InitializeTarget instead of in constructor, and have some fallback mechanism if the wanted layoutrenderer is not yet available.

That would definitely work. Any way of "checking" a layout without trying to set the value and catch an ArgumentException?
private void TrySetLayout(Action<Layout> layout, string value, string fallback)
{
    try
    {
        layout(value);
    }
    catch (ArgumentException)
    {
        layout(fallback);
    }
}

Can you give the exception-callstack, when you assign a text-string to a Layout that contains an unknown layout-renderer ?
Would actually think NLog would ignore unknown layout-renderers. Since that is the behavior when loading from Nlog.config-file.

LayoutRenderer cannot be found: 'aspnet-response-statuscode'. Is NLog.Web not included?

Would actually think NLog would ignore unknown layout-renderers.

I think that would be a better solution overall.

I think that would be a better solution overall.

Think the ignore-behavior only happens when throwExceptions=false and throwConfigExceptions=false.
Must admit I kind of like that throwConfigExceptions makes NLog complain about typos and other errors.

Didn't know about throwConfigExceptions=false. I had it in the sample code. Must have been a copy-paste from somewhere. It now makes total sense to me and I agree. If users set this to true, NLog should indeed complain.

This issue should probably just be closed then. I'll keep my TrySetLayout-method to support people who want to set throwConfigExceptions to true and not include the NLog.AspNet package.

This issue should probably just be closed then

👍


 The Nuckelavee however can't touch clear water (for example from a river)
Phase 1: every attack on the Nuckelavee stacks the negative regen, making you lose health/mana faster and faster untill you cleanse yourself from it in a nearby water source (which is in the boss area).
Phase 2: The Nuckelavee also breaths out poisonous clouds, hitting them stacks up the debuff. Also the debuff now reduces your attack speed and increase your cooldowns (getting rid of the debuff also lowers your cooldowns again)
Phase 3: the Nuckelavee becomes stronger the more poison stacks are on targets in the arena, getting life steal, attack damage and damage reduction.

Putting aside balanxe for a moment, this hero is one dimensional. He does 1 thing. He snowballs. Not very fun. I assume the clean water dispells every debuff, not just the one from stage 1? If so, then the major counterplay to this hero is to run away. And the punishment for running away is that the boss snowballs even more bc your team is at a disadvantage.
I imagine this boss fight devolving into a perpetual water camping clownfest or being simply not very fun in general. Maybe chose one of the gimmicks and expand it into something with more substance.

I like the idea of using this character. Not all the boss fights need advanced mechanics.. There could be a few that just hit really hard, require magic absorption, etc.

Simplicity and one-dimensionality are two different concepts.
Putting that aside for a moment. Take your degen attack for instance. It forces the caster/support heroes to never attack it or else they won't have enough HP/Mana to finish the fight. But why? Why punish them for nothing? You assert that my suggestion is that your boss be more complicated when it's actually the opposite. Your boss is a rare bird that is simultaneously complicated and one-dimensional - you're punished for attacking, you're punished for taking agro (your tank is gonna die - period), you're not given any choice in the matter there's no counterplay. Pools of water aren't a counterplay, just a chore. And when the battle comes to an end and everybody's almost dead the fucker gets increased lifesteal?

"You assert that my suggestion is that your boss be more complicated" -- Did he even reply to you?

Wow. I didn't notice the OP and the reply were from two different people. Satisfied?

I thought he might have deleted it.

Phase 1 and 3 interesting, phase 2 isn't. Gonna keep this open.

With the sea monster sort of theme, you might have where it turns into a sea (through a transparent visual effect), and have stage 2 take place in a whirlpool where you're being sucked toward the center and back out, and you have to dodge the clouds while being tugged around by the current.

Idea for phase 2: give it a Heart of Tarrasque like regeneration - if the players leave it be for too long, it starts regenerating quickly.

From Town Hall: change poison to fire, combine with #5

Essentially a DPS race. Closed.


BIGTOP-4310: Executing {hadoop_mapreduce_home}/bin/mapred would occur ERROR that the mapred-config.sh doesn't exist

Description of PR
Details see: BIGTOP-4310
Fix the issue that executing {hadoop_mapreduce_home}/bin/mapred would occur ERROR.
How was this patch tested?
Applying this changes，
And executing the command: {hadoop_mapreduce_home}/bin/mapred --help, would print messages, as fallow.
Usage: mapred [OPTIONS] SUBCOMMAND [SUBCOMMAND OPTIONS]
 or    mapred [OPTIONS] CLASSNAME [CLASSNAME OPTIONS]
  where CLASSNAME is a user-provided Java class

  OPTIONS is none or any of:
......

For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'BIGTOP-3638. Your PR title ...')?
[x] Make sure that newly added files do not have any licensing issues. When in doubt refer to https://www.apache.org/licenses/

Bigtop provides /usr/bin/mapred as a wrapper script for setting up required environment such as HADOOP_LIBEXEC_DIR and JAVA_HOME. If users want to directly invoke /usr/lib/hadoop-mapreduce/bin/mapred, they should set required environment on their own. Same as /usr/bin/hadoop and /usr/bin/yarn. @guluo2016

Understand, thanks for your reply.


Polish AST Print with better indentation handling

Codecov Report

Merging #14 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #14   +/-   ##
=======================================
  Coverage   84.67%   84.67%
=======================================
  Files         143      143
  Lines        7282     7282
=======================================
  Hits         6166     6166
  Misses       1116     1116


Write a Dart program to print numbers from 1 to n

Description
Write a Dart program to print numbers from 1 to n
Input  : 5
Output : 1 2 3 4 5

How to contribute

Save the solution in print_numbers_from_1_to_n.dart file
Add print_numbers_from_1_to_n.dart file in program/print-numbers-from-1-to-n folder

!assign


chore: link with cookiecutter template

Coverage decreased (-2.5%) to 90.933% when pulling 28f7c19abb323fb5f174a56f6ad8ecdc4777fa37 on chore/link-cookiecutter-template into 50d228dd91aca40acc7f9293808b1e87cb645e5d on master.


Update create-process-on-windows.yml mapping to ATT&CK T1106

Just adding the att&ck category for T1106 to the create-process-on-windows.yml rule.
I think this makes sense for the Windows rule.
https://attack.mitre.org/techniques/T1106/

Hm, interesting. The way I read this is that we could assign this category to every rule using a Windows API call?
So I am not sure if this is really helpful.

Hm, I'm still not convinced here.
https://attack.mitre.org/tactics/TA0002/ describes Execution as "The adversary is trying to run malicious code."
That also applies to potentially all rules we have.

Cool, totally understand. Thank you for taking the time to consider and review.

Of course, please keep ideas, suggestions, and feedback coming!


[Journey] App crashes when tap on 'Search for a journey'

Environment: iOS 13.7, 14.7 (appetize simulator).
Reproducible: always
Build found: 1.0.98
Steps to reproduce

Open app.
Tap on 'Search for a journey' picture

Actual result
App crashes
Expected result
Search for a journey page should open
User story and test case links
User story #12

Duplicate of #307


Update README.md

Modified the opening paragraphs to match what is on the home page of ballerina.io - this is the refined, shorter version of the value prop.

@samgnaniah - if this PR gets approved, please update the readme for all of our other repos, too. I presume that they all have the same opening statements about Ballerina as this one, and then after the summary, they divert into repo-specific discussions?

@TylerJewell - Ack. Will do.

@samgnaniah I have merged this PR.

@hasithaa - Ack. Thanks for checking this.


Updating helm chart to be in line with stable requirements

Moving chart towards [stable requirements]https://github.com/helm/charts/blob/master/CONTRIBUTING.md#technical-requirements
Added post deploy usage in NOTES.txt
Added logo from info message output by helm lint
Added description of all parameters
Added livenessProbe and readinessProbe

The chart does not currently deploy successfully unless certificates are generated first. This doesn't meet the "Must successfully launch with default values (helm install .)" charts technical requirement so will need to be sorted. I think running without client certificate authentication by default and then using certificates if they are provided would be the usual approach to this.

This can be tested using helm install -n grafeas grafeas-charts/ and following the output.

The chart does not currently deploy successfully unless certificates are generated first. This doesn't meet the "Must successfully launch with default values (helm install .)" charts technical requirement so will need to be sorted. I think running without client certificate authentication by default and then using certificates if they are provided would be the usual approach to this.

I agree. I'm open to the contribution to enable this in this PR or a separate one.

https://github.com/grafeas/grafeas/pull/376/commits/764a16af0b97556d18cd1ffe70b876f16647e017
https://github.com/grafeas/grafeas/pull/376/commits/d6b2cc88992753783807cff456d3b6a1fac8fead
https://github.com/grafeas/grafeas/pull/376/commits/2f8b8cb66f6ff54f4aa4d71a4330ecca7090a932
https://github.com/grafeas/grafeas/pull/376/commits/5bfaa7d9f2827e968cbd521094a197b87a2e8d24

The chart does not currently deploy successfully unless certificates are generated first. This doesn't meet the "Must successfully launch with default values (helm install .)" charts technical requirement so will need to be sorted. I think running without client certificate authentication by default and then using certificates if they are provided would be the usual approach to this.

I agree. I'm open to the contribution to enable this in this PR or a separate one.

(This is done)


Close resolved issues.

This issues are resolved:
#9
#8 (i think)
#5
#2
#1

Thanks @RenatoSiqueira !
Let's review:
#9 is resolved
#8 is resolved
#5 is resolved
#2 is not resolved, I just tested here and we still have extra lines
#1 is not resolved, I just tested here and we still have extra spaces


[release-0.6] server/home: create home workspace with final URL on '~' access

Backport of https://github.com/kcp-dev/kcp/pull/1530

/lgtm

/retest


phantomjs: uninitialized constant #Class:0x00000001013fd048::MinimumMacOSRequirement

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.

[X] are reporting a bug others will be able to reproduce and not asking a question. If you're not sure or want to ask a question do so on our Discourse: https://discourse.brew.sh
[X] ran a brew command and reproduced the problem with multiple formulae? If it's a problem with a single, official formula (not cask) please file this issue at Homebrew/homebrew-core: https://github.com/Homebrew/homebrew-core/issues/new/choose. If it's a brew cask problem please file this issue at https://github.com/Homebrew/homebrew-cask/issues/new/choose. If it's a tap (e.g. Homebrew/homebrew-php) problem please file this issue at the tap.
[X] ran brew update and can still reproduce the problem?
[X] ran brew doctor, fixed all issues and can still reproduce the problem?
[X] ran brew config and brew doctor and included their output with your issue?

To help us debug your issue please explain:

What you were trying to do (and why)
What happened (include command output)
What you expected to happen
Step-by-step reproduction instructions (by running brew commands)

Currently, I can see that
$ brew install phantomjs
Error: No available formula with the name "phantomjs"
It was migrated from homebrew/core to homebrew/cask.

On my computer, I have installed phantomjs back in the days when it was available in homebrew core. Now, after updating my MacBook Pro to Mojave I try to run brew upgrade and it is all fine. It fails when I run brew bundle containing "old" and unexisting phantomjs:
# https://github.com/Homebrew/homebrew-bundle

tap "caskroom/cask"
tap "caskroom/versions"
tap "homebrew/bundle"
tap "homebrew/core"
tap "homebrew/services"
tap "thoughtbot/formulae"
tap "universal-ctags/universal-ctags"
tap "caskroom/fonts"

# Homebrew
# https://github.com/Homebrew/brew/
brew "phantomjs"

I receive output:
$ brew bundle
Tapping caskroom/cask
Tapping caskroom/versions
Using homebrew/bundle
Using homebrew/core
Using homebrew/services
Using thoughtbot/formulae
Using universal-ctags/universal-ctags
Tapping caskroom/fonts
Error: phantomjs: uninitialized constant #<Class:0x0000000102a979c0>::MinimumMacOSRequirement
Did you mean?  MaximumMacOSRequirement

I have solved the issue by running
$ brew uninstall phantomjs
Uninstalling /usr/local/Cellar/phantomjs/2.1.1... (50 files, 50.3MB)

and now when I run brew bundle I receive
$ brew bundle
Tapping caskroom/cask
Tapping caskroom/versions
Using homebrew/bundle
Using homebrew/core
Using homebrew/services
Using thoughtbot/formulae
Using universal-ctags/universal-ctags
Tapping caskroom/fonts
Error: No available formula with the name "phantomjs"

Here is my brew config:
 brew config
HOMEBREW_VERSION: 1.8.2
ORIGIN: https://github.com/Homebrew/brew
HEAD: 6f458aaed494eaab92622f41f9f1967add00b0f4
Last commit: 7 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 7e1ae2335f27ee2cd8f917b331b9a6b9db6552ca
Core tap last commit: 5 hours ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_AWS_ACCESS_KEY_ID: set
HOMEBREW_AWS_SECRET_ACCESS_KEY: set
HOMEBREW_CASK_OPTS: --appdir=/Applications
CPU: octa-core 64-bit kabylake
Homebrew Ruby: 2.3.7 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.7/bin/ruby
Clang: 10.0 build 1000
Git: 2.19.1 => /usr/local/bin/git
Curl: 7.54.0 => /usr/bin/curl
macOS: 10.14.1-x86_64
CLT: <IP_ADDRESS>.1.1539992718
Xcode: N/A

brew doctor did not have any warning before and after I have uninstalled phantomjs. brew outdated also did not report any issues.

@MikeMcQuaid 	I'm sorry but why did you close the issue? I think that error
Error: phantomjs: uninitialized constant #<Class:0x0000000102a979c0>::MinimumMacOSRequirement
Did you mean?  MaximumMacOSRequirement

is very misleading and should be changed/handled in a different manner. What do you think? People will come across this issue again with different casks

I'll improve the error messages in https://github.com/Homebrew/homebrew-bundle/pull/402.


Subcategory icons are still black sometimes

This is on <IP_ADDRESS>.

this is weird

try build 12

Still happening under build 12. Reproducible by simply installing the latest auto-config and opening it's example config gui.

Reproduced when the textfield is in front of the subcategory

Fixed in Build 13


fix(converter) check if css parameter is not missing

@OsoianMarcel please rebase 👍

@cordoval Hi, but it is only one commit, just merge it with master (without conflicts) 😃


[bug] The command exit code always be '0' in windows container

The command exit code always be '0' in windows container. Because the wait function on windows only check the err but not with the exit code of the stat (The error might be redirect).
https://github.com/argoproj/argo-workflows/blob/aa59b43748f78e599709add871af7ec14e1fd3c1/workflow/executor/os-specific/signal_windows.go#L29
I tried to fix this by the code below:
func Wait(process *os.Process) error {
	stat, err := process.Wait()
	if stat.ExitCode() != 0 {
		var errStr string
		if err != nil {
			errStr = err.Error()
		} else {
			errStr = "<nil>"
		}

		return errors.New(fmt.Sprintf("Exited failed. Error:%s", errStr))
	}
	return nil
}

This is just a temporary fix, so I just show my code here. Hope helpful.
The yaml below will reproduce the issue.
apiVersion: argoproj.io/v1alpha1
kind: WorkflowTemplate
metadata:
  namespace: default
  name: mytemplate

spec:
  templates:
    - name: test
      template: test-main

    - name: test-main
      nodeSelector:
        kubernetes.io/os: windows
      container:
        - name: main
          image: mcr.microsoft.com/windows/nanoserver:ltsc2022 # I used the nanoserver-ltsc2022 but the issue should also exist on ltsc2019
          command: ["cmd", "/s", "/c"]
          args: ["call exit /b 1"]

Please submit PR to fix!

Please submit PR to fix!

https://github.com/argoproj/argo-workflows/pull/9806
PR submitted


Do not pass sync interval by default

What changes are proposed in this pull request?
Before this PR, if we did not config alluxio.user.file.metadata.sync.interval in client side, and config it in master side while we disable the alluxio.user.conf.cluster.default.enabled meanwhile, the metadata sync interval will always set to -1.
After this PR, when alluxio.user.conf.cluster.default.enabled=false, master side alluxio.user.file.metadata.sync.interval will be accepted if user didn't set it.

Does this PR introduce any user facing changes?
No

Hi @maobaolong, thanks for the contribution! Would you mind explaining a bit more about the reason for this change in the PR's description?

@ChunxuTang I paste the related code to the description.

@tcrain @elega PTAL, thanks!

thanks for the fix LGTM

@jiacheliu3 PTAL

alluxio-bot, merge this please

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


arduino-cli lib install git-url

Great build tasks! There is issue with the current implementation with the lib install task, in that the command formed for library install using git is using
--git-url="...."
but should be
--git-url "..."
the equal sign breaks the command. Easy fix though in lib_install/lib_installV1/index.js, line 41. The same is probably true for the zip option (line 50) even though I did not try that.
I am not sure whether this equal sign vs. space issue also screws up the other command line options. According to the docs, space should be used to separate args from key....
Related: library install (and zip install) will only work if unsafe installs are enabled. This has to be done prior to calling the action. There are really two ways to accomplish that, adding a arduino-cli.yaml file to the repo with the appropriate keys or creating the file on the build agent on the fly using bash and
$(RUNNER_TOOLSDIRECTORY)/arduino-cli/$(arduino-cli-version)/Linux_64bit/arduino-cli config init
$(RUNNER_TOOLSDIRECTORY)/arduino-cli/$(arduino-cli-version)/Linux_64bit/arduino-cli config set library.enable_unsafe_install true
$(RUNNER_TOOLSDIRECTORY)/arduino-cli/$(arduino-cli-version)/Linux_64bit/arduino-cli config set board_manager.additional_urls ....
          ... # other config keys

I do the latter, because I do not really want the config file in the repo, as for desk side development I use VSCode with full arduino. It be great to add an action for creating the config file and adding the key/value. There are only like a handful of keys or so :)

Thank you! Great input for a new release.


Update sanitize-html from 2.1.0 to 2.12.1 fails. Module parse failed: Unexpected token.

To Reproduce
Step by step instructions to reproduce the behavior:

npm install
npm run build

Expected behavior

Clean build.

Describe the bug

Details
Version of Node.js:

16.18.1

Server Operating System:

MacOs Monterey 12.5.1. Github Actions script.

Additional context:

Relates to https://github.com/apostrophecms/sanitize-html/issues/592#issuecomment-1374020808 and https://github.com/apostrophecms/sanitize-html/issues/592

Two comments:

Node16 is past the EOL - you should upgrade to 18 or 20. This might solve your problem.
I'm not sure what the build script is. It isn't part of the core repo.
I thought we had done something to pin the version of htmlparser2, but I can't find it in the changelog. My latest version is installing 8.0.1.

Can you provide better steps to replicate the error? Again, there isn't a build script.

As state here the issue (I'm having the same one) is your webpack version.
You should update to v5. However, im not really satisfied with the author's answer and I'm wondering whether htmlparser2 should have a devDependency on webpack ^5

Thanks @Insomniak313! - I thought it was something with the version of htmlparser2.

I am using sanitize-html in backend (...it explodes badly on non esm base with the update) and frontend (...this issue).
Updating Node.js or Webpack is not possible for us on all environments, or restructuring our backend for that matter. Webpack 5 behaves badly with circular imports (our project has over 500 of these) while Webpack 4 just chews 'em all good and produces a working bundle.
What works currently is locking this module using "sanitize-html": "~2.7.3" on the package.json dependencies, then I manually apply commit c5dbdf77fe8b836d3bf4554ea39edb45281ec0b4 using patch-package to get rid of the security vulnerability.
I was comparing the source code of the project and not much changed. So another solution might be to force at the package.json overrides (npm) and/or resolutions (yarn) "htmlparser2": "^6.0.0" so the good old sub-dependency version is used.
My two cents. Alternatively just make a version 2.7.4 with the security patch. Thank you.

Could this issue be fixed by bumping htmlparser2 to 9.0.0 or 9.1.0?

Ran into the same issue, downgrading to 2.1.0 for now helped (as we are stuck on Node 16 for the time being).
Problem is not present when using a higher version of Node/Webpacker, and in our branch with upgraded Node/Webpacker version it works fine with 2.12.1.

@adamreisnz read my comment https://github.com/apostrophecms/sanitize-html/issues/663#issuecomment-2249942595

What works currently is locking this module using "sanitize-html": "~2.7.3" on the package.json dependencies, then I manually apply commit c5dbdf77fe8b836d3bf4554ea39edb45281ec0b4 using patch-package to get rid of the security vulnerability.

Using earlier versions will make you vulnerable to exploits.

Am aware, but as I said we have no choice as stuck on Node 16 for a while
longer.
On Sat, 9 Nov 2024, 00:45 Jorge Oliveira, @.***> wrote:

@adamreisnz https://github.com/adamreisnz read my comment #663 (comment)
https://github.com/apostrophecms/sanitize-html/issues/663#issuecomment-2249942595
What works currently is locking this module using "sanitize-html":
"~2.7.3" on the package.json dependencies, then I manually apply commit
c5dbdf7
https://github.com/apostrophecms/sanitize-html/commit/c5dbdf77fe8b836d3bf4554ea39edb45281ec0b4
using patch-package https://www.npmjs.com/package/patch-package to get
rid of the security vulnerability.
Using earlier versions will make you vulnerable to exploits.
—
Reply to this email directly, view it on GitHub
https://github.com/apostrophecms/sanitize-html/issues/663#issuecomment-2464519519,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADXYQWRBFTUQPWBP7YBWLDZ7SP6JAVCNFSM6AAAAABGKSVWASVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRUGUYTSNJRHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Version 2.7.3 works on node.js 16 (runs as far back as node.js 12).
Use this instead of 2.1.0 if you can. https://github.com/apostrophecms/sanitize-html/issues/663#issuecomment-2249942595

Thanks, I'll check it out.
On Sat, 9 Nov 2024, 07:13 Jorge Oliveira, @.***> wrote:

Version 2.7.3 works on node.js 16 (runs as far back as node.js 12).
Use this instead of 2.1.0 if you can. #663 (comment)
https://github.com/apostrophecms/sanitize-html/issues/663#issuecomment-2249942595
—
Reply to this email directly, view it on GitHub
https://github.com/apostrophecms/sanitize-html/issues/663#issuecomment-2465470910,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADXYQTBACVMWJJ6YC7P66LZ7T5K7AVCNFSM6AAAAABGKSVWASVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRVGQ3TAOJRGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ended up using DOMPurify, as we were also running into https://github.com/apostrophecms/sanitize-html/issues/639 in the latest version using Vite.


Update README.md

Test PR to test branch protections, please disregard.

Why did these checks work? 🤔


marketPriceRequest returning integer values for RDMFieldDictionary

Hi,
We came across a strange issue when using this library. When we try to do a marketPriceRequest and in the response, we get the integer value of the RDMFieldDictionary for most fields. So instead of 'PROD_PERM', we get 1  and so on for all fields in https://raw.githubusercontent.com/devcartel/pyrfa/24ca27ec6e0cb9b1f87a9390f8d219409ccaea9f/pyrfa/RDMFieldDictionary
Sample code:
    p.marketPriceRequest(','.join(rics))
    # Wait for the snapshot to be complete
    start, elapsed = time.time(), 0
    images, completed = OrderedDict(), set()
    while completed != set(rics) and elapsed < timeout:
        logging.debug('Checking for events...')
        for msg in p.dispatchEventQueue(1000):
            if msg:
                if msg['MTYPE'] == 'IMAGE':
                    images[msg['RIC']] = OrderedDict(msg)```

in the msg, we have integers in the keys instead of the fields, and the values are Unknown DataBuffer: (1, 'Unknown DataBuffer')
This is happening for all keys apart from ['RIC', '/VXRT.O'), ('SERVICE', 'EED'), ('MTYPE', 'IMAGE')]) in the msg.

Is there something in the allocation of the string to encode the dictionary values going wrong? Anything I can do to enable some more logging to debug this better. We also have another C++ library imported to by python via pybind, can that cause issues as well?

Looks like PyRFA cannot decode the data using dictionary provided.


Atlasmap Support for Quarkus

Hi Team,
I am trying to run a small quarkus camel project for using atlas map.
Any plans of creating a quarkus camel supporting below error.
"No endpoint could be found for: atlas://map/simple-mapping.adm, please check your classpath contains the needed Camel component jar"

<route id="xml-atlasmap">
    <from uri="timer:simple?repeatCount=1"/>
    <to uri="language:constant:resource:classpath:/data/individual.xml"/>
    <to uri="atlas:map/simple-mapping.adm"/>
    <log message="result is:\n ${body}"/>
</route>

Added maven dependency below
<dependency>
      <groupId>io.atlasmap</groupId>
      <artifactId>camel-atlasmap</artifactId>
      <version>1.42.4</version>
</dependency>

Regards,
Rahul

As far as I know you are the first trying it - contribution is welcome!

I am able to successfully create an application
https://github.com/rahgadda/ApacheCamel/tree/master/03-MyDev/quarkus-no-camel/atlasmap/json-to-json/transformation

I am able to successfully create an application
https://github.com/rahgadda/ApacheCamel/tree/master/03-MyDev/quarkus-no-camel/atlasmap/json-to-json/transformation

I don't think anyone have tried to get AtlasMap work with Quarkus. Reopening.

I don't think anyone have tried to get AtlasMap work with Quarkus. Reopening.

There is a JVM support in camel-quarkus since 1.5 version. You can use the camel-quarkus-atlasmap depency for Quarkus JVM mode.
I'm working currently on porting it to native mode, see camel-quarkus-1989, so it will be soon available as a stable extension.

The PR for native support has been merged.

Native support has been introduced since camel-quarkus 1.7.0

You can close this issue


Add support for JSON_INPUT in gzipped configMap.

This patch adds support for JSON_INPUT in gizipped configMap. It should be backwards compatible with env version. Switch is based on ending with ".gz" which is not a valid JSON.

This patch seems to work in OpenShift. LGTM

Seems to work with both vanilla Kubernetes and OpenShift. Code changes seem non-controversial as well. No Java code changed either, so just merging this

Seems to work with both vanilla Kubernetes and OpenShift. Code changes seem non-controversial as well. No Java code changed either, so just merging this

this is not correct. Java code PR (tesk-api) will follow, there are more changes in Java codes. I need to fix test first ;)


Updating lock json  for CF metadataplugin

The project wasn't installing with the current lock json files.
After the updates the project is now installing successfully.
Running "npm install" in the custom-functions-metadata-package directory results in the error message:
ERR! Cannot read property 'match' of undefined
It appears that the package-lock.json file is corrupt and needs to be regenerated.
To fix this, I deleted the package-lock.json file and ran npm install to generate it again.

You need to provide clearer details about the problem and why this is a solution. Which project are you referring to when you say it "doesn't install". This Office-Addin-Scripts repository, or a Yo Office project using this. Which version of Node and NPM are you using?
It is nearly impossible to review the diffs of package-lock.json changes. Why do these changes fix the problem? What is the problem that it is fixing?

Sure, sorry for not providing that many details.
The problem is that this project Office-Addin-Scripts doesn't build. The package that has the problem is the custom functions metadata plugin. When using npm install, it throws an error in lock-json file and simply updating it solves this issue.

Did you check the other packages as well?

Did you check the other packages as well?

I'm thinking this change alone didn't fix it. When I retry the npm install it gives me the same error again!
I will do some more testing and investigate this bug, for now I will make this PR a draft.


"not enough arguments for format string" error

Hi,
This query works directly with MySQL but through squealy gets the error "not enough arguments for format string" and I suspect it's because of the use of '%' in the format_date function call is causing problem for jinjasql. Is there a way to tell jinjasql to ignore the '%' ?

Hi chris,
Yes you're correct. This issue is due to the '%' character that is reserved for python formatting.
Solution - Use '%%' instead of '%'.
So, your query would be -> select DATE_FORMAT(createTimestamp, '%%Y-%%m') AS ....
Hope this solves your issue. Thanks.

Yes that worked, thx!


DELIA-55604: Fix for terminate called without an active exception in …

…HDMICEC
Reason for change: Thread join for created updated and poll thread
Test Procedure: Activate and deactivate HDMICEC plugin number of times
Risks: Medium
Signed-off-by: Kumar Santhanam<EMAIL_ADDRESS>

address the merge conflict in the github ui. Merging it.


Error when trying to quantize Llama3 70B instruct: module 'exllamav2_ext' has no attribute 'sim_anneal'

When trying to quantize llama 3 instruct from HF to exl2 I get this error:

-- Input: /home/ubuntu/text-generation-webui/models/meta-llama_Meta-Llama-3-70B-Instruct
-- Output: /home/ubuntu/temp/exl2/
-- Using default calibration dataset
-- Target bits per weight: 8.0 (decoder), 6 (head)
-- Max shard size: 8192 MB
-- Full model will be compiled to: /home/ubuntu/text-generation-webui/models/meta-llama_Meta-Llama-3-70B-Instruct_exl2_8.0bpw
-- Optimizing...
-- Optimizing:    1/ 240
Traceback (most recent call last):
File "/home/ubuntu/exllamav2/convert.py", line 248, in
optimize(job, save_job, model)
File "/home/ubuntu/exllamav2/conversion/optimize.py", line 119, in optimize
s_, si_, p_, c_, m_ = ext_c.sim_anneal(slots,
^^^^^^^^^^^^^^^^
AttributeError: module 'exllamav2_ext' has no attribute 'sim_anneal'

Using this command to quantize:
python convert.py -i /home/ubuntu/text-generation-webui/models/meta-llama_Meta-Llama-3-70B-Instruct -o /home/ubuntu/temp/exl2/ -cf /home/ubuntu/text-generation-webui/models/meta-llama_Meta-Llama-3-70B-Instruct_exl2_8.0bpw -b 8.0
Using the latest version as per git pull of main branch

You have an older version of the exllamav2 package installed, along with exllamav2_ext. You can either uninstall the package (pip uninstall exllamav2) and use the JIT version if you have the CUDA toolkit installed, or you can install the most recent prebuilt wheel from here.

you're correct. Thanks!


Psibase remote boot is a large request

Trying to call push_boot with a local psibase on a remote server running nginx proxying to psinode fails because the request is ~8m in size. I could bump up the max allowed size of requests in nginx, but why is this request so big?
From nginx access log
[error] 33#33: *296 client intended to send too large body: 8544049 bytes, client: <IP_ADDRESS>, server: psibase.io, request: "POST /native/push_boot HTTP/2.0", host: "psibase.io"

The request:
psibase -a https://psibase.io boot --no-doc -k PUB_K1_7jTdMYEaHi66ZEcrh7To9XKingVkRdBuz6abm3meFbGw8zFFve -p myproducer

It this really different from #239 other than smaller limit? psinode's http request limit is 20 MiB. maximum block size is 16 MiB.

Guess not, any reason to throttle webservers below the block limit?


XmlNodeConverter failing on invalid XML element names with ':'

When converting a JSON object containing properties which are not compatible as XML element names, XmlNodeConverter errors: The ':' character, hexadecimal value 0x3A, cannot be included in a name.'
Edit:
Upon research, it looks like XmlConvert.EncodeName is being used. However, this doesn't consider : characters, and it leaves them untouched so they could potentially be valid prefixes. XmlConvert.EncodeLocalName is probably what should be used.

I've fixed this with a new setting called EncodeSpecialCharacters.
const string initialJson = @"{""Be:fore:After!"":""Value!""}";

XDocument xmlNode = JsonConvert.DeserializeXNode(initialJson, null, false, encodeSpecialCharacters: true);

Assert.AreEqual("<Be_x003A_fore_x003A_After_x0021_>Value!</Be_x003A_fore_x003A_After_x0021_>", xmlNode.ToString());

string json = JsonConvert.SerializeXNode(xmlNode);

Assert.AreEqual(initialJson, json);

When it is set to true, all special characters in JSON property names are ignored and are encoded into the XML element name.

"root" instead of null will prevents another error message...
XDocument xmlNode = JsonConvert.DeserializeXNode(initialJson, _"Root"_, false, encodeSpecialCharacters: true);


[BUG] Tooltip displayed for unmounted component.

Describe the bug
When an element is set to display a tooltip (e.g. using data-tooltip-id), and that tooltip has a delayShow (e.g. 300ms), when the user triggers the tooltip (by e.g. hovering over the element), but the element is unmounted before the delayShow interval (e.g. after ~200ms), the tooltip for another element (always seems to be the element which is declared first) will be displayed.
Version of Package
v5.14.0
To Reproduce
See repro repo: https://github.com/knubie/react-tooltip-demo
It seems that the mouse should not move before the "phantom" tooltip is displayed.
Expected behavior
When the element is unmounted, no tooltip should be displayed.
Screenshots
https://github.com/ReactTooltip/react-tooltip/assets/431251/8b46a8c6-8f19-4d69-9451-8cc07022efed
Desktop (please complete the following information if possible or delete this section):

OS: macOS
Browser firefox
Version 114
Frameworks React 18.2

Thanks for reporting.
Some timeouts aren't being cleared on unmount. Should work with<EMAIL_ADDRESS>We'll close this once #1043 gets merged.
Let us know if you see anything else.

Awesome! Thanks for the quick fix. Just tested it out and it resolves the problem for me.

Available on official release<EMAIL_ADDRESS>


Routing improvements

Description
Design: https://www.notion.so/alchemotion/Error-and-latency-based-routing-in-the-Satsuma-node-gateway-3a60d896c5844f82bf90f68570b00608
Tracker: https://www.notion.so/alchemotion/Project-tracker-Error-latency-based-routing-1999d7ff3e5c430d9af3a7149923d8b9
Type of change

[ ] 🐛 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 not work as expected)
[ ] ⚒ Refactor (no functional changes)
[ ] 📖 Documentation (updating or adding docs)

Should we update the PR title to reflect accurately? Being a bit nitty on this since this repo and its PRs are all open to the public.


Select for  syntax highlighting language

Some cms have xyz.module instead of .php and syntax highlighting doesn't work.
Other editor / IDEs have a selector to manually select the needed language.

This is also already on my roadmap and something in the next version like mentioned in #73


Fix ruby warnings

For uninitialized @count ivar

Coverage remained the same at 100.0% when pulling f56f3add20775aef968f2a5281a2056dcb2ade33 on fix-warnings into 242a30fdb539dba91254ec833613a9cf69c1a9cd on master.

Coverage remained the same at 100.0% when pulling f56f3add20775aef968f2a5281a2056dcb2ade33 on fix-warnings into 242a30fdb539dba91254ec833613a9cf69c1a9cd on master.

Coverage remained the same at 100.0% when pulling f56f3add20775aef968f2a5281a2056dcb2ade33 on fix-warnings into 242a30fdb539dba91254ec833613a9cf69c1a9cd on master.

👍


Feature Request: Platform option

esbuild has an option to specify the output platform.
I want to specify neutral as a platform option for building SSR applications.
Platform - esbuild.github.io
Motivation
Currently, esm.sh always seems to specify the browser platform, even when the client is Deno.
https://github.com/esm-dev/esm.sh/blob/v131/server/build.go#L319
In certain cases, this behavior might be undesirable because this option automatically adds the browser condition.

When the platform is set to browser (the default value):

The conditions setting automatically includes the browser condition. This changes how the exports field in package.json files is interpreted to prefer browser-specific code.

For instance, the<EMAIL_ADDRESS>package.json has an exports field like the one below:
{
  "exports": {
    ".": {
      "module": {
        "worker": "./dist/emotion-cache.worker.esm.js",
        "browser": "./dist/emotion-cache.browser.esm.js",
        "default": "./dist/emotion-cache.esm.js"
      },
      "default": "./dist/emotion-cache.cjs.js"
    },
    "./package.json": "./package.json"
  }
}

I want to use the module.default script, but it is given lower priority compared to module.browser, so it is never resolved to module.default.
The neutral platform option can resolve this.

When the platform is set to neutral:

The conditions setting does not automatically include any platform-specific values.

The current workaround for this issue is to specify the subpath directly:
https://esm.sh/v128/<EMAIL_ADDRESS>
However, this workaround needs to be applied for all dependent packages as shown in the following example:
// import-map.json
{
  "imports": {
    "@mui/material": "https://esm.sh/v128/@mui/material@5.11.7?alias=react:preact/compat,@types/react:preact/compat,react-dom:preact/compat&external=preact,@emotion/cache,@emotion/react,@emotion/styled",

    // workarounds to import "@mui/material"
    "@emotion/cache": "https://esm.sh/v128/@emotion/cache@11.10.5/dist/emotion-cache.esm.js?external=@emotion/utils",
    "@emotion/react": "https://esm.sh/v128/@emotion/react@11.10.5/dist/emotion-react.esm.js?alias=react:preact/compat&external=preact,@emotion/cache,@emotion/use-insertion-effect-with-fallbacks,@emotion/utils",
    "@emotion/styled": "https://esm.sh/v128/@emotion/styled@11.10.5/dist/emotion-styled.esm.js?alias=react:preact/compat&external=preact,@emotion/react,@emotion/use-insertion-effect-with-fallbacks,@emotion/utils",
    "@emotion/use-insertion-effect-with-fallbacks": "https://esm.sh/v128/@emotion/use-insertion-effect-with-fallbacks@1.0.0/dist/emotion-use-insertion-effect-with-fallbacks.esm.js?alias=react:preact/compat&external=preact",
    "@emotion/utils": "https://esm.sh/v128/<EMAIL_ADDRESS>  }
}

For further context, please refer to this related comment: https://github.com/denoland/fresh/discussions/363#discussioncomment-6394957

{
  "exports": {
    ".": {
      "module": {
        "worker": "./dist/emotion-cache.worker.esm.js",
        "browser": "./dist/emotion-cache.browser.esm.js",
        "default": "./dist/emotion-cache.esm.js"
      },
      "default": "./dist/emotion-cache.cjs.js"
    },
    "./package.json": "./package.json"
  }
}

In this case, the worker": "./dist/emotion-cache.worker.esm.js" is used for Deno. The priority looks like:
deno -> worker -> module -> import -> es2015 -> default(under `module` condition) -> browser

i don't think changing the platform option to neutral can help


[Myanimelist Search] Add Anilist Search support

Extension
https://www.raycast.com/luknl/myanimelist-search
Description
Adding AniList Search would be a great idea, since now Anilist has a good amount of user base. The extension can provide a choice to the users to select a default source, i.e, either MyAnimeList or AniList.
Who will benefit from this feature?
More and more streaming services are providing features to sync their watch history with AniList (as the API of AniList seems better than MAL's) and thus AniList users are exponentially growing. Thus, it would be easier for anime fans to look up a title on Raycast using this extension from AniList.
Anything else?
Sometimes a title doesn't show up when the English name of the anime is searched for. In such cases, the title would pop up when the Japanese Romanized is searched for. I'm not sure if this is a bug or the extension can only scrape MAL for Romanized names. (example: "Your Name" doesn't show up. Instead, I have to type it as "Kimi No Na Wa")

Were you thinking of adding support to Anilist in this current extension, or create a new extension?

I'm not sure which would be better and what would most users prefer. It'd be great to have both in one extension. Maybe you can provide a default search (MAL or AniList) and the extension would search for the input from the default set source. And let users change between MAL and AniList using Command + P within Raycast or probably have an action within Command + K to change the search source.
Or you could also create a new extension which uses the AniList API.
Whether you implement it in this extension or create a new extension, I think you'll probably have a better idea than me about what's more feasible and what the users would prefer.


Is the emore datasets a public face dataset?

Hi, I'm a rookie in this field. I wonder if this data set is a public data set? I didn't find it on Google.

google MS1MV2, refer to arcface paper v3.


Check out of findOne

I'm doing simple catching when I want to check store before doing ajax request.
How do I check if findOne returns something or nothing. Currently as I can see it always returns an ObjectProxy for empty object, but how do I check if it is valid?
I think i can do store.findOne('user').get('source.length') but that looks too complicated

you could do store.findOne('user').get('firstObject')

@antulik great find - the issue w/ the ObjectProxy is that it's never falsy :(
I've done either what @matthewnessworthy mentioned or this in my repository like classes to check if the object is cached
  var user = store.findOne('user');
  if(user.get('id')) {
    //this would prove the object has an id (something ObjectProxy doesn't have by default)
  }

Thanks, that works for me.


edit connections (improvement)

Operating system or device - Godot version:
2.0.x
Issue description (what happened, and what was expected):
It would be great if you will be able to add their signals and then connect to it.
Steps to reproduce:
Link to minimal example project (optional but very welcome):

Sorry, I don't get it. Connecting to signals is exactly the purpose of this tool, so I don't see what you want as an improvement.

user custom signal create and connect

add to button "create signal" in editor.

For what I get: in GDScript you can add custom signals. This is asking to do this via editor, so you can connect custom signals with the GUI.

I made a button. and to process its signals I have to connect to it from the script. but it is more convenient to do it in the editor as well as with a conventional button.

There's already an interface for defined signals, is this what you want:

Or is it for custom signals?

I do not see the version 2.1 and do not know what is happening there. but it seems that this is not exactly what I wanted. I need the ability to (create and) connect to user signals via the editor.

Btw, custom signals will be added to the signal editor automatically if you define the signal at the top of the script like so:
signal my_signal

So that will at least save you from having to connect via code.

@Hinsbart  - a splendid news :)

and when it is scheduled to release a stable version 2.1. really want to quickly use the new features? :)

The feature is in 2.0 already, there's not much new functionality wise for this in 2.1.

@akien-mga  no this feature in 2.0.

That's a screenshot from 2.0.3...

Add a signal from editor is useless, since you still need an script to emit them, but connect signals defined in an script is worth

@MarianoGnu that is already possible.

This seems to me a request for something that is already on Godot, so no reason to keep open. If I misunderstood then comment clearly about what you really want.

What I like:
1 Create a signal from the script (methods may be different, I suggest to create in the editor is not very well)
signal  user_signal_name(param_signal)

2 When you open the Connection Editor, this signal appears there
3 We can podkryuchitsya editor of this signal.
But as I understood (from the message @Hinsbart) is already implemented in version 2.1. Unless of course I understand correctly

It's quite hard to understand what you are requesting for. If it's not editing signal connections from the editor, then I guess what you are asking for is signal declaration from the editor.
If that's the case, I don't think it makes sense. If you are creating a custom signal, you will have to emit it from the script no matter what, so I don't see the problem with declaring the signal also in the script.

later I make pictures showing what I suggest

As i understand he wants to declare a signal on script and connect it from editor. I tryed this on the past, but was not able to do it, probably becase i'm used to call add_user_signal instead of the keyword?


How do specify the index to which the logs are to be written?

Logs by default appear in logstash index. I would like the logs for each service be in their own index.

You would need to configure that in your logstash configuration, not in the logstash-logback-encoder config.
See here for an example based on type.  You can also base it on other fields in the events.


Add superuser and token/api key creation to dummy data

Cut down version of #2610

Backend checklist

[x] Formatted my code by running autoflake --exclude src/proto -r -i --remove-all-unused-imports src && isort . && black . in app/backend
[ ] Added tests for any new code or added a regression test if fixing a bug
[ ] All tests pass
[x] Run the backend locally and it works
[x] Added migrations if there are any database changes, rebased onto develop if necessary for linear migration history

Closed #1731


Procmon issue - UI misbehaves

When you have a process properties page up and click the up or down arrows, instead of starting at the line you are inspecting, it starts at line 1. This used to work as expected (up or down from the line you are currently inspecting).

Thanks for reporting this. We'll check it out.

You bet, I think it started occurring when the UI changed to have the more modern buttons.

Any update on this? the bug is pretty frustrating. I can no longer use up/down arrows to move around reliably :(


Add exponential formatting to BigInt

Should close #6.

Would this PR be merged soon? I'm looking for similar functionality on this.


Add Tajik Language (tg_TJ) Support to Flutter Localization (l10n)

Use case
As a developer, I want to create a Flutter application that supports multiple languages, including Tajik, so that I can provide a localized experience for Tajik-speaking users.
User Story:
As a Tajik-speaking user, I should be able to interact with mobile applications in my native language, making it easier to navigate, understand, and use the app’s features.
Currently, Flutter does not natively support Tajik in its flutter_localizations package, limiting developers' ability to create fully localized apps for Tajik-speaking communities.
Problem:
The lack of Tajik language support in Flutter localization forces developers to rely on custom implementations or external libraries, which may not be as robust or future-proof as native Flutter localization support.
Proposed Solution:
Implement and integrate the Tajik language (ISO code: tg) into the flutter_localizations package.
Add Tajik translations for date, time, and number formatting, as well as standard UI elements such as buttons, dialogs, alerts, etc.
Provide Tajik language support for Material components and widgets where localization is required.
Refs:
https://api.flutter.dev/flutter/flutter_localizations/GlobalMaterialLocalizations-class.html
https://github.com/flutter/flutter/blob/master/packages/flutter_localizations/lib/src/l10n/README.md
Proposal
We propose adding support for the Tajik language (ISO 639-1 code: tg) to the Flutter localization (flutter_localizations) package. Tajik is a variant of Persian spoken by over 9 million people primarily in Tajikistan and surrounding regions. Adding support for Tajik would enable developers to create localized Flutter applications that cater to Tajik-speaking users, ensuring wider accessibility and inclusivity in the global market.

Hello @abberdeen ,
sorry if this question of mine may come as rude, but can i know how you came to conclude flutter does not support Tajik ?
I'm not a Tajik speaker, neither do i know anyone from there, i'm just asking, since my knowledge is limited,
I'm not really sure the approach you took was the right one (like i said, tell me if i'm wrong)
once you have your "arb" file that contains all your text in your flutter app translated to Tajik,
you can add Tajik's language code to the Locale's your app is going to be supporting
for example:
  static final all = [     const Locale('en'), // English     const Locale('es'), // Spanish... if you app support Spanish too     const Locale('tg_TJ'), // Tajik   ];
you can try this short video:
https://youtu.be/zugxpAcbe4U?si=l1X0K6l6WQmH5bDR
if it works out for you, please let me know

Hello @TheCodeDaniel,
you can check for yourself that Tajik is not supported here: https://api.flutter.dev/flutter/flutter_localizations/GlobalMaterialLocalizations-class.html
Thank you the example you described is correct, but since Tajik is not supported it will not work.

Hello @TheCodeDaniel,
you can check for yourself that Tajik is not supported here: https://api.flutter.dev/flutter/flutter_localizations/GlobalMaterialLocalizations-class.html
Thank you the example you described is correct, but since Tajik is not supported it will not work.

Ok, i see, thanks for pointing that out, there is another option i think you should try.
you may have noticed, but normal "Google translate" supports Tajik.. take a look here:
https://translate.google.com/?sl=en&tl=tg&text=Hello&op=translate
there could still be one more way you can translate.
if you are familiar with "Google cloud console", there is an API in the "Library" you could use,
it's called "Cloud translation API".
see the package below for a full description of how it works
https://pub.dev/packages/google_translator.

Hello @TheCodeDaniel,
you can check for yourself that Tajik is not supported here: https://api.flutter.dev/flutter/flutter_localizations/GlobalMaterialLocalizations-class.html
Thank you the example you described is correct, but since Tajik is not supported it will not work.

Ok, i see, thanks for pointing that out, there is another option i think you should try.
you may have noticed, but normal "Google translate" supports Tajik.. take a look here: https://translate.google.com/?sl=en&tl=tg&text=Hello&op=translate
there could still be one more way you can translate.
if you are familiar with "Google cloud console", there is an API in the "Library" you could use, it's called "Cloud translation API".
see the package below for a full description of how it works
https://pub.dev/packages/google_translator.

the reason why i think this may work is because, the translation is done by google's API which already supports Tajik,
the translation is not handled directly with flutter, google does it and gives back a response.

https://pub.dev/packages/translator
there is also this package i came across too,
it's a translator package,
I've not used this one is particular, but it seems like it has a decent amount of popularity.
you can use it see, the implementation looks straightforward.
You can try the two options i suggested and see if anyone works for you,
let me know if i'm wrong somewhere of if the provided solutions will resolve your issue

Hello @TheCodeDaniel,
you can check for yourself that Tajik is not supported here: https://api.flutter.dev/flutter/flutter_localizations/GlobalMaterialLocalizations-class.html
Thank you the example you described is correct, but since Tajik is not supported it will not work.

Ok, i see, thanks for pointing that out, there is another option i think you should try.
you may have noticed, but normal "Google translate" supports Tajik.. take a look here: https://translate.google.com/?sl=en&tl=tg&text=Hello&op=translate
there could still be one more way you can translate.
if you are familiar with "Google cloud console", there is an API in the "Library" you could use, it's called "Cloud translation API".
see the package below for a full description of how it works
https://pub.dev/packages/google_translator.

Thank you for trying to help. However, I just need the language code, locale, tg to be added. And as a temporary solution I can create a splitting (https://docs.flutter.dev/ui/accessibility-and-internationalization/internationalization#adding-support-for-a-new-language), but it is inconvenient and especially difficult for beginners.

@abberdeen Oh, i see you just want Tajik to be supported by the intl package right ? that's fine,
although, i think you should move your issue to this repo instead "https://github.com/dart-lang/i18n/issues"
the flutter team don't directly handle the "intl" package, so please refer to that link and replicate your issue there, so they could help you fix it sooner, wishing you the best 👍


Description of the metrics for Microsoft.Storage/storageAccounts/queueServices

Descriptions of the metrics for Microsoft.Storage/storageAccounts/queueServices are different from ones in the following link:
Azure Storage monitoring data reference | Microsoft Docs
https://docs.microsoft.com/en-us/azure/storage/common/monitor-storage-reference?toc=/azure/storage/blobs/toc.json#queue-storage
Which is correct?

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

ID: a0a586c4-71fd-e60b-9c4c-1a3eb2443c74
Version Independent ID: 2f3706cc-073b-a310-8da4-2a5a90da6473
Content: Azure Monitor supported metrics by resource type - Azure Monitor
Content Source: articles/azure-monitor/platform/metrics-supported.md
Service: azure-monitor
Sub-service: metrics
GitHub Login: @rboucher
Microsoft Alias: robb

@kazshi Thanks for your comment! We will review and provide an update as appropriate.

Hi @kazshi. The descriptions in this article (metrics supported) are correct. But they don't seem that different to me.  They say essentially the same thing. Are you looking at translated versions of the two articles? Adding @normesta for visibility.

@rboucher Thanks for your comment. I'm looking at English page, the description of QueueMessageCount in this page says "The approximate number of queue messages in the storage account's Queue service.", but the description of QueueMessageCount in the Storage page says "The number of unexpired queue messages in the storage account." - which is accurate?

#assign: @normesta

This still seems to be a problem.

Checking with product group and Norm.

#label:"TriagedFeb2021"

@kazshi
This one is correct.
"The number of unexpired queue messages in the storage account."
we will update the docs appropriately.
Sorry for the huge delay.

#please-close


Update to latest changes in Spine Storage API

This changeset migrates the library code to the changes in RecordSpec and related types, provided in scope of Spine Storage API.
In particular, in this PR the following changes were applied:

Usages of Storage API were migrated to new RecordSpec.
All common Spine dependencies were updated. Some of them were additionally forced.
Recently added grpc-inprocess was also added as a test-implementation-level dependency to whom it concerns.

The library version is set to 2.0.0-SNAPSHOT.174 to match the one of core-java artifacts.

@alexander-yevsyukov PTAL.
CI builds will pass once core-java#1533 is re-reviewed and merged.


Rocketchat user management scripts

Two scripts

Dump user data (display name, username, user id, UTC offset, email, last login) (optional)

Not managed to test email or last login as I don't have the privileges

Process (user, channel) pairs to add users to channels

~Tested and should work, but waiting on exact format of channel information dump (edit: woops, missed #220, will test)~ works!

Relates to https://github.com/acl-org/acl-2020-virtual-conference/issues/208

@GuyAglionby Can you move this PR to https://github.com/acl-org/acl-2020-virtual-conference-tools/?


#38 Add Express Support

What Does This PR Do?
This PR adds support for express server
Description Of Task To Be Completed
This PR adds the configurations that will be needed if a user decides to create his app with an express server.

Add tests for all created components
Fix packages dependencies issues

Any Background Context You Want To Provide?
N/A
How can this be manually tested?

clone this repo git clone https://github.com/andela-iamao/react-raise.git
navigate to cloned repo cd react-raise
install dependencies npm install
Run tests npm test
Run npm link
Navigate to another directory
Start command raise new appname
Run npm install
Run npm test
Run npm run start-dev

Screenshot(s)
N/A
What are the relevant github issues?
#38

Coverage decreased (-0.3%) to 99.096% when pulling d14fcd4769bfd894f6c2756d4f7c4a5af1bcc014 on ft-configure-express-38 into e3b84d9d6f82a8c4ce04c20e851dfad9e3fd6663 on develop.


SQL Injection

Hi there,
I really like the look of this project, and it could be very helpful for me. However does the approach you are taking protect against any sort of SQL injection?
Generally I would expect the SQL statement to be defined as such, notice the ? instead of  {{ .Id }}
{{ define "SelectBlog" }}
select
        id,
        title,
        author_id
from blog
where id = ? limit 1
{{ end }}

Then the Id argument to be passed into the query:
	rows, err := m.DB.Query(rawSql, r.Id)
	defer rows.Close()
        ...

It looks as though this is not happening and raw input is just being passed straight into the query. Have you thought about a way to prevent an attack?
Or am I just missing something?
Thanks for the work!

Very good point and you would be correct.
Templating makes parameterised queries a little tricky, especially with if statements or for loops in the template.
I think the best solution is to let the developer indicate which inputs are to be parametrized. This can be done with a template helper function such as:
{{ define "SelectBlog" }} select id, title, author_id from blog  where id =   {{ .Id | param }}  {{ end }}
this would create the following query
select id, title, author_id from blog  where id ?
which would be prepared by database/sql.
The reason I would go with a "param" function is because a field in your request message may not be an input to the query, but rather an augmentation to the query itself. Here is a simple example:
{{ define "SelectBlog" }} select id, title, author_id from blog      {{ if .OrderBy }}      order by id     {{ end }} {{ end }}
Let me know your opinion.

I do like the templating approach and the logic that it allows.
And in this scenario the param function would gather the correct set of arguments to pass to the query in order? I do wonder about how the ordering of the function would be called on executing the template
Overall sounds like a reasonable approach to me.

I would take care of the ordering. It's more about whether this solution is valid.
I'll take care of it.

release v.0.4.0 resolves this issue


How does this node cliednt to the parity server?

Please suggest with an example

Hi @anujnetwork! This is not a complete client, just a protocol implementation to use in a client. For an example of usage, see Genesis Replicator.