The stable diffusion example encountered an error after being upgraded to version 0.4.

commannd(mlx-example: 47dd6bd17f3cc7ef95672ea16e443e58ce5eb1bf)
python txt2image.py "A photo of an astronaut riding a horse on Mars." --n_images 1 --n_rows 1 --steps 1

error(all):
$ python txt2image.py "A photo of an astronaut riding a horse on Mars." --n_images 1 --n_rows 1 --steps 1

100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 1/1 [00:09<00:00,  9.25s/it]
  0%|                                                                                | 0/1 [00:00<?, ?it/s]
libc++abi: terminating due to uncaught exception of type std::runtime_error: [METAL] Command buffer execution failed: Invalid Resource (00000009:kIOGPUCommandBufferCallbackErrorInvalidResource)
[1]    7474 abort      python txt2image.py "A photo of an astronaut riding a horse on Mars."  1  1
/Users/xxx/.pyenv/versions/3.10.13/lib/python3.10/multiprocessing/resource_tracker.py:224: UserWarning: resource_tracker: There appear to be 1 leaked semaphore objects to clean up at shutdown
  warnings.warn('resource_tracker: There appear to be %d '

My environment:

mac mini, m2 pro
macOS: 14.3.1
python: 3.10.13
mlx: 0.4.0
clang:

Apple clang version 15.0.0 (clang-15<IP_ADDRESS>.5)
Target: arm64-apple-darwin23.3.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

I attempted to locate the error by adding a log, and I found it in vae.py:Decoder:__call__. However, when I tried to investigate further by adding another log(add print(x) at this line), the error disappeared. It seems that some variable is released too early?

I'm attempting to reproduce the issue. It did not fail with the same command on an M1 Max. How much RAM does your M2 pro have?

It did fail on an M1 mini with 8 GB...presumably it's an OOM issue but the message is not helpful

I'm attempting to reproduce the issue. It did not fail with the same command on an M1 Max. How much RAM does your M2 pro have?

The memory of my M2 pro is 16GB. This error does not occur in mlx0.3, suggesting that memory may not be the main cause, I think. And I have two devices(16GB M2 air and 16GB M2 Pro mini). The error occurs on both devices.

Hmm actually I was able to repro the bug in 0.3 and 0.4. I believe the fix is in https://github.com/ml-explore/mlx/pull/752


Library award

I noticed that Physiolibrary applied for the award at https://modelica.org/events/modelica2014/proceedings/html/sessions/session_libraries.html. Do you know the winning three libs? Unfortunately I cannot find any public record for it. Thanks.

Physiolibrary was first, ThermoCycle was second. I am not sure about the third place.

Thanks.


Loss name not using names from the output dictionary

Click to expand!
Issue Type
Support
Source
binary
Tensorflow Version
v2.8.0-rc1-32-g3f878cff5b6 2.8.0
Custom Code
No
OS Platform and Distribution
Linux Arch 5.17.4-arch1-1
Mobile device
No response
Python version
3.9.12
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
I am not sure if this is expected behavior or not, but when I have custom layer that returns dict of tensors then I would like to loss use those names because order could change it I outputs change.

For example if I set `my_layer_1_mse` as a monitor variable in early stopping and change other outputs this will no longer work if `out2` changes from `my_layer_1_mse` to e.g. `my_layer_2_mse`

I would like to know if it is possible to have loss and metrics be named `out1_loss` and `out2_mse` in this specific case. Or at least `my_layer_out1_loss` and `my_layer_out2_loss`

Another problem is that I would like for model outputs to follow same naming convention if possible when exporting model to, for example, to tfjs. Currently outputs will be named `my_layer` and `my_layer_1` instead of `out1` and `out2` respectively.

Thank you for your help.

Standalone code to reproduce the issue
import tensorflow as tf

class Layer(tf.keras.layers.Layer):
    def __init__(self, **kwargs):
        super().__init__(**kwargs)

    def call(self, inputs):
        batch_size = tf.shape(inputs)[0]
        return {"out1": tf.repeat(1, batch_size), "out2": tf.repeat(2, batch_size)}

def build_model():

    input = tf.keras.layers.Input(shape=(1,))
    out = Layer(name="my_layer")(input)

    return tf.keras.Model(
        inputs=input,
        outputs=out,
    )

model = build_model()

model.compile(
    optimizer=tf.keras.optimizers.Adam(),
    loss={"out1": "mse"},
    metrics={"out2": "mse"},
)

data = (
    # x
    [5, 5, 5, 5, 5, 5, 5, 5],
    # y
    {
        "out1": [5, 5, 5, 5, 5, 5, 5, 5],
        "out2": [5, 5, 5, 5, 5, 5, 5, 5],
    },
)

ds = tf.data.Dataset.from_tensor_slices(data)
ds = ds.batch(2)

model.fit(ds)

Relevant log output
4/4 [==============================] - 0s 2ms/step - loss: 16.0000 - my_layer_loss: 16.0000 - my_layer_1_mse: 9.0000

@eugen-vusak ,
I was able to reproduce the issue in tf v2.7, v2.8 and nightly.Please find the gist of it here.

@eugen-vusak ,
Thanks for opening this issue. Development of keras moved to another repository.
Can you please post this issue on keras-team/keras repo.
To know more refer:
https://discuss.tensorflow.org/t/keras-project-moved-to-new-repository-in-https-github-com-keras-team-keras/1999
Thank you!


Use name param for avatar alt text instead of hard-coded value.

I'm a complete newb to both Hugo, this theme, and half the languages used. So if I've screwed something up or misunderstood some underlying concept, please let me know. Thanks!


Testing To Do list: part 1

Testing add task and remove task

Hi @ichala, please I think we are doing group project and the previous code as not been approved yet and we can use anyone's repository to submit the first task.

Hello, @Fayob I'm sorry I can't proceed with the review since you guys requested one before already you can ask for a second opinion review from this form. thank you


API and request updates

updated BASE_URL to vesyncs new API addresses, and corrected all API requests to match vesync's native app requests (extracted from Proxie app). get_devices(), turn_on(), and turn_off() now work 100%

I still get the occasional problem where the switches will turn on and then immediately back off, then have to wait ~10-15s before I can get them to switch back on.
I've opened an issue for myself to track on my own repo, but wanted to see if anyone else was experiencing these problems, and if so, if they ever solved them.
https://github.com/jslay88/pyhap_vesync/issues/2


Date Nullable

Hello,
the plainToClass doesnt not seams to deals with nullable Date:
Example  :
@Type(() => Date)
public Act_Date_Reunion_Exp: Date;

=> if jsonObject Act_Date_Reunion_Exp is null , set classObject to "1970-01-01 01:00:00.000" Date
Example 2 :
// no typing @Type(() => Date)
public Act_Date_Reunion_Exp: Date;

=> if jsonObject Act_Date_Reunion_Exp is null , set classObject to new Date() Date
Example 3
 @Transform(value =>moment(value), { toClassOnly: false })
 public Act_Date_Reunion_Exp: Moment;
=>throw error : target.prototype in findMetadatas() , target is null

Maybe i'm not doing this right
Any Ideas ?

@colbeaf Helllo, maybe use this:
@Transform(value =>moment(value).isValid() ? moment(value).isValid() : null, { toClassOnly: false })
 public Act_Date_Reunion_Exp?: Moment;

@artemln I suppose your solution should look like this:
@Transform(value =>moment(value).isValid() ? moment(value) : null, { toClassOnly: false })
 public Act_Date_Reunion_Exp?: Moment;

Hi @colbeaf!
The @Type operator is for user defined classes. You dont need to specify them for builtins.
I cannot reproduce your issue, I have tried with the following code:
import { Type, plainToClass } from 'class-transformer';

class Example {
  public date: Date;
}

plainToClass(Example, { date: null });

This will result in the expected Example {date: null} object.
If you need data validation you should take a look at typestack/class-validator.
I will go ahead and close this, if you have any more question regarding the topic, please feel free to continue the discussion.


http://docs.romana.io/ - do not work

attempt to open http://docs.romana.io/ gives next error:
403 Forbidden

    Code: AllAccessDisabled
    Message: All access to this object has been disabled
    RequestId: 9D0BAC3B14F370D5
    HostId: VvORaqgUfQvTj+F7RcsLW6Y2jafRW7HVD2rUKopLyoGHZi3yplqXbksIQZmHy5gF8q5WQm5Kvf8=

Sorry about that. The link is broken. The docs were moved to http://romana.readthedocs.io

https://github.com/romana/romana/pull/187

Great! Now it is better :-) I am closing this issue.


insert gecomplete search results into a mysql database

i want to use search field and map in a registration form to find a person location and then insert the result town, region, country, latitude and longitude into a mysql table. Anyone know how to do this?

This is a pure JS plugin. We won't add server or DB support.


Duplicate entity found

