IndexError on running pipenv install

With a fresh install on Windows 10, I get the following failure and traceback. This is with Python 3.6 installed for all users, with Pipenv installed from pip install --user pipenv. I'm curious if this perhaps has something to do with the "find_windows_executable" function.
PS C:\Users\mccar\Documents\Software Development\bittorrent> pipenv install
Creating a virtualenv for this project…

Virtualenv location:
Creating a virtualenv for this project…

Virtualenv location:
Pipfile.lock not found, creating…
Locking [dev-packages] dependencies…
Traceback (most recent call last):
  File "c:\program files (x86)\python36-32\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "c:\program files (x86)\python36-32\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Users\mccar\AppData\Roaming\Python\Python36\Scripts\pipenv.exe\__main__.py", line 9, in <module>
  File "C:\Users\mccar\AppData\Roaming\Python\Python36\site-packages\pipenv\vendor\click\core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "C:\Users\mccar\AppData\Roaming\Python\Python36\site-packages\pipenv\vendor\click\core.py", line 697, in main
    rv = self.invoke(ctx)
  File "C:\Users\mccar\AppData\Roaming\Python\Python36\site-packages\pipenv\vendor\click\core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "C:\Users\mccar\AppData\Roaming\Python\Python36\site-packages\pipenv\vendor\click\core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "C:\Users\mccar\AppData\Roaming\Python\Python36\site-packages\pipenv\vendor\click\core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "C:\Users\mccar\AppData\Roaming\Python\Python36\site-packages\pipenv\cli.py", line 1785, in install
    do_init(dev=dev, allow_global=system, ignore_pipfile=ignore_pipfile, system=system, skip_lock=skip_lock, verbose=verbose, concurrent=concurrent, deploy=deploy)
  File "C:\Users\mccar\AppData\Roaming\Python\Python36\site-packages\pipenv\cli.py", line 1293, in do_init
    do_lock(system=system)
  File "C:\Users\mccar\AppData\Roaming\Python\Python36\site-packages\pipenv\cli.py", line 1022, in do_lock
    python=python_version(which('python', allow_global=system)),
  File "C:\Users\mccar\AppData\Roaming\Python\Python36\site-packages\pipenv\cli.py", line 1404, in which
    p = find_windows_executable(os.path.join(location, 'Scripts'), command)
  File "C:\Users\mccar\AppData\Roaming\Python\Python36\site-packages\pipenv\utils.py", line 820, in find_windows_executable
    return exec_files[0]
IndexError: list index out of range
PS C:\Users\mccar\Documents\Software Development\bittorrent>

Thanks for the report! I think you’re spot on. I wrote this functionality and am working on a fix for some window path issues as well. Can you tell me the output of pipenv —version?

My version is 8.2.6
pipenv, version 8.2.6```

If it helps, I've experienced this error with Python installed through chocolatey, and I uninstalled chocolatey and installed python through the default 32 bit installer from Python.org, with the box checked to set environmental variables.
Could I manually perhaps add an environmental variable with the path to whatever executives this function wants?  I'd like to work around this if possible.

Can you provide the output of pipenv install --verbose as well as pipenv --venv?  This should tell us exactly what is going wrong here.  I suspect it has to do with having a space in your path, which will be fixed whenever #893 is pulled and released (currently waiting on me to write the tests for it).

Marking this as bug!  Thanks for the work @techalchemy!

I've got the same issue, @mccartykim, I wonder if you found some workaround? Thanks

When i tried to run  tests on windows i got the same error and other tests failed because of pew not finding psutil

It helped me to uninstall Python, remove all Python user folder, remove all related PATH entries and Re-install.

I hate python on Windows

@mccartykim can you verify whether this is resolved?

Hi, yes, I can confirm it works as expected now. I'm not on my Windows machine at the moment, but I can confirm I was able to pipenv install and pipenv shell after updating my version with no errors. I tried it in just Powershell, however.  If you'd like me to try CMD, I'll try it next time I'm at that machine.

@mccartykim thanks for the update, I will go ahead and close this out no need to try it out again!


Only serialize a populated field if it exists

I have a field in my schema that is populated by fastjoin but may or may-not exist, so ideally if it doesn't exist I just want to return a null value. However I want to use serialize compute on one of the populated fields if it doesn't exist.
I can't see a way of doing this currently, is there anything you can suggest to get around it
At the moment in the example below it will return an object with a null price value but I don't want to return an object at all I just want to ignore the computed field if finish is empty.
  names: {
    // Populated array
    finish: {
      // Populate field that doesn't always exist
      computed: {
        price: (finish, hook) => {
          return finish ? finish.price / 100 : null;
        }
      }
    }
  }

fastJoin is based on GraphQL. In GraphQL every defined field must have a value or be null.
You'll need a hook to look for and remove null fields.


Merge lora adaptor leads to saving the model in FP32 though base model is of FP16/BF16

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
After merge the final merged model should have been of same precision as the base model but it is FP32 this leads to size of the merged model almost doubling as compared to size of the base model.
Current behaviour
After merge the final merged model should have been of same precision as the base model but it is FP32 this leads to size of the merged model almost doubling as compared to size of the base model.
Steps to reproduce
Generate a lora adaptor.
Below is the config i used for Gemma 2B Pretraining with Lora.
after pt i try to merge the adaptor generated with base model.
In merged models config.json i see torch.dtype as FP32. Thus the size of merged model is 10 GB when base model is of 5GB
Config yaml
base_model: google/gemma-2b
model_type: AutoModelForCausalLM
tokenizer_type: AutoTokenizer

load_in_8bit: false
load_in_4bit: true
strict: false

datasets:
  - path: /content/test_txt_data-10exmpl.json
    type: completion
    field: text
dataset_prepared_path:
val_set_size: 0
output_dir: ./qlora-out

adapter: qlora
lora_model_dir:

sequence_len: 600
sample_packing: true
pad_to_sequence_len: true

lora_r: 64
lora_alpha: 128
lora_dropout: 0.05
lora_target_modules:
  - q_proj
  - v_proj
  - k_proj
  - o_proj
  - gate_proj
  - down_proj
  - up_proj
#lora_modules_to_save:
#  - embed_tokens
#  - lm_head
lora_target_linear: true
lora_fan_in_fan_out:

gradient_accumulation_steps: 1
micro_batch_size: 1
num_epochs: 1
optimizer: adamw_bnb_8bit
lr_scheduler: cosine
learning_rate: 0.0002

train_on_inputs: false
group_by_length: false
bf16: false
fp16: false
tf32: false

gradient_checkpointing: true
early_stopping_patience:
resume_from_checkpoint:
local_rank:
logging_steps: 1
xformers_attention:
flash_attention: False

warmup_ratio: 0.1
evals_per_epoch: 1
eval_table_size:
eval_max_new_tokens: 128
eval_sample_packing: False
saves_per_epoch: 1
debug:
deepspeed:
weight_decay: 0.0
fsdp:
fsdp_config:
special_tokens:

save_safetensors: True

Possible solution
No response
Which Operating Systems are you using?

[X] Linux
[ ] macOS
[ ] Windows

Python Version
3.10
axolotl branch-commit
latest
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this bug has not been reported yet.
[X] I am using the latest version of axolotl.
[X] I have provided enough information for the maintainers to reproduce and diagnose the issue.

code mistake. not an issue.

This is an issue still there. earlier i thought this is an issue with my script but it is not. axolotl is saving in FP32 at the end of pretraining with lora. Both merged model and adaptor is stored in FP32 precision though base model is in FP16.


Support for crc pause

It would be great to implement support for pausing the CRC virtual machine.
In development laptops, where CRC consumes a significant amount of resources/batteries, this would allow for an easy (and MUCH quicker than stop/start) pause/resume mechanism to interact with the local OCP environment.
A couple of failed attempts to pause the VM form the virsh perspective:
❯ sudo virsh managedsave 1
error: Failed to save domain '1' state
error: Requested operation is not valid: cannot migrate domain: non-migratable device: 0000:00:02.0:00.0/vhost-user-fs

❯ sudo virsh dompmsuspend 1 disk
error: Domain 'crc' could not be suspended
error: argument unsupported: QEMU guest agent is not configured

This might have unforeseen consequences of time-skew and possibly expired certificates, plus not all hypervisors support this. Also, not sure why but the guest agent does not start in the Microshift image. This I also noticed yesterday when debugging an issue around the startup phase

Also, not sure why but the guest agent does not start in the Microshift image. This I also noticed yesterday when debugging an issue around the startup phase.

We configure qemu-guest-agent to use vsock for communication, but we don't add a vsock device to our VM at least on linux iirc, maybe this is what you saw.

maybe this is what you saw.

Does this prevent the guest-agent from functioning properly, as the service on Linu-hosted VMs fails.

Is there any other use of the guest-agent? if not, perhaps we have to remove it, or fix the start definition, as at the moment it shows 'degraded' as state from systemd.

similar tools do not provide a pause either; prefer to use stop and start.


Execute function instead of whole script

Is it possible to execute a specific function in a script instead of the entire script? Something like vm.ExecuteFunction with the name of the function as a parameter would be nice.

I think there are other ways to solve this issue. For example, first run in the script with all the functions, then can run a script with just the wanted function. Could also look into using modules.

I'm not sure I understand. I can run a whole script, but how would I then run the wanted function only?
Modules look basically like classes to me. Unless I can interact with them from outside Anko, I'm not sure that modules would help me.

Split the script into two part. First run in the part with all the functions. Then run a script with just the function you want to call.

Are the variables shared between the two parts?
Basically I have a function (or in this case script) that gets called in regular intervals and with each execution of the script, all variables are reset. My thought was that if I just call a function and not the whole script, I can have variables outside of that function that don't reset between calls.

Any script can be run in any env, the env is where the script get run. Can keep running scripts as long as you want in the same env.
package main

import (
	"fmt"
	"log"

	"github.com/mattn/anko/env"
	"github.com/mattn/anko/vm"
)

func main() {
	e := env.NewEnv()

	err := e.Define("println", fmt.Println)
	if err != nil {
		log.Fatalf("Define error: %v\n", err)
	}

	script := `
a = 1
	`

	_, err = vm.Execute(e, nil, script)
	if err != nil {
		log.Fatalf("Execute error: %v\n", err)
	}

	script = `
