Something went wrong during completion. Reason: 'NoneType' object is not subscriptable

mongo-express         | Mongo Express server listening at http://<IP_ADDRESS>:8081
mongo-express         | Server is open to allow connections from anyone (<IP_ADDRESS>)
mongo                 | {"t":{"$date":"2023-03-04T05:13:52.593+00:00"},"s":"I",  "c":"NETWORK",  "id":22943,   "ctx":"listener","msg":"Connection accepted","attr":{"remote":"<IP_ADDRESS>:51322","uuid":"e0994f27-2006-4d7e-9560-9b2e2eac37ed","connectionId":4,"connectionCount":4}}
mongo                 | {"t":{"$date":"2023-03-04T05:13:52.595+00:00"},"s":"I",  "c":"NETWORK",  "id":51800,   "ctx":"conn4","msg":"client metadata","attr":{"remote":"<IP_ADDRESS>:51322","client":"conn4","doc":{"driver":{"name":"PyMongo","version":"4.3.3"},"os":{"type":"Linux","name":"Linux","architecture":"x86_64","version":"4.18.0-408.el8.x86_64"},"platform":"CPython 3.8.16.final.0"}}}
chatgpt_telegram_bot  | Something went wrong during completion. Reason: 'NoneType' object is not subscriptable

Need to select a mode first.

it works


correct menu bar item offset

As soon as the viewControllers of the PageController were set a second
time the items of the menu bar were not centered correctly anymore.

Thanks, @FBente . And sorry too late.


Persistent sort order

When I call the loadData function, the sort order does not persist. Is there an easy way to do this? The data gets refreshed when I get an event from the database but I'd like to keep the grid where it was. It also looks like the grid is trying to persist the sort as the icon for the column sorted is still present, however, the data isn't updated to reflect that.

@ragerory This worked! Thank you so much!. I've been searching everywhere and banging my head trying to find a solution for sorting. Thank you!

Thanks! That works, and I'm really curious why it does not work when writing to the done() in the controller.


IPv6 support

Hi,
do the Network and Network Range columns support IPv6?

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

ID: 2f8119c9-61af-b469-af98-ddd37764c0ca
Version Independent ID: 7d829a74-9df7-5b6f-8e1c-44d3e5d38f75
Content: Adding and updating Locations data
Content Source: Teams/learn-more-about-site-upload.md
Service: msteams
GitHub Login: @tonysmit
Microsoft Alias: lolaj

@eimann
Thank you for submitting this feedback.
I will get this over to for investigation. Thank you for reporting and making the docs better. Much appreciated.

@officedocsbot assign @scanum

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

@officedocsbot close

Hi @scanum,
thanks for your feedback. I don't have a service issue, I have a documentation issue.
From my understanding the documentation should reflect if there's feature parity between IPv4 and IPv6 for Network Locations or not. In my opinion, this is something product management or engineering can provide for this specific component.
Cheers,
Dominik


npm test fails on Windows

VSCode Version: 1.4
OS Version: Windows 10

I don't know if I'm doing this right, but I'm trying to set up my extension up on AppVeyor to run tests. It works fine on Jenkins (Linux and OS X) but fails on AppVeyor. I can repro on my Windows desktop.
I'm running npm test (I presume this is correct - it seems to do the expected thing on Linux/Mac) but I get this output:
npm test --silent
### VS Code Extension Test Run ###
Current working directory: C:\projects\dart-code
Downloading VS Code into "C:\projects\dart-code\.vscode-test" from: https://vscode-update.azurewebsites.net/1.4.0/linux-x64/stable
Running extension tests: C:\projects\dart-code\.vscode-test\VSCode-linux-x64\code C:\projects\dart-code\out\test --extensionDevelopmentPath=C:\projects\dart-code --extensionTestsPath=C:\projects\dart-code\out\test
Failed to execute tests: Error: spawn C:\projects\dart-code\.vscode-test\VSCode-linux-x64\code ENOENT
Tests exited with code: -4058
Command exited with code 1

It seems to be downloading a Linux version of code. I tracked this back to node_modules/vscode/bin/test which has a platform check that only supports Linux and Mac:
var platform = (process.platform === 'darwin') ? 'darwin' : 'linux-x64';

So, two questions:

What's the correct way to run tests on Windows/AppVeyor (I found this though I'm not sure if relevant)
Is there any reason npm test can't be made to work? All the pieces are cross-platform, it would be nice if this just worked the same everywhere

Sorry, but we do currently not support running integration tests for extensions on Windows. See https://code.visualstudio.com/docs/extensions/testing-extensions

This issue was moved to Microsoft/vscode-extension-vscode#30


duplicate entry: android/support/v13/view/DragAndDropPermissionsCompat.class

phonegap-plugin-push seems to be conflicting with two other plugins. I have been fighting this odd issue for days now and I have no idea what the problem is.
Expected Behaviour
The build should pass.
Actual Behaviour
The build fails with this error message:
BUILD FAILED

Total time: 45.755 secs
Execution failed for task ':transformClassesWithJarMergingForDebug'.
> com.android.build.api.transform.TransformException: java.util.zip.ZipException: duplicate entry: android/support/v13/view/DragAndDropPermissionsCompat.class

Reproduce Scenario
Steps to Reproduce
Create a new Cordova project:
cordova create test
cd test

