setting AZCOPY_JOB_PLAN_LOCATION causes autologin to be skipped or fail

Which version of the AzCopy was used?
AzCopy 10.16.1
Note: The version is visible when running AzCopy without any argument
Which platform are you using? (ex: Windows, Mac, Linux)
Windows
What command did you run?
azcopy.exe sync frompath topath --recersive
Note: Please remove the SAS to avoid exposing your credentials. If you cannot remember the exact command, please retrieve it from the beginning of the log file.
What problem was encountered?
I am using Autologin with SPN as well as setting the AZCOPY_LOG_LOCATION and AZCOPY_JOB_PLAN_LOCATION. The result of this is is the error "Cannot perform sync due to error: no cached token found, please log in with azcopy's login command"
the expected result is the sync will complete without error.
How can we reproduce the problem in the simplest way?
     try { $auth = ($endpoint | ConvertFrom-Json -ErrorAction SilentlyContinue).Auth.parameters; } catch {}; # ConvertFrom-Json does not honor -ErrorAction SilentlyContiune so need to use try catch
      if ($auth -and $auth.authenticationType -eq "spnKey")
      {
        Write-Output '*** set service principal auto login ***';
        $env:AZCOPY_AUTO_LOGIN_TYPE = 'SPN';
        $env:AZCOPY_SPA_APPLICATION_ID = $auth.serviceprincipalid;
        $env:AZCOPY_SPA_CLIENT_SECRET = $auth.serviceprincipalkey;
      }

       if ($azCopyLogPath)
      {
        Remove-Item $azCopyLogPath -Recurse -ErrorAction SilentlyContinue; # ensure the log folder is empty. this ensures only logs for this run are published in the pipeline.
        $env:AZCOPY_LOG_LOCATION = $azCopyLogPath;
        $env:AZCOPY_JOB_PLAN_LOCATION = Join-Path $azCopyLogPath 'plans';
        New-Item -Path $env:AZCOPY_JOB_PLAN_LOCATION -ItemType Directory;
      }
 $containers | ForEach-Object{
      Invoke-Expression "$azcopypath sync `"https://$sourceStorageAccount.blob.core.windows.net/$_`" `"https://$targetStorageAccount.blob.core.windows.net/$_`" --recursive --delete-destination $deleteDestination";
}

Have you found a mitigation/solution?
If I don't set  AZCOPY_JOB_PLAN_LOCATION the command login works as expected and syncs each container. As soon as i set the plan location it is like Auto_Login is skipped.

Not being able to send AZCOPY_JOB_PLAN_LOCATION will cause files to be created outside the DevOps working directory and could cause the drive to fill up.

I just realized I somehow removed setting AZCOPY_TENANT_ID so autologin was never working it must have been getting the last token from the plans folder.


Fix SSL Breakage

Fixes the fact that SSL throws a ton of errors and doesn't work, by commenting out default truststore expecation (the file doesn't exist).

Nevermind, I'll do something else.


Autorest Regen Preview to 3.0.0-alpha.20230728.2 by Dapeng Zhang from refs/pull/3354/merge

Triggered from https://github.com/Azure/autorest.csharp/pull/3354

API change check
API changes are not detected in this pull request.

We will also need to manually add back this method for openai since its already part of the public api surface so it doesn't change.
This can be done in the real regen PR.


[custom UCP] LCZ parameter error

Hi,
I'm trying to run w2w with a custom UCP table but run into the following issue.
w2w -n 25 ./ lcz_wudapt_ponche.tif geo_em.d04.nc --lcz-ucp custom_lcz_ucp.csv
--> Set data, arguments and files
> Seems you are using a LCZ map produced by https://lcz-generator.rub.de/
> I therefor use the Gaussian filtered default 'LczFilter' layer
--> Check LCZ integrity, in terms of class labels, projection and extent
> LCZ labels as expected (1 to 17)
ERROR 1: PROJ: proj_identify: Cannot find proj.db
ERROR 1: PROJ: proj_identify: Cannot find proj.db
> LCZ provided as WGS84 (EPSG:4326)
> LCZ domain is covering WRF domain
ERROR 1: PROJ: proj_create_from_name: Cannot find proj.db
--> Replace WRF MODIS urban LC with surrounding natural LC
Looping through urban grid pixels: 100%|█████████████████████████████████████████████████████████████████████████████████████████████████| 197/197 [00:01<00:00, 136.76it/s]
--> Create LCZ-based geo_em file
Traceback (most recent call last):
  File "/home2020/home/geo/fbreton/w2w_netcdf3/lib/python3.8/site-packages/pandas/core/indexes/base.py", line 3621, in get_loc
    return self._engine.get_loc(casted_key)
  File "pandas/_libs/index.pyx", line 136, in pandas._libs.index.IndexEngine.get_loc
  File "pandas/_libs/index.pyx", line 163, in pandas._libs.index.IndexEngine.get_loc
  File "pandas/_libs/hashtable_class_helper.pxi", line 5198, in pandas._libs.hashtable.PyObjectHashTable.get_item
  File "pandas/_libs/hashtable_class_helper.pxi", line 5206, in pandas._libs.hashtable.PyObjectHashTable.get_item