a = a + 1
println(a)
`

	_, err = vm.Execute(e, nil, script)
	if err != nil {
		log.Fatalf("Execute error: %v\n", err)
	}

	_, err = vm.Execute(e, nil, script)
	if err != nil {
		log.Fatalf("Execute error: %v\n", err)
	}

	_, err = vm.Execute(e, nil, script)
	if err != nil {
		log.Fatalf("Execute error: %v\n", err)
	}

}

Output:
2
3
4

Okay, that solves this issue. Thanks!


Error in examples in documentation

First of all, I would like to thank you for your project. Great job. It looks very promising.

Describe The Bug:

In the documentation, in the examples section, you refer to the event.name and c.name that do not exist. The correct name of the property is serviceName.
To Reproduce:
Expected behavior:
Logs:
{
  aid: 4,
  iid: 9,
  uuid: 'xxx',
  type: 'ContactSensor',
  humanType: 'Contact Sensor',
  serviceName: 'xxx',
  serviceCharacteristics: [
    {
      aid: 4,
      iid: 11,
      uuid: 'xxx',
      type: 'ContactSensorState',
      serviceType: 'ContactSensor',
      serviceName: 'xxx',
      description: 'Contact Sensor State',
      value: 0,
      format: 'uint8',
      perms: [Array],
      unit: undefined,
      maxValue: 1,
      minValue: 0,
      minStep: 1,
      canRead: true,
      canWrite: false,
      ev: true,
      setValue: [AsyncFunction (anonymous)],
      getValue: [AsyncFunction (anonymous)]
    },
    {
      aid: 4,
      iid: 12,
      uuid: 'xxx',
      type: 'StatusLowBattery',
      serviceType: 'ContactSensor',
      serviceName: 'xxx',
      description: 'Status Low Battery',
      value: 0,
      format: 'uint8',
      perms: [Array],
      unit: undefined,
      maxValue: 1,
      minValue: 0,
      minStep: 1,
      canRead: true,
      canWrite: false,
      ev: true,
      setValue: [AsyncFunction (anonymous)],
      getValue: [AsyncFunction (anonymous)]
    },
    {
      aid: 4,
      iid: 13,
      uuid: 'xxx',
      type: 'StatusTampered',
      serviceType: 'ContactSensor',
      serviceName: 'xxx',
      description: 'Status Tampered',
      value: 0,
      format: 'uint8',
      perms: [Array],
      unit: undefined,
      maxValue: 1,
      minValue: 0,
      minStep: 1,
      canRead: true,
      canWrite: false,
      ev: true,
      setValue: [AsyncFunction (anonymous)],
      getValue: [AsyncFunction (anonymous)]
    }
  ],
  accessoryInformation: {
    Manufacturer: 'TuYa',
    Model: 'TS0203',
    Name: 'xxx',
    'Serial Number': 'xxx',
    'Firmware Revision': '?',
    'Hardware Revision': '?'
  },
  values: { ContactSensorState: 0, StatusLowBattery: 0, StatusTampered: 0 },
  linked: undefined,
  instance: {
    name: 'homebridge',
    username: 'xxx,
    ipAddress: 'xxx',
    port: 1234,
    services: [],
    connectionFailedCount: 0
  },
  uniqueId: 'xxx',
  refreshCharacteristics: [Function (anonymous)],
  setCharacteristic: [Function (anonymous)],
  getCharacteristic: [Function (anonymous)]
}

Plugin Config:
Screenshots:
Environment:

Plugin Version: 0.1.1
Homebridge Version: 1.7.0
Node.js Version: 18.19.0
NPM Version:
Operating System: Raspbian

❤ Thank you, fixed

fwiw i'm running this in my home now, verified working:
automation.listen(function (event) {
  if (event.serviceName === "Motion sensor") {
    if (
      event.serviceCharacteristics.find(
        (c) => c.type === "MotionDetected" && c.value === 1,
      )
    ) {
      // Find the light we want
      const light = automation.services.find(
        (s) => s.serviceName === "Tall Lamp",
      );
      if (light) {
        const on = light.serviceCharacteristics.find((s) => s.type === "On");
        if (on) {
          automation.set(light.uniqueId, on.iid, 1);
        }
      }
    }
  }
});

Huge painpoint getting the exactly correct service names and drilling through to find the characteristics to flip. Open to ideas on how we could improve the automation interrface for easier authoring.


broken layout on Internet Explorer 11

Hello,
see screenshot attached.

regards
Kai

Thanks for reporting this bug, we are looking into it.

Assigning this to Christian so his team can take a look at it.


enableVirtualBackground for IRtcEngine is now working with Expo SDK 50

Describe the bug
Hi, I'm using Agora for video calls using react-native-agora and a Fork of Agora UI Kit.
Working with Expo SDK 49 working perfectly blurring background, if we wanted, but after upgrade to Expo SDK 50 ("react-native": "0.73.6" ) call is still working but blur background has dissapear, only shows a completely normal background.
I've been researching documentation and I can't find anything. Also, this function is returning Promise void so I cannot include logs to know if there's a problem or not blurring.
To Reproduce
Steps to reproduce the behavior:

Create a minimal proyect with Expo SDK 50 and agora-react-native using this dependency https://github.com/elenitaex5/agora-video-uikit-react-native.
enableBlurBackground is an rtcProp passing them to UIKit
Starts a call
White Blur Background has to be included in video image
No blur Background is shown, only video background.

Expected behavior
After starting the video White Blur background has to be included in video image.
Screenshots

Smartphone (please complete the following information):

Device: Both Android or iOS
OS: All of them we've tried with several iphones and Android devices.
Version 4.3.2

Additional context
Add any other context about the problem here.

@elenitaex5 We have some samples in https://github.com/AgoraIO-Extensions/react-native-agora/blob/main/example/src/examples/advanced/VirtualBackground/VirtualBackground.tsx
Can you confirm is that work in the issue device?

@guoxianzhe  I've just tried in a new EXPO SDK 50 project and include only Virtual Background components (and dependencies or other components needed) with react-native-agora etc.
And this sample doesn't work. It only have expo sdk 50 new project chore and structure and  component in main screen and it doesn't work with any type of Background Source.
Attaching a picture. I've tried to enable background several times with multiple types of bacgkround with dropdown selectors.

I've just tried in a new EXPO SDK 50 project and include only Virtual Background components (and dependencies or other components needed) with react-native-agora etc.

@elenitaex5 Can you share this code to me?

Hi @guoxianzhe thanks for helping. This is my testing repository. As I said just create a new Expo SDK 50 project and copy-paste  component class and components and dependencies needed.
Route and Navigation are not added to BaseComponent, not needed for real test and our appId etc are hardcoded in Virtual Background component and not comes from config.  You can see it. It was a fast test.
This is test repository:
https://github.com/elenitaex5/expo-sdk-50

@elenitaex5 I saw your code and I found that you are using a file(agora-logo.png) that is not existing in your project. Please use a exist file to enableVirtualBackground
for example in android: https://github.com/AgoraIO-Extensions/react-native-agora/blob/main/example/android/app/src/main/assets/agora-logo.png

@guoxianzhe thanks for your answer, I didn't include the background image because I only want to try a Blur background or particular color background with no image and they don't work either. In example above you can check I'm trying blur background with a background color not image color.

@elenitaex5 You should delete source parameter like this.

I tested it in my real android device and it's works.

your code has passed a un-existing file and it will cause error.

Hi @guoxianzhe Thanks a lot for your help. This code is working in a clean project. Hope I can reproduce it in my real project.


ENH: warn user that quad() ignores points=... when weight=... is …

…specified

Reference issue

What does this implement/fix?
integrate.quad() allows user to specify points of local difficulty (points=...)
and a weighting function (weight=...). These are handled by different quadpack
routines and are mutually exclusive. However this was not explicit in the docstrings.
Providing both resulted in points of local difficulty being silently ignored.
Updated docstrings to reflect this; added code to display warning message if
both options are provided. Otherwise behaviour remains unchanged: we attempt
to evaluate weighted integrand without reference to breakpoints.
Additional information

thanks, merged


UnicodeEncodeError on submission

I submit like this:
kaggle competitions submit -c favorita-grocery-sales-forecasting -f submission.csv -m "/54/submission.csv submitted using kaggle api"

And get this error:
Traceback (most recent call last):
  File "/home/user/anaconda2/bin/kaggle", line 11, in <module>
    sys.exit(main())
  File "/home/user/anaconda2/lib/python2.7/site-packages/kaggle/cli.py", line 20, in main
    print(out,end='')
UnicodeEncodeError: 'ascii' codec can't encode character u'\xf3' in position 35: ordinal not in range(128)

When I submit this using the browser interface it is accepted but I get a score I have trouble believing (I have run multiple models, have both validation and test sets withheld from kaggle training data, both giving similar score ~0.52 but every model I upload gets about 1.1 on the public scoreboard)
Here are the first few lines of my submission file:
id,unit_sales
125497040,1.5736053353290398
125497041,1.793600498069159
125497042,2.63803404720867
125497043,1.6771344406059487
125497044,3.157071568983687
125497045,5.286638692777691
125497046,10.47901030240068
125497047,3.779807986274869
125497048,1.932556247291067
125497049,1.5810675114261763
125497050,2.4403552601948544
125497051,5.050816071983492
125497052,3.0398113039515886
125497053,1.5767793422420588
125497054,1.61912920839818
125497055,1.9439467429349728
125497056,1.6638621400716964
125497057,1.9610603072622248
125497058,2.864015139306119
125497059,3.701055296168941
125497060,4.666667300117952

I should have a fix out for this soon, but it looks like a python 2 only issue.  I can't guarantee continued support for python 2 problems.

Should be fixed now - please let me know if you have any additional problems.

Thanks for looking into this but I'm still getting the same problem :-(
user@silverstone:~/Data/Kaggle/Favorita/54$ kaggle competitions submit -c favorita-grocery-sales-forecasting -f submission.csv -m "/54/submission.csv submitted using kaggle api"
Traceback (most recent call last):
  File "/home/user/anaconda2/bin/kaggle", line 11, in <module>
    sys.exit(main())
  File "/home/user/anaconda2/lib/python2.7/site-packages/kaggle/cli.py", line 20, in main
    print(out,end='')
UnicodeEncodeError: 'ascii' codec can't encode character u'\xf3' in position 35: ordinal not in range(128)

@ablaom, what version of the API are you running?

Hi Rebecca,
My mistake - I forgot to update the kaggle api before testing. My file now successfully uploads. Many thanks.
Anthony
On 27/04/2018, at 11:02 AM, Rebecca Ysteboe<EMAIL_ADDRESS>wrote:
@ablaom https://github.com/ablaom, what version of the API are you running?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Kaggle/kaggle-api/issues/22#issuecomment-384814774, or mute the thread https://github.com/notifications/unsubscribe-auth/AdGnYC5BwA5P6Q8FCiQf-StKt5TRO8-rks5tslIfgaJpZM4TJGit.

Dr. Anthony Blaom
Mathematician
and Data Scientist
Auckland


dibs: I will implement exercise Matrix

Go Frida, go!

Hey @FridaTveit, would you mind connecting with me over email (on my GitHub profile)?  I have an Exercism-related question for you.

I'm curious to see what you choose to do with the problem interface here, @FridaTveit - I have pondered this puzzle a little in the past!

You've pondered the puzzle of this problem's protocol? (sorry, couldn't help myself... carry on)

Hah, indeed! In preparation for pontification!

I added a commit with the implementation a while ago but I rebased it on top of master instead of merging it, is that a problem?

Ah, no, that's perfectly fine, I had just missed it! I normally update the PR title to exercise-slug: add to track once I had an implementation that's ready for review, just to call out the difference :)

Ah, okay :)

Hey @stkent I'm assuming that you're reviewing this PR; that your intent?

Sure, I'm happy to; probably won't get to it for a couple days though, so if you want to jump in, that's fine by me.

@FridaTveit some comments in; mostly musing out loud 😄 . Let me know what you prefer in the cases where I suggested alternatives!

@FridaTveit thanks, changes look great! I added one more question that I didn't spot the first time around - let me know what you would like to do with that. Once that's been discussed and/or resolved, let's get this merged!

(Small merge with master also needed now :)

I took care of those config conflicts using the new inline GitHub merge tool. It worked well! Merging on successful build 🏆


When disable the Spell, you can't back to the "Enable" Mode

VSCode Version: Code 1.7.1 (02611b40b24c9df2726ad8b33f5ef5f67ac30b44, 2016-11-03T13:53:14.822Z)
OS Version: Windows_NT ia32 6.1.7601
Extensions:

Extension
Author
Version

csharp
ms-vscode
1.4.1

Spell
seanmcbreen
0.8.6

Steps to Reproduce:
1.Click on Enable Spell to Diable it
2.Try to Enable again

@cidisaac In the latest insiders, I improved the behaviour to revert back the action. It means, if you disable an extension and you can revert it back by clicking enable action before restarting VS Code.


Please update this KB with correct Ubuntu version

https://docs.microsoft.com/en-us/azure/virtual-network/virtual-network-multiple-ip-addresses-portal#linux-ubuntu
Can you please update this KB for Ubuntu section, saying the process is valid for Ubuntu 14/16. Ubuntu18 is using netplan to manage Network Interfaces as opposed to /etc/network/interfaces

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

ID: b3b4d2d7-88a3-c493-68f9-39c89fbbcf96
Version Independent ID: 6dff4842-197d-9e60-3c80-b34cc6aaab3e
Content: Multiple IP addresses for Azure virtual machines - Portal
Content Source: articles/virtual-network/virtual-network-multiple-ip-addresses-portal.md
Service: virtual-network
GitHub Login: @anavinahar
Microsoft Alias: annahar

@arunkp89 Thank you for your feedback! We will review and provide an update as appropriate.

Done, will be live by tomorrow at the very latest.


can i import alertify in react

I suppose so.
We use webpack and this is what we do,
import alertify from 'alertify.js';
And use alertify as documented.

I'm using this way:
var alertify = require('alertify.js')
And displaying alerts:
alertify().alert('I am an alert')

Closing this as it seems to have been answered. If not, feel free to comment and/or re-open.


Meta links not displaying

Hi,
I like the meta feature (http://blog.imdone.io/post/111622263565/imdone-is-meta) but for some reason when I try using it everything works except for the link... no errors are thrown, the link icon just isn't displayed:

Thanks!

Do you have it configured in .imdone/config.json?

Yes - I'm actually just trying to use the meta descriptors that you have included by default.  In this case I'm trying to use the user to construct the github link:
Here is the meta json from the .imdone/config.json
"meta": {
"user": {
"urlTemplate": "https://github.com/%s",
"titleTemplate": "github profile for %s"
},
"issue": {
"urlTemplate": "https://github.com/imdone/imdone-atom/issues/%s",
"titleTemplate": "github issue #%s"
}
}
Here is my todo item:
#TODO:10 Fill out description upon new task creation user:10101

I can't seem to reproduce it.  Can you post your full config.js file?

Hmm... it could be an odd Windows-related bug.  I am using the default config.json file (haven't edited it)

{
"exclude": [
"^(node_modules|bower_components|\.imdone|target|build|dist|logs)[\/\\]?|\.(git|svn|hg|npmignore)|\~$|\.(jpg|png|gif|swp|ttf|otf)$"
],
"watcher": true,
"code": {
"include_lists": [
"TODO",
"DOING",
"DONE",
"PLANNING",
"FIXME",
"ARCHIVE",
"HACK",
"CHANGED",
"XXX",
"IDEA",
"NOTE",
"REVIEW"
]
},
"lists": [
{
"name": "CHANGED",
"hidden": true
},
{
"name": "BACKLOG",
"hidden": true
},
{
"name": "TODO",
"hidden": false
},
{
"name": "DOING",
"hidden": false
},
{
"name": "READY",
"hidden": true
},
{
"name": "DONE",
"hidden": true
}
],
"marked": {
"gfm": true,
"tables": true,
"breaks": false,
"pedantic": false,
"smartLists": true,
"langPrefix": "language-"
},
"meta": {
"user": {
"urlTemplate": "https://github.com/%s",
"titleTemplate": "github profile for %s"
},
"issue": {
"urlTemplate": "https://github.com/imdone/imdone-atom/issues/%s",
"titleTemplate": "github issue #%s"
}
}
}

Are you editing the correct .imdone/config.json?  The one you should edit is in [Your project folder] > .imdone > config.json

Oh, you're right.  I can't believe I missed that!  It works great now that I'm editing the right json file :)  Thanks again for your work on this package.


Donation account-Loader and Web view issues.

Loader is loding continously when we refresh the page for latest transaction.It doesn't stop even after killing the app and coming back.It is happening on android device.
Also when we go to web view it is not showing the Total funds raised and Total amount remaining.It is always showing 0 sats.
Tested on dev app v2.0.01(374)

Donation Web is now resolved.

Verified this issue on live app v2.0.2(376)


new version of this package.

hi, i have rewritten this package but without the cargo check function. i would appreciate it if you would point out any errors since you have experience with this, thank you! my repository is here. i understand that FichteFoll has made a rewrite for this, but i have no idea how to test plugins out on a previous version of sublime text.

I no longer use Sublime so I am not in a good position to point out errors. However, I'll leave this open for anyone looking for alternatives.


Release 0.1 ?

@astrofrog - How about releasing v0.1 today?
I'm sure that with announcing new users / bug / feature requests will show up, and having a release
/ conda-forge package would help e.g. with hips, I could switch over to healpix and also make a   release now.
Gammapy / reproject or other packages could then wait for a more complete and well-tested v0.2 release.

I'm happy to release today but I think we might want to rename first. I asked the coordination committee if there were any objections to using astropy-healpix and I didn't get any.

(the main issue we'll have is that conda install -c conda-forge healpix won't work because that is the C package!)

I'll be on hangouts all afternoon if we want to coordinate a release

Done!


Can you provide the  'model_best.model.pkl'?

'model_best.model.pkl'  is  not provided，you just provide ' FractureSegModel.model'，Can you provide the file of 'model_best.model.pkl'，thanks

The same request！'model_best.model.pkl'  is an indispensable file for running the test code. By the way, hope the training code will be provided soon!


oppo find x2 teams have no notification, no message

On my oppo find x2 phone, teams always have no notification. And now have no message update even I open teams and manually scroll down to let refresh
Teams downloaded from original oppo market ap

文档详细信息
⚠ 请勿编辑此部分。 docs.microsoft.com ➟ GitHub 问题链接必须具有此部分。

ID: 0c86f401-cff4-93b3-ff69-b9787a87e644
Version Independent ID: c0808bd3-cf63-0625-4c3c-65469c26fb3e
Content: 在中国获取 Microsoft Teams for Android - Microsoft Teams
Content Source: Teams/get-teams-android-in-china.md
Service: msteams
GitHub Login: @SerdarSoysal
Microsoft Alias: serdars

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

OK, thanks


feature/ add a check to open in a tab if in tabs mode otherwise open buffer

Relates and hopefully fixes #1675, I added a check to open with a regular edit command if a user is using buffers mode otherwise it uses the tabedit command which for all I know is probably not the desired effect, the tab drop command does nothing for me locally though I wasn't using tabs so not sure if it would have been working for tab users.
The main aim of this PR is to have the quick open open files depending on the users tabs.mode settings, happy to revert the tab drop change as it likely does exactly what was intended but seemed to do nothing locally so attempted a fix for that ^^

From my brief understanding of tab drop it seems to be working, so may be worth leaving in.
Ie :
:e LICENSE
:vsplit oni.sh
:split package.json
:tabe appveyor.yml

Calling :tabe package.json opens a new tab, calling :tab drop package.json swaps me back over to the "workspace" with my 3 splits in.

@CrossR thanks for the explanation had no idea what it did, happy to revert it since it works as it should

Reverted 👍 just checks tabs mode now

Cool, the fix looks good to me! Thanks @Akin909 and @CrossR  👍


Fixes Issue #683

Added the OrganizationID and the Organization to the Person so that there is an Organization associated with the TestUser.

thanks @TheTopher! once I see the build pass on appveyor I will merge.

@TheTopher does this fix in all cases? I saw the same error when I created a new (non-test) user and tried going to that page as well.

@cmgutmanis I wonder if there is code in creating a user (registering) that doesn't require an org to be picked.


Minor improvements

Q
A

Bug Fix?
yes

New Feature?
no

BC Breaks?
no

Deprecations?
no

Fixed Tickets

9.07 score on scrutinizer :dancer: nice :yum:


[FEATURE REQ] SqlDatabaseResource.ExportAsync() / DatabaseExportDefinition to allow Managed Identity

Library name
Azure.ResourceManager.Sql
Please describe the feature.
Currently, the DatabaseExportDefinition class constructor will only accept an Azure Storage access key and SQL Server username and password.
Two of the recommendations from Microsoft Defender for Cloud are: "Storage accounts should prevent shared key access" and "Azure SQL Database should have Azure Active Directory Only Authentication enabled". Both of these are considered "critical" recommendations.
We have a WebJob that triggers a database export each night, which I don't think is an un-common scenario. With the export functionality as it is, we could not possibly achieve the two recommendations of Microsoft Defender for Cloud. This seems especially important on the destination storage account, because it contains customer data in .bacpac files not protected in any other way.

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Hi @johnburgato , .NET SDK is auto-generated from service swagger, if you need service to support the Export operation with ManagedIdentity, you could open an Azure support request to get further help.


Design files

add brand and design files
Things added

Hack font
webpack.svg
sketch files

Simple Branding
Terminal and Editor states
Website

Hey @zxhadow! Thanks for your hard work! I love the design in Sketch for the dark editor and the CLI. However, I have some remarks about the usage of this plugin.

The user install the plugin locally doing:
shell
npm install config-webpack-plugin --save-dev

The user configure the plugin in his/her webpack.config.js telling to the plugin which config file to intercept during webpack's compilation time.
webpack.config.js
const ConfigPlugin = require('webpack-config-plugin');

module.exports = {
    // Among other stuff...
    plugins: [ new ConfigPlugin('./config.js') ]
}

The user trigger webpack's compilation running: webpack command. During that compilation time, the plugin will read the specified configuration file and will merge matching properties with current environment variables and will ‘write the changes to disk’.

That said, I think we need to help the audience to understand what the plugin does in a simple way that is: intercept a config file, replace config properties with matching variable and, write the modified config file disk.

TL;DR.

I ❤ the banner style for the branding. The logo of webpack along with the blank hexagon makes a good job transmitting the message of a plugin.
The ‘process-env‘ from Windows Pages are not accurate. The user don't need to type process.env (that's just a property from inside Node.js). In the ‘how it works’ image that I prepared I was trying to illustrate what are the current environment variables that are visible to NodeJS typing process.env inside a NodeJS session. Maybe, I failed in transmitting that idea. Apart, from the installation, the only required command to kickoff the plugin is: webpack which can be ignored because that's how all webpack plugin runs.
From my previous point. We need to show the user how a config file looks with props, some current environment variables and the result of the tiny work of the plugin.

Please, don't hesitate to ask anything about this entire comment (or blog post) 😅

Thanks @rmariuzzo for this well explained comment. See these states for clarification if I got it right this time.
These are not the explanation itself, but we will use them to create a little gif or another better idea.

Ok, I like the idea for the GIF and the graphics.

I will be very busy this week, but we can close this PR and create another one with the assets so we could upload the files in case someone wants to help with them.

@zxhadow I will create the gh-pages branch which will hold the landing page and the sources files. That way we don't mix them with the library.


Train and test WaveGAN

Modify WaveGAN model to train on emotion dataset and test model

[ ] Test model
[ ] Train various model


Get-ServiceNowIncident does not appear to include the CI name

Is this project dead?  Have not seen any activity since 9/24/19.
It would appear that the Get-ServiceNowIncident does not appear to include the CI name that may or may not be defined on the INC.
Any chance we can get that?

Found it in another place

The two of us that were putting the most work in both got hit with other things in life.  So kinda dead, yeah, but not completely ignored.
Nothing should be hidden from a Get command if you dig under the formatting.  There can be some confusion where field names are returned with a database name so they're a tad harder to spot, but the module should be aligned with the app.  So like a Set will have a friendly name just like the UI does, but a Get will return what the field is called in their database because that's what the API delivers.


mv does not have a -r option

I do not believe the mv command has a -r option. It is recursive by default if you move a *
https://github.com/milesburton/teslacam/blob/aab37fb9b133a21da5e2f0bf394ef8f61e9a6ba0/src/dashcam-monitor.js#L87
Error:
mv: invalid option -- 'r'
Try 'mv --help' for more information.

pi@raspberrypi:~/teslacam/video $ mv --help
Usage: mv [OPTION]... [-T] SOURCE DEST
  or:  mv [OPTION]... SOURCE... DIRECTORY
  or:  mv [OPTION]... -t DIRECTORY SOURCE...
Rename SOURCE to DEST, or move SOURCE(s) to DIRECTORY.

Mandatory arguments to long options are mandatory for short options too.
      --backup[=CONTROL]       make a backup of each existing destination file
  -b                           like --backup but does not accept an argument
  -f, --force                  do not prompt before overwriting
  -i, --interactive            prompt before overwrite
  -n, --no-clobber             do not overwrite an existing file
If you specify more than one of -i, -f, -n, only the final one takes effect.
      --strip-trailing-slashes  remove any trailing slashes from each SOURCE
                                 argument
  -S, --suffix=SUFFIX          override the usual backup suffix
  -t, --target-directory=DIRECTORY  move all SOURCE arguments into DIRECTORY
  -T, --no-target-directory    treat DEST as a normal file
  -u, --update                 move only when the SOURCE file is newer
                                 than the destination file or when the
                                 destination file is missing
  -v, --verbose                explain what is being done
  -Z, --context                set SELinux security context of destination
                                 file to default type
      --help     display this help and exit
      --version  output version information and exit

The backup suffix is '~', unless set with --suffix or SIMPLE_BACKUP_SUFFIX.
The version control method may be selected via the --backup option or through
the VERSION_CONTROL environment variable.  Here are the values:

  none, off       never make backups (even if --backup is given)
  numbered, t     make numbered backups
  existing, nil   numbered if numbered backups exist, simple otherwise
  simple, never   always make simple backups

GNU coreutils online help: <http://www.gnu.org/software/coreutils/>
Full documentation at: <http://www.gnu.org/software/coreutils/mv>
or available locally via: info '(coreutils) mv invocation'

System info:
pi@raspberrypi:~/teslacam/video $ uname -a
Linux raspberrypi 4.14.98+ #1200 Tue Feb 12 20:11:02 GMT 2019 armv6l GNU/Linux

Agreed I was experimenting following a bug, this must have crept in
On Tue, 18 Jun 2019 at 04:25, Jim Karg<EMAIL_ADDRESS>wrote:

I do not believe the mv command has a -r option. It is recursive by
default if you move a *
https://github.com/milesburton/teslacam/blob/aab37fb9b133a21da5e2f0bf394ef8f61e9a6ba0/src/dashcam-monitor.js#L87
Error:
mv: invalid option -- 'r'
Try 'mv --help' for more information.
pi@raspberrypi:~/teslacam/video $ mv --help
Usage: mv [OPTION]... [-T] SOURCE DEST
or:  mv [OPTION]... SOURCE... DIRECTORY
or:  mv [OPTION]... -t DIRECTORY SOURCE...
Rename SOURCE to DEST, or move SOURCE(s) to DIRECTORY.
Mandatory arguments to long options are mandatory for short options too.
--backup[=CONTROL]       make a backup of each existing destination file
-b                           like --backup but does not accept an argument
-f, --force                  do not prompt before overwriting
-i, --interactive            prompt before overwrite
-n, --no-clobber             do not overwrite an existing file
If you specify more than one of -i, -f, -n, only the final one takes effect.
--strip-trailing-slashes  remove any trailing slashes from each SOURCE
argument
-S, --suffix=SUFFIX          override the usual backup suffix
-t, --target-directory=DIRECTORY  move all SOURCE arguments into DIRECTORY
-T, --no-target-directory    treat DEST as a normal file
-u, --update                 move only when the SOURCE file is newer
than the destination file or when the
destination file is missing
-v, --verbose                explain what is being done
-Z, --context                set SELinux security context of destination
file to default type
--help     display this help and exit
--version  output version information and exit
The backup suffix is '~', unless set with --suffix or SIMPLE_BACKUP_SUFFIX.
The version control method may be selected via the --backup option or through
the VERSION_CONTROL environment variable.  Here are the values:
none, off       never make backups (even if --backup is given)
numbered, t     make numbered backups
existing, nil   numbered if numbered backups exist, simple otherwise
simple, never   always make simple backups
GNU coreutils online help: http://www.gnu.org/software/coreutils/
Full documentation at: http://www.gnu.org/software/coreutils/mv
or available locally via: info '(coreutils) mv invocation'
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/milesburton/teslacam/issues/8?email_source=notifications&email_token=AAJGHM7YHYE5LYYAILV73EDP3BITXA5CNFSM4HY36DP2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G2A6LFQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJGHM4KCBYE4M2JUPM3R53P3BITXANCNFSM4HY36DPQ
.


Roll Plugins from 7d1923f4528e to 3647ba11cbee (1 revision)

https://github.com/flutter/plugins/compare/7d1923f4528e...3647ba11cbee
2022-04-04<EMAIL_ADDRESS>[camera] Migrate deprecated Scaffold methods to ScaffoldMessenger (flutter/plugins#5151)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


fix: remove refMap and fix infinite looping case

Codecov Report

Merging #492 (6c0c5b8) into master (6a69f44) will decrease coverage by 0.01%.
 Consider uploading reports for the commit aebeed8 to get more accurate results

@@            Coverage Diff             @@
##           master     #492      +/-   ##
==========================================
- Coverage   98.95%   98.94%   -0.02%
==========================================
  Files           2        2
  Lines          96       95       -1
  Branches       27       27
==========================================
- Hits           95       94       -1
  Misses          1        1

Impacted Files
Coverage Δ

src/dereferencer.ts
98.90% <100.00%> (-0.02%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

:tada: This PR is included in version 1.5.7 :tada:
The release is available on:

GitHub release
1.5.7
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


adding cron job support for daily update of usage data

adding cron job for supporting daily refresh of student_dashboard database tables, which contains information of course, user, file, file usage, assignment, and grade, etc.

I'd do this as a new management command (dashboard/management/commands) similar to the createuser example there. Maybe something like populatedata ?
Then change the cron.py to be called directly be this and remove all of the URL's.
Then the simple way would just let django-crontab schedule this on the individual nodes like
https://pypi.org/project/django-crontab/
And later set it up to run on something like Jenkins, especially if we need to scale out horizontally.
Was this something you were working on or had ideas about @zqian ? I could work on this later this week otherwise.

compared to django-crontab with last release version in 2016, I think the django-cron(https://django-cron.readthedocs.io/en/latest/) is more up-to-date


Images similar to the one already inserted

Hello,
I was wondering if there is a repository of svg images in the style you already put in the illustration folder.
It would be amazing since of course those two images don't fit every use case.
Thanks a lot for the amazing work

yes,I replaced both of them myself
demo

@EdoardoVimercati @y377 Thanks, I'm glad you like the theme!
@EdoardoVimercati from memory the illustrations I used in this theme come from https://icons8.com/illustrations - Amazing stuff.
@y377 If you like my work and would like to support me then check out my premium themes https:/www.zerostatic.io - Unfortunately I'm not currently available to do paid customisation.


New NPM packages

Find a useful NPM package that we have not covered in class.

Found passport, fullcalendar and cron


featureImportances not serialized while serializing spark tree based models. Any resolution?

@ancasarb ????

do we need to write the serializer by ourselves or you guys are providing?


Turn off Anchor Free in Training

@FateScript
I have a quick question. Is there an option to turn off the anchor-free?
I looked at the prediction on my document (image) data. Usually, objects are well separated but paired. It shows some patterns for the poor performance

used to miss the object inside the big object
used to miss empty objects (no content inside the bounding box)

I am wondering if the anchor-free approach might not help in my case.
Thanks.

YOLOX is anchor free model, so it's impossisble to use anchor-based model(unless you write a new model in get_model).


Batch update functionality using gorm.DB.Save() is not working

GORM Playground Link

https://github.com/go-gorm/playground/pull/1
Description

There is an issue with batch update functionality using gorm.DB.Save(). It occurs when we try to insert into SQL server DB. As you can see in the image below, passing an array of objects to Save() function will be translated to SQL query as MERGE statement.  The error message clearly describes the root case "the table name is missing schema". Save() function works correctly using an object instead of an array.

Please submit a correct Plaground PR as your linked PR is not correct.


Sequence lengths other than powers of 2?

In your demo, you write
# sequence lengths must be a power of 2 of lengths between 32 and 65536
# hit me up if you need different lengths!

What should we do if we have a sequence length that's not a power of 2?

Would padding at the end of the sequence to the nearest power of two work for you? If you don't have any gradient on later entries, the padding doesn't affect the learning.


deserialization issue following moving to new requests and responses

Search is running into issues deseiralizing following our pr moving over to new requests and responses. The issue isn't the new requests and responses, the issue is that we are now only deserializing with pipeline_response.http_response instead of pipeline_response. Roll back part of this change, because we don't want to modify core behavior. This is in search

cc @xiangyan99

I infer search SDK is generated with cmd autorest D:\dev\azure-rest-api-specs\specification\search\data-plane\Azure.Search\readme.md   --python --python-sdks-folder=D:\dev\azure-sdk-for-python\sdk --use=<EMAIL_ADDRESS>--use=<EMAIL_ADDRESS>--version=3.10.2 --version-tolerant=False --tag=package-2021-04-searchindex-preview

We have test cover this scenario https://github.com/Azure/autorest.python/blob/029547c6f97bdb153cecc495a14ff3aaa233a188/packages/autorest.python/test/vanilla/legacy/Expected/AcceptanceTests/BodyInteger/bodyinteger/operations/_int_operations_operations.py#L401 and I don't reproduce the deserialization issue. Could you provide more details about the issue? @iscai-msft @xiangyan99


Add clustering check method

We have image pairs or triplets that represent replicates of each other. One way to measure the performance of our feature space is to check that replicate images appear closer in feature space than random pairs of images. In effect, we have a partial ground truth on the image clustering.

Closed by #12


Adds a device and user count the the all devices page.

Adds an indicator for the number of devices (online vs total) and the number of users on the 'All Devices' page.
Before:

After:

What do you think?

Sounds good to me 👍🏼 🚀


Update Composer dependencies (2023-11-21-00-15)

Visual regression test passed!
View the visual regression test report


not seeing anything

Hi!
Your tool seems pretty cool, but does not work on my side.
I converted it to swift like this:
   if let myAwesomeVideo = TSHAlphaVideoController.video(withRGBVideoFile: "Alpha-TestVideo",
                                                       with:self)
    {
        self.view.addSubview(myAwesomeVideo.view)
        myAwesomeVideo.play()
    }

I tried to play an mp4 with alpha channel, but I dont see anything (yes, my background is blue).
Does the mp4 have to be prepared somehow?
Thanks for your support.
Regards,
Werner

Hey @pawelnathan yes, there is a very specific way you have to encode the video. The instructions are in the readme under Video Processing
If it's not working for you, feel free to attach your video here and I'll take a look.


Use current locale to determine decimal separator

Hi Ian,
we like your approach of implementing a custom number pad for SwiftUI based watchOS apps :)
Since our watch app is mainly used in countries with the , as the decimal separator we would like to introduce the following code changes to your implementation that basically determine the decimal separator based on the specified Locale.

Thanks in advance

@patrickniepel is your app publicly available on the App Store in the United States? Id love to see it!


HELP: How to remove default flags "--quiet, --version, --ansi, --no-ansi, --no-interaction, --verbose" from commands?

How to remove the default "--quiet, --version, --ansi, --no-ansi, --no-interaction, --verbose" flags from a custom command that inherites from "from cleo.commands.command import Command", like the GreetCommand showed in the demo on README?
I created a command and don't want to show these default flags to users.

This is currently not available.

Could you show me where in the code these flags are created? So I could add this functionality of suppressing the flags.

Could you show me where in the code these flags are created? So I could add this functionality of suppressing the flags.

https://github.com/python-poetry/cleo/blob/675ec7e1d89aa65ce6dbca0938a89b4788db1045/src/cleo/application.py#L549


Bedrock

Do you have plan to support model in bedrock?
I think that would open up even more possibilities.
Thanks :)

Bedrock will be supported once my pull request is approved.


WinRM timeout in test-kitchen on Windows

I'm currently using a homegrown cookbook to install Nodejs for windows. I can then use a community LWRP to install npm packages on Windows.  To cover my bases, I've linked my issue on the community cookbook as well here: https://github.com/redguide/nodejs/issues/148
Using test-kitchen kitchen converge -l debug, when using resource nodejs_npm 'gulp' I find the debug output:
       [2016-12-09T07:24:40+00:00] DEBUG: Providers for generic execute resource enabled on node include: [Chef::Provider::Execute]
       [2016-12-09T07:24:40+00:00] DEBUG: Provider for action run on resource execute[install NPM package gulp] is Chef::Provider::Execute

             [execute] npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
                npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
                npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use 'npm ls gr
aceful-fs' to find it in the tree.
                C:\Users\azure\AppData\Roaming\npm\gulp -> C:\Users\azure\AppData\Roaming\npm\node_modules\gulp\bin\gulp.js
                C:\Users\azure\AppData\Roaming\npm
                `--<EMAIL_ADDRESS>