Install<EMAIL_ADDRESS>cordova platform add<EMAIL_ADDRESS>
Install the following plugins:
cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>cordova plugin add<EMAIL_ADDRESS>
Both of those other plugins alone will cause the issue but they work just fine together without the push plugin.
Add your google-services.json file so you don't get errors about that.
Attempt to build (it shouldn't work):
cordova build android

Now remove phonegap-plugin-push and build again (it should work):
cordova plugin rm phonegap-plugin-push
cordova build android

Cordova CLI version and cordova platform version
cordova --version                                    8.0.0
cordova platform version android                     6.3.0

Plugin version
cordova plugin version | grep phonegap-plugin-push   2.1.3

Android SDK packages installed:

Path
Version

build-tools;27.0.3
27.0.3

emulator
27.1.10

extras;android;m2repository
47.0.0

extras;google;g...e_play_services
46

extras;google;m2repository
58

patcher;v4
1

platform-tools
27.0.1

platforms;android-26
2

platforms;android-27
1

tools
26.0.1

The solution in this comment on #2229 seems to fix the problem!
I'm going to leave the issue open though because the solution is only a workaround.

update this plugin to 2.2.1 solved this issue for me

I believe that solved the issue for me too but unfortunately, that is not a viable option. 2.2.1 requires cordova-android 7.1.0 which I cannot upgrade to because of lack of compatibility with other plugins.

Upgrading til 2.2.1 fixed my problems as well

@nprail oh really ?!! where did u read that ? i checked all changes and can't find something like that
because if that true i will have a big problem too

@alaa-alshamy : Did you try it ? If yes which parameter you changed ?
I mean in config.xml file

Did you changed spec version of this plugin ?

yep sure i tried that, u can make so in several ways u can remove the plugin and reinstall it or u can change the spec in the config.xml and run 'cordova prepare' or whatever u have to reinstall the plugins
@nprail and yea if what did u say true it should throw an error i think but i installed the plugin and made a build successfully

@nprail same root cause as #2229. I was travelling all weekend so I didn't get a chance to reproduce. I'll try to get to the bottom of it today or tomorrow.


Move towards pyproject.toml standard

I am packacking the dicarlo conversion and this is a good time to address this long standing issue and getting more familiar with the packaging and dependency managment process.
This is the first attempt at moving our static configuration to pyproject. The dynamic configuration is the spiky part and I expect this to fail until I tune that:
https://setuptools.pypa.io/en/latest/userguide/pyproject_config.html#dynamic-metadata
Let's see

Thanks for taking this on
Good luck!
I'd also be willing to reopen the old discussion about whether or not we should only have local requirements.txt files per data format, that magically get parsed during a very programmatic setup call, instead of having one central list of all optional dependencies
The benefit of a single list is (i) it's easier to check when cutting a release if we have any dev branch pins, (ii) it's easier to spot conflicts in version ranges across the repo
Granted though, I wonder, is there a way in pyproject.toml to also declare full (or other keys) to be set-based intersection of other optional dependencies? Or must it all be fully explicit?

Good luck!

x D

I'd also be willing to reopen the old discussion about whether or not we should only have local requirements.txt files per data format, that magically get parsed during a very programmatic setup call, instead of having one central list of all optional dependencies

The benefit of a single list is (i) it's easier to check when cutting a release if we have any dev branch pins, (ii) it's easier to spot conflicts in version ranges across the repo

That was my preference then as is my preference now. It will be a lot of work to change though. Back then also Ben pointed out that this was better to achieve a pluggin architecture. Not sure I understood completly but I thought that it was about only doing local changes.

Granted though, I wonder, is there a way in pyproject.toml to also declare full (or other keys) to be set-based intersection of other optional dependencies? Or must it all be fully explicit?

You can do stuff like this but I am not sure it works as intended.
dev = [
    "spikeinterface[test]",
    "spikeinterface[test_core]",
    "spikeinterface[docs]",
    "black",
    "pre-commit",
]

Relevant:
https://setuptools.pypa.io/en/latest/build_meta.html#backend-wrapper

OK, the installation part with pip locally works quite well. We have to ensure that the build process for sdist and wheels works.

This should be ready @CodyCBakerPhD

Interesting, so you're saying we can mix custom Python logic for finding extra dependencies in setup.py as well as explicitly written extras in the .toml?

Interesting, so you're saying we can mix custom Python logic for finding extra dependencies in setup.py as well as explicitly written extras in the .toml?

I am kind of suprised it works tbh. I was expecting that defining some optional dependencies in the pyproject would be exclusive or with extras_require with the setup.py but at last this does not seem to be the case.


Error upon creating World.

Upon creating a new world the console gets filled with a largeeee amount of error messagges  and the server stops .
Mods/Plugins present :
Pixelmon 4.2.5 -
Custom NPCS 1.8.9 -
EssentialCmds-8.1.1 -
spongeforge-1.8.9-1808-4.1.0-BETA-1303 -
worldedit-forge-mc1.8.9-6.1.1-dist -
Error report 1 : http://pastebin.com/7v2572PY
Error report 2 : http://pastebin.com/aNf76NHM

What Forge version are you running on?

Is this still happening?


Allow default stubs to render slot content

What problem does this feature solve?
Please check this example: https://codesandbox.io/s/6zmj357qrn
In my app a have pages designed with various layouts, each layout is a component composed of other tiny components (logo, profile, navigation etc.). A page itself is a component matching a route (e.g. /jobs > Jobs);
So I decided to go with slots:
I have a set of "layout" components describing visual wrappers. And a set of "page" components that describe app's routes.
When I test "page" component's logic, I should not care of "layout" (should be tested separately, right?) So I'd really like to use shallowMount everywhere and don't stub all the children. The problem is that shallowMount doesn't seem to allow that, while mount does.
Is there any way to use this technique and still be able to go with shallowMount over mount?
What does the proposed API look like?
Not sure

This is actualy already implemented by https://github.com/vuejs/vue-test-utils/pull/782!
This will be released in beta.21 which will be published within a week.


poetry install issue on windows

Collecting uvloop<0.18.0,>=0.17.0 (from appthreat-chen)
  Downloading uvloop-0.17.0.tar.gz (2.3 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 2.3/2.3 MB 2.6 MB/s eta 0:00:00
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [20 lines of output]
      Traceback (most recent call last):
        File "C:\Users\prabhu\AppData\Local\Programs\Python\Python311\Lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 353, in <module>
          main()
        File "C:\Users\prabhu\AppData\Local\Programs\Python\Python311\Lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "C:\Users\prabhu\AppData\Local\Programs\Python\Python311\Lib\site-packages\pip\_vendor\pyproject_hooks\_in_process\_in_process.py", line 118, in get_requires_for_build_wheel
          return hook(config_settings)
                 ^^^^^^^^^^^^^^^^^^^^^
        File "C:\Users\prabhu\AppData\Local\Temp\pip-build-env-kr6yxyu5\overlay\Lib\site-packages\setuptools\build_meta.py", line 355, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=['wheel'])
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "C:\Users\prabhu\AppData\Local\Temp\pip-build-env-kr6yxyu5\overlay\Lib\site-packages\setuptools\build_meta.py", line 325, in _get_build_requires
          self.run_setup()
        File "C:\Users\prabhu\AppData\Local\Temp\pip-build-env-kr6yxyu5\overlay\Lib\site-packages\setuptools\build_meta.py", line 507, in run_setup
          super(_BuildMetaLegacyBackend, self).run_setup(setup_script=setup_script)
        File "C:\Users\prabhu\AppData\Local\Temp\pip-build-env-kr6yxyu5\overlay\Lib\site-packages\setuptools\build_meta.py", line 341, in run_setup
          exec(code, locals())
        File "<string>", line 8, in <module>
      RuntimeError: uvloop does not support Windows at the moment
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

@cerrussell, I have made some improvements to make the project install on windows. But could you also take a look at your convenience?

This workflow might be helpful for testing the Windows install...
 As you can see, the poetry install runs ok, but chen --download and chen --download --with-science have some issues.
As of now, python 3.8 - 3.10 are able to do chen --download while 3.11 and 3.12 cannot as they can't build a wheel for rocksdb-py.
Chen --download --with-science doesn't work on any of the versions due to not being able to build a wheel for pygraphviz.

Try the new conda-based installer which is working reliably.


Build task not detected on first try

VSCode Version: Code - Insiders 1.15.0-insider (ae73f8ee6f8ddb87b5e30d31655d9f55a53f4a95, 2017-07-26T07:47:53.848Z)
OS Version: Windows_NT x64 10.0.15063
Extensions:

Steps to Reproduce:

Start a fresh vscode
Have a build task
ctrl-shift-b
Error:

ctrl-shift-b again
now it is detected

Dupe: https://github.com/Microsoft/vscode/issues/31250


simplify types with recursive type

This PR removes the limitation of the number of parameters.

@sokra Can you review this PR? Thanks.


Fix potential NPE if code is executed concurrently

This PR is an attempt to fix the NPEs seen on flaky tests like https://builds.gradle.org/viewLog.html?buildId=34721592&buildTypeId=Gradle_Check_Quick_1_bucket5
I couldn't reproduce the NPE locally on the same tests and JDK, but from the location of the NPE in the code one reasonable explanation is that for some reason this code could be executed concurrently. Therefore this commits adds synchronization around the datastructure to avoid NPEs.
If this works it's a candidate for a potential 6.4.1.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.


Packages fail to upload

  00:53:44 | [ERROR] ('Storage requirements exceeded<PHONE_NUMBER>00 bytes).', 402)

@kgugala @PiotrZierhoffer do you have a clue of what we did last time to solve this?

Removing some of the older https://anaconda.org/LiteX-Hub/open_pdks.sky130a packages.


[RFC] Allow an empty for statement to be used as an infinite loop.

Most C-family languages don't have a dedicated infinite loop. Instead they use constructions such as for(;;), while(1), while(true) or do..while(true) of which perhaps the first one is the most common.
An exception to this is Go which uses for as a dedicated infinite loop.
Currently, in Wren most folks seem to use while (true) for this purpose though the more cryptic while (1) or even while (0) - as any number is truthy - are also options.
What I'd like to propose is that we also allow an empty for statement, for () to represent an infinite loop. This is shorter and looks more like a dedicated loop than while (true) and would bring us more in line with what other languages do. If we had it, I personally would use it all the time for this purpose.
There will be those who will say that we shouldn't be influenced by what other languages do. However, the truth is that Wren isn't developed in a vacuum and there's a reason why most of our keywords are the same as in other C-family languages, namely familiarity.
I look forward to hearing your views on this.

Considering low usage of it, and the lalel'ing effort that is going on, it is an ultra low priority for now to determine the correct syntax for this. It is a nice optimization to have a forever loop, but I would like to see numbers before accepting such change.

Well,  as we don't have do..while(cond), I tend to use infinite loops more in Wren than when programming say in C:
while (true) {
    // blah
    if (cond) break
}

Although I'd consider Rust to be a C-family language, like Go, it has some unusual syntax so I agree that it's not a good language to compare Wren with.

If we want a new construct for infinite loop (although I don't think this is needed), then I think for () has no advantage over while (true). I would use loop instead, since they both stands for an infinite loop ("forever" or "while true"), but the intention is to break - so loop is a better name.

I agree, and that's why I think we shouldn't do it.

I don't think for () is a good idea for the language, as it isn't self-explanatory. It could just as easily mean "skip this", "perform this action once" or "loop forever"
while(true) is a lot clearer in its behaviour as an infinite loop.

OK, I think it's already clear that the use of for() as an infinite loop is going to lack the necessary support so I'll close this now as I don't want to clutter up the issues list.
Thanks everyone for your comments.

My preference would be a simple while block
while {
  my_infinite_instructions.call()
}

I find it less readable than while (true).

Also you'd always need to use a block. In the example you give, you couldn't really put the single line of code on the same line as while because, without the following parentheses, it might just be a syntax error.


Loftsman as an Kubernetes operator?

We've done a fair amount design-wise with the manifest schema and otherwise to support the idea of making Loftsman an operator running in the Kubernetes cluster. Tracking this possibility officially here.

One idea around this is to actually encapsulate ability to deploy the operator into a cluster via the CLI, a given version of the CLI still encapsulating the version of stuff to deploy or pointers to those artifacts at least, so a workflow that looks like:
$ loftsman operator deploy  [--namespace] [other options]

which would deploy the operator as a step that is setting up your cluster not unlike initializing the cluster otherwise with needed CNIs, other base setup/config.
And then further operations would simply apply manifests to manage things in the cluster moving forward, e.g.
kubectl apply loftmanmanifest -n [loftsman manifest tracking namespace] -f ./manifest.yaml


If FTP Connection fails, there should be a non-zero return

I'm working to incorporate dploy in the new Bitbucket Pipelines. For whatever reason the FTP connection is failing. Bitbucket Pipelines seems to think everything went fine because dploy seems to be emitting an exit code of 0. For continuous integration, any process failure should result in a non-zero exit code.

Output from bitbucket pipelines (Note that $FTP_PASSWORD is a bitbucket pipeline that is successfully set up):
+ printf '$FTP_PASSWORD' | npm run deploy
npm info it worked if it ends with ok
npm info using<EMAIL_ADDRESS>npm info using<EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>
npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>
><EMAIL_ADDRESS>deploy /opt/atlassian/bitbucketci/agent/build
> dploy

– Enter the password for ftp.materialpokedex.com:

Connecting to live...
Connection failed.
Connection failed.

npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info ok

What was ok about this deploy?


Improve the jsdoc index page.

This alters the jaguarjs-jsdoc template by replacing the mainpage and other templates and adding some css.  The resulting index page lists the GeoJS version and has links to all classes and typedefs.

It looks great.


x/build/cmd/gomote: add gomote session manager package

Add a component which manages gomote instances. This component should:

[x] Manage the lifecycle of gomote sessions (handle timeouts)

This is a component of the project to revamp the security model used by gomote #47521
@golang/release

This issue has been edited. We no longer plan to manage SSH keys in the coordinator. We've shifted over to using certificates for authentication. This work has been completed.


COLR: incorporate revised semantics for sweep gradients to align with CSS

In discussion with Google, it was found that the spec for sweep gradients was potentially confusing because of ways in which it diverged from the way sweep gradients are expressed in CSS and also in Skia. This was due to misunderstanding of the  (unfortunately vague) Skia spec when COLR v1 formats were being defined. To minimize confusion, and since implementations for COLR v1 are still very early in their development, Microsoft and Google agreed to make a breaking change in the semantics for representation of sweep gradients.
Note: The formats for representing sweep gradients (PaintSweepGradient, PaintVarSweepGradient) do not need to change. The only change is to how member values are interpreted.
The draft changes from https://github.com/googlefonts/colr-gradients-spec/pull/362 need to be incorporated into OT 1.9.1.

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

ID: be879f46-c026-4a9d-1156-a3c98ae8f429
Version Independent ID: 28be1ce2-e487-b652-632f-e4dc5bb17f42
Content: COLR - Color Table (OpenType 1.9) - Typography
Content Source: typographydocs/opentype/spec/colr.md
Product: typography
GitHub Login: @PeterCon
Microsoft Alias: alib

See the OT 1.9.1 alpha for draft revisions addressing this issue.

This is fixed in OT 1.9.1. See the OT 1.9.1 beta for draft revisions addressing this issue.


[CSL-981] Fix tcp timeouts in yaml config (ms vs mcs)

Here are definitions from Constants.hs (main repo and infra):
networkConnectionTimeout :: Microsecond
networkConnectionTimeout = ms . fromIntegral . ccNetworkConnectionTimeout $ compileConfig
...
networkReceiveTimeout :: Microsecond
networkReceiveTimeout = ms . fromIntegral . ccNetworkReceiveTimeout $ infraConstants

Values in .yaml config were ~10^3, so several milliseconds as parsed/used.

LGTM but maybe they should be even longer.

No, 1000000 # 1 sec is actually going to be 1000 sec because we use ms in Constants.hs. ms converts milliseconds to microseconds.

1 second is still too short in my opinion, particularly for production.


disabled prop

Is your feature request related to a problem? Please describe.
I want to use disabled prop in a typescript project.
Describe the solution you'd like
One line code would be perfect disabled={inprogress}
Describe alternatives you've considered
Show/hide DatePicker with a boolean value. Please let me know if there is a better alternative?
Additional context
I think it would be great if we could pass input attributes from DatePicker component down into actual <input > tag. I know one way to do it is using renderInput but that requires more line of codes.

In style
.DatePicker {
   pointer-events: none;
}


error message fixed

#528 ... fix the error message from call

Found the same problem - will use ([system.String]::Join("`n", $result)) instead (to get line breaks)

The change showed the real problem:
[22922070] The following SQL Server error or errors occurred:  40607,"HY000",[Microsoft][SQL Server Native Client 11.0][SQL Server]Windows logins are not supported in this version of SQL Server.
at <ScriptBlock>, <No file>: line 24

#528 Generate-SymbolsInNavContainer does not support AzureSQL with configured container Authentication for DB (User/Password)

Thanks


Undocumented dependency on new-ish pip

As of now, anyone running "pip install --upgrade setuptools" on a CentOS 6 machine will find it fails horribly:
pip install --upgrade setuptools
Downloading/unpacking setuptools from https://pypi.python.org/packages/b1/d1/3df86b914d72b18d37266f964a11576f135eadbb7d2e3c5165377e4263bc/setuptools-34.0.1.zip#md5=9e907950a94d38c0295df0da2507c280
  Downloading setuptools-34.0.1.zip (617kB): 617kB downloaded
  Running setup.py egg_info for package setuptools
    Traceback (most recent call last):
      File "<string>", line 3, in <module>
      File "setuptools/__init__.py", line 10, in <module>
        from six.moves import filter, map
    ImportError: No module named six.moves
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):

  File "<string>", line 3, in <module>

  File "setuptools/__init__.py", line 10, in <module>

    from six.moves import filter, map

ImportError: No module named six.moves

I discovered this when a long-functioning build script started failing all of a sudden.
I gather from #937 and #938 that a more recent pip is required, and indeed installing it with pip install --upgrade pip does allow installation of setuptools to proceed.
May I suggest that the README should specify what version of pip is required, as it is a newer version than is present on many systems.

I don't think there's a specific version of pip that's required. It probably varies by platform and maybe by Python version. At the very least, whatever pip version is used needs to rely on the wheel format and not the sdist. If you would like to try different versions, I could update the README or the CHANGES to indicate which version to use.

I can tell you that pip 1.4.1 does not work, and pip 9.0.1 does work.
If the required version was not known/is not knowable at the point the new dependency was introduced and pushed into the wild, I guess my best suggestion is a big health warning that people should install the latest pip if they're using the latest setuptools.
Alternatively, you could indicate which version of setuptools people should pin to in order to have something that will run on stable distributions that ship an older pip.

We're seeing this too with pip 1.5.4

I'm also seeing this issue with our builds on CircleCI, running Ubuntu 12.04 (Precise).

We have a build pipeline that packs up all the sdists for our application and its dependencies and installs then using only the local index.
Does the workaround above (upgrade pip first) really depend on installing from the wheel format? I.e. is the sdist broken for setuptools?

Upgrading pip first doesn't depend on installing from wheel format. Upgrading pip is necessary to install setuptools from a wheel. If you don't install setuptools from a wheel, you need to install setuptools' dependencies first (currently appdirs, packaging, six).

Ok. Very helpful. Perhaps worthy of a release note?

@zroadhouse-rmn You mean beyond the release notes already made? The second paragraph of the change for v34.0.0 already mentions "pre-installing the Setuptools dependencies". Perhaps you could suggest some wording that would be clearer?

I think the clear wording would be something like (at the top, not in the body of text):
Important: setuptools is no longer installable with older versions of pip.  If you are installing setuptools on an older linux distribution, you may need to install the latest pip before attempting to use it to install setuptools.  If you seen an error like "ImportError: No module named six.moves", try updating pip before installing setuptools.
Because people will be running into this issue for a long time to come (these older versions of pip aren't going away any time soon), it would probably be prudent to put an ImportError handler in the code that gives people a clue about what's going on -- instead of "ImportError: No module named six.moves", it could say "Dependencies not found, are you trying to install with an older version of pip? "

@jcsp Would you be willing to put together a PR?

@jaraco - I re-read the description in changes:

Users are welcome to contrive other means to install or upgrade Setuptools using other means, such as pre-installing the Setuptools dependencies with pip or a bespoke bootstrap tool, but such usage is not recommended and is not supported.

From this description, my use case is no longer supported (installing/upgrading setuptools from sdist). I'll likely proceed with the workaround you suggested above (installing pyparsing, packaging, appdirs, and six prior to installing setuptools). Longer term, I'll need to rethink our application packaging solution, which relies on sdist vs wheel to produce platform independent packages that fully contain all python dependencies.

Ultimately fixed with setuptools 36 (see #980).


How to Build a Deep Learning Model Using Tensorflow with Python

Article checklist
Attention
Our Peer Reviewer and Content Moderator teams do NOT provide any revisions services. All revisions and edits should be completed on your own forked repo (as to not take up room in the queue) in order for our team to review them all in a timely manner.
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.

As a rule of thumb - please be sure to only submit articles (pull requests) that are fully polished and ready to be published. Be sure to go through our resources documents for extra and 3rd party (vetted) resources to help improve overall technical writing.

Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x ] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x ] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels, under 300KB in size, and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

This is close #4194

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

this is an update of #4828

Hi @lalith1403 the required changes are in line

hello @lalith1403 the requested changes are in line

@TimedH The content in the article is lacking, as discussed on slack. Please work on adding more information that explores a topic/project in depth. Thank you.

Plagiarism check complete

Hello @lalith1403 i can't see anything that's factually incorrect in this PR
cc: @ahmadmardeni1 @hectorkambow @WanjaMIKE

import TensorFlow

import TensorFlow

i used the command to import tensorflow on keras


added filter kwarg to VectorStoreIndexWrapper query and query_with_so…

Description: added filter to query methods in VectorStoreIndexWrapper for filtering by metadata (i.e. search_kwargs)
Tag maintainer: @rlancemartin, @eyurtsev

Updated the doc snippet on this topic as well.  It took me a long while to figure out how to filter the vectorstore by filename, so this might help someone else out.

thanks @ChrisPappalardo!


Addition of STM32 (H7) example project

Hello,
Thanks for this great repository! I was planning to create a similar implementation until I found this repository. Great stuff! exactly what I was looking for :)
Because I want to use this embedded-cli on STM32's MCU, I've created a compatibility layer + example project for STM32 MCU's.
Since I was documenting it for myself for later use anyways, I thought I'd create a pull request for other to use as well.
The README.md file explains how I setup the project, so others can re-create this example using other STM32's than I did. The steps should be fairly similar on all STM32's.
Feel free to change anything about the example you'd like.
Thanks again for the great functioning code!

