MLX_lm：generate() or stream_generate() how to setting system prompt and model config?

I attempted to utilize the mlx_lm library's generate or stream_generate functions to run the llm model, however, I faced challenges when setting up the model configuration.
I've examined the source code but couldn't find where the configuration is being set.
I noticed that some people have mentioned setting the configuration in server.py.
How to setting model config and system prompt to generate or stream_generate? or generate or stream_generate not support?
Use stream_generate code:
from mlx_lm import load, generate, stream_generate
model, tokenizer = load(
    path_or_hf_repo = "/Users/astl/RemoteFolder/LLM/LLM-Model/Meta-Llama-3.1-8B-Instruct-8bit/",
    )
prompt = """"Write a story about Einstein"""
for t in stream_generate(model, tokenizer, prompt, max_tokens=500):
    print(t, end="", flush=True)
print()

Use ServerRequests code :

python -m mlx_lm.server --model ~/RemoteFolder/LLM/LLM-Model/Meta-Llama-3.1-8B-Instruct-8bit
python main.py

# main.py
import requests
import json
from typing import List, Dict

# Function to send a request to the server and get a response
def get_response(
    server_url: str,
    messages: List[Dict[str, str]],
    temperature: float = 0.7,
    top_p: float = 0.9,
    max_tokens: int = 4096,
    stream: bool = True,
) -> str:
    headers = {"Content-Type": "application/json"}
    data = {
        "messages": messages,
        "temperature": temperature,
        "top_p": top_p,
        "max_tokens": max_tokens,
        "stream": stream,
    }

    # Send POST request to the server
    response = requests.post(
        f"{server_url}/v1/chat/completions",
        headers=headers,
        data=json.dumps(data),
        stream=stream,
    )
    response.raise_for_status()  # Ensure the request was successful

    if stream:
        content = ""
        for line in response.iter_lines():
            if line:
                decoded_line = line.decode("utf-8").lstrip("data: ")
                try:
                    json_line = json.loads(decoded_line)
                    if "choices" in json_line and len(json_line["choices"]) > 0:
                        delta = json_line["choices"][0].get("delta", {})
                        content_piece = delta.get("content", "")
                        content += content_piece
                        print(content_piece, end="", flush=True)
                except json.JSONDecodeError:
                    continue
        print()  # Ensure the next prompt starts on a new line
        return content
    else:
        result = response.json()
        if "choices" in result and len(result["choices"]) > 0:
            return result["choices"][0]["message"]["content"]
        else:
            return ""

# Function to run the chatbot
def chatbot(
    server_url: str,
    system_instructions: str = "",
    temperature: float = 0.0 , #0.7,
    top_p: float = 0.9,
    max_tokens: int = 12800, #4096,
    stream: bool = True,
):
    messages = [{"role": "system", "content": system_instructions}]

    prompt = """Write a story about Einstein"""

    messages.append({"role": "user", "content": prompt})
    print("Assistant: ", end="")
    response = get_response(
        server_url, messages, temperature, top_p, max_tokens, stream
    )
    messages.append({"role": "assistant", "content": response})

if __name__ == "__main__":
    server_url = "http://localhost:8080"
    chatbot(server_url=server_url)

Here is the code that sets the prompt when using the mlx_lm.generate CLI.
The prompt is expected to be set before you call the functions generate or stream_generate. Basically you have to do this:
messages = [{"role": "user", "content": prompt}]
prompt = tokenizer.apply_chat_template(
    messages, tokenize=False, add_generation_prompt=True
)

The mlx_lm.generate CLI will set the prompt for you using the model's chat template
The mlx_lm.server will apply the model's chat template given the messages field in the request. (The server also has a CLI arg which let's you specify the chat template to use --chat-template).

Let us know if you have any more questions about it.

Sorry for not checking the code carefully.
I understand your explanation but I need to use Python instead of CLI.
Thanks, I've figured out how to adjust the parameters.
The stream_generate function is working correctly with args.
from mlx_lm import load, stream_generate

model_path_or_hf_repo = "/Users/astl/RemoteFolder/LLM/LLM-Model/mlx-model/Meta-Llama-3.1-8B-Instruct-8bit/"
model, tokenizer = load(model_path_or_hf_repo)

system_instructions = """your system instructions"""
input_prompt = """your_input_prompt"""

messages = [{"role": "system", "content": system_instructions},{"role": "user", "content": input_prompt}]
prompt = tokenizer.apply_chat_template(
    messages, tokenize=False, add_generation_prompt=True
)

for response in stream_generate(model, tokenizer, prompt, max_tokens=4096, temp=0.3, top_p=0.9, repetition_penalty=1.5):
    print(response, end="", flush=True)
print()


Bing Maps broken?

The Bing Maps example is broken. What is up?
http://openlayers.org/en/master/examples/bing-maps.html

Looks like our key expired again
olc_11({"authenticationResultCode":"DeniedCredentials","brandLogoUri":"http:\/\/dev.virtualearth.net\/Branding\/logo_powered_by.png","copyright":"Copyright © 2016 Microsoft and its suppliers. All rights reserved. This API cannot be accessed and the content and any results may not be used, reproduced or transmitted in any manner without express written permission from Microsoft Corporation.","errorDetails":["The request was forbidden.  Your credentials may be denied or suspended."],"resourceSets":[],"statusCode":403,"statusDescription":"Forbidden","traceId":"1d54b91e760b4888b92d78b1b8cfd927|DB40051719|<IP_ADDRESS>|"})

Use your own key and you should be fine.

The Bing example works, can we close this issue?


How to get the wx_id when user had wx account?

1. Expected behavior
Question: What behavior do you expect?
Answer: Get wx_id from Contact.id not wx account
2. Actual behavior
Question: What actual behavior did you experience?
Answer: When user had set the wx account (ex: test), it will return 'test'. But, I want the wx id ,not the wx account.
Case, wx_id is unique, wx account can be changed

Question: Which puppet are you using?

Get wx_id from Contact.id not wx account

Yes, using Contact.id just can get contact unique id according to the puppet, see more:  #1644 #90 #1276 #133 #1307
If you want to get weixin id, you can use Contact.weixin(), but this isn't supported at this time, it is coming very soon! see #1647

If you are using iPad version, Contact.id is unique ID of and always the same id across sessions. It means when you login again, you always get this unique id.
If you are using Web version, Contact.id is unique id but it will change across sessions. It means when you login again, you may get a different id even it is the same user.

Thanks


device disconnected

i have read the FAQ and all this stuff but il took 3 charger and when i plug my phone with the 3 of them it still gets the same error they put disconnected device and i tried older and newer versions but it didn't work idk if there is way to fix but please help

Post the full console output.

the red is my name and that's what im getting

https://github.com/Genymobile/scrcpy/blob/master/FAQ.md#exception

what's that i don't see my error in this page

what's that i don't see my error in this page

On your screenshot, at android.media.MediaCodec.native_dequeueOutputBuffer(Native Method)

how can we put a lower definition

It's explained in the FAQ.
scrcpy -m 1024

If you don't know how command line works on Windows, it's also explained in the FAQ: https://github.com/Genymobile/scrcpy/blob/master/FAQ.md#command-line-on-windows

It works thanks but can i put a higher resolution but not so high because i want to stream with a good quality

thanks


Sidebar image not displaying correctly on Firefox (low-res)

please read the full discussion at mitmproxy/mitmproxy#1077

After seeing this too, upgrading firefox to version 46.0.1 fixed it for me.
Can you confirm?

@raffomania What did you use to test it? Mitmproxy has patched their doc image as stated in the linked issue discussion, so I have no test material.

I'm afraid this was for a private repo, so I can't give a link.
If no one else has experienced this, maybe we can just close it ;)

well don't just do that! Give me some time :+1:

Closing this, if you still have an issue please reopen this.


Flow passes / fails depending on where curried type annotation occurs

If I type the return type of hofAB as a function that returns a function of the type  () => {| ...T, meta: Object |} then flow complains.
Try
function hofAB<T: Object>(actionCreator: () => T): () => {| ...T, meta: Object |} {
  return () => ({
    ...actionCreator(),
    meta: {}
  });
}

type A = {
  type: "A"
};
type B = {
  type: "B"
};

type AB = {| ...A, meta: Object |} | {| ...B, meta: Object |};

const a: AB = hofAB(() => ({ type: "A" }))();

However if I annotate the returned function with the return type (dropping the () =>) {| ...T, meta: Object |} then flow is fine.
Try
function hofAB<T: Object>(actionCreator: () => T) {
  return ():  {| ...T, meta: Object |} => ({
    ...actionCreator(),
    meta: {}
  });
}

type A = {
  type: "A"
};
type B = {
  type: "B"
};

type AB = {| ...A, meta: Object |} | {| ...B, meta: Object |};

const a: AB = hofAB(() => ({ type: "A" }))();

And if I add both then flow complains again:
Try
function hofAB<T: Object>(actionCreator: () => T): () => {| ...T, meta: Object |} {
  return ():  {| ...T, meta: Object |} => ({
    ...actionCreator(),
    meta: {}
  });
}

type A = {
  type: "A"
};
type B = {
  type: "B"
};

type AB = {| ...A, meta: Object |} | {| ...B, meta: Object |};

const a: AB = hofAB(() => ({ type: "A" }))();

In this specific repro flow is complaining about the exactness of types; we were getting a lot of indexer property is missing errors but I couldn't get it to reproduce the error we were getting in a simple test case.
I'm guessing this it to do with scoping types over closures but it took aaaaages to figure out. Typically the errors were pretty unclear and pointed to weird call sites but also, at a glance (or even at a long stare) it's not clear why changing the types in this way fixed the issue.

Now it works if you annotate the return as requested by recent versions of flow


set_ciphersuites function should be static or renamed ssl_internal_set_ciphersuites

See ssl_ciph.c:1304
int set_ciphersuites(STACK_OF(SSL_CIPHER) **currciphers, const char *str)
This should be static and renamed ssl_internal_set_ciphersuites ...

I can make that change for you. However, there are other static int functions which don't carry the internal in their name. So I would leave the name unchanged if that's ok for you.

E.g. static int update_cipher_list_by_id(...) and static int update_cipher_list(...).

See #7253.


[3.6][Bug] MS SQL columns cannot be ordered

The columns are default non sortable.
I added 'orderable' => true to the column. Then the order icon is visible, but clicking on the column name does not sort the result.

I'm using backpack/crud 3.5.13 and laravel v5.7.19.

Wow - thanks a lot for getting to the bottom of this @howest-ward & @brianjohnhanna . As you know MSSQL is not officially supported by Backpack (only MySQL), but 98% of the features do work, and I know a lot of people use it with MSSQL nevertheless.
I'm glad to merge PR #2183, and bring this to 99% 😄
Thanks again for the PR @brianjohnhanna !
Cheers!


[BUG] Code coverage event based report is disabled

What is the bug?
Code coverage is broken for the repo. Report is showing some data, but latest commit that has been consumed is 6 months old: https://app.codecov.io/gh/opensearch-project/neural-search. Also recent PRs are missing this report, check out the most recent PR merged at time of writing https://github.com/opensearch-project/neural-search/pull/971