D      [WinRM] retrying receive request after timeout

If I let it wait, I'll just get the last line [WinRM] error forever.
When logging in locally and running the same command npm install -g gulp I get the below output:
PS C:\Users\azure> npm install -g gulp
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update t
o<EMAIL_ADDRESS>as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.
C:\Users\azure\AppData\Roaming\npm\gulp -> C:\Users\azure\AppData\Roaming\npm\node_modules\gulp\bin\gulp.js
C:\Users\azure\AppData\Roaming\npm
`--<EMAIL_ADDRESS>
PS C:\Users\azure>

Outputs are the same (and the command runs 100% successfully), it just seems that WinRM doesn't release the shell back to test-kitchen after npm is done installing. Seen this issue before? It may be nodejs specific, but I'm curious if this is an issue with WinRM not letting test-kitchen know that the resource is done.
For good merit, here's my .kitchen.yml file:
---
driver:
  name: azurerm

driver_config:
  subscription_id: "123-guid-goes-here-456"
  location: "East US"
  machine_size: "Standard_DS1"

provisioner:
  name: chef_zero

platforms:
  - name: windows2012-r2
    driver_config:
      image_urn: MicrosoftWindowsServer:WindowsServer:2012-R2-Datacenter:latest
    transport:
      name: winrm
      elevated: true

suites:
  - name: default
    run_list:
      - recipe[nap-nodejs::default]
    attributes:

After digging through, this is an issue with upstream code it appears, not with the driver. Closing the ticket. Apologies for opening an unneeded issue :)


Adding UML, ER, & Activity Diagrams to the Wiki

When the diagram issues are closed, we then need to add the relevant information to the wiki.

Activity diagram done, needs to be added to wiki later.

Activity diagram done, needs to be added to wiki later.


Add plugins directory path to tailwind config.

Remix with Vite now comes with Tailwind installed by default. However, only the app directory path is included. So to save newbies some time, it is better if it is specified to add the plugins directory path to the tailwind config file.

Thank you! Good idea.


Use static members on Stylesheet and Values

Big API change but works really nicely for:

autocomplete
namespacing (floatLeft -> .left)

Could take this further by defining static colors on Color...but waiting till we figure out the Auto/Inherit/None sitch.

(We also may wanna rename Stylesheet back to Css for nicer qualified naming again.)


pm: simplify state management

Summary:

Remove usage of local states, the device PM subsystem already holds device PM state
The "ctrl_command" is removed, since the PM control callback will now always be called to set the state, not to retrieve it.
The state is passed as a value, since it does not need to be written by the PM control callback
The callback is only called for a state change, thus simplifying driver operation (no need for state change checks)

Commits summary:
pm: device: remove usage of local states

The device PM subsystem already holds the device state, so there is no
need to keep duplicates inside the device. The pm_device_state_get has
been refactored to just return the device state. Note that this is still
not safe, but the same applied to the previous implementation. This
problem will be addressed later.

pm: device: remove usage of ctrl_command

The callback is now invoked to set the device PM state in all cases, so
the usage of ctrl_command is redundant.

pm: device: remove ctrl_command callback argument

The ctrl_command is not used anymore, so remove it from the callback
signature.

pm: device: remove pointer usage for state

Since the state is no longer modified by the device PM callback, just
use the state value.

pm: device: improve docs and arg names for state set

Improve the docstrings of the pm_device_state_set function and change to
shorter argument variable names.

doc: pm: update documentation

Update documentation so that latest changes are reflected.

pm: device: do not call device if already on the given state

If the device is already at the given state, do not call the device PM
control function. This makes sure that devices are only called to change
from one state to another.

Even though asynchronous device PM is completely broken, transitional
states are considered too.

pm: simplify state change check logic

The device PM control function will only be called if the requested
state is different from the current one. A significant amount of drivers
were checking for state changes, now unnecessary. This patch removes all
this redundant logic.

This PR is part of a series of device PM improvements, see #36125 for more details.

Before doing any more changes, I'd like to have a clear roadmap on where we want to go, otherwise every PR will take ages to merge and as a result we'll keep having a device PM subsys that is far from solving the PM problem. I think that if the goal is clear, and the current PM subsys status is well understood/recognized, things will be easier. I have the feeling that device PM in particular is sold as ready when it is clearly not, and, as far as I know, I'm not the first person to mention that.
Let's take a couple of examples to illustrate my argument:

The ctrl_command argument was, in theory, added to support arbitrary commands. The reality, however, is that it has only been used to set/get the device state (with a command name PM_DEVICE_STATE_GET/SET, same prefix as actual device states). I think everybody agrees that keeping local states is, in general, a bad idea. Removing the "get", we're left with a single command to set the state. So in reality, the callback is de-facto being used with an action, that is, we combine a "set" command with a "state" argument. An example to make it clear: "set + active" == "resume". The void *context was replaced in the last release in favor of a uint32_t *state and now with enum pm_device_state *state, meaning a generic command can hardly be implemented now. So the reason to keep ctrl_command doesn't hold anymore. What this PR does in the end is to continue the same design pattern (use states as actions), just removing a redundancy (the ctrl_command), so I don't see what is wrong with this approach as an intermediate step to a proper solution as suggested in #36125.
The runtime API is another example of something that has existed, but nobody (except a single driver) uses it. It lacks proper documentation and usage guidelines, it doesn't handle device dependencies, etc. Let's talk about a problem I have already mentioned with this API. It basically puts devices into ACTIVE or SUSPEND states. But what about LOW_POWER? Let's see: according to the specs, LOW_POWER should be used when context is not lost. In many devices this holds for low power states, so a driver for such device will pick LOW_POWER. But then, how can runtime PM be implemented? What happened in reality: most drivers handle all the "low power states" the same way, which is wrong and leads to inconsistent behavior (it's incompatible to implement low power/suspend/off the same way). This is a common pattern in the PM subsystem: there are many APIs that have been created without really analyzing if they can solve a real problem.

My feeling is that the PM subsystem was designed in a rush, with a quick implementation that likely solved immediate problems, but it is now showing its limitations and requires some fundamental changes. We're not talking about some fixes, API extensions, etc. but a refactor/re-write in some areas. This is difficult, painful and likely means a period of some instability, but it is sometimes necessary to make progress. And let's be honest: we'll not either get it 100% right this time.

The cleanup is fine, that is definitely not the final version of this control API, but lets move forward.

@ceolin @nashif I've added all the changes up to initial support of actions.


Tries to create folder in X:\

The primary HDD is C:, although my company has X:\ as well. Nanobox tries to create the folder at X:.nanobox. Is there any way to change that?

It seems to use https://github.com/mitchellh/go-homedir/blob/master/homedir.go to determine home directory. Which seems to use environment variables. You should be able to set a HOMEDRIVE and HOMEPATH environment variable to change it to what ever you want.
See https://github.com/mitchellh/go-homedir/blob/master/homedir.go#L121

Thank you, but I can't set that variable. I see if I can contact an IT Admin.

Thanks for finding this. The help is greatly appreciated!
Nanobox has been completely re-written from the ground up, and we just released the v1 into beta this week. Nanobox no longer depends on Vagrant, and so this is no longer an issue. We would love to get your feedback on the new version. The new guides are still a WIP but should be usable within the next week. If you're interested, we'd love to talk with you on irc #nanobox or we can bring you into our slack channel as well.
Additionally, Nanobox and Nanobox Cloud have been merged into a unified process. If you would be interested in early access to the private beta please let me know and I can get you an access token.
Just reach out to<EMAIL_ADDRESS>and we can get you going.
I really appreciate your help on this. I'll go ahead and close this ticket and hope to hear from you directly.


Some questions about the data part

Great work, I have some questions below:
class RetroMAECollator(DataCollatorForWholeWordMask):
max_seq_length: int = 512
encoder_mlm_probability: float = 0.15
decoder_mlm_probability: float = 0.15
def __call__(self, examples):
    input_ids_batch = []
    attention_mask_batch = []
    encoder_mlm_mask_batch = []
    decoder_labels_batch = []
    decoder_matrix_attention_mask_batch = []

    tgt_len = self.max_seq_length - self.tokenizer.num_special_tokens_to_add(False)

    for e in examples:
        e_trunc = self.tokenizer.build_inputs_with_special_tokens(e['token_ids'][:tgt_len])
        tokens = [self.tokenizer._convert_id_to_token(tid) for tid in e_trunc]

        self.mlm_probability = self.encoder_mlm_probability
        text_encoder_mlm_mask = self._whole_word_mask(tokens)

        self.mlm_probability = self.decoder_mlm_probability
        mask_set = []
        for _ in range(min(len(tokens), 256)):
            mask_set.append(self._whole_word_mask(tokens))

First, what does 256 mean here, why is it set to 256 ?
Second, e['token_ids'] has several blocks according to process.py.   It seems that e['token_ids'] is  list[list[int]], so (e['token_ids'][:tgt_len] seems not right?

Thanks for your interest in RetroMAE!
For the enhanced decoder, the collater needs to prepare a mask list for each token. If the number of tokens is large, the data processing may be time-consuming. Therefore, I set a mask set with a limited max size of 256 (maybe another hyper-parameter is better),  and then mask list will be sampled from this set.
Blocks returned by pad_each_line function will be assigned to multiple rows ({'token_ids': list[list[int]]} will be [{'token_ids':list[int]}, ..]), so the e['token_ids'] is list[int].


CI Improvements

WHY are these changes introduced?
Our CI runs take a lot of time, this is an effort to improve that.
This PR reduces CI times from 12-16min to 6-9min.
The bottleneck is in acceptance tests (in mac os), they are slow and do real network calls, if we want to further improve CI times we'd need to discuss what we want to do with them if the plan is to add more (maybe not run them in every PR?)

WHAT is this pull request doing?
Revamp the github action workflows to parallelize as much as possible and remove unnecessary steps:

Don't make a workflow depend on any other -> run all at the same time.
Only install ruby if specifically needed for the current workflow -> only for acceptance tests.
Run acceptance tests in a new workflow -> this is what takes most of the time, no need to wait for unit tests.
Only fetch the last commit when cloning the repo -> we were fetch EVERYTHING.
Stop using nx affected -> it didn't improve the CI times and forces us to fetch all commits of the repo.
Update some external github actions.
Remove node 16 from the CI matrix.
Disable some GitDiff tests that were failing on windows.
Use the built-in cache of action/setup-node instead of our own solution

How to test your changes?

Post-release steps

Measuring impact
How do we know this change was effective? Please choose one:

[ ] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[ ] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[ ] I've considered possible documentation changes
[ ] I've made sure that any changes to dev or deploy have been reflected in the internal flowchart.

Cool!
If the problem is the acceptance test, I would move them to be executed during the nightly generation. In case of receiving an alert we only need to put the focus on the PRs merged the last day that should not be a big amount


dejaVu share URL generated by gh-pages app is incorrect

The URL generated is of the form
http://appbaseio.github.io#?input_state=...
It should be of the form
http://appbaseio.github.io/dejaVu/live#?input_state=...

Fixed in v0.7.0.


OpenAPI schema validation

 [INFO] --- openapi-generator-maven-plugin:7.0.1:generate (default) @ horreum-api ---
Warning:  There were issues with the specification, but validation has been explicitly disabled.
 I'll close #804

A lot of these issues are caused by the internal API.  I have not cleaned that yet, and is not the API spec that we publish

The options for resolving the operationId is repeated is to rename the methods so that they are unique, or to use a different operation generation strategy, but that would result in an openapi spec that looks like this;

instead of the current;

The client api calls would also change from dataset. listBySchema() to dataset.DatasetService_listBySchema()

The only way to resolve these messages is to fully document the internal API.  There are 2x as many internal endpoints as there are external.
We agreed to split api into internal/external so that we just had to document the external tools api.
I think this is a nice to have, but not essential for the 0.10 release


Add widget menu not closable

Dev Home version
0.503.261.0
Windows build number
10.0.22621.2428
Other software
none
Steps to reproduce the bug
go on add widget button
try to click close
Expected result
closes
Actual result
doesn't close
Included System Information
No response
Included Extensions Information
no extensions

If you press the 'Escape' key will it close?


Add KeepAcceptEncoding option, prevents proxy from changing Accept-Encoding sent by clients

Hi,
This PR adds a new flag to the proxy, KeepAcceptEncoding. When disabled, proxy behaviour remains the same to avoid breaking existing clients.
The default behaviour is that the Accept-Encoding header on client requests is removed. The proxy outbound transport will then add an "Accept-Encoding: gzip" header to outbound requests.
With KeepAcceptEncoding enabled the proxy will pass the header through. Note that if you want the proxy to never modify Accept-Encoding, you should also set DisableCompression true on the proxy outbound transport (otherwise it will ask for gzip when the client does not send the header).
Fixes https://github.com/elazarl/goproxy/issues/102 for me.

I now wonder why I added accept encoding, and not just send client's accept encoding.
In retrospect it looks like a mistake.
What does squid do?

This feature has been integrated in https://github.com/elazarl/goproxy/pull/585. Closing this PR.


User.Identity.Name is null after login

User.Identity.Name is null after login
Using .NET Core 3.0 rc1. I created a MVC project using VS 2019 Preview MVC template with Individual Authentication. I created a user using Register form, and Logged in. But User.Identity.Name returns null in my controller. I've tried httpContextAccessor, but it is still null.
Edit: I'm using JWT.

This issue was moved to aspnet/AspNetCore#14160


Auth

Added Authentication for a simple password/email. Then authorization around 3 newly created roles. Admin (CRUD), Contributor (CR), and Viewer (R).

Good to go


Changing the mpm module results in a fatal configuration error

Changing the mpm module to be used for a httpd_service instance (at least on Ubuntu 14.04) results in 2 enabled mpm module load files and this fatal error while running sudo chef-client:
      STDOUT: * Starting web server apache2
       *
       * The apache2-example configtest failed.
      STDERR: Output of config test was:
      AH00534: apache2: Configuration error: More than one MPM loaded.
      Action 'configtest' failed.

Deleting one of the mpm module load files in /etc/apache2-example/mods-enabled/ and executing sudo chef-client again solves the problem.
I guess Chef should delete the obsolete mpm module load file if you change the mpm module in your Chef cookbook/recipe, right?
More detailed output:
  * httpd_service_debian_sysvinit[example] action start
    * template[/etc/init.d/apache2-example] action create (up to date)
    * service[apache2-example] action start

      ================================================================================
      Error executing action `start` on resource 'service[apache2-example]'
      ================================================================================

      Mixlib::ShellOut::ShellCommandFailed
      ------------------------------------
      Expected process to exit with [0], but received '1'
      ---- Begin output of /etc/init.d/apache2-example start ----
      STDOUT: * Starting web server apache2
       *
       * The apache2-example configtest failed.
      STDERR: Output of config test was:
      AH00534: apache2: Configuration error: More than one MPM loaded.
      Action 'configtest' failed.
      The Apache error log may have more information.
      ---- End output of /etc/init.d/apache2-example start ----
      Ran /etc/init.d/apache2-example start returned 1

      Cookbook Trace:
      ---------------
      /var/chef/cache/cookbooks/compat_resource/files/lib/chef_compat/copied_from_chef/chef/provider.rb:119:in `compile_and_converge_action'

      Resource Declaration:
      ---------------------
      # In /var/chef/cache/cookbooks/httpd/libraries/httpd_service_debian_sysvinit.rb

       19:       service "#{apache_name}" do
       20:         supports restart: true, reload: true, status: true
       21:         provider Chef::Provider::Service::Init::Debian
       22:         action [:start, :enable]
       23:       end
       24:     end

      Compiled Resource:
      ------------------
      # Declared in /var/chef/cache/cookbooks/httpd/libraries/httpd_service_debian_sysvinit.rb:19:in `block in <class:HttpdServiceDebianSysvinit>'

      service("apache2-example") do
        provider Chef::Provider::Service::Debian
        action [:start, :enable]
        supports {:restart=>true, :reload=>true, :status=>true}
        retries 0
        retry_delay 2
        default_guard_interpreter :default
        service_name "apache2-example"
        pattern "apache2-example"
        declared_type :service
        cookbook_name "solid_webserver"
      end

    ================================================================================
    Error executing action `start` on resource 'httpd_service_debian_sysvinit[example]'
    ================================================================================

    Mixlib::ShellOut::ShellCommandFailed
    ------------------------------------
    service[apache2-example] (/var/chef/cache/cookbooks/httpd/libraries/httpd_service_debian_sysvinit.rb line 19) had an error: Mixlib::ShellOut::ShellCommandFailed: Expected process to exit with [0], but received '1'
    ---- Begin output of /etc/init.d/apache2-example start ----
    STDOUT: * Starting web server apache2
     *
     * The apache2-example configtest failed.
    STDERR: Output of config test was:
    AH00534: apache2: Configuration error: More than one MPM loaded.
    Action 'configtest' failed.
    The Apache error log may have more information.
    ---- End output of /etc/init.d/apache2-example start ----
    Ran /etc/init.d/apache2-example start returned 1

    Cookbook Trace:
    ---------------
    /var/chef/cache/cookbooks/compat_resource/files/lib/chef_compat/copied_from_chef/chef/provider.rb:119:in `compile_and_converge_action'

    Resource Declaration:
    ---------------------
    # In /var/chef/cache/cookbooks/solid_webserver/recipes/httpd.rb

      2: httpd_service 'example' do
      3:   mpm 'prefork'
      4:   action [:create, :start]
      5: end
      6:

    Compiled Resource:
    ------------------
    # Declared in /var/chef/cache/cookbooks/solid_webserver/recipes/httpd.rb:2:in `from_file'

    httpd_service_debian_sysvinit("example") do
      action [:create, :start]
      updated true
      retries 0
      retry_delay 2
      default_guard_interpreter :default
      declared_type :httpd_service
      cookbook_name "solid_webserver"
      recipe_name "httpd"
      mpm "prefork"
      version "2.4"
      package_name "apache2"
      instance "example"
      run_user "www-data"
      run_group "www-data"
      servername "localhost"
      maxclients "0"
      maxconnectionsperchild "0"
      maxrequestworkers "150"
      maxspareservers "10"
      minspareservers "5"
      startservers "5"
      modules ["authz_core", "authz_host", "authn_core", "auth_basic", "access_compat", "authn_file", "authz_user", "alias", "dir", "autoindex", "env", "mime", "negotiation", "setenvif", "filter", "deflate", "status"]
      timeout "400"
      log_level "warn"
      keepalive true
      maxkeepaliverequests "100"
      keepalivetimeout "5"
      hostname_lookups "off"
      listen_addresses ["<IP_ADDRESS>"]
      listen_ports ["80"]
    end

