Missing information

I was onboarding Windows Server 2012 R2 (down-level) and ran into issues whereby the server wouldn't onboard correctly. Following a lot of troubleshooting I believe the two main issues were the Seamless SSO wasn't set up on that particular domain and also that the down-level devices were not synchronised in Azure AD Connect despite being in the synched OU. This was resolved using the guidance in these links: - https://learn.microsoft.com/en-us/azure/active-directory/hybrid/how-to-connect-sso-quick-start & https://learn.microsoft.com/en-us/microsoft-365/security/defender-endpoint/troubleshoot-security-config-mgt?view=o365-worldwide#instructions-for-applying-computer-join-rule-in-aad-connect. I believe it would be worth point to those links somewhere in the requirements/pre-requisites to save extended troubleshooting traumas for other technical people.

Oh apologies, it said give feedback about this page, I assumed it would give you the URL. I believe it is this one. https://learn.microsoft.com/en-us/microsoft-365/security/defender-endpoint/configure-server-endpoints?view=o365-worldwide.
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg

From: Denise Vangel-MSFT @.>
Sent: Thursday, March 30, 2023 5:55:48 PM
To: MicrosoftDocs/microsoft-365-docs @.>
Cc: terryhugill @.>; Mention @.>
Subject: Re: [MicrosoftDocs/microsoft-365-docs] Missing information (Issue #11520)
Hello @terryhugillhttps://github.com/terryhugill. Can you please tell us to which article your feedback applies? This information will help us route the issue to the correct writer/team. Thank you!
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/microsoft-365-docs/issues/11520#issuecomment-1490626789, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AF4HDR4WQNNOONJ2CKNAXWTW6W3JJANCNFSM6AAAAAAWNA7JD4.
You are receiving this because you were mentioned.Message ID: @.***>

Thanks for letting me know
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg

From: Sriraman M S @.>
Sent: Monday, April 3, 2023 9:33:10 AM
To: MicrosoftDocs/microsoft-365-docs @.>
Cc: terryhugill @.>; Mention @.>
Subject: Re: [MicrosoftDocs/microsoft-365-docs] Missing information (Issue #11520)
@terryhugillhttps://github.com/terryhugill Thank you for your feedback. We have made changes to the document via PR##11557https://github.com/MicrosoftDocs/microsoft-365-docs/pull/11557
Once the author approves, the changes will be live.
Thanks
Sri
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/microsoft-365-docs/issues/11520#issuecomment-1493822920, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AF4HDR7GBWR6IM44AIUMVM3W7J4LNANCNFSM6AAAAAAWNA7JD4.
You are receiving this because you were mentioned.Message ID: @.***>


reward.quest.title not working on server

I want to make global notification when player finishing important quest.
My ZS code sending global message in chat, so everyone on server knows progress of other players.
There is my code: https://github.com/Krutoy242/Enigmatica2Expert-Extended/blob/5307a7e9483b9191ea83b196b9c3615745e16536/scripts/mods/ftbquests_custom_rewards.zs#L48
Im using events.onCustomReward and getting e.reward.quest.title for quest title.
Seems like (but not sure) since i have lang entries instead of actual titles {q.gates.conflux_i.name}, ftbquests try to translate it but unable to and print this error into crafttweaker.log:
[SERVER_STARTED][SERVER][ERROR] net/minecraft/client/resources/I18n
java.lang.NoClassDefFoundError: net/minecraft/client/resources/I18n
	at com.feed_the_beast.ftbquests.quest.QuestObjectBase.getTitle(QuestObjectBase.java:378)
	at youyihj.zenutils.api.ftbq.CTQuestObjectBase.getTitle(CTQuestObjectBase.java:39)
	at CrafttweakerReloadablemods\Ftbquests_custom_rewards186.handle(mods/ftbquests_custom_rewards.zs:48)
	at CrafttweakerReloadablemods\Ftbquests_custom_rewards186.handle(mods/ftbquests_custom_rewards.zs)
	at crafttweaker.util.EventList.publish(EventList.java:51)
	at youyihj.zenutils.api.ftbq.FTBQEventManager$Handler.onCustomReward(FTBQEventManager.java:104)
	at net.minecraftforge.fml.common.eventhandler.ASMEventHandler_1374_Handler_onCustomReward_CustomRewardEvent.invoke(.dynamic)

How i can fix this error? Is this means i cant notify players with lang entries, since lang can only being translated on clients?

Probably it would make sense to have special getter to get raw lang data of text fields, like e.reward.quest.titleRaw.

Maybe im using it wrong, but i still see untranslated entries.
There is my code:
  /**
  * Endorse player with message to whole server as its finished chapter
  */
  if (e.reward.tags has 'chapcomplete') {
    server.commandManager.executeCommandSilent(server,
      '/say §l' ~ e.player.name
      ~ '§r has fully completed the §n' ~ e.reward.quest.chapter.titleText.formattedText
      ~ '§r chapter after §l' ~ formatPlayTime(e.player) ~ '§r of play!§r ```Congrats!```'
    );
  }

Im using command /say because I need my Discord Bot to show in discord chat that player finished chapter.
But server on .titleText.formattedText returning lang entry instead of real chapter name.
Also, players see in chat lang entry instead of translated name too.

Do not use formattedText getter on server side. Pass the text component in ICommandSender#sendRichTextMessage

In your use case, in lang files e2ee.congratulation=%s has fully completed the %s chapter after %s of play! Congrats!
in script
player.sendRichTextMessage(ITextComponent.fromTranslation("e2ee.congratulation", [player.name, e.reward.quest.chapter.titleText, formatPlayTime(e.player)])

For the sake of history, for someone who will find this issue on similar task, there is a code that send message to all players, translating it even on server side, making nice messages in discord.
The key is server.broadcastMessage function.
titleCode is something like e.reward.quest.chapter.titleText.formattedText.
function notifyEveryone(player as IPlayer, langCode as string, titleCode as string) as string {
  server.broadcastMessage(crafttweaker.text.ITextComponent.fromData([{
    translate: langCode,
    with     : [
      player.name,
      { translate: titleCode },
      formatPlayTime(player)
  ]}]));
}


All svelte transitions should be disabled by default

I think all svelte transitions should be disabled by default, and the developer should choose whether to use a transition and which one they should use. I found svelte transitions in HeaderAction and HamburgerMenu.
Some developers want to optimize their user interfaces so they're super fast, so they don't want any transitions at all. Others would maybe want to use other transitions than those currently hard-coded in the library.
Such a feature can be can be easily implemented using the in and out directives: https://svelte.dev/examples#in-and-out and custom js transitions: https://svelte.dev/examples#custom-js-transitions
Update: in my initial message I didn't specify svelte transitions, just "transitions". I updated this message to reflect that I meant svelte transitions.

Currently, the transitions are controlled via CSS from carbon-components. FWIW, Carbon also uses the prefers-reduced-motion API to reduce animations if the user opts in. For example, if prefers-reduced-motion is triggered, the Skeleton states will not animate.
As for overriding transitions, would the following work?
* {
  transition-duration: 0 !important;
}

Sorry, I meant svelte transitions, as in those specified using the transition directive: https://svelte.dev/tutorial/transition
I have updated the issue title and my initial message to reflect this.

I stand corrected – you're right.
I find it difficult to disagree.

@weaseldotro
Released in v0.25:

the fly transition for HamburgerMenu has been removed
disable the HeaderAction transition by setting transition to false Usage example

I think all svelte transitions should be disabled by default

I'll consider this for the next major release.

Nicely done. I suppose you have to keep this feature in sync with the parent carbon library, so maybe some transitions must remain. But it's fine as long as they can be disabled by the developer.


Can Deploy Presearch without a PRESEARCH_REGISTRATION_CODE using terraform

presearch deployment success without inserting a PRESEARCH_REGISTRATION_CODE using terraform
steps made to test this case

using terrafrom go to examples/resources/presearch
insert your ssh and disk deployments info and nodes
[run command](run: terraform init && terraform apply -parallelism=1) in your terminal
the machine is up and you can ssh on it without a presearch node attached

in this case it will act like a normal VM, and its a user responsibility to provide the reg code


Need fix hidden class for TOC (false)

Hi! :octocat:
I think need fix class in theme when a TOC component was disable on page, bcz without this we have big empty place in right, it's not beautiful if we want use index page like a homepage...
I am play in browser console and change a class nx-hidden to hidden and get full-size :)
Before

After

I guess it is per design, if you want that main content takes width of toc, use the following setting theme: { layout: 'full' }
https://github.com/dimaMachina/graphql-eslint/blob/b2f9eb4600abcdca5b9d1082f4eb33a71b306e7a/website/src/pages/rules/_meta.ts#L2


[Event Hubs] SASCredential that follows TokenCredential

This issue is to track the investigation on whether the SAS Credential object we now have can be modified to implement the TokenCredential interface.
Apparently our friends at .Net & Java already do this.
Having the SAS Credential object follow TokenCredential will allow us to have a simpler constructor story for EventProcessor

cc @chradek

Since we have prior art via storage libraries for vending a SharedKeyCredential, we're going to continue using that rather than force it to follow the TokenCredential interface. Either way, it would be an event hubs-specific credential and not portable to other services.


Please look at the tactic changes and merge. Please just look at the others.

Most of your comments are known bugs/design issues.

In another branch, I've been working on issue #26 (Horrible Error Messages), but it's introducing massive slow downs. I can't tell if that's due to Redex or because I'm abusing syntax parse, but I suspect the former.
The tactic system does not provide support for checking whether a theorem you've state is actually proved.
The tactics should not throw errors, since that really prevents a try tactic from existing.

Oh and I seem to have disabled error checking in the interactive tactic, which may explain the crashes. I don't remember why.


How to configure a Relational Database Server on AWS

Proposed title of the article
How to configure a Relational Database Server on AWS
Introduction paragraph:
There are numerous reasons why you should set up the database for your next project on AWS. AWS includes a slew of features for scaling and securing your databases. Finding your way around AWS configuration settings can be difficult. The docs may not always be able to point you in the right direction.
In this post, we will go over how to set up an RDS on AWS. We will use a flask application to connect to the RDS.
Prerequisites
AWS account
Python
Flask
Flask-MySQL
Key takeaways:
Setting up and scaling relational databases in the cloud
Configuring a database on AWS
Setting up an application with Flask
Integrating a Relational Database with Flask
References:
https://flask.palletsprojects.com/en/2.0.x
https://amzn.to/3zyRDBr

@Bamimore-Tomi
Sounds like a helpful topic - lets please be sure it add value beyond what is in the official docs and what is covered in other blog sites.
Please be sure to double check that it does not overlap with any existing EngEd articles or incoming topic suggestions (if you haven't already) to avoid any potential article closure, please references any relevant EngEd articles in yours. - Approved


Performance of ol.Collection clear method

I have ol.source.Cluster with ol.source.Vector as a source. The ol.Collection represents the features property of the vector source.
1.) If I do not create the new instance of ol.Collection after every time the clear method of ol.Collection is called, the ol.Collection gets very slow on next push method calls.
http://jsfiddle.net/mattti/bca9t92c/1/
2.) If the clear method of ol.source.Vector is not called before the clear method of the ol.Collection is called,  the ol.Collection clear method performs very slow.
http://jsfiddle.net/mattti/bca9t92c/2/
3.) The only solution i found is to create a new ol.Collection, which in turn requires me to create new instances of ol.source.Cluster and ol.source.Vector, as these objects do not have the observable setters setSource and setFeatures respectively(please consider adding these methods to the relevant objects).
http://jsfiddle.net/mattti/bca9t92c/
In fiddle open the browser console to see the timer outputs of clear and push methods of each scenario.