The PR has been merged. @martin-gaievski Can you check whether this issue has been resolved?

I want to keep this open for some time. I see for my PR coverage got updated, but one after PR has failed. Let's gather more info before we conclude if that is fixed or not

I'm seeing the coverage report got uploaded for some recent PRs (https://github.com/opensearch-project/neural-search/pull/978), I think we're good to close this issue


ddf.drop method fails if columns is an empty list

Hello,
I have a comment regarding the drop function at https://github.com/dask/dask/blob/master/dask/dataframe/core.py
    def drop(self, labels=None, axis=0, columns=None, errors="raise"):
        axis = self._validate_axis(axis)
        if (axis == 1) or (columns is not None):
            return self.map_partitions(
                drop_by_shallow_copy, columns or labels, errors=errors
            )
        raise NotImplementedError(
            "Drop currently only works for axis=1 or when columns is not None"
        )

In case columns is an empty list, we enter the  map_partitions fun and get the error 'metadata inference failed in drop_by_shallow_copy. which is not very explicit.
I don't know if it's the desired behaviour, but in case it's not, then a test on columns and an update of the NotImplementedError error msg could be more explicit

Thanks @Sylvain-chateau - I agree that a metadata-inference failure is not the desired behavior here.  I would expect dask to provide the same behavior as pandas: return a copy of the same DataFrame.
Did you have any interest in providing a PR to handle the columns == [] case in the code you quoted above?  If not, I should be able to follow up later today or Monday.


add more details about a line in the line profile page

extract data from json, and show it in a visually appealing way

line origin / destination
bus company logo
etc

see #193 for more ideas

Please assign this issue to me :)
@NoamGaash

Thanks @YahavSofer ! I'm happy you're taking this one :muscle:

@YahavSofer hi, how is it going?

@NoamGaash
Is this still relevant? Can I have this one?

@TuvalSimha Sure! Good luck

@TuvalSimha let us know where are you standing and if you need any help! If you're still on this issue, please let us know and you'll be re-assigned
Thanks!

is it still relevant? i need a bit more information.
it looks like the data from the JSON is already in use. (logo, line origin / destination)
So, do I need to use another API call to show the table with the bus schedule?

@dorinzrihen Hi Dorin, thanks for raising this question!
Actually, I don't really get the idea of that table. The schedule of the bus can (theoretically) change every day - so why bother with a table?
I do believe that having a list of stations can be nice, but the page looks way better compared to the screenshot made when this issue was originally opened thanks to the great work of many volunteers.
Actually, I don't see the planned route - do we have a bug?
Anyway I guess this issue is not very relevant - perhaps it would be better to move on and make similar page for operators (see #458).

this pi will get the stops list, if we want to show something fancy :)
https://open-bus-stride-api.hasadna.org.il/docs#/gtfs/list__gtfs_stops_list_get

@dorinzrihen nice to see you in this issue too :-)

@NoamGaash
Is this still relevant? Can I have this one?

@Tami-Co sure!

@NoamGaash
Could it be that this issue has already been resolved?
I checked several lines, and each time the details (logo, origin, destination) appeared.
Attached is a sample image.

@Tami-Co Oh, my bad. Seems you're right, the page is almost complete. Adding the list of stops can be a nice addition, but I guess it could be opened as a separate issue - anyway, would you like to try handling this task?
Also, seems like the map doesn't render nicely in the screenshot you've attached. Is this a bug? It seems like it is

@NoamGaash
I can take on the task of adding the stops.
Regarding the map, it was a one-time issue on my side. I checked, and it's fine now. I'm attaching a picture.

Cool, glad to hear that!
If you will manage to reproduce the issue in the future, opening a new issue about it with clear steps to reproduce the issue could be very helpful

@NoamGaash
Regarding the task of adding the station list, have you already opened an issue about this?
Also, I would appreciate more details on what the task should include.
thanks!

@Tami-Co I don't think such issue was opened, feel free to contribute an issue. I'm not sure what this list should include, for start we can show a plain html list of all stations, but the sky is the limit and any functionality can be added (for example, hovering a station name can highlight the same station on the map)


Toward hostname configuration consistent with bosh-dns

Bosh-deployed machines currently have /etc/hostname is set to the Bosh agent id.
This results in hostname syscalls (hostname, uname, cat /proc/sys/kernel/hostname) returning the Bosh agent id.
The Bosh agent id is also appended to the localhost line in /etc/hosts according to the template
<IP_ADDRESS> localhost {{ . }}
# The following lines are desirable for IPv6 capable hosts
::1 localhost ip6-localhost ip6-loopback {{ . }}
fe00::0 ip6-localnet
ff00::0 ip6-mcastprefix
ff02::1 ip6-allnodes
ff02::2 ip6-allrouters
ff02::3 ip6-allhosts

This results in hostname syscalls for the fully qualified domain name (FQDN) returning "localhost". This format is slightly off since Debian documentation states that the hostname should appear on a seperate line in /etc/hosts with the machine's external ip address. This means creating a seperate line in /etc/hosts with the ip address return of server.address and the contents of /etc/hostname.
When bosh-dns enters the picture, each machine has FQDNs that are resolvable by other hosts.
These FQDNs appear in the /etc/hosts file and server.address resolves to the FQDN or the machine's external ip address depending on whether use_dns_addresses is set to true or false. I think this behavior is inconsistent with Bosh hostname behavior.
Here is what I think the behavior should be when bosh-dns is enabled.

hostname syscalls should return the machine instance id
hostname syscalls for the FDQN should return <instance id>.<domain name>
server.address resolves to the FQDN <instance id>.<domain name> or the machine's permanent ip address depending on whether use_dns_addresses is set to true or false (This one is the current bosh-dns behavior)

This way, hostname syscalls for the FQDN and server.address match.
If backward compatibility could be completely disregarded, I would make the following changes to achieve this.
bosh-agent:

Set /etc/hostname to the instance id instead of the bosh agent id.
Remove the bosh agent id from /etc/hosts
Create /etc/hosts entry with the machine's external ip address and the instance id. This would read <ip address> <instance id>

bosh-dns:

Populate /etc/hosts so the above line reads <ip address> <instance id>.<domain> <instance id>

I realize there might be some reliance on the bosh agent id though. Therefore the folowing changes might be more palatable.
bosh-agent:

Remove dependencies on the hostname being the bosh agent id
Keep /etc/hostname set to the bosh agent id
Create /etc/hosts entry with <IP_ADDRESS> or <IP_ADDRESS> and the bosh agent id
Remove the bosh agent id from the "localhost" line in /etc/hosts
Create /etc/hosts entry with the machine's external ip address and the instance id

bosh-dns:

Set /etc/hostname to the instance id instead of the bosh agent id.
Populate /etc/hosts so the above line reads <ip address> <instance id>.<domain> <instance id>

The latter course of action makes sure the bosh agent id is still resolvable. It also stops the current behavior of FQDN requests returning "localhost". Then, if bosh-dns is being used, hostname behavior is consistent with the rest of bosh-dns.
This is from the Linux point of view (specifically debian), so I'm unsure what actions are needed to stay consistent across operating systems. I am also aware that this has been brought up here before, but I thought I would formalise it a bit more.

+1 I haven’t thought through all the trade offs on the proposed changes but I’ve been meaning to say for a while that mapping hostname to localhost is a no-no that has caused me misery in the past

one of the challenges we have with setting hostname is that it's not clear what we should set it to since there could be multiple networks configured for each instance. concept of fqdn doesnt translate well in this scenario. additionally we have more advanced dns functionality for doing certain level of filtering, etc. so it would be problematic to hardcode just one configuration for fqdn.
@mlmitch what do you think about job specifically setting hostname and bosh-agent relinquishing ownership of it once it sees that it's different from the agent-id (ie making bosh-agent only set it on the first time)? job could run hostname <%= ... %> though im still trying to figure out how to best set hostname -f...

@cppforlife you've mentioned the issue of multiple networks before. It makes sense to me that you would set it in the same manner as the <%=server.address%> return when use_dns_addresses is set to true. Although, I don't know how that is set when there are multiple networks.
I like the idea of a hostname job or add-on for now. That might enable any changes to be opt-in.
Assuming the return of <%=server.address%> is going to be the FQDN, the instance id should be the hostname and it should be appended to the end of the /etc/hosts line that contains <%=server.address%>. That would correctly set hostname -f or any other equivalent syscalls.
Regardless of the above, I consider returning "localhost" for the FDQN to be a bug. Though, that one is easily fixed by putting the agent id on a separate line in /etc/hosts, even if it still resolves to the same ip address.

Bosh documentation on multi-homed VMs indicates a possible approach. A hostname value could be added to the network's default property. Then the hostname would be chosen in a similar way to gateway or dns which are similarly network specific things.

I've created a pull request to solve FQDN requests returning "localhost".

We're running into a similar issue with any machine deployed with Bosh returns localhost as the hostname of the machine. An unfortunate side affect of this especially for monitoring within a cluster of EC2 instances is that there's no way to differentiate between the various nodes. This has hampered our ability accurately assess a failing node. Working with NewRelic support team they have mentioned that we could override the /etc/hosts file to fix it but unfortunately since the file is managed by Bosh any manual changes do get overridden when a deployment occurs.
e.g. https://docs.newrelic.com/docs/agents/java-agent/troubleshooting/host-links-missing-java-apps-apm-overview

We'll track this via Story #159974467 and try to discuss it a bit more soon.

This is breaking New Relic monitoring on Java for us as well.  Can this be prioritized for a fix soon?

It would be very helpful to make the FQDN configurable with BOSH DNS as some software relies on this being a constant value. E.g. mongodb uses the FQDN to determine it's identity in a replicaset.

Any ETA for this yet?  We have production workloads we can't fully monitor in New Relic and it seems like this is a fairly trivial fix in the BOSH automation (just need to add a name into /etc/hosts).

Hey @dpb587-pivotal - any update for this yet your original ticket said that you would discuss it more?

cc @mgadiya @luan as this is in the realm of your work and I'm not sure where it falls in priorities. The issue description gives a pretty good summary of needs, the earlier-referenced story refers to another Slack discussion around this, and the suggestion of using default is probably something to keep in mind if you're considering this. Let me know if I can help provide context or additional ideas.

has there been any progress on this?

I'd actually prefer if one can overwrite the hostname and fqdn. E.g. the network is shared but the vms might belong to different domains.


Death Knight strike

The  death knight blood macro keeps changing heart strike to rune strike on its own and there is no rune strike in the game.  It does the same thing with festering strike with unholy death knight

This is not an issue.  It’s doing it on purpose for a reason.  See https://wowlazymacros.com/t/two-most-asked-questions-base-spells-and-macro-locks/17772


Fix/smartbond timer counter test build fails

