Play audio from channel (server)

According to the api wiki it says this:
MumbleOutputStream - Audio output stream for streaming audio from the server.
How do I use this?:
https://github.com/Rantanen/node-mumble/blob/6b873788f12509f7928f2a2dbd1b57d67fb8b384/lib/MumbleOutputStream.js
I simply want to be able to sit on my laptop and say something and be able to hear it from my desktop running this npm package. Is this possible? Could you give me a pre-made example which just works out of the box? If not, could you lead me in the right direction?
I was looking through the source and could not find out how this node package is getting input from the inputStream, how does it know what microphone to use?

This is possible. You would need to somehow get access to your PCs audio
devices, via some other NPM package. You could then connect to mumble using
this package, create a stream as already mentioned by you, connect the
microphone stream to the mumble stream and you're good to go. There are
many examples available. For example this one:
https://github.com/Rantanen/node-mumble/blob/master/examples/stream.js
On Tue, 7 Jul 2020, 02:57 Daniel Gleason<EMAIL_ADDRESS>wrote:

According to the api wiki it says this:
MumbleOutputStream - Audio output stream for streaming audio from the
server.
How do I use this?:
https://github.com/Rantanen/node-mumble/blob/6b873788f12509f7928f2a2dbd1b57d67fb8b384/lib/MumbleOutputStream.js
I simply want to be able to sit on my laptop and say something and be able
to hear it from my desktop running this npm package. Is this possible?
Could you give me a pre-made example which just works out of the box? If
not, could you lead me in the right direction?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Rantanen/node-mumble/issues/120, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALME5Q2RLUK5ONISLC6FF3R2JXH3ANCNFSM4OSIMPVA
.

I was thinking about using this to play the PCMData from the 'voice' event:
https://github.com/TooTallNate/node-speaker
Do you think this would work? Then, I could simply find another way of getting live audio (probably using another npm package, and just pipe it into the input stream?)

Also, is there a way to 'create' new channels?

I'm trying to use this: https://github.com/TooTallNate/node-speaker
So I  wrote this code ion my init event:
            channels: 1,          // 2 channels
            bitDepth: 16,         // 16-bit samples
            sampleRate: 48000 //44100     // 44,100 Hz sample rate
        });
        connection.outputStream().pipe(speaker);

Now it works, but the quality is off, it sounds slow, laggy and robotic. Any ideas?


API commands debt

VS Code has API commands like vscode.open or vscode.executeCompletionItemProvider. We call them "API commands" because they accept and return types from the API, like vscode.TextDocumentShowOptions etc. Know that in theory all our commands are API - keybindings, executeCommandApi, or menus depend on them but the most don't accept arguments and are therefore simpler to handle.
API commands (those with arguments/return-types from the API) are usually defined inside src/vs/workbench/api/common/extHostApiCommands.ts and they usually have a counter part in the renderer. The counter part is doing the actual work while api command transforms api-types to internal-types and back. For instance the vscode.executeDocumentHighlights-command transforms vscode.Uri and vscode.Position into URI and IPosition before calling _executeDocumentHighlights and then transforms the result modes.DocumentHighlight[] into vscode.DocumentHighlight[]. Because of these transformations api commands are implemented in the extension host which has full access to the API types.
There also exists src/vs/workbench/api/common/apiCommands.ts which is for commands that are implemented in the extension host and in the renderer. This is due to a specific bug fix where some API command references would escape the extension host, namely when command-links are used in markdown. We are now in a state where we can fix this bug better but it appears that apiCommands became popular and needs to be clean-up first.
Ask yourself the following questions and act accordingly
Does my API command use types from the API?

YES 👉 use ExtHostCommands#registerApiCommand to register your command. A popular place is still extHostApiCommands.ts but in fact you can (and likely should) use it from anywhere, like within your feature area.

NO 👈 register your command in the renderer via ICommandRegistry#registerCommand  (just like any other command). Be nice and add command description and validate (sample).

[x] OpenFolderAPICommand @bpasero
[x] NewWindowAPICommand @aeschli
[x] OpenWithAPICommand @mjbvz
[x] RemoveFromRecentlyOpenedAPICommand @aeschli
[x] OpenIssueReporter @RMacfarlane
[x] SetEditorLayoutAPICommand @bpasero
[ ] MoveViewsAPICommand @eamodio
[ ] OpenAPICommand @jrieken
[ ] DiffAPICommand @jrieken

Does my API command use types from the API?

Answer this question with NO when yous command expects arguments but when those are outside the APIs, like primitives or simple object literals

@jrieken to clairfy, both my SetEditorLayoutAPICommand and OpenFolderAPICommand already use CommandsRegistry.registerCommand. Do I still need to do anything? Move them to a different file?

Yes - remove this then and move the command metadata into "the one" place. Also, consider to move the command into its feature layer

Ok I have changes in https://github.com/microsoft/vscode/pull/110714, please let me know if that is what you had in mind

ping @eamodio

ping @eamodio

Fixed mine in https://github.com/microsoft/vscode/commit/a6e6e1f70b3276e5c555c47153ff59aae9830cc2

Last round for _extensionTests.getLogLevel and _extensionTests.setLogLevel. @bpasero please move those command into a more domain-specific place, after that we can delete apiCommands.ts

@joaomoreno I also had to move your _workbench.fetchJSON command, so please check that out:
https://github.com/microsoft/vscode/blob/0804afd00ee8e993494386ab32b2c15719bc1749/src/vs/workbench/services/request/browser/requestService.ts#L48-L59
(from https://github.com/microsoft/vscode/commit/d60dbbf646ae6a1affe0de44538a9a294e7a3399#diff-9bc927a30b92207292bb17a10e58c2ffe05cc22d5ea5ddcd7595cb19dbe50696)


Missing setuptools dependency

setuptools is required for pkg_resources to be available

are you saying to add it to requirements?

Yes, because it is a requirement.

Thank you for your help fixed the issue


Bump to Python 3.10

Open Forms itself upgraded to Py 3.10.

Too good to be true that I wouldn't have missed anything :stuck_out_tongue:


Test and extend the support for LiteLLM with the Python GitHub App

Currently, only the OpenAI API seems to be readily supported, and integrating LiteLLM might prove to be more useful with regards to using Huggingface models (such as https://huggingface.co/tiiuae/falcon-40b-instruct, which is released under the Apache 2.0 license, and for which the community provides guidance for finetuning).

Hey @jrgriffiniii love this! Are you using litellm today?


chore: update @idos-network/idos-sdk

That looks odd to me because I got the same behavior. Let me dig deeper into it

Locally it works. Let's hold this a bit

@shelegdmitriy mynearwallet keeps me in a loop.  I'm getting an entirely different error on near.org with both mynearwallet and here.


Protect against stateful mappers returning nothing in the raw executor

Fixes #5303.

:+1: on green.


[stable/nfs-client-provisioner] F0928 provisioner.go:180 Error getting server version

Describe the bug
command: /usr/local/bin/helm install --values ~/nfs-client-provisioner_values.yml --name nfs-client-provisioner stable/nfs-client-provisioner
[devopsdev@kmasterdev ~]$ kubectl describe pod nfs-client-provisioner-68cf8787d6-j5cfk | tail -4
Events:
  Type     Reason   Age                    From                 Message
  ----     ------   ----                   ----                 -------
  Warning  BackOff  4m50s (x385 over 98m)  kubelet, knode02dev  Back-off restarting failed container

[root@kmasterdev ~]# kubectl -n company-com-devops-dev get pods
NAME                                      READY   STATUS             RESTARTS   AGE
nfs-client-provisioner-68cf8787d6-j5cfk   0/1     CrashLoopBackOff   27         129m

[devopsdev@kmasterdev ~]$ kubectl logs nfs-client-provisioner-68cf8787d6-j5cfk
F0928 22:09:05.159493       1 provisioner.go:180] Error getting server version: Get https://<IP_ADDRESS>:443/version?timeout=32s: dial tcp <IP_ADDRESS>:443: i/o timeout

Version of Helm and Kubernetes:
[devopsdev@kmasterdev ~]$ kubectl version
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.4", GitCommit:"67d2fcf276fcd9cf743ad4be9a9ef5828adc082f", GitTreeState:"clean", BuildDate:"2019-09-18T14:51:13Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.4", GitCommit:"67d2fcf276fcd9cf743ad4be9a9ef5828adc082f", GitTreeState:"clean", BuildDate:"2019-09-18T14:41:55Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}