What you're describing is all expected. If you consider that calling ol.Collection#clear() triggers an event with the removal of every item, which will in turn change the underlying vector source of the cluster source, which will cause the cluster to be recalculated.
The preferred solution to replace the features of a vector source is
source.clear();
source.addFeatures(newFeatures);

There is also no good reason in this scenario to use an ol.Collection for the vector source's features. It just adds unnecessary overhead.

Still the problem 1.) persists and has nothing to do with the performance of clear method itself. It is strange that the ol.Collection gets slow after the clear method was called. It is empty and all the following push methods are significantly slower. Why?


More fluent chinese expression

Signed-off-by: koushiro<EMAIL_ADDRESS>Fix some Chinese translation to make it closer to the idioms of native speaker.

As always, thanks a bunch :)


Issues with Installing PHP Extensions in Custom Docker Image

Hi,
I would like to build a custom Docker image for FrankenPHP and followed the instructions provided in the FrankenPHP documentation.
I attempted to install both the PHP extensions and the Caddy module for Cloudflare DNS. While the addition of the Caddy module was successful, I encountered issues with the PHP extensions. When visiting my WordPress site and checking the site health status, I receive the following warnings and error messages:

Warning: The optional module, imagick, is not installed or has been disabled.
Warning: The optional module, zip, is not installed or has been disabled.
Error: The required module, gd, is not installed or has been disabled.

# Extract the binary:
docker cp $(docker create --name static-frankenphp-tmp static-frankenphp):/usr/local/bin/frankenphp ~/src/frankenphp

# Check version
./frankenphp --version

# Replace
sudo rm /usr/local/bin/frankenphp
sudo mv frankenphp /usr/local/bin
sudo systemctl restart frankenphp

# Clean up the container:
docker rm static-frankenphp-tmp

Could you please help me troubleshoot why the PHP extensions are not being recognized in my WordPress installation? Thank you!

I think you have to do run the install-php-extensions part in the runner image, otherwise the libraries from the extensions will be missing in the final image.
...
FROM dunglas/frankenphp AS runner

# Replace the official binary by the one containing your custom modules
COPY --from=builder /usr/local/bin/frankenphp /usr/local/bin/frankenphp

# Now install PHP extensions
RUN install-php-extensions ...

I followed your suggestion and explored other methods, but the PHP extensions are still missing from the FrankenPHP binary. I'm building the Docker image inside an Incus container. Could this be causing the issue? I'm running out of ideas, and since I'm not a Docker expert, perhaps compiling FrankenPHP from source might be an alternative approach, as explained in the documentation: https://frankenphp.dev/docs/compile/.

Looking at your Docker file and command, this isn't doing what you think it is doing. You are copying a FrankenPHP that is not statically compiled to your system. You need to statically compile FrankenPHP: https://frankenphp.dev/docs/embed/
Instead, you are using a dynamically compiled version of FrankenPHP which is using your system's installed version of PHP. If this is a non-zts build of PHP on your system, you will suffer severe performance issues and use your installed extensions. If it is a zts build of PHP and the same version from the container (same minor version with no special patches) you should be able to use it just fine this way. In fact, this is what you would end up with if you compile FrankenPHP from scratch.

Thanks to everyone who offered their help.
I decided to take a different approach and build a FrankenPHP binary without using Docker. Following the instructions at https://frankenphp.dev/docs/compile/, I downloaded and built PHP from source, installed the required dependencies on Debian (including Go and xcaddy), and successfully created a binary that meets my WordPress needs.
Since I've achieved my goal, I believe this issue is no longer necessary.


Proposal: Allow for non-datetime formats for duration in JobAttributes

Currently, JobAttributes takes a duration kwarg that must be of type datetime.timedelta. While this makes sense, it is posing some challenges for me because datetime.timedelta is not JSON serializable, which is problematic for how some workflow engines operate. It would be much easier to handle if duration could be supplied as an alternate type. Perhaps the easiest route would be a str formatted as DD:HH:SS or even easier, an int or float in a pre-defined unit (e.g. minutes) that would be transformed to a date time object when the JobAttributes class isinstantiated.
What I'm thinking is something like:
self.duration = duration if isinstance(duration, timedelta) else timedelta(minutes=duration)

I've got opinions! :)
But I'll abstain for now.
Would JobAttributes(duration=parse_walltime('12:00')) work?
Also, can you tell me more about the serialization challenges? There is serialization code already in psij-python, which may help depending on the exact problem.
Allowing the duration parameter be a str is an option, but I worry a bit about adding too many formats at the core level since it trickles back into the formal specification which would ideally be kept short.

Thanks for the reply! Your opinions are most certainly welcome and appreciated. That's why I posted here after all 😄

Would JobAttributes(duration=parse_walltime('12:00')) work?

I'm not sure I follow this 100% --- apologies. What would parse_walltime be in this hypothetical scenario?

Also, can you tell me more about the serialization challenges? There is serialization code already in psij-python, which may help depending on the exact problem.

I can try, but I must admit that I'm in the middle of a bug-squashing session and have not delved too deeply into this particular issue (because I have a workaround).
In short, in this scenario, I am using a workflow packaged called Covalent and a custom executor plugin I developed around PSI/J called covalent-hpc-plugin. Covalent works by decorating a function and specifying the various execution parameters in a class that is passed to the decorator.
As is common among workflow packages, one of the criteria is that the (decorated) function must be JSON serializable, I believe. However, passing in a class object with a datetime.timedelta object causes a JSON serialization error.
The way around this, of course, is for me (the plugin developer) to allow the user to pass in a JSON serializable parameter (str, int, float, whatever) and then --- when the plugin ultimately calls PSI/J and makes the submission script --- instantiates the timedelta object when it's needed on the remote machine. This is what I'm doing, and it's fine, but it made me wonder if timedelta was the most user-friendly option in general (of course, it is perhaps the most unambiguous).
I recognize this may not be super insightful! Apologies if so --- I don't know the inner workings of Covalent's serialization approaches.

Allowing the duration parameter be a str is an option, but I worry a bit about adding too many formats at the core level since it trickles back into the formal specification which would ideally be kept short.

This is fair. Having a multitude of options is a potential concern, although I don't think DD:HH:SS as a str would be too outlandish seeing as that is how most job schedulers conventionally accept the walltime anyway and is what most end-users are probably used to by force of habit. I certainly prefer str over something like int/float because then PSI/J would have to specify a unit (e.g. minute, hour, whatever), which seems not great.

Thanks for the reply! Your opinions are most certainly welcome and appreciated. That's why I posted here after all 😄

Would JobAttributes(duration=parse_walltime('12:00')) work?

I'm not sure I follow this 100% --- apologies. What would parse_walltime be in this hypothetical scenario?

It would be a function provided by psij-python.

Also, can you tell me more about the serialization challenges? There is serialization code already in psij-python, which may help depending on the exact problem.

I can try, but I must admit that I'm in the middle of a bug-squashing session and have not delved too deeply into this particular issue (because I have a workaround).
In short, in this scenario, I am using a workflow packaged called Covalent and a custom executor plugin I developed around PSI/J called covalent-hpc-plugin. Covalent works by decorating a function and specifying the various execution parameters in a class that is passed to the decorator.
As is common among workflow packages, one of the criteria is that the (decorated) function must be JSON serializable, I believe. However, passing in a class object with a datetime.timedelta object causes a JSON serialization error.
The way around this, of course, is for me (the plugin developer) to allow the user to pass in a JSON serializable parameter (str, int, float, whatever) and then --- when the plugin ultimately calls PSI/J and makes the submission script --- instantiates the timedelta object when it's needed on the remote machine. This is what I'm doing, and it's fine, but it made me wonder if timedelta was the most user-friendly option in general (of course, it is perhaps the most unambiguous).
I recognize this may not be super insightful! Apologies if so --- I don't know the inner workings of Covalent's serialization approaches.