This PR is a hotfix and may not be the ideal.
Fixes #73685
I'm not sure why, but setting CONFIG_SMARTBOND_TIMER=n in a .conf or on command line is not able to disable the Kconfig. The only way I could get it to disable was by changing that default case in the Kconfig.defconfig as seen in the PR.

A "hotfix" for a bug labelled as low priority seems a bit too streched.
I think it may be better to let the assignee, @ioannis-karachalios , review.

A "hotfix" for a bug labelled as low priority seems a bit too streched. I think it may be better to let the assignee, @ioannis-karachalios , review.

my understanding is priority has nothing to do with it, hotfix is for issues blocking development, which this is, ci is failing on PRs

@ioannis-karachalios @andrzej-kaczmarek @blauret can any of you please take a quick look at this PR?


TOC reorganization

@sascala I've cherry-picked your changes and added some additions:

renamed sections: Using Marathon vs. Administration vs. Development
moved articles to the (most) appropriate  section
restored alphabetical order (as before)
fixed markdown metadata syntax

Can you please check if I missed something?

Thanks for your changes, @meichstedt. I'm not sure about alphabetizing the TOC sections -- I tried to order the topics according to logical workflow (eg, install --> configure --> upgrade).
BTW the other changed files in this PR are TOC-related -- I created some separate troubleshooting pages and made a few other tweaks.

OK, in the interest of getting the revised TOC published, I'll table the alphabetical order question for the moment.
Assuming that @meichstedt 's Readiness Checks document is also ready for publication, could you please merge, @aquamatthias ?

Thanks!


Add ui to display all companies

fixes #30

@gruz0
Would it be better to show the archived date instead of the status?

@Aditya-A-G, please display only the following fields: display name, comment and status. Also don't forget to add "edit" button like in EmployeesList component. I think created_at, updated_at and archived_at are not required right now, we can omit them in the interface :-)


Not able to pass file name as regex in npx npx github-actions-ctrf command

@Ma11hewThomas I am using Windows OS and electron project for WebdriverIO. I am able to generate ctrf file which is having format ctrf/ctrf-report-end-to-end-login.spec-2024-07-05T05-57-42-778Z.json.
I want to use command like  npx github-actions-ctrf ctrf/ctrf-report-*.json but i am getting error Failed to read file: Error: ENOENT: no such file or directory, open 'ctrf/ctrf-report-*.json'
I also tried to fix file name by using below option but it didn't generate file name as ctrf-report.json but still generating file name as ctrf-report-end-to-end-login.spec-2024-07-05T05-57-42-778Z.json
reporters: [
	['ctrf-json',
		{
			outputFile: 'ctrf-report.json',
			outputDir: 'ctrf',
		}
	]

],

Please suggest.

Hi @lalitsvm,
Currently this package only supports a single ctrf report.
Due to the architecture of WDIO, the filename can not be defined, you have two options -
If running in parallel and generating multiple ctrf reports:
The ctrf-cli package provides a method to merge the multiple json files into a single file and you can state the file name.
If always generating a single report, use a bash script or similar to find and rename the file.
Thanks


Python >= 3.9 support

Hi
First of all, really like your implementation of hierarchical classifiers. Bravo. I am constrained by other package requirements to have python >= 3.9.
Would it be possible to bump the python version requirements or is there any major blocker for this?

Hi @flo-ri-an
Thanks for the interest in HiClass. One of the dependencies, Ray, was incompatible with Python 3.9 a couple of months ago but it seems that it is compatible now. Will give it a try and let you know if it works.
Kind regards,
Fabio

Fortunately it worked with python 3.9 now. With python 3.10 it cannot find any installation candidate for Ray so at least for now not possible.

Cool, thank you so much for the fast update!


Message retention policy (realm-global)

This is a rebase of #2172.  Notes on things I still want to do with this:

[ ] Use RETURNING or something on functions like move_expired_user_messages_to_archive to get the list of message IDs moved to delete_expired_messages and friends not ridiculously slow once this has been running for a while. (Currently, it sends the list of all expired message IDs to the database every time, not just the ones it just deleted...).  I think the concept here is laid out pretty clearly in https://github.com/zulip/zulip/pull/2172#issuecomment-288959720.
[ ] Do the efficient cross-realm bot logic suggested in that comment as well.
[ ] Maybe add some sort of auditing assertions; more thought required here.
[ ] We have a lot of blocks that look like this in the tests:

+        for realm_instance in Realm.objects.filter(message_retention_days__isnull=False):
+            move_expired_messages_to_archive(realm_instance)
+            move_expired_user_messages_to_archive(realm_instance)

that basically are running half of the archive_messages code; we should instead split archive_messages into two functions and call those two helper functions in these test code paths.

[ ] There is a weird pattern in the tests of making local variables like archived_attachment = ArchivedAttachment.objects.all() that we should remove (and just name the table each time).
[ ] ￼ + INNER JOIN zerver_userprofile ON zerver_usermessage.user_profile_id = zerver_userprofile.id
[ ] We need to archive any other tables with Message foreign keys added since, e.g. Reaction
￼
It seems like this would archive messages where are least one recipient has been archived, which is likely not correct in the case of a group private message with users in multiple realms; or am I missing something?  We should add a test for this possibly.

Seems like an important corner case.

@mateuszmandera I think this PR has nothing of value not already integrated, so closing.


Fullrow Select disable

Hello,
thank you so much for this amazing tool !
I finally have been able to set columns as I always wanted to, instead of trying to script registry tweaks that are often unreliable.
The only thing I've not been able to change (yet) is the fullrow select which I've always hated. On W7 I managed to do it following this post: https://superuser.com/a/1083303
On W10-11 is more complex and doing it by hand messes up the shell bags and causes me continuous explorer crashes (with drag & drop and/or dealing with the trash bin). I suspect the crashes are related to messing up with shell bags but I'm not 100% sure. It's really hard to troubleshoot this issue and I ended up reinstalling the OS many times.
A tested working option would be using Ultimate Windows Tweaker 4 (for W10) and Ultimate Windows Tweaker 5 (for W11). But I dream of a scriptable solution and WinSetView seems to be capable of doing that easily.
Is there a way to your knowledge to disable FullRow select via WinSetView ?
Thanks again and
have a nice day,
V.

I looked at this once before and concluded it caused problems with other settings, but I don't recall the details. I'm taking another look at it now and I'll see if I can add an option. I did notice that the GUID {57807898-8C4F-4462-BB63-71042380B109} only shows up in Bag node entries on Windows 7 and appears to be a unused leftover (maybe from XP) because it references nothing and never appears on Windows 10 or 11.

I'm curious. I couldn't see any functional difference between full row select enabled or disabled. Is it just a cosmetic thing? For me, I like seeing the highlight all the way across. Am I missing something?

Interesting. The row spacing is tighter when full row select is enabled, especially in List view. Also, those FFLAGS enable free (temporary) arranging in Details and Icons views. At least on Windows 10. I still have to test Windows 11.

Version 2.90 is now available to download. You will see a new option:
Legacy row and icon spacing
Check that and then you will also see:
Disable full row select
See the manual for details.

Amazing speed ! Thank you sooo much !!!

You're welcome. 🙂


feat: added navbar loader

PR Fixes:

Added Navbar loader in CMS

https://github.com/code100x/cms/assets/22127725/814421b4-3688-4c5d-930e-933711860e82
Resolves : Added Navbar loader in CMS which was discussed in live stream.
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

/bounty $5


Unable to set log level on default logger

Because logger is a private type, we're unable to adjust the log level of the default logger:
keygen.Logger.Level = keygen.LogLevelDebug

We should add a setter to LeveledLogger.

Closed in https://github.com/keygen-sh/keygen-go/commit/5049fa5f511db893bc42b5873bb69b42661e1346.


[AutoPR security/resource-manager] Adding more documentation to ASC Pricing API

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6691

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/5434


Support player.twitch.tv-style URLs

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.12.22. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[X] I've verified and I assure that I'm running youtube-dl 2016.12.22

Before submitting an issue make sure you have:

[X] At least skimmed through README and most notably FAQ and BUGS sections
[X] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[X] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

Description of your issue, suggested solution and other information
This is sort of a mix between a bug report and a site request. Basically,  youtube-dl already suppports twitch.tv videos fairly well, but I noticed one important caveat:
URLs like https://player.twitch.tv/?video=v109751819 don't get detected by youtube-dl as valid twitch URLs.
It works if I resolve that URL to https://www.twitch.tv/mmsrhino/v/109751819 (which the player.twitch.tv frame links to), but the unfortunate part is that this isn't a purely mechanical transformation since it relies on knowing the channel name.
It would be great if youtube-dl could support the player.twitch.tv-style URLs as well, since they're often embedded on websites such as speedrun.com.

Thanks!


update to mozim 0.2.1

A fixes for the breaking changes:

timeout is u32
DhcpV4Config::new_proxy() no longer return an Result type

This also fixes the build issue on i686.
Fixes #578

@mheon @flouthoc PTAL

LGTM

/lgtm
/approve


Tectonic Console Failure Over Time

Tectonic Version
1.8.4-tectonic.3
Environment
VMware ESXi 6.5 U1
Expected Behavior
The Tectonic Console will remain operational over time.
Actual Behavior
On a daily basis the Tectonic Console will fail... the nearest relevant log entries seem to show that the PODS are having issues with DNS resolution.
The logon prompt will appear, you enter the correct credentials, and then it recycles.
No error is shown, though I suspect it is buried in the re-direct.  It seems to be due to the failed handshake between the Tectonic Identity and the Console.  But this may be a red herring as the Console seems to have DNS issues, but only eventually...
kubectl (and thus I assume the API) continue to work unaffected.
Numerous people have reported seeing this issue, but it is always confused by the login prompt re-appearing - some have taken this as a user/password issue.  Also a red herring.
See #192, #237, #232 and #192.
There is a common thread of having a fully functional cluster one day, doing nothing, and then returning to have this issue.  As a relative newcomer, I am also experiencing it after initial installs that otherwise appear to be fully functional.  Frustrating to diagnose as the usual channels produce no useful output.  journalctl -xe on the controller offers nothing.
kubectl logs on the Identity server showed nothing wrong.  The only clue I could find was the Tectonic Console POD logs...
2018/02/12 15:57:44 auth: unable to verify auth code with issuer: Post https://core-ingress.idstudios.local/identity/token: dial tcp: lookup core-ingress.idstudios.local on <IP_ADDRESS>:53: no such host
2018/02/12 15:57:44 server: no token found for apis/apps/v1beta2/daemonsets: token not found in request (Sec-WebSocket-Protocol or Authorization Header)
2018/02/12 15:57:44 server: no token found for apis/apps/v1beta2/deployments: token not found in request (Sec-WebSocket-Protocol or Authorization Header)
2018/02/12 15:57:44 server: no token found for /api/tectonic/crds: token not found in request (Sec-WebSocket-Protocol or Authorization Header)
2018/02/12 15:57:44 server: no token found for /version: token not found in request (Sec-WebSocket-Protocol or Authorization Header)
2018/02/12 16:09:06 server: no token found for /api/tectonic/crds: token not found in request (Sec-WebSocket-Protocol or Authorization Header)
2018/02/12 16:09:06 server: no token found for apis/apps/v1beta2/deployments: token not found in request (Sec-WebSocket-Protocol or Authorization Header)
2018/02/12 16:09:06 server: no token found for api/v1/proxy/namespaces/tectonic-system/services/prometheus:9090/api/v1/query?query=count(increase(kube_pod_container_status_restarts%5B1h%5D)%20%3E%205): token not found in request (Sec-WebSocket-Protocol or Authorization Header)