I'm seeing this too.  Indeed, changing the mpm module should clean up mods enabled symlinks, not just conf enabled symlinks.


[3.2] Android: Keyboard modifier and arrow key support for text edit

Based on https://github.com/godotengine/godot/pull/40398 for 3.2

@m4gr3d not sure if Im supposed to tag you here 😅

Thanks!


Write Terraform CLI logs to container stdout

Description of your changes
Added optional field enableTerraformCLILogging to the Workspace resource to enable writing of Terraform CLI logs to the container stdout in JSON format to assist with debugging and to view detailed information about Terraform operations.

Fixes https://github.com/upbound/provider-terraform/issues/163
I have:

[x] Run make reviewable to ensure this PR is ready for review.
[x] Run make e2e with UPTEST_EXAMPLE_LIST="examples/workspace-enable-logging.yaml"

How has this code been tested

On the local cluster, applied the Workspace resource present in the examples workspace-enable-logging.yaml
Observed that the container logs has the expected Terraform Plan and Apply logs
Made changes to the Workspace YAML and re-applied
Observed that the container logs are updated with new Terraform Plan and Apply logs
Changed enableTerraformCLILogging to false, made few other Terraform code changes in the Workspace YAML and re-applied
Observed that the container logs doesn't have the new Terraform Plan and Apply logs
Toggled enableTerraformCLILogging to true and re-applied the Workspace
Deleted the Workspace resource, Observed that the container logs has the expected Terraform Destroy logs.