[devopsdev@kmasterdev ~]$ helm version
Client: &version.Version{SemVer:"v2.14.3", GitCommit:"0e7f3b6637f7af8fcfddb3d2941fcc7cbebb0085", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.14.3", GitCommit:"0e7f3b6637f7af8fcfddb3d2941fcc7cbebb0085", GitTreeState:"clean"}

Which chart:
stable/nfs-client-provisioner
What happened:
The Pod BackOff
What you expected to happen:
To have the Pod running stable
How to reproduce it (as minimally and precisely as possible):
From a Debian based distro like Linux Mint or Linux MX and from a non root user run these ansible commands:
ansible-galaxy install ernestomedina17.ansible_role_kube_cluster_3vms_vagrant --force
ansible-galaxy install ernestomedina17.ansible_role_kube_cluster_3vms_common --force
ansible-galaxy install ernestomedina17.ansible_role_kube_cluster_3vms_kmaster --force
ansible-galaxy install ernestomedina17.ansible_role_kube_devops_namespace --force
ansible-galaxy install ernestomedina17.ansible_role_kube_devops_tools --force
ansible-playbook .ansible/roles/ernestomedina17.ansible_role_kube_cluster_3vms_vagrant/playme01.yml
ansible-playbook .ansible/roles/ernestomedina17.ansible_role_kube_cluster_3vms_common/playme02.yml
ansible-playbook .ansible/roles/ernestomedina17.ansible_role_kube_cluster_3vms_kmaster/playme03.yml
ansible-playbook .ansible/roles/ernestomedina17.ansible_role_kube_devops_namespace/playme04.yml
ansible-playbook .ansible/roles/ernestomedina17.ansible_role_kube_devops_tools/playme05.yml

then you can ssh and use kubectl as shown below:
ssh ansibledev@kmasterdev
sudo su -
kubectl -n company-com-devops-dev describe pod $(kubectl -n company-com-devops-dev get pods | grep nfs | awk '{ print $1 }')

Anything else we need to know:
Relevant files that are applied before running the helm install command are:
~/.ansible/roles/ernestomedina17.ansible_role_kube_cluster_3vms_kmaster/files/tiller.yml
~/.ansible/roles/ernestomedina17.ansible_role_kube_devops_namespace/templates/helm.yml.j2
~/.ansible/roles/ernestomedina17.ansible_role_kube_devops_namespace/tasks/main.yml
~/.ansible/roles/ernestomedina17.ansible_role_kube_devops_tools/templates/nfs-client-provisioner_values.yml.j2
~/.ansible/roles/ernestomedina17.ansible_role_kube_devops_tools/tasks/main.yml

These are the full events on the pod:
Events:
  Type     Reason     Age                From                 Message
  ----     ------     ----               ----                 -------
  Normal   Scheduled  77s                default-scheduler    Successfully assigned company-com-devops-dev/nfs-client-provisioner-68cf8787d6-765r5 to knode01dev
  Normal   Pulling    76s                kubelet, knode01dev  Pulling image "quay.io/external_storage/nfs-client-provisioner:v3.1.0-k8s1.11"
  Normal   Pulled     72s                kubelet, knode01dev  Successfully pulled image "quay.io/external_storage/nfs-client-provisioner:v3.1.0-k8s1.11"
  Normal   Created    40s (x2 over 72s)  kubelet, knode01dev  Created container nfs-client-provisioner
  Normal   Pulled     40s                kubelet, knode01dev  Container image "quay.io/external_storage/nfs-client-provisioner:v3.1.0-k8s1.11" already present on machine
  Normal   Started    39s (x2 over 71s)  kubelet, knode01dev  Started container nfs-client-provisioner
  Warning  BackOff    9s                 kubelet, knode01dev  Back-off restarting failed container

Today after bringing up the VMs luckily I noticed new Warning with more info on the events:
Events:
  Type     Reason                  Age                  From                 Message
  ----     ------                  ----                 ----                 -------
  Normal   Scheduled               18h                  default-scheduler    Successfully assigned company-com-devops-dev/nfs-client-provisioner-68cf8787d6-765r5 to knode01dev
  Normal   Pulling                 18h                  kubelet, knode01dev  Pulling image "quay.io/external_storage/nfs-client-provisioner:v3.1.0-k8s1.11"
  Normal   Pulled                  18h                  kubelet, knode01dev  Successfully pulled image "quay.io/external_storage/nfs-client-provisioner:v3.1.0-k8s1.11"
  Normal   Created                 18h (x5 over 18h)    kubelet, knode01dev  Created container nfs-client-provisioner
  Normal   Started                 18h (x5 over 18h)    kubelet, knode01dev  Started container nfs-client-provisioner
  Normal   Pulled                  18h (x4 over 18h)    kubelet, knode01dev  Container image "quay.io/external_storage/nfs-client-provisioner:v3.1.0-k8s1.11" already present on machine
  Warning  BackOff                 17h (x198 over 18h)  kubelet, knode01dev  Back-off restarting failed container
  Warning  FailedCreatePodSandBox  118s                 kubelet, knode01dev  Failed create pod sandbox: rpc error: code = Unknown desc = failed to set up sandbox container "dbb17a6c3e78b9154f17048f86d87b1be5cb6150c7eb73e83f623f09ecd505a9" network for pod "nfs-client-provisioner-68cf8787d6-765r5": NetworkPlugin cni failed to set up pod "nfs-client-provisioner-68cf8787d6-765r5_company-com-devops-dev" network: open /run/flannel/subnet.env: no such file or directory
  Warning  FailedCreatePodSandBox  110s                 kubelet, knode01dev  Failed create pod sandbox: rpc error: code = Unknown desc = failed to set up sandbox container "e80c61c0726e862ad33da112c796f2ff32d276fb6cb6ac73d3ee04caa816b5bf" network for pod "nfs-client-provisioner-68cf8787d6-765r5": NetworkPlugin cni failed to set up pod "nfs-client-provisioner-68cf8787d6-765r5_company-com-devops-dev" network: open /run/flannel/subnet.env: no such file or directory
  Normal   SandboxChanged          109s (x3 over 2m5s)  kubelet, knode01dev  Pod sandbox changed, it will be killed and re-created.
  Warning  BackOff                 17s (x3 over 75s)    kubelet, knode01dev  Back-off restarting failed container
  Normal   Pulled                  5s (x3 over 108s)    kubelet, knode01dev  Container image "quay.io/external_storage/nfs-client-provisioner:v3.1.0-k8s1.11" already present on machine
  Normal   Created                 5s (x3 over 107s)    kubelet, knode01dev  Created container nfs-client-provisioner
  Normal   Started                 5s (x3 over 106s)    kubelet, knode01dev  Started container nfs-client-provisioner

Me bad, the problem was that I initiated the cluster using a network cidr different from the default flannel network, and since they are different components which its configuration needs to match, that made the network nonfunctional. Closing.


testing Max NaN check

adds nan check to Tensor.max()
adds max nan CI testing
upped kernel limits

*** onnx thinks chicken is a lizard
*** webgpu and metal don't handle nan the same
benchmarks:
Master:
max  4096x 4096  3.71 ms (4.52 GFLOPS  18.09 GB/s) in torch,  8.73 ms (1.92 GFLOPS 7.68 GB/s) in tinygrad,  2.35x slower      16.78 MOPS  67.11 MB
This Patch:
max  4096x 4096  3.76 ms (8.93 GFLOPS  35.70 GB/s) in torch, 17.36 ms ( 1.93 GFLOPS 7.73 GB/s) in tinygrad,  4.62x slower  33.55 MOPS  134.22 MB
Also, made an implementation in mlops, but it generated more kernels.

Changes made in tinygrad/:
------------------------------------------------------------
files                             insertions       deletions
------------------------------------------------------------
tinygrad/tensor.py                         1               1
------------------------------------------------------------
lines added in the tinygrad folder: 0

Big perf hit, I'm not going to merge this.


feat: support htmltools.browsable() methods

In R {htmltools}, there is a way to preview html results in browsers from console via htmltools::browsable(). However, {py-htmltools} does not seem to have htmltools.browsable() methods.
This feature could be possibly integrated in {great_tables} to support some preview results, such as https://github.com/posit-dev/great-tables/issues/100

Hi @jooyoungseo. The closest thing we have to browsable() is the .show() method, which should be on any Tag/TagList:
>>> import htmltools
>>> htmltools.div("hello").show()

@cpsievert -- This is cool! For those coming from R {htmltools} package, are you open to alias this method to browsable() as well?


Ability to add grouping rules for other repos

Is your feature request related to a problem? Please describe.
This project has a really cool feature where you don't have to maintain multiple versions since a library group is known to use the same version for all of its artifacts. Good example is retrofit2. All of it's versions will be simply stored in version.retrofit2 and be shared.
Describe the solution you'd like
The same applies to some of group of artifacts too. refreshVersions can provide rules functionality to support this. And versions.properties will become easier to maintain.

Another example: Netty

Another example: Netty

This is already supported but not documented. You can see an example in the sample-kotlin sample.
FYI, we have been thinking of different approaches where this could partially be inferred, or where it could be defined from code, but both of these approaches seem tricky to implement.

This is already supported but not documented. You can see an example in the sample-kotlin sample.
FYI, we have been thinking of different approaches where this could partially be inferred, or where it could be defined from code, but both of these approaches seem tricky to implement.

Is it possible to link to the example? I couldn't see how it is done. I just checked. It seems like the following version is defined 2 times back to back
https://github.com/jmfayard/refreshVersions/blob/main/sample-kotlin/versions.properties#L84

Is it possible to link to the example? I couldn't see how it is done. I just checked. It seems like the following version is defined 2 times back to back
https://github.com/jmfayard/refreshVersions/blob/main/sample-kotlin/versions.properties#L84

Is it possible to link to the example? I couldn't see how it is done. I just checked. It seems like the following version is defined 2 times back to back
https://github.com/jmfayard/refreshVersions/blob/main/sample-kotlin/versions.properties#L84

@tasomaniac If you remove lines 84-86, the sample seems to work correctly, using the version from version.apache.poi. However, I'd expect the plugin to remove those duplicate lines by itself since they are covered by a shorter name. @LouisCAD Is this a bug, or is it intended?

Is it possible to link to the example? I couldn't see how it is done. I just checked. It seems like the following version is defined 2 times back to back
https://github.com/jmfayard/refreshVersions/blob/main/sample-kotlin/versions.properties#L84

@tasomaniac If you remove lines 84-86, the sample seems to work correctly, using the version from version.apache.poi. However, I'd expect the plugin to remove those duplicate lines by itself since they are covered by a shorter name. @LouisCAD Is this a bug, or is it intended?

@joharei Probably a bug, though in future versions, we might have a different behavior as we might stop auto-removing unused entries, just adding a warning/info comment instead.

@joharei Probably a bug, though in future versions, we might have a different behavior as we might stop auto-removing unused entries, just adding a warning/info comment instead.

I've documented how it works here
https://github.com/jmfayard/refreshVersions/issues/356#issuecomment-843276157


Emails created from plain text Outlook messages lose formatting in emailToEML()

I create an email object from a .msg file that was in Outlook's plaintext format using outlookMsgToEmail(). When I then try to use emailToEML() to save the email as a .eml, all formatting (e.g. line breaks) is lost.
On investigation it appears that outlookMsgToEmail() is creating an object with plain text content available using both getPlainText() and getHTMLText(). I'm not sure whether this is a problem in outlookMsgToEmail() or in the actual Outlook plain text file format. It then seems that emailToEML() is using the content which is plain text, but treating it as HTML, therefore losing the line breaks.
This problem doesn't occur when creating an email object from emlToEmail(), as here getPlainText() produces the plain text format but getHTMLText() shows as null, and emailToEml() seems to treat the plain text as plain text and keep line breaks intact.
	public static void viewMsg() throws IOException {

		Email em1 = EmailConverter.outlookMsgToEmail(new File("inputPlainText.msg"));

		String htmlText = em1.getHTMLText();
		String plainText = em1.getPlainText();

		System.out.println("plainText: " + plainText);
		System.out.println("htmlText: " + htmlText);

		String emlStr = EmailConverter.emailToEML(em1);

		FileOutputStream outputStream = new FileOutputStream("output1.eml");
		byte[] strToBytes = emlStr.getBytes();
		outputStream.write(strToBytes)
		outputStream.close();

	}

	public static void viewEml() throws IOException {

		Email em1 = EmailConverter.emlToEmail(new File("inputPlainText.eml"));

		String htmlText = em1.getHTMLText();
		String plainText = em1.getPlainText();

		System.out.println("plainText: " + plainText);
		System.out.println("htmlText: " + htmlText);

		String emlStr = EmailConverter.emailToEML(em1);

		FileOutputStream outputStream = new FileOutputStream("output2.eml");
		byte[] strToBytes = emlStr.getBytes();
		outputStream.write(strToBytes)
		outputStream.close();

	}

samples.zip

I'm looking into it, but the Outlook msg actually isn't just plain text; it contains an RTF body, which is converted to HTML by the Outlook message parser library. The RTF text has endlines containg \r\n, which are not converted to  elements. And since HTML body is prioritized over text bodies by email clients (unless you specifically configured them to ignore HTML), you see the content with the missing line breaks.
I'm currently looking into whether the RTF conversion is correct. Stay tuned.

It's caused by this bug: https://github.com/bbottema/rtf-to-html/issues/6

Given the discussion at bbottema/rtf-to-html#6, is it possible for me to choose the RTFToHTML converter that emailToEML() uses, so if I am processing an MSG plain text file I can use the RTF2HTMLConverterJEditorPane which produces formatted HTML?

Really the only reason the swing converter is still there because of my OCD completionist tendency (or research), but it should really not be considered for real world application. It really doesn't handle RTF properly.
That said, the other don't either in this particular (exotic?) case.
I'm just not keen on making this part of the builder api. Perhaps making it a property is enough? Like a sneaky backdoor config for Outlook RTF conversion.


Remove component definitions in Java codes.

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[ ] Update the CHANGES log.

Remove component definitions in Java codes. component-libraries.yml controls the ownership now.

I noticed this from several PRs about adding component IDs.


Resolved templating issue with cic disabled

cic:
  required: false

gives a: duplicated resource error

Hi @BobVanB , Thanks for pointing out the issue in the template of Helm chart.
We have released the correct fix for this issue and it is available in our latest Helm Charts.
Please use them and let us know if there are any further queries.

@priyankash-citrix i proposed the fix, but have not tested it with cic.required=True. Can you please make shure the fix was correct?

Hi @BobVanB, the issue reported by you was there but the fix done in your PR was not the correct fix for that issue.
Moreover this needed to be fixed in other helm charts as well, that is why I have checked-in the correct fix for the issue in all the helm charts. And the fix available in the latest helm charts should work for both cic.required=True and cic.required=False.

Glad to know, thank you.


Add method to automatically create integer IDs

Added a method to generates integer resource IDs automatically.

Having automatically generated ids is a good idea, but I'm not loving parsing directory to generate it. It could lead to massive slowdowns. Instead we could generate an id (UUID, timestamp hash, etc...) to avoid the overhead of listing/parsing the directory

@glinton yeah i thought about UUIDs, but for my usecase (the datastore behind a slackbot) I need to be able to refer to items by simple IDs. A UUID would be cumbersome to type into slack. The other benefit of integer IDs is creation order is implicit (I'm only really interested in the last 20 or so files, so not having to ls -l to find the latest ones is useful).
Perhaps the driver should just be updated to cache the last used auto-generated ID? While auto-generating an ID, it could try to write a new file (not overwrite an existing one), and only fall back to inspecting the filesystem if a file with the next ID already exists. This would be a simple way of improving performance without needing complicated IDs.

I think parsing the files in the directory to establish the starting integer is a much better solution than reading every time for sure. Another concern regarding parsing a directory would be if an empty file exists, should scribble free that id up or just move on and take the highest found number then add 1? If you would like to add the bit to parse the directory on startup and determine the id, we can merge that in. Otherwise, i'll update this pr to merge to a feature branch where I'll finish that bit and use what you have for the id use.
Let me know what you prefer/have time for.

@glinton I think regarding empty files scribble should just take the next available ID. No point in making too many assumptions about why empty files exist.
One thing I'd probably change is I just picked a largish number of zeros to pad IDs by, but that should probably end up in config on the driver with a method to centralise generating padded IDs. There are a few fmt.Sprintf() calls floating around which aren't intuitive, especially in client code.
I'm not going to have time to work on this until at least the week after next, so if you have time please go ahead.
Thanks!

Sounds good, I'll get on it


sequencer: prevent one bundle revert causing entire batch to revert

currently if there is a batch of N action bundles, and one of those bundles reverts ... the entire batch reverts.
only the one bundle should revert, not the entire batch.

this was resolved in #11


Contributing a deep-learning, BERT-based analyzer

Description
Hi,
We are building an open-source custom Hebrew/Arabic analyzer (lemmatizer and stopwords), based on a BERT model. We'd like to contribute this to this repository. How can we do that and be accepted? Can we compile it to native code and use JNI or Panama ? If not, what is the best approacch?
https://github.com/apache/lucene/issues/12502#issuecomment-1675084211
@uschindler would be very happy to hear what you think

@lmessinger I don't see why text tokenization would need any native code. Word piece is pretty simple and just a dictionary look up.
Do y'all not have a Java one?
Or does this model actually need inference to do the lemmatization? (e.g. https://huggingface.co/dicta-il/dictabert-joint) ?

Hi,
Got it. Pointing to the project from the documentation would actually be
very valuable to the Hebrew community. How can that be done? is the
documentation also on github, so we can add it there as PR for approval?
thanks!
Lior
On Tue, Feb 6, 2024 at 10:06 PM Dawid Weiss @.***>
wrote:

It will be a major headache to maintain native bindings for all major
platforms. I think such an analyzer should be a downstream project (then
you can restrict the platforms on which it's available to whatever you wish
to maintain). We can point at such a project from Lucene documentation, for
example.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/lucene/issues/13065#issuecomment-1930667697,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAM5MHJY3T5ODVXTMPJVEMDYSKEKZAVCNFSM6AAAAABCU5UD2KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZQGY3DONRZG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Lior Messinger
+1-646-3730044
+972-546-888401

How can that be done?

This is a question that is much harder to answer than I thought... Lucene doesn't have a tutorial/user guide. The only place I could think of was here, in the javadocs:
https://github.com/apache/lucene/blob/main/lucene/analysis/common/src/java/overview.html
An alternative would be to include an empty package for Hebrew and only add the package-info.java file, telling folks where they can find downstream Hebrew analyzers. I really don't have any better ideas.

How about something with the source maintained in the sandbox dir (along
with instructions to build), but no corresponding official release artifact?
On Fri, 9 Feb, 2024, 1:20 am Dawid Weiss, @.***> wrote:

How can that be done?
This is a question that is much harder to answer than I thought... Lucene
doesn't have a tutorial/user guide. The only place I could think of was
here, in the javadocs:
https://github.com/apache/lucene/blob/main/lucene/analysis/common/src/java/overview.html
An alternative would be to include an empty package for Hebrew and only
add the package-info.java file, telling folks where they can find
downstream Hebrew analyzers. I really don't have any better ideas.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/lucene/issues/13065#issuecomment-1934831122,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDCR5FBPK7ZCAOW5U2YYOTYSUT65AVCNFSM6AAAAABCU5UD2KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZUHAZTCMJSGI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

How can that be done?

This is a question that is much harder to answer than I thought... Lucene doesn't have a tutorial/user guide. The only place I could think of was here, in the javadocs:
https://github.com/apache/lucene/blob/main/lucene/analysis/common/src/java/overview.html
An alternative would be to include an empty package for Hebrew and only add the package-info.java file, telling folks where they can find downstream Hebrew analyzers. I really don't have any better ideas.

It would be better located in the module "analysis" (which is just the parent of all analyzers). Unfortunately this module does not create javadocs, so analysis-common is the only location.
I think it would be a good idea to add there a list to external resources of analysis components. Lucene is a flexible library with extension points through SPI, so we can list all external contributions there.
This page is also missing an overview on the analysis submodules.
An alternative (an in my opinion better) idea is to put a list as Markdown file into the documentation package: https://github.com/apache/lucene/tree/main/lucene/documentation/src/markdown
All md files there are compiled to HTML and can be linked in the template file for index.html, too.

How about something with the source maintained in the sandbox dir (along with instructions to build), but no corresponding official release artifact?

I don't think this is a good idea. It won't be tested (as we can't run the build) and also it is inconsequent.
We had that in the past for the DirectIODirectory and WindowsDirectory. All those were not maintained -- and did not build anymore, although there were build scripts. The Java parts were building, the JNI parts were not longer matching the Java implementations. I may be wrong, but when we looked into this, it was almost impossible to make it work again.
Luckily they were rewritten using Java 11+ APIs and are now part of official distribution.


How to remember password key after reboot

I have ente the password for my private key after reboot pc. How to save my password private key after reboot?, like this:
$ ssh bhh@bhost  -p1234
(without enter password priv key after executing command)

I'm not certain how to force it to remember the key passphrase but I would advise against this. The whole point of having a passphrase for a key file is for additional security. If you don't need this extra layer of security or are too annoyed by it, then I would suggest just remove the passphrase from the key file entirely.

Sorry, but it not a solution. Now i use keychain, he is more useful.


Enable tree sitter code only when it's enabled

This does not work for me. When I apply it locally, flycheck reports that rust-mode-treesitter-derive) is not defined

Perhaps that should be an or?

Adding (require 'rust-mode) gets rid of the warning, but the problem remains.
Turning the and into an or just makes rust-mode not load at all.
I am reluctant to turn on native tree-sitter mode, as I develop some tree-sitter grammars and have the non-native tree-sitter support enabled.

@alanz When do you actually see that warning that was reported by you ? Is it during installation or during some other step (Your latest comment indicates it's during flycheck) ? I can try reproducing it locally for my setup.
I don't think doing or is the right thing here.

I agree or is not the right step, it broke completely.
I see the error on starting up emacs.  I guess I need to dig more deeply into the actual origin. And/or go full native tree-sitter.
I will not be able to do any investigation today though

And it seems the warning is benign, I have the functionality I need regardless.

see the error on starting up emacs.

Looking at my Messages buffer - I can see a similar type of warning:
 ■  Warning (treesit): Cannot activate tree-sitter, because language grammar for rust is unavailable (not-found): (libtree-sitter-rust libtree-sitter-rust.0 libtree-sitter-rust.0.0 libtree-sitter-rust.so libtree-sitter-rust.so.0 libtree-sitter-rust.so.0.0) No such file or directory

I receiving the warning even though I'm using a tree sitter based setup. Probably it's able to find the grammars subsequently, not sure.

And it seems the warning is benign, I have the functionality I need regardless.

Thanks for the confirmation, I assumed something was broken. Given the situation, I will close this PR. I would be happy to accept any PR that fixes the warning. :-)

FYI, I switched to emacsformacosx, which has tree-sitter support build in. Without explicitly asking for the tree-sitter mode, I am getting
⛔ Warning (treesit): Cannot activate tree-sitter, because language grammar for rust is unavailable (not-found): (libtree-sitter-rust.so libtree-sitter-rust.so.0 libtree-sitter-rust.so.0.0 libtree-sitter-rust.dylib libtree-sitter-rust.dylib.0 libtree-sitter-rust.dylib.0.0) No such file or directory

Which I guess means I need to work out how to install the language. Is it documented somewhere? A quick scan of the README does not show anything

And it looks like the steps are
init.el
  (setq treesit-language-source-alist
      '((rust "https://github.com/tree-sitter/tree-sitter-rust")
        ))

In emacs
M-x treesit-install-language-grammarPickrust`

My final piece of barrage.
Putting the following into my init.le
(use-package rust-ts-mode
  :custom
  (rust-mode-treesitter-derive t)
...
)
...
(use-package lsp-mode
  :hook (rust-mode rust-ts-mode)
...
)

And basically using rust-ts-mode instead of rust-mode in all config restored

semantic highlighting
compile buffer regexp matching
basic rust-mode operation, such as knowledge of what the normal indent and comment markers are


Ultra Prison Core - Features & Bugs

Chat
Add a option to allow placeholders so we can show prestige in front of our nicknames in chat using essentialsx.
Pickaxe
Add a option to allow the placeholder pickaxe level to be in the name of the pickaxe.
Add a option to allow us to change the command to bring up the enchantment menu, such as  /etshop or /enchant instead of on click. When clicking with the pickaxe in hand it brings up the enchantment menu, however some of us use npcs to run commands and mcmmo on prison. We cannot use these because it triggers the menu instead.
Add a option to allow the pickaxe to automatically change depending on what block your hitting.
I don't mean that if you have the pickaxe and another tool in your inventory, that it would switch to it. I am saying that the actual pickaxe changes. For example, when breaking certain blocks it would be a pickaxe, and then automatically change to a axe when hitting any wood block, sword when hitting players or entities, shovel and so on.
Disenchanting
Add a option to allow your pickaxe to be fully unenchanted with a confirm gui for the disenchanter.
This would be a option and the gui would only show if we have this option toggled. If you don't, then you will get the way in comes to where you will be able to take off certain enchants.
Mines
Add a command to go to the current mine that you are at, using /mine.
Add a command to see all the available and unavailable mines along with their per drop, using /mines.
Add a option that mine resets are on a timer or by the percent of blocks that are mined.
Boosters
Add the option for a global booster to where you can boost 2x and 3x for a time limit with the options of sell and etokens with optional xp.  This would be a physical item that is credited to the players inventory. In doing this, add /booster that brings up a gui to see all active boosters for the server. Make it so only one booster of each type can be active at one time per server. Boosters do not stack in time. Make it so you must wait to activate a booster until the current one expires.
Backpacks:
Add the option for backpacks can be bought using e-tokens in either a shop gui or at least by a command so we can make our own shops.
Add the option for backpacks of our choosing to be merged into higher tiers, giving us more slots by using /bpmerge.
Add the option to only allow blocks that are mined go into the backpack, and make it so they automatically go there.
Add the option that if you place a backpack to the ground that all blocks inside of it go into your inventory. The backpack is not able to be placed on the ground.
Add the option to allow to choose what material the backpack is in inventory.
Enchants:
Efficiency - Allows pick to break blocks faster.
Fortune - Increases drops per blocks broken.
Jackhammer - Chance to clear one whole layer of the mine.
Explosive - Spawn TnT explosions as you mine.
Regrowth - Recover health and hunger while using your pickaxe.
Speed - Allows you to run faster while using your pickaxe.
Haste - Allows you to swing your pickaxe faster.
Auto Mine - Like explosive except automatic.
Merchant - Chance to sell a block for 2x it's value.
Enrage - Chance to throw you into a rage spawning an RPG everytime you swing your pickaxe for 3s.
Auto Sell - Every 6 seconds, you will sell 3x block, upgradeable based on yoru level.
Laser - Chance to shoot a laser that mines out in all directions and doesn't stop until it hits the mine's edge or air.

Great list. i'd like to add to enchants:
Silktouch - Allows you to silktouch blocks like vanilla minecraft.

Updated

Updated

Hello, please do not post feature requests in there, instead post them on our Discord. Also lot of there things are already in the core.


Superset v0.24.0 translation for Chinese language is broken for Chart menu part

Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if any
[x] I have reproduced the issue with at least the latest released version of superset
[x] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
v0.24.0
Expected results
Thanks a lot for the great Superset team to provide brilliant functionalities, and features continuously!
Background,  in our company we are popularize Superset to finance division, so I am doing Chinese translation work based on v0.24.0. I know github repo already has a Chinese translation, and I'd like to expect: the Chinese messages could be shown on Superset webpage, especially the menu implemented by React, so that users could have a better understanding of how to use Superset.
Actual results
With Superset v0.24.0, even if we have switched to Chinese locale on the Superset webpage,  user still cannot see Chinese translation content on the chart menu part.
So the Chinese translation part is broken in v0.24.0.
Steps to reproduce

Open Superset webpage, and switch to Chinese locale on the top-right button of the webpage.
Open a Chart(Slice), check the left part of the webpage and see it is still Englis prompt, no Chinese messages.

Below is what I have seen after the above reproducing:
Top bar menus are good in Chinese, but left menus are still in English(PS: the left menus indeed have Chinese translation in superset/translations/zh/LC_MESSAGES/messages.po/json

Below is how I solve the Chinese translation missing issue:
I followed below procedure guide and I have the Chinese translation worked for Chart menu part.
https://github.com/apache/incubator-superset/blob/master/CONTRIBUTING.md#translations

Go to top directory of superset directory.
(venv) [root@centos4 site-packages]# pwd
/root/Dev/Superset/venv/lib/python2.7/site-packages
Modify babel.cfg to add static for js/jsx path, OTHERWISE "fabmanager babel-extract" will NOT extract the messages for translation in js/jsx files.

A big question, why there is NO static in the github repo?
Without the static in https://github.com/apache/incubator-superset/blob/master/superset/translations/babel.cfg, how could the messages in js/jsx be extracted?
Could you please help answer?
(venv) [root@centos4 translations]# cat babel.cfg
[ignore: superset/assets/node_modules/**]
[python: superset/**.py]
[jinja2: superset/**/templates/**.html]
[javascript: superset/static/assets/javascripts/**.js]
[javascript: superset/static/assets/javascripts/**.jsx]
[javascript: superset/static/assets/visualizations/**.js]
[javascript: superset/static/assets/visualizations/**.jsx]
encoding = utf-8

Extract the messages marked for translation
(venv) [root@centos4 site-packages]# fabmanager babel-extract --target superset/translations/ --output superset/translations/messages.pot --config superset/translations/babel.cfg -k _ -k __ -k t -k tn -k tct
Compile and generate json file.
(venv) [root@centos4 site-packages]# fabmanager babel-compile --target superset/translations/
(venv) [root@centos4 site-packages]# po2json -d superset -f jed1.x superset/translations/zh/LC_MESSAGES/messages.po
Restart Superset and select Chinese locale.

After above operation, the translation messages could been seen on menu, as below:

If needed, I could contribute to solve this issue for Chinese locale, and submit to Superset github repo and catch a patch or later release, since I am familiar with Chinese.
Please don't hesitate if anything needed!
Thanks a lot in advance!
Regards,
Jiaxu
刘佳旭

When doing development work you should work on a git checkout and not on an installed superset :) Said that looking forward to your PR!

Just FYI translations / i18n is mostly community contributed and not a top priority for the core committers. Though we're happy to enable others to translate and make translations work.

@CasperLiu
Hello, please, why do I follow your steps in the result of translation, the result still is English，
I'm doing the third step：
Extract the messages marked for translation
(venv) [root@centos4 site-packages]# fabmanager babel-extract --target superset/translations/ --output superset/translations/messages.pot --config superset/translations/babel.cfg -k _ -k __ -k t -k tn -k tct
It's running all the time，I stopped by direct ctrl+c；
Fourth steps  ：(venv) [root@centos4 site-packages]# fabmanager babel-compile --target superset/translations/
It's running all the time，I stopped by direct ctrl+c；
Is it related to this again?
How long will the two orders be carried out

@CasperLiu Have you solved it?

Looks like translations stopped working somewhere between 0.25.6 and 0.26.0rc1

I think https://github.com/apache/incubator-superset/pull/5008 broke translations

Make sure these boxes are checked before submitting your issue - thank you!

[x]  I have checked the superset logs for python stacktraces and included it here as text if any
[x]  I have reproduced the issue with at least the latest released version of superset
[x]  I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
v0.24.0
Expected results
Thanks a lot for the great Superset team to provide brilliant functionalities, and features continuously!
Background, in our company we are popularize Superset to finance division, so I am doing Chinese translation work based on v0.24.0. I know github repo already has a Chinese translation, and I'd like to expect: the Chinese messages could be shown on Superset webpage, especially the menu implemented by React, so that users could have a better understanding of how to use Superset.
Actual results
With Superset v0.24.0, even if we have switched to Chinese locale on the Superset webpage, user still cannot see Chinese translation content on the chart menu part.
So the Chinese translation part is broken in v0.24.0.
Steps to reproduce

Open Superset webpage, and switch to Chinese locale on the top-right button of the webpage.
Open a Chart(Slice), check the left part of the webpage and see it is still Englis prompt, no Chinese messages.

Below is what I have seen after the above reproducing:
Top bar menus are good in Chinese, but left menus are still in English(PS: the left menus indeed have Chinese translation in superset/translations/zh/LC_MESSAGES/messages.po/json

Below is how I solve the Chinese translation missing issue:
I followed below procedure guide and I have the Chinese translation worked for Chart menu part.
https://github.com/apache/incubator-superset/blob/master/CONTRIBUTING.md#translations

Go to top directory of superset directory.
(venv) [root@centos4 site-packages]# pwd
/root/Dev/Superset/venv/lib/python2.7/site-packages
Modify babel.cfg to add static for js/jsx path, OTHERWISE "fabmanager babel-extract" will NOT extract the messages for translation in js/jsx files.

A big question, why there is NO static in the github repo?
Without the static in https://github.com/apache/incubator-superset/blob/master/superset/translations/babel.cfg, how could the messages in js/jsx be extracted?
Could you please help answer?
(venv) [root@centos4 translations]# cat babel.cfg
[ignore: superset/assets/node_modules/]
[python: superset/.py]
[jinja2: superset//templates/.html]
[javascript: superset/static/assets/javascripts/.js]
[javascript: superset/static/assets/javascripts/.jsx]
[javascript: superset/static/assets/visualizations/.js]
[javascript: superset/static/assets/visualizations/.jsx]
encoding = utf-8

Extract the messages marked for translation
(venv) [root@centos4 site-packages]# fabmanager babel-extract --target superset/translations/ --output superset/translations/messages.pot --config superset/translations/babel.cfg -k _ -k __ -k t -k tn -k tct
Compile and generate json file.
(venv) [root@centos4 site-packages]# fabmanager babel-compile --target superset/translations/
(venv) [root@centos4 site-packages]# po2json -d superset -f jed1.x superset/translations/zh/LC_MESSAGES/messages.po
Restart Superset and select Chinese locale.

After above operation, the translation messages could been seen on menu, as below:

If needed, I could contribute to solve this issue for Chinese locale, and submit to Superset github repo and catch a patch or later release, since I am familiar with Chinese.
Please don't hesitate if anything needed!
Thanks a lot in advance!
Regards,
Jiaxu
刘佳旭

hi,
i have tried in .33 version of superset,but its not working.can you please help me on this


rename uintmem to physaddr

It was named uintmem because uint is 32-bits and we needed a
physical address type that was kind of uint but really for physical
addresses and hence 64 bits, while, at the same time, not being
tied to a bit length (so u64 would not do).
This all goes back to the days of the 32-bit transition for K10 IIRC.
This is all very confusing so give it a name related to its function.
Signed-off-by: Ronald G. Minnich<EMAIL_ADDRESS>

May I recommend calling this u64 instead? We found when we needed an integral type to hold physical addresses that made the most sense (aside from newtype'ing the thing in Rust). Our rationale was that a physical address is just a 64-bit integer, as opposed to a virtual address, that could be represented in a usize.

Dan, reason I went with physaddr was that jmk, or charles, not sure who, long ago, felt it was important to distinguish that small set of uses/definitions.
Happy to revisit this at some point but I'd rather let it stay distinguished for now ... I have a feeling they had a reason for it. But maybe not, who knows.

Dan, I have asked Charles about this, and depending on the answer, we can see if your suggestion makes more sense.
I feel happy having someone with your experience look at this stuff and raise the questions, many thanks.


types/purl.go

https://github.com/docker/index-cli-plugin/blame/ec13fbbed3010abbd8dd3cca7d4f6952bafb2ba4/types/purl.go#L29

https://github.com/docker/index-cli-plugin/blame/ec13fbbed3010abbd8dd3cca7d4f6952bafb2ba4/types/purl.go#L29


Add options to element interaction rules

Add options to no-static-element-interactions and no-noninteractive-element-interactions rules that allow an implementor to specify the handlers that should be considered when triggering the rule.
I plan to start with just onClick for FB initially and expand to more handlers as I fix violations and understand patterns of use better. The options allow for a slower roll-out and less developer friction.

Coverage increased (+0.002%) to 99.524% when pulling f91fa0846a53ed9b7ea7d4a23975487e39dd4edb on jessebeach:options-for-interactive-element-rules into a36d8b08c2812f1b3856a270ba99bd7710f88615 on evcohen:master.

on another note, should this be a blacklist or a whitelist?

There are 68 possible handler props. The whitelist approach is less prone to accidental omitting and is shorter to specify. The recommended configuration provides a good baseline. It's where I want to get to in the FB codebase. I've found that elements with onMouseOver and onMouseOut aren't always the best site to also include focus/blur handling. Guidance here isn't always straightforward.

Coverage increased (+0.001%) to 99.523% when pulling ed0074dbcd4df8a5ec6287aaaecd4e35af650b56 on jessebeach:options-for-interactive-element-rules into a36d8b08c2812f1b3856a270ba99bd7710f88615 on evcohen:master.

@evcohen requested changes have been made. The argument to keep the options as objects is a good one. I'll keep it in mind for future changes.

Coverage increased (+0.002%) to 99.524% when pulling 71299a0ed10a2a060507a9b35bb648e17af96d6e on jessebeach:options-for-interactive-element-rules into a36d8b08c2812f1b3856a270ba99bd7710f88615 on evcohen:master.

LGTM! ah one more thing- lets add this to the docs

I am literally the worst at remembering to do this!

@evcohen docs updated.

Coverage increased (+0.002%) to 99.524% when pulling acb6f31121f3e43e82989207d08e844282536728 on jessebeach:options-for-interactive-element-rules into a36d8b08c2812f1b3856a270ba99bd7710f88615 on evcohen:master.


GOPATH's multiple paths

bo-mbp:bo [~/src/go/src]I> go-bootstrap -dir foobar/
2015/05/12 14:47:20 Creating /Users/bo/.gocode/src/foobar...
2015/05/12 14:47:20 Copying a blank project to /Users/bo/.gocode/src/foobar...
2015/05/12 14:47:20
chdir /Users/bo/.gocode:/Users/bo/src/go/src/github.com/go-bootstrap/go-bootstrap/blank: no such file or directory

My $GOPATH contains a list of paths, so everything I go get ends up in the first one, everything I do myself is located in the second. So when I go get go-bootstrap, it's in ~/.gocode/, but when I want it to create a new project, it should be in ~/src/go/.
So the error is in main.go#L43, this should use the gopath variable. But before I change stuff I wanted to check back if you'd be ok with -dir being a relative path to the last part of $GOPATH. Maybe we can even check if $PWD is inside of a part and use that one.
What do you think?

But before I change stuff

whatever :)

Apparently filepath.Abs(filepath.Dir(os.Args[0])) doesn't work. Please don't merge this yet.

By default, We use the first $GOPATH, does that work with you? See: https://github.com/go-bootstrap/go-bootstrap/blob/b5ccd89ab3571490eb7cdb7c681f1feca1eb9d4f/main.go#L25
Oh, you exposed the bug where we actually didn't use the first $GOPATH on https://github.com/go-bootstrap/go-bootstrap/blob/b5ccd89ab3571490eb7cdb7c681f1feca1eb9d4f/main.go#L43. I just fixed that.

By default, We use the first $GOPATH, does that work with you? See: #L25

The problem is: you cannot depend that you're located in $GOPATH or even the first one. I installed go-bootstrap in the second part, because that's the one I'm working in, it didn't work. The first part is the one go get uses.

Out of curiosity, which dependency management library do you use? gvm? or is it homegrown?

Currently none, I'm looking into gb though.
It's just that the GOPATH is intended to work with different directories, but you simply don't support any other directory than the first one. I just felt this needed fixing. :)

I just added a flag, -gopath, it allows you to choose which path in $GOPATH to use.
Does that work for you?

I think we have a misunderstanding.
The problem is this:
go-bootstrap needs to know where it's directory is. It assumes it's go getted, although that might not be the case.
My suggestion is:
Let the program determine where it is and use that directory. That way you don't even have to think about that. If that's possible why should the user even bother?

If you don't like the way I was locating the binary, maybe we could change that.
Something like filepath.Abs() and if it's not found, check PATH. Like they do it in Gogs' ExecPath

Sorry, last night work was really busy.
To be honest , I am having a hard time picturing this use case, maybe you can play with it and provide tests afterward so I can see and play with it?

The use case is: having multiple GOPATHs and not working in the first one. It's that simple.

The second suggestion, filepath.Abs(), is that path to the binary?

path/filepath.Abs() returns the absolute path to a given path. That's for calling via relative or absolute path, and os/exec.LookPath() searches for the given executable in PATH, if you called go-bootstrap via your PATH.

You should use https://github.com/kardianos/osext for locating binary path.

@matrixik Good find, you're right.

@didip So would you be ok with that?

Seems like using osext is the way to go. Give it a try!

Just committed. While I was at it I handles a few other edge cases and dropped the previous commits, I hope you're ok with that.

Thanks for the contribution, I will definitely look at it over the weekend!

@Knorkebrot Can you move all the helper functions under /helpers package?

Done, sorry that this took so long.


[ERROR] Wait for scheduler end, this may take some time...

The python version I am currently using is 2.7+, I installed vlcp.
When I run the following code, there is a lot of wait notifications. Windows10 and Linux 6.5 are the same.
pip install vlcp
python -m vlcp.service.debugging.console
Wait for scheduler end, this may take some time...
Wait for scheduler end, this may take some time...
Wait for scheduler end, this may take some time...
Wait for scheduler end, this may take some time...
....

Thanks for report, seems a recent modification breaks the debugging console.

I'v fixed it in master, and tested it in Windows and Linux. Can you try the master branch and confirm it?


Merge dav1d assembly for AArch64, by release

Releases of dav1d

[ ]  0.1.0
[ ]   0.2.0
[ ]   0.2.1
[ ]   0.2.2
[ ]   0.3.0
[ ]   0.3.1
[ ]   0.4.0
[ ]   0.5.0

Extract dav1d subtree history
for TAG in `git -C dav1d tag -l`
do git -C dav1d subtree split -P src/arm/64 -b "src-ar-$TAG" "$TAG"
done

Apply to rav1e, restoring the prefix
git -C dav1d format-patch --no-stat --keep-subject --stdout --full-index -M100% \
--src-prefix=a/src/arm/64/ --dst-prefix=b/src/arm/64/ ..src-arm-64-0.1.0 -- '*.S' |
git -C rav1e am --keep

Reference #1750

This process will be a little bit different since we don't have ARM assembly in rav1e yet. But adding dav1d's ARM assembly is a great plan!

@shssoichiro Yeah,
We have been trying to make the build system better for AArch64
and with this we are slowy having the dav1d ARM Assembly.

@vibhoothiiaanand Great work, it would be amazing if rav1e works faster on ARMv8 CPUs, especially as rav1e's multithreading gets better.

Ok,
Here are the new updates from this week
We have tried to incorporate CDEF changes but it is not so doable easily, we have to rework on memory buffers to incorporate changeable stride for input buffer which would take time and would be a blocker. So what we are going to do now is, have all x86 integrated changes for AArch64 also and have it pass all tests and do a benchmark.
Here is the summary of changes from upstream

0.2.0 has CDEF, ARM Optimisations for MC, LR Improvements
0.2.1 has smart padding for CDEF
0.2.2 has SGR Looprestoration, Loopfiltering
0.3.0 No changes for src/arm/
0.3.1 has cold attribute changes+ msac_decode_symbol_adapt
0.4.0 has msac_decode_bool, blend, w_mask, inv_txfm_add
0.5.0 has msac optimisations, blend_h,blend_h,w_mask,intra_pred_(dc/h/v),paeth,smooth, palette,filter,cfl_pred,cfl_ac.

So interesting parts comes from 0.4.0
List of commits including 0.2.0 till 0.5.0 which are relevant for integrating are in green and the commits which will be added but not going to be integrated is in red, this is made in accordance based on x86 intregration from dav1d.
- arm64: ipred: NEON implementation of the cfl_ac functions
+ arm64: ipred: NEON implementation of the cfl_pred functions
- arm64: ipred: NEON implementation of the filter function
- arm64: ipred: NEON implementation of palette prediction
+ arm64: ipred: NEON implementation of smooth prediction
+ arm64: ipred: NEON implementation of paeth prediction
- arm64: mc: Use addp instead of addv+trn1 in warp
- arm64: cdef: Improve find_dir
- arm64: cdef: Calculate two initial parameters in the same vector
- arm64: cdef: Use loads with postincrement in more places in the padding function
- arm64: cdef: Rewrite an expression slightly
- arm64: mc: Schedule instructions better in the warp8x8 functions
- arm64: mc: Use sbfx instead of ubfx+sxth in the warp function
+ arm64: ipred: NEON implementation of dc/h/v prediction modes
+ arm64: itx: Fix overflows in idct
+ arm64: itx: Consistently use the factor 2896 in adst
+ arm64: itx: Use smull+smlal instead of addl+mul
+ arm64: itx: Do the final calculation of adst4/adst8/adst16 in 32 bit to avoid too narrow clipping
- arm64: mc: NEON implementation of w_mask_444/422/420 function
- arm64: mc: NEON implementation of blend, blend_h and blend_v function
- Add msac optimizations
+ arm64: itx: Add NEON optimized inverse transforms
+ arm64: Consistently name macro arguments tX for temporaries in transposes
- arm64: msac: Add handwritten versions of msac_decode_bool functions
- arm64: msac: Fix a typo in a comment
- Add __attribute__((cold)) to rarely used functions
- arm64: remove invalid macro argument delimiter
- arm64: msac: Implement NEON msac_decode_symbol_adapt
- arm64: loopfilter: Implement NEON loop filters
- arm64: looprestoration: Add a NEON implementation of SGR
- arm64: cdef: Clarify a slightly confusing comment
- arm64: cdef: Use a smarter padding constant
- arm64: cdef: Do saturating subtractions to avoid max operations with 0
+ fix dav1d spelling
- arm64/ios: use prefixed dav1d_mc_warp_filter symbol
- arm64: mc: NEON implementation of warp8x8{,t}
- arm64: cdef: NEON implementation of the dir function
- arm64: cdef: NEON optimized cdef filter function
- arm64: looprestoration: Optimize loop termination checks in copy_narrow_neon
- arm64: looprestoration: Simplify the horizontal filtering of one pixel at a time
- arm64: looprestoration: Simplify the setup of wiener_filter_v_neon
- arm64: looprestoration: Fix the loop condition in copy_narrow_neon
- arm64: looprestoration: Fix comment typos
- arm64: looprestoration: Avoid unnecessary alignment of the mid buffer
+ arm64: mc: Optimize mc_8tap_regular_w4_hv_8bpc for A53
+ arm64: mc: Simplify the 8tap_2w_hv code slightly
+ arm64: mc: Optimize the mul_mla_8_* macros for Cortex A53
+ arm64: mc: Improve a comment
+ arm64: mc: Remove unused/unnecessary macro args
- arm64: mc: Use ubfx instead of ubfm, for consistency with arm
- arm64: looprestoration: NEON optimized wiener filter
- arm64: mc: Implement 8tap and bilin functions  ```


Searching camel case method names with include comments yields odd results with operator AND enabled

The number of results is much higher, since it's not ANDing the split terms together. This is likely due to something on how MultiFieldQueryParser works.
The results don't AND all the terms in the camel casing, like it does with the standard parser (when include comments is off).
See what can be done to make more consistent.

If you put spaces between each camel case part, you get the expected results. However, you shouldn't have to add spaces, since the camel case would break into separate tokens


Support versioning of persistent data structures

This PR adds versioning support of persistent data structures via flatbuffers. It addresses the problem of changing serialized on-disk structures with version upgrades. To date, VAST cannot access data of older incompatible versions. Using flatbuffers allows for specifying the binary format in portable manner that supports a robust upgrade path.
There's also a performance benefit. Due to aligned data representation and
ability to perform random access, it plays well with our builtin memmory-mapped archive.

@tobim after reviewing, this is now ready for merge into the epic/integration branch.

Thanks for the nix patch! Is there a way to specify a minimum version? We require 1.11.0.

As discussed in person, it is not necessary / natural to set a version for each package. Instead, all packages are versioned in combination by the version https://github.com/NixOS/nixpkgs, which we pin to a specific commit at nix/pinned.nix.


Feature request: Digital number only option for data OCR.

Describe the pain point and your solution
I found that English OCR may fail on digitals numbers, An digital only mode maybe help.
Mode which would include change

[ ] Settings Window

Screenshots or sketches
OCR result Example:

What mode are you using when doing the capture? Fullscreen capture or Grab Frame?

If you highlight the text blocks you want you can right click and choose, "convert to number" to fix common number/letter conversions


More aggressive banner style...

that also looks nicer, not tested.
Is more consistent with the banners on things were you fill out question.
Old:

New:

Slightly different green:


Remove or polyfill the use of Symbol in the examples

The  real word example makes use of JavaScript Symbols. This doesn't work in older browsers¸ also Create React App doesn't provide any polyfill for Symbols.
I believe that this should be removed from the example. Because It is only used once, and a polyfill needs some optimizations require it.
What do you think?

Sure, I don't think it's important there. Send a PR?


Remove spaces from file name listing

[current]
there are currently 32 char padded with file name
[expectation]
kindly remove that many spaces when the file names are not long enough.

try to keep length of padding to the most lengthy file name + 2 spaces should be enough.

resolved by 9e6c92


Add node affinity feature enhancements  REP

The value of introducing the Labels mechanism for scheduling was discussed in this REP（[Add labels mechanism] #13 ） before. NodeAffinity is now discussed separately from this REP.
In fact, everyone should know about NodeAffinity before. I think we can reach an agreement on the following points in this REP.

[ ] Agreed on API format
[ ] The main implementation plan is agreed
[ ] Ask specific single-point questions, and then discuss and reach a consensus
[ ] Some specific details can be discussed during the coding process.

This REP seems like it would be blocked on the autoscaler / GCS refactoring proposal, right @scv119 ? Do we need to have that out first before reviewing this one?

This REP seems like it would be blocked on the autoscaler / GCS refactoring proposal, right @scv119 ? Do we need to have that out first before reviewing this one?

Thanks for checking it so quickly, Now i'm still just a draft, i finished it today, and i'll @you to help review it.

@scv119  @ericl @stephanie-wang

We understand that you also have demands for Runtime Env acceleration, and I have also added a proposal to use Node Labels to achieve Runtime Env acceleration, you can take a look.

Do you have any comments on the two REPs Node affinity and Actor Affinity? Can you help advance the progress of these two REPs? This way I can start developing as soon as possible.

Use Node Labels to achieve Runtime Env acceleration:

oh I think it is the same comment as https://github.com/ray-project/enhancements/pull/22#discussion_r1101633839 actually. So I believe it is addressed


[incubator/zookeeper] Fixes compatibility with 1.16

Make zookeeper compatible with 1.16. A quick overview of deprecated APIs can be found on their blog post: https://kubernetes.io/blog/2019/07/18/api-deprecations-in-1-16/.
Signed-off-by: Xiang Dai<EMAIL_ADDRESS>

/ok-to-test

@zanhsieh test pass!

@lachie83
@kow3ns
Could you guys approve this? Thank you.


Dummy backend on linux permissions issues

I've noticed that on linux, there are still lingering issues with the dummy backend, particularly during cleanup. Since the container runs as root, any files written into the bind mount from the container are owned by root and usually write protected. This causes issues when the host tries to clean up the staging directory after killing the cluster.
I think the easiest workaround would be a two part approach:

Change root's umask in the docker image to be more permissive
Invoke chmod -R in the controller right before cleanup

I like that this doesn't involve the use of sudo, which may not be available to a user

What about adding the user to the group docker, and having all containers run under the user, rather than under root?
This is the default behavior for the DockerTransient backend, and it works well.

I think we can assume they're already in the docker group if the dummy backend successfully launches, but wouldn't baking their uid into the image only work for non-docker jobs that run directly in a worker container that we control? If someone tries to run a script that uses docker we'll run into the same problem

Oh right. Let's go with the chmod -R, and pass in the UID/GID of the user through an environment variable (-e flag):
We do this in wolF here: https://github.com/getzlab/wolF/blob/5e7ee233e913024a705f306a2d660ac6affcbf69/wolf/task.py#L303


Exception on Bluebird 3.1.2 and node 0.12.2

Hi,
I am getting this exception after upgrading to Bluebird 3.1.2:
TypeError: Cannot redefine property: length
at defineProperty (native)
at Function.Promise.coroutine (/home/ubuntu/services/sender/source/node_modules/bluebird/js/release/generators.js:188:5)
Looks like something that was added in https://github.com/petkaantonov/bluebird/commit/6bd757e8595c9d58191d0d22d66100f1dee3b234
Notice that I am using node 0.12.2, maybe length cannot be overridden there ?

Sigh. I had a feeling it was too good to be true that there was a solution to modify .length

Node 0.12 has generators? I didn't even remember that, is that with a flag or without it?
Anyway, this concerns me since having programs behave differently in two different versions of node is probably a bad idea.

with --harmony

@petkaantonov to be fair, it's supposed to be configurable http://www.ecma-international.org/ecma-262/6.0/index.html#sec-functioninitialize so 0.12 is just using an old buggy version of v8 here - we can put a caveat for Node users with --harmony.

I am fine with just adding a warning. 0.12 is indeed really old.

@benjamingr still breaking code is not worth it here at all, it's just a pointless "nice to have" thing.

Thanks for the notice.


add sagemaker-experiments

add sagemaker-experiments dependency to tf-2
related https://github.com/aws/sagemaker-tensorflow-container/pull/286

LGTM.


Visitor Pattern

In some cases, especially when copying between groups or searching for something in a group, it helps to have a visitor pattern by which one can visit all objects in a Zarr group and do something with them. h5py has the nice visit and visititems methods for Group. It would be really handy to have these same methods for Zarr Groups.

Yes this would be nice. Unfortunately I don't have time to work on this at
the moment but would be happy to review a PR.
On Wednesday, November 30, 2016, jakirkham<EMAIL_ADDRESS>wrote:

In some cases, especially when copying between groups or searching for
something in a group, it helps to have a visitor pattern by which one can
visit all objects in a Zarr group and do something with them. h5py has the
nice visit
http://docs.h5py.org/en/latest/high/group.html?highlight=recurse#Group.visit
and visititems
http://docs.h5py.org/en/latest/high/group.html?highlight=recurse#Group.visititems
methods for Group. It would be really handy to have these same methods for
Zarr Groups.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/alimanfoo/zarr/issues/92, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAq8Qq9wq75ksR6uSleLeyXfN5O1bEbNks5rDbcCgaJpZM4LAfF9
.

--
Alistair Miles
Head of Epidemiological Informatics
Centre for Genomics and Global Health http://cggh.org
The Wellcome Trust Centre for Human Genetics
Roosevelt Drive
Oxford
OX3 7BN
United Kingdom
Email<EMAIL_ADDRESS>Web: http://purl.org/net/aliman
Twitter: https://twitter.com/alimanfoo
Tel: +44 (0)1865 287721

Ok, cool. I'll take a look.

Rough implementation of these in PR ( https://github.com/alimanfoo/zarr/pull/122 ).


Please add 1st-party

Please change first sentence to:
Azure NetApp Files is an Azure native, 1st-party, enterprise-class, high-performance file storage service.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: f836e991-a695-12c2-2336-84d4e9efe93e
Version Independent ID: 6826b468-635a-1f28-a6d9-7ead166b50d7
Content: What is Azure NetApp Files
Content Source: articles/azure-netapp-files/azure-netapp-files-introduction.md
Service: azure-netapp-files
GitHub Login: @b-hchen
Microsoft Alias: anfdocs

@gvanteylingen
Thanks for your feedback! We will investigate and update as appropriate.

Fixing in PR https://github.com/MicrosoftDocs/azure-docs-pr/pull/224873

PR merged for publishing.

#please-close

#please-close


Doesn't let me use it in Hardcore mode

I tried turning on the Datapacks only to find out the options weren't there! And I was also playing on Singleplayer! Is it a bug or it's only made for Single-SINGLEPlayer? Not Hardcore?

What?

Didcha fix it? If not, reopen the issue.


Unable to run funnel --help

Issue Template
After installation, when I try to run funnel --help, i get error saying:
funnel : The term 'funnel' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of
the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1

funnel --help

  + CategoryInfo          : ObjectNotFound: (funnel:String) [], CommandNotFoundException
  + FullyQualifiedErrorId : CommandNotFoundException

Steps to Reproduce

Hi There,
Could you give us more information about your OS/hardware? I suspect this is Windows?
If it is Windows, please let us know exactly how you installed JobFunnel? Are you using PowerShell? CommandLine prompt?Windows Terminal? Also did you install The new JobFunnel 3.0? That has not been released yet and nor has it been thoroughly/peer-reviewed tested, so you should not be using that version yet. I will be reviewing later it today, but I suspect it will be some time before we can release it because Paul probably wants as many people to review it as possible.
Like I said, please give us more details about the environment you have JobFunnel running under.

Hi..
I ran this command:
pip install git+https://github.com/PaulMcInnis/JobFunnel.git
OS: Windows
I ran it on the powershell terminal of VS code.
Also I have a folder generated after I run the above command: JobFunnel-2.1.9-py3.8.egg-info
So I am guessing the version is 2.1.9..
Sorry I am kind of a noob. Just trying to experiment on Linkedin
Thanks in advance

I recommend not installing JobFunnel through VS code's powershell. Try to install it by using the stand-alone PowerShell terminal. I don't use Windows, but I highly suspect VS Code is isolating JobFunnel on its own virtual environment. Again, not a Windows user, just a hypothesis.
Also, regarding LinkedIn, are you trying to write a scraper for LinkedIn? That would be really awesome to have, but I don't recommend it. Last I checked, they really do not like people to scrape their site.
I would check in with @PaulMcInnis(the maintainer of the repo) about writing a scraper for LinkedIn. Because for all I know, I could be wrong about the whole LinkedIn situation.
I don't have a Windows Environment ready at the moment, but I will try to reproduce your issue if I get time. No promises though. I highly suspect this might be because of VS Code.
It might also help if you give us your Python and pip version.
Hope this helps!

Thank you.. will try that..
Regarding Linkedin,.. i am just trying to learn about web scraping... if Linkedin doesn't like it, probably will try it with other websites..
My python version: 3.8
Pip: 20.1
Again, I appreciate all ur help!

Not a problem! That's really awesome you're implementing a new web scraper for JobFunnel. I know I said 3.0 isn't ready yet, but if it is for development, then I do suggest using 3.0 :smile:. For development, NOT regular use. The reason I really think you should be using 3.0 is because we have a a much better architecture for the code base that makes it easier for web scraper authors like you to deal with stuff like internationalization  and dynamic/static scraping. 3.0 is currently being worked on(and ALMOST ready to be shipped). 3.0 is currently on the ABCJobFunnel branch.
Since you are trying to develop on JobFunnel, do NOT install it  with pip install git+https://github.com/PaulMcInnis/JobFunnel.git. That's for regular use. To make your life easier as a developer, install it like this:
git clone git@github.com:PaulMcInnis/JobFunnel.git jobfunnel
pip install -e ./jobfunnel
funnel --help

Like I said, 3.0 is not ready yet. It is currently under review on #90. So be on the lookout for any changes/decisions that are made for this new release if you will be developing for JobFunnel.
Happy coding!

Hey there @manohar52, I'll chime in to echo @thebigG and recommend that you check out the branch ABCJobFunnel on this repo, though there are still some bugs with CLI handling currently, it is quite functional. Once this branch is merged into master it should be simple enough to rebase.
To write a scraper, you should start by taking a look at the indeed scraper and the base scraper, these should be enough to get you started.
There is currently an issue #52 where we are keeping a list of websites we want to scrape jobs from, google's job site looks like a really good one.

As for the bug you are reporting, it seems that there may be an issue with VSCode and Powershell, which may be worth fixing if we can determine what's causing it. For further discussion on writing a scraper feel free to create a new issue with the feature label.


Refactor TextStyle

Right now to add/remove properties to TextStyle requires a bit of boilerplate. I'd like to change it to be simpler, where all you need to do is add a property and it works.

@englercj Is this still needing work?
Could you give and example of the boilerplate required and what the outcome should be?
I'll be happy to pick this up with a bit more info.

This is still something we'd like to do.
@bigtimebuddy had a good idea about ultimately having a single TextStyle that could work across different types, like Text and BitmapText that we have now. These classes would need a way to register to TextStyle which properties they support. And TextStyle could generate a lot of the setters / getters in code, rather than having to list them all out.

Just to see if im understanding this correctly, would something like this be what you're looking for?
Creating a text style with bitmap set to true would allow access to properties unique to Bitmap Text, and the default would access regular text properties?
new PIXI.TextStyle({ bitmap: true, tint: #ccc })


[test] Add integration_test for PandasAgent

confirm creation
confirm functionality with a simple dimension check.

The test now is calling OpenAI API directly, but learning from @vowelparrot that we’re caching the requests, so that it’s not that expensive. I also found we’re calling OpenAI api in other integration tests. Please lmk if there is any concern of real external API calls. I can alternatively make a fake LLM for this test. Thanks

Yeah, I was thinking of adding a scope=class fixture function which can
mock up LLM or Agent for all test functions once together in the very
beginning, but later I realized this kind of mock up doesn’t exist much
yet. To make it right and reasonable, I changed my plan to start the
mocking with LLM models first. When the proposal is aligned with everyone
else, I’ll gradually add it to all tests from LLM, Chains to Agents here.
Just decided to move forward in a longer but safer path. :)
On Wed, May 3, 2023 at 6:02 PM Davis Chase @.***> wrote:

@.**** commented on this pull request.
thanks @skcoirz https://github.com/skcoirz! quick question
In tests/integration_tests/agent/test_pandas_agent.py
https://github.com/hwchase17/langchain/pull/4056#discussion_r1184440919:

df: DataFrame

dim: int

class Config:

   arbitrary_types_allowed = True

@.***(scope="function")
+def data() -> TestData:

dim = 4
random_data = np.random.rand(dim, dim)
df = DataFrame(random_data, columns=["name", "age", "food", "sport"])
return TestData(df=df, dim=4)

+class TestPandasAgent:
out of curiosity, why implement as class instead of just having test
functions?
—
Reply to this email directly, view it on GitHub
https://github.com/hwchase17/langchain/pull/4056#pullrequestreview-1412102550,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO64MH2E2LG5RQ2NXRDDZ5LXEL52ZANCNFSM6AAAAAAXU2KOLQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

The benefit of this mock up is that we can avoid calling to the external
APIs and at the same time make each LLM output more certain (the real model
might generate different answer every a while even when the temperature is
0) I think this can be helpful for multi-tools collaboration functionality
test where all tools will be called one by one with a series of
predetermined LLM outputs.
This mock up won’t replace the real API call, but when our focus is on the
tooling functionality inside of Agent, we will need the certainty to make
sure all of them can be called in the expected way.
On Wed, May 3, 2023 at 6:33 PM Mike Wang @.***> wrote:

Yeah, I was thinking of adding a scope=class fixture function which can
mock up LLM or Agent for all test functions once together in the very
beginning, but later I realized this kind of mock up doesn’t exist much
yet. To make it right and reasonable, I changed my plan to start the
mocking with LLM models first. When the proposal is aligned with everyone
else, I’ll gradually add it to all tests from LLM, Chains to Agents here.
Just decided to move forward in a longer but safer path. :)
On Wed, May 3, 2023 at 6:02 PM Davis Chase @.***>
wrote:

@.**** commented on this pull request.
thanks @skcoirz https://github.com/skcoirz! quick question
In tests/integration_tests/agent/test_pandas_agent.py
https://github.com/hwchase17/langchain/pull/4056#discussion_r1184440919
:

df: DataFrame

dim: int

class Config:

   arbitrary_types_allowed = True

@.***(scope="function")
+def data() -> TestData:

dim = 4
random_data = np.random.rand(dim, dim)
df = DataFrame(random_data, columns=["name", "age", "food", "sport"])
return TestData(df=df, dim=4)

+class TestPandasAgent:
out of curiosity, why implement as class instead of just having test
functions?
—
Reply to this email directly, view it on GitHub
https://github.com/hwchase17/langchain/pull/4056#pullrequestreview-1412102550,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO64MH2E2LG5RQ2NXRDDZ5LXEL52ZANCNFSM6AAAAAAXU2KOLQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

The benefit of this mock up is that we can avoid calling to the external APIs and at the same time make each LLM output more certain (the real model might generate different answer every a while even when the temperature is 0) I think this can be helpful for multi-tools collaboration functionality test where all tools will be called one by one with a series of predetermined LLM outputs. This mock up won’t replace the real API call, but when our focus is on the tooling functionality inside of Agent, we will need the certainty to make sure all of them can be called in the expected way.
…
On Wed, May 3, 2023 at 6:33 PM Mike Wang @.> wrote: Yeah, I was thinking of adding a scope=class fixture function which can mock up LLM or Agent for all test functions once together in the very beginning, but later I realized this kind of mock up doesn’t exist much yet. To make it right and reasonable, I changed my plan to start the mocking with LLM models first. When the proposal is aligned with everyone else, I’ll gradually add it to all tests from LLM, Chains to Agents here. Just decided to move forward in a longer but safer path. :) On Wed, May 3, 2023 at 6:02 PM Davis Chase @.> wrote: > @.**** commented on this pull request. > > thanks @skcoirz https://github.com/skcoirz! quick question > ------------------------------ > > In tests/integration_tests/agent/test_pandas_agent.py > <#4056 (comment)> > : > > > + df: DataFrame > + dim: int > + > + class Config: > + arbitrary_types_allowed = True > + > + > @.(scope="function") > +def data() -> TestData: > + dim = 4 > + random_data = np.random.rand(dim, dim) > + df = DataFrame(random_data, columns=["name", "age", "food", "sport"]) > + return TestData(df=df, dim=4) > + > + > +class TestPandasAgent: > > out of curiosity, why implement as class instead of just having test > functions? > > — > Reply to this email directly, view it on GitHub > <#4056 (review)>, > or unsubscribe > https://github.com/notifications/unsubscribe-auth/AO64MH2E2LG5RQ2NXRDDZ5LXEL52ZANCNFSM6AAAAAAXU2KOLQ > . > You are receiving this because you were mentioned.Message ID: > @.> >

sorry i still dont understand why having the test as a class instead of functions helps
this is partially on us - we need to better define how tests should be written, and make sure they are done consistently


Feature/lua #42

This PR adds Lua as an editor scripting language, enabling you to write plugins and edit keybinds from an init.lua file instead of through Java and GUIs. This is all possible due to gudzpoz's fantastic LuaJava project.
Notes

The init.lua file must be located in EditorActions.getConfigDir() + "/lua"

On MacOS: ~/Library/Application Support/Graphedit/lua
On Linux: ~/.local/graphedit/lua
On Windows: %APPDATA%\graphedit\lua (untested for now)

Take a look at the lua bindings types repository to get an idea of what is possible

Minimal Example
ge.log:info("Hello " .. ge.info.name .. " " .. ge.info.version .. "!")

-- Get stuff from the Dependency Injection container
-- TODO: manually importing like this may become a bit cumbersome, maybe we should add a helper function for this
local IBufferContainer = java.import("dk.gtz.graphedit.viewmodel.IBufferContainer")
ge.api.dep:get(IBufferContainer)

-- Add plugins
ge.api.add_plugin({
    name = "Example",
    description = "Example plugin",
    on_start = function()
        ge.log:info("Hello from the Example lua plugin!")
    end,
    on_destroy = function()
        ge.log:info("Goodbye from the Example lua plugin!")
    end,
    -- TODO: These are not supported yet, but I will add them if interest shows.
    -- syntax_factories = {},
    -- panels = {},
    -- importer = function(paths) end,
    -- exporter = function(resource, new_path) end,
})

-- This will overwrite the default keymap if you add a keymap with the same key combination
ge.api.keymap.set("Shortcut+K", function() ge.api.toast:success("Hello from the keymap!") end, "Greeter")
ge.api.keymap.set_in_category("Shortcut+Shift+L", function() ge.api.action:openKeybinds() end, "Open Keybinds", "help")

This PR fixes #42


500 error when title includes unicode characters

Reported on behalf of an enterprise customer:
Hello
Noticed PDF renderer service throws 500 Internal Server Error error if there special character in title field
actual value used for title: Syngenta vetőmag ajánlat...Test user th
note: it works well if I remove special chars from the string

This appears to be caused by an errant space character in a regular expression that should be restricting the filename to a very simple set of supported characters.

Closed with PR #29


[RFC] Replacing unit tests by E2E tests?

Currently we are facing some troubles with unit tests. They are quite hard to write (for instance, mocking requestAnimationFrame is not trivial when testing zoom behavior). And they are also not easy to understand.
That's why I'm wondering if we shouldn't replace all of them by some E2E tests? It would be far slower, yet it would allow to test more easily our code.
What do you think all?

We reworked the whole test suite in 1.0.0, making tests easier to write. Adding E2E may be a good idea, but replacing unit tests by E2E is no more a requirement.


fix: allow show action to be chained

The description of this PR is below within line comments.

Good explanation. Makes sense to me.


Change docker image to the Node 20 UBI-image

LIIKUNTA-624.
Add documentation of how to use the docker composer locally with the cached image. Add some missing arguments to the docker composer build time arguments. Added the esbuild dependency to fix local docker builds.
(Try to) redo (better 😄 ) what was reverted here in https://github.com/City-of-Helsinki/events-helsinki-monorepo/pull/649/.

EVENTS GRAPHQL PROXY is deployed to platta: https://events-graphql-proxy-pr653.dev.hel.ninja :rocket::rocket::rocket:

VENUE GRAPHQL PROXY is deployed to platta: https://venue-graphql-proxy-pr653.dev.hel.ninja :rocket::rocket::rocket:

Diff between main just before reverting last UBI image changes and this branch:
https://github.com/City-of-Helsinki/events-helsinki-monorepo/compare/e5c8c03152a37edc920728c6c32911787c7ffe72...LIIKUNTA-624-docker-image-update

Diff between main just before reverting last UBI image changes and this branch:
e5c8c03...LIIKUNTA-624-docker-image-update

According to this proxy.Dockerfile is still using helsinkitest image.

VENUE GRAPHQL PROXY is deployed to platta: https://venue-graphql-proxy-pr653.dev.hel.ninja :rocket::rocket::rocket:

Looks ok to me, we need to test this in the CI/CD pipelines though so they work too!

Yep, I've been trying to find a solution how to update the base image of the CI-pipeline. It's not a simple thing, since the pipeline configuration of the baser image creation cannot be overridden any how. 😞 This leads to a situation, that the base image would use the old "Helsinki-test-image", but the app-build would use the ubi-image (that would then base on the helsinki-test-image). I think there are 3 different approaches to fix this:

Delete the cache image so it would be recreated properly. The problem with this is that no-one I know has privileges to do that (and it should be done with some terminal tools).
Disable the cache image usage for while and re-enable it once all the images are using the UBI. I'm not sure whether or not we would still have the issue when re-enabling it.
Merge this so that the cache-image can be forced to use the ubi-images only, because the cache-image pipeline is using app's master branch (and we cannot change it).

I will now try the 2nd and 3rd approaches: if the 2nd does not work, I'll continue with the 3rd.

Events-Helsinki branch is deployed to platta: https://tapahtumat-pr653.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://liikunta-pr653.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://harrastukset-pr653.dev.hel.ninja :laughing::tada::tada::tada:

VENUE GRAPHQL PROXY is deployed to platta: https://venue-graphql-proxy-pr653.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://tapahtumat-pr653.dev.hel.ninja :laughing::tada::tada::tada:

EVENTS GRAPHQL PROXY is deployed to platta: https://events-graphql-proxy-pr653.dev.hel.ninja :rocket::rocket::rocket:

EVENTS GRAPHQL PROXY is deployed to platta: https://events-graphql-proxy-pr653.dev.hel.ninja :rocket::rocket::rocket:

VENUE GRAPHQL PROXY is deployed to platta: https://venue-graphql-proxy-pr653.dev.hel.ninja :rocket::rocket::rocket:


Create Weather-App.md

📝 Description
App that allows users to check the current weather conditions for a specific location. The app fetches weather data from the OpenWeatherMap API and displays it in a easy-to-read format.
💣 Is this a breaking change (Yes/No):

[ ] Yes
[x] No

🚧 How to migrate?
Don't need.
📝 Additional Information
hacktoberfest

Hello, I think this data may not be very accurate. Because The 3.5 model is not real-time, we cannot merge this.

Hello, I think this data may not be very accurate. Because The 3.5 model is not real-time, we cannot merge this.

Thank you for reviewing the PR. I'd like to clarify that I am not using GPT and this is not an ai-agent. The weather info is based on the data provided by openweathermap API.

@nabinbhatt
Hello,It's my bad.I replied to the wrong PR.LOL

No worries at all!
By the way, I noticed the label on the PR is currently set to "ILLA with AI", but I believe it should be "build with ILLA". Could you please update the label accordingly?


Saving model with moment dates as attributes

I'm converting time stamp fields to moment objects in parse like this:
        parse: function(response) {
            // Remove fields which have null value.
            response = _.omit(response, _.isNull);

            // snake_case db columns -> camelCase attributes
            var attrs = objectKeysToCase(response, 'camel');
            if (attrs.createdAt) {
                attrs.createdAt = moment(attrs.createdAt);
            }
            if (attrs.updatedAt) {
                attrs.updatedAt = moment(attrs.updatedAt);
            }

            return attrs;
        }

Do I need to convert them back to normal Date objects in model's format function? Or is it enough that moment object provides the toJSON() function?

What happens if you don't convert them back to normal Date objects in model's format function? Are you having any problems with that?

I don't yet have any problems but I wanted to know how this is done internally in Bookshelf. I tried to skim through some code but didn't find the place where date attribute is converted to timestamp field for database.

I'm almost certain that is handled by Knex (or even the underlying database adapters) and not Bookshelf. Take a look here and here.

Thanks. Looking at the code: typeof moment() is "object" which means knex calls JSON.stringify to the object. This doesn't look very good:
> JSON.stringify(moment())
'"2015-06-15T11:40:47.369Z"'
> moment().toJSON()
'2015-06-15T11:40:58.349Z'

So, there's a bug after all? What exactly is the problem?

I'll get back to this at some point. Sorry but have no time to test it in more detail now. I'm not encountering any bug currently. But by looking at the code, it looks like a risk of bug:
If I don't handle moment objects in  parse function, dates are(or it seems so) be passed to database driver as correct ISO dates but with surrounding quotes.

JSON.stringify calls Object#toJSON() internally, so you're all good. More info on MDN.

Relevant discussion here: #246.

Closing this for lack of activity. If you need any further clarification about this issue we can reopen it.


[Bug] Unable to load campaign thumbnails

Describe the bug
Image thumbnail links on campaigns page appear to break sometimes
To Reproduce[hard]
Visit the campaigns page from time to time

Click on "Дарете/Donate"
See (can't) campaign thumbnail images.

Screenshots

Desktop::

OS: Mac OS X 12.4
Browser: Safari
Version: 15.5

Additional context
Does not always seem to happen, but It happened to me two times in the last 15 min so I am reporting it as a bug.

This does not seems to be currently valid.
Also according to the new design the campaign cards and the images are going to be updated.
I think we can close this issue. If the problem with the images is still there after the update, we can open another one referring to the new UI.


Godot 4.0 causes windows to crash/freeze when importing projects from 3.x

Godot version
v4.0.dev.custom_build [de4ad63e6]
System information
Windows 10 Home (20H2), Nivida GT 1030 (Vulkan), 12GBs of RAM.
Issue description
When Importing projects that contain a lot of asset files Godot appears at first to not be doing anything and throws out an error saying it's unable to enable active plugins (if your project has one). But in the background Godot seems to be hoarding all of the available RAM slowly until Windows starts stuttering and eventually freeze completely or crash as windows closes everything and signs you off. The CPU utilization is constant (around 13-14% on my i7-4790k) but there is no active disk read or writes during that time.
This project I was importing from 3.3 was around 100MBs large but still had a lot of sprites, gdscripts, sound files and large scenes and in 3.3 the project would take up to a minute and thirty seconds to load on startup, but in Godot 4 it refuses to load and does this behavior that I described above.
Steps to reproduce

Import a project that contains a lot of asset files from 3.x in the Project window of 4.0
If you have active plugins, The first thing you'll see is Godot showing you an error saying it's unable to enable these plugins.
Observe Godot slowly hoarding all your memory until Godot causes windows to stutter or crash.

Minimal reproduction project
I've used my own project as an example.
https://github.com/MaxIsJoe/TWCRemake

Update :
I tested with a 32-bit build of Godot 4.0, same godot version in the issue report above, Godot crashes after a few seconds now instead of infinitely hoarding memory until crashing the OS.
The console also spam these two types of errors very frequently until the editor fully loads.

Tried deleting the .import folder to see if it makes a change but it doesn't change anything.


rework process handling

migrate the process handling to c++ where appropriate
move code definition / usage closer together
add function to portably detect external processes (not spawned by us) and whether we may
send them signals
add portable code to send signals to processes
for linux match what impact a signal will have on the process we send it
check the status of the process we sent the signal, and return if its non-deadly or the process terminated
after 8 retries with 1 second sleep send a hard kill.
killExternal() now returns the same structure as statusExternal()
don't block signals on spawned processes

@dothebart


Template Strings

JSBin doesn't support template strings. Simple programs such as var x = This is a template string; will cause it to throw a large number of errors.
https://jsbin.com/tomidu/edit?js

Apparently this is because your browser doesn't support template strings.
Try switching from JavaScript mode to es6/babel

(I'm using the latest Firefox version. It does support ES6.)
/*jshint esnext: true */ solves the problem.

The warning on jsbin says something like "let is available in ES6, use esnext option" but doesn't actually tell you how to "use an option"... and picking ES6/babel from the menu doesn't fix it... so thank you for explaining how to actually do it


[NEAR] Deployment

will need assistance to handle the .env file for production

will need assistance to handle the .env file for production

@mikeplotean please jump in here! thanks!

@SuperBatata please merge everything so the web-wallet can be integrated with https://nftkit.walt-test.cloud/v2/nftkit/swagger