Which seems to suggest an issue down in the virtual networking DNS resolution.
Reproduction Steps
The install is based on the Tectonic Graphical Install for Bare Metal, which completes successfully.

Deploy PXE based Tectonic CoreOS cluster
Verify that the cluster is functioning as expected
Deploy workloads the cluster, configure ingress... all will be fine.  I've deployed both Stateful backends and web servers, all running fine.
Wait a day.

At the point in time where the Tectonic Console can no longer resolve the IP for:
 lookup core-ingress.idstudios.local on <IP_ADDRESS>:53: no such host

The name core-ingress.idstudios.local resolves just fine from the CoreOS nodes.  So it is something farther down in the bowels of the Virtual Network that is failing... and I am at a loss.
Workaround
Rebooting my single controller (master) fixes the issue.  It seems to be the only fix that works.  If I try killing the Tectonic Console PODS, they will not restart... and I will begin to get "502 Bad Gateway" errors from Nginx.
Hard to know where to begin looking for something that just breaks over time with little error output.  Be nice if the Tectonic Console could spit out an error clue.

This seems to be tied to Dex key rotation.  I can trigger the issue after a key rotation by logging out of the browser session for Tectonic console.

I think I solved this one... DHCP configuration issue I believe... though I'm not certain I didn't just mask the problem with longer lease times.
Two things may have contributed:

I had left a small pool of dynamic addresses available, in addition to the static reservations for the nodes.  For some reason, the VMware bridged vnics would try to pull a dynamic IP before getting assigned the static one.  It was only evident in the dhcpd server logs.  I think this may have confused the virtual network layer that sits on top of the bridged nic.  I removed this pool from the Bootp DHCP server.
The default lease time was 7200s... which I changed to 90 days :)

Since those changes the Tectonic Console has worked as expected, and I've seen key rotation without losing the ability to log back into the console.  I expect the latter change made the difference - won't know for 90 days :)
Seems these sorts of issues always relate to the underlying DNS system in some way.  Might be good to have something in a FAQ, as it is so easy to mistake the symptoms for being login related.
Thanks @aaronlevy for the kind cc on this.  Tectonic CoreOS is amazing.
After spending the past year building Docker CE/EE Swarms (and the IaC to support them), I really appreciate the polish in Tectonic.  I hope RedHat doesn't mess it up :)

@seanhig where exactly did you set the default lease time ? Thanks


chore(callback): Remove old peft saving code

Previously, peft does not save the adapter, hence we needed this code. After peft updated a while back, this callback unnecessarily saves the adapter another time.
We can now remove it.

Per winglian's comment in Discord, relora might need this code. I think I need some confirmation what relora usually saves at as I haven't trained one before. And if it's necessary for only relora, we can change the condition for only then.

Seems like relora already saves its own adapter_model

Rebased!


WebApp:webapp up compile failure for some directory structures

Describe the bug
The compile step of az webapp up fails if a csproj references a sibling folder -- that is, if the reference is ../SomeClassLibrary.
For example, consider this solution structure:
BugTestSln
    ClassLib
    Server

If Server code references ClassLib the build and deployment will fail.  Unlike dotnet build, the build process used here seems to assume that all relevant files are in the Server directory, or subdirectories.
Among the log messages are:
Skipping project \"D:\\local\\Temp\\zipdeploy\\ClassLib\\ClassLib.csproj\" because it was not found.
Program.cs(9,7): error CS0246: The type or namespace name 'ClassLib' could not be found (are you missing a using directive or an assembly reference?) [D:\\local\\Temp\\zipdeploy\\extracted\\Server.csproj]
Done Building Project \"D:\\local\\Temp\\zipdeploy\\extracted\\Server.csproj\" (default targets) -- FAILED.

To Reproduce
Set up a solution.  Begin by executing the following on a commandline.  They should all succeed.
mkdir BugTestSln
cd BugTestSln/
mkdir Server
mkdir ClassLib
dotnet new sln
dotnet new classlib -o ClassLib/
dotnet new webapi -o Server/
dotnet sln add Server/Server.csproj
dotnet sln add ClassLib/ClassLib.csproj
cd Server/
dotnet add reference ../ClassLib/ClassLib.csproj

Now modify Program.cs and add the following line near the top (with the other using statements):
using ClassLib;

Observe that the solution compiles, using the following command:
dotnet build

Observe that the program runs correctly locally by running as follows,
dotnet run

And then point a web browser to the following address:
http://localhost:5000/weatherforecast

Now try to deploy to Azure.  Here's where things go wrong:
az webapp up --sku F1 --name BugTestServer11235

You'll get a failure message, similar to:
The webapp 'BugTestServer11235' doesn't exist
Creating AppServicePlan 'jesse_asp_Windows_centralus_0' ...
Creating webapp 'BugTestServer11235' ...
Configuring default logging for the app, if not already enabled
Creating zip with contents of dir C:\Users\jesse\tinker\BugTestSln\Server ...
Getting scm site credentials for zip deployment
Starting zip deployment. This operation can take a while to complete ...
Deployment endpoint responded with status code 202
Configuring default logging for the app, if not already enabled
Zip deployment failed. {'id': '8501c6cb58b348aaa41a7bcc9e8331ee', 'status': 3, 'status_text': '', 'author_email': 'N/A', 'author': 'N/A', 'deployer': 'ZipDeploy', 'message': 'Created via a push deployment', 'progress': '', 'received_time': '2020-12-30T15:53:59.3811158Z', 'start_time': '2020-12-30T15:53:59.5854641Z', 'end_time': '2020-12-30T15:54:26.1331Z', 'last_success_end_time': None, 'complete': True, 'active': False, 'is_temp': False, 'is_readonly': True, 'url': 'https://bugtestserver11235.scm.azurewebsites.net/api/deployments/latest', 'log_url': 'https://bugtestserver11235.scm.azurewebsites.net/api/deployments/latest/log', 'site_name': 'BugTestServer11235', 'provisioningState': None}. Please run the command az webapp log deployment show -n BugTestServer11235 -g jesse_rg_Windows_centralus

Expected behavior
The app should deploy successfully.  Or, if the system won't support this directory structure, it should fail gracefully and direct the user to a web page explaining what is and isn't supported.
Environment summary
azure-cli 2.17.0
dotnet 3.1.300
Windows 10 Pro 1909
mintty 3.1.6

webapp

Thanks for reporting.Will look into this.

@calvinsID  can you take a look at this?

Hello. Is there any update on this issue?

This scenario is not supported in az webapp up, as it assumes csproj is in the same directory/subdirectories. We should add better error handling to gracefully error

Closing since the associated work item has been merged. Please re-open if there are still issues.


cannot load products in safari, js has errors

When visiting products/category/blah/ in safari (tested in iphone, ipad, windows), the products in that cateogy are not listed. There are some errors in the js.

Could you share content of these errors?

Please see the upon image. BTW, the safari used in windows is 5.1.7, ios is 8.3.

@osess Thanks for reporting this issue. We've just merged PR #838 that should fix it.


[BUG] Azure KeyVault: Expecting element 'root' from namespace ''.. Encountered 'None'  with name '', namespace ''.

Describe the bug
I've been experiencing issues with acquiring an Azure Authentication token in Visual Studio 2019. While I appear to be able to acquire tokens for other services (e.g. ADLS) with no issue, Azure Key Vault fails to correctly utilise a token provided using ServiceClientCredentials.
I have attempted to authenticate using both Azure CLI and the Visual Studio built-in authentication mechanism, but both return the same exception.
Expected behavior
Key Vault client will authenticate successfully using client credentials.
Actual behavior (include Exception or Stack Trace)
[3/2/2020 10:24:26 PM] Parameters: Connection String: [No connection string specified], Resource: https://vault.azure.net, Authority: . Exception Message: Tried to get token using Visual Studio. Access token could not be acquired.
[3/2/2020 10:24:26 PM] Exception for Visual Studio token provider Microsoft.Asal.TokenService.exe : Token response is not in the expected format. Exception Message: Expecting element 'root' from namespace ''.. Encountered 'None'  with name '', namespace ''.
[3/2/2020 10:24:26 PM] Parameters: Connection String: [No connection string specified], Resource: https://vault.azure.net, Authority: . Exception Message: Tried to get token using Azure CLI. Access token could not be acquired. Token response is not in the expected format. Exception Message: Expecting element 'root' from namespace ''.. Encountered 'None'  with name '', namespace ''.
[3/2/2020 10:24:26 PM] .

To Reproduce
Attempt to acquire an AzureServiceTokenProvider while debugging in Visual Studio 2019.
Environment:

Name and version of the Library package used: [e.g. Azure.Storage.Blobs 12.2.0] Microsoft.Azure.KeyVault v3.0.5
Hosting platform or OS and .NET runtime version (dotnet --info output for .NET Core projects): Windows 10 .NET Core 2.2
IDE and version : Visual Studio 16.4.5

I've only encountered this issue while remoting into my workstation via RDP, not sure if it's a red herring but just in case that info is useful...

I see that MSAL is failing to parse a document. If this only happens when you RDP in, I wonder if the path is resolved differently. Could you run procmon and see where it's looking in both cases?
Another option is to upgrade to Azure.Security.KeyVault.* version 4. These are our new libraries under active development that are faster with fewer allocations, more extensible (logging, HTTP pipeline, etc.), and easier authentication. In this case, however, we don't yet have VS authentication but that is coming very soon.
We're only making critical fixes to Microsoft.Azure.* package where Azure.* packages exist, though it seems in this case if this is an MSAL bug we'd need to get a fix and take a new version if that is indeed the case.

Thanks for taking this up @heaths! I think the paths are being resolved correctly here - the MSAL path (%LOCALAPPDATA%\Local.IdentityService\msal.cache) appears to be valid on RDP when checking with procmon.
I could try the new SDK - do you reckon it would address this particular issue?