Sample Logs:
{"level":"info","ts":"2024-04-10T20:24:04.678Z","logger":"provider-terraform","msg":"\nTerraform used the selected providers to generate the following execution\nplan. Resource actions are indicated with the following symbols:\n  + create\n\nTerraform will perform the following actions:\n\n  # random_id.example_id will be created\n  + resource \"random_id\" \"example_id\" {\n      + b64_std     = (known after apply)\n      + b64_url     = (known after apply)\n      + byte_length = 7\n      + dec         = (known after apply)\n      + hex         = (known after apply)\n      + id          = (known after apply)\n    }\n\n  # random_password.password will be created\n  + resource \"random_password\" \"password\" {\n      + bcrypt_hash = (sensitive value)\n      + id          = (known after apply)\n      + length      = 16\n      + lower       = true\n      + min_lower   = 0\n      + min_numeric = 0\n      + min_special = 0\n      + min_upper   = 0\n      + number      = true\n      + numeric     = true\n      + result      = (sensitive value)\n      + special     = true\n      + upper       = true\n    }\n\nPlan: 2 to add, 0 to change, 0 to destroy.\n\nChanges to Outputs:\n  + random_id_hex   = (known after apply)\n  + random_password = (sensitive value)\n","request":"example-random-generator"}

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 4 committers have signed the CLA.:white_check_mark: katronquillo:white_check_mark: suvaanshkumar:white_check_mark: suramasamy:x: Rami RadaidehRami Radaideh seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