That's a big improvement, thanks for your contribution!
There are some things I'd like to improve first:

Is it possible to ignore all HAL library files? I think they can be generated by CubeMX. So no reason to include Drivers dir. And maybe some other files.
It's a bad idea to include embedded_cli.h in example. We will need to update it each time library changes. It's better to ignore it and point out in README how to get it.

I'll look at other changes more closely later.

No worries, thank you for the cli.
Yeah, that would certainly be possible. I think to cover both of your improvements, best would be to completely remove all the H750IBK6 stuff in total, and just add my cli_setup.c/.h and cli_binding.c/.h files to the example folder, and then just explain in the README.md how to re-create the whole ordeal for a specific STM32 controller.
That would be the quickest/easiest way at-least. Especially if you don't like driver/MCU specific stuff being in the repository. Let me know what you think, and I can change the request to match that.
Kind regards.

I see there are three different groups of files in this example:

Application specific code (like cli_binding.h or cli_setup.c)
Controller specific code to setup hardware that is generated from cubemx project settings (like Core/Inc/usart.h or Core/Inc/stm32h7xx_hal_conf.h)
Library code that is copy-pasted by CubeMX (like whole Drivers dir)

First group must be present in project, there is no doubt. And third group must not since it's just copy pasted library. With second group it's a bit harder, there are some reasons to include it in VCS. But since CubeMX can generate it and there is no user code in it (at least I didn't see it), it's perfectly fine to remove it and let CubeMX generate it.
So I think it's better to keep example H7 specific and keep as much code as possible, but ignore files from groups 2 and 3. The only requirement is that user can generate whole project from ioc file, copy paste embedded_cli.h and then everything should compile (for H7).
And it would be even better in addition to this example dir in embedded-cli repo to create separate repository and check into VCS all files necessary to build project (groups 2, 3 and IDE specific files). So that repository can be downloaded and compiled without downloading CubeMX. And then put into README link with that example of example.
This way this repo will keep concise example of launching CLI on stm32 cpu without hardware specifics while in other repo user can download reference implementation of that example.

Okay, I've added the changes you suggested. This is indeed a way cleaner way to have it included into your examples folder.
I've tested both the build your own using the .ioc approach, as well as the fully implemented import (from my reference implementation repository) method. Both seem to work.
I've also linked both repositories to each other using their README.md files.
If there is anything missing, or unclear let me know. I'll probably have some time tomorrow to fix it, before going back to work on monday :)

Thanks for the feedback!
I've worked through them, and gave a 'thumbs up' for each point that was processed.
There was one feedback point which I did not understand the need for. This might need some extra explaining for me :)
I've attached a video in that conversation, for you to see how the CLI looks on my terminal.
Have a good Saturday. Cheers!