Hey guys,
I'm trying to pass two ligands to the model, but I'm encountering the following error. Has anyone tried passing multiple ligands on a local installation? (Multiple proteins work fine for me.) I'm curious whether I'm doing something wrong with the input or if I should dig deeper into the code.
Traceback (most recent call last):
File "/home/andrey/chai-lab/examples/predict_structure.py", line 51, in
candidates = run_inference(
File "/home/andrey/anaconda3/envs/chai/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/home/andrey/chai-lab/chai_lab/chai1.py", line 474, in run_inference
return run_folding_on_context(
File "/home/andrey/anaconda3/envs/chai/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/home/andrey/chai-lab/chai_lab/chai1.py", line 910, in run_folding_on_context
save_to_cif(
File "/home/andrey/chai-lab/chai_lab/data/io/cif_utils.py", line 151, in save_to_cif
new_context_to_cif_atoms(
File "/home/andrey/anaconda3/envs/chai/lib/python3.10/site-packages/jaxtyping/_decorator.py", line 559, in wrapped_fn
return wrapped_fn_impl(args, kwargs, bound, memos)
File "/home/andrey/anaconda3/envs/chai/lib/python3.10/site-packages/jaxtyping/_decorator.py", line 483, in wrapped_fn_impl
out = fn(*args, **kwargs)
File "/home/andrey/chai-lab/chai_lab/data/io/cif_utils.py", line 242, in new_context_to_cif_atoms
dumper.write(open(out_path, "w"), systems=[system])
File "/home/andrey/anaconda3/envs/chai/lib/python3.10/site-packages/modelcif/dumper.py", line 870, in write
return ihm.dumper.write(fh, systems, format, dumpers, variant)
File "/home/andrey/anaconda3/envs/chai/lib/python3.10/site-packages/ihm/dumper.py", line 3989, in write
d.finalize(system)
File "/home/andrey/anaconda3/envs/chai/lib/python3.10/site-packages/ihm/dumper.py", line 370, in finalize
raise ValueError("Duplicate entity %s found" % entity)
ValueError: Duplicate entity <ihm.Entity(ligand-2)> found
FASTA file I'm using:
example_fasta = """
>protein|name=example-of-long-protein
MANFTPVNGSSGNQSVRLVTSSSHNRYETVEMVFIATVTGSLSLVTVVGNILVMLSIKVNRQLQTVNNYFLFSLACADLIIGAFSMNLYTVYIIKGYWPLGAVVCDLWLALDYVVSNASVMNLLIISFDRYFCVTKPLTYPARRTTKMAGLMIAAAWVLSFVLWAPAILFWQFVVGKRTVPDNQCFIQFLSNPAVTFGTAIAAFYLPVVIMTVLYIHISLASRSRVHKHRPEGPKEKKAVARKFASIARNQVRKKRQMAARERKVTRTIFAILLAFILTWTPYNVMVLVNTFCQSCIPDTVWSIGYWLCYVNSTINPACYALCNATFKKTFRHLLLCQYRNIGTAR
>ligand|name=ligand-1
CN1CCN(CC1)C(=O)COc1nc2sc(c(c2c(c1Cl)C)N)C(=O)NC1CC1
>ligand|name=ligand-2
C[N+](CC#CCOC1=NOCC1)(C)C
""".strip()

It seems a similar bug has recently been introduced for homopolymers.  See https://github.com/chaidiscovery/chai-lab/issues/67#issuecomment-2543386006 .
Rolling back (git checkout f77f44165ef59bba0685f176d03ee4b02e793967) resolves the issue.

@mosayebi thanks for reporting, please try PR https://github.com/chaidiscovery/chai-lab/pull/251 , problem should be fixed there


Issue: 401 - Invalid client secret is provided

When I run this task, I keep getting a 401. I am using a service connection which I've tried specifying in YAML (in build) and in the UI (for release), but neither are working. I ran it with debug and
What seems to be the relevant error:
error_description":"AADSTS7000215: Invalid client secret is provided.\r\nTrace ID: 6ef7f788-7671-4305-97cb-f3f5b7384900
I also see that some of these are blank:
"spnObjectId":"","appObjectId":""
So I guess something is wrong with my service connection?
This is the full dump:
2019-04-08T16:52:42.2928624Z ##[debug]Evaluating condition for step: 'Create Azure Blueprint'
2019-04-08T16:52:42.2942327Z ##[debug]Evaluating: succeeded()
2019-04-08T16:52:42.2942558Z ##[debug]Evaluating succeeded:
2019-04-08T16:52:42.2942968Z ##[debug]=> True
2019-04-08T16:52:42.2943260Z ##[debug]Result: True
2019-04-08T16:52:42.2943836Z ##[section]Starting: Create Azure Blueprint
2019-04-08T16:52:42.3272834Z ==============================================================================
2019-04-08T16:52:42.3273128Z Task         : Create Azure Blueprint
2019-04-08T16:52:42.3273203Z Description  : Creates an Azure Blueprint
2019-04-08T16:52:42.3273287Z Version      : 1.0.0
2019-04-08T16:52:42.3273351Z Author       : Neil Peterson
2019-04-08T16:52:42.3273429Z Help         :
2019-04-08T16:52:42.3273498Z ==============================================================================
2019-04-08T16:52:44.2848539Z ##[debug]VstsTaskSdk 0.11.0 commit 7ff27a3e0bdd6f7b06690ae5f5b63cb84d0f23f4
2019-04-08T16:52:44.7671692Z ##[debug]INPUT_CONNECTEDSERVICENAME: '2010be28-4fba-4774-bc8d-bd4a611057da'
2019-04-08T16:52:44.7830810Z ##[debug]ENDPOINT_URL_2010be28-4fba-4774-bc8d-bd4a611057da: 'https://management.azure.com/'
2019-04-08T16:52:44.7870804Z ##[debug]ENDPOINT_AUTH_2010be28-4fba-4774-bc8d-bd4a611057da: '********'
2019-04-08T16:52:44.7972407Z ##[debug]ENDPOINT_DATA_2010be28-4fba-4774-bc8d-bd4a611057da: '{"subscriptionId":"d56e652e-758d-480a-8f0d-47f230264b4c","subscriptionName":"Visual Studio Ultimate with MSDN","environment":"AzureCloud","scopeLevel":"Subscription","creationMode":"Manual","azureSpnRoleAssignmentId":"","azureSpnPermissions":"","spnObjectId":"","appObjectId":"","environmentUrl":"https://management.azure.com/","galleryUrl":"https://gallery.azure.com/","serviceManagementUrl":"https://management.core.windows.net/","resourceManagerUrl":"https://management.azure.com/","activeDirectoryAuthority":"https://login.microsoftonline.com/","environmentAuthorityUrl":"https://login.windows.net/","graphUrl":"https://graph.windows.net/","managementPortalUrl":"https://manage.windowsazure.com/","armManagementPortalUrl":"https://portal.azure.com/","activeDirectoryServiceEndpointResourceId":"https://management.core.windows.net/","sqlDatabaseDnsSuffix":".database.windows.net","AzureKeyVaultDnsSuffix":"vault.azure.net","AzureKeyVaultServiceEndpointResourceId":"https://vault.azure.net","StorageEndpointSuffix":"core.windows.net","EnableAdfsAuthentication":"false"}'
2019-04-08T16:52:44.8066962Z ##[debug]INPUT_PUBLISHBLUEPRINT: 'true'
2019-04-08T16:52:44.8098395Z ##[debug]INPUT_VERSION: 'Increment'
2019-04-08T16:52:44.8128261Z ##[debug]INPUT_BLUEPRINTCREATIONLOCATION: 'managementGroup'
2019-04-08T16:52:44.8197174Z ##[debug]INPUT_MANAGEMENTGROUPNAME: 'root'
2019-04-08T16:52:44.8221667Z ##[debug]INPUT_BLUEPRINTNAME: 'AppNetwork'
2019-04-08T16:52:44.8228970Z ##[debug]INPUT_BLUEPRINTPATH: './Example Blueprints/AppNetwork/blueprint.json'
2019-04-08T16:52:44.8258581Z ##[debug]INPUT_ARTIFACTSPATH: './Example Blueprints/AppNetwork/artifacts'
2019-04-08T16:52:44.8503832Z ##[debug]POST https://login.microsoftonline.com/***/oauth2/token with -1-byte payload
2019-04-08T16:52:45.3588019Z ##[debug]Caught exception from task script.
2019-04-08T16:52:45.3631545Z ##[debug]Error record:
2019-04-08T16:52:45.4888869Z ##[debug]Invoke-RestMethod : {"error":"invalid_client","error_description":"AADSTS7000215: Invalid client secret is provided.\r\nTrace ID: 6ef7f788-7671-4305-97cb-f3f5b7384900\r\nCorrelation ID: 56dc967c-8e7a-41c5-8dfe-e0eaed3cf1f8\r\nTimestamp: 2019-04-08 16:52:44Z","error_codes":[7000215],"timestamp":"2019-04-08 16:52:44Z","trace_id":"6ef7f788-7671-4305-97cb-f3f5b7384900","correlation_id":"56dc967c-8e7a-41c5-8dfe-e0eaed3cf1f8"}
2019-04-08T16:52:45.4901926Z ##[debug]At D:\a\_tasks\CreateBlueprint_1a8623d7-1d5c-423c-9d92-5e9587bbb02a\1.0.0\createBlueprint.ps1:35 char:10
2019-04-08T16:52:45.4916056Z ##[debug]+ $Token = Invoke-RestMethod -Method Post -Uri $RequestAccessTokenUri - ...
2019-04-08T16:52:45.4932626Z ##[debug]+          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2019-04-08T16:52:45.4948113Z ##[debug]    + CategoryInfo          : InvalidOperation: (System.Net.HttpWebRequest:Htt    pWebRequest) [Invoke-RestMethod], WebException
2019-04-08T16:52:45.4963833Z ##[debug]    + FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShe    ll.Commands.InvokeRestMethodCommand
2019-04-08T16:52:45.4994170Z ##[debug]Script stack trace:
2019-04-08T16:52:45.5041822Z ##[debug]at <ScriptBlock>, D:\a\_tasks\CreateBlueprint_1a8623d7-1d5c-423c-9d92-5e9587bbb02a\1.0.0\createBlueprint.ps1: line 35
2019-04-08T16:52:45.5056221Z ##[debug]at <ScriptBlock>, <No file>: line 1
2019-04-08T16:52:45.5072381Z ##[debug]at <ScriptBlock>, <No file>: line 22
2019-04-08T16:52:45.5086346Z ##[debug]at <ScriptBlock>, <No file>: line 18
2019-04-08T16:52:45.5100626Z ##[debug]at <ScriptBlock>, <No file>: line 1
2019-04-08T16:52:45.5124471Z ##[debug]Exception:
2019-04-08T16:52:45.5175720Z ##[debug]System.Net.WebException: The remote server returned an error: (401) Unauthorized.
2019-04-08T16:52:45.5189823Z ##[debug]   at Microsoft.PowerShell.Commands.WebRequestPSCmdlet.GetResponse(WebRequest request)
2019-04-08T16:52:45.5203090Z ##[debug]   at Microsoft.PowerShell.Commands.WebRequestPSCmdlet.ProcessRecord()
2019-04-08T16:52:45.5825597Z ##[error]The remote server returned an error: (401) Unauthorized.
2019-04-08T16:52:45.5835545Z ##[debug]Processed: ##vso[task.logissue type=error]The remote server returned an error: (401) Unauthorized.
2019-04-08T16:52:45.5851075Z ##[debug]Processed: ##vso[task.complete result=Failed]
2019-04-08T16:52:45.6411594Z ##[section]Finishing: Create Azure Blueprint

Are you creating the blueprint at a management group and if so can you make the service principal an owner of the management group?

And then that the pipeline service connection is configured to use this service principal? This requires creating the service connection with the ‘advanced’ options.

The blueprint is at an MG and the service principal is an owner of that MG, I double checked, and I’m pretty confident the service connection is configured correctly (see screenshots).
Is it expected that those two properties are empty strings like that? It seems like it is not grabbing the SPN/service connection info properly.

I've just tested in my environment and these values are also blank (however task was successful).
Walking through the configuration from scratch, end to end, I noticed that when creating the Azure Pipeline Service Connection we can create a new service connection (without providing creds) and scope it to a Management Group, which will create the SP and grant ownership over the management group.
I've tried this and the task was successful. Can you give this a shot to troubleshoot any potential service principal issues?

@alex-frankel sorry, advice in the last comment is not good.
I've updated the readme with end to end instructions which have worked in my environment. I can help troubleshoot next week in yours.
Thanks

So given what was troubleshooted, it's possible there still may be an issue with using an existing SPN. Though it's also possible since I was not upgraded automatically from 1.0 to 1.1 that could have resolved it to.
Also, DevOps updating their service connection UX to support MGs also could have resolved it.
I think you can consider this closed until I can repro it again

Closing this for now.


AP-1747: Update benefit association of bank transactions

What
Link to story
Update the StateBenefitAnalyserService
Update the regex handling, remove the applicants nino and only check for
words that match the known DWP codes.
This also meant removing the handling for unknown DWP codes
as we only match on existing ones.
NOTE: This should not be merged until after the CFE end
NOTE II: commit dcad022 should be removed before merging, it allows the apply UAT instance to talk to the CFE UAT instance but will affect any subsequent branches if merged!
NOTE III:

[x] Needs rebasing on master

Checklist
Before you ask people to review this PR:

[x] Tests and rubocop should be passing: bundle exec rake
[x] Github should not be reporting conflicts; you should have recently run git rebase master.
[x] There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
[x] The PR description should say what you changed and why, with a link to the JIRA story.
[x] You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
[x] You should have checked that the commit messages say why the change was made.

Rebased to remove temp redirect in UAT


update packages

:x: Build TheIdServer 7.4.7-fix-dependencies1-0010 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/1208c84f72 by @invalid-email-address)

:white_check_mark: Build TheIdServer 7.4.7-fix-dependencies.1+10.build.3409 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/1208c84f72 by @invalid-email-address)


Numpy build fails

The numpy --no-binary flag causes the following error when building the Docker image:
ERROR: Service 'base' failed to build: The command '/bin/sh -c pip3 install . --no-binary numpy,rasterio -t $PACKAGE_PREFIX -U' returned a non-zero code: 1
The error occurs when compiling numpy with gcc:
error: Command "gcc -pthread -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fPIC -Ibuild/src.linux-x86_64-3.6/numpy/core/src/common -Inumpy/core/include -Ibuild/src.linux-x86_64-3.6/numpy/core/include/numpy -Inumpy/core/src/common -Inumpy/core/src -Inumpy/core -Inumpy/core/src/npymath -Inumpy/core/src/multiarray -Inumpy/core/src/umath -Inumpy/core/src/npysort -I/var/lang/include/python3.6m -Ibuild/src.linux-x86_64-3.6/numpy/core/src/common -Ibuild/src.linux-x86_64-3.6/numpy/core/src/npymath -Ibuild/src.linux-x86_64-3.6/numpy/core/src/common -Ibuild/src.linux-x86_64-3.6/numpy/core/src/npymath -c build/src.linux-x86_64-3.6/numpy/core/src/npysort/radixsort.c -o build/temp.linux-x86_64-3.6/build/src.linux-x86_64-3.6/numpy/core/src/npysort/radixsort.o -MMD -MF build/temp.linux-x86_64-3.6/build/src.linux-x86_64-3.6/numpy/core/src/npysort/radixsort.o.d" failed with exit status 1
Removing the --no-binary flag for numpy fixes this error (see here).  This fix is based off of Seth's rasterio lambda layer.

I've seen this recently I had to force numpy==0.16 to build it.

I've added a temporary fix here https://github.com/developmentseed/cogeo-mosaic/blob/master/Dockerfile#L13


IBM-Swift/Kitura-CouchDB#58 Remove check on uninitialized variable

The change in #11 removed the calls to set CURLOPT_HEADER, but left in the subsequent check if (rc == CURLE_OK) on the uninitialized CURLcode rc; variable, which meant the subsequent calls were never invoked.
This is breaking several dependent repos (See https://github.com/IBM-Swift/Kitura-CouchDB/issues/58). Please tag a patch with this fix if it looks ok.

@shmuelk reviewed this and said it looks fine. Merging.


Develop

after run composer require masmaleki/laravel-product-finder in Laravel project
run php artisan vendor:publish --tag=product-finder-migrations
run php artisan vendor:publish --tag=productfinder-views
run php artisan vendor:publish --tag=product-finder-assets

You will see the result page in DOMAIN/product-finder


Trash & BulkTrash operation trait in CRUD Package

WHY
Trash & BulkTrash operation trait in CRUD Package

to use same path like other operations
to check backpack pro for usage

A new inspection was created.

The inspection completed: 2 new issues, 4 updated code elements

Pedro pinged me on this - I don't think this is needed, to be honest. It's ok for the PRO operations to have a different namespace. We only introduced this for the previous operations to prevent a breaking change... but we might move them in v6 anyway.


Resolve name collisions

At the moment when two items to be saved have the same directory name, restic
only saves the first one to the repo. Let's say we have a structure like this:
dir1
└── subdir
    └── file
dir2
└── subdir
    └── file

When restic is run on dir1/subdir and dir2/subdir, it will only save the
first subdir:
$ restic backup dir1/subdir dir2/subdir
[...]

$ restic ls latest
drwxr-xr-x  1000   100      0 2017-08-27 20:56:39 /subdir
-rw-r--r--  1000   100     17 2017-08-27 20:56:39 /subdir/file

That's obviously a bad thing, caused by an early decision to strip the full
path to the files/dirs to save and only leave the last directory.
This commit partly resolves this by handling colliding names and resolving the
conflicts. Restic will now append a counter (-123) until the conflict is
resolved. So in the example above, we'll end up with the following structure:
$ restic ls latest
drwxr-xr-x  1000   100      0 2017-08-27 20:56:39 /subdir
-rw-r--r--  1000   100     17 2017-08-27 20:56:39 /subdir/file
drwxr-xr-x  1000   100      0 2017-08-27 20:56:46 /subdir-1
-rw-r--r--  1000   100     17 2017-08-27 20:56:46 /subdir-1/file

This partly resolves #549 and closes #1179.

Codecov Report

Merging #1209 into master will increase coverage by 0.05%.
The diff coverage is 70%.

@@            Coverage Diff             @@
##           master    #1209      +/-   ##
==========================================
+ Coverage   52.92%   52.97%   +0.05%
==========================================
  Files         126      123       -3
  Lines       12060    12052       -8
==========================================
+ Hits         6383     6385       +2
+ Misses       4932     4925       -7
+ Partials      745      742       -3

Impacted Files
Coverage Δ

internal/archiver/archiver.go
64.82% <70%> (-0.14%)
:arrow_down:

internal/restic/node_darwin.go

internal/restic/progress_unix_with_siginfo.go

cmd/restic/background.go

internal/backend/test/tests.go
58.35% <0%> (+0.65%)
:arrow_up:

internal/backend/s3/s3.go
61.03% <0%> (+1.37%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #1209 into master will increase coverage by 0.05%.
The diff coverage is 70%.

@@            Coverage Diff             @@
##           master    #1209      +/-   ##
==========================================
+ Coverage   52.92%   52.97%   +0.05%
==========================================
  Files         126      123       -3
  Lines       12060    12052       -8
==========================================
+ Hits         6383     6385       +2
+ Misses       4932     4925       -7
+ Partials      745      742       -3

Impacted Files
Coverage Δ

internal/archiver/archiver.go
64.82% <70%> (-0.14%)
:arrow_down:

internal/restic/node_darwin.go

cmd/restic/background.go

internal/restic/progress_unix_with_siginfo.go

internal/backend/test/tests.go
58.35% <0%> (+0.65%)
:arrow_up:

internal/backend/s3/s3.go
61.03% <0%> (+1.37%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #1209 into master will increase coverage by 0.26%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1209      +/-   ##
==========================================
+ Coverage   52.92%   53.19%   +0.26%
==========================================
  Files         126      126
  Lines       12060    12137      +77
==========================================
+ Hits         6383     6456      +73
  Misses       4932     4932
- Partials      745      749       +4

Impacted Files
Coverage Δ

internal/archiver/archiver.go
65.47% <100%> (+0.51%)
:arrow_up:

internal/filter/filter.go
79.36% <0%> (+6.5%)
:arrow_up:

cmd/restic/cmd_restore.go
67.64% <0%> (+7.64%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #1209 into master will decrease coverage by 0.36%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1209      +/-   ##
==========================================
- Coverage   53.03%   52.67%   -0.37%
==========================================
  Files         126      123       -3
  Lines       12102    12094       -8
==========================================
- Hits         6418     6370      -48
- Misses       4937     4987      +50
+ Partials      747      737      -10

Impacted Files
Coverage Δ

internal/archiver/archiver.go
65.3% <100%> (+0.51%)
:arrow_up:

cmd/restic/cmd_mount.go
12.79% <0%> (-44.19%)
:arrow_down:

internal/backend/s3/s3.go
59.65% <0%> (-1.38%)
:arrow_down:

internal/restic/node.go
33.12% <0%> (-1.07%)
:arrow_down:

internal/restic/node_darwin.go

internal/restic/progress_unix_with_siginfo.go

cmd/restic/background.go

Continue to review full report at Codecov.

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

This is great stuff. Clean and simple and the expected way for it to be dealt with when there's conflicting file/directory names. Also will only happen on the first level in the restore, which is already a place where you know you'll have duplicate names to address anyway.

Codecov Report

Merging #1209 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1209      +/-   ##
==========================================
+ Coverage   53.03%   53.06%   +0.03%
==========================================
  Files         126      123       -3
  Lines       12102    12094       -8
==========================================
  Hits         6418     6418
+ Misses       4937     4930       -7
+ Partials      747      746       -1

Impacted Files
Coverage Δ

internal/archiver/archiver.go
65.47% <100%> (+0.67%)
:arrow_up:

internal/restic/node_darwin.go

internal/restic/progress_unix_with_siginfo.go

cmd/restic/background.go

Continue to review full report at Codecov.

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

@fd0 ,
The fix helps when there is a collision but it looks like it treats the "collision" as temporary situation. It's better than previous behavior as it does not ignores/removes files from backup but it will still confuses the restore. Which subdir applies to which path? It might get untreatable. For example, image a user wants to backup its /etc/ directory. However, it uses opensuse that uses btrfs and snapshots periodically root fs. The user could ask restic to backup
restic backup /.snapshots/*/snapshot/etc

Now I'll get a bunch of etc-xxx. I guess restic would even reuse the counter (or even /etc) when the snapshot that used it get pruned. So, a new /etc could be newer than /etc-1. I was just a cheap example. It could be many more.
I guess the correct solution would be to expand any relative path and save the full/absolute path. Go-lang libraries could do it for you.
In the example you mentioned,
machine1:/xxx/yyy $ restic backup ../foo foo

Should never collide as they should be saved as:

machine1:/xxx/foo
machine1:/xxx/yyy/foo

I didn't get the idea why using only dirname was a good solution. What was the situation it was trying to solve?

You're right, this PR does not solve the issue completely. The second part (which will be done in a separate PR) will change the code so that the complete path (if available) is saved.
In this your case, running
$ restic backup /.snapshots/*/snapshot/etc

will yield a snapshot that contains:
.snapshots/foo/snapshot/etc
.snapshots/bar/snapshot/etc
[...]

So there's no collision any more. That's what tar does, and it works very well.
The problem with the other example is that determining what .. means is not so easy as it may sound. So we'll probably keep the current behavior: When restic is instructed to save ../foo and foo, it will still rename the second foo to foo-1. Users are warned during backup that this happened, and they can easily remedy the situation by providing an absolute path, e.g. /home/user/foo and /home/user/work/foo so the collision does not occur.
Another option that I'd like to implement (again, similar to tar) is --strip-components, so that a number of path elements are removed. Your example again:
$ restic backup --strip-components=1 /.snapshots/*/snapshot/etc

will yield a snapshot that contains:
foo/snapshot/etc
bar/snapshot/etc
[...]

But when users then instruct restic to strip e.g. three path components, collisions happen again (several etc directories at the top level).
As far as I can see there are several options what restic can do:

Bail out, print an error message, exit with an error code
Silently archive several directories with the same name at the same level. That's possible in the data structures we use.
Warn the use, archive the files, but make sure the names are unique.

I don't like the first option, it's not user friendly. The second option is bad because it makes restore very awkward: Which etc directory should be restored? We'd need to find a way (maybe with include/exclude filters?) to address a specific directory when there are several with the same name. That's not good.
The third option is what I like best so far, and that's what we're going to build (at least that's the plan).

@fd0 , this is the only (known) issue that prevent me from using restic. The fullpath feature seems to be a very long-time requested feature. Isn't just a matter of storing all the path passed as arg in the field where today only the directory name is stored?
If / or . are special character, simply replace it with any other character (like '-' or \1). It's hacky but way better than simply append a different number for each conflicted backup. If the user asked to backup '../../etc', warn the user, replace ".." with "__" and go on, storing it as --etc. I'll give you time to refactor the repo structure for a long-term solution. Users need a deterministic "backup path", not dir-1, dir-2, dir-3... that depends on the order of args. If dir-2 is gone, dir-3 becomes dir-2. It's a maintenance nightmare. It would be better to only apply this kind of replace-string solution where it is needed. If repo can store ../../etc or /etc/, save it just like the user passed. Leave the hacks for where it is really needed (fuse mount?)
A feature like '--fullpath' that resolves relative path into full paths is an needed feature for any serious backup solution. I think that it is the way to go. Keep the "last directory name" as default behavior for now if you wish but allow the user to ask restic to resolve the path (I still prefer to simply stores what user passed as arg). Any language library allow to convert relative path into fullpath for existing paths, and so does go-lang. You can easily resolve any path in unix with "readllnk -f relative-or-absolute-path", even with non-existing path (-m).

FYI: I'm already working on this.


Fix vppbranch var to point to right versions of Calico VPP

Fix vppbranch var to point to right versions of Calico VPP
Product Version(s):
v3.27.0
Issue:

Link to docs preview:

SME review:

[ ] An SME has approved this change.

DOCS review:

[ ] A member of the docs team has approved this change.

Additional information:

Merge checklist:

[ ] Deploy preview inspected wherever changes were made
[ ] Build completed successfully
[ ] Test have passed

@ctauchen, could you review and merge this please :)

LGTM, thanks @onong


feat: add postgresql support

add postgresql support
dashboard chat prompt update

@lozzo Thank you for your patience. Please address the code formatting issues. You can install black with pip install black, and then run black . in the project directory to format the Python code.

@lozzo Thank you for your patience. Please address the code formatting issues. You can install black with pip install black, and then run black . in the project directory to format the Python code.

done


[GHSA-3xq5-wjfh-ppjc] Luxon Inefficient Regular Expression Complexity vulnerability

Updates

Affected products

Comments
Wrong luxon 1.x version, it is not 1.38.1 but 1.28.1. See https://www.npmjs.com/package/luxon?activeTab=versions

Hi there @icambron! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

oops, thought i already fixed that. Thanks


Use structured data class for CPU backend target

Update the sample CPU backend to use structured data class (GPUTarget) following https://github.com/openai/triton/commit/5162346487b3e3ebc062d9697429bafad25f22f6.
Fixes #130

Yeah this is not ideal but is good to start. It's probably best to have a base class TritonTarget and let GPUTarget and CPUTarget inherit from it instead.


Update repo to require CMake 3.16

We have been requiring the use of CMake 3.16 for robotology repos since September 2020 (see https://github.com/robotology/robotology-superbuild/issues/462). Now it could be a good time to review this repo and requiring CMake 3.16 also here, as these will drastically simplify some boilerplate.

The first step after that is to finally accept https://github.com/robotology/how-to-export-cpp-library/pull/50 .


Write tests for the libraries

As we start to know what path this library will make, is it possible to write tests.
We should write integration tests to verify that stuff works the way we expect them to do.

Closed in #36


Auto center map

The map is oriented lefthand side, so the right side of the map is cropped on my mobile device:

Here the same on the dekstop:

Can you add an option to plot the map not oriented on the lefthand side but  oriented centered ?
Then artefacts all around are would be cropped, which are not of interest at all.
Where as now the most left outer artefact is used to start the lefthand side oriented plot and parts of the map on the right hand side are cropped.
Thank you.

I'm running into the same issue but my overspray artefacts are not equal on all sides so I think finding a center would not make sense in my case.
It would be helpful if I could just use crop to cut down the map and the resulting cropped square could then be centered in the container.


"++" operator not recognized in pre-increment position, (++myVar)

Challenge Name

https://www.freecodecamp.com/challenges/increment-a-number-with-javascript#?solution=
var myVar %3D 87%3B

%2F%2F Only change code below this line
myVar%2B%2B%3B

Issue Description

The test:
Use the ++ operator
fails when the pre-increment version of the "++" operator is used (i.e. "++myVar;") rather than the post-increment version.  Nothing in the description of the task precludes the pre-increment version being correct.
Browser Information

Browser Name, Version: various
Operating System: various
Mobile, Desktop, or Tablet: multiple

Your Code
// If relevant, paste all of your challenge code in here
var myVar = 87;

// Only change code below this line
++myVar;

Screenshot

@dmm10 thanks for the issue. Because we're targeting beginner coders, we wanted the challenges to be simple enough to teach a concept but not overwhelm learners with other details. Same goes for #8562.
cc/ @FreeCodeCamp/issue-moderators worth accepting this prefix operation?

I remember this was discussed earlier and more times in the chat, we could add separate challenges for pre/post increments/decrements.
But this is going to be confusing for beginners. Maybe we could relax the tests to allow pre but not change the test case message.
So advanced campers who are attempting these as in this case can pass the challenge and beginners are not overwhelmed as well.

@raisedadead I agree on just accepting the answer for flexibility but not change the test messages.
I think editing this line to the following will work. I'm not sure about escaping the parentheses but whoever takes this will have to check this.
assert(/([+]{2})?\\s*myVar\\s*([+]{2})?/.test(code)

The code will break for ++myVar++ anyways so I think the above should be okay.

@erictleung Won't the above code pass with just myVar?   Since both ++ sections are optional, that will match just myVar.
Better to do:
/([+]{2}\s*myVar|myVar\s*[+]{2}/

@SaintPeter good point. I was being careless 😄  Thanks!

Thanks ya’ll!  I couldn’t have asked for a better outcome.
Having poked your backend with alternate ‘correct' solutions to prior challenges and been impressed with your robust handling of them this situation stuck out a bit.  And I was hoping to contribute to the polish of your beautiful project.
I appreciate the peak behind the curtain.  It confirmed that the modification only required a simple regex edit and that ya’ll really are a copacetic group.
Kudos all around!

On May 11, 2016, at 12:39 PM, Eric Leung<EMAIL_ADDRESS>wrote:
@SaintPeter good point. I was being careless 😄 Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

var myVar = 11;
// Only change code below this line
myVar--;


Update readme to remove install from source

Fixes # .
Motivation
Update readme to remove install from source since we want to install from binary moving forward
Solution

Open questions

Pull Request Test Coverage Report for Build 4460

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

Totals

Change from base Build 4456:
0.0%

Covered Lines:
158

Relevant Lines:
188

💛  - Coveralls


[ONNX] Fix test_roi_align failure

RoiAlign-16 introduces coordinate_transformation_mode, which should be set to 'output_half_pixel' to omit the pixel shift for the input (for a backward-compatible behavior). This PR should fix the failure in https://ci.tlcpack.ai/job/docker-images-ci/job/docker-image-run-tests/231/testReport/junit/cython.tests.python.frontend.onnx/test_forward/Test___frontend__GPU_3_of_6___test_roi_align_cuda_/
Moved #12869 to an in-repo branch

Please review @driazati, thanks!


Example not working

The example for cogs is not working anymore.
Statcord stat posting error:
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/statcord/client.py", line 103, in _post_loop
    await self.post_stats()
  File "/usr/local/lib/python3.8/dist-packages/statcord/client.py", line 169, in post_stats
    raise Exception(f"Statcord server response status was not 200 OK:\n{await resp.text()}")
Exception: Statcord server response status was not 200 OK:
{"error":true,"message":"Custom 1 Parameter(s) are Missing or in the Wrong Format. (0). API Docs --> https://docs.statcord.com/#/api?id=post-bot-stats"}

First, I just tried this:
    def __init__(self, bot):
        self.bot = bot
        self.statcord_client = StatcordClient(bot, "statcord.com-TOKEN")

    def cog_unload(self):
        self.statcord_client.close()

After that I also tried this:
    def __init__(self, bot):
        self.bot = bot
        self.statcord_client = StatcordClient(bot, "statcord.com-TOKEN", self.custom_graph_1)

    def cog_unload(self):
        self.statcord_client.close()

    async def custom_graph_1(self):
        return 1 + 2 + 3

I believe I have fixed this in 1.5.2. Please let me know if there's still an issue.

Sadly it's still not working.

Yea, now its working. Thanks. :D


Max content length

to be merged after #34
this ensures that clients don't read more than 10 MB from payload

Hey, I've merged #34, can you rebase now and I'll get this in? Thanks!

@derekdowling rebased

👍


Add new threadpool executor with EWMA enabled

Is your feature request related to a problem? Please describe.
The executor of our new SiFi resizable search queue is OpenSearchThreadPoolExecutor which unfortunately does not keep track of queueSize/EWMA execution time.
Describe the solution you'd like
We should ideally Add EWMA enabled threadpool executor for search/write queue to keep track of EWMA time of each request.
Describe alternatives you've considered
n/a
Additional context
n/a

Could you please share what problem are we solving for OpenSearch with this issue? Is there a generic way to solve this issue for OpenSearch instead of just SiFi feature?

Hi @rguo-aws, can you please look into this?

Closing this issue, @rguo-aws feel free to reopen if this is still needed


update dev version to 0.8.6

Now release-candidate had 0.8.5, time to update dev to 0.8.6.

:+1:


SSH permission denied, VNC denied

Running Docker on Windows 10, started with
$ docker run -d -P --name android tracer0tong/android-emulator
docker ps shows the container and available ports just fine.
$ docker ps
CONTAINER ID        IMAGE                          COMMAND             CREATED             STATUS              PORTS                                                                                NAMES
a494f19f07a4        tracer0tong/android-emulator   "/entrypoint.sh"    32 minutes ago      Up 32 minutes       <IP_ADDRESS>:32772->22/tcp, <IP_ADDRESS>:32771->5037/tcp, <IP_ADDRESS>:32770->5554/tcp, <IP_ADDRESS>:32769->5555/tcp, <IP_ADDRESS>:32768->5900/tcp   android

When I try SSHing into it using user root it gives me "connection refused".
$ ssh<EMAIL_ADDRESS>-p 32722
ssh: connect to host <IP_ADDRESS> port 32722: Connection refused

Trying to connect via VNC (using RealVNC client and port 5900) it gives me "denied by remote host".

Ok, seems the SSH issue can be solved by not using <IP_ADDRESS> but the Docker IP instead.

You are right. There is a difference between Windows and Linux environments, Windows currently will run container in Oracle Virtual Box, that's why you should use another IP address.
Yury Leonychev

On 16 Oct 2016, at 06:17, Lars Schwarz<EMAIL_ADDRESS>wrote:
Ok, seems the SSH issue can be solved by not using <IP_ADDRESS> but the Docker IP instead.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks, still can't connect to the VNC server though, I see the process running, but when trying to connect with a VNC client it's always refusing the connection. Might be a similar issue with Docker on Windows though and I'll close this "issue".


ASP.NET Core metrics

ASP.NET Core metrics
Add info to What's new in ASP.NET Core 8.0
Not sure this is the best place.

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

ID: 90384677-5801-05d3-3f16-78392fc3ffea
Version Independent ID: 7ecc5731-9d3c-1f7a-9920-3ce2a37434d1
Content: Performance Diagnostics Tools
Content Source: aspnetcore/performance/diagnostic-tools.md
Product: aspnet-core
Technology: aspnetcore-performance
GitHub Login: @mjrousos
Microsoft Alias: riande

Associated WorkItem - 90669

@JamesNK I was thinking of documenting metrics in an include file, then adding the include to this perf doc, health doc, SignalR, and Kestrel. Or do you think each area needs a tailored doc?


Installation fails with yarn

First off, thank you for this library! It is so helpful and sometimes I ask myself, why many of the features are missing in core protractor.
We are currently switching to from npm to yarn. With yarn the installation of protractor-helper fails with following error:
error /Users/michl/dev/angular/node_modules/protractor-helpers: Command failed.
Exit code: 127
Command: webdriver-manager update --standalone --versions.chrome 2.28 --gecko false
Arguments:

Is there any trick how to install this package with yarn?
maybe somewhat related to #52

I get on TeamCity Exit code 1
EXEC error C: \BuildAgent\work\981498\project\node_modules\protractor-helpers: Command failed.
Exit code: 1
Command: webdriver-manager update --standalone --versions.chrome 2.28 --gecko false
Arguments:
Directory: C:\BuildAgent\work\981498\project\node_modules\protractor-helpers
Output:
'webdriver-manager' is not recognized as an internal or external command,
operable program or batch file.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
C:\BuildAgent\work\981498\project\Install\deploy.project.tasks(160, 2): error MSB3073: The command "yarn install" exited with code 1.

For what you need install-script in package.json?

Same thoughts, this is related to this issue #52
Running this script via yarn in our CI platform (gitlab) causes it to crash. Will be great to delegate updating the webdriver to the library using protractor-helpers.


Environment Configuration

I've been having problems configuring the environment according to your steps, can you please send your image package?

I've updated the env.yaml file, you can try conda env create -f env.yaml again. Recommended to use linux. If that doesn't work, install the package manually to resolve the dependencies

Thank you very much, the problem has been solved


Update jenkins version?

Hi there,
The Dockerfile is based on the latest long-term support (LTS) Jenkins build, so each time it builds you should get the current latest.

Yeah thanks, I asked without checkign the Dockerfile cause i thought it was using an old version 😄


wsk trigger update on a feed trigger calls the feed action again

is this intended new behavior?
 wsk trigger create f --feed /whisk.system/utils/echo

i create a trigger with a bogus feed (need to fix #134 seeing the activation result of the feed is annoying enough i might just fix it :)
then wsk trigger update f and wsk trigger get f both call the feed - it's not clear in the first that my trigger was updates, for example to add a parameter that i specified. Further, on the get, I don't see my trigger either.
OK granted this is pathological I called an action as the feed that isn't a feed - but is this the intended behavior especially for the get?

OK from inspecting the CLI code, an update and get are routed to the feed - not the openwhisk API host. That explains the observed results.

see also https://github.com/apache/incubator-openwhisk/pull/3318 for a related CLI change to wsk trigger update to now take the --feed parameter.


DateTime is returning wrong value

When I am trying to convert milliseconds to datetime object. It is returning wrong values -
DateTime dd = new DateTime.fromMicrosecondsSinceEpoch(millis);
debugPrint("dd -> $dd");
When I am passing -><PHONE_NUMBER>000
I am getting print like this -> dd -> 1970-01-19 10:06:31.260
If I convert same value form a online tool - I get 23 Oct 2019 - Which is correct.
Then why DateTime class is returning wrong value. If I am doing something wrong, please correct me.

Sorry, I found the issue - I have to use fromMillisecondsSinceEpoch instead of fromMicrosecondsSinceEpoch


Some crystals are too strong for this generator

Pack: Direwolf20110-1.5.1-1.10.2
Forge: <IP_ADDRESS>5
DeepResonance: 1.1x-1.4.5
Mcjtylib: 1.1x-2.3.6-SNAPSHOT
I have two 100x3 crystals and 3 generators and get the "some crystals are too strong for this generator" error and one crystal will not fire. Since both crystals are just under 20k output shouldn't this work or am I not understanding the documentation?

Yea I'm a derp.  I added a forth generator and problem solved.  Big difference in 10k and 100k max for each generator block.


Please don't suggest to use px values for font-size

Use rem for Global Sizing; Use em for Local Sizing -> YES!
But… html { font-size: 16px; } -> pleaaaase noooo! :-/
https://nicolas-hoizey.com/2016/03/people-don-t-change-the-default-16px-font-size-in-their-browser.html

@nhoizey thanks for the feedback. Your post is interesting. Are you advocating more for just html { font-size: 1em; } to let the default value be interited per browser?

@AllThingsSmitty thanks for reading it! ;-)
html { font-size: 1em; } is useless, as it is the default value. So I would advocate for nothing at all.
To be honest, I now use something a little more complex on the body to add some fluid responsiveness to my font-size, but I don't touch the html font-size at all and the value on body is meant to be equal to 1em or more, depending on viewport sizes.


VTK Extension: Tool buttons are not activated / deactivated

The buttons (Crosshair, Levels) are just calling onClick handlers currently:
https://github.com/OHIF/Viewers/blob/24a8d7bca2e8daa119da324fc46595418013892e/extensions/ohif-vtk-extension/src/ConnectedToolbarSection.js#L16-L20
This means that the active tool is never highlighted in the toolbar. It also means the expanded tool menu doesn't close when a button is clicked.
More generally, this is a problem of not knowing how to use something like setToolActive for the VTK Extension.

Getting there. No default button shows as active, but each updates well enough.


Add protocol to readme link

Current master readme directs you to:
https://github.com/rspec/rspec-rails/blob/master/rspec.info/blog/2016/07/rspec-3-5-has-been-released/#rails-support-for-rails-5
This PR readme will now direct you to:
http://rspec.info/blog/2016/07/rspec-3-5-has-been-released/

@JonRowe updated all http readme links 👍

Thanks :)


Stackblitz demo init not working

Describe the bug
Seems like there is an update on that breaks the demo app, and the overall package as well.
Maybe linked to https://github.com/dotansimha/graphql-code-generator/issues/8200 ?
Your Example Website or App
https://stackblitz.com/edit/jycouet-kitql-p7cpk8?file=README.md&title=KitQL Demo 2
Steps to Reproduce the Bug or Issue
Go to the demo init app
Expected behavior
I expect it to be working, I tried on local as well.
Screenshots or Videos
failed to load config from /home/projects/jycouet-kitql-fskbe2/vite.config.ts
error when starting dev server:
file:///home/projects/jycouet-kitql-fskbe2/node_modules/@graphql-codegen/cli/esm/codegen.js:1
import { DetailedError, normalizeOutputParam, normalizeInstanceOrArray, normalizeConfig, getCachedDocumentNodeFromSchema, isDetailedError, } from '@graphql-codegen/plugin-helpers';
         ^^^^^^^^^^^^^
SyntaxError: The requested module '@graphql-codegen/plugin-helpers' does not provide an export named 'DetailedError'
    at async loadConfigFromBundledFile (file:///home/projects/jycouet-kitql-fskbe2/node_modules/vite/dist/node/chunks/dep-0ebb5606.js:63560:21)
    at async loadConfigFromFile (file:///home/projects/jycouet-kitql-fskbe2/node_modules/vite/dist/node/chunks/dep-0ebb5606.js:63455:28)
    at async resolveConfig (file:///home/projects/jycouet-kitql-fskbe2/node_modules/vite/dist/node/chunks/dep-0ebb5606.js:63072:28)
    at async createServer (file:///home/projects/jycouet-kitql-fskbe2/node_modules/vite/dist/node/chunks/dep-0ebb5606.js:62138:20)
 ELIFECYCLE  Command failed with exit code 1.

Platform

OS: [e.g. macOS, Windows, Linux]
NodeJS: [e.g. 18.1.0]
@kitql/* version(s): [e.g. 2.5.1]

Additional context
No response

I'll fix it this week.
Thx for reporting 👍

Did you find a solution for this? I'm having the same issue...
` ERROR  The requested module '@graphql-codegen/plugin-helpers' does not provide an export named 'DetailedError'
import { DetailedError, createNoopProfiler } from '@graphql-codegen/plugin-helpers';`

Let me fix it now, give me a few minutes 👍

It should be fixed.
Try it again (make sure it's using "@kitql/all-in": "0.8.0-next.21")
Maybe you can try it locally with:
npx degit jycouet/kitql/examples/demo-init demo-init

I close the issue, if you have any issue please comment / re open / open a new ticket. 👍


deno.land/std website is often throwing 500 errors when retrieving dependencies

I have a Github actions script that runs a Deno command every five minutes. The most frequent cause of failure is a 500 error being generated by the web server serving up https://deno.land/std.... It would be great if the source of these errors could be determined and the issue fixed. I've seen this error at least 20 times in the last week in script logging (and I'm sure I'm not alone).
It doesn't seem to be for any particular file. And it has also been observed for https://deno.land/x dependencies.
error: Import<EMAIL_ADDRESS>failed: 500 Internal Server Error
    at https://deno.land/std@0.98.0/fs/mod.ts:2:0
Error: Process completed with exit code 1.

Thanks

We have put in place a temporary mitigation for the issue. I'll get back to you with more info and a permanent fix once it is ready.

This should now be resolved fully. If you encounter any 5xx class errors in the future please re-open the issue! Thanks for the report :-)

Hi @lucacasonato, We've been regular 500 errors today. Examples include:
error: Import<EMAIL_ADDRESS>failed: 500 Internal Server Error
    at https://deno.land/std@0.162.0/fs/expand_glob.ts:11:32

error: Import<EMAIL_ADDRESS>failed: 500 Internal Server Error
    at https://deno.land/std@0.119.0/node/fs.ts:4:34

error: Import<EMAIL_ADDRESS>failed: 500 Internal Server Error
    at https://deno.land/std@0.119.0/node/_errors.ts:16:45

Retrying often fixes the problem


adding common library logging config functionality

looking into travis build fail now

@andrewmil see my comments for what should be the fix


Missing classification

Owners: @Shopify/operational-excellence (This repo has no owners)
Service: graphql-js-client/production
Please set a classification.
You can add it in service.yml, under the key classification.
Refer to the detailed documentation for more information.

Classification check "Missing classification" passed.


define ClusterInfo as a CRD

Reduce gap between single-cluster and multi-cluster branches by porting ClusterInfo CRD and controller to the single-cluster branch. Modify test suite to obtain cluster capacity from ClusterInfo CRD.

PR looks good. As a next step, we should probably replace the use of the default namespace for the clusterinfo object with the mcad-controller namespace.


Add discord and bitbucket provider

Description
This pull request adds support for 🗝Discord and 🗝Bitbucket single sign-on (SSO) providers to fastapi-sso. These additions extend authentication options. I added it because I personally needed it 😊, and I think it will be useful to others.
Changes Made

Discord SSO Provider: A new integration has been added to allow users to sign by using their Discord accounts. See fastapi_sso/sso/discord.py

Atlassian Bitbucket SSO Provider: A new integration has been added to allow users to sign by using their BitBucket accounts. See fastapi_sso/sso/bitbucket.py

Tests Performed

[x] Pytest 👌
[x] Personal integration test 👌

Typechecking

[x] black 👌
[x] pylint 👌
[x] mypy 👌
[x] pyclean 👌
[x] pre-commit 👌

Exemples

[x] MKDocs has been updated 👍

Examples of use have been added :

Discord exemples/discord.py
BitBucket exemples/bitbucket.py

I think I've applied the changes correctly, but please let me know if you need any further changes.

It looks perfect now, could you please try rebasing from the master? I switched to ruff, I hope it won't break anything, but the pipeline should now pass.
git checkout master
git pull
git checkout -
git rebase master

That should do it. There will probably be some conflicts, I can help you resolve them if you have any troubles.

I felt bad for requesting so many changes, so I went ahead and created a PR into your PR https://github.com/afi-dev/fastapi-sso/pull/2 😁  it's all rebased, you only have to accept all incoming to resolve the conflicts. If you'd like to resolve yourself, feel free to close the pull request.

no problem

Hey @afi-dev, this PR turned out to be such a failure on my side, I am so, so much sorry. If this is still relevant, is it ok if I take your commits so that you can keep the contribution, rebase it with latest changes and remove the changes to the docs generation as it's not relevant to Discord and Bitbucket?


Color fix needed?

That's how it looks in my terminal, can read the text only after highlighting them!
The color of the text is gray!

+1

+1

+1
But after 30min of reading these related issues I think it might be the time to ditch Solarized Darko for iTem2.

https://github.com/chriskempson/base16-iterm2

+1

Does the base16 Solarized Dark theme have the same issues?

It's fine with base16 because that makes use of extended colors.  You have
to do a little more work with base16-shell to get everything set up.

Cool, thanks for the quick answer.

@hydrozen 👍👍👍

+1 contrast fix

It is worth noting that Ubuntu 15.04 now officially includes Solarized in the standard selections for gnome-terminal color schemes. If you use Chalk or other themes that are not Solarized compatible you are choosing to be incompatible with a standard Ubuntu theme at this point. My experience has been only Chalk applications are subject to this issue. I have never encountered even an old app that uses terminal colors that is as broken as most Chalk applications.

htop
Am 24.08.2015 4:40 nachm. schrieb "Rob Muhlestein"<EMAIL_ADDRESS>
:

It is worth noting that Ubuntu 15.04 now officially includes Solarized in
the standard selections for gnome-terminal color schemes. If you use Chalk
or other themes that are not Solarized compatible you are choosing to be
incompatible with a standard Ubuntu theme at this point. My experience has
been only Chalk applications are subject to this issue. I have never
encountered even an old app that uses terminal colors that is as broken as
most Chalk applications.
—
Reply to this email directly or view it on GitHub
https://github.com/altercation/solarized/issues/220#issuecomment-134229185
.

Thanks @iamolivinius haven't been using htop (mostly because I believe in teaching students tools that have more ubiquity, like top and sar, etc.) but this is indeed unfortunate. But I certainly won't be missing it.

As a heads up to people using iTerm2 that have this issue and don't want to modify the theme, you can bump up the contrast about 10-20% in your terminal colour profile settings, and it made the invisible text readable without making a mess of the other colours.  It does get a little brighter looking, but it's reasonable.

+1 I'm having the same issue.

3 years after...
I fixed this using base16-solarized.dark.itermcolors found here : https://github.com/chriskempson/base16-iterm2
It use extended color : https://github.com/altercation/solarized/issues/220#issuecomment-95610140

I'm using Ubuntu 15.04, Guake Terminal and Terminator with Solarized Dark.
htop, goaccess and many another program has this issue.
I read whole thread but don't see how to fix this. And this thread was 3 years ago

+1 with gulp

In Mac OS X Terminal Preferences > Profiles > Solarized Dark, just manually change the Color Ansi (8) to whatever best suits your eyes.

:+1:

having similar issues with gnome-terminal on ubuntu 15.10

having the same issue with htop, any progress?

It's quite obvious this project is abandoned, last commit is from 2013.

Hi @desbma , it there any alternative project? I want to use it with MinTTY.

@altercation do you have any opinion on this? I'm a huge fan of solarized, but with this issue still open, I can't really justify recommending it to friends atm.
If you don't have the time to work on this anymore, maybe try to find a new maintainer?

@nicolasmccurdy thanks for the tip.  just spent 30 mins trying to get that to work. (similarly awful salmonesque results in vim)

@nicolasmccurdy thanks for the tip.  just spent 30 mins trying to get that to work. (similarly awful salmonesque results in vim)

Huh, so does their solarized theme seem to be a good alternative in general?

Huh, so does their solarized theme seem to be a good alternative in general?

I can't get base16 to work in my terminal in the first place, so I can't say.  Looks great on the site!

I can't get base16 to work in my terminal in the first place, so I can't say.  Looks great on the site!

i have come to see this problem as un-solvable: the solarized color scheme is a hack, and it changes colors fundamentally in a way that breaks applications that expect certain colors to have certain contrast, by using weird background colors (e.g. iptraf). most of the time, i can switch color schemes in those to be simpler (e.g. B&W in iptraf), so i have mostly given up on this issue.

i have come to see this problem as un-solvable: the solarized color scheme is a hack, and it changes colors fundamentally in a way that breaks applications that expect certain colors to have certain contrast, by using weird background colors (e.g. iptraf). most of the time, i can switch color schemes in those to be simpler (e.g. B&W in iptraf), so i have mostly given up on this issue.

+1 for adjusting contrast in iterm2 to solve this

@dfournier You can still see the changes at #240.
I solved this (at least among the tools I use) on urxvt by mapping color8 to base01 in my Xresources and tweaking the vim theme (pgarrison/vim-colors-solarized).

If someone is still having trouble with this issue and since hydrozen's fork mentioned that fixes this is no longer available (the user is shown as @ghost now and the old https://github.com/hydrozen/solarized repo is gone), you might want to try the Solarized Dark Patched theme available in https://github.com/mbadolato/iTerm2-Color-Schemes . Just download this file, enter your iTerm preferences -> Profiles -> Colors -> Load Presets... -> Import... and select & activate the theme. That's how I managed to solve the issue for myself.

I moved to https://github.com/chriskempson/base16 with solorized theme.
Works perfectly!
On Wed, Mar 2, 2016 at 10:39 PM Dražen Lučanin<EMAIL_ADDRESS>wrote:

If someone is still having trouble with this issue and since hydrozen's
fork mentioned that fixes this is no longer available (the user is shown as
@ghost https://github.com/ghost now and the old
https://github.com/hydrozen/solarized repo is gone), you might want to
try the Solarized Dark Patched theme available in
https://github.com/mbadolato/iTerm2-Color-Schemes . Just download this
file
https://raw.githubusercontent.com/mbadolato/iTerm2-Color-Schemes/master/schemes/Solarized Dark - Patched.itermcolors,
enter your iTerm preferences -> Profiles -> Colors -> Load Presets... ->
Import... and select & activate the theme. That's how I managed to solve
the issue for myself.
—
Reply to this email directly or view it on GitHub
https://github.com/altercation/solarized/issues/220#issuecomment-191444392
.

+1 for fixing this issue if possible. @cuviper's solution of switching which ANSI codes base02 base03 are mapped to seems the most reasonable.
I think Solarized works so well because of the careful colour choices, so any solution that changes the colours would not be the best, IMO. Switching the ANSI mappings would mean that any theme/colourscheme in a cli program that counts on the Solarized theme to be the terminal colours would have to be adjusted, sure.
But compared to the alternatives (choose a new colour that doesn't fit the thinking, change to a different colour scheme, change the whole contrast of the scheme, or just lose users over time) it's reasonable: it's a small change for "downstream" themes, and at least brblack will be visible.
Sure, one can argue that others shouldn't be putting important information in brblack, but it should at least be visible.
It's a non-backwards compatible-change, sure. These things happen in software projects. Let's handle it.

@hemanth Can you please change the name of this issue? (Clearly, the fix is required. That would help in avoiding opening of new issues. I searched for the issue that I was facing and ended up opening #351 , until @zeorin informed me that the issue has already been reported.) Thanks!
And yeah, +1 for fixing this issue.

for the record, i have mostly alleviated the problems here by switching from xterm to rxvt-unicode...

Hi all,
I don’t think that, outside of forking this project ourselves, this issue or any other issues are likely to be fixed.
So that’s exactly what I’m proposing: let’s create a community-maintained fork!
I’ve created a new issue for the discussion of this idea: https://github.com/altercation/solarized/issues/355
Please do chime in.

Left hand side is OSX terminal app and left hand side is iterm2. Font size and color scheme is same..
But color and font rendering is different. Any idea how i can fix OSX terminal color rendering.

Here's a simple command to reproduce this:
if you have your terminal set up with the Solarized theme, just do
echo -e 'Normal text. \033[0;30mInvisible text'

The text "Invisible text" will not be visible.

i love this theme but it has been 4 years since this issue was created, are there any plans to fix this?

@altercation Its been 4 years since you said you would take a look. Has there been any progress on this, or should we abandon solarized dark?

hey @tyrsius, the possibility of this being fixed is remote. You should try using base16, I switched to that when I first encountered this issue and it's been pretty good. :smile:

@ivanmanolov90's solutions seems to works perfectly for me - at least with the tools I use.

I "fixed" this by setting iTerm2's black light colour to 0387ad (as suggested by @ivanmanolov90 ) and then using the solarized-8 theme with the following settings:
set background=dark
let g:solarized_use16 = 1
let g:solarized_termtrans = 1
colorscheme solarized8

I know this won't work for everyone but if your setup is like mine then maybe it'll save you 15 minutes of frustration.

I copied over the colors from gnome-terminal to iTerm2 and created a new color scheme. here. I tried using both base16 and base256 for iTerm2, but the first one simply didn't show some text and the second one had a weird problem where the difference between folders and files was very small (they were both white and not noticeable at all)
I have been using this for a couple days now, and it's not shown up any problems. I will report here if it does.
P.S the copying over process was pretty simple and you should absolutely do it if you have a preferred setup on a computer that works fine and want to emulate the setup elsewhere. (Not sure if this was already discussed here, I was unable to find it anywhere) :thinking:

I copied over the colors from gnome-terminal to iTerm2 and created a new color scheme. here.

I see. You're using the normal colors as the bright ones. I'm going to do the same. At least, the bright colors are not grey anymore.

Mocha is pretty much the only tool where this issue still occurs


DataLayer.reloadData() connects my end points

I have a data set of points drawn using a FlowLine (extended from DataLayer). It draws just fine. When I call the reloadData() function it redraws all the data but it also connects my end points.
I've whipped up a quick jsFiddle to show what I mean. http://jsfiddle.net/wzfp1mLh/ Click the "DataLayer.reloadData()" text above the map.
I should note: I am not actually that interested in calling reloadData() directly. Originally I was using the DataLayer.setDisplayOption() function to alter the opacity of the layer, which in turn calls reloadData().  When the extra line segment appeared, I took steps to isolate the cause.

I've only just gotten back to working on this again. I can confirm the problem is gone. Thanks.


Cannot update SAML Inbound authentication configs

Add a SAML SP and Save
Then on the saved SP, click 'Edit' on SAML inbound configs. This does not load the complete configuration that we configured previously. 'Issuer' is missing
Try to re-save the config. That will give an error as the 'Issuer' is missing.
This blocks updating SAML SP inbound configurations

Labels:
Affected/5.7.0-Alpha

Tested with 5.7.0-alfa3 and scenario works fine. So resolving the issue.


TimeSeries: move ui_add_event to ui_edit_events

[X] History of event names to add
[X] Add event with another name
[X] Remove event
[ ] Remove all events of a given name --> switch to a button dialog with all current event names
[X] Move event
[X] Save and close
[X] Cancel

 test

I won't do the point
[ ] Remove all events of a given name --> switch to a button dialog with all current event names
for now, so that I can release 0.6 faster.


Allow for only one storage extra

Fixes #14
Successfully starts up if only mongo is installed.
Should there be a check and raise if neither is installed? I thought about writing a test that deletes modules to test this. But I don't know how to do that gracefully.

Codecov Report

Merging #15 into master will decrease coverage by 0.18%.
The diff coverage is 66.66%.

@@             Coverage Diff             @@
##            master      #15      +/-   ##
===========================================
- Coverage   100.00%   99.81%   -0.19%
===========================================
  Files           89       89
  Lines         2201     2211      +10
===========================================
+ Hits          2201     2207       +6
- Misses           0        4       +4

Impacted Files
Coverage Δ

py_abac/storage/__init__.py
66.66% <66.66%> (-33.34%)
:arrow_down:

Continue to review full report at Codecov.

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

I think that wont be necessary. Once the 4.0RC branch is merged, will keep the In-Memory as default storage. That way there will always be at least one type of storage imported.

Just curious, this whole PR won't be required, or the check that at least one storage extra got imported?
Also, am I reading this right, that currently mongo/sql will get get queried for a list of policies every time that PDP.is_allowed() is call, but the in-memory might be able to cache them? That would be cool.
I'm playing around with this stuff partly to get some experience with ABAC and to evaluate it as my implementation for a web portal, so a mongo query on every single is_allowed on every resource seems like a lot.

Sorry for not being clear earlier, I meant the check for at least one storage.
Regarding reduce database queries, there is a plan to add storage cache as a feature. The in-memory storage does not act as a cache. Its just there in case the user does not want to use a database for storage. The plan is to add a redis storage and a general purpose cache for policy caching in the subsequent releases. However, due to being preoccupied with another project, currently I am not able to spend much time on it.
If you would like to get the cache into the next release I can give that priority. The plan is to support three types of caches:

in-memory cache: policies are caches in-memory.

This type of cache is perfect for deployment where single instance of the application is running. It can not be used for multiple replica deployments like in microservice on kubernetes. To get rid of this limitation the following will be supported:

memcache cache: this a cache which uses memcache as backend.
redis cache: this cache uses redis as backend.

The storage-cache branch will be used for the implementation. Any contributions and feedback are welcomed! :)

I think the travis CI build is failing for python 3.5. It seams to be cause due to some pytest version issues. Can you have a look. I think the error says version 4.6 or greater is required. Thanks!

Oops, I was going to take a look, but didn't get around to it. But I see you merged, so it resolved maybe. Cool. I really like what you're doing with this project, and am prototyping its use this week. Cheers!


Include almost all source files in sdist (#482)

Presently, the sdist published to pypi cannot be built (failing as shown in #482), because hatch-fancy-pypi-readme is configured to read README.md but that file is not included in the source distribution.
Here we make the sdist almost all-inclusive (differing from git archive output only by leaving out only hidden files and directories -- presently .github, .devcontainers, .python-version, and .stats.yml).

Thanks for the suggestion/contribution, we'll take a look!

thanks for the suggestion/contribution, this change has been incorporated!


fix: avoid showing git error on startup

Even though execSync is wrapped in a try...catch it will still output error messages in docker console.
This will introduce a check for the git command in the shell, if it doesn't exist, then exit 0 else run the git command. This avoids throwing and catching errors and keeps the logs clean.

@zwave-js-bot fix lint

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 6 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 26.978%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/utils.ts
0
6
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
3651

Relevant Lines:
14263

💛  - Coveralls


Code for finetuning the instruct models

@loubnabnl The code for finetuning the base models are provided in this repo. However, I guess that the same code cannot be used for finetuning the instruct models. So, please provide code for finetuning the instruct models.

Hi, the current script should work if you change the dataset content column with properly formatted with ChatML conversations. But we added the recipe for building SmolLM2 instruct  with the alignment handbook here https://github.com/huggingface/alignment-handbook/blob/main/recipes/smollm2/README.md


⚠ Bofrost.fr has degraded performance

In 806a234, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1286 ms

Resolved: Bofrost.fr performance has improved in 3829997 after 6 minutes.


Enable 'no-implicit-dependencies' lint rule

It checks that everything we depend on has a package.json entry.


Map editor

Map editor tool to create new maps.
The first tool prototype can be a simple bitmap editor allowing the user to create the files required for a battle map:
base.png
feature.png
height.png
place_name.png
with the brush colours restricted to the RGB code matching each terrain type.

Other files with relevant data for maps are:
info_en.csv
music_event.csv
source_en.csv
eventlog_en.csv
map_event.csv
troop_pos.csv
weather.csv
As seen in example:
data/module/historical/map/preset/atestmap
https://github.com/remance/Masendor/tree/main/data/module/historical/map/preset/atestmap

This pygame project is quite similar to what you suggest: https://www.pygame.org/project/5157/8204
I haven't test it yet myself though. But it may help us formulate our own tool.

Wow 😳
Based on Deluxe Paint. A classic
It's going to be interesting having a look.
I already started writing a prototype with pyqt, I know the game engine is based on pygame but I considered it would be easier for me to just make the editor app with qt.
As soon as I get a minimal working version I will upload it for you to review

Looking forward to it.


Setter not working

You need to use the this qualifier to reference the dbType field otherwise dbType references the method argument.

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


Enum default starts @1

Search Terms
enum
compiler options
enum default
Suggestion
I would like enum default behavior to start assignment at numeric 1, or to enumerate by default with strings.
Default behavior currently has an enum start at numeric 0, this causes a danger for lazy existence checking.
Use Cases
Although perhaps a personal preference, I think enum's should function as purely logical constructs, their actual value being effectively irrelevant, they are ideally used to make an assignment From an enum to some variable or property, and later using it to do a conditionalized check against the enum definition (not directly checking based on the internal value it is compiled to)  This works fine as-is with direct identity checks.  This has a vulnerability for existence checks.
Specifically performing a fuzzy existence check on a variable containing an optional enum will result in a failed check if the enum happens to internally be assigned as 0.
Examples
enum Dangerous {
Hi,
There
}
// internally Hi has been assigned 0, and There has been assigned 1
let test: Dangerous | undefined = Dangerous.Hi
.
.  // do some stuff
.
if(test) {
console.log('Success, there is an enum present!  test: ', test);
} else {
console.log('Failure, there is not an enum present!  test: ', test);
}
// Console Output:
//       Failure, there is not an enum present!  test: 0
// Running the same example with test = Dangerous.There
//  Console Output:
//       Success, there is an enum present!  test: 1
//  Running the same example with test = undefined
// Console Output:
//       Failure, there is not an enum present!  test: undefined
I would like both cases where an enum is in-fact  present, to succeed the fuzzy existence check.
Checklist
[X] This wouldn't be a breaking change in existing TypeScript/JavaScript code
Adding a compiler option would allow users to set project wide a default behavior for assigning enums, stopping this potentially dangerous javascript quirk.
--EnumStartsAtOne
have all enums (unless directly defined by the proggrammer) start assignment at 1.
or
--EnumString
set enum values as the string version of the numeric's typically assigned.   enum Ex { First: '0', Second: '1' }
or the compiler could assign the key as the value in those cases:  enum Ex { First: 'First', Second: 'Second' }
Some flavor of those changes, so an enum never is assigned 0 ( unless directly assigned by the programmer )
[X] This wouldn't change the runtime behavior of existing JavaScript code
Going the compiler option route would avoid possible conflicts.
[X] This could be implemented without emitting different JS based on the types of the expressions
Yes

[X] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)

Yes

[X] This feature would agree with the rest of TypeScript's Design Goals.

I believe this is inline with those goals in that it will reduce the likely hood of failures due to existence checks.
If this could be made the initialization behavior by default I think that would be even better, but then it would cause a breaking change, so it seems like a compiler option is the best route.
Thank you for your consideration.

As an aside, I know this can be achieved by assigning a value to the first enum, ie.
enum SlightlyAnnoying = {
Stuff = 1,
Things,
Blorp,
Blip
}
however this seems not ideal to have to manually define a value to all your enums to avoid the potential pitfall.

I think enum's should function as purely logical constructs, their actual value being effectively irrelevant

That's nice, but a bit divorced from the actual reality of things; code can and will do whatever it wants to in the js runtime, and the default numbering order of enums is something that's been guaranteed by Typescript since the bronze age.
So --enumStartsAtOne would be difficult or impossible to use if you're bringing anyone else's library code into your compilation; you could break a library's assumption about enum values.
On the other hand, this sounds like an ideal case for a linter rule:  You could have any enum without a value specified for the first member cause a lint warning or error, and then you could apply that to your code as your preference, and others could choose not to use that linter warning if they wanted to.

Thanks for the idea, however this is not possible to do without breaking a lot of code, as @nmain mentioned.
It also would break the expectations for people coming from other languages because all the languages I can think of have enums which start at 0.

So --enumStartsAtOne would be difficult or impossible to use if you're bringing anyone else's library code into your compilation; you could break a library's assumption about enum values.

I was unaware typescript applied a local projects TS compile settings onto npm dependencies found in node_modules.  Changing that behavior even as a compile option would indeed be very dangerous.


[Autopilot] approval for rule pgbench/test1-pvc-052bb032-2625-4c79-a48a-bc763d40ad83 rule: test1

This is a request to approve an autopilot action. The request was triggered based on an AutopilotRule test1 defined in your cluster.
What actions will be taken
Action: resize

Params: map[scalepercentage:1]

ExpectedResult
PVC will resize from 10Gi to<PHONE_NUMBER>2
What objects will get affected

PersistentVolumeClaim pgbench/pgbench-data (pvc-052bb032-2625-4c79-a48a-bc763d40ad83)

Object Owner(s):

ReplicaSet pgbench-57f9d94764

How do I approve
Once you review the above,

To approve, simply approve and merge this PR
To declined, close the PR

Autopilot will be watching for the merged specs in the cluster and will proceed with the action if approved and declined the action if not.

[Autopilot] Closing PR as the approval pgbench/test1-pvc-052bb032-2625-4c79-a48a-bc763d40ad83 was cancelled in the Kubernetes cluster. The cancellation could be due to one of below reasons:

The approval state was change by a user in the spec to cancelled.
The rule conditions for the object: pvc-052bb032-2625-4c79-a48a-bc763d40ad83 are no longer met in the cluster..


cranelift: Support derived instruction builder functions

Add the InsnBuilderDerived trait as a place to add new derived builder functions. Also add support in the instruction specificaton dsl for marking instructions as raw, which indicates that their generated builder function is not meant to be the primary interface for generating that instruction.
As an example, port over the jump builder function with an implementation that dispatches directly to the raw jump_ builder function. This functionality will greatly simplify landing #5464, where the largest portion of the diff is due to the arguments of the jump builder function changing.

So this version requires anyone who wants to call the jump method to explicitly use the new InstBuilderDerived trait, right? We talked about trying to name the new trait InstBuilder and rename the auto-generated trait to something else. I take it that didn't work out?
How about this: make the generated inst_builder.rs contain only the method definitions, not the surrounding pub trait InstBuilder etc. Move the pub trait declaration to the hand-written code, so it can have hand-written methods in it, and stick include!(concat!(env!("OUT_DIR"), "/inst_builder.rs")); into the middle of that trait so all the generated methods are also in the same trait. Does that make this diff smaller?

So this version requires anyone who wants to call the jump method to explicitly use the new InstBuilderDerived trait, right? We talked about trying to name the new trait InstBuilder and rename the auto-generated trait to something else. I take it that didn't work out?

I found that the previous InstBuilder methods weren't available without importing it as well, which led me to give the derived functions a new name.

How about this: make the generated inst_builder.rs contain only the method definitions, not the surrounding pub trait InstBuilder etc. Move the pub trait declaration to the hand-written code, so it can have hand-written methods in it, and stick include!(concat!(env!("OUT_DIR"), "/inst_builder.rs")); into the middle of that trait so all the generated methods are also in the same trait. Does that make this diff smaller?

I think that would make the diff smaller! The downside would be that we'd still need to manually port the documentation over for any hand-written wrappers, as the docs wouldn't be automatically generated for them. I think I'm going to try a different approach, and generate the version of the builder that takes the Block and &[Value] arguments separately back on #5464. That will avoid any issues of documentation sync, and should make it easy enough to generate multiple versions of the same function if necessary.


Handle string and boolean pointers in attributes

I took a stab at adding support for this, but let me know if this isn't the behavior we'd expect from the API.
Resolves #647

LGTM, thanks for that.


single cycle nop insertion on jump/branch commands

single cycle nop insertion is not possible, double cycles insertion is possible, already implemented.


[BUG] fvm flutter does'nt work

macOS 15.6
fvm version: 1.2.3

You have to "use" first and you should run inside your project folder

You have to "use" first and you should run inside your project folder

But I just want to upgrade flutter :/

@tbm98 this seems to be related to https://github.com/leoafarias/fvm/issues/129
Can you provide the global path that you configured?

the global path is
/Users/tbm98/fvm/default/bin/flutter

in ~/fvm/versions don't have /bin folder

Default is a symlink.. can you tell me which flutter version the /default/ folder points to?
On Sep 18, 2020, 9:37 AM -0600, tbm98<EMAIL_ADDRESS>wrote:

the global path is
/Users/tbm98/fvm/default/bin/flutter
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Versions should have the cached versions. If there is nothing there  it means no flutter version was installed through fvm
On Sep 18, 2020, 9:40 AM -0600, tbm98<EMAIL_ADDRESS>wrote:

in ~/fvm/versions don't have /bin folder
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Default is a symlink.. can you tell me which flutter version the /default/ folder points to?
…
On Sep 18, 2020, 9:37 AM -0600, tbm98 @.***>, wrote: the global path is /Users/tbm98/fvm/default/bin/flutter — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

it is tbm98/fvm/versions/beta

Versions should have the cached versions. If there is nothing there  it means no flutter version was installed through fvm
…
On Sep 18, 2020, 9:40 AM -0600, tbm98 @.***>, wrote: in ~/fvm/versions don't have /bin folder — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

Yes. I just use fvm to use global, never try install to flutter project before.

@tbm98 please test the latest prerelease version https://pub.dev/packages/fvm/versions/1.3.0-dev.0
pub global activate fvm 1.3.0-dev.0

I believe the release provides better information on what is going on. I will go ahead and close this issue. But please let me know if the issue persists and I will reopen it.

Thank you! It worked well :D


Add support for Gutenberg

If Gutenberg is detected, make sure Analysis still works.

See commit: https://github.com/nextbuzz/buzz-seo/commit/af6dec038f14798c7971a2d8cfe3c6a4dfaa207a


Issue running any awslocal command on Windows 10

This is the response for any awslocal command.

Same here. Had no chance to solve it so far...any ideas?

Thanks for reporting @mgrantsmarsh @skergetd . We've pushed a new version that should hopefully fix this issue. Can you please do a pip install --upgrade --no-cache 'awscli-local>=0.6' and give it another try? Thanks

Hi all, I'm also having issues using the tool, would like to give you more information but I see only error code in my case

Bat script on windows needs to turn echoing off, example:
@echo off
python %~dp0\awslocal %*

Thanks @4lex1v - can you please elaborate why this is required? Closing this issue, as we haven't heard about this issue in a while. If the issue persists, please feel free to open a pull request.. Thanks for your help!