New log Structure: Which includes the {"operation":value} block
{"level":"debug","ts":"2024-05-10T16:34:07.437-0400","logger":"provider-terraform","msg":"Reconciling","controller":"providerconfig/providerconfig.tf.upbound.io","request":{"name":"default"}}
{"level":"info","ts":"2024-05-10T16:34:09.895-0400","logger":"provider-terraform","msg":"Refreshing Terraform state in-memory prior to plan...\nThe refreshed state will be used to calculate this plan, but will not be\npersisted to local or remote state storage.\n\n\n------------------------------------------------------------------------\n\nAn execution plan has been generated and is shown below.\nResource actions are indicated with the following symbols:\n  + create\n\nTerraform will perform the following actions:\n\n  # random_id.example_id will be created\n  + resource "random_id" "example_id" {\n      + b64_std     = (known after apply)\n      + b64_url     = (known after apply)\n      + byte_length = 8\n      + dec         = (known after apply)\n      + hex         = (known after apply)\n      + id          = (known after apply)\n    }\n\n  # random_password.password will be created\n  + resource "random_password" "password" {\n      + bcrypt_hash = (sensitive value)\n      + id          = (known after apply)\n      + length      = 16\n      + lower       = true\n      + min_lower   = 0\n      + min_numeric = 0\n      + min_special = 0\n      + min_upper   = 0\n      + number      = true\n      + numeric     = true\n      + result      = (sensitive value)\n      + special     = true\n      + upper       = true\n    }\n\nPlan: 2 to add, 0 to change, 0 to destroy.\n\nChanges to Outputs:\n  + random_id_hex   = (known after apply)\n  + random_password = (sensitive value)\n\n------------------------------------------------------------------------\n\nNote: You didn't specify an "-out" parameter to save this plan, so Terraform\ncan't guarantee that exactly these actions will be performed if\n"terraform apply" is subsequently run.\n\n","request":"example-random-generators","operation":"plan"}
{"level":"info","ts":"2024-05-10T16:34:11.517-0400","logger":"provider-terraform","msg":"random_id.example_id: Creating...\nrandom_id.example_id: Creation complete after 0s [id=CS5FbALoRxU]\nrandom_password.password: Creating...\nrandom_password.password: Creation complete after 0s [id=none]\n\nApply complete! Resources: 2 added, 0 changed, 0 destroyed.\n\nOutputs:\n\nrandom_id_hex = 092e456c02e84715\nrandom_password = \n","request":"example-random-generators","operation":"apply"}

