Anyone try fine-tuning 13B model?

Training the 7B model takes about 18GB of RAM.
I tried training the 13B model, and ran out of VRAM on my 24GB card.  I suspect, will need at least 32GB of VRAM.
Has anyone else been successful with fine-tuning a 13B model?

https://github.com/tloen/alpaca-lora/issues/14#issuecomment-1471273729
@ItsLogic what's your experience been like?

It has been quite good although understandably slow to train. I have a 4090 so a 24GB card is enough. The only thing you need to change to make it work is MICRO_BATCH_SIZE which I have set to 2.
The whole time training I was within a few hundred MB of an OOM so you might need to close a few background tasks when you decide to train.
Training time is ~10 hours for the full three epochs. I trained a single epoch(406 steps) in 3 hours 15 mins and got these results on 13B:

13B with lora

13B normal

Just a heads up the provided export_state_dict_checkpoint.py has the parameters set for 7B so you will need to change those to match the 13B params before you can use it.
It also only outputs one file at the end but the llama to HF conversion script works fine as long as you change the 13B shard count to 1 you plan on using transformers.

@ItsLogic Great results! Can you share your way to make it as a chat with memorized history? Do you include the whole chat history to "### Input:" field with the next prompt or what is the idea? It would be great if you can share your chat logic code!
Btw, how big chat history it can handle with decent memorization and ability to recover info from previous messages?

@ItsLogic Great results! Can you share your way to make it as a chat with memorized history? Do you include the whole chat history to "### Input:" field with the next prompt or what is the idea? It would be great if you can share your chat logic code!

I'm just using https://github.com/oobabooga/text-generation-webui with the --cai-chat launch arg as my GUI and it handles everything

Btw, how big chat history it can handle with decent memorization and ability to recover info from previous messages?

Honestly not sure. Haven't had any super long conversations so I can't speak for it's memory.

I'm just using https://github.com/oobabooga/text-generation-webui with the --cai-chat launch arg as my GUI and it handles everything

It's my understanding that you have to format your prompt in a specific way for this model, as is done here. I don't think text-generation-webui does that (yet).

@tloen @ItsLogic Guys, do you remember what was the minimal loss when you stopped the training?
And do you know (from papers or experience) what loss is considered great for such model/training goal? Thank you!

@baleksey Trained a 13B lora with one epoch as well, my loss was around 0.75 lowest. Text gen results don't feel too different for me than 7B though

@ItsLogic Guys, do you remember what was the minimal loss when you stopped the training?

I hovered around low 0.8 and high 0.7 from about 20% through the first epoch. It ended just above 0.8.
I just finished epoch 3 and got 0.78. took 9 hours and 20 mins.

Thanks! Just finished my 13B as well with 0.79.
Trying to finetune 33B with A6000. It uses 48.4GB out of 49 and says that 1 epoch will take about 12 hours (if there will be no OOMs definitely), oh my.. So interesting how all 7/13/33 model will be different in difficult questions and instructions.

you may need to also add max_memory={0: 18} to LlamaForCausalLM.from_pretrained if u run into OOM errors when fine-tuning the 13B model

Any hero out there got Alpaca-13B distributed yet?

Any hero out there got Alpaca-13B distributed yet? For those of us that lack a 24GB GPU sweat

I uploaded my epoch 1 and epoch 3 loras
https://huggingface.co/Draff/llama-alpaca-stuff/tree/main/Alpaca-Loras

Forgive my ignorance, but how do I download this? from_pretrained() doesn't support subdirectories.

@ItsLogic Forgive my ignorance, but how do I download this? from_pretrained() doesn't support subdirectories.

just download adapter_config.json and adapter_model.bin manually and put them in a folder and then edit the path in generate.py to point to that folder.

Can report the same as @ItsLogic e1 and e3 feels roughly the same, probably because the loss are both ~0.78.
Somewhat related, when I was trying the model out with textgen-webui, the outputs are incredibly short. Don't know if it's a problem with the webui or the model itself.

Any hero out there got Alpaca-13B distributed yet? For those of us that lack a 24GB GPU sweat

Not so sure the 13B model is gonna perform much better than the 7B right now, the stanford dataset has a ton of issues. I've been going through trying to fix them.

Any hero out there got Alpaca-13B distributed yet? For those of us that lack a 24GB GPU sweat

Not so sure the 13B model is gonna perform much better than the 7B right now, the stanford dataset has a ton of issues. I've been going through trying to fix them.
I've done a first best effort to resolve the issues, and I'm training a new 7b model right now, but my GPU is a potato, so I wont have anything to show until tomorrow.

Could you please elaborate on the issues you are seeing with the Stanford Alpaca dataset?

Hi @gururise

but my GPU is a potato

I'm co-founder of qblocks.cloud. We would love to offer to you some GPU credits to help with your research and experimentation on alpaca / lora. Can we connect some way?

Hi @gururise

but my GPU is a potato

I'm co-founder of qblocks.cloud. We would love to offer to you some GPU credits to help with your research and experimentation on alpaca / lora. Can we connect some way?

Would love to take you up on your offer to generate some fine-tuned Alpaca models using my cleaned dataset. I've sent you an email.

@tloen @ItsLogic Guys, do you remember what was the minimal loss when you stopped the training? And do you know (from papers or experience) what loss is considered great for such model/training goal?
Fine-tuning my 13B at the moment and guessing when it's "enough" :)

This is my loss curve figure for 13B model. Wondering if others are the same

Training time 7.5h on 3090.

I suppose Kaggle grandmaster's lesson of using 5-fold cross-validation also for DL models is out of the window then:)