I think it's very useful.
At first I thought that json-serializability is indeed a requirement of covalent, and I wrote a reply based on that. As I was prepared to insert the evidence in the reply, in the form of pointers to code, I started having some doubts.
I'm looking at _workflow/Lattice.__init__ and seeing self.workflow_function = TransportableObject.make_transportable(self.workflow_function). This leads me to TransportableObject.__init__ and b64object = base64.b64encode(cloudpickle.dumps(obj)).
I would think that cloudpickle would support more than what json.dump does, but there is still much uncertainty in my head. If you had a stack trace, perhaps that could help pinpoint where in covalent the issue is.
In any event, it's a fair point that properties of psij objects are vanilla json-serializable except for pathlib.Path and timedelta objects.

Allowing the duration parameter be a str is an option, but I worry a bit about adding too many formats at the core level since it trickles back into the formal specification which would ideally be kept short.

This is fair. Having a multitude of options is a potential concern, although I don't think DD:HH:SS as a str would be too outlandish seeing as that is how most job schedulers conventionally accept the walltime anyway and is what most end-users are probably used to by force of habit. I certainly prefer str over something like int/float because then PSI/J would have to specify a unit (e.g. minute, hour, whatever), which seems not great.

I think the universal struggle in designing an API is to have a set of principles that sufficiently constrain the result so as to avoid an incoherent mess. And by "incoherent mess" I mean the result coming from the persistent uncertainty when confronted with the question of "why don't we do it this way instead", because that invariably happens.
We have the job arguments as a list of strings. It may be more intuitive, especially if you come straight from the command line, to have it as a single string that gets magically tokenized, just like *sh would do it. Of course, you don't get that for free. It comes with having to deal with arguments with spaces, quotes, etc (in some previous incantation of something like PSI/J, we never could convince Condor to pass arguments correctly because of such an issue). We decided to opt for formalism in the argv case and other cases because, ultimately, psij is a tool presumably sitting between one formal specification (workflow) and another formal specification (the scheduler or other job execution mechanisms).
It's not too dissimilar with walltimes. I honestly never know what the exact format a given scheduler accepts. Is "aa:bb" interpreted as "hh:mm" or "mm:ss". Is "aa:bb:cc" "dd:hh:mm" or "hh:mm:ss"? If it's "hh:mm", can hh > 24 or do I have to switch to "dd:hh:mm"?
So the question here is: should we maintain the formalism* principle, break it in some cases, or substitute it with another one? If we break it in some cases, is there something else that we could anchor the change on, such as, for example, the difficulties in applying a vanilla json.dump on a JobSpec? Even so, is the requirement imposed by and external library that generic objects be json-serializable something that passes whatever threshold of reasonableness we set?
*) It's more like "simple formalism". Surely one can also formalize the myriad of "aa:bb[:cc]" versions or the precise grammar of argument strings.

At first I thought that json-serializability is indeed a requirement of covalent

I know some workflow engines like Prefect require JSON serializability, but my understanding is actually what you mentioned --- that Covalent requires objects to be pickle-able. This is where I started to have some doubts as well.
Since it seems you were curious enough to code-diving, here's an example:
pip install covalent
pip install covalent-hpc-plugin

import covalent as ct
from datetime import timedelta
executor = ct.executor.HPCExecutor(
    address="doesntmatter",
    username="alsodoesntmatter",
    job_attributes_kwargs={"duration": timedelta(minutes=10)}
)

@ct.electron(executor=executor)
def add(a, b):
    return a + b

@ct.lattice
def workflow(a, b):
    return add(a, b)

dispatch_id = ct.dispatch(workflow)(1, 2)
result = ct.get_result(dispatch_id,wait=True)

The traceback:
TypeError                                 Traceback (most recent call last)
Cell In[2], line 18
     13 @ct.lattice
     14 def workflow(a, b):
     15     return add(a, b)
---> 18 dispatch_id = ct.dispatch(workflow)(1, 2)
     19 result = ct.get_result(dispatch_id,wait=True)

File ~/software/miniconda/envs/quacc/lib/python3.10/site-packages/covalent/_dispatcher_plugins/local.py:130, in LocalDispatcher.dispatch.<locals>.wrapper(*args, **kwargs)
    127 lattice.build_graph(*args, **kwargs)
    129 # Serialize the transport graph to JSON
--> 130 json_lattice = lattice.serialize_to_json()
    132 # Extract triggers here
    133 json_lattice = json.loads(json_lattice)

File ~/software/miniconda/envs/quacc/lib/python3.10/site-packages/covalent/_workflow/lattice.py:103, in Lattice.serialize_to_json(self)
    101 attributes["transport_graph"] = None
    102 if self.transport_graph:
--> 103     attributes["transport_graph"] = self.transport_graph.serialize_to_json()
    105 attributes["args"] = []
    106 attributes["kwargs"] = {}

File ~/software/miniconda/envs/quacc/lib/python3.10/site-packages/covalent/_workflow/transport.py:400, in _TransportGraph.serialize_to_json(self, metadata_only)
    397                 data["links"][idx].pop("edge_name", None)
    399 data["lattice_metadata"] = encode_metadata(self.lattice_metadata)
--> 400 return json.dumps(data)

File ~/software/miniconda/envs/quacc/lib/python3.10/json/__init__.py:231, in dumps(obj, skipkeys, ensure_ascii, check_circular, allow_nan, cls, indent, separators, default, sort_keys, **kw)
    226 # cached encoder
    227 if (not skipkeys and ensure_ascii and
    228     check_circular and allow_nan and
    229     cls is None and indent is None and separators is None and
    230     default is None and not sort_keys and not kw):
--> 231     return _default_encoder.encode(obj)
    232 if cls is None:
    233     cls = JSONEncoder

File ~/software/miniconda/envs/quacc/lib/python3.10/json/encoder.py:199, in JSONEncoder.encode(self, o)
    195         return encode_basestring(o)
    196 # This doesn't pass the iterator directly to ''.join() because the
    197 # exceptions aren't as detailed.  The list call should be roughly
    198 # equivalent to the PySequence_Fast that ''.join() would do.
--> 199 chunks = self.iterencode(o, _one_shot=True)
    200 if not isinstance(chunks, (list, tuple)):
    201     chunks = list(chunks)

File ~/software/miniconda/envs/quacc/lib/python3.10/json/encoder.py:257, in JSONEncoder.iterencode(self, o, _one_shot)
    252 else:
    253     _iterencode = _make_iterencode(
    254         markers, self.default, _encoder, self.indent, floatstr,
    255         self.key_separator, self.item_separator, self.sort_keys,
    256         self.skipkeys, _one_shot)
--> 257 return _iterencode(o, 0)

File ~/software/miniconda/envs/quacc/lib/python3.10/json/encoder.py:179, in JSONEncoder.default(self, o)
    160 def default(self, o):
    161     """Implement this method in a subclass such that it returns
    162     a serializable object for ``o``, or calls the base implementation
    163     (to raise a ``TypeError``).
   (...)
    177
    178     """
--> 179     raise TypeError(f'Object of type {o.__class__.__name__} '
    180                     f'is not JSON serializable')

TypeError: Object of type timedelta is not JSON serializable

So, I believe you are correct that the function arguments and returns have to be pickle-able, but I didn't investigate why this JSON serialization issue came up.

So the question here is: should we maintain the formalism^* principle, break it in some cases, or substitute it with another one? If we break it in some cases, is there something else that we could anchor the change on, such as, for example, the difficulties in applying a vanilla json.dump on a JobSpec? Even so, is the requirement imposed by and external library that generic objects be json-serializable something that passes whatever threshold of reasonableness we set?