This almost feels like it should be extrapolated as a provider config option across all providers so you can like opt into all json logging if you want that. Having it as a flag on the workspace feels too low level here.

This almost feels like it should be extrapolated as a provider config option across all providers so you can like opt into all json logging if you want that. Having it as a flag on the workspace feels too low level here.

Debugging tends to happen at the individual Workspace level so I think it's appropriate to enable/disable logging at that level.

Hi @bobh66 and @zach-source, I think there is a bit of a miscommunication as there are two separate questions here:

The first question was, for Terraform CLI output, should the option to control whether it is logged or not be at the Workspace level or the ProviderConfig level. As @bobh66 said, the Workspace level makes the most sense because that's where debugging of Terraform configurations tends to happen.
Regarding @ramiradaideh's question: this PR makes a change to the provider that appears to switch all of the controller's log output from plain text to structure/JSON format. Given that we can't know all the possible ways that users of this provider may have set up their log monitoring systems and alerts, we probably shouldn't change this unconditionally.

To address the second item, I think an option to select the controller's log format should be introduced. Correct me if I'm wrong, but I don't think the option can be put at the Workspace level or the ProviderConfig level, since that will result in mixed-format output from the controller process. I think the only place the option can be is as an option to the controller process itself; it's something that could be changed via an environment variable on the Deployment for the controller.
Thoughts?