@ItsLogic Can you show what your trainer args and hyper params are for the 13B training run? My models seem to take WAY longer than 10 hours to train. on 3090ti. Like a couple days at a time. :(

@ItsLogic nevermind. The longer training time definitely stemmed from cutoff len going from 256 to 512.

@ItsLogic I get the error while fine-tuning 13B on 2 quadro 24GB. Setting the micro_batch_size to 2 does not help either. What are your training params? the same code works fine for distributed finetuning of 7B model. So, I suspect I would have to reduce the size, not sure which one.
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 26.00 MiB (GPU 0; 23.65 GiB total capacity; 21.88 GiB already allocated; 8.31 MiB free; 22.82 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF
Traceback (most recent call last):
File "finetune.py", line 317, in
fire.Fire(train)

Update. bitsandbytes==0.37.2 solved the problem. I had bitsandbytes==0.39.0 previously.


Validation doesn't catch multiple services with the same container_name

The following configuration declares two services, each of which has the container_name specified.
But those container names are the same.
version: '3.4'
services:
  webservice1:
    container_name: foo
    image: nginx
  webservice2:
    container_name: foo
    image: nginx

I can run docker compose up with the above configuration.
But there will be an error at runtime from Docker,
so one of the two containers will not start.
Why doesn't compose validate this and issue an error before running any containers?

Thank you so much for the fast response! One question: would this fix make it to the v1 line of releases? Will there be any v1 further releases at all?

Hi there.

So docker compose up device-controllers would spin up
the required Compose services.
As you're aware containerised apps address services in
other containers by container name. Our containerised
controller app makes requests to the API server by
container name. The controller does not (and shouldn't)
care of what Compose service nor what Docker image is
driving this API server.
Now, starting containers with same names should throw an
error at runtime as it does. But is this not enough? What
is the issue with simply defining the services with same
names?
With this validation you're taking the logic flexibility out
of the docker-compose.yml file, and breaking our software.
Even though spinning up of conflicting containers would never
be attempted.


Matlab (R2015b) Compilation of Caffe-0.999

Hi,
I've successfully installed Caffe in $RCNN/external/caffe . However, when I run make matcaffe, I get the below error:
/usr/local/MATLAB/R2015b/bin/mex matlab/caffe/matcaffe.cpp build/lib/libcaffe.a
CXXFLAGS="$CXXFLAGS -pthread -fPIC -DNDEBUG -O2 -I/usr/include/python2.7 -I/usr/lib/python2.7/dist-packages/numpy/core/include -I/usr/local/include -Ibuild/src -I./src -I./include -I/usr/local/cuda/include -Wall"
CXXLIBS="$CXXLIBS -L/usr/lib -L/usr/local/lib -L/usr/lib -L/usr/local/cuda/lib64 -L/usr/local/cuda/lib -lcudart -lcublas -lcurand -lpthread -lglog -lprotobuf -lleveldb -lsnappy -lboost_system -lhdf5_hl -lhdf5 -lopencv_core -lopencv_highgui -lopencv_imgproc -lopenblas" -o matlab/caffe/caffe.mexa64
Unknown MEX argument '-o'.
make: *** [matlab/caffe/caffe.mexa64] Error 255
I am using Ubuntu 14.04 and Matlab R2015b. My LD_LIBRARY_PATH includes path to cuda.
export LD_LIBRARY_PATH=/usr/local:/usr/local/cuda-7.5/lib64:/usr/lib/nvidia-352:
Is this issue because of my Matlab version and Caffe-0.999? I was able to successfully compile the latest version of Caffe (obtained from https://github.com/BVLC/caffe)
Any help would be appreciated. Thanks!

Hi deeptigp,
I had the same problem with Matlab R2014a, our versions seem to updated for caffe 0.999. I fixed the problem by changing the Makefile like this:
change the line $(MATLAB_DIR)/bin/mex $(MAT$(PROJECT)_SRC) $(STATIC_NAME)
with	$(MATLAB_DIR)/bin/mex $(MAT$(PROJECT)_SRC)
this remains the same: CXXFLAGS="$$CXXFLAGS $(MATLAB_CXXFLAGS)"
change			CXXLIBS="$$CXXLIBS $(LDFLAGS)" -o $@
with			CXXLIBS="$$CXXLIBS $(STATIC_NAME) $(LDFLAGS)" -output $@
see also https://github.com/rbgirshick/caffe/commit/c28823f7829b91c5dd56060579dda2cecbc22d7b

Hi, @deeptigp
I had the same problem with Matlab R2014b, Linux 14.04, caffe v0.999. I fixed the problem by changing the Makefile like this,
change CXXLIBS="$$CXXLIBS $(LDFLAGS)" -o $@
to CXXLIBS="$$CXXLIBS $(STATIC_NAME) $(LDFLAGS)" -output $@
which meaning add $(STATIC_NAME) before $(LDFLAGS) and change -o to -output
It works for me!

why i can't start matlab being in R-CNN directory , in installation guide it is mentioned that we have to start matlab being in R-CNN directory but i m unable to locate matlab in r-cnn, any help will be appreciated. thanks


texlive.bin.core-big: fix luajittex on aarch64-linux

Description of changes
This should fix the luajittex crashes on aarch64-linux unveiled by @xworld21's systematic testing.
At least it works on my machine!
@ofborg build tests.texlive

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg build tests.texlive


[Regression] Talk to recent models are not updated if to use model from recent list

EPAM AI DIAL chat version
0.6.11
What happened?
Probably the bug appears because of #873 fix!

Create 5 chats based on different models/applications, so they all appear in recent Talk to models
For example,

Create a conversation and use the 5th model from the recent Talk to list, e.g. Cohere Command Text V14
Create a conversation and check the list in Talk to -> Cohere Command Text V14 is the 5th model -> bug

Expected: Cohere Command Text V14 is the 1st model
Note: the list is updated ok if to select a model from See full list...
https://github.com/epam/ai-dial-chat/assets/111180458/df4d0b44-025b-46a2-bd58-1fa795851b8c
Confidential information

[X] I confirm that do not share any confidential information

Verified on stg, no issues


Compiling OpenCV with Windows Intel OpenCL

Good day,
I am having issues compiling OpenCV 3.1.0 with the Intel Windows OpenCL SDK 2.0. I have set my CMAKE Options as such:
WITH_OPENCL=OFF
HAVE_OPENCL=ON
HAVE_OPENCL_STATIC=ON
OPENCL_LIBRARIES=C:/Program Files (x86)/Intel/OpenCL SDK/5.3/lib/x64/OpenCL.lib
OPENCL_INCLUDE_DIRS=C:/Program Files (x86)/Intel/OpenCL SDK/5.3/include/
It seems to give warnings saying that clCreateCommandQueue and clEnqueueTask was declared deprecated. Is this an issue of 1.2 vs 2.0?

Looks like some things were changed in Windows SDK or MSVS compiler.
Just disable Direct3D support via CMake parameter: -DWITH_DIRECTX=OFF.
OpenCL 2.0 deprecates some calls of OpenCL 1.1/1.2. You can ignore these messages.
Also OpenCL 2.0 support is not well tested. So you should build and run OpenCV tests to pass some basic checks.

I now have issues with Eigen and OpenCL and OpenMP, getting this error where I would get an Internal Compiler Error:
http://eigen.tuxfamily.org/bz/show_bug.cgi?id=1096


Jesse: Watch digitaltechisMe(?#twitter)

Jesse requests the watch of the watch_keyword digitaltechisMe(?#twitter). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

digitaltechisMe(?#twitter) has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Registry: Impossible to edit long keys

When a registry key is so long that it overflows the page, there is no way to get to the button to save the key. There should probably be a scroll bar in the edit window in such a case instead of just overflowing the edit window.
As an example, see the key couplingPts at registry/Jim/AliceBobRes/Cooldown151230/160222/Bc/:

Well this wasn't a use case I'd considered. I think it should be an easy fix though...

Jim, could you do me a favor and past the contents of this registry key into here?  It should look like this, correct?

If so, it's just a matter of updating the polymer components and recompiling.
@maffoo can you just run  $ bower update paper-input and recompile the server? I think this should fix it.

@joshmutus, how you display the key is indeed what I'm asking for.

I've updated the server, but it still doesn't work for me. @jwenner, can you give it another try yourself?

@joshmutus, if we do need newer versions of the paper components, we should update bower.json. Right now the versions there are all just ^1.0.0.

@joshmutus and @maffoo, the version on matrix-reloaded still doesn't work for me either.

Right now matrix-reloaded has 1.1.7:
$ bower list | grep paper-input
│ ├─┬ paper-input#1.1.7
│ │ ├─┬ paper-input#1.1.7

@joshmutus, what version are you using locally?

Just a sec, bower list is failing on me for some reason
 Failed to execute "git ls-remote --tags --heads git://github.com/nathanfaucett/require.js.git", exit code of #128 fatal: remote error:    Repository not found.

I can check the bower.json manually in the paper-input folder and I see I'm on 1.1.7

$ bower list | grep paper-input
│ ├─┬ paper-input#1.1.7
│ │ ├─┬ paper-input#1.1.7

$ bower list | grep polymer
│ │ └─┬ polymer#1.2.3 (1.3.0 available)
│ │ │ └── polymer#1.2.3

I double checked and I'm definitely on master. I've done a gulp clean to make sure my server's not in a strange state. I hope this isn't another mac vs. pc vs. linux rendering bug.

@maffoo to get bower.json updated to my current version numbers should I just run bower init ? There must a better way than manually inserting these version numbers.

@joshmutus I think it's something like bower install --save

Ugh, looks like this has to be done for every component. I'll make an issue of this and do it later.

@maffoo have you tried rendering the registry key locally off of master? matrix-reloaded isn't on master, correct?

I think that screen resolution has something to do with this. When I opened the page on linux through my workstation the scrollbar didn't show up.
It seems like these dialogs are a big pain in general and we should think about going to popup windows or window.prompt(). I think @maffoo suggested this originally...

As I mentioned in #97, I'm still encountering this issue in the version on matrix-reloaded. In fact, when I try to select part of the key in the edit screen to go up or down, what scrolls is the device page (listing all keys) instead of the edit screen.

With the version that is now on matrix-reloaded as of this evening, this is now fixed. @maffoo, which version is on matrix-reloaded? If it is master, then we can go ahead and close this.

Thanks for checking again, @jwenner. Yes, the version currently running on matrix is master, so I'll close this.


Don't allow editing entities with datastore viewer

The editing feature needs additional testing to ensure certain inputs do not have unintended side effects.

https://ocd.appscale.com:8080/job/Daily Build/4772/


Detect the duplicated schema issue in the spec PR pipeline

Control plane SDK is generated based on a tag defined in the readme. It might include multiple swaggers within a tag. If the combination of all these referenced swaggers within a tag contain same schema name but different values, then this issue would block the SDK generation.
Is lintDiff tool still an extension of autorest or it gets rid of autorest dependency? If using autorest, it should be able to identify this schema duplication issue. Otherwise, can we add a check to run autorest modeler 4 in the spec PR pipeline?

@raych1 can you share a URL of a PR that has this problem and the breaking change tool is not detecting it?

@konrad-jamrozik , duplicated schema issue should be caught by lintDiff instead of breaking change detection tool. This is an example PR with the issue.

@raych1 looks to me this is a candidate for a new LintDiff rule to be owned by SDK automation team. I'll assign this to you - feel free to reassign to appropriate person on your team. Maybe there is some existing rule that is not working as it should.


How much memory to allocate ?

Hi,
Total zipkin newbie here. The docker image is used in kubernetes manifests of the knative project:
https://github.com/knative/serving/blob/master/config/monitoring/tracing/zipkin/100-zipkin.yaml
We are trying to set a memory request so that the pod can get scheduled if we have lower than 1 Gb available.
What would be a sane amount of memory to allocate to this container ?
thank you,

#79 old but related so tagging. no need to look at it.
sorry I don't know for sure, but we have done some work on performance and overhead with elasticsearch recently. I recall doing a load test of 128 clients with 4 threads against the stock server and it is something in the couple hundreds of megs.
If you want to tune down the amount of heap, what I'd recommend is running prometheus and the grafana dashboard we have. Maybe start with 300MiB or so on the zipkin server. Then in your environment, you can look at or alert on heap related things as they are tracked with real usage.
In order buckle down is easier than unbuckling.
if you want to chat more, please join gitter https://gitter.im/openzipkin/zipkin

thank you


Fishing rod pulling.

Remove fishing rod pulling and just leave fishing rod knock-back, fishing rod pulling is a new feature in 1.9+ version which a lot of people don't like, and there's currently no plugin on spigot which entirely disables it, so I think that would be pretty cool to see in this plugin features.
By fishing rod pulling I mean a "thing" added in 1.9+ Minecraft versions where when you use fishing rod on a player you pull him closer to yourself. This was not a thing back in Minecraft 1.7/1.8. In Minecraft 1.7/1.8 when you used fishing rod on a player you would pull him further away from you not closer.
Here is a video example: https://www.youtube.com/watch?v=s7t46VEACIY

I have added this in OCM version 1.6.2


Disassembling Furby Connect

You don't need to disassemble your expensive Furby Connect to see inside it - the FCC has done the hard work for you:
https://fccid.io/document.php?id=3031176
You can see the Nordic chip (n51822) on a breakout board. Unfortunately the General Plus chip seems to be hidden under a big black blob.
Here are all the filings: https://fccid.io/RS4-B6083A

The blob: Epoxy. Ridiculous, abhorrent stuff.
I know a select few people have been able to rip it off. http://natashenka.ca/ got somebody to take off a Tamagotchi's epoxy blob when they wanted to hack it. Anyone willing to try it with their Furby?


Error: Page crashed!

When all dependencies are installed, use npm run start the following error pops up at startup
The error report is as follows
错误 Error: Page crashed!
at PuppetWeChat. (/mnt/g/wechat-assistant-pro/node_modules/wechaty/dist/src/wechaty.js:271:44)
at PuppetWeChat.emit (events.js:315:20)
at PuppetWeChat.EventEmitter.emit (domain.js:467:12)
at Bridge. (/mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/wechaty-puppet-wechat/dist/src/puppet-wechat.js:202:45)
at Bridge.emit (events.js:315:20)
at Bridge.EventEmitter.emit (domain.js:467:12)
at /mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/wechaty-puppet-wechat/dist/src/bridge.js:169:36
at /mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/puppeteer/lib/cjs/vendor/mitt/src/index.js:51:62
at Array.map ()
at Object.emit (/mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/puppeteer/lib/cjs/vendor/mitt/src/index.js:51:43)
at Page.emit (/mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/puppeteer/lib/cjs/puppeteer/common/EventEmitter.js:72:22)
at Page._onTargetCrashed (/mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/puppeteer/lib/cjs/puppeteer/common/Page.js:268:14)
at /mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/puppeteer/lib/cjs/puppeteer/common/Page.js:154:57
at /mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/puppeteer/lib/cjs/vendor/mitt/src/index.js:51:62
at Array.map ()
at Object.emit (/mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/puppeteer/lib/cjs/vendor/mitt/src/index.js:51:43)
at CDPSession.emit (/mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/puppeteer/lib/cjs/puppeteer/common/EventEmitter.js:72:22)
at CDPSession._onMessage (/mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/puppeteer/lib/cjs/puppeteer/common/Connection.js:225:18)
at Connection._onMessage (/mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/puppeteer/lib/cjs/puppeteer/common/Connection.js:101:25)
at WebSocket. (/mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/puppeteer/lib/cjs/puppeteer/node/NodeWebSocketTransport.js:13:32)
at WebSocket.onMessage (/mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/ws/lib/event-target.js:132:16)
at WebSocket.emit (events.js:315:20)
at WebSocket.EventEmitter.emit (domain.js:467:12)
at Receiver.receiverOnMessage (/mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/ws/lib/websocket.js:835:20)
at Receiver.emit (events.js:315:20)
at Receiver.EventEmitter.emit (domain.js:467:12)
at Receiver.dataMessage (/mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/ws/lib/receiver.js:437:14)
at Receiver.getData (/mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/ws/lib/receiver.js:367:17)
at Receiver.startLoop (/mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/ws/lib/receiver.js:143:22)
at Receiver._write (/mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/ws/lib/receiver.js:78:10)
at writeOrBuffer (internal/streams/writable.js:358:12)
at Receiver.Writable.write (internal/streams/writable.js:303:10)
at Socket.socketOnData (/mnt/g/wechat-assistant-pro/node_modules/wechaty/node_modules/ws/lib/websocket.js:910:35)
at Socket.emit (events.js:315:20)
at Socket.EventEmitter.emit (domain.js:467:12)
at addChunk (internal/streams/readable.js:309:12)
at readableAddChunk (internal/streams/readable.js:284:9)
at Socket.Readable.push (internal/streams/readable.js:223:10)
at TCP.onStreamRead (internal/stream_base_commons.js:188:23)
推送错误成功 Pagecrashed!

Thanks for the reporting.
It will be better if you can past the log messages as Text format instead of a Image because it would be more easy to be read and searched.

Thanks for the reporting.
It will be better if you can paste the log messages in a text format instead of an image because it would be easier to be read and searched.
I'd like to suggest that you can follow https://wechaty.js.org/docs/getting-started/quick-start to get started on the cloud, then try to run in your local environment again.

os envoriment
root@Jason-Office:/mnt/g/wechat-assistant-pro# node --version
v14.16.1
root@Jason-Office:/mnt/g/wechat-assistant-pro# npm -v
7.11.1
root@Jason-Office:/mnt/g/wechat-assistant-pro# uname -a
Linux Jason-Office 4.4.0-18362-Microsoft #836-Microsoft Mon May 05 16:04:00 PST 2020 x86_64 x86_64 x86_64 GNU/Linux
root@Jason-Office:/mnt/g/wechat-assistant-pro# cat /etc/
Display all 187 possibilities? (y or n)
root@Jason-Office:/mnt/g/wechat-assistant-pro# cat /etc/sys
sysctl.conf  sysctl.d/    systemd/
root@Jason-Office:/mnt/g/wechat-assistant-pro# cat /etc/os-release
NAME="Ubuntu"
VERSION="20.04.1 LTS (Focal Fossa)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 20.04.1 LTS"
VERSION_ID="20.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=focal
UBUNTU_CODENAME=focal


Any suggestion about model ensemble for faster rcnn?

I have trained several faster rcnn models with different backbones, then I want to ensemble these models for a better performance, but I am totally confused about how to do this, can you give me some suggestion or reference for this? Thank you anyway.

In detection_test.py, we aggregate all output_record with the same im_id in all_output, as shown in here. If you want to do ensemble testing, you should save the all_output as pickle for all models you tested. And concate them as a single all_output for downstream evaluation.


Add test to prevent regression of being able to run tests while the server is running

Originally posted by @DiamondJoseph in https://github.com/DiamondLightSource/blueapi/pull/241#pullrequestreview-1443720087

isn't this solved by separating unit tests from functional / feature  / system / integration tests?

@DiamondJoseph is this still an issue?


Add error checking, path and user handling.

Modulus Pull Request
Description

added Python dist path resolution.
handle a case when the script runs under non-root user.
error checking.

Checklist

[x] I am familiar with the Contributing Guidelines.
[ ] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[ ] The CHANGELOG.md is up to date with these changes.
[ ] An issue is linked to this pull request.

Dependencies

Not applicable anymore.


Tekken Crashes When Trying to use Record Moves/Inputs

After using the tekken overlay's repete action every time i try use record move inside tekken the game crashes consistanly. the only fix is to not use overlay at all. i have tried openeing overlay before the game starts, after the game starts, complete redownload of tekken overlay.

Try newer release: https://github.com/TekkenOverlay/TekkenOverlay/releases/latest

Hello. I tried the latest version and the side wall crash doesn't seem to happen which is good.
Going back into the main menu from Practice and online waiting lobby does not result in a crash, also good.
But when starting the overlay mod first the game will crash when going back to waiting room/lobby.
Terminal:
Opening process... Successful.
Injecting DLL... Successful.
Lost process handle!
Searching for process ID... NOT FOUND!
However this issue does not happen if I start the game first and then the overlay program.

I'm going to assume that the update fixed the problem.


Add object command in array commands_object before execute.

🎉 Feature Request
Summary
Hello everyone! I'm not exactly sure if this can be done
        if ($update_type === 'message') {
            $message = $this->update->getMessage();
            $type    = $message->getType();

            // Let's check if the message object has the type field we're looking for...
            $command_tmp = $type === 'command' ? $message->getCommand() : $this->getCommandFromType($type);
            // ...and if a fitting command class is available.
            $command_obj = $command_tmp ? $this->getCommandObject($command_tmp) : null;

            // Empty usage string denotes a non-executable command.
            // @see https://github.com/php-telegram-bot/core/issues/772#issuecomment-388616072
            if (
                ($command_obj === null && $type === 'command')
                || ($command_obj !== null && $command_obj->getUsage() !== '')
            ) {
                $command = $command_tmp;
            }
        } elseif ($update_type !== null) {
            $command = $this->getCommandFromType($update_type);
        }

        //Make sure we don't try to process update that was already processed
        $last_id = DB::selectTelegramUpdate(1, $this->update->getUpdateId());
        if ($last_id && count($last_id) === 1) {
            TelegramLog::debug('Duplicate update received, processing aborted!');
            return Request::emptyResponse();
        }

        DB::insertRequest($this->update);

        return $this->executeCommand($command);

When executed this line
$command_obj = $command_tmp ? $this->getCommandObject($command_tmp) : null;
We can create instance of  command.
And maybe we can add to the end in functions something this
    DB::insertRequest($this->update);

    if(isset($command_obj) && $command_obj instanceof Command) {
        $this->commands_objects[$command] = $command_obj;
    }

    return $this->executeCommand($command);

Because next command $this->executeCommand($command);  sometimes again create this instance.

I'm not sure what you're trying to achieve. Can you explain what problems you have and what you want to achieve?

Closing this due to lack of response


Deployer

Adds a deployer component, which uses the bgentry/que-go to schedule deployments using different strategies (currently one-by-one or all-at-once).
Resolves #602, resolves #603, resolves #63.

@flynn/devs This is now integrated with the controller and ready for another review pass

Please move 3a094cc422b66901db004fcac92dfba1fafa328b and eae30dd28b65f268fd383f671ad767ec4d8dcae6 into a separate PR.

Comments addressed.

@lmars @titanous Rollback test is added, I think this should be enough to merge this PR, and then follow up with more patches.

Okay, will review in a bit.

LGTM

Please pull in the latest que-go and pgx before merging.


neomutt: 20170128 -> 20170306

Motivation for this change
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Looks rather mergeable to me :)
On 11 March 2017 16:29:22 CET, Mention Bot<EMAIL_ADDRESS>wrote:

@magnetophon, thanks for your PR! By analyzing the history of the files
in this pull request, we identified @fpletz, @hiberno and @Mic92 to be
potential reviewers.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/NixOS/nixpkgs/pull/23753#issuecomment-285875018

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Thanks!


Add a py.typed file to add typing support

This is needed for typing support in projects using this library.
Example https://github.com/kolonialno/sdxp/blob/main/src/sdxp/py.typed

Fixed here: https://github.com/kolonialno/troncos/pull/10


kotlin: 1.9.10 -> 1.9.20

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/JetBrains/kotlin/releases.
meta.description for kotlin is: General purpose programming language
meta.homepage for kotlin is: https://kotlinlang.org/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.9.20 in filename of file in /nix/store/d46m3jw5j0015g7j63aky037bfzqqz4k-kotlin-1.9.20

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
kotlin

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/d46m3jw5j0015g7j63aky037bfzqqz4k-kotlin-1.9.20 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A kotlin https://github.com/r-ryantm/nixpkgs/archive/2c83eef4b9197f3ef4e959cef90bdc86952e9870.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/2c83eef4b9197f3ef4e959cef90bdc86952e9870#kotlin

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/d46m3jw5j0015g7j63aky037bfzqqz4k-kotlin-1.9.20
ls -la /nix/store/d46m3jw5j0015g7j63aky037bfzqqz4k-kotlin-1.9.20/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

kotlin

Maintainer pings
cc @SubhrajyotiSen for testing.

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

1 package built:

kotlin


Add WireGuard PrivateKey/PresharedKey rule

This is a rule I regularly use to look for WireGuard .conf files (specifically private and preshared WG keys). Maybe it makes sense to add it to the defaults. I generated the 2 keys in the example myself just for this purpose.

Thank you @gemesa! This looks really good. This will allow Nosey Parker to detect more secrets with precision.
Would you please split this new WireGuard PrivateKey/PresharedKey rule into two new rules? They are conceptually different things, even if they have the same format. WireGuard Private Key and WireGuard Preshared Key are good names for them.

I split them and fixed the review findings, please check it out.

Thank you for your contribution, @gemesa!


Netty client will report an error when connecting to the server through a domain name

Netty version
4.1.79.Final
JVM version (e.g. java -version)
jdk8

This is expected... strip the http:// from the hostname


[Event Request] codeunit 2580 "Dimension Correction Mgt" - OnAfterVerifyIfDimensionCanBeChanged

Hi team, please add OnAfterVerifyIfDimensionCanBeChanged to the procedure VerifyIfDimensionCanBeChanged

This will be used to add more checks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Guidance for links to empty relationships

The spec currently gives guidance on how to structure relationships when the exist, but gives no guidance for when they do not.
It is obvious (but not stated in the spec) that when the value of a relationship is a related resource URL, it should be null when missing:
Present
// ...
{
  "type": "articles",
  "id": "1",
  "title": "Rails is Omakase",
  "links": {
    "comments": {
      "resource": "http://example.com/articles/1/comments"
    }
  }
}
// ...

Empty
// ...
{
  "type": "articles",
  "id": "1",
  "title": "Rails is Omakase",
  "links": {
    "comments": {
      "resource": null
    }
  }
}
// ...

Things are less clear when a relationship is provided as a link object because of the self link. Even when the related resource is empty, the self link is a valid link for modifying the relationship.
Present
// ...
{
  "type": "articles",
  "id": "1",
  "title": "Rails is Omakase",
  "links": {
    "self": "http://example.com/articles/1",
    "author": {
      "self": "http://example.com/articles/1/links/author",
      "resource": "http://example.com/articles/1/author",
      "type": "people",
      "id": "9"
    }
  }
}
// ...

What does this look like if the author relationship is empty? There are two strategies: omitting keys, or null values.
Missing, omitting keys
// ...
{
  "type": "articles",
  "id": "1",
  "title": "Rails is Omakase",
  "links": {
    "self": "http://example.com/articles/1",
    "author": {
      "self": "http://example.com/articles/1/links/author"
    }
  }
}
// ...

Missing, null values
// ...
{
  "type": "articles",
  "id": "1",
  "title": "Rails is Omakase",
  "links": {
    "self": "http://example.com/articles/1",
    "author": {
      "self": "http://example.com/articles/1/links/author",
      "resource": "http://example.com/articles/1/author",
      "type": null,
      "id": null
    }
  }
}
// ...

I propose that the null value approach be taken by the spec. @dgeb work on PR https://github.com/json-api/json-api/pull/357 might help to inform where we go on this?

To signify an empty to-one relationship, you're supposed to set id: null. If it's a to-many relationship, you say ids: []. If you normally use data, I guess you could have an empty array in data, too. If you only use URLs, then you'd probably set resource: null. A URL doesn't signal cardinality anyway, so this is fine.
@ethanresnick reinitiated the discussion on unifying id, ids, and data in #312, which, if realized, would simplify this system.

@bintoro good catch on the related resource URLs. I've updated the example.

This is a duplicate of https://github.com/json-api/json-api/issues/348, I think


numfmt: prohibit --header=0 to align with GNU

Adjust header option handling to prohibit passing a value of 0 to align
with GNU numfmt. Also report header option parse errors as GNU does.
closes #1708

great, thanks!


Make message sender box textarea instead of input

I would like to add multi-line text input for the chat widget's sender box. Is there any easy way to do this? If not it seems like a pretty straight forward feature to add (add a multiLine prop that controls whether the underlying HTML element is a textarea or an input).

Hi Cris!  I hope you are all right. Did you find the solution for this issue or some other way? Thanks.

Hi Cris!  I hope you are all right. Did you find the solution for this issue or some other way? Thanks.

Hi @mromauricio
I did not. I actually ended up just pulling the html and css out of this package and writing my own logic, the built in options were way to restrictive. Also, the way to add text messages made no sense, React is declarative and this sorta goes against that paradigm by using the addMessage methods or whatever they were called.

Hi @mromauricio
I did not. I actually ended up just pulling the html and css out of this package and writing my own logic, the built in options were way to restrictive. Also, the way to add text messages made no sense, React is declarative and this sorta goes against that paradigm by using the addMessage methods or whatever they were called.

Thank you very much for the quick reply.

Thank you very much for the quick reply.


v0.20.0

v0.20.0 October 31, 2022

Enhancements

Replace use of deprecated append method for dataframes and series with concat method (#1533)

Fixes

Fixed bug relating to dependence calculations to ensure columns exist in dataframe (#1534)
Small typo fix in select docstring (#1544)
Fix TypeValidationError message (#1557)
Set dask version below 2022.10.1 (#1558)

@Cmancuso they just pushed this hotfix out. I think we can wait until the next release before unpinning. Filed this issue to track.


[Jenkinsfile] Change tests run command

currently tests starting with command - ./run_tests_qa.sh -r
need to change it to call maven directly,
it will help us to catch tests fails.

done


CFE constant initializers sometimes have duplicate optional argument keys.

For the following snippet:
const a = A();

class A extends Z<int> {
  const A() : super();

  @override
  final int defaultValue = 100;
}

class Z<T> {
  final T defaultValue;
  const Z({this.defaultValue});
}

main() {
  print(a.defaultValue);
}

the CFE generates:
main = sta::main;
library from "org-dartlang-app:/static1.dart" as sta {

  class A extends sta::Z<core::int*> {
    @#C1
    final field core::int* defaultValue = 100;
    const constructor •() → sta::A*
      : super sta::Z::•()
      ;
  }
  class Z<T extends core::Object* = dynamic> extends core::Object {
    final field sta::Z::T* defaultValue;
    const constructor •({sta::Z::T* defaultValue = #C2}) → sta::Z<sta::Z::T*>*
      : sta::Z::defaultValue = defaultValue, super core::Object::•()
      ;
  }
  static const field sta::A* a = #C4;
  static method main() → dynamic {
    core::print(sta::a.{sta::A::defaultValue});
  }
}
constants  {
  #C1 = core::_Override {}
  #C2 = null
  #C3 = 100
  #C4 = sta::A {defaultValue:#C3, defaultValue:#C2}
}

The VM and DDC print 100, but DDK prints null because it emits both defaultValue arguments from left-to-right without any de-duping logic.
Are default parameters supposed to be de-duped in the CFE, or is there some sort of prescribed evaluation order (right-to-left here)? This pattern is used a bit in google3.

The CFE outputs references to the actual fields, which are distinct. The textual Kernel output contains just the names of the fields, which is confusing, but it is not related to the problem we see in DDK.
When a field is overridden, DDK outputs an object with duplicate keys:
dart.defineLazy(CT, {
  get C0() {
    return C0 = dart.const({
      __proto__: $37839.A.prototype,
      defaultValue: 100,
      defaultValue: null
    });
  }
});

In this case, where the field is virtual (as defined by ClassPropertyModel.virtualFields), it should output the actual keys of the underlying fields, i.e.
dart.defineLazy(CT, {
  get C0() {
    return C0 = dart.const({
      __proto__: $37839.A.prototype,
      [Symbol("A.defaultValue")]: 100,
      [Symbol("Z.defaultValue")]: null
    });
  }
});

I couldn't figure out how to get at these names, in particular since the ClassPropertyModel seems to only be available while compiling the class.
But there is a workaround that we can implement quickly, which will work as long as the overriding class does not bypass the field override explicitly by using super. We can simply output the fields in the opposite order, which will output subclasses after superclasses. This is not a specified and enforced order in the Kernel output, but it is what the CFE constant evaluator does, so it can work until we establish a more complete solution. I uploaded a CL for the workaround here: https://dart-review.googlesource.com/c/sdk/+/113060

An example of the code the doesn't work with the workaround is:
class A {
  final field;
  const A(this.field);
}
class B extends A {
  final field;
  const B(this.field, fieldA) : super(fieldA);
  get fieldA => super.field;
}
main() {
  print(const B(1,2).fieldA); // prints 'null' instead of '2' with the workaround.
}

We filed bug a bit after this one that might actually be the same issue manifested differently. The DDK property model seems to be missing something: https://github.com/dart-lang/sdk/issues/37855

@Markzipan - with the workaround landed, is this still an internal blocker for DDK?

Unless someone's doing what Johnni pointed out internally, technically no. But I won't know that until tonight's presubmit.

Thanks @askeksa-google !  We'll need to fix in DDK side - and add tracking to get to the underlying field.
@Markzipan - can you update priority after tonight's presubmit?

Potential fix:
https://dart-review.googlesource.com/c/sdk/+/113136

@vsmenon That breaks if for some reason the superclass has a variable with the same scheme.
E.g.:
const a = A();

class A extends Z<int> {
  const A() : super();

  @override
  final int defaultValue = 100;
}

class Z<T> {
  final T defaultValue;
  final _A_defaultValue = 300;
  int get value => _A_defaultValue;
  const Z({this.defaultValue});
}

main() {
  print(a.defaultValue);
  print(a.value);
}

@Markzipan - any updated data from a presubmit?

I haven't been able to look through all the failures, but it seems the old ones have been resolved. I'm also not sure yet if that superclass pattern's being used.


Argsort does not support axis

I have managed to minimise the code that triggers #2013 in isolation. It is due to argsort being passed the axis argument.
import numpy as np

# pythran export c3(float64[:])
def c3(data: np.ndarray):
    all_indexes = data.argsort(axis=0)
    return all_indexes

The issue also happens when data is 2D, for all valid values of the axis argument, and regardless of wether I use the array method or the numpy function.

Can you confirm 1a517ac2048eb85c3ca6e4cea4927066f0a7734b fixed that issue?

It does, thanks.


Text in a Column ignores its textAlign attribute

I'm creating a column with text like so:
                        new Column(
                            mainAxisAlignment: MainAxisAlignment.center,
                            children: [
                              new Text('foo',
                                  style: theme.primaryTextTheme.body1,
                                  textAlign: TextAlign.left),
                              new Text('barbazbop',
                                  style: theme.primaryTextTheme.body1,
                                  textAlign: TextAlign.left),
                              new Text('fizzbuzz',
                                  style: theme.primaryTextTheme.caption,
                                  textAlign: TextAlign.left)
                            ])

however, the strings all line up differently.
To get them to line up on the left side of the column, I had to set crossAxisAlignment on the column.
Is it possible to set the default crossAxisAlignment on Column such that the textAlign property of child textViews can be respected?

It's easier to understand what's going on if you put the text in a container with a background.
The simplest solution is to use a Block instead of a Column.

@Hixie would the "Debug drawing" checkbox in Atom also have shown the issue?

Yes, but only if you know what you're looking for. The Container demonstrates it in a better way because it becomes more obvious that the text alignment is aligning the text inside a box that's as wide as the text.

Not sure what we can do to make this clearer, so I'm going to close this for now. If you have any concrete ideas for how to make this more intuitive, please don't hesitate to file a new bug.

I just wrapped the Text in to a Row with a Spacer()
       Row(
              children: <Widget>[
                Text("Highlights",
                    textAlign: TextAlign.left,
                    style: TextStyle(fontSize: 20)),
                Spacer()
              ],
            )

This was posted in 2016 but still doesn't seem to work properly.  Clearly the text should be aligned in accordance with the width of the outer Container:
Container(
  width: 200,
  child: Column(
    children: <Widget>[
      Text('One', textAlign: TextAlign.left),
      Text('Two', textAlign: TextAlign.right),
    ],
  ),
),

Putting a Container with a width setting around each individual Text widget seems to make much less sense since the width is already defined by the outer Container.

@RedBrogdon I wonder if flutter pad would be useful for easily replyign to these sorts of bugs with an example? :)

What exactly is Flutter pad?

We've experimented with adding Flutter support to DartPad now that we have some basic Flutter for web support.  Informally we joke about such as being a "flutter pad".

https://dartpad.dev/embed-flutter.html?id=34c7fa6377fe6ecc28bc3d479e3e2a56
Did someone say "Flutter in DartPad?"
@hasen6, I know that what you're seeing might seem counterintuitive at first (especially if you're used to a UI system that doesn't work the same way), but it makes sense given the way Flutter's columns and rows lay things out.
We've got a whole codelab that uses the new features in DartPad to show how the properties on Rows and Columns work, if you're interested:
https://flutter.dev/docs/codelabs/layout-basics
There's a survey at the bottom, if you have any additional feedback!

@eseidelGoogle Ok I was gonna say. That would be great though, since languages like Javascript have that that are used in many frameworks. All it would need would be to cope with basic layouts, no need for animations or custom painting or whatever.
It would be especially useful for Flutter since as you RedBrogdon says the ui is counterintuitive. So many questions on stackoverflow with things like how to remove all the built in padding around every widget, when is TextAlign not ignored, or how to get everything lined up as you want it etc.
@RedBrogdon I'll check out the codelab on layout anyway, I didn't know about that. Although it doesn't seem to mention the word textalign anywhere so not sure if it clears up the issue in this thread.

@RedBrogdon Ok no problem.

Hi guys i think the best bet in this situation would be using this approach.
....................................................................................
    child: Container(
      width: MediaQuery.of(context).size.width,
      child: Column(

        children: [
          Text("Lets Get Started?",textAlign: TextAlign.center,),
          Text("Lets Get Started?",style: Theme.of(context).textTheme.headline1,textAlign: TextAlign.center,),
        ],
      ),
    ),

...........................................................................................

width: MediaQuery.of(context).size.width,
child: Column(

I think you'll find that if you try to align to start or remove the textAlign value that nothing changes.

Hi guys i think the best bet in this situation would be using this approach.
....................................................................................
    child: Container(
      width: MediaQuery.of(context).size.width,
      child: Column(

        children: [
          Text("Lets Get Started?",textAlign: TextAlign.center,),
          Text("Lets Get Started?",style: Theme.of(context).textTheme.headline1,textAlign: TextAlign.center,),
        ],
      ),
    ),

...........................................................................................

I think you'll find that if you try to align to start or remove the textAlign value that nothing changes.

All Possible working solution added [Solved] textAlign not working flutter


nohup: improve test coverage

as requested in #1857
I was able to simulate a terminal such that we can test the code in function "replace_fds"

CI/CD reported +15.46% testcoverage for nohup.rs :-)

I don't know whether you have seen that the test fails on FreeBSD/macOS?

I don't know whether you have seen that the test fails on FreeBSD/macOS?

thanks for pointing out. there are so many unrelated error currently that it gets difficult to see the real issues

closed in favor of https://github.com/uutils/coreutils/pull/5869


UI improvements

Problem description
As commit e60d3d00f6c18352c51fc3943565155a27eef843, the user interface is not really clear and functional:

"Number of iterations" (reachability) and "Max parameter splits" (synthesis) options are placed in the same block;
execution (for both reachability and synthesis), stop (which is unnecessary when sapo is not computing), temporal logics (only for synthesis), and plots are all in the position and take useful vertical space;
"Variable declarations" block wastes a lot of space;
dynamics are not located near their variables. Thus, the user must move from "variable declarations" to "system of equations" for each new variable;
parameter names can be deduced from dynamics. Thus, the "Parameter declaration" block wastes space;
The "Modify Approximation Method Matrix" block may appear to be a little bit mysterious to users;
The "L-matrix" modal is not usable when the model involves a large number of variables.

Proposal
My suggestion is to wait for issue dreossi/sapo#6 to be closed and re-write the user interface as depicted in the following image

Moreover, I recommend using the material design components provided by mui.

Commit e3b6b17f3683d0cf015c93e9fbad971e34e08884 goes in the right direction, but some changes are still needed in the Parameter block


Tapping on map markers on mobile doesn't open the popup

For some reason, trying this when emulating touches in chrome works fine, so this needs to be tested on a device.

From #28:
Steps to reproduce:

Load the site on a mobile device
Search for a county, and view the county map
Tapping on a placemark does not show any pop-up

Also, for testing on a device using localhost (pretty sure I didn't know how to do this a year ago when I made this bug report):

Run grunt serve  on the app on your computer like normal
Make sure your computer and your device are on the same network
Find your computer's local IP address using ifconfig or another method
Change line 77 of angular/Gruntfile.js to `hostname: '<IP_ADDRESS>' or your local IP address
Go to [your-ip-address]:9000 in your mobile browser


weapons no spawn randomly in loot chests

I used 1.6.0 but found weapons no spawn randomly in loot chests

I used 1.6.0 but found weapons no spawn randomly in loot chests

I've completely redone the whole mod from the ground up and have tested that they do in fact spawn in loot chests. I've tested this by doing /setblock ~ ~2 ~ minecraft:chest 1 0 {LootTable:"chests/simple_dungeon"} You can see all the possible loot chests that it will spawn in [HERE] (https://github.com/AzureZhen/dotHackWeapons/tree/master/src/main/resources/assets/rcraft/loot_tables/inject)

Sorry, I understand it wrong, I understand loot chests in Mine and Slash.

Sorry, I understand it wrong, I understand loot chests in Mine and Slash.

They should be spawning in loot boxes of Mine and Slash as well, I just tested them.


Switches Not working

Students report that the switch input is not working:
https://github.com/orgs/neu-ece-4534-sp23/discussions/9
Can you please look into it?

I can't access that link.

Tried a couple of things in QEMU source, the event from the web client side reaches where it should, i.e. it tries to correctly write a MMR for the correct gpio chip, but that has no side effects. Reading that memory location immediately after shows no change.
I need to run QEMU under gdb for more information.

The problem was in QEMU source, not zpanel


Edit Community -> Community not found

Describe the bug
When editing a community, I receive the error notification "Community not found."
To Reproduce
Steps to reproduce the behavior:

Go to a community
Edit the community
Click Edit button after making changes
See error

Expected behavior
Expected edits to be saved and returned to community
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Windows
Browser: Edge

Additional context
Add any other context about the problem here.

Resolved


Minimum stock of parts on a (sub-)location

Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

Problem statement
I have the need to track the amount of parts on different locations. By now, we can set a minimum stock of a part globally. But what if I want to have some sublocation stocks to don't fall below their own minimum?
Suggested solution
The stock items could have a minimum quantity. Or the location itself could have minimum quantities for certain parts.
Describe alternatives you've considered
I can't see a way to achive this by now.
Examples of other systems
No response
Do you want to develop this?

[ ] I want to develop this.

Sounds interesting

@msch-alpgis the current implementation is very simple, compared to this approach. Can you suggest how the data structures to support this might work? My first impression is that this would be quite complex to implement (especially regarding edge cases).

@msch-alpgis the question is more about how to efficiently match this with nested locations for multiple items/nested boms. The queries have to scale as there might be a lot of items in one query. Could you provide a PR of your approach?

@msch-alpgis there are a lot of higher priority tasks for us right now. If you want to see this implemented quickly, you could consider sponsoring the feature. Otherwise, it will sit in our roadmap until a developer gives attention to it :)

Not stale

Not stale


Multi-dimensional Slice

Should it be possible to aggregate over arbitrary dimensions (for example 2 out of 4)? Following on, is cubeing functionality necessary?

@poojke is this closed by the Along/Over multiple dimensions?

Thanks @hgrano, yes this can be closed. Good find!

Closed by https://github.com/CommBank/grimlock/pull/88


Image stuck directly on vr screen

Its hard to explain, but I did a build out and now the image of the scene is stuck on the scren and moves with the head a still frame... I dunno what happened?

After further testing only seems to happen in Microsoft edge


Facebook Messenger looks awful and logs out constantly

Bug description:
<Facebook Messenger has no true dark mode (despite standalone Messenger Web having dark mode and the fake dark mode looks horrendous. Also, it logs me out EVERY. SINGLE. TIME.  that I close Ferdi.>
Steps to reproduce:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior

Screenshots

Environment:

Operating System:

Ferdi Version:

Server:

Debug information:

What is this ticket even. Completely false.

Messenger web does NOT have a dark mode. The messenger app has it, that's not the same.
It logs you out because you don't click "save your login" on the login screen. The checkbox is not visible in dark mode. Turn off dark mode before you log in and check it.


Fix organization case switching renames

#2163

Coverage decreased (-2.8%) to 66.876% when pulling 016b0ed172302ce2e4c1b49904994ab93dfeba63 on feature/fix_case_switch_org into 70f33dfe5a3a741cf511142dc07f11bcaecec541 on develop.


[glib] build failure in Alpine

Package: glib[core]:x64-linux -> 2.74.0
Host Environment

Host: x64-linux
Compiler: GNU 10.3.1
vcpkg-tool version: 2999-12-31-unknownhash
vcpkg-scripts version: c72cefb25 2022-10-26 (4 days ago)

To Reproduce
VCPKG_FORCE_SYSTEM_BINARIES=1 vcpkg install glib
Failure logs
-- Using cached glib-2.74.0.tar.xz.
-- Extracting source /home/default/code/vcpkg/downloads/glib-2.74.0.tar.xz
-- Applying patch use-libiconv-on-windows.patch
-- Applying patch libintl.patch
-- Using source at /home/default/code/vcpkg/buildtrees/glib/src/glib-2-22354247da.clean
-- Getting CMake variables for x64-linux-dbg
-- Getting CMake variables for x64-linux-rel
-- Configuring x64-linux-dbg
-- Configuring x64-linux-dbg done
-- Configuring x64-linux-rel
-- Configuring x64-linux-rel done
-- Package x64-linux-dbg
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:96 (message):
    Command failed: /home/default/code/vcpkg/downloads/tools/ninja/1.10.2-linux/ninja install -v
    Working Directory: /home/default/code/vcpkg/buildtrees/glib/x64-linux-dbg
    Error code: No such file or directory
    See logs for more information:

Call Stack (most recent call first):
  scripts/cmake/vcpkg_install_meson.cmake:33 (vcpkg_execute_required_process)
  ports/glib/portfile.cmake:47 (vcpkg_install_meson)
  scripts/ports.cmake:147 (include)

Additional context
Seems related with the ninja version downloaded by vcpkg:
DESKTOP-BABMP59:~/code/vcpkg$ /home/default/code/vcpkg/downloads/tools/ninja/1.10.2-linux/ninja --version
-ash: /home/default/code/vcpkg/downloads/tools/ninja/1.10.2-linux/ninja: not found
DESKTOP-BABMP59:~/code/vcpkg$ ldd /home/default/code/vcpkg/downloads/tools/ninja/1.10.2-linux/ninja
        /lib64/ld-linux-x86-64.so.2 (0x7f5209988000)
        libstdc++.so.6 => /usr/lib/libstdc++.so.6 (0x7f52097e7000)
        libm.so.6 => /lib64/ld-linux-x86-64.so.2 (0x7f5209988000)
        libgcc_s.so.1 => /usr/lib/libgcc_s.so.1 (0x7f52097cd000)
        libc.so.6 => /lib64/ld-linux-x86-64.so.2 (0x7f5209988000)

Indeed the problem is that the vcpkg ninja version is not statically compiled and uses libc so it doesn't work under alpine (uses musl). The solution would be to use statically compiled versions for cmake and ninja.

VCPKG_FORCE_SYSTEM_BINARIES=1 vcpkg install glib

You tell vcpkg that your system provides ninja but it doesn't.

I had Ninja but it wasn't the expected version to build. When that was detected vcpkg downloaded a built ninja version but that version was not statically compiled so it couldn't run. Having vcpkg using statically compiled version of cmake and ninja would fix this problem.

I'm gonna close this ticket but my recommendation would be for vcpkg to use a statically compiled version of cmake so it will work in every Linux distro


Enhance the appearance of suggestions

I have updated beta version to obsidian-various-complements-plugin-beta v5.1.0-beta1 🚀

Released at v5.1.0 🚀
https://github.com/tadashi-aikawa/obsidian-various-complements-plugin/releases/tag/5.1.0


Incorrect Detection of dGPU (Laptops)

While running the tool on laptop with dGPU, incorrectly will only find iGPU, even if Windows is set to run tool in context of dGPU.
This leads to the launch option message never appearing, so VRAM will be incorrectly set to 512mb

What gpu you run it with shouldn't play a role - the app checks every GPU with Vulkan support, or it should anyway. Can you provide me with values from the info button?

What gpu you run it with shouldn't play a role - the app checks every GPU with Vulkan support, or it should anyway. Can you provide me with values from the info button?
Could you also check if the same occurs on the Python version?

For python version, what do you need?

What gpu you run it with shouldn't play a role - the app checks every GPU with Vulkan support, or it should anyway. Can you provide me with values from the info button?
Could you also check if the same occurs on the Python version?

For python version, what do you need?

Try installing DXVK via the python version, send me the logs after you're done

Python version detects 8g of video memory, unlike WPF. Here is full output

Python version detects 8g of video memory, unlike WPF. Here is full output

Mm, the WPF version probably fails to check the other json or doesnt even try to acquire it. Will look into it later, contact me on discord for helping

I don't need assistance, I have it manually configured, just wanted to report to further improve the program.
Thank you for all the work you put into making the AIO GTA IV guide!

Actually I don't either. I forgot I have another double-gpu laptop in the house, lol.

Fixed in the upcoming release. Stupidly obvious typo, thanks for bringing my attention to this.


Pod install error migrating to swift 3

Hi,
I'm getting the following error when trying to install PromiseKit for Swift 3:
Analyzing dependencies
Pre-downloading: `AlamofireImage` from `https://github.com/Alamofire/AlamofireImage.git`, branch `swift3`
Pre-downloading: `PromiseKit` from `https://github.com/mxcl/PromiseKit.git`, branch `swift-3.0`
[!] The `PromiseKit` pod failed to validate due to 1 error:
    - ERROR | attributes: Unable to validate `dependencies` (Illformed requirement `"swift3"`).

This is what i have in my pod file:
pod 'Alamofire', :git => 'https://github.com/Alamofire/Alamofire.git', :branch => 'swift3'
pod 'AlamofireImage', :git => 'https://github.com/Alamofire/AlamofireImage.git', :branch => 'swift3'
pod 'PromiseKit', :git => 'https://github.com/mxcl/PromiseKit.git', :branch => 'swift-3.0'

I noticed in the pod spec that the swift3 branch of Alamofire is required but as you can see from the section of my pod file I already have it specified.

Pushed a fix.


upgrade to zmq ~2.2.0

this is a trivial one, but without this dependency I cannot npm install m2node

Guess nobody cares about this. I certainly don't anymore :)


Update docs to use an approved 1Password logo

Problem description
Our friends from 1Password reached out to have this page updated to include their official logo (it appears that we grabbed an old logo).

I saw a product documentation issue on the following URL: https://www.pulumi.com/docs/esc/

Other:

Affected product version(s)
N/A
Suggestions for a fix
Updated the infographic with the logo attached.

Seems redundant with https://github.com/pulumi/pulumi-hugo/issues/3962

Closing this one, I have the new esc image back from our designer I'll wrap this one up in https://github.com/pulumi/pulumi-hugo/issues/3962


Automatic deploy via custom script plugin on pull request

Automatic deploy didn't start on pull request.
I make pull request from develop to staging branch and it only runs npm tests. After tests are successfully passed it stops.
Than I need to manually run "Test & Deploy" to run deploy from "Deploy" section in "Custom Script" plugin.
Thanks in advance

The project settings page has the following option:

make sure you have it enabled.

@knownasilya everything is checked ( active ).

Which provider? And have you updated your plugins?

Provider: bitbucket
Plugins: all up to date

And Strider is on the latest version?

@knownasilya
Version is 1.9.4
Is there easy way to update to latest 1.9.5
Can I do that with npm update?

You should be able to just npm install -g strider, if that's how you installed it initially.

@knownasilya
There was issue on our side. We have strider-middleware which hooks tests or test and deploy functionallity after pull requests.
There was if, else. In case if it is master branch than make test and deploy and for all other do tests only.
So, I just extended if branch master or staging make test and deploy and it works.
Sorry for causing mess with this.

No problem


Tests failing

The image tests are failing, but the bash is too complex for me to really debug. I think it's ok for now.

I don't think it's the tests, I think there's a genuine error. A clean rebuild of main leads to the desktop failing to appear, and a Javascript error Uncaught Error: Cannot find module '@novnc/novnc/core/rfb'

Hmmm.... maybe there are multiple errors, https://github.com/jupyterhub/jupyter-remote-desktop-proxy/pull/120 is still failing

Did a new version of websockify arrive 1 month ago perhaps? Tests fail when websockify is involved, but for both tiger and turbovnc, and in a test without this project proxying things involved.

I've removed websockify: https://github.com/jupyterhub/jupyter-remote-desktop-proxy/pull/119


Improve error message in case too much nesting

Fixes https://github.com/rinja-rs/rinja/issues/28.

Updated!


[Docker] Update Dockerfile to force-use transformers main

What does this PR do?
Optimum pins transformers to <4.35.0, leading our docker images to have transformers==4.34.1, which is not intended since we want to test PEFT against transformers and accelerate main
This PR fixes it, there is no need to update the cpu docker file as optimum is not installed there
cc @pacman100 @BenjaminBossan

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@BenjaminBossan as far as I know we've been always testing on transformers main: https://github.com/huggingface/peft/blob/main/docker/peft-gpu/Dockerfile#L38 except until the latest optimum release

Oh, so this is more of a problem of the order in which the packages are installed. Is there a reason why bitsandbytes optimum auto-gptq are installed in a separate block further down below? Maybe they should be installed in the first block, and then we add a second block to install every package that needs to be installed from source? Otherwise, we are prone to run into similar issues in the future if any dependency adds another pin to a version.

Makes sense yes, let me update that as well

Thanks @BenjaminBossan !


Can I consider mistral as llama?

Hi,
I want to ask can I apply all of quantization method to mistral following the example of llama?

You can have a try, it will report an error if the flags are not supported.


Feature request: moving data between remote and local

Awesome work on this plugin; it's crazy to me that it works so well out of the box.
One thing that might make it even better would be a way to transmit data between the two processes (not just the text display of data), e.g. something like this
julia> x = 1
julia> @sendremote x
remote> y = x + 1
julia> @fetchremote y
julia> @assert y == 2

This would be super useful for situations where you are doing compute intensive work on a server but want to e.g. plot the results locally.

We could certainly have a way to do this as I already use Serialization for serializing values.
Regarding syntax, how about some "REPL magic" for sharing values.  Maybe get and put?
remote> %get x  # send x from remote to local
remote> %put x  # send x from local to remote

A prefix % is invalid Julia syntax so it won't clash with anything.
We could have send and recv or receive but they're longer to type and not necessarily clearer.
One caveat is that you won't be able to send and receive user-defined types seamlessly unless you have the same packages loaded on both the local and remote sides. But that's just a standard problem also shared by Distributed.jl and any other library which does similar things.

Looking at the verbs above, they seem a bit backward... perhaps we need
remote> %get x  # get x from local to remote
remote> %put x  # put x to local from remote

The same ambiguity exists for send and recv as well. I wonder whether there's some other verbs we could use.
There's another design issue here which is the use of special syntax vs some macros — the good thing about special syntax is that it's scoped to a particular remote session so there's some way to find which particular remote Connection we're talking about (in the future it might be sensible to allow multiple remote connections)

We could also support expressions, so you could do simple renaming, eg getting variable y from local and putting it into x on the remote:
remote> %get x = y

or arbitrary expressions, eg getting x on the remote by computing y+z locally and sending the result:
remote> %get x = y+z

I agree it makes sense for both get/put to be done from remote> mode and that "get" should mean local->remote.
If we want to be cute (and maximally efficient), we could have a dedicated mode for these operations:
julia> x = 1
julia> >
remote> >
get> x
get> <
put> y = x + 1
put> [backspace]
remote> [backspace]
julia> @assert y == 1

Having written this out, I think that your %get syntax probably makes more sense, but there it is.
Agreed expressions would be nice.

Another thought: as an alternative to %get, you could use variable prefixes to indicate that the variable should be taken from the local process.
julia> x = 1
remote> y = %x + 1

This could conceivably be used for %put as well, although it seems a bit weirder in this case.
remote> %z = y + 1
julia> @assert z == 2

I agree it makes sense for both get/put to be done from remote> mode and that "get" should mean local->remote.

Thanks, good to know that makes some sense outside my own head :)

as an alternative to %get, you could use variable prefixes to indicate that the variable should be taken from the local process.

I think we shouldn't be too clever with syntax here to avoid parsing difficulties. The benefit of a simple %get and %put is that it's strictly a prefix to the line, and everything after that can be parsed with the normal Julia parser. I think this will also make it simpler to understand.

have a dedicated mode for these operations

That could work, though I don't think there's repl sub-modes so this might be hard to implement.
I've implemented the %get and %put syntax in #13

Another thought: as an alternative to %get, you could use variable prefixes to indicate that the variable should be taken from the local process.
julia> x = 1
remote> y = %x + 1

I was just thinking a little more about this and I agree something like this would be pretty slick. The main reason would be to allow a REPL workflow with single-line commands which mix local and remote variables, without the need to manually transfer them in a precusor step.
The $ syntax is actually a pretty good match for this kind of interpolation in normal Juila code. For example, here $y would come from the remote and x would be local:
julia> @get_remote plot(x, $y)

The main problem with this is having a clean way to infer which Connection we're referring to. A global variable holding the current connection would work, it's just not very satisfying!

Yes this would be pretty nifty. I thought of $ first but didn't suggest it because its rlready legal julia syntax so it might be harder to parse. Although it's basically the same thing as in Distributed.jl so maybe you could reuse that code.

The main problem with this is having a clean way to infer which Connection we're referring to.

Is this not equally true for using > to enter remote mode?

I thought of $ first but didn't suggest it because its rlready legal julia syntax so it might be harder to parse.

It's not so much hard to parse as it is ambiguous with the existing meaning of $ to mean interpolation in other contexts. But it would be ok to steal the meaning provided there's a macro outside the expression.  Other macros like Benchmarks.@btime do the same thing.
The parsing itself is easy, for example:
julia> Meta.parse(raw"@get_remote plot(x, $y)")
:(#= none:1 =# @get_remote plot(x, $(Expr(:$, :y))))

Is this not equally true for using > to enter remote mode?

True.
I was wondering how it would relate to multiple remote connections and whether there'd be a repl mode for each remote... But I guess that doesn't make a lot of sense. It would make more sense to have a single RemoteREPL mode connected to a particular client, with a menu to switch between clients.

Agreed. I think it makes sense to have one "active connection" at a time, with a menu to switch between them. Or perhaps you could just use connect_repl for both the initial connection and switching (with some kind of cacheing to treat the second call differently). I imagine that most users will only want to connect to one remote in each session though.

I reopened this because I think we could add a macro. Perhaps I was a bit hasty with releasing a version with %get and %put, but we can always replace them if we find something more convenient :-)
Here's my current thoughts about how this might work. We'd add a single macro which interpolates any $-expressions with the remote evaluation of those expressions. Ideally this would appear to work symmetrically on both the remote and local sides (though the implementation maybe be slightly asymmetric internally). For example:
julia> @fetch plot($x, $y)

remote> @fetch a,b = $data

Naming-wise, @fetch does clash with Distributed.@fetch though they're semantically similar https://docs.julialang.org/en/v1/stdlib/Distributed/#Distributed.@fetch
So we could use something else.  Ideas

@with_remote
@rget ("remote get")

There's some value in having this short — given dependence on a global connection variable it's a tool for interactive use rather than an API for serious code — so @rget might be a good option.
Implementation-wise, we need a way of getting the "current remote" side of the connection.

In the local client julia> REPL, it's just found via the current remote in a global variable (the client may be connected to multiple servers, but only one will be active in the remote> REPL at any given time)
On the server side, we can set a task local variable so the client making the call becomes the current remote (the server may be serving multiple clients at once). This would be a pull-based implementation. Alternatively we could do some syntax analysis of the AST on the client side and eagerly expand any calls to @rget before sending the AST. This push-based option would have less latency though it's also a little strange to subvert the normal macro expansion process and partially split it between client and server.

From a user perspective, @rget (name and semantics) makes a lot of sense to me. I think this is strictly better than the %put/%get syntax.
I'm not familiar with the implementation (nor with macros in general), so I can't comment on the relative merits of the push- vs pull-based strategies. Excited to try this out though!

This looks like a great solution to me!


[SPARK-31037][SQL] refine AQE config names

What changes were proposed in this pull request?

When introducing AQE to others, I feel the config names are a bit incoherent and hard to use.
This PR refines the config names:

remove the "shuffle" prefix. AQE is all about shuffle and we don't need to add the "shuffle" prefix everywhere.
targetPostShuffleInputSize is obscure, rename to advisoryShufflePartitionSizeInBytes.
reducePostShufflePartitions doesn't match the actual optimization, rename to coalesceShufflePartitions
minNumPostShufflePartitions is obscure, rename it minPartitionNum under the coalesceShufflePartitions namespace
maxNumPostShufflePartitions is confusing with the word "max", rename it initialPartitionNum
skewedJoinOptimization is too verbose. skew join is a well-known terminology in database area, we can just say skewJoin

Why are the changes needed?

Make the config names easy to understand.
Does this PR introduce any user-facing change?

deprecate the config spark.sql.adaptive.shuffle.targetPostShuffleInputSize
How was this patch tested?

N/A

cc @JkSelf @maryannxue

Test build #119314 has started for PR 27793 at commit 186a6af.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24054/
Test PASSed.

Test build #119314 has finished for PR 27793 at commit 186a6af.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119314/
Test FAILed.

Test build #119316 has started for PR 27793 at commit 2ebd979.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24055/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119316/
Test FAILed.

Test build #119324 has started for PR 27793 at commit 9ce3f04.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24061/
Test PASSed.

Test build #119324 has finished for PR 27793 at commit 9ce3f04.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119324/
Test FAILed.

Test build #119325 has started for PR 27793 at commit 2dffcd5.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24062/
Test PASSed.

Test build #119325 has finished for PR 27793 at commit 2dffcd5.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119325/
Test FAILed.

Test build #119363 has started for PR 27793 at commit 3daf7df.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24101/
Test PASSed.

Test build #119363 has finished for PR 27793 at commit 3daf7df.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119363/
Test FAILed.

Test build #119366 has started for PR 27793 at commit 5a63fdb.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24103/
Test PASSed.

Test build #119366 has finished for PR 27793 at commit 5a63fdb.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119366/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24105/
Test PASSed.

Test build #119368 has started for PR 27793 at commit f2dea40.

Test build #119368 has finished for PR 27793 at commit f2dea40.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119368/
Test FAILed.

Test build #119381 has started for PR 27793 at commit 8d18db8.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24118/
Test PASSed.

Test build #119381 has finished for PR 27793 at commit 8d18db8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119381/
Test PASSed.

thanks for review, merging to master/3.0!


[IGListIndexSetResult hasChanges]` should instead be a readonly property #402

Changed method to read only property.

this is breaking

Changes Unknown when pulling 958c234ab4d1adc495a283b3ded1418f6d95d573 on AdamRobertsEF:master into ** on Instagram:master**.

Changes Unknown when pulling 958c234ab4d1adc495a283b3ded1418f6d95d573 on AdamRobertsEF:master into ** on Instagram:master**.

Changes Unknown when pulling 958c234ab4d1adc495a283b3ded1418f6d95d573 on AdamRobertsEF:master into ** on Instagram:master**.

@rnystrom I have some github app issue and reforked. I didn't expect it to close the commit, since you already imported.


Add service account names to credentials request manifest

For a new way of managing OpenShift credentials on AWS to work, using Security
Token Service, we need external tooling to know the service account names from
the Credentials Request manifest so that it can create IAM Roles that can be
assumed only by those specific service accounts.
Managing credentials using STS ref: https://github.com/openshift/cloud-credential-operator/blob/master/docs/sts.md
Tooling design ref: https://issues.redhat.com/browse/CCO-60
/cc @joelddiaz

/hold

/unhold
Added service accounts to new CredentialsRequest field serviceAccountNames

/retest

/retest

/lgtm

/retest

/retest

/retest

/retest


Addressing the Issue of Missing Boundaries in Small Countries/Regions

This is the case with Nauru.
I believe we will encounter the same issues with other extremely small countries due to the oversimplification.

Download the topojson from https://github.com/worldbank/WB-rezoning-explorer/tree/staging/app/public/zones/country. Nauru iso code is NRU, so you will need to download NRU.topojson
Open it in QGIS, and it will look like this.

Since this data is useless, we need to download the layer from the internet. I found files for some countries in GeoBoundaries. We need the adm1 layer, try to find the simplified ones.
If you have downloaded the layer, update the properties to include 'id' and 'name'.

I downloaded this file from geoboundaries, and changed from shapeName to name, shapeIso to id.
Convert to topojson

This data should be extracted from the GADM 4.0 dataset which was (supposed
to be) used as an updated global boundary layer for this version of the
tool. Can you please confirm this is what is being used?
I am surprised to hear that the geometry is not valid for some countries.
Thanks!!
On Mon, Jun 12, 2023 at 9:53 AM Dimas Ciputra @.***>
wrote:

This is the case with Nauru.
I believe we will encounter the same issues with other extremely small
countries due to the oversimplification.

Download the topojson from
https://github.com/worldbank/WB-rezoning-explorer/tree/staging/app/public/zones/country.
Nauru iso code is NRU, so you will need to download NRU.topojson
Open it in QGIS, and it will look like this.
[image: image]
https://user-images.githubusercontent.com/1979569/245054518-8168a16a-631d-4e25-baf6-72fa3b34a815.png
Since this data is useless, we need to download the layer from the
internet. I found files for some countries in GeoBoundaries. We need the
adm1 layer.
If you have downloaded the layer, update the properties to include
'id' and 'name'.
[image: image]
https://user-images.githubusercontent.com/1979569/245054904-6fef9ca0-3e1d-4a65-9e71-c89084e17155.png
I downloaded this file from geoboundaries, and changed from shapeName
to name, shapeIso to id.
Convert to topojson

—
Reply to this email directly, view it on GitHub
https://github.com/kartoza/rezoning-2-project/issues/85, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACHU3HPGJRURVNLXE2N6AUTXK24G3ANCNFSM6AAAAAAZC5GRZ4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi @ClaraIV, yes, the data was downloaded from GADM, and the layers have been simplified using the same factor. While this approach works well for larger countries, it renders extremely small countries practically unusable.
So, I believe that for now, we should manually adjust the small countries to save time, rather than tweaking the factor


Remove keywords from Cargo.toml

error: failed to publish to registry at https://crates.io/
Caused by:
  the remote server responded with an error: expected at most 5 keywords per crate

Also noticed that the categories don't exist on crates.io. Will change that just in case.

Looking at generally similar libraries, "date", "time", "calendar", "timezone", and "duration" might be best.
Lib.rs

Yeah, we can publish using those and modify them if needed in the future.


Update http4s-jdk-http-client to 0.3.3

Updates org.http4s:http4s-jdk-http-client from 0.3.2 to 0.3.3.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s", artifactId = "http4s-jdk-http-client" } ]

labels: test-library-update, semver-patch

Superseded by #102.

Superseded by #102.


Need to adjust this comment and test wandb logging

Could use some help with testing wandb logging, and other parameters in this section:
https://github.com/chipGPT/nanoGPT/blob/3e6b8ac0f0958026d3ea6c9232b93f317a003c39/train.py#L84C1-L85C1

Hi @gkielian , can i work on this issue

@gkielian is this still a good first issue?

Thanks! Looking at this again, thinking that adding more data types and datasets might be the best way to get started.
Updating the labels


Single Node.js instance

As discussed at https://github.com/runtimejs/runtime/issues/75#issuecomment-120875129, implement a module to use a single instance of Node.js. Probably there's something similar available, but the idea here is to "replace" the Node.js executable itself so all the new executions are redirected to the same Node.js instance. Probably this will not be really useful until threated WebWorkers support don't arribe to io.js/Node.js-next, but until that some proof of concept can be done with child_process.fork creating new processes.

Another option is to use the multi-context feature, available from the vm module.

I'll see about making a gist tonight. It shouldn't be absurdly difficult to get a proof of concept

:+1: I did a quick search and only found pidfile module, but it's too primitive and focused on instance of an application and not node.js itself, so it's useless to us.

https://github.com/avz/node-mkfifo This is the sort of thing your were interested in correct?
Apparently using mkfifo is dangerous territory. I'm currently implementing a simple version that just creates a version listening to a file.
As for the multi-context environment, I had gotten started on that earlier. Here is the most relevent file to it
According to strong loop, it seems as though net and other native modules are already being handled. However, when "closing" a context seems to still be a needed feature. As a result, keeping track of all asyncronous aspects created is pretty important and involves a lot more work : /. Threads appear to be a golden opportunity from a clean up perspective.

https://github.com/avz/node-mkfifo This is the sort of thing your were interested in correct?
Apparently using mkfifo is dangerous territory. I'm currently implementing a simple version that just creates a version listening to a file.

Yes, that's what I was talking about :-) Didn't know about the flaws of mkfifo, but as in your second link is suggested, you can mostly do the same with domain sockets that are already supported by Node.js. I'm not sure about the diferences between them (maybe named pipes are lighter because they don't involve the network stack? maybe better compatibility of domain sockets with network-oriented applications?) but for this use case both are valid ones, so it's ok for me to use domain sockets :-)

As for the multi-context environment, I had gotten started on that earlier. Here is the most relevent file to it

I read it a while ago but didn't understood too much about what you were propossing, sorry :-P I'll need to take it another look... :-)

According to strong loop, it seems as though net and other native modules are already being handled. However, when "closing" a context seems to still be a needed feature. As a result, keeping track of all asyncronous aspects created is pretty important and involves a lot more work : /

The article is somewhat old (pre v0.12) so probably things got better now.

Threads appear to be a golden opportunity from a clean up perspective.

Definitelly, I'm too much into processes but seems here is the right way to go :-)

Do we still want to attempt this?


Remove PhysX 2.8

As it's not available any more, and apparently the PhysX 3 module is working.

:+1:

Handled with #1899


Notification Icon oversized

I reported this issue back when I was using Xubuntu 15.10 Beta 2 (https://github.com/Yubico/yubioath-desktop/issues/53) and it seemed to go away with the official released version of 15.10, but it seems to be back.

I've tried uninstalling (apt-get remove and apt-get --purge) with reboot and it still shows the oversized icon in the notification area after a fresh install.  I can't seem to figure this one out.  Could this possibly be a theme issue where it's cached somewhere?
System info:

It does sound like something might be cached somewhere that isn't getting purged. I haven't been able to reproduce this myself, and I've run it on multiple versions of XUbuntu, including 15.10. Unfortunately I have no idea as to what is causing it in your case.

I'm still experiencing this problem, even on a completely different system (LinuxMint 17.3).

Is this still the case with the new version (4.0.0) ?
https://developers.yubico.com/yubioath-desktop/Releases/

I will test this out and let you know.  I'm currently having a problem with the latest version that was just released (4.0.0).  It upgraded successfully, but it won't launch.  I'll create a new ticket.

I'm closing this since the systray icon was dropped in the latest version. See #125


Fix Lax missing link in README

Pull Request Test Coverage Report for Build 421

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.654%

Totals

Change from base Build 416:
0.0%

Covered Lines:
4890

Relevant Lines:
4907

💛  - Coveralls


VM test - could not find a pecan.xml

library(PEcAn.all)
settings.file <- c("/home/dlebauer/dev/biocro_regional/vignettes/illinois_mxg_settings.xml")
settings <- read.settings(settings.file)
2015-08-31 09:04:20 SEVERE [read.settings] :
Could not find a pecan.xml file
Error in logger.severe("Could not find a pecan.xml file") :
Could not find a pecan.xml file

sorry - that workflow is not on the PEcAn VM. Here is what to do:

If you haven't already, and the PEcAn VM is connected to the world, change the username/password for the carya user

Set up an ssh key-pair with Github

clone the biocro_regional repository
 cd ${HOME}
 git clone git@github.com:ebimodeling/biocro_regional

cd into the pecan directory, checkout the biocro-module branch, and build PEcAn
 cd ${HOME}/pecan
 git pull
 git fetch --all
 git checkout -b biocro-module2
 ./scripts/build.sh --dependencies

Change the file path in the workflow from
 `settings.file <- c("/home/dlebauer/dev/biocro_regional/vignettes/illinois_mxg_settings.xml")`

to
    `settings.file <- c("~/biocro_regional/vignettes/illinois_mxg_settings.xml")`

.... proceed, and let me know if this resolves the issue ...

How can I configure PEcAn VM to submit jobs to ROGER?

create a new user account
git clone git@github.com:pecanproject/pecan
./pecan/scripts/sshkey.sh

follow instructions (roger-login.ncsa.illinois.edu; username, copy-paste last line in terminal beginning with cat " ....  ... to send public key to roger

now this should work ...

Thanks, David! It works.

The sshkey setting looks okay, however, after I reload setting file and run to the following command,
start.model.runs(settings, settings$database$bety$write) # Start ecosystem model runs
It is still using my account instead of Tao's to submit jobs, any steps I missed?

Oh, just figured out I need to change the .ssh/config user name option, it works now.


Sign Up Page Issue

Not Sign up Page Forgot Password Not Working
And Sign Up Link On Nav is Not Working

I want to Fix This Issue Under GSSOC '23 . Could you please assign me this issue?

I want to fix this issue, could you please assign this issue to me?

Similar issue is already being assigned!


Update predefined macros

Use correct value of __STDC_VERSION__
Add placeholder values for __DATE__ and __TIME__
Update test runner to create a new compilation for each test since the predefined macros can change based on the test.

Updated it to use the epoch API you linked, for when that gets merged.
I changed the way the test runner handles the predefined macros but I'm not entirely sure about what you mean regarding removing the magic numbers from Compilation - I assume you mean the + 2's but I don't know how adding a new Id builtin lets me get rid of those.

Thanks! I fixed a small off-by-one error in __DATE__, month_day.day_index is [0,30] we want [1,31].


isJSON() or isParsable()

Hi,
This would be very helpful to have either isJSON() or isJSONParsable() to know if we can run deserialize on the input. ?
What do you all think of this ?
Thanks

I think it's out of scope for Underscore.


Added instructions to run elasticsearch on latest macos

Summary

Added instructions to run ElasticSearch on the latest macOS

Does CI-SKIP work on docs changes yet? If not please add support

Sure. Will check and will make the necessary changes.


fix: issue with end_idx inferring int type from variant type

var types cannot be inferred using := from variant types so I used mini instead of min to correctly return an int type

@Ark2000 this is a tiny fix, should be easy to merge. If there is an specific way you'd like MR's please let me know as I would love to contribute to your project. I can have a look at the open issues and roadmap in the future, this is more of a tiny fix to test your process

Sorry for delay, thank you for contribution.