Okay - I tried using the new KeyVault SDK and authenticated using Azure CLI az login` (is this supported?) and received the following exception instead:
Service request failed.
Status: 400 (Bad Request)

Content:
<!DOCTYPE html>

        <title>Runtime Error</title>
        <meta name="viewport" content="width=device-width" />

Server Error in '/' Application.

 Runtime Error

             Description: An application error occurred on the server. The current custom error settings for this application prevent the details of the application error from being viewed remotely (for security reasons). It could, however, be viewed by browsers running on the local server machine.

            Details: To enable the details of this specific error message to be viewable on remote machines, please create a <customErrors> tag within a "web.config" configuration file located in the root directory of the current web application. This <customErrors> tag should then have its "mode" attribute set to "Off".

<!-- Web.Config Configuration File -->

<configuration>
    <system.web>
        <customErrors mode="Off"/>
    </system.web>
</configuration>

            Notes: The current error page you are seeing can be replaced by a custom error page by modifying the "defaultRedirect" attribute of the application's <customErrors> configuration tag to point to a custom error page URL.

<!-- Web.Config Configuration File -->

<configuration>
    <system.web>
        <customErrors mode="RemoteOnly" defaultRedirect="mycustompage.htm"/>
    </system.web>
</configuration>

Headers:
Cache-Control: private
Server: Microsoft-IIS/10.0
X-AspNet-Version: 4.0.30319
X-Powered-By: ASP.NET
Strict-Transport-Security: max-age=31536000;includeSubDomains
X-Content-Type-Options: nosniff
Date: Tue, 03 Mar 2020 01:57:26 GMT
Content-Type: text/html; charset=utf-8
Content-Length: 3420

Looks like a generic 5xx response, despite the 400 response code. I'll try using a service principal as well, but perhaps a more informative exception would be helpful here.

Nevermind - I tried using the recommendation of an AAD application, but received the same result.

Azure.Security.KeyVault will support az login and VS very soon. We'll have news to share shortly. I will have to track down owners for ASAL as this seems to be a problem with their library.

Sounds good. I think this has something to do with my connection to my work environment. If it helps, here's how I'm connected:
Home PC (not domain-joined to MSFT) > Hyper-V VM (MSFTVPN connected) > RDP to work computer (in REDMOND domain)

@nonik0 could you have a look, please? Looks like ASAL is having problems parsing a file when a user is RDP'd into a session.

@heaths @nonik0 I've done some further investigation, and have the following useful info:

I went into the office today and tested my code on my desktop (without RDP/VMs) and encountered the same issue. So I doubt that RDP is actually the culprit here.
This issue only appears to be affecting my local environment; the same binaries deployed to Azure work with no issue.
I tried running my code on a different machine (my laptop) and encountered the same issue.
I only started encountering this issue recently, despite making no code changes to our key vault client since the issues began.

This is starting to look more like a service error, rather than a client error. Thoughts?

The client is trying to get cached access tokens before it can authenticate to the service. I'm wondering if maybe Visual Studio took an update and broke the format that this version of ASAL is expecting. Does az login work for you instead?

az login doesn't appear to work here, unfortunately - the same error is returned, which is somewhat suspect as the access tokens in this instance would be different, right?
I'm wondering if the Internal Service Error message I'm receiving on the new Key Vault client is the underlying issue, and ASAL is just failing to parse the page as it is invalid JSON/XML.

From your original stack,
[3/2/2020 10:24:26 PM] Parameters: Connection String: [No connection string specified], Resource: https://vault.azure.net, Authority: . Exception Message: Tried to get token using Visual Studio. Access token could not be acquired.
[3/2/2020 10:24:26 PM] Exception for Visual Studio token provider Microsoft.Asal.TokenService.exe : Token response is not in the expected format. Exception Message: Expecting element 'root' from namespace ''.. Encountered 'None'  with name '', namespace ''.

Are you getting a different stack now?
It seems like it's having problems parsing a file (likely XML; I really don't know, though). I know, conceptually, there are a cache of files that contain access tokens we try to reuse to send to services. Apart from that, @nonik0 would need to take a look.

The original stack is what appears on Microsoft.Azure.KeyVault, and the internal service error I'm referring to here appears on Azure.Security.KeyVault.

I did mention above that,

In this case, however, we don't yet have VS authentication but that is coming very soon.

So no authentication credential is sent for Azure.* yet, but will be very soon.

That's true, though I did attempt to use AAD application authentication and received the same result:
    "AZURE_CLIENT_ID": "812xxx",
    "AZURE_CLIENT_SECRET": "xxx",
    "AZURE_TENANT_ID": "72fxxx"

Without sample code, it's hard to say. Did you use a DefaultAzureCredential and were those environment variables set such that the process picked them up. If you set them on your machine but without restarting VS (if you're debugging/testing your app), they won't get used. We use service principals in our live tests every night, as well as ad-hoc testing.
But that result page you got doesn't look right either. What are you passing for the vault URI?

I tried using DefaultAzureCredential, and I believe so - I'm using the same mechanism provided by local.settings.json that works fine for other environment variables and picks up the latest values on each run.
The key vault URL I'm testing on is: https://InsertVaultNameHere.vault.azure.net

I'll need to see more: either a sample projects, or you can contact me internally with project details. You could also enable tracing, which in Azure with Application Monitoring enabled should be automatic; otherwise, see https://heaths.dev/azure/2020/02/04/trace-azure-sdk-for-net.html.

Okay, I found out why the odd Internal Server Error response was being received - I hadn't updated GetSecretAsync from the old interface of (string vaultBaseUrl, string secretName) to (string name, string version). Perhaps there was a parsing error with the inputs here - some user validation might make sense here on the client end to make code upgrades from the old library to the new one smoother.
The environment credentials seem to work on the new SDK after all, which is good to know! Where can I track the use of Azure CLI/Visual Studio authentication being made public, by the way?

@jublair , I tried this using Microsoft.Azure.KeyVault v 3.0.5, Microsoft.Azure.Services.AppAuthentication v 1.4.0, Visual Studio 16.4.5 and it worked fine for me.
Could it be a transient issue? Can you try again?
This is the code I used:
var azureServiceTokenProvider1 = new AzureServiceTokenProvider("RunAs=Developer; DeveloperTool=VisualStudio");
var kv = new KeyVaultClient(new KeyVaultClient.AuthenticationCallback(azureServiceTokenProvider1.KeyVaultTokenCallback));
Console.WriteLine(await kv.GetSecretAsync("https://myvault.vault.azure.net/secrets/mysecret/8bfe9cbc568b49988155969ba628a800"));

You might track it through https://github.com/Azure/azure-sdk-for-net/issues/8934; otherwise, @schaabs, do you have an epic tracking this?

Thanks for the help folks! Turns out this was a transient issue with (perhaps) AAD issuing bad tokens. Either way, the issue appears to have subsided, so I'll close out this issue and monitor the provided issue to track when Azure CLI will be available on the new package.


Guides can be filtered by state, user, content owner, or keyword.

It looks good generally. A few comments:

Ideally this would be split into two commits, one for switching to sql schema, one for adding the feature
We use "Published by" and "Content Owner" as synonyms. I think it would be nice to keep calling it :content_owners at the code level and only use the term "Published by" on for labels.
Would be cool to have a test that proves editions other than the latest edition do not get searched e.g. you have a guide that has been published in the past but currently has a draft state. It should not show up when searching for 'published' guides etc.


New project structure

This pull request contains the project structure needed for the new architecture discussed in #31
General Changes

Introduce seperation of frontend and backend
Introduce Docker (files still empty)
Created docs section which outlines the targeted architecture and design decisions made

Frontend Changes
I didnt touch the frontend as this is not my area of expertise.
Backend Changes
Switch to uv as a package manager
I switched from Poetry to uv as a package manager, as it is the current best solution. Following a comparison between uv and poetry:
Benefits of uv

Speed: Faster installation and resolution due to Rust implementation.
Simplicity: Easy transition from pip with a straightforward interface.
Unified Tooling: Combines multiple tasks into one tool.

Drawbacks of uv

Limited Features: Lacks some advanced features found in poetry.
Maturity: Newer tool with potentially less stability and documentation.
Compatibility Issues: May require adjustments for existing poetry projects.

Benefits of Poetry

Comprehensive Management: Excellent dependency resolution and handling.
Native Dependency Support: Better for projects needing compiled modules.
Established Ecosystem: Strong community support and extensive documentation.

Drawbacks of Poetry

Performance Overhead: Slower than uv in some cases.
Complexity: More features can lead to unnecessary complexity for simple projects.

Add Ruff for linting
I added Ruff and some basic ruff config as a python linter as it is the commonly agreed best linter currently available. Some reasons on why to use Ruff:

Speed: 10-100x faster than traditional linters.
Consolidation: Replaces multiple tools (e.g., Flake8, isort) in one package.
Feature Parity: Offers a wide range of checks for style and security.
Configurability: Easy setup via pyproject.toml.
Community Adoption: Increasingly used by popular projects like FastAPI or pandas.

Github Action Changes

Put all autocomments in one action autocomment-on-events.yml to clean up the action directory
Added Ruff formatting
Added pytest testing

Todos
I think those can be put in different issues and handled seperately. This list is not comprehensive.

[ ] Fill the .gitignore
[ ] Update the CONTRIBUTING.md and the README.md
[ ] Create the Docker deployments
[ ] Finish documenting the architecture
[ ] Add the frontend structure
[ ] Implement the frontend
[ ] Implement the backend

Also added the logos as in the repository.

@dmotts @swarupn17 @shawaiz-khan @Brijeshthummar02 @team What do you think about these changes?

@HomerusJa Tremendous work 👏! I'll push your work and we'll try to get a demo up as soon as possible.

I can do that, however, the .idea folder only contains IDE-specific settings and there is no risk at all of submitting API keys. If you dont want to have any IDE settings in this repository, we should communicate that clearly.

Ima create issues for the different tasks mentioned above.

@HomerusJa that would be easy to assist.

Can you start from the bottom?


Error rendering the formula

Hello good time
To render, I use the formulas from the code below
window.MathJax.typeset()

This code sometimes gives this error
TypeError: Cannot read properties of null (reading 'replaceChild')

Can you help me what is this error?
using mathjax-4.0.0-beta.4/mathjax/mml-chtml.js
mathjax config:
(window.MathJax = {
            options: {
                enableMenu: !1,
            },
            loader: {
                paths: {
                    "mathjax-fira": "https://cdn.chista.me/chista-mathjax-4.0.0-beta.4/mathjax-fira-font",
                }
            },
            chtml: {
                displayAlign: 'right',
                scale: 1,
                merrorInheritFont: !0,
                matchFontHeight: false,
                mtextFont: 'fontRegular, Noto Sans, sans-serif',
                mtextInheritFont: false,
                unknownFamily: 'fontRegular, Noto Sans, sans-serif',
                font: 'mathjax-fira',
            },
            mml: {
                allowHtmlInTokenNodes: true
              },

            startup: {
                ready() {
                    const { ChtmlMn: e } = MathJax._.output.chtml.Wrappers.mn;
                    e.prototype.remapChars = function (t) {
                        const a = [];
                        for (const e of t) {
                            const t = this.font.getRemappedChar("mn", e);
                            a.push(...(t ? this.unicodeChars(t, this.variant) : [e]));
                        }
                        return a;
                    };
                    const { FontData: n } = MathJax._.output.common.FontData,
                        o = n.defaultMnMap;
                    for (var s = 0; s < 10; s++) o[48 + s] = String.fromCodePoint(1776 + s);
                    const { CHTML } = MathJax._.output.chtml_ts;
                    Object.assign(CHTML.prototype, {
                        _unknownText: CHTML.prototype.unknownText,
                        unknownText(text, variant, width = null, rscale = 1) {
                            const node = this._unknownText(text, variant, width, rscale);
                            if (width !== null) {
                                this.adaptor.setStyle(node, 'width', this.fixed(width * this.math.metrics.scale) + 'em');
                            }
                            return node;
                        },
                        measureTextNode(textNode) {
                            const adaptor = this.adaptor;
                            const text = adaptor.clone(textNode);
                            adaptor.setStyle(text, 'font-family', adaptor.getStyle(text, 'font-family').replace(/MJXZERO, /g, ''));
                            const em = this.math.metrics.em;
                            const style = {
                                position: 'absolute', top: 0, left: 0,
                                'white-space': 'nowrap',
                                'font-size': this.fixed(em, 3) + 'px'
                            };
                            const node = this.html('mjx-measure-text', { style }, [text]);
                            adaptor.append(adaptor.parent(this.math.start.node), this.container);
                            adaptor.append(this.container, node);
                            let w = adaptor.nodeSize(text, em)[0];
                            adaptor.remove(this.container);
                            adaptor.remove(node);

                            return { w: w, h: .6, d: .2 };
                        }
                    });
                    const { MmlMath } = MathJax._.core.MmlTree.MmlNodes.math;
                    const { MmlMstyle } = MathJax._.core.MmlTree.MmlNodes.mstyle;
                    MmlMath.defaults.scriptsizemultiplier = MmlMstyle.defaults.scriptsizemultiplier = 1;
                    MathJax.startup.defaultReady();
                    const params = MathJax.startup.document.outputJax.font.params;
                    params.rule_thickness = .130;
                    params.rule_factor = .75;
                    params.sub1 = .6;
                },
            },
        });

Sometimes this command window.MathJax.typeset(); I run it, I get this error and the formulas are not rendered

@dpvc Hello
I have similar issue but for mathjax/3.2.2/tex-chtml-full-speech.js
Could you please provide example how i can make sure  MathJax.typesetPromise() is resolved before modifying the content on the page?

@Lumeniz, it is hard to do that without knowing how your code works and is organized.  But the basic idea is that you either use
MathJax.typesetPromise().then(() => {
  // your code that runs after MathJax is complete
});

or
await MathJax.typesetPromise();
// your code that runs after MathJax is complete

to synchronize your code with MathJax.  Note, however, that the functions that perform these actions will complete before MathJax has finished its typesetting,  so the functions that call these functions can't assume that MathJax is finished.  That means your whole workflow may need to be aware of the asynchronous nature of these calls, and may need to be broken into pieces in a way that handles that properly.  That might not be the way it is currently organized.
For example, if your code responds to use interaction to modify the page, you may have to prevent additional interactions from occurring until after MathJax has finished typesetting.  One way would be to disable buttons or other means of triggering updates until the MathJax promise is resolved, at which point you re-enable them.
If you are responding to updates in an input area and updating a typeset preview, for example, you might need to suspend the changes until MathJax has completed, and then make a single change that includes all the cached changes and typeset again.  That takes a little more code than just updating immediately when the input changes, but is certainly doable.
This are the kinds off things you need to be aware of when you work with promises.


miss  numpy package

🐛 Bug Description
qlib/qlib/contrib/backtest/position.py /
161        cash = pd.Series(dtype=np.float)

NameError: name 'np' is not defined
To Reproduce
Steps to reproduce the behavior:

import numpy  at head

import numpy as np

Expected Behavior

Screenshot

Environment
Note: User could run cd scripts && python collect_info.py all under project directory to get system information
and paste them here directly.

Qlib version:
Python version:
OS (Windows, Linux, MacOS):
Commit number (optional, please provide it if you are using the dev version):

Additional Notes

Hi, @zhang787jun
Thanks so much for submitting this issue.
Would you mind sending a PR to fix this issue and become one of the contributors of Qlib!


Assert PlayerView is attached

Problem
We have a number of calls that rely on calling methods on the PlayerView. When calling on the Player, another class that can be null, we assert and throw should this be null.
Potential Solution
Check if PlayerView is null and throw informing clients that they must attach a PlayerView in order to interact with it.
Impact
Consistency in the codebase to how we handle @Nullable. More informative error messaging for a user should they forget to attach a PlayerView rather than a NullPointer.

Is the issue still open ? If yes please assign it to me id like to work on it! Thanks.

I have a feeling that this was achieved mostly in the newest update of the library, @ouchadam can you confirm?

We're only asserting when loadVideo is called in the future we would like the library to be able to handle playback without a playerView for example continuing video playback as audio only.

We should drop this issue then. I'll open a new one for what we plan in the future. Thanks @ouchadam. Sorry @Kartik2016 for the misinformation on this issue.


Button shouldn't say save changes if no changes were made

To Reproduce
Steps to reproduce the behavior:

Receive subsequent occurrence of recurring expense email and click through
Without making any changes, click next through all the steps
Button at the end says 'save changes'

Expected behavior
If no changes were made it should say 'submit'
Screenshots

P3 high frequency, low impact

New condition needs to be added here: https://github.com/opencollective/opencollective-frontend/blob/b14156e76c5d806716266b4cb9241ef12db2ede8/pages/expense.js#L690

New condition needs to be added here: https://github.com/opencollective/opencollective-frontend/blob/b14156e76c5d806716266b4cb9241ef12db2ede8/pages/expense.js#L690

@Betree : I believe the button should always be Submit Expense because we don't let the user just "Save" the edits the made without submitting. WDYT? 🤔

@SudharakaP Well, "saving" and "submitting" is the same thing, just different words.
The initial proposal was:

If no changes were made it should say 'submit'

Calling a button just "Submit" is a bad UX practice: https://uxmovement.com/forms/why-your-form-buttons-should-never-say-submit.
The alternative proposal, Submit Expense, works but could let you think you're about to submit a new expense.
I suggest going with Save Expense as a label, regardless if there are changes or not in the form.


[construct identifiers] form an identifier by using arbitrary data

Description
On various places in our codebase we have the need to construct request/look up keys that are based on some other arbitrary (but deterministically defined) data. One of the examples is on the block_remover component the construct_request_key method which forms a key based on provided block (certificate) ids. Those ids are used to form a unique request id which is further used on map look ups. This is now done in a bit naive way by just concatenating the id bytes, which is not ideal since:

it leads to variable length keys
it can lead to quite big keys

As part of this task we want to define a way of creating more concise keys.
Note: Ideally we would like to implement a proper request/response pattern so we don't have to handle this logic on the core application level.

FYI @huitseeker @bmwill @asonnino that's the ticket around finding a better solution for constructing an identifier (which we kind of do via the Hashing approach we followed on some of our PRs. Ideally though we want to implement the request/response pattern we talked today and is mentioned on the task here: https://github.com/MystenLabs/narwhal/issues/143

Yeah I would hope that the networking layer/abstraction would be able to just handle this and be transparent to the actual application layer. We'll see how close we can get with the new networking library once I get to integrating it.


[BUG] Chip-tool cannot connect to matter device, crashes after some time

Reproduction steps
I have an ESP32 device with the examples/light example from esp-matter connected to the wifi network via idf.py monitor. From there I can commission and connect via the ESP Matter app, but if I try to connect from the CHIP tool, it hangs and then eventually quits with a critical error. Here is the output (after running chip-tool pairing code-wifi 0x7283 <ssid> <password><PHONE_NUMBER>2):

Logs
[1682968103.753160][30473:30473] CHIP:DL: ChipLinuxStorage::Init: Using KVS config file: /tmp/chip_kvs
[1682968103.753432][30473:30473] CHIP:DL: ChipLinuxStorage::Init: Using KVS config file: /tmp/chip_factory.ini
[1682968103.753563][30473:30473] CHIP:DL: ChipLinuxStorage::Init: Using KVS config file: /tmp/chip_config.ini
[1682968103.753656][30473:30473] CHIP:DL: ChipLinuxStorage::Init: Using KVS config file: /tmp/chip_counters.ini
[1682968103.753847][30473:30473] CHIP:DL: writing settings to file (/tmp/chip_counters.ini-DQPbgq)
[1682968103.754190][30473:30473] CHIP:DL: renamed tmp file to file (/tmp/chip_counters.ini)
[1682968103.754294][30473:30473] CHIP:DL: NVS set: chip-counters/reboot-count = 31 (0x1F)
[1682968103.754660][30473:30473] CHIP:DL: Got Ethernet interface: eth0
[1682968103.754824][30473:30473] CHIP:DL: Found the primary Ethernet interface:eth0
[1682968103.755032][30473:30473] CHIP:DL: Failed to get WiFi interface
[1682968103.755082][30473:30473] CHIP:DL: Failed to reset WiFi statistic counts
[1682968103.755123][30473:30473] CHIP:IN: UDP::Init bind&listen port=0
[1682968103.755240][30473:30473] CHIP:IN: UDP::Init bound to port=45250
[1682968103.755289][30473:30473] CHIP:IN: UDP::Init bind&listen port=0
[1682968103.755412][30473:30473] CHIP:IN: UDP::Init bound to port=40937
[1682968103.755449][30473:30473] CHIP:IN: BLEBase::Init - setting/overriding transport
[1682968103.755456][30473:30473] CHIP:IN: TransportMgr initialized
[1682968103.755492][30473:30473] CHIP:FP: Initializing FabricTable from persistent storage
[1682968103.755588][30473:30473] CHIP:TS: Last Known Good Time: 2023-05-01T12:07:15
[1682968103.755817][30473:30473] CHIP:FP: Fabric index 0x1 was retrieved from storage. Compressed FabricId 0xAA240AF72C9494DA, FabricId 0x0000000000000001, NodeId 0x000000000001B669, VendorId 0xFFF1
[1682968103.756773][30473:30473] CHIP:ZCL: Using ZAP configuration...
[1682968103.757546][30473:30473] CHIP:CTL: System State Initialized...
[1682968103.757609][30473:30473] CHIP:CTL: Stopping commissioning discovery over DNS-SD
[1682968103.757627][30473:30473] CHIP:CTL: Setting attestation nonce to random value
[1682968103.757684][30473:30473] CHIP:CTL: Setting CSR nonce to random value
[1682968103.758949][30473:30475] CHIP:DL: CHIP task running
[1682968103.759053][30473:30475] CHIP:DL: HandlePlatformSpecificBLEEvent 32784
[1682968103.759323][30473:30475] CHIP:CTL: Stopping commissioning discovery over DNS-SD
[1682968103.759410][30473:30475] CHIP:CTL: Setting attestation nonce to random value
[1682968103.759467][30473:30475] CHIP:CTL: Setting CSR nonce to random value
[1682968103.761223][30473:30475] CHIP:CTL: Generating NOC
[1682968103.761727][30473:30475] CHIP:FP: Validating NOC chain
[1682968103.762386][30473:30475] CHIP:FP: NOC chain validation successful
[1682968103.762492][30473:30475] CHIP:FP: Updated fabric at index: 0x1, Node ID: 0x000000000001B669
[1682968103.762505][30473:30475] CHIP:TS: Last Known Good Time: 2023-05-01T12:07:15
[1682968103.762562][30473:30475] CHIP:TS: New proposed Last Known Good Time: 2021-01-01T00:00:00
[1682968103.762587][30473:30475] CHIP:TS: Retaining current Last Known Good Time
[1682968103.763763][30473:30475] CHIP:FP: Metadata for Fabric 0x1 persisted to storage.
[1682968103.764848][30473:30475] CHIP:TS: Committing Last Known Good Time to storage: 2023-05-01T12:07:15
[1682968103.765692][30473:30475] CHIP:CTL: Joined the fabric at index 1. Fabric ID is 0x0000000000000001 (Compressed Fabric ID: AA240AF72C9494DA)
[1682968103.771138][30473:30475] CHIP:CTL: Setting wifi credentials from parameters
[1682968103.771174][30473:30475] CHIP:CTL: Setting attestation nonce to random value
[1682968103.771205][30473:30475] CHIP:CTL: Setting CSR nonce to random value
[1682968103.771239][30473:30475] CHIP:CTL: Stopping commissioning discovery over DNS-SD
[1682968103.771289][30473:30475] CHIP:CTL: Starting commissioning discovery over BLE
[1682968103.771343][30473:30475] CHIP:CTL: Starting commissioning discovery over DNS-SD
[1682968103.772720][30473:30474] CHIP:DL: FAIL: get bus sync in StartupEndpointBindings, error: Could not connect: No such file or directory
[1682968103.772856][30473:30475] CHIP:DL: PlatformBlueZInit init success
[1682968103.772877][30473:30475] CHIP:CTL: Commissioning discovery over BLE failed: ../../src/platform/Linux/BLEManagerImpl.cpp:682: CHIP Error 0x00000003: Incorrect state
[1682968103.772957][30473:30475] CHIP:-: ../../src/platform/Linux/BLEManagerImpl.cpp:682: CHIP Error 0x00000003: Incorrect state at ../../src/controller/SetUpCodePairer.cpp:299
[1682968103.772986][30473:30475] CHIP:BLE: No adapter available for new connection establishment
[1682968133.786533][30473:30475] CHIP:CTL: Discovery timed out
[1682968133.786737][30473:30475] CHIP:CTL: Stopping commissioning discovery over DNS-SD
[1682968133.786769][30473:30475] CHIP:TOO: Secure Pairing Failed
[1682968133.788015][30473:30473] CHIP:CTL: Shutting down the commissioner
[1682968133.788184][30473:30473] CHIP:CTL: Stopping commissioning discovery over DNS-SD
[1682968133.788212][30473:30473] CHIP:CTL: Shutting down the controller
[1682968133.788234][30473:30473] CHIP:IN: Expiring all sessions for fabric 0x1!!
[1682968133.788253][30473:30473] CHIP:FP: Forgetting fabric 0x1
[1682968133.788560][30473:30473] CHIP:TS: Pending Last Known Good Time: 2023-05-01T12:07:15
[1682968133.788894][30473:30473] CHIP:TS: Previous Last Known Good Time: 2023-05-01T12:07:15
[1682968133.789002][30473:30473] CHIP:TS: Reverted Last Known Good Time to previous value
[1682968133.789070][30473:30473] CHIP:CTL: Shutting down the commissioner
[1682968133.789163][30473:30473] CHIP:CTL: Stopping commissioning discovery over DNS-SD
[1682968133.789256][30473:30473] CHIP:CTL: Shutting down the controller
[1682968133.789355][30473:30473] CHIP:CTL: Shutting down the System State, this will teardown the CHIP Stack
[1682968133.789599][30473:30473] CHIP:DMG: All ReadHandler-s are clean, clear GlobalDirtySet
[1682968133.789737][30473:30473] CHIP:BLE: BleConnectionDelegate::CancelConnection is not implemented.
[1682968133.789775][30473:30473] CHIP:FP: Shutting down FabricTable
[1682968133.789869][30473:30473] CHIP:TS: Pending Last Known Good Time: 2023-05-01T12:07:15
[1682968133.790053][30473:30473] CHIP:TS: Previous Last Known Good Time: 2023-05-01T12:07:15
[1682968133.790147][30473:30473] CHIP:TS: Reverted Last Known Good Time to previous value
[1682968133.790364][30473:30473] CHIP:DL: writing settings to file (/tmp/chip_counters.ini-sR7BLr)
[1682968133.790856][30473:30473] CHIP:DL: renamed tmp file to file (/tmp/chip_counters.ini)
[1682968133.790972][30473:30473] CHIP:DL: NVS set: chip-counters/total-operational-hours = 0 (0x0)
[1682968133.790998][30473:30473] CHIP:DL: Inet Layer shutdown
[1682968133.791015][30473:30473] CHIP:DL: BLE shutdown

(process:30473): GLib-GObject-CRITICAL **: 13:08:53.791: g_object_unref: assertion 'G_IS_OBJECT (object)' failed
[1682968133.791368][30473:30473] CHIP:DL: System Layer shutdown
[1682968133.791912][30473:30473] CHIP:TOO: Run command failure: ../../examples/chip-tool/commands/pairing/PairingCommand.cpp:240: CHIP Error 0x00000003: Incorrect state

Is this the result of user error on my part, or a bona fide bug? Let me know if there are any more details I should supply.
Bug prevalence
Whenever I do this
GitHub hash of the SDK that was being used
bd17b9f52f41eeac5aaa74c1ecf3a76e06dd6958
Platform
windows
Platform Version(s)
Windows 10, WSL2 running Ubuntu 20.04
Anything else?
No response

The problem seems to be right here:
[1682968103.772720][30473:30474] CHIP:DL: FAIL: get bus sync in StartupEndpointBindings, error: Could not connect: No such file or directory

which is due to  this call failing:
    conn = g_bus_get_sync(G_BUS_TYPE_SYSTEM, nullptr, &error);

@marcelgoh That part seems like an issue with gdbus or bluez setup on your system, at first glance.  But it's possible that you might need to sudo to connect to the system bus, depending on the gdbus version.
This part:
(process:30473): GLib-GObject-CRITICAL **: 13:08:53.791: g_object_unref: assertion 'G_IS_OBJECT (object)' failed

looks like a real bug in the bluez bits; if init fails, it's trying to clean up something that never got set up.  Would be useful to know which exact unref this is coming from....  Is this from the:
    g_object_unref(apEndpoint->mpAdapter);

in ShutdownBluezBleLayer in src/platform/Linux/bluez/Helper.cpp?

@bzbarsky-apple
Do you know which the dbus version that can support system bus connection without sudo?

@jacqueshsu I don't know, sorry.  You need someone more familiar with the Linux bits.

Platform seems to be Windows 10 with WSL ... while it maybe work, SDK maintainers do not currently have a windows maintainer.

I will have to close this one: very old bug and we do not generally have WSL support. We only really can debug Ubuntu as this is what we use for our docker images (and maybe some other linux distro on a case by case basis depending on what the main maintainers run on their system)


k8s issue with rbac with non admin users in 1.6

In 1.6, k8s environment with I have three users:
leouser1(Member)
leouser2(Read-Only)
leouser3(Restricted)

when I login as leouser2, I see kubernetes in "Setting up Kubernetes" state though I have a valid running cluster
when I login as leouser3, I see kubernetes but when I access dashboard I see errors User "leouser3" cannot list blah blah blah

when I login as leouser2, I see kubernetes in "Setting up Kubernetes" state though I have a valid running cluster

This is expected as users won't have access to the k8s API. If you want to set up readonly users in k8s, you need to turn on k8s RBAC and then use read only roles for k8s.

when I login as leouser3, I see kubernetes but when I access dashboard I see errors User "leouser3" cannot list blah blah blah

This sounds like k8s RBAC is already turned on. If so, restricted users are not automatically given any privileges and you'd need to set up the RBAC for those users through k8s.

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


change cap. RequiresMACAddress to true

Related to https://github.com/projectcalico/calicoctl/issues/1817
Libnetwork plugin is not functional since docker 18.03
Description
With RequiresMACAddress set to false docker ignores mac change.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@CharlieR-o-o-t sorry for letting this one slip through the cracks...
To be honest I don't fully understand this one - do we still need it? The linked issue is closed (by me?) so wondering if it's still useful.
Let me know - if you'd like to merge then I think you'll need to sign the CLA to make the bot happy. Otherwise I will close this.

@caseydavenport , Hello.
Wow, one year passed. I have found that it's docker libnetwork bug, not calico and fixed this stuff in this PR:
https://github.com/docker/libnetwork/pull/2205
Thank you for reminder to close.


start_date and end_date not working on double picker

I am using the props start_date and end_date on a double picker but it still only shows the default selected range. I can see using Vue Dev Tools that the prop is set, but it's also set in the data function to the default range. Any ideas how I can overcome this?

Update: I just figured out it is a race condition. When I use a static date in the props it works as expected, but when I set them dynamically with a function in the mounted process it does not apply. Any recommendations?

Final update: I learned that I can bind it directly to my prop that contains the start and end date strings without parsing them with the moment library.
:start_date="prop.startDate"


Several IOS errors

Downloaded the file - Ran pod install - Tried compiling it but getting several errors..

Hi!
This is example is super old and no longer maintained. You should check out a more up-to-date example like FriendlyChat for how to get started!
Allt he best,
abe


feat(*): add v0.8 action data filter's useResults field

Signed-off-by: lsytj0413<EMAIL_ADDRESS>Many thanks for submitting your Pull Request :heart:!
What this PR does / why we need it:

add v0.8 actionDataFilter.useResults field

Special notes for reviewers:
Additional information (if needed):
Refs: https://github.com/serverlessworkflow/sdk-go/issues/70

@spolti wanna take a look too?

3ks!

I see, thx :)

issue: https://github.com/serverlessworkflow/sdk-go/issues/70


Support specifying compression type

based of issue #31
Changes in this PR:

Add config table to chose write_compression for CTAS

Hello,
Can this be merged?

@ignacioreyna Thanks, this look great (also wanted to try out zstd compression myself) . Could you fix the conflict?

Waiting on the magic merge :+1: :100:

Done!

Thanks @ignacioreyna @Antauri !

@Antauri
We are a bit limited in time to maintain this repository.
If you or someone else wants to step in, feel free to let us know, @Tomme is willing to add you as maintainer.
Quoting the message of @Tomme  here:

Firstly, I have had to change my dbt Slack account, so apologies if I have missed any of your recent messages.
Secondly, As you may have noticed over the past six months I have been less responsive regarding the upkeep and maintenance of the dbt-athena adapter, this was largely due to me simply using Athena less and less in my day to day work. This is set to change, I am back working with Athena regularly now and hope to implement timely improvements to the adapter and incorporate the pending improvement people have kindly submitted!
If anyone is interested in being added to the project as a maintainer please do just reach out, any help is appreciated.
If there is any open Pull Requests people would like me to prioritise please point me to them,
Note I am open to transferring ownership of the project if it can be more community driven and/or vendor supported, e.g. dbt Labs or Amazon Web Services - feel free to drop me a message if interested.
And lastly, a huge thank you to all contributors, whether it be issues or pull requests - I really did not anticipate the amount of use this adapter would get!
In brief, I hope to improve maintenance for the dbt-athena adapter! (edited)

Sure! When you can, if you'd like, please add me as a maintainer/reviewer. We're using Athena (with Dagster/dbt) for the foreseeable future, which should be helpful in testing some of the pull requests/contributions in a real eco-system.

I'm getting an error after this change. When I do a dbt run ... I get:
06:43:35  Runtime Error in model <omitted> (models/<omitted>.sql)
06:43:35    HIVE_UNSUPPORTED_FORMAT: Unknown ORC compression type . You may need to manually clean the data at location 's3://<omitted>/tables/74c18181-e106-4d3e-8750-87f45371f5a4' before retrying. Athena will not delete data in your account.

If I remove the line related to the write_compression in https://github.com/ignacioreyna/dbt-athena/blob/a77bfbe18889f05c34cddee2df194159c8f004ab/dbt/include/athena/macros/materializations/models/table/create_table_as.sql#L25-L27 everything is back to working.
I'm wondering (but I have very little knowledge of the dbt-athena code) if this has to do with that https://github.com/Tomme/dbt-athena/pull/53 doesn't add the initial set variable line that is done for other variables, i.e. the format variable  {%- set format = config.get('format', default='parquet') -%} has https://github.com/ignacioreyna/dbt-athena/blob/a77bfbe18889f05c34cddee2df194159c8f004ab/dbt/include/athena/macros/materializations/models/table/create_table_as.sql#L7
Maybe you should add  {%- set write_compression = config.get('write_compression', default='snappy') -%}?

The  {%- set write_compression = config.get('write_compression', default=none) -%} was part of the original commit https://github.com/Tomme/dbt-athena/pull/53/commits/4893009b4d9fcd2b27b5571b785dd83269967e5b#diff-c741e161a85cd7b35c19884cff9f6790dd483de64ac687cf4c44e1947d478fa1R15 and must have been lost in a later one. So, I'm pretty sure that should go into the master branch.
Btw, can https://github.com/Tomme/dbt-athena/issues/31 be closed now that this is merged?

hi, yep. I have the same issue.
I investigated the same issue and got to the same point.

Also encountered the issue about write_compression, ref. https://github.com/Tomme/dbt-athena/pull/53#issuecomment-1080268443.
Seems like there's no open issue about it yet, no PR either? Fix seems to be simple. Is anyone planning to make that fix?


Update schema.cdm.json

deleted row 166 because it caused an error in parsing (comment)

Thanks for reporting this!  FYI, @JeffBern is about to submit updates to the schema which fixes this along with other enhancements.

Thanks for feedback!

We will still merge your small change, as Jeff's change needs a little more time.


build: refactored build steps using rollup

Bump dependencies version. I also moved typescript, ts-node, and typedoc to the devDependencies because you don't need them when you're using them. users have to install typescript themself if they're using typescript.
Added module, exports, and files key on package.json. The only thing you'll need when publishing the package is just (1) update the version on package.json and npm publish. You don't have to worry about files that will go to the npm anymore.
Added rollup and rollup-plugin-typescript2 as the build dependencies

I use rollup so there's less hassle for you when compiling the files
It also lets you generate both CommonJS and ESM module files.
Below are the screenshots of the generated files, along with the /dist directory with typescript typings

Removed./src/internal.ts as I think it's kinda useless that you import the file somewhere then export it again somewhere. You could just export it straight away from the source.
Using esbuild-register instead of ts-node because as of now, ts-node has some problem with modules.
Proof that npm run test and npm run build are working:

I'm hoping you could merge this and publish it to npm soon. Thanks!

Sorry for taking so long to answer! I'll try to give this a review tomorrow, thanks for the PR :)

Closes #4


Potential bug?

Hi,
Thank you for sharing the code.
When I was using your code, I think I found some problems. In the code here, you try to get all the env{i}_out_Accuracies/acc_net. It was called here.
I think there is no such metric in the results, and the logic here is not right. If my test_env=0, then after first iteration, the for loop will break, and fails to collect the rest env_i_acc.
Can you take a look at it?
Best regards.

Thank you for the feedback, and yes there was an inconsistency in the code. The default value of metric_key="acc" was appropriate. It's now fixed in the latest commit 8ee9724ae4fc3ec07910d176659cd0f11521a616.


Make chat color and presence message an option

It caused a server crash

seems like owens pr #7 removed crash fix :/

@RhysB Drawing attention to this, merged but overwritten


Update resize-vhd.md

Correct Sizebytes Uint64 value

@nenonix @JohanFreelancer9 Minor copy editing is needed for this PR. Thanks.

As the parameter is named -SizeBytes, instead of referring to the value as UInt64, can you mention the value is in bytes and as you did, provide an example of 1TB = nnn bytes?

ok with @tseward suggestions

Did you test it? I tested and looks like 1TB, 300GB, etc works.
Also, looks like it works for them:
https://www.considerednormal.com/2014/09/resizing-vhd-files-the-easy-way/
https://www.altaro.com/hyper-v/resize-virtual-hard-disks-hyper-v-2016/

Did you test it? I tested and looks like 1TB, 300GB, etc works.
Also, looks like it works for them:
https://www.considerednormal.com/2014/09/resizing-vhd-files-the-easy-way/
https://www.altaro.com/hyper-v/resize-virtual-hard-disks-hyper-v-2016/

cc @msbemba

@

Did you test it? I tested and looks like 1TB, 300GB, etc works.
Also, looks like it works for them:
https://www.considerednormal.com/2014/09/resizing-vhd-files-the-easy-way/
https://www.altaro.com/hyper-v/resize-virtual-hard-disks-hyper-v-2016/

cc @msbemba

@e0i  As per peers it doesn't require change. Was able to test it like Dario mentioned.

@AndreaBarr The PR is ready to be merged. Thanks.

Docs Build status updates of commit d1fe215:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/hyper-v/resize-vhd.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Allow baseurl to be overridable in stable and current recipes, like i…

Allows the baseurl to be overwritten as an attribute in the current and stable recipes, like in the default recipe.  This allows for the override the baseurl so the cookbook will run on platforms such as amazon linux.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.

I've signed the CLA

Thanks @DrHappyAngry for your contribution. As this cookbook is meant to be used exactly as it is with separate recipes for each situation (current, stable), it's explicitly meant to be hard coded to reflect the specific package repositories. This means we won't be taking this PR. Thank you so much for contributing. Apologies that it took so long to get to reviewing this PR. We're working on fixing that, and in the future we should be more on top of these things.


A couple of comments

Hey Anton,
I just looked through the source code and was impressed by the number of smartest metaprogramming tricks. It literally made my brain exploded.
I have only a couple of comments to discuss:

First, the requirement of using pod type seems to be too restrictive. IIUC what we need from a type is to be aggregate initializable and be able to convert via reinterpret_cast to an arbitrary structure. For this, standard_layout requirement seems to be good enough and would allow flat_type to be _somewhat_trivial*. Consider the following case:

struct A { // not trivial, yet aggregate_initializable class (in C++14)
    int a = 8;
    double b = im_not_trivial_but_long_running_function();
};
boost::pfr::flat_get<1>(A{});

On top of that, as flat types are ubiquitously used in constexpr context, it would be nice to static_assert on is_literal_type in the first place.
2. has_reference_members seems to just check whether members can be constructed from temporary objects. it doesn't cover cases when a reference is const or rvalue reference. I'm not sure, maybe it is not intented. Anyways, this requrement is already checked by is_pod, as is_pod implies is_standard_layout, which explicitly prohibits reference members.

First, the requirement of using pod type seems to be too restrictive. IIUC what we need from a type is to be aggregate initializable and be able to convert via reinterpret_cast to an arbitrary structure.

Good point. I'm trying to implement that in a user-friendly manner right now (with readable error message in border-cases).
Ideally I'd like to see the following problem solved: we need a trait std::is_aggregate_initializable here, otherwise there's no way to distinguish between
struct test_me {
    int i = 2;
    short s = 14;
};

and
struct test_me {
    int i = 2;
    short s = 14;

    test_me(){} // not aggregate initializable, PFR can not work with that, compile time error with unreadable message
};

has_reference_members seems to just check whether members can be constructed from temporary objects. it doesn't cover cases when a reference is const or rvalue reference. I'm not sure, maybe it is not intented. Anyways, this requrement is already checked by is_pod, as is_pod implies is_standard_layout, which explicitly prohibits reference members.

Removed the check. Thanks!

New technique was integrated into the library :) Limitation removed: non-flat functions work with any constexpr aggregate initializable function in C++14.


specify integer type to be compatible with 32-bit systems.

Fixes #64
I think that this was a 32-bit issue, rather than a Windows issue. Will need to run tests after merging #62 to test on Windows.

Codecov Report

Merging #65 into develop will decrease coverage by 0.45%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop      #65      +/-   ##
===========================================
- Coverage    83.07%   82.62%   -0.46%
===========================================
  Files           11       11
  Lines         1176     1145      -31
===========================================
- Hits           977      946      -31
  Misses         199      199

Impacted Files
Coverage Δ

pyDeltaRCM/init_tools.py
94.93% <100.00%> (ø)

pyDeltaRCM/shared_tools.py
19.09% <0.00%> (-7.96%)
:arrow_down:

pyDeltaRCM/model.py
88.76% <0.00%> (-1.98%)
:arrow_down:

Continue to review full report at Codecov.

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

I'm just merging this because it's a simple bug fix. Hope y'all are cool with that.


Fallout of serialize traits changes

Requires https://github.com/rust-lang/rustc-serialize/pull/16 to land

Nice!


Force warnings to be errors in Travis CI

This came up in the following discussion: https://github.com/mfem/mfem/pull/1529#discussion_r440983510
I think we can add the following extra options to catch some common warnings: -pedantic -Wall -Werror. The simplest way to do this (independent of the debug option) is to set the makefile variable CPPFLAGS.

That's fine with me

I understand the context correctly, you would like this flag in Travis only, right?
Then I suggest an extra job in the check phase that builds with these flags, and thus check for warnings. Does it make sense to both of you?

@adrienbernede, I was thinking of adding it to all GNU make builds, e.g. by adding CPPFLAGS += " -pedantic -Wall -Werror" before this https://github.com/mfem/mfem/blob/36c7d5f973bc089374b53c9550b5d60566537f76/.travis.yml#L389-L390
However, it makes sense to add it in the "checks" phase too, e.g. here https://github.com/mfem/mfem/blob/36c7d5f973bc089374b53c9550b5d60566537f76/.travis.yml#L68
to prevent unnecessary runs of all "tests".
Using these options in the "tests" phase too is still a good idea, since those test different builds and compilers, so they may fail even if the build in the "checks" phase works fine.

Discussion continued in #1600