Hello @ytsarev @bobh66,
Could you please help in reviewing this PR, Thanks so much!

Does it make sense to encapsulate the recurring logic
	switch cmd.ProcessState.ExitCode() {
	case 0:
		if h.EnableTerraformCLILogging {
			h.Logger.Info(string(log), "operation", "<operation>")
		}
	default:
		ee := &exec.ExitError{}
		errors.As(err, &ee)
		if h.EnableTerraformCLILogging {
			h.Logger.Info(string(ee.Stderr), "operation", "<operation>")
		}
	}

into the runCommand function? A nice side effect is, that all commands are logged automatically. i.e. currently the workspace cmd is not logged.
Furthermore it would be nice to emit also a log before the command is executed including the cmd name and all the args. This would help a lot during troubleshooting, because one can reproduce with the exact commands. I think it makes also sense to implement this generally in the runCommand function.
What do you think @tammy-young @suramasamy  @bobh66  ?

I know it's late in the game, but it might be worth taking a look at https://github.com/crossplane/crossplane-runtime/pull/754.
I like the pattern we landed on for that feature (provider change logs):

The typical, existing provider logs are written to the main provider container's stdout. These logs are structured, but heterogenous in their schema - different log lines might have different fields.
The new change logs are sent over RPC to a sidecar container, which writes them to its stdout. These logs are structured and homogenous.

This way our change logs aren't mixed in with regular logs. You can easily configure fluent bit or similar to scrape all logs from the sidecar container and do something different with them from regular logs.


Logging issues in RestConf driver

Logging issues in RestConf driver

Checked in code with branch release-2.5.0 with issue #110


[Feature]: taq compile storage .jsligo 'Set.empty as set'

Describe the feature you would like
compile storage from jsligo expression to the config.json configuration for the contract .jsligo
on config.json output could be javascript like today or michelson expression
Is this feature solving a problem?
actual init storage require manual update and knowledge of taquito storage representation whereas we develop on jsligo , for example
Please share any additional details you have?
Details about the problem your feature solves
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@zamrokk this is definitely something we want to add. Thank you for logging this feature request.

Even better, we could be able to compile a storage referencing an environment. Example :
in ligo , we have
ligo compile storage ./contracts/pokeGame.jsligo 'Set.empty as set<address>' --output-file ./artifacts/pokeGameStorage.tz --protocol jakarta

in taqueria I would liek to have :
taq compile storage pokeGame.jsligo 'Set.empty as set<address>' -e "development"

and it should produce on .taq/config.json file, the storage section below:
"development": {
            "networks": [],
            "sandboxes": [
                "local"
            ],
            "storage": {
                "pokeGame.tz": []
            }
        },

taqueria will have to read the Michelson .tz output and transform it direclty to taquito javascript representation. It will be transparent to the user

@mweichert @jchenche, thoughts on this? ^^

@zamrokk https://github.com/ecadlabs/taqueria/issues/963 and an upcoming plan to include the compile expression and compile storage command/task will solve this.

Closing for now


Adding journal files to agenda search

Other than open journal file buffers, the org-agenda search does not include my journal files. I understand that org-agenda-file-regexp needs to be set and (org-journal-update-auto-mode-alist) should be called after this, but I am not able to get it working.
Setup:

All my org files (and agenda files) are in ~/my_org
org-journal directory is a subdirectory of the above: ~/my_org/journal/
I already have one modification to org-agenda-file-regexp to include gpg files

(unless (string-match-p "\\.gpg" org-agenda-file-regexp)
  (setq org-agenda-file-regexp
        (replace-regexp-in-string "\\\\\\.org" "\\\\.org\\\\(\\\\.gpg\\\\)?"
                                  org-agenda-file-regexp)))

I have also defined additional directories for the text search. This works fine for directories Other than the journal directory. I would have thought this to be sufficient.

(setq org-agenda-text-search-extra-files (apply 'append
						(mapcar
						 (lambda (directory)
						   (directory-files-recursively
						    directory org-agenda-file-regexp))
						 '("~/my_org/journal/"
                                                 "~/my_projects/" ))))

After all the above is run, my org-journal setup is run.

(use-package org-journal
  :ensure t
  :defer t
  :custom
  (org-journal-dir "~/my_org/journal/")
  (org-journal-file-format "%Y%m%d")
  (org-journal-enable-agenda-integration t)
  (add-to-list 'auto-mode-alist '(".*/[0-9]*$" . org-mode))

I've tried adding the following:
(setq org-agenda-file-regexp ".*/[0-9]*$")
(org-journal-update-auto-mode-alist)

I would be glad for advice for the correct method to go about including journal files in the org-agenda search.
My thinking is that modifying the regex above for including .gpg files to also include "./[0-9]$" should solve the problem.

This should work for you
(use-package org-journal
  :ensure t
  :defer t
  :customs:
  (org-journal-dir "~/my_org/journal/")
  (org-journal-enable-agenda-integration t)
  :config
  (setq org-agenda-file-regexp "\\`\\\([^.].*\\.org\\\|[0-9]\\\{8\\\}\\\(\\.gpg\\\)?\\\)\\'")
  (add-to-list 'org-agenda-files org-journal-dir))

I have updated the readme as it is incorrect.
I'll close this issue as it works, but don't hesitate to post more question here.

I just tried this. It does not work. Note: there is a typo in :custom
I tried removing the earlier snippet to include .gpg files since it is now set in the org journal config. Now gpg files are also not showing up in the agenda, or even in agenda search.

Works for me, not sure what else have changed!?

With the default org-journal-file-format-pattern value this should actually work! I tested it my self.

I've just copied your snippet and removed the earlier snippet for gpg inclusion. No other changes :) Also tried reloading Emacs for good measure.
I have an extension to my question:
I was going through the updated Readme and realised that I had not implemented complete agenda integration earlier as it was suggested that the inbuilt search is much faster. I do not want to slow down my agenda, and I have 189 journal files today.
However, my understanding was that the journal files would be included as extra locations Only for text search via agenda, and not agenda generation itself (unless there is an active date tag)
My goal is a unified search of my agenda files, journal files, and project files. Do you have any suggestions how I could do so?

You could switch to weekly/monthly/yearly journal files if you like,  to make it faster.

However, my understanding was that the journal files would be included as extra locations Only for text search via agenda, and not agenda generation itself (unless there is an active date tag)

Nope! Like any other org file!

Hmm - by using org-agenda-file-regexp : I presume i am including everything in the agenda.
However with org-agenda-text-search-extra-files , I believe these file lists are only used for search. why are my journal files not showing up using this >> Is it because I do not use an .org extension? My org based project files show up.
(setq org-agenda-text-search-extra-files (apply 'append
						(mapcar
						 (lambda (directory)
						   (directory-files-recursively
						    directory org-agenda-file-regexp))
						 '("~/my_org/journal/"
                                                 "~/my_projects/" ))))

My goal is a unified search of my agenda files, journal files, and project files. Do you have any suggestions how I could do so?

Not at the moment, but I'll dig a bit around in the next time and will let you know whether I found something or not.

Hmm - by using org-agenda-file-regexp : I presume i am including everything in the agenda.
However with org-agenda-text-search-extra-files , I believe these file lists are only used for search. why are my journal files not showing up using this >> Is it because I do not use an .org extension? My org based project files show up.

Hmm - by using org-agenda-file-regexp : I presume i am including everything in the agenda.
However with org-agenda-text-search-extra-files , I believe these file lists are only used for search. why are my journal files not showing up using this >> Is it because I do not use an .org extension? My org based project files show up.

Hmmm, I was not aware of the variable org-agenda-text-search-extra-files. Let me have quick look at it!

Well, I tested it and it seems setting just org-agenda-text-search-extra-files is sufficient, to search in journal files... as well, fabulous.

Content of my scratch buffer:
(setq org-agenda-text-search-extra-files (apply 'append
						(mapcar
						 (lambda (directory)
						   (directory-files-recursively
						    directory org-agenda-file-regexp))
						 '("~/emacs/org/journal"))))
("/home/cschwarzgruber/emacs/org/journal/20150101.gpg" "/home/cschwarzgruber/emacs/org/journal/20170101.gpg" "/home/cschwarzgruber/emacs/org/journal/20180101.gpg" "/home/cschwarzgruber/emacs/org/journal/20190101.gpg")
org-agenda-files
("/home/cschwarzgruber/emacs/org/agenda")
org-agenda-file-regexp
"\\`[^.].*\\.org\\'"

I finally got it working. :dancer:  I  had to set org-agenda-file-regexp to include the org journal format first. So I guess it means that setting org-agenda-file-regexp only dictates the format of the files and does not include them in the agenda itself.
Anyway - the results via agenda text search for multiple directories and only ~200 journal files is VERY slow, compared to using grep / rgrep / ag. The problem will only increase with more journal files. If the journal folder is within the same directory as org files - it is easy to use the above tools via Emacs. I'm trying to find a way to enable searching multiple directories.
Thank you for your quick help today !

That's why we included org-journal-search, which is super fast even for large numbers of files. The problem is that the agenda search opens every file in org-mode, and org-mode is pretty slow to load files. Using a weekly/monthly journal cuts down on the number of files, which makes it faster. But I prefer my daily journal.

My journal currently contains 1013 files, since 2016. org-journal-search is still incredibly snappy!

Thanks for sharing your workflow Bastian. I'm happy to find that my current workflow is quite similar. I was initially sceptical about the intermittent narrative approach and spreading information to daily files.  However, I've found it to be much more efficient and functional to maintain a detailed daily narrative, and review this regularly, rather than capture active date tagged notes in  large files, and then remember to archive them and etc.
I agree it is a good idea to  think 'wide' and incorporate as many keywords as possible, otherwise it becomes quite frustrating. I've found this becomes better only with practice and time, i.e when you actually search for something and find you did not record the one thing that you need now!


Add england west midlands group

Will check that I got this right first and then move on to adding other UK resources.

Looks great, thank you @RobJN 👍