You were completely right about the _write function. I've removed it, and created a printf like string formatted output for embeddedCliPrint() instead. See above conversation on video, showing that it does work correctly now.
Great catch, and thanks for all the feedback. I appreciate it :)

I cleaned up code a little bit (no major changes) and merged it into master. I also checked that STM32CubeMX correctly generates all library code and project compiles (didn't flash real chip though). You can try to adapt your sample repository to this new version.
Thanks again for this great submission!

Ah good catch on placing the CommandBinding variables in local scope.
I've also merged these changes into my reference project.
No worries, thank you for the feedback & awesome repository!

It won't compile in my setup anymore. I've noticed that when I copy the single header version in my project, and use your altered versions of my cli_binding.c/.h and cli_setup.c/.h:

The implementation of getCliPointer() is missing in the source file. It is still in the header file, but I get undefined reference errors.
In your single header version of the embedded-cli, you create the definition #define UNUSED(x) (void)x, but the STM HAL also defines this (in the file stm32h7xx_hal_def.h). Maybe add a #ifndef UNUSED clause around that definition.

Any specific reason why the implementation of getCliPointer() is missing? Or just accidentally removed? :P
Cheers.

You're right, it's missing. Don't actually know how that happened, I've added it back.
I've added ifndef to c file. But since single header version not yet released, grab it from artifacts of latest build:
https://github.com/funbiscuit/embedded-cli/actions/runs/5524447549
If it works correctly, I'll make a new release.

Perfect, thanks.
I just tested that it does work when using the new single-header file from the artifact of the latest build.
No more missing references or redefinition of UNUSED.


Postgresql error with ; in string

System information:

Windows 7 Pro SP1
DBeaver <IP_ADDRESS>910201831
No additional extensions

Connection specification:

PostgreSQL 9.5.16
Driver: org.postgresql:postgresql:RELEASE [9.4.1212.jre7]; net.postgis:postgis-jdbc:RELEASE [2.2.1]
No tunnels or proxies

When string in query contains ";" an error occurs "SQL Error: Internal jdbc driver error"
Example SELECT ');'

Thanks for the bug report.
This seems to be caused by recent strings parser.
Workaround: select your query before execute.

verified

We just released hotfix version with the fix (<IP_ADDRESS>910211155)


Don't process direct nodes when we've seen them already from another node in ExprProcessor

This fixes an OOM that we have been seeing, where calling this from within FinallyProcessor could lead to many new objects being created, overloading the garbage collector. I'm not 100% confident that this is the right fix, so I'll test a bit more on my own.

Codecov Report

Merging #159 (376650d) into develop/1.9.0 (bebc65f) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@                 Coverage Diff                 @@
##             develop/1.9.0     #159      +/-   ##
===================================================
- Coverage            77.57%   77.55%   -0.03%
+ Complexity            7651     7648       -3
===================================================
  Files                  201      201
  Lines                24408    24410       +2
  Branches              6359     6359
===================================================
- Hits                 18934    18930       -4
- Misses                3554     3557       +3
- Partials              1920     1923       +3

Impacted Files
Coverage Δ

...a/decompiler/modules/decompiler/ExprProcessor.java
87.52% <100.00%> (-0.75%)
:arrow_down:

...decompiler/modules/decompiler/stats/Statement.java
81.06% <0.00%> (-0.43%)
: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 bebc65f...376650d. Read the comment docs.


Added community forums page

Issues Identification
Closes: #818
Description
added new CSS styles for the comments section, including styles for the comment form and individual comments.
Created a new "Comments" section below the existing content, which includes all the feature in basic community forum discussion.
A form with a text area for users to write their comments
A submit button to post the comment
An empty unordered list where new comments will be displayed
Summary
A brief description of what this pull request does and why it is needed.
Details
Include any detailed information about the changes in this pull request.
Types of Changes
Please check the boxes that apply

[X] Bugfix (non-breaking change that fixes an issue)
[X] New feature (non-breaking change that adds functionality)
[X] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] Documentation update (Documentation content changed)
[] Other (please describe):

Checklist
Please check the boxes that apply

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] My changes do not break the current system and pass all existing test cases
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Screenshots
If applicable, please attach screenshots of the changes made to the user interface.
Additional Information
Please provide any other information that is relevant to this pull request.

@aditya-bhaumik can you please check this

@aditya-bhaumik  can you please check the changes

you have not shared screenshot or recording how will i know what changes you did

i cant find the new changes in the deployment

https://github.com/user-attachments/assets/59b3d3c4-7485-4425-81db-2ab6be2ffc3a

i mentioned community forums you didnt check

@aditya-bhaumik please check once again

Please press on commit suggestion

No need to make separate commit just press on commit suggestion

@aditya-bhaumik i made changes


POJOs should be modeled as LrEntities, not ObjEntities

With the new metadata layer, there's no need to fake POJOs as ObjEntities. Let's switch POJO support to the new mapping.

UPGRADE NOTE: as a result of this change, POJO responses no longer include a duplicate of id property. Instead it is served under "id" key (in the past both "id" and "realPropertyName" keys were included)


nixos/systemd-lib: don't fail on systemd.packages duplicates

In some cases like we've noticed in https://github.com/NixOS/nixpkgs/issues/76169,
having duplicate packages in systemd.packages like
systemd.packages = [ gnome-shell gnome-shell gnome-session ];

breaks.
Here we use an associative array to ensure no
duplicate paths when we symlink all the units listed
in systemd.packages.

Motivation for this change
A certain situation popped up where gnome3 adds gnome-shell and gnome-session to systemd.packages, but gdm also depends on gnome-shell there.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg test systemd  gnome3

Systemd test was broken already https://hydra.nixos.org/job/nixos/trunk-combined/nixos.tests.systemd.x86_64-linux.


[ALLUXIO-2672] Better error message when journal write failed

https://alluxio.atlassian.net/browse/ALLUXIO-2672
This is not a fix to ALLUXIO-2672 but just a slightly improvement in error messages users see.
Instead of showing error like this on client side (e.g., in Spark logs) which is not clear where is wrong, at least point out it is journal problem and likely due to insufficient space on the space to write journals.
2017-03-20 02:33:00,219 ERROR logger.type (RpcUtils.java:call) - Unexpected error running rpc
java.lang.RuntimeException: alluxio.exception.UnexpectedAlluxioException: java.lang.RuntimeException: java.io.IOException: Failed to replace a bad datanode on the existing pipeline due to no more good datanodes being available to try. (Nodes: current=[<IP_ADDRESS>:50010, <IP_ADDRESS>:50010], original=[<IP_ADDRESS>:50010, <IP_ADDRESS>:50010]). The current failed datanode replacement policy is DEFAULT, and a client may configure this via 'dfs.client.block.write.replace-datanode-on-failure.policy' in its configuration.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14104/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14105/
Test PASSed.

Build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14128/
Test PASSed.


[T2-A3][W15-A4]Nikhil Suresh Chandiramani

Inline all methods in main
Inline isValidFilePath method in setupGivenFileForStorage

I realise that this also covers a little of A4 as well.

it is not as smelly as expected for A4 :-P but accepted nevertheless :-)


Initial artifacts doc

This PR adds documentation on how we store builds, and their artifacts 👍

Issue to track the implementation: https://github.com/fastlane/ci/issues/75
Initial discussion https://github.com/fastlane/ci/issues/40
Also describes the concept of build context https://github.com/fastlane/ci/issues/94

Done, did some follow-up changes 👍


✨ Support memory evict strategy

Signed-off-by: Jason Liu<EMAIL_ADDRESS>Ⅰ. Describe what this PR does
Close #54
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews

/area koordlet

/lgtm
/approve


Use musl libc for x86_64 and arm build

Is that what you meant @thomaseizinger ?

seems that it fails on the ring crate.
Perhaps using a dedicated container (like this: https://hub.docker.com/r/ekidd/rust-musl-builder/ ) could help?
It states:

Basic compilation for armv7 using musl-libc. Not all libraries are supported at the moment, however


Disable unaligned load & store for debug build only

This is an alternative PR to https://github.com/polydbms/sheetreader-core/pull/1.
I haven't tested, that this version would enable MINIZ_USE_UNALIGNED_LOADS_AND_STORES for the release build though -- only that it is disabled for the debug (which is to be expected because of #ifndef NDEBUG).
So please consider this PR as a draft. & maybe you have a better idea, how do the check for debug vs. release build.

Opted for disabling by default (#1)


fix: remove theme fallback logic

变动类型

[x] 日常 bug 修复

需求背景和解决方案
changelog

去掉多余的主题 fallback 逻辑

Codecov Report

Merging #188 (2a4247f) into main (2d25c3e) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #188      +/-   ##
==========================================
- Coverage   59.40%   59.40%   -0.01%
==========================================
  Files        1180     1180
  Lines       72354    72352       -2
  Branches    14978    14977       -1
==========================================
- Hits        42983    42980       -3
- Misses      26768    26769       +1
  Partials     2603     2603

Impacted Files
Coverage Δ

...kages/theme/src/browser/workbench.theme.service.ts
72.20% <100.00%> (-0.17%)
:arrow_down:

packages/theme/src/browser/theme-data.ts
63.30% <0.00%> (-0.29%)
: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 2d25c3e...2a4247f. Read the comment docs.


WANTED: People to help maintain supercsv

Dear super-csv community, we are looking for people to help maintain super-csv. Time and motivation is at low, for both James and I.
James has moved the project tremendously and I if only we can find people with as much enthusiasm and forward thinking as he, it would be great :)
You don't need to be a supercsv expert to jump along, all you need is the time and motivation to do great stuff with a great library. There are plenty of issues to start solving, or if your clock it ticking in a different direction, raise new issues and lets see if we can agree that they are indeed good :)

Hi. I am a computer science Ph.D student. I am interested in maintaining SuperCSV. Maybe we can talk more?

Sounds great! There are plenty of issues here to start working on. Anything in particular you want to start improving?

@kbilsted I think it's better to start with fixing issues. So I can get familiar with everything through fixing issues. Do you have anything special or urgent that need to be fixed? If no, I will pick one of the issues to work on.

@ChongTang Do what you find the most fun..one interesting thing: how well is the support for Chinese source code. Maybe there are issues similar to #16
Please raise issues before coding away on stuff not already reported so we can agree that it is a good idea 😋
Also issue #26 may be a good starting point

@kbilsted OK. I will take a look at these two issues first.

@ChongTang  I'm looking forward to seeing how you spell get and set in Chinese then. :)

Hi @kbilsted,
I am a fun of this library and I will try to help you guys as much as I will able to. I have even started working on issue #16 but it is not easy to reproduce. I will try to figure out how to reproduce it and fix.

@ZioberMichal  That sounds fantastic. With you and maybe @ChongTang  on-board it is important to first discuss in an issue what you are working on so we don't all do the same work. Looking at #16 is a great start. Thanks!

I am in love with this library. I would like to contribute on any issue. I'm not an expert in java, but would take the issues which I can easily to start with my part of contribution to super-csv community. Thanks.

@email2vimalraj  great! just pick something from the issue list or add your own issues

@kbilsted I also don't mind to join if still needed :)

@longtimeago  I can't see why not ;) I see you already have  an idea in the open - I think its a great idea to expand the API that utilizes the new features of java.

I just submitted a PR at https://github.com/super-csv/super-csv/pull/103 and am happy to do a little tit-for-tat work on this project to get my PR merged.
Is there anything I can do for folks in exchange for getting code review on my PR from e.g. @kbilsted or @jamesbassett ?  I'm happy to do a ticket review to try to clear out old stuff, fill in some docs where people think they're lacking, or do a code review myself for someone else's outstanding pull request.  In general, help this project out as it's struggling for attention from the maintainers.
Any suggestions?

@ash211  sounds good. At the moment it is @ZioberMichal  who is doing all the heavy lifting. I'd gladly review your code changes and welcome you review others' PR's. Im sure we can all learn from each others points of views.
Are you asking for initial tasks to do ?

I've got one task in progress that I'm selfishly interested in seeing through to completion - #103 - and am offering to do another similarly-sized task as a "pay it forward" to the community.

@ash211  yeah thats how this project has always run - doing the things one think is fun to do ;)