This is all a very reasonable take! And I largely agree! My only potential caveat to it is that PSI/J specifically exists within the context of reducing a very commonly used pattern across workflow management tools (broadly defined). While Covalent itself may not strictly require JSON-serializability, I believe both Prefect and FireWorks do, and I imagine there might be others. So, on one hand, that could be an argument to try to maintain JSON-serializability with PSI/J. At the same time, however, maybe this is relatively rare (I can't say for sure!).
Anyway, for me, I am pretty flexible. I can address this "on my end," and I don't really see it as a major dealbreaker or anything. But perhaps food for thought. Ultimately, I am happy with whatever decision you make --- it's a good conversation to have either way 😄

So, I believe you are correct that the function arguments and returns have to be pickle-able, but I didn't investigate why this JSON serialization issue came up.

Oh, I see. The function is pickled, but the executor is json-serialized.
This seems to be triggered in _AbstractBaseExecutor.to_dict.
Correct me if I'm wrong, but I think the following apply here:

in principle, HPCExecutor could override to_dict and deal with timedeltas.
the data that is shipped remotely for the executor needs not actually be psij-python objects, since things get templated.
allowing strings in the constructor might not fix the problem, since json.dump(s) looks at the internal property type, which will still be timedelta.
this would also apply to pathlib.Path objects in JobSpec (i.e., json.dumps(pathlib.Path('/tmp')) also fails).
the various PSI/J classes, such as JobSpec, JobAttributes, etc., will remain problematic even if all their properties are json-serializable (i.e., json.dumps(JobAttributes()) fails).
the json package does not have a mechanism to make a class serializable except for a custom JSONEncoder which is not something that can be easily inserted into a third-party library like covalent
it might help to expose the various to/from dict methods in the psij serialization module/classes.
an electron is not always an electron

So the question here is: should we maintain the formalism^* principle, break it in some cases, or substitute it with another one? If we break it in some cases, is there something else that we could anchor the change on, such as, for example, the difficulties in applying a vanilla json.dump on a JobSpec? Even so, is the requirement imposed by and external library that generic objects be json-serializable something that passes whatever threshold of reasonableness we set?

This is all a very reasonable take! And I largely agree! My only potential caveat to it is that PSI/J specifically exists within the context of reducing a very commonly used pattern across workflow management tools (broadly defined). While Covalent itself may not strictly require JSON-serializability, I believe both Prefect and FireWorks do, and I imagine there might be others. So, on one hand, that could be an argument to try to maintain JSON-serializability with PSI/J. At the same time, however, maybe this is relatively rare (I can't say for sure!).

This discussion has helped quite a bit. I think there are two relevant issues from above with json serializability that I will restate:

serializing properties (e.g., json.dump(obj.__dict__)) would require the internal types to be json-serializable; changing the constructor to accept a more lenient type and convert it to the otherwise non-json-serializable representation will not quite work
psij classes will still be non-json-serializable without a custom JSONEncoder; in other words, from my understanding of the json package, json.dumps(job_spec) will simply not work. So if that statement appears in a library, there is nothing that can be done short of replacing all relevant psij-python objects with dicts.

We could consider the idea of doing class JobSpec(dict) and storing all properties in the dict itself, but the idea of emulating classes with dicts for the sole purpose of making a specific method of serialization work seems a bit like an anti-pattern.

Anyway, for me, I am pretty flexible. I can address this "on my end," and I don't really see it as a major dealbreaker or anything. But perhaps food for thought. Ultimately, I am happy with whatever decision you make --- it's a good conversation to have either way 😄

Indeed a good conversation.

Correct me if I'm wrong, but I think the following apply here:

In general, all the points you mentioned are correct.
The one caveat is that the way I have currently structured HPCExecutor is that everything PSI/J-related is written out as a file on the remote machine and are not instantiated locally. As such, there are no actual PSI/J objects being called when the executor is specified. See here for an example of what I mean. That's also why none of the serialization issues are a problem for me at the moment, once I ensured that the user is not passing in a timedelta (unless I override .to_dict() like you suggested). This is obviously a specific example and may not be a representative usage pattern (and also, this will inevitably change when PSI/J remote is made/released).

psij classes will still be non-json-serializable without a custom JSONEncoder; in other words, from my understanding of the json package, json.dumps(job_spec) will simply not work. So if that statement appears in a library, there is nothing that can be done short of replacing all relevant psij-python objects with dicts.

Good point. This seems like a bad game of whack-a-mole. From this conversation, I would lean towards not pushing for further JSON serializability of PSI/J inputs and instead requiring the plugin developer (e.g. me) to either work with the workflow engine (e.g. Covalent) to address any issues or design around it (like I've done by having the user pass in a dictionary of keyword arguments --- not the JobAttributes or ResourceSpec itself! --- to instantiate PSI/J classes).

The one caveat is that the way I have currently structured HPCExecutor is that everything PSI/J-related is written out as a file on the remote machine

I think that's what I meant by the second bullet point.
I think, should it come to it, there is a somewhat distasteful but possible solution to this in general: subclassing dicts. But perhaps it may be wise to defer making such a change while reasonable workarounds are possible.
That said, some of the suggestions that popped up here (e.g., the parse_walltime function, easier access to existing serialization routines, etc.) may still be worth considering.


Invalid URI error: arrival-and-departure-for-stop

Stack:
lib/oba/server.rb:21 in arrival_and_departure
app/controllers/alarms_controller.rb:8 in create

Error:
bad URI(is not URI?): http://api.tampa.onebusaway.org/api/api/where/arrival-and-departure-for-stop/Hillsborough Area Regional Transit_7690.json?app_uid=C071187D-67E0-458C-A1DA-CADE062AE667&app_ver=20170105.12&key=org.onebusaway.iphone&serviceDate=1488258000000&stopSequence=6&tripId=Hillsborough+Area+Regional+Transit_234879&vehicleId=Hillsborough+Area+Regional+Transit_2603&version=2

Duplicate of #7?

@sayhiben Fixed by the URL escaping fix that'll work for #7, but I want to add a separate test for this to validate that it keeps working properly.

I'm going to close this and I'll deal with the other issue appropriately.


Opening dev tools with a shortcut does not work anymore

Discord username
No response
What type of issue is this?
Permanent – Ocurring repeatably
Is this issue blocking you from using Enso?

[ ] Yes, I can't use Enso because of this issue.

Is this a regression?

[ ] Yes, previous version of Enso did not have this issue.

What issue are you facing?
After changes to Electron the shortcut to open dev tools in IDE stopped working. Theoretically it is enabled in our code:
https://github.com/enso-org/enso/blob/e7668ebc3ae39a489149bf0d0a51e45b7ef7b427/app/ide-desktop/lib/client/src/index.ts#L353
Expected behaviour
The default shortcut to opening dev tools should work in IDE.
How we can reproduce it?
No response
Screenshots or screencasts
No response
Enso Version
2023.1.1-nightly.2023.4.12
Browser or standalone distribution
Standalone distribution
Browser Version or standalone distribution
standalone
Operating System
MacOS
Operating System Version
No response
Hardware you are using
No response

Can we confirm this is a bug and not a deliberate decision to disable dev-tools by default? @sylwiabr @wdanilo
The reason is this line: https://github.com/enso-org/enso/blob/e7668ebc3ae39a489149bf0d0a51e45b7ef7b427/app/ide-desktop/lib/client/src/index.ts#L209
Removing it will fix the issue.

I confirmed it with @wdanilo on Tuesday :)


⚠ Bofrost.gr has degraded performance

In bb0f00e, Bofrost.gr (https://www.bofrost.gr) experienced degraded performance:

HTTP code: 200
Response time: 3748 ms

Resolved: Bofrost.gr performance has improved in 5b02175 after 8 minutes.


Create a project tracking board for KEP Improvements

Work item for https://github.com/kubernetes/features/issues/617
/kind feature
/sig pm
/milestone v1.13
/assign

Done: https://github.com/orgs/kubernetes/projects/5
/close


Simplify how we define rules that need to run on all files

Recipe is what we actually want. It includes it all.
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

@coderanger Any objection to this cleanup.


RFE - Explain what bigrest does

Hi, what does this actually provide? I know it provides some things that apparently make it easier to work with both BIGIP and BIGIQ but more examples are needed. The reason behind its creation.... to provide a unified REST API to BIG-IP products perhaps?

Documentation updated with "What is useful for?" and "Why create another SDK?" sections.
https://bigrest.readthedocs.io/introduction.html


Also add trailing slash to directory link's titles

Just a minor change.
Before:

After:

I did this is another commit. Thanks for the PR though!


Add touch gesture support for opening and closing panels

For better mobile experience, we need to implement support for touch gestures.
See https://hammerjs.github.io/ or similar.

[x] left panel should open on panright
[x] left panel should close on panleft
[x] right panel should close on panright

Base functionality is set up & working. Some fine tunning/polishing will be needed.


Delete inactive subscriptions automatically

Motivation
Some our tenants keep unnecessary subscriptions.
It may cause lack of storages, producer_request_hold and so on.
Modifications
Added subscriptionExpirationTimeSeconds to broker setting.
Result
If subscriptionExpirationTimeSeconds > 0, inactive subscriptions will be deleted automatically.

@merlimat

Another question is: how can this be enforced across brokers restarts?

Did you mean that a scheduler for inactive subscription checker is reset when a broker restarts ?

retest this please

@merlimat

I know it's not exactly the same but the effect is very similar to TTL on messages.

The aim is the same: to keep the backlog size as low as possible.
However I think auto-unsub achieves less storage usage than TTL since the inactive subscriptions are removed and there are no messages to retain, while "living"(not expired) messages always remain by TTL.

@merlimat ping

@yush1ga @nkurihar Sorry for delay on this one. The only concern I have is that the "timeout" for the delete subscription is reset each time a broker is restarted. So if you have a 2 weeks auto-deletion, and restart brokers frequently that would not be applied, or much later that one wold expect.
And so the amount of storage would be higher (compared to the TTL approach).

@merlimat
Should we make subscriptionExpiryCheckIntervalInMinutes settable like message TTL?
If we set this frequently enough compared to the frequency of restarting brokers, subscription expiry will be checked rightly.

@merlimat @nkurihar
I added lastActive to ManagedCursor.
We can detect the last time that consumers connect or disconnect in the subscription and it is used for deleting inactive subscription.

retest this please

retest this please

retest this please

Adding lastActive looks good to me.

@merlimat
Any thoughts?

@merlimat are we going to include this in 2.0?

Merged with current master and fixed conflicts.

Retest this please

Why are getLastActive and updateLastActive not named getLastActiveTime and updateLastActiveTime? The current naming is very confusing if you don't look at the comments. @yush1ga

In the interface ManagedCursor, the method getLastActive is in conflict with the definition of active and the method isActive。


DO_NOT_MERGE feat(PX-4502): adjust the display for order details page

This is very much work in progress here.
Starting to adjust this display. Want to see how many specs are failing here.
We still need to finalize a lot of what is supposed to be displayed here but the idea is there.

Ok. Resolved some TODOs here in logic.
Still need:

fix specs
add new specs
link tracking id if tracking URL is present
account for the case where no tracking URL or tracking Id is present.

cc @annacarey @jpotts244 @pepopowitz

Ok friends! @annacarey @jpotts244 @pepopowitz - removing WIP here as I added minimal spec and the state is currently much better than what we had before!
There is still more things to do on this specific one:

link tracking id if tracking URL is present
account for the case where no tracking URL or tracking Id is present.
write even more specs!
But I think all those can come as a follow up PRs. The main logic here is ok and I think it would be helpful to have it on staging to look at things.


Local Assets/Images Fail to Render

Thanks again for maintaining this great plugin; I really like the ability in the recent version to control CSS styling.
However, it seems that v0.12 has an issue where local images fail to render.  This was working fine in the prior version.
Local images display correctly when a card is viewed via VS Code's built-in Markdown: Open Preview.
This has been reproduced on a completely fresh install of VS Code, with no other extensions installed.  I have tried toggling on/off the various built-in markdown styles (shown below) to no avail.

Below is an example flashcard.  The URL images display just fine, but local images (both relative and absolute paths) fail to display.
---
recall: header
---

### URL Test

![Markdown Picture](https://picsum.photos/id/237/536/354)

### Relative Local Test

![Markdown Picture](./assets/image.png)

### Absolute Local Test

![Markdown Picture](/Users/sam/recall_test/assets/image.png)

Failed images look like this:

Hi. It is fixed in v0.12.2. Thanks for reporting this.

Thanks for the fix, working perfectly now.


[BEAM-7339] Enable 1Gb input for Python wordcount benchmark

Change input size of current wordcount benchmark to 1Gb. Apply to both py27 and py35 benchmarks. This will give us better performance signal than testing with current small input.
+R: @tvalentyn

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

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


Asset Detail Page

i'm thinking we could get rid of this /Utility/bix.jsx completely .. idk if it's used anywhere else .. but for it don't make much sense to


Executive overview for Version-by-File support

@Rick-Anderson @tdykstra @wadepickett ... Per internal discussion with PU management, I'll be preparing an executive overview to try and get support for implementing Version-by-File support. IIRC, there were two problems that I faced with my implementation that remained unsupported, thus forcing me to abandon it ...

Localization was either broken or heavily delayed.
The sidebar ToC wouldn't highlight the correct article that the dev was reading for non-current-release articles.

Otherwise, everything seemed to work great. Cross-links and navigation worked great, and there was no duplicate heading problem with copied content because each copy of the content was in a separate file. Docs would've been easy to update and maintain because all docs for a given release were in one folder named for its version (i.e., 3.1, 5.0, 6.0, etc.). You could simply set a search filter on a given release folder to isolate content for find-'n-replace updates (e.g., sample cross-links). It was really nice, and I could see how it would save a lot of time ... thus 💰💰💰💰.
I won't reach this for a few weeks. I need to get a Blazor tutorial written and work up the Pre4 Blazor coverage first. The purpose of this issue is primarily to track it as a work item. If you have input on this, you can email me. I'll email the draft to you for comments when it's ready.

Localization was either broken or heavily delayed.

Loc is gone, there is no more loc.

I won't reach this for a few weeks.

Let me know a couple days before you start and I'll arrange for support.

@Rick-Anderson ...

Loc is gone, there is no more loc.

What does that mean? Articles are still localized, so I don't follow what you're saying.

Placing a few links here for my tracking ...

https://github.com/dotnet/AspNetCore.Docs/commits/f39e314537c492bdb88ec53ad2942c4baa2af264/aspnetcore
https://github.com/dotnet/AspNetCore.Docs/pull/22776
https://github.com/dotnet/AspNetCore.Docs/pull/22784
https://github.com/dotnet/AspNetCore.Docs/issues/22975

Email sent ... closing as completed 🎉.

What does that mean? Articles are still localized, so I don't follow what you're saying.

They are not. I found a bunch of loc repo's that weren't deleted and had them deleted. Our content is no longer localized. Open an issue if you find localized content and I'll get the repo deleted as the loc content is not updated.

Not according to dev tools. Look at the server response for a page request and see that the content is coming down translated.
Besides, it's not the actual translation that was the problem for Version-by-File. The problem was that the links to the articles were malformed. IIRC, the version segment wasn't being removed. It looked like to me that the feature wasn't implemented for Version-by-File (or "Group Topic Versioning" if that's what they call it).

Note that when I say "localized," I mean translated. I probably should've used the word "translated" in the first place to avoid confusion. Yes, browsers handle localization of numbers, currencies, etc. I was referring to MS's loc/culture routing system ... that's what wasn't working for the feature.


[BT] DRep registration doesn't finish. (80 chars long title not cut, not submittitng)

Context & versions
staging.sanchonet; Demos 1.7
Video
https://www.loom.com/share/efffe5bdceb14f4c920f4f569030b0db?sid=92b94338-7560-41ea-a943-0f0fd5281d78
Steps to reproduce

Try to become a DRep
Go to end of flow and click "Submit"

Actual behavior
nothing happens
Expected behavior
Registration is complete
Other Issues

Url field needs to return over in download button
Error messages need to be more friendlier

OK NOW:
used json file:
https://raw.githubusercontent.com/mpawel79/testrepo/master/test_test_test_t.jsonld


ROX-23257: remove rocksdb from konflux

Description
Removes rocksdb from konflux builds
Checklist

[ ] Investigated and inspected CI test results
~- [ ] Unit test and regression tests added~
~- [ ] Evaluated and added CHANGELOG entry if required~
~- [ ] Determined and documented upgrade steps~
~- [ ] Documented user facing changes (create PR based on [openshift/openshift-docs]~(https://github.com/openshift/openshift-docs) and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change
If CI works, it works.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

/retest

/retest

failing test are known that was pushed to master yesterday.


IconFamily should default direction to LTR

Describe the bug
The default behavior when using the iconFamily format for specifying an icon ({ name: '' }) is auto, meaning that the icon will be flipped when the language is RTL.
What is the expected behavior?
Icons should not flip by default.
What are the steps to reproduce?

Open the showcase
Configure an icon using the icon family
Flip the direction to RTL
observe the icon is flipped

Anything else to add?
I'm logging this more as a question about what we want to be the default. If I'm not mistaken, I think the majority of icons should not flip in RTL, meaning we should default to that behavior and have user identify just those icons that they do want to flip to save them work.

@huayunh What are your thoughts on the  "Anything else to add" section here?

Technically, this would be a breaking change and would necessitate a new major version.

Correct; most icons should not be flipped, with a few exceptions. For what to flip, see here

The prop for direction should default to 'ltr' instead of 'auto' so that most icons will not be flipped in RTL languages (only a handful of icons should be flipped).


I have problem when I run "generate_data.py"

Process Process-1:
Traceback (most recent call last):
File "/usr/lib/python3.5/multiprocessing/process.py", line 249, in _bootstrap
self.run()
File "/usr/lib/python3.5/multiprocessing/process.py", line 93, in run
self._target(*self._args, **self._kwargs)
File "/home/hungdv/Documents/graphNN/TIES_DataGeneration/TFGeneration/GenerateTFRecord.py", line 266, in write_tf
driver = Firefox(options=opts)
File "/usr/local/lib/python3.5/dist-packages/selenium/webdriver/firefox/webdriver.py", line 174, in init
keep_alive=True)
File "/usr/local/lib/python3.5/dist-packages/selenium/webdriver/remote/webdriver.py", line 157, in init
self.start_session(capabilities, browser_profile)
File "/usr/local/lib/python3.5/dist-packages/selenium/webdriver/remote/webdriver.py", line 252, in start_session
response = self.execute(Command.NEW_SESSION, parameters)
File "/usr/local/lib/python3.5/dist-packages/selenium/webdriver/remote/webdriver.py", line 321, in execute
self.error_handler.check_response(response)
File "/usr/local/lib/python3.5/dist-packages/selenium/webdriver/remote/errorhandler.py", line 242, in check_response
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.WebDriverException: Message: newSession

Your problem is about the version of selenium, firefox and geckodriver. You can reference #16

F


Refactor risk and return models

Currently, there is a lot of repeated code within  risk_models.py and expected_returns.py.
Almost all of the functions therein take prices as inputs, before processing them into returns, with the following couple of lines repeated a lot.
    if not isinstance(prices, pd.DataFrame):
        warnings.warn("prices are not in a dataframe", RuntimeWarning)
        prices = pd.DataFrame(prices)
    daily_returns = prices.pct_change().dropna(how="all")

In the spirit of DRY, I'd like to refactor this without complicating the API. Haven't decided the best way of proceeding. I suppose I could put these lines into a function, but that would probably need to go in a separate file (not very elegant IMO).

Revisiting this issue, I think there are two solutions I could use:
1. Put the whole code block into a function and have each risk model call that function.
Pros: reduces repetition.
Cons: a bit ugly because each function will pass prices strait to the format_prices function. Also, people implementing new risk models will probably have to call that function?
2. Refactor into a RiskModels class, then handle formatting in the constructor
Pros: probably the standard way of doing it
Cons: for people to use their own risk model, they may end up having to inherit etc – a bit more of a hassle. Also, import paths become longer, e.g risk_models.RiskModels.function rather than risk_models.function.
All said, I am probably leaning towards the first one, but would appreciate a second opinion.

Hi Rob, thanks for the amazing work in this.
Might not be direclty related to this post -
When running risk_models.CovarianceShrinkage I get the error message "cov_matrix is not a dataframe or array". Whereas the other risk models work fine.
Are there constraints to the datasets when running CovarianceShrinkage, or should it be called differently?
Many thanks for your help.

@Paposss Hi, could you post full error/traceback then it easy to debug, I took a look at risk_models code but cant find your error message so I believe that it's not a problem with this package. Also, if it's not relate to this issue, please open a new one.
@robertmartin8 Hi, IMO the second approach is preferred, like the class CovarianceShrinkage, user can use it as
from risk_models import RiskModels
RiskModels.function(...)

@Paposss @88d52bdba0366127fffca9dfa93895
Actually the error message is from PyPortfolioOpt, in the EfficientFrontier constructor. This error is raised if the input data is not in the correct format – @Paposss could you please double check how you are passing the data?
CovarianceShrinkage calls functions from pandas, and as such it may be a bit more picky regarding the format.

@88d52bdba0366127fffca9dfa93895
thanks for the feedback, there is currently a refactor PR that I am merging ASAP, so it should improve usability.

Hi,
I like the first approach and I think that an "input valldator" would work nicely (something that both receives and returns a dataframe for prices, similar to check_array in sklearn.utils).
But I also like the second (object-oriented) approach because it seems very "global" (all models in the same place). I think both can exist at the same time, if there's a way of not repeating oneself. I have two suggestions in this regard:

What about doing something similar to the fit/predict interface of sklearn? This would require a reinterpretation of their approach, but it seems natural to me.
Have you considered "one-size-fits-all" functions? scipy does that all the time (see scipy.optimize.minimize: a function receives a method parameter and an if statement transparently runs the appropriate function underneath:

# I have no idea how this should be named!
def risk_matrix(prices, method="sample", *args, **kwargs):
    if method == "sample":
        return sample_covariance(prices, *args, **kwargs)
    elif ...:
        ...
    else:
        raise NotImplementedError(f"method {method} not implemented")

Any "global" solution (either object-oriented or scipy-like) makes it very easy to do backtests and compare many different approaches with little boilerplate code. Besides, there's an easy solution for allowing user-provided, custom methods: receiving a callable as parameter (the following was taken from the scipy codebase, but comments are mine):
def minimize(fun, x0, ...):
    ...
    # Lots of input validation
    ...
    if meth == '_custom':
        return method(fun, x0, args=args, jac=jac, hess=hess, hessp=hessp,
                      bounds=bounds, constraints=constraints,
                      callback=callback, **options)
    elif meth == 'nelder-mead':
        return _minimize_neldermead(fun, x0, args, callback, **options)
    elif meth == 'powell':
        return _minimize_powell(fun, x0, args, callback, **options)
    ...
    # Many more elif statemets
    ...

I personally think that this last possibility is a very elegant solution.

@schneiderfelipe
Thanks for the suggestions! I'll have a think about this. I'm not a huge fan of the fit/predict interface because I don't think it makes intuitive much sense for covariance matrices.
Your second suggestion is very interesting, and probably how I'd do it if I decide to make any changes. I'm still trying to figure out whether it's worth it because it'd be a breaking change.

Alternatively, I could just get rid of all of the input sanitisation and trust the user to enter a pandas dataframe...

Thanks for the suggestions! I'll have a think about this. I'm not a huge fan of the fit/predict interface because I don't think it makes intuitive much sense for covariance matrices.

I totally agree :)
Anyway, a check_dataframe function could be useful.

Your second suggestion is very interesting, and probably how I'd do it if I decide to make any changes. I'm still trying to figure out whether it's worth it because it'd be a breaking change.

I think it doesn't need to be a breaking change. Both ways of using risk and return models can exist simultaneously, e.g., the following lines would be equivalent:
S = risk_models.semicovariance(prices)
S = risk_models.risk_matrix(prices, method="semicovariance")  # not sure about this name "risk_matrix"


Feedback for “Command-line utilities”

I find the use of the term bytecode to describe the flat encoding of UPLC confusing: It's not bytecode, it's just binary encoding of the UPLC program into some specific format (eg. flat encoding). The term bytecode should be reserved to describe instruction sets for virtual machine. This is important because whereas a bytecode-encoded program can be interpreted (or rather executed) directly, using raw bytes, against some VM "architecture", a serialised program needs first to be deserialised to be interpreted.

I agree, let's get that fixed. Thanks for the feedback.


Save queued updates to memory only

Details
When trying to save the response of OpenApp we are saving a huge amount of data in one go during the queuedOnyxUpdates key setting. This will deprecate that key and make it so these queued updates only exist in memory.
Fixed Issues
$ https://github.com/Expensify/App/issues/25612
Relevant Slack thread: https://expensify.slack.com/archives/C035J5C9FAP/p1692141387843329
Tests

Basic testing of the app

Sign out / sign in

Verify API requests are succeeding and data is updating normally.

[ ] Verify that no errors appear in the JS console

Offline tests
N/A
QA Steps
No specific QA apart from regular regression tests is required.

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

Merge freeze is over now so getting this into review.

Hmm @tgolen can you please look at my changes here and maybe help me determine if the linked issue is still a concern? I took a quick look at the recent changes and can't say that I am having an easy time following them.

I think since we hit this:
https://github.com/Expensify/App/blob/d012c5499e2500dd4cf16bd8b3a1fd19deb95a73/src/libs/Middleware/SaveResponseInOnyx.js#L48-L50
Which should be fine for the OpenApp case since it's in requestsToIgnoreLastUpdateID 🤷♂
But we also have this now:
https://github.com/Expensify/App/blob/191bda19b9acd42e83c5f87a870a483c30da7cc5/src/libs/actions/OnyxUpdates.ts#L84-L87
Which is saving the entire response, but I assume these updates will not be the huge ones I was concerned about storing for no reason so maybe they are fine.

I think it should be fine. You're right that the big requests for OpenApp and ReconnectApp will skip that. The updates that get temporarily stored in Onyx will be the "incremental updates", so they should usually be relatively small.
This was the result of kind of a nasty work-around to avoid a cyclical dependency of the network modules and it was easiest to use Onyx as a temporary storage place for the updates.
This could be refactored to hold the objects in memory and use a more "custom built" solution with a basic pub/sub interface. (I know that's short on details, so I can expand on that if you want)

@marcaaron please fix conflict

@situchan Fixed can you please prioritize reviewing this?

Oof sorry I missed this @situchan thanks for your patience with me.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/tgolen in version: 1.3.76-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 1.3.76-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 1.3.77-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Storage blocks and crafting grids show different item amounts.

Storage blocks and crafting grids show different item amounts.
Under some circumstances, a storage block shows fewer items than it actually contains.
Test Setup:
Controller
Crafting Grid
Disk Drive with one 1K disk
One 1K storage block
Exporter -> chest (2x speed upgrades)
Importer <- chest (2x speed upgrades)
Powered by Ender IO conduits (plenty of RF)
I was working on a system to move items out of disk-based storage into dedicated storage blocks with void items set to limit the total amount stored in the system. To simplify moving items I did this (using cocoa beans as an example):
Set the empty storage block to whitelist cocoa beans, import only, void items, priority 5.
Set the exporter to whitelist cocoa beans.
Leave the importer with default settings.
Add a 1K disk with 1,000 cocoa beans, priority 0.
As expected, the exporter pulls all cocoa beans out of the disk storage, and the importer puts them into the 1K block. Here's where it gets weird.
Flip the 1K storage block to import/export mode. The exporter/importer will start to push/pull cocoa beans between the chest and the 1K block. Wait a short time, and then remove the whitelist entry from the exporter. Let the chest empty completely. The total items shown in the crafting grid and 1K block no longer match.
Breaking/reinstalling the controller has no effect.
Emptying the system with the exporter produces 1,000 items, so nothing is lost.
Refilling the storage block from scratch fixes the issue.
Version (Make sure you are on the latest version before reporting):

Minecraft: 1.10.2
Forge: <IP_ADDRESS>2
Refined Storage: 1.2.16
Using FTB Skyblock 3 mod pack

Does this issue occur on a server? [yes/no]
Tested on a server, haven't tried single player.
Setup:
http://i1062.photobucket.com/albums/t490/mthomas768/Random Odd Images/RS_setup_zpsvc7csc0j.jpg
Crafting grid display:
http://i1062.photobucket.com/albums/t490/mthomas768/Random Odd Images/RS_craftinggrid_zpsgttfjwla.jpg
Storage block display:
http://i1062.photobucket.com/albums/t490/mthomas768/Random Odd Images/RS_storageblock_zpsjm6ov55l.jpg
PS - thanks for the awesome mod! :)

That is quite the issue. I guess something is being odd in the storage block item calculation (could just be a counting error). As long as no items are lost :p

I don't think the amount stored variable is thread-safe (in ItemStorageNBT). Ideally, we should abolish it and calculate the amount dynamically.

Even if it's a display error, I still can't recover the part from the casing by shift clicking.

Might have something to do with old nbt from removed mod items? I'm also having this issue after removing a mod and my drive is stuck at 41 - the number of said items I had stored before removing the mod.

@TehSeph Can you drop the drive on the ground and (tell a server operator to) execute this and post the input here?

/entitydata @e[type=Item,r=3] {}

I know what causes this issue, no debugging is necessary.

Fixed by 8330a36858305571aa0cd8d42dbbaa129557067e


Time to deprecate?

Given that Tower duplicates/replicates a lot of the functionality expressed here, should we simply deprecate this crate?  Leave it up, but add the word deprecated somewhere in the repo description and/or README.
Nobody seems to ask about it in Gitter, and code-wise it's clearly stagnated.
/cc @carllerche

Actually, it looks like maybe just archiving the repo would be the right move.

Yes, before we archive, could you submit a PR to this repo and tokio-proto w/ a big deprecated warning? Similar to this: https://github.com/carllerche/eventual/blob/master/README.md#this-crate-is-deprecated
tokio-service can point to tower. tokio-proto probably should point to the "what to do w/ tokio-proto" issue.


Autocomplete Load More and hasNextPage Problems

Reproduction
https://formkit.link/6f676e2de505d2356a06098a173f49d3
Describe the bug
According to the documentation, if there is no search parameter, an empty array is returned. However, in some cases, we need to send a request and list the incoming data, even if the search is an empty string. In this case, we encounter a bug. What can we do to solve this problem?
Do this steps when search is empty string.
Steps for first problem:

Focus and see dropdownWrapper. You will see the page (skip) as 1
Lose focus and close dropdownWrapper and focus it again. You will see the page (skip) as 2

Video: https://streamable.com/mkz3hd
Steps for second problem:

Focus and see dropdownWrapper. You will see the page (skip) as 1
Click the load more button.
Lose focus and close dropdownWrapper and focus it again. You will see the page (skip) as 3

Video: https://streamable.com/uxd93f
Environment
• OS: Windows
• Browser Chrome
• Version 1.5.x

@kadirboylu Good question! I can see why this would be a bit confusing.
The autocomplete input by default has the prop always-load-on-open set to true. This means anytime the autocomplete is expanded, we are directly calling the options loader function. This is why you are seeing the page increment erroneously (the load options function is being called too many times).
By setting the prop always-load-on-open to false, you will see this function behave more like how you would expect: https://formkit.link/01e8552a38d3a03faf8f187275f2ff1a
We could maybe point out this prop always-load-on-open a bit more in the autocomplete and taglist docs specifically.
Let me know if you have any more questions.

@sashamilenkovic thank you for your quick response and detailed explanation 🙂 Is there an easy way to change the default value of the always-load-on-open prop for all autocomplete inputs? I dont want to add :always-load-on-open="false" by one by

@kadirboylu No problem at all! Here's an example of using a FormKit plugin to always set always-load-on-open to false for the autocomplete: https://formkit.link/33c6705a1f7143317e0b636f50e3a7af.

@sashamilenkovic thank you for everything!! 🥳😄


invalid C++: imagebufalgo_pixelmath.cpp tries to allocate std::vector

Problem
imagebufalgo_pixemath.cpp declares at two places a array_view<const TypeDesc>, which resolves to a std::vector<const TypeDesc>, but STL containers must contain assignable values.
The compiler may or may not accept this, depending on the STL implementation.
Older GCC versions don't accept it: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=16875
Expected behavior: [What you expected to happen]
Use a array_view<TypeDesc> instead. This doesn't break anything.
Actual behavior: [What actually happened]
build error:
https://gist.github.com/devernay/58f93957598ab6db31dc
Steps to Reproduce

[First Step]
[Second Step]
[and so on...]

Versions

OIIO branch/version: 1.6.10
OS: Mac OS X 10.6
C++ compiler: gcc 4.2.1

Solution proposed in #1367 (patch against 1.6, will port to master if it checks out with you).

that's what I did too, and it worked.
I think this stuff has been clarified in C++11, though I cannot find an exact reference.

Le 23 févr. 2016 à 02:07, Larry Gritz<EMAIL_ADDRESS>a écrit :
Solution proposed in #1367 https://github.com/OpenImageIO/oiio/pull/1367 (patch against 1.6, will port to master if it checks out with you).
—
Reply to this email directly or view it on GitHub https://github.com/OpenImageIO/oiio/issues/1364#issuecomment-187460943.

Merged.
I'm only putting this in 1.6.  The code in this area is different for master, and given that master will require C++11 eventually (probably soon), I don't think we need to accommodate older gcc limitations.


Rebrand bluemix to ibmcloud

Coverage remained the same at 78.922% when pulling 5d6c2e968c13336c3d0760a86e3a8dff74628f61 on rebranding into 1bf5ce09f739cd8f0c6b2a10b7788de388d12333 on development.


release tags that we can depend on from other projects' Package.swift

I'm running into a specific problem that would be fixed by release tags: https://github.com/tensorflow/swift-models/pull/640 causes error "swift-benchmark is required using two different revision-based requirements, which is not supported" if we try to depend on swift-models (which depends on a specific commit hash of swift-benchmark) and penguin (which depends on master of swift-benchmark at the same time (as we do in SwiftFusion).

Hi @marcrasi, thanks for raising this issue, we need to do a 0.1 release soon!

Hi @shabalind, any updates on this?


ESU Enablement for Year 2

Would it be possible to add the ability to enable year 2 ESU based on the new guidance from MSFT and the updated API.  https://learn.microsoft.com/en-us/rest/api/hybridcompute/licenses/create-or-update?view=rest-hybridcompute-2024-07-10&tabs=HTTP#volumelicensedetails

I merged a pull request from Courney Vallentine adding the feature you requested.


yann.lecun.com/exdb/mnist/ down

The website of Yann LeCun is down at the moment.
Therefore the MNIST-Script for downloading the files doesn't work either. (-> learn/python/learn/datasets/mnist.py won't work)
Are there any mirrors?

Apparently (according to @kakawait) you can download dataset via archive machine, e.g. https://web.archive.org/web/20160828233817/http://yann.lecun.com/exdb/mnist/index.html
You will need to download and copy the files to your training data directory manually.


ensuring complex_ior_fresnel() always returns a valid result

the complex_ior_fresnel function can sometimes produce a negative result
eg
complex_ior_fresnel( 1.0f, 0.0f, 0.670869052f ) produces -1.67196e-08
this change clamps the final result to valid range.

Thank you very much for your report. We have fixed the issue in our internal sources and will release the fix with an upcoming version of the MDL-SDK.


cross_val_predict from sklearn gives an interesting error for the entity extractor

All the relevant information can be found in this notebook to reproduce and document the error.

This was a funny one. It seems in the general domain (scenario), there aren't enough tagged entities to train a model. The error wasn't so helpful to determine this:
The Kernel crashed while executing code in the the current cell or a previous cell. Please review the code in the cell(s) to identify a possible cause of the failure. Click here for more info. View Jupyter log for further details.

Canceled future for execute_request message before replies were done

To fix this, I added the following into get_entity_reports_for_domains:
def check_percent_of_non_entities(domain_df):
        number_of_nan_entities = domain_df['entity_types'].isnull().sum()
        total_size = len(domain_df)
        percent_no_entities = number_of_nan_entities / total_size
        return percent_no_entities

    for domain in domains:
        print(f'Evaluating entity classifier for {domain}')
        domain_df = data_df[data_df['scenario'] == domain]

        percent_of_non_entities = check_percent_of_non_entities(domain_df)

        if percent_of_non_entities > 0.90:
            print(
                f'{domain} has {percent_of_non_entities}% non-entities. It is too sparse to evaluate.'
                )
            continue

I''m not sure if this is the right amount needed. It may also depend on the size of the data set. But this quick fix works for now. If anyone ever gets this error, this is the reason and the percent_of_non_entities threshold will have to be changed.


Invalidate query string parameters which don't exist

webargs allows the web user to pass parameter names which are not reflected in the use_args schema. This isn't favorable because ideally you'd invalidate field names which don't exist in schema, in case there's a typo or something small/hard-to-notice.

This would break argument insertion, namely the ability to limit free users' collection searches by inserting an argument into flask args.


kernel: timer: cycle64: kernel.timer.cycle64 reports timeout on nrf54l15pdk/nrf54l15/cpuapp

Describe the bug
Test tests/kernel/timer/cycle64/kernel.timer.cycle64 reports timout.
Observed for nrf54l15pdk/nrf54l15/cpuapp.
To Reproduce
Steps to reproduce the behavior:

have nrf54l15pdk/nrf54l15/cpuapp connected
go to your zephyr dir
call ./scripts/twister -T tests/kernel/timer/cycle64 -p nrf9160dk/nrf9160/ns --device-testing --device-serial /dev/ttyACM0 -v --inline-logs
See console output with error

Expected behavior
Valid console output
Impact
Not clear
Logs and console output
*** Booting Zephyr OS build v3.6.0-5099-g127cb9edb6c6 ***
Running TESTSUITE cycle64_tests
===================================================================
START - test_32bit_wrap_around
32-bit wrap-around should occur every 4294s
[ddd:hh:mm:ss.0ms]
[000:00:03:27.029]: cycle: 000000000c5706ce

Environment (please complete the following information):

OS: Ubuntu 20.04.1 LTS
Toolchain: Zephyr SDK 0.16.5-1
Commit SHA or Version used: v3.6.0-5099-g127cb9edb6c6

nrf_triage_done

https://nordicsemi.atlassian.net/browse/NRFX-5564


Project_RPAChallenge_PDD

@paruA32  please review code

Hi yash .Your PDD looks good .

Thank u !
On Tue, 8 Aug 2023 at 7:33 PM, paruA32 @.***> wrote:

Closed #819
https://github.com/vajrang-b/RPA-Developer-in-30-Days/pull/819.
—
Reply to this email directly, view it on GitHub
https://github.com/vajrang-b/RPA-Developer-in-30-Days/pull/819#event-10036045307,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBWWCR7DTE3ELBAM7QRUEJTXUJBMPANCNFSM6AAAAAA3H7A64E
.
You are receiving this because you authored the thread.Message ID:
<vajrang-b/RPA-Developer-in-30-Days/pull/819/issue_event/10036045307@
github.com>


Enforce tag field constraints

Right now we only support text tag field with no tokenization (= raw tokenizer).
The reason why we do not handle integer types is just due to the lack of tag generation logic in the packager.
The reason why we do not allow for tokenizer, is because

they might generate a lot of string
the tokenizers that have some normalization logic (lowercasing, stemming, etc.) would require a treatment different for all splits, preventing the pruning to happen in the root node.

The reason why we do not handle integer types is just due to the lack of tag generation logic in the packager.
I'm not sure to understand that point. Currently, a tag is a concatenated string tag_field:tag_value where tag_value is a string representation of the typed value. What do you mean by handling integer types?

Right now the packager code just tries to interpret the term dictionary
bytes as utf8.
It truncates the result if it is not possible.
For integer, the values are big endian encoded over 8 bytes.
As a result, we generate pretty weird tags.
Generally speaking we want a unit test too.
Le ven. 17 déc. 2021 à 17:33, François Massot @.***> a
écrit :

The reason why we do not handle integer types is just due to the lack of
tag generation logic in the packager.
I'm not sure to understand that point. Currently, a tag is a concatenated
string tag_field:tag_value where tag_value is a string representation of
the typed value. What do you mean by handling integer types?
—
Reply to this email directly, view it on GitHub
https://github.com/quickwit-inc/quickwit/issues/912#issuecomment-996531750,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHZMQRYRQJII4UVAFAJOHTURLYWVANCNFSM5KIDRPNQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


support for laravel 5.3

Does it work for Laravel 5.3? seems like i can't composer require this library for laravel 5.3.
Thanks.

Add support 5.3 version. Will test functional for new version soon as possible.

Thank. But try to install by using composer and still fail for requirement not meet for laravel  V5.3.2.
Thank You.

I install fresh version of laravel 5.3(5.3.28), add to composer "langaner/materialized-path": "dev-master" and run composer update. Build pass without version compatible errors.

It work for now. Many thanks for your help.

you are welcome ))


NodeExec: Create task skeleton

Create task skeleton as given here

Yep and make sure mandatory fields are asserted for. We should ensure that we group the tasks with a default name and let users override it if they want to.

Lets leave out incremental build support. I'm guessing users can define this in their custom tasks. Everyone has a different setup for file organization and that is better left to users. Considering this what we have in our private setup,
exec {
	workingDir "$projectDir"
	commandLine = generateCommandLineParams([nodeJsExecPath, "${nodeModulesPath}/less/bin/lessc", "-x", sourceFilePath, targetFilePath])
}

this could be simplified in this repo as,
nodeExec {
          module = "lessc"
          command = "-x ${sourceFilePath} ${targetFilePath}"
}


SABnzbd Memory Allocation Error

Restarting the SABnzbd server from the web interface results in this hanging crash.
Exception in thread Thread-27:
Traceback (most recent call last):
File "/usr/lib/python2.7/threading.py", line 801, in __bootstrap_inner
self.run()
File "/usr/lib/python2.7/threading.py", line 754, in run
self.__target(*self.__args, **self.__kwargs)
File "/home/public/sabnzbd/sabnzbd/init.py", line 478, in trigger_restart
cherrypy.engine._do_execv()
File "/home/public/sabnzbd/cherrypy/process/wspbus.py", line 408, in _do_execv
os.execv(sys.executable, args)
OSError: [Errno 12] Cannot allocate memory

@nosajthenitram - Thanks for posting.  Could you please follow the instructions in our issue template?

Closing issue, feel free to reopen after adding the required info from the template.

@benhillis Unable to reopen an issue closed by you. Can you please reopen this issue if you feel as though it's adequately updated?

@nosajthenitram - You can help Ben with:
strace -ff python SABnzbd.py 2> SABnzbd.strace
# do whatever 'Configure sabnzbd' means. I quit at:
#    "Please enter in the details of your primary usenet provider."
# ...watch it crash, kill the `strace` with ctrl-c if necessary, then:
gzip SABnzbd.strace

Attach SABnzbd.strace.gz to this github issue.

Hi I am having this issue how can I help?
I have seen this over the years but I can reproduce this almost all the time with a specific setup I have been testing over years. Now I am switched over to this set up and been using it fine but have this issue.
I am using Windows for Linux Sub-system (WSL) and I am running SAB, SickRage and Couch from the Ubuntu bash and all works well and is much fast then older methods I used to run phython source from Windows. Windows was slower but I never had issues.
WSL is so much fast and I have good support for my 60+TB Raid for my media. Believe it or not the driver support is a pain in the butt in Linux so my server runs Windows and WSL is a blessing!
That being said let me know what I can I can capture or help to potentially fix. I get this 100% of the time when I restart using the GUI. It then hands until I break the command line and then I re-run and life is good as long as I do not restart.
Hope I can help, cheers.

Assuming...:
$ sudo apt install python3 python3-cheetah python3-feedparser python3-cherrypy3 python3-portend python3-sabyenc \
    unrar par2 git # probably else
$ git clone https://github.com/sabnzbd/sabnzbd.git
$ cd sabnzbd
$ python3 SABnzbd.py

"Click restart SABnzbd button" takes on recent WSL1 (sic). If the scenario was once not working, it appears to be now (call it 19042).


[mini.files] Preview on selected file extension only

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md

Module(s)
mini.files
Description
I would like to have the ability to select a couple of file extensions to show preview (files with text only). Sometimes I move over some large binary or object files and that cause Neovim to lag or freeze. Is there a way to do this?

Thanks for the suggestion!
I can not reproduce any visible lag (for example, with 4.5Gb zip files or 1.5Gb mkv files).
Working with non-text files is already fairly optimized and large size should not be a factor here. Providing ability to filter which files are previewed should be not necessary (ideally).
I am inclined to close this, but first would you mind answering couple of questions?

Can you describe files with which you experience delay?
Does preview show 'Non-text-file-------' string or some random characters?

Hi, it seems that I can't consistently reproduce the problem, so I think I will close this issue for now.


Add "--tray" option to allow app to stay in the system tray

#304
Maybe we can improve this with a support of badges and counters, like to change the icon with an dynamically generated icon with the badge... (ref : https://github.com/electron/electron/issues/3148)

Changes icon dynamically using canvas...
a sample code:
const { createCanvas, loadImage } = require('canvas')
const canvas = createCanvas(image.width, image.height)
const ctx = canvas.getContext('2d')
ctx.drawImage(image, 0, 0, image.width, image.height)
var centerX = canvas.width / 2;
var centerY = canvas.height / 2;
centerX = centerX + (centerX / 2);
centerY = centerY + (centerY / 2);
var radius = 100;
ctx.beginPath();
ctx.arc(centerX, centerY, radius, 0, 2 * Math.PI, false);
ctx.fillStyle = 'red';
ctx.fill();
ctx.lineWidth = 5;
ctx.strokeStyle = '#003300';
ctx.stroke();
ctx.font = '165px Arial'
ctx.fillStyle = 'black';
var value = 6;
ctx.fillText(value, centerX - (value >= 10 ? 90 : 45), centerY + 60)
buffer = canvas.toBuffer()


Text Edit: Wrong cursor size if line height is set and return pressed

In TextField:

Set textStyle = TextStyle(lineHeight = 360.sp)
Press Enter (⏎)
Cursor is too large

fixed  in https://github.com/JetBrains/androidx/pull/277

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


MIZSelect() does not work with inlineformset created by mizdb-inlines

post_sub_level = forms.ModelChoiceField(
PostSubjectLevel.objects.all(),
widget=MIZSelect(PostSubjectLevel, url='mizdb_autocomplete_view'),
)

Please show where you are declaring that inline formset. Where is the code snippet from that you've posted? Where are you declaring that widget?

For reference, if  you use the InlineFormsetMixin from mizdb-inlines, something like this should just work:
class Pizza(models.Model):
    name = models.CharField(max_length=50)
    toppings = models.ManyToManyField("Topping")

class Topping(models.Model):
    name = models.CharField(max_length=50)

class PizzaView(InlineFormsetMixin, CreateView):
    model = Pizza
    fields = "__all__"
    template_name = "base.html"
    success_url = "/"

    formset_classes = [
        inlineformset_factory(
            Pizza,
            Pizza.toppings.through,
            fields="__all__",
            extra=1,
            widgets={"topping": MIZSelect(Topping, url="autocomplete")}
        )
    ]

With a template like this:
<!DOCTYPE html>
{% load static mizdb_inlines django_bootstrap5 %}
<html lang="en">
<head>
    <meta charset="UTF-8">
    <title>My Awesome Form</title>
    {{ combined_media }}
    <script src="{% static 'mizdb_inlines/js/mizdb_inlines.js' %}"></script>
    {% bootstrap_css %}
    {% bootstrap_javascript %}
</head>
<body>
<form class="container mt-3" method="post">
    {% csrf_token %}

    {% bootstrap_form form %}

    {% for formset in formsets %}
        {% inline_formset formset layout="horizontal" %}
    {% endfor %}
    {% bootstrap_button button_type="submit" button_class="success" content="Save" %}
    {% bootstrap_button button_type="reset" button_class="warning" content="Reset" %}
</form>
</body>
</html>

Ok thanks for your response. I have one issue in formset , widget MIZSelect not  working in added form of formsets. And filter_by option not working in formsets forms properly.

I have corrected my code


Add basic UI for itinerary page

Codecov Report
Base: 50.07% // Head: 49.22% // Decreases project coverage by -0.85% :warning:

Coverage data is based on head (0d706af) compared to base (f0e9288).
Patch coverage: 2.38% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #83      +/-   ##
============================================
- Coverage     50.07%   49.22%   -0.86%
  Complexity      422      422
============================================
  Files            97       99       +2
  Lines          1909     1944      +35
  Branches        199      205       +6
============================================
+ Hits            956      957       +1
- Misses          912      946      +34
  Partials         41       41

Impacted Files
Coverage Δ

...n/java/seedu/waddle/model/itinerary/Itinerary.java
54.20% <0.00%> (-3.80%)
:arrow_down:

...ain/java/seedu/waddle/storage/JsonAdaptedItem.java
0.00% <0.00%> (ø)

src/main/java/seedu/waddle/ui/ItemGroupCard.java
0.00% <0.00%> (ø)

.../main/java/seedu/waddle/ui/ItemGroupListPanel.java
0.00% <0.00%> (ø)

src/main/java/seedu/waddle/ui/MainWindow.java
0.00% <0.00%> (ø)

src/main/java/seedu/waddle/model/item/Day.java
13.51% <50.00%> (+2.08%)
:arrow_up:

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

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


[Java] StringView Initial Implementation

Describe the enhancement requested
StringView is already supported in C++, and providing support for Java users is important. Providing core StringView functionality via ViewVarCharVector and ViewVarBinaryVector is the objective.
Component(s)
Java

Issue resolved by pull request 40340
https://github.com/apache/arrow/pull/40340


Formatting numbers, dates etc

We've always said we would provide a module for formatting stuff. I have a partially implemented module called ceylon.format which makes a start. I need to get this finished.

Between the ceylon.locale module and the Integer.format() and Float.format() functions added in 1.3.1, do we still need a separate module? I could see maybe wanting one for custom date formats, if those should be allowed at all. (I've wished they weren't, in a previous job!)


Bug: Remove flicker when loading a series

Low priority:
When I select a tripos/part from the home page, the first series of the first module is loaded correctly, however during the loading flow the the main content screen goes black, then shows the home screen again then the series data is shown.
Ideally we should put in a simple loading bar for the main content page at the start of the loading sequence, and only remove it when the series data is loaded, eliminating the flicker.

Assigning to myself to have a look

Might be fixed with https://github.com/CUL-DigitalServices/grasshopper-ui/pull/292

Closing. I checked and this is fixed in #292 .