KeyError: 'MH_URB2D'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
  File "/home2020/home/geo/fbreton/w2w_netcdf3/bin/w2w", line 8, in <module>
    sys.exit(main())
  File "/home2020/home/geo/fbreton/w2w_netcdf3/lib/python3.8/site-packages/w2w/w2w.py", line 179, in main
    nbui_max = create_lcz_params_file(
  File "/home2020/home/geo/fbreton/w2w_netcdf3/lib/python3.8/site-packages/w2w/w2w.py", line 1226, in create_lcz_params_file
    dst_data = _add_frc_lu_index_2_wrf(
  File "/home2020/home/geo/fbreton/w2w_netcdf3/lib/python3.8/site-packages/w2w/w2w.py", line 1152, in _add_frc_lu_index_2_wrf
    frc_urb = _ucp_resampler(
  File "/home2020/home/geo/fbreton/w2w_netcdf3/lib/python3.8/site-packages/w2w/w2w.py", line 678, in _ucp_resampler
    SW, BW = _get_SW_BW(ucp_table)
  File "/home2020/home/geo/fbreton/w2w_netcdf3/lib/python3.8/site-packages/w2w/w2w.py", line 630, in _get_SW_BW
    SW = ucp_table['MH_URB2D'] / ucp_table['H2W']
  File "/home2020/home/geo/fbreton/w2w_netcdf3/lib/python3.8/site-packages/pandas/core/frame.py", line 3505, in __getitem__
    indexer = self.columns.get_loc(key)
  File "/home2020/home/geo/fbreton/w2w_netcdf3/lib/python3.8/site-packages/pandas/core/indexes/base.py", line 3623, in get_loc
    raise KeyError(key) from err
KeyError: 'MH_URB2D'

The custom UCP table is below.
   ,FRC_URB2D ,MH_URB2D_MIN ,MH_URB2D ,MH_URB2D_MAX ,BLDFR_URB2D ,H2W
1  ,0.95      ,40           ,50       ,60           ,0.5         ,2.25
2  ,0.9       ,15           ,17.5     ,20           ,0.55        ,0.8
3  ,0.85      ,5            ,6.4      ,10           ,0.55        ,0.67
4  ,0.65      ,40           ,50       ,60           ,0.3         ,1.17
5  ,0.7       ,15           ,17.5     ,20           ,0.3         ,0.67
6  ,0.6       ,5            ,6.4      ,10           ,0.3         ,0.49
7  ,0.85      ,5            ,5        ,5            ,0.75        ,0.2
8  ,0.85      ,5            ,6.4      ,10           ,0.4         ,0.22
9  ,0.3       ,5            ,6.4      ,10           ,0.15        ,0.148
10 ,0.55      ,5            ,10       ,15           ,0.25        ,0.42
11 ,0         ,0            ,0        ,0            ,0           ,0
12 ,0         ,0            ,0        ,0            ,0           ,0
13 ,0         ,0            ,0        ,0            ,0           ,0
14 ,0         ,0            ,0        ,0            ,0           ,0
15 ,1         ,5            ,5        ,5            ,0.05        ,1
16 ,0         ,0            ,0        ,0            ,0           ,0
17 ,0         ,0            ,0        ,0            ,0           ,0

Is this some kind of syntax error? I just copied the default UCP table and changed the values.
Florentin

hey, please use the issue template next time, it's there for a reason :)
This is a bug on our end indeed. We don't handle the non standard, padded-out csv format properly. If you remove the spaces e.g. like this, it should work fine.
,FRC_URB2D,MH_URB2D_MIN,MH_URB2D,MH_URB2D_MAX,BLDFR_URB2D,H2W
1,0.95,40,50,60,0.5,2.25
2,0.9,15,17.5,20,0.55,0.8
3,0.85,5,6.4,10,0.55,0.67
4,0.65,40,50,60,0.3,1.17
5,0.7,15,17.5,20,0.3,0.67
6,0.6,5,6.4,10,0.3,0.49
7,0.85,5,5,5,0.75,0.2
8,0.85,5,6.4,10,0.4,0.22
9,0.3,5,6.4,10,0.15,0.148
10,0.55,5,10,15,0.25,0.42
11,0,0,0,0,0,0
12,0,0,0,0,0,0
13,0,0,0,0,0,0
14,0,0,0,0,0,0
15,1,5,5,5,0.05,1
16,0,0,0,0,0,0
17,0,0,0,0,0,0

If you want to have a look, it's this line that's initially causing it and needs fixing.
https://github.com/matthiasdemuzere/w2w/blob/a90c686fb4bb8aa24e3b06c384b0761a0409e8f5/w2w/w2w.py#L156
When you look at ucp_table.columns, there's most likely trailing whitespace somewhere causing the column names to be incorrect.
We can probably use this to fix it: https://pandas.pydata.org/pandas-docs/stable/reference/api/pandas.Series.str.strip.html

Thank you for the superfast response! :) (and sorry for the template, guilty)
I removed the spaces and reran the code.
`(w2w_netcdf3) (base) [fbreton@hpc-login1 w2w_ncdf3_wudapt]$ w2w -n 25 ./ lcz_wudapt_ponche.tif geo_em.d04.nc --lcz-ucp custom_lcz_ucp.csv
--> Set data, arguments and files

Seems you are using a LCZ map produced by https://lcz-generator.rub.de/
I therefor use the Gaussian filtered default 'LczFilter' layer
--> Check LCZ integrity, in terms of class labels, projection and extent
LCZ labels as expected (1 to 17)
ERROR 1: PROJ: proj_identify: Cannot find proj.db
ERROR 1: PROJ: proj_identify: Cannot find proj.db
LCZ provided as WGS84 (EPSG:4326)
LCZ domain is covering WRF domain
ERROR 1: PROJ: proj_create_from_name: Cannot find proj.db
--> Replace WRF MODIS urban LC with surrounding natural LC
Looping through urban grid pixels: 100%|█████████████████████████████████████████████████████████████████████████████████████████████████| 197/197 [00:01<00:00, 150.44it/s]
--> Create LCZ-based geo_em file
Processing LP_URB2D ...
Processing MH_URB2D ...
Processing STDH_URB2D ...
Processing HGT_URB2D ...
Processing LB_URB2D ...
Processing LF_URB2D ...
Processing HI_URB2D ...
Traceback (most recent call last):
File "/home2020/home/geo/fbreton/w2w_netcdf3/bin/w2w", line 8, in
sys.exit(main())
File "/home2020/home/geo/fbreton/w2w_netcdf3/lib/python3.8/site-packages/w2w/w2w.py", line 179, in main
nbui_max = create_lcz_params_file(
File "/home2020/home/geo/fbreton/w2w_netcdf3/lib/python3.8/site-packages/w2w/w2w.py", line 1272, in create_lcz_params_file
ucp_res, nbui_max = _hi_resampler(
File "/home2020/home/geo/fbreton/w2w_netcdf3/lib/python3.8/site-packages/w2w/w2w.py", line 971, in _hi_resampler
df_hi = _compute_hi_distribution(info, ucp_table=ucp_table)
File "/home2020/home/geo/fbreton/w2w_netcdf3/lib/python3.8/site-packages/w2w/w2w.py", line 916, in _compute_hi_distribution
hi_sample = _get_truncated_normal_sample(
File "/home2020/home/geo/fbreton/w2w_netcdf3/lib/python3.8/site-packages/w2w/w2w.py", line 833, in _get_truncated_normal_sample
hi_sample = hi_inst.rvs(SAMPLE_SIZE)
File "/home2020/home/geo/fbreton/w2w_netcdf3/lib/python3.8/site-packages/scipy/stats/_distn_infrastructure.py", line 467, in rvs
return self.dist.rvs(*self.args, **kwds)
File "/home2020/home/geo/fbreton/w2w_netcdf3/lib/python3.8/site-packages/scipy/stats/_distn_infrastructure.py", line 1066, in rvs
raise ValueError(message)
ValueError: Domain error in arguments. The scale parameter must be positive for all distributions, and many distributions have restrictions on shape parameters. Please see the scipy.stats.truncnorm documentation for details.
`

The new issue seems to be with statistical fitting that is unhappy with the input parameters. The input parameters could be out of bounds of what's expected by the statistical model, or the model expects varying parameter values (e.g. my LCZ 15 with constant building height broke the computation).
Cheers,
Flo

Hi @FloBre33,
Thanks for reporting about this.
To be honest, this user-specific template functionality has not been tested extensively before.
E.g., I haven't really tested the influence of changing the parameter values downstream, such as in the building height distribution derivation ...
But your assessment is correct: scipy.stats.truncnorm - that is used to estimate the building height probabilities per height bin -  builds a normal distribution around a min, mean, max, yet also requires a standard deviation. If all heights are the same, no distribution can be developed, and the system will crash.
I'll have to check how to make this clear. Either in the documentation, or as a warning in case someone uses a user-specific look-up table!

via #95


[tool_crash] StateError: Bad state: No context with the running Dart application.

Command
flutter run -d chrome

Steps to Reproduce

...
...
...

Logs
StateError: Bad state: No context with the running Dart application.
#0      RemoteDebuggerExecutionContext.id (package:dwds/src/debugging/execution_context.dart:47:7)
<asynchronous suspension>
#1      AppInspector._getExtensionRpcs (package:dwds/src/debugging/inspector.dart:486:20)
<asynchronous suspension>
#2      AppInspector._initialize (package:dwds/src/debugging/inspector.dart:93:34)
<asynchronous suspension>
#3      AppInspector.initialize (package:dwds/src/debugging/inspector.dart:156:5)
<asynchronous suspension>
#4      ChromeProxyService.createIsolate (package:dwds/src/services/chrome_proxy_service.dart:188:18)
<asynchronous suspension>

[✓] Flutter (Channel beta, 1.25.0-8.2.pre, on Microsoft Windows [Version 10.0.18363.1256], locale en-IN)
    • Flutter version 1.25.0-8.2.pre at C:\src\flutter
    • Framework revision b0a2299859 (17 hours ago), 2021-01-05 12:34:13 -0800
    • Engine revision 92ae191c17
    • Dart version 2.12.0 (build 2.12.0-133.2.beta)

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\suman\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    ✗ Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[✓] Android Studio (version 4.1.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[✓] VS Code (version 1.52.1)
    • VS Code at C:\Users\suman\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (2 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 87.0.4280.88
    • Edge (web)   • edge   • web-javascript • Microsoft Edge 87.0.664.66

! Doctor found issues in 1 category.

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: beta
Creation framework version: b0a22998593fc605c723dee8ff4d9315c32cfe2c
Plugins
integration_test

duplicate of https://github.com/flutter/flutter/issues/60495
see there's an open issue addressing the case you described.
Please follow up on that issue, I'm closing the current one as a duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you

duplicate of https://github.com/flutter/flutter/issues/60495
see there's an open issue addressing the case you described.
Please follow up on that issue, I'm closing the current one as a duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you


TestArbitraryPackageAttack TearDownClass failure

From PR #1113 CI run:
======================================================================
70ERROR: tearDownClass (test_arbitrary_package_attack.TestArbitraryPackageAttack)
71----------------------------------------------------------------------
72Traceback (most recent call last):
73  File "C:\projects\tuf\tests\test_arbitrary_package_attack.py", line 101, in tearDownClass
74    shutil.rmtree(cls.temporary_directory)
75  File "C:\Python38\lib\shutil.py", line 730, in rmtree
76    return _rmtree_unsafe(path, onerror)
77  File "C:\Python38\lib\shutil.py", line 603, in _rmtree_unsafe
78    _rmtree_unsafe(fullname, onerror)
79  File "C:\Python38\lib\shutil.py", line 603, in _rmtree_unsafe
80    _rmtree_unsafe(fullname, onerror)
81  File "C:\Python38\lib\shutil.py", line 603, in _rmtree_unsafe
82    _rmtree_unsafe(fullname, onerror)
83  File "C:\Python38\lib\shutil.py", line 608, in _rmtree_unsafe
84    onerror(os.unlink, fullname, sys.exc_info())
85  File "C:\Python38\lib\shutil.py", line 606, in _rmtree_unsafe
86    os.unlink(fullname)
87PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'C:\\projects\\tuf\\tests\\tmpmsjfh32h\\TestArbitraryPackageAttack_axb40vsy\\repository\\targets\\file1.txt'

Looking at the code that seems logical:
  @classmethod
  def tearDownClass(cls):
    # tearDownModule() is called after all the test cases have run.
    # http://docs.python.org/2/library/unittest.html#class-and-module-fixtures

    # Remove the temporary repository directory, which should contain all the
    # metadata, targets, and key files generated of all the test cases.
    shutil.rmtree(cls.temporary_directory)

    # Kill the SimpleHTTPServer process.
    if cls.server_process.returncode is None:
      logger.info('Server process ' + str(cls.server_process.pid) + ' terminated.')
      cls.server_process.kill()
      cls.server_process.wait()

If the server has files open in the temp directory, rmtree cannot succeed. Server should be killed (and waited on) first.

I am thinking that this problem could appear in multiple places.
With a quick search of the line shutil.rmtree(cls.temporary_directory) I found a couple of files which contain this line and are using a server subprocess:

tests/test_arbitrary_package_attack.py
tests/test_endless_data_attack.py
tests/test_extraneous_dependencies_attack.py
tests/test_indefinite_freeze_attack.py
tests/test_key_revocation_integration.py
tests/test_mix_and_match_attack.py
tests/test_replay_attack.py
tests/test_updater_root_rotation_integration.py
tests/test_updater.py

rmtree is only an issue because the server is still running in this case: so switching the order of rmtree and server kill here (and elsewhere if there's other places where the order is wrong) should help

rmtree is only an issue because the server is still running in this case: so switching the order of rmtree and server kill here (and elsewhere if there's other places where the order is wrong) should help

Yes, the files I have mentioned above are in the same error: remove the folder before stop the subprocess.

@joshuagl I forgot to use the Fixes keyword in Log subproceses stdout and stderr in temp files #1104 pr to automatically close this one. You can close it.


Provide way to explicitly mark receiver as ready to start handling messages

Context

Currently the messenger receiver starts listening on import
An application may have application-specific setup/initialization to do before it's ready to receive messages
Currently the application has to be careful not to import the messenger receiver until the rest of the setup has occurred. This is tricky if there are transitive dependencies to the messenger

Related Discussion

https://github.com/pixiebrix/pixiebrix-extension/pull/4061#issue-1342427732

Related Issues

https://github.com/pixiebrix/webext-messenger/issues/74

I’ve been working on #74. The messenger will be ready when we call registerMethods, which we do once per context.
Having a secondary setMessengerReady() function means we still need to make sure to not add further registerMethods calls after that, so it has a similar risk. Given that we only have one messenger/methods.ts file per context, it’s not a problem at the moment

That will be fixed for https://github.com/pixiebrix/webext-messenger/issues/74, which will make the messenger ignore communication until the first registerMethods call.

IIRC, registerMethods is called by the initInternalApi. So this is implying that the initInternalApi call won't call registerMethods anymore?

Having a secondary setMessengerReady() function means we still need to make sure to not add further registerMethods calls after that, so it has a similar risk. Given that we only have one messenger/methods.ts file per context, it’s not a problem at the moment.

I'm OK to proceed either way. In both cases, I think the would be a warning to designate mis-use

Warn if registerMethods is called multiple times
Warn if registerMethods is called after setMessengerReady (if we choose to add that method)

What's an ETA on a fix? I'm planning the PixieBrix 1.7.5 release schedule

Since you merged https://github.com/pixiebrix/pixiebrix-extension/pull/4061, this is no longer a blocking issue. I will likely finish it tomorrow. Parcel (tests) are giving me grief so I had to take a break.

this is no longer a blocking issue

It solves the case where it fails 100% of the time, but doesn't completely solve the problem (one user had actually been seeing the bug pre-1.7.4). We've already deployed workarounds for our enterprise customers, so will look to get the real fix into the 1.7.5 release

But I think this wouldn't fix what you're describing. We already have exactly one registerMethod call per context, which currently happens before any init function except for the content script (and that has been fixed by your recent PR).
Even if the private call happens long before registerMessenger, it still happens in the same tick, there is no chance that a message is received between a webext-messenger import and registerMessenger call currently. That's why this change is only to future-proof it rather than fixing current bugs.


Can not package SWC for Mobile Platform

I want package swc file use from mobile platform.
But,I do not know how.
I can only use type='swc' or type='mobile',this looks has problem.
for example ,I want package 'https://code.google.com/p/stagewebviewbridge/' with gradlefx.

If you want to create a library for use in a mobile project, you should either use 'swc' or 'swcAir' as the project type. These will package your project into a swc file. You can manually specify all mobile specific dependencies which you require.
Is there a specific problem you're facing?

Thank you.
Now the situation is，I want package a library used for air project. and use type='swcAir', but build failure. error log show:cannot find class flash.filesystem.File .

Have you specified the AIR SDK as a dependency?

Do you mean specified flexHome like this?
subprojects {
apply plugin: 'gradlefx'
flexHome = '/Users/liurui/develop/sdk/flex/4.14.1'
srcDirs = ['src']
}

indeed, like that, or by specifying the SDK as a dependency as described here: http://doc.gradlefx.org/en/latest/sdk_auto_install.html

I try use sdk auto install,like this:
type = 'swcAir'
repositories {
ivy {
name 'Apache'
artifactPattern 'http://apache.cu.be/flex/[revision]/binaries/[module]-[revision]-bin.[ext]'
}
}
dependencies {
flexSDK group: 'org.apache', name: 'apache-flex-sdk', version: '4.14.1', ext: 'tar.gz'
airSDK files('/Users/liurui/develop/sdk/flex/AdobeAIRSDK.tbz2')
}
but error log show:
Apache Flex compc (Component Compiler)
Version 4.14.1 build 20150325
Copyright 2015 The Apache Software Foundation.
/Users/liurui/.gradle/gradleFx/sdks/ebca91370fb119f48e5867aa9b6ade0ce98949a5/frameworks/flex-config.xml(65): ERROR:cannot open“libs/player/16.0/playerglobal.swc”
    </external-library-path>

Reason for this issue is that GradleFx only downloads the 11.1 version of playerglobal, and the AIR SDK only provides the 17.0 version of the playerglobal.swc. Therefore it can't find the 16.0 version.
I suggest you add the following to your build, so that it will use player version 17.0:
playerVersion = "17.0"
additionalCompilerOptions = [
        '-target-player=17.0'
]

I use your suggestions,and now error log show: 'cannot find class flash.filesystem.File' like before.
So ,I think it should not error caused sdk files.
The project url is https://github.com/aruis/stagewebviewbridge.git ，it can reappear the issue.

Found the issue. Turned out that the flex-config.xml was used for the configuration of a swcAir project using Flex, which is incorrect since it should be using air-config.xml. This was the reason you got the "cannot find class flash.filesystem.File" error.
I've fixed this issue with commit https://github.com/GradleFx/GradleFx/commit/b8fc8bf1e7d06686dc74511fe45e620ee8dd39dd
A snapshot release is now available on our snapshot repo as version 1.2.1-SNAPSHOT

@SlevinBE
"Turned out that the flex-config.xml was used for the configuration of a swcAir project using Flex, which is incorrect since it should be using air-config.xml."
actually seems, that it uses it also for swc. so, is I use airsdk version 18.0, it will be impossible to build (it doesn't have frameworks/flex-config.xml, it has only airsdk.xml).

If I remember correctly then you are using the AIR SDK without the Flex SDK. In that case you have to use the default AIR SDK provided by Adobe (AIRSDK_Compiler), not the one for Flex users (AdobeAIRSDK): http://www.adobe.com/devnet/air/air-sdk-download.html
The AIR SDK for Flex does indeed not have the *-config.xml files in it, because it assumes they are provided by the Flex SDK. The non-Flex AIR SDK has those files in them.

Thank you @SlevinBE . It works in 1.2.1-SNAPSHOT.


45 change the proccess page into storage page

I love the changes you did, really like it! :) Thank you!
And I re-assigned you as reviewer! :)

yeah saw the typo as well, thank you for spotting it! :)


Automator: update istio/client-go@master dependency in istio/istio@master

Generated by Automator - 2021-05-25T15:53:19+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-operator-controller-tests_istio
6c8ac89d6266cb5988fd70708340d3eda501d9df
link
/test integ-operator-controller-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Can't build on raspian

BinaryContainer, ProjectLayout, BuildConfigSpec)

Could not resolve all files for configuration ':nativeDeps'.
Could not find wpiutil-cpp-linuxnativearm.zip (edu.wpi.first.wpiutil:wpiutil-cpp:3.0.0-20180209143927-2-g4e9e7ec).
Searched in the following locations:
http://first.wpi.edu/FRC/roborio/maven/development/edu/wpi/first/wpiutil/wpiutil-cpp/3.0.0-20180209143927-2-g4e9e7ec/wpiutil-cpp-3.0.0-20180209143927-2-g4e9e7ec-linuxnativearm.zip

Appears the cpp binaries don't exist at that location?

Building with gradle on other platforms is not currently supported, as we don't distribute binaries for those platforms.  You can use the following repo to build using cmake instead of gradle: https://github.com/ThadHouse/CmakeWpilib

Will be fixed by wpilibsuite/allwpilib#1015, which brings back cmake builds.


New version has broken map tiles

Hi
We are currently loading in webp map tiles from a third party provider.
This is done like so
  this.observedImageLayerUpperGround = new Cesium.UrlTemplateImageryProvider({
            url: '[THE URL]',
            rectangle: Cesium.Rectangle.fromDegrees(west, south, east, north),
            ImagerySplitDirection: Cesium.ImagerySplitDirection.RIGHT,
            alpha: 0.5
        });
        this.observedImageLayerUpperGround.defaultAlpha = 1;
        layers.addImageryProvider(this.observedImageLayerUpperGround);

This was working great up until 1.92.0
However now we receive the following error.

I have rolled version back to 1.91.0 and it works again. Anyone have an idea what could have broken it in 1.92.0

Hi there, we updated to native promises in 1.92 which changed a few small things with how Promises work in Cesium. The issue you ran into may have been fixed in https://github.com/CesiumGS/cesium/pull/10268. Would you be able to check out main and five it a try?


docker_client: don't pull image and create container per tc command

Optimise the flow when using a separate container to run tc commands, by only pulling the container image once, creating a single container for the commands, and execing each command in that container.

Codecov Report
Patch coverage: 75.00% and project coverage change: +0.11% :tada:

Comparison is base (f3f7422) 40.11% compared to head (8a6cac5) 40.22%.
Report is 8 commits behind head on master.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #251      +/-   ##
==========================================
+ Coverage   40.11%   40.22%   +0.11%
==========================================
  Files          37       37
  Lines        1249     1218      -31
==========================================
- Hits          501      490      -11
+ Misses        748      728      -20

Files Changed
Coverage Δ

pkg/container/docker_client.go
75.68% <75.00%> (+3.85%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Hello @w-miller thank you for the PR.
But I would like to understand: what problem are you trying to solve with this PR? Is there any performance or stability issue with the current flow? Why do you think reusing the same tc container is a better option, than creating a new instance for every command? Did you measure the performance impact of this change?

@alexei-led thanks for your response. Our pumba workflow involves using a separate container to run the tc commands (with --tc-image=gaiadocker/iproute2), and setting specific target IP addresses with the --target filter. We then typically use a long-ish duration for the pumba netem invocation, and terminate the container when we want to undo the rules. The motivation for this pull request was to reduce the time between invoking pumba, and the netem tc rules being succesfully applied to the target container.
Explanation of the specific change in this patch: before the patch, pumba was attempting to do the following for every tc command it executes:

pull the image specified in --tc-image. Assuming that you already have the image present, this adds the overheads of making a network call to your container registry, and validating that the local image is still up to date.
create a new container to run the single tc command, execute the command, then stop and remove the container again.

I've run a benchmark locally with and without this patch to calculate the time between starting pumba and the netem tc rules being successfully applied. I compared these results between the code with and without the patch in this PR. N.B. in order to measure this timespan, I also needed to cherry-pick the patch in https://github.com/alexei-led/pumba/pull/250 to both of the pumba builds I tested, since the time at which the log message netem command started appears is the time at which the tc rules have all been applied, and there was no existing message logged at this point. I created a dummy container to be used as the target for the tc rules - a trivial container from the ubuntu image with the entrypoint sleep infinity. This container had name funny_napier. I used the following command, replacing pumba_image with a docker image built from each of the two versions of the code. I ran each variant 10 times
docker run -t -v /var/run/docker.sock:/var/run/docker.sock -it <pumba_image> --json --log-level=debug netem --duration=60s --tc-image=gaiadocker/iproute2 --target=<IP_ADDRESS> loss --percent 10 funny_napier  | grep --line-buffered 'netem command started' | head -n1 | ts -s '%.s'
The full results are below, but the outcome was that the average time between pumba invocation and netem command started being logged was:

10.86 seconds for the code without this patch
3.24 seconds for the code with this patch

This result mirrors the speedup of around 3x that we saw in our testing workflows that use pumba after applying this patch.


various fixes and refactorings

see the individual commit messages for what's going on.

We could rename GlobalEvalContext to EvalContext now that it's the only type.

done

done

Great cleanups, thanks!


Travis errors and using conda environments

I took a look at the errors that Travis is spitting out.
I was wondering if they might be related to the fact that you are using both virtualenv and conda (or is it that travis always uses virtualenv?). For example virtualenv usues python 2.7.9, but the miniconda version has python 2.7.10 - so there are actually two python versions on the machine. There is also some noise in the installation process - numpy (version 1.10) is first installed via conda but then pip install -r requirements.txt uninstalls it and installs the required version 1.9.
Maybe using conda environments is worth trying out?
For example:
conda create -n testenv --yes pip python=2.7
conda update conda --yes
source activate testenv
conda install --yes --file requirements.txt

You can also take a look at the travis script that seaborn is using
I would be happy to help with this, but I am using a Windows machine :cry:

Thanks Mikolaj!  Good eye. I’m now hoping you’re spot on. (It’s a simple answer.)
The Travis script is something I hacked together,  just trying for that 'green bar’ as fast as possible. :) I’ll try conda environs (post-SFN).

On Oct 15, 2015, at 4:05 AM, Mikolaj Magnuski<EMAIL_ADDRESS>wrote:
I took a look at the errors that Travis is spitting out.
I was wondering if they might be related to the fact that you are using both virtualenv and conda (or is it that travis always uses virtualenv?). For example virtualenv usues python 2.7.9, but the miniconda version has python 2.7.10 - so there are actually two python versions on the machine. There is also some noise in the installation process - numpy (version 1.10) is first installed via conda but then pip install -r requirements.txt uninstalls it and installs the required version 1.9.
Maybe using conda environments is worth trying out?
For example:
conda create -n testenv --yes pip python=2.7
conda update conda --yes
source activate testenv
conda install --yes --file requirements.txt
You can also take a look at the travis script that seaborn is using https://github.com/mwaskom/seaborn/blob/master/.travis.yml
I would be happy to help with this, but I am using a Windows machine
—
Reply to this email directly or view it on GitHub https://github.com/voytekresearch/pacpy/issues/19.

Looks like it was was conda/pip install nonsense dance I created in the first .yml draft. Now resolved. Thanks @mmagnuski!

:+1:


chore: fix Actix Web typos

Just fixes a nitpick of mine. Accompanies https://github.com/shuttle-hq/shuttle/pull/1093.

Hey @robjtede, nice to see you here! :)
Thanks for submitting this, I'll merge it real quick and ensure that we use Actix Web in future cases (and in-retro if we dig them out in the upcoming days)!


Link is broken the router

Describe the bug
I'm facing an issue while using Link. It's working with all gateways except near.org. So, if I have this:
const Link = styled("Link")``;

return (
  <Link href="?path=feature-overview">
    Go to Feature Overview
  </Link>
);

When I hover or click, this is going to work as expected (except for near.org):

https://near.social/alem-lib.near/widget/Index?path=feature-overview
And this is what I'm getting when using near.org:

To Reproduce
Steps to reproduce the behavior:

Go to https://near.org/alem-lib.near/widget/Index
Click on "Learn Além"
See error

You can do the same using a diff gateway, and it's going to work, e.g.: https://near.social/alem-lib.near/widget/Index
Expected behavior
Should render the path correctly.
Desktop (please complete the following information):

OS: macOS
Browser chrome
Version 124.0.6367.119 (Official Build) (x86_64)

hey @wpdas Link is a gateway-defined custom element and the near.org implementation is a bit different than that of near.social.  Are you able to add the path + the query params?

Re-opening because this seems to affect multiple apps on dev.near.org.  See PotLock

hey @shelegdmitriy let's pull this into this week. What might need to change in our custom Link component to have parity with that of near.social?


feat: adding ssm provision and reservation

Adds an SSM provision and reservation service
Noted that LocalProvisionService should replace static hosts with a HostProvider

Related to #46

Codecov Report

Merging #54 (2ac6cf0) into main (d5a9f1d) will increase coverage by 0.41%.
The diff coverage is 97.82%.

@@             Coverage Diff              @@
##               main      #54      +/-   ##
============================================
+ Coverage     89.39%   89.80%   +0.41%
- Complexity      416      433      +17
============================================
  Files            59       60       +1
  Lines          1782     1874      +92
  Branches         72       77       +5
============================================
+ Hits           1593     1683      +90
- Misses          162      163       +1
- Partials         27       28       +1

Impacted Files
Coverage Δ

.../philcali/device/pool/ssm/ProvisionServiceSSM.java
97.82% <97.82%> (ø)

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 d5a9f1d...2ac6cf0. Read the comment docs.


N1 queries

Changes

Added bullet gem to check for N+1 queries.
Fixed N+1 query occurrences.

Side changes

Added some style to food_path's buttons

Status: Approved!! 🎉🎉
There's nothing else to say than it's time to merge it!!

Status: Approved!! 🎉🎉
There's nothing else to say than it's time to merge it!!


Show built in environment variables when listing env vars

$ modulus env list should show the built-in environment variables (where possible).

Is this request to include env vars that are not related to the CLI (stored in a user's shell configuration)?

Just wanted to ping on this one... Refer to comment above

I believe this is in reference to the fact that we set additional environment variables at runtime that are not exposed in this CLI command, (eg: PORT, APP_DIR, or SERVO_ID).

Hm. I think those environment variables are exposed inside of the servo only...


Backpropagation resets weights to 0

Steps to reproduce:
Create net. Call net = net.backpropagate()
Expected behaviour:
weights of the net will change slightly toward the targetValue depending on learningRate
Actual behavior:
weights of net all become zero.
Notes:
the output of the broken net is 0.5 because sigmoid of 0 is 0.5.

The problem was in the layer constructor. It was instantiating an empty weights array despite being passed new weights.


Docs: minor fixes in mock response guide

Grammar fixes "nor ... or" -> "neither ... nor", "an response" -> "a response"
Tidy up sentence structure to hopefully be more readable
"string" and "number" were the wrong way around

It's a lot better than my Italian, I'm sure! :)


standard qft qubit ordering convention is not the same for vector and circuit modes

Informations

Qiskit Aqua version: 0.4.1
Python version: 3.6.8
Operating system: MacOS High Sierra

What is the current behavior?
In aqua>components>qfts>standard.py, it seems that the qubit ordering convention when calling the 'vector' mode and the 'circuit' mode aren't the same. Since the function does not include the final swaps, I would expect this behavior:
if one compares the matrix obtained with the "vector mode", and the one obtain by "circuit mode + unitary simulator processing", the second one should have two ROWS switched compared to the first one. Instead two COLUMNS are switched. This means that the swaps should be applied before the QFT to obtain identical matrices. This can be confusing.
Steps to reproduce the problem
from qiskit import Aer, execute
from qiskit.aqua.components.qfts.standard import Standard
QFT = Standard(num_qubits)
qft_vec_mode = QFT.construct_circuit('vector')
circuit_qft = QFT.construct_circuit('circuit')
simulator = Aer.get_backend('unitary_simulator')
result = execute(circuit_qft, simulator).result()
qft_circ_mode = result.get_unitary(circuit_qft)
print(qft_vec_mode)
print(qft_circ_mode)
What is the expected behavior?
expect to have a qft+swaps unitary that is identical to the vector mode of qft.
Suggested solutions
change ordering of the qubits in the circuit definition of qft:
for j in range(self._num_qubits-1,-1,-1):
for k in range(self._num_qubits-1,j,-1):

@polline thx for reporting this! I tried as you suggested and also got different results from 'circuit' and 'vector' modes. I did try your suggested solution but unfortunately couldn't get things to work.
Aqua's circuit construction for qft was actually adapted from Terra:
# Define methods for making QFT circuits
def qft(circ, q, n):
    """n-qubit QFT on q in circ."""
    for j in range(n):
        for k in range(j):
            circ.cu1(math.pi / float(2**(j - k)), q[j], q[k])
        circ.h(q[j])

could you use this method instead for the 'circuit' mode and verify your suggestion solution?

@hushaohan Thanks for your reply
Indeed we also need to switch the indices in the power of 2 in the qft. Here is my suggested solution, I run some tests, seems to work:
          for j in range(self._num_qubits-1,-1,-1):                 for k in range(self._num_qubits-1,j,-1):                     lam = 1.0 * pi / float(2 ** (k-j))                      circuit.u1(lam / 2, qubits[j])                     circuit.cx(qubits[j], qubits[k])                     circuit.u1(-lam / 2, qubits[k])                     circuit.cx(qubits[j], qubits[k])                     circuit.u1(lam / 2, qubits[k])                 circuit.u2(0, np.pi, qubits[j])
I think the same should also be applied in the standard iqft method.

@polline could you take a look at this PR https://github.com/Qiskit/qiskit-aqua/pull/447 and let me know if it works for you? I basically added the swaps (before qft and after iqft) and the options to toggle them on/off.


Different behavior when canceling a token

Hey @mayuki
We encountered another issue after updating YetAnotherHttpHandler. When canceling a token, an exception occurs on the client with different contents, depending on the environment
This is a client-side code:
private async UniTask UpdateCameraPreviewModeAsync(CancellationToken cancellationToken, CameraModeParams nextParams)
{
    try
    {
        var parameters = new CameraPreviewParametersDto
        {
            ScanMode = nextParams.CurrentScanMode,
            CameraPositionType = nextParams.CurrentCameraPositionType
        };
        await grpcClientService.Settings_UpdateCameraPreviewModeAsync(parameters, cancellationToken);
    }
    catch (RpcException exception) when (exception.StatusCode == StatusCode.Cancelled)
    {
        logger.LogTrace("Update of camera preview mode was canceled");
    }
    catch (RpcException exception)
    {
        logger.LogErrorWithException("Cannot update the camera preview mode", exception);
    }
    catch (Exception exception)
    {
        logger.LogErrorWithException("Error when trying to update the camera preview mode", exception);
    }
}

And a server-side code:
public async UnaryResult<Nil> Settings_UpdateCameraPreviewModeAsync(CameraPreviewParametersDto previewParametersDto)
{
    var request = new UpdateCameraPreviewCommand(previewParametersDto);
    await mediator.Send(request, Context.CallContext.CancellationToken);
    return Nil.Default;
}

The error we get when canceling the token on the client if we test through Unity Editor:
unityEditorException.txt
The error we get when canceling the token on the client if we test the application on Android:
androidException.txt
The main difference is that on Android, the TaskCanceledExceptio turns into an HttpRequestException and we receive the status code not Grpc.Core.StatusCode.Cancelled, but Grpc.Core.StatusCode.Internal
Is this behavior expected? Do we need to handle this behavior for Android additionally?
Also, for the sake of the experiment, I tried to change the server code and throw the exception myself. In this case, both in Unity editor and on Android, we receive the status Grpc.Core.StatusCode.Cancelled:
public async UnaryResult<Nil> Settings_UpdateCameraPreviewModeAsync(CameraPreviewParametersDto previewParametersDto)
{
    throw new TaskCanceledException();
    var request = new UpdateCameraPreviewCommand(previewParametersDto);
    await mediator.Send(request, Context.CallContext.CancellationToken);
    return Nil.Default;
}

For easier testing on the client, I used a token with cancellation after 100 milliseconds:
private async UniTask UpdateCameraPreviewModeAsync(CancellationToken cancellationToken, CameraModeParams nextParams)
{
    var cts = new CancellationTokenSource(TimeSpan.FromMilliseconds(100));

    try
    {
        var parameters = new CameraPreviewParametersDto
        {
            ScanMode = nextParams.CurrentScanMode,
            CameraPositionType = nextParams.CurrentCameraPositionType
        };
        await grpcClientService.Settings_UpdateCameraPreviewModeAsync(parameters, cts.Token);
    }
    ...
}

Which version of YetAnotherHttpHandler and Unity is causing this problem?

@mayuki We use latest 1.5.1 version of YetAnotherHttpHandler and Unity 2022.3.29f1

Thank you for the additional information. How are you passing CancellationToken in grpcClientService.Settings_UpdateCameraPreviewModeAsync?
Normally, CancellationToken is not propagated from the client to the remote, so I'm curious about how it is handled.

I was able to reproduce a similar problem, so I will continue to investigate.

We have released 1.5.2, which fixes the cancellation behavior in IL2CPP. Please try this version.

You rock @mayuki! ❤


Minor UI-adjustments for detection view

User story

As a user
I want / need have self explanatory and intuitive icons used for features
So that I can quickly access a desired feature

Acceptance criteria

[ ] Delete "Scan"-button on the top right corner
[ ] Next to each saved device, place two buttons that navigate to the signal strength or geolocation feature respectively:
[ ]

Definition of done (DoD)

It builds without errors
code has been reviewed
Build has been made and deployed on a testing environment
Code coverage >75%
All acceptance criteria were met
Documentation updated whenever needed within planning documents (Tabs: Bill of materials, Documentations)
Branch/feature was merged into dev and then main

DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

The scan button is removed and the itempage and itemdetailpage show the desired icons. Sorry for the late reply, i've been sick.


missing some values when set the permission to user

get the latest packages, there're some values missing when set the permission to user.
can add these two missing values in the Localization

Hi, @davidnasar. Welcome to create a PR for it.

created it already


Refresh access_token Service

Bolt's access_token expires after a while making some of the API calls we send to Bolt unable to process. With this PR we introduce a service that will refresh the access_token after expiry and rework the code to call said service instead of using session[:bolt_access_token].
Fixes https://github.com/nebulab/solidus_bolt/issues/101
Requires https://github.com/nebulab/omniauth-bolt/pull/2

The logic looks good to me. But I have bring it to draft because we have to update the omniauth-bolt gem, otherwise, we cannot update the reference to that gem


Improve graph cycle detection

NOTE
The current cycle detection algorithm is computationally expensive, but IIUC, all we have to do is quite simple and check if we can write a path without referencing the same node 🙂  I've introduced a new variable (visited) to memorize the path to the current node to make it more effective (O(N)).

No, the current algorithm is wrong. Let me think about it a little more deeply 🤦♂

I try to create the same fixing based this PR #16 . Please see it.

Closing in response to https://github.com/mercari/grpc-federation/pull/17


added staircase challenge in C++

#1

Problem Link: https://hackerrank.com/challenges/staircase/problem


Support decoding optional values hyperbee skips (en/de)coding

The value in bee.put(key, value) is optional and is not encoded via the valueEncoding option when the node is appended. Because of this, hyperbee-diff-stream threw an error trying to decode a null assuming it was a buffer. A new check for null skips decoding allowing it to pass through.
For reference, here is where hyperbee currently skips decoding: https://github.com/holepunchto/hyperbee/blob/ba9b6cde9dcfd8c639769b4118593b3b8c269b31/index.js#L325

Sorry for the late reply, I had misconfigured my notifications and didn't receive one for PR's for this repo (feel free to ping me if that happens again).
This looks like a sensible change to me, to keep it consistent with Hyperbee. I'll have a proper look tomorrow

Out in 1.0.4. Thanks for the PR!


./refine does not trigger a build

Describe the bug
If the software has not yet been built, running ./refine no longer automatically builds the software before running
To Reproduce

At the command line type ./refine

Current Results
./refine
You have 5631M of free memory.
Your current configuration is set to use 1400M of memory.
OpenRefine can run better when given more memory. Read our FAQ on how to allocate more memory here:
https://github.com/OpenRefine/OpenRefine/wiki/FAQ:-Allocate-More-Memory
/Library/Java/JavaVirtualMachines/jdk1.8.0_72.jdk/Contents/Home/bin/java -cp server/classes:server/target/lib/*   -Xms1400M -Xmx1400M -Drefine.memory=1400M -Drefine.max_form_content_size=1048576 -Drefine.verbosity=info -Dpython.path=main/webapp/WEB-INF/lib/jython -Dpython.cachedir=/Users/damyantiandowen/.local/share/google/refine/cachedir -Drefine.autoreload=true -Dbutterfly.autoreload=true -Xdock:icon=graphics/icon/openrefine.icns -Drefine.webapp=main/webapp -Drefine.port=3333 -Drefine.host=<IP_ADDRESS> com.google.refine.Refine
Starting OpenRefine at 'http://<IP_ADDRESS>:3333/'

Error: Could not find or load main class com.google.refine.Refine

Expected behavior
Expected software to attempt a build, and if successful, run refine
Desktop (please complete the following information):

OS: macOS Mojave 10.14.1 (18B75)
JRE or JDK Version:
java version "1.8.0_72"
Java(TM) SE Runtime Environment (build 1.8.0_72-b15)
Java HotSpot(TM) 64-Bit Server VM (build 25.72-b15, mixed mode)

OpenRefine (please complete the following information):

Version: Latest Master
NB this works on the 3.1 Beta release, so it is a change since that point

That is curious. I don't see why any of the commits since 3.1-beta introduced this - I think this is probably due to a half-built copy of OpenRefine. At the moment we detect whether OR should be built with ls $REFINE_LIB_DIR | grep openrefine, that can surely be improved to match your case.
Can you paste the output of ls server/target/lib from a copy of OR where ./refine fails?

Away from computer right now but will check later

I just tried on fresh Ubuntu install... works fine.
thad@DESKTOP-TJUMQT0:~/OpenRefine$ ./refine
You have 16308M of free memory.
Your current configuration is set to use 1400M of memory.
OpenRefine can run better when given more memory. Read our FAQ on how to allocate more memory here:
https://github.com/OpenRefine/OpenRefine/wiki/FAQ:-Allocate-More-Memory
ls: cannot access 'server/target/lib': No such file or directory
[INFO] Scanning for projects...
[WARNING]
[WARNING] Some problems were encountered while building the effective model for org.openrefine:packaging:jar:3.1-SNAPSHOT
[WARNING] The expression ${version} is deprecated. Please use ${project.version} instead.
[WARNING] The expression ${version} is deprecated. Please use ${project.version} instead.
[WARNING] The expression ${version} is deprecated. Please use ${project.version} instead.
[WARNING]
[WARNING] It is highly recommended to fix these problems because they threaten the stability of your build.
[WARNING]
[WARNING] For this reason, future Maven versions might no longer support building such malformed projects.
[WARNING]
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Build Order:
[INFO]
[INFO] OpenRefine                                                         [pom]
[INFO] OpenRefine - main                                                  [jar]
[INFO] OpenRefine - server                                                [jar]
[INFO] OpenRefine - extensions                                            [pom]
[INFO] OpenRefine - Jython extension                                      [jar]
[INFO] OpenRefine - Wikidata extension                                    [jar]
[INFO] OpenRefine - Database extension                                    [jar]
[INFO] OpenRefine - Gdata extension                                       [jar]
[INFO] OpenRefine - PC-axis extension                                     [jar]
[INFO] OpenRefine - packaging                                             [jar]
[INFO]
[INFO] ---------------------< org.openrefine:openrefine >----------------------
[INFO] Building OpenRefine 3.1-SNAPSHOT                                  [1/10]
[INFO] --------------------------------[ pom ]---------------------------------
Downloading from central: https://repo.maven.apache.org/maven2/org/codehaus/mojo/build-helper-maven-plugin/1.8/build-helper-maven-plugin-1.8.pom

[snip][snip]

@ostephens if you are OK with losing work in there... git reset --hard @{u}
or you could 1st try not losing work git reset --soft @{u}
ref: https://git-scm.com/docs/git-reset#git-reset-emgitresetemltmodegtltcommitgt

Yeah - I don't often push the master to https://github.com/ostephens/OpenRefine (I have done just now), but on my local repo I keep it up to date
When I say "copying the content of the repo": I copy the Git repo to another directory to do test builds to avoid the built files ending up in my branch. If there is a better / different way of doing this part of the workflow then I'd definitely like to learn!
Trying both git reset soft & hard still doesn't fix the problem

@ostephens when I copy the repository to a different location in your computer, I find it useful to use git clone my_old_location my_new_location rather than an actual copy such as with cp -r my_old_location my_new_location. That ensures that only the files tracked in the repository get copied over. (Not sure if you are already doing that or if it fits your workflow at all of course!)

Ah - I see /server/classes and others all in Git ignore - so I'm guessing that there is something in there...
Yep - removing /server/classes from my end sorts the problem

So currently having any /server/classes directory stops the build even if it is empty. Not sure it is worth changing this behaviour tbh

I got this error
I do all your procedure but in my repo nothing happend

Try to clean your clone with ./refine clean maybe?

Try to clean your clone with ./refine clean maybe?

Thanks, but I redownload the repo
And now it works


[C#] Builder Design Pattern revamp

Continuing with the revamps in #18, this is the implementation of Builder Design Pattern.
Changes implemented

Created a dedicated Builder project, with an example in Program.cs to show demo code.
Sealed all concrete classes.
Use File-Scoped namespaces, resulting in one less indentation level.
XML Documentation on main classes and methods
Separated Nested classes for SlackNotification and MessengerNotification, as well as removing duplicated code.
Remove Getter methods in favor of Auto Properties for less boilerplate code.
Removed all nullability warnings by setting string properties to nullable versions string?
Printed Notification objects using JSON Serialization using built-in System.Text.Json and boosted its performance using Source-Generated implementation Reference
[Important] Removed ProductService example as C# HttpClient is not designed to use the builder pattern.

@mahyoussef This PR was a little bit challenging, So I am interested in your thoughts about it.

There's also a conflict in the PR.


Support  IAM roles for service accounts

The new feature which supports assigning IAM roles directly to pods was recently released:
https://aws.amazon.com/blogs/opensource/introducing-fine-grained-iam-roles-service-accounts/
Currently, the kubernetes-external-secrets pod is not able to assume the role and is using the node assigned role instead, although the aws-sdk version in nodejs supports the new feature.

@dudicoco Have you checked all the kinks in the previous issue? https://github.com/godaddy/kubernetes-external-secrets/issues/164

@Flydiverny I have now checked all of the related issues.
Everything seems to be configured properly on my side, but I am experiending the same issue as

Somehow I managed to link to the wrong issue was meant to refer to #161
I haven't tested this myself but my understanding is that if kiam or something else provides credentials to the pod, these are used before the IRSA credentials.
As seen here:
https://github.com/aws/aws-sdk-js/blob/v2.575.0/lib/node_loader.js#L61-L69
Please verify that you are not using other means of providing AWS credentials to the pod that have higher priority than TokenFileWebIdentityCredentials which should be what is used for IRSA

Closing this as it seems resolved by discussion in #200


DEPR: strtools no longer applies as the package is for Python>=3.9

Problem
The package was originally for a Python<=3.8 environment, hence the strtools methods. This is no longer required.
Tasks

[ ] Move the module to a new package
[ ] Remove the module from zolltools

I don't think I even need to move this into a separate module because I won't be using this in the future. I'm just going to remove that task and remove the module from this package.

Removed the module. Now I just need to make sure it passes all tests before merging and pushing.

Fixed with https://github.com/Jython1415/zolltools/commit/97e2cc3c8e3b4d2bf01c6df7abe96364fbb12f2f


NTR: negative regulation of posttranslational protein targeting to membrane, translocation (and parent)

Hi,
I would like to request the following term 'negative regulation of posttranslational protein targeting to membrane, translocation' be added to the process ontology. This will require an additional term be created first in the process ontology 'regulation of posttranslational protein targeting to membrane, translocation' using the regulates relationship. Authors present evidence that yeast STE24, a metalloprotease involved in N-terminal proteolytic processing and a component of the ER quality control mechanism that removes faulty proteins clogging ER translocation channels, also functions as a negative regulator of N-terminal signal peptide–independent translocation into the endoplasmic reticulum
Here is what I am thinking:
GO:0031204 posttranslational protein targeting to membrane, translocation
----->GO:XXXXXXX regulation of posttranslational protein targeting to membrane, translocation	(regulates)
----------->GO:YYYYYYY negative regulation of post-translational protein targeting to endoplasmic reticulum membrane, translocation (is_a)
GO:XXXXXXX	regulation of posttranslational protein targeting to membrane, translocation
Definition: 	Any process that modulates the frequency, rate or extent of posttranslational protein translocation through the ER membrane.
Synonyms:     	regulation of SRP-independent endoplasmic reticulum protein-membrane targeting, translocation
regulation of N-terminal signal peptide–independent translocation into the ER
regulation of posttranslational endoplasmic reticulum membrane protein translocation
GO:YYYYYYY	negative regulation of posttranslational protein targeting to membrane, translocation
Definition: 	Any process that stops, prevents or reduces the frequency, rate or extent of posttranslational protein translocation through the ER membrane.
Synonyms:     	negative regulation of SRP-independent endoplasmic reticulum protein-membrane targeting, translocation
negative regulation of N-terminal signal peptide–independent translocation into the ER
negative regulation of posttranslational endoplasmic reticulum membrane protein translocation
Reference:
Hosomi A, et al. (2020) The ER-associated protease Ste24 prevents N-terminal signal peptide-independent translocation into the ER in Saccharomyces cerevisiae. J Biol Chem.
https://www.yeastgenome.org/reference/S000275006
PMID: 32513868
Cheers,
Rob

Hi @robnash
Thanks for providing so much info about what terms you need, it makes it very easy!
You didn't mention whether the synonyms should be EXACT, NARROW, etc. and it seemed to me that the ones like this should be narrow, since they are limited to signal-peptide independent mechanisms while the terms you requested aren't obviously limited in that way:

regulation of SRP-independent endoplasmic reticulum protein-membrane targeting, translocation
regulation of N-terminal signal peptide–independent translocation into the ER

while the ones like this could be exact:

regulation of posttranslational endoplasmic reticulum membrane protein translocation

but if you disagree with what I put in, let me know what they should be instead.
Thanks,
-Karen


Add docs for NU1300, 1301, 1302, 1303, 1304, 1307

write docs for the work done to address: Improve CLI + VS auth failure errors #9688

The change hasn't been made yet, so this PR is not relevant at this point.


Is there a way to create multiples tables based on one simple class?

I have a library project, with common functionalities and then i have multiple projects that depends on it.
Is there a way, for example, from one class "ArticleDTO" create multiple tables representing the categories that i have in each app?
Can i achieve this by using the table config file?

Can you elaborate a little? Ormlite is an ORM tool that stores your objects in a database. I don's see how you can split up an object to store it in multiple tables without splitting up the class itself.

Short answer is "no". Going to close this.


New GOST identificators

This pull request provides OIDs and NIDs necessary for new GOST-related standards.

Oh...actually, it looks like a "make update" is required for this PR

yes. fuzz/oids  okay to do 'make update' as part  of the commit?

Sure

thanks.  done and merged.


Fixing some readability issues.

on standard_paths function:

I used os.path.isdir instead of try: ... except OSError: pass  for checking if lib-dynload exists.

on  standard_package_names function:

I used name.endswith instead of using the complex expression name.rsplit('.')[-1]

Coverage decreased (-0.1%) to 99.327% when pulling 3d0346aeab09487d48b6ce157f5f57b06d965f2e on hadizakialqattan:patch-2 into b330072a26bb13aea8843f8083cee5332d71f78f on myint:master.

Coverage increased (+0.0006%) to 99.438% when pulling 3d0346aeab09487d48b6ce157f5f57b06d965f2e on hadizakialqattan:patch-2 into b330072a26bb13aea8843f8083cee5332d71f78f on myint:master.

Also the test coverage has increased.


Server-side testing is missing

Server-side testing
We need to add server-side testing framework with tests for the features that we are adding. Once the framework is in place, any new feature we add should also include test cases for that feature.
Describe the solution you'd like
Specifically, we need to add tests for issue #16 and issue #9. Since the server-side is written in Python, pytest is a natural choice for the testing framework. The test should be automatically run on any push to the repository, via GitHub actions.

We'll want to have tests for use_cases and rest components of the server side application.

Fixed with pr #27


Replace base classes of 'collections.UserList' with 'list'

This issue was originally created at: 2009-02-10 22:14:16.
This issue was reported by: gregnoel.
gregnoel said at 2009-02-10 22:14:16

It's now possible to create classes based on built-in types.  The classes based
on 'collections.UserList' should be converted to use 'list' as a base type.

gregnoel said at 2009-02-10 22:14:57

Set dependency.

stevenknight said at 2009-02-22 08:10:02

Reprioritizing and reassigning per bug party triage.
(Actually, we didn't specifically settle on assigning this to you, Greg.  I
assigned it to you because it feels of the same piece as the Fixer work you've
been spearheading.  Kick it back if you feel that's inappropriate.)

stevenknight said at 2009-02-22 08:13:49

Actually, I see now that we suggested Nate takes this (and 2238) but I'm not
sure if he has a tigris.org username.  Please reassign as appropriate.

garyo said at 2012-09-01 10:01:49

Bumping all old issues targeted for past releases to 2.x.

garyo said at 2012-09-01 10:04:01

de-assigning all tickets assigned to Greg Noel (no longer working on SCons)

gregnoel said this issue depends on #2337 at 2009-02-10 22:14:57.

Turns out this is hard to do, due to subtleties with how Python handles lists (it cheats, for purposes of optiimization), and also hard due to some of the ways we use some of the lists based on UserList. Despite the wide number of comments that you should just sublass list, UserList (and the suitable abstract base class in collections.abc) are designed for subclassing.
Here's a discussion by one experienced Pythonista: https://treyhunner.com/2019/04/why-you-shouldnt-inherit-from-list-and-dict-in-python/
Suggest we drop this one.

Yes. I don't see a reasonable way to do this unless python's list's implementation makes more internal visible. Closing.


Emails redesign

https://chat.zulip.org/#narrow/stream/107-kandra/topic/marketing.20email.20design

A few thoughts:

Can we use a slightly darker purple?  The text of links feels a bit hard to read, which was a problem with the green too.
We probably should change the button colors too; you can see all the emails in /emails
It's going to be hard to do a cutover for all of our emails without changing the colors for the line drawing illustrations, which I'm not sure is easy, so maybe we use this just for the newsletter for now.

I don't know what is causing this backend failure.

The error message is terrible, but It's just because we have an assertion for some text/HTML that included the old color.

I merged the heading and color changes as f452e176daf3d4e0a26ce3f2ac100420b5a0a7a4, after fixing that CI issue.

I merged the CSS change as dff5e8ff6e2c34fdcb9738050744d3e2d695a55a after testing that it worked great (and playing with the cutoff to find what seemed optimal).  Huge thanks for making this work.
For future reference, it is possible to send real emails in a development via the option documented here:

@hackerkid implemented it so should be able to help if you run into any trouble getting it working (though you may not need it at all for this effort).
Huge thanks for making this happen!


frontend styling

This includes removing the global styling and making sure individual components look clean, as well as making sure everything is logically placed. One thing some of us have already noticed in that regard is that the current placement for the tours and garbage collections does not make a lot of sense, we could either place these two under a new dropdown in the main header or just directly place both of them on the main header since these will be used a lot.

Make sure to make links more obvious. E.g. in this example, it is not obvious that there is a link to click:


level10 Hollow Knight

Output log:
Import: Game file 'level10' has been found
Import: Dependency 'unity_builtin_extra' has been loaded
Import: Dependency 'unity default resources' has been loaded
Import: Dependency 'globalgamemanagers.assets' has been loaded
Import: Dependency 'sharedassets0.assets' has been loaded
Import: Dependency 'resources.assets' has been loaded
Import: Dependency 'sharedassets5.assets' has been loaded
Import: Dependency 'sharedassets1.assets' has been loaded
Import: Dependency 'sharedassets6.assets' has been loaded
Import: Dependency 'sharedassets7.assets' has been loaded
Import: Dependency 'sharedassets9.assets' has been loaded
Import: Dependency 'sharedassets10.assets' has been loaded
Import: Asset collection has versions probably incompatible with each other. Here they are:
Import: 2017.4.6f1
Import: 2017.3.0b10
Import: System.ArgumentException: Illegal characters in path.
at System.IO.Path.CheckInvalidPathChars(String path, Boolean checkAdditional)
at System.Security.Permissions.FileIOPermission.CheckIllegalCharacters(String[] str)
at System.Security.Permissions.FileIOPermission.AddPathList(FileIOPermissionAccess access, AccessControlActions control, String[] pathListOrig, Boolean checkForDuplicates, Boolean needFullPath, Boolean copyPathList)
at System.Security.Permissions.FileIOPermission..ctor(FileIOPermissionAccess access, String[] pathList, Boolean checkForDuplicates, Boolean needFullPath)
at System.IO.DirectoryInfo.Init(String path, Boolean checkHost)
at System.IO.DirectoryInfo..ctor(String path)
at uTinyRipper.FileUtils.GetUniqueName(String dirPath, String fileName)
at uTinyRipper.AssetExporters.ExportCollection.GetUniqueFileName(ISerializedFile file, Object asset, String dirPath)
at uTinyRipper.AssetExporters.AssetExportCollection.Export(ProjectAssetContainer container, String dirPath)
at uTinyRipper.AssetExporters.ProjectExporter.Export(String path, FileCollection fileCollection, IEnumerable1 assets, ExportOptions options)    at uTinyRipper.GameStructure.Export(String exportPath, Func2 filter)
at uTinyRipperGUI.MainWindow.ExportFiles(Object data)
Go to: Create issue

Please upload game files

Solution https://github.com/mafaca/UtinyRipper/issues/304


Improving compatibility with VS2017 and .NETCore projects

The existing package has a number of problems:

Usage of powershell for adding content files: these files
are no longer executed by nuget (IIRC, as of NuGet v3+)
Since they are no longer run, not only does the sample .cs
does not get injected, but neither does the target run at
all since it's also the PS1 that did the injection of an
MSBuild target.

The latter is unnecessary since NuGet already supports providing
targets via nugets. This is added by providing the new InjectModuleInitializer.targets
as part of the nuget package.
WRT to the sample content file, since it adds little value and
it will be tricky to maintain in a compatible way across content vs
contentFiles and packages.config vs project.json/PackageReference in
VS2015+, I think it's simply better to just improve the readme.txt.
After all, it's just a simple class with one method. No big deal copying
it from the readme after installing.

Thanks for the pull request. Will merge it and publish when I can, right now I don't have a Windows machine available, not sure if I will be able to compile this with .NET Core.

Hi there, it's been a while ;)

Will be available as soon as nuget.org indexes it at https://www.nuget.org/packages/InjectModuleInitializer.Unofficial as v2.0.0


Published a version, unpublished it, then made a code change and republished it with the same version number.  VSCode Extension install is not picking up the newer published version.

I published n Extension version 1.0.2 with a specific Publisher value in the package.json. Then I unpublished it 5 minutes later, made a code change to alter the publisher to our company standard,  rebuilt the VSIX and republished with the same version number.
I then attempted to install this new published version from within VSCode  but VSCode appears to be caching the older version and installed that instead.
I went to the Marketplace website for the extension - where you have the option to Install the Extension or 'Download Extension"
I chose now to download the Extension and a new VSIX File was downloaded to my desktop.
I then proceeded to install that VSIX and it works fine and it has the correct publisher value.
After 4 hours, I tried to install the Extension via VSCode again and it still fails by picking up some older cached version.
It does appear that publishing, then unpublishing and republishing with the same version number does cause VSCode some grief but it should still permit it having unpublished the original.  I can find no documentation on this issue.
Having installed the extension from within VSCode , I  went to the .vscode directory on my laptop  and looked at the code that was installed and sure enough it was the code prior to my code change.
Desktop (please complete the following information):

OS:  Windows 10
Browser Edge
VSCode version : 1.62.3

Hi @davidkjackson54 ,
Thanks for reaching VS Marketplace Team, are you still facing the issue?
Regards,
VS Marketplace Team

Hi @davidkjackson54,
Since we have not received any response from you, we are closing the issue. If the issue still persists, please reopen or create a new request.
Regards,
VS Marketplace Team


[Chore] Remove the outdated Create React App

https://www.geeksforgeeks.org/migrating-from-create-react-app-to-nextjs-a-practical-guide/
https://stackoverflow.com/questions/77621320/im-trying-to-deploy-my-react-app-with-vercel-i-did-every-step-in-my-terminal-b
https://react.dev/learn/start-a-new-react-project
https://stackoverflow.com/questions/78646710/how-can-i-get-a-compatible-react-package-ecosystem-having-both-react-scriptsx-a
https://dev.to/henriquejensen/migrating-from-create-react-app-to-vite-a-quick-and-easy-guide-5e72

related: #76

https://github.com/CreativeTechGuy/ReactTemplate

temporary solve with adding the overrides
https://github.com/1chooo/1chooo.com/blob/59c96f9b1e043abce3d447011a7c82fc43712928/package.json#L28-L30


Provide an easy way to disable command decorations

Currently you can disable them via:
  "terminal.integrated.shellIntegration.commandIconDefault": "",
  "terminal.integrated.shellIntegration.commandIconError": "",
  "terminal.integrated.shellIntegration.commandIcon": "",

But this is not particularly easy and the decorations still end up rendering. We should have a switch that disables them outright, including adding them to xterm.

It is enabled even though shell integration is off... But thanks for the fix, now i can go back to using the integrated terminal!

@pitust they're rectangles so you're probably on a pretty old build (1 week+?), updating will probably fix it for you.

@meganrogge are there any verification steps?

It's still unreleased, but use terminal.integrated.commandDecorationsEnabled:false to disable them.


Fix button padding

WHAT
Inconsequential update to the spec
WHY
Tiny diff from the spec
HOW
Checklist :clipboard:

[ ] Add explicit visibility modifier and explicit return types for public declarations
[ ] Run spotless check
[ ] Run tests
[ ] Update metalava's signature text files

Trivial, not waiting for review.


Errors in XCode 7

In XCode7 I get quite a lot of errors in SobrCameraView.swift. Any plans to update ?
Thanks
Frank

Hi Frank, atm it isn't updated to Swift 2.0. At the moment there aren't any plans to upgrade it, because there are several solutions which are already upgraded. Nevertheless feel free to open a pull request.

Hi, what do you mean by "there are several solutions which are already upgraded"?

What are the other solutions?  I can't seem to find any other similar libraries in swift...

@michaelbina : I switched back to IPDFCameraViewController (in ObjC). Compiles fine in XCode7.

What was the point of creating this library as an alternative to IPDFCameraViewController if its not going to be updated?

Pod updated to version 0.2.0. Example is working now with Xcode 7.1.1

Thanks @katunch !


toco crashed while change model to lite

I use toco to change model to lite format, it crashed like the following statck (PS: toco was compiled in my own machine with the tensorflow code. The git commit is 9fb9ac66ce5236ff045630cad6793f0531bf9d2c, with date Sun Jan 21 07:43:08 2018 +0800 )
Program received signal SIGSEGV, Segmentation fault.
0x000000000046bdf8 in void toco::(anonymous namespace)::EvaluateBinaryOperatorOnConstantInputs<(toco::ArrayDataType)2, (toco::ArrayDataType)2>(toco::Model*, toco::Operator const*) ()
(gdb) bt
#0  0x000000000046bdf8 in void toco::(anonymous namespace)::EvaluateBinaryOperatorOnConstantInputs<(toco::ArrayDataType)2, (toco::ArrayDataType)2>(toco::Model*, toco::Operator const*) ()
#1  0x000000000046e3e9 in toco::ResolveConstantBinaryOperator::Run(toco::Model*, unsigned long) ()
#2  0x0000000000489dd3 in toco::(anonymous namespace)::GraphTransformationsPass(int, toco::Model*, toco::GraphTransformationsSet const&) ()
#3  0x000000000048aae0 in toco::RunGraphTransformations(toco::Model*, std::string const&, toco::GraphTransformationsSet const&) ()
#4  0x000000000042b608 in toco::Transform(toco::TocoFlags const&, toco::Model*) ()
#5  0x000000000041c5e8 in main ()
It can reproduce by unzip my attached file
test.zip
, and run the command
toco --input_file=/home/jeyawn/ML/test.pb --output_file=/home/jeyawn/ML/test.lite --input_format=TENSORFLOW_GRAPHDEF --output_format=TFLITE --inference_type=FLOAT --input_shape=1,368,368,3 --input_array=image --input_data_type=FLOAT --output_arrays=Openpose/MConv_Stage6_L1_5_pointwise/BatchNorm/FusedBatchNorm,Openpose/MConv_Stage6_L2_5_pointwise/BatchNorm/FusedBatchNorm
Following is my env:
== cat /etc/issue ===============================================
Linux jeyawn-virtual-machine 3.13.0-128-generic #177-Ubuntu SMP Tue Aug 8 11:40:23 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
VERSION="14.04.5 LTS, Trusty Tahr"
VERSION_ID="14.04"
== are we in docker =============================================
No
== compiler =====================================================
c++ (Ubuntu 4.8.4-2ubuntu1~14.04.3) 4.8.4
Copyright (C) 2013 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
== uname -a =====================================================
Linux jeyawn-virtual-machine 3.13.0-128-generic #177-Ubuntu SMP Tue Aug 8 11:40:23 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
== check pips ===================================================
numpy (1.8.2)
protobuf (2.5.0)
== check for virtualenv =========================================
False
== tensorflow import ============================================
Traceback (most recent call last):
File "", line 1, in
ImportError: No module named tensorflow
== env ==========================================================
LD_LIBRARY_PATH is unset
DYLD_LIBRARY_PATH is unset
== nvidia-smi ===================================================
nvidia-smi: command not found
== cuda libs  ===================================================

Nagging Awaiting TensorFlower: It has been 14 days with no activity and the awaiting tensorflower label was assigned. Please update the label and/or status accordingly.

Nagging Awaiting TensorFlower: It has been 14 days with no activity and the awaiting tensorflower label was assigned. Please update the label and/or status accordingly.

Nagging Awaiting TensorFlower: It has been 14 days with no activity and the awaiting tensorflower label was assigned. Please update the label and/or status accordingly.

Nagging Assignee @cy89: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @cy89: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @cy89: It has been 114 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @cy89: It has been 129 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

I ran your test.zip file through toco and it worked for me. Please reopen if it still doesn't work when you run through a recent version of TF Lite.


How expose error from API

Environment

Ember Version: 2.18
Ember CLI Version: 2.18.1
Ember CP Validations Version: v4.0.0-beta.6

Steps to Reproduce
How I can handle with validation from server.
My backend return errors like this:
return new Response(422, { 'Content-Type': 'application/json' }, {
        error: {
          message: {
            amount: ['amount error']
          },
          status: 422
        }
      });

this format is totally different from the ember-cp-validations standard. How can we handle client side and server side validations uniformly?

cc: @offirgolan

I am facing a similar issue. I am using ember-apollo-client to fetch errors using the graphql spec. I understand that by default the errors are picked up from DS.Model. But I am using an EmberObject. If you could at least point me in the right direction, that would be great.
Thanks for the amazing library!

@villander Even I am looking for something on the similar grounds. Please check if https://github.com/offirgolan/ember-cp-validations/issues/110 is useful.


Ambient CNI informer seems to get spurious EventDelete

Bug Description
In this function: https://github.com/istio/istio/blob/master/cni/pkg/ambient/informers.go#L82 I have noticed (via the addition of debug logging) that weirdly, we can get spurious EventDeletes for ztunnel pods during creation even if the pod is not being deleted or rescheduled.
The spurious EventDeletes will have a nil event.Old - "normal" EventDeletes from a real ztunnel pod deletion will not - current code will implicitly ignore the former case, but it still seems strange.
Version
Current `master` as of 5/4/2023

Additional Information
No response
Affected product area

[X] Ambient
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Control Plane Revisions

https://github.com/istio/istio/blob/73c4ea182b611bb54c10ac4c5cc4cf30f84f6588/cni/pkg/ambient/informers.go#L74 would give no Old which seems like a bug (with no impact, as far as I can tell).
doesn't explain spurious though

When a ztunnel spins up, istio-cni will reconcile all pods on the node
https://github.com/istio/istio/blob/05580f5d5e248a9aa46079909661837089a6c1a5/cni/pkg/ambient/server.go#L252
and trigger delete events for pods without ambient enabled
https://github.com/istio/istio/blob/05580f5d5e248a9aa46079909661837089a6c1a5/cni/pkg/ambient/informers.go#L73-L78
Not sure why we need this

@dddddai ah that's it. Also not sure. We don't AFAICT, that seems like junk events.


Different overlay locations for resizable mode

Overlays such as boost information and screen markers "move" position depending on whether the user is in fixed or resizable mode. They don't actually move, but they shift to different locations since the client resizes. This leaves them possibly off-screen or moved to other locations depending on how the client is resized.

I'm pretty sure we've long since updated our overlay code to ensure overlays don't get pushed offscreen when switching from resizable to fixed view. Can you provide a clear example of this happening?

They don't seem to move off-screen anymore (as you stated), but they still move. I'm really just asking for a sort of toggle or mode specific location.

Here is the fixed version, the blue square is for deposit all in bank. The green square is the same, but when in resizeable. Shown here

Really a QoL to maybe be able to have positions for both fixed and resizeable for overlays.


Update version number for previous breaking changes

https://github.com/szeged/blurz/pull/8 added a parameter to some public method, so code built with 0.2.2 does not compile with 0.2.3. or 0.2.4. Please yank these two versions and republish with an appropriate version number.

Done

Thanks!


Haoyus_Pizza_Challenge

Make Jan's ISlinfitter benchmark run-able, to fit Dan Welty's data but assuming we already know there is going to be 5 components.

Restart from a new master branch


Install Fails With Peer Dependency Warning

$ npm -g i gatsby-plugin-s3
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
$ gatsby --version
2.4.11
$ npm run build

><EMAIL_ADDRESS>build /mysite
> gatsby build

success open and validate gatsby-configs â 0.331 s
error Unable to find plugin "gatsby-plugin-s3". Perhaps you need to install its package?

  Error: Unable to find plugin "gatsby-plugin-s3". Perhaps you need to install its package?
...
...
...
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `gatsby build`
npm ERR! Exit status 1
...
...
$

You're trying to install gatsby-plugin-s3 globally (with the -g flag). Try installing it locally instead.

Ok. That worked.


Add user facing setting for persisting headers

Description
Adds a user control in the settings dialog for persisting headers. This is only visible if the shouldPersistHeaders is not passed to the GraphQL component.
Demo and brief explanation: https://www.loom.com/share/b958369673eb4ecbafcd2d1fe1fa0fe9
I also found 2 potential issues when working on this:

https://github.com/graphql/graphiql/issues/2891 (Accessibility related, I think the ButtonGroup we use in the settings dialog doesn't have the correct accessibility behavior and attributes. I decided to keep it as a separate issue to keep this PR smaller, I'll work on that one once this one is completed)
https://github.com/graphql/graphiql/issues/2892

Note for the future: I think it would be nice to somehow refactor this out into a user settings module, not the UI but the logic to store and access user settings. At the moment we have Theme and now Persist Headers, but there could be more user controlled settings going forward and having a central place for this logic (including defaults and such) could be nice.
Questions

As headers are not persisted as a security measure, should we include a warning for the user in the dialog?
I added logic to cleanup the headers storage if this option is turned off, do we also need to do the same for tabs state?

thank you so much for this @TheMightyPenguin !

should we include a warning for the user in the dialog?
I think so, perhaps just a warning to make sure they trust the device they are persisting these secrets to.

do we also need to do the same for tabs state?
it would be lovely if you could cover that case as well!

@TheMightyPenguin merged an upstream PR where @jonathanawesome fixed the act() issue, sorry for the merge conflicts!

This is only visible if the shouldPersistHeaders is not passed to the GraphQL component.

This seems like a sensible choice, however - what if this prop just determines the default value, and the user can choose to override it?

This is only visible if the shouldPersistHeaders is not passed to the GraphQL component.

this seems sensible! what do you think of this?

if shouldPersistHeaders isfalse explicitly, we should not show the setting
otherwise, if it is true or not specified, the user defined setting is available, defaulting to enabled/disabled respectively

I like it, seems considerate with the end user being in control of their privacy, I'll include this logic in the next commit!

@acao updated PR, now we only show the setting if the property is not set (undefined) or true, and when it's set to false we also clean up the tabs storage, removing any headers key in there, I added tests for this logic.
I also added a warning message (using the warning color, let me know is the styling is not right 😅):

I'm not sure if we should add the state for user-controlling this setting to the EditorContextProvider. I feel that this is more of a thing that should happen in user-land (from the perspective of @graphiql/react).
To give an example: If I'm building my own custom GraphQL IDE with the components from @graphiql/react, I might have a clear opinion on how I want to persist headers or not. I already have the shouldPersistHeaders prop that I can use to control this behavior. Changing this state with the setShouldPersistHeaders method added in this PR gives me too much control that I don't need.
My proposal here is to move the state for the new toggle button in the settings dialog to the GraphiQL component. This component can combine this state with the value it receives as shouldPersistHeaders prop and pass the "final" value for that prop to the GraphiQLProvider. It also passes the value and the state-setter down to the GraphiQLInterface component which will use it to render the toggle button in the settings dialog. That way we don't need to expose any more state or methods from the EditorContext which keep it more easy to use.
Finally: Having a chunk of logic in the EditorContextProvider that clears the persisted header state when the shouldPersistHeaders prop changes is a fantastic addition, so we should definitely leave that in! (In fact, I would even consider it a bug that we're not doing it yet.)

@thomasheyenbrock I tried implementing this, but there's an issue as the GraphiQL would not have access to the storage context. We could make this work somehow but not sure how clean the solution would be 😅. Gonna think this one a bit more to see what workaround we could use.
Additionally, I'd say we could always show the setting in the GraphiQL component, removing the logic we have for showing this in only certain scenarios (e.g. when the prop is true or undefined) would simplify the code.

@acao @thomasheyenbrock pushed a few commits addressing the comments!

Looks good to me, thanks @TheMightyPenguin for the thorough PR!

@acao @thomasheyenbrock @jonathanawesome anything missing from my side in this PR? (just making sure I am not blocking anything)


👷 chore(ci): Add missing CI step for Playwright

Current dependencies on/for this PR:

new-dawn

PR #1059   👈

This stack of pull requests is managed by Graphite.


Not working on dezeen.com

Example:
https://www.dezeen.com/2022/05/03/studio-drift-elbphilharmonie-drone-installation-cancelled-breaking-waves/
Working fine everywhere else.

Oh, I didn't remember this, but I just checked and:

When I click the extension, the pane that appears and lets me edit and download the clipping, only shows the loading animation and doesn't load anything
It happens on all Dezeen pages, I can't replicate the error with any other web page right now, I don't remember if it happened anywhere else
I reinstalled Windows 10 last week and just installed MarkDownload when I read your message, so this should be a clean install, I haven't even touched the config
This is not really a problem for me, MD works fine except for Dezeen, I just wanted to inform about a possible bug

I encoutered the excat same thing with this website under edge and brave: https://wikipedia.thetimetube.com/?lang=de&q=Radbruchsche_Formel
Can you confirm?

Confirmed, same problem with this site. Tested with Brave.


problem with new migration 2.0.6

hi all im trying this
'id' => Schema::TYPE_STRING . '(64) PRIMARY KEY',
but in the new migration i cant make this
'id' => $this->string(64)->primaryKey();
is a bug or this not exist in this case i cant define string to primary key ?
my suggestion is modify the primary key for support int and strings
$this->string()->primaryKey()->autoincrement(false);
or
$this->string->primaryKey($autoincrement = false);
i need this for the table sessions in this case
$this->createTable('session',[
            'id' => Schema::TYPE_STRING . '(64) PRIMARY KEY',
            'user_id' => Schema::TYPE_INTEGER,
            'ip' => Schema::TYPE_STRING,
            'is_trusted' => Schema::TYPE_BOOLEAN,
            'expire' => Schema::TYPE_INTEGER,
            'data' => Schema::TYPE_STRING,
            'controllerCurrent' => Schema::TYPE_STRING,
            'actionCurrent' => Schema::TYPE_STRING,
            'pageCurrent' => Schema::TYPE_STRING,
            'is_guest' => Schema::TYPE_BOOLEAN,
        ],$tableOptions);

in this case i don't need autoincrement,etc and the string is primary key

The issue related with yiisoft/db, primaryKey is a part of Yiisoft\Db\Schema\Builder\AbstractColumn class.
https://github.com/yiisoft/db/blob/4f1dbb95c9551b4359cdb2b188d0094bd70aa138/src/Schema/SchemaInterface.php#L124
https://github.com/yiisoft/db-mysql/blob/d48ca3a6ad83abf6ed97e146185c76c5ce2e55ab/src/QueryBuilder.php#L20
And this should be solved there


creates file watcher with trailing slash causes problems for other extensions

Investigating https://github.com/microsoft/vscode-spring-boot-dashboard/issues/324 and https://github.com/spring-projects/sts4/issues/1231 revealed that the Project Manager for Java extension in VSCode seems to create a file watcher that contains a trailing slash for the root path, which should not be done (according to https://github.com/spring-projects/sts4/issues/1231#issuecomment-2061156798).
It causes problems down the road with other extensions, since future file events when contain a wrong URI for the changed file, containing a duplicated / between the root path and the relative file path in the workspace.

@testforstephen Would be awesome if you could have a look.

LGTM, thanks for this fix. Much appreciated.


day 5, part 2 bug

I found your coding while debugging mine and yours day5 part2 gives too high results. Just letting know :)

Thanks. I guess it has to do with the issue I noted in grow_seed():

Note: This currently doesn't properly support splitting when "coming from below". For example, something like (11, 11) would not be correctly split at 20. However, these situations do not seem to happen with the full tables.

This didn't affect me and my input, but I guess it can affect some inputs. I'll find a more solid implementation and let you know :)

Thanks again for your note, @misiek08 .
I think my new version should handle this edge case properly.


fix categorical labels handling in what if panel

Fixes issue:
https://github.com/microsoft/responsible-ai-widgets/issues/314
By passing indexes instead of raw values for true_y and predicted_y columns when needed.
The what if panel now looks like:

Would be nice to have tests for this. :-)


Create new packaged helm repos to support terraform installations for multiple release versions

Terraform helm provider installations need packaged helm repos to download and install charts and since we have only one such repo https://charts.rancher.io/ it is not possible to support multiple released versions at the same time.
This is leading to charts missing in the packaged repo.
Ex issue: https://github.com/rancher/rancher/issues/37601
To fix this, we need to create additional repos for each version.
SURE-5346

I did some testing in Rancher 2.5, 2.6, and 2.7. I noticed that for Rancher 2.5, the package repository points to the master branch of the rancher/charts repository and we cannot change it due to some incompatibility. However, for Rancher 2.6 and 2.7, it is possible to point to specific branches of rancher/charts, therefore allowing installation of specific charts releases.
For Rancher 2.6, I did the following test:

Install Rancher.
In the app repositories, create a new repository named new-charts pointing to rancher/charts branch release-v2.7.
Create a Terraform template that installs a package version that exists only in release-v2.7.
Try to install from the rancher-charts repository (v2.6).
Check that it fails as expected.
Change the template to point to the new-charts repository.
Try applying the template again and check that it succeeds as expected.

@mitulshah-suse since the Rancher documentation currently states that Rancher 2.5 is at end of maintenance, and this requirement works for more recent versions, do you think it would be alright to close this issue?

Closing due to the original reason for filing is no longer relevant. See the previous comment for more details.


EC2: More filters for DescribeNetworkAcls

Adds support for additional filters for EC2 DescribeNetworkAcls operation (documented here)

Codecov Report

Merging #5049 (d7ee0ed) into master (61a5d5c) will decrease coverage by 0.00%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master    #5049      +/-   ##
==========================================
- Coverage   95.80%   95.79%   -0.01%
==========================================
  Files         663      663
  Lines       66468    66476       +8
==========================================
+ Hits        63680    63684       +4
- Misses       2788     2792       +4

Flag
Coverage Δ

servertests
36.56% <0.00%> (-0.01%)
:arrow_down:

unittests
95.73% <50.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

moto/ec2/_models/network_acls.py
83.46% <50.00%> (-2.25%)
: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 61a5d5c...d7ee0ed. Read the comment docs.

The describenetworkAcls API seems to define more filters.  Is there a reason to exclude them, asking out of curiosity.  Thanks