@longtimeago  how are you doing? Find anything interesting to start poking at?

@kbilsted I'm fine, thanks! Just pretty busy on my main job now, sorry

@longtimeago  I know the feeling ;) Let us know when you (if ever) get more time on your hands ;) I'll delete the comments as to keep this thread down in size

@kbilsted thank you for understanding!

@kbilsted I want to submit a new cellprocessor but GitHub is proving to be a very steep wall to climb just to submit 25 lines of code.

Just post it here then:)

I've attached the cellprocessor.  Basically, it's a way to work around values that might have leading zeroes like ZIP codes.  It wraps the text with double quotes with an equals sign prepended like this...
="08109"
...this prevents the leading zeroes from getting lost when opening the CSV.  I'm not totally sure of the mechanics of how it works, but I think that it's forcing Excel to treat the field like a formula.
I tried the other "trick" of putting a single apostrophe at the beginning of the data to preserve the leading zeroes, but SuperCSV doesn't seem to allow doing that in a cellprocessor.  It seemed to keep adding a second "closing" apostrophe when I didn't want one.
More info on this leading zero issue...
https://social.msdn.microsoft.com/Forums/en-US/aae07b39-865f-4c68-a07f-7cad2dfd6733/how-do-i-open-csv-using-excel-without-deleting-leading-zeros?forum=isvvba
PreserveLeadingZeroes.zip

有没有操作csv文件的框架啊？

If you like you can add me as contributor

Hi @tvand  not sure how we would go about it

@tvand  https://commons.apache.org/proper/commons-csv/source-repository.html is broken

@tvand https://commons.apache.org/proper/commons-csv/source-repository.html is broken (this is the link to "Source repository" in commons csv main menu)

See https://commons.apache.org/proper/commons-csv/scm.html

Hi @tvand not sure how we would go about it

If you were interested, I could go investigate. AFAIK, an Intellectual property clearance would be needed. See http://incubator.apache.org/ip-clearance/ on how something like this looks.
Bye, Thomas.

But the code is already under apache license. And they have another csv implementation incompatible with this one.

Yes and that probably makes it easier. We would never "just take" your code, even if the license said we could. We would rather have you donate it. The ASF prefers to have your written consent. I would offer to organize most of the required steps, if you want.
Bye, Thomas.

@tvand  who can contribute that are prevented from contributing today? And why wouldn't they want to contribute today already?

@tvand who can contribute that are prevented from contributing today? And why wouldn't they want to contribute today already?

I'm just hoping that people already contributing to commons-csv or using it will be attracted by the significantly more sophisticated features of super-csv - under the umbrella of Apache. I cannot offer more than this hope at this time.
Bye, Thomas.

@kbilsted are you still looking for contributors?

@moarcode  yes that would be great. But I don't see you doing any public work for the past 3 years on GH?

Hi, @kbilsted
Are you still looking for a maintainer? I'm interested in it.

@dota17  sounds great!

I could help, too. What is on the backlog?
On Sat, 4 Jan 2020, 3:54 AM Kasper B. Graversen<EMAIL_ADDRESS>wrote:

@dota17 https://github.com/dota17 sounds great!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/super-csv/super-csv/issues/47?email_source=notifications&email_token=AAHK5HICAASHJG2AGW3JJY3Q35GQ5A5CNFSM4BFEKG3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIBJHUQ#issuecomment-570594258,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHK5HNYFU4ZYM5TOQB3AYLQ35GQ5ANCNFSM4BFEKG3A
.

@SchlauFuchs fantastic . We need to have fun. So whatever tickles you...
Also We need to deal with issues, review the pr's and make ready for a v2.50 release before long

Hi, @kbilsted
What features and issues are more urgent for version 2.5.0? I can prioritize. And what is your plan for 2.5.0?

@SchlauFuchs  found anything of interest?

The first thing I found (looking for the help wanted tag) was the one about
empty rows made of commas. Someone has done work on it already, I wonder
why it wasn't merged yet.
I did not look into the code, only the backlog.
I could tackle this
On Mon, 13 Jan 2020, 12:31 PM Kasper B. Graversen<EMAIL_ADDRESS>wrote:

@SchlauFuchs https://github.com/SchlauFuchs found anything of interest?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/super-csv/super-csv/issues/47?email_source=notifications&email_token=AAHK5HOHRGGEVVMCBN7CX33Q5OR6DA5CNFSM4BFEKG3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIXHMDI#issuecomment-573470221,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHK5HIQBOMKMX5D6U7ORMDQ5OR6DANCNFSM4BFEKG3A
.

Hi,
Kasper, can you review/merge my PR at
https://github.com/super-csv/super-csv/pull/196 ?
I had a quick a quick review of some sources, especially wondering that the
java classlevel of the project is still 5. I had to dig out a JDK8 from
deep in the web to be able to build this under such settings. I really
recommend to upgrade the project to at least classlevel 8 for compatibility
reasons, but that is already so much EOL that going to 11 would make sense.
I start a branch in my fork for that and give the project the
necessary changes.
Cheers,
Kai
On Mon, 13 Jan 2020 at 12:31, Kasper B. Graversen<EMAIL_ADDRESS>wrote:

@SchlauFuchs https://github.com/SchlauFuchs found anything of interest?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/super-csv/super-csv/issues/47?email_source=notifications&email_token=AAHK5HOHRGGEVVMCBN7CX33Q5OR6DA5CNFSM4BFEKG3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIXHMDI#issuecomment-573470221,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHK5HIQBOMKMX5D6U7ORMDQ5OR6DANCNFSM4BFEKG3A
.

I just reviewed https://github.com/super-csv/super-csv/pull/216 (all 7 commits).
There seem to be people wanting to contribute to this project, but even low risk pull requests don't get approved. While I'm not volunteering to take this project in any special direction, I wouldn't mind reviewing low risk stuff that doesn't change the architecture so at least basic maintenance would still be happening (bugfixes & staying up to date with versions of libraries or Java).
It would be a pitty to see this project end up dead. It's probably still the best library available, even if it doesn't have annotation bells and whistles.

As the maintainer(s) still seems to be unresponsive, we should think about forking this project, apply the PR/fixes there and finally release a new version.

@patric-r , cc: @kbilsted :
I second that, but seeing the amount of issues and pull requests, I was wondering if it's even possible to copy all that good input to a fork. It would be easier to first get things cleaned up on this fork and then pass it on to another maintainer. Maybe others can get more rights on this fork from the original maintainers first?

@brbog If you think the PRs will get merged soon, I agree.
But to be honest and when considerung the previous years - I doubt that.
Have a look at this issue. The original author asked 2015 for a handover but nothing really happened so far even though there were several promising volunteers. Last action from @kbilsted had been in 2020...
Therefore I vote for what will happen sooner or later anyhow (if SuperCSV has a future at all):
A community-drive active fork needs to get created.
Maybe a deadline should be issued to the original author beforehand.
PRs from the original project can still get merged...

@patric-r : I'm afraid you are right and @kbilsted probably has a mail filter to ignore all messages originating from super-csv (addressing him is a bit of a test for this actually).
Maybe people can answer on this thread:

if they are willing to help get everything code reviewed/fixed into a new fork
if they know of someone who deserves the maintainer-role (that would become the account where the fork is maintained)

A little effort (2h or so) from multiple people could help a lot (it's a nice straightforward library), I don't mind helping out a little.

I would propose that whoever interested folks go about forking via:

pick a new name for this project that's a slight variant, like super-csv2
create the github project+repo for it, like github.com/super-csv2/super-csv2
push this code over and do a first release with the same version number of the last super-csv release: 2.4.0

this way super-csv 2.4.0 and super2-csv 2.4.0 are identical code (helps people with migration)

port PRs from super-csv over to super-csv2 and carry the project on in whatever way the new contributors desire

Sounds good. But I still think we should offer @kbilsted a deadline to help the community by either adding people as maintainers to this repo (preferred) or at least accept the most important PRs.
If he doesn't react, we should go for a fork.
@SchlauFuchs @dota17 What do you think?

Hey, I had a permission to the organisation but it looks like it was removed and now I can not add anyone as a maintainer.

Also, I can try to accept most important MR. Please list them and help me to approve. We can try to release version 2.5.0 together.
WDYT, @SchlauFuchs , @dota17 , @patric-r , @ash211

I tried to approve MR created by dependency bot but it is still blocked so it means my collaborator role is not enough to push it further.

Many thanks to all the people who recently showed interest in contributing. It's great to see such energy ;-)
I am very interested in moving the project onwards. As many of you know, the package is fairly wide-spread so the last thing we want is to some troll taking over and committing malware into the package. And there has been struggles with people signaling interest in the project only to disappear unfortunately.
Other struggles in the past with PR's is that people simply code before discussing the validity of their ideas and/refusing to implement tests that show the new feature is actually working. Working tests significantly reduces the work code reviews entails. Finally, the importance of performance in the library is a topic not all grasp.
I hope all you future maintainers share these points of views. If not please voice them now, perhaps there are better values we should cultivate?
Personally, I'd like to participate in discussions and reviews, but realistically won't be coding much myself. If someone will help me setup the tool-chain I can do the releases perhaps in conjunction with project veterans. That be for starters. Naturally more people should gain access when we know each other a little better.

Hej @kbilsted,  great to see you alive here again ;)
While it cannot be completely ruled out, I don't see any big risks about "malware commiters".
This rarely happens. The risk supercsv is slowly going to die if you do nothing is much higher :)
Of course you should choose wisely who will gain commiter privileges but if you do this for people which are active on github for a long time and especially contributing to this project for years, the risk is rather theoretical for me.
And at the end, giving others commiter access doesn't mean that you're out of the game.
Let's face the reality:
Either you're going to still keep the keys (which is my favorite option) - but then should at least react on on the most important PRs (and at the end, ideally merging them) and get a 2.5.0 release out as soon as possible.
Or you hand some of your keys over to other volunteers and see where things are going.
I strongly believe that there are many other clever people here who will respect your architectural and quality concerns.
If neither of the options work for you, we should continue with a fork (hopefully there is still enough momentum left for that).
It would be a pitty if we would need to move on to another csv library...
@ash211 @brbog  @ZioberMichal  What do you think?

I agree with @patric-r. Let's release version 2.5.0 which is almost ready, add some version upticks for libraries suggested by dependency bot, and maybe some small MR and release it.
After that we can discuss who can lead the project after @kbilsted. Some time ago I thought I will be able to do that but I could not find time for it. Nothing changed since then. Right now, I can only play a support role.

@patric-r : well worded.
@kbilsted (and others)
Current reality in Belgium is that most people will conservatively select the Apache Commons CSV library or look at articles like https://www.baeldung.com/java-csv in which supercsv is not mentioned. Anyone else will probably be looking for something that satisfies their "latest-syntax-hype-fetish".
The weakest spot of supercsv is that it looks abandoned/unmaintained, so it has become nearly impossible to suggest using it in a professional environment. Most people are sensible enough to avoid doing crazy things with csv, so using other libs with fingers crossed and "fix problems only if they appear in practice" actually works, but I rather like a library that can handle every weird use-case imaginable while still being easy to use :-). If it would get minor updates once in a while (Java 8 is reaching end of life), that would be a great marketing boost.
@kbilsted : If you are afraid of getting a stained legacy, you could consider moving to a 3.0.0 release so you can easily demarcate your involvement until the last 2.x version (weirder things have happened with version numbers, see http://www.slackware.com/faq/do_faq.php?faq=general#0 ).
I'm hoping that we can get supercsv up to date with minimal effort. The library was great from the beginning and anyone who believes in a fundamentally different design will have wandered off to a different library already I think :-).


Make sure go dep can also be ran easily outside the toolchain

It'd be very useful if we could make sure that go dep can also easily be executed outside the go toolchain.
This implies making sure that it doesn't rely on any toolchain internals but can also be built as a standalone binary.
This would mean that users can update the tool outside of the usual go release cycle to get features they care about that were merged in recently.

we'll start thinking about this once we're actually close to moving into the toolchain. it's definitely on the radar...but i don't think we need to keep an open issue for it 😄


How do I get VcXsrv to work with wayland-0

would like to get my full xfce4 desktop like I have with windows 10 and https://www.tinyurl.com/donofrioworkremmina2020 shows WSL1 working great.  In windows 11 I've yet to get WSL2 to work even when I do the display mod's for the env setup, but that was trying x11, now in Insider w11 dev channel I guess we now get Wayland native but how to get a full DE like xfce4 so far all I get is:
(xfwm4:3779): Gtk-WARNING **: 05:58:38.765: cannot open display: wayland-0

Works for me!

would like to get my full xfce4 desktop like I have with windows 10 and https://www.tinyurl.com/donofrioworkremmina2020 shows WSL1 working great. In windows 11 I've yet to get WSL2 to work even when I do the display mod's for the env setup, but that was trying x11, now in Insider w11 dev channel I guess we now get Wayland native but how to get a full DE like xfce4 so far all I get is:
(xfwm4:3779): Gtk-WARNING **: 05:58:38.765: cannot open display: wayland-0

If you want a full desktop I recommend you this project: https://github.com/DesktopECHO/xWSL

yah works for me when I do not use VcXsrv but built-in Wayland support - https://github.com/microsoft/wslg/issues/843 - well almost perfect, cannot close Wayland once opened lol


No such file or directory

I want to finetune qqp and here comes an error:
File "/private-transformers-main/examples/classification/run_classification.py", line 545, in main
train_dataset = FewShotDataset(data_args, tokenizer=tokenizer, mode="train", use_demo=use_demo)
File "/private-transformers-main/examples/classification/src/dataset.py", line 377, in init
with FileLock(lock_path):
File "/home/anaconda3/envs/fuck/lib/python3.8/site-packages/filelock/_api.py", line 214, in enter
self.acquire()
File "/home/anaconda3/envs/fuck/lib/python3.8/site-packages/filelock/_api.py", line 170, in acquire
self._acquire()
File "/home/anaconda3/envs/fuck/lib/python3.8/site-packages/filelock/_unix.py", line 35, in _acquire
fd = os.open(self._lock_file, open_mode)
FileNotFoundError: [Errno 2] No such file or directory: 'classification/data/original/QQP/cached_train_RobertaTokenizer_256_qqp_few_shot.lock'
how can I get this file? thanks.

Thanks for the interest.
Have you run the data downloading commands listed in the README.md file?
cd data
bash download_dataset.sh

Closed as I was able to run this by first downloading and processing the dataset.


Db interface cachedir

I fix both the DB interface cachedir bug and loss function bug for train_alexnet in newest version of tensorflow. See issue #35 and issue #36

I slightly modified your solution for cross_entropy.


CVE-2024-43491 marked as being on the KEV, but was removed.

CVE-2024-43491
CISA added this vulnerability to the KEV, but has since removed it. It's still tagged as being on the KEV.
https://msrc.microsoft.com/update-guide/vulnerability/CVE-2024-43491
Publicly disclosed:  No
Exploited:  Yes
Exploitability assessment:  Exploitation Detected
This CVE documents the rollback of fixes that addressed vulnerabilities which affected some Optional Components for Windows 10 (version 1507). Some of these CVEs were known to be exploited, but no exploitation of CVE-2024-43491 itself has been detected.

@CSMurray-CISA I'm marking this as a bug and I'm taking a look at what is going on with this entry.

A fix has pushed out and should show up everywhere within an hour.


Set welcome message on subscription metadata

see https://github.com/praekelt/seed-stage-based-messaging/issues/9

@gsvr review please

@gsvr I've just realised I've sent the SMS as a prepend too instead of immediately as per https://github.com/praekelt/hellomama-registration/issues/36 - will resolve tomorrow, please review the rest though.

I'm happy with it in principle.  If you want to you can land this and then we can change the welcome SMS sending in #36, otherwise we could do the work for #36 here.

@gsvr ready for re-review

:+1: when travis happy


Querying a function

How would I generate something like:
SELECT * FROM OPENJSON('["xxx","yyy","zzz"]') as codes
I think I could fudge it sqlsevercompiler wasn't adding square brackets, but it does and I can't find a way to do it.

I need to join result of such a query to real tables in the database - so the query above is just one of subqueries of a big join.

Not an issue. Can be resolved with "WithRaw" method, just found it on the documentation page.


Remove several deprecations in org.hibernate.boot

Signed-off-by: Jan Schatteman<EMAIL_ADDRESS>

Checked the failures locally on hsqldb and mariadb, and at these for those db's they don't seem related to my commit (ran the tests without it).


#new not working with transform_types

I'm not sure if I'm using transform_types correctly.
I want to do some type transformations on an array on initalization and later use #new to create a copy of the struct, but I couldn't get this to work. Am I doing it wrong? It seems to work as expected without transform_type.
Please let me know if you need more information or if my question is not clear to you. Please find the code and the output below.
Cheers, Simon
# running ruby 2.4.0p0 (2016-12-24 revision 57164) [x86_64-linux]
require 'bundler/inline'

gemfile do
  source "https://rubygems.org"
  gem "dry-struct", "0.5.0"
end

module Types
  include Dry::Types.module
end

class Tag < Dry::Struct
  transform_keys(&:to_sym)
  attribute :name, Types::Strict::String
end

class EntityWorking < Dry::Struct
  transform_keys(&:to_sym)

  attribute :name, Types::Strict::String
  attribute :tags, Types::Strict::Array.of(Tag)
end

class EntityFailing < Dry::Struct
  transform_keys(&:to_sym)

  transform_types do |type, name|
    case name
    when :tags then type.constructor do |tags|
      tags.map { |t| Tag.new(name: t) }
    end
    else
      type
    end
  end

  attribute :name, Types::Strict::String
  attribute :tags, Types::Strict::Array.of(Tag)
end

p x = EntityWorking.new("name" => "Franz", tags: [ {name: "red"}])
p x.new(name: "asdf")

p x = EntityFailing.new("name" => "Franz", tags: %w(red))
p x.new(name: "asdf")

#<EntityWorking name="Franz" tags=[#<Tag name="red">]>
#<EntityWorking name="asdf" tags=[#<Tag name="red">]>
#<EntityFailing name="Franz" tags=[#<Tag name="red">]>
/home/simon/.rvm/gems/ruby-2.4.0@guidebook-rails/gems/dry-struct-0.5.0/lib/dry/struct/class_interface.rb:208:in `rescue in new': [EntityFailing.new] [#<Tag name="red">] (Array) has invalid type for :tags violates constraints ([Tag.new] #<Tag name="red"> (Tag) has invalid type for :name violates constraints (type?(String, #<Tag name="red">) failed) failed) (Dry::Struct::Error)
        from /home/simon/.rvm/gems/ruby-2.4.0@guidebook-rails/gems/dry-struct-0.5.0/lib/dry/struct/class_interface.rb:202:in `new'
        from /home/simon/.rvm/gems/ruby-2.4.0@guidebook-rails/gems/dry-struct-0.5.0/lib/dry/struct/class_interface.rb:218:in `call'
        from /home/simon/.rvm/gems/ruby-2.4.0@guidebook-rails/gems/dry-struct-0.5.0/lib/dry/struct.rb:174:in `new'
        from dry_problem.rb:46:in `<main>'

shell returned 1

@lister I manage to fix work script by replacing EntityFailing with:
class EntityFailing < Dry::Struct
  transform_keys(&:to_sym)

  transform_types do |type, name|
    case name
    when :tags
      then type.constructor do |tags|
        tags.map { |t| t.class == Tag ? t : Tag.new(name: t) }
      end
    else
      type
    end
  end

  attribute :name, Types::Strict::String
  attribute :tags, Types::Strict::Array.of(Tag)
end

I think I can explain what is happening to the first time in line x = EntityFailing.new("name" => "Franz", tags: %w(red)) will execute the block of transform_types and when execute the line x.new(name: "asdf") will use the previous attributes {name: "Franz" tags: [#<Tag name="red">]} and merge {name: "asdf"} and trigger tranform_types so the second times the block parameters tags is already a Tag entity.
I would not jump to conclusions saying that this is a bug or the way you are using transform_type is the best way, but I'm sure @flash-gordon could throw some light on the subject.

thx for the clarification. i will close this issue.

The issue has nothing to do with transform_types, you'll get the same result with
class EntityFailing < Dry::Struct
  transform_keys(&:to_sym)

  attribute :name, Types::Strict::String
  attribute :tags, Types::Strict::Array.of(Tag).constructor { |tags|
    tags.map { |t| Tag.new(name: t) }
   }
end

@lister your type for the :tags attribute doesn't support array of Tags as input, it works with arrays of strings only. That's fine when you instantiate a struct for the first time with strings. However, when you call x.new(update) what it does essentially is Entity.new(x.attributes.merge(update)), here x.attributes[:tags] contains an array of Tag objects rather than an array of strings, hence the error. @GustavoCaso worked around that by making constructor work with both types of arrays.

@flash-gordon thanks for the comment and the clarification 😄


Create Signup and Success HTML files

Create HTML files named Signup.html and Success.html with the following attributes:

Signup.html:

Load Static and Staticfiles
Make a form with the method as POST and action directed to the URL name = signup

Success.html:

Load Static and Staticfiles
Have a button that redirects to the URL name = ``

Refer to #2 and ensure proper integration
This file goes inside templates/Main directory

I'll take a crack at this, would quite like a chance to get to grips with Django as a new technology


WSL hangs when deleting a file behind a Windows junction

Your Windows build number:  (Type ver at a Windows Command Prompt)
10.0.16299.64

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)
Executing unlink /mnt/d/Documents/VS2017/g.txt hangs WSL. In the command: /mnt/d/Documents/VS2017 is a junction pointing to /mnt/d/Documents/Visual Studio 2017, g.txt is a plain text file created with a previous command.

What's wrong / what should be happening instead:
The file should've been deleted.

Strace of the failing command, if applicable:  (If some_command is failing, then run strace -o some_command.strace -f some_command some_args, and link the contents of some_command.strace in a gist here)
N/A

Notes
The same command does not hang WSL when the path is the original one (without the junction). After the WSL hangs, there is no other option except restarting the computer.

I was not able to reproduce this in 17711, and I suspect this is fixed in 1803 (17134).  Please reopen if you can hit this on 1803.


whether we need to retry sending message when system busy.

sometime, we can get system busy exception when sending message.
I know rocketmq won't retry if system busy according to #1196
but we don't know whether we need to retry beasuce we don't know whether rocketmq can save messages even if system busy.

The default strategy of retried queue is from another broker.
If a service node indicates that it is busy, retrying may aggravate this situation


ERROR: for [name]  Cannot start service [service]: OCI runtime start failed: container process is already dead: unknown

hypriotos-rpi-v1.10.0.img.zip

$ docker -v
Docker version 18.06.3-ce, build d7080c1

Error message:
ERROR: for [name]  Cannot start service [service]: OCI runtime start failed: container process is already dead: unknown

docker-compose.yml
version: '3'

services:
  postgres:
    image: postgres:11.2
    ports:
      - 5432:5432
    environment:
      POSTGRES_PASSWORD:
  redis:
    image: redis:5.0.3
    ports:
      - 6379:6379
    command: redis-server --appendonly yes
  mongo:
    image: amd64/mongo:4.1-xenial
    ports:
      - 27017:27017

Solutions
https://success.docker.com/article/error-oci-runtime-create-failed-docker-1809

At least this image amd64/mongo:4.1-xenial is not available for the Raspberry Pi CPU, also the mongo:4.1 only provides an amd64 and arm64 image.

Closing due to inactivity


[Bug]: Toml quoted keys are not parsed properly

Description
For each toml quoted key, I can see additional double quotes around it.
[{"basePath":"/api/entitlement-service","url":["https://abc.com/api/entitlement-service"],"config":{""foo.bar.baz"":"value"}}]

Config.toml
[[endpoints]]
basePath = "/api/entitlement-service"
url = ["https://abc.com/api/entitlement-service"]
config = {"foo.bar.baz" = "value"}

Steps to Reproduce
Use the below code snippet
type Endpoint record {|
    string basePath;
    string[] url;
    map<string> config;
|};

configurable Endpoint[] endpoints = [];

Config.toml
[[endpoints]]
basePath = "/api/entitlement-service"
url = ["https://abc.com/api/entitlement-service"]
config = {"foo.bar.baz" = "value"}

Affected Version(s)
2201.5.0
OS, DB, other environment details and versions
No response
Related area
-> Compilation
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

This will be a blocker pretty soon. We need this to implement service registration and discovery proposal.

Had a look at this today.
From toml parsers side, As I remember, adding the quotes into quoted keys was intentional. This is due to complexities when retrieving via dotted keys.
Consider the following sample -
"foo.bar" = "value"

[foo]
bar = "value1"

Both the entries are different right? First one is a key value par in root level, and 2nd one is inside a table. But when we want to retrieve via dotted keys. we always have to use "foo.bar" for both cases. I believe that's why we made surrounding quotes required when we wanna retrieve quoted strings. This is the same behavior in parsers like Toml4j as well.
However, in Config.toml case this doesn't really matter. Had a chat with @HindujaB today. We'll introduce an API without quotes for the Configurables to use in the AST level.

I have a minor concern.
Since we support identifiers with special characters in Ballerina and if we have a configurable variable with quotes as follows,
(This is not yet supported)
configurable string \"foo\.bar\" = ?;

What should be the Config.toml representation?
"foo.bar" = "hello-foo.bar"

or
"\"foo.bar\"" = "hello-foo.bar"

As " in Toml spec is used to enclose toml reserved characters inside a key, I suppose the user has to enclose the entire variable name as option 2. In that case, we can remove the inconsistency from the above example in the issue and separately handle the identifier with special characters.
WDYT?
@sameerajayasoma @warunalakshitha

@HindujaB Option 2 would be more user friendly. But if there are any ambiguities we can support both


User/ben/handoff task

Also dont forget to add that new repo to the rosinstall. just follow the format already in that file


WiP: Update resources to VEC 1.2.1

Pull Request

[x] I have checked for similar PRs.
[x] I have read the contributing guidelines.

Changes

[x] Code
[ ] Documentation
[x] Other: XSD files

Description
Updates the resources (xsd and xjb files) to the VEC 1.2.1. Also improved test code to use the factory methods of the SchemaFactory to get a Schema more easily,

Closed with #66 and #67.


__gnu_cxx recursive init error on multiple goroutines creating isolates at the same time

When creating new isolates in multiple goroutines at the same time, I get the following crash message:
terminate called recursively
terminate called after throwing an instance of '__gnu_cxx::recursive_init_error'
  what():  std::exception
SIGABRT: abort
PC=0x7ff2f869722d m=27 sigcode=18446744073709551610
signal arrived during cgo execution

goroutine 3463 [syscall]:
runtime.cgocall(0x773bdf0, 0xc0005b9808)
	/usr/local/go/src/runtime/cgocall.go:158 +0x5c fp=0xc0005b97e0 sp=0xc0005b97a8 pc=0x286007c
rogchap.com/v8go._Cfunc_NewIsolate()
	_cgo_gotypes.go:745 +0x49 fp=0xc0005b9808 sp=0xc0005b97e0 pc=0x73a3a89
rogchap.com/v8go.NewIsolate()
	/go/pkg/mod/rogchap.com/v8go@v0.7.1-0.20221102201510-1f00b5007d95/isolate.go:59 +0x39 fp=0xc0005b9858 sp=0x
c0005b9808 pc=0x73acb59
github.com/solo-io/gloo-mesh-enterprise/pkg/translator/plugins/graphql.NewStitchingScriptRunner()

I've tried to do a little testing myself and believe that multiple goroutines are hanging on this line: https://github.com/rogchap/v8go/blob/master/v8go.cc#L154

Why version of this package are you using? I am on v0.7.0 and I can create a new isolate per http request (which runs in its own goroutine)


[WINDOWS10/CPU/CUDA] float type issues

I have been trying to generate just one image but keep getting issues related to float. I am executing the following command.
python39 ./scripts/txt2img.py --prompt "a professional photograph of an astronaut riding a horse" --ckpt .\768-v-ema.ckpt --config  .\configs\stable-diffusion\v2-inference-v.yaml --H 200 --W 200

or this (cuda)
python39 ./scripts/txt2img.py --prompt "a professional photograph of an astronaut riding a horse" --ckpt .\768-v-ema.ckpt --config  .\configs\stable-diffusion\v2-inference-v.yaml --H 200 --W 200  --precision full --device cuda

I get this error.
RuntimeError: expected scalar type BFloat16 but found Float

I have read that "--precision full" may help so I tried it but I get this.
RuntimeError: Input type (struct c10::Half) and bias type (float) should be the same

It is quite weird because UI version works, and I just wonder why. Is it running on some kind of container? Why am I getting these errors? Am I using the wrong commands?

i added "--bf16" to the command but now i am getting out of memory error. Solved but not done with my journey yet.

try to add --device cuda

--device cuda

Hello! Ive tried it but I got this:
NotImplementedError: No operator found for memory_efficient_attention_forward with inputs:
query       : shape=(30, 9216, 1, 64) (torch.float16)
key         : shape=(30, 9216, 1, 64) (torch.float16)
value       : shape=(30, 9216, 1, 64) (torch.float16)
attn_bias   : <class 'NoneType'>
p           : 0.0
cutlassF is not supported because:
xFormers wasn't build with CUDA support
flshattF is not supported because:
xFormers wasn't build with CUDA support
tritonflashattF is not supported because:
xFormers wasn't build with CUDA support
triton is not available
requires A100 GPU
smallkF is not supported because:
xFormers wasn't build with CUDA support
dtype=torch.float16 (supported: {torch.float32})
max(query.shape[-1] != value.shape[-1]) > 32
unsupported embed per head: 64

--device cuda

Hello! Ive tried it but I got this:
NotImplementedError: No operator found for memory_efficient_attention_forward with inputs: query : shape=(30, 9216, 1, 64) (torch.float16) key : shape=(30, 9216, 1, 64) (torch.float16) value : shape=(30, 9216, 1, 64) (torch.float16) attn_bias : <class 'NoneType'> p : 0.0 cutlassF is not supported because: xFormers wasn't build with CUDA support flshattF is not supported because: xFormers wasn't build with CUDA support tritonflashattF is not supported because: xFormers wasn't build with CUDA support triton is not available requires A100 GPU smallkF is not supported because: xFormers wasn't build with CUDA support dtype=torch.float16 (supported: {torch.float32}) max(query.shape[-1] != value.shape[-1]) > 32 unsupported embed per head: 64

Have you solved this problem?

--device cuda

Hello! Ive tried it but I got this:
NotImplementedError: No operator found for with inputs: query : shape=(30, 9216, 1, 64) (torch.float16) key : shape=(30, 9216, 1, 64) (torch.float16) value : shape=(30, 9216, 1, 64) (torch.float16) attn_bias : <class 'NoneType'> p : 0.0 is not supported because: xFormers wasn't build with CUDA support is not supported because: xFormers wasn't build with CUDA support is not supported because: xFormers wasn't build with CUDA support triton is not available requires A100 GPU is not supported because: xFormers wasn't build with CUDA support dtype=torch.float16 (supported: {torch.float32}) max(query.shape[-1] != value.shape[-1]) > 32 unsupported embed per head: 64memory_efficient_attention_forward``cutlassF``flshattF``tritonflashattF``smallkF

My problem is the same as yours. Has it been solved?

I received the following error message.
WARNING[XFORMERS]: xFormers can't load C++/CUDA extensions. xFormers was built for:
PyTorch 2.4.0+cu121 with CUDA 1201 (you have 2.4.0+cpu)
Python  3.12.4 (you have 3.12.2)
Please reinstall xformers (see https://github.com/facebookresearch/xformers#installing-xformers)
Memory-efficient attention, SwiGLU, sparse and more won't be available.
Set XFORMERS_MORE_DETAILS=1 for more details
D:\Depth-Anything\run.py:42: FutureWarning: You are using torch.load with weights_only=False (the current default value), which uses the default pickle module implicitly. It is possible to construct malicious pickle data which will execute arbitrary code during unpickling (See https://github.com/pytorch/pytorch/blob/main/SECURITY.md#untrusted-models for more details). In a future release, the default value for weights_only will be flipped to True. This limits the functions that could be executed during unpickling. Arbitrary objects will no longer be allowed to be loaded via this mode unless they are explicitly allowlisted by the user via torch.serialization.add_safe_globals. We recommend you start setting weights_only=True for any use case where you don't have full control of the loaded file. Please open an issue on GitHub for any issues related to this experimental feature.
depth_anything.load_state_dict(torch.load('./checkpoint/depth_anything_vitl14/pytorch_model.bin'))
Total parameters: 335.32M
0%|          | 0/19 [00:00<?, ?it/s]
Traceback (most recent call last):
File "D:\Depth-Anything\run.py", line 87, in
depth = depth_anything(image)
^^^^^^^^^^^^^^^^^^^^^
File "E:\ANACONDA\envs\p111\Lib\site-packages\torch\nn\modules\module.py", line 1553, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "E:\ANACONDA\envs\p111\Lib\site-packages\torch\nn\modules\module.py", line 1562, in _call_impl
return forward_call(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "D:\Depth-Anything\depth_anything\dpt.py", line 158, in forward
features = self.pretrained.get_intermediate_layers(x, 4, return_class_token=True)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "D:\Depth-Anything\torchhub/facebookresearch_dinov2_main\vision_transformer.py", line 308, in get_intermediate_layers
outputs = self._get_intermediate_layers_not_chunked(x, n)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "D:\Depth-Anything\torchhub/facebookresearch_dinov2_main\vision_transformer.py", line 277, in _get_intermediate_layers_not_chunked
x = blk(x)
^^^^^^
File "E:\ANACONDA\envs\p111\Lib\site-packages\torch\nn\modules\module.py", line 1553, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "E:\ANACONDA\envs\p111\Lib\site-packages\torch\nn\modules\module.py", line 1562, in _call_impl
return forward_call(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "D:\Depth-Anything\torchhub/facebookresearch_dinov2_main\dinov2\layers\block.py", line 247, in forward
return super().forward(x_or_x_list)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "D:\Depth-Anything\torchhub/facebookresearch_dinov2_main\dinov2\layers\block.py", line 105, in forward
x = x + attn_residual_func(x)
^^^^^^^^^^^^^^^^^^^^^
File "D:\Depth-Anything\torchhub/facebookresearch_dinov2_main\dinov2\layers\block.py", line 84, in attn_residual_func
return self.ls1(self.attn(self.norm1(x)))
^^^^^^^^^^^^^^^^^^^^^^^^
File "E:\ANACONDA\envs\p111\Lib\site-packages\torch\nn\modules\module.py", line 1553, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "E:\ANACONDA\envs\p111\Lib\site-packages\torch\nn\modules\module.py", line 1562, in call_impl
return forward_call(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "D:\Depth-Anything\torchhub/facebookresearch_dinov2_main\dinov2\layers\attention.py", line 76, in forward
x = memory_efficient_attention(q, k, v, attn_bias=attn_bias)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "E:\ANACONDA\envs\p111\Lib\site-packages\xformers\ops\fmha_init.py", line 276, in memory_efficient_attention
return memory_efficient_attention(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "E:\ANACONDA\envs\p111\Lib\site-packages\xformers\ops\fmha_init.py", line 395, in _memory_efficient_attention
return memory_efficient_attention_forward(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "E:\ANACONDA\envs\p111\Lib\site-packages\xformers\ops\fmha_init.py", line 414, in _memory_efficient_attention_forward
op = _dispatch_fw(inp, False)
^^^^^^^^^^^^^^^^^^^^^^^^
File "E:\ANACONDA\envs\p111\Lib\site-packages\xformers\ops\fmha\dispatch.py", line 119, in _dispatch_fw
return _run_priority_list(
^^^^^^^^^^^^^^^^^^^
File "E:\ANACONDA\envs\p111\Lib\site-packages\xformers\ops\fmha\dispatch.py", line 55, in _run_priority_list
raise NotImplementedError(msg)
NotImplementedError: No operator found for memory_efficient_attention_forward with inputs:
query       : shape=(1, 2073, 16, 64) (torch.float32)
key         : shape=(1, 2073, 16, 64) (torch.float32)
value       : shape=(1, 2073, 16, 64) (torch.float32)
attn_bias   : <class 'NoneType'>
p           : 0.0
ckF is not supported because:
device=cpu (supported: {'cuda'})
dtype=torch.float32 (supported: {torch.float16, torch.bfloat16})
operator wasn't built - see python -m xformers.info for more info


Build wheels for Python 3.13

Hi! With python 3.13 coming out (hopefully) later today, it would be nice to have wheels for the new version available.
This PR adds Python 3.13 to the versions for which tiktoken should be built (for now together with CIBW_PRERELEASE_PYTHONS: True, since only 3.13.0rc2 is available to date). It also bumps pyo3 to version 0.22 which supports Python 3.13, together with the gil-refs feature flag for backwards compatibility.
I suppose a discussion could be had about whether support for Python 3.8 should be dropped due to its end-of-life, but that's for another day.
Let me know if there's anything else that should be accounted for, my experience with Rust is very limited :)

Thanks for the PR! The internal version already uses the new PyO3 Bound API, I'll sync the codebase.
Not sure what's regressed the QEMU aarch64 builds

tiktoken 0.8 ships 3.13 wheels. Needed a few things to be different from this diff. Thanks again!


initialUrl config option

It would be useful to specify a url to open in the app so you can deep link into it for your tests. In particular this is useful to integrate with Exponent, so you can install the Exponent client and then if initialUrl is provided the app is opened with fbsimctl open <initialUrl> otherwise it just launches the app directly.

This is already implemented internally. Not sure if already published in JS. @rotemmiz ?
Due to the way we inject Detox into iOS apps, we cannot use fbsimctl open. We have special API where you can launch your app using a URL, and it is rerouted to the app delegate as you expect from a launch with URL.

@brentvatne do you mean something like this ? https://github.com/wix/detox/wiki/Mocking-User-Notifications-and-URLs#mocking-app-launch-from-a-url

nice :) that seems like it would do the trick


Simplify GitHub Actions

Remove the "pip install" test. pip install with Poetry is pretty routine now
Also doing the development tests without Poetry's poetry CLI tool. We don't need the virtual environments for GHA tests

Codecov Report

Merging #54 (5bef86c) into master (49dc964) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 5bef86c differs from pull request most recent head f97978